Agglayer bridging tracking PR #2107
Draft
Conversation
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
Update `agglayer` branch to `next`
…erBridgeOut` contract -> `BURN` note (#2023) * feat: stubbed out B2AGG note & bridge out functionality * feat: create BURN note during B2AGG consumption * feat: stub out mmr_frontier for bridgeout * refactor: add padding comments to B2AGG note * refactor: cleanup B2AGG notescript implementation * Update crates/miden-lib/asm/agglayer/note_scripts/B2AGG.masm Co-authored-by: Copilot <[email protected]> * refactor: update B2AGG note doc comments * Update crates/miden-lib/asm/agglayer/account_components/bridge_out.masm Co-authored-by: Copilot <[email protected]> * Update crates/miden-testing/tests/agglayer/bridge_out.rs Co-authored-by: Copilot <[email protected]> * refactor: clean up test logic * feat: implement ntx faucet tag in masm & compute BURN serial_num * refactor: make BURN note inputs a constant * fix: fix docs CI * feat: hash b2agg note details using keccak256 * feat: add comments to bridge out component procedures * feat: add B2AGG reclaim test * fix: clippy warning * Update crates/miden-lib/asm/agglayer/account_components/bridge_out.masm Co-authored-by: Copilot <[email protected]> * Update crates/miden-lib/asm/agglayer/account_components/bridge_out.masm Co-authored-by: Copilot <[email protected]> * Update crates/miden-lib/asm/agglayer/account_components/bridge_out.masm Co-authored-by: Copilot <[email protected]> * Revert "Update crates/miden-lib/asm/agglayer/account_components/bridge_out.masm" This reverts commit 733883d. * refactor: cleanup build.rs updates * refactor: remove added comment from build.rs * feat: use build_note_tag_for_network_account in bridge_out component * refactor: update procedure comment * Update crates/miden-lib/asm/agglayer/account_components/bridge_out.masm Co-authored-by: Marti <[email protected]> * Update crates/miden-lib/asm/agglayer/account_components/bridge_out.masm Co-authored-by: Marti <[email protected]> * Update crates/miden-lib/asm/agglayer/account_components/bridge_out.masm Co-authored-by: Marti <[email protected]> * Update crates/miden-lib/asm/agglayer/account_components/bridge_out.masm Co-authored-by: Marti <[email protected]> * refactor: rm convert_b2agg_to_u32 & use store_word_u32s_le * fix: spelling in masm comment * fix: rm debug mode from build.rs * refactor: rm unused imports in bridge_out * fix: update doc comment * refactor: import std::word * refactor: use swapw instead of padw in B2AGG * refactor: rename B2AGG_NOTE_INPUTS_NUMBER to B2AGG_NOTE_INPUTS_COUNT * refactor: address stack comments & optimize cycles * refactor: add B2AGG input comments * refactor: update B2AGG input comments & stack layout * refactor: fix stack state before note::build_note_tag_for_network_account * refactor: modularize agglayer bridge * fix: rustfmt * refactor: add add_asset_message procedure * refactor: use different stack comment notation for B2AGG inputs * refactor: modify B2AGG description doc comment * Apply suggestions from code review * Update crates/miden-lib/asm/agglayer/account_components/bridge_out.masm Co-authored-by: Marti <[email protected]> * refactor: add todo comments in masm --------- Co-authored-by: Copilot <[email protected]> Co-authored-by: Marti <[email protected]>
Co-authored-by: Andrey Khmuro <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.