@polycode-projects/the-mechanical-code-talker 3.1.3 → 3.1.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@polycode-projects/the-mechanical-code-talker",
-  "version": "3.1.3",
+  "version": "3.1.4",
   "private": false,
   "type": "module",
   "description": "The Mechanical Code Talker (tmct) — a tolerant, offline, $0 chat surface that guides you toward precision queries about a software repository. ELIZA/PARRY-style but domain-obsessed with code. No model calls; indexes a repo on request (tmct index) or reads any producer's graph.",

package/src/adapters/wink-model.mjs

@@ -3,14 +3,23 @@
 //
 // A static `import "wink-nlp"` would drag the ~1 MB model into the base/viewer bundle.
 // Instead, `registerWinkModel` lets a browser/bundler entry hand in an already-imported
-// `{ winkNLP, model }` pair once; Node instead resolves lazily via `createRequire`.
+// `{ winkNLP, model }` pair once; Node instead resolves lazily via `createRequire`,
+// fetched at call time from `process.getBuiltinModule("node:module")` rather than a
+// top-level `import { createRequire } from "node:module"`. A top-level import declares
+// the `createRequire` binding in this module's own output, and a downstream consumer
+// that bundles tmct alongside other CJS dependencies (esbuild, `--format=esm`) can
+// inject its own auto-generated `createRequire` shim into the same bundle scope,
+// producing a duplicate top-level declaration and a `SyntaxError` at Node module-parse
+// time. Resolving it lazily via `process.getBuiltinModule` keeps the bundled output free
+// of any top-level `createRequire` declaration, so there's nothing left to collide with.
+// Don't "simplify" this back to a top-level import. `process.getBuiltinModule` needs
+// Node 22.3.0/20.16.0+; this package's `engines.node` floor (">=24") already clears
+// that, so lowering the floor later needs to keep clearing it too.
 //
 // The loader stays synchronous. Failure is cached as null — a checkout without the
 // optional deps, or a page that never registered a model, runs adapter-less rather
 // than throwing.
 
-import { createRequire } from "node:module";
-
 let injected; // browser/bundler-supplied `() => ({ winkNLP, model })`, or undefined
 let cached; // undefined = not tried yet; null = unavailable (tried once, honestly off)
 
@@ -40,7 +49,7 @@ function loadWinkModel() {
 /** Node fallback: resolve wink through the module system (never a guessed path).
  *  CJS deps, so `createRequire`. */
 function nodeRequireWink() {
-  const require = createRequire(import.meta.url);
+  const require = process.getBuiltinModule("node:module").createRequire(import.meta.url);
   return {
     winkNLP: require("wink-nlp"),
     model: require("wink-eng-lite-web-model"),