Conversation
Clears all 14 Snyk security findings (2 critical, 7 high, 5 moderate) and all 6 npm audit findings. Every vulnerability was transitive. Upgrades next, react and react-dom within their existing majors, and adds undici and js-yaml overrides. Raises the stale qs and sharp overrides, which were pinning versions that had themselves become vulnerable. No major version bumps. Snyk's suggested isomorphic-dompurify 4.0.0 major was rejected in favour of a same-major undici override covering the same 11 findings. The build failure on this branch is pre-existing and environmental: it reproduces identically on unmodified main via npm ci, and is caused by build-time Contentstack fetches requiring real credentials. TypeScript and lint both pass on the upgraded tree. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
🔒 Security Scan Results
⏱️ SLA Breach Summary
ℹ️ Vulnerabilities Without Available Fixes (Informational Only)The following vulnerabilities were detected but do not have fixes available (no upgrade or patch). These are excluded from failure thresholds:
✅ BUILD PASSED - All security checks passed |
netrajpatel
approved these changes
Sep 10, 2026
aniket-shikhare-cstk
approved these changes
Sep 10, 2026
aniket-shikhare-cstk
approved these changes
Sep 10, 2026
amit-kanswal-cs
approved these changes
Sep 11, 2026
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.
Problem
Snyk reported 14 security vulnerabilities on
main(2 critical, 7 high, 5 moderate), andnpm auditreported 6 (3 high, 3 moderate). Every one was transitive.The dominant source was a single package:
undici@8.10.0, reached viaisomorphic-dompurify > jsdom, accounted for 11 of the 14 findings including both criticals.Two entries in the existing
overridesblock had also gone stale and were themselves pinning vulnerable versions:qspinned at^6.15.3— inside the affected range>=6.14.2 <=6.15.3sharppinned at0.35.3— inside the affected range<0.35.4(heap overflow)Fix
Direct dependency upgrades (patch/minor only, no majors):
next^16.2.12→^16.3.4react^19.2.8→^19.3.0react-dom^19.2.8→^19.3.0Transitive fixes via
overrides:undici→^8.10.2(new) — clears 11 findings incl. both criticalsjs-yaml→^4.3.2(new) — clears 1 high (CPU DoS, GHSA-2883-xcg3-v3hh, via@eslint/eslintrc)qs^6.15.3→^6.16.0— clears 2 moderatesharp0.35.3→0.35.4— clears 1 highVersion bumped
1.3.3→1.3.4.Deliberately not taken: Snyk's suggested remedy for the
undicicluster was a major bump ofisomorphic-dompurify3.23.0 → 4.0.0. Rejected in favour of a same-majorundicioverride that fixes all 11 identical issues.Build verification — please read
npm run buildfails on this branch, and it fails identically on unmodifiedmain. This was verified by checking out pristineHEADinto a separate directory and runningnpm ciagainst the original lockfile: byte-identical error,Error: API key for Stack is requiredatlib/contentstack.ts:8, followed byFailed to collect page data for /.The cause is environmental, not a dependency regression: this SSG kickstart fetches Contentstack content at build time, so
next buildrequires valid Stack credentials, which are not available in the environment this ran in.AGENTS.mdin this repo states its CI "focuses on security and policy, not app build verification" — there is no build workflow, because it cannot run without a stack.Everything a dependency upgrade could plausibly break did pass on the upgraded tree: TypeScript compiled successfully in 90s, and
npm run lintpassed clean. The failure occurs strictly after compilation, at the data-fetch step.Please run
npm run buildwith valid Contentstack credentials before merging.Remaining findings (not addressable here)
@img/sharp-libvips-*@1.3.2vianext > sharp. License metadata only; no non-LGPL alternative exists.isomorphic-dompurifystays^3.23.0(latest is 4.2.0, a major). No outstanding vulnerability, but 3.23.0 is deprecated upstream and raised its Node floor to^22.22.2without a major bump, so it emitsEBADENGINEon Node 22.21.x.typescriptstays^5(latest is 7.x, a major). Not vulnerable.snyk code test(SAST) returned findings for files that do not exist in this repo (src/main.ts, a knex service spec, a DB migration) — this repo contains onlylib/andpages/. This looks like a server-side project-mapping collision and was not actioned; treat that SAST output as unreliable for this repo.🤖 Generated with Claude Code