feat(tokens): add tSKHY/wtSKHY to the Base production token list - #263
Conversation
Adds the 21st Base production token — SK hynix Inc. ADR ST0x (tSKHY/wtSKHY) — to LibTokenInvariants: the receipt / receipt-vault / wrapped-token-vault pins and the receipt vault into productionReceiptVaults(). So the receipt-vault uniformity invariants cover it and the V4 receipt-vault upgrade (20260623-upgrade-receipt-vaults-to-v4) sets the authoriser on it too (its setAuthorizer tx count derives from productionReceiptVaults().length). Verified live on Base: wtSKHY.asset() == receiptVault, receiptVault owned by the token-owner Safe, on the V3 authoriser (pre-swap) like the other 20; LibTokenInvariantsTest uniform owner + authoriser pass with 21 vaults. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VPs1hCTxusmaSeFKvoc4Kr
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: ⛔ Files ignored due to path filters (12)
📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe production receipt vault invariant universe expands from 20 to 21 entries by adding the SK hynix receipt vault, allowing existing owner and authoriser checks to include it. ChangesSKHY vault coverage
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Poem
🚥 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 |
Adds the 21st Base production token (SK hynix ADR, tSKHY/wtSKHY) to
LibTokenInvariants— pins +productionReceiptVaults()— so the receipt-vault uniformity invariants and the V4 receipt-vault upgrade'ssetAuthorizercover it. Verified live on Base (correct wiring, Safe-owned, on V3 pre-swap);LibTokenInvariantsTestpasses with 21 vaults.Summary by CodeRabbit