root - chore: upgrade docula - #97
Conversation
- docula ^2.0.0 → ^2.1.0 pnpm build and pnpm test:ci pass (612 tests, 100% statement/line coverage); docs site builds cleanly via pnpm website:build. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XqhnMb2Khr8eYnWmYbAmtd
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the docula dependency from version 2.0.0 to 2.1.0 in package.json and updates the corresponding lockfile pnpm-lock.yaml with various dependency upgrades, including @ai-sdk packages, @Cacheable packages, and others. There are no review comments, and I have no additional feedback to provide.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #97 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 8 8
Lines 1399 1399
Branches 303 303
=========================================
Hits 1399 1399 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 477160a933
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| engines: {node: ^22.18.0 || >=24.0.0} | ||
| docula@2.1.0: | ||
| resolution: {integrity: sha512-lNsIM9ZaLU7+r7sHpHnBgjt6pOSl2Yc2QLbBrCvjLEn8SIlvnhLiXyQ7o4rmw+36CvSB/ZkF2QXABOpcyAM8nQ==} | ||
| engines: {node: ^22.19.0 || >=24.0.0} |
There was a problem hiding this comment.
This upgrade now installs docula@2.1.0, whose lockfile entry requires node: ^22.19.0 || >=24.0.0, while the root package still advertises "node": ">=22". In a Node 22.18 environment that satisfies package.json and the previous Docula 2.0.0 lock entry, docs commands such as pnpm website:build now run outside the upgraded tool's supported engine range (and undici@8.4.1 in this same lockfile also requires >=22.19.0). Please either keep Docula on a version supporting the advertised range or update the root engine/CI guidance to >=22.19.
Useful? React with 👍 / 👎.
Align engines.node with the toolchain after the docula 2.1.0 upgrade (#97). docula 2.1.0 (and its transitive undici) require Node ^22.19.0 || >=24; .nvmrc (24) and CI (node 22/24/26, where 22.x is >=22.19) already satisfy this. - engines.node: >=22 -> >=22.19.0 pnpm build and pnpm test:ci pass (612 tests, 100% statement/line coverage). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XqhnMb2Khr8eYnWmYbAmtd
1 participant
Summary
Upgrade the documentation site generator.
Versions
docula2.0.0→2.1.0Tests
pnpm test:cipasses (612 tests, 100% statement/line coverage)pnpm buildpassespnpm website:buildbuilds the docs site cleanly🤖 Generated with Claude Code
https://claude.ai/code/session_01XqhnMb2Khr8eYnWmYbAmtd
Generated by Claude Code