refactor(testing): drop duplicate abstract fork name declaration

[tcoratger]

Motivation

The fork name() was declared abstract twice in forks/base.py:

• on the metaclass (BaseForkMeta), so its own __repr__ could call cls.name(), and
• as the @classmethod @abstractmethod on BaseFork that concrete forks override.

The metaclass declaration was not dead, though — removing it outright makes the type checker fail on cls.name() inside __repr__ (Self has no attribute name). It existed only so the metaclass methods could resolve name.

What this does

Types the metaclass methods' cls parameter as type[BaseFork] (which it always is — BaseForkMeta is only ever BaseFork's metaclass). The __repr__ and __le__ methods then resolve name through the single classmethod, so the metaclass-level abstract declaration can be dropped.

The @classmethod @abstractmethod name() on BaseFork remains the one source of truth and still enforces that every concrete fork implements it.

Behavior preserved

Lstar.name() = Lstar
repr(Lstar)  = Lstar
Lstar <= Lstar = True
BaseFork()            -> abstract (cannot instantiate)
class F(BaseFork): pass; F()  -> abstract (name still required)

Testing

• just check passes (lint, format, ty, codespell, mdformat).
• Runtime checks confirm name, repr, ordering, and ABC enforcement are unchanged.
• Fill smoke: the emitted network field is still "Lstar".

🤖 Generated with Claude Code