Fix Builder MP SEO: noindex auth pages, promote bills listing - #92
Merged
Merged
Conversation
Searching "Builder MP" returned /bills/sign-in instead of the listing. The sign-in page had no metadata and inherited the layout default title, while the listing was titled "Home · Builder MP". - noindex sign-in, unauthorized and bill edit pages - Give the bills listing an absolute Builder MP title, description, keywords and WebApplication JSON-LD; rename H1 to lead with Builder MP - Mention Builder MP in layout descriptions - Raise /bills sitemap priority; disallow /bills/api/ in robots
# Conflicts: # src/app/bills/page.tsx # src/app/robots.ts
|
greptile-apps
Bot
dismissed
their stale review
September 21, 2026 17:36
Dismissed because a newer commit was pushed; Greptile will re-review the current head.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Searching "Builder MP" on Google returns
/bills/sign-inranked above the bills listing.Root cause: the sign-in page had no metadata, so it inherited the bills layout default title "Builder MP" verbatim. The listing itself was titled "Home · Builder MP", which Google rewrote to the H1 "45th Canadian Federal Parliament".
Changes
noindex, nofollowon/bills/sign-in,/bills/unauthorizedand/bills/[id]/edit(layouts added where the page is a client component)/billssitemap priority 0.8 → 0.9;/bills/api/added to robots disallowSign-in is intentionally not added to robots disallow: Google must crawl it to see the noindex tag.
Testing
tsc --noEmitand eslint pass/billsand/bills/sign-inin Search Console