Skip to content

Fix input queue deadlock after session rebound - #168

Merged
Pigbibi merged 1 commit into
mainfrom
fix/rebound-input-confirmation-deadlock
Sep 1, 2026
Merged

Fix input queue deadlock after session rebound#168
Pigbibi merged 1 commit into
mainfrom
fix/rebound-input-confirmation-deadlock

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Sep 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • scope unresolved input confirmation locks to the authoritative bound window after a topic rebind
  • retire superseded confirmations after a read-only transcript check, without replaying old text
  • keep route-wide serialization while a rebind is unconfirmed or ambiguous
  • add regression coverage for the production deadlock and durable target filtering

Root cause

A submitted_unconfirmed record tied to a hibernated tmux window continued locking the entire Telegram topic after the topic resumed in a new window. Later messages entered the Bot FIFO but its drainer could never start, so users saw Queued until the agent is ready indefinitely instead of native steer/queue handling.

Verification

  • rtk ruff check src/ tests/
  • rtk ruff format --check src/ tests/
  • rtk pyright src/telegram_agent_bot/
  • rtk pytest (816 passed)

Co-Authored-By: Codex <noreply@openai.com>
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

🤖 Codex PR Review

⚠️ Review skipped: The Codex review could not be completed.

No Codex service URL or API key configured. Set CODEX_AUDIT_SERVICE_URL, ANTHROPIC_API_KEY, or OPENAI_API_KEY.

Please ensure a human reviewer checks this PR before merging.

@Pigbibi
Pigbibi merged commit 97820b9 into main Sep 1, 2026
7 checks passed
@Pigbibi
Pigbibi deleted the fix/rebound-input-confirmation-deadlock branch September 1, 2026 06:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant