Skip to content

fix(giga): match panic recovery to v2#3045

Draft
arajasek wants to merge 3 commits intomainfrom
asr/giga-misc-fixes
Draft

fix(giga): match panic recovery to v2#3045
arajasek wants to merge 3 commits intomainfrom
asr/giga-misc-fixes

Conversation

@arajasek
Copy link
Contributor

@arajasek arajasek commented Mar 10, 2026

Describe your changes and provide context

  • Seq mode needs to trap panics
  • OCC mode needs to lower panics to the appropriate error code: ErrOCCAbort, ErrOutOfGas, or ErrPanic

Testing performed to validate your change

tests pass

@arajasek arajasek changed the title fix(giga) fix(giga): match panic recovery to v2 Mar 10, 2026
@github-actions
Copy link

github-actions bot commented Mar 10, 2026

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedMar 11, 2026, 1:56 PM

@arajasek arajasek marked this pull request as draft March 10, 2026 13:15
@arajasek
Copy link
Contributor Author

Draft until confirmed it syncs mainnet well

@codecov
Copy link

codecov bot commented Mar 10, 2026

Codecov Report

❌ Patch coverage is 0% with 112 lines in your changes missing coverage. Please review.
✅ Project coverage is 58.30%. Comparing base (2bb2537) to head (29ab486).

Files with missing lines Patch % Lines
app/app.go 0.00% 112 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3045      +/-   ##
==========================================
- Coverage   58.33%   58.30%   -0.03%     
==========================================
  Files        2079     2079              
  Lines      171885   171941      +56     
==========================================
- Hits       100268   100256      -12     
- Misses      62671    62749      +78     
+ Partials     8946     8936      -10     
Flag Coverage Δ
sei-chain-pr 47.97% <0.00%> (?)
sei-db 70.41% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
app/app.go 66.90% <0.00%> (-3.70%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@arajasek arajasek force-pushed the asr/giga-misc-fixes branch from a57cdc2 to 29ab486 Compare March 11, 2026 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant