Skip to content

chore: Remove unused patches#26039

Merged
Gudahtt merged 1 commit intomainfrom
remove-unused-patches
Feb 12, 2026
Merged

chore: Remove unused patches#26039
Gudahtt merged 1 commit intomainfrom
remove-unused-patches

Conversation

@Gudahtt
Copy link
Copy Markdown
Member

@Gudahtt Gudahtt commented Feb 12, 2026

Description

These were not referenced anywhere, and had no lockfile impact.

Changelog

CHANGELOG entry: null

Related issues

N/A

Manual testing steps

N/A

Screenshots/Recordings

N/A

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

Note

Low Risk
Deletes unreferenced patch artifacts only; risk is limited to the unlikely case a patch was implicitly relied on despite not being wired into dependency resolution.

Overview
Removes several unused Yarn patch files under .yarn/patches, including patches previously targeting @metamask/* controllers and appwright/Appium configuration tweaks.

No runtime or dependency behavior is intended to change, since these patches were not referenced by package.json resolutions/lockfile.

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

These were not referenced anywhere, and had no lockfile impact.
@Gudahtt Gudahtt marked this pull request as ready for review February 12, 2026 21:10
@Gudahtt Gudahtt requested a review from a team as a code owner February 12, 2026 21:10
@metamaskbot metamaskbot added the team-core-platform Core Platform team label Feb 12, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🔍 Smart E2E Test Selection

  • Selected E2E tags: None (no tests recommended)
  • Selected Performance tags: None (no tests recommended)
  • Risk Level: low
  • AI Confidence: 95%
click to see 🤖 AI reasoning details

E2E Test Selection:
This PR is a housekeeping change that removes obsolete yarn patch files from the .yarn/patches directory. The deleted patches are for older versions of packages that have since been upgraded:

  1. @metamask/bridge-controller v61.0.0 patch - Package is now at v65.2.0
  2. @metamask/network-enablement-controller v2.1.0 and v3.0.0 patches - Package is now at v4.1.0
  3. @metamask/smart-transactions-controller v21.0.0 and v21.1.0 patches - Package is now at v22.5.0
  4. Three obsolete appwright patches - The current appwright uses a different patch (appwright-npm-0.1.45-f282bc1c1b.patch)

These patch files are no longer referenced anywhere in package.json (neither in dependencies nor resolutions). The deletion of these unused files has no functional impact on the application - no code behavior changes, no package versions change, and no build configuration changes.

No E2E tests are required because:

  • No application code is modified
  • No runtime behavior is affected
  • The patches were already not being used (orphaned files)
  • This is purely a repository cleanup operation

Performance Test Selection:
No performance tests are needed. The deleted appwright patches are obsolete and not referenced by the current appwright configuration. The current appwright uses a different patch file (appwright-npm-0.1.45-f282bc1c1b.patch) which remains in place. This cleanup has no impact on performance testing infrastructure or app performance.

View GitHub Actions results

@sonarqubecloud
Copy link
Copy Markdown

@Gudahtt Gudahtt enabled auto-merge February 12, 2026 23:17
Copy link
Copy Markdown
Contributor

@Cal-L Cal-L left a comment

Choose a reason for hiding this comment

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

LGTM

@Gudahtt Gudahtt added this pull request to the merge queue Feb 12, 2026
Merged via the queue into main with commit f52f17b Feb 12, 2026
77 checks passed
@Gudahtt Gudahtt deleted the remove-unused-patches branch February 12, 2026 23:36
@github-actions github-actions bot locked and limited conversation to collaborators Feb 12, 2026
@metamaskbot metamaskbot added the release-7.67.0 Issue or pull request that will be included in release 7.67.0 label Feb 12, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-7.67.0 Issue or pull request that will be included in release 7.67.0 size-M team-core-platform Core Platform team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants