Repository navigation
Conversation
|
⏹️ Reflex agent status: Stopped
This PR was created by Reflex. This comment updates in place as agents work. |
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Claude replay re-dispatched historical control requests after
control_cancel_request, causing a cancelled AskUserQuestion to park again on reconnect. Correlate agent cancellations by their top-levelrequest_id, alongside existing user response pairing, so only pending requests reach application handlers.An SSE disconnect before the captured replay head also used to flush requests before their responses or cancellations could arrive. Keep the replay buffer with the transport's resume cursor, finish correlation after reconnect, and clear it on close. This shared transport correction also applies to Codex.
Validation:
bun run check,bun run typecheck,bun run build, andbun run testpass (28 files, 662 tests). New contract tests instantiate the real Claude connection, transport, and read loop against a fake Axon boundary. They cover the REF-249 cancellation fixture with full/bounded replay, allow/deny/error responses, request-specific cancellation and origin checks, pending questions preceding newer queries, replacement connections preserving request identity, and disconnects during replay.Upstream prerequisite for REF-249; this PR does not complete the issue. Reflex still needs shared browser/MCP lifecycle interpretation and request-aware recovery bounds, with its application integration coverage. Its current newest-prompt bound can omit an older pending request. No broker change is included. Package versioning follows the repository's Release Please workflow; Reflex currently consumes the latest published version, 0.4.13.