chore: Use tsdx instead of plain rollup

Author: lopezjurip

__tests__/index.test.ts

@@ -1,4 +1,4 @@
-import { proxy } from "../src/proxy";
+import { proxy } from "../src";
 
 describe("proxy", () => {
   it("wraps URL that may required proxy", () => {

package.json

@@ -8,13 +8,17 @@
     "proxy",
     "typescript"
   ],
-  "main": "dist/proxy.es5.js",
-  "module": "dist/proxy.esm.js",
-  "typings": "dist/proxy.d.ts",
-  "sideEffects": false,
+  "main": "dist/index.js",
+  "typings": "dist/index.d.ts",
+  "module": "dist/index.esm.js",
   "files": [
-    "dist"
+    "dist",
+    "src"
   ],
+  "engines": {
+    "node": ">=12"
+  },
+  "sideEffects": false,
   "author": "Patricio López Juri <[email protected]>",
   "repository": {
     "type": "git",
@@ -23,12 +27,13 @@
   "license": "MIT",
   "private": false,
   "scripts": {
-    "prebuild": "rimraf dist",
-    "build": "rollup -c rollup.config.ts",
-    "prepublishOnly": "yarn run build",
-    "start": "rollup -c rollup.config.ts -w",
+    "start": "tsdx watch",
+    "build": "tsdx build",
+    "test": "tsdx test",
     "lint": "eslint '*/**/*.{js,ts,tsx}'",
-    "test": "jest"
+    "prepare": "tsdx build",
+    "size": "size-limit",
+    "analyze": "size-limit --why"
   },
   "husky": {
     "hooks": {
@@ -43,37 +48,16 @@
   "dependencies": {},
   "peerDependencies": {},
   "devDependencies": {
-    "@flyyer/eslint-config": "^2.0.0",
-    "@types/jest": "^27.0.1",
+    "@flyyer/eslint-config": "^2.1.1",
+    "babel-jest": "^27.2.0",
     "eslint": "^7.32.0",
+    "eslint-plugin-jest": "^24.4.0",
     "husky": "^4.3",
-    "jest": "^27.2.0",
     "lint-staged": "^11.1.2",
     "prettier": "^2.4.0",
-    "rollup": "^2.56.3",
-    "rollup-plugin-commonjs": "^10.1.0",
-    "rollup-plugin-json": "^4.0.0",
-    "rollup-plugin-node-resolve": "^5.2.0",
-    "rollup-plugin-sourcemaps": "^0.6.3",
-    "rollup-plugin-typescript2": "^0.30.0",
     "ts-jest": "^27.0.5",
-    "ts-node": "^10.2.1",
+    "tsdx": "^0.14.1",
+    "tslib": "^2.3.1",
     "typescript": "^4.4.3"
-  },
-  "jest": {
-    "transform": {
-      ".(ts|tsx)": "ts-jest"
-    },
-    "testEnvironment": "jsdom",
-    "testRegex": "(/__tests__/.*|\\.(test|spec))\\.(ts|tsx|js)$",
-    "moduleFileExtensions": [
-      "ts",
-      "tsx",
-      "js"
-    ],
-    "coveragePathIgnorePatterns": [
-      "/node_modules/",
-      "/test/"
-    ]
   }
 }

rollup.config.ts

@@ -1,3 +1,14 @@
+/**
+ * https://stackoverflow.com/a/61934195/3416691
+ */
+export const FORCE_HTTPS = (url: string) =>
+  url.replace(/^(?:(.*:)?\/\/)?(.*)/i, (match, schema, nonSchemaURL) => (schema ? match : `https://${nonSchemaURL}`));
+
+/**
+ * https://stackoverflow.com/a/19709846/3416691
+ */
+export const EXTERNAL_URL_REGEX = new RegExp("^(?:[a-z]+:)?//", "i");
+
 /**
  * Returns an URL to proxy request via flyyer network.
  *
@@ -34,14 +45,3 @@ export function proxy(src: string): string {
     return src;
   }
 }
-
-/**
- * https://stackoverflow.com/a/61934195/3416691
- */
-export const FORCE_HTTPS = (url: string) =>
-  url.replace(/^(?:(.*:)?\/\/)?(.*)/i, (match, schema, nonSchemaURL) => (schema ? match : `https://${nonSchemaURL}`));
-
-/**
- * https://stackoverflow.com/a/19709846/3416691
- */
-export const EXTERNAL_URL_REGEX = new RegExp("^(?:[a-z]+:)?//", "i");

src/proxy.ts renamed to src/index.ts

@@ -1,28 +1,35 @@
 {
+  // see https://www.typescriptlang.org/tsconfig to better understand tsconfigs
+  "include": ["src", "types"],
   "compilerOptions": {
-    "moduleResolution": "node",
-    "target": "es5",
-    "module":"es2015",
-    "alwaysStrict": true,
+    "module": "esnext",
+    "lib": ["dom", "esnext"],
+    "importHelpers": true,
+    // output .d.ts declaration files for consumers
+    "declaration": true,
+    // output .js.map sourcemap files for consumers
+    "sourceMap": true,
+    // match output dir to input dir. e.g. dist/index instead of dist/src/index
+    "rootDir": "./src",
+    // stricter type-checking for stronger correctness. Recommended by TS
     "strict": true,
-    "strictNullChecks": true,
-    "strictPropertyInitialization": true,
-    "strictFunctionTypes": true,
-    "strictBindCallApply": true,
-    "noImplicitAny": true,
-    "noImplicitThis": true,
-    "noUncheckedIndexedAccess": true,
-    "noFallthroughCasesInSwitch": true,
+    // linter checks for common issues
     "noImplicitReturns": true,
-    "preserveConstEnums": true,
-    "sourceMap": true,
-    "declaration": true,
-    "allowSyntheticDefaultImports": true,
-    "declarationDir": "dist/types",
-    "outDir": "dist/lib",
-    "typeRoots": [
-      "node_modules/@types", "types"
-    ],
-  },
-  "include": ["types/**/*.d.ts", "src/**/*.ts", "src/**/*.tsx"]
+    "noFallthroughCasesInSwitch": true,
+    // noUnused* overlap with @typescript-eslint/no-unused-vars, can disable if duplicative
+    "noUnusedLocals": true,
+    "noUnusedParameters": true,
+    // use Node's module resolution algorithm, instead of the legacy TS one
+    "moduleResolution": "node",
+    // transpile JSX to React.createElement
+    "jsx": "react",
+    // interop between ESM and CJS modules. Recommended by TS
+    "esModuleInterop": true,
+    // significant perf increase by skipping checking .d.ts files, particularly those in node_modules. Recommended by TS
+    "skipLibCheck": true,
+    // error out if import and file system have a casing mismatch. Recommended by TS
+    "forceConsistentCasingInFileNames": true,
+    // `tsdx build` ignores this option, but it is commonly used when type-checking separately with `tsc`
+    "noEmit": true,
+  }
 }