Conversation
🧪 E2E Test StatusE2E tests are non-blocking and validate real blockchain interactions. Failures may occur due to network issues, RPC rate limits, or external service downtime. Test Runs (Newest First): |
PR SummaryMedium Risk Overview
Written by Cursor Bugbot for commit cbc787d. This will update automatically on new commits. Configure here. |
|
bump @lzJxhn |
Co-authored-by: nazreen.mohamad <nazreen.mohamad@layerzerolabs.org>
St0rmBr3w
left a comment
There was a problem hiding this comment.
debug.ts and index.ts changes look good. The "Use Pathway Defaults" label for 0 confirmations is a nice touch, and silencing the warning when the deployment file doesn't exist yet makes the create flow much cleaner. Changesets are appropriate.
On updateMetadata: the auto-detection logic (SPL Token -> Metaplex, Token-2022 -> probe for Metaplex PDA first, fall back to Token Extensions) is solid. The early hasFieldUpdates check prevents the rent-only footgun that bugbot flagged, and keypair resolution goes through getSolanaKeypair() as expected.
Looks good to merge.
Solana OFT debug script: state that value
0for confirmations means 'use default'Closes DEVREL-1238
When running:
Before:
After:
Solana OFT - updateMetadata script supports Token-2022 Metadata Update
Closes DEVREL-1239
Problem
lz:oft:solana:update-metadataonly supported Metaplex metadata, not Token-2022.Solution
updateTokenExtensionsMetadata()with authority validation and rent handling for multisig.Changes
updateMetadata.ts: Auto-detect and support Token-2022 metadata with authority checks and rent top-up.Example Run