Skip to content

feat: create merger and factory for the Caip25Permission#5283

Merged
ffmcgee725 merged 28 commits intomainfrom
jc/merger-caip25-permission
Feb 25, 2025
Merged

feat: create merger and factory for the Caip25Permission#5283
ffmcgee725 merged 28 commits intomainfrom
jc/merger-caip25-permission

Conversation

@ffmcgee725
Copy link
Copy Markdown
Member

@ffmcgee725 ffmcgee725 commented Feb 5, 2025

Explanation

Addressing https://github.com/MetaMask/MetaMask-planning/issues/4017

We want to create a merger for the Caip25Permission specification in order to enable wallet_addEthereumChain and wallet_switchEthereumChain to call to the PermissionController rather than the ApprovalController for incremental permission requests (i.e. for the permittedChains flow).

References

Extension: MetaMask/metamask-extension#30042

Changelog

@metamask/multichain-api

  • BREAKING: Renamed mergeScopes to mergeNormalizedScopes
  • ADDED: Added merger to CaveatSpecification returned by caip25CaveatBuilder()
  • ADDED: Added mergeInternalScopes which merges two InternalScopesObjects

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 highlighted breaking changes using the "BREAKING" category above as appropriate
  • I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes

Loading
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.

4 participants