Skip to content

[deps]: Update @actions/artifact to v6#617

Open
renovate[bot] wants to merge 2 commits intomainfrom
renovate/actions-artifact-6.x
Open

[deps]: Update @actions/artifact to v6#617
renovate[bot] wants to merge 2 commits intomainfrom
renovate/actions-artifact-6.x

Conversation

@renovate
Copy link
Contributor

@renovate renovate bot commented Feb 16, 2026

This PR contains the following updates:

Package Change Age Confidence
@actions/artifact (source) 5.0.16.2.0 age confidence

Release Notes

actions/toolkit (@​actions/artifact)

v6.2.0

  • Support uploading single un-archived files (not zipped). Direct uploads are only supported for artifacts version 7+ (based on the major version of actions/upload-artifact). Callers must pass the skipArchive option to uploadArtifact. Only single files can be uploaded at a time right now. Default behavior should remain unchanged if skipArchive = false. When skipArchive = true, the name of the file is used as the name of the artifact for consistency with the downloads: you upload artifact.txt, you download artifact.txt.

v6.1.0

  • Support downloading non-zip artifacts. Zipped artifacts will be decompressed automatically (with an optional override). Un-zipped artifacts will be downloaded as-is.

v6.0.0

  • Breaking change: Package is now ESM-only
    • CommonJS consumers must use dynamic import() instead of require()

v5.0.3

  • Bump @actions/http-client to 3.0.2

Configuration

📅 Schedule: Branch creation - "every 2nd week starting on the 2 week of the year before 4am on Monday" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate renovate bot requested a review from a team as a code owner February 16, 2026 02:54
@renovate renovate bot requested a review from pixman20 February 16, 2026 02:54
@bitwarden-bot bitwarden-bot changed the title [deps]: Update @actions/artifact to v6 [PM-32292] [deps]: Update @actions/artifact to v6 Feb 16, 2026
@bitwarden-bot
Copy link

Internal tracking:

@renovate renovate bot changed the title [PM-32292] [deps]: Update @actions/artifact to v6 [deps]: Update @actions/artifact to v6 Feb 16, 2026
@renovate renovate bot force-pushed the renovate/actions-artifact-6.x branch from e8c0728 to 40f678d Compare February 17, 2026 14:23
@renovate renovate bot force-pushed the renovate/actions-artifact-6.x branch from 40f678d to a6a6492 Compare March 4, 2026 19:14
@renovate renovate bot force-pushed the renovate/actions-artifact-6.x branch from a6a6492 to d732bf3 Compare March 13, 2026 10:40
@pixman20 pixman20 added the ai-review Request a Claude code review label Mar 13, 2026
@pixman20
Copy link
Contributor

@claude review this

@claude
Copy link

claude bot commented Mar 13, 2026

Claude finished @pixman20's task in 6m 45s —— View job


I'll analyze this and get back to you.

@github-actions
Copy link

github-actions bot commented Mar 13, 2026

Logo
Checkmarx One – Scan Summary & Details8de7b3b4-8a1f-4e4a-b94e-6dd8e424ce41

Great job! No new security vulnerabilities introduced in this pull request

@sonarqubecloud
Copy link

@claude
Copy link

claude bot commented Mar 13, 2026

Review Summary

PR: [deps]: Update @actions/artifact to v6
Type: Dependency update (major version bump)
Risk: High - This is a major version bump with breaking changes that cannot be merged as-is.


Findings

1. CRITICAL: Committed node_modules not updated

The download-artifacts/node_modules/ directory is committed to the repository (5,087 tracked files) and is used directly at runtime by the GitHub Action (runs.using: node20). This PR only updates package.json and package-lock.json but does not update node_modules/. As a result, this dependency bump has no actual runtime effect -- the action would still execute with @actions/artifact v2.0.0.

After updating package.json, npm install must be run and the resulting node_modules/ changes must be committed.

2. CRITICAL: ESM/CJS incompatibility will break the action

The @actions/artifact v6.0.0 release notes (included in this PR description) state:

Breaking change: Package is now ESM-only. CommonJS consumers must use dynamic import() instead of require()

However, download-artifacts/main.js uses CommonJS require() at the module level:

// download-artifacts/main.js, line 3
const artifact = require('@actions/artifact')

After running npm install to update node_modules, Node.js will throw ERR_REQUIRE_ESM when the action starts, crashing the entire action before any code executes. The main.js file must be migrated to use ESM import syntax (with "type": "module" in package.json) or use dynamic import().

3. IMPORTANT: API breaking changes in artifact.create()

download-artifacts/main.js line 10 calls artifact.create():

const artifactClient = artifact.create()

This factory method does not exist in the v6 DefaultArtifactClient API. The v6 package exports a pre-instantiated client as the default export. The downloadAction function (lines 9-20) and its downloadArtifact(name, path, downloadOptions) call signature also differ from the v6 API, which uses downloadArtifact(artifactId, options). While this code path is only used as a fallback, it needs to be updated.


Recommendation

This major version bump requires code changes to download-artifacts/main.js beyond a simple dependency update:

  1. Convert main.js to ESM (or use dynamic import() for the artifact package)
  2. Update the downloadAction function to use the v6 API
  3. Run npm install and commit the updated node_modules/
  4. Verify all test workflows pass with the updated code

This likely cannot be handled by Renovate alone and needs manual developer intervention.

@renovate
Copy link
Contributor Author

renovate bot commented Mar 13, 2026

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@pixman20 pixman20 added the hold label Mar 20, 2026
@pixman20
Copy link
Contributor

This needs additional work beyond the Renovate updates:
https://bitwarden.atlassian.net/browse/BRE-1713

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-review Request a Claude code review hold major-update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants