fix(workflows): keep follow-up Working visible

[flora131]

Summary

Keep the attached workflow-stage Working indicator visible for workflow-authored follow-up turns from admission through retained-session restore, prompt startup, pre-turn compaction, and agent handoff. Correlated delivery ownership also prevents late or overlapping lifecycle events from clearing or restarting the indicator at the wrong time.

Changes

• Add a workflow delivery-activity channel with replay for late attach and remount.
• Start Working before lazy retained-session restore and settle it on success, no-turn, failure, replacement, or disposal.
• Preserve Working through pre-turn compaction and restore the terminal stale-event fence after the final delivery settles.
• Prevent one overlapping delivery from clearing a lifecycle generation still owned by another delivery.
• Cover direct attach, late attach, post-mortem restore, pause, terminal races, overlapping leases, compaction, stale events, and cleanup with deterministic tests.
• Update the workflow UI contract and [Unreleased] changelog entry.

Validation

• Focused lifecycle suite: 146 passed, 0 failed
• bun run test:unit: 4,386 passed, 2 skipped, 0 failed
• bun run test:integration: 288 passed, 1 skipped, 0 failed
• bun run typecheck: passed
• bun run lint: passed
• bun run check:file-length: passed
• bunx tsgo -p packages/coding-agent/tsconfig.build.json --noEmit: passed
• git diff --check origin/main...HEAD: passed
• Two fresh reviewers approved the final repair.

Tmux evidence

This is the exact tmux capture-pane output from a fresh deterministic terminal replay. It runs the real workflow delivery channel and StageChatView lifecycle in a Bun test harness; it is not a live-provider/full Atomic CLI screenshot. The replay proves that Working remains visible while delivery B still owns the reconciled lifecycle, clears after the final settlement, and cannot be restarted by a stale terminal event.

Fresh tmux replay

AGENT=1 bun test /tmp/workflow-overlap-review-opus.test.ts; echo "EXIT=$?"
bun test v1.3.14 (0d9b296a)

../../../../private/tmp/workflow-overlap-review-opus.test.ts:
delivery_start A (workflow-authored sendUserMessage accepted)
delivery_start B (overlapping accepted delivery)
pre-turn compaction reconciled -> status: ∀ Working...
after settling A (B accepted, agent_start still pending) -> glyph=true animation=true status: ∀ Working...
B public agent_start -> animation=true
after final settlement -> glyph=false animation=false status: <no status line>
stale post-terminal start -> glyph=false animation=false
REVIEWER REPLAY OK

 1 pass
 0 fail
Ran 1 test across 1 file. [2.89s]
EXIT=0

A second independent tmux replay produced the same state sequence:

delivery_start A
delivery_start B
compaction_start
compaction_end / reconciled
after first settlement: Working=true, animation=true
after final settlement: Working=false, animation=false
stale terminal start: Working=false, animation=false
1 pass, 0 fail, EXIT=0

Notes

No package versions or generated artifacts changed. Commits are conventional and carry Assistant-model attribution.

---

^{Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.}

Greptile Summary

Implements a correlated delivery-activity lifecycle for workflow-authored follow-up turns.

• Replays active delivery leases when a stage chat attaches or remounts.
• Keeps Working visible through retained-session restoration, prompt startup, compaction, and agent handoff.
• Coordinates overlapping delivery ownership and fences stale lifecycle events after terminal settlement.
• Adds deterministic unit and integration coverage plus workflow documentation and changelog updates.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

T-Rex Logs

What T-Rex did

• Baseline revision failed both integration lifecycle cases because Working was absent before agent_start.
• The PR revision passed both integration lifecycle cases.
• The complete focused plan passed 27/27 tests with exit code 0.
• Four log artifacts were uploaded to document the test results and evidence.

_{Ran code and verified through T-Rex}

Important Files Changed

Filename	Overview
packages/workflows/src/runs/foreground/stage-delivery-activity.ts	Adds correlated active-delivery leases with synchronous replay for late subscribers and deterministic settlement.
packages/workflows/src/runs/foreground/stage-runner-controller.ts	Integrates retained-session preparation and delivery activity with serialized message admission and disposal.
packages/workflows/src/runs/foreground/stage-runner-send-user-message.ts	Emits activity around admitted idle delivery paths while leaving streaming follow-up and steering deliveries under the existing turn lifecycle.
packages/workflows/src/tui/stage-chat-view-delivery-activity.ts	Maps correlated delivery leases onto host lifecycle generations, including overlap, compaction reconciliation, and terminal fencing.
packages/workflows/src/tui/stage-chat-view-live-events.ts	Reconciles delivery-owned activity after compaction and suppresses stale terminal lifecycle starts.
packages/workflows/src/tui/stage-chat-view-state.ts	Subscribes to delivery activity before SDK events and cleans up lifecycle state during terminal transitions and disposal.
packages/coding-agent/src/modes/interactive/components/chat-session-host-runtime.ts	Exposes generation-aware APIs for beginning, reasserting, and settling externally submitted prompt activity.

Sequence Diagram

sequenceDiagram
  participant Workflow
  participant Controller as StageSessionController
  participant Activity as StageDeliveryActivity
  participant View as StageChatView
  participant Host as ChatSessionHost
  participant Session as AgentSession
  Workflow->>Controller: sendUserMessage()
  Controller->>Activity: delivery_start(id)
  Activity-->>View: delivery_start(id)
  View->>Host: beginExternalPromptLifecycle()
  Host-->>View: lifecycle generation
  Controller->>Session: restore and deliver prompt
  Session-->>View: compaction / agent lifecycle events
  View->>Host: reconcile lifecycle after compaction
  Session-->>Controller: delivery settles
  Controller->>Activity: delivery_settled(id)
  Activity-->>View: delivery_settled(id)
  View->>Host: settle lifecycle after final owner

Loading

_{Reviews (2): Last reviewed commit: "Merge origin/main into fix/workflow-foll..." | Re-trigger Greptile}

[mintlify]

Preview deployment for your docs. Learn more about Mintlify Previews.

Project	Status	Preview	Updated (UTC)
bastani	🟢 Ready	View Preview	Jul 29, 2026, 12:39 AM

💡 Tip: Enable Workflows to automatically generate PRs for you.