From 4739542397fe9887b18639b25048a1945c85e759 Mon Sep 17 00:00:00 2001 From: Codeman maintainer Date: Fri, 6 Nov 2026 16:30:59 +0100 Subject: [PATCH] chore(local-model): drop build-time wording from two comments The label list was "frozen for the parallel build" and the card's test said the route answers 501: both were true while the pieces were written side by side and are not any more. Comments only. Co-Authored-By: Claude Fable 5.1 --- src/types/local-model.ts | 6 +++--- test/local-model-ui.test.ts | 7 ++++--- 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/src/types/local-model.ts b/src/types/local-model.ts index 77352d64..faf42777 100644 --- a/src/types/local-model.ts +++ b/src/types/local-model.ts @@ -221,9 +221,9 @@ export interface DeviceBudget { /** * Every sentence and label the server may put into the payload, in English (the i18n source - * language). Frozen for the parallel build: the zh-CN dictionary in src/web/public/i18n.js has - * one entry per value, keyed by this exact text, and the browser's i18n test imports this - * constant to list them. Adding a key is fine; changing a text means changing its zh-CN key too. + * language). The zh-CN dictionary in src/web/public/i18n.js has one entry per value, keyed by + * this exact text, and test/local-model-i18n.test.ts imports this constant to list them. Adding + * a key is fine; changing a text means changing its zh-CN key too. * * `{placeholder}` slots are filled by the browser from `LocalModelMessage.vars`. `{name}` is * taken (the translator fills it with the display name), so no slot here is called `name`. diff --git a/test/local-model-ui.test.ts b/test/local-model-ui.test.ts index 1689d6ff..00daa696 100644 --- a/test/local-model-ui.test.ts +++ b/test/local-model-ui.test.ts @@ -16,8 +16,9 @@ * - the dismissal is a per-device localStorage key and survives a new app instance; * - one request a minute at most, shared while in flight, and a failed refresh keeps the answer. * - * The route answers 501 until the server stage lands, so the payloads here are fixtures typed - * with the contract's `LocalModelStatus`, built from `LOCAL_MODEL_LABELS`. + * The payloads here are fixtures typed with `LocalModelStatus` and built from + * `LOCAL_MODEL_LABELS`, not the output of the route: what the server computes for a given Mac is + * pinned by test/local-model-recommend.test.ts and test/routes/local-model-routes.test.ts. * * Port: none. */ @@ -131,7 +132,7 @@ const settle = () => new Promise((done) => setTimeout(done, 0)); // ---------- Fixtures (the contract's types) ---------- -/** The MacBook of the probe: 48 GB as sold, a chip that is in no chips row. */ +/** The MacBook of the probe: 48 GB as sold. The candidates below give it an unknown speed, as for a chip in no chips row. */ const HARDWARE: HardwareFacts = { platform: 'darwin', arch: 'arm64',