fix: add view image to plan mode allow list (#1496)

This commit is contained in:
Christina Tong
2026-03-23 17:53:48 -07:00
committed by GitHub
parent e3ad1a3cf6
commit ad56c3d273
2 changed files with 23 additions and 0 deletions

View File

@@ -300,6 +300,9 @@ class PermissionModeManager {
"Grep",
"NotebookRead",
"TodoWrite",
// Image tools (read-only)
"ViewImage",
"view_image",
// Plan mode tools (must allow exit!)
"ExitPlanMode",
"exit_plan_mode",

View File

@@ -233,6 +233,26 @@ test("plan mode - allows Read", () => {
expect(result.matchedRule).toBe("plan mode");
});
test("plan mode - allows ViewImage", () => {
permissionMode.setMode("plan");
const permissions: PermissionRules = {
allow: [],
deny: [],
ask: [],
};
const result = checkPermission(
"ViewImage",
{ path: "/Users/test/image.png" },
permissions,
"/Users/test/project",
);
expect(result.decision).toBe("allow");
expect(result.matchedRule).toBe("plan mode");
});
test("plan mode - allows TaskOutput", () => {
permissionMode.setMode("plan");