fix(vscode): fix Open in Editor button in Documents viewer

[brunoagatao]

What changed

openRelativeFile silently did nothing when the file path was absolute (or when root was unset), because resolveInside returns undefined for any absolute input. The Documents viewer's go-to-file action sends the document's path as-is, which is often absolute, so clicking the button had no effect.

openRelativeFile now opens absolute paths directly via openFileInEditor, instead of requiring the path to resolve inside a workspace root.

Why

Fixes #14597. Users clicking the "Open file in editor" icon in the Kilo Documents webview panel saw no editor tab open and no error.

Testing

• Added unit tests in packages/kilo-vscode/tests/unit/review-utils.test.ts covering absolute paths with and without a root.
• bun run typecheck, bun run lint, and the targeted bun test tests/unit/review-utils.test.ts all pass.

[kilo-code-bot]

Code Review Summary

Status: No Issues Found | Recommendation: Merge

The incremental commit collapses openRelativeFile's absolute-path early return into a single ternary (path.isAbsolute(...) ? relativePath : root && resolveInside(...)). Behavior is unchanged for every input — absolute path, unset root, empty root, and paths inside/outside the selected repository — so the original fix still holds and no new issues were introduced. Memory-leak/shared-state check: the function only forwards a one-shot vscode.open command and retains nothing.

Files Reviewed (3 files)

• .changeset/documents-open-in-editor.md
• packages/kilo-vscode/src/review-utils.ts
• packages/kilo-vscode/tests/unit/review-utils.test.ts

Previous Review Summary (commit 029802e)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit 029802e)

Status: No Issues Found | Recommendation: Merge

The fix correctly restores the go-to-file action: openRelativeFile previously returned early via resolveInside for any absolute input, so the Documents viewer silently did nothing. Opening absolute paths directly matches the existing openFileInEditor contract, and the new unit tests cover both the unset-root and outside-root cases. No memory-leak or shared-state concerns: the added branch only forwards a one-shot vscode.open command.

Files Reviewed (3 files)

• .changeset/documents-open-in-editor.md
• packages/kilo-vscode/src/review-utils.ts
• packages/kilo-vscode/tests/unit/review-utils.test.ts

---

_{Reviewed by deepseek-v4.1-flash · Input: 0 · Output: 0 · Cached: 0}

_{Review guidance: REVIEW.md from base branch main}