fix(deps): drop unused remark-mdx-frontmatter and bump browserslist override - #2204
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Essentials Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (4)
💤 Files with no reviewable changes (2)
Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review. 📝 WalkthroughWalkthroughThe PR removes the unused ChangesMDX dependency cleanup
Browserslist version override
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to This removes an unused MDX dependency and constrains browserslist to the intended patched version, with no current merge-readiness risk identified. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Closes three high-severity Dependabot alerts: 1315 (
toml, prototype pollution, CVSS 8.2) and 1316 / 1317 (browserslist, uncaught crash plus prototype write, and unbounded cache growth, CVSS 7.5 each).tomlreached us only throughremark-mdx-frontmatter, whichpackages/cli/src/cli/loaders/mdx.tsimported but never used — the two.use(remarkMdxFrontmatter)calls were dropped in #656 and the import line was left behind. Removing the import and the dependency takestomlout of the tree entirely, including for anyone runningnpm i lingo.dev, which a pnpm override cannot do.browserslistis deep transitive under Babel, autoprefixer and cssnano, so it gets an override to>=4.28.8 <5; every dependent already asks for^4.2x, so this is a patch bump inside ranges they already declare.Full monorepo build passes (19/19 tasks, including the Next 16 and Vite demos) and so do the tests (20/20 tasks — 955 CLI, 244 compiler).
pnpm auditreports no remaining high-severity findings.Summary by CodeRabbit
Bug Fixes
Chores