Conversation
WalkthroughVersion bump from 4 to 5 and addition of two TRC20 Tron tokens (USDT-tron, USDC-tron); schema, types, and validation updated to recognize the Tron network and TRC20 token type. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
No actionable comments were generated in the recent review. 🎉 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 |
Greptile OverviewGreptile SummaryThis PR appends two TRON stablecoin entries (USDT and USDC) to However, the repository’s current token-list schema and CI validation logic appear to only support EVM-style networks/tokens (plus ISO4217 fiat). The new tokens introduce Confidence Score: 2/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant Dev as Contributor
participant TokenList as tokens/token-list.json
participant CI as GitHub Actions (pr.yml)
participant Validator as src/validation/validate.ts
participant Schema as src/schemas/token-list-schema.json
participant Types as src/types/index.ts
Dev->>TokenList: Add USDT-tron + USDC-tron entries
CI->>Validator: Run npm run validate
Validator->>Schema: AJV compile + validate(tokenList)
Schema-->>Validator: Reject (network/type enums)
Validator-->>CI: Exit 1 (schema errors)
CI-->>Dev: Checks fail
note over Validator,Types: Even if schema updated,
Validator->>Types: isValidNetwork / isValidTokenType / CHAIN_IDS lookup
Types-->>Validator: No tron/TRC20/chainId mapping
Validator->>Validator: ethers.isAddress(TRON base58)
Validator-->>CI: Exit 1 (token validation)
|
MantisClone
left a comment
There was a problem hiding this comment.
LGTM. Token addresses, chain ID (728126428), and decimals verified against official sources.
Closes RequestNetwork/private-issues#224
Summary by CodeRabbit