Repository navigation
De-template the seven comparison pages and fix their copy slips - #273
Merged
Merged
Conversation
- Each page states the pricing sentence once, in its call-to-action area, with the free tools linked by name instead of a bare "/tools". - The shared "That is the part DebateOwl is built for" sentence becomes one sentence per page; the ticked pick card stays on two pages, and the others get an "If you are… pick…" paragraph, a checklist or a one-line callout. - /best-ai-for-debate gets a one-sentence meta description with quoted "If they say", and the llms.txt blurb quotes it too. - /chatgpt-vs-claude-for-debate's title loses the broken ": And". - The homepage heading "Everything you need to win" becomes "Everything for your next round" (en, es, hu). - scripts/check_templating.py fails when a sentence of more than 12 words is on more than two of the seven pages; npm test runs it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Warning Review limit reachedEnable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. Next included review available in 59 minutes. View limit details
Note 🎁 Summarized by CodeRabbit Free
Comment |
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.
Task 2 of the 29 Oct AEO audit. Stacked on #272, so the base is
seo/aeo-hubs-1029and this diff shows only Task 2.What changed
Compare DebateOwl, ChatGPT, Claude and other AI tools for debate cases, "If they say" replies and timed practice, with facts as of October 2026.That is one sentence and 144 characters, with no "top pick".ChatGPT vs Claude for Debate (2026), and When to Use a Debate Tool | DebateOwl(66 characters before the suffix). og:title and the JSON-LDnamefollow the title. The JSON-LDheadlinemirrors the H1 ("ChatGPT vs Claude for debate prep"), which never contained the broken "And", so it is unchanged.scripts/check_templating.pysplits the visible text of the 7 pages into sentences. It fails if any sentence longer than 12 words appears on more than 2 pages. The pricing sentence, the byline, the disclosure and the "Last updated" lines are exempt.src/test/seo/comparison-templating.test.tsruns it, sonpm test(and CI'snpm teststep) cover it. The generator also refuses a page that shows the pricing sentence more or less than once, or shows a bare "/tools".best-ai-for-debate.test.tsnow expects the new description and the linked pricing sentence, shown exactly once.guide-content.test.tsalready exempted the required pricing sentence. Its allow-list entry now uses the linked wording.Acceptance checks (local build of this branch)
check_templating.pyexists, runs innpm test, and passes. Before this change it flagged exactly the shared sentence above, on 3 pages./toolscount is 0 on all 7 pages. Before: 9, 6, 3, 5, 4, 6 and 3."If they say, and has no "top pick". The llms.txt blurb has the quote marks.<title>has no ": And".Verification
NODE_OPTIONS=--max-old-space-size=3072 npm run verifypassed locally: 757 vitest tests, collaboration and edge suites, build with check-site and check-links clean, SW smoke, and 47 Playwright @smoke tests. After that run, one CSS rule was added so the ChatGPT vs Claude verdict block spans both grid columns (screenshots showed the heading stranded in one column). After it I regenerated the pages and reran the templating check and the SEO tests (20 passed). This is a local pass, not CI: GitHub Actions is blocked by billing.🤖 Generated with Claude Code