Skip to content

fix: state: safely handle access list reverts#2627

Merged
arajasek merged 1 commit intomainfrom
asr/fix-accesslist
Dec 17, 2025
Merged

fix: state: safely handle access list reverts#2627
arajasek merged 1 commit intomainfrom
asr/fix-accesslist

Conversation

@arajasek
Copy link
Copy Markdown
Contributor

Describe your changes and provide context

Similar to #2626, we should handle potential duplicates safely in access list changes.

Testing performed to validate your change

Added a new test.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 17, 2025

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

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedDec 17, 2025, 6:34 PM

@codecov
Copy link
Copy Markdown

codecov bot commented Dec 17, 2025

Codecov Report

❌ Patch coverage is 50.00000% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 43.61%. Comparing base (18a5fff) to head (b7331d3).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
x/evm/state/journal.go 20.00% 2 Missing and 2 partials ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2627      +/-   ##
==========================================
+ Coverage   42.55%   43.61%   +1.05%     
==========================================
  Files        1557     1899     +342     
  Lines      124605   158396   +33791     
==========================================
+ Hits        53031    69083   +16052     
- Misses      66761    82969   +16208     
- Partials     4813     6344    +1531     
Flag Coverage Δ
sei-chain 45.51% <50.00%> (-0.01%) ⬇️
sei-cosmos 38.04% <ø> (+<0.01%) ⬆️
sei-db 69.06% <ø> (ø)
sei-tendermint 47.50% <ø> (?)

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

Files with missing lines Coverage Δ
x/evm/state/accesslist.go 91.52% <100.00%> (+0.14%) ⬆️
x/evm/state/journal.go 89.18% <20.00%> (-10.82%) ⬇️

... and 348 files with indirect coverage changes

🚀 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 enabled auto-merge (squash) December 17, 2025 18:34
@arajasek arajasek merged commit f07bc96 into main Dec 17, 2025
39 checks passed
@arajasek arajasek deleted the asr/fix-accesslist branch December 17, 2025 19:02
arajasek added a commit that referenced this pull request Jan 29, 2026
arajasek added a commit that referenced this pull request Jan 30, 2026
arajasek added a commit that referenced this pull request Jan 30, 2026
arajasek added a commit that referenced this pull request Feb 2, 2026
arajasek added a commit that referenced this pull request Feb 4, 2026
arajasek added a commit that referenced this pull request Feb 5, 2026
arajasek added a commit that referenced this pull request Feb 6, 2026
arajasek added a commit that referenced this pull request Feb 10, 2026
arajasek added a commit that referenced this pull request Feb 18, 2026
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.

3 participants