fix(core): readsAsWallClock answers from the spec's ClockTimeValueSchema

The private wall-clock regex in temporal-comparand.ts was a second spelling
of the time-of-day set the spec's ClockTimeValueSchema defines. It now asks
that schema, so one rule answers the comparand door, the record validator's
time arm and the spec's time default gate. Pins: 23:59:59.999 read, 24:00
and 10:00Z refused, through isUninterpretableTemporalComparand.

Claude-Session: https://claude.ai/code/session_01DEvba2nBuD4tWzfq8r8NFY
Co-authored-by: Claude <noreply@anthropic.com>

Author: claude

packages/core/src/utils/temporal-comparand.test.ts

@@ -372,3 +372,21 @@ describe('[#20480] isUninterpretableTemporalComparand — a time column\'s insta
     }
   });
 });
+
+// [#20771] The bare-wall-clock half of the `time` rule is the spec's
+// `ClockTimeValueSchema` (`@objectstack/spec/data`), not a second regex in
+// core: the private `readsAsWallClock` asks that schema. These are the
+// boundary cases on each side, asserted through the one public door that
+// reaches `readsAsWallClock`. None of them is an ISO instant, so the
+// instant half of the rule cannot rescue one: the verdict is the wall-clock
+// half's alone.
+describe('[#20771] isUninterpretableTemporalComparand — the wall-clock half is the spec\'s time-of-day rule', () => {
+  it('reads the last wall clock of the day, to the fraction', () => {
+    expect(isUninterpretableTemporalComparand('time', '23:59:59.999')).toBe(false);
+  });
+
+  it('refuses the hour past the last one, and a time of day with a zone', () => {
+    expect(isUninterpretableTemporalComparand('time', '24:00')).toBe(true);
+    expect(isUninterpretableTemporalComparand('time', '10:00Z')).toBe(true);
+  });
+});

packages/core/src/utils/temporal-comparand.ts

@@ -134,7 +134,7 @@
  *   test), so it is left alone too.
  */
 
-import { classifyFilterToken } from '@objectstack/spec/data';
+import { ClockTimeValueSchema, classifyFilterToken } from '@objectstack/spec/data';
 import { isOutsideTemporalYearRange, temporalStorageForm } from './temporal-storage-form.js';
 
 /** Which temporal storage rule a declared field takes. */
@@ -246,11 +246,16 @@ function readsAsCalendarDay(s: string): boolean {
  * `temporalStorageForm`'s `time` reading: a bare wall clock whose components are in
  * range. Out-of-range (`25:00`) is uninterpretable — the rule it mirrors
  * returns such a value untouched rather than wrapping it.
+ *
+ * [#20771] The set is the spec's: `ClockTimeValueSchema`, the `time` stored
+ * form — `HH:MM[:SS[.fraction]]`, hours 00..23, minutes and seconds 00..59, no
+ * time zone (ADR-0053 D-C1). This function asks that schema rather than
+ * spelling a second regex, so the comparand door, the record validator's
+ * `time` arm and the spec's `time` default gate answer from one rule and
+ * cannot drift apart on an edit to either side.
  */
 function readsAsWallClock(s: string): boolean {
-  const m = /^(\d{2}):(\d{2})(?::(\d{2})(?:\.(\d+))?)?$/.exec(s);
-  if (!m) return false;
-  return Number(m[1]) <= 23 && Number(m[2]) <= 59 && Number(m[3] ?? '0') <= 59;
+  return ClockTimeValueSchema.safeParse(s).success;
 }
 
 /**