fix: auto-approve skill tool (no user confirmation required)

hannesrudolph · hannesrudolph · commit 5d6388913680 · 2026-01-29T11:56:10.000-07:00
Skills are informational content that don't perform any actions,
so they should be auto-approved like updateTodoList.
diff --git a/src/core/auto-approval/index.ts b/src/core/auto-approval/index.ts
@@ -151,6 +151,10 @@ export async function checkAutoApproval({
 			return { decision: "approve" }
 		}
 
+		if (tool.tool === "skill") {
+			return { decision: "approve" }
+		}
+
 		if (tool?.tool === "switchMode") {
 			return state.alwaysAllowModeSwitch === true ? { decision: "approve" } : { decision: "ask" }
 		}

Original file line number	Diff line number	Diff line change
`@@ -151,6 +151,10 @@ export async function checkAutoApproval({`
`151`	`151`	`return { decision: "approve" }`
`152`	`152`	`}`
`153`	`153`
	`154`	`+ if (tool.tool === "skill") {`
	`155`	`+ return { decision: "approve" }`
	`156`	`+ }`
	`157`	`+`
`154`	`158`	`if (tool?.tool === "switchMode") {`
`155`	`159`	`return state.alwaysAllowModeSwitch === true ? { decision: "approve" } : { decision: "ask" }`
`156`	`160`	`}`