remove tsdx, add rollup build (#595)

Author: NiloCK

packages/client/package.json

@@ -4,27 +4,19 @@
   "private": true,
   "license": "MIT",
   "main": "dist/index.js",
+  "module": "dist/index.esm.js",
   "typings": "dist/index.d.ts",
   "files": [
     "dist",
     "src"
   ],
   "engines": {
-    "node": ">=10"
+    "node": ">=12"
   },
   "scripts": {
-    "start": "tsdx watch",
-    "build": "tsdx build",
-    "test": "tsdx test",
-    "lint": "tsdx lint",
-    "prepare": "tsdx build",
-    "size": "size-limit",
-    "analyze": "size-limit --why"
-  },
-  "husky": {
-    "hooks": {
-      "pre-commit": "tsdx lint"
-    }
+    "build": "rimraf dist && tsc && rollup --bundleConfigAsCjs -c",
+    "dev": "rollup --bundleConfigAsCjs -c -w",
+    "lint": "eslint 'src/**/*.ts'"
   },
   "prettier": {
     "printWidth": 80,
@@ -33,23 +25,24 @@
     "trailingComma": "es5"
   },
   "author": "Colin Kennedy",
-  "module": "dist/client.esm.js",
-  "size-limit": [
-    {
-      "path": "dist/client.cjs.production.min.js",
-      "limit": "10 KB"
-    },
-    {
-      "path": "dist/client.esm.js",
-      "limit": "10 KB"
-    }
-  ],
   "dependencies": {
     "axios": "^1.7.4",
     "command-line-args": "^5.2.1"
   },
   "devDependencies": {
+    "@rollup/plugin-commonjs": "^25.0.7",
+    "@rollup/plugin-node-resolve": "^15.2.3",
+    "@rollup/plugin-typescript": "^11.1.6",
+    "@types/jest": "^29.5.12",
     "@types/node": "^20.2.5",
-    "tsdx": "^0.14.1"
+    "@typescript-eslint/eslint-plugin": "^7.4.0",
+    "@typescript-eslint/parser": "^7.4.0",
+    "eslint": "^8.57.0",
+    "jest": "^29.7.0",
+    "rimraf": "^5.0.5",
+    "rollup": "^4.12.1",
+    "ts-jest": "^29.1.2",
+    "tslib": "^2.6.2",
+    "typescript": "^5.4.2"
   }
 }

packages/client/rollup.config.js

@@ -0,0 +1,39 @@
+import resolve from '@rollup/plugin-node-resolve';
+import commonjs from '@rollup/plugin-commonjs';
+import typescript from '@rollup/plugin-typescript';
+import { defineConfig } from 'rollup';
+
+export default defineConfig({
+  input: 'src/index.ts',
+  output: [
+    {
+      file: 'dist/index.js',
+      format: 'cjs',
+      sourcemap: true,
+    },
+    {
+      file: 'dist/index.esm.js',
+      format: 'esm',
+      sourcemap: true,
+    },
+  ],
+  // Mark all Node.js built-ins and dependencies as external
+  external: [
+    /node_modules/,
+    'fs',
+    'path',
+    'buffer',
+    'axios',
+    'command-line-args',
+  ],
+  plugins: [
+    resolve({
+      preferBuiltins: true,
+    }),
+    commonjs(),
+    typescript({
+      tsconfig: './tsconfig.json',
+      sourceMap: true,
+    }),
+  ],
+});

packages/client/src/index.ts

@@ -1,14 +1,13 @@
 import * as fs from 'fs';
-// import { addNote55 } from '../../vue/src/db/courseDB';
-// import PouchDb from 'pouchdb-core';
-import console from 'console';
-import Blob from 'buffer';
 import axios from 'axios';
+
 // import { addNote55 } from 'new-skuilder';
 
 // const spellingCourseID = 'a9fae15687220aa6ce62018005087c95';
 // testAPI();
 
+type AudioData = Buffer;
+
 makeSpellingNotes();
 makeSelectionNotes();
 
@@ -59,7 +58,7 @@ function isCVCeWord(w: string): boolean {
 
 function cardText(w: string): string {
   return `Spell the word:
-  
+
 {{ ${w} }}`;
 }
 
@@ -114,7 +113,7 @@ function spellingCardTags(w: string): string[] {
 
 type spellingNote = {
   md: string;
-  audio: Blob.Blob;
+  audio: AudioData;
   tags: string[];
 };
 

packages/client/tsconfig.json

@@ -1,37 +1,25 @@
 {
-  // see https://www.typescriptlang.org/tsconfig to better understand tsconfigs
-  "include": ["src", "types"],
   "compilerOptions": {
-    "module": "esnext",
-    "lib": ["dom", "esnext"],
-    "importHelpers": true,
-    // output .d.ts declaration files for consumers
-    "declaration": true,
-    // output .js.map sourcemap files for consumers
+    "target": "es2018",
+    "module": "esnext", // Changed from "commonjs" to "esnext"
+    "lib": ["es2018"],
+    "moduleResolution": "node",
     "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,
-    // linter checks for common issues
+    "declaration": true,
+    "esModuleInterop": true,
     "noImplicitReturns": true,
-    "noFallthroughCasesInSwitch": true,
-    // noUnused* overlap with @typescript-eslint/no-unused-vars, can disable if duplicative
+    "noImplicitThis": true,
+    "noImplicitAny": true,
+    "strictNullChecks": true,
     "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
+    "allowSyntheticDefaultImports": true,
+    "outDir": "dist",
+    "declarationDir": "dist",
+    "resolveJsonModule": true,
     "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,
-    // for yarn 2 monorepo support
-    "composite": true
-  }
+    "forceConsistentCasingInFileNames": true
+  },
+  "include": ["src"],
+  "exclude": ["node_modules", "dist"]
 }