website - chore: upgrade docula to 2 (breaking)#1676
Conversation
docula ^1.14.0 -> ^2.1.0 (major; @cacheable/website devDep, docs-site generator). No config or code changes required: the existing site/docula.config.ts schema is accepted by docula 2, and `pnpm website:build` completes successfully (pages, docs, sitemap, llms.txt, and search index all built). Website-only; no impact on the published @cacheable/* packages. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01P1TBuMiXotBjpNHYcbwvqf
There was a problem hiding this comment.
Code Review
This pull request updates the devDependency docula from ^1.14.0 to ^2.1.0 in packages/website/package.json and updates the lockfile pnpm-lock.yaml to reflect this upgrade along with several other dependency updates, including zod, react, and html-react-parser. No review comments were provided, and I have no feedback to offer.
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1676 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 29 29
Lines 3504 3504
Branches 808 795 -13
=========================================
Hits 3504 3504 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Re: Socket "Obfuscated code" alert on
The "Socket Security: Pull Request Alerts" check is a Warn-level advisory. As before, I've not self-issued a Generated by Claude Code |
Please check if the PR fulfills these requirements
What kind of change does this PR introduce?
Maintenance (
chore) — major upgrade ofdocula, the@cacheable/websitedocs-site generator. Website-onlydevDependency; no impact on the published@cacheable/*packages. Marked(breaking)because the major version changed, though no migration was needed (see below).Versions
docula^1.14.0→^2.1.0(@cacheable/website, dev)Breaking notes
site/docula.config.ts(DoculaOptions:githubPath,siteTitle,siteDescription,siteUrl,themeMode) is accepted unchanged by docula 2.Verification
pnpm website:buildcompletes successfully on docula 2 — builds the pages, docs for all packages,sitemap.xml,robots.txt,feed.xml,llms.txt/llms-full.txt, and the search index (✔ Build completed).pnpm build+node scripts/test-build.mjspass for all published packages, andpnpm teststays at 100% coverage — confirming the shared lockfile change doesn't affect anything outside the website.Generated by Claude Code