fix(web): drop unused Discord badge class

[Hmbown]

Summary

Current main already contains the substantive copy, locale, navigation, and metadata improvements from the original branch. Rebuilding the PR cleanly left one honest cleanup: remove the stale paper-discord-badge class from the Discord nav button, whose styling is fully owned by site-discord-link.

Validation

• npm run lint -- components/nav.tsx
• git diff --check origin/main...HEAD
• final diff read back: one class-name removal

No-Issue: tiny cleanup found while rebuilding the older web branch on current main.

Agent-assisted; verified against current web styles.

[cursor]

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

[claude]

Claude finished @Hmbown's task in 3m 40s —— View job

---

Review of PR #5289

• Gather context (diff, CLAUDE.md/AGENTS.md rules)
• Review nav/CSS changes (nav-links.tsx, nav.tsx, globals.css)
• Review EN copy changes (home.ts, chrome.ts, docs/runtime pages)
• Review native locale rewrites (9 locales × home.ts/chrome.ts) for token/structure integrity
• Check types.ts change
• Post final review

This PR is entirely in web/ (Next.js i18n site), not the Rust workspace, so I reviewed it as a web/content change: correctness first, then reuse/consistency and content precision.

Bug: duplicated "Codewhale" in the OG/social card

IDENTITY_PHRASE in web/lib/page-meta.ts:10-11 changed from "Dive into the deep so you don't have to — …" to "Codewhale dives into the deep so you don't have to — …" — it now includes the brand name itself.

But two call sites still prepend SITE_NAME in front of it:

• web/lib/page-meta.ts:18 — alt: \${SITE_NAME} — ${IDENTITY_PHRASE}`(used as the default OG imagealtfor every locale/page viabuildPageMetadata`)
• web/app/opengraph-image.tsx:4 — export const — ${IDENTITY_PHRASE}\;`

This renders as "Codewhale — Codewhale dives into the deep so you don't have to — any model, on your machine." — and the actual generated PNG (opengraph-image.tsx:47-58) visually stacks it too: a big "Codewhale" heading directly above body text that now starts with "Codewhale" again. This is the image/alt text shared across every locale and route (via OG_IMAGE/buildPageMetadata), so it's a live regression on every social share card, not just English.

page-meta.test.ts:48 won't catch this — it asserts against the same ${SITE_NAME} — ${IDENTITY_PHRASE} template expression, so it stays green regardless of what IDENTITY_PHRASE contains.

Fix: either drop "Codewhale" back out of IDENTITY_PHRASE (so it composes cleanly with SITE_NAME —), or stop prepending SITE_NAME at the two call sites since the phrase now carries the brand itself.

Fix this →

Minor / non-blocking

• web/lib/i18n/dictionaries/es/chrome.ts:33 — dateLocale changed from "es" to "es-ES", but the file's own doc comment says this pack is "neutral (pan-Hispanic) Spanish." es-ES biases Intl date formatting toward Spain conventions; if neutrality matters here, es-419 (or leaving it locale-generic) would match the stated intent better than es-ES.
• aria-label="Discord" on the new icon-only Discord button (web/components/nav.tsx:100) is hardcoded rather than sourced from the chrome dict like the other nav aria-labels (chrome.starsAria, etc.) — pre-existing pattern (the visible "Discord" text was already hardcoded English before this PR), so not a regression, just an inconsistency now that it's icon-only across all locales.
• .paper-discord-badge (still applied in nav.tsx:99) has no matching CSS rule in globals.css — pre-existing dead class reference, not introduced by this PR.

Verified good

• All nine locale rewrites (es, id, ja, ko, pt-BR, ru, uk, vi, zh) preserve every {brand}/{tag}/{count}/{date}/{mode}/{label}/{handle} interpolation token — diffed every changed line's token set against its predecessor and found no mismatches.
• Nav baseline alignment (nav-links.tsx) and the new .site-discord-link/.brand-mark CSS are consistent with the existing .site-github-link/.paper-star-badge pattern; SVGs are aria-hidden with the label carried on the parent Link, which is correct.
• Key parity and "local-first" removal look complete — grepped for lingering local-first/localized equivalents; the only remaining hits are in doc comments explicitly noting the old framing is gone, not live copy.

I wasn't able to install dependencies in this environment (no node_modules, and installing was out of scope for a review), so I didn't re-run npm test/tsc/eslint myself — I verified the above by reading the diff and tracing call sites directly instead.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b9f8c4df98

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Remove the duplicated brand from OG image alt text

Because both OG_IMAGE.alt here and app/opengraph-image.tsx prepend SITE_NAME, adding “Codewhale” to IDENTITY_PHRASE makes every social-card alternative read Codewhale — Codewhale dives into the deep…. Screen-reader users encounter this duplicated label on every localized page; either keep the identity phrase brand-free or stop prepending the site name.

Useful? React with 👍 / 👎.

[claude]

Claude finished @Hmbown's task in 33s —— View job

---

Review of PR #5289

• Gather context (diff, CLAUDE.md/AGENTS.md rules)
• Review the diff (web/components/nav.tsx)
• Post final review

Summary

Clean, minimal fix. The diff removes paper-discord-badge from the Discord nav link's className (web/components/nav.tsx:99), leaving site-discord-link as the sole class:

-            className="site-discord-link paper-discord-badge"
+            className="site-discord-link"

I verified .paper-discord-badge has no matching rule anywhere in web/app/globals.css or elsewhere in web/ — it was dead weight. .site-discord-link fully owns the styling (globals.css:514-524, plus the responsive display: none at globals.css:953), so removing the stray class changes no rendered behavior. This is exactly the "pre-existing dead class reference" flagged as a minor/non-blocking note in the prior review of this branch (see comment above from 2026-08-08).

This is a one-line, no-behavior-change cleanup — no correctness bugs, no provider/model/route safety concerns (not applicable to this file), no reuse/duplication issues, and no new behavior requiring test coverage. Nothing else to flag.

Verified

• git diff origin/main...HEAD shows only this one-line change (1 file, +1/-1).
• Grepped web/ for paper-discord-badge and site-discord-link: only the removed reference existed for the former; the latter has full CSS ownership.

Looks good to merge.

[Hmbown]

Closing as superseded: the substantive copy, navigation, locale, and metadata improvements from this lane are already on main (5bdfeb7 localized navigation, eb5faf7 EN copy + icon buttons + locale rewrites, 74b5192 accessibility/structured-data/metadata fixes). The only remaining diff is the non-behavioral Discord badge-class cleanup, which isn't worth a merge on its own — it can ride along with the next web PR if it still matters. Thanks for keeping the web surface moving!