Skip to content

Chore | +Claude | Pin js-yaml via npm override to resolve Snyk high-severity finding - #218

Merged
AryanBansal-launch merged 1 commit into
developmentfrom
CL-snyk-fixes/10-Aug-2026
Aug 10, 2026
Merged

Chore | +Claude | Pin js-yaml via npm override to resolve Snyk high-severity finding#218
AryanBansal-launch merged 1 commit into
developmentfrom
CL-snyk-fixes/10-Aug-2026

Conversation

@AryanBansal-launch

Copy link
Copy Markdown
Contributor

Summary

  • Fixes SNYK-JS-JSYAML-18593780 (Inefficient Algorithmic Complexity, High severity) in js-yaml@4.3.0, pulled in transitively via @contentstack/cli-utilities, eslint-config-oclif, and mocha.
  • Adds a flat js-yaml: ^4.3.1 npm override — all declared ranges (^4.1.0, etc.) already permit the patch bump, so no code changes were needed.
  • Updates the .talismanrc checksum for package-lock.json to match the regenerated lockfile hash.

Verification

  • npm ci — clean install against the updated lockfile
  • npm run build — passes
  • npm test — 129/129 Jest tests pass; mocha suite unchanged from baseline (pre-existing local-only failures gated on ORG/ENVIRONMENT/PROJECT env vars not present in this environment, unrelated to this change)
  • snyk test — 0 vulnerable paths (was 1 issue / 2 vulnerable paths)

Test plan

  • Confirmed via a clean npm ci + re-scan that the fix holds on a fresh install
  • No source code changes — dependency-only fix

🤖 Generated with Claude Code

…ding
js-yaml@4.3.0 (pulled in transitively via @contentstack/cli-utilities,
eslint-config-oclif, and mocha) is affected by SNYK-JS-JSYAML-18593780
(Inefficient Algorithmic Complexity, High severity), fixed in 4.3.1.
All declared ranges (^4.1.0 etc.) already permit the patch bump, so a
flat override is enough — no code changes required.
Updates the .talismanrc checksum for package-lock.json to match the
new lockfile hash (Talisman's secret scanner flags dependency hash
changes for review).
Verified with npm ci, npm run build, npm test, and a clean re-scan
(snyk test: 0 vulnerable paths, was 1 issue / 2 vulnerable paths).
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
CopilotAI lite review requested due to automatic review settings August 10, 2026 10:13
@snyk-io

snyk-ioBot commented Aug 10, 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.

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

Pins js-yaml via npm overrides to address a high-severity Snyk finding in the transitive dependency graph, and updates the lockfile and secret-scan allowlist checksum accordingly.

Changes:

  • Adds an npm override for js-yaml to bump it to ^4.3.1.
  • Regenerates package-lock.json to reflect the overridden resolution (including pruning now-unneeded nested deps).
  • Updates .talismanrc checksum for the updated lockfile.

Reviewed changes

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

FileDescription
package.jsonAdds a js-yaml entry under overrides to force a patched version.
package-lock.jsonUpdates resolved js-yaml version to 4.3.1 and prunes no-longer-needed nested packages.
.talismanrcUpdates the stored checksum for 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
@AryanBansal-launchAryanBansal-launch changed the title chore: pin js-yaml via npm override to resolve Snyk high-severity findingChore | +Claude | Pin js-yaml via npm override to resolve Snyk high-severity findingAug 10, 2026
@AryanBansal-launch
AryanBansal-launch merged commit 6d9d639 into developmentAug 10, 2026
6 of 7 checks passed
@AryanBansal-launch
AryanBansal-launch deleted the CL-snyk-fixes/10-Aug-2026 branch August 10, 2026 10:21
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.

4 participants

@AryanBansal-launch@Harshi-Shah-CS@rohan-agrawal-cs