feat(cct-sdk): Add get token info op solana - #405
Conversation
…-withdraw-liquidity
|
You must have Developer access to commit code to Chainlink Labs on Vercel. If you contact an administrator and receive Developer access, commit again to see your changes. Learn more: https://vercel.com/docs/accounts/team-members-and-roles/access-roles#team-level-roles |
CI Test Report✅ 2506/2512 tests passed (650 suites) in 5m 2s SummaryCoverage report |
Overview✅ APPROVE-WITH-NITS — 0 blockers, 0 correctness majors. Well-typed, sound, not a duplicate. Adds a
Architecture: Read-only query ( Live Devnet Verification:
Note on CI: The red "Run npm ci" is a trunk-wide [COMMENT 1/5] 🔴 Medium —
|
[COMMENT 2/5] 🟡 Low — Nondeterministic error type + double-fetch from
|
[COMMENT 3/5] 🔵 Low (Completeness) —
|
[COMMENT 4/5] 🔵 Nits (DX) — Cross-references, @throws, parity note
Example addition to facade: /**
* Reads an SPL token mint's metadata, program, supply, and mint/freeze authorities.
*
* @remarks
* Metadata comes from {@link SolanaChain.getTokenInfo}; mint state comes directly from
* the SPL Token or Token-2022 mint account. Supply is in base units. **Solana-only** — no
* EVM equivalent exists.
*
* @see {@link setTokenAuthority} — sets the authorities returned here
* @see {@link updateMetadataAuthority} — updates metadata authority
* @see {@link getTokenPoolState} — reads pool configuration (not mint)
* @see {@link SolanaChain.getTokenInfo} — core implementation (symbol/decimals only)
*
* @throws {@link CCTParamsInvalidError} If `tokenAddress` is not a valid Solana public key.
* @throws {@link CCIPSplTokenInvalidError} If the mint is not owned by an SPL Token program.
* @throws {@link CCIPTokenMintNotFoundError} If the mint account does not exist.
* @throws {@link CCIPTokenMintInvalidError} If the mint account is not a valid mint.
* @throws {@link CCIPTokenDataParseError} If the mint data cannot be parsed.
*/ |
[COMMENT 5/5] 📋 Informational — CI status & cross-family contextCI Status (not this PR)The red "Run npm ci" failure is a trunk-wide
Fix (once, at Cross-Family ParityThis PR adds a Solana-only read. There is no EVM CCT |
@aelmanaa Addressed all comments except for #5 which was already handled by different PR. |
* feat: add createPoolSignerATA option to deploy token pool op * fix: address comments
What
getTokenInfooperation and facade methodWhy