fix(tui): consult stripped name for activation on prefixed runner miss

A harness-namespaced call (default.<stripped>) misses the registry under its original name even when <stripped> was activated. Check isActivated with the stripped form too so an activated-but-unmounted tool reports reconnecting instead of unknown tool. Miss-branch strings still use the original name; never-activated misses keep the exact unknown-tool string. Adds keeper tests: dotted registration wins over bare suffix, prefixed activated miss reports reconnecting under the original name, gate-reject on a normalized call reports the stripped name.

Author: TheGreatAxios

src/tui/dynamic-tool-runner.test.ts

@@ -254,6 +254,64 @@ describe("harness namespace prefix", () => {
     expect(result.isError).toBe(true);
     expect(result.content).toBe("unknown tool: default.mcp__gone__tool");
   });
+
+  test("an exact dotted registration wins over the bare suffix (anti-misrouting)", async () => {
+    const runner = createDynamicToolRunner([
+      stringTool("mcp__acme__do", "bare"),
+      stringTool("default.mcp__acme__do", "dotted"),
+    ]);
+    runner.setCallGate(() => true);
+
+    const result = await runner.run(
+      { id: "1", name: "default.mcp__acme__do", arguments: {} },
+      new AbortController().signal,
+    );
+
+    expect(result.isError).toBeUndefined();
+    expect(result.content).toBe("dotted");
+  });
+
+  test("a prefixed miss for an activated-but-unmounted stripped tool reports reconnecting under the original name", async () => {
+    const runner = createDynamicToolRunner([
+      stringTool("read_file", "core"),
+      stringTool("mcp__acme__do", "blind-result"),
+    ]);
+    const activated = new Set(["mcp__acme__do"]);
+    runner.setCallGate((name) => name === "read_file" || activated.has(name), {
+      isActivated: (name) => activated.has(name),
+    });
+
+    // The server dropped between search and call, so the bare tool left the
+    // registry; the model still emits the harness-namespaced form.
+    runner.removeTools(["mcp__acme__do"]);
+    const result = await runner.run(
+      { id: "1", name: "default.mcp__acme__do", arguments: {} },
+      new AbortController().signal,
+    );
+
+    expect(result.isError).toBe(true);
+    expect(result.content).toContain("default.mcp__acme__do");
+    expect(result.content).toContain("reconnecting");
+    expect(result.content).toContain("Retry the call shortly");
+  });
+
+  test("a normalized call rejected by the gate reports the stripped name", async () => {
+    const runner = createDynamicToolRunner([
+      stringTool("read_file", "core"),
+      stringTool("mcp__acme__do", "blind-result"),
+    ]);
+    runner.setCallGate((name) => name === "read_file");
+
+    const result = await runner.run(
+      { id: "1", name: "default.mcp__acme__do", arguments: {} },
+      new AbortController().signal,
+    );
+
+    expect(result.isError).toBe(true);
+    expect(result.content).toContain("mcp__acme__do");
+    expect(result.content).toContain("tool_search");
+    expect(result.content).not.toContain("default.mcp__acme__do");
+  });
 });
 
 describe("terminal control stripping", () => {

src/tui/dynamic-tool-runner.ts

@@ -92,14 +92,6 @@ export function createDynamicToolRunner(
         byName.has(name),
       );
       if (resolved === undefined) {
-        return {
-          callId: call.id,
-          content: `unknown tool: ${call.name}`,
-          isError: true,
-        };
-      }
-      const found = byName.get(resolved);
-      if (found === undefined) {
         // The name was promoted (gate-activated) but the registry no longer
         // holds it — the server dropped between search and call. Say so: the
         // model already has the schema from the tool_search card and only
@@ -114,6 +106,31 @@ export function createDynamicToolRunner(
             isError: true,
           };
         }
+        // A harness-namespaced call (`default.<stripped>`) misses the registry
+        // under its original name even though the model was shown `<stripped>`:
+        // consult activation with the stripped form too, but keep the original
+        // name in the message so the transcript matches what the model emitted.
+        const dot = call.name.indexOf(".");
+        if (dot > 0) {
+          const stripped = call.name.slice(dot + 1);
+          if (stripped.length > 0 && isActivated?.(stripped) === true) {
+            return {
+              callId: call.id,
+              content:
+                `Error: ${call.name} is not currently available - its server may ` +
+                `still be reconnecting. Retry the call shortly.`,
+              isError: true,
+            };
+          }
+        }
+        return {
+          callId: call.id,
+          content: `unknown tool: ${call.name}`,
+          isError: true,
+        };
+      }
+      const found = byName.get(resolved);
+      if (found === undefined) {
         return {
           callId: call.id,
           content: `unknown tool: ${call.name}`,