From 7a1c721d43d8418cf645f809489d1bc43080fc73 Mon Sep 17 00:00:00 2001
From: Ned Twigg <ned.twigg@diffplug.com>
Date: Sat, 31 Oct 2026 09:25:06 -0700
Subject: [PATCH] Pin the access-log story's timestamps to the story clock

`PairedPhonesAndAccessLog` built its log in a `parameters` literal, which runs at module load, before the meta's `beforeEach` pins `Date.now` to `STORY_NOW`. The log therefore carried the CI runner's wall-clock time, and every PR showed an Argos diff on `modals-remotecontrolsection--paired-phones-and-access-log`. `pairedDevices` and `accessLogEntries` now take an optional `now`, and the story passes `STORY_NOW`; `OneTimeEndedNetworkNotAllowed`'s refusal time gets the same treatment.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 lib/src/host/remote/test-burrow-link.ts          | 11 +++++++----
 lib/src/stories/RemoteControlSection.stories.tsx |  8 +++++---
 2 files changed, 12 insertions(+), 7 deletions(-)

diff --git a/lib/src/host/remote/test-burrow-link.ts b/lib/src/host/remote/test-burrow-link.ts
index a916957b2..cada13061 100644
--- a/lib/src/host/remote/test-burrow-link.ts
+++ b/lib/src/host/remote/test-burrow-link.ts
@@ -155,9 +155,10 @@ const HOUR_MS = 60 * 60_000;
  * first paired an hour ago and connected now; each after it paired a day
  * before the last and was last seen two hours after pairing. Relative to now,
  * like {@link setupQrResult}, so "Last seen" reads the same in every story.
+ * A caller that runs before a story pins the clock (a `parameters` literal is
+ * built at module load) passes its pinned `now`.
  */
-export function pairedDevices(count: number): PairedDevice[] {
-  const now = Date.now();
+export function pairedDevices(count: number, now = Date.now()): PairedDevice[] {
   return Array.from({ length: count }, (_, index) => {
     const pairedAt = now - HOUR_MS - index * 24 * HOUR_MS;
     return {
@@ -176,7 +177,10 @@ export function pairedDevices(count: number): PairedDevice[] {
  * one's pairing, the second's session that went direct and ended, a one-time
  * connection, an unpaired phone refused, and the first's session open now.
  */
-export function accessLogEntries(devices: PairedDevice[] = pairedDevices(2)): AccessLogEntry[] {
+export function accessLogEntries(
+  devices: PairedDevice[] = pairedDevices(2),
+  now = Date.now(),
+): AccessLogEntry[] {
   const entries: AccessLogEntry[] = [];
   for (const [index, { label: device, id: deviceId, pairedAt, lastSeenAt }] of devices.entries()) {
     entries.push({ at: pairedAt, event: 'paired', device, deviceId });
@@ -192,7 +196,6 @@ export function accessLogEntries(devices: PairedDevice[] = pairedDevices(2)): Ac
       { at: lastSeenAt, event: 'ended', device, deviceId, path: 'direct', durationMs: 4 * 60_000 },
     );
   }
-  const now = Date.now();
   const oneTime = 'Dormouse Pocket (browser)';
   entries.push(
     { at: now - 3 * HOUR_MS, event: 'one-time-connected', device: oneTime, path: 'direct' },
diff --git a/lib/src/stories/RemoteControlSection.stories.tsx b/lib/src/stories/RemoteControlSection.stories.tsx
index 8e6dcdd7d..c465d3f92 100644
--- a/lib/src/stories/RemoteControlSection.stories.tsx
+++ b/lib/src/stories/RemoteControlSection.stories.tsx
@@ -39,7 +39,9 @@ function RemoteControlStory() {
 }
 
 /** The clock every story here reads. A setup code encodes its expiry, so a real
- *  clock would draw a different QR on every run. */
+ *  clock would draw a different QR on every run. `beforeEach` pins it only
+ *  around render: a `parameters` literal is built at module load, under the
+ *  real clock, so a timestamp there must use `STORY_NOW` itself. */
 const STORY_NOW = Date.UTC(2026, 0, 1);
 
 const meta: Meta<typeof RemoteControlStory> = {
@@ -342,7 +344,7 @@ export const PairedPhonesAndAccessLog: Story = {
   parameters: {
     primedBurrow: {
       status: enrolledStatus({ pairedClients: 2 }),
-      accessLog: accessLogEntries(pairedDevices(2)),
+      accessLog: accessLogEntries(pairedDevices(2, STORY_NOW), STORY_NOW),
     },
     docs: { story: { height: '620px' } },
   },
@@ -703,7 +705,7 @@ export const OneTimeEndedNetworkNotAllowed: Story = {
       oneTime: {
         status: 'ended',
         reason: 'network-not-allowed',
-        refusal: { at: Date.now(), kind: 'path-refused', end: 'remote', address: '172.58.12.9', addressSource: 'observed' },
+        refusal: { at: STORY_NOW, kind: 'path-refused', end: 'remote', address: '172.58.12.9', addressSource: 'observed' },
       },
     },
     docs: { story: { height: '360px' } },