refactor: extract command execution logic into buildComment function

src/main.ts

@@ -13,6 +13,24 @@ export type stepResponse = { output: string; error: boolean };
 export const failedEmoji = "❌";
 export const passedEmoji = "✅";
 
+export const buildComment = async (
+  commands: { label: string; command: string }[],
+): Promise<string[]> => {
+  let commentBody = "\n";
+  let errorMessages = "";
+  for (const { label, command } of commands) {
+    const result = await runCommand(command, label);
+    if (result) {
+      commentBody += `<li>${failedEmoji} - ${label}
+<details><summary>See details</summary>${result}</details></li>`;
+      errorMessages += `${result}`;
+    } else {
+      commentBody += `<li>${passedEmoji} - ${label}\n</li>`;
+    }
+  }
+  return [commentBody, errorMessages];
+};
+
 export const runCommand = async (
   command: string,
   label: string,

src/scripts/analyze.ts

@@ -1,26 +1,14 @@
 import { setFailed } from "@actions/core";
-import { stepResponse, failedEmoji, passedEmoji, runCommand } from "src/main";
+import { stepResponse, buildComment } from "src/main";
 
 export const analyze = async (): Promise<stepResponse> => {
-  const results = [
+  const commands = [
     { label: "Custom Elements Manifest Analyzer", command: "npm run analyze" },
     { label: "ESLint", command: "npm run lint" },
     { label: "Lit Analyzer", command: "npm run lint:lit-analyzer" },
   ];
 
-  let commentBody = "\n";
-  let errorMessages = "";
-
-  for (const { label, command } of results) {
-    const result = await runCommand(command, label);
-    if (result) {
-      commentBody += `<li>${failedEmoji} - ${label}
-<details><summary>See details</summary>${result}</details></li>`;
-      errorMessages += `${result}`;
-    } else {
-      commentBody += `<li>${passedEmoji} - ${label}\n</li>`;
-    }
-  }
+  let [commentBody, errorMessages] = await buildComment(commands);
 
   if (errorMessages) {
     setFailed(errorMessages.trim());

src/scripts/formatting.ts

@@ -1,15 +1,15 @@
-import { exec } from "@actions/exec";
 import { setFailed } from "@actions/core";
-import { stepResponse, failedEmoji, passedEmoji } from "src/main";
+import { stepResponse, buildComment } from "src/main";
 
 export const formatting = async (): Promise<stepResponse> => {
-  try {
-    // Run prettier
-    await exec("npm run prettier");
+  const commands = [{ label: "Prettier", command: "npm run prettier" }];
 
-    return { output: `<li>${passedEmoji} - Formatting</li>`, error: false };
-  } catch (error) {
-    if (error instanceof Error) setFailed(error.message);
-    return { output: `<li>${failedEmoji} - Formatting</li>`, error: true };
+  let [commentBody, errorMessages] = await buildComment(commands);
+
+  if (errorMessages) {
+    setFailed(errorMessages.trim());
+    return { output: commentBody.trim(), error: true };
+  } else {
+    return { output: commentBody.trim(), error: false };
   }
 };

src/scripts/testing.ts

@@ -1,24 +1,13 @@
 import { setFailed } from "@actions/core";
-import { stepResponse, failedEmoji, passedEmoji, runCommand } from "src/main";
+import { stepResponse, buildComment } from "src/main";
 
 export const testing = async (): Promise<stepResponse> => {
-  const results = [
+  const commands = [
     { label: "Testing", command: "npm run test -- --coverage" },
     { label: "TSDoc", command: "npm run docs" },
   ];
 
-  let commentBody = "\n";
-  let errorMessages = "";
-
-  for (const { label, command } of results) {
-    const result = await runCommand(command, label);
-    if (result) {
-      commentBody += `<li>${failedEmoji} - ${label}</li>`;
-      errorMessages += `${result}`;
-    } else {
-      commentBody += `<li>${passedEmoji} - ${label}</li>`;
-    }
-  }
+  let [commentBody, errorMessages] = await buildComment(commands);
 
   if (errorMessages) {
     setFailed(errorMessages.trim());