fix: update CLI tool renderer to recognize skill tool and add auto-approval comment

hannesrudolph · hannesrudolph · commit 3d07128e36f8 · 2026-01-28T16:58:14.000-07:00
- Replace fetchInstructions/fetch_instructions with skill in CLI types.ts
- Update CLI utils.ts display names for skill tool
- Add explanatory comment for skill auto-approval policy
diff --git a/apps/cli/src/ui/components/tools/types.ts b/apps/cli/src/ui/components/tools/types.ts
@@ -16,15 +16,7 @@ export type ToolCategory =
 	| "other"
 
 export function getToolCategory(toolName: string): ToolCategory {
-	const fileReadTools = [
-		"readFile",
-		"read_file",
-		"fetchInstructions",
-		"fetch_instructions",
-		"listFilesTopLevel",
-		"listFilesRecursive",
-		"list_files",
-	]
+	const fileReadTools = ["readFile", "read_file", "skill", "listFilesTopLevel", "listFilesRecursive", "list_files"]
 
 	const fileWriteTools = [
 		"editedExistingFile",
diff --git a/apps/cli/src/ui/components/tools/utils.ts b/apps/cli/src/ui/components/tools/utils.ts
@@ -50,8 +50,7 @@ export function getToolDisplayName(toolName: string): string {
 		// File read operations
 		readFile: "Read",
 		read_file: "Read",
-		fetchInstructions: "Fetch Instructions",
-		fetch_instructions: "Fetch Instructions",
+		skill: "Load Skill",
 		listFilesTopLevel: "List Files",
 		listFilesRecursive: "List Files (Recursive)",
 		list_files: "List Files",
diff --git a/src/core/auto-approval/index.ts b/src/core/auto-approval/index.ts
@@ -151,6 +151,9 @@ export async function checkAutoApproval({
 			return { decision: "approve" }
 		}
 
+		// The skill tool only loads pre-defined instructions from built-in, global, or project skills.
+		// It does not read arbitrary files - skills must be explicitly installed/defined by the user.
+		// Auto-approval is intentional to provide a seamless experience when loading task instructions.
 		if (tool.tool === "skill") {
 			return { decision: "approve" }
 		}

Original file line number	Diff line number	Diff line change
`@@ -151,6 +151,9 @@ export async function checkAutoApproval({`
`151`	`151`	`return { decision: "approve" }`
`152`	`152`	`}`
`153`	`153`
	`154`	`+ // The skill tool only loads pre-defined instructions from built-in, global, or project skills.`
	`155`	`+ // It does not read arbitrary files - skills must be explicitly installed/defined by the user.`
	`156`	`+ // Auto-approval is intentional to provide a seamless experience when loading task instructions.`
`154`	`157`	`if (tool.tool === "skill") {`
`155`	`158`	`return { decision: "approve" }`
`156`	`159`	`}`