Skip to content

fix: Update glob via overrides#72

Merged
kodiakhq[bot] merged 3 commits intomainfrom
fix/update_glob_via_overrides
Feb 26, 2026
Merged

fix: Update glob via overrides#72
kodiakhq[bot] merged 3 commits intomainfrom
fix/update_glob_via_overrides

Conversation

@erezrokah
Copy link
Member

@erezrokah erezrokah added the automerge Add to automerge PRs once requirements are met label Feb 26, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the glob package indirectly by pinning @vercel/nft to version 1.3.2 via npm overrides. The package @vercel/nft is a transitive dependency brought in by the ava testing framework. This change mirrors a similar fix implemented in the plugin-sdk-javascript repository (PR #355), suggesting a coordinated effort to address a known issue, likely a security vulnerability or bug in older versions of glob.

Changes:

  • Added npm override to pin @vercel/nft to version 1.3.2
  • Updated @vercel/nft from 0.29.4 to 1.3.2, which brings glob from v10.4.5 to v13.0.6 and updates related dependencies (minimatch, minipass, etc.)

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
package.json Added overrides section to pin @vercel/nft to version 1.3.2
package-lock.json Reflects dependency tree changes from the override, including updated versions of @vercel/nft, glob, minimatch, minipass, and new nested dependencies; several devDependencies marked as "peer": true

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@erezrokah erezrokah requested a review from mnorbury February 26, 2026 15:48
@kodiakhq kodiakhq bot merged commit 01ee439 into main Feb 26, 2026
8 checks passed
@kodiakhq kodiakhq bot deleted the fix/update_glob_via_overrides branch February 26, 2026 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Add to automerge PRs once requirements are met

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants