Add Codex subagent management bar - #211
Merged
Merged
Conversation
0xSolarPunk
force-pushed
the
feat/codex-subagent-panel
branch
from
June 23, 2026 05:49
4150fea to
00d7ae5
Compare
Contributor
Author
|
Review/fix + dogfood pass completed. No existing review comments were present. Fixed a real edge case in commit 00d7ae5: |
7 tasks
Merge current origin/main into feat/codex-subagent-panel and resolve the renderer test conflict. Keep namespaced app dynamic tools external, avoid hidden tool DOM wrappers, count only subagents in the management bar, and support Bun runtimes without Web CompressionStream during compression validation.
yongkangc
marked this pull request as ready for review
June 24, 2026 02:49
# Conflicts: # server/lib/__tests__/http-compression.test.js # server/lib/__tests__/http-route.test.js # server/lib/http-compression.ts # web/src/lib/components/chat/ConversationWorkspace.svelte
Replaces the always-expanded subagent chip strip with a compact "Agents" trigger that opens a popover listing root and subagents. Removes the toolbar collision and per-chip truncation, surfaces an aggregate status dot on the collapsed trigger, and keeps jump-to-origin on each subagent row. Updates screenshots and component tests.
yongkangc
enabled auto-merge (squash)
June 25, 2026 11:29
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
2 participants
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.
Focused follow-up to the merged Codex subagent tool-rendering work (#210), adding a read-only subagent management surface derived from the existing transcript.
What changed
Agentscontrol above the chat feed that collapses to a single trigger (count + aggregate status dot) and opens a popover listing the root and each subagent with status, model, and path. Keeps the surface read-only.Screenshots
Desktop:
Mobile:
Validation
bun run --cwd web test -- src/lib/chat/__tests__/subagent-management.test.ts src/lib/components/chat/__tests__/SubagentManagementBar.test.ts src/lib/components/chat/tools/__tests__/ChatToolEventRenderer.test.ts-> 26 passedbun run --cwd web test-> 1373 passedbun run check-> passedGARCON_HTTP_COMPRESSION=false bun run start --port 0 --disable-auth --config-dir /tmp/garcon-review-dogfood-fixture-> rebuilt, started on an OS-assigned port, and verified the popover desktop/mobile (root + 2 agents in Waiting state, no fakelist_agents/multi-target entries, jump-to-origin from each row)Known inherited local failure:
bun run testserver suite fails locally inserver/lib/__tests__/http-compression.test.jsbecause this Bun environment lacksCompressionStream; GitHub checks are green.