Skip to content

feat(statics): add CoinFeature[] overload to getFilteredFeatures#8391

Merged
manas-at-bitgo merged 1 commit intomasterfrom
feat/ams-getfilteredfeatures-overload-cshld-559
Apr 2, 2026
Merged

feat(statics): add CoinFeature[] overload to getFilteredFeatures#8391
manas-at-bitgo merged 1 commit intomasterfrom
feat/ams-getfilteredfeatures-overload-cshld-559

Conversation

@manas-at-bitgo
Copy link
Copy Markdown
Contributor

@manas-at-bitgo manas-at-bitgo commented Apr 1, 2026

Summary

  • Added a TypeScript method overload to getFilteredFeatures in modules/statics/src/ofc.ts to accept CoinFeature[] directly, in addition to the existing string suffix lookup
  • Added ofcFeatures?: CoinFeature[] field to Erc20WithOfcConfig in generateERC20.ts so callers can explicitly control OFC coin features without relying on derived onChain.features

Test plan

  • Existing unit tests for getFilteredFeatures should pass unchanged
  • Verify TypeScript overloads resolve correctly for both string and CoinFeature[] call sites
  • Verify generateERC20 coins using ofcFeatures get expected filtered features on the OFC coin

Jira: CSHLD-559

🤖 Generated with Claude Code

Allow callers with an existing CoinFeature[] to pass it directly to
getFilteredFeatures, bypassing the coin map lookup. Also add ofcFeatures
field to Erc20WithOfcConfig so generateERC20 callers can explicitly
control OFC coin features.

Jira: CSHLD-559

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@manas-at-bitgo manas-at-bitgo requested a review from a team as a code owner April 1, 2026 12:45
@manas-at-bitgo manas-at-bitgo merged commit 8bb1fa4 into master Apr 2, 2026
21 checks passed
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