@knighted/duel 4.0.0 → 4.0.2-rc.0

package/dist/cjs/duel.cjs

@@ -9,7 +9,6 @@ const node_child_process_1 = require("node:child_process");
 const promises_1 = require("node:fs/promises");
 const node_crypto_1 = require("node:crypto");
 const node_perf_hooks_1 = require("node:perf_hooks");
-const glob_1 = require("glob");
 const find_up_1 = require("find-up");
 const module_1 = require("@knighted/module");
 const get_tsconfig_1 = require("get-tsconfig");
@@ -44,6 +43,13 @@ const logDiagnostics = (diags, projectDir, hazardAllowlist = null) => {
     }
     return hasError;
 };
+const collectGlob = async (pattern, options) => {
+    const files = [];
+    for await (const file of (0, promises_1.glob)(pattern, options)) {
+        files.push(file);
+    }
+    return files;
+};
 const duel = async (args) => {
     const ctx = await (0, init_js_1.init)(args);
     if (ctx) {
@@ -587,7 +593,7 @@ const duel = async (args) => {
                  * Transform ambiguous modules for the target dual build.
                  * @see https://github.com/microsoft/TypeScript/issues/58658
                  */
-                const toTransform = await (0, glob_1.glob)(`${subDir.replace(/\\/g, '/')}/**/*{.js,.jsx,.ts,.tsx}`, {
+                const toTransform = await collectGlob(`${subDir.replace(/\\/g, '/')}/**/*{.js,.jsx,.ts,.tsx}`, {
                     ignore: `${subDir.replace(/\\/g, '/')}/**/node_modules/**`,
                 });
                 let transformDiagnosticsError = false;
@@ -659,12 +665,14 @@ const duel = async (args) => {
                 }
                 catch (err) {
                     if (err?.code === 'ENOENT') {
-                        throw new Error(`Dual build output not found at ${shadowDualOutDir}`);
+                        throw new Error(`Dual build output not found at ${shadowDualOutDir}`, {
+                            cause: err,
+                        });
                     }
                     throw err;
                 }
                 const dualGlob = dualTarget === 'commonjs' ? '**/*{.js,.cjs,.d.ts}' : '**/*{.js,.mjs,.d.ts}';
-                const filenames = await (0, glob_1.glob)(`${absoluteDualOutDir.replace(/\\/g, '/')}/${dualGlob}`, {
+                const filenames = await collectGlob(`${absoluteDualOutDir.replace(/\\/g, '/')}/${dualGlob}`, {
                     ignore: `${absoluteDualOutDir.replace(/\\/g, '/')}/**/node_modules/**`,
                 });
                 const rewriteSyntaxMode = dualTarget === 'commonjs' ? true : syntaxMode;
@@ -687,7 +695,7 @@ const duel = async (args) => {
                     onRewrite: (from, to) => logVerbose(`Rewrote specifiers in ${from} -> ${to}`),
                 });
                 if (dirs && originalType === 'commonjs') {
-                    const primaryFiles = await (0, glob_1.glob)(`${primaryOutDir.replace(/\\/g, '/')}/**/*{.js,.cjs,.d.ts}`, {
+                    const primaryFiles = await collectGlob(`${primaryOutDir.replace(/\\/g, '/')}/**/*{.js,.cjs,.d.ts}`, {
                         ignore: `${primaryOutDir.replace(/\\/g, '/')}/**/node_modules/**`,
                     });
                     await (0, resolver_js_1.rewriteSpecifiersAndExtensions)(primaryFiles, {

package/dist/cjs/init.cjs

@@ -99,7 +99,7 @@ const printHelp = () => {
     }
 };
 const init = async (args) => {
-    let parsed = null;
+    let parsed;
     try {
         const { values } = (0, node_util_1.parseArgs)({
             args,
@@ -176,8 +176,8 @@ const init = async (args) => {
     else {
         const { project, 'pkg-dir': pkgDir, dirs, exports: exportsOpt, 'exports-config': exportsConfig, 'exports-validate': exportsValidate, 'rewrite-policy': rewritePolicy, 'detect-dual-package-hazard': detectDualPackageHazard, 'dual-package-hazard-allowlist': dualPackageHazardAllowlist, 'dual-package-hazard-scope': dualPackageHazardScope, verbose, mode, 'copy-mode': copyMode, } = parsed;
         let configPath = (0, node_path_1.resolve)(project);
-        let stats = null;
-        let pkg = null;
+        let stats;
+        let pkg;
         if (mode && !['none', 'globals', 'full'].includes(mode)) {
             (0, util_js_1.logError)('--mode expects one of: none | globals | full');
             return false;

package/dist/cjs/util.cjs

@@ -7,7 +7,6 @@ const node_fs_1 = require("node:fs");
 const node_child_process_1 = require("node:child_process");
 const node_process_1 = require("node:process");
 const node_path_1 = require("node:path");
-const glob_1 = require("glob");
 const find_up_1 = require("find-up");
 const COLORS = {
     reset: '\x1b[0m',
@@ -181,12 +180,14 @@ const readExportsConfig = async (configPath, pkgDir) => {
             ? (0, node_path_1.resolve)(pkgDir, configPath)
             : (0, node_path_1.resolve)((0, node_process_1.cwd)(), configPath);
     const raw = await (0, promises_1.readFile)(abs, 'utf8');
-    let parsed = null;
+    let parsed;
     try {
         parsed = JSON.parse(raw);
     }
     catch (err) {
-        throw new Error(`Invalid JSON in --exports-config (${configPath}): ${err.message}`);
+        throw new Error(`Invalid JSON in --exports-config (${configPath}): ${err.message}`, {
+            cause: err,
+        });
     }
     const { entries, main } = parsed;
     if (!entries ||
@@ -279,7 +280,12 @@ const generateExports = async (options) => {
             return;
         }
         const baseEntry = baseMap.get(baseKey) ?? {};
-        baseEntry[kind] = withDot;
+        if (kind === 'types') {
+            baseEntry.types = baseEntry.types ?? withDot;
+        }
+        else {
+            baseEntry[kind] = withDot;
+        }
         baseMap.set(baseKey, baseEntry);
         const subpath = useEntriesSubpaths
             ? ensureDotSlash(stripKnownExt(relFromRoot))
@@ -289,7 +295,8 @@ const generateExports = async (options) => {
             const mappedSubpath = baseToSubpath.get(baseKey);
             if (mappedSubpath) {
                 const subEntry = subpathMap.get(mappedSubpath) ?? {};
-                subEntry.types = useWildcard ? toWildcardValue(withDot) : withDot;
+                const nextType = useWildcard ? toWildcardValue(withDot) : withDot;
+                subEntry.types = subEntry.types ?? nextType;
                 subpathMap.set(mappedSubpath, subEntry);
             }
             return;
@@ -301,10 +308,9 @@ const generateExports = async (options) => {
             baseToSubpath.set(baseKey, subpath);
         }
     };
-    const esmFiles = await (0, glob_1.glob)(`${esmRootPosix}/**/*.{js,mjs,d.ts,d.mts}`, {
+    for await (const file of (0, promises_1.glob)(`${esmRootPosix}/**/*.{js,mjs,d.ts,d.mts}`, {
         ignore: esmIgnore,
-    });
-    for (const file of esmFiles) {
+    })) {
         if (/\.d\.(ts|mts)$/.test(file)) {
             recordPath('types', file, esmRoot);
         }
@@ -312,10 +318,9 @@ const generateExports = async (options) => {
             recordPath('import', file, esmRoot);
         }
     }
-    const cjsFiles = await (0, glob_1.glob)(`${cjsRootPosix}/**/*.{js,cjs,d.ts,d.cts}`, {
+    for await (const file of (0, promises_1.glob)(`${cjsRootPosix}/**/*.{js,cjs,d.ts,d.cts}`, {
         ignore: cjsIgnore,
-    });
-    for (const file of cjsFiles) {
+    })) {
         if (/\.d\.(ts|cts)$/.test(file)) {
             recordPath('types', file, cjsRoot);
         }

package/dist/esm/duel.js

@@ -3,10 +3,9 @@ import { argv } from 'node:process';
 import { pathToFileURL } from 'node:url';
 import { join, dirname, resolve, relative, sep } from 'node:path';
 import { spawn } from 'node:child_process';
-import { writeFile, rm, mkdir, cp, access, readdir } from 'node:fs/promises';
+import { writeFile, rm, mkdir, cp, access, readdir, glob } from 'node:fs/promises';
 import { createHash } from 'node:crypto';
 import { performance } from 'node:perf_hooks';
-import { glob } from 'glob';
 import { findUp } from 'find-up';
 import { transform, collectProjectDualPackageHazards } from '@knighted/module';
 import { getTsconfig, parseTsconfig } from 'get-tsconfig';
@@ -41,6 +40,13 @@ const logDiagnostics = (diags, projectDir, hazardAllowlist = null) => {
     }
     return hasError;
 };
+const collectGlob = async (pattern, options) => {
+    const files = [];
+    for await (const file of glob(pattern, options)) {
+        files.push(file);
+    }
+    return files;
+};
 const duel = async (args) => {
     const ctx = await init(args);
     if (ctx) {
@@ -584,7 +590,7 @@ const duel = async (args) => {
                  * Transform ambiguous modules for the target dual build.
                  * @see https://github.com/microsoft/TypeScript/issues/58658
                  */
-                const toTransform = await glob(`${subDir.replace(/\\/g, '/')}/**/*{.js,.jsx,.ts,.tsx}`, {
+                const toTransform = await collectGlob(`${subDir.replace(/\\/g, '/')}/**/*{.js,.jsx,.ts,.tsx}`, {
                     ignore: `${subDir.replace(/\\/g, '/')}/**/node_modules/**`,
                 });
                 let transformDiagnosticsError = false;
@@ -656,12 +662,14 @@ const duel = async (args) => {
                 }
                 catch (err) {
                     if (err?.code === 'ENOENT') {
-                        throw new Error(`Dual build output not found at ${shadowDualOutDir}`);
+                        throw new Error(`Dual build output not found at ${shadowDualOutDir}`, {
+                            cause: err,
+                        });
                     }
                     throw err;
                 }
                 const dualGlob = dualTarget === 'commonjs' ? '**/*{.js,.cjs,.d.ts}' : '**/*{.js,.mjs,.d.ts}';
-                const filenames = await glob(`${absoluteDualOutDir.replace(/\\/g, '/')}/${dualGlob}`, {
+                const filenames = await collectGlob(`${absoluteDualOutDir.replace(/\\/g, '/')}/${dualGlob}`, {
                     ignore: `${absoluteDualOutDir.replace(/\\/g, '/')}/**/node_modules/**`,
                 });
                 const rewriteSyntaxMode = dualTarget === 'commonjs' ? true : syntaxMode;
@@ -684,7 +692,7 @@ const duel = async (args) => {
                     onRewrite: (from, to) => logVerbose(`Rewrote specifiers in ${from} -> ${to}`),
                 });
                 if (dirs && originalType === 'commonjs') {
-                    const primaryFiles = await glob(`${primaryOutDir.replace(/\\/g, '/')}/**/*{.js,.cjs,.d.ts}`, {
+                    const primaryFiles = await collectGlob(`${primaryOutDir.replace(/\\/g, '/')}/**/*{.js,.cjs,.d.ts}`, {
                         ignore: `${primaryOutDir.replace(/\\/g, '/')}/**/node_modules/**`,
                     });
                     await rewriteSpecifiersAndExtensions(primaryFiles, {

package/dist/esm/init.js

@@ -96,7 +96,7 @@ const printHelp = () => {
     }
 };
 const init = async (args) => {
-    let parsed = null;
+    let parsed;
     try {
         const { values } = parseArgs({
             args,
@@ -173,8 +173,8 @@ const init = async (args) => {
     else {
         const { project, 'pkg-dir': pkgDir, dirs, exports: exportsOpt, 'exports-config': exportsConfig, 'exports-validate': exportsValidate, 'rewrite-policy': rewritePolicy, 'detect-dual-package-hazard': detectDualPackageHazard, 'dual-package-hazard-allowlist': dualPackageHazardAllowlist, 'dual-package-hazard-scope': dualPackageHazardScope, verbose, mode, 'copy-mode': copyMode, } = parsed;
         let configPath = resolve(project);
-        let stats = null;
-        let pkg = null;
+        let stats;
+        let pkg;
         if (mode && !['none', 'globals', 'full'].includes(mode)) {
             logError('--mode expects one of: none | globals | full');
             return false;

package/dist/esm/util.js

@@ -1,10 +1,9 @@
 import { pathToFileURL } from 'node:url';
-import { realpath, readFile, writeFile, symlink, rm } from 'node:fs/promises';
+import { realpath, readFile, writeFile, symlink, rm, glob } from 'node:fs/promises';
 import { existsSync, rmSync } from 'node:fs';
 import { spawnSync } from 'node:child_process';
 import { cwd, platform } from 'node:process';
 import { join, resolve, relative, parse as parsePath, posix, isAbsolute, sep, normalize as normalizePath, } from 'node:path';
-import { glob } from 'glob';
 import { findUp } from 'find-up';
 const COLORS = {
     reset: '\x1b[0m',
@@ -168,12 +167,14 @@ const readExportsConfig = async (configPath, pkgDir) => {
             ? resolve(pkgDir, configPath)
             : resolve(cwd(), configPath);
     const raw = await readFile(abs, 'utf8');
-    let parsed = null;
+    let parsed;
     try {
         parsed = JSON.parse(raw);
     }
     catch (err) {
-        throw new Error(`Invalid JSON in --exports-config (${configPath}): ${err.message}`);
+        throw new Error(`Invalid JSON in --exports-config (${configPath}): ${err.message}`, {
+            cause: err,
+        });
     }
     const { entries, main } = parsed;
     if (!entries ||
@@ -264,7 +265,12 @@ const generateExports = async (options) => {
             return;
         }
         const baseEntry = baseMap.get(baseKey) ?? {};
-        baseEntry[kind] = withDot;
+        if (kind === 'types') {
+            baseEntry.types = baseEntry.types ?? withDot;
+        }
+        else {
+            baseEntry[kind] = withDot;
+        }
         baseMap.set(baseKey, baseEntry);
         const subpath = useEntriesSubpaths
             ? ensureDotSlash(stripKnownExt(relFromRoot))
@@ -274,7 +280,8 @@ const generateExports = async (options) => {
             const mappedSubpath = baseToSubpath.get(baseKey);
             if (mappedSubpath) {
                 const subEntry = subpathMap.get(mappedSubpath) ?? {};
-                subEntry.types = useWildcard ? toWildcardValue(withDot) : withDot;
+                const nextType = useWildcard ? toWildcardValue(withDot) : withDot;
+                subEntry.types = subEntry.types ?? nextType;
                 subpathMap.set(mappedSubpath, subEntry);
             }
             return;
@@ -286,10 +293,9 @@ const generateExports = async (options) => {
             baseToSubpath.set(baseKey, subpath);
         }
     };
-    const esmFiles = await glob(`${esmRootPosix}/**/*.{js,mjs,d.ts,d.mts}`, {
+    for await (const file of glob(`${esmRootPosix}/**/*.{js,mjs,d.ts,d.mts}`, {
         ignore: esmIgnore,
-    });
-    for (const file of esmFiles) {
+    })) {
         if (/\.d\.(ts|mts)$/.test(file)) {
             recordPath('types', file, esmRoot);
         }
@@ -297,10 +303,9 @@ const generateExports = async (options) => {
             recordPath('import', file, esmRoot);
         }
     }
-    const cjsFiles = await glob(`${cjsRootPosix}/**/*.{js,cjs,d.ts,d.cts}`, {
+    for await (const file of glob(`${cjsRootPosix}/**/*.{js,cjs,d.ts,d.cts}`, {
         ignore: cjsIgnore,
-    });
-    for (const file of cjsFiles) {
+    })) {
         if (/\.d\.(ts|cts)$/.test(file)) {
             recordPath('types', file, cjsRoot);
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@knighted/duel",
-  "version": "4.0.0",
+  "version": "4.0.2-rc.0",
   "description": "TypeScript dual packages.",
   "type": "module",
   "main": "dist/esm/duel.js",
@@ -70,13 +70,13 @@
     }
   },
   "devDependencies": {
-    "@eslint/js": "^9.39.1",
+    "@eslint/js": "^10.0.1",
     "@tsconfig/recommended": "^1.0.10",
     "@types/node": "^24.10.1",
-    "c8": "^10.1.3",
+    "c8": "^11.0.0",
     "cross-spawn": "^7.0.6",
-    "eslint": "^9.39.1",
-    "eslint-plugin-n": "^17.23.1",
+    "eslint": "^10.0.3",
+    "eslint-plugin-n": "^17.24.0",
     "globals": "^16.3.0",
     "husky": "^9.1.7",
     "lint-staged": "^16.2.7",
@@ -84,7 +84,7 @@
     "prettier": "^3.6.2",
     "tsx": "^4.20.6",
     "typescript": "^5.9.3",
-    "vite": "^7.2.4"
+    "vite": "^7.3.1"
   },
   "dependencies": {
     "@jridgewell/gen-mapping": "^0.3.13",
@@ -92,7 +92,6 @@
     "@knighted/module": "^1.5.0",
     "find-up": "^8.0.0",
     "get-tsconfig": "^4.13.0",
-    "glob": "^13.0.0",
     "magic-string": "^0.30.21",
     "read-package-up": "^12.0.0"
   },