fix: correctly parse a limit of 0 and handle negative limit

Admyral-Security · Nov 20, 2024 · f932b89 · f932b89
1 parent c10ead9
commit f932b89
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/web/src/hooks/use-list-workflow-runs-api.ts b/web/src/hooks/use-list-workflow-runs-api.ts
@@ -8,7 +8,7 @@ import { HTTPMethod } from "@/types/api";
 
 // GET /api/v1/runs/<workflow-id>
 const ListWorkflowRunsRequest = z.object({
-	limit: z.number().optional(),
+	limit: z.number().min(0).optional(),
 });
 const ListWorkflowRunsResponse = z.array(WorkflowRunMetadata);
 
@@ -29,7 +29,7 @@ export const useListWorkflowRunsApi = (workflowId: string, limit?: number) => {
 	return useQuery({
 		queryKey: ["workflowRuns", workflowId, limit],
 		queryFn: () => {
-			const params = limit ? { limit } : {};
+			const params = typeof limit === "number" ? { limit } : {};
 			return buildListWorkflowRunsApi(workflowId)(params);
 		},
 		refetchInterval: REFETCH_INTERVAL_1_SECOND,