fix(devx): check:i18n fails on an undeclared authoring key, not just on bundle drift (#4804) (#4847)

The #4167 unknown-authoring-key lint saw all nine offending
`scripts/i18n-extract.config.ts` files that #4736 later cleaned. It printed
`stack.name: 'name' is not a declared stack key, so its value is dropped at
load` once per package, on every run — inside a fully green `check:i18n`, so it
read as noise nine times running. #4736 removed the symptom; nothing stopped the
tenth copy.

Two things made the warning unenforceable, and both are fixed here:

  1. The lint writes to **stderr**, and the gate used `execFileSync`, which
     surfaces stderr only on the throw path. On a zero-exit run the stream went
     straight to the terminal — seen by nobody, judged by nothing. Now
     `spawnSync`, so both streams are read on both paths; stderr lines that are
     not the lint signature are re-emitted, so capturing costs no diagnostics.
  2. The gate judged bundle drift only. It now also fails on the
     unknown-authoring-key signature, in its own verdict.

Option A per the issue: the failure lands in the gate script. `os i18n extract`
keeps its exit code (B would write an internal hygiene rule into the public CLI
contract) and `ObjectStackDefinitionSchema` stays non-strict (C would silence
the lint itself — see `metadata-authoring-lint.ts`).

The two verdicts never merge. Drift keeps its section and its remedy
(`--write`); the new class gets its own, naming the package, the config path,
the key, and the consequence that matters — the value is dropped at load, so
whatever it was meant to configure is not in effect and never was. Regenerating
bundles does not fix it, and the message says so. Coverage needs no manifest:
`findConfigs` walks `packages/`, so the tenth config is gated the day it lands.

Proven able to go red, not merely observed green — the failure mode of #4690.
`--self-test` (now wired into `check:i18n` ahead of the real run) drives both
classifiers over recorded CLI output, asserts the drift text produces no key
finding and the key text produces no drift finding, and pins the exact
ffab803^ stderr line so a reworded message breaks the test rather than
disarming the gate. A reworded signature that still matches the weak pattern
fails as `unattributed` instead of passing. Reverse-verified against the nine
configs restored from ffab803^: all nine reported, exit 1.

Fixes #4804


Claude-Session: https://claude.ai/code/session_018iARDqtrhQgz6fVHDeDkbQ

Co-authored-by: Claude <noreply@anthropic.com>

Author: xuyushun441-sys

.changeset/check-i18n-fails-on-undeclared-authoring-key.md

@@ -0,0 +1,45 @@
+---
+---
+
+chore(devx): `pnpm check:i18n` now FAILS on an undeclared authoring key, not just on bundle drift
+
+Releases nothing — the change is confined to `scripts/check-i18n-bundles.mjs` and
+the root `check:i18n` script. No package source, no published behaviour, and
+deliberately **not** the `os i18n extract` exit code (that would write an internal
+hygiene rule into the public CLI contract).
+
+#4736 cleaned nine `scripts/i18n-extract.config.ts` files that all opened their
+`defineStack({ … })` with the same undeclared `name:` key. Nine, because the same
+mistake was copied from the first one — and nothing stopped any of them. The
+#4167 unknown-authoring-key lint *saw* every single one: it printed
+`stack.name: 'name' is not a declared stack key, so its value is dropped at load`
+on stderr, once per package, on every run. But the CLI exited 0 and the gate only
+judged bundle drift, so those nine warnings appeared inside a **fully green**
+`check:i18n` and were read as noise nine times. A warning that nine authors
+filtered out is not a control; #4736 cleaned the symptom, this closes the hole.
+
+**What changed.** The gate now reads the extractor's **stderr** — which it
+previously let flow straight through to the terminal, seen by nobody and judged by
+nothing — and fails on the unknown-authoring-key signature. Coverage needs no
+manifest: `findConfigs` walks `packages/`, so the tenth config is gated the day it
+lands.
+
+**The two verdicts stay separate.** Bundle drift keeps its own section and its own
+remedy (`--write`); the new class gets its own, naming the package, the config
+path, the key, and the consequence that matters — *the value is dropped at load,
+so whatever it was meant to configure is not in effect and never was*.
+Regenerating bundles does not fix it, and the message says so.
+
+**The gate is proven able to go red**, not merely observed green — the failure
+mode of `check:react-declaration-parity` (#4690), which exited 0 with nothing to
+check. Two proofs: `node scripts/check-i18n-bundles.mjs --self-test` (now wired
+into `check:i18n`, ahead of the real run) drives both classifiers over recorded
+CLI output, including a case asserting neither verdict matches the other's output;
+and the gate was run against the nine configs restored from `ffab8033b^`, the
+commit before #4803 deleted the keys, where it reports all nine and exits 1.
+
+Fixing an offending config means deleting the key at the producer. If a key is
+genuinely wanted it gets declared in `packages/spec` deliberately — not
+accommodated by a consumer-side fallback, and not silenced by making
+`ObjectStackDefinitionSchema` strict (which would mute the lint itself; see
+`metadata-authoring-lint.ts`).

package.json

@@ -30,7 +30,7 @@
     "objectui:clean": "rm -rf packages/console/dist .cache/objectui-*",
     "lint": "eslint . --no-inline-config",
     "i18n:extract": "tsx packages/cli/bin/run-dev.js i18n extract packages/platform-objects/scripts/i18n-extract.config.ts --locales=zh-CN,ja-JP,es-ES --fill=default --out=packages/platform-objects/src/apps/translations",
-    "check:i18n": "node scripts/check-i18n-bundles.mjs",
+    "check:i18n": "node scripts/check-i18n-bundles.mjs --self-test && node scripts/check-i18n-bundles.mjs",
     "check:i18n-coverage": "node scripts/check-i18n-coverage.mjs",
     "check:nul-bytes": "node scripts/check-nul-bytes.mjs",
     "check:doc-authoring": "node scripts/check-doc-authoring.mjs",