Phase D — make the rework cycle disappear#15
Merged
Conversation
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
TrajectoryAccumulator folds the event stream (Bash commands with ok outcomes, tool counts, duration, final text) into an atomic <slug>.trajectory.json next to the raw log at stream end. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- restack_stack: dependency-ordered sequential restack with restack-progress events; conflict dispatches the resolver and halts; restack_pr shares the same helper - notify poller: config-gated background fetch reusing the command's no-clobber path, board-updated event, coalesced Rust-side OS notifications for external transitions only (Reworked stays with agent-completed), first cycle seeds silently - get_trajectory_summary + evidence agent_ran folded from the persisted trajectory Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…I, board refresh pairRequests maps each dispatched request to the interdiff span that answers it (side-aware, ±10 tolerance); AddressedRequests becomes a clickable answered/total checklist with an informational unmatched warning beside Approve (never blocking, §9). AgentPanel shows a compact last-run trajectory card (summary-first, raw log stays the escape hatch). StackContainer gains confirmed sequential restack with live progress. board-updated refreshes silently; notify_poll_secs in Settings. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…ed commands Per-child + root agent re-checks mid-sequence (halt with detail reason); restack button re-enables after terminal conflict/error; unresolved Bash commands are counted, not shown as passing; cast style; interdiff-gated addressed panel. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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.
What (ROADMAP Phase D)
Review round
Reviewer: ship-ready with one MAJOR (TOCTOU on the mid-sequence agent guard) — fixed with per-child + root re-checks and a detail-carrying halt event. Also fixed: sticky conflict note hiding the restack button, pending-at-end commands reading as ✓, cast style, checklist flash while the interdiff loads.
Verified
cargo fmt/clippy -D warnings/test --all (445+ & 11 poller-diff tests) · tsc strict · vitest 193 · vite build · reviewer pass + fix round.
🤖 Generated with Claude Code