Skip to content

feat(spec-specs, tests): Add EIP-7981 Increase Access List Cost#2877

Merged
marioevz merged 13 commits into
forks/amsterdamfrom
eips/amsterdam/eip-7981
May 19, 2026
Merged

feat(spec-specs, tests): Add EIP-7981 Increase Access List Cost#2877
marioevz merged 13 commits into
forks/amsterdamfrom
eips/amsterdam/eip-7981

Conversation

@felix314159

@felix314159 felix314159 commented May 19, 2026

Copy link
Copy Markdown
Contributor

🗒️ Description

Merge EIP-7981 into forks/amsterdam.

🔗 Related Issues or PRs

✅ Checklist

  • All: Ran fast static checks to avoid unnecessary CI fails, see also Code Standards and Enabling Pre-commit Checks: 
    just static
  • All: PR title adheres to the repo standard - it will be used as the squash commit message and should start type(scope):.
  • All: Considered updating the online docs in the ./docs/ directory.
  • All: Set appropriate labels for the changes (only maintainers can apply labels).
  • Tests: Ran mkdocs serve locally and verified the auto-generated docs for new tests in the Test Case Reference are correctly formatted.
  • Tests: For PRs implementing a missed test case, update the post-mortem document to add an entry the list.
  • Ported Tests: All converted JSON/YML tests from ethereum/tests or tests/static have been assigned @ported_from marker.

Cute Animal Picture

Cute Animal Picture

felix314159 and others added 13 commits May 19, 2026 12:20
@felix314159
feat: devnet script optimization to avoid merge conflicts
d799234
@felix314159
feat: mario feedback
f35d365
@felix314159
refactor: 8037+7981 harmonization, add zero-valued access list token …
3be95b5 
…scaffolding
@felix314159
fix: add comment
6b523b7
@felix314159
feat(tests): adds EIP-7981 test and required framework changes (#2144)
60b3d3f 
* feat: adds 7981 src changes + tests

* fix: sam feedback

* fix: ethereum-spec-lint: 'the item .. has changed relative positions', do we really need this check

* feat: EIPs 11340 update implemented

* fix: sam feedback
@marioevz @felix314159
refactor(tests-eip-7981): Constrain tests to EIP inclusion
0e17de5
@felix314159
fix: harmonize with 8037, correct access list token placement and rem…
25a7eb3 
…ove docstring conflicts
@nerolation @felix314159
fix(specs,tests): EIP-7981 - Update implementation and specs (#2682)
91b751b 
* fix(specs,tests): EIP-7981 - Update implementation and specs

* fix: tooling+unused

* fix: ACCESS_LIST_ADDRESS_FLOOR_TOKENS and ACCESS_LIST_STORAGE_KEY_FLOOR_TOKENS are now populated

---------

Co-authored-by: Felix H <felix314159@users.noreply.github.com>
@felix314159
chores: update refspec
5895117
@spencer-tb @felix314159
fix(tests): drop GAS_ prefix from gas_costs attribute references
0c5a811 
Align the EIP-7981 mixin and tests with the GasCosts dataclass rename
on forks/amsterdam.
@felix314159
chore: update refspec (both 706296e06bf9b700bf046cfaa80bd5c3ebbc89c3 …
554c4f1 
…and 71d3353c3f6776485539fe2ec526b7cc80780bfc would not change the fixtures we currently output, cosmetic only)
@felix314159
chore: update refspec (cosmetic change, no action required)
ec63104
@chfast @felix314159
feat(tests): EIP-7981 zero-byte calldata floor cost rejection (#2869)
7d41e4e
@felix314159 felix314159 mentioned this pull request May 19, 2026
Open
8 tasks
@codecov

codecov Bot commented May 19, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.16%. Comparing base (852715c) to head (7d41e4e).

Additional details and impacted files 
@@               Coverage Diff                @@
##           forks/amsterdam    #2877   +/-   ##
================================================
  Coverage            87.16%   87.16%           
================================================
  Files                  586      586           
  Lines                35792    35798    +6     
  Branches              3364     3364           
================================================
+ Hits                 31199    31205    +6     
  Misses                3943     3943           
  Partials               650      650
Flag Coverage Δ
unittests 87.16% <100.00%> (+<0.01%) ⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

marioevz
marioevz approved these changes May 19, 2026
View reviewed changes

@marioevz marioevz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@marioevz marioevz merged commit db42a89 into forks/amsterdam May 19, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@marioevz marioevz marioevz approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@felix314159 @marioevz @nerolation @spencer-tb @chfast