Agents: drop bash tool alias
This commit is contained in:
committed by
Peter Steinberger
parent
15620b1092
commit
ff52aec38e
@@ -255,11 +255,6 @@ export function createClawdbotCodingTools(options?: {
|
|||||||
}
|
}
|
||||||
: undefined,
|
: undefined,
|
||||||
});
|
});
|
||||||
const bashTool = {
|
|
||||||
...(execTool as unknown as AnyAgentTool),
|
|
||||||
name: "bash",
|
|
||||||
label: "bash",
|
|
||||||
} satisfies AnyAgentTool;
|
|
||||||
const processTool = createProcessTool({
|
const processTool = createProcessTool({
|
||||||
cleanupMs: cleanupMsOverride ?? execConfig.cleanupMs,
|
cleanupMs: cleanupMsOverride ?? execConfig.cleanupMs,
|
||||||
scopeKey,
|
scopeKey,
|
||||||
@@ -280,7 +275,6 @@ export function createClawdbotCodingTools(options?: {
|
|||||||
: []),
|
: []),
|
||||||
...(applyPatchTool ? [applyPatchTool as unknown as AnyAgentTool] : []),
|
...(applyPatchTool ? [applyPatchTool as unknown as AnyAgentTool] : []),
|
||||||
execTool as unknown as AnyAgentTool,
|
execTool as unknown as AnyAgentTool,
|
||||||
bashTool,
|
|
||||||
processTool as unknown as AnyAgentTool,
|
processTool as unknown as AnyAgentTool,
|
||||||
// Channel docking: include channel-defined agent tools (login, etc.).
|
// Channel docking: include channel-defined agent tools (login, etc.).
|
||||||
...listChannelAgentTools({ cfg: options?.config }),
|
...listChannelAgentTools({ cfg: options?.config }),
|
||||||
|
|||||||
@@ -30,12 +30,6 @@
|
|||||||
"title": "Exec",
|
"title": "Exec",
|
||||||
"detailKeys": ["command"]
|
"detailKeys": ["command"]
|
||||||
},
|
},
|
||||||
"bash": {
|
|
||||||
"emoji": "🛠️",
|
|
||||||
"title": "Exec",
|
|
||||||
"label": "exec",
|
|
||||||
"detailKeys": ["command"]
|
|
||||||
},
|
|
||||||
"process": {
|
"process": {
|
||||||
"emoji": "🧰",
|
"emoji": "🧰",
|
||||||
"title": "Process",
|
"title": "Process",
|
||||||
|
|||||||
@@ -6,8 +6,8 @@ describe("tool-policy", () => {
|
|||||||
const expanded = expandToolGroups(["group:runtime", "BASH", "apply-patch", "group:fs"]);
|
const expanded = expandToolGroups(["group:runtime", "BASH", "apply-patch", "group:fs"]);
|
||||||
const set = new Set(expanded);
|
const set = new Set(expanded);
|
||||||
expect(set.has("exec")).toBe(true);
|
expect(set.has("exec")).toBe(true);
|
||||||
expect(set.has("bash")).toBe(true);
|
|
||||||
expect(set.has("process")).toBe(true);
|
expect(set.has("process")).toBe(true);
|
||||||
|
expect(set.has("bash")).toBe(false);
|
||||||
expect(set.has("apply_patch")).toBe(true);
|
expect(set.has("apply_patch")).toBe(true);
|
||||||
expect(set.has("read")).toBe(true);
|
expect(set.has("read")).toBe(true);
|
||||||
expect(set.has("write")).toBe(true);
|
expect(set.has("write")).toBe(true);
|
||||||
|
|||||||
@@ -17,7 +17,7 @@ export const TOOL_GROUPS: Record<string, string[]> = {
|
|||||||
// Basic workspace/file tools
|
// Basic workspace/file tools
|
||||||
"group:fs": ["read", "write", "edit", "apply_patch"],
|
"group:fs": ["read", "write", "edit", "apply_patch"],
|
||||||
// Host/runtime execution tools
|
// Host/runtime execution tools
|
||||||
"group:runtime": ["exec", "bash", "process"],
|
"group:runtime": ["exec", "process"],
|
||||||
// Session management tools
|
// Session management tools
|
||||||
"group:sessions": [
|
"group:sessions": [
|
||||||
"sessions_list",
|
"sessions_list",
|
||||||
|
|||||||
Reference in New Issue
Block a user