fix: When issue updates fail, immediately halt execution (#23)

Fixes the issue I flagged in
github/continuous-ai-for-accessibility#18 (comment)
(Hubber access only):
> I discovered today that workflows can be green even if “File” fails to
file anything (e.g. if you provide a `token` without “contents: write”
access to the provided `repository`):
https://github.com/github/accessibility-sandbox/actions/runs/17081630847
[(Hubber access only)]

This PR makes it so that—as soon as issue closure, creation, reopen, or
assignment fails—the workflow exits immediately (i.e. subsequent issue
updates are not attempted) and when it exits, it is red.

Implementation details:
-
[`core.error`](https://github.com/actions/toolkit/blob/227b1ce741a925ae9f5fdef91fd10415f2d0ca45/packages/core/src/core.ts#L270-L284)
— note this neither sets an exit code nor exits the process.
-
[`core.setFailed`](https://github.com/actions/toolkit/blob/227b1ce741a925ae9f5fdef91fd10415f2d0ca45/packages/core/src/core.ts#L240-L249)
— note this calls `core.error` and sets an exit code, but it _doesn’t_
exit the process.

Author: smockle

.github/actions/file/src/index.ts

@@ -1,4 +1,5 @@
 import type { Finding } from "./types.d.js";
+import process from "node:process";
 import core from "@actions/core";
 import { Octokit } from '@octokit/core';
 import { toFindingsMap } from "./toFindingsMap.js"
@@ -31,7 +32,8 @@ export default async function () {
         closedIssueUrls.push(response.data.html_url);
         core.info(`Closed issue: ${response.data.title} (${repoWithOwner}#${response.data.number})`);
       } catch (error) {
-        core.error(`Failed to close issue for finding: ${error}`);
+        core.setFailed(`Failed to close issue for finding: ${error}`);
+        process.exit(1);
       }
     }
   }
@@ -52,7 +54,8 @@ export default async function () {
         core.info(`Created issue: ${response.data.title} (${repoWithOwner}#${response.data.number})`);
       }
     } catch (error) {
-      core.error(`Failed to open/reopen issue for finding: ${error}`);
+      core.setFailed(`Failed to open/reopen issue for finding: ${error}`);
+      process.exit(1);
     }
   }
 

.github/actions/fix/src/index.ts

@@ -1,3 +1,4 @@
+import process from "node:process";
 import core from "@actions/core";
 import { Octokit } from '@octokit/core';
 import { fixIssue } from "./fixIssue.js";
@@ -16,7 +17,8 @@ export default async function () {
       await fixIssue(octokit, repoWithOwner, issueUrl);
       core.info(`Assigned ${repoWithOwner}#${issueUrl.split('/').pop()} to Copilot!`);
     } catch (error) {
-      core.error(`Failed to assign ${repoWithOwner}#${issueUrl.split('/').pop()} to Copilot: ${error}`);
+      core.setFailed(`Failed to assign ${repoWithOwner}#${issueUrl.split('/').pop()} to Copilot: ${error}`);
+      process.exit(1);
     }
   }
   core.info("Finished 'fix' action");