Skip to content

refactor: migrate {Authentication,UserStorage}Controller to @metamask/messenger - #6533

Merged
FrederikBolding merged 6 commits into
mainfrom
cryptodev-2s/messenger/profile-sync-controller
Oct 27, 2025
Merged

refactor: migrate {Authentication,UserStorage}Controller to @metamask/messenger#6533
FrederikBolding merged 6 commits into
mainfrom
cryptodev-2s/messenger/profile-sync-controller

Conversation

@cryptodev-2s

@cryptodev-2scryptodev-2s commented Sep 9, 2025

Copy link
Copy Markdown
Contributor

Explanation

This PR migrates AuthenticationController, UserStorageController to the new @metamask/messenger message bus, as opposed to the one exported from @metamask/base-controller.

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

Note

Migrates AuthenticationController and UserStorageController to @metamask/messenger, renames state metadata anonymous -> includeInDebugSnapshot, and updates tests, configs, and deps.

  • Profile Sync Controller:
    • Migrate AuthenticationController and UserStorageController from @metamask/base-controller messenger to @metamask/messenger (messagingSystem -> messenger; new handler/subscribe/call usage; updated types).
    • Update imports to @metamask/base-controller/next.
    • Rename state metadata key anonymous to includeInDebugSnapshot and adjust tests accordingly.
    • Revise tests/mocks to construct root/child messengers and delegate permissions; adapt mock handlers (e.g., SnapController:handleRequest).
  • Dependencies/Config:
    • Add @metamask/messenger dependency and TS project references; update yarn.lock.
    • Update README dependency graph to include @metamask/messenger and new edges.

Written by Cursor Bugbot for commit b09e552. This will update automatically on new commits. Configure here.

@cryptodev-2scryptodev-2s self-assigned this Sep 9, 2025
@cryptodev-2s
cryptodev-2s requested review from a team as code ownersSeptember 9, 2025 14:21
@cryptodev-2scryptodev-2s mentioned this pull request Sep 9, 2025
43 tasks
@cryptodev-2s
cryptodev-2sforce-pushed the cryptodev-2s/messenger/profile-sync-controller branch from 7ebbd49 to 5aaaf72CompareSeptember 10, 2025 11:36
@cryptodev-2s
cryptodev-2sforce-pushed the cryptodev-2s/messenger/profile-sync-controller branch from 5aaaf72 to 550de82CompareOctober 7, 2025 10:23
@cryptodev-2s
cryptodev-2s requested a review from a team as a code ownerOctober 7, 2025 10:23
@cryptodev-2s
cryptodev-2sforce-pushed the cryptodev-2s/messenger/profile-sync-controller branch from 550de82 to 05e4815CompareOctober 7, 2025 10:24
mcmire
mcmire previously approved these changes Oct 7, 2025

@mcmiremcmire left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

cursor[bot]

This comment was marked as outdated.

@FrederikBolding
FrederikBolding added this pull request to the merge queueOct 27, 2025
Merged via the queue into main with commit 7d6e178Oct 27, 2025
256 of 258 checks passed
@FrederikBolding
FrederikBolding deleted the cryptodev-2s/messenger/profile-sync-controller branch October 27, 2025 12:51
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@cryptodev-2s@mcmire@Gudahtt@Mrtenz@FrederikBolding@mikesposito