more withState state -> forkyState

[tersec]

Followup to #4026 (explanation), #4038 (first round of updates), and #4104 (second round of updates).

This completes the test suite, state_sim, block_sim, and nimbus_beacon_node in pure compilation-or-not terms, but to finish removing the state accessor, it needs to be audited for any remaining withState(state): foo(state) cases where they'll compile but not run correctly. Those work fine now, still, because the accessor template is still there.

Furthermore, some of the other make targets such as ncli_db will require a few more changes, with again an audit for the withState(state): foo(state) pattern in some contexts where it compiles successfully but to incorrect code.

This PR, on its own, is safe from such issues.

[github-actions]

Unit Test Results

       12 files  ±0       860 suites  ±0   59m 47s ⏱️ - 10m 46s
  1 982 tests ±0    1 835 ✔️ ±0  147 💤 ±0  0 ❌ ±0 
10 662 runs  ±0  10 472 ✔️ ±0  190 💤 ±0  0 ❌ ±0 

Results for commit 78f1281. ± Comparison against base commit 8be964a.