Skip to content

fix: sitemap#785

Merged
tannerlinsley merged 1 commit intomainfrom
sitemap-fix
Mar 26, 2026
Merged

fix: sitemap#785
tannerlinsley merged 1 commit intomainfrom
sitemap-fix

Conversation

@LadyBluenotes
Copy link
Copy Markdown
Member

@LadyBluenotes LadyBluenotes commented Mar 26, 2026

simplify sitemap generation by removing unused route handling and enhancing high-value page entries

Summary by CodeRabbit

  • Bug Fixes

    • Updated crawler directives to allow search engine access to API and OAuth endpoints.
  • Refactor

    • Simplified canonical URL path normalization to streamline URL handling.

…te handling and enhancing high-value page entries
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 26, 2026

Deploy Preview for tanstack ready!

Name Link
🔨 Latest commit 09e3183
🔍 Latest deploy log https://app.netlify.com/projects/tanstack/deploys/69c4b65ea165d60008dfcb35
😎 Deploy Preview https://deploy-preview-785--tanstack.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 38 (🔴 down 16 from production)
Accessibility: 90 (no change from production)
Best Practices: 83 (🔴 down 9 from production)
SEO: 97 (no change from production)
PWA: 70 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: d40dd9b6-3b5a-42b5-a3d7-70691c2fda53

📥 Commits

Reviewing files that changed from the base of the PR and between 333b238 and 09e3183.

📒 Files selected for processing (2)
  • src/utils/seo.ts
  • src/utils/sitemap.ts
💤 Files with no reviewable changes (2)
  • src/utils/seo.ts
  • src/utils/sitemap.ts

📝 Walkthrough

Walkthrough

Simplified SEO utility functions by removing library canonicalization logic from the SEO module and removing API/OAuth path disallow rules from robots.txt generation. The changes result in straightforward path normalization and permissive crawl directives.

Changes

Cohort / File(s) Summary
SEO & Sitemap Utilities
src/utils/seo.ts, src/utils/sitemap.ts
Removed library path transformation logic and API/OAuth disallow rules. getCanonicalPath now returns normalized input paths directly without version rewriting; generateRobotsTxt no longer restricts /api/ and /oauth/ endpoints.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 The paths grow simpler, straight and true,
No winding canonicals to pursue,
The robots now hop without restraint,
Through APIs freely—what a quaint
New garden to explore anew! 🌱

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix: sitemap' is directly related to the changeset, which removes unused route handling (disallow rules) from robots.txt and removes canonical-path transformation logic in SEO utilities related to sitemap generation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch sitemap-fix

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.

@tannerlinsley tannerlinsley merged commit e71479b into main Mar 26, 2026
8 checks passed
@tannerlinsley tannerlinsley deleted the sitemap-fix branch March 26, 2026 04:37
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