seo: stop the description echoing the title, and clear the copy-voice defects - #15
Merged
Merged
Conversation
… defects Four low-severity findings from the 2026-09-05 end-to-end audit, all of them text or attribute level. The home meta description spent its last sentence restating the title tag almost character for character, so a reader who had already seen the title learned nothing from it. That closing clause now carries the sourcing attribution instead, which is a fact the description did not otherwise state. The About page joined clauses with em dashes, and three pages carried the absolutes the estate copy-voice rule bars. Each one is recast with a comma, a period, or a scoped phrase. No claim, number, name, or date moves. The header wordmark link labelled itself "Suede home" while displaying SUEDE/DNA, so its accessible name did not contain its visible label and Lighthouse scored label-content-name-mismatch at 0. Dropping the override lets the name come from the visible text, which is what the rule asks for. Findings: dna.suedeai.ai#4, #5, #6, #7 (label half).
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
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
1 participant
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.
Second pass on the 2026-09-05 end-to-end audit for dna.suedeai.ai. Four LOW findings,
all text or attribute level. No claim, number, name, date, or sentence subject changes.
dna.suedeai.ai#4 — home meta description restated the title verbatim.
src/lib/seo.tshomePageDescription()ended with "Suede DNA — signal chains, archived.",a near character-for-character echo of the
<title>. That clause now carries the sourcingattribution the description did not otherwise state. The same function feeds the
CollectionPageJSON-LDdescription, so both surfaces move together.dna.suedeai.ai#5 — em dashes as clause connectors on /about.
The audit named three, the file had seven in body prose. All are recast with a comma or a
period. The two remaining em dashes in the file are the
About — Suede DNAOG and Twittertitle separators, which the finding exempts.
dna.suedeai.ai#6 — absolutes ("every", "nothing") in body copy.
grep -niE '\bevery\b|\bnothing\b'across the three named files now returns nothing.The audit listed two instances on /about; there were three. The FAQ answer is also the
FAQPageacceptedAnswer.text, so the JSON-LD moves with the rendered text.One deviation from the suggested rewrite: for the Clapton article the audit proposed
"Some details of that story may be embellished...", which asserts embellishment the
original left open. Kept the original's agnostic framing instead:
"Whether the details of that story are precise or slightly burnished...".
dna.suedeai.ai#7 — Lighthouse mobile
label-content-name-mismatch, applied.The header wordmark link carried
aria-label="Suede home"while displayingSUEDE/DNA,so its accessible name did not contain its visible label. It was the only element on the
served homepage with a name-from-content role and an overriding
aria-label; the otherfive sit on
nav,aside, androle="group"containers, which the rule does not cover.Dropping the override lets the name come from the visible text, which matches by
construction.
dna.suedeai.ai#7 —
target-size, not fixed. Measured on the live page at 375x812:16 targets under 24x24 CSS px, in three groups — the five header nav links (22.4x16.5,
inside a bar pinned to
var(--header-main-h)), ten rig-card source links (325x16.5), andthe footer link (104.7x14). Every one is a
.mono-labelelement whose height comes fromthe type scale in
src/styles/tokens.css. Raising them means changing spacing tokens andlayout across the header, the cards, and the footer, which is a design decision under the
visual overhaul spec rather than a mechanical fix, so it is left for that work.
Gates:
pnpm typecheck,pnpm lint,pnpm test(111 passed),pnpm buildall pass.