refactor: represent slot intervals with enum

[dicethedev]

🗒️ Description / Motivation

• Replaces raw 0..4 interval dispatch in blockchain tick handling with a typed SlotInterval enum.
• This makes interval duties self-documenting and lets Rust enforce exhaustive handling when intervals change.
• Closes the gap where interval-specific behavior was matched using integer literals.

What Changed

• crates/blockchain/src/lib.rs

  • Added SlotInterval with from_slot_index as the integer slot-to-enum constructor.
  • Updated BlockChainServer::on_tick to compare and match on SlotInterval variants.
  • Kept interval logging as numeric output through Display.

• crates/blockchain/src/store.rs

  • Reused SlotInterval for the store-level tick interval match.

Correctness / Behavior Guarantees

• No behavior change intended.
• The same interval indices map to the same duties:
  • 0 → block publication
  • 1 → attestation production
  • 2 → aggregation
  • 3 → safe target update
  • 4 → end of slot
• Interval matches are now exhaustive, so adding/removing interval variants requires updating all dispatch sites at compile time.

Tests Added / Run

• No tests added; this is a refactor with no intended behavior change.
• Ran:
  • cargo fmt
  • cargo check -p ethlambda-blockchain

Related Issues / PRs

• Closes Use an enum to represent the different intervals in BlockChainServer::on_tick #469
• Related to #

✅ Verification Checklist

• Ran make fmt — clean
• Ran make lint (clippy with -D warnings) — clean
• Ran cargo test --workspace --release — all passing

[greptile-apps]

No reviewable files after applying ignore patterns.

[MegaRedHand]

Left some comments. The direction is good, but there are some improvements I think are necessary before we merge this

[dicethedev]

Left some comments. The direction is good, but there are some improvements I think are necessary before we merge this

Alright Mega. I will push a fix a few minutes with your suggestions.

[MegaRedHand]

Good stuff!