-
-
Notifications
You must be signed in to change notification settings - Fork 275
feat: bump accounts deps + use new AccountProvider.createAccounts
#7857
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
d367787 to
a425eca
Compare
| - **BREAKING:** Use new `AccountProvider.createAccounts` method with `CreateAccountOptions` ([#7857](https://github.com/MetaMask/core/pull/7857)) | ||
| - All account providers now accept `CreateAccountOptions` with `type` field. | ||
| - Added `capabilities` property to all account providers defining supported account creation types. | ||
| - **BREAKING:** Bump `@metamask/eth-snap-keyring` from `^18.0.0` to `^19.0.0` ([#7857](https://github.com/MetaMask/core/pull/7857)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While not being used right now, I plan to use it right in a follow-up PR, and this would really become breaking at this point.
So, I just prefer to mark it as breaking right now, it feels more appropriate.
|
@metamaskbot publish-preview |
|
Preview builds have been published. See these instructions for more information about preview builds. Expand for full list of packages and versions. |
AccountProvider.createAccountsaccounts deps + use new AccountProvider.createAccounts
cryptodev-2s
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor suggestions otherwise looks good to me
Co-authored-by: cryptodev-2s <109512101+cryptodev-2s@users.noreply.github.com>
Co-authored-by: cryptodev-2s <109512101+cryptodev-2s@users.noreply.github.com>
Explanation
Bumping all accounts deps.
AND:
Use of the new
createAccountsmethod (from the keyring V2 ADR).This will allow to slowly integrate new account creation constructors (such as the new
bip44:derive-index-rangeto batch account creations).References
N/A
Checklist
Note
Medium Risk
Touches core account creation/alignment paths and upgrades several keyring/account dependency majors, so runtime compatibility issues or mis-specified
CreateAccountOptions.typecould break account provisioning across providers.Overview
Updates
@metamask/*account/keyring dependencies across controllers (notably@metamask/account-apito^1.0.0,@metamask/keyring-apito^21.5.0,@metamask/keyring-internal-apito^10.0.0,@metamask/eth-snap-keyringto^19.0.0, and@metamask/keyring-snap-clientto^8.2.0), with correspondingCHANGELOGentries andyarn.lockrefresh.Refactors
multichain-account-serviceproviders to the newAccountProvider.createAccounts(options: CreateAccountOptions)signature by threading an explicittype(currentlyAccountCreationType.Bip44DeriveIndex) through wallet/group alignment and creation flows, adding per-providercapabilitiesmetadata, and validating supported creation types viaassertCreateAccountOptionIsSupported(with new negative tests).AccountProviderWrappernow forwardscapabilitiesand accepts the newCreateAccountOptionsshape.Minor tooling cleanup includes removing an obsolete ESLint suppression entry for
src/tests/providers.ts.Written by Cursor Bugbot for commit 3e0dd00. This will update automatically on new commits. Configure here.