-
Notifications
You must be signed in to change notification settings - Fork 123
feat(agglayer): store and use metadata hash for bridge-out leaves #2583
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
+627
−169
Merged
Changes from 4 commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
a05e38a
feat(agglayer): store and use metadata hash for bridge-out leaves
claude c416907
refactor: address self-review feedback on metadata hash
claude d6d9127
chore: allow CC0-1.0 license in cargo-deny for tiny-keccak
claude 5c717fd
chore: add changelog entry for metadata hash feature
claude 5a2bbd8
refactor: replace tiny-keccak with miden-crypto and alloy-sol-types f…
claude da64d05
refactor: replace truncate_stack with explicit drops in get_metadata_…
claude c57ca64
docs: simplify TODO comment and compute METADATA_HASH inline in Solid…
claude 57fd363
chore: consolidate tests
mmagician 6eb0e62
chore: address Copilot review comments
claude 0623afa
Update crates/miden-agglayer/asm/agglayer/faucet/mod.masm
mmagician 5b902f0
fix: explicit pad before FPI call in metadata hash retrieval
claude 6df58ee
fix: explicit pad and reorder FPI setup for metadata hash retrieval
claude 696b0b2
chore: explicit padding comments; load proc-ref before faucet ID
mmagician a130feb
chore: rearrange mem writing sections
mmagician f55b3d9
merge: integrate agglayer branch (storage helpers refactor)
claude 24dba22
chore: no need to store faucet ID to mem
mmagician de38d2f
Apply suggestion from @mmagician
mmagician 7318011
Update deny.toml
mmagician 2347d58
merge agglayer branch
mmagician 8e7380d
refactor: encode token metadata in Solidity test vectors, read from J…
claude 8e38a5e
Merge branch 'agglayer' into mmagician-claude/agglayer-metadata-hash
mmagician ac582b3
fix: add (4) annotations to word-sized stack comments in bridge_out
claude 1f43497
Merge remote-tracking branch 'origin/agglayer' into mmagician-claude/…
claude 3b7587d
Revert "fix: add (4) annotations to word-sized stack comments in brid…
claude 09079f1
chore: clean up stack comments for storing leaf data
mmagician File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.