Honor cancellation for LangGraph state writes - #1138
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Contributor
blove
enabled auto-merge (squash)
September 23, 2026 15:16
Contributor
Contributor
Forward the existing AbortSignal to LangGraph state writes and cover cancellation through normal and protected transports with real HTTP regressions. Include the generated API documentation and parity inventory update.
blove
force-pushed
the
codex/state-write-cancellation
branch
from
September 23, 2026 19:21
54c3472 to
f91f5c8
Compare
Contributor
Contributor
Contributor
This branch was successfully 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
1 participant
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.
FetchStreamTransport.updateStateaccepted an abort signal but never passed it to the LangGraph SDK. Pre-aborted writes still reached the server, and cancelling an in-flight write left its HTTP response open.Forward the existing signal alongside values/asNode. Six real-SDK tests against a local HTTP server cover pre-abort, in-flight cancellation/connection closure, and unchanged successful request bodies through normal and protected transports. Cancellation reporting remains suppressed. Four cancellation cases fail before the fix; all six pass after it.
Validation: all 594 runtime tests, runtime/public type checks, LangGraph lint, the legacy transport test target, inventory and source boundaries pass. The inventory refresh records the transport change without changing capability assignments. Independent source review approved the change and separately ran all six HTTP tests successfully.
This honors request cancellation; it does not roll back a state write already accepted by the server. Durable late tool settlement retains its deliberately independent signal. No new checkpoint execution API is introduced.
Integration: current main is included and all PR commits are signed. Head
d570471b5passed 594 runtime tests, runtime/public types, 183 workflow/scope/review-verification tests, inventory and source boundaries. Current head420c30f8e552adfcc772c88f0ce6bc73ae3bfa86adds only main's website comment corrections; the fresh 594-test runtime run and website font-variable target pass. Required CI run 35913390368 must pass on this exact head before merge. The diff remains limited to the transport, six HTTP regressions, inventory fingerprint and generated API parameter name.