Skip to content

Commit 2c08f8b

Browse files
authored
feat: add proposer proof aggregation (#284)
This PR adds true attestation aggregation on the proposer block building pipeline. As part of this, the attestation byte cap we added was removed since we now have the 16 attestation+proof cap.
1 parent 1a4bf9d commit 2c08f8b

File tree

5 files changed

+476
-146
lines changed

5 files changed

+476
-146
lines changed

CLAUDE.md

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -338,10 +338,9 @@ cargo test -p ethlambda-blockchain --test forkchoice_spectests -- --test-threads
338338
## Common Gotchas
339339

340340
### Aggregator Flag Required for Finalization
341-
- At least one node **must** be started with `--is-aggregator` to finalize blocks in production (without `skip-signature-verification`)
341+
- At least one node **must** be started with `--is-aggregator` to finalize blocks
342342
- Without this flag, attestations pass signature verification and are logged as "Attestation processed", but the signature is never stored for aggregation (`store.rs:368`), so blocks are always built with `attestation_count=0`
343343
- The attestation pipeline: gossip → verify signature → store gossip signature (only if `is_aggregator`) → aggregate at interval 2 → promote to known → pack into blocks
344-
- With `skip-signature-verification` (tests only), attestations bypass aggregation and go directly to `new_aggregated_payloads`, so the flag is not needed
345344
- **Symptom**: `justified_slot=0` and `finalized_slot=0` indefinitely despite healthy block production and attestation gossip
346345

347346
### Signature Verification

0 commit comments

Comments
 (0)