fix(api): simplify date comparison in getDueCronTriggers using lte for improved clarity

Author: bchapuis

apps/api/src/db/queries.ts

@@ -6,7 +6,7 @@ import {
   WorkflowExecutionStatus,
 } from "@dafthunk/types";
 import * as crypto from "crypto";
-import { and, desc, eq, inArray, SQL, sql } from "drizzle-orm";
+import { and, desc, eq, inArray, lte, SQL, sql } from "drizzle-orm";
 import { alias } from "drizzle-orm/sqlite-core";
 import { v7 as uuidv7 } from "uuid";
 
@@ -1207,12 +1207,7 @@ export async function getDueCronTriggers(
       selectedDeployment: selectedDeployment,
     })
     .from(cronTriggers)
-    .where(
-      and(
-        eq(cronTriggers.active, true),
-        sql`${cronTriggers.nextRunAt} <= ${Math.floor(now.getTime() / 1000)}`
-      )
-    )
+    .where(and(eq(cronTriggers.active, true), lte(cronTriggers.nextRunAt, now)))
     .innerJoin(workflows, eq(workflows.id, cronTriggers.workflowId))
     .innerJoin(latestByWorkflow, eq(latestByWorkflow.workflowId, workflows.id))
     .innerJoin(