@konneal/engine 0.2.137 → 0.2.139

package/dist/mcp-proto.d.ts

@@ -1,4 +1,10 @@
 export declare const PROTOCOL_VERSION = "2025-06-18";
+/** Registry tool name → the MCP-presented name. */
+export declare function mcpName(registryName: string): string;
+/** MCP-presented name → the name the handlers dispatch on. Raw
+ *  registry names pass through (transition tolerance for early
+ *  consumers of the unprefixed surface). */
+export declare function registryName(presented: string): string;
 export interface McpTool {
     name: string;
     description: string;
@@ -8,7 +14,7 @@ export interface McpTool {
         required: string[];
     };
 }
-export declare const TOOLS: McpTool[];
+export declare function buildTools(): McpTool[];
 export type McpResult = {
     ok: true;
     result: unknown;

package/dist/worker_public/src/index.js

@@ -961,41 +961,66 @@ async function deleteAttachment(deps, sub, id) {
 
 // workers/worker_public/src/mcp-proto.ts
 var PROTOCOL_VERSION = "2025-06-18";
-var TOOLS = [
-  {
-    name: "ask",
-    description: "Ask the corpus a question; returns a citation-grounded answer with the passages it rests on.",
-    inputSchema: {
-      type: "object",
-      properties: {
-        query: { type: "string", description: "The question (1-8000 chars)" },
-        lang: { type: "string", description: "Answer language hint (e.g. en, fr)" }
-      },
-      required: ["query"]
-    }
-  },
-  {
-    name: "retrieve",
-    description: "Retrieve the top passages for a query (hybrid dense + metadata steering, reranked).",
-    inputSchema: {
-      type: "object",
-      properties: {
-        query: { type: "string" },
-        k: { type: "number", description: "Hits to return (default 5)" }
-      },
-      required: ["query"]
-    }
-  },
-  ...TOOLS_REGISTRY.filter((t) => t.audiences.includes("mcp")).map((t) => ({
-    name: t.name,
-    description: t.description,
-    inputSchema: {
-      type: "object",
-      properties: Object.fromEntries(t.params.map((p) => [p.key, { type: "string", description: p.description }])),
-      required: t.params.filter((p) => p.required).map((p) => p.key)
-    }
-  }))
-];
+var mcpPrefix = () => `${P().publisher.id}_`;
+var mcpOverrides = () => ({
+  retrieve: `${P().publisher.id}_search`,
+  // the published retrieval contract
+  "documents.family": `${P().publisher.id}_documents`
+  // the published registry contract
+});
+function mcpName(registryName2) {
+  const overrides = mcpOverrides();
+  return overrides[registryName2] ?? `${mcpPrefix()}${registryName2.replace(/\./g, "_")}`;
+}
+function registryName(presented) {
+  const pid = P().publisher.id;
+  if (presented === `${pid}_ask`) return "ask";
+  const overrides = mcpOverrides();
+  for (const [reg, mc] of Object.entries(overrides)) if (mc === presented) return reg;
+  if (presented.startsWith(`${pid}_`)) {
+    const rest = presented.slice(pid.length + 1);
+    const hit = TOOLS_REGISTRY.find((t) => t.audiences.includes("mcp") && t.name.replace(/\./g, "_") === rest);
+    if (hit) return hit.name;
+  }
+  return presented;
+}
+function buildTools() {
+  return [
+    {
+      name: mcpName("ask"),
+      description: "Ask the corpus a question; returns a citation-grounded answer with the passages it rests on.",
+      inputSchema: {
+        type: "object",
+        properties: {
+          query: { type: "string", description: "The question (1-8000 chars)" },
+          lang: { type: "string", description: "Answer language hint (e.g. en, fr)" }
+        },
+        required: ["query"]
+      }
+    },
+    {
+      name: mcpName("retrieve"),
+      description: "Retrieve the top passages for a query (hybrid dense + metadata steering, reranked).",
+      inputSchema: {
+        type: "object",
+        properties: {
+          query: { type: "string" },
+          k: { type: "number", description: "Hits to return (default 5)" }
+        },
+        required: ["query"]
+      }
+    },
+    ...TOOLS_REGISTRY.filter((t) => t.audiences.includes("mcp")).map((t) => ({
+      name: mcpName(t.name),
+      description: t.description,
+      inputSchema: {
+        type: "object",
+        properties: Object.fromEntries(t.params.map((p) => [p.key, { type: "string", description: p.description }])),
+        required: t.params.filter((p) => p.required).map((p) => p.key)
+      }
+    }))
+  ];
+}
 function dispatch(method, params, callTool) {
   switch (method) {
     case "initialize":
@@ -1004,13 +1029,13 @@ function dispatch(method, params, callTool) {
     case "ping":
       return Promise.resolve({ ok: true, accepted: true });
     case "tools/list":
-      return Promise.resolve({ ok: true, result: { tools: TOOLS } });
+      return Promise.resolve({ ok: true, result: { tools: buildTools() } });
     case "tools/call": {
       const name = typeof params?.name === "string" ? params.name : "";
-      if (!TOOLS.some((t) => t.name === name)) {
+      if (!buildTools().some((t) => t.name === name)) {
         return Promise.resolve({ ok: false, code: -32602, message: `unknown tool: ${name}` });
       }
-      return callTool(name, params?.arguments ?? {}).then(
+      return callTool(registryName(name), params?.arguments ?? {}).then(
         (payload) => ({ ok: true, result: { content: [{ type: "text", text: JSON.stringify(payload) }] } })
       );
     }

package/package.json

@@ -27,7 +27,7 @@
     "playwright": "^1.62.1",
     "yaml": "^2.9.1"
   },
-  "version": "0.2.137",
+  "version": "0.2.139",
   "description": "The Konneal engine: the publisher-agnostic build pipeline and API plane for standards intelligence (retrieval, answer contract, verdicts, evaluation).",
   "license": "BSD-3-Clause",
   "type": "module",

package/workers/worker_public/src/mcp-proto.ts

@@ -2,8 +2,45 @@
 // dependency-free so plain node can load it for unit tests — the
 // route adapter (mcp.ts) owns the handler wiring.
 import { TOOLS_REGISTRY } from "./tools.ts";
+import { P } from "./profile.ts";
 export const PROTOCOL_VERSION = "2025-06-18";
 
+// The MCP naming convention (2026-10-12): every tool the publisher
+// exposes over MCP carries the publisher prefix — MCP clients render
+// tool names in a SHARED namespace, so generic names ("ask") collide
+// across servers. The three names docs/MCP.md already publishes keep
+// their exact spelling; the mechanical transform covers the rest.
+const mcpPrefix = () => `${P().publisher.id}_`;
+// per call like everything else here — a module-level map froze the
+// fixture prefix into the two published contracts (the freeze test
+// caught this one AFTER catching the TOOLS array)
+const mcpOverrides = (): Record<string, string> => ({
+  retrieve: `${P().publisher.id}_search`,          // the published retrieval contract
+  "documents.family": `${P().publisher.id}_documents`, // the published registry contract
+});
+
+/** Registry tool name → the MCP-presented name. */
+export function mcpName(registryName: string): string {
+  const overrides = mcpOverrides();
+  return overrides[registryName] ?? `${mcpPrefix()}${registryName.replace(/\./g, "_")}`;
+}
+
+/** MCP-presented name → the name the handlers dispatch on. Raw
+ *  registry names pass through (transition tolerance for early
+ *  consumers of the unprefixed surface). */
+export function registryName(presented: string): string {
+  const pid = P().publisher.id;
+  if (presented === `${pid}_ask`) return "ask";
+  const overrides = mcpOverrides();
+  for (const [reg, mc] of Object.entries(overrides)) if (mc === presented) return reg;
+  if (presented.startsWith(`${pid}_`)) {
+    const rest = presented.slice(pid.length + 1);
+    const hit = TOOLS_REGISTRY.find((t) => t.audiences.includes("mcp") && t.name.replace(/\./g, "_") === rest);
+    if (hit) return hit.name;
+  }
+  return presented;
+}
+
 export interface McpTool {
   name: string;
   description: string;
@@ -11,10 +48,16 @@ export interface McpTool {
 }
 
 // the registry's MCP-audience tools are GENERATED here (OCP: adding a
-// tool registers it; this list is never hand-extended for them)
-export const TOOLS: McpTool[] = [
+// tool registers it; this list is never hand-extended for them).
+// BUILT PER CALL, never at module load: the names carry the publisher
+// prefix, and the deployment's setProfile runs AFTER this module's
+// imports evaluate — a module-level array froze the fixture prefix
+// into production's tools/list (caught by the 2026-10-12 deploy smoke's
+// tools/list line, which the "ask" substring grep was too weak to fail).
+export function buildTools(): McpTool[] {
+  return [
   {
-    name: "ask",
+    name: mcpName("ask"),
     description: "Ask the corpus a question; returns a citation-grounded answer with the passages it rests on.",
     inputSchema: {
       type: "object",
@@ -26,7 +69,7 @@ export const TOOLS: McpTool[] = [
     },
   },
   {
-    name: "retrieve",
+    name: mcpName("retrieve"),
     description: "Retrieve the top passages for a query (hybrid dense + metadata steering, reranked).",
     inputSchema: {
       type: "object",
@@ -40,7 +83,7 @@ export const TOOLS: McpTool[] = [
   ...TOOLS_REGISTRY
     .filter((t) => t.audiences.includes("mcp"))
     .map((t): McpTool => ({
-      name: t.name,
+      name: mcpName(t.name),
       description: t.description,
       inputSchema: {
         type: "object",
@@ -48,7 +91,8 @@ export const TOOLS: McpTool[] = [
         required: t.params.filter((p) => p.required).map((p) => p.key),
       },
     })),
-];
+  ];
+}
 
 export type McpResult =
   | { ok: true; result: unknown }
@@ -69,13 +113,13 @@ export function dispatch(
     case "ping":
       return Promise.resolve({ ok: true, accepted: true });
     case "tools/list":
-      return Promise.resolve({ ok: true, result: { tools: TOOLS } });
+      return Promise.resolve({ ok: true, result: { tools: buildTools() } });
     case "tools/call": {
       const name = typeof params?.name === "string" ? params.name : "";
-      if (!TOOLS.some((t) => t.name === name)) {
+      if (!buildTools().some((t) => t.name === name)) {
         return Promise.resolve({ ok: false, code: -32602, message: `unknown tool: ${name}` });
       }
-      return callTool(name, params?.arguments ?? {}).then(
+      return callTool(registryName(name), params?.arguments ?? {}).then(
         (payload) => ({ ok: true, result: { content: [{ type: "text", text: JSON.stringify(payload) }] } }) as McpResult,
       );
     }