## Motivation The [leanSpec](https://github.com/leanEthereum/leanSpec) reference implementation renamed the XMSS signing/verification parameter from `epoch` to `slot` ([commit `3b9d054`](leanEthereum/leanSpec@3b9d054), part of [PR #412](leanEthereum/leanSpec#412)). In ethlambda, the actual value passed to this parameter is always a **slot number** (e.g., `attestation.data.slot as u32`), so the `epoch` naming was misleading — in Ethereum consensus, an "epoch" typically means 32 slots, but leanConsensus has no epoch concept at all. The XMSS scheme uses this parameter as a one-time signature index to prevent key reuse, and leanConsensus uses the slot number for this purpose. This is part of the post-devnet-3 alignment work tracked in #155 (pq-devnet-4 preparation). ## Description Pure parameter rename: `epoch` → `slot` across production code, documentation, and tests. **No behavioral or wire-format change** — the `leansig` crate uses positional arguments, so the external dependency is unaffected. ### Changes by file | File | Scope | Changes | |------|-------|---------| | `crates/common/types/src/signature.rs` | `is_valid()`, `sign()` | Parameter + doc rename | | `crates/common/crypto/src/lib.rs` | `aggregate_signatures()`, `verify_aggregated_signature()` | Parameter + doc rename | | `crates/common/crypto/src/lib.rs` | Unit tests + cross-client ream tests | Variable, constant (`REAM_EPOCH` → `REAM_SLOT`), test name, comment, and assert message renames | | `crates/blockchain/src/key_manager.rs` | `sign_attestation()`, `sign_message()` | Parameter + doc + local variable rename | | `crates/blockchain/src/store.rs` | `verify_signature()`, `verify_aggregated_attestation()`, block verification | Local variable rename at 4 call sites | ### What was NOT renamed - **`activation_epoch`** variables in test helpers — these are parameters to `generate_keypair_and_sign()` for leansig keypair setup, not part of our signing/verification API. - **`leansig` internals** — the external crate's parameter names are unchanged; we only pass our renamed variable positionally. ## How to test ```bash cargo check --workspace # Compiles clean cargo clippy --workspace -- -D warnings # No warnings cargo test --workspace --release # All 92 tests pass (5 pre-existing ignores) ``` ## Related issues - Part of #155 (pq-devnet-4 preparation, section 1.2) - Companion PR: #183 (rename `SignedAttestation.message` → `.data`) - leanSpec reference: [commit `3b9d054`](leanEthereum/leanSpec@3b9d054) --------- Co-authored-by: Tomás Grüner <47506558+MegaRedHand@users.noreply.github.com>

Add support for pq-devnet-4 #155

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions

Add support for `pq-devnet-4` #155