From ff52aec38e8aaff9f8159964c8924b23364e2e5d Mon Sep 17 00:00:00 2001 From: Tak hoffman Date: Sat, 24 Jan 2026 01:21:50 -0600 Subject: [PATCH] Agents: drop bash tool alias --- src/agents/pi-tools.ts | 6 ------ src/agents/tool-display.json | 6 ------ src/agents/tool-policy.test.ts | 2 +- src/agents/tool-policy.ts | 2 +- 4 files changed, 2 insertions(+), 14 deletions(-) diff --git a/src/agents/pi-tools.ts b/src/agents/pi-tools.ts index b8c328e40..ba0fe13bc 100644 --- a/src/agents/pi-tools.ts +++ b/src/agents/pi-tools.ts @@ -255,11 +255,6 @@ export function createClawdbotCodingTools(options?: { } : undefined, }); - const bashTool = { - ...(execTool as unknown as AnyAgentTool), - name: "bash", - label: "bash", - } satisfies AnyAgentTool; const processTool = createProcessTool({ cleanupMs: cleanupMsOverride ?? execConfig.cleanupMs, scopeKey, @@ -280,7 +275,6 @@ export function createClawdbotCodingTools(options?: { : []), ...(applyPatchTool ? [applyPatchTool as unknown as AnyAgentTool] : []), execTool as unknown as AnyAgentTool, - bashTool, processTool as unknown as AnyAgentTool, // Channel docking: include channel-defined agent tools (login, etc.). ...listChannelAgentTools({ cfg: options?.config }), diff --git a/src/agents/tool-display.json b/src/agents/tool-display.json index f2206e205..3fea81405 100644 --- a/src/agents/tool-display.json +++ b/src/agents/tool-display.json @@ -30,12 +30,6 @@ "title": "Exec", "detailKeys": ["command"] }, - "bash": { - "emoji": "🛠️", - "title": "Exec", - "label": "exec", - "detailKeys": ["command"] - }, "process": { "emoji": "🧰", "title": "Process", diff --git a/src/agents/tool-policy.test.ts b/src/agents/tool-policy.test.ts index a0a234c9f..9e5aff717 100644 --- a/src/agents/tool-policy.test.ts +++ b/src/agents/tool-policy.test.ts @@ -6,8 +6,8 @@ describe("tool-policy", () => { const expanded = expandToolGroups(["group:runtime", "BASH", "apply-patch", "group:fs"]); const set = new Set(expanded); expect(set.has("exec")).toBe(true); - expect(set.has("bash")).toBe(true); expect(set.has("process")).toBe(true); + expect(set.has("bash")).toBe(false); expect(set.has("apply_patch")).toBe(true); expect(set.has("read")).toBe(true); expect(set.has("write")).toBe(true); diff --git a/src/agents/tool-policy.ts b/src/agents/tool-policy.ts index c2ead21d4..ac2b1a91c 100644 --- a/src/agents/tool-policy.ts +++ b/src/agents/tool-policy.ts @@ -17,7 +17,7 @@ export const TOOL_GROUPS: Record = { // Basic workspace/file tools "group:fs": ["read", "write", "edit", "apply_patch"], // Host/runtime execution tools - "group:runtime": ["exec", "bash", "process"], + "group:runtime": ["exec", "process"], // Session management tools "group:sessions": [ "sessions_list",