Release/1020.0.0#8999
Merged
Merged
Conversation
|
✅ No changelog changes needed. |
ccharly
commented
Jun 4, 2026
Contributor
Author
There was a problem hiding this comment.
There's a ### Added that adds the keyring-controller dep explicitly. I don't think this really justify bumping the minor (given it was already in-use, just not declared really).
Or do we want to bump the minor instead?
cryptodev-2s
approved these changes
Jun 4, 2026
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.
Explanation
accounts-controllerto start using the new events from thesnap-account-serviceinstead of theSnapKeyringlegacy instancesnap-account-serviceto publish those eventsReferences
N/A
Checklist
Note
High Risk
Breaking messenger contract on accounts-controller affects all clients and downstream controllers; incorrect event registration would break snap account balance/asset/transaction updates.
Overview
Monorepo release
1020.0.0— bumps published package versions, changelogs,package.jsondependency ranges, andyarn.lockacross the workspace (no new application logic in this diff).The headline consumer change is
@metamask/accounts-controllerv39 (breaking): account asset/balance/transaction updates are re-published fromSnapAccountServiceevents instead of legacySnapKeyringevents, so messenger wiring must register the Snap Account Service events and can drop SnapKeyring equivalents.@metamask/snap-account-servicev0.3.0 adds the correspondingSnapAccountService:account{AssetList,Balances,Transactions}Updatedevents;@metamask/multichain-account-serviceand dependents pick that up via dependency bumps.Everything else in the release is mostly patch/minor version bumps (e.g.
transaction-controller66.0.1,account-tree-controller7.5.1, bridge/assets packages) propagating the accounts-controller and snap-account-service updates through the dependency graph.Reviewed by Cursor Bugbot for commit 3b1d708. Bugbot is set up for automated code reviews on this repo. Configure here.