Skip to content

chore(deps): bump fumadocs-core and fumadocs-ui to 16.15.9 - #1906

Open
clay-good wants to merge 1 commit into
mainfrom
chore/website-fumadocs-16.15.9
Open

clay-good wants to merge 1 commit into
mainfrom
chore/website-fumadocs-16.15.9

Conversation

@clay-good

@clay-good clay-good commented Sep 17, 2026

Copy link
Copy Markdown
Collaborator

Status: LGTM, ready for review. Supersedes Dependabot #1825.

What was wrong
#1825 bumps fumadocs-core and fumadocs-ui from 16.15.5 to 16.15.9, but its Cloudflare Pages build fails. In fumadocs 16.15.9, llms(source).index() returns a Promise<string> (that release added async renderPage support to llms()), so website/app/llms.txt/route.ts stopped type-checking:

app/llms.txt/route.ts(7,23): error TS2345: Argument of type 'Promise<string>' is not assignable to parameter of type 'BodyInit | null | undefined'.

The fix can't be pushed to the Dependabot branch: a manual push makes Dependabot stop maintaining the PR.

What this does

  • Makes the same two version bumps, on current main. The lockfile was regenerated with the repo's pnpm 10.34.5 and resolves to 16.15.11, the latest in range. Transitive changes come from fumadocs itself: cnfast is replaced by cn, and lucide-react 1.47.0 is added as a fumadocs-ui dependency. The top-level lucide-react stays at 1.34.0.
  • llms.txt route: GET is now async and awaits index().

Proof it works

Notes / nits

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Bug Fixes

    • Improved reliability when retrieving the site’s AI-readable documentation index.
  • Improvements

    • Updated documentation site components to incorporate the latest maintenance and compatibility improvements.

Carries dependabot #1825 onto current main. fumadocs 16.15.9 made
llms(source).index() return a Promise, which broke the website type check
(the Cloudflare Pages failure on #1825); the llms.txt route now awaits it.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 17, 2026

Copy link
Copy Markdown

Deploying openspec-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: b4bec44
Status: ✅  Deploy successful!
Preview URL: https://17c7cedd.openspec-docs.pages.dev
Branch Preview URL: https://chore-website-fumadocs-16-15.openspec-docs.pages.dev

View logs

@clay-good
clay-good requested a review from a team as a code owner September 17, 2026 16:05
@clay-good
clay-good requested review from TabishB and removed request for a team September 17, 2026 16:05
@coderabbitai

coderabbitai Bot commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: c1975a98-a05b-4f20-af16-07451dcf85fe

📥 Commits

Reviewing files that changed from the base of the PR and between bae58cf and b4bec44.

⛔ Files ignored due to path filters (1)
  • website/pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (2)
  • website/app/llms.txt/route.ts
  • website/package.json

Included review availability: Your plan provides up to 10 included reviews per hour; 8 remain after this review.


📝 Walkthrough

Walkthrough

Changes

Website LLM route update

Layer / File(s) Summary
Fumadocs dependency update
website/package.json
Updates fumadocs-core and fumadocs-ui from ^16.15.5 to ^16.15.9.
Async LLM route handling
website/app/llms.txt/route.ts
Makes GET asynchronous and awaits llms(source).index() before returning a Response.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to b4bec

The documentation route remains compatible with the upgraded Fumadocs packages and has no identified user-impacting regression.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. (1 skipped: 1 … Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the primary change: upgrading fumadocs-core and fumadocs-ui to version 16.15.9. It is concise and related to the pull request objectives.
Linked Issues check ✅ Passed The pull request satisfies the coding requirements in [#1825]. website/package.json upgrades fumadocs-core and fumadocs-ui from ^16.15.5 to ^16.15.9. The llms.txt route awaits the promise …
Out of Scope Changes check ✅ Passed The changes stay within [#1825]. The dependency changes implement the requested Fumadocs upgrade. The asynchronous llms.txt route change adapts website code to the upgraded API. No unrelated source …
Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. (1 skipped: 1 unsupported.)

  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@openspec-cloud

Copy link
Copy Markdown
Contributor

No PR-relevant drift confirmed.

AI-generated · A citation proves the line exists, not that it makes the case — verify before acting.
Checked the 0 requirements selected for this PR at b4bec44 (255 total).
This is not a full-repository clean result; see the check for coverage and any broader findings.
View results · Click Refresh, then Scan again in the check. Or comment /openspec-cloud.

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.

1 participant