fix: treat mint limit errors as definitive during issue recovery#2113
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 411dfde52b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
411dfde to
5da5eb8
Compare
|
@codex review |
|
Codex Review: Didn't find any major issues. 🚀 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2113 +/- ##
==========================================
+ Coverage 71.41% 71.48% +0.06%
==========================================
Files 356 356
Lines 73757 73857 +100
==========================================
+ Hits 52676 52798 +122
+ Misses 21081 21059 -22 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Issue saga recovery now stops on definitive replay failures such as max output/input rejections instead of falling back to restore. Decode those mint error responses into structured variants so recovery preserves and surfaces the original failure.
5da5eb8 to
bfb5b8d
Compare
|
Successfully created backport PR for |
Issue saga recovery now stops on definitive replay failures such as max output/input rejections instead of falling back to restore. Decode those mint error responses into structured variants so recovery preserves and surfaces the original failure.
Description
Notes to the reviewers
Suggested CHANGELOG Updates
CHANGED
ADDED
REMOVED
FIXED
Checklist
just quick-checkbefore committingcrates/cdk-ffi)