Skip to content

fix(docs): reformat breadcrumbs#7510

Merged
mhartington merged 4 commits intomainfrom
fix/search-breadcrumbs
Feb 17, 2026
Merged

fix(docs): reformat breadcrumbs#7510
mhartington merged 4 commits intomainfrom
fix/search-breadcrumbs

Conversation

@mhartington
Copy link
Member

@mhartington mhartington commented Feb 17, 2026

Summary by CodeRabbit

  • Bug Fixes

    • Breadcrumbs now reflect actual URL segments—root or empty paths no longer insert a default "Docs" entry, improving navigation accuracy.
  • Chores

    • Minor formatting change to the docs deployment configuration file (trailing newline added) with no functional impact on redirects.

@vercel
Copy link

vercel bot commented Feb 17, 2026

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Feb 17, 2026 3:31pm

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 17, 2026

No actionable comments were generated in the recent review. 🎉


Walkthrough

Updated breadcrumb logic in apps/docs/src/app/api/search/route.ts to stop returning a default ['Docs'] for empty/root paths and instead return [] when no segments exist; added a trailing newline at end of file in apps/docs/vercel.json (no functional changes).

Changes

Cohort / File(s) Summary
Breadcrumb Generation Logic
apps/docs/src/app/api/search/route.ts
Changed getBreadcrumbsFromUrl to no longer default to ['Docs'] for empty or root URLs; it now returns [] when no path segments remain. Updated doc comment to reflect actual breadcrumb formation.
Repository Metadata / EOF
apps/docs/vercel.json
Added a trailing newline at end of file only; no changes to redirects or JSON content.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ 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 main change: updating breadcrumb formatting logic in the search route to return empty arrays instead of defaulting to ['Docs'].

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


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 and usage tips.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 17, 2026

🍈 Lychee Link Check Report

3660 links: ✅ 2975 OK | 🚫 19 errors | 🔀 0 redirects | 👻 664 excluded

❌ Errors

./apps/docs/content/docs.v6/(index)/prisma-orm/add-to-existing-project/mongodb.mdx

./apps/docs/content/docs.v6/(index)/prisma-orm/quickstart/mongodb.mdx

./apps/docs/content/docs.v6/ai/index.mdx

./apps/docs/content/docs.v6/orm/more/comparisons/prisma-and-drizzle.mdx

./apps/docs/content/docs.v6/orm/more/dev-environment/editor-setup.mdx

  • [503] https://fig.io/ | Rejected status code (this depends on your "accept" configuration): Service Unavailable

./apps/docs/content/docs.v6/orm/overview/introduction/should-you-use-prisma.mdx

./apps/docs/content/docs.v6/orm/overview/prisma-in-your-stack/graphql.mdx

./apps/docs/content/docs.v6/orm/prisma-client/setup-and-configuration/databases-connections/index.mdx

./apps/docs/content/docs.v6/orm/prisma-client/testing/integration-testing.mdx

./apps/docs/content/docs.v6/orm/prisma-client/testing/unit-testing.mdx

./apps/docs/content/docs.v6/orm/prisma-schema/overview/generators.mdx

./apps/docs/content/docs/(index)/prisma-orm/add-to-existing-project/mongodb.mdx

./apps/docs/content/docs/(index)/prisma-orm/quickstart/mongodb.mdx

./apps/docs/content/docs/ai/index.mdx

./apps/docs/content/docs/orm/more/comparisons/prisma-and-drizzle.mdx

./apps/docs/content/docs/orm/more/dev-environment/editor-setup.mdx

  • [503] https://fig.io/ | Rejected status code (this depends on your "accept" configuration): Service Unavailable

./apps/docs/content/docs/orm/prisma-client/testing/integration-testing.mdx

./apps/docs/content/docs/orm/prisma-client/testing/unit-testing.mdx

./apps/docs/content/docs/orm/prisma-schema/overview/generators.mdx


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

coderabbitai[bot]
coderabbitai bot previously approved these changes Feb 17, 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: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@apps/docs/vercel.json`:
- Line 289: Remove the duplicate redirect entry for
"/orm/more/upgrade-guides/upgrading-versions/upgrading-to-prisma-7" →
"/docs/guides/upgrade-prisma-orm/v7" (the JSON object shown in the diff) so only
the original mapping remains; also fix the minor formatting nit by removing the
extra space before the comma after the "source" value in that object to match
the file's style.

mhartington and others added 2 commits February 17, 2026 10:25
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
@mhartington mhartington merged commit 0024be4 into main Feb 17, 2026
6 of 10 checks passed
@mhartington mhartington deleted the fix/search-breadcrumbs branch February 17, 2026 15:32
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