use artifact prefix to avoid conflicts

Author: jeremydubreil

.github/workflows/integration-test.yml

@@ -45,6 +45,7 @@ jobs:
           target: push
           sources: ${{ github.workspace }}
           debug: true
+          artifact-prefix: ${{ matrix.os }}
       - name: Check run succeeded
         env:
           RUN_OUTPUT: ${{ steps.run-action.outputs.push-completed }}
@@ -58,7 +59,7 @@ jobs:
       - name: Download results
         uses: actions/download-artifact@v4
         with:
-          name: ${{ runner.name }}-results-push
+          name: ${{ matrix.os }}-results-push
           path: artifact
       - name: Check results
         working-directory: artifact

action.yaml

@@ -23,6 +23,10 @@ inputs:
     description: 'Show vulnerabilities found in transitive dependencies'
     required: false
     default: false
+  artifact-prefix:
+    description: 'Prefix for the artifact name'
+    required: false
+    default: ''
 outputs:
   old-completed:
     description: 'If running a target called old, whether the analysis for this was completed'
@@ -92,3 +96,4 @@ runs:
         token: '${{ inputs.token || github.token }}'
         footer: '${{ inputs.footer }}'
         eval-indirect-dependencies: '${{ inputs.eval-indirect-dependencies }}'
+        artifact-prefix: '${{ inputs.artifact-prefix }}'

src/index.ts

@@ -76,7 +76,12 @@ async function runAnalysis() {
   toUpload.push(scaReport)
 
   const uploadStart = Date.now()
-  await uploadArtifact('results-' + target, ...toUpload)
+  const artifactPrefix = getInput('artifact-prefix')
+  if (artifactPrefix !== '') {
+    await uploadArtifact(artifactPrefix + '-results-' + target, ...toUpload)
+  } else {
+    await uploadArtifact('results-' + target, ...toUpload)
+  }
   telemetryCollector.addField('duration.upload-artifacts', (Date.now() - uploadStart).toString())
   setOutput(`${target}-completed`, true)
 }