diff --git a/packages/core/test/tool-edit.test.ts b/packages/core/test/tool-edit.test.ts index 1e1d806d93..aaecf9766f 100644 --- a/packages/core/test/tool-edit.test.ts +++ b/packages/core/test/tool-edit.test.ts @@ -1,7 +1,6 @@ import fs from "fs/promises" import path from "path" -import { fileURLToPath } from "url" -import { describe, expect, test } from "bun:test" +import { describe, expect } from "bun:test" import { Effect, Layer } from "effect" import { AppNodeBuilder } from "@opencode-ai/core/effect/app-node-builder" import { LayerNode } from "@opencode-ai/util/effect/layer-node" @@ -460,25 +459,3 @@ describe("EditTool", () => { ), ) }) - -test("keeps the locked edit schema, semantics docstring, and deferred TODOs visible", async () => { - const source = (await fs.readFile(new URL("../src/tool/edit.ts", import.meta.url), "utf8")).replaceAll("\r\n", "\n") - const definition = await Effect.runPromise( - withTool(path.dirname(fileURLToPath(import.meta.url)), (registry) => toolDefinitions(registry)), - ) - const schema = definition[0]?.inputSchema as { readonly properties?: Record } - - expect(Object.keys(schema.properties ?? {}).sort()).toEqual(["newString", "oldString", "path", "replaceAll"]) - expect(source).toContain( - "absolute external paths retain mutation capability through a separate\n * external_directory approval before edit approval.", - ) - for (const todo of [ - "Port V1 fuzzy correction strategies only after exact-edit behavior is established: line-trimmed matching, block-anchor fallback, indentation correction, and similarity-threshold review.", - "Add formatter integration after V2 formatter runtime exists.", - "Publish watcher/file-edit events after V2 watcher integration exists.", - "Add snapshots / undo after design exists.", - "Add LSP notification and diagnostics after V2 LSP runtime exists.", - ]) { - expect(source).toContain(`TODO: ${todo}`) - } -}) diff --git a/packages/core/test/tool-shell.test.ts b/packages/core/test/tool-shell.test.ts index 8d79cde7a9..039680a8d1 100644 --- a/packages/core/test/tool-shell.test.ts +++ b/packages/core/test/tool-shell.test.ts @@ -1,7 +1,7 @@ import fs from "fs/promises" import { realpathSync } from "node:fs" import path from "path" -import { describe, expect, test } from "bun:test" +import { describe, expect } from "bun:test" import { DateTime, Deferred, Duration, Effect, Fiber, Layer, Scope, Stream } from "effect" import { Money } from "@opencode-ai/schema/money" import { AppNodeBuilder } from "@opencode-ai/core/effect/app-node-builder" @@ -643,20 +643,3 @@ describe("ShellTool", () => { ), ) }) - -test("keeps locked deferred parity TODOs visible", async () => { - const source = await fs.readFile(new URL("../src/tool/shell.ts", import.meta.url), "utf8") - for (const todo of [ - "Port tree-sitter bash / PowerShell parser-based approval reduction.", - "Port BashArity reusable command-prefix approvals.", - "Replace token-based command-argument external-directory advisories with parser-based detection.", - "Restore PowerShell and cmd-specific invocation/path handling on Windows.", - "Add plugin shell.env environment augmentation once V2 plugin hooks exist.", - "Persist job status and define restart recovery before exposing remote observation.", - "Revisit process-group cleanup and platform coverage with shell-specific tests if current AppProcess semantics do not fully cover it.", - "Revisit binary output handling if stdout/stderr decoding is text-only.", - "Stream full shell output into managed storage while retaining only a bounded in-memory preview.", - ]) { - expect(source).toContain(`TODO: ${todo}`) - } -}) diff --git a/packages/core/test/tool-write.test.ts b/packages/core/test/tool-write.test.ts index 6e4d61c57c..67a001e0eb 100644 --- a/packages/core/test/tool-write.test.ts +++ b/packages/core/test/tool-write.test.ts @@ -1,7 +1,6 @@ import fs from "fs/promises" import path from "path" -import { fileURLToPath } from "url" -import { describe, expect, test } from "bun:test" +import { describe, expect } from "bun:test" import { Effect, Layer } from "effect" import { FileMutation } from "@opencode-ai/core/file-mutation" import { AppNodeBuilder } from "@opencode-ai/core/effect/app-node-builder" @@ -363,25 +362,3 @@ describe("WriteTool", () => { ), ) }) - -test("keeps the locked write schema, semantics docstring, and deferred UX TODOs visible", async () => { - const source = (await fs.readFile(new URL("../src/tool/write.ts", import.meta.url), "utf8")).replaceAll("\r\n", "\n") - const definition = await Effect.runPromise( - withTool(path.dirname(fileURLToPath(import.meta.url)), (registry) => toolDefinitions(registry)), - ) - const schema = definition[0]?.inputSchema as { readonly properties?: Record } - - expect(Object.keys(schema.properties ?? {}).sort()).toEqual(["content", "path"]) - expect(source).toContain( - "absolute external paths retain mutation capability through a separate\n * external_directory approval before edit approval.", - ) - for (const todo of [ - "Revisit whether model-facing mutation schemas should prefer absolute `filePath` naming for trained-in compatibility after evaluating model behavior.", - "Add formatter integration after V2 formatter runtime exists.", - "Publish watcher/file-edit events after V2 watcher integration exists.", - "Add snapshots / undo after design exists.", - "Add LSP notification and diagnostics after V2 LSP runtime exists.", - ]) { - expect(source).toContain(`TODO: ${todo}`) - } -})