Skip to content

feat(vscode): add Agent Manager document inspector - #13245

Merged
marius-kilocode merged 16 commits into
mainfrom
implement-markdown-file-viewer-sidebar
Aug 20, 2026
Merged

feat(vscode): add Agent Manager document inspector#13245
marius-kilocode merged 16 commits into
mainfrom
implement-markdown-file-viewer-sidebar

Conversation

@marius-kilocode

Copy link
Copy Markdown
Collaborator

Agent Manager can now open and review Markdown and text artifacts without leaving the session view. This makes generated plans and other agent-produced documents easier to inspect, discuss, and send back to the agent.

The document inspector provides:

  • Plan and artifact links that open in a dedicated Documents panel.
  • Per-worktree document tabs using the standard file-type icons.
  • Rendered Markdown and source views, plus safe handling for text, image, binary, and oversized files.
  • Inline Markdown review comments that can be drafted, edited, deleted, or sent to the active session.
  • Preview actions from changed Markdown files in the diff viewer.
  • Documents toolbar visibility scoped to the current worktree, including the empty-panel close flow.
  • Localized document and plan labels.

The document UI is grouped under webview-ui/agent-manager/documents/ and document reads use containment and size checks before returning file contents.

Plan links open as document links in Agent Manager

Agent Manager documents inspector with multiple Markdown tabs

Rendered Markdown document preview

Inline document review comment editor

Review comments listed for a document

@marius-kilocode
marius-kilocode force-pushed the implement-markdown-file-viewer-sidebar branch from 796d23f to 65a025b Compare August 19, 2026 14:29
@marius-kilocode
marius-kilocode force-pushed the implement-markdown-file-viewer-sidebar branch from e4e4129 to 5aa2cb8 Compare August 19, 2026 14:34
Comment thread packages/kilo-vscode/webview-ui/agent-manager/documents/state.ts Outdated
Comment thread packages/kilo-vscode/webview-ui/agent-manager/documents/state.ts Outdated
Comment thread packages/kilo-vscode/webview-ui/documents/state.ts
Comment thread packages/kilo-vscode/src/agent-manager/worktree-diff-controller.ts Outdated
Comment thread packages/kilo-vscode/src/agent-manager/worktree-diff-controller.ts Outdated
Comment thread packages/kilo-vscode/webview-ui/documents/DocumentPanel.tsx
Comment thread packages/kilo-vscode/webview-ui/documents/DocumentPanel.tsx
Comment thread packages/kilo-vscode/src/diff/DiffViewerProvider.ts Outdated
Comment thread packages/kilo-vscode/webview-ui/diff-viewer/FullScreenDiffView.tsx Outdated
Comment thread .changeset/rendered-document-inspector.md
@kilo-code-bot

kilo-code-bot Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • packages/kilo-vscode/src/DocumentViewerProvider.ts
  • packages/kilo-vscode/webview-ui/documents/index.tsx
Previous Review Summaries (5 snapshots, latest commit 6369249)

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

Previous review (commit 6369249)

Status: 2 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 2
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
packages/kilo-vscode/src/DocumentViewerProvider.ts 101 document.openFile reopens the inspector instead of the editor
packages/kilo-vscode/webview-ui/documents/index.tsx 36 Sending comments both re-queues them locally and posts them twice
Files Reviewed (18 files)
  • packages/kilo-vscode/src/DocumentViewerProvider.ts - 1 issue
  • packages/kilo-vscode/webview-ui/documents/index.tsx - 1 issue
  • packages/kilo-vscode/esbuild.js
  • packages/kilo-vscode/knip.json
  • packages/kilo-vscode/src/KiloProvider.ts
  • packages/kilo-vscode/src/agent-manager/worktree-diff-controller.ts
  • packages/kilo-vscode/src/documents/document-reader.ts
  • packages/kilo-vscode/src/extension.ts
  • packages/kilo-vscode/tests/unit/agent-manager-arch.test.ts
  • packages/kilo-vscode/tests/unit/agent-manager-documents.test.ts
  • packages/kilo-vscode/tests/unit/agent-manager-i18n.test.ts
  • packages/kilo-vscode/tests/unit/document-reader.test.ts
  • packages/kilo-vscode/webview-ui/agent-manager/AgentManagerApp.tsx
  • packages/kilo-vscode/webview-ui/agent-manager/documents/DocumentPanelHost.tsx
  • packages/kilo-vscode/webview-ui/documents/DocumentPanel.tsx
  • packages/kilo-vscode/webview-ui/documents/state.ts
  • packages/kilo-vscode/webview-ui/src/types/messages/extension-messages.ts
  • packages/kilo-vscode/webview-ui/src/types/messages/webview-messages.ts

Fix these issues in Kilo Cloud

Previous review (commit 343622c)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 files)
  • packages/kilo-vscode/webview-ui/src/components/chat/ReviewComments.tsx

Previous review (commit 0415f72)

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
packages/kilo-vscode/webview-ui/src/components/chat/ReviewComments.tsx 45 Fallback openFile drops sessionID, so worktree comments can open the wrong file
Files Reviewed (6 files)
  • packages/kilo-vscode/webview-ui/src/components/chat/ReviewComments.tsx - 1 issue
  • packages/kilo-vscode/webview-ui/agent-manager/documents/DocumentPanel.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/agent-manager/documents/DocumentPanelHost.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/agent-manager/AgentManagerApp.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/agent-manager/agent-manager.css - 0 issues
  • packages/kilo-vscode/webview-ui/diff-viewer/FullScreenDiffView.tsx - 0 issues

Fix these issues in Kilo Cloud

Previous review (commit 056db46)

Status: 10 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 9
SUGGESTION 1
Issue Details (click to expand)

WARNING

File Line Issue
packages/kilo-vscode/webview-ui/agent-manager/documents/state.ts 170 preventDefault() cancels editor fallback before the inspector accepts the file
packages/kilo-vscode/webview-ui/agent-manager/documents/state.ts 168 Open event sessionID is ignored, so citations can preview the wrong worktree
packages/kilo-vscode/webview-ui/agent-manager/documents/state.ts 34 Document tabs/comments are not project-namespaced and can leak across projects
packages/kilo-vscode/src/agent-manager/worktree-diff-controller.ts 215 Unresolved worktree/session ids fall back to the main checkout
packages/kilo-vscode/src/agent-manager/worktree-diff-controller.ts 216 Missing root posts nothing, leaving the inspector stuck on loading
packages/kilo-vscode/webview-ui/agent-manager/documents/DocumentPanel.tsx 173 Failed/empty reread deletes all staged comments for that file
packages/kilo-vscode/webview-ui/agent-manager/documents/DocumentPanel.tsx 154 Draft state is not reset on tab/worktree change and blocks new comments
packages/kilo-vscode/src/diff/DiffViewerProvider.ts 61 Non-markdown Changes opens force markdown render off
packages/kilo-vscode/webview-ui/diff-viewer/FullScreenDiffView.tsx 239 initialFile is ignored unless the current selection is empty/stale

SUGGESTION

File Line Issue
.changeset/rendered-document-inspector.md 2 CLI plan-citation change is not included in the changeset
Files Reviewed (38 files)
  • .changeset/rendered-document-inspector.md - 1 issue
  • packages/kilo-vscode/src/agent-manager/document-reader.ts - 0 issues
  • packages/kilo-vscode/src/agent-manager/worktree-diff-controller.ts - 2 issues
  • packages/kilo-vscode/src/agent-manager/AgentManagerProvider.ts - 0 issues
  • packages/kilo-vscode/src/agent-manager/types.ts - 0 issues
  • packages/kilo-vscode/src/diff/DiffViewerProvider.ts - 1 issue
  • packages/kilo-vscode/src/diff/types.ts - 0 issues
  • packages/kilo-vscode/src/KiloProvider.ts - 0 issues
  • packages/kilo-vscode/src/kilo-provider/editor-actions.ts - 0 issues
  • packages/kilo-vscode/src/extension.ts - 0 issues
  • packages/kilo-vscode/webview-ui/agent-manager/documents/state.ts - 3 issues
  • packages/kilo-vscode/webview-ui/agent-manager/documents/DocumentPanel.tsx - 2 issues
  • packages/kilo-vscode/webview-ui/agent-manager/documents/DocumentPanelHost.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/agent-manager/AgentManagerApp.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/agent-manager/DiffPanel.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/agent-manager/TabBar.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/agent-manager/ClosableTab.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/agent-manager/side-panel-layout.ts - 0 issues
  • packages/kilo-vscode/webview-ui/agent-manager/agent-manager.css - 0 issues
  • packages/kilo-vscode/webview-ui/src/App.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/src/components/chat/AssistantMessage.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/diff-viewer/DiffViewerApp.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/diff-viewer/FullScreenDiffView.tsx - 1 issue
  • packages/kilo-vscode/webview-ui/diff-viewer/MarkdownAnnotationLayer.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/diff-viewer/MarkdownDiffView.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/src/types/messages/extension-messages.ts - 0 issues
  • packages/kilo-vscode/webview-ui/src/types/messages/webview-messages.ts - 0 issues
  • packages/kilo-vscode/webview-ui/src/styles/plan-exit.css - 0 issues
  • packages/kilo-vscode/tests/unit/document-reader.test.ts - 0 issues
  • packages/kilo-vscode/tests/unit/agent-manager-arch.test.ts - 0 issues
  • packages/kilo-vscode/tests/unit/agent-manager-i18n.test.ts - 0 issues
  • packages/kilo-ui/src/components/message-part.tsx - 0 issues
  • packages/kilo-ui/src/components/message-part.css - 0 issues
  • packages/opencode/src/kilocode/session/prompt.ts - 0 issues
  • packages/ui/src/components/markdown.css - 0 issues
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/* - 0 issues
  • packages/ui/src/i18n/* - 0 issues

Fix these issues in Kilo Cloud

Previous review (commit 5aa2cb8)

Status: 10 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 9
SUGGESTION 1
Issue Details (click to expand)

WARNING

File Line Issue
packages/kilo-vscode/webview-ui/agent-manager/documents/state.ts 170 preventDefault() cancels editor fallback before the inspector accepts the file
packages/kilo-vscode/webview-ui/agent-manager/documents/state.ts 168 Open event sessionID is ignored, so citations can preview the wrong worktree
packages/kilo-vscode/webview-ui/agent-manager/documents/state.ts 34 Document tabs/comments are not project-namespaced and can leak across projects
packages/kilo-vscode/src/agent-manager/worktree-diff-controller.ts 215 Unresolved worktree/session ids fall back to the main checkout
packages/kilo-vscode/src/agent-manager/worktree-diff-controller.ts 216 Missing root posts nothing, leaving the inspector stuck on loading
packages/kilo-vscode/webview-ui/agent-manager/documents/DocumentPanel.tsx 173 Failed/empty reread deletes all staged comments for that file
packages/kilo-vscode/webview-ui/agent-manager/documents/DocumentPanel.tsx 154 Draft state is not reset on tab/worktree change and blocks new comments
packages/kilo-vscode/src/diff/DiffViewerProvider.ts 61 Non-markdown Changes opens force markdown render off
packages/kilo-vscode/webview-ui/diff-viewer/FullScreenDiffView.tsx 239 initialFile is ignored unless the current selection is empty/stale

SUGGESTION

File Line Issue
.changeset/rendered-document-inspector.md 2 CLI plan-citation change is not included in the changeset
Files Reviewed (38 files)
  • .changeset/rendered-document-inspector.md - 1 issue
  • packages/kilo-vscode/src/agent-manager/document-reader.ts - 0 issues
  • packages/kilo-vscode/src/agent-manager/worktree-diff-controller.ts - 2 issues
  • packages/kilo-vscode/src/agent-manager/AgentManagerProvider.ts - 0 issues
  • packages/kilo-vscode/src/agent-manager/types.ts - 0 issues
  • packages/kilo-vscode/src/diff/DiffViewerProvider.ts - 1 issue
  • packages/kilo-vscode/src/diff/types.ts - 0 issues
  • packages/kilo-vscode/src/KiloProvider.ts - 0 issues
  • packages/kilo-vscode/src/kilo-provider/editor-actions.ts - 0 issues
  • packages/kilo-vscode/src/extension.ts - 0 issues
  • packages/kilo-vscode/webview-ui/agent-manager/documents/state.ts - 3 issues
  • packages/kilo-vscode/webview-ui/agent-manager/documents/DocumentPanel.tsx - 2 issues
  • packages/kilo-vscode/webview-ui/agent-manager/documents/DocumentPanelHost.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/agent-manager/AgentManagerApp.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/agent-manager/DiffPanel.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/agent-manager/TabBar.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/agent-manager/ClosableTab.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/agent-manager/side-panel-layout.ts - 0 issues
  • packages/kilo-vscode/webview-ui/agent-manager/agent-manager.css - 0 issues
  • packages/kilo-vscode/webview-ui/src/App.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/src/components/chat/AssistantMessage.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/diff-viewer/DiffViewerApp.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/diff-viewer/FullScreenDiffView.tsx - 1 issue
  • packages/kilo-vscode/webview-ui/diff-viewer/MarkdownAnnotationLayer.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/diff-viewer/MarkdownDiffView.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/src/types/messages/extension-messages.ts - 0 issues
  • packages/kilo-vscode/webview-ui/src/types/messages/webview-messages.ts - 0 issues
  • packages/kilo-vscode/webview-ui/src/styles/plan-exit.css - 0 issues
  • packages/kilo-vscode/tests/unit/document-reader.test.ts - 0 issues
  • packages/kilo-vscode/tests/unit/agent-manager-arch.test.ts - 0 issues
  • packages/kilo-vscode/tests/unit/agent-manager-i18n.test.ts - 0 issues
  • packages/kilo-ui/src/components/message-part.tsx - 0 issues
  • packages/kilo-ui/src/components/message-part.css - 0 issues
  • packages/opencode/src/kilocode/session/prompt.ts - 0 issues
  • packages/ui/src/components/markdown.css - 0 issues
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/* - 0 issues
  • packages/ui/src/i18n/* - 0 issues

Fix these issues in Kilo Cloud


Reviewed by grok-4.6 · Input: 178.2K · Output: 8.7K · Cached: 173.6K

Review guidance: REVIEW.md from base branch main

…le-viewer-sidebar

# Conflicts:
#	packages/kilo-vscode/webview-ui/agent-manager/AgentManagerApp.tsx
#	packages/kilo-vscode/webview-ui/diff-viewer/FullScreenDiffView.tsx
#	packages/kilo-vscode/webview-ui/src/components/chat/ReviewComments.tsx
@marius-kilocode
marius-kilocode force-pushed the implement-markdown-file-viewer-sidebar branch from c5d3827 to 9026260 Compare August 20, 2026 08:58
@marius-kilocode
marius-kilocode force-pushed the implement-markdown-file-viewer-sidebar branch from 13150e2 to 55333d4 Compare August 20, 2026 09:07
…le-viewer-sidebar

# Conflicts:
#	packages/kilo-vscode/webview-ui/agent-manager/AgentManagerApp.tsx
#	packages/kilo-vscode/webview-ui/agent-manager/TabBar.tsx
Comment thread packages/kilo-vscode/webview-ui/src/components/chat/ReviewComments.tsx Outdated
@marius-kilocode
marius-kilocode force-pushed the implement-markdown-file-viewer-sidebar branch from 0415f72 to 8e26de0 Compare August 20, 2026 10:12
@marius-kilocode
marius-kilocode force-pushed the implement-markdown-file-viewer-sidebar branch from 343622c to 3be279f Compare August 20, 2026 11:10
@marius-kilocode
marius-kilocode force-pushed the implement-markdown-file-viewer-sidebar branch from ffa2c01 to eda1e9c Compare August 20, 2026 11:28
Comment thread packages/kilo-vscode/src/DocumentViewerProvider.ts Outdated
Comment thread packages/kilo-vscode/webview-ui/documents/index.tsx Outdated
@marius-kilocode
marius-kilocode force-pushed the implement-markdown-file-viewer-sidebar branch from 6369249 to ba1c3c6 Compare August 20, 2026 11:55
@marius-kilocode
marius-kilocode merged commit 6f87e9c into main Aug 20, 2026
36 checks passed
@marius-kilocode
marius-kilocode deleted the implement-markdown-file-viewer-sidebar branch August 20, 2026 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants