Repository navigation
Agent: distinguish sidebar task activity states - #1213
Merged
Merged
Conversation
doowta
marked this pull request as draft
October 22, 2026 16:02
Collaborator
|
I'm picking this up to work on it. |
Collaborator
|
Video of it implemented. maple-activity-indicator.mov |
A task paused on an ask_user question looked the same as one still running, and a run that failed off screen left no trace in the list. The row indicator now has four states: a coral dot while a question waits, the spinner while a run goes, an alert once a run failed unseen, and the green dot once one completed unseen. The screen pushes them as one TaskActivity. A question outranks the run it paused; a run outranks how the previous one ended. The failed marker follows the unread one: a failure off screen sets it, opening or archiving the task clears it, and a stop sets nothing. Questions arriving, answered, skipped, resolved, or dropped with their run now update the row. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The turning arc steps visibly at the shared clock's low frame rate. A running task's row now shows a coral ring whose center dot fades in and out on that same clock, which reads as smooth at the same cost. Only the sidebar row changes; the loading spinner elsewhere stays. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Rows for another computer's tasks only knew running and unread, so a task waiting on a question or one that failed looked idle there. The computer snapshot now carries the same task activity as this computer's sidebar, and a question arriving or going away refreshes it. Deleting a task or archiving its project also left a failed marker behind, so both now clear it along with the unread one. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Archiving a project clears its tasks' unseen markers, but no test checked the failed one, or that another project's markers stay. The project removal test now seeds both markers in each project and checks both sides. The queued questions test also checks the sidebar row goes from needs-you to idle once the last question is answered, like the skip test does. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
marksftw
force-pushed
the
ux/sidebar-status
branch
from
October 29, 2026 22:36
ec35e1c to
3615d6f
Compare
marksftw
force-pushed
the
sidebar-spacing
branch
from
October 29, 2026 22:36
749d0e6 to
7c84aff
Compare
marksftw
marked this pull request as ready for review
October 29, 2026 22:42
Contributor
|
Approving. Thanks Michael and Mark. Telling "needs you" apart from "running" at a glance is a real improvement. I tested it the way it merges now: current master (with #1373) plus this PR, against master, using real builds, the Pro test account and a second computer paired as Seen live:
The code is UI only: no protocol, settings or persisted-state changes. The breathing ring uses the same shared clock the spinner used, so idle repaint cost is unchanged. Clippy and all 1,175 app tests pass on the merged tree. Not blocking, noted in case they come up later
🤖 Generated with Claude Code |
AnthonyRonning
approved these changes
Oct 31, 2026
AnthonyRonning
left a comment
Contributor
There was a problem hiding this comment.
Approved; details and screenshot in the comment above.
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.

Tasks waiting on a question previously looked the same as running tasks, and an unseen failure had no sidebar marker. This change gives those states distinct indicators across connected computers.
Markers clear when their task is seen, archived or deleted; cancellation adds no marker. Question resolution restores the appropriate running state. The activity model preserves current paging, task placement and bounded navigation snapshots.
Validation:
Stack: layer 2 of 2, based on #1373 (
sidebar-spacing). Layout and cross-computer task controls are reviewed in the lower layer. The four original indicator commits remain separate, with their authorship preserved.In each image, one task waits on a question and the other is running.
Scroll sideways to compare. Click an image for full size.
Full-size images