Skip to content

Conversation

@danroc
Copy link
Contributor

@danroc danroc commented Jan 8, 2026

This PR adds the tron namespace and its CAIP-2 specification.

Rendered files:

@danroc danroc force-pushed the add-tron-ns branch 3 times, most recently from abf759e to 32f105a Compare January 8, 2026 08:28
@danroc danroc marked this pull request as ready for review January 8, 2026 08:28
@danroc danroc changed the title Add TRON namespace and CAIP-2 specification Add tron namespace and CAIP-2 specification Jan 8, 2026
@bumblefudge
Copy link
Collaborator

Made minor comments for formatting and made obvious stuff painfully explicit (it's been my experience that these get read very quickly!).

One editorial aside that might be worth mentioning is that proper handling of large ChainIDs is a little hard to guarantee/assume. I remember there have been many discussions over the years (mostly linked from this eth-mag thread ) about what the proper or optimal maximum value should be for chainId, but I also remember (can't find the link right now) that viem and other major libraries validate chainIds internally, making a defacto maximum value of... whatever library a given dapp uses 😅

This is all probably overkill for a TRON namespace, but just a possible footnote if you're already/also tracking that thread elsewhere...

@bumblefudge
Copy link
Collaborator

@obstropolos care to double-check i'm not missing anything?

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.

2 participants