Skip to content

fix: validate broker frames before entering the send phase - #414

Draft
ejgallego wants to merge 4 commits into
mainfrom
codex/bt128-436-scope-20261103
Draft

ejgallego wants to merge 4 commits into
mainfrom
codex/bt128-436-scope-20261103

Conversation

@ejgallego

@ejgallego ejgallego commented Nov 3, 2026 •

Copy link
Copy Markdown
Collaborator

This PR returns a correlated structured error when an oversized broker result or validation reply is rejected before any frame bytes are written. Previously, a completed result such as a large hover could surface as brokerUnavailable even though the Lean backend remained healthy.

  • Prepare size-checked frame bytes before changing output state, including greetings and recovery replies; preserve connection abandonment after a send begins.
  • Preserve request correlation in validation-error recovery and retain the machine-readable operation_may_have_completed warning only on result-delivery failures.
  • Cover byte boundaries, committed effects, stopped-reader send deadlines, and raw-protocol recovery followed by strict EOF.

@ejgallego ejgallego added this to the 0.2.1 milestone Nov 3, 2026 — with ChatGPT Codex Connector
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