chore: kill unused tool (#23701)

Author: rekram1-node

packages/opencode/specs/effect/instance-context.md

@@ -224,7 +224,6 @@ These tools mostly use direct getters for path resolution and repo-relative disp
 - `src/tool/bash.ts`
 - `src/tool/edit.ts`
 - `src/tool/lsp.ts`
-- `src/tool/multiedit.ts`
 - `src/tool/plan.ts`
 - `src/tool/read.ts`
 - `src/tool/write.ts`

packages/opencode/specs/effect/schema.md

@@ -216,7 +216,6 @@ emitted JSON Schema must stay byte-identical.
 - [ ] `src/tool/grep.ts`
 - [ ] `src/tool/invalid.ts`
 - [ ] `src/tool/lsp.ts`
-- [ ] `src/tool/multiedit.ts`
 - [ ] `src/tool/plan.ts`
 - [ ] `src/tool/question.ts`
 - [ ] `src/tool/read.ts`

packages/opencode/specs/effect/tools.md

@@ -46,7 +46,6 @@ These exported tool definitions currently use `Tool.define(...)` in `src/tool`:
 - [x] `grep.ts`
 - [x] `invalid.ts`
 - [x] `lsp.ts`
-- [x] `multiedit.ts`
 - [x] `plan.ts`
 - [x] `question.ts`
 - [x] `read.ts`
@@ -82,7 +81,6 @@ Notable items that are already effectively on the target path and do not need se
 - `write.ts`
 - `codesearch.ts`
 - `websearch.ts`
-- `multiedit.ts`
 - `edit.ts`
 
 ## Filesystem notes

packages/opencode/src/config/agent.ts

@@ -93,7 +93,7 @@ const normalize = (agent: z.infer<typeof Info>) => {
   const permission: ConfigPermission.Info = {}
   for (const [tool, enabled] of Object.entries(agent.tools ?? {})) {
     const action = enabled ? "allow" : "deny"
-    if (tool === "write" || tool === "edit" || tool === "patch" || tool === "multiedit") {
+    if (tool === "write" || tool === "edit" || tool === "patch") {
       permission.edit = action
       continue
     }

packages/opencode/src/config/config.ts

@@ -660,7 +660,7 @@ export const layer = Layer.effect(
           const perms: Record<string, ConfigPermission.Action> = {}
           for (const [tool, enabled] of Object.entries(result.tools)) {
             const action: ConfigPermission.Action = enabled ? "allow" : "deny"
-            if (tool === "write" || tool === "edit" || tool === "patch" || tool === "multiedit") {
+            if (tool === "write" || tool === "edit" || tool === "patch") {
               perms.edit = action
               continue
             }

packages/opencode/src/permission/index.ts

@@ -307,7 +307,7 @@ export function merge(...rulesets: Ruleset[]): Ruleset {
   return rulesets.flat()
 }
 
-const EDIT_TOOLS = ["edit", "write", "apply_patch", "multiedit"]
+const EDIT_TOOLS = ["edit", "write", "apply_patch"]
 
 export function disabled(tools: string[], ruleset: Ruleset): Set<string> {
   const result = new Set<string>()

packages/opencode/src/tool/multiedit.ts

@@ -474,7 +474,7 @@ test("legacy tools config converts to permissions", async () => {
   })
 })
 
-test("legacy tools config maps write/edit/patch/multiedit to edit permission", async () => {
+test("legacy tools config maps write/edit/patch to edit permission", async () => {
   await using tmp = await tmpdir({
     config: {
       agent: {

packages/opencode/src/tool/multiedit.txt

@@ -1427,35 +1427,6 @@ test("migrates legacy patch tool to edit permission", async () => {
   })
 })
 
-test("migrates legacy multiedit tool to edit permission", async () => {
-  await using tmp = await tmpdir({
-    init: async (dir) => {
-      await Filesystem.write(
-        path.join(dir, "opencode.json"),
-        JSON.stringify({
-          $schema: "https://opencode.ai/config.json",
-          agent: {
-            test: {
-              tools: {
-                multiedit: false,
-              },
-            },
-          },
-        }),
-      )
-    },
-  })
-  await Instance.provide({
-    directory: tmp.path,
-    fn: async () => {
-      const config = await load()
-      expect(config.agent?.["test"]?.permission).toEqual({
-        edit: "deny",
-      })
-    },
-  })
-})
-
 test("migrates mixed legacy tools config", async () => {
   await using tmp = await tmpdir({
     init: async (dir) => {