Skip to content

Fix review feedback on docs SEO remediation#7833

Open
mhessdev wants to merge 1 commit intocodex/seo-docs-remediationfrom
codex/seo-docs-remediation-followup
Open

Fix review feedback on docs SEO remediation#7833
mhessdev wants to merge 1 commit intocodex/seo-docs-remediationfrom
codex/seo-docs-remediation-followup

Conversation

@mhessdev
Copy link
Copy Markdown
Contributor

@mhessdev mhessdev commented Apr 27, 2026

Summary

  • fix the review feedback on the first docs SEO remediation PR
  • remove the early duplicate /docs/optimize/:path* redirect that shadowed specific optimize recommendation redirects
  • harden the redirect audit path normalization and fix the failing external blog link

Validation

  • ran pnpm --filter docs run audit:redirects
  • verified https://www.prisma.io/blog/serverless-database-drivers-KML1ehXORxZV returns 200

Stack

Summary by CodeRabbit

  • Documentation

    • Enhanced cursor-based pagination documentation to properly handle edge cases when retrieving subsequent pages
    • Updated external reference links in database support section
    • Corrected formatting in ORM reference documentation
  • Chores

    • Improved URL redirect routing and query parameter handling for better accuracy

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 27, 2026

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

Project Deployment Actions Updated (UTC)
blog Ready Ready Preview, Comment Apr 27, 2026 6:18pm
docs Ready Ready Preview, Comment Apr 27, 2026 6:18pm
eclipse Ready Ready Preview, Comment Apr 27, 2026 6:18pm
site Ready Ready Preview, Comment Apr 27, 2026 6:18pm

Request Review

@github-actions
Copy link
Copy Markdown
Contributor

🍈 Lychee Link Check Report

165 links: ✅ 34 OK | 🚫 0 errors | 🔀 4 redirects | 👻 127 excluded

✅ All links are working!


Full Statistics Table
Status Count
✅ Successful 34
🔀 Redirected 4
👻 Excluded 127
🚫 Errors 0
⛔ Unsupported 0
⏳ Timeouts 0
❓ Unknown 0

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 7c6d4aa3-4b0d-4eec-afbe-fe2c9de520da

📥 Commits

Reviewing files that changed from the base of the PR and between 1c8bd82 and 9f17ba1.

📒 Files selected for processing (5)
  • apps/docs/content/docs/orm/more/comparisons/prisma-and-drizzle.mdx
  • apps/docs/content/docs/orm/prisma-client/queries/pagination.mdx
  • apps/docs/content/docs/orm/reference/prisma-client-reference.mdx
  • apps/docs/scripts/audit-redirects.mjs
  • apps/docs/vercel.json
💤 Files with no reviewable changes (1)
  • apps/docs/vercel.json

Walkthrough

This pull request updates documentation pages and scripts in the Prisma docs app. Changes include a URL parameter update, pagination logic guard clause, punctuation cleanup, route normalization enhancement for query/hash handling, and removal of a wildcard redirect rule.

Changes

Cohort / File(s) Summary
Documentation Content
apps/docs/content/docs/orm/more/comparisons/prisma-and-drizzle.mdx, apps/docs/content/docs/orm/prisma-client/queries/pagination.mdx, apps/docs/content/docs/orm/reference/prisma-client-reference.mdx
URL case identifier update, pagination guard clause to handle empty lastPost state, and trailing space removal in reference link text.
Script & Configuration
apps/docs/scripts/audit-redirects.mjs, apps/docs/vercel.json
Route normalization logic now strips query and hash segments before applying trailing-slash rules; wildcard redirect from /docs/optimize/:path* removed from Vercel configuration.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 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%. 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 accurately reflects the main objective: addressing review feedback on docs SEO remediation work, which includes redirect fixes, audit hardening, and link corrections.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ 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.

@argos-ci
Copy link
Copy Markdown

argos-ci Bot commented Apr 27, 2026

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build Status Details Updated (UTC)
default (Inspect) ⚠️ Changes detected (Review) 1 changed Apr 27, 2026, 6:24 PM

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