bump: snap-sdk to 12.0.1 + snap-utils to 12.6.0 + keyring-api to 24.1.0 + snaps-execution-environments to 11.3.0 + metamask/snaps-jest to 10.2.1 + metamask/keyring-snap-sdk to 10.0.0 - #214
Conversation
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
@metamaskbot publish-preview |
|
Preview builds have been published. Learn how to use preview builds in other projects. Expand for full list of packages and versions. |
| "@metamask/snaps-sdk": "12.0.1", | ||
| "@metamask/snaps-utils": "12.6.0", |
There was a problem hiding this comment.
-
resolve snaps-utils to 12.6.0 to enable PK export with base32
-
resolve snaps-sdk to 12.0.1 to enable PK export with base32
-
resolve metamask/snaps-execution-environments to 11.3.0 to align the snap sdk
| "snap_getPreferences": {} | ||
| }, | ||
| "platformVersion": "11.2.0", | ||
| "platformVersion": "12.0.1", |
There was a problem hiding this comment.
when we build the snap,
it may appear a warning ,
but soon will be resolved, as Snap team will update the client
The specified platform version "12.0.1" is not supported in the production version of MetaMask. The current maximum supported version is "12.0.0". To resolve this, downgrade `@metamask/snaps-sdk` to a
compatible version.
There was a problem hiding this comment.
Pull request overview
This PR updates the monorepo’s Snap dependencies to newer @metamask/snaps-sdk / @metamask/snaps-utils versions and bumps the Snap manifest platformVersion/shasum for the affected wallet snaps to align with the new SDK release.
Changes:
- Bump
@metamask/snaps-sdkto12.0.1and@metamask/snaps-utilsto12.6.0(via rootresolutions+yarn.lock). - Update
platformVersionto12.0.1and refreshshasumin multiple Snap manifests.
Reviewed changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
package.json |
Pins @metamask/snaps-sdk / @metamask/snaps-utils via Yarn resolutions. |
yarn.lock |
Updates lock entries to resolve @metamask/snaps-sdk@12.0.1 and @metamask/snaps-utils@12.6.0. |
packages/bitcoin-wallet-snap/snap.manifest.json |
Updates bundle shasum and platformVersion to 12.0.1. |
packages/solana-wallet-snap/snap.manifest.json |
Updates bundle shasum and platformVersion to 12.0.1. |
packages/stellar-wallet-snap/snap.manifest.json |
Updates bundle shasum and platformVersion to 12.0.1. |
packages/tron-wallet-snap/snap.manifest.json |
Updates bundle shasum and platformVersion to 12.0.1. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
snap-sdk to 12.0.1 + bump snap-utils to 12.6.0 snap-sdk to 12.0.1 + bump snap-utils to 12.6.0 + bump keyring-api to 24.1.0
snap-sdk to 12.0.1 + bump snap-utils to 12.6.0 + bump keyring-api to 24.1.0snap-sdk to 12.0.1 + snap-utils to 12.6.0 + keyring-api to 24.1.0
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 12 out of 13 changed files in this pull request and generated no new comments.
Suppressed comments (1)
package.json:88
resolutionspins@metamask/snaps-sdkto12.0.1, but the lockfile still shows multiple transitive packages declaring@metamask/snaps-sdkdependencies as^11.x(e.g.@metamask/snaps-jestdepends onnpm:^11.2.0atyarn.lock:3517, and@metamask/snaps-controllersdepends onnpm:^11.1.1atyarn.lock:3464). Forcing a major version outside dependents’ semver ranges viaresolutionscan mask real incompatibilities; it’s usually safer to bump those dependent@metamask/*packages to versions that explicitly support snaps-sdk 12 (or drop the resolution if it’s no longer needed).
"@metamask/snaps-execution-environments": "11.2.0",
"@metamask/snaps-sdk": "12.0.1",
"@metamask/snaps-utils": "12.6.0",
snap-sdk to 12.0.1 + snap-utils to 12.6.0 + keyring-api to 24.1.0snap-sdk to 12.0.1 + snap-utils to 12.6.0 + keyring-api to 24.1.0 + snaps-execution-environments to 11.3.0 + metamask/snaps-jest to 10.2.1
| "@metamask/snaps-cli": "^8.4.1", | ||
| "@metamask/snaps-jest": "^10.2.0", | ||
| "@metamask/snaps-sdk": "^11.2.0", | ||
| "@metamask/snaps-jest": "^10.2.1", |
There was a problem hiding this comment.
update metamask/snaps-jest to 10.2.1 is to align the snap sdk
|
@metamaskbot publish-preview |
|
Warning MetaMask internal reviewing guidelines:
|
|
Preview builds have been published. Learn how to use preview builds in other projects. Expand for full list of packages and versions. |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 12 out of 13 changed files in this pull request and generated 1 comment.
Suppressed comments (1)
package.json:88
resolutionspins@metamask/snaps-sdkto12.0.1, but the lockfile still includes dependencies that declare@metamask/snaps-sdk: npm:^11.x(e.g.@metamask/account-tree-controller). This forces those packages to run against a major version outside their declared compatibility range, which is a high risk for runtime/type breakage; consider upgrading those transitive packages to versions compatible with Snaps SDK v12 (or avoid forcing a global resolution if they cannot be upgraded yet).
"resolutions": {
"@metamask/snaps-execution-environments": "11.3.0",
"@metamask/snaps-sdk": "12.0.1",
"@metamask/snaps-utils": "12.6.0",
| "@metamask/snaps-execution-environments": "11.2.0", | ||
| "@metamask/snaps-sdk": "11.2.0", | ||
| "@metamask/snaps-execution-environments": "11.3.0", | ||
| "@metamask/snaps-sdk": "12.0.1", |
There was a problem hiding this comment.
Why is a resolution needed?
There was a problem hiding this comment.
good catch, i think we dont need to resolve the metamask/snaps-sdk
for metamask/snaps-execution-environments may still need?
There was a problem hiding this comment.
I don't understand why you would need it for any of the packages
There was a problem hiding this comment.
oh actually it need
snap cli 8.4.1 is using metamask/snaps-sdk 10.4.0
There was a problem hiding this comment.
Hmm, I would bump the CLI eventually, but you can do a resolution for now I guess
There was a problem hiding this comment.
snaps-execution-environments is becoz
asset controller -> account tree controlller -> snaps-execution-environments
| "snap_getPreferences": {} | ||
| }, | ||
| "platformVersion": "11.2.0", | ||
| "platformVersion": "12.0.1", |
There was a problem hiding this comment.
Updating this will now pass originMetadata to the keyring Snaps, we should double check that there is no validation preventing this. If we are using only keyring-api for validation, then we should be good.
There was a problem hiding this comment.
verified Safe for all snap
|
Is there a reason we are not bumping @metamask/keyring-snap-sdk also ? (Since it depends on the @metamask/keyring-api 24 that is bumped in this PR) |
|
@Julink-eth keyring API is peer dependency of keyring-snap-sdk so keyring-snap-sdk doesnt need to bump |
snap-sdk to 12.0.1 + snap-utils to 12.6.0 + keyring-api to 24.1.0 + snaps-execution-environments to 11.3.0 + metamask/snaps-jest to 10.2.1snap-sdk to 12.0.1 + snap-utils to 12.6.0 + keyring-api to 24.1.0 + snaps-execution-environments to 11.3.0 + metamask/snaps-jest to 10.2.1 + metamask/keyring-snap-sdk to 10.0.0
…snaps into bump/snap-sdk
Explanation
This PR bumps
Origin metadatato keyring api submit request, but none of our validation is blocking it, so it is safe to bumpWith those bumps, they add
Test bump PR on client
MetaMask/metamask-extension#45822
References
Checklist