Skip to content

Clear the npm-audit advisories and align the dependency posture with Docsy - #518

Merged
chalin merged 3 commits into
docsy:mainfrom
chalin:chalin-m24-npm-audit-alignment-2026-1006
Oct 6, 2026
Merged

chalin merged 3 commits into
docsy:mainfrom
chalin:chalin-m24-npm-audit-alignment-2026-1006

Conversation

@chalin

@chalin chalin commented Oct 6, 2026 •

Copy link
Copy Markdown
Collaborator
  • Scope:
    • clears every advisory npm audit reports for the committed lock (0 remain)
    • aligns the override spec, advisory-package versions and npm floor with Docsy
  • Out of scope:
    • Node, Hugo, theme and Renovate bumps: not applicable to this change
    • an audit gate for this repo (the theme's tests/npm-audit.test.mjs idiom): tracked on the Docsy supply-chain task board
  • Takes the postcss-cli major: the only route off chokidar 3, the braces chain's root
    • the theme has no postcss-cli to mirror, so the version stands on its own age (past the cooldown)
    • the RTL build output (rtlcss + autoprefixer) is byte-identical before and after
  • Drops the fsevents allowScripts row: the package left the lock with chokidar 3, and allowScripts stays exact to the locked install-script inventory
  • Takes source-map-js on a one-time release-age exception (inside the 7-day min-release-age): the tarball, identical by integrity, Docsy vetted byte-for-byte against upstream in Clear npm-audit advisories for katex, smol-toml, and source-map-js docsy#2849
  • Exposure: all dev-only (PostCSS/Autoprefixer build tooling, hugo-extended's installer, sass's optional fallback); nothing shipped

chalin added 3 commits October 6, 2026 15:48
- Widens the adm-zip override to Docsy's caret form so later patch fixes resolve without a manifest edit; the lock keeps resolving the same version
- Raises the npm floor to where allowScripts support landed, matching the theme and docsy-starter
- Refreshes source-map-js within sass's declared range, on a one-time release-age exception: the same tarball Docsy vetted against upstream and merged earlier today
- Takes the major because chokidar 3, the braces path, leaves only with it; the theme has no postcss-cli, so there is no spec to mirror
- Drops the fsevents deny row: the package left the lock with chokidar 3, and allowScripts stays exact to the locked install-script inventory
@chalin
chalin requested a review from a team October 6, 2026 20:16
@netlify

netlify Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for goldydocs ready!

Name Link
🔨 Latest commit 295879f
🔍 Latest deploy log https://app.netlify.com/projects/goldydocs/deploys/6ac55719106cb30008ebbd38
😎 Deploy Preview https://deploy-preview-518--goldydocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@chalin
chalin merged commit c9fb2ff into docsy:main Oct 6, 2026
8 checks passed
@chalin
chalin deleted the chalin-m24-npm-audit-alignment-2026-1006 branch October 6, 2026 22:30
Sign up for free to 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.

1 participant