diff options
| author | Ariane Emory <[email protected]> | 2026-03-25 16:18:52 -0400 |
|---|---|---|
| committer | GitHub <[email protected]> | 2026-03-25 15:18:52 -0500 |
| commit | 66a56551beb9299b0de694e5070afe22ab6bcad9 (patch) | |
| tree | a249dd1a4735389860795866b51718604b728036 | |
| parent | 7123aad5a8c8957ee5ae34a0d82c9e6800f7109e (diff) | |
| download | opencode-66a56551beb9299b0de694e5070afe22ab6bcad9.tar.gz opencode-66a56551beb9299b0de694e5070afe22ab6bcad9.zip | |
fix(task): respect agent permission config for todowrite tool (#19125)
| -rw-r--r-- | packages/opencode/src/tool/task.ts | 17 |
1 files changed, 11 insertions, 6 deletions
diff --git a/packages/opencode/src/tool/task.ts b/packages/opencode/src/tool/task.ts index f2a83507f..af130a70d 100644 --- a/packages/opencode/src/tool/task.ts +++ b/packages/opencode/src/tool/task.ts @@ -64,6 +64,7 @@ export const TaskTool = Tool.define("task", async (ctx) => { if (!agent) throw new Error(`Unknown agent type: ${params.subagent_type} is not a valid agent type`) const hasTaskPermission = agent.permission.some((rule) => rule.permission === "task") + const hasTodoWritePermission = agent.permission.some((rule) => rule.permission === "todowrite") const session = await iife(async () => { if (params.task_id) { @@ -75,11 +76,15 @@ export const TaskTool = Tool.define("task", async (ctx) => { parentID: ctx.sessionID, title: params.description + ` (@${agent.name} subagent)`, permission: [ - { - permission: "todowrite", - pattern: "*", - action: "deny", - }, + ...(hasTodoWritePermission + ? [] + : [ + { + permission: "todowrite" as const, + pattern: "*" as const, + action: "deny" as const, + }, + ]), ...(hasTaskPermission ? [] : [ @@ -131,7 +136,7 @@ export const TaskTool = Tool.define("task", async (ctx) => { }, agent: agent.name, tools: { - todowrite: false, + ...(hasTodoWritePermission ? {} : { todowrite: false }), ...(hasTaskPermission ? {} : { task: false }), ...Object.fromEntries((config.experimental?.primary_tools ?? []).map((t) => [t, false])), }, |
