kilocode/TESTS.md

PR #10507 — Kilo-Specific Test Removal Check

Summary

No Kilo-specific tests were removed. All ~110 Kilo-owned test files under packages/opencode/test/kilocode/ and other Kilo paths are preserved.

Kilo-Specific Test Files Touched in PR

packages/opencode/test/kilocode/

Both files modified — content preserved, only the cancel stub signature was updated to match a Kilo-side type change (TaskPromptOps.cancel now returns an Effect):

• test/kilocode/task-nesting.test.ts — single-line tweak: cancel() {} → cancel: () => Effect.void
• test/kilocode/tool-task-model.test.ts — same single-line tweak

No assertions removed, no test cases dropped.

packages/opencode/test/server/auth.test.ts

kilocode_change marker preserved. The test "defaults to the kilo username" is intact — the comment was just moved from a trailing position to its own line.

packages/kilo-ui/src/components/reasoning-heading.test.ts

Formatting-only change (line break for prettier). Test logic untouched.

Deleted Test Files

One test file was deleted in the diff:

• packages/opencode/test/workspace/workspace-restore.test.ts — upstream-originated test (added by upstream PR #22837, "fix: add a few more tests for sync and session restore"). It was already marked describe.skip(...) on the Kilo side with a kilocode_change comment explaining the upstream Workspace refactor broke the spy-based fixtures. Upstream removed it entirely in this version; we are inheriting that deletion. Not a Kilo-specific test.

No files under test/kilocode/ were deleted, and no other test files were deleted.

Test Run

Root bun install fails due to a native build (tree-sitter-powershell needs make, missing in this sandbox). Worked around with bun install --ignore-scripts. The full bun test suite cannot reasonably finish in the available time window in this environment (>3 min and counting), but the two Kilo-specific tests modified in this PR were run directly and pass:

$ bun test test/kilocode/task-nesting.test.ts
 2 pass
 0 fail
 7 expect() calls
Ran 2 tests across 1 file. [8.89s]

$ bun test test/kilocode/tool-task-model.test.ts
 8 pass
 0 fail
 32 expect() calls
Ran 8 tests across 1 file. [12.02s]

Conclusion

The PR makes only mechanical adjustments to Kilo-owned tests (signature update to match an Effect-returning cancel, plus a comment reflow). The single deleted test file (workspace-restore.test.ts) is upstream-originated, not Kilo-specific, and had already been skipped on our side. The Kilo test suite remains intact.