Repository navigation
Deepen the four format guides and shrink the shared About block - #233
Merged
Merged
Conversation
Lincoln-Douglas, Policy, British Parliamentary and Congress each get a speech-time table that matches the free timer's format data (Congress from the NSDA manual, since the timer has no preset), a worked example on an original practice motion, common mistakes, a 15-minute drill, their own FAQs and links to the NSDA manual or the WUDC manual. Their lead answers are rewritten so the sections do not repeat them. The generator gains a table block and https links to primary sources. The two-paragraph About block now appears only on the pricing page; every other guide gets one line linking to the landing page and pricing. CONTENT_DATE is now a floor for every guide, because that shared copy changed on all of them. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Caution Review failedThe pull request is closed.
Note 🎁 Summarized by CodeRabbit Free
Comment |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The worked example labelled the permutation and solvency deficit as the 2AR answer, but they are new arguments, and the same page says judges discount new arguments in the last rebuttal. They belong in the 2AC; the 1AR and 2AR now extend and weigh them instead. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Programcode
added a commit
that referenced
this pull request
Oct 26, 2026
#233 and #235 deepened the guides and retired five of them. The generator keeps main's per-page FAQs, primary-source links, tables, short About block and CONTENT_DATE floor, and this branch's clusters, in-body links and entity JSON-LD. LINKS_DATE is gone: CONTENT_DATE already dates every page from 2026-10-26, which covers the shared related links. The retired slugs leave CLUSTERS; their phrases now link the guides their 301s point to. In-body links also go in list and step items, since the deepened guides put much of their prose there. GuideHub renders nothing on /es and /hu routes, which do not exist yet on main. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Programcode
added a commit
that referenced
this pull request
Oct 26, 2026
Conflicts resolved:
- i18n.ts: keep both default variable sets, product/brand/host from
main and trialDays/proMonthly from this branch.
- en/es/hu.json: main's new hero headline and subheadline, this
branch's {{trialDays}} trial badge.
- prerender/entry.tsx, check-site.mjs, docs/operations.md: keep both
sides (main's entity graph and FAQ checks, this branch's localized
pages and hreflang check).
GuideHub on /hu and /es now shows only the free tools at their
localized addresses, plus one link to the English guides on the home
page ("Angol nyelvű útmutatók", "Guías en inglés"). The Hungarian tool
labels use this branch's names (Vitaidőmérő, Vitatétel-generátor,
Jegyzetlap), and "cáfolást" follows the review.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Oct 26, 2026
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.
Spec 2, part 1 of 2 (formats first, as the spec asks once it grows past about 6 pages). Part 2 retires the 5 weak pages and deepens the 6 skills guides. It is #235, a separate draft PR stacked on this one.
What changed
src/lib/free-tools/formats.ts. LD, Policy and BP link to the timer with that format preselected (/tools/debate-timer?format=ld|policy|bp). The timer has no Congress preset, so the Congress table comes from the NSDA manual and the page points to the timer's custom format.tableblock, and[label](https://...)links to primary sources./and/debatelab-pricingon every guide except the pricing page, which keeps the definition and plan facts./andllms.txtare unchanged.CONTENT_DATEmoves to 2026-10-26 and is now a floor for every guide'smodified, because the aside changed on every page. Before this change, a page with its own oldermodifieddate would have kept a stale<lastmod>.{product}, set inseo_guides.py.Word counts
These count visible words in
<main>, leaving out the breadcrumb, the related-links block and the About aside.Note for Spec 1 (
seo/internal-links-brand-entity)internal_links()is untouched. The sections for these 4 slugs now live inscripts/seo_guides.py, andGUIDESreplaces a page'ssectionswholesale. Put contextual in-body links for these pages inseo_guides.py. If you add them to thePAGESentries ingenerate_seo_content.py, they will be silently overwritten.Review follow-up
In the Policy worked example, the permutation and solvency deficit row is now labelled "2AC answer:" instead of "2AR answer:". The page itself says judges discount new arguments in the last rebuttal, so a 2AR label contradicted it. A new "1AR and 2AR:" row extends and weighs those answers instead.
Verification
npm run verifyran locally at this head (11d7732) and passed. GitHub Actions is billing-blocked, so this is a local pass, not a CI pass.Below 760px, tables stack each row into labelled cells, because a 3-column table didn't fit at 390px. I checked that layout and desktop in a real browser, and confirmed that
?format=bppreselects British Parliamentary in the timer.🤖 Generated with Claude Code