fix(docs): patch mermaid, fast-uri, postcss advisories#229
Conversation
Bump mermaid to 11.15.0 and pin transitive fast-uri >=3.1.2 and postcss >=8.5.10 to clear 7 Dependabot alerts in the docs site.
📝 WalkthroughWalkthroughThe PR updates the ChangesDependency Version Updates
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Suggested labels
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
docs/package.json (1)
40-45: ⚡ Quick winConsider using caret ranges for more predictable dependency resolution.
The current
>=ranges ensure that any patched version is used, which is good for security. However, using caret ranges (e.g.,^3.1.2,^8.5.10) would prevent potential breaking changes from future major versions while still ensuring the security fixes are applied.♻️ Suggested refinement
"pnpm": { "overrides": { - "fast-uri@<3.1.2": ">=3.1.2", - "postcss@<8.5.10": ">=8.5.10" + "fast-uri@<3.1.2": "^3.1.2", + "postcss@<8.5.10": "^8.5.10" } }This approach ensures:
- Security patches are applied (versions >= 3.1.2 and >= 8.5.10)
- Future major version breaking changes won't be automatically pulled in
- More predictable behavior during future dependency updates
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@docs/package.json` around lines 40 - 45, Update the pnpm overrides to use caret ranges instead of open-ended greater-than ranges: in the pnpm.overrides object replace the ">=3.1.2" value for the "fast-uri@<3.1.2" override with "^3.1.2" and replace the ">=8.5.10" value for the "postcss@<8.5.10" override with "^8.5.10" so the overrides still apply security patches but avoid automatically pulling future major versions.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@docs/package.json`:
- Around line 40-45: Update the pnpm overrides to use caret ranges instead of
open-ended greater-than ranges: in the pnpm.overrides object replace the
">=3.1.2" value for the "fast-uri@<3.1.2" override with "^3.1.2" and replace the
">=8.5.10" value for the "postcss@<8.5.10" override with "^8.5.10" so the
overrides still apply security patches but avoid automatically pulling future
major versions.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 5b97e6dc-1872-4178-a9f0-076accb73ff6
⛔ Files ignored due to path filters (1)
docs/pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (1)
docs/package.json
Summary
Resolves the 7 open Dependabot alerts, all in the docs site's dependency tree (
docs/pnpm-lock.yaml). The dashboard, Rust core, and Python package are unaffected.mermaid(direct)fast-uri(transitive)postcss(transitive)Changes
mermaiddependency range to^11.15.0.pnpm.overrides(fast-uri@<3.1.2→>=3.1.2,postcss@<8.5.10→>=8.5.10) to lift the vulnerable transitive copies while leaving already-patched instances untouched.docs/pnpm-lock.yaml; no vulnerable versions remain.Verification
pnpm --dir docs types:check,pnpm --dir docs lint, andpnpm --dir docs build(full static export, mermaid diagrams included) all pass.Summary by CodeRabbit