[pull] main from vercel:main - #539
Merged
Merged
Conversation
…ng forever (#3742) * fix(world-vercel): fail the run on a lost event payload instead of retrying A frame stream that dies mid-body reaches us as a truncated response, which is exactly what a dropped socket looks like. So an event whose stored payload is permanently gone was indistinguishable from a transient blip, and the runtime kept redelivering a replay that could never succeed: one run re-read a single missing payload 12,932 times in 26 minutes, and the backend query behind each attempt throttled its table. The World now sends a terminal `{_error: 1, code}` frame for failures that a retry cannot fix. Handle it: - `payload-missing` raises `CorruptedEventLogError`, so the run fails with `CORRUPTED_EVENT_LOG` rather than looping. The log does reference a payload nothing can produce. - An unknown code raises a `WorkflowWorldError` with no retryable code and no status, which is also terminal. A future code stays safe without needing a client release first. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> * Revert the world-vercel URL override to empty Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> * fix(world-vercel): classify terminal stream errors Signed-off-by: Alex Langenfeld <alex.langenfeld@vercel.com> --------- Signed-off-by: Alex Langenfeld <alex.langenfeld@vercel.com> Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com> Co-authored-by: Alex Langenfeld <alex.langenfeld@vercel.com>
* Make the python docs codeownership more targetted I had originally left it broad so that things would be smooth when we start reorganizing the python docs, but we'll adjust that when it happens. * Update .github/CODEOWNERS Co-authored-by: vercel[bot] <35613825+vercel[bot]@users.noreply.github.com> Signed-off-by: Michael J. Sullivan <sully@msully.net> --------- Signed-off-by: Michael J. Sullivan <sully@msully.net> Co-authored-by: vercel[bot] <35613825+vercel[bot]@users.noreply.github.com>
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
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.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )