Skip to content

fix: fall back to zero-address for native tokens without slip44 - #9833

Open
n3ps wants to merge 5 commits into
mainfrom
n3ps/native-caip-fallback
Open

fix: fall back to zero-address for native tokens without slip44#9833
n3ps wants to merge 5 commits into
mainfrom
n3ps/native-caip-fallback

Conversation

@n3ps

@n3ps n3ps commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Explanation

EVM native tokens without a slip44 mapping (e.g. Chiliz) left assetId undefined.

Fall back to the zero-address ERC-20 CAIP-19 form (eip155:<chainId>/erc20:0x000…000) so clients can resolve based still based on assetId by deriving the chain id.

References

N/A

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

Made with Cursor

@n3ps
n3ps requested review from a team as code owners August 12, 2026 03:21
@n3ps
n3ps deployed to default-branch August 12, 2026 03:21 — with GitHub Actions Active
@n3ps n3ps changed the title fix: fall back to zero-address CAIP for EVM natives without slip44 fix: fall back to zero-address for native tokens without slip44 Aug 12, 2026
@n3ps
n3ps enabled auto-merge August 12, 2026 03:23

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit bc5aa76. Configure here.

Comment thread packages/client-utils/src/mappers/helpers/caip.ts
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