fix(session): render completed agent rows below the done toggle - #557
Merged
Merged
Conversation
Completed agent rows in the Task Activity work graph were rendered above the active rows, so expanding the "N done" toggle revealed them at the top instead of directly beneath the toggle. Split active and completed rows into separate render regions so expanding the toggle shows completed rows immediately below it.
astyfx
requested review from
dodonmountain and
heath-s
and removed request for
a team
September 18, 2026 04:08
astyfx
enabled auto-merge (squash)
September 18, 2026 04:08
|
| CVE | Package | Version | Fix |
|---|---|---|---|
| CVE-2026-18446 | fast-uri |
3.1.0 |
3.1.5 |
| CVE-2026-13676 | fast-uri |
3.1.0 |
3.1.3 |
| CVE-2026-6322 | fast-uri |
3.1.0 |
3.1.2 |
| CVE-2026-76172 | fast-uri |
3.1.0 |
3.1.6 |
| CVE-2026-16221 | fast-uri |
3.1.0 |
3.1.4 |
| CVE-2026-75975 | fast-uri |
3.1.0 |
3.1.6 |
| CVE-2026-6321 | fast-uri |
3.1.0 |
3.1.1 |
View full analysis in Upwind Console
Scan completed in 11s
Scan history (1 scan)
| Commit | Scanned at | New | Resolved | Net |
|---|---|---|---|---|
0d6f504 < |
2026-09-18 04:08 UTC | +7 | 0 | +7 |
Last scanned: 0d6f504 · 2026-09-18 04:08 UTC
|
| Commit | Scanned at | New | Resolved | Net |
|---|---|---|---|---|
0d6f504 < |
2026-09-18 04:08 UTC | 0 | 0 | 0 |
Last scanned: 0d6f504 · 2026-09-18 04:08 UTC
Merged
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
In the Task Activity work graph, completed agent rows rendered above the active rows, so expanding the "N done" toggle revealed them at the top of the section instead of directly beneath the toggle. This splits active and completed rows into separate render regions so the expanded completed rows appear immediately below the toggle.
Changes
Fixes
Refactors / Chores
Test Evidence
bun test tests/work-graph-tree-render.test.tsx— 14 pass, 0 fail (adds a regression assertion thatLive auditprecedes the1 donetoggle, which precedesFinished audit, and that the completed rows region ishiddenwhile collapsed)bun run typecheck— passesbun run check:design-system— passes (ratchet: 0 files contain utility syntax)Notes
bun run check:licensesfails locally due to a missinglicense-checkermodule in this worktree'snode_modules(pre-existing environment issue, unrelated to this diff).