fix(aiRunFilterService): enhance date validation in AI filter response

- Improved date handling in the AIRunFilterService to ensure that invalid date formats do not silently pass through.
- Added error logging for invalid date inputs and updated the response structure to reflect success or failure based on date validation.
- Adjusted the test timeout for runsReplicationService to improve reliability in long-running tests.

Author: danton267

apps/webapp/app/v3/services/aiRunFilterService.server.ts

@@ -268,14 +268,35 @@ export class AIRunFilterService {
         };
       }
 
+      // `from`/`to` are validated as strings, so a malformed value (e.g. the
+      // model returning a non-ISO date) would survive safeParse and then
+      // produce NaN here. NaN serializes to `null` over JSON, silently dropping
+      // the date constraint while still reporting success — so reject it.
+      const from = validationResult.data.from
+        ? new Date(validationResult.data.from).getTime()
+        : undefined;
+      const to = validationResult.data.to
+        ? new Date(validationResult.data.to).getTime()
+        : undefined;
+
+      if ((from !== undefined && Number.isNaN(from)) || (to !== undefined && Number.isNaN(to))) {
+        logger.error("AI filter returned an invalid datetime", {
+          from: validationResult.data.from,
+          to: validationResult.data.to,
+        });
+
+        return {
+          success: false,
+          error: "AI response contained an invalid date",
+        };
+      }
+
       return {
         success: true,
         filters: {
           ...validationResult.data,
-          from: validationResult.data.from
-            ? new Date(validationResult.data.from).getTime()
-            : undefined,
-          to: validationResult.data.to ? new Date(validationResult.data.to).getTime() : undefined,
+          from,
+          to,
         },
       };
     } catch (error) {

apps/webapp/test/runsReplicationService.part2.test.ts

@@ -513,6 +513,7 @@ describe("RunsReplicationService (part 2/2)", () => {
 
   containerTest(
     "should be able to handle processing transactions for a long period of time",
+    { timeout: 60_000 * 5 },
     async ({ clickhouseContainer, redisOptions, postgresContainer, prisma }) => {
       await prisma.$executeRawUnsafe(`ALTER TABLE public."TaskRun" REPLICA IDENTITY FULL;`);
 
@@ -614,8 +615,7 @@ describe("RunsReplicationService (part 2/2)", () => {
       expect(result?.length).toBeGreaterThanOrEqual(50);
 
       await runsReplicationService.stop();
-    },
-    { timeout: 60_000 * 5 }
+    }
   );
 
   containerTest(