feat(session): open a detail dialog for subagents, workers, and child tasks - #558
Merged
Merged
Conversation
… tasks Clicking an agent row in the Agent Tree, Collaboration panel, and Turn Activity now opens a read-only detail dialog instead of only jumping to a turn event. The dialog shows the assignment, live and saved activity, the returned result, and the model and reasoning effort in use, labelled with where each value came from (requested, configured, or runtime-reported). Saved subagent transcripts are loaded read-only over a new provider.read-agent-history contract: the Claude path validates subagent membership before returning messages, and the Codex path reads a stored thread with turns without resuming it and rejects a thread from another working directory. Transcripts are kept as inert text, oversized entries are truncated at the persisted-event ceiling, and the reader is never exposed as a resume, fork, or start path. Advisor rows keep their existing dedicated log path.
astyfx
requested review from
JongSikLim and
heath-s
and removed request for
a team
September 18, 2026 04:43
astyfx
enabled auto-merge (squash)
September 18, 2026 04:43
|
| Commit | Scanned at | New | Resolved | Net |
|---|---|---|---|---|
ff1361e < |
2026-09-18 04:43 UTC | 0 | 0 | 0 |
Last scanned: ff1361e · 2026-09-18 04:43 UTC
|
| Commit | Scanned at | New | Resolved | Net |
|---|---|---|---|---|
ff1361e < |
2026-09-18 04:44 UTC | 0 | 0 | 0 |
Last scanned: ff1361e · 2026-09-18 04:44 UTC
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.
Summary
Clicking an agent row in the Agent Tree, Collaboration panel, and Turn Activity now opens a read-only detail dialog that shows the assignment, live and saved activity, the result, and the model and reasoning effort in use, instead of only jumping to a turn event.
Changes
Features
requested,configured, or runtime-reported— so a displayed value is no longer mistaken for confirmed execution telemetry.provider.read-agent-historycontract that loads a subagent transcript without resuming, forking, or starting an agent. Claude validates subagent membership vialistSubagentsbefore returning messages; Codex reads a stored thread with turns (includeTurns) and rejects a thread from another working directory.Fixes
spawnAgent.Refactors / Chores
AgentNodeand the reducer carrymodel,effort, andmodelEvidence; tool-use parts carryagentId,ownerAgentId, andparentToolUseIdthrough replay and the persisted schema.Breaking Changes
Test Evidence
bun run typecheck: clean.bun test tests/agent-history.test.ts: 11 pass — request-contract bounds, inert-text handling, provider mapping, and 64 KiB truncation.bun testacross the directly-affected suites (agent-history, codex-worker-activity, codex-app-server-runtime, work-graph, delegation/exchange, turn-activity, provider-event-replay): 329 pass, 0 fail.bun test: 5843 pass. The 62 failures are load-based timeouts and subprocess-timing flakes (ACP, executable-probe, CDP guest removal, untracked-working-tree diff) in files this PR does not touch; each passes when run outside the saturated full-suite run.check:licensesfails only becauselicense-checkeris absent from this worktree'snode_modules; no dependency or license file changed.Notes