Skip to content

chore: Disallow Git dependencies#641

Merged
FrederikBolding merged 1 commit into
mainfrom
fb/disallow-git-deps
May 13, 2026
Merged

chore: Disallow Git dependencies#641
FrederikBolding merged 1 commit into
mainfrom
fb/disallow-git-deps

Conversation

@FrederikBolding
Copy link
Copy Markdown
Member

@FrederikBolding FrederikBolding commented May 13, 2026

Bump Yarn to v4 and disallow Git dependencies. Additionally add npmMinimalAgeGate and other properties from the module template.


Note

Medium Risk
Changes package manager version and Yarn install policies (Git dependency allowlist and npm age gate), which can break installs or CI if dependencies/plugins rely on previous Yarn behavior.

Overview
Updates the repo to Yarn 4.14.1 and removes the vendored @yarnpkg/plugin-constraints bundle from .yarn/plugins.

Tightens dependency sourcing rules by disallowing all Git-based dependencies via approvedGitRepositories: [], and adds an NPM publish-age policy (npmMinimalAgeGate) with an allowlist (npmPreapprovedPackages) in .yarnrc.yml.

Reviewed by Cursor Bugbot for commit e94509f. Bugbot is set up for automated code reviews on this repo. Configure here.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 13, 2026

Commit e94509f770f45f2ac56307d5b22c19526364becd successfully pushed to staging website:

https://metamask.github.io/snaps-directory-staging/pr/641/

@FrederikBolding FrederikBolding marked this pull request as ready for review May 13, 2026 08:03
@FrederikBolding FrederikBolding requested a review from a team as a code owner May 13, 2026 08:03
@FrederikBolding FrederikBolding enabled auto-merge (squash) May 13, 2026 09:41
@FrederikBolding FrederikBolding merged commit de7b9e1 into main May 13, 2026
21 checks passed
@FrederikBolding FrederikBolding deleted the fb/disallow-git-deps branch May 13, 2026 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants