Repository navigation
fix(deps): resolve Socket alerts for npm - #4
Open
ls-valentinas-bakaitis wants to merge 3 commits into
Open
ls-valentinas-bakaitis wants to merge 3 commits into
ls-valentinas-bakaitis wants to merge 3 commits into
Conversation
…pawn, js-yaml, brace-expansion, minimatch (Socket alerts)
|
You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool. What Enabling Code Scanning Means:
For more information about GitHub Code Scanning, check out the documentation. |
ls-valentinas-bakaitis
marked this pull request as ready for review
November 10, 2026 02:21
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixed
All dev-only transitive deps, updated in package-lock.json via
npm update.Already resolved
None
Verification
CI=true npm run lintpassed;npm run buildpassed;npm run testpassed (19/19). Same results before and after. Installed with--ignore-scripts.Left for a human
Socket's stated targets for js-yaml (4.3.1, applied to the 3.x copy), brace-expansion (5.0.11) were not met for the older-major copies (js-yaml 3.15.2, brace-expansion 1.x/2.x), since parents constrain majors. Re-check Socket after merge. The form-data (3.0.5) and cross-spawn (6.0.6) targets were lower than installed majors, so latest in the installed major was used.
Other changes
package.jsongains anoverridesentry forcing minimatch ^9.0.7 under @typescript-eslint/typescript-estree (it pinned 9.0.3).Generated by
socket-alerts(AI agent). Runvend__reactular__npm/20261110T004613Z. Review before merging.🛡️ Release Checklist
1. Merge Timing (Required)
When will this PR be merged?
2. Feature Flag
Does this PR require a feature flag to be created?
3. Environment Variables
Does this PR require environment variables to be added/removed/updated?
4. Database Migration
Does this PR require a database migration applied to production BEFORE it lands? (Production usually has no auto-deploy.)
5. Rollback Safety
Is this PR safe to rollback?
6. Dependencies
Is this PR self-contained, or does it depend on another PR not yet in master/main?
7. Risk Assessment (Select exactly one)
8. Urgency (Select exactly one)