fix(i18n): drop the dead pages.chatPage.dismiss_upload_error key - #8951
Conversation
The chat pane's upload-error strip now renders through ErrorNotice, which carries its own dismiss control, so the hand-rolled dismiss button and the aria-label key it read were removed together — except the key, which stayed in every catalog. The dead-key ratchet now counts 30 against a baseline of 29 and fails the frontend shard on main and on every PR rebased onto it. Remove the key from the thirteen catalogs; the baseline is unchanged.
Design Review (Fable 5) — ✅ PASSDesign-level review of Design-Verdict: PASS Pure dead-key deletion across all thirteen catalogs, restoring the dead-key ratchet to its baseline; no code references remain and no design surface changes. [DESIGN-REVIEWED] 02fa7c2 |
UX Review (Fable 5) — ✅ PASSUX-level review of UX-Verdict: PASS Deletes one unreferenced i18n key from 13 catalogs; [UX-REVIEWED] 02fa7c2 |
First Principles Review (Fable 5) — ✅ PASSPremise-level review of All verification checks out: the key is referenced nowhere in the repo (grep for First-Principles-Verdict: PASS A pure deletion that unwedges the red What this change shipsIntent: turn
The claim checks out mechanically: grep for [FIRST-PRINCIPLES-REVIEWED] 02fa7c2 |
Opus 4.8 Review — ✅ no blocking findingsReviewed Verdict parsed from the review's SHA-scoped output markers for commit False positive or not applicable? A repository writer can comment: |
GPT 5.6 Review — ✅ no blocking findingsGPT 5.6 completed its review of This comment is updated in place on each push. Review detailsNo findings. False positive or not applicable? A repository writer can comment: |
Summary
mainis red onFrontend Tests (3)since #8859:src/i18n/deadKeys.test.tscounts 30 unreferenced catalog keys against a baseline of 29. The new dead key ispages.chatPage.dismiss_upload_error— #8859 replaced the chat pane's hand-rolled upload-error strip (and its dismiss button, whosearia-labelread this key) withErrorNotice, which carries its own dismiss control, but the key stayed in every catalog.This removes the key from the 13 catalogs (12 authored + regenerated
en-XA).BASELINEis unchanged at 29. No UI change.Verification
npx vitest run src/i18n— 42 files / 663 tests passed (deadKeys, catalogParity, keyReference, pluralKeys, changedValueQa included).Every open PR rebased onto current
mainfails the same shard until this lands (e.g. #8943).