fix(CLI-BH): seed dedup set from initial fetch, fix test reproducibility

- Add onInitialLogs callback to FollowConfig so trace follow mode
  seeds the seenWithoutTs set from the initial batch, preventing
  duplicates on the first poll cycle.
- Replace Math.random() in property tests with fast-check combinators
  (oneof, constant, tuple) for deterministic reproducibility.

Author: BYK

src/commands/log/list.ts

@@ -195,6 +195,11 @@ type FollowConfig<T extends LogLike> = {
   fetch: (statsPeriod: string, afterTimestamp?: number) => Promise<T[]>;
   /** Extract only the genuinely new entries from a poll response */
   extractNew: (logs: T[], lastTimestamp: number) => T[];
+  /**
+   * Called with the initial batch of logs before polling begins.
+   * Use this to seed dedup state (e.g., tracking seen log IDs).
+   */
+  onInitialLogs?: (logs: T[]) => void;
 };
 
 /**
@@ -315,6 +320,7 @@ function executeFollowMode<T extends LogLike>(
         if (initialLogs[0]) {
           lastTimestamp = initialLogs[0].timestamp_precise ?? lastTimestamp;
         }
+        config.onInitialLogs?.(initialLogs);
         scheduleNextPoll();
       })
       .catch((error: unknown) => {
@@ -479,6 +485,13 @@ export const listCommand = buildListCommand("log", {
               seenWithoutTs.add(l.id);
               return true;
             }),
+          onInitialLogs: (logs) => {
+            for (const l of logs) {
+              if (l.timestamp_precise === undefined) {
+                seenWithoutTs.add(l.id);
+              }
+            }
+          },
         });
       } else {
         await executeTraceSingleFetch(stdout, org, flags.trace, flags);

test/lib/trace-log-schema.property.test.ts

@@ -8,12 +8,14 @@
 
 import { describe, expect, test } from "bun:test";
 import {
+  constant,
   assert as fcAssert,
   nat,
   oneof,
   option,
   property,
   string,
+  tuple,
 } from "fast-check";
 import {
   DetailedSentryLogSchema,
@@ -33,33 +35,30 @@ function numberOrString(arb = nat()) {
 }
 
 /**
- * Pick a random representation for a numeric field:
- * number, string, or undefined (omitted).
+ * Arbitrary that produces a number as either the number itself, its
+ * string representation, or undefined — simulating optional numeric
+ * fields that the API may omit or serialize as strings.
  */
-function randomNumericField(value: number): number | string | undefined {
-  const r = Math.random();
-  if (r < 0.3) return;
-  if (r < 0.65) return value;
-  return String(value);
+function optionalNumberOrString(arb = nat()) {
+  return oneof(arb, arb.map(String), constant(undefined));
 }
 
 /** Arbitrary for a valid trace-log entry with mixed number/string numeric fields */
-const traceLogEntryArb = nat().chain((projectId) =>
-  nat().chain((sevNum) =>
-    nat().chain((tsPrecise) =>
-      option(string(), { nil: undefined }).map((msg) => ({
-        id: "test-log-id",
-        "project.id": Math.random() > 0.5 ? projectId : String(projectId),
-        trace: "aaaa1111bbbb2222cccc3333dddd4444",
-        severity_number: randomNumericField(sevNum),
-        severity: "info",
-        timestamp: "2025-01-30T14:32:15+00:00",
-        timestamp_precise: randomNumericField(tsPrecise),
-        message: msg ?? null,
-      }))
-    )
-  )
-);
+const traceLogEntryArb = tuple(
+  numberOrString(),
+  optionalNumberOrString(),
+  optionalNumberOrString(),
+  option(string(), { nil: undefined })
+).map(([projectId, sevNum, tsPrecise, msg]) => ({
+  id: "test-log-id",
+  "project.id": projectId,
+  trace: "aaaa1111bbbb2222cccc3333dddd4444",
+  severity_number: sevNum,
+  severity: "info",
+  timestamp: "2025-01-30T14:32:15+00:00",
+  timestamp_precise: tsPrecise,
+  message: msg ?? null,
+}));
 
 describe("property: TraceLogSchema coercion", () => {
   test("always parses successfully with number or string numeric fields", () => {