fix: re-export ESM types in CommonJS (#416)

Author: fasttime

packages/config-array/rollup.config.js

@@ -16,8 +16,10 @@ export default {
 	plugins: [
 		copy({
 			targets: [
-				{ src: "src/types.ts", dest: "dist/cjs", rename: "types.cts" },
-				{ src: "src/types.ts", dest: "dist/esm" },
+				{
+					src: "src/types.ts",
+					dest: "dist/esm",
+				},
 			],
 		}),
 	],

packages/config-helpers/rollup.config.js

@@ -16,8 +16,10 @@ export default {
 	plugins: [
 		copy({
 			targets: [
-				{ src: "src/types.ts", dest: "dist/cjs", rename: "types.cts" },
-				{ src: "src/types.ts", dest: "dist/esm" },
+				{
+					src: "src/types.ts",
+					dest: "dist/esm",
+				},
 			],
 		}),
 	],

packages/core/package.json

@@ -17,7 +17,7 @@
     "access": "public"
   },
   "scripts": {
-    "build:cts": "node -e \"fs.cpSync('dist/esm/types.d.ts', 'dist/cjs/types.d.cts')\"",
+    "build:cts": "node ../../tools/build-cts.js dist/esm/types.d.ts dist/cjs/types.d.cts",
     "build": "tsc && npm run build:cts",
     "lint:types": "attw --pack",
     "pretest": "npm run build",

packages/object-schema/rollup.config.js

@@ -16,8 +16,10 @@ export default {
 	plugins: [
 		copy({
 			targets: [
-				{ src: "src/types.ts", dest: "dist/cjs", rename: "types.cts" },
-				{ src: "src/types.ts", dest: "dist/esm" },
+				{
+					src: "src/types.ts",
+					dest: "dist/esm",
+				},
 			],
 		}),
 	],

packages/plugin-kit/rollup.config.js

@@ -16,8 +16,10 @@ export default {
 	plugins: [
 		copy({
 			targets: [
-				{ src: "src/types.ts", dest: "dist/cjs", rename: "types.cts" },
-				{ src: "src/types.ts", dest: "dist/esm" },
+				{
+					src: "src/types.ts",
+					dest: "dist/esm",
+				},
 			],
 		}),
 	],

templates/package/rollup.config.js

@@ -16,8 +16,10 @@ export default {
 	plugins: [
 		copy({
 			targets: [
-				{ src: "src/types.ts", dest: "dist/cjs", rename: "types.cts" },
-				{ src: "src/types.ts", dest: "dist/esm" },
+				{
+					src: "src/types.ts",
+					dest: "dist/esm",
+				},
 			],
 		}),
 	],

tools/build-cts.js

@@ -1,14 +1,14 @@
 /**
- * @fileoverview Rewrites import expressions for CommonJS compatibility.
- * This script creates "dist/cjs/index.d.cts" from "dist/esm/index.d.ts" by modifying imports
- * from `"./types.ts"` to `"./types.cts"`.
+ * @fileoverview Creates a CommonJS type declaration file that re-exports ESM types.
  *
- *    node tools/build-cts.js /path/to/esm/index.d.ts path/to/cjs/index.d.cts
+ * Usage:
+ *    node tools/build-cts.js /path/to/esm/index.d.ts /path/to/cjs/index.d.cts
  *
  * @author Francesco Trotta
  */
 
-import { readFile, writeFile } from "node:fs/promises";
+import { mkdir, writeFile } from "node:fs/promises";
+import path from "node:path/posix";
 
 const filename = process.argv[2];
 const newFilename = process.argv[3];
@@ -23,7 +23,12 @@ if (!newFilename) {
 	process.exit(1);
 }
 
-const oldSourceText = await readFile(filename, "utf-8");
-const newSourceText = oldSourceText.replaceAll('"./types.ts"', '"./types.cts"');
+const newDir = path.dirname(newFilename);
+const esmPath = path.relative(newDir, filename).replace(/\.d\.ts$/u, ".js");
+const newSourceText = `
+import type * as types from "${esmPath}" with { "resolution-mode": "import" };
+export = types;
+`.trimStart();
 
+await mkdir(newDir, { recursive: true });
 await writeFile(newFilename, newSourceText);