Actually respect open_tracking_issues

Author: JoyceZhu

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

@@ -24,6 +24,7 @@ export default async function () {
   const cachedFilings: (ResolvedFiling | RepeatedFiling)[] = JSON.parse(
     core.getInput("cached_filings", { required: false }) || "[]",
   );
+  const shouldOpenTrackingIssues = core.getBooleanInput("open_tracking_issues");
   core.debug(`Input: 'findings: ${JSON.stringify(findings)}'`);
   core.debug(`Input: 'repository: ${repoWithOwner}'`);
   core.debug(`Input: 'cached_filings: ${JSON.stringify(cachedFilings)}'`);
@@ -70,13 +71,15 @@ export default async function () {
         response = await openIssue(octokit, repoWithOwner, filing.findings[0]);
         (filing as any).issue = { state: "open" } as Issue;
         // Track for grouping
-        const problemShort: string = filing.findings[0].problemShort;
-        if (!newIssuesByProblemShort[problemShort])
-          newIssuesByProblemShort[problemShort] = [];
-        newIssuesByProblemShort[problemShort].push({
-          url: response.data.html_url,
-          id: response.data.number,
-        });
+        if (shouldOpenTrackingIssues) {
+          const problemShort: string = filing.findings[0].problemShort;
+          if (!newIssuesByProblemShort[problemShort])
+            newIssuesByProblemShort[problemShort] = [];
+          newIssuesByProblemShort[problemShort].push({
+            url: response.data.html_url,
+            id: response.data.number,
+          });
+        }
       } else if (isRepeatedFiling(filing)) {
         // Reopen the filing’s issue (if necessary)
         response = await reopenIssue(octokit, new Issue(filing.issue));
@@ -98,39 +101,41 @@ export default async function () {
     }
   }
 
-  // Open tracking issues for groups with >1 new issue and link back from each new issue
-  for (const [problemShort, issues] of Object.entries(
-    newIssuesByProblemShort,
-  ) as [string, { url: string; id: number }[]][]) {
-    if (issues.length > 1) {
-      const title: string = `${problemShort} issues`;
-      const body: string =
-        `# ${problemShort} issues\n\n` +
-        issues.map((issue) => `- [ ] ${issue.url}`).join("\n");
-      try {
-        const trackingResponse = await octokit.request(
-          `POST /repos/${repoWithOwner}/issues`,
-          {
-            owner: repoWithOwner.split("/")[0],
-            repo: repoWithOwner.split("/")[1],
-            title,
-            body,
-          },
-        );
-        const trackingUrl: string = trackingResponse.data.html_url;
-        trackingIssueUrls[problemShort] = trackingUrl;
-        core.info(
-          `Opened tracking issue for '${problemShort}' with ${issues.length} issues.`,
-        );
-      } catch (error) {
-        core.warning(
-          `Failed to open tracking issue for '${problemShort}': ${error}`,
-        );
+  // Open tracking issues for each root cause and link back from each newly-created issue
+  if (shouldOpenTrackingIssues) {
+    for (const [problemShort, issues] of Object.entries(
+      newIssuesByProblemShort,
+    ) as [string, { url: string; id: number }[]][]) {
+      if (issues.length > 1) {
+        const title: string = `${problemShort} issues`;
+        const body: string =
+          `# ${problemShort} issues\n\n` +
+          issues.map((issue) => `- [ ] ${issue.url}`).join("\n");
+        try {
+          const trackingResponse = await octokit.request(
+            `POST /repos/${repoWithOwner}/issues`,
+            {
+              owner: repoWithOwner.split("/")[0],
+              repo: repoWithOwner.split("/")[1],
+              title,
+              body,
+            },
+          );
+          const trackingUrl: string = trackingResponse.data.html_url;
+          trackingIssueUrls[problemShort] = trackingUrl;
+          core.info(
+            `Opened tracking issue for '${problemShort}': ${issues.length} issues.`,
+          );
+        } catch (error) {
+          core.warning(
+            `Failed to open tracking issue for '${problemShort}': ${error}`,
+          );
+        }
       }
     }
-  }
 
-  core.setOutput("filings", JSON.stringify(filings));
-  core.debug(`Output: 'filings: ${JSON.stringify(filings)}'`);
-  core.info("Finished 'file' action");
+    core.setOutput("filings", JSON.stringify(filings));
+    core.debug(`Output: 'filings: ${JSON.stringify(filings)}'`);
+    core.info("Finished 'file' action");
+  }
 }