feat(src): EIP-7843 (SLOTNUM) Draft#2007

Merged

fselmo merged 2 commits intoethereum:eips/amsterdam/eip-7843from

felix314159:eip7843

Jan 21, 2026

Contributor

felix314159 commented Jan 12, 2026 •

edited

Loading

🗒️ Description

uv run fill tests/amsterdam/eip7843_slotnum/test_slotnum.py -v --clean --until=Amsterdam.
successfully tested against geth's bal-devnet-2 branch after applying these fixes. btw in hive/clients/go-ethereum i used the genesis that ethpandaops provided for bal-devnet-1

🔗 Related Issues or PRs

Solves #1876

✅ Checklist

All: Ran fast tox checks to avoid unnecessary CI fails, see also Code Standards and Enabling Pre-commit Checks:
```
uvx tox -e 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 adding an entry to CHANGELOG.md.
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

marioevz requested a review from danceratopz

January 12, 2026 21:59

marioevz assigned danceratopz

felix314159 force-pushed the eip7843 branch from da70bec to 6a5f173 Compare

January 19, 2026 09:54

felix314159 changed the base branch from eips/amsterdam/eip-7843 to forks/amsterdam

January 19, 2026 10:00

felix314159 changed the base branch from forks/amsterdam to eips/amsterdam/eip-7843

January 19, 2026 10:09

Contributor Author

felix314159 commented Jan 19, 2026

Rebased the remote eip7843 branch on forks/amsterdam (which now includes BAL) and then rebased this PR on it

felix314159 marked this pull request as ready for review

January 19, 2026 16:25

marioevz approved these changes

View reviewed changes

Member

marioevz left a comment

Looks fantastic! I added two small comments only. Thanks!

tests/amsterdam/eip7843_slotnum/test_slotnum.py Outdated Show resolved Hide resolved

tests/amsterdam/eip7843_slotnum/test_slotnum.py Outdated Show resolved Hide resolved

marioevz mentioned this pull request

chore(testing/vm): Remove UndefinedOpcodes #2044

Merged

7 tasks

marioevz added this to the bal devnet-2 milestone

felix314159 force-pushed the eip7843 branch from 8942980 to c693d2e Compare

January 20, 2026 10:46

felix314159 changed the base branch from eips/amsterdam/eip-7843 to forks/amsterdam

January 20, 2026 10:47

Contributor Author

felix314159 commented Jan 20, 2026

I rebased on top of forks/amsterdam. I think just like BAL there is value in merging this into forks/amsterdam instead of its own eip branch because it affects the block header. It is not SFI'd officially, but I think it could simplify our workflow when working on the other eips

jsign pushed a commit to jsign/execution-specs that referenced this pull request


          feat(consume): Add consume sync tests to sync clients after engine …

…tests (ethereum#2007)

jsign pushed a commit to jsign/execution-specs that referenced this pull request


          feat(consume): Add consume sync tests to sync clients after engine …

bbe459a

…tests (ethereum#2007)

jsign pushed a commit to jsign/execution-specs that referenced this pull request


          feat(consume): Add consume sync tests to sync clients after engine …

7527fc5

…tests (ethereum#2007)

felix314159 changed the base branch from forks/amsterdam to eips/amsterdam/eip-7843

January 20, 2026 15:13

Contributor Author

felix314159 commented Jan 20, 2026

Ok the target EIP branch is now also rebased on forks/amsterdam

spencer-tb reviewed

View reviewed changes

Contributor

spencer-tb left a comment

Some small comments.

Quick tests to include in another PR:

Fork transitions tests, verify invalid pre fork, valid on fork block and after.
Call context, verify slot_number is consistent in nested call.
Initcode context, verify slot_number in constructor via direct tx and via CREATE/CREATE2.

tests/amsterdam/eip7843_slotnum/spec.py Show resolved Hide resolved

packages/testing/src/execution_testing/vm/opcodes.py Show resolved Hide resolved

src/ethereum/forks/amsterdam/vm/instructions/block.py Show resolved Hide resolved

spencer-tb mentioned this pull request

STEEL Weekly Meeting, Wed 21st Jan ethsteel/pm#39

Closed

felix314159 added 2 commits

January 21, 2026 10:04


          feat(amsterdam): Implement EIP-7843 SLOTNUM opcode

449e320


          mario feedback

77e5ffa

fselmo force-pushed the eip7843 branch from c693d2e to 77e5ffa Compare

January 21, 2026 17:36

fselmo approved these changes

View reviewed changes

Contributor

fselmo left a comment

I rebased this and it lgtm 👍🏼. I'm going to address a failing test for 7934 and Spencer's comments in a followup PR.

fselmo merged commit 8682b5d into ethereum:eips/amsterdam/eip-7843

21 of 22 checks passed

fselmo added a commit to fselmo/execution-specs that referenced this pull request


          fix(tests): minor updates to eip-7843 following ethereum#2007

f6df50b

This was referenced Jan 21, 2026

Make EIP-7934 tests fork agnostic #1972

Open

fix(tests): minor fixes and updates for eip7843 #2057

Merged

fselmo added a commit that referenced this pull request


          fix(tests): minor fixes and updates for eip7843 (#2057)

8e344e4

* fix(tests): minor updates to eip-7843 following #2007

* fix: account for slotnum in genesis creation

* fix: append slot_number in fixture_loader.py

github-actions bot pushed a commit that referenced this pull request


          feat(src): Initial EIP-7843 (SLOTNUM) Implementation (#2007)

b995d34

* feat(amsterdam): Implement EIP-7843 SLOTNUM opcode

* mario feedback

github-actions bot pushed a commit that referenced this pull request


          fix(tests): minor fixes and updates for eip7843 (#2057)

d857334

* fix(tests): minor updates to eip-7843 following #2007

* fix: account for slotnum in genesis creation

* fix: append slot_number in fixture_loader.py

fselmo pushed a commit that referenced this pull request


          feat(src): Initial EIP-7843 (SLOTNUM) Implementation (#2007)

df36fb7

* feat(amsterdam): Implement EIP-7843 SLOTNUM opcode

* mario feedback

fselmo added a commit that referenced this pull request


          fix(tests): minor fixes and updates for eip7843 (#2057)

88515e6

* fix(tests): minor updates to eip-7843 following #2007

* fix: account for slotnum in genesis creation

* fix: append slot_number in fixture_loader.py

github-actions bot pushed a commit that referenced this pull request


          feat(src): Initial EIP-7843 (SLOTNUM) Implementation (#2007)

2aa7f25

* feat(amsterdam): Implement EIP-7843 SLOTNUM opcode

* mario feedback

github-actions bot pushed a commit that referenced this pull request


          fix(tests): minor fixes and updates for eip7843 (#2057)

726bb57

* fix(tests): minor updates to eip-7843 following #2007

* fix: account for slotnum in genesis creation

* fix: append slot_number in fixture_loader.py

github-actions bot pushed a commit that referenced this pull request


          feat(src): Initial EIP-7843 (SLOTNUM) Implementation (#2007)

2a53c94

* feat(amsterdam): Implement EIP-7843 SLOTNUM opcode

* mario feedback

github-actions bot pushed a commit that referenced this pull request


          fix(tests): minor fixes and updates for eip7843 (#2057)

a64c42a

* fix(tests): minor updates to eip-7843 following #2007

* fix: account for slotnum in genesis creation

* fix: append slot_number in fixture_loader.py

marioevz pushed a commit that referenced this pull request


          feat(src): Initial EIP-7843 (SLOTNUM) Implementation (#2007)

2f7daf1

* feat(amsterdam): Implement EIP-7843 SLOTNUM opcode

* mario feedback

marioevz pushed a commit that referenced this pull request


          fix(tests): minor fixes and updates for eip7843 (#2057)

7a58fab

* fix(tests): minor updates to eip-7843 following #2007

* fix: account for slotnum in genesis creation

* fix: append slot_number in fixture_loader.py

fselmo pushed a commit that referenced this pull request


          feat(src): Initial EIP-7843 (SLOTNUM) Implementation (#2007)

9d8463c

* feat(amsterdam): Implement EIP-7843 SLOTNUM opcode

* mario feedback

fselmo added a commit that referenced this pull request


          fix(tests): minor fixes and updates for eip7843 (#2057)

8978c6d

* fix(tests): minor updates to eip-7843 following #2007

* fix: account for slotnum in genesis creation

* fix: append slot_number in fixture_loader.py

spencer-tb pushed a commit that referenced this pull request


          feat(src): Initial EIP-7843 (SLOTNUM) Implementation (#2007)

44689bd

* feat(amsterdam): Implement EIP-7843 SLOTNUM opcode

* mario feedback

spencer-tb pushed a commit that referenced this pull request


          fix(tests): minor fixes and updates for eip7843 (#2057)

2e9b948

* fix(tests): minor updates to eip-7843 following #2007

* fix: account for slotnum in genesis creation

* fix: append slot_number in fixture_loader.py

fselmo pushed a commit to fselmo/execution-specs that referenced this pull request


          feat(src): Initial EIP-7843 (SLOTNUM) Implementation (ethereum#2007)

cff4b8d

* feat(amsterdam): Implement EIP-7843 SLOTNUM opcode

* mario feedback

fselmo added a commit to fselmo/execution-specs that referenced this pull request


          fix(tests): minor fixes and updates for eip7843 (ethereum#2057)

b0298ae

* fix(tests): minor updates to eip-7843 following ethereum#2007

* fix: account for slotnum in genesis creation

* fix: append slot_number in fixture_loader.py

fselmo pushed a commit that referenced this pull request


          feat(src): Initial EIP-7843 (SLOTNUM) Implementation (#2007)

1e811bb

* feat(amsterdam): Implement EIP-7843 SLOTNUM opcode

* mario feedback

fselmo added a commit that referenced this pull request


          fix(tests): minor fixes and updates for eip7843 (#2057)

466be96

* fix(tests): minor updates to eip-7843 following #2007

* fix: account for slotnum in genesis creation

* fix: append slot_number in fixture_loader.py

fselmo pushed a commit that referenced this pull request


          feat(src): Initial EIP-7843 (SLOTNUM) Implementation (#2007)

3d6d93a

* feat(amsterdam): Implement EIP-7843 SLOTNUM opcode

* mario feedback

fselmo added a commit that referenced this pull request


          fix(tests): minor fixes and updates for eip7843 (#2057)

2f37c31

* fix(tests): minor updates to eip-7843 following #2007

* fix: account for slotnum in genesis creation

* fix: append slot_number in fixture_loader.py

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet