Skip to content

Bump @actions/cache to 5.1.0, log cache write denied#1569

Open
jasongin wants to merge 3 commits into
actions:mainfrom
jasongin:update-actions-cache-5.1.0
Open

Bump @actions/cache to 5.1.0, log cache write denied#1569
jasongin wants to merge 3 commits into
actions:mainfrom
jasongin:update-actions-cache-5.1.0

Conversation

@jasongin

Copy link
Copy Markdown

Description:

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

@jasongin jasongin requested a review from a team as a code owner June 18, 2026 22:23
Copilot AI review requested due to automatic review settings June 18, 2026 22:23

Copilot AI left a comment

Copy link
Copy Markdown

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 setup-node’s caching behavior to align with @actions/cache@5.1.0, including improved handling/logging when cache writes are denied (for example, read-only tokens on fork PRs). It also bumps the action’s minor version and refreshes generated artifacts and dependency licensing metadata.

Changes:

  • Bump @actions/cache dependency to ^5.1.0 and update action version to 6.5.0.
  • Add a debug log when saveCache returns -1 (cache not saved) and update tests accordingly.
  • Regenerate dist bundles and update the cached dependency license file.
Show a summary per file
File Description
src/cache-save.ts Adds a debug log when @actions/cache.saveCache indicates the cache was not saved (-1).
__tests__/cache-save.test.ts Updates test expectations to validate the new debug log behavior.
package.json Bumps action version to 6.5.0 and updates @actions/cache to ^5.1.0.
package-lock.json Locks @actions/cache to 5.1.0 and updates the root version metadata.
dist/setup/index.js Regenerated bundle reflecting the updated @actions/cache behavior.
dist/cache-save/index.js Regenerated bundle reflecting the new cache-write-denied handling and the new debug log.
.licenses/npm/@actions/cache.dep.yml Updates recorded dependency version for @actions/cache to 5.1.0.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 4/7 changed files
  • Comments generated: 1

Comment thread __tests__/cache-save.test.ts
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