Use Node20

action.yml

@@ -8,22 +8,6 @@ inputs:
   path:
     description: 'Destination path'
     required: false
-  github-token:
-    description: The GitHub token used to download the artifact
-    default: ${{ github.token }}
-    required: false
-  repository:
-    description: 'The repository owner and the repository name joined together by "/".
-      This specifies the repository that artifacts will be downloaded from. If downloading artifacts from external workflow runs or repositories then the above download-token must be permissions to this repository.'
-    required: false
-    default: ${{ github.repository }}
-  run-id:
-    description: 'The id of the workflow run where the desired download artifact was uploaded from. If downloading artifacts from anything other than the current workflow run then this needs to be overwritten.'
-    required: false
-    default: ${{ github.run_id }}
-outputs:
-  download-path:
-    description: 'Path of artifact download'
 runs:
-  using: 'node16'
+  using: 'node20'
   main: 'dist/index.js'

package.json

@@ -1,6 +1,6 @@
 {
   "name": "download-artifact",
-  "version": "3.0.0",
+  "version": "3.1.0",
   "description": "Download a build artifact that was previously uploaded in the workflow by the upload-artifact action",
   "main": "dist/index.js",
   "scripts": {
@@ -28,9 +28,8 @@
   },
   "homepage": "https://github.com/actions/download-artifact#readme",
   "dependencies": {
-    "@actions/artifact": "^2.0.0",
+    "@actions/artifact": "^1.1.3",
-    "@actions/core": "^1.10.0",
+    "@actions/core": "^1.10.0"
-    "@actions/github": "^5.1.1"
   },
   "devDependencies": {
     "@types/node": "^12.12.6",

src/constants.ts

@@ -1,11 +1,7 @@
 export enum Inputs {
   Name = 'name',
-  Path = 'path',
+  Path = 'path'
-  GitHubToken = 'github-token',
-  Repository = 'repository',
-  RunID = 'run-id'
 }
-
 export enum Outputs {
   DownloadPath = 'download-path'
 }

src/download-artifact.ts

@@ -1,100 +1,61 @@
-import * as os from 'os'
-import * as path from 'path'
 import * as core from '@actions/core'
 import * as artifact from '@actions/artifact'
+import * as os from 'os'
+import {resolve} from 'path'
 import {Inputs, Outputs} from './constants'
 
-const PARALLEL_DOWNLOADS = 5
-
-export const chunk = <T>(arr: T[], n: number): T[][] =>
-  arr.reduce((acc, cur, i) => {
-    const index = Math.floor(i / n)
-    acc[index] = [...(acc[index] || []), cur]
-    return acc
-  }, [] as T[][])
-
 async function run(): Promise<void> {
-  const inputs = {
+  try {
-    name: core.getInput(Inputs.Name, {required: false}),
+    const name = core.getInput(Inputs.Name, {required: false})
-    path: core.getInput(Inputs.Path, {required: false}),
+    const path = core.getInput(Inputs.Path, {required: false})
-    token: core.getInput(Inputs.GitHubToken, {required: true}),
-    repository: core.getInput(Inputs.Repository, {required: true}),
-    runID: parseInt(core.getInput(Inputs.RunID, {required: true}))
-  }
 
-  if (!inputs.path) {
+    let resolvedPath
-    inputs.path = process.env['GITHUB_WORKSPACE'] || process.cwd()
+    // resolve tilde expansions, path.replace only replaces the first occurrence of a pattern
+    if (path.startsWith(`~`)) {
+      resolvedPath = resolve(path.replace('~', os.homedir()))
+    } else {
+      resolvedPath = resolve(path)
     }
-
-  if (inputs.path.startsWith(`~`)) {
-    inputs.path = inputs.path.replace('~', os.homedir())
-  }
-
-  const resolvedPath = path.resolve(inputs.path)
     core.debug(`Resolved path is ${resolvedPath}`)
 
-  const [owner, repo] = inputs.repository.split('/')
-  if (!owner || !repo) {
-    throw new Error(
-      `Invalid repository: '${inputs.repository}'. Must be in format owner/repo`
-    )
-  }
-
     const artifactClient = artifact.create()
-  let artifacts: artifact.Artifact[] = []
+    if (!name) {
-
+      // download all artifacts
-  if (inputs.name) {
+      core.info('No artifact name specified, downloading all artifacts')
-    const {artifact: targetArtifact} = await artifactClient.getArtifact(
+      core.info(
-      inputs.name,
+        'Creating an extra directory for each artifact that is being downloaded'
-      inputs.runID,
+      )
-      owner,
+      const downloadResponse = await artifactClient.downloadAllArtifacts(
-      repo,
+        resolvedPath
-      inputs.token
+      )
+      core.info(`There were ${downloadResponse.length} artifacts downloaded`)
+      for (const artifact of downloadResponse) {
+        core.info(
+          `Artifact ${artifact.artifactName} was downloaded to ${artifact.downloadPath}`
         )
-
-    if (!targetArtifact) {
-      throw new Error(`Artifact '${inputs.name}' not found`)
       }
-
-    core.debug(
-      `Found named artifact '${inputs.name}' (ID: ${targetArtifact.id}, Size: ${targetArtifact.size})`
-    )
-
-    artifacts = [targetArtifact]
     } else {
-    const listArtifactResponse = await artifactClient.listArtifacts(
+      // download a single artifact
-      inputs.runID,
+      core.info(`Starting download for ${name}`)
-      owner,
+      const downloadOptions = {
-      repo,
+        createArtifactFolder: false
-      inputs.token
+      }
+      const downloadResponse = await artifactClient.downloadArtifact(
+        name,
+        resolvedPath,
+        downloadOptions
       )
-
+      core.info(
-    if (listArtifactResponse.artifacts.length === 0) {
+        `Artifact ${downloadResponse.artifactName} was downloaded to ${downloadResponse.downloadPath}`
-      throw new Error(
-        `No artifacts found for run '${inputs.runID}' in '${inputs.repository}'`
       )
     }
-
+    // output the directory that the artifact(s) was/were downloaded to
-    core.debug(`Found ${listArtifactResponse.artifacts.length} artifacts`)
+    // if no path is provided, an empty string resolves to the current working directory
-    artifacts = listArtifactResponse.artifacts
-  }
-
-  const downloadPromises = artifacts.map(artifact =>
-    artifactClient.downloadArtifact(artifact.id, owner, repo, inputs.token, {
-      path: path.join(resolvedPath, artifact.name)
-    })
-  )
-
-  const chunkedPromises = chunk(downloadPromises, PARALLEL_DOWNLOADS)
-  for (const chunk of chunkedPromises) {
-    await Promise.all(chunk)
-  }
-
-  core.info(`Total of ${artifacts.length} artifact(s) downloaded`)
     core.setOutput(Outputs.DownloadPath, resolvedPath)
-  core.info('Download artifact has finished successfully')
+    core.info('Artifact download has finished successfully')
+  } catch (err) {
+    core.setFailed(err.message)
+  }
 }
 
-run().catch(err =>
+run()
-  core.setFailed(`Unable to download artifact(s): ${err.message}`)
-)