feat(extension): add browser agent side panel#4164
Open
iscekic wants to merge 112 commits into
Open
Conversation
![kilo-code-bot[bot]](https://avatars.githubusercontent.com/in/2193792?s=60&v=4){kind=small}
Contributor
Code Review SummaryStatus: 1 Issue Found | Recommendation: Address before merge Overview
Fix these issues in Kilo Cloud Issue Details (click to expand)WARNING
Files Reviewed (2 files)
Previous Review Summaries (26 snapshots, latest commit bcf0cde)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit bcf0cde)Status: No Issues Found | Recommendation: Merge Files Reviewed (3 files)
Previous review (commit 933bd7f)Status: No Issues Found | Recommendation: Merge Files Reviewed (4 files)
Previous review (commit ba14af1)Status: No Issues Found | Recommendation: Merge Files Reviewed (1 file)
Previous review (commit 80a5733)Status: 1 Issue Found | Recommendation: Address before merge Overview
Fix these issues in Kilo Cloud Issue Details (click to expand)WARNING
Files Reviewed (1 file)
Previous review (commit cb03ce6)Status: No Issues Found | Recommendation: Merge Files Reviewed (4 files)
Previous review (commit c1a6acc)Status: 1 Issue Found | Recommendation: Address before merge Overview
Fix these issues in Kilo Cloud Issue Details (click to expand)WARNING
Files Reviewed (4 files)
Previous review (commit a3035c8)Status: 1 Issue Found | Recommendation: Address before merge Overview
Fix these issues in Kilo Cloud Issue Details (click to expand)WARNING
Files Reviewed (1 file)
Previous review (commit e5015a3)Status: 1 Issue Found | Recommendation: Address before merge Overview
Fix these issues in Kilo Cloud Issue Details (click to expand)WARNING
Files Reviewed (1 file)
Previous review (commit 521f62e)Status: 1 Issue Found | Recommendation: Address before merge Overview
Fix these issues in Kilo Cloud Issue Details (click to expand)WARNING
Files Reviewed (3 files)
Previous review (commit f1614fe)Status: No Issues Found | Recommendation: Merge Files Reviewed (1 file)
Previous review (commit 7bfacc3)Status: 1 Issue Found | Recommendation: Address before merge Overview
Fix these issues in Kilo Cloud Issue Details (click to expand)WARNING
Files Reviewed (2 files)
Previous review (commit e864cd0)Status: 1 Issue Found | Recommendation: Address before merge Overview
Fix these issues in Kilo Cloud Issue Details (click to expand)WARNING
Files Reviewed (2 files)
Previous review (commit 3a454df)Status: No Issues Found | Recommendation: Merge Files Reviewed (4 files)
Previous review (commit 105ad8d)Status: No Issues Found | Recommendation: Merge Files Reviewed (4 files)
Previous review (commit 42f8fa4)Status: 1 Issue Found | Recommendation: Address before merge Overview
Fix these issues in Kilo Cloud Issue Details (click to expand)WARNING
Files Reviewed (6 files)
Previous review (commit 9d1483c)Status: 1 Issue Found | Recommendation: Address before merge Overview
Fix these issues in Kilo Cloud Issue Details (click to expand)WARNING
Files Reviewed (3 files)
Previous review (commit 9d20e19)Status: 2 Issues Found | Recommendation: Address before merge Overview
Fix these issues in Kilo Cloud Issue Details (click to expand)WARNING
Files Reviewed (11 files)
Previous review (commit ffb2e2a)Status: No Issues Found | Recommendation: Merge Files Reviewed (4 files)
Previous review (commit 6f5f0bc)Status: 1 Issue Found | Recommendation: Address before merge Overview
Fix these issues in Kilo Cloud Issue Details (click to expand)WARNING
Files Reviewed (10 files)
Previous review (commit eaebdd5)Status: No Issues Found | Recommendation: Merge Files Reviewed (9 files)
Previous review (commit 31f5646)Status: No Issues Found | Recommendation: Merge Files Reviewed (2 files)
Previous review (commit 08a436f)Status: No Issues Found | Recommendation: Merge Files Reviewed (2 files)
Previous review (commit 31d1edb)Status: 2 Issues Found | Recommendation: Address before merge Executive SummaryThe new Chrome and Firefox reload scenarios both reintroduce the same async-persistence race, so they can pass or fail nondeterministically without proving conversation tabs survive a side-panel refresh. Fix these issues in Kilo Cloud Overview
Issue Details (click to expand)WARNING
Files Reviewed (11 files)
Previous review (commit edd91e2)Status: 3 Issues Found | Recommendation: Address before merge Executive SummaryThe React Query auth refactor can strand approved sign-ins in the pending view, and the related polling/test changes still leave stale extension state insufficiently guarded. Fix these issues in Kilo Cloud Overview
Issue Details (click to expand)WARNING
Files Reviewed (16 files)
Previous review (commit 4564e5c)Status: 3 Issues Found | Recommendation: Address before merge Executive SummaryThe React Query auth refactor can strand approved sign-ins in the pending view, and the related polling/test changes still leave stale extension state insufficiently guarded. Fix these issues in Kilo Cloud Overview
Issue Details (click to expand)WARNING
Files Reviewed (16 files)
Previous review (commit 64b3f95)Status: 3 Issues Found | Recommendation: Address before merge Executive SummaryThe React Query auth refactor can strand approved sign-ins in the pending view, and the related polling/test changes still leave stale extension state insufficiently guarded. Fix these issues in Kilo Cloud Overview
Issue Details (click to expand)WARNING
Files Reviewed (16 files)
Additional previous summary content was truncated to keep this comment within platform limits. Reviewed by gpt-5.4-20260305 · Input: 96.6K · Output: 10.9K · Cached: 743.4K Review guidance: REVIEW.md from base branch |
| if (attached) { | ||
| try { | ||
| await debuggerApi.detach(target); | ||
| } catch { |
Contributor
There was a problem hiding this comment.
WARNING: Catch-all detach suppression can hide leaked debugger attachments
The benign cases here are the tab closing or already being detached, but this unconditional catch also swallows genuine chrome.debugger.detach() failures. When detach really does fail, the tab can remain attached and the next dangerous-mode run on that tab will start failing to reattach while this call still reports success. Restrict the suppression to the known closed/already-detached error shapes so real cleanup failures still surface.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds the new browser extension app under
apps/extension, including a native Chrome side panel, Kilo auth, model loading, tab selection, dangerous-mode eval tooling through Chrome debugger, and a gateway-backed streaming LLM harness. The side panel includes compact Kilo-styled controls, virtualized conversation history, fixed shell scrolling, and E2E coverage for the core extension flows.Verification
Manual verification not run; no manual browser screenshots were captured for this extension scaffold.
Visual Changes
N/A
Reviewer Notes
Review focus areas: Chrome extension permissions, debugger/eval execution path, streaming chat-completions parsing, auth-token validation, and the new WXT/Playwright package setup.