-
Notifications
You must be signed in to change notification settings - Fork 20
Update @actions package support to latest versions #250
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- 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
Semantic Version Check Passed ✅Version in manifest file This comment will be automatically updated as changes are pushed to this PR branch. |
There was a problem hiding this 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_COUNTto the afterEach hook.
afterEach(() => {
jest.resetAllMocks()
delete process.env.GITHUB_WORKSPACE
delete process.env.GITHUB_SERVER_URL
})
✅MegaLinter analysis: Success
See detailed reports in MegaLinter artifacts |

@actions/artifactsupport to5.0.2@actions/cachesupport to5.0.3@actions/coresupport to2.0.2@actions/githubsupport to7.0.0Primarily this involves updating transitive dependencies for better Node.js support. Outside of that, there are some minor updates. Refer to the
@actions/toolkitrepository for mode details.