Skip to content

Conversation

@renovate
Copy link
Contributor

@renovate renovate bot commented Nov 10, 2025

This PR contains the following updates:

Package Type Update Change
actions/download-artifact action major v5.0.0 -> v6.0.0
actions/upload-artifact action major v4.6.0 -> v5.0.0
actions/upload-artifact action major v4.6.2 -> v5.0.0

Release Notes

actions/download-artifact (actions/download-artifact)

v6.0.0

Compare Source

What's Changed

BREAKING CHANGE: this update supports Node v24.x. This is not a breaking change per-se but we're treating it as such.

New Contributors

Full Changelog: actions/download-artifact@v5...v6.0.0

actions/upload-artifact (actions/upload-artifact)

v5.0.0

Compare Source

v4.6.2

Compare Source

What's Changed

  • Update to use artifact 2.3.2 package & prepare for new upload-artifact release by @​salmanmkc in #​685

New Contributors

Full Changelog: actions/upload-artifact@v4...v4.6.2

v4.6.1

Compare Source

What's Changed

Full Changelog: actions/upload-artifact@v4...v4.6.1


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.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • 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 November 10, 2025 03:34
@renovate renovate bot requested a review from djsmith85 November 10, 2025 03:34
@bitwarden-bot bitwarden-bot changed the title [deps]: Update GitHub Artifact Actions (major) [PM-27974] [deps]: Update GitHub Artifact Actions (major) Nov 10, 2025
@bitwarden-bot
Copy link

Internal tracking:

@codecov
Copy link

codecov bot commented Nov 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 40.92%. Comparing base (a4d7735) to head (4376788).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #17305      +/-   ##
==========================================
- Coverage   40.93%   40.92%   -0.01%     
==========================================
  Files        3544     3544              
  Lines      101725   101725              
  Branches    15240    15240              
==========================================
- Hits        41637    41636       -1     
  Misses      58334    58334              
- Partials     1754     1755       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@renovate renovate bot changed the title [PM-27974] [deps]: Update GitHub Artifact Actions (major) [deps]: Update GitHub Artifact Actions (major) Nov 10, 2025
@renovate renovate bot force-pushed the renovate/major-github-artifact-actions branch 2 times, most recently from 02186d5 to 8f38ec2 Compare November 12, 2025 18:23
@djsmith85 djsmith85 requested review from a team and gitclonebrian and removed request for a team November 17, 2025 14:20
Copy link
Contributor

@djsmith85 djsmith85 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

They mention, a breaking change by supporting Node 24. From the the way it is phrased I don't we are required to bump our workflows to use Node24 and can stay on Node 22. Currently I'd say this is save, but wanted to check with @bitwarden/dept-bre first. Any concerns?

@djsmith85 djsmith85 self-requested a review November 17, 2025 14:22
@claude
Copy link

claude bot commented Nov 17, 2025

Claude finished @renovate[bot]'s task —— View job


Analysis of GitHub Artifact Actions Update

I've reviewed this dependency update PR and analyzed the Node 24 compatibility concern raised by djsmith85.

Summary

This PR is safe to merge. The Node 24 support mentioned in the release notes does not require any changes to your workflows or Node version configuration.

Findings

Finding 1: No Node version changes required 💭

The release notes state "BREAKING CHANGE: this update supports Node v24.x. This is not a breaking change per-se but we're treating it as such." This phrasing means:

  • The actions now support Node 24 (they can run on Node 24 runners)
  • This does NOT require you to upgrade from Node 22
  • Your project uses Node 22 (.nvmrc:1) and workflows reference this dynamically
  • The artifact actions run on the GitHub Actions runner, independent of your Node version

Finding 2: Changes are version updates only

The PR updates only the action versions across 5 workflow files:

  • upload-artifact: v4.6.0/v4.6.2 → v5.0.0
  • download-artifact: v5.0.0 → v6.0.0

No other workflow configuration changes are included. The updates consist of SHA hash changes that correspond to the new versions.

Finding 3: Workflows dynamically read Node version

All workflows retrieve the Node version from .nvmrc at runtime:

- name: Get Node Version
  id: retrieve-node-version
  run: |
    NODE_NVMRC=$(cat .nvmrc)
    NODE_VERSION=${NODE_NVMRC/v/''}
    echo "node_version=$NODE_VERSION" >> "$GITHUB_OUTPUT"

This means your workflows will continue using Node 22 regardless of the artifact action versions.

Recommendation

Safe to merge - No BRE concerns identified. The updated artifact actions are backward compatible with Node 22 and will continue working with your current Node version configuration.

@github-actions
Copy link
Contributor

Logo
Checkmarx One – Scan Summary & Details384acedf-170f-4bc9-8421-45e424f7d749

New Issues (2)

Checkmarx found the following issues in this Pull Request

Severity Issue Source File / Package Checkmarx Insight
MEDIUM CVE-2025-12443 Npm-electron-37.7.0
detailsRecommended version: 39.1.0
Description: Out of bounds read in WebXR in Google Chrome prior to 142.0.7444.59 allowed a remote attacker to perform an Out of Bounds memory read via a crafted...
Attack Vector: NETWORK
Attack Complexity: LOW

ID: 7XAYCrowQ3XEHyxhJwGieDQ55lRC5bIGiYx7MPsF8d0%3D
Vulnerable Package
MEDIUM CVE-2025-12447 Npm-electron-37.7.0
detailsRecommended version: 39.1.0
Description: Incorrect security UI in Omnibox in Google Chrome on Android prior to 142.0.7444.59 allowed a remote attacker who convinced a user to engage in spe...
Attack Vector: NETWORK
Attack Complexity: HIGH

ID: Rc1n95wlW1sBkgDC6cJ7VTW5lTX%2FaHTnPsi23pTvEJ4%3D
Vulnerable Package

@renovate renovate bot force-pushed the renovate/major-github-artifact-actions branch from 4376788 to 0253e95 Compare November 18, 2025 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants