Skip to content

chore(website): refresh docs dependencies#154

Merged
leehack merged 1 commit into
mainfrom
chore/website-audit-cleanup
May 17, 2026
Merged

chore(website): refresh docs dependencies#154
leehack merged 1 commit into
mainfrom
chore/website-audit-cleanup

Conversation

@leehack

@leehack leehack commented May 17, 2026

Copy link
Copy Markdown
Owner

Summary

  • refresh website Docusaurus packages to 3.10.1
  • add the Mermaid ELK layout peer dependency required by the newer Docusaurus Mermaid theme
  • scope npm overrides for serialize-javascript@7.0.5 to the Docusaurus bundler's webpack plugin subtrees so the website audit no longer resolves the vulnerable 6.x line
  • document the website Node.js floor as Node 20, matching CI docs workflows

Existing issue check

  • No matching open issue/PR found for npm audit, Docusaurus dependency, serialize-javascript, or website dependency cleanup.

Test Plan

Review notes

  • Local audit now reports found 0 vulnerabilities for website.
  • serialize-javascript@7.0.5 is resolved through scoped npm overrides under copy-webpack-plugin and css-minimizer-webpack-plugin.
  • Independent review found no blocking dependency mistakes; the Node 20 engine declaration is included because CI already uses Node 20 for docs workflows.
  • Copilot's initial override-scope comment was fixed by replacing the global override with scoped overrides and the outdated thread was resolved.

Copilot AI review requested due to automatic review settings May 17, 2026 01:57

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the website/ Docusaurus dependency set, adds the required Mermaid ELK layout peer dependency for newer Mermaid theme versions, and introduces an npm override intended to keep serialize-javascript on a non-vulnerable release line while documenting a Node.js 20+ baseline for the docs site.

Changes:

  • Bump Docusaurus packages (@docusaurus/core, @docusaurus/preset-classic, @docusaurus/theme-mermaid) to ^3.10.1.
  • Add @mermaid-js/layout-elk dependency to satisfy newer Mermaid theme requirements.
  • Add npm overrides for serialize-javascript and declare Node >=20 in engines.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread website/package.json Outdated
@codecov-commenter

codecov-commenter commented May 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.50%. Comparing base (8058797) to head (ab53f04).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #154   +/-   ##
=======================================
  Coverage   78.50%   78.50%           
=======================================
  Files          76       76           
  Lines        9885     9885           
=======================================
  Hits         7760     7760           
  Misses       2125     2125           
Flag Coverage Δ
unittests 78.50% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@leehack leehack force-pushed the chore/website-audit-cleanup branch from ea03eb8 to ab53f04 Compare May 17, 2026 02:10
@leehack leehack requested a review from Copilot May 17, 2026 02:32

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 2 changed files in this pull request and generated no new comments.

@leehack leehack merged commit ce004ea into main May 17, 2026
10 checks passed
@leehack leehack deleted the chore/website-audit-cleanup branch May 17, 2026 02:37
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.

3 participants