Uh oh!
There was an error while loading. Please reload this page.
fix(drive): import nullifier constants instead of duplicating them - #3202
Conversation
Replace locally-defined NULLIFIERS_KEY_U8 and COMPACTED_NULLIFIERS_KEY_U8 with imports from saved_block_transactions::queries to avoid divergence. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Uh oh!
There was an error while loading. Please reload this page.
Issue being fixed or feature implemented
NULLIFIERS_KEY_U8andCOMPACTED_NULLIFIERS_KEY_U8were defined locally in verify modules instead of importing fromsaved_block_transactions::queries. If the canonical value changes, these local copies would silently diverge, breaking proof verification.What was done?
Replaced local constant definitions with imports from
crate::drive::saved_block_transactions.How Has This Been Tested?
cargo check -p drive— compiles cleanlyBreaking Changes
None.
Checklist: