Skip to content

feat: add Tempo and Hyperliquid chain aliases#134

Merged
njdawn merged 2 commits intoopen-wallet-standard:mainfrom
franklad:feat/tempo-chain-alias
Apr 9, 2026
Merged

feat: add Tempo and Hyperliquid chain aliases#134
njdawn merged 2 commits intoopen-wallet-standard:mainfrom
franklad:feat/tempo-chain-alias

Conversation

@franklad
Copy link
Copy Markdown
Contributor

@franklad franklad commented Mar 26, 2026

Summary

  • Add Tempo (eip155:4217) and Hyperliquid (eip155:999) to KNOWN_CHAINS registry as EVM chains
  • Add default RPC endpoints (rpc.tempo.xyz, rpc.hyperliquid.xyz/evm)
  • Update supported chains docs with new entries and shorthand aliases

Test plan

  • cargo test -p ows-core — 71 tests pass
  • Full cargo build compiles cleanly
  • Alias resolution tests for both friendly names and CAIP-2 IDs
  • RPC endpoint lookup tests for both chains

@franklad franklad requested a review from njdawn as a code owner March 26, 2026 01:21
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 26, 2026

@franklad is attempting to deploy a commit to the MoonPay Team on Vercel.

A member of the Team first needs to authorize it.

njdawn
njdawn requested changes Mar 31, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@njdawn njdawn left a comment

Choose a reason for hiding this comment

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

overall looks good.

can be merged when existing tests that assume eip155:999 is unknown must be updated

@franklad
Copy link
Copy Markdown
Contributor Author

franklad commented Mar 31, 2026

Tests already updated — the old eip155:999 miss test in config.rs now uses eip155:99999, and the ows-pay tests use eip155:999999 (6 nines) which isn't affected. Full suite passes: 217 passed, 0 failed.

@franklad franklad requested a review from njdawn April 1, 2026 11:55
@njdawn
Copy link
Copy Markdown
Contributor

njdawn commented Apr 1, 2026

fix merge conflicts! then good to go!

@franklad franklad force-pushed the feat/tempo-chain-alias branch from 09c1ebe to 1a20ac0 Compare April 2, 2026 02:52
njdawn
njdawn requested changes Apr 3, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@njdawn njdawn left a comment

Choose a reason for hiding this comment

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

some final merge conflicts, then lgtm

@franklad franklad force-pushed the feat/tempo-chain-alias branch from 1a20ac0 to c62177d Compare April 4, 2026 18:08
@franklad franklad requested a review from njdawn April 5, 2026 12:58
njdawn
njdawn requested changes Apr 8, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@njdawn njdawn left a comment

Choose a reason for hiding this comment

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

one more merge conflict -> then can merge! appreciate the patience

@franklad franklad force-pushed the feat/tempo-chain-alias branch from c62177d to fc29700 Compare April 8, 2026 17:38
franklad added 2 commits April 8, 2026 13:43
@franklad
feat: add Tempo and Hyperliquid chain aliases
56c3b87 
Register Tempo (eip155:4217) and Hyperliquid (eip155:999) as known EVM
chains with friendly name aliases and default RPC endpoints.
@franklad
style: run cargo fmt on config.rs
7708e9d
@franklad franklad force-pushed the feat/tempo-chain-alias branch from fc29700 to 7708e9d Compare April 8, 2026 17:43
@franklad
Copy link
Copy Markdown
Contributor Author

franklad commented Apr 8, 2026

one more merge conflict -> then can merge! appreciate the patience

Done! nw

@njdawn njdawn merged commit dc95cd0 into open-wallet-standard:main Apr 9, 2026
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@njdawn njdawn njdawn requested changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@franklad @njdawn