Skip to content

apollo_starknet_os_program: add qm31 blake#14121

Open
einat-starkware wants to merge 1 commit into
mainfrom
einat/privacy/qm31-blake
Open

apollo_starknet_os_program: add qm31 blake#14121
einat-starkware wants to merge 1 commit into
mainfrom
einat/privacy/qm31-blake

Conversation

@einat-starkware
Copy link
Copy Markdown
Contributor

No description provided.

@reviewable-StarkWare
Copy link
Copy Markdown

This change is Reviewable

Copy link
Copy Markdown
Contributor Author

einat-starkware commented May 20, 2026

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 20, 2026

Artifacts upload workflows:

@einat-starkware einat-starkware marked this pull request as ready for review May 20, 2026 14:24
@cursor
Copy link
Copy Markdown

cursor Bot commented May 20, 2026

PR Summary

Medium Risk
Introduces new Cairo0 hashing logic and cross-language parity tests; risk is mainly around subtle serialization/mod-reduction mismatches that could break downstream verifier assumptions. Changes are otherwise isolated to test-program compilation and test coverage.

Overview
Adds a new Cairo0 test program qm31_blake that computes a Blake2s-256 digest over QM31 inputs (serialized as 4 LE u32 limbs per element) and reduces the 8 digest words modulo 2^31-1.

Wires this program into the test-program build (compile_program.rs + QM31_BLAKE_BYTES) and adds a Rust reference implementation plus a parity test in blake2s_test.rs to assert Cairo output matches the expected QM31 hashing semantics across several cases.

Reviewed by Cursor Bugbot for commit 143bc30. Bugbot is set up for automated code reviews on this repo. Configure here.

@einat-starkware einat-starkware force-pushed the einat/privacy/qm31-blake branch from aaa90c8 to 143bc30 Compare May 20, 2026 18:47
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.

2 participants