@frockbot/plugin-tools 0.3.8 → 0.3.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@frockbot/plugin-tools",
-  "version": "0.3.8",
+  "version": "0.3.10",
   "private": false,
   "type": "module",
   "exports": {
@@ -17,7 +17,7 @@
     "typecheck": "tsc --noEmit -p tsconfig.json"
   },
   "dependencies": {
-    "@frockbot/kernel-contracts": "0.3.8",
+    "@frockbot/kernel-contracts": "0.3.10",
     "cordis": "4.0.0-rc.8"
   },
   "devDependencies": {

package/src/dynamic-tools.test.ts

@@ -590,3 +590,45 @@ describe("progressive tool disclosure", () => {
     );
   });
 });
+
+describe("a dynamic tool called by its bare name", () => {
+  test("is answered with the envelope, not a dead-end Unknown tool", async () => {
+    // `applet_list` is listed to the model by bare name in
+    // `<dynamic_tool_namespaces>`, so the model reaches for it that way. The
+    // refusal used to be `Unknown tool: applet_list` and nothing else — a dead
+    // end for a tool that was right there.
+    const root = await rootWithTools();
+    root.tools.register(dynamicTool("applets", "applet_list"));
+
+    const result = await invoke(root, "applet_list", {});
+
+    expect(result.isError).toBe(true);
+    expect(result.content).toContain("applet_list");
+    expect(result.content).toContain(CALL_DYNAMIC_TOOL_NAME);
+    expect(result.content).toContain(
+      '{"namespace":"applets","toolName":"applet_list"',
+    );
+    expect(result.content).toContain(GET_DYNAMIC_TOOLS_NAME);
+  });
+
+  test("names every namespace the bare name is in", async () => {
+    const root = await rootWithTools();
+    root.tools.register(dynamicTool("mail-one", "search"));
+    root.tools.register(dynamicTool("mail-two", "search"));
+
+    const result = await invoke(root, "search", {});
+
+    expect(result.isError).toBe(true);
+    expect(result.content).toContain("mail-one");
+    expect(result.content).toContain("mail-two");
+  });
+
+  test("a name in no namespace still says only that it is unknown", async () => {
+    const root = await rootWithTools();
+    root.tools.register(dynamicTool("applets", "applet_list"));
+
+    const result = await invoke(root, "not_a_tool", {});
+
+    expect(result.content).toBe("Unknown tool: not_a_tool");
+  });
+});

package/src/tools.ts

@@ -552,6 +552,30 @@ export class ToolRegistry extends Service implements ToolExecution {
     return { kind: "denied", call, result: { content, isError: true } };
   }
 
+  /**
+   * Why a name is not callable — and, when it *is* callable through the
+   * envelope, how.
+   *
+   * A dynamic tool is listed to the model by bare name in
+   * `<dynamic_tool_namespaces>`, so the model reaches for it by that name and
+   * used to get `Unknown tool: applet_list` and nothing else: a dead end for a
+   * tool that was right there, costing a step every time. The refusal now
+   * hands back the exact envelope for the namespace the name is actually in.
+   */
+  private unknownToolRefusal(name: string): string {
+    const namespaces = [...this.dynamicDefinitions]
+      .filter(([, definitions]) => definitions.has(name))
+      .map(([namespace]) => namespace)
+      .sort();
+    const first = namespaces[0];
+    if (first === undefined) return `Unknown tool: ${name}`;
+    const where =
+      namespaces.length === 1
+        ? `namespace "${first}"`
+        : `namespaces ${listOrNone(namespaces)} — pick one`;
+    return `Unknown tool: ${name}. It is a dynamic tool in ${where}; call it through ${CALL_DYNAMIC_TOOL_NAME} as {"namespace":"${first}","toolName":"${name}","arguments":{ … the tool's own input … }}. Call ${GET_DYNAMIC_TOOLS_NAME}({"namespace":"${first}","toolName":"${name}"}) first if you do not have its schema.`;
+  }
+
   private async prepareRegistered(
     call: ToolCall,
     registered: RegisteredTool | undefined,
@@ -566,7 +590,10 @@ export class ToolRegistry extends Service implements ToolExecution {
           return {
             kind: "denied",
             call,
-            result: { content: `Unknown tool: ${call.name}`, isError: true },
+            result: {
+              content: this.unknownToolRefusal(call.name),
+              isError: true,
+            },
           };
         }
         // Defence in depth: the catalog was already trimmed, so a call that