feat(deps): update @actions/cache, @actions/core, @actions/exec, and zod#39
feat(deps): update @actions/cache, @actions/core, @actions/exec, and zod#39
Conversation
TypeScript 6 requires explicit "types": ["node"] in tsconfig.json for @types/node to be resolved.
Move test config into the main vite.config.ts and remove the separate vitest.config.ts file.
There was a problem hiding this comment.
Pull request overview
This PR updates the GitHub Action’s runtime dependencies to newer major versions (notably ESM-only @actions/* packages and zod@4) and adjusts the code to match Zod v4’s error API.
Changes:
- Bump
@actions/cache,@actions/core, and@actions/execto major ESM-only versions. - Bump
zodto v4 and update error handling (ZodError.errors→ZodError.issues). - Refresh
pnpm-lock.yamlto reflect the dependency upgrades.
Reviewed changes
Copilot reviewed 4 out of 7 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
src/inputs.ts |
Updates Zod error property usage for Zod v4 compatibility. |
package.json |
Raises dependency ranges to the new major versions. |
pnpm-lock.yaml |
Lockfile updates for the new dependency graph (including transitive updates). |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "dependencies": { | ||
| "@actions/cache": "^4.1.0", | ||
| "@actions/core": "^1.11.1", | ||
| "@actions/exec": "^1.1.1", | ||
| "@actions/cache": "^6.0.0", | ||
| "@actions/core": "^3.0.0", | ||
| "@actions/exec": "^3.0.0", | ||
| "@actions/glob": "^0.6.0", | ||
| "yaml": "^2.8.2", | ||
| "zod": "^3.25.76" | ||
| "zod": "^4.0.0" | ||
| }, |
There was a problem hiding this comment.
The test plan notes the action hasn’t been validated in a real workflow run yet. Given the major upgrades to ESM-only @actions/* packages and zod v4, please run the action in a sample workflow (or add evidence/logs) before merging to confirm runtime behavior on GitHub Actions.
fengmk2
changed the title
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 9 changed files in this pull request and generated 1 comment.
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| export default defineConfig({ | ||
| test: { | ||
| include: ["src/**/*.test.ts"], | ||
| }, | ||
| staged: { | ||
| "*": "vp check --fix", | ||
| }, | ||
| pack: { | ||
| entry: ["./src/index.ts"], | ||
| format: ["esm"], | ||
| outDir: "dist", | ||
| noExternal: [/.*/], | ||
| inlineOnly: false, | ||
| deps: { | ||
| alwaysBundle: [/.*/], | ||
| onlyBundle: false, | ||
| }, | ||
| clean: true, | ||
| minify: true, | ||
| }, | ||
| lint: { | ||
| ignorePatterns: ["dist/**/*"], | ||
| options: { | ||
| typeAware: true, | ||
| typeCheck: true, | ||
| }, |
There was a problem hiding this comment.
The PR description only mentions dependency bumps and the ZodError property rename, but this diff also changes the toolchain/config surface (moves test config into vite.config.ts, updates pack bundling options, enables type-aware/type-check linting, and adjusts TS module settings). Please update the PR description to reflect these additional behavioral/config changes (or split them into a separate PR) so reviewers/users understand the full impact.
Summary
@actions/cachefrom ^4.1.0 to ^6.0.0@actions/corefrom ^1.11.1 to ^3.0.0@actions/execfrom ^1.1.1 to ^3.0.0zodfrom ^3.25.76 to ^4.0.0ZodError.errors→ZodError.issuesinsrc/inputs.tsConsolidates the following Renovate PRs:
Test plan
vp run check:fixpassesvp run testpasses (86 tests, 5 files)vp run buildproduces updateddist/index.mjs