Skip to content

correct list of codefi assets#8887

Open
bergarces wants to merge 3 commits into
mainfrom
correct-price-api-list
Open

correct list of codefi assets#8887
bergarces wants to merge 3 commits into
mainfrom
correct-price-api-list

Conversation

@bergarces
Copy link
Copy Markdown
Contributor

@bergarces bergarces commented May 21, 2026

Explanation

References

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

Note

Medium Risk
Updates the static chain-id→asset-id mapping used for native token spot-price queries; mistakes here can disable or misroute price fetching for affected networks. Scope is limited to constants and documentation, with no behavioral logic changes beyond which chains are considered supported.

Overview
Updates Codefi Price API v3 /spot-prices support mappings by adding inEVM Mainnet (0x9dd / chain 2525) and ICB Network (0x11d9b / chain 73115), fixing Blast Mainnet’s chain-id key (0x13c310x13e31), and removing Forma (0xf043a / chain 984122) since it’s not supported by the endpoint.

Also tightens maintainer guidance for SPOT_PRICES_SUPPORT_INFO (verification, comments, sorted-by-chain-id) and does minor ordering cleanup in the native-token address map; changelog is updated accordingly.

Reviewed by Cursor Bugbot for commit da3a844. Bugbot is set up for automated code reviews on this repo. Configure here.

@bergarces bergarces requested a review from a team as a code owner May 21, 2026 20:50
@bergarces bergarces temporarily deployed to default-branch May 21, 2026 20:50 — with GitHub Actions Inactive
'0x11d9b': 'eip155:73115/erc20:0x0000000000000000000000000000000000000000', // ICB Network - Native symbol: ICBX,
'0x138de': 'eip155:80094/erc20:0x0000000000000000000000000000000000000000', // Berachain - Native symbol: Bera',
'0x13c31': 'eip155:81457/slip44:60', // Blast Mainnet - Native symbol: ETH
'0x13e31': 'eip155:81457/slip44:60', // Blast Mainnet - Native symbol: ETH
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one had a typo in the key.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reordered the object keys, added a couple of missing chains for which we have support and removed one that had no prices (other than Tempo, which is a valid exception as there's no native asset).

'0x63564c40': 'eip155:1666600000/slip44:1023', // Harmony Mainnet Shard 0 - Native symbol: ONE
'0xdef1': 'eip155:57073/slip44:60', // Ink Mainnet - Native symbol: ETH
'0x3dc': 'eip155:988/erc20:0x779ded0c9e1022225f8e0630b35a9b54be713736', // Stable - Native symbol: USDT0
'0xf043a': 'eip155:984122/slip44:984122', // Forma - Native symbol: TIA (Celestia)
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed Forma as it is not supported by price api.

Reordered the other keys.

@bergarces bergarces requested a review from a team as a code owner May 21, 2026 20:59
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