fix(seo): align /tools/{provider} titles with H1s - #481
Draft
JayZeeDesign wants to merge 1 commit into
Draft
JayZeeDesign wants to merge 1 commit into
JayZeeDesign wants to merge 1 commit into
Conversation
- Remove 'API pricing:' lead-in from metered/mixed provider page titles
- Metered titles now: '{Provider}: {n} tools from {price} | treg.to'
- Mixed titles now: '{Provider}: {n} tools, platform or your own key | treg.to'
- Own-account titles unchanged: '{Provider}: connect your own account | treg.to'
- Update seo.md docs to reflect Title=H1 rule
- Rename test_provider_title_leads_with_pricing -> test_provider_title_matches_h1
Co-authored-by: Jason Zhou <JayZeeDesign@users.noreply.github.com>
This was referenced Sep 16, 2026
Draft
|
Folded into #551. |
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
2 participants
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.
What this does
Removes leftover formulaic
API pricing:chrome from/tools/{provider}page titles so Title matches H1.Before:
DataForSEO API pricing: $0.00012/call, platform + BYOK | treg.toHunter API pricing: $X, platform + BYOK | treg.toAfter:
DataForSEO: 217 tools from $0.00012/call | treg.to(matches H1)Hunter: 35 tools, platform or your own key | treg.to(matches H1){Provider}: connect your own account | treg.toThe seo.md fragment documented Title and H1 as matching, but the code still built titles with the
API pricing:lead-in that H1s never used. This fix removes that leftover chrome.How it was tested
uv run --frozen python -m pytest tests/test_seo.py tests/test_provider_pages.py -v # 77 passedVerified rendered titles on
/tools/hunterand/tools/dataforseomatch their H1s.Checklist
uv run --with pytest-xdist pytest -n auto -qpasses locallydocs/context/fragment (if a subsystem changed).envvalues)