fix(provider): normalize terminal failures#907
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
dcramer
marked this pull request as ready for review
July 15, 2026 17:35
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
dcramer
force-pushed
the
fix/uniform-provider-errors
branch
from
July 24, 2026 16:23
22698f0 to
2f93dc4
Compare
dcramer
changed the title
dcramer
force-pushed
the
fix/uniform-provider-errors
branch
from
July 24, 2026 16:34
2f93dc4 to
ad53f6c
Compare
dcramer
force-pushed
the
fix/uniform-provider-errors
branch
from
July 24, 2026 17:03
ad53f6c to
8248f3c
Compare
dcramer
force-pushed
the
fix/uniform-provider-errors
branch
from
July 24, 2026 19:51
8248f3c to
daa9242
Compare
dcramer
force-pushed
the
fix/uniform-provider-errors
branch
from
July 24, 2026 20:04
daa9242 to
d1efef6
Compare
dcramer
force-pushed
the
fix/uniform-provider-errors
branch
from
July 24, 2026 22:02
e9ab39f to
23f4d24
Compare
dcramer
force-pushed
the
fix/uniform-provider-errors
branch
from
July 24, 2026 22:13
23f4d24 to
5cfdf56
Compare
dcramer
force-pushed
the
fix/uniform-provider-errors
branch
from
July 24, 2026 22:29
5cfdf56 to
2b4d4d2
Compare
dcramer
force-pushed
the
fix/uniform-provider-errors
branch
from
July 24, 2026 22:53
2b4d4d2 to
ecd2ef6
Compare
dcramer
force-pushed
the
fix/uniform-provider-errors
branch
from
July 24, 2026 23:32
ecd2ef6 to
d4e6361
Compare
Classify provider errors once for retry policy, telemetry, and sanitized terminal replies. Preserve retry hints without leaking upstream payloads to users. Refs JUNIOR-53
Classify known connection and timeout codes through nested error causes so transient gateway failures retain retry behavior.
Keep the normalized provider classifier compatible with Pi's transient provider and transport patterns while retaining Junior's typed error kinds.
dcramer
force-pushed
the
fix/uniform-provider-errors
branch
from
July 25, 2026 02:33
d4e6361 to
04bfe43
Compare
dcramer
force-pushed
the
fix/uniform-provider-errors
branch
from
July 25, 2026 02:53
04bfe43 to
33fa92d
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 33fa92d. Configure here.
Delegate assistant retry recognition to Pi, keep Junior's terminal vetoes narrow, and avoid recording raw provider payloads in normalized errors. Refs JUNIOR-53
dcramer
force-pushed
the
fix/uniform-provider-errors
branch
from
July 25, 2026 03:04
33fa92d to
491dd8a
Compare
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.
Provider failures now use one structured contract across Gateway, Pi, and the
turn runtime. Retry decisions recognize verified transient status and transport
signals, honor bounded
retry-afterhints, and no longer depend on rewritingError.name.Terminal provider failures carry safe classification fields into Sentry and
show actionable, sanitized user copy with the existing event reference. Unknown
failures keep the generic fallback, partial model-authored replies keep their
existing interruption behavior, and Gateway fallback configuration remains out
of scope.
Refs JUNIOR-53.
Requested by immutable dcramer via Junior.
--
View Junior Session in Sentry