Skip to content

Skip inactive TUI threads without pending user interaction - #35525

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/388968277bde34bedc43187c1d1746d76166131e
Jul 26, 2026
Merged

Skip inactive TUI threads without pending user interaction#35525
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/388968277bde34bedc43187c1d1746d76166131e

Conversation

@copyberry

@copyberry copyberry Bot commented Jul 26, 2026

Copy link
Copy Markdown

Skip inactive TUI threads without pending user interaction

What changed

Only collect buffered requests from inactive threads whose event store reports
pending user input or approval. This keeps unrelated requests from being
considered when pending interactions are surfaced after leaving a side
conversation.

Testing

Extend the side-conversation routing test with an unrelated dynamic tool request
and verify that only the pending subagent approval is selected.

## What changed

Only collect buffered requests from inactive threads whose event store reports
pending user input or approval. This keeps unrelated requests from being
considered when pending interactions are surfaced after leaving a side
conversation.

## Testing

Extend the side-conversation routing test with an unrelated dynamic tool request
and verify that only the pending subagent approval is selected.

GitOrigin-RevId: 388968277bde34bedc43187c1d1746d76166131e
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/388968277bde34bedc43187c1d1746d76166131e branch from b1642fa to 0896bf6 Compare July 26, 2026 22:20
@copyberry
copyberry Bot merged commit 0896bf6 into main Jul 26, 2026
14 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/388968277bde34bedc43187c1d1746d76166131e branch July 26, 2026 22:22
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 26, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants