Use constants instead of magic numbers for timeouts / retries.

Author: Codex-

src/main.ts

@@ -5,6 +5,8 @@ import * as api from "./api";
 import { LogZip } from "./zip";
 
 const DISTINCT_ID = uuid();
+const WORKFLOW_FETCH_TIMEOUT_MS = 60 * 1000;
+const WORKFLOW_LOGS_RETRY_MS = 2500;
 
 async function run(): Promise<void> {
   try {
@@ -26,7 +28,6 @@ async function run(): Promise<void> {
     await api.dispatchWorkflow(DISTINCT_ID);
 
     const timeoutMs = config.workflowTimeoutSeconds * 1000;
-    const workflowFetchTimeoutMs = 60 * 1000;
     let attemptNo = 0;
     let elapsedTime = Date.now() - startTime;
     core.info("Attempt to extract run ID from logs...");
@@ -39,7 +40,9 @@ async function run(): Promise<void> {
       // Get all runs for a given workflow ID
       const workflowRunIds = await api.retryOrDie(
         () => api.getWorkflowRunIds(workflowId),
-        workflowFetchTimeoutMs > timeoutMs ? timeoutMs : workflowFetchTimeoutMs
+        WORKFLOW_FETCH_TIMEOUT_MS > timeoutMs
+          ? timeoutMs
+          : WORKFLOW_FETCH_TIMEOUT_MS
       );
 
       core.debug(`Attempting to get logs for Run IDs: [${workflowRunIds}]`);
@@ -77,7 +80,9 @@ async function run(): Promise<void> {
         `Exhausted fetched logs for known runs, attempt ${attemptNo}...`
       );
 
-      await new Promise((resolve) => setTimeout(resolve, 1000));
+      await new Promise((resolve) =>
+        setTimeout(resolve, WORKFLOW_LOGS_RETRY_MS)
+      );
     }
 
     throw new Error("Timeout exceeded while attempting to get Run ID");