Skip to content

feat(tempo): add Zone E contracts#4891

Merged
jxom merged 5 commits into
mainfrom
jxom/add-zone-e
Jul 23, 2026
Merged

feat(tempo): add Zone E contracts#4891
jxom merged 5 commits into
mainfrom
jxom/add-zone-e

Conversation

@jxom

@jxom jxom commented Jul 23, 2026

Copy link
Copy Markdown
Member

Adds Zone E metadata and RPC configuration, with Portal and Messenger addresses exposed through shared registries and derived into the chain contract configuration for downstream Zone clients.

@changeset-bot

changeset-bot Bot commented Jul 23, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: fefc9a2

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
viem Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel

vercel Bot commented Jul 23, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
viem Ready Ready Preview, Comment Jul 23, 2026 2:12am

Request Review

@pkg-pr-new

pkg-pr-new Bot commented Jul 23, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/viem@4891

commit: fefc9a2

@jxom
jxom marked this pull request as ready for review July 23, 2026 01:37

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7ef15fe7cc

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/tempo/zones/zone.ts Outdated
Comment on lines +37 to +38
messenger: {
address: Addresses.messenger[tempoModerato.id][1],

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Key Messenger by its source chain

Zone E’s Messenger is a source-chain contract just like the Portal, but this exposes it as a direct contracts.messenger.address. Consumers following the chain-contract convention will look for direct contracts on the zone chain/RPC, while source-chain contracts are keyed by the parent chain ID (as portal is below and as the local Zone genesis provisions Messenger alongside Portal/Factory on the parent Tempo chain). Keep Messenger keyed under tempoModerato.id so callers can resolve and call it on the source chain.

Useful? React with 👍 / 👎.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2b8e4c75e6

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@@ -0,0 +1,5 @@
---
'viem': patch

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Mark portalAddresses removal as major

This changeset explicitly documents a breaking removal of portalAddresses, and the commit also deletes that symbol from the public viem/tempo/zones barrel. Leaving the bump as patch lets Changesets publish a semver patch that breaks consumers importing the removed export, so this should be a major changeset or the export should be retained as a deprecated alias.

Useful? React with 👍 / 👎.

@jxom
jxom merged commit c067f1a into main Jul 23, 2026
44 checks passed
@jxom
jxom deleted the jxom/add-zone-e branch July 23, 2026 02:18
@github-actions github-actions Bot mentioned this pull request Jul 23, 2026
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