Skip to content

fix(deps): resolve 6 Snyk vulnerabilities via npm overrides (+Claude) - #219

Closed
dhruv-parekh-cs wants to merge 1 commit into
mainfrom
CL-snyk-fixes/12-Aug-2026
Closed

fix(deps): resolve 6 Snyk vulnerabilities via npm overrides (+Claude)#219
dhruv-parekh-cs wants to merge 1 commit into
mainfrom
CL-snyk-fixes/12-Aug-2026

Conversation

@dhruv-parekh-cs

Copy link
Copy Markdown

Automated Snyk remediation pass (/snyk-fix, run date 12-Aug-2026). Surface in scope: Node.js. Public repo — Conventional Commits used.

Summary

All six findings are transitive (nothing vulnerable is declared directly), so each is pinned through the existing overrides block. Caret ranges throughout, so future compatible patches aren't blocked.

IssueSeverityPackagePathOverride addedClass
SNYK-JS-BRACEEXPANSION-18512280Highbrace-expansion2.1.2@oclif/core > ejs > jake > filelist > minimatch@5.1.9 > brace-expansion"minimatch@5": { "brace-expansion": "^2.1.4" }Non-fixable
SNYK-JS-BRACEEXPANSION-18313044Highbrace-expansion5.0.7@oclif/core > minimatch@10.2.5 > brace-expansion"minimatch@10": { "brace-expansion": "^5.0.9" }Non-fixable
SNYK-JS-BRACEEXPANSION-18512280Highbrace-expansion5.0.7@oclif/core > minimatch@10.2.5 > brace-expansionsame minimatch@10 entry (^5.0.9 satisfies both advisories)Non-fixable
SNYK-JS-FASTURI-18021349Highfast-uri3.1.3@contentstack/cli-utilities > conf > ajv@8.20.0 > fast-uri"fast-uri": "^3.1.5"Non-fixable
SNYK-JS-FASTURI-18506908Highfast-uri3.1.3samesame (^3.1.5 satisfies both advisories)Non-fixable
SNYK-JS-JSYAML-18593780Highjs-yaml4.3.0@contentstack/cli-utilities > js-yaml"js-yaml": "^4.3.1"Non-fixable

Why brace-expansion is path-scoped rather than a single top-level override

The tree holds two brace-expansion majors at once — 2.1.2 under minimatch@5 and 5.0.7 under minimatch@10 — needing ≥2.1.4 and ≥5.0.9 respectively. A single top-level "brace-expansion": "^5.0.9" would force minimatch@5's copy across a major boundary. The two path-scoped entries patch each in place instead, matching the "minimatch@3": { … } selector style already used elsewhere in this repo's overrides.

Application-code changes

None. package.json / package-lock.json plus one .talismanrc line — see below.

.talismanrc change (please review)

The Talisman pre-commit hook already allowlists package-lock.json by content checksum. The lockfile content changed, so its recorded checksum went stale and blocked the commit (Talisman flags npm integrity fields — base64 SHA-512 digests — as "base64 encoded texts"). Only that one checksum was updated to the value Talisman itself reported; no new file was allowlisted and no scan was bypassed.

Validation

  • npm install — pass, no peer-dependency warnings
  • npm test — pass
  • npm run build — pass
  • snyk test --all-projects re-scan — clean, 0 issues remaining (the repo's own pre-commit Snyk gate also reported "Tested 463 dependencies for known issues, no vulnerable paths found")

Needs human review

None for this repo.


🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
CopilotAI lite review requested due to automatic review settings August 12, 2026 07:20
@dhruv-parekh-cs
dhruv-parekh-cs requested review from a team as code ownersAugust 12, 2026 07:20
@snyk-io

snyk-ioBot commented Aug 12, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

StatusScan Engine Critical High Medium LowTotal (0)
Open Source Security0000 0 issues
Licenses0000 0 issues
Code Security0000 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@github-actions

Copy link
Copy Markdown

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check TypeCount (with fixes)Without fixesThresholdResult
🔴 Critical Severity0010✅ Passed
🟠 High Severity0025✅ Passed
🟡 Medium Severity00500✅ Passed
🔵 Low Severity001000✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

SeverityBreaches (with fixes)Breaches (no fixes)SLA Threshold (with/no fixes)Status
🔴 Critical0015 / 30 days✅ Passed
🟠 High0030 / 120 days✅ Passed
🟡 Medium0090 / 365 days✅ Passed
🔵 Low00180 / 365 days✅ Passed

✅ BUILD PASSED - All security checks passed

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates dependency resolution to remediate six transitive Snyk findings in this Node.js CLI repo by adding/adjusting npmoverrides, regenerating the lockfile accordingly, and updating the Talisman allowlist checksum for the modified lockfile.

Changes:

  • Added overrides entries to force patched versions of fast-uri, js-yaml, and brace-expansion (scoped by minimatch major).
  • Updated package-lock.json to reflect the new override-driven dependency graph.
  • Updated .talismanrc checksum for package-lock.json so pre-commit secret scanning does not block the changed lockfile.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

FileDescription
package.jsonAdds new overrides pins/scoped overrides to remediate transitive vulnerabilities.
package-lock.jsonLockfile refreshed to reflect overridden versions (e.g., fast-uri, js-yaml, brace-expansion).
.talismanrcUpdates stored checksum for the modified package-lock.json.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment threadpackage.json
"tmp": "^0.2.4"
"tmp": "^0.2.4",
"fast-uri": "^3.1.5",
"js-yaml": "^4.3.1",
Sign up for freeto 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.

2 participants

@dhruv-parekh-cs