fix: CJS export issue, CI cleanup (#88)

* fix: remove problematic flag

tsup already treeshakes! this flag is causing discrepancies with the CJS build vs types

* ci: re-add typechecking

Author: kanadgupta

.github/workflows/ci.yml

@@ -24,6 +24,4 @@ jobs:
         with:
           node-version: ${{ matrix.node-version }}
 
-      - run: npm ci
-      - run: npm run build
-      - run: npm test
+      - run: npm cit

package.json

@@ -27,13 +27,13 @@
   },
   "scripts": {
     "build": "tsup",
-    "lint": "eslint . --ext .js,.ts",
+    "lint": "tsc --noEmit && eslint . --ext .js,.ts",
     "prebuild": "rm -rf dist/",
     "prepack": "npm run build",
     "prepare": "husky install",
-    "pretest": "npm run build; npm run lint",
+    "pretest": "npm run lint",
     "prettier": "prettier --list-different --write \"./**/**.{js,ts}\"",
-    "test": "vitest --coverage"
+    "test": "vitest run --coverage"
   },
   "devDependencies": {
     "@commitlint/cli": "^17.6.5",

tsup.config.ts

@@ -16,5 +16,4 @@ export default defineConfig((options: Options) => ({
   silent: !options.watch,
   sourcemap: true,
   splitting: true,
-  treeshake: true,
 }));