Skip to content

lazer/iota: Mainnet deploy#3921

Open
matej-douro wants to merge 4 commits into
matej/lazer-iota-sdkfrom
matej/iota-lazer-mainnet-deploy
Open

lazer/iota: Mainnet deploy#3921
matej-douro wants to merge 4 commits into
matej/lazer-iota-sdkfrom
matej/iota-lazer-mainnet-deploy

Conversation

@matej-douro

@matej-douro matej-douro commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary

Rationale

How has this been tested?

  • Current tests cover my changes
  • Added new tests
  • Manually tested the code

Open in Devin Review

@vercel

vercel Bot commented Jul 22, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
api-reference Ready Ready Preview, Comment Jul 22, 2026 12:30pm
component-library Ready Ready Preview, Comment Jul 22, 2026 12:30pm
developer-hub Ready Ready Preview, Comment Jul 22, 2026 12:30pm
4 Skipped Deployments
Project Deployment Actions Updated (UTC)
entropy-explorer Skipped Skipped Jul 22, 2026 12:30pm
insights Skipped Skipped Jul 22, 2026 12:30pm
proposals Skipped Skipped Jul 22, 2026 12:30pm
staking Skipped Skipped Jul 22, 2026 12:30pm

Request Review

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 1 potential issue.

Open in Devin Review

Comment on lines +17 to 21
{
"chain": "iota_mainnet",
"stateId": "0x7fa7eb76e4c840102d2a57589b658c44ac506847f972c6fdacdf824cb8f53cb4",
"type": "IotaWormholeContract"
}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔍 Second iota_mainnet wormhole entry does not collide with existing one

The PR adds a second iota_mainnet IotaWormholeContract entry (contract_manager/src/store/contracts/IotaWormholeContracts.json:17-21) alongside the existing one. The store keys contracts by getId() which combines chain and stateId (contract_manager/src/core/contracts/iota.ts:411-413), so the two entries have distinct IDs and won't trigger the Multiple contracts with id error in contract_manager/src/node/utils/store.ts:246-251. This mirrors the pre-existing pattern where iota_testnet already has two entries. No collision, but reviewers should confirm that consumers looking up a wormhole contract by chain (rather than by full id) handle multiple iota_mainnet entries as intended.

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant