refactor: improve logging

Author: Codex-

src/__snapshots__/return-dispatch.spec.ts.snap

@@ -2,7 +2,7 @@
 
 exports[`return-dispatch > handleAction > getRunIdAndUrl > called fetchWorkflowRunIds with the provided workflowId and branch 1`] = `
 [
-  "Attempting to identify Run ID for workflow.yml (123)",
+  "Attempting to identify run ID for workflow.yml (123)",
 ]
 `;
 
@@ -20,7 +20,7 @@ exports[`return-dispatch > handleAction > getRunIdAndUrl > called fetchWorkflowR
 
 exports[`return-dispatch > handleAction > getRunIdAndUrl > should call retryOrTimeout with the larger workflowTimeoutMs timeout value 1`] = `
 [
-  "Attempting to identify Run ID for workflow.yml (123)",
+  "Attempting to identify run ID for workflow.yml (123)",
 ]
 `;
 
@@ -40,15 +40,15 @@ exports[`return-dispatch > handleAction > getRunIdAndUrl > should retry until an
 
 exports[`return-dispatch > handleAction > getRunIdAndUrl > should retry until an ID is found 2`] = `"No Run IDs found for workflow, attempt 1..."`;
 
-exports[`return-dispatch > handleAction > getRunIdAndUrl > should retry until an ID is found 3`] = `"Attempting to identify Run ID for workflow.yml (123)"`;
+exports[`return-dispatch > handleAction > getRunIdAndUrl > should retry until an ID is found 3`] = `"Attempting to identify run ID for workflow.yml (123)"`;
 
 exports[`return-dispatch > handleAction > getRunIdAndUrl > should retry until an ID is found 4`] = `"No Run IDs found for workflow, attempt 2..."`;
 
 exports[`return-dispatch > handleAction > getRunIdAndUrl > should retry until an ID is found 5`] = `"Attempting to get step names for Run IDs: [0]"`;
 
 exports[`return-dispatch > handleAction > getRunIdAndUrl > should return the ID when found 1`] = `
 [
-  "Attempting to identify Run ID for workflow.yml (123)",
+  "Attempting to identify run ID for workflow.yml (123)",
 ]
 `;
 
@@ -66,7 +66,7 @@ exports[`return-dispatch > handleAction > getRunIdAndUrl > should return the ID
 
 exports[`return-dispatch > handleAction > getRunIdAndUrl > should timeout when unable failing to get the run IDs 1`] = `
 [
-  "Attempting to identify Run ID for workflow.yml (123)",
+  "Attempting to identify run ID for workflow.yml (123)",
 ]
 `;
 
@@ -80,7 +80,7 @@ exports[`return-dispatch > handleAction > getRunIdAndUrl > should timeout when u
 
 exports[`return-dispatch > handleAction > getRunIdAndUrl > should timeout when unable to find over time 2`] = `"Exhausted searching IDs in known runs, attempt 1..."`;
 
-exports[`return-dispatch > handleAction > getRunIdAndUrl > should timeout when unable to find over time 3`] = `"Attempting to identify Run ID for workflow.yml (123)"`;
+exports[`return-dispatch > handleAction > getRunIdAndUrl > should timeout when unable to find over time 3`] = `"Attempting to identify run ID for workflow.yml (123)"`;
 
 exports[`return-dispatch > handleAction > getRunIdAndUrl > should timeout when unable to find over time 4`] = `"Attempting to get step names for Run IDs: [0]"`;
 

src/api.spec.ts

@@ -547,7 +547,7 @@ describe("API", () => {
           Repository: owner/repo
           Workflow Run ID: 0
           Jobs Fetched: [0]
-          Steps Fetched: [Test Step 1, Test Step 2]"
+          Steps Fetched: ["Test Step 1", "Test Step 2"]"
       `,
       );
     });

src/api.ts

@@ -244,7 +244,7 @@ export async function fetchWorkflowRunJobSteps(
         `  Repository: ${config.owner}/${config.repo}\n` +
         `  Workflow Run ID: ${runId}\n` +
         `  Jobs Fetched: [${jobs.map((job) => job.id).join(", ")}]\n` +
-        `  Steps Fetched: [${steps.join(", ")}]`,
+        `  Steps Fetched: [${steps.map((step) => `"${step}"`).join(", ")}]`,
     );
 
     return steps;

src/return-dispatch.spec.ts

@@ -593,7 +593,7 @@ describe("return-dispatch", () => {
         assertOnlyCalled(coreDebugLogMock, coreInfoLogMock);
         expect(coreDebugLogMock).toHaveBeenCalledTimes(2);
         expect(coreDebugLogMock.mock.calls[0]?.[0]).toMatchInlineSnapshot(
-          `"Attempting to identify Run ID for workflow.yml (123)"`,
+          `"Attempting to identify run ID for workflow.yml (123)"`,
         );
         expect(coreDebugLogMock.mock.calls[1]?.[0]).toMatchInlineSnapshot(
           `"Attempting to get step names for Run IDs: [0]"`,

src/return-dispatch.ts

@@ -146,7 +146,7 @@ export async function getRunIdAndUrl({
   );
 
   core.info("Attempting to identify run ID from steps...");
-  core.debug(`Attempting to identify Run ID for ${workflow} (${workflowId})`);
+  core.debug(`Attempting to identify run ID for ${workflow} (${workflowId})`);
 
   let attemptNo = 0;
   let elapsedTime = Date.now() - startTime;