From 7b17ec819a3f5603c6aa18a20889f5b8eb866b04 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marc-Andr=C3=A9=20Bombeck?= Date: Mon, 19 Oct 2026 14:57:00 +0200 Subject: [PATCH] chore: drop personal names from comments and test fixtures Comments referred to the maintainer by name, and a test fixture carried a full real name. Comments describe the behaviour; fixtures use placeholders. --- src/app/page-client.tsx | 2 +- .../daily/__tests__/today-hero.test.tsx | 2 +- src/components/daily/today-hero.tsx | 2 +- .../document-coach-launch-wiring.test.ts | 2 +- .../documents/__tests__/auto-stage-labs.test.ts | 2 +- src/lib/documents/auto-stage-labs.ts | 2 +- src/lib/jobs/__tests__/coach-nudge.test.ts | 16 ++++++++-------- 7 files changed, 14 insertions(+), 14 deletions(-) diff --git a/src/app/page-client.tsx b/src/app/page-client.tsx index 1e32be413..9fae5214a 100644 --- a/src/app/page-client.tsx +++ b/src/app/page-client.tsx @@ -295,7 +295,7 @@ export default function DashboardPageClient({ const digestQuery = useDailyDigest(isAuthenticated); // v1.29.1 — the v1.29.0 selected-score-ring cluster is removed from the web - // hero (Marc, live-use: uneven, wasted tile space). The snapshot still + // hero (live use: uneven, wasted tile space). The snapshot still // resolves `scoreRings` server-side for the iOS selection contract; the web // hero just no longer renders them, so no client-side ring ordering here. diff --git a/src/components/daily/__tests__/today-hero.test.tsx b/src/components/daily/__tests__/today-hero.test.tsx index a334795ed..7e58ad751 100644 --- a/src/components/daily/__tests__/today-hero.test.tsx +++ b/src/components/daily/__tests__/today-hero.test.tsx @@ -318,7 +318,7 @@ describe("", () => { }); // v1.29.1 — the v1.29.0 selected-score-ring cluster was removed from the web - // hero (Marc, live-use: uneven, wasted tile space). Only the main + // hero (live use: uneven, wasted tile space). Only the main // health-score ring paints now; the cluster's data-slots are gone. it("renders no score-ring cluster — only the health-score ring", () => { const html = render(); diff --git a/src/components/daily/today-hero.tsx b/src/components/daily/today-hero.tsx index d2325fd0c..936232eef 100644 --- a/src/components/daily/today-hero.tsx +++ b/src/components/daily/today-hero.tsx @@ -14,7 +14,7 @@ * - the day's read: the health `ScoreRing` (`flat`, md) with its * server-computed band and an honest provisional/final face — a null * score paints the ring's own provisional state, never a zero; - * - v1.29.1 (Marc, live-use): the v1.29.0 selected-score-ring cluster is + * - v1.29.1 (live use): the v1.29.0 selected-score-ring cluster is * removed from the web hero — it read as uneven and wasted tile space. * The health `ScoreRing` is the hero's only ring now. The score-ring * SELECTION contract stays server-side (`selectedScoreRings` on the diff --git a/src/components/documents/__tests__/document-coach-launch-wiring.test.ts b/src/components/documents/__tests__/document-coach-launch-wiring.test.ts index f0b59d9ce..626fc38a0 100644 --- a/src/components/documents/__tests__/document-coach-launch-wiring.test.ts +++ b/src/components/documents/__tests__/document-coach-launch-wiring.test.ts @@ -5,7 +5,7 @@ import { join } from "node:path"; /** * v1.28.52 (Documents R3) — "Ask the Coach" opens the REAL fenced coach * conversation in the shared SIDE DRAWER, scoped to the document, instead of a - * full-page nav to `/coach?doc=` (the v1.28.51 behaviour Marc corrected). + * full-page nav to `/coach?doc=` (the v1.28.51 behaviour, since corrected). * * The wiring is source-structural (the send-path fence itself is covered by the * coach-send-target unit tests — untouched here). These guards pin: diff --git a/src/lib/documents/__tests__/auto-stage-labs.test.ts b/src/lib/documents/__tests__/auto-stage-labs.test.ts index ebcde0792..90a9943f0 100644 --- a/src/lib/documents/__tests__/auto-stage-labs.test.ts +++ b/src/lib/documents/__tests__/auto-stage-labs.test.ts @@ -1,7 +1,7 @@ import { beforeEach, describe, expect, it, vi } from "vitest"; /** - * S8 — auto-stage lab facts after the auto-index job. Pins Marc's sign-off + * S8 — auto-stage lab facts after the auto-index job. Pins the sign-off * (auto-STAGE, never auto-COMMIT), the double module gate, the reuse of the * auto-index consent gate, the "looks like a lab" heuristic, idempotency (a * non-STORED or already-staged document is left alone), and the guarded diff --git a/src/lib/documents/auto-stage-labs.ts b/src/lib/documents/auto-stage-labs.ts index fc694a56e..a30714219 100644 --- a/src/lib/documents/auto-stage-labs.ts +++ b/src/lib/documents/auto-stage-labs.ts @@ -5,7 +5,7 @@ * document that LOOKS LIKE a lab result is run through the SAME inbound * extraction the manual "Extract" button uses — but the facts land PENDING for * the existing human-review screen. Nothing is committed here: the user still - * taps one confirm before a value becomes a `LabResult`. Marc's sign-off is + * taps one confirm before a value becomes a `LabResult`. The sign-off is * explicit — auto-STAGE, never auto-COMMIT. * * Gates (ALL must hold, otherwise the manual extract button is left untouched): diff --git a/src/lib/jobs/__tests__/coach-nudge.test.ts b/src/lib/jobs/__tests__/coach-nudge.test.ts index 8543a4340..074263377 100644 --- a/src/lib/jobs/__tests__/coach-nudge.test.ts +++ b/src/lib/jobs/__tests__/coach-nudge.test.ts @@ -279,19 +279,19 @@ describe("buildCoachNudgePayload", () => { it("opens the title with a localized morning greeting and the name", () => { const { title, body } = buildCoachNudgePayload("bp", "de", { - name: "Marc", + name: "Alex", openerSeed: 0, }); - expect(title).toBe("Guten Morgen, Marc"); + expect(title).toBe("Guten Morgen, Alex"); expect(body.length).toBeGreaterThan(0); }); it("rotates to the casual opener on an odd seed", () => { const { title } = buildCoachNudgePayload("bp", "de", { - name: "Marc", + name: "Alex", openerSeed: 1, }); - expect(title).toBe("Hey Marc"); + expect(title).toBe("Hey Alex"); }); it("drops the name from the greeting when none is given", () => { @@ -361,11 +361,11 @@ describe("resolveGreetingName", () => { it("prefers the display name's first token", () => { expect( resolveGreetingName({ - displayName: "Marc B.", - fullName: "Marc Bombeck", - username: "marc", + displayName: "Alex B.", + fullName: "Alex Example", + username: "alex", }), - ).toBe("Marc"); + ).toBe("Alex"); }); it("falls back to the full name's given token", () => {