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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@polycode-projects/the-mechanical-code-talker",
-  "version": "3.1.3",
+  "version": "3.2.0",
   "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"),

package/src/services/chat-session.mjs

@@ -128,6 +128,12 @@ export async function createSession({
   // already-resolved value; full precedence (this param > TMCT_MEMORY_BACKEND
   // env > tmct.toml > the sqlite default) resolved below.
   memoryBackend = null,
+  // The tool-name prefix a host's own follow-up hints should use in place of
+  // "tmct_" (e.g. seonix's own tool names) — threaded onto `config` below so
+  // dispatchTool's handlers (kit.mjs/tmct-search.mjs/tmct-members.mjs/
+  // tmct-context.mjs) render hints under the host's own names. Omitted (the
+  // default) preserves today's "tmct_" behavior exactly.
+  toolNamePrefix,
 } = {}) {
   // EPHEMERAL mode (--ephemeral, or TMCT_EPHEMERAL=1): read the target graph but
   // write NOTHING back into it. The shipped examples run this way so a demo never
@@ -193,6 +199,10 @@ export async function createSession({
       ({ config, toml } = await resolveRuntimeConfig({ argv, cwd, env, gitRoot }));
     }
   }
+  // Every `config` branch above ends up here — set once so every downstream
+  // hint-rendering call site (dispatchTool's handlers, via config.toolNamePrefix)
+  // sees the same value regardless of which branch built `config`.
+  config.toolNamePrefix = toolNamePrefix || "tmct_";
 
   // Every game's tuning knobs (spider-fly's mass economy, guess-the-number's
   // bounds, the shared plan lane's search-depth cap), resolved once per
@@ -465,11 +475,12 @@ export async function runChat({
   narrate = false,
   liveReference = false,
   memoryBackend = null,
+  toolNamePrefix,
 } = {}) {
   // createSession's first-run seed (~2-3s) produces ZERO output until it fully
   // resolves, which otherwise reads as `npm run chat` hanging with total silence.
   output.write("tmct — starting…\n");
-  const session = await createSession({ repoPath, graphPaths, configPath, source, env, cwd, gitRoot, ephemeral, narrate, liveReference, memoryBackend });
+  const session = await createSession({ repoPath, graphPaths, configPath, source, env, cwd, gitRoot, ephemeral, narrate, liveReference, memoryBackend, toolNamePrefix });
 
   const dim = (s) => (env.NO_COLOR || !output.isTTY ? s : `\x1b[2m${s}\x1b[0m`);
   for (const line of session.bannerLines) output.write(dim(line) + "\n");