Replace hardcoded fork paths with `Hardfork` class

[SamWilsn]

There are a few locations that manually import or otherwise reference hard forks:

Non-exhaustive location list.

• execution-specs/src/ethereum_optimized/__init__.py

  Line 38 in cdbe21f

  slow_state = cast(Any, import_module("ethereum." + fork_name + ".state"))

• execution-specs/src/ethereum_optimized/__init__.py

  Line 57 in cdbe21f

  slow_spec = import_module("ethereum." + fork_name + ".fork")

• execution-specs/src/ethereum_optimized/fork.py

  Line 44 in cdbe21f

  mod = cast(Any, import_module("ethereum." + _fork_name + ".fork"))

• execution-specs/src/ethereum_optimized/state_db.py

  Lines 60 to 61 in cdbe21f

  	types_mod = cast(Any, import_module("ethereum." + fork + ".fork_types"))
  	state_mod = cast(Any, import_module("ethereum." + fork + ".state"))

• execution-specs/src/ethereum_spec_tools/evm_tools/loaders/fork_loader.py

  Line 30 in cdbe21f

  return importlib.import_module(f"ethereum.{self._fork_module}.{name}")

• execution-specs/src/ethereum_spec_tools/evm_tools/loaders/fork_loader.py

  Line 36 in cdbe21f

  if fork.name == "ethereum." + self._fork_module:

• execution-specs/src/ethereum_spec_tools/evm_tools/loaders/fork_loader.py

  Line 46 in cdbe21f

  if fork.name == "ethereum." + self._fork_module:

• execution-specs/src/ethereum_spec_tools/evm_tools/t8n/__init__.py

  Line 177 in cdbe21f

  if self.fork.is_after_fork("ethereum.london"):

• execution-specs/src/ethereum_spec_tools/evm_tools/t8n/__init__.py

  Line 185 in cdbe21f

  if self.fork.is_after_fork("ethereum.cancun"):

• execution-specs/src/ethereum_spec_tools/evm_tools/t8n/__init__.py

  Line 262 in cdbe21f

  if self.fork.is_after_fork("ethereum.cancun"):

• execution-specs/src/ethereum_spec_tools/evm_tools/t8n/__init__.py

  Line 282 in cdbe21f

  if not self.fork.is_after_fork("ethereum.paris"):

• execution-specs/src/ethereum_spec_tools/evm_tools/t8n/__init__.py

  Line 290 in cdbe21f

  if self.fork.is_after_fork("ethereum.shanghai"):

• execution-specs/src/ethereum_spec_tools/evm_tools/t8n/__init__.py

  Line 295 in cdbe21f

  if self.fork.is_after_fork("ethereum.prague"):

• execution-specs/src/ethereum_spec_tools/evm_tools/t8n/env.py

  Line 79 in cdbe21f

  if t8n.fork.is_after_fork("ethereum.cancun"):

• execution-specs/src/ethereum_spec_tools/evm_tools/t8n/env.py

  Line 98 in cdbe21f

  if not t8n.fork.is_after_fork("ethereum.cancun"):

• execution-specs/src/ethereum_spec_tools/evm_tools/t8n/env.py

  Line 133 in cdbe21f

  if t8n.fork.is_after_fork("ethereum.osaka"):

• execution-specs/src/ethereum_spec_tools/evm_tools/t8n/env.py

  Line 172 in cdbe21f

  if t8n.fork.is_after_fork("ethereum.london"):

• execution-specs/src/ethereum_spec_tools/evm_tools/t8n/env.py

  Line 214 in cdbe21f

  if t8n.fork.is_after_fork("ethereum.paris"):

• execution-specs/src/ethereum_spec_tools/evm_tools/t8n/env.py

  Line 235 in cdbe21f

  if t8n.fork.is_after_fork("ethereum.shanghai"):

• execution-specs/src/ethereum_spec_tools/evm_tools/t8n/env.py

  Line 250 in cdbe21f

  if t8n.fork.is_after_fork("ethereum.paris"):

• execution-specs/src/ethereum_spec_tools/evm_tools/t8n/env.py

  Line 269 in cdbe21f

  if t8n.fork.is_after_fork("ethereum.byzantium"):

• execution-specs/src/ethereum_spec_tools/evm_tools/t8n/env.py

  Line 289 in cdbe21f

  t8n.fork.is_after_fork("ethereum.prague")

• execution-specs/src/ethereum_spec_tools/evm_tools/t8n/t8n_types.py

  Line 140 in cdbe21f

  if t8n.fork.is_after_fork("ethereum.berlin"):

• execution-specs/src/ethereum_spec_tools/evm_tools/t8n/t8n_types.py

  Line 183 in cdbe21f

  if t8n.fork.is_after_fork("ethereum.berlin"):

• execution-specs/src/ethereum_spec_tools/evm_tools/t8n/t8n_types.py

  Line 208 in cdbe21f

  if t8n.fork.is_after_fork("ethereum.berlin"):

• execution-specs/src/ethereum_spec_tools/evm_tools/t8n/t8n_types.py

  Line 215 in cdbe21f

  if t8n.fork.is_after_fork("ethereum.spurious_dragon"):

These should be converted to use Hardfork instead. Finding fork packages is non-trivial, so we should centralize that logic in one place.

[Peponks9]

hi @SamWilsn would like to contribute to it :)

[omegajudith]

Hi @SamWilsn, I’d like to take on this issue. I’ll start by replacing the hardcoded fork references with the Hardfork class where applicable, and then open a PR. Please let me know if anyone is already working on it. Thanks!

[SamWilsn]

We're happy to accept outside contributions. No one on our team is working on this yet.

[omegajudith]

Thank you so much @SamWilsn

[Anuoluwapo25]

Hi @SamWilsn is this issue still open for contribution, i would like to contribute to this. Thanks

[omegajudith]

yes iam working on it

[Anuoluwapo25]

Alright thank you for your swift response

…

On Wed, Oct 8, 2025 at 2:40 PM judith Omega ***@***.***> wrote: *omegajudith* left a comment (ethereum/execution-specs#1426) <#1426 (comment)> yes iam working on it — Reply to this email directly, view it on GitHub <#1426 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/A55OV7QXUXC7G4QVZMTALAL3WUH3TAVCNFSM6AAAAACGSITTM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTGOBRGYYDANRXGQ> . You are receiving this because you commented.Message ID: ***@***.***>

[bomanaps]

Hello @SamWilsn I went over the implementation steps in regards to this with @omegajudith, so this is my thoughts - Add a helper method to Hardfork class (src/ethereum_spec_tools/forks.py):
Add get_by_short_name(name) method to find a fork by name like "frontier"
Fix 3 main locations where fork paths are hardcoded:
src/ethereum_optimized/ (3 files) - change to use Hardfork.get_by_short_name() then fork.module()
src/ethereum_spec_tools/evm_tools/loaders/fork_loader.py - already imports Hardfork, just use it
tests/json_infra/ (5 test files) - update to use Hardfork
please can you confirm this is the right approach if it's not please clarify?

[SamWilsn]

Yep, that sounds exactly right! I'd go with something like:

@classmethod
def by_short_name(cls: Type[H], name: str) -> H:
    ...