🤖 fix: prevent concurrent agent banner wake flicker#3713
Conversation
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9876c1478d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
Addressed the inactive-workspace startup gap by adding an authoritative @codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d8c11fb0a4
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
Addressed the activity-list race: @codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c7f785131a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
Seeded @codex review |
|
Seeded @codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9dd4f14ca3
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
Cleared the live pending wake signal after terminal-attention notifications transition to delivered (and after re-pending on pre-stream failure), so normal @codex review |
|
Codex Review: Didn't find any major issues. 🎉 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Summary
Keep the concurrent local-agent warning stable while another agent cycles through background wake-ups, including the handoff gap after a one-shot monitor or terminal task notification is consumed but before the synthetic turn starts streaming.
Background
The warning originally only considered an interruptible stream active. Background monitor matches and workflow/task terminal messages transition the owning workspace through waiting, queued/preparing, and streaming states. For inactive workspaces, pre-stream startup was not exposed in activity snapshots, so the banner could still unmount and remount on every wake cycle.
Implementation
pendingBackgroundWakeworkspace activity field.Validation
make typecheckmake static-checkRisks
Moderate and localized to workspace activity signaling. The pending flag is recomputed from durable stores and live session state, and is cleared on stream start, cancellation, supersession, or consumed terminal attention; versioned refreshes prevent older async reads from restoring stale state.
Generated with
mux• Model:openai:gpt-5.6-sol• Thinking:high• Cost:$2.94