Remove string I/O backward compat, use file-based I/O exclusively

- Remove string inputs (findings, cached_filings, issues) and outputs
  (findings, filings, fixings) from find, file, fix actions
- Make file inputs required (findings_file, issues_file)
- Remove core.setOutput calls for string versions
- Simplify TypeScript to read only from files, no fallback logic

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: lindseywild

.github/actions/file/action.yml

@@ -2,23 +2,17 @@ name: "File"
 description: "Files GitHub issues to track potential accessibility gaps."
 
 inputs:
-  findings:
-    description: "List of potential accessibility gaps, as stringified JSON"
-    required: false
   findings_file:
-    description: "Path to a JSON file containing findings (alternative to 'findings' for large datasets)"
-    required: false
+    description: "Path to a JSON file containing the list of potential accessibility gaps"
+    required: true
   repository:
     description: "Repository (with owner) to file issues in"
     required: true
   token:
     description: "Token with fine-grained permission 'issues: write'"
     required: true
-  cached_filings:
-    description: "Cached filings from previous runs, as stringified JSON. Without this, duplicate issues may be filed."
-    required: false
   cached_filings_file:
-    description: "Path to a JSON file containing cached filings (alternative to 'cached_filings' for large datasets)"
+    description: "Path to a JSON file containing cached filings from previous runs. Without this, duplicate issues may be filed."
     required: false
   screenshot_repository:
     description: "Repository (with owner) where screenshots are stored on the gh-cache branch. Defaults to the 'repository' input if not set. Required if issues are open in a different repo to construct proper screenshot URLs."
@@ -29,10 +23,8 @@ inputs:
     default: "false"
 
 outputs:
-  filings:
-    description: "List of issues filed (and their associated finding(s)), as stringified JSON"
   filings_file:
-    description: "Path to a JSON file containing filings (use for large datasets to avoid output size limits)"
+    description: "Path to a JSON file containing the list of issues filed (and their associated finding(s))"
 
 runs:
   using: "node24"

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

@@ -18,22 +18,20 @@ const OctokitWithThrottling = Octokit.plugin(throttling)
 
 export default async function () {
   core.info("Started 'file' action")
-  const findingsFile = core.getInput('findings_file', {required: false})
-  const findings: Finding[] = findingsFile
-    ? JSON.parse(fs.readFileSync(findingsFile, 'utf8'))
-    : JSON.parse(core.getInput('findings', {required: !findingsFile}))
+  const findingsFile = core.getInput('findings_file', {required: true})
+  const findings: Finding[] = JSON.parse(fs.readFileSync(findingsFile, 'utf8'))
   const repoWithOwner = core.getInput('repository', {required: true})
   const token = core.getInput('token', {required: true})
   const screenshotRepo = core.getInput('screenshot_repository', {required: false}) || repoWithOwner
   const cachedFilingsFile = core.getInput('cached_filings_file', {required: false})
   const cachedFilings: (ResolvedFiling | RepeatedFiling)[] = cachedFilingsFile
     ? JSON.parse(fs.readFileSync(cachedFilingsFile, 'utf8'))
-    : JSON.parse(core.getInput('cached_filings', {required: false}) || '[]')
+    : []
   const shouldOpenGroupedIssues = core.getBooleanInput('open_grouped_issues')
-  core.debug(`Input: 'findings: ${JSON.stringify(findings)}'`)
+  core.debug(`Input: 'findings_file: ${findingsFile}'`)
   core.debug(`Input: 'repository: ${repoWithOwner}'`)
   core.debug(`Input: 'screenshot_repository: ${screenshotRepo}'`)
-  core.debug(`Input: 'cached_filings: ${JSON.stringify(cachedFilings)}'`)
+  core.debug(`Input: 'cached_filings_file: ${cachedFilingsFile}'`)
   core.debug(`Input: 'open_grouped_issues: ${shouldOpenGroupedIssues}'`)
 
   const octokit = new OctokitWithThrottling({
@@ -137,12 +135,10 @@ export default async function () {
     }
   }
 
-  core.setOutput('filings', JSON.stringify(filings))
-
   const filingsPath = path.join(process.env.RUNNER_TEMP || '/tmp', 'filings.json')
   fs.writeFileSync(filingsPath, JSON.stringify(filings))
   core.setOutput('filings_file', filingsPath)
 
-  core.debug(`Output: 'filings: ${JSON.stringify(filings)}'`)
+  core.debug(`Output: 'filings_file: ${filingsPath}'`)
   core.info("Finished 'file' action")
 }

.github/actions/find/action.yml

@@ -24,10 +24,8 @@ inputs:
     required: false
 
 outputs:
-  findings:
-    description: 'List of potential accessibility gaps, as stringified JSON'
   findings_file:
-    description: 'Path to a JSON file containing findings (use for large datasets to avoid output size limits)'
+    description: 'Path to a JSON file containing the list of potential accessibility gaps'
 
 runs:
   using: 'node24'

.github/actions/find/src/index.ts

@@ -46,13 +46,11 @@ export default async function () {
     core.info(`Found ${findingsForUrl.length} findings for ${url}`)
   }
 
-  core.setOutput('findings', JSON.stringify(findings))
-
   const findingsPath = path.join(process.env.RUNNER_TEMP || '/tmp', 'findings.json')
   fs.writeFileSync(findingsPath, JSON.stringify(findings))
   core.setOutput('findings_file', findingsPath)
 
-  core.debug(`Output: 'findings: ${JSON.stringify(findings)}'`)
+  core.debug(`Output: 'findings_file: ${findingsPath}'`)
   core.info(`Found ${findings.length} findings in total`)
   core.info("Finished 'find' action")
 }

.github/actions/fix/action.yml

@@ -2,12 +2,9 @@ name: "Fix"
 description: "Attempts to fix issues with Copilot."
 
 inputs:
-  issues:
-    description: "List of issues to attempt to fix, as stringified JSON"
-    required: false
   issues_file:
-    description: "Path to a JSON file containing issues (alternative to 'issues' for large datasets)"
-    required: false
+    description: "Path to a JSON file containing the list of issues to attempt to fix"
+    required: true
   repository:
     description: "Repository (with owner) containing issues"
     required: true
@@ -16,10 +13,8 @@ inputs:
     required: true
 
 outputs:
-  fixings:
-    description: "List of pull requests filed (and their associated issues), as stringified JSON"
   fixings_file:
-    description: "Path to a JSON file containing fixings (use for large datasets to avoid output size limits)"
+    description: "Path to a JSON file containing the list of pull requests filed (and their associated issues)"
 
 runs:
   using: "node24"

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

@@ -13,13 +13,11 @@ const OctokitWithThrottling = Octokit.plugin(throttling)
 
 export default async function () {
   core.info("Started 'fix' action")
-  const issuesFile = core.getInput('issues_file', {required: false})
-  const issues: IssueInput[] = issuesFile
-    ? JSON.parse(fs.readFileSync(issuesFile, 'utf8'))
-    : JSON.parse(core.getInput('issues', {required: !issuesFile}) || '[]')
+  const issuesFile = core.getInput('issues_file', {required: true})
+  const issues: IssueInput[] = JSON.parse(fs.readFileSync(issuesFile, 'utf8'))
   const repoWithOwner = core.getInput('repository', {required: true})
   const token = core.getInput('token', {required: true})
-  core.debug(`Input: 'issues: ${JSON.stringify(issues)}'`)
+  core.debug(`Input: 'issues_file: ${issuesFile}'`)
   core.debug(`Input: 'repository: ${repoWithOwner}'`)
 
   const octokit = new OctokitWithThrottling({
@@ -61,12 +59,10 @@ export default async function () {
     }
   }
 
-  core.setOutput('fixings', JSON.stringify(fixings))
-
   const fixingsPath = path.join(process.env.RUNNER_TEMP || '/tmp', 'fixings.json')
   fs.writeFileSync(fixingsPath, JSON.stringify(fixings))
   core.setOutput('fixings_file', fixingsPath)
 
-  core.debug(`Output: 'fixings: ${JSON.stringify(fixings)}'`)
+  core.debug(`Output: 'fixings_file: ${fixingsPath}'`)
   core.info("Finished 'fix' action")
 }