feat(chains): add Zetrix L2 Testnet (chainId 938748) - #152
Conversation
Adds Zetrix L2 Testnet (EVM, chainId 938748, native ZETRIX2, 18 decimals) so TrustVC-based verifiers can validate transferable records / eBL issued on the chain out of the box. - CHAIN_ID.zetrixL2Testnet = '938748' + SUPPORTED_CHAINS entry - networkName 'zetrixL2Testnet' + networkCurrency 'ZETRIX2' - placeholder chain icon (to be replaced with the official Zetrix asset) - test coverage for the new entry RPC https://z2-test-node.zetrix.com, explorer https://z2-test-explorer.zetrix.com (EIP-3091).
📝 WalkthroughWalkthroughAdds Zetrix L2 testnet support by defining its network and currency, exporting a placeholder icon, registering chain ID ChangesZetrix L2 testnet support
Estimated code review effort: 2 (Simple) | ~10 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
|
There was a problem hiding this comment.
🧹 Nitpick comments (1)
src/utils/static/icons.ts (1)
16-18: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low valueReminder: Replace the placeholder icon before merge.
The comment indicates that this placeholder SVG should be replaced with the official Zetrix brand asset before or at merge. Please ensure this is updated before the PR is completed.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/utils/static/icons.ts` around lines 16 - 18, The iconZetrix export currently uses a placeholder SVG; replace its data URI with the official Zetrix brand asset while preserving the existing export and usage contract.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@src/utils/static/icons.ts`:
- Around line 16-18: The iconZetrix export currently uses a placeholder SVG;
replace its data URI with the official Zetrix brand asset while preserving the
existing export and usage contract.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 39202072-ce1c-4e23-8dc7-bb1aa27c83f7
📒 Files selected for processing (4)
src/utils/network/index.tssrc/utils/static/icons.tssrc/utils/supportedChains/index.tssrc/utils/supportedChains/supportedChains.test.ts



What
Adds Zetrix L2 Testnet (EVM, chainId 938748, native currency ZETRIX2, 18 decimals) to the supported-chains registry so TrustVC-based issuers/verifiers can validate transferable records (eBL) issued on the chain out of the box, without a runtime
registerChainshim.Changes
src/utils/supportedChains/index.ts—CHAIN_ID.zetrixL2Testnet = '938748'+SUPPORTED_CHAINSentry (label, typetest, currencyZETRIX2, RPC, explorer, nativeCurrency).src/utils/network/index.ts— addzetrixL2TestnettonetworksandZETRIX2tonetworkCurrency.src/utils/static/icons.ts— chain icon (currently a placeholder SVG; happy to swap in the official Zetrix brand asset before merge — just let me know the preferred source).src/utils/supportedChains/supportedChains.test.ts— test coverage for the new entry.Details
https://z2-test-node.zetrix.comhttps://z2-test-explorer.zetrix.com(EIP-3091)938748is registered inethereum-lists/chains(PR Add Zetrix L2 Testnet (chainId 938748) ethereum-lists/chains#8523).Tests
npx vitest --run src/utils/supportedChains/supportedChains.test.ts→ 15 passed (incl. the newzetrixL2Testnetcase).Summary by CodeRabbit
New Features
Tests