Skip to content

correct keyGen expansion logic and fix extended test failures#99

Open
chetanyb wants to merge 1 commit intomasterfrom
test-fix
Open

correct keyGen expansion logic and fix extended test failures#99
chetanyb wants to merge 1 commit intomasterfrom
test-fix

Conversation

@chetanyb
Copy link
Contributor

@chetanyb chetanyb commented Feb 4, 2026

Summary

  • Fix keyGen expansion logic to match Rust (remove incorrect * 128 multiplier)
  • Remove faulty TopTreeStartIndexMismatch check that broke signing after advancePreparation
  • Add missing neg method, fix SSZ test parameters, fix API calls in extended tests

Test plan

  • All 29 core tests pass (zig build test)
  • All 10 extended tests pass (zig build test-extended)
  • All 4 lifetime tests pass (zig build test-lifetimes)

@chetanyb chetanyb requested a review from ch4r10t33r February 4, 2026 17:37
@chetanyb chetanyb changed the title fix: correct keyGen expansion logic and fix extended test failures correct keyGen expansion logic and fix extended test failures Feb 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant