Bump actions/upload-artifact from 5 to 6

[dependabot]

Bumps actions/upload-artifact from 5 to 6.

Release notes

Sourced from actions/upload-artifact's releases.

v6.0.0

v6 - What's new

[!IMPORTANT] actions/upload-artifact@v6 now runs on Node.js 24 (runs.using: node24) and requires a minimum Actions Runner version of 2.327.1. If you are using self-hosted runners, ensure they are updated before upgrading.

Node.js 24

This release updates the runtime to Node.js 24. v5 had preliminary support for Node.js 24, however this action was by default still running on Node.js 20. Now this action by default will run on Node.js 24.

What's Changed

• Upload Artifact Node 24 support by @​salmanmkc in actions/upload-artifact#719
• fix: update @​actions/artifact for Node.js 24 punycode deprecation by @​salmanmkc in actions/upload-artifact#744
• prepare release v6.0.0 for Node.js 24 support by @​salmanmkc in actions/upload-artifact#745

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

Commits

• b7c566a Merge pull request #745 from actions/upload-artifact-v6-release
• e516bc8 docs: correct description of Node.js 24 support in README
• ddc45ed docs: update README to correct action name for Node.js 24 support
• 615b319 chore: release v6.0.0 for Node.js 24 support
• 017748b Merge pull request #744 from actions/fix-storage-blob
• 38d4c79 chore: rebuild dist
• 7d27270 chore: add missing license cache files for @​actions/core, @​actions/io, and mi...
• 5f643d3 chore: update license files for @​actions/artifact@​5.0.1 dependencies
• 1df1684 chore: update package-lock.json with @​actions/artifact@​5.0.1
• b5b1a91 fix: update @​actions/artifact to ^5.0.0 for Node.js 24 punycode fix
• Additional commits viewable in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

---

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

• @dependabot rebase will rebase this PR
• @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
• @dependabot merge will merge this PR after your CI passes on it
• @dependabot squash and merge will squash and merge this PR after your CI passes on it
• @dependabot cancel merge will cancel a previously requested merge and block automerging
• @dependabot reopen will reopen this PR if it is closed
• @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
• @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
• @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

---

Resolves #745
Resolves #744
Resolves actions/upload-artifact#719
Resolves actions/upload-artifact#744
Resolves actions/upload-artifact#745

[github-actions]

🤖 Claude Code Review

Code Review Analysis

Code Quality

• ✅ Code follows style guide: The changes are minimal YAML configuration updates. The formatting and structure are consistent with GitHub Actions conventions.

• ✅ No commented-out code: No commented code present in the diff.

• ✅ Meaningful variable names: N/A - This is configuration, not code with variables.

• ✅ DRY principle followed: The same update is applied consistently across all three workflow files, which is appropriate.

• ✅ Identify Defects: No defects identified. The version bump from actions/upload-artifact@v5 to @v6 is a straightforward dependency update. No logic errors, race conditions, or security vulnerabilities introduced.

• ✅ Project memory configuration: No .claude/CLAUDE.md file found in the repository to consider.

Testing

• ✅ Unit tests for new functions: N/A - This is a GitHub Actions dependency version bump, not functional code changes.

• ✅ Integration tests for new endpoints: N/A - No new endpoints added.

• ✅ Edge cases covered: N/A - Configuration change only.

• ✅ Test coverage > 80%: Not applicable to this change, but the workflows themselves run comprehensive tests across Darwin, Linux, and Windows platforms.

Documentation

• ✅ Readme updated if needed: Not required - this is an internal CI/CD dependency update that doesn't affect user-facing functionality.

• ✅ API docs updated: N/A - No API changes.

• ✅ Inline comments for complex logic: N/A - YAML workflow files are self-documenting.

• ⚠️ CHANGELOG.md updated: No CHANGELOG.md update detected. While this is a minor dependency bump, it's worth considering whether dependency updates should be tracked in the changelog.

• ✅ Markdown formatting: No markdown files were modified in this PR.

Security

• ✅ No hardcoded credentials: No credentials present.

• ✅ Input validation implemented: N/A - Configuration change only.

• ✅ Proper error handling: The workflows include if: always() conditions on test log uploads, which is good practice.

• ✅ No sensitive data in logs: No sensitive data logging introduced.

• ✅ No license files (.lic) checked in: No license files or AQAAAD strings in the diff.

Summary

This is a clean and safe PR that updates the actions/upload-artifact action from v5 to v6 across three GitHub Actions workflow files:

• .github/workflows/go-test-darwin.yaml:132,138
• .github/workflows/go-test-linux.yaml:100,106
• .github/workflows/go-test-windows.yaml:129,135

Recommendation: APPROVE ✅

The only minor consideration is whether to document this dependency update in CHANGELOG.md, but for GitHub Actions internal dependencies, this is typically not required unless it impacts functionality or introduces breaking changes.

Automated code review analyzing defects and coding standards

[github-actions]

🤖 Claude Code Review

Code Review Analysis

Code Quality

✅ Code follows style guide

No Go code changes to evaluate. Only dependency updates in GitHub Actions workflows and go.mod/go.sum files.

✅ No commented-out code

No commented-out code found in the diff.

✅ Meaningful variable names

No new variables introduced.

✅ DRY principle followed

Changes are repetitive but appropriate - updating the same action version across multiple workflow files follows the principle of consistent dependency management.

✅ Identify Defects

No bugs, logic errors, or security vulnerabilities identified. The changes are:

1. GitHub Actions upgrade (v5→v6): The actions/upload-artifact upgrade from v5 to v6 is a standard maintenance update that maintains backward compatibility
2. Dependency updates: All go.mod/go.sum changes are routine security and maintenance updates with no breaking changes evident

⚠️ Project Memory Configuration

No .claude/CLAUDE.md file found in the repository to evaluate.

Testing

❌ Unit tests for new functions

Not applicable - No new functions added. This is a dependency update PR.

❌ Integration tests for new endpoints

Not applicable - No new endpoints added.

❌ Edge cases covered

Not applicable - No new logic to test.

❌ Test coverage > 80%

Cannot verify - No test coverage changes shown in diff. The workflows contain test execution but coverage metrics aren't visible in this diff.

Documentation

✅ README updated if needed

Not required - Dependency updates typically don't require README updates unless they introduce breaking changes or new features.

✅ API docs updated

Not required - No API changes.

✅ Inline comments for complex logic

Not applicable - No complex logic added.

❌ CHANGELOG.md updated

Issue found: No CHANGELOG.md update detected in the diff. For transparency and maintainability, dependency updates should be documented:

• GitHub Actions: actions/upload-artifact v5→v6 across all workflow files
• Go dependencies: Multiple version bumps including security updates

✅ Markdown files follow CommonMark

Not applicable - No markdown files modified in this PR.

Security

✅ No hardcoded credentials

No credentials found.

✅ Input validation implemented

Not applicable - No new input handling.

✅ Proper error handling

Not applicable - No new error handling paths.

✅ No sensitive data in logs

No sensitive data exposure detected.

✅ No license files (.lic) or AQAAAD strings

No .lic files or AQAAAD strings found in the diff.

---

Summary

Recommendation: ✅ APPROVE with minor documentation request

This is a clean dependency maintenance PR with:

• ✅ Consistent GitHub Actions upgrade across all platform workflows (.github/workflows/go-test-{darwin,linux,windows}.yaml)
• ✅ Routine Go dependency updates following semantic versioning
• ✅ No security concerns
• ✅ No breaking changes evident

Minor Issue:

• Missing CHANGELOG.md entry to document these dependency updates

Files affected:

• .github/workflows/go-test-darwin.yaml:132,137
• .github/workflows/go-test-linux.yaml:100,105
• .github/workflows/go-test-windows.yaml:129,134
• go.mod (multiple dependency version bumps)
• go.sum (corresponding checksum updates)

---

Automated code review analyzing defects and coding standards