chore(docs) - Remove dupe entries in sitemap by kodster28 · Pull Request #52083 · supabase/supabase · GitHub
Skip to content

chore(docs) - Remove dupe entries in sitemap - #52083

Open
kodster28 wants to merge 3 commits into
masterfrom
docs-sitemap-dupes
Open

chore(docs) - Remove dupe entries in sitemap#52083
kodster28 wants to merge 3 commits into
masterfrom
docs-sitemap-dupes

Conversation

@kodster28

@kodster28 kodster28 commented Oct 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Problem

Dupe entries for /docs/reference/cli pages due to I guess them getting grabbed in the Reference Sitemap generation as well?

Solution

Removes CLI sitemap generation script + reference from the sitemap generation.

Preview links

Production

  • 2898 entries
  • 2773 uniques
image image

Preview

  • 2773 entries
  • 2773 uniques
image image

@vercel

vercel Bot commented Oct 21, 2026 •
edited
Loading

Copy link
Copy Markdown

@coderabbitai

coderabbitai Bot commented Oct 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: supabase/supabase/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: b4e7fb8d-c727-4c21-827a-80937b79a7a4
📥 Commits

Reviewing files that changed from the base of the PR and between 4c4471d and cb39a52.

📒 Files selected for processing: 2
  • apps/docs/internals/files/cli.ts
  • apps/docs/internals/generate-sitemap.ts
💤 Files with no reviewable changes: 1
  • apps/docs/internals/files/cli.ts

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

📝 Walkthrough

The CLI page-generation module was removed. Sitemap generation no longer imports or includes CLI pages. The sitemap write remains synchronous.

Changes: CLI sitemap exclusion

Layer / File(s) Summary
Remove CLI pages from sitemap generation
apps/docs/internals/files/cli.ts, apps/docs/internals/generate-sitemap.ts		 The CLI page generator and its sitemap integration were removed. The synchronous-write suppression comment was also removed; sitemap writing remains synchronous.

Priority: ➖ Normal

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Suggested reviewers: alexisintech

Merge Risk: ⚪ Minimal · up to cb39a

No actionable merge-blocking risk was identified in the CLI sitemap cleanup; it is ready for normal checks.

Pre-merge checks: Passed 4 · Failed 1

Failed 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. Write docstrings for the functions missing them to satisfy the coverage threshold.
Passed Passed checks: 4 passed
Check name Status Explanation
Title check Passed The title clearly identifies the main change: removing duplicate sitemap entries from the documentation.
Description check Passed The description includes complete Problem and Solution sections, plus production and preview sitemap evidence. It omits the template's Review instructions and Checklist sections, but the core change a…
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.
Full details: Description check

Explanation

The description includes complete Problem and Solution sections, plus production and preview sitemap evidence. It omits the template's Review instructions and Checklist sections, but the core change and validation details are clear.

✨ 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
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Oct 21, 2026
@kodster28
kodster28 marked this pull request as ready for review October 21, 2026 13:43
@kodster28
kodster28 requested a review from a team as a code owner October 21, 2026 13:43

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@kodster28