Extracts issue body generation in prepration for adding screenshots

Author: lindseywild

.github/actions/file/src/generateIssueBody.ts

@@ -0,0 +1,31 @@
+import type { Finding } from "./types.d.js";
+
+export function generateIssueBody(finding: Finding, repoWithOwner: string): string {
+  const solutionLong = finding.solutionLong
+      ?.split("\n")
+      .map((line: string) =>
+        !line.trim().startsWith("Fix any") &&
+        !line.trim().startsWith("Fix all") &&
+        line.trim() !== ""
+          ? `- ${line}`
+          : line
+      )
+      .join("\n");
+    const acceptanceCriteria = `## Acceptance Criteria
+  - [ ] The specific axe violation reported in this issue is no longer reproducible.
+  - [ ] The fix MUST meet WCAG 2.1 guidelines OR the accessibility standards specified by the repository or organization.
+  - [ ] A test SHOULD be added to ensure this specific axe violation does not regress.
+  - [ ] This PR MUST NOT introduce any new accessibility issues or regressions.
+  `;
+    const body = `## What
+  An accessibility scan flagged the element \`${finding.html}\` on ${finding.url} because ${finding.problemShort}. Learn more about why this was flagged by visiting ${finding.problemUrl}.
+
+  To fix this, ${finding.solutionShort}.
+  ${solutionLong ? `\nSpecifically:\n\n${solutionLong}` : ''}
+
+  ${acceptanceCriteria}
+  `;
+
+  return body;
+}
+

.github/actions/file/src/openIssue.ts

@@ -1,5 +1,6 @@
 import type { Octokit } from '@octokit/core';
 import type { Finding } from './types.d.js';
+import { generateIssueBody } from "./generateIssueBody.js";
 import * as url from 'node:url'
 const URL = url.URL;
 
@@ -23,32 +24,10 @@ export async function openIssue(octokit: Octokit, repoWithOwner: string, finding
   const labels = [`${finding.scannerType} rule: ${finding.ruleId}`, `${finding.scannerType}-scanning-issue`];
   const title = truncateWithEllipsis(
     `Accessibility issue: ${finding.problemShort[0].toUpperCase() + finding.problemShort.slice(1)} on ${new URL(finding.url).pathname}`,
-    GITHUB_ISSUE_TITLE_MAX_LENGTH
+    GITHUB_ISSUE_TITLE_MAX_LENGTH,
   );
-  const solutionLong = finding.solutionLong
-    ?.split("\n")
-    .map((line) =>
-      !line.trim().startsWith("Fix any") &&
-      !line.trim().startsWith("Fix all") &&
-      line.trim() !== ""
-        ? `- ${line}`
-        : line
-    )
-    .join("\n");
-  const acceptanceCriteria = `## Acceptance Criteria
-- [ ] The specific axe violation reported in this issue is no longer reproducible.
-- [ ] The fix MUST meet WCAG 2.1 guidelines OR the accessibility standards specified by the repository or organization.
-- [ ] A test SHOULD be added to ensure this specific axe violation does not regress.
-- [ ] This PR MUST NOT introduce any new accessibility issues or regressions.
-`;
-  const body = `## What
-An accessibility scan flagged the element \`${finding.html}\` on ${finding.url} because ${finding.problemShort}. Learn more about why this was flagged by visiting ${finding.problemUrl}.
 
-To fix this, ${finding.solutionShort}.
-${solutionLong ? `\nSpecifically:\n\n${solutionLong}` : ''}
-
-${acceptanceCriteria}
-`;
+  const body = generateIssueBody(finding, repoWithOwner);
 
   return octokit.request(`POST /repos/${owner}/${repo}/issues`, {
     owner,