Skip to content

fix(i18n): drop the dead pages.chatPage.dismiss_upload_error key - #8951

Merged
bolichen97 merged 1 commit into
mainfrom
fix/drop-dead-dismiss-upload-error-key
Sep 6, 2026
Merged

fix(i18n): drop the dead pages.chatPage.dismiss_upload_error key#8951
bolichen97 merged 1 commit into
mainfrom
fix/drop-dead-dismiss-upload-error-key

Conversation

@bolichen97

Copy link
Copy Markdown
Collaborator

Summary

main is red on Frontend Tests (3) since #8859: src/i18n/deadKeys.test.ts counts 30 unreferenced catalog keys against a baseline of 29. The new dead key is pages.chatPage.dismiss_upload_error#8859 replaced the chat pane's hand-rolled upload-error strip (and its dismiss button, whose aria-label read this key) with ErrorNotice, 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). BASELINE is 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 main fails the same shard until this lands (e.g. #8943).

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.
@bolichen97
bolichen97 requested a review from a team as a code owner September 6, 2026 08:34
@github-actions github-actions Bot added readiness: checking Automated validation is still running readiness: action required A blocking check or review needs attention and removed readiness: checking Automated validation is still running labels Sep 6, 2026
@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Design Review (Fable 5) — ✅ PASS

Design-level review of 02fa7c2011f07f97ab60f28939ebe75da56ed255 — updated in place on each push. A BLOCK verdict blocks PR readiness; PASS/CONCERNS are advisory.

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

@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

UX Review (Fable 5) — ✅ PASS

UX-level review of 02fa7c2011f07f97ab60f28939ebe75da56ed255 — updated in place on each push. A BLOCK verdict blocks PR readiness; PASS/CONCERNS are advisory.

UX-Verdict: PASS

Deletes one unreferenced i18n key from 13 catalogs; dismiss_upload_error has zero references in website/src, so no rendered string, control, or state changes.

[UX-REVIEWED] 02fa7c2

@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

First Principles Review (Fable 5) — ✅ PASS

Premise-level review of 02fa7c2011f07f97ab60f28939ebe75da56ed255 — why this exists and whether the shipped surface is the smallest honest version. Updated in place on each push. A BLOCK verdict blocks PR readiness; PASS/CONCERNS are advisory.

All verification checks out: the key is referenced nowhere in the repo (grep for dismiss_upload_error across the whole checkout: 0 hits outside the diff's deletions), all 13 non-manual catalogs are covered plus en.manual.json never carried the key, and deadKeys.test.ts:106 pins the count exactly to BASELINE = 29 — so main was red at 30 and this deletion restores exactly 29 with no baseline edit. This is a pure subtraction that unwedges a red build, which the contract treats as derived by definition.

First-Principles-Verdict: PASS

A pure deletion that unwedges the red deadKeys shard: one orphaned key removed from every catalog, baseline untouched, nothing rides along.

What this change ships

Intent: turn Frontend Tests (3) green again by deleting the catalog key #8859 orphaned. This is a FIX.

  1. The unused "Dismiss upload error" string is deleted from all 13 locale catalogs — justified

The claim checks out mechanically: grep for dismiss_upload_error across the repo finds zero references (the ErrorNotice replacement in #8859 carries its own dismiss label), en.manual.json never held the key, and deadKeys.test.ts pins the dead count to exactly BASELINE = 29 in both directions, so this single deletion is the precise fix — no baseline bump, no rider, no smaller version exists. The 29 remaining dead keys are pre-existing debt the ratchet already tracks; they are not siblings of this cause (#8859's removal of the strip) and belong to no in-scope fix.

[FIRST-PRINCIPLES-REVIEWED] 02fa7c2

@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Opus 4.8 Review — ✅ no blocking findings

Reviewed 02fa7c2011f07f97ab60f28939ebe75da56ed255 — this comment is updated in place on each push.

Review details

No findings.

[OPUS-REVIEWED] 02fa7c2

Verdict parsed from the review's SHA-scoped output markers for commit 02fa7c2011f07f97ab60f28939ebe75da56ed255.

False positive or not applicable? A repository writer can comment:
/ai-review override fable 02fa7c2011f07f97ab60f28939ebe75da56ed255: <one-sentence reason>

@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

GPT 5.6 Review — ✅ no blocking findings

GPT 5.6 completed its review of 02fa7c2011f07f97ab60f28939ebe75da56ed255 and found no blocking issues.

This comment is updated in place on each push.

Review details

No findings.
[GPT-REVIEWED] 02fa7c2

False positive or not applicable? A repository writer can comment:
/ai-review override gpt 02fa7c2011f07f97ab60f28939ebe75da56ed255: <one-sentence reason>

@bolichen97
bolichen97 merged commit 8aef8fe into main Sep 6, 2026
60 of 62 checks passed
@bolichen97
bolichen97 deleted the fix/drop-dead-dismiss-upload-error-key branch September 6, 2026 08:45
@github-actions github-actions Bot removed the readiness: action required A blocking check or review needs attention label Sep 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant