feat: ERC721 token support for EVM coins#7707
Merged
rohitsaw115 merged 1 commit intomasterfrom Dec 22, 2025
Merged
Conversation
rohitsaw115
force-pushed
the
erc721
branch
2 times, most recently
from
7dbdff9 to
3b2cdbc
Compare
ArunBala-Bitgo
previously approved these changes
Dec 8, 2025
Contributor
ArunBala-Bitgo
left a comment
ArunBala-Bitgo
left a comment
There was a problem hiding this comment.
Could we add test cov for the new changes
rohitsaw115
force-pushed
the
erc721
branch
2 times, most recently
from
e6f722c to
0c52dd8
Compare
There was a problem hiding this comment.
Pull request overview
This PR adds ERC721 (NFT) token support for EVM-compatible coins, specifically enabling it for Hedera EVM (both mainnet and testnet). The implementation follows the existing ERC20 token pattern and introduces the necessary infrastructure to support NFTs across EVM chains.
Key Changes:
- Added
SUPPORTS_ERC721coin feature andERC721underlying asset type - Implemented
EthLikeERC721Tokenclass and factory function with non-divisible token support (0 decimal places) - Registered ERC721 token constructors and configured token mappings for supported chains
- Fixed Hedera EVM base unit from
HBAR(tinybar) toETH(wei) to align with EVM standards
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| modules/statics/src/base.ts | Added SUPPORTS_ERC721 coin feature enum value |
| modules/statics/src/account.ts | Added EthLikeERC721Token class, Erc721ConstructorOptions interface, and erc721Token factory function with proper documentation |
| modules/statics/src/tokenConfig.ts | Added ERC721 token configuration functions and integration into token map structure |
| modules/statics/src/coins.ts | Added ERC721 chain-to-name mapping and token creation logic in createToken function |
| modules/statics/src/allCoinsAndTokens.ts | Enabled SUPPORTS_ERC721 feature for Hedera EVM (mainnet and testnet) and corrected base unit from HBAR to ETH |
| modules/sdk-coin-evm/src/ethLikeErc721Token.ts | Implemented EthLikeErc721Token class extending EthLikeToken with ERC721-specific functionality |
| modules/sdk-coin-evm/src/register.ts | Added ERC721 token constructor registration for EVM SDK |
| modules/sdk-coin-evm/src/index.ts | Exported EthLikeErc721Token class |
| modules/bitgo/src/v2/coinFactory.ts | Added ERC721 token registration, testnet/mainnet mapping, and token constructor logic |
| modules/bitgo/src/v2/coins/index.ts | Exported EthLikeErc721Token for use in BitGo SDK |
| modules/bitgo/test/browser/browser.spec.ts | Updated test exclusions to include EthLikeErc721Token |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
rohitsaw115
force-pushed
the
erc721
branch
10 times, most recently
from
7f71c74 to
c58e5ee
Compare
rohitsaw115
force-pushed
the
erc721
branch
5 times, most recently
from
86f67bc to
0b361aa
Compare
ArunBala-Bitgo
previously approved these changes
Dec 10, 2025
rohitsaw115
force-pushed
the
erc721
branch
2 times, most recently
from
25a45dc to
fa1189b
Compare
rohitsaw115
requested review from
ArunBala-Bitgo,
kisslove-dewangan and
mohammadalfaiyazbitgo
Contributor
Author
|
@claude please review this |
ArunBala-Bitgo
approved these changes
Dec 22, 2025
ArunBala-Bitgo
approved these changes
Dec 22, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ticket: win-8154
Changes:-