Skip to content

Fix sitemap indexing consistency - #111

Merged
profilesearch merged 1 commit into
mainfrom
fix/search-console-sitemap-hygiene
Jul 19, 2026
Merged

Fix sitemap indexing consistency#111
profilesearch merged 1 commit into
mainfrom
fix/search-console-sitemap-hygiene

Conversation

@profilesearch

Copy link
Copy Markdown
Owner

Summary

  • remove one duplicate URL from the public sitemap
  • remove an intentionally noindex article from the sitemap while leaving the page unchanged
  • add non-vacuous regression coverage for unique sitemap URLs and sitemap/indexing consistency
  • parse the sitemap as XML and robots metadata as HTML instead of relying on regular expressions

Search Console finding

The current robots policy only disallows /cgi-bin/. A read-only Search Console URL Inspection audit covered every unique live sitemap URL and found zero sitemapped URLs blocked by robots.txt. The email alert is therefore consistent with intentional blocking of invalid /cgi-bin/ paths, not a production-content crawl defect. robots.txt is intentionally unchanged.

Test plan

  • Targeted RED/GREEN regression tests
  • 390 backend tests passed
  • 81 browser tests passed; 5 skipped
  • Security static checks passed
  • Public-shell checks passed
  • Performance budgets passed
  • Dependency audit found no known vulnerabilities
  • Sitemap XML parsed successfully
  • Local sitemap has 137 entries and 137 unique URLs
  • git diff --check passed

Risk

Low. The production change only removes two <url> blocks from sitemap.xml: one duplicate and one URL whose HTML already opts out of indexing. No page content, routing, robots policy, API, payments, or user data changes.

@vercel

vercel Bot commented Jul 19, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
gohire-humans Ready Ready Preview, Comment Jul 19, 2026 2:57am

Request Review

@profilesearch
profilesearch merged commit 4583ea1 into main Jul 19, 2026
5 checks passed
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