fix(Fix): Allow empty array inputs

Author: smockle

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

@@ -10,18 +10,20 @@ const OctokitWithThrottling = Octokit.plugin(throttling);
 export default async function () {
   core.info("Started 'fix' action");
   /** @deprecated Use `issues` instead. */
-  const issueUrls: string[] = JSON.parse(
-    core.getInput('issue_urls', { required: false }) || "[]"
+  const issueUrls: string[] | null = JSON.parse(
+    core.getInput("issue_urls", { required: false }) || "null"
   );
-  const issues: IssueInput[] = JSON.parse(
-    core.getInput('issues', { required: false }) || JSON.stringify(issueUrls.map(url => ({ url })))
+  const issues: IssueInput[] | null = JSON.parse(
+    core.getInput("issues", { required: false }) || "null"
   );
-  if (issues.length === 0) {
-    core.setFailed("Neither 'issues' nor 'issue_urls' was provided, but one is required.");
+  if (issueUrls === null && issues === null) {
+    core.setFailed(
+      "Neither 'issues' nor 'issue_urls' was provided, but one is required."
+    );
     process.exit(1);
   }
-  const repoWithOwner = core.getInput('repository', { required: true });
-  const token = core.getInput('token', { required: true });
+  const repoWithOwner = core.getInput("repository", { required: true });
+  const token = core.getInput("token", { required: true });
   core.debug(`Input: 'issues: ${JSON.stringify(issues)}'`);
   core.debug(`Input: 'issue_urls: ${JSON.stringify(issueUrls)}'`);
   core.debug(`Input: 'repository: ${repoWithOwner}'`);
@@ -30,26 +32,34 @@ export default async function () {
     auth: token,
     throttle: {
       onRateLimit: (retryAfter, options, octokit, retryCount) => {
-        octokit.log.warn(`Request quota exhausted for request ${options.method} ${options.url}`);
+        octokit.log.warn(
+          `Request quota exhausted for request ${options.method} ${options.url}`
+        );
         if (retryCount < 3) {
           octokit.log.info(`Retrying after ${retryAfter} seconds!`);
           return true;
         }
       },
       onSecondaryRateLimit: (retryAfter, options, octokit, retryCount) => {
-        octokit.log.warn(`Secondary rate limit hit for request ${options.method} ${options.url}`);
+        octokit.log.warn(
+          `Secondary rate limit hit for request ${options.method} ${options.url}`
+        );
         if (retryCount < 3) {
           octokit.log.info(`Retrying after ${retryAfter} seconds!`);
           return true;
         }
       },
-    }
+    },
   });
-  for (const issueInput of issues) {
+  const issueInputs: IssueInput[] =
+    issues ?? issueUrls?.map((url) => ({ url })) ?? [];
+  for (const issueInput of issueInputs) {
     try {
       const issue = new Issue(issueInput);
       await fixIssue(octokit, issue);
-      core.info(`Assigned ${issue.owner}/${issue.repository}#${issue.issueNumber} to Copilot!`);
+      core.info(
+        `Assigned ${issue.owner}/${issue.repository}#${issue.issueNumber} to Copilot!`
+      );
     } catch (error) {
       core.setFailed(`Failed to assign ${issueInput.url} to Copilot: ${error}`);
       process.exit(1);