Repository navigation
Spell the brand "DebateOwl" where Google reads it first; drop DebateLab-only sameAs; larger favicon frames - #274
Merged
Merged
Conversation
- The homepage's first sentence now starts with the name: "DebateOwl helps you prepare cases and practice rounds with AI." (en, es, hu). The H1 and <title> already used it. - Organization.sameAs drops the SaaSHub and Uneed listings: checked live on 29 Oct 2026, both still say "DebateLab" and never "DebateOwl". With no profile left, the node omits sameAs; check-site and the brand test now allow that but still reject a non-https or parent URL. - favicon.ico gains 64, 96, 128 and 256 px frames of the owl next to the existing 16, 32 and 48 px ones (20 KB, PNG-compressed), rasterized from src/assets/owl/owl-favicon.svg. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Caution Review failedThe pull request is closed.
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 3, the brand entity fix. In a US sample, Google spell-corrects "debateowl" to "debate owl". This PR makes the places it reads first use the one-word name, and keeps profiles that spell it differently out of
sameAs. It branches offmainand touches none of #272/#273's generator files (a trialgit merge-treewith #273 merges cleanly).What changed
Homepage H1,
<title>and first sentence. The H1 and title were alreadyDebateOwl: AI debate prep and practice. The first sentence of body copy (the hero subheadline) was "Prepare cases and practice rounds with AI." It is now "DebateOwl helps you prepare cases and practice rounds with AI.", with es and hu to match.Organization
sameAs. I checked both profiles live on 29 Oct 2026:/debateowladdresses on both sites return 404, and nothing in the repo, the footer or humans.txt names another profile. A web search for "DebateOwl" found no profiles either.Both entries are dropped, so
sameAsinsite.jsonis now[], andorganizationNode()leaves the key out instead of shipping an empty list.check-site.mjsandbrand-entity.test.tsused to require a non-emptysameAs. They now accept nosameAs, but still reject a non-https URL and the parent (Pi-Stack). That is a deliberate relaxation of an existing check, made for this spec.docs/operations.mdstates the rule: list a profile only after checking that its live page says "DebateOwl".favicon.ico. Production served 16, 32 and 48 px frames. The file now holds 16, 32, 48, 64, 96, 128 and 256 px frames. All are PNG-compressed, for 20 KB in total (the old file was 3 KB; the 273 KB favicon from the October performance review stays gone). The new frames come from
src/assets/owl/owl-favicon.svg(rasterized in Chromium), the same source as the existing frames, which stay byte-identical. The icon links (favicon.ico, the 48 px PNG, the 192 px PNG,apple-touch-icon) were already on every page, so they are unchanged. Changing them would regenerate every guide and collide with Lead the hub pages with DebateOwl and link the comparison cluster both ways #272/De-template the seven comparison pages and fix their copy slips #273.Left as is, for Adam to decide
alternateName: "Debate Owl"stays on the Organization, WebSite and SoftwareApplication nodes. It tells Google the two-word form is the same entity, which helps people who type it that way, but it could also reinforce the spell-correction. Out of scope here.sameAs, rename the SaaSHub and Uneed listings to DebateOwl, then add their URLs tosite.json.Verification
NODE_OPTIONS=--max-old-space-size=3072 npm run verifypassed locally at this head (756 vitest tests, collaboration, edge, build with check-site and check-links clean, SW smoke, 47 Playwright @smoke tests). The only later change is a comment inowl-favicon.svgthat lists the new frame sizes. In the builtdist/index.html, the title and H1 contain "DebateOwl", the first sentence after the H1 is "DebateOwl helps you…", the Organization node has nosameAs, anddist/favicon.icois the 7-frame file. The sitemap has 62 URLs. This is a local pass, not CI: GitHub Actions is blocked by billing.🤖 Generated with Claude Code