Skip to content
This repository has been archived by the owner on Oct 7, 2024. It is now read-only.

Remove @metamask/snaps-ui from dependency tree #204

Merged
merged 2 commits into from
Nov 21, 2023

Conversation

mcmire
Copy link
Contributor

@mcmire mcmire commented Nov 20, 2023

  • The versions of @metamask/snaps-controllers and @metamask/snaps-rpc-methods that this package uses rely on @metamask/snaps-ui, which has recently been deprecated. The deprecation causes the dependencies audit on the extension to fail.
  • Global objects have been moved out of @metamask/snaps-rpc-methods to @metamask/snaps-sdk. Specifically, SnapsGlobalObject was renamed to SnapsProvider.

Given these changes, this commit bumps @metamask/snaps-controllers and replaces @metamask/snaps-rpc-methods with @metamask/snaps-sdk, using SnapsGlobalObject instead of SnapsProvider.

@mcmire mcmire requested a review from a team as a code owner November 20, 2023 19:55
@mcmire mcmire mentioned this pull request Nov 20, 2023
1 task
Copy link

socket-security bot commented Nov 20, 2023

Removed dependencies detected. Learn more about Socket for GitHub ↗︎

🚮 Removed packages: @metamask/snaps-rpc-methods@3.3.0, @metamask/snaps-utils@3.3.0

@mcmire

This comment was marked as outdated.

@mcmire mcmire marked this pull request as draft November 20, 2023 20:00
- The versions of `@metamask/snaps-controllers` and
  `@metamask/snaps-rpc-methods` that this package uses rely on
  `@metamask/snaps-ui`, which has recently been deprecated. The
  deprecation causes the dependencies audit on the extension to fail.
- Global objects have been moved out of `@metamask/snaps-rpc-methods` to
  `@metamask/snaps-sdk`. Specifically, `SnapsGlobalObject` was renamed
  to `SnapsProvider`.

Given these changes, this commit bumps `@metamask/snaps-controllers` and
replaces `@metamask/snaps-rpc-methods` with `@metamask/snaps-sdk`, using
`SnapsGlobalObject` instead of `SnapsProvider`.
@mcmire mcmire force-pushed the replace-snaps-rpc-methods branch from fc48169 to 3f98d7b Compare November 20, 2023 20:18
@mcmire mcmire changed the title @metamask/snaps-rpc-methods -> @metamask/snaps-sdk Remove @metamask/snaps-ui from dependency tree Nov 20, 2023
@mcmire mcmire marked this pull request as ready for review November 20, 2023 20:19
@danroc danroc added this pull request to the merge queue Nov 21, 2023
@danroc danroc removed this pull request from the merge queue due to a manual request Nov 21, 2023
@danroc danroc added this pull request to the merge queue Nov 21, 2023
Merged via the queue into main with commit ce33fc9 Nov 21, 2023
17 checks passed
@danroc danroc deleted the replace-snaps-rpc-methods branch November 21, 2023 12:56
legobeat added a commit that referenced this pull request Dec 6, 2023
legobeat added a commit that referenced this pull request Dec 6, 2023
danroc pushed a commit that referenced this pull request Dec 7, 2023
danroc pushed a commit that referenced this pull request Dec 7, 2023
github-merge-queue bot pushed a commit that referenced this pull request Dec 7, 2023
* 2.0.0

* update changelog

* format changelog

* add changelog entries for #204 and #213

---------

Co-authored-by: github-actions <github-actions@github.com>
Co-authored-by: legobt <6wbvkn0j@anonaddy.me>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants