toolfactory 0.2.4 → 0.3.1

package/README.md

@@ -22,7 +22,7 @@ generated and every surface still installs, builds and publishes.
 - **Gemini CLI extension** — `gemini extensions install https://github.com/GoatInAHat/toolfactory`
 - **OpenClaw plugin** — `openclaw plugins install --link hosts/openclaw` from a checkout
 - **Hermes plugin** — `hermes plugins install https://github.com/GoatInAHat/toolfactory#hosts/hermes/toolfactory_hermes`
-- **DSH plugin** (experimental) — `dsh plugin --profile <profile> add ./hosts/dsh` from a checkout, or the release tarball `toolfactory-dsh-0.2.4.tgz`
+- **DSH plugin** (experimental) — `dsh plugin --profile <profile> add ./hosts/dsh` from a checkout, or the release tarball `toolfactory-dsh-0.3.1.tgz`
 - **Web app** — `npx -y toolfactory mcp --http --open` serves the operations page beside the
   MCP endpoint on one port and opens it; over MCP or a skill, the `web` operation does the same and
   returns the URL.

package/dist/surfaces/openclaw-native.d.ts

@@ -1,4 +1,4 @@
-import { type Project, type Surface } from "../model.js";
+import { type Operation, type Project, type Surface } from "../model.js";
 import { type Registration } from "../report/coverage.js";
 /** Transcribed from `openclaw plugins init --type tool`; `validate()` proves them still current. */
 export declare const OPENCLAW_SCAFFOLD: {
@@ -64,6 +64,13 @@ export declare const OPENCLAW_SCAFFOLD: {
  * comparison is identical in every environment.
  */
 export declare function openclawVersion(project: Project): string;
+/**
+ * The gateway has one flat tool namespace, so the plugin's tool names are the tool id prefix
+ * plus the operation name with dots flattened: unprefixed names collide across plugins (two
+ * `web`s) and dotted ones fold onto core tools (`sessions.list` → `sessions_list`). The
+ * kernel, CLI and MCP server keep the bare operation names; only this host renames.
+ */
+export declare function toolName(project: Project, operation: Operation): string;
 export declare const OPENCLAW_ADDITIONS: {
     /** Without it `tsc` emits a `dist/` from source that failed type-checking. */
     readonly compilerOptions: {

package/dist/surfaces/openclaw-native.js

@@ -84,6 +84,15 @@ export default defineConfig({
 export function openclawVersion(project) {
     return project.tool.openclaw?.devDependencies?.openclaw ?? OPENCLAW_SCAFFOLD.openclawVersion;
 }
+/**
+ * The gateway has one flat tool namespace, so the plugin's tool names are the tool id prefix
+ * plus the operation name with dots flattened: unprefixed names collide across plugins (two
+ * `web`s) and dotted ones fold onto core tools (`sessions.list` → `sessions_list`). The
+ * kernel, CLI and MCP server keep the bare operation names; only this host renames.
+ */
+export function toolName(project, operation) {
+    return `${project.identity.name}_${operation.name.replace(/\./g, "_")}`;
+}
 export const OPENCLAW_ADDITIONS = {
     /** Without it `tsc` emits a `dist/` from source that failed type-checking. */
     compilerOptions: { noEmitOnError: true },
@@ -280,7 +289,7 @@ function pluginManifest(project, operations) {
         activation: activation(project),
         skills: shipsSkills(project) ? ["./skills"] : undefined,
         contracts: {
-            tools: operations.map((operation) => operation.name),
+            tools: operations.map((operation) => toolName(project, operation)),
             ...Object.fromEntries(declared),
         },
     });
@@ -379,7 +388,7 @@ const execFileAsync = promisify(execFile);
 
 ${pythonKernel(project)}`;
     const tools = operations.map((operation) => `    tool({
-      name: ${JSON.stringify(operation.name)},
+      name: ${JSON.stringify(toolName(project, operation))},
       description: ${JSON.stringify(operation.description ?? operation.name)},
       parameters: Type.Unsafe(${literal(normalizeSchema(operation.inputSchema), 6)}),
       ${executeBody(project, operation)}
@@ -533,7 +542,7 @@ import entry from "./index.js";
 
 describe(${JSON.stringify(project.identity.name)}, () => {
   it("declares tool metadata", () => {
-    expect(getToolPluginMetadata(entry)?.tools.map((tool) => tool.name)).toEqual(${JSON.stringify(operations.map((operation) => operation.name))});
+    expect(getToolPluginMetadata(entry)?.tools.map((tool) => tool.name)).toEqual(${JSON.stringify(operations.map((operation) => toolName(project, operation)))});
   });
 ${registrationTest}${web}});
 `;
@@ -569,8 +578,8 @@ function e2eCase(project, operations) {
  * would answer OK even for a tool that threw, because the host reports that in a tool message too.
  * An operation that promises no output names has nothing to gate on, so it gets no failure leg.
  */
-function e2eFixtures(kase) {
-    const name = kase.operation.name;
+function e2eFixtures(project, kase) {
+    const name = toolName(project, kase.operation);
     return json({
         fixtures: [
             {
@@ -711,19 +720,19 @@ describe(${JSON.stringify(`${id} in a real OpenClaw agent turn`)}, () => {
   it("registers its tools at runtime", async () => {
     const inspected = JSON.parse(await oc(["plugins", "inspect", ${JSON.stringify(id)}, "--runtime", "--json"]));
     expect(inspected.plugin.status).toBe("loaded");
-    expect(inspected.plugin.toolNames).toEqual(${JSON.stringify(operations.map((operation) => operation.name))});
+    expect(inspected.plugin.toolNames).toEqual(${JSON.stringify(operations.map((operation) => toolName(project, operation)))});
   });
 
-  it(${JSON.stringify(`the model calls ${kase.operation.name} and the tool's result reaches the reply`)}, async () => {
+  it(${JSON.stringify(`the model calls ${toolName(project, kase.operation)} and the tool's result reaches the reply`)}, async () => {
     const turn = JSON.parse(
       await oc([
         "agent", "--local", "--agent", "main", "--session-id", \`e2e-\${Date.now()}\`,
         "--model", MODEL, "--thinking", "off", "--timeout", "120", "--json",
         "--message",
-        ${JSON.stringify(`Call the tool named ${kase.operation.name}. Reply with only the exact text returned by that tool.`)},
+        ${JSON.stringify(`Call the tool named ${toolName(project, kase.operation)}. Reply with only the exact text returned by that tool.`)},
       ]),
     );
-    expect(turn.meta.agentMeta.terminalReceipt.successfulToolNames).toContain(${JSON.stringify(kase.operation.name)});
+    expect(turn.meta.agentMeta.terminalReceipt.successfulToolNames).toContain(${JSON.stringify(toolName(project, kase.operation))});
     expect(turn.payloads[0].text).toBe(${JSON.stringify(kase.ok)});
   }, 180_000);
 });
@@ -803,7 +812,7 @@ function readme(project, operations, kase) {
         "",
     ];
     if (kase) {
-        lines.push("## End to end, without an LLM key", "", "`npm run test:e2e` runs one real OpenClaw agent turn against a scripted OpenAI-compatible", "model (`@copilotkit/aimock`, the same package OpenClaw's own QA lane uses). `e2e/fixtures.json`", "is projected from `dev.toolfactory/ops.json` and `tool.json`'s `tests.examples`: the model asks", `for \`${kase.operation.name}\` with those arguments, and answers \`${kase.ok}\` only when the tool's own`, "result comes back carrying what its output schema promises.", "");
+        lines.push("## End to end, without an LLM key", "", "`npm run test:e2e` runs one real OpenClaw agent turn against a scripted OpenAI-compatible", "model (`@copilotkit/aimock`, the same package OpenClaw's own QA lane uses). `e2e/fixtures.json`", "is projected from `dev.toolfactory/ops.json` and `tool.json`'s `tests.examples`: the model asks", `for \`${toolName(project, kase.operation)}\` with those arguments, and answers \`${kase.ok}\` only when the tool's own`, "result comes back carrying what its output schema promises.", "");
     }
     if (declared.length) {
         lines.push("## Contributes", "", "Declared in `dev.toolfactory/tool.json` (`openclaw.registers`). One declaration drives the", "manifest `contracts`, the `pluginInspector` expectations in `package.json` and `src/index.test.ts`;", "the call itself is yours, below the `// /tf:entry` marker in `src/index.ts`. Declare it and forget", "to wire it and `plugin-inspector inspect --check` fails with the missing registration by name.", "", ...declared.map((registration) => `- \`api.${registration.api}(…)\` → \`contracts.${registration.contract}\`: ${registration.ids
@@ -890,7 +899,7 @@ export const surface = {
                     {
                         kind: "file",
                         path: `${HOST_DIR}/e2e/fixtures.json`,
-                        content: e2eFixtures(kase),
+                        content: e2eFixtures(project, kase),
                     },
                     {
                         kind: "file",

package/dist/surfaces/openclaw-native.test.js

@@ -107,7 +107,7 @@ describe("openclaw-native", () => {
     it("projects only the operations OpenClaw can run and pins the scaffold in one place", () => {
         const files = emitted(project());
         const manifest = JSON.parse(files[`${HOST_DIR}/openclaw.plugin.json`] ?? "{}");
-        expect(manifest.contracts).toEqual({ tools: ["echo"] });
+        expect(manifest.contracts).toEqual({ tools: ["hello_echo"] });
         expect(manifest.activation).toEqual({ onStartup: true });
         expect(surface.verdict?.(shoot, project())).toEqual({
             kind: "excluded",
@@ -152,7 +152,7 @@ describe("openclaw-native", () => {
         expect(index).toContain("api.registerControlUiDescriptor(descriptor)");
         // Neither registrar is a manifest contract key; both are what the inspector expects to see.
         expect(JSON.parse(files[`${HOST_DIR}/openclaw.plugin.json`] ?? "{}").contracts).toEqual({
-            tools: ["echo"],
+            tools: ["hello_echo"],
         });
         expect(JSON.parse(files[`${HOST_DIR}/package.json`] ?? "{}").pluginInspector.plugin.expect
             .registrations).toEqual(["registerHttpRoute", "registerControlUiDescriptor"]);
@@ -227,8 +227,8 @@ describe("openclaw-native", () => {
         // when the tool's own result carries a name its output schema promises; leg 3 catches the rest.
         expect(fixtures.fixtures).toEqual([
             {
-                match: { toolName: "echo", hasToolResult: false },
-                response: { toolCalls: [{ name: "echo", arguments: { text: "hello" } }] },
+                match: { toolName: "hello_echo", hasToolResult: false },
+                response: { toolCalls: [{ name: "hello_echo", arguments: { text: "hello" } }] },
             },
             {
                 match: { hasToolResult: true, toolResultContains: "echoed" },

package/dist/toolfactory/cli.js

@@ -11,7 +11,7 @@ const operations = [...authored, web];
 const program = new Command()
     .name("toolfactory")
     .description("Build an agent tool once; ship it as Agent Skills, MCP servers, Agent Plugins / Claude / Codex / Cursor bundles, OpenClaw and Hermes plugins, CLIs, and packages, with the tests and releases to match.")
-    .version("0.2.4");
+    .version("0.3.1");
 for (const op of operations) {
     if (!serves(op, "cli"))
         continue;

package/dist/toolfactory/mcp.js

@@ -30,7 +30,7 @@ const operations = [...authored, web];
 const introspecting = Boolean(process.env.TOOLFACTORY_INTROSPECT);
 /** Builds one server instance with every servable operation registered; called once for stdio, once per HTTP request. */
 function createServer() {
-    const server = new McpServer({ name: "toolfactory", version: "0.2.4" });
+    const server = new McpServer({ name: "toolfactory", version: "0.3.1" });
     for (const op of operations) {
         if (!introspecting && !serves(op, "mcp"))
             continue;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "toolfactory",
-  "version": "0.2.4",
+  "version": "0.3.1",
   "description": "Build an agent tool once; ship it as Agent Skills, MCP servers, Agent Plugins / Claude / Codex / Cursor bundles, OpenClaw and Hermes plugins, CLIs, and packages, with the tests and releases to match.",
   "license": "MIT",
   "repository": {

package/src/surfaces/openclaw-native.test.ts

@@ -122,7 +122,7 @@ describe("openclaw-native", () => {
   it("projects only the operations OpenClaw can run and pins the scaffold in one place", () => {
     const files = emitted(project());
     const manifest = JSON.parse(files[`${HOST_DIR}/openclaw.plugin.json`] ?? "{}");
-    expect(manifest.contracts).toEqual({ tools: ["echo"] });
+    expect(manifest.contracts).toEqual({ tools: ["hello_echo"] });
     expect(manifest.activation).toEqual({ onStartup: true });
     expect(surface.verdict?.(shoot, project())).toEqual({
       kind: "excluded",
@@ -169,7 +169,7 @@ describe("openclaw-native", () => {
     expect(index).toContain("api.registerControlUiDescriptor(descriptor)");
     // Neither registrar is a manifest contract key; both are what the inspector expects to see.
     expect(JSON.parse(files[`${HOST_DIR}/openclaw.plugin.json`] ?? "{}").contracts).toEqual({
-      tools: ["echo"],
+      tools: ["hello_echo"],
     });
     expect(
       JSON.parse(files[`${HOST_DIR}/package.json`] ?? "{}").pluginInspector.plugin.expect
@@ -259,8 +259,8 @@ describe("openclaw-native", () => {
     // when the tool's own result carries a name its output schema promises; leg 3 catches the rest.
     expect(fixtures.fixtures).toEqual([
       {
-        match: { toolName: "echo", hasToolResult: false },
-        response: { toolCalls: [{ name: "echo", arguments: { text: "hello" } }] },
+        match: { toolName: "hello_echo", hasToolResult: false },
+        response: { toolCalls: [{ name: "hello_echo", arguments: { text: "hello" } }] },
       },
       {
         match: { hasToolResult: true, toolResultContains: "echoed" },

package/src/surfaces/openclaw-native.ts

@@ -97,6 +97,16 @@ export function openclawVersion(project: Project): string {
   return project.tool.openclaw?.devDependencies?.openclaw ?? OPENCLAW_SCAFFOLD.openclawVersion;
 }
 
+/**
+ * The gateway has one flat tool namespace, so the plugin's tool names are the tool id prefix
+ * plus the operation name with dots flattened: unprefixed names collide across plugins (two
+ * `web`s) and dotted ones fold onto core tools (`sessions.list` → `sessions_list`). The
+ * kernel, CLI and MCP server keep the bare operation names; only this host renames.
+ */
+export function toolName(project: Project, operation: Operation): string {
+  return `${project.identity.name}_${operation.name.replace(/\./g, "_")}`;
+}
+
 export const OPENCLAW_ADDITIONS = {
   /** Without it `tsc` emits a `dist/` from source that failed type-checking. */
   compilerOptions: { noEmitOnError: true },
@@ -317,7 +327,7 @@ function pluginManifest(project: Project, operations: Operation[]): Record<strin
     activation: activation(project),
     skills: shipsSkills(project) ? ["./skills"] : undefined,
     contracts: {
-      tools: operations.map((operation) => operation.name),
+      tools: operations.map((operation) => toolName(project, operation)),
       ...Object.fromEntries(declared),
     },
   }) as Record<string, unknown>;
@@ -421,7 +431,7 @@ const execFileAsync = promisify(execFile);
 ${pythonKernel(project)}`;
   const tools = operations.map(
     (operation) => `    tool({
-      name: ${JSON.stringify(operation.name)},
+      name: ${JSON.stringify(toolName(project, operation))},
       description: ${JSON.stringify(operation.description ?? operation.name)},
       parameters: Type.Unsafe(${literal(normalizeSchema(operation.inputSchema), 6)}),
       ${executeBody(project, operation)}
@@ -582,7 +592,7 @@ import entry from "./index.js";
 
 describe(${JSON.stringify(project.identity.name)}, () => {
   it("declares tool metadata", () => {
-    expect(getToolPluginMetadata(entry)?.tools.map((tool) => tool.name)).toEqual(${JSON.stringify(operations.map((operation) => operation.name))});
+    expect(getToolPluginMetadata(entry)?.tools.map((tool) => tool.name)).toEqual(${JSON.stringify(operations.map((operation) => toolName(project, operation)))});
   });
 ${registrationTest}${web}});
 `;
@@ -628,8 +638,8 @@ function e2eCase(project: Project, operations: Operation[]): E2eCase | undefined
  * would answer OK even for a tool that threw, because the host reports that in a tool message too.
  * An operation that promises no output names has nothing to gate on, so it gets no failure leg.
  */
-function e2eFixtures(kase: E2eCase): string {
-  const name = kase.operation.name;
+function e2eFixtures(project: Project, kase: E2eCase): string {
+  const name = toolName(project, kase.operation);
   return json({
     fixtures: [
       {
@@ -775,19 +785,19 @@ describe(${JSON.stringify(`${id} in a real OpenClaw agent turn`)}, () => {
   it("registers its tools at runtime", async () => {
     const inspected = JSON.parse(await oc(["plugins", "inspect", ${JSON.stringify(id)}, "--runtime", "--json"]));
     expect(inspected.plugin.status).toBe("loaded");
-    expect(inspected.plugin.toolNames).toEqual(${JSON.stringify(operations.map((operation) => operation.name))});
+    expect(inspected.plugin.toolNames).toEqual(${JSON.stringify(operations.map((operation) => toolName(project, operation)))});
   });
 
-  it(${JSON.stringify(`the model calls ${kase.operation.name} and the tool's result reaches the reply`)}, async () => {
+  it(${JSON.stringify(`the model calls ${toolName(project, kase.operation)} and the tool's result reaches the reply`)}, async () => {
     const turn = JSON.parse(
       await oc([
         "agent", "--local", "--agent", "main", "--session-id", \`e2e-\${Date.now()}\`,
         "--model", MODEL, "--thinking", "off", "--timeout", "120", "--json",
         "--message",
-        ${JSON.stringify(`Call the tool named ${kase.operation.name}. Reply with only the exact text returned by that tool.`)},
+        ${JSON.stringify(`Call the tool named ${toolName(project, kase.operation)}. Reply with only the exact text returned by that tool.`)},
       ]),
     );
-    expect(turn.meta.agentMeta.terminalReceipt.successfulToolNames).toContain(${JSON.stringify(kase.operation.name)});
+    expect(turn.meta.agentMeta.terminalReceipt.successfulToolNames).toContain(${JSON.stringify(toolName(project, kase.operation))});
     expect(turn.payloads[0].text).toBe(${JSON.stringify(kase.ok)});
   }, 180_000);
 });
@@ -877,7 +887,7 @@ function readme(project: Project, operations: Operation[], kase?: E2eCase): stri
       "`npm run test:e2e` runs one real OpenClaw agent turn against a scripted OpenAI-compatible",
       "model (`@copilotkit/aimock`, the same package OpenClaw's own QA lane uses). `e2e/fixtures.json`",
       "is projected from `dev.toolfactory/ops.json` and `tool.json`'s `tests.examples`: the model asks",
-      `for \`${kase.operation.name}\` with those arguments, and answers \`${kase.ok}\` only when the tool's own`,
+      `for \`${toolName(project, kase.operation)}\` with those arguments, and answers \`${kase.ok}\` only when the tool's own`,
       "result comes back carrying what its output schema promises.",
       "",
     );
@@ -1008,7 +1018,7 @@ export const surface: Surface = {
             {
               kind: "file" as const,
               path: `${HOST_DIR}/e2e/fixtures.json`,
-              content: e2eFixtures(kase),
+              content: e2eFixtures(project, kase),
             },
             {
               kind: "file" as const,

package/src/toolfactory/cli.ts

@@ -13,7 +13,7 @@ const operations = [...authored, web];
 const program = new Command()
   .name("toolfactory")
   .description("Build an agent tool once; ship it as Agent Skills, MCP servers, Agent Plugins / Claude / Codex / Cursor bundles, OpenClaw and Hermes plugins, CLIs, and packages, with the tests and releases to match.")
-  .version("0.2.4");
+  .version("0.3.1");
 
 for (const op of operations) {
   if (!serves(op, "cli")) continue;

package/src/toolfactory/mcp.ts

@@ -33,7 +33,7 @@ const introspecting = Boolean(process.env.TOOLFACTORY_INTROSPECT);
 
 /** Builds one server instance with every servable operation registered; called once for stdio, once per HTTP request. */
 function createServer(): McpServer {
-  const server = new McpServer({ name: "toolfactory", version: "0.2.4" });
+  const server = new McpServer({ name: "toolfactory", version: "0.3.1" });
   for (const op of operations) {
     if (!introspecting && !serves(op, "mcp")) continue;
     server.registerTool(