From 2d1009136a0c8bc8bee7bdabb2d9e9f40a609523 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marc-Andr=C3=A9=20Bombeck?= Date: Thu, 12 Nov 2026 12:54:20 +0100 Subject: [PATCH] chore: drop the em dashes this release added to prose and test titles The notification categories' contract description, which lands in the published API document, and a handful of new code comments and test titles used em dashes; the house style keeps them out of new prose. --- src/__tests__/display-transform-rounding-guard.test.ts | 2 +- src/app/api/telegram/webhook/__tests__/route.test.ts | 4 ++-- .../__tests__/medication-intake-quick-add.test.tsx | 2 +- src/components/documents/document-picker-sheet.tsx | 6 +++--- src/lib/ai/provider.ts | 2 +- src/lib/insights/__tests__/blood-pressure-status.test.ts | 2 +- src/lib/insights/__tests__/features.test.ts | 2 +- src/lib/openapi/routes/notifications.ts | 2 +- 8 files changed, 11 insertions(+), 11 deletions(-) diff --git a/src/__tests__/display-transform-rounding-guard.test.ts b/src/__tests__/display-transform-rounding-guard.test.ts index 8d0a4a5fd..b24003542 100644 --- a/src/__tests__/display-transform-rounding-guard.test.ts +++ b/src/__tests__/display-transform-rounding-guard.test.ts @@ -45,7 +45,7 @@ const REGISTRY = "lib/measurements/display-transform.ts"; /** * The one module allowed to import the unrounded escape hatch. The target * adapter needs the unrounded pound edge of a canonical guardrail so it can - * round it INWARD — rounding to the nearest tenth first can put the pound + * round it INWARD: rounding to the nearest tenth first can put the pound * edge outside the kilogram window, and the server rejects what converts * back from it. */ diff --git a/src/app/api/telegram/webhook/__tests__/route.test.ts b/src/app/api/telegram/webhook/__tests__/route.test.ts index 4226f6012..f9fa42f50 100644 --- a/src/app/api/telegram/webhook/__tests__/route.test.ts +++ b/src/app/api/telegram/webhook/__tests__/route.test.ts @@ -1272,7 +1272,7 @@ describe("POST /api/telegram/webhook — record-only medications", () => { * silent, so a reminder button left in the chat, the /add list and the * "taken" keyword do nothing either. */ -describe("POST /api/telegram/webhook — Medications module off", () => { +describe("POST /api/telegram/webhook with the Medications module off", () => { function textUpdate(text: string) { return { update_id: 400, @@ -1323,7 +1323,7 @@ describe("POST /api/telegram/webhook — Medications module off", () => { * since ended, which would also take a unit off the stock. The list side is * pinned in `src/lib/__tests__/telegram-dose-medications.test.ts`. */ -describe("POST /api/telegram/webhook — ended course", () => { +describe("POST /api/telegram/webhook on an ended course", () => { const DAY_MS = 24 * 60 * 60 * 1000; const ENDED = { ...MEDICATION, diff --git a/src/components/dashboard/__tests__/medication-intake-quick-add.test.tsx b/src/components/dashboard/__tests__/medication-intake-quick-add.test.tsx index 28ab5a317..aa768cdd7 100644 --- a/src/components/dashboard/__tests__/medication-intake-quick-add.test.tsx +++ b/src/components/dashboard/__tests__/medication-intake-quick-add.test.tsx @@ -408,7 +408,7 @@ describe("submit invalidation — the quick-add is a capture-picker surface, not }); it("forces the inactive Today hero + dashboard reads to refetch after submit", async () => { - // The helper the submit path calls — asserted directly, same shape as + // The helper the submit path calls, asserted directly, same shape as // the take-all-due suite: the bundle fans out (N keys), then the two // daily reads are invalidated with `refetchType: "inactive"`, which is // what actually refetches them while unmounted. diff --git a/src/components/documents/document-picker-sheet.tsx b/src/components/documents/document-picker-sheet.tsx index 41bf5f182..140750ccf 100644 --- a/src/components/documents/document-picker-sheet.tsx +++ b/src/components/documents/document-picker-sheet.tsx @@ -2,9 +2,9 @@ /** * The vault document picker sheet: a `ResponsiveSheet` (bottom sheet on - * phones) with a debounced search over the person's own vault — the same - * recall path as the vault page, one page (50) of matches with a line that - * says when more exist, the search input as the way to an older document — + * phones) with a debounced search over the person's own vault (the same + * recall path as the vault page, one page of 50 matches with a line that + * says when more exist, the search input as the way to an older document) * and one toggle row per document. * * Two pickers used to carry this whole body as copies: linking documents to diff --git a/src/lib/ai/provider.ts b/src/lib/ai/provider.ts index d51551682..11b5b90b0 100644 --- a/src/lib/ai/provider.ts +++ b/src/lib/ai/provider.ts @@ -1355,7 +1355,7 @@ async function resolveProviderForTestUnbound( // Empty selection ("test my saved config" — the ai-section calls // /api/ai/test with an empty body): resolve via the SAME path generation - // uses — the first entry of `resolveProviderChain`. The legacy codex→admin + // uses, the first entry of `resolveProviderChain`. The legacy codex→admin // fallback used to probe a different provider than generation ran // (observed: test→admin-key while generation→codex), so a green test gave // no signal about whether overnight generation would work. diff --git a/src/lib/insights/__tests__/blood-pressure-status.test.ts b/src/lib/insights/__tests__/blood-pressure-status.test.ts index 872190a88..dccc3209e 100644 --- a/src/lib/insights/__tests__/blood-pressure-status.test.ts +++ b/src/lib/insights/__tests__/blood-pressure-status.test.ts @@ -346,7 +346,7 @@ describe("generateBloodPressureStatusForUser — no rate without an expectation" }); }); -describe("generateBloodPressureStatusForUser — the person's own target", () => { +describe("generateBloodPressureStatusForUser and the person's own target", () => { const readings = [1, 2, 3].flatMap((daysAgo) => { const measuredAt = new Date(Date.now() - daysAgo * dayMs); return [ diff --git a/src/lib/insights/__tests__/features.test.ts b/src/lib/insights/__tests__/features.test.ts index fbdd0fdd6..25470b91c 100644 --- a/src/lib/insights/__tests__/features.test.ts +++ b/src/lib/insights/__tests__/features.test.ts @@ -422,7 +422,7 @@ describe("extractFeatures — weight target direction (#1006)", () => { }); }); -describe("extractFeatures — blood pressure against the person's own target", () => { +describe("extractFeatures: blood pressure against the person's own target", () => { // 135/85 on three days: above the age default's ceiling, inside a target // the person's doctor agreed (up to 140/90). const bpRows = [1, 2, 3].flatMap((daysAgo) => { diff --git a/src/lib/openapi/routes/notifications.ts b/src/lib/openapi/routes/notifications.ts index b3ab7d429..d83d218e4 100644 --- a/src/lib/openapi/routes/notifications.ts +++ b/src/lib/openapi/routes/notifications.ts @@ -76,7 +76,7 @@ const notificationPreferencesResponse = z eventTypes: z .array(z.string()) .describe( - "The event categories this account is shown a switch for, in declaration order — the list a client renders rows for. A category nothing sends is not listed, and `preferences` carries no stored row for it.", + "The event categories this account is shown a switch for, in declaration order: the list a client renders rows for. A category nothing sends is not listed, and `preferences` carries no stored row for it.", ), eventDefaults: z .record(eventTypeEnum, z.boolean())