Skip to content

Commit 4b4dd2b

Browse files
authored
fix: Add apply_patch to EDIT_TOOLS filter (#18009)
1 parent bc949af commit 4b4dd2b

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

packages/opencode/src/permission/next.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ export namespace PermissionNext {
6969
return S.evaluate(permission, pattern, ...rulesets)
7070
}
7171

72-
const EDIT_TOOLS = ["edit", "write", "patch", "multiedit"]
72+
const EDIT_TOOLS = ["edit", "write", "apply_patch", "multiedit"]
7373

7474
export function disabled(tools: string[], ruleset: Ruleset): Set<string> {
7575
const result = new Set<string>()

packages/opencode/test/permission/next.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -395,17 +395,17 @@ test("disabled - disables tool when denied", () => {
395395
expect(result.has("read")).toBe(false)
396396
})
397397

398-
test("disabled - disables edit/write/patch/multiedit when edit denied", () => {
398+
test("disabled - disables edit/write/apply_patch/multiedit when edit denied", () => {
399399
const result = PermissionNext.disabled(
400-
["edit", "write", "patch", "multiedit", "bash"],
400+
["edit", "write", "apply_patch", "multiedit", "bash"],
401401
[
402402
{ permission: "*", pattern: "*", action: "allow" },
403403
{ permission: "edit", pattern: "*", action: "deny" },
404404
],
405405
)
406406
expect(result.has("edit")).toBe(true)
407407
expect(result.has("write")).toBe(true)
408-
expect(result.has("patch")).toBe(true)
408+
expect(result.has("apply_patch")).toBe(true)
409409
expect(result.has("multiedit")).toBe(true)
410410
expect(result.has("bash")).toBe(false)
411411
})

0 commit comments

Comments
 (0)