update .gitignore, enhance VSCode launch configuration, and refactor …

…error handling in main.ts and testing.ts

.gitignore

@@ -104,4 +104,5 @@ __tests__/runner/*
 # TS Docs
 docs
 
+# VSCode
 .vscode

.vscode/launch.json

@@ -7,12 +7,13 @@
       "request": "launch",
       "runtimeExecutable": "npx",
       "cwd": "${workspaceRoot}",
-      "args": ["local-action", ".", "src/main.ts", ".env"],
+      "args": ["local-action", ".", "src/main.ts", ".env", "-- --local"],
       "console": "integratedTerminal",
       "skipFiles": ["<node_internals>/**", "node_modules/**"],
       "env": {
         "NODE_OPTIONS": "--experimental-specifier-resolution=node"
-      }
+      },
+      "preLaunchTask": "npm: install"
     }
   ]
 }

src/main.ts

@@ -1,4 +1,4 @@
-import { getBooleanInput, getInput, setFailed } from "@actions/core";
+import { getBooleanInput, getInput, setFailed, debug } from "@actions/core";
 import { exec } from "@actions/exec";
 import { getOctokit, context } from "@actions/github";
 import { analyze } from "./scripts/analyze";
@@ -13,6 +13,26 @@ export type stepResponse = { output: string; error: boolean };
 export const failedEmoji = "❌";
 export const passedEmoji = "✅";
 
+export const runCommand = async (
+  command: string,
+  label: string,
+): Promise<string | boolean> => {
+  try {
+    await exec(command);
+    return false;
+  } catch (error: unknown) {
+    if (error instanceof Error) {
+      debug(`${label} failed: ${error.message}`);
+      return error.message;
+    } else if (typeof error === "string") {
+      debug(`${label} failed: ${error}`);
+      return error;
+    } else {
+      return true;
+    }
+  }
+};
+
 /**
  * The main function for the action.
  * @returns {Promise<void>} Resolves when the action is complete.

src/scripts/analyze.ts

@@ -1,26 +1,5 @@
-import { exec } from "@actions/exec";
-import { setFailed, debug } from "@actions/core";
-import { stepResponse, failedEmoji, passedEmoji } from "src/main";
-
-const runCommand = async (
-  command: string,
-  label: string,
-): Promise<string | boolean> => {
-  try {
-    await exec(command);
-    return false;
-  } catch (error: unknown) {
-    if (error instanceof Error) {
-      debug(`${label} failed: ${error.message}`);
-      return error.message;
-    } else if (typeof error === "string") {
-      debug(`${label} failed: ${error}`);
-      return error;
-    } else {
-      return true;
-    }
-  }
-};
+import { setFailed } from "@actions/core";
+import { stepResponse, failedEmoji, passedEmoji, runCommand } from "src/main";
 
 export const analyze = async (): Promise<stepResponse> => {
   const results = [

src/scripts/testing.ts

@@ -1,18 +1,30 @@
 import { exec } from "@actions/exec";
 import { setFailed } from "@actions/core";
-import { stepResponse } from "src/main";
+import { stepResponse, failedEmoji, passedEmoji, runCommand } from "src/main";
 
 export const testing = async (): Promise<stepResponse> => {
-  try {
-    // Run tests and generate coverage
-    await exec("npm run test -- --coverage");
+  const results = [
+    { label: "Testing", command: "npm run test -- --coverage" },
+    { label: "TSDoc", command: "npm run docs" },
+  ];
 
-    // Test tsdoc
-    await exec("npm run docs");
+  let commentBody = "\n";
+  let errorMessages = "";
 
-    return { output: "Testing complete", error: false };
-  } catch (error) {
-    if (error instanceof Error) setFailed(error.message);
-    return { output: "Testing failed", error: true };
+  for (const { label, command } of results) {
+    const result = await runCommand(command, label);
+    if (result) {
+      commentBody += `${failedEmoji} - ${label}\n`;
+      errorMessages += `${result}\n`;
+    } else {
+      commentBody += `${passedEmoji} - ${label}\n`;
+    }
+  }
+
+  if (errorMessages) {
+    setFailed(errorMessages.trim());
+    return { output: commentBody.trim(), error: true };
+  } else {
+    return { output: commentBody.trim(), error: false };
   }
 };