Repository navigation
Bump transitive npm deps to close Dependabot alerts #81–#83, #86–#89 - #134
Conversation
There was a problem hiding this comment.
🟡 Changes recommended
The lockfile refresh removes required libc platform metadata from native resolver bindings.
0 open findings
What changed in this PR
Updates transitive npm dependencies to remediate Dependabot alerts.
Changes:
- Pins patched dependency versions through npm overrides.
- Refreshes resolved versions and integrity hashes.
| File | Description |
|---|---|
| package.json | Updates transitive dependency overrides. |
| package-lock.json | Records patched dependency resolutions. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Optional linux libc fields were dropped from native optional packages during the lockfile refresh; restore them from main so platform resolution metadata stays intact.
62f8304 to
a9116dc
Compare
|
comphy-bot review status for Review completed for this exact pull request head. |
comphy-bot
left a comment
There was a problem hiding this comment.
Review: dependency security refresh
Reviewed a9116dcacb26ad6bc7b4d750bf82e33d1c728777 (+41/−39 across two files).
Findings
No verified P0/P1 findings in the supplied changes. The manifest overrides and corresponding lockfile version updates are consistent; the Linux native-binding libc selectors remain present. The earlier peer concern about removed platform metadata does not apply to this head.
Verification
- Inspected the manifest changes and dependency-resolution metadata, including the KaTeX/Commander update. The new Commander dependency declares Node
>=22.12.0; environments installing these development dependencies should satisfy that requirement. - Exact-head checks captured for this review: build, markdown-quality and dependency/security separation passed. Other reviewer checks were still pending; these are not represented as completed checks.
- Patch conflict-marker and trailing-whitespace checks passed.
- No PR code, installation scripts or local test commands were executed. The PR author's test results are reported evidence, not independently rerun verification.
No blocking issue identified; approved for this exact head.
Summary
Closes open Dependabot alerts on npm development dependencies in
package-lock.json:source-map-jsnpm update; parents already allow^1.2.1)postcss-selector-parsernpm update; stylelint allows^7.1.0)brace-expansionoverridessmol-toml~1.7.1→~1.9.1(parent pins exact versions)katex~0.18.2(micromark-extension-mathstill asks for^0.16.0)katexis only a transitive ofmarkdownlint-cli2→micromark-extension-math. It is not vendored underassets/or loaded as a site runtime.No unrelated major upgrades. Remaining
npm auditfindings (e.g.colord,fast-uri, picomatch-chain noise) are out of scope for this PR.Follow-ups in this PR
libcmetadata thatnpmhad stripped from the lockfile (Copilot note).package.json+package-lock.json).Testing
./scripts/lint-check.sh— passed (3 pre-existing unused-var warnings)npm test -- --runInBand— 20 suites, 166 tests passed./scripts/build.sh— Jekyll production build succeeded (Ruby 3.4.10 / Bundler 2.5.23)Notes
gh api .../dependabot/alertsreturned 403 for the Cursor integration token; patched versions taken fromnpm audit/ GHSA advisories matching the alert numbers above.Summary by CodeRabbit