Skip to content

fix: skip Solana wallet standard registration when Metamask extension exists#231

Draft
ffmcgee725 wants to merge 1 commit intomainfrom
jc/WAPI-1083
Draft

fix: skip Solana wallet standard registration when Metamask extension exists#231
ffmcgee725 wants to merge 1 commit intomainfrom
jc/WAPI-1083

Conversation

@ffmcgee725
Copy link
Member

@ffmcgee725 ffmcgee725 commented Mar 17, 2026

Explanation

When both the MetaMask extension and SDK are present, registering the SDK wallet would create a duplicate "MetaMask" entry in wallet adapters, confusing users. This change ensures only one MetaMask wallet appears in the wallet selection UI.

This PR brings back the change from #164 to prevent duplicate wallet registration when the MetaMask browser extension is already installed and registered in the Solana wallet-standard registry.

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, highlighting breaking changes as necessary
  • I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes

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