@knighted/duel 1.0.0-alpha.3 → 1.0.0-alpha.4

package/README.md

@@ -6,7 +6,7 @@
 
 Node.js tool for creating a TypeScript dual package.
 
-Early stages of development. Inspired by https://github.com/microsoft/TypeScript/issues/49462.
+Inspired by https://github.com/microsoft/TypeScript/issues/49462.
 
 ## Requirements
 
@@ -79,8 +79,8 @@ Options:
 
 ## Gotchas
 
-* Unfortunately, TypeScript doesn't really build [dual packages](https://nodejs.org/api/packages.html#dual-commonjses-module-packages) very well in regards to preserving module system by file extension. For instance, it will **always** create CJS exports when `--module commonjs` is used, _even on files with an `.mts` extension_, which is contrary to [how Node determines module systems](https://nodejs.org/api/packages.html#determining-module-system). The `tsc` compiler is fundamentally broken in this regard. One reference issue is https://github.com/microsoft/TypeScript/issues/54573. If you use `.mts` extensions to enforce an ESM module system, this will break in the corresponding dual CJS build. There is no way to fix this until TypeScript fixes their compiler.
+These are definitely edge cases, and would only really come up if your project mixes file extensions. For example, if you have `.ts` files combined with `.mts`, and/or `.cts`. For most project, things should just work as expected.
 
-* If targeting a dual CJS build, and you are using [top level `await`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/await#top_level_await), you will most likely encounter the compilation error `error TS1378: Top-level 'await' expressions are only allowed when the 'module' option is set to 'es2022', 'esnext', 'system', 'node16', or 'nodenext', and the 'target' option is set to 'es2017' or higher.` during the CJS build. This is because `duel` creates a temporary `tsconfig.json` from your original and necessarily overwrites the `--module` and `--moduleResolution` based on the provided `--target-ext`. There is no workaround other than to **not** use top level await if you want a dual build.
+* Unfortunately, TypeScript doesn't really build [dual packages](https://nodejs.org/api/packages.html#dual-commonjses-module-packages) very well in regards to preserving module system by file extension. For instance, there doesn't appear to be a way to convert an arbitrary `.ts` file into another module system, _while also preserving the module system of `.mts` and `.cts` files_. In my opinion, the `tsc` compiler is fundamentally broken in this regard, and at best is enforcing usage patterns it shouldn't. If you want to see one of my extended rants on this, check out this [comment](https://github.com/microsoft/TypeScript/pull/50985#issuecomment-1656991606). This is only mentioned for transparency, `duel` will correct for this and produce files with the module system you would expect based on the files extension, so that it works with [how Node.js determines module systems](https://nodejs.org/api/packages.html#determining-module-system).
 
 * If doing an `import type` across module systems, i.e. from `.mts` into `.cts`, or vice versa, you might encounter the compilation error ``error TS1452: 'resolution-mode' assertions are only supported when `moduleResolution` is `node16` or `nodenext`.``. This is a [known issue](https://github.com/microsoft/TypeScript/issues/49055) and TypeScript currently suggests installing the nightly build, i.e. `npm i typescript@next`.

package/dist/duel.cjs

@@ -15,7 +15,6 @@ var _glob = require("glob");
 var _specifier = require("@knighted/specifier");
 var _init = require("./init.cjs");
 var _util = require("./util.cjs");
-// TypeScript is defined as a peer dependency.
 const tsc = (0, _nodePath.join)((0, _nodeProcess.cwd)(), 'node_modules', '.bin', 'tsc');
 const runBuild = project => {
   const {
@@ -45,7 +44,8 @@ const duel = async args => {
       projectDir,
       tsconfig,
       targetExt,
-      configPath
+      configPath,
+      absoluteOutDir
     } = ctx;
     const startTime = _nodePerf_hooks.performance.now();
     (0, _util.log)('Starting primary build...\n');
@@ -58,13 +58,15 @@ const duel = async args => {
       } = tsconfig.compilerOptions;
       const dualConfigPath = (0, _nodePath.join)(projectDir, `tsconfig.${hex}.json`);
       const dualOutDir = isCjsBuild ? (0, _nodePath.join)(outDir, 'cjs') : (0, _nodePath.join)(outDir, 'mjs');
+      // Using structuredClone() would require node >= 17.0.0
       const tsconfigDual = {
         ...tsconfig,
         compilerOptions: {
           ...tsconfig.compilerOptions,
           outDir: dualOutDir,
-          module: isCjsBuild ? 'CommonJS' : 'NodeNext',
-          moduleResolution: isCjsBuild ? 'Node' : 'NodeNext'
+          module: isCjsBuild ? 'CommonJS' : 'ESNext',
+          // Best way to make this work given how tsc works
+          moduleResolution: 'Node'
         }
       };
       await (0, _promises.writeFile)(dualConfigPath, JSON.stringify(tsconfigDual, null, 2));
@@ -74,7 +76,8 @@ const duel = async args => {
         force: true
       });
       if (success) {
-        const filenames = await (0, _glob.glob)(`${(0, _nodePath.join)(projectDir, dualOutDir)}/**/*{.js,.d.ts}`, {
+        const absoluteDualOutDir = (0, _nodePath.join)(projectDir, dualOutDir);
+        const filenames = await (0, _glob.glob)(`${absoluteDualOutDir}/**/*{.js,.d.ts}`, {
           ignore: 'node_modules/**'
         });
         for (const filename of filenames) {
@@ -96,6 +99,49 @@ const duel = async args => {
             force: true
           });
         }
+
+        /**
+         * This is a fix for tsc compiler which doesn't seem to support
+         * converting an arbitrary `.ts` file, into another module system,
+         * while also preserving the module systems of `.mts` and `.cts` files.
+         *
+         * Hopefully it can be removed when TS updates their supported options,
+         * or at least how the combination of `--module` and `--moduleResolution`
+         * currently work.
+         *
+         * @see https://github.com/microsoft/TypeScript/pull/50985#issuecomment-1656991606
+         */
+        if (isCjsBuild) {
+          const mjsFiles = await (0, _glob.glob)(`${absoluteOutDir}/**/*.mjs`, {
+            ignore: ['node_modules/**', `${absoluteDualOutDir}/**`]
+          });
+          for (const filename of mjsFiles) {
+            const relativeFn = (0, _nodePath.relative)(absoluteOutDir, filename);
+            await (0, _promises.copyFile)(filename, (0, _nodePath.join)(absoluteDualOutDir, relativeFn));
+          }
+        } else {
+          const cjsFiles = await (0, _glob.glob)(`${absoluteOutDir}/**/*.cjs`, {
+            ignore: ['node_modules/**', `${absoluteDualOutDir}/**`]
+          });
+          for (const filename of cjsFiles) {
+            const relativeFn = (0, _nodePath.relative)(absoluteOutDir, filename);
+            await (0, _promises.copyFile)(filename, (0, _nodePath.join)(absoluteDualOutDir, relativeFn));
+          }
+
+          /**
+           * Now copy the good .mjs files from the dual out dir
+           * to the original out dir, but build the file path
+           * from the original out dir to distinguish from the
+           * dual build .mjs files.
+           */
+          const mjsFiles = await (0, _glob.glob)(`${absoluteOutDir}/**/*.mjs`, {
+            ignore: ['node_modules/**', `${absoluteDualOutDir}/**`]
+          });
+          for (const filename of mjsFiles) {
+            const relativeFn = (0, _nodePath.relative)(absoluteOutDir, filename);
+            await (0, _promises.copyFile)((0, _nodePath.join)(absoluteDualOutDir, relativeFn), filename);
+          }
+        }
         (0, _util.log)(`Successfully created a dual ${targetExt.replace('.', '').toUpperCase()} build in ${Math.round(_nodePerf_hooks.performance.now() - startTime)}ms.`);
       }
     }

package/dist/duel.js

@@ -1,16 +1,14 @@
 #!/usr/bin/env node
 import { argv, cwd } from 'node:process';
-import { join } from 'node:path';
+import { join, relative } from 'node:path';
 import { spawnSync } from 'node:child_process';
-import { writeFile, rm } from 'node:fs/promises';
+import { writeFile, copyFile, rm } from 'node:fs/promises';
 import { randomBytes } from 'node:crypto';
 import { performance } from 'node:perf_hooks';
 import { glob } from 'glob';
 import { specifier } from '@knighted/specifier';
 import { init } from './init.js';
 import { getRealPathAsFileUrl, logError, log } from './util.js';
-
-// TypeScript is defined as a peer dependency.
 const tsc = join(cwd(), 'node_modules', '.bin', 'tsc');
 const runBuild = project => {
   const {
@@ -40,7 +38,8 @@ const duel = async args => {
       projectDir,
       tsconfig,
       targetExt,
-      configPath
+      configPath,
+      absoluteOutDir
     } = ctx;
     const startTime = performance.now();
     log('Starting primary build...\n');
@@ -53,13 +52,15 @@ const duel = async args => {
       } = tsconfig.compilerOptions;
       const dualConfigPath = join(projectDir, `tsconfig.${hex}.json`);
       const dualOutDir = isCjsBuild ? join(outDir, 'cjs') : join(outDir, 'mjs');
+      // Using structuredClone() would require node >= 17.0.0
       const tsconfigDual = {
         ...tsconfig,
         compilerOptions: {
           ...tsconfig.compilerOptions,
           outDir: dualOutDir,
-          module: isCjsBuild ? 'CommonJS' : 'NodeNext',
-          moduleResolution: isCjsBuild ? 'Node' : 'NodeNext'
+          module: isCjsBuild ? 'CommonJS' : 'ESNext',
+          // Best way to make this work given how tsc works
+          moduleResolution: 'Node'
         }
       };
       await writeFile(dualConfigPath, JSON.stringify(tsconfigDual, null, 2));
@@ -69,7 +70,8 @@ const duel = async args => {
         force: true
       });
       if (success) {
-        const filenames = await glob(`${join(projectDir, dualOutDir)}/**/*{.js,.d.ts}`, {
+        const absoluteDualOutDir = join(projectDir, dualOutDir);
+        const filenames = await glob(`${absoluteDualOutDir}/**/*{.js,.d.ts}`, {
           ignore: 'node_modules/**'
         });
         for (const filename of filenames) {
@@ -91,6 +93,49 @@ const duel = async args => {
             force: true
           });
         }
+
+        /**
+         * This is a fix for tsc compiler which doesn't seem to support
+         * converting an arbitrary `.ts` file, into another module system,
+         * while also preserving the module systems of `.mts` and `.cts` files.
+         *
+         * Hopefully it can be removed when TS updates their supported options,
+         * or at least how the combination of `--module` and `--moduleResolution`
+         * currently work.
+         *
+         * @see https://github.com/microsoft/TypeScript/pull/50985#issuecomment-1656991606
+         */
+        if (isCjsBuild) {
+          const mjsFiles = await glob(`${absoluteOutDir}/**/*.mjs`, {
+            ignore: ['node_modules/**', `${absoluteDualOutDir}/**`]
+          });
+          for (const filename of mjsFiles) {
+            const relativeFn = relative(absoluteOutDir, filename);
+            await copyFile(filename, join(absoluteDualOutDir, relativeFn));
+          }
+        } else {
+          const cjsFiles = await glob(`${absoluteOutDir}/**/*.cjs`, {
+            ignore: ['node_modules/**', `${absoluteDualOutDir}/**`]
+          });
+          for (const filename of cjsFiles) {
+            const relativeFn = relative(absoluteOutDir, filename);
+            await copyFile(filename, join(absoluteDualOutDir, relativeFn));
+          }
+
+          /**
+           * Now copy the good .mjs files from the dual out dir
+           * to the original out dir, but build the file path
+           * from the original out dir to distinguish from the
+           * dual build .mjs files.
+           */
+          const mjsFiles = await glob(`${absoluteOutDir}/**/*.mjs`, {
+            ignore: ['node_modules/**', `${absoluteDualOutDir}/**`]
+          });
+          for (const filename of mjsFiles) {
+            const relativeFn = relative(absoluteOutDir, filename);
+            await copyFile(join(absoluteDualOutDir, relativeFn), filename);
+          }
+        }
         log(`Successfully created a dual ${targetExt.replace('.', '').toUpperCase()} build in ${Math.round(performance.now() - startTime)}ms.`);
       }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@knighted/duel",
-  "version": "1.0.0-alpha.3",
+  "version": "1.0.0-alpha.4",
   "description": "TypeScript dual packages.",
   "type": "module",
   "main": "dist",
@@ -44,7 +44,7 @@
     "url": "https://github.com/knightedcodemonkey/duel/issues"
   },
   "peerDependencies": {
-    "typescript": "^5.0.0"
+    "typescript": ">=4.0.0"
   },
   "devDependencies": {
     "babel-dual-package": "^1.0.0-rc.5",