@nubjs/loader 0.9.0-canary.20260908.508 → 0.9.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nubjs/loader",
-  "version": "0.9.0-canary.20260908.508",
+  "version": "0.9.0",
   "description": "Standalone TypeScript loader for Node.js from the Nub project — TypeScript, JSX, tsconfig paths, and data-format imports through a native transform, registered the way tsx and ts-node are",
   "license": "MIT",
   "repository": "https://github.com/nubjs/nub",
@@ -49,13 +49,13 @@
     "@oxc-project/runtime": "0.140.0"
   },
   "optionalDependencies": {
-    "@nubjs/loader-darwin-arm64": "0.9.0-canary.20260908.508",
-    "@nubjs/loader-darwin-x64": "0.9.0-canary.20260908.508",
-    "@nubjs/loader-linux-x64": "0.9.0-canary.20260908.508",
-    "@nubjs/loader-linux-x64-musl": "0.9.0-canary.20260908.508",
-    "@nubjs/loader-linux-arm64": "0.9.0-canary.20260908.508",
-    "@nubjs/loader-linux-arm64-musl": "0.9.0-canary.20260908.508",
-    "@nubjs/loader-win32-x64": "0.9.0-canary.20260908.508",
-    "@nubjs/loader-win32-arm64": "0.9.0-canary.20260908.508"
+    "@nubjs/loader-darwin-arm64": "0.9.0",
+    "@nubjs/loader-darwin-x64": "0.9.0",
+    "@nubjs/loader-linux-x64": "0.9.0",
+    "@nubjs/loader-linux-x64-musl": "0.9.0",
+    "@nubjs/loader-linux-arm64": "0.9.0",
+    "@nubjs/loader-linux-arm64-musl": "0.9.0",
+    "@nubjs/loader-win32-x64": "0.9.0",
+    "@nubjs/loader-win32-arm64": "0.9.0"
   }
 }

package/preload-async-hooks.mjs

@@ -30,7 +30,7 @@ import "./floor-builtin.mjs";
 import {
   TRANSPILE_EXTS, PLAIN_JS_EXTS, CLOBBER_MAP, dataExtsFor,
   extname, isFileUrl, resolveSpec, loadTranspile, maybeTranspilePlainJs, loadData, loadTextImport, isDependency,
-  noteRuntimeV8FlagSource, outerHookOwnsFormat,
+  noteRuntimeV8FlagSource,
 } from "./transform-core.mjs";
 import { createRequire, isBuiltin } from "node:module";
 import { existsSync } from "node:fs";
@@ -114,11 +114,6 @@ async function loadInner(url, context, nextLoad) {
   // fast-tier hook.
   if (context?.importAttributes?.type === "text" && isFileUrl(url)) return loadTextImport(url);
   const ext = extname(url);
-  // A user loader above nub's assigned this TypeScript file a bare module format
-  // and will transform it itself from the raw source (tsx's pattern; see
-  // `outerHookOwnsFormat`). This tier has no registration counter like the fast
-  // tier's and needs none: nothing but such a hook produces the bare form here.
-  if (outerHookOwnsFormat(context?.format, ext) && !isDependency(url)) return nextLoad(url, context);
   // node_modules deps are NEVER transpiled (the byte-parity boundary). This guard is
   // make-or-break now that TRANSPILE_EXTS includes `.js`/`.mjs`/`.cjs`: without it,
   // the compat tier would route every dependency `.js` through oxc. (loadTranspile's

package/preload-common.cjs

@@ -231,10 +231,6 @@ function installWatchReporting(core) {
 // `type` (transpile — there is no user hook to do it, and Node's strip-only mode
 // can't handle enums/namespaces). See makeHooks().load.
 let __userHooksRegistered = false;
-// Narrower: a user registration that brought a `load` hook, so it can transform
-// what nub hands back. A resolve-only user hook that labels a `.ts` file with a
-// bare `commonjs`/`module` still relies on nub as the transformer.
-let __userLoadHookRegistered = false;
 function installUserHookDetector() {
   if (typeof module_.registerHooks !== "function") return;
   const orig = module_.registerHooks;
@@ -242,10 +238,7 @@ function installUserHookDetector() {
   let seen = 0;
   const wrapped = function (...args) {
     // Call #1 is nub's own preload registration; #2+ are user hooks.
-    if (seen >= 1) {
-      __userHooksRegistered = true;
-      if (args[0] && typeof args[0].load === "function") __userLoadHookRegistered = true;
-    }
+    if (seen >= 1) __userHooksRegistered = true;
     seen += 1;
     return orig.apply(this, args);
   };
@@ -705,8 +698,8 @@ function makeHooks(core, watchReporting, foreignLoaderFlagPresent = foreignAsync
       const source = readFileSync(path);
       const pkgType = core.getPackageType(dirname(path));
       const format = core.moduleFormatFor(ext, pkgType, path, source.toString("utf8"));
-      if (format === "commonjs") return { format: "commonjs", source: null, responseURL: url, shortCircuit: true };
-      return { format, source, responseURL: url, shortCircuit: true };
+      if (format === "commonjs") return { format: "commonjs", source: null, shortCircuit: true };
+      return { format, source, shortCircuit: true };
     } catch {
       return null;
     }
@@ -780,22 +773,7 @@ function makeHooks(core, watchReporting, foreignLoaderFlagPresent = foreignAsync
     // user's outer hook, which does the real ESM->CJS conversion, matching Node.
     // Native 'module-typescript'/'commonjs-typescript' formats still fall through to
     // nub's transpile below, so normal augmentation is unchanged.
-    //
-    // tsx 4.2x writes the bare `commonjs`/`module` form instead of 'typescript'
-    // (`outerHookOwnsFormat`), and the hook that wrote it must also be able to
-    // transform what it gets back: a user registration WITH a load hook
-    // (`__userLoadHookRegistered`), or a loader registered through
-    // `module.register` (`userAsyncLoaderActive`), which the registerHooks counter
-    // cannot see and which nub has no way to inspect. A resolve-only user hook
-    // that labels a `.ts` file keeps nub as its transformer. The 'typescript'
-    // branch keeps its own narrower gate: a non-transpiling async loader (a
-    // telemetry `--import`) must not turn a bare 'typescript' from Node's own
-    // CJS loader into a step-aside.
-    if (context && (
-      (__userHooksRegistered && context.format === "typescript") ||
-      ((__userLoadHookRegistered || userAsyncLoaderActive(foreignLoaderFlagPresent)) &&
-        core.outerHookOwnsFormat(context.format, ext) && !core.isDependency(url))
-    )) {
+    if (__userHooksRegistered && context && context.format === "typescript") {
       return nextLoad(url, context);
     }
 
@@ -1049,36 +1027,10 @@ function installCjsRequireHooks(core, withClassicTranspile) {
   // missing. An explicit `require("dep/x/sub.cjs")` is unaffected either way — an
   // exact path is found by stat, without consulting the extension list at all.
   const NUB_ADDED_EXTS = [".ts", ".cts", ".mts", ".tsx", ".jsx", ".cjs"];
-  // A `require.extensions` handler that is ALREADY registered for one of these
-  // when this runs belongs to a transpiler the user chose, and it stays in charge
-  // of that extension: nub's classic shim neither replaces it nor pre-judges its
-  // files as ES modules at resolve time. Every `--require` runs before any `--import`, so on
-  // the compat tier — where nub's own preload is an `--import` — tsx's
-  // `--require`d preflight installs its `.ts` handler FIRST; nub then overwrote
-  // it, and a file tsx would have compiled to CJS (mixed `import` + `require`,
-  // the shape its ESM hook hands to the CJS loader) died in nub's handler as
-  // "Cannot require() this file — it is an ES module". Node itself registers only
-  // `.js`/`.json`/`.node`, so nothing but a user transpiler holds one of these
-  // keys here.
-  const foreignExts = new Set(
-    [...core.TRANSPILE_EXTS, ...core.allDataExts(), ...NUB_ADDED_EXTS].filter((ext) =>
-      ext !== ".js" && ext !== ".json" && ext !== ".node" && Object.hasOwn(module_._extensions, ext)),
-  );
-  // Ownership of a key is decided per lookup, never frozen at start-up: user code
-  // may install or replace a handler after nub's preload (`--import tsx` runs
-  // after every `--require`), and from then on that key is the user's own
-  // widening of LOAD_AS_FILE, exactly as under plain Node plus their handler —
-  // `require("dep/sub")` finding a dependency's `sub.ts` through it is the answer
-  // to preserve. A key is nub's only while BOTH hold: nub introduced it (it was
-  // not in `foreignExts` — a user's `.cjs` handler that nub merely wraps still
-  // counts as theirs) and it still holds a function nub registered below
-  // (`nubHandlers`). Anything else is not nub's to strip, nor to pre-judge.
-  const nubHandlers = new Set();
-  const nubOwnsExt = (ext) => !foreignExts.has(ext) && nubHandlers.has(module_._extensions[ext]);
   const withoutNubAddedExtensions = (fn) => {
     const saved = [];
     for (const ext of NUB_ADDED_EXTS) {
-      if (nubOwnsExt(ext)) {
+      if (Object.hasOwn(module_._extensions, ext)) {
         saved.push([ext, module_._extensions[ext]]);
         delete module_._extensions[ext];
       }
@@ -1091,7 +1043,7 @@ function installCjsRequireHooks(core, withClassicTranspile) {
   };
   const isDepAddedExtHit = (filename) =>
     typeof filename === "string" &&
-    nubOwnsExt(pathExtname(filename)) &&
+    NUB_ADDED_EXTS.includes(pathExtname(filename)) &&
     core.isDependency(pathToFileURL(filename).href);
 
   module_._resolveFilename = function (request, parent, isMain, options) {
@@ -1114,8 +1066,7 @@ function installCjsRequireHooks(core, withClassicTranspile) {
       // and 4 from `require.resolve` up to 22.14, but 22.15 and 22.16 pass 4 for both
       // while still lacking native TS — and the translator crash is present on
       // exactly those versions. A clean error beats an opaque crash, so this stays.
-      if (withClassicTranspile && nubOwnsExt(pathExtname(resolved)) &&
-          core.requireTargetIsEsm(resolved, pathExtname(resolved))) {
+      if (withClassicTranspile && core.requireTargetIsEsm(resolved, pathExtname(resolved))) {
         throw requireEsmError(resolved);
       }
       return resolved;
@@ -1244,7 +1195,6 @@ function installCjsRequireHooks(core, withClassicTranspile) {
     if (core.TRANSPILE_EXTS.has(ext)) return transpileExtension(mod, filename);
     return nativeJs.call(module_._extensions, mod, filename);
   };
-  nubHandlers.add(nubExtension);
 
   // Registered for every extension either path may claim, so the dispatcher is
   // reached at all; `nubExtension` then decides. Node's own `.js`/`.json`/`.node`
@@ -1265,7 +1215,7 @@ function installCjsRequireHooks(core, withClassicTranspile) {
   // resolution identical across tiers.
   const CODE_EXTS = new Set([".ts", ".cts", ".mts", ".tsx", ".jsx"]);
   for (const ext of new Set([...core.TRANSPILE_EXTS, ...core.allDataExts()])) {
-    if (ext === ".js" || ext === ".json" || ext === ".node" || foreignExts.has(ext)) continue;
+    if (ext === ".js" || ext === ".json" || ext === ".node") continue;
     Object.defineProperty(module_._extensions, ext, {
       value: nubExtension,
       enumerable: CODE_EXTS.has(ext),
@@ -1293,7 +1243,7 @@ function installCjsRequireHooks(core, withClassicTranspile) {
   // (`nativeJs` is captured above, before the TS handlers are registered.)
   for (const ext of [".js", ".cjs"]) {
     const origExtension = module_._extensions[ext] || nativeJs;
-    const plainJsExtension = (mod, filename) => {
+    module_._extensions[ext] = (mod, filename) => {
       // (0) The project pointed this extension at a data loader (`{".js":"text"}`),
       // which the ESM path honors. These two extensions are skipped by the
       // registration loop above because THIS wrapper owns them and runs after it,
@@ -1316,8 +1266,6 @@ function installCjsRequireHooks(core, withClassicTranspile) {
       }
       return origExtension.call(module_._extensions, mod, filename); // (3)
     };
-    module_._extensions[ext] = plainJsExtension;
-    nubHandlers.add(plainJsExtension);
   }
 }
 

package/transform-core.mjs

@@ -207,20 +207,6 @@ export const TRANSPILE_EXTS = new Set([".ts", ".tsx", ".mts", ".cts", ".jsx"]);
 // `maybeTranspilePlainJs` gate); a no-op plain-JS file falls through to Node's
 // native loader untouched, byte-identical. node_modules is excluded at the gate.
 export const PLAIN_JS_EXTS = new Set([".js", ".mjs", ".cjs"]);
-// A bare `commonjs`/`module` format on a file nub would transpile can only have
-// been assigned by a hook layered ABOVE nub's. Node's resolver labels a `.ts` file
-// `commonjs-typescript`/`module-typescript` (or `typescript` when the package has
-// no `type`) and leaves `.jsx` unlabelled, and nub's own resolve returns no format;
-// the bare form is what tsx's resolve hook writes (getFormatFromFileUrl), and its
-// load hook then expects the RAW source back from `nextLoad` so it can run its own
-// module-format transform — a mixed `import` + `require` file becomes CJS there,
-// where nub's syntax detection would make it ESM and `require` undefined. Both
-// tiers step aside on this signal, the fast tier only once the user registration
-// is known to carry a load hook (preload-common.cjs). Plain JS is excluded: Node
-// assigns those the bare form itself.
-export function outerHookOwnsFormat(format, ext) {
-  return (format === "commonjs" || format === "module") && TRANSPILE_EXTS.has(ext) && !PLAIN_JS_EXTS.has(ext);
-}
 // The data loaders nub SHIPS — a runtime feature, not a project setting, so they stay
 // in force inside node_modules too (see dataExtsFor).
 const BUILTIN_DATA_EXTS = { ".jsonc": "jsonc", ".json5": "json5", ".toml": "toml", ".yaml": "yaml", ".yml": "yaml", ".txt": "txt" };
@@ -1021,15 +1007,7 @@ export function loadTranspile(url, ext, source) {
     const details = result.errors.map((e) => e.codeframe || e.message).join("\n\n");
     throw new Error(`Transpile error in ${filePath}:\n${details}`);
   }
-  // `responseURL` is what an OUTER user hook keys on. Node's default load sets it
-  // to the file URL, and tsx's load hook takes its CommonJS branch only when the
-  // result `nextLoad` hands back carries a `file:` responseURL — without it, tsx
-  // re-transformed a `.ts` file nub had already emitted as CJS in its ESM branch,
-  // and `require` was undefined at run time (`tsx script.ts` under `nub run`).
-  // Node itself defaults a missing responseURL to the URL, so only a hook layered
-  // above nub's can observe the difference; every file-URL result nub
-  // short-circuits carries it for that reason.
-  return { format: result.format, source: result.code, responseURL: url, shortCircuit: true };
+  return { format: result.format, source: result.code, shortCircuit: true };
 }
 
 // Project-source plain JS (`.js`/`.mjs`/`.cjs`) gate. Returns a transpiled load
@@ -1148,7 +1126,7 @@ export function loadData(url, ext) {
   const parsed = dataValue(url, ext);
 
   if (parsed == null) {
-    return { format: "module", source: "export default undefined;\n", responseURL: url, shortCircuit: true };
+    return { format: "module", source: "export default undefined;\n", shortCircuit: true };
   }
 
   // Default export only. Data modules deliberately do NOT emit per-key named
@@ -1158,7 +1136,7 @@ export function loadData(url, ext) {
   // default — `import cfg from "./c.yaml"; const { host } = cfg;` — which the
   // `@nubjs/types` `Record<string, unknown>` default type makes sound.
   const code = `export default ${JSON.stringify(parsed)};\n`;
-  return { format: "module", source: code, responseURL: url, shortCircuit: true };
+  return { format: "module", source: code, shortCircuit: true };
 }
 
 // Import Text: `import s from "./any.file" with { type: "text" }` → the raw file
@@ -1174,5 +1152,5 @@ export function loadData(url, ext) {
 const __textDecoder = new TextDecoder();
 export function loadTextImport(url) {
   const text = __textDecoder.decode(readFileSync(fileURLToPath(url)));
-  return { format: "module", source: `export default ${JSON.stringify(text)};\n`, responseURL: url, shortCircuit: true };
+  return { format: "module", source: `export default ${JSON.stringify(text)};\n`, shortCircuit: true };
 }