Repository navigation
feat(agents): preview where interrupted agents were working before resume - #3494
Merged
Merged
Conversation
…sume the resume notice gains a collapsed mention-count disclosure. opening it reads the saved mentions once with the same fresh relay read resume uses, then lists them by channel and thread root with a plain-text excerpt. mentions the read does not return show details unavailable. toast notices take an optional details slot between the message and actions. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Signed-off-by: Kalvin Chau <kalvin@block.xyz>
…able key preview reads by agent so snapshot rebuilds neither cancel nor repeat them, keep failed reads distinct and read them again on the next open, and make the overflowing details list a named, focusable scroll region. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Signed-off-by: Kalvin Chau <kalvin@block.xyz>
wpfleger96
approved these changes
Nov 13, 2026
wpfleger96
left a comment
Member
There was a problem hiding this comment.
🤖 looks good. the preview does the same fresh, proof-filtered readRemote as Resume and never touches the saved turns or follow-ups. reads are kept per agent, so snapshot updates don't restart them, and a failed read (including before the community connects) is read again when the disclosure reopens.
call the real readinterrupted after a stopped app restarts: it reads the saved ids fresh, drops events without relay proof, and leaves the saved turns, run count and interrupted list unchanged. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Signed-off-by: Kalvin Chau <kalvin@block.xyz>
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.
The "Resume interrupted agent?" notice now shows where each interrupted agent was working before you choose to resume.
Change
InterruptedNoticegains a collapsed "N mentions" disclosure (sharedAccordion,activityvariant). N counts the saved mention turns.messagePreview, spoiler-safe).AgentRunners.interruptedTurns(key)exposes the saved turns.Agents.interruptedTurns(agent)andAgents.readInterrupted(agent, signal)back the notice.ToastNoticetakes an optionaldetailsslot between the message and its actions.docs/agents.mdanddocs/buzz-help.mddescribe the preview.Screenshots
Many mentions in WebKit, with the details list focused and scrolled by keyboard:
Evidence
Component tests (
InterruptedNotice.test.tsx):The lifecycle and keyboard cases fail against the first implementation and pass now.
Browser runs (Chromium and WebKit): real reload → notice → expand → Resume/Don't resume against the production browser broker and a modeled relay. Also covered:
bin/pnpm checkpasses.No browser journeys were added or removed; the browser checks above were run with a separate harness.
🤖 Generated with Claude Code