Use unique filenames for output files to avoid collisions

lindseywild · Copilot · lindseywild · commit 0d870fd748a5 · 2026-03-31T12:58:21.000-04:00
When an action is invoked multiple times in the same job, a fixed
filename would cause later invocations to overwrite earlier outputs.
Use crypto.randomUUID() to generate unique filenames.

Co-authored-by: Copilot &lt;223556219+Copilot@users.noreply.github.com&gt;
diff --git a/.github/actions/file/src/index.ts b/.github/actions/file/src/index.ts
@@ -135,7 +135,7 @@ export default async function () {
     }
   }
 
-  const filingsPath = path.join(process.env.RUNNER_TEMP || '/tmp', 'filings.json')
+  const filingsPath = path.join(process.env.RUNNER_TEMP || '/tmp', `filings-${crypto.randomUUID()}.json`)
   fs.writeFileSync(filingsPath, JSON.stringify(filings))
   core.setOutput('filings_file', filingsPath)
 
diff --git a/.github/actions/find/src/index.ts b/.github/actions/find/src/index.ts
@@ -46,7 +46,7 @@ export default async function () {
     core.info(`Found ${findingsForUrl.length} findings for ${url}`)
   }
 
-  const findingsPath = path.join(process.env.RUNNER_TEMP || '/tmp', 'findings.json')
+  const findingsPath = path.join(process.env.RUNNER_TEMP || '/tmp', `findings-${crypto.randomUUID()}.json`)
   fs.writeFileSync(findingsPath, JSON.stringify(findings))
   core.setOutput('findings_file', findingsPath)
 
diff --git a/.github/actions/fix/src/index.ts b/.github/actions/fix/src/index.ts
@@ -59,7 +59,7 @@ export default async function () {
     }
   }
 
-  const fixingsPath = path.join(process.env.RUNNER_TEMP || '/tmp', 'fixings.json')
+  const fixingsPath = path.join(process.env.RUNNER_TEMP || '/tmp', `fixings-${crypto.randomUUID()}.json`)
   fs.writeFileSync(fixingsPath, JSON.stringify(fixings))
   core.setOutput('fixings_file', fixingsPath)
 

Original file line number	Diff line number	Diff line change
`@@ -135,7 +135,7 @@ export default async function () {`
`135`	`135`	`}`
`136`	`136`	`}`
`137`	`137`
`138`		`- const filingsPath = path.join(process.env.RUNNER_TEMP \|\| '/tmp', 'filings.json')`
	`138`	+ const filingsPath = path.join(process.env.RUNNER_TEMP \|\| '/tmp', `filings-${crypto.randomUUID()}.json`)
`139`	`139`	`fs.writeFileSync(filingsPath, JSON.stringify(filings))`
`140`	`140`	`core.setOutput('filings_file', filingsPath)`
`141`	`141`
Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ export default async function () {`
`46`	`46`	core.info(`Found ${findingsForUrl.length} findings for ${url}`)
`47`	`47`	`}`
`48`	`48`
`49`		`- const findingsPath = path.join(process.env.RUNNER_TEMP \|\| '/tmp', 'findings.json')`
	`49`	+ const findingsPath = path.join(process.env.RUNNER_TEMP \|\| '/tmp', `findings-${crypto.randomUUID()}.json`)
`50`	`50`	`fs.writeFileSync(findingsPath, JSON.stringify(findings))`
`51`	`51`	`core.setOutput('findings_file', findingsPath)`
`52`	`52`
Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ export default async function () {`
`59`	`59`	`}`
`60`	`60`	`}`
`61`	`61`
`62`		`- const fixingsPath = path.join(process.env.RUNNER_TEMP \|\| '/tmp', 'fixings.json')`
	`62`	+ const fixingsPath = path.join(process.env.RUNNER_TEMP \|\| '/tmp', `fixings-${crypto.randomUUID()}.json`)
`63`	`63`	`fs.writeFileSync(fixingsPath, JSON.stringify(fixings))`
`64`	`64`	`core.setOutput('fixings_file', fixingsPath)`
`65`	`65`