Repository navigation
Finish the visible DebateLab to DebateOwl rename - #241
Conversation
The header, footer, titles, manifest, emails, and link preview now use the product name from site.json. The one-time notice still says DebateLab is becoming DebateOwl. Routes, storage keys, and the domain stay the same. Co-authored-by: Gáborjáni-Szabó Ádám <Programcode@users.noreply.github.com>
|
Caution Review failedThe pull request is closed.
Note 🎁 Summarized by CodeRabbit Free
Comment |
Codex review (gpt-6-luna)Independent read-only review of BlockerNone Should-fixNone Nit
Verdict: Adam, the rebrand changes look consistent; only a stale SVG title remains. |
Sonnet 5.5 review (claude-sonnet-5-5)Independent read-only review of BlockerNone Should-fix
Nit
Checked and fine
Verdict: Safe to merge once the Hungarian suffixes are hyphenated, with the hosted email templates and the |
DebateOwl takes Hungarian case endings with a hyphen. The owl mark title matches the new name, and og:image and twitter:image use hero-visual.jpg?v=2. The production canary still requires a DebateOwl title and reloads for the promotion window. Co-authored-by: Gáborjáni-Szabó Ádám <Programcode@users.noreply.github.com>
|
Sonnet review: hyphenated the Hungarian suffixes on DebateOwl ( |
|
Codex review: updated |
# Conflicts: # .claude/skills/verify-debatelab/features/auth-demo-login.md # .claude/skills/verify-debatelab/recipes/auth.spec.ts # .claude/skills/verify-debatelab/recipes/current-debate.spec.ts # e2e/first-run.spec.ts # e2e/full-stack/first-run.spec.ts # e2e/hungarian-parity.spec.ts # src/lib/debateowl-intro.ts # src/locales/en.json # src/locales/es.json # src/locales/hu.json # src/test/pages/SharedPage.test.tsx
Summary
The visible product name is DebateOwl in English, Hungarian, and Spanish.
site.jsonis the source (product/brand); the header, footer, startup shell, page titles, and PDF header read it instead of a hardcoded word. Routes, the domaindebate.pi-stack.org, package and repo names, storage keys, and API identifiers are unchanged.The one-time notice still leads with the old name so it stays true after the UI already says DebateOwl: “DebateLab is becoming DebateOwl”, with the same meaning in Spanish and Hungarian. “Slowly” is gone, and the Google line is in the present tense. Hungarian keeps “A DebateLabből DebateOwl lesz”: DebateLab is pronounced as spelled, so -ből stays attached. Suffixes on DebateOwl are hyphenated (DebateOwl-ba, DebateOwl-t, DebateOwl-ban, DebateOwl-hoz, DebateOwl-nak), matching the existing DebateOwl-fiókod.
The link-preview file is still
public/hero-visual.jpg(1200×630).og:imageandtwitter:imagepoint athttps://debate.pi-stack.org/hero-visual.jpg?v=2so scrapers fetch the new card. The JSON-LD image uses the same query. The alt text is “DebateOwl wordmark with the owl, for AI debate prep and practice”.public/owl-mark.svgtitle is “DebateOwl owl”.The production canary still requires a DebateOwl title. It reloads the homepage for up to 3 minutes, because
deployment_statuscan run while the previous HTML is still what the edge serves. Other canary checks are unchanged. This PR does not deploy.Hosted auth emails still say DebateLab
Nothing in the hosted production or staging Supabase projects was read or changed. A merge does not update Auth email templates. The only two templates this repo configures, which on
mainstill say DebateLab, are:[auth.email.template.magic_link]supabase/templates/magic_link.html)[auth.email.template.confirmation]supabase/templates/confirmation.html)The repo copies of those two now say DebateOwl. There is no recovery, invite, or email-change template in the repo. Adam can decide whether to update the two hosted subjects and bodies.
Checks
Re-run after the review fixes:
npm run lintnpm run typechecknpm testnpm run test:edgenpm run buildcheck-site55 HTML files,check-links52 sitemap URLs). Prerendered HU pages havehero-visual.jpg?v=2and no glued DebateOwl suffix.@smokeThe unit failure is the same unrelated
src/test/scripts/backend-status.test.tshook timeout (10s) while that file builds a throwaway git repo. It is not in this diff. The same test passes alone.Changed screens
Landing (EN / HU / ES), sign-in, terms, tools hub, debate timer, the static practice guide and pricing pages, the JS-disabled startup shell, the signed-in app header, and the rename notice in EN / HU / ES. Hungarian sign-in and terms were re-shot at 390, 820, and 1440 after the hyphenation (no horizontal overflow). The notice copy did not change.
Hungarian sign-in, 390px
Landing after the rename, 1440px
Rename notice, 390px
New link-preview image
Likely conflicts
design/library-nameeditssrc/locales/en.json,es.json, andhu.json, plus the first-run specs.design/bottom-tab-bareditsAppLayout.tsxande2e/home-navigation.spec.ts.design/onboarding-mergeeditsdebateowl-intro.ts, its test,App.tsx, and the verify skill.design/debateowl-animationseditsApp.tsx.design/debateowl-polishandfeat/case-structure-v2are not on the remote. Polish is expected to touch the locale files, prerendered HTML, and the owl image. Case structure is expected to touch export; this PR sets the PDF header tosite.product.DebateLab strings kept on purpose
debatelab.aiin the pricing FAQ./debatelab-pricing.@debatelab/core.workspace@debatelab.local.To show artifacts inline, enable in settings.