chore(clerk-react,shared,types): Switch to tsc for type generation

.changeset/itchy-avocados-push.md

@@ -0,0 +1,7 @@
+---
+'@clerk/shared': patch
+'@clerk/clerk-react': patch
+'@clerk/types': patch
+---
+
+Internal change on how the TypeScript types are generated. If you run into type issues, please open an issue.

packages/backend/tsconfig.json

@@ -2,8 +2,6 @@
   "compilerOptions": {
     "allowSyntheticDefaultImports": true,
     "baseUrl": ".",
-    "declaration": true,
-    "declarationMap": false,
     "esModuleInterop": true,
     "forceConsistentCasingInFileNames": true,
     "importHelpers": true,

packages/nextjs/tsconfig.json

@@ -2,8 +2,6 @@
   "compilerOptions": {
     "allowJs": true,
     "baseUrl": ".",
-    "declaration": true,
-    "declarationMap": true,
     "esModuleInterop": true,
     "importHelpers": true,
     "isolatedModules": true,

packages/react/package.json

@@ -25,7 +25,7 @@
   "exports": {
     ".": {
       "import": {
-        "types": "./dist/index.d.mts",
+        "types": "./dist/index.d.ts",
         "default": "./dist/index.mjs"
       },
       "require": {
@@ -35,7 +35,7 @@
     },
     "./internal": {
       "import": {
-        "types": "./dist/internal.d.mts",
+        "types": "./dist/internal.d.ts",
         "default": "./dist/internal.mjs"
       },
       "require": {
@@ -45,7 +45,7 @@
     },
     "./errors": {
       "import": {
-        "types": "./dist/errors.d.mts",
+        "types": "./dist/errors.d.ts",
         "default": "./dist/errors.mjs"
       },
       "require": {
@@ -64,11 +64,11 @@
   "scripts": {
     "build": "tsup",
     "postbuild": "node ../../scripts/subpath-workaround.mjs react",
-    "clean": "rimraf ./dist",
+    "build:declarations": "tsc -p tsconfig.declarations.json",
     "dev": "tsup --watch",
     "dev:publish": "pnpm dev --env.publish",
     "lint": "eslint src",
-    "lint:attw": "attw --pack .",
+    "lint:attw": "attw --pack . --ignore-rules false-cjs",
     "lint:publint": "publint",
     "publish:local": "pnpm yalc push --replace  --sig",
     "test": "vitest run",

packages/react/tsconfig.declarations.json

@@ -0,0 +1,11 @@
+{
+  "extends": "./tsconfig.json",
+  "compilerOptions": {
+    "declaration": true,
+    "declarationDir": "./dist",
+    "declarationMap": true,
+    "emitDeclarationOnly": true,
+    "skipLibCheck": true
+  },
+  "exclude": ["**/__tests__/**/*"]
+}

packages/react/tsconfig.json

@@ -1,10 +1,7 @@
 {
   "compilerOptions": {
     "baseUrl": ".",
-    "declaration": true,
-    "declarationMap": false,
     "esModuleInterop": true,
-    "emitDeclarationOnly": true,
     "importHelpers": true,
     "isolatedModules": true,
     "jsx": "react",

packages/react/tsup.config.ts

@@ -1,20 +1,20 @@
+import type { Options } from 'tsup';
 import { defineConfig } from 'tsup';
 
+import { runAfterLast } from '../../scripts/utils';
 import { version as clerkJsVersion } from '../clerk-js/package.json';
 import { name, version } from './package.json';
 
 export default defineConfig(overrideOptions => {
   const isWatch = !!overrideOptions.watch;
   const shouldPublish = !!overrideOptions.env?.publish;
 
-  return {
+  const options: Options = {
     entry: {
       index: 'src/index.ts',
       internal: 'src/internal.ts',
       errors: 'src/errors.ts',
     },
-    dts: true,
-    onSuccess: shouldPublish ? 'pnpm publish:local' : undefined,
     format: ['cjs', 'esm'],
     bundle: true,
     clean: true,
@@ -28,4 +28,6 @@ export default defineConfig(overrideOptions => {
       __DEV__: `${isWatch}`,
     },
   };
+
+  return runAfterLast(['pnpm build:declarations', shouldPublish && 'pnpm publish:local'])(options);
 });

packages/shared/package.json

@@ -12,7 +12,7 @@
   "exports": {
     ".": {
       "import": {
-        "types": "./dist/index.d.mts",
+        "types": "./dist/index.d.ts",
         "default": "./dist/index.mjs"
       },
       "require": {
@@ -22,7 +22,7 @@
     },
     "./*": {
       "import": {
-        "types": "./dist/*.d.mts",
+        "types": "./dist/*.d.ts",
         "default": "./dist/*.mjs"
       },
       "require": {
@@ -32,7 +32,7 @@
     },
     "./react": {
       "import": {
-        "types": "./dist/react/index.d.mts",
+        "types": "./dist/react/index.d.ts",
         "default": "./dist/react/index.mjs"
       },
       "require": {
@@ -42,7 +42,7 @@
     },
     "./utils": {
       "import": {
-        "types": "./dist/utils/index.d.mts",
+        "types": "./dist/utils/index.d.ts",
         "default": "./dist/utils/index.mjs"
       },
       "require": {
@@ -52,7 +52,7 @@
     },
     "./workerTimers": {
       "import": {
-        "types": "./dist/workerTimers/index.d.mts",
+        "types": "./dist/workerTimers/index.d.ts",
         "default": "./dist/workerTimers/index.mjs"
       },
       "require": {
@@ -62,7 +62,7 @@
     },
     "./dom": {
       "import": {
-        "types": "./dist/dom/index.d.mts",
+        "types": "./dist/dom/index.d.ts",
         "default": "./dist/dom/index.mjs"
       },
       "require": {
@@ -121,12 +121,12 @@
   "scripts": {
     "build": "tsup",
     "postbuild": "node ../../scripts/subpath-workaround.mjs shared",
-    "clean": "rimraf ./dist",
+    "build:declarations": "tsc -p tsconfig.declarations.json",
     "dev": "tsup --watch",
     "dev:publish": "pnpm dev -- --env.publish",
     "postinstall": "node ./scripts/postinstall.mjs",
     "lint": "eslint src",
-    "lint:attw": "attw --pack .",
+    "lint:attw": "attw --pack . --ignore-rules false-cjs",
     "lint:publint": "publint",
     "publish:local": "pnpm yalc push --replace  --sig",
     "test": "jest",

packages/shared/tsconfig.declarations.json

@@ -0,0 +1,11 @@
+{
+  "extends": "./tsconfig.json",
+  "compilerOptions": {
+    "declaration": true,
+    "declarationDir": "./dist",
+    "declarationMap": true,
+    "emitDeclarationOnly": true,
+    "skipLibCheck": true
+  },
+  "exclude": ["**/__tests__/**/*"]
+}

packages/shared/tsconfig.json

@@ -9,12 +9,8 @@
     "preserveWatchOutput": true,
     "skipLibCheck": true,
     "strict": true,
-    "emitDeclarationOnly": true,
-    "declaration": true,
-    "declarationMap": true,
     "outDir": "dist",
     "resolveJsonModule": true,
-    "declarationDir": "dist/types",
     "jsx": "react",
     "lib": ["ES6", "DOM", "WebWorker"],
     "allowJs": true

packages/shared/tsup.config.ts

@@ -23,7 +23,7 @@ export default defineConfig(overrideOptions => {
     clean: true,
     minify: false,
     sourcemap: true,
-    dts: true,
+    onSuccess: 'pnpm build:declarations',
     target: 'es2020',
     external: ['react', 'react-dom'],
     esbuildPlugins: [WebWorkerMinifyPlugin as any],

packages/types/package.json

@@ -25,18 +25,19 @@
   "author": "Clerk",
   "main": "dist/index.js",
   "module": "dist/esm/index.js",
-  "types": "dist/index.d.ts",
+  "types": "dist/types/index.d.ts",
   "files": [
     "dist"
   ],
   "scripts": {
     "build": "tsup --env.NODE_ENV production",
-    "clean": "rimraf ./dist",
+    "build:declarations": "tsc -p tsconfig.declarations.json",
     "dev": "tsup --watch",
-    "lint": "eslint src"
+    "lint": "eslint src/",
+    "lint:attw": "attw --pack ."
   },
   "dependencies": {
-    "csstype": "3.1.3"
+    "csstype": "3.1.1"
   },
   "devDependencies": {},
   "engines": {

packages/types/tsconfig.declarations.json

@@ -0,0 +1,11 @@
+{
+  "extends": "./tsconfig.json",
+  "compilerOptions": {
+    "declaration": true,
+    "declarationDir": "./dist/types",
+    "declarationMap": true,
+    "emitDeclarationOnly": true,
+    "skipLibCheck": true
+  },
+  "exclude": ["**/__tests__/**/*"]
+}

packages/types/tsconfig.json

@@ -3,8 +3,8 @@
     "outDir": "./dist",
     "baseUrl": "./src",
     "lib": ["es6", "dom"],
-    "module": "commonjs",
-    "moduleResolution": "node",
+    "module": "nodenext",
+    "moduleResolution": "nodenext",
     "declaration": true,
     "declarationMap": true,
     "noImplicitReturns": true,

packages/types/tsup.config.ts

@@ -10,6 +10,6 @@ export default defineConfig(() => {
     sourcemap: true,
     format: ['cjs', 'esm'],
     legacyOutput: true,
-    dts: true,
+    onSuccess: 'pnpm build:declarations',
   };
 });