docs(spec): unified agent panel 2b implementation plan#206
Merged
Conversation
This was referenced Jul 18, 2026
Merged
BunsDev
marked this pull request as ready for review
July 18, 2026 13:21
Contributor
There was a problem hiding this comment.
Pull request overview
Docs-only PR adding the Phase 2b implementation plan for the unified agent panel (AgentPanelView) that merges CLI + daemon conversations into a single feature-flagged surface while keeping the two existing panels untouched.
Changes:
- Adds a 7-task, TDD-oriented plan for implementing
app/src/agent_panel/behind a newFeatureFlag::UnifiedAgentPanel. - Specifies wiring
stream-history.json → sqlitemigration intoChatModelbootstrap so daemon conversations appear in the merged list. - Includes intended workspace wiring (actions, panel field/render path, keybinding, menu) and a layout-safety test.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+600
to
+605
| .with_context_predicate(id!("Workspace")) | ||
| .with_group(bindings::BindingGroup::Navigation.as_str()) | ||
| .with_enabled(crate::agent_panel::feature_flag::is_enabled) | ||
| .with_custom_action(CustomAction::ToggleUnifiedAgentPanel) | ||
| .with_mac_key_binding("cmd-shift-U") | ||
| .with_linux_or_windows_key_binding("ctrl-shift-U"), |
Comment on lines
+611
to
+617
| CustomAction::ToggleUnifiedAgentPanel => { | ||
| if OperatingSystem::get().is_mac() { | ||
| Keystroke::parse("cmd-shift-U").ok() | ||
| } else { | ||
| Keystroke::parse("ctrl-shift-U").ok() | ||
| } | ||
| } |
|
|
||
| ## Done criteria (2b) | ||
|
|
||
| - A new `FeatureFlag::UnifiedAgentPanel` gates `app/src/agent_panel/AgentPanelView`, reachable via `cmd-shift-U` / `ctrl-shift-U` and the menu, rendered at the right dock (360px) — the two old panels untouched and still reachable. |
Comment on lines
+425
to
+427
| use warpui::App; | ||
| use warp::test_util::{terminal::initialize_app_for_terminal_view, add_window_with_terminal}; | ||
|
|
| #[cfg(test)] | ||
| mod view_tests; | ||
|
|
||
| pub use view::mod_view::AgentPanelView; |
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
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.
Description
Docs-only. The 2b implementation plan for the unified agent panel —
specs/castcodes-unified-agent-panel/PLAN-2b.md. Continues the Phase 2 design in #204; the 2a shared core it builds on is #205.2b scope: a new, feature-flagged
app/src/agent_panel/AgentPanelView— a merged conversation list (CLI + migrated Coven-daemon conversations, each badged) + the sharedagent_transcripttranscript + a composer shell (active for live CLI conversations, placeholder otherwise) — reachable by one keybinding, with the two existing panels left untouched. It wires 2a'sstream-history.json → sqlitemigration intoChatModelbootstrap so daemon conversations surface in the list, and adds theViewlayout-safety coverage Phase 1 deferred.Deferred (fenced explicitly in the plan): per-backend composer routing + the live daemon conversation source + stream-mode prompt delivery → 2c; deleting the old panels / making this the default → 2d.
The plan is 7 TDD tasks with exact file paths, signatures (from a full map of the existing
cli_chatview layer + registration points), test code, and per-task signed commits.Linked Issue
Continues the reconciliation roadmap (design #204; 2a core #205; Phase 1 #201). No separate tracking issue, per the #204 precedent.
Testing
./script/check_ai_attribution+./script/check_rebrandpass../script/run(n/a — plan document).Agent Mode