-
Notifications
You must be signed in to change notification settings - Fork 263
DEVREL-1299 fix(devtools-move): bump initia.js and update deployer derivation #1928
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
base: main
Are you sure you want to change the base?
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 Adds unit tests covering the new Initia derivation/helpers and changes Written by Cursor Bugbot for commit fdb9cbf. This will update automatically on new commits. Configure here. |
Co-authored-by: beer-1 <147697694+beer-1@users.noreply.github.com>
Co-authored-by: beer-1 <147697694+beer-1@users.noreply.github.com>
Co-authored-by: beer-1 <147697694+beer-1@users.noreply.github.com>
Co-authored-by: beer-1 <147697694+beer-1@users.noreply.github.com>
Co-authored-by: beer-1 <147697694+beer-1@users.noreply.github.com>
ae5e04f to
20cdbc5
Compare
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Problem
Initia changed object creation seed derivation (v1.2.0+): it now includes a deployment count.
MoveVM change: initia-labs/movevm@8ad3db1
initiad fix: initia-labs/initia@d3557bd
To deploy new objects reliably, users need:
Additional Details
Solution
SHA3-256(address + 0x22 + "initia_std::object_code_deployment" + sequence(u64 LE) + deployCount(u64 LE) + 0xFE)
Other Changes
Verify-contract’s build failed in CI (and locally) because Rollup 4.40.0 now errors on a malformed sourcemap shipped in
@solidity-parser/parser@0.16.2.That map contains duplicatesrc/index.tsentries with conflicting sourcesContent, so Rollup throws “Multiple conflicting contents for sourcemap source”.Failed run: https://github.com/LayerZero-Labs/devtools/actions/runs/21777097176/job/62835345680#step:8:893
Fix: upgrade
@solidity-parser/parser in packages/verify-contractto^0.17.0, which ships a clean sourcemap; after reinstalling,pnpm -C packages/verify-contractrun build succeeds.Proof of Test
Deployments
Initia (Move)
0x928841F0A922D6BCA4C07FAD0F7EA73924B8A58BF980BE2822FACBC3F26591C6Base (EVM)
0xF9F9F83C77Eb39DfA100004db82cea9c0d9521c4Wiring
EVM → Move (Base → Initia), from
transactions/move.layerzero.config.ts/broadcast/latest.jsonMove → EVM (Initia → Base)
Cross-Chain Send Test
Base → Initia (OFT send)
Initia → Base (OFT send)