fix: satisfy cron retry CI checks

joshavant · joshavant · commit 121ca8b2a62d · 2026-06-11T15:39:34.000-05:00
diff --git a/src/cron/service/timer.ts b/src/cron/service/timer.ts
@@ -1166,7 +1166,7 @@ function isRunnableJob(params: {
     const lastRun = job.state.lastRunAtMs;
     const nextRun = job.state.nextRunAtMs;
     if (isScheduledTerminalOneShotRetry(job, lastRunStatus, lastRun, nextRun)) {
-      return nowMs >= nextRun;
+      return typeof nextRun === "number" && nowMs >= nextRun;
     }
     return false;
   }
diff --git a/src/gateway/server-cron.test.ts b/src/gateway/server-cron.test.ts
@@ -44,23 +44,23 @@ const {
   retireSessionMcpRuntimeMock: vi.fn(async () => true),
 }));
 
-function enqueueSystemEvent(...args: unknown[]) {
-  return enqueueSystemEventMock(...args);
+function enqueueSystemEvent(text: string, opts?: unknown) {
+  return enqueueSystemEventMock(text, opts);
 }
 
-function enqueueSystemEventEntry(...args: unknown[]) {
-  const result = enqueueSystemEventMock(...args);
+function enqueueSystemEventEntry(text: string, opts?: unknown) {
+  const result = enqueueSystemEventMock(text, opts);
   if (result === false || result === null) {
     return null;
   }
   return {
-    text: String(args[0] ?? ""),
+    text,
     ts: Date.now(),
   };
 }
 
-function consumeSelectedSystemEventEntries(...args: unknown[]) {
-  return consumeSelectedSystemEventEntriesMock(...args);
+function consumeSelectedSystemEventEntries(sessionKey: string, entries: readonly unknown[]) {
+  return consumeSelectedSystemEventEntriesMock(sessionKey, entries);
 }
 
 function requestHeartbeat(...args: unknown[]) {

Original file line number	Diff line number	Diff line change
`@@ -1166,7 +1166,7 @@ function isRunnableJob(params: {`
`1166`	`1166`	`const lastRun = job.state.lastRunAtMs;`
`1167`	`1167`	`const nextRun = job.state.nextRunAtMs;`
`1168`	`1168`	`if (isScheduledTerminalOneShotRetry(job, lastRunStatus, lastRun, nextRun)) {`
`1169`		`- return nowMs >= nextRun;`
	`1169`	`+ return typeof nextRun === "number" && nowMs >= nextRun;`
`1170`	`1170`	`}`
`1171`	`1171`	`return false;`
`1172`	`1172`	`}`