Skip to content

chore: align chain config to chainlist registry#49

Merged
github-actions[bot] merged 1 commit into
mainfrom
chore/align-chain-config-to-chainlist
May 10, 2026
Merged

chore: align chain config to chainlist registry#49
github-actions[bot] merged 1 commit into
mainfrom
chore/align-chain-config-to-chainlist

Conversation

@satyakwok
Copy link
Copy Markdown
Member

Summary

Standardize Sentrix Chain identity across every frontend app to match the ethereum-lists/chains submission (PR #8266). Three sources had each invented their own variant for the network name, native currency, and RPC URL pattern.

Final canonical values

Mainnet Testnet
name Sentrix Chain Sentrix Testnet
nativeCurrency.name Sentrix Sentrix
nativeCurrency.symbol SRX SRX
RPC https://rpc.sentrixchain.com https://testnet-rpc.sentrixchain.com
Explorer https://scan.sentrixchain.com https://scan-testnet.sentrixchain.com

Notable fixes

  • apps/chain-landing/public/chain-testnet.json was declaring nativeCurrency.symbol: "tSRX" — that ticker has never existed on chain. Now SRX.
  • apps/chain-landing/src/components/ui/add-to-wallet-button.tsx had nativeCurrency.name: "SRX" (duplicating the symbol) — now "Sentrix", matching the EIP-3085 convention used by Ethereum (name: "Ether", symbol: "ETH").
  • Testnet block explorer URL switched from scan.sentrixchain.com to scan-testnet.sentrixchain.com. Fixes the chainlist reviewer's complaint that testnet tx links opened the mainnet view — the two-host pattern (already used by apps/scan/lib/network-context.ts) makes EIP-3091 routing unambiguous.
  • Mainnet RPC URLs now bare (https://rpc.sentrixchain.com), no /rpc suffix. Both forms work on the live Caddy LB; bare is the standard EVM convention and matches what most wallets paste in.
  • Drops the retired Blockscout entry from chain.json.

Files

12 files changed, all surfacing chain identity to wallets, RPC clients, or end-user UI:

  • Canonical chain configs: packages/wallet-config/src/chain.ts, apps/scan/lib/chain.ts, apps/airdrop/src/lib/chain.ts, apps/coinblast/src/lib/chain.ts, apps/landing/lib/chain.ts
  • EIP-3085 add-network params: apps/chain-landing/src/components/ui/add-to-wallet-button.tsx
  • chainagnostic chain-meta JSON: apps/chain-landing/public/chain.json, apps/chain-landing/public/chain-testnet.json
  • Display labels: apps/coinblast/src/app/create/page.tsx, apps/solux/src/components/NetworkHealth.tsx
  • /rpc cleanup: apps/coinblast/src/lib/ws.ts, apps/landing/components/sections/developers.tsx

Test plan

  • pnpm -w build (all apps build)
  • Open landing one-click "Add to Wallet" — MetaMask shows "Sentrix Chain" / "Sentrix Testnet" with correct RPC + explorer URLs
  • In MetaMask after add, send 1 SRX — balance label reads "Sentrix" not "SRX"
  • Coinblast token-create dropdown shows "Sentrix Chain" (not "Sentrix Mainnet")
  • Solux network panel header shows "Sentrix Chain"
  • Verify scan-testnet.sentrixchain.com routes correctly for testnet tx links

Standardize the canonical chain identity across every frontend app to
match the ethereum-lists/chains submission (PR #8266). Three sources had
each invented their own variant.

Mainnet name: "Sentrix Chain" everywhere (was a mix of "Sentrix",
"Sentrix Mainnet", "Sentrix Chain"). Testnet stays "Sentrix Testnet".
Native currency: name "Sentrix", symbol "SRX" on both networks
(chain-testnet.json was claiming "tSRX" — wrong, never existed on chain).
RPC URLs: bare hostname, no /rpc suffix (both work, bare is the standard
EVM convention). Testnet block explorer: scan-testnet.sentrixchain.com,
not scan.sentrixchain.com — fixes the EIP-3091 routing complaint where
testnet tx links opened the mainnet view.

Also drops the retired Blockscout entry from chain.json.

Display labels in coinblast network selector and solux NetworkHealth
panel updated to match.
@github-actions github-actions Bot enabled auto-merge (squash) May 10, 2026 17:39
@github-actions github-actions Bot merged commit b98f200 into main May 10, 2026
4 checks passed
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