Repository navigation
chore(deps-dev): update every dev dependency and set the unfixable braces advisory aside - #462
Merged
Merged
Conversation
All 28 dev dependencies are at their latest release, including the eslint-plugin-ava 18 and eslint-plugin-jsdoc 65 majors, which need no config changes here. A lockfile refresh moves smol-toml to 1.9.1, which clears its advisory. package.json declares npm 11.16's allowScripts, denying unrs-resolver's postinstall: it only downloads the native resolver binding when npm hasn't installed it, and npm installs it as an optional dependency. The built dist/ is byte-identical to the published 6.3.0. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NfrA9rAkb8oHNcNHDaPVC4
markdownlint-cli2 0.23.3, its latest release, pins markdown-it 15.0.1 exactly. Two high advisories published on 2026-10-20 cover that version (GHSA-gfr2-rv44-r23c and GHSA-r7fv-28h4-cvq7, both quadratic-time parsing), and 15.0.3 fixes both, so it is forced the same way smol-toml already is. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NfrA9rAkb8oHNcNHDaPVC4
…dvisory npm audit can't set an advisory aside, and braces <= 3.0.3 (GHSA-vfj7-8cjw-p6xm) has no patched release. audit-ci keeps the same high-severity gate and reads its one exception, with the reason and an expiry of 2027-01-20, from audit-ci.jsonc. braces reaches n2words only through ava and markdownlint-cli2, which hand it patterns from this repo's own config, and the package ships no dependencies. The audit now runs through npx before npm ci, as audit-ci recommends, so no dependency's install script runs ahead of it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NfrA9rAkb8oHNcNHDaPVC4
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.
Updates all 28 dev dependencies to their latest release. This supersedes #459.
check-indentation, flags nothing here.allowScripts: npm 11.16 added this field. It only warns today, and npm says a future release will block install scripts that aren't listed. The one dependency with an install script isunrs-resolver(via eslint-plugin-import-x), so it's denied: its postinstall only downloads the native binding when npm hasn't installed it, and npm installs it as an optional dependency.npm cino longer warns.The built
dist/is byte-identical to the published 6.3.0 (all 354 files, diffed against the npm tarball).npm run lint,typecheckandnpm testpass.The security audit
braces <= 3.0.3. 3.0.3 is the latest release and the latest ava and markdownlint-cli2 still depend on it, so no update can clear it. It doesn't apply to n2words:package.jsondeclares none andnpm packhas nothing undernode_modules, so users never install braces.npm audit --omit=devfinds nothing.filesglobs, markdownlint-cli2's config globs, and the staged file paths lint-staged passes to it.audit-ci.jsonc. Once that date passes, the advisory fails the gate again. It runs throughnpxbeforenpm ci, as audit-ci recommends, so no install script runs ahead of the audit. audit-ci is pinned at 7.1.0, but its own dependencies resolve fresh on each run. The contributor checklist inSECURITY.mdnow points at this gate instead of plainnpm audit.Checked against this lockfile: with braces set aside and markdown-it still at 15.0.1, the gate fails on the two markdown-it advisories. With 15.0.3 it passes, and with the expiry moved to yesterday it fails on braces again.
🤖 Generated with Claude Code
https://claude.ai/code/session_01NfrA9rAkb8oHNcNHDaPVC4