fix(core): harden RunEvent schema and acknowledge contract change risk

Author: bharathkumar39293

packages/core/src/v3/schemas/api-type.test.ts

@@ -204,6 +204,12 @@ describe("RunEvent Schema", () => {
     expect(result.startTime.toISOString()).toBe("2024-03-14T00:00:00.000Z");
   });
 
+  it("fails on invalid startTime", () => {
+    const event = { ...validEvent, startTime: "not-a-date" };
+    const result = RunEvent.safeParse(event);
+    expect(result.success).toBe(false);
+  });
+
   it("allows optional/null parentId", () => {
     const eventWithoutParent = { ...validEvent };
     delete (eventWithoutParent as any).parentId;

packages/core/src/v3/schemas/api.ts

@@ -1646,23 +1646,25 @@ export type TaskEventLevel = z.infer<typeof TaskEventLevel>;
 export const NanosecondTimestampSchema = z
   .union([z.string(), z.number(), z.bigint()])
   .transform((val) => {
-    // If it's already a Date, return it (though union doesn't include it)
-    if (typeof val === "object" && val instanceof Date) return val;
-
     const str = val.toString();
 
+    let date: Date;
+
     // 19 digits is nanoseconds (e.g., 1710374400000000000)
     if (str.length === 19) {
-      return new Date(Number(BigInt(str) / 1000000n));
+      date = new Date(Number(BigInt(str) / 1000000n));
+    } else if (str.length === 13) {
+      // 13 digits is milliseconds
+      date = new Date(Number(str));
+    } else {
+      // Default to standard date coercion
+      date = new Date(str);
     }
 
-    // 13 digits is milliseconds
-    if (str.length === 13) {
-      return new Date(Number(val));
-    }
-
-    // Default to standard date coercion
-    return new Date(val as any);
+    return date;
+  })
+  .refine((d) => !isNaN(d.getTime()), {
+    message: "Invalid timestamp",
   });
 
 export const RunEvent = z.object({