Skip to content

Conversation

@aidankmcalister
Copy link
Member

@aidankmcalister aidankmcalister commented Feb 12, 2026

Summary by CodeRabbit

  • New Features

    • Documentation pages now prepend generated frontmatter (title, description, URL) to content for more consistent rendering.
  • Improvements

    • Docs build now merges content from multiple source versions with deduplication and leaf-page filtering, improving coverage and backward compatibility.
    • Static page parameter generation is now deterministic and returned synchronously, streamlining the build process.

@vercel
Copy link

vercel bot commented Feb 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Feb 12, 2026 7:29pm

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 12, 2026

Walkthrough

Route updated to resolve pages from primary source with fallback to sourceV6, prepend generated Markdown frontmatter (title, description, url) to LLM content, and replace async static-param generation with a deduplicating, combined synchronous array of slug params from both v7 and v6 sources.

Changes

Cohort / File(s) Summary
LLM Route Handler Updates
apps/docs/src/app/llms.mdx/[[...slug]]/route.ts
Added public import of sourceV6; use source.getPage(slug) with fallback to sourceV6.getPage(slug). Prepend generated YAML frontmatter (title, description, url) to content from getLLMText(page) via escapeYaml. Rewrote generateStaticParams() to synchronously return a deduplicated array merging slugs from both v7 and v6 sources and filter out non-leaf slugs; changed return type from Promise<Array<{ slug?: string[] }>> to Array<{ slug?: string[] }>.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Merge Conflict Detection ⚠️ Warning ❌ Merge conflicts detected (18 files):

⚔️ apps/docs/content/docs.v6/orm/overview/prisma-in-your-stack/is-prisma-an-orm.mdx (content)
⚔️ apps/docs/content/docs.v6/orm/prisma-client/deployment/traditional/deploy-to-heroku.mdx (content)
⚔️ apps/docs/content/docs.v6/orm/prisma-client/deployment/traditional/deploy-to-koyeb.mdx (content)
⚔️ apps/docs/content/docs.v6/orm/prisma-client/observability-and-logging/opentelemetry-tracing.mdx (content)
⚔️ apps/docs/content/docs.v6/orm/prisma-client/queries/pagination.mdx (content)
⚔️ apps/docs/content/docs.v6/orm/prisma-client/queries/relation-queries.mdx (content)
⚔️ apps/docs/content/docs.v6/postgres/database/caching.mdx (content)
⚔️ apps/docs/content/docs.v6/postgres/database/direct-connections.mdx (content)
⚔️ apps/docs/content/docs.v6/postgres/database/prisma-studio/embedding-studio.mdx (content)
⚔️ apps/docs/content/docs.v6/postgres/integrations/vscode.mdx (content)
⚔️ apps/docs/content/docs/guides/postgres/vscode.mdx (content)
⚔️ apps/docs/content/docs/orm/prisma-client/deployment/traditional/deploy-to-heroku.mdx (content)
⚔️ apps/docs/content/docs/orm/prisma-client/deployment/traditional/deploy-to-koyeb.mdx (content)
⚔️ apps/docs/content/docs/orm/prisma-client/observability-and-logging/opentelemetry-tracing.mdx (content)
⚔️ apps/docs/content/docs/orm/prisma-client/queries/relation-queries.mdx (content)
⚔️ apps/docs/content/docs/postgres/database/caching.mdx (content)
⚔️ apps/docs/content/docs/postgres/database/direct-connections.mdx (content)
⚔️ apps/docs/content/docs/studio/integrations/embedding.mdx (content)

These conflicts must be resolved before merging into main.
Resolve conflicts locally and push changes to this branch.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the primary change: adding frontmatter generation to the llms.mdx route handler. It's specific, concise, and directly reflects the main objective.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


No actionable comments were generated in the recent review. 🎉


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

@github-actions
Copy link
Contributor

github-actions bot commented Feb 12, 2026

🍈 Lychee Link Check Report

3660 links: ✅ 2992 OK | 🚫 2 errors | 🔀 0 redirects | 👻 664 excluded

❌ Errors

./apps/docs/content/docs.v6/guides/tanstack-start.mdx

./apps/docs/content/docs/guides/frameworks/tanstack-start.mdx


Full Statistics Table
Status Count
✅ Successful 2992
🔀 Redirected 0
👻 Excluded 664
🚫 Errors 2
⛔ Unsupported 2
⏳ Timeouts 0
❓ Unknown 0

coderabbitai[bot]
coderabbitai bot previously requested changes Feb 12, 2026
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Fix all issues with AI agents
In `@apps/docs/src/app/llms.mdx/`[[...slug]]/route.ts:
- Around line 12-19: The frontmatter string built in route.ts interpolates raw
values (page.data.title, page.data.description, page.url) which can break YAML;
update the frontmatter construction (the frontmatter constant) to escape/quote
these values safely before insertion—e.g., implement a helper (escapeYaml or
quoteYaml) and apply it to page.data.title, page.data.description and page.url
(or render multi-line description as a YAML block scalar) so special characters
like colons, quotes and newlines are handled correctly when building the
frontmatter string returned by getLLMText/page route.
- Around line 31-41: The current deduplication only removes prefix routes via
isPrefix but doesn't remove identical slugs coming from v7Params and v6Params;
update the params pipeline that builds allParams/allSlugs to first deduplicate
by the full slug path (e.g., use a Set of joined segments or JSON-stringify the
slug arrays) so generateParams outputs unique slug arrays before applying the
isPrefix filter; update references to v7Params, v6Params, allParams, allSlugs
and keep the existing isPrefix/prefix-filter logic but operate on the
deduplicated list returned from generateStaticParams.

@aidankmcalister aidankmcalister merged commit d347057 into main Feb 12, 2026
5 of 10 checks passed
@aidankmcalister aidankmcalister deleted the feat/frontmatter-to-llmsmdx branch February 12, 2026 19:28
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.

2 participants