add celestia arabica + edentestnet #1150
Conversation
🦋 Changeset detectedLatest commit: 202aa06 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
|
Mocha was added as |
|
agreed @christopherbrumm, see arabica devnet page |
📝 WalkthroughWalkthroughAdds a changeset and two new chain registry metadata files registering Celestia Arabica Devnet and Eden Testnet with their network endpoints, asset conventions, block parameters, and testnet flags. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes
Suggested reviewers
Poem
Pre-merge checks and finishing touches✅ Passed checks (2 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (3)
🚧 Files skipped from review as they are similar to previous changes (3)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (7)
.changeset/old-papers-rule.md (1)
5-5: Spell out the additions for easier release notes.Consider naming both additions explicitly, e.g., “Add Celestia Arabica Devnet (celestiadevnet) and Eden Testnet (edentestnet) to the registry,” to avoid ambiguity when scanning the changelog later.
chains/celestiadevnet/addresses.yaml (1)
1-2: Align the comment with the display name + add trailing newline.
- Match metadata’s displayName (“Celestia Arabica Devnet”) for consistency.
- YAML linter flags missing newline at EOF.
-# Hyperlane contract addresses for Celestia Arabica +# Hyperlane contract addresses for Celestia Arabica Devnet # These will be populated once contracts are deployed +chains/edentestnet/addresses.yaml (1)
1-2: Add trailing newline to keep linters happy.Nothing else to tweak here.
# Hyperlane contract addresses for Eden Testnet # These will be populated once contracts are deployed +chains/edentestnet/metadata.yaml (1)
14-14: Add trailing newline.Tiny paper cut, keeps lint green.
chains/celestiadevnet/metadata.yaml (3)
16-19: Avoid conflicting gas price settings.You set a base gasPrice of 0.1 utia, then override to 0.0. That can lead to underpriced txs on some Cosmos validators.
Pick one; simplest is to drop the override:
gasPrice: amount: "0.1" denom: utia @@ -transactionOverrides: - gasPrice: "0.0"Also applies to: 38-39
32-35: Endpoints look good; consider redundancy.If another RPC/REST mirror exists, adding it improves resilience.
1-39: Add trailing newline.Keeps YAML linters from grumbling.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (2)
chains/celestiadevnet/logo.svgis excluded by!**/*.svgchains/edentestnet/logo.svgis excluded by!**/*.svg
📒 Files selected for processing (5)
.changeset/old-papers-rule.md(1 hunks)chains/celestiadevnet/addresses.yaml(1 hunks)chains/celestiadevnet/metadata.yaml(1 hunks)chains/edentestnet/addresses.yaml(1 hunks)chains/edentestnet/metadata.yaml(1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-07-23T21:28:47.407Z
Learnt from: xeno097
PR: hyperlane-xyz/hyperlane-registry#788
File: chains/unichain/metadata.yaml:3-4
Timestamp: 2025-07-23T21:28:47.407Z
Learning: In the hyperlane-registry repository, API keys in chain metadata YAML files (like in chains/*/metadata.yaml) are intended for public usage and are not sensitive credentials that need to be hidden.
Applied to files:
chains/celestiadevnet/metadata.yaml
🪛 YAMLlint (1.37.1)
chains/celestiadevnet/addresses.yaml
[error] 2-2: no new line character at the end of file
(new-line-at-end-of-file)
chains/celestiadevnet/metadata.yaml
[error] 39-39: no new line character at the end of file
(new-line-at-end-of-file)
chains/edentestnet/metadata.yaml
[error] 14-14: no new line character at the end of file
(new-line-at-end-of-file)
chains/edentestnet/addresses.yaml
[error] 2-2: no new line character at the end of file
(new-line-at-end-of-file)
🔇 Additional comments (5)
chains/celestiadevnet/metadata.yaml (3)
24-26: Name matches guidance; good call using “celestiadevnet.”This lines up with the team’s preference for Arabica “devnet” naming. No changes needed.
If Mocha appears elsewhere as “celestiatestnet,” consider a follow-up PR to align naming across the board.
21-23: Sanity‑check the index.from height.Starting at 8,214,286—confirm this is a safe height with finalized data and that downstream indexers won’t miss prior events.
14-15: All green — domainId derivation verified.
Confirmed: parseInt('0x'+Buffer.from('ARAB').toString('hex')) % 4294967295 === 1095909698, so the metadata value is correct.chains/edentestnet/metadata.yaml (2)
12-14: Prefer HTTPS if available — RPC host reachable (HEAD -> 405)File: chains/edentestnet/metadata.yaml (lines 12–14). Probed http://rpc-evreth-sequencer-edennet-1-testnet.binary.builders:8545/ and received "HTTP/1.1 405 Method Not Allowed" (HEAD), so the host is reachable but rejects HEAD. Action: switch rpcUrls to an HTTPS endpoint if one exists; otherwise confirm the HTTP URL is intentional and validate the JSON‑RPC with a POST (e.g. curl -sS -X POST -H 'Content-Type: application/json' --data '{"jsonrpc":"2.0","method":"eth_chainId","params":[],"id":1}' http://rpc-evreth-sequencer-edennet-1-testnet.binary.builders:8545/).
2-6: No collisions found — confirm chainId/domainId 3735928814 is intentional.All clean: 3735928814 appears only in chains/edentestnet/metadata.yaml (chainId and domainId); name "edentestnet" is unique. Confirm both IDs are intended as-is or change to a different unused ID in chains/edentestnet/metadata.yaml.
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
chains/edentestnet/metadata.yaml (1)
14-14: Technical stack "other" may be too vague.Most chains specify a more concrete technical stack (e.g., "cosmos", "ethereum", "solana"). Marking it as "other" might not be descriptive enough. What's the actual stack Eden Testnet uses? Even an ogre's got layers—so should this metadata.
Consider specifying the actual technical stack, or confirm with maintainers whether "other" is the appropriate value for this testnet.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
chains/celestiadevnet/metadata.yaml(1 hunks)chains/edentestnet/metadata.yaml(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- chains/celestiadevnet/metadata.yaml
🔇 Additional comments (1)
chains/edentestnet/metadata.yaml (1)
12-13: No changes needed—structure is consistent with all other chains.The
http:nested key you flagged is actually the standard format used throughout the entire registry. I checked multiple chains (zora, zksync, zklink, zircuit, zeronetwork, zetachain, and others), and they all follow the same pattern: a list entry with anhttp:key pointing to the RPC URL. The edentestnet file is spot-on.
blasrodri
force-pushed
the
add-edentenestnet-celestia-arabica
branch
from
646a305 to
321f96c
Compare
- Add celestiadevnet chain configuration and logo - Add edentestnet chain configuration and logo - Set edentestnet domainId to 60901 - Add changeset for new chains
blasrodri
force-pushed
the
add-edentenestnet-celestia-arabica
branch
from
321f96c to
202aa06
Compare
|
@christopherbrumm i've changed the |
There was a problem hiding this comment.
Could you use the Eden logo instead of Paradex?
3 participants
Description
Adding Celestia Arabica and Eden Testnet into the registry
Backward compatibility
Testing
Summary by CodeRabbit
New Features
Chores