Skip to content

Conversation

@ncalteen
Copy link
Collaborator

Primarily this involves updating transitive dependencies for better Node.js support. Outside of that, there are some minor updates. Refer to the @actions/toolkit repository for mode details.

- Add getMaxArtifactListCount function
- Add StreamExtractResponse interface return type
- Add digest return props
- Add jsdoc comments to interfaces
- No actual changes required
- No actual changes required
- No actual changes required
@ncalteen ncalteen self-assigned this Jan 16, 2026
Copilot AI review requested due to automatic review settings January 16, 2026 17:15
@github-actions
Copy link

Semantic Version Check Passed ✅

Version in manifest file package.json is valid.

This comment will be automatically updated as changes are pushed to this PR branch.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates support for the latest versions of GitHub Actions toolkit packages (@actions/artifact, @actions/cache, @actions/core, and @actions/github) from v2-v6 to v5-v7, primarily to improve Node.js compatibility. The update involves syncing stub implementations with the latest toolkit commits, adding new functionality like artifact digest support and configurable artifact list counts, and updating dependency versions.

Changes:

  • Updated package dependencies to latest versions (@actions/artifact 5.0.2, @actions/cache 5.0.3, @actions/core 2.0.2, @actions/github 7.0.0)
  • Added digest support to artifact download/list operations with SHA256 hash verification
  • Implemented configurable maximum artifact list count via environment variable
  • Fixed pagination logic in artifact listing to properly handle page boundaries

Reviewed changes

Copilot reviewed 34 out of 35 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
package.json Updated @actions package versions and moved undici to overrides
src/stubs/artifact/internal/shared/interfaces.ts Added comprehensive JSDoc comments and restored digest/expectedHash fields
src/stubs/artifact/internal/shared/config.ts Added getMaxArtifactListCount function with validation
src/stubs/artifact/internal/find/list-artifacts.ts Fixed pagination loop logic and added digest field support
src/stubs/artifact/internal/download/download-artifact.ts Implemented SHA256 hash computation and digest mismatch detection
src/stubs/cache/internal/tar.ts Updated Last Reviewed Date metadata
tests/stubs/artifact/internal/shared/config.test.ts Added comprehensive tests for getMaxArtifactListCount
README.md Updated package version table and Node.js version support (20/22 → 22/24)
CHANGELOG.md Added v7 changelog entry
.env.example Added ACTIONS_ARTIFACT_MAX_ARTIFACT_COUNT documentation
Comments suppressed due to low confidence (1)

tests/stubs/artifact/internal/shared/config.test.ts:18

  • The afterEach hook does not clean up the ACTIONS_ARTIFACT_MAX_ARTIFACT_COUNT environment variable that is set in the new test cases (lines 73, 78, 85). This could cause test pollution if tests run in a different order or if new tests are added that rely on the default value. Add delete process.env.ACTIONS_ARTIFACT_MAX_ARTIFACT_COUNT to the afterEach hook.
  afterEach(() => {
    jest.resetAllMocks()

    delete process.env.GITHUB_WORKSPACE
    delete process.env.GITHUB_SERVER_URL
  })

@github-actions
Copy link

github-actions bot commented Jan 16, 2026

MegaLinter analysis: Success

Descriptor Linter Files Fixed Errors Warnings Elapsed time
✅ ACTION actionlint 5 0 0 0.96s
✅ JAVASCRIPT prettier 19 0 0 3.68s
✅ JSON npm-package-json-lint yes no no 0.77s
✅ JSON prettier 29 0 0 1.52s
✅ MARKDOWN markdownlint 10 0 0 1.13s
✅ REPOSITORY checkov yes no no 26.17s
✅ REPOSITORY gitleaks yes no no 2.05s
✅ REPOSITORY git_diff yes no no 0.02s
✅ REPOSITORY grype yes no no 50.31s
✅ REPOSITORY secretlint yes no no 1.31s
✅ REPOSITORY syft yes no no 6.46s
✅ REPOSITORY trivy yes no no 16.31s
✅ REPOSITORY trivy-sbom yes no no 6.52s
✅ REPOSITORY trufflehog yes no no 30.08s
✅ TYPESCRIPT prettier 109 0 0 4.26s
✅ YAML prettier 25 0 0 0.52s
✅ YAML yamllint 25 0 0 0.62s

See detailed reports in MegaLinter artifacts

MegaLinter is graciously provided by OX Security

@ncalteen ncalteen merged commit 23a1eb6 into main Jan 16, 2026
15 checks passed
@ncalteen ncalteen deleted the ncalteen/updates branch January 16, 2026 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants