Revert "Update actions/cache version to 4.2.2 to resolve caching warning pack…"

This reverts commit 9ccf8a2650.

.github/workflows/ci.yml

@@ -19,7 +19,7 @@ jobs:
           - ubuntu
           - windows
     steps:
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v4
       - uses: ./
       - if: failure() && runner.debug == '1'
         uses: mxschmitt/action-tmate@v3

.github/workflows/release.yml

@@ -15,8 +15,8 @@ jobs:
   release:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v4
-      - uses: actions/setup-node@v6
+      - uses: actions/setup-node@v4
         with:
           cache: npm
           node-version-file: package.json

.tool-versions

@@ -1 +1 @@
-nodejs 24.14.0
+nodejs 22.14.0

README.md

@@ -6,7 +6,7 @@ and provides an advanced fine-grained caching to improve workflows performance.
 ## Usage
 
 ```yaml
-- uses: bazel-contrib/setup-bazel@0.18.0
+- uses: bazel-contrib/setup-bazel@0.14.0
   with:
     # Avoid downloading Bazel every time.
     bazelisk-cache: true
@@ -39,7 +39,7 @@ Default `""`.
   #### Install Bazelisk 1.x
 
   ```yaml
-  - uses: bazel-contrib/setup-bazel@0.18.0
+  - uses: bazel-contrib/setup-bazel@0.14.0
     with:
       bazelisk-version: 1.x
   ```
@@ -47,7 +47,7 @@ Default `""`.
   #### Install exact Bazelisk version
 
   ```yaml
-  - uses: bazel-contrib/setup-bazel@0.18.0
+  - uses: bazel-contrib/setup-bazel@0.14.0
     with:
       bazelisk-version: 1.19.0
   ```
@@ -67,7 +67,7 @@ Default `""`.
   #### Enable Bzlmod
 
   ```yaml
-  - uses: bazel-contrib/setup-bazel@0.18.0
+  - uses: bazel-contrib/setup-bazel@0.14.0
     with:
       bazelrc: common --enable_bzlmod
   ```
@@ -75,7 +75,7 @@ Default `""`.
   #### Add colors and timestamps
 
   ```yaml
-  - uses: bazel-contrib/setup-bazel@0.18.0
+  - uses: bazel-contrib/setup-bazel@0.14.0
     with:
       bazelrc: |
         build --color=yes
@@ -83,30 +83,6 @@ Default `""`.
   ```
 </details>
 
-### `cache-save`
-
-Whether to save caches at the end of the workflow.
-
-Set to `false` for pull requests to allow cache restoration without saving,
-which prevents PRs from polluting the cache while still benefiting from it.
-
-Default `true`.
-
-<details>
-  <summary>Examples</summary>
-
-  #### Disable cache saving on pull requests
-
-  ```yaml
-  - uses: bazel-contrib/setup-bazel@0.18.0
-    with:
-      bazelisk-cache: true
-      disk-cache: ${{ github.workflow }}
-      repository-cache: true
-      cache-save: ${{ github.event_name != 'pull_request' }}
-  ```
-</details>
-
 ### `disk-cache`
 
 Enable [`disk_cache`][2] and store it on GitHub based on contents of `BUILD` files.
@@ -121,7 +97,7 @@ Default `false`.
   #### Share a single disk cache
 
   ```yaml
-  - uses: bazel-contrib/setup-bazel@0.18.0
+  - uses: bazel-contrib/setup-bazel@0.14.0
     with:
       disk-cache: true
   ```
@@ -129,9 +105,9 @@ Default `false`.
   #### Separate disk caches between workflows
 
   ```yaml
-  - uses: bazel-contrib/setup-bazel@0.18.0
+  - uses: bazel-contrib/setup-bazel@0.14.0
     with:
-      disk-cache: ${{ github.workflow }}
+      disk-cache: ${{ github.workflow }}}
   ```
 </details>
 
@@ -153,7 +129,7 @@ Default `false`.
   #### Enable external repositories caches
 
   ```yaml
-  - uses: bazel-contrib/setup-bazel@0.18.0
+  - uses: bazel-contrib/setup-bazel@0.14.0
     with:
       external-cache: true
   ```
@@ -161,7 +137,7 @@ Default `false`.
   #### Cache NPM repositories based on `package-lock.json` contents
 
   ```yaml
-  - uses: bazel-contrib/setup-bazel@0.18.0
+  - uses: bazel-contrib/setup-bazel@0.14.0
     with:
       external-cache: |
         manifest:
@@ -171,7 +147,7 @@ Default `false`.
   #### Do not cache Ruby on Windows
 
   ```yaml
-  - uses: bazel-contrib/setup-bazel@0.18.0
+  - uses: bazel-contrib/setup-bazel@0.14.0
     with:
       external-cache: |
         manifest:
@@ -191,7 +167,7 @@ Default `""`.
   #### Authenticate via key
 
   ```yaml
-  - uses: bazel-contrib/setup-bazel@0.18.0
+  - uses: bazel-contrib/setup-bazel@0.14.0
     with:
       google-credentials: ${{ secrets.GOOGLE_CLOUD_KEY }}
   ```
@@ -222,7 +198,7 @@ Default is one of the following:
   #### Use `C` drive letter
 
   ```yaml
-  - uses: bazel-contrib/setup-bazel@0.18.0
+  - uses: bazel-contrib/setup-bazel@0.14.0
     with:
       output-base: C:/_bazel
   ```
@@ -242,7 +218,7 @@ Default `false`.
   #### Store a single repository cache
 
   ```yaml
-  - uses: bazel-contrib/setup-bazel@0.18.0
+  - uses: bazel-contrib/setup-bazel@0.14.0
     with:
       repository-cache: true
   ```
@@ -250,19 +226,10 @@ Default `false`.
   #### Store a repository cache from a custom location
 
   ```yaml
-  - uses: bazel-contrib/setup-bazel@0.18.0
+  - uses: bazel-contrib/setup-bazel@0.14.0
     with:
       repository-cache: examples/gem/WORKSPACE
   ```
-
-  #### Store a repository cache from a list of custom locations
-  ```yaml
-  - uses: bazel-contrib/setup-bazel@0.18.0
-    with:
-      repository-cache: |
-        - MODULE.bazel
-        - requirements_locked.txt
-  ```
 </details>
 
 ## Migrating from [`bazelbuild/setup-bazelisk`][6]
@@ -276,7 +243,6 @@ However, if you used a `bazel-version` input before, you will need to remove it
 To build action, run the following command:
 
 ```sh
-$ npm install
 $ npm run build
 ```
 

action.yml

@@ -13,10 +13,6 @@ inputs:
     description: Extra contents to write to user .bazelrc
     required: false
     default: ""
-  cache-save:
-    description: Whether to save caches. Set to false for pull requests to allow restores but prevent saves.
-    required: false
-    default: "true"
   cache-version:
     description: Version of all caches
     required: false
@@ -54,4 +50,3 @@ runs:
   using: node20
   main: dist/main/index.js
   post: dist/post/index.js
-  post-if: "!cancelled()"

config.js

@@ -5,8 +5,8 @@ const core = require('@actions/core')
 const github = require('@actions/github')
 
 const bazeliskVersion = core.getInput('bazelisk-version')
-const cacheSave = core.getBooleanInput('cache-save')
 const cacheVersion = core.getInput('cache-version')
+const externalCacheConfig = yaml.parse(core.getInput('external-cache'))
 const moduleRoot = core.getInput('module-root')
 
 const homeDir = os.homedir()
@@ -16,12 +16,7 @@ const platform = os.platform()
 let bazelOutputBase = core.getInput('output-base')
 if (!bazelOutputBase) {
   if (platform === 'win32') {
-    // check if GITHUB_WORKSPACE starts with D:
-    if (process.env.GITHUB_WORKSPACE?.toLowerCase()?.startsWith('d:')) {
     bazelOutputBase = 'D:/_bazel'
-    } else {
-      bazelOutputBase = `C:/_bazel`
-    }
   } else {
     bazelOutputBase = `${homeDir}/.bazel`
   }
@@ -46,24 +41,21 @@ switch (platform) {
     break
 }
 
-const baseCacheKey = `setup-bazel-${cacheVersion}-${platform}-${arch}`
+const baseCacheKey = `setup-bazel-${cacheVersion}-${platform}`
 const bazelrc = core.getMultilineInput('bazelrc')
 
 const diskCacheConfig = core.getInput('disk-cache')
 const diskCacheEnabled = diskCacheConfig !== 'false'
 let diskCacheName = 'disk'
 if (diskCacheEnabled) {
-  // Before Bazel 6.3, providing --disk_cache to common is an error,
+  bazelrc.push(`common --disk_cache=${bazelDisk}`)
-  // with Bazel 6.3 and onwards, common accepts any legal Bazel option
-  // https://github.com/bazelbuild/bazel/issues/3054
-  bazelrc.push(`build --disk_cache=${bazelDisk}`)
   if (diskCacheName !== 'true') {
     diskCacheName = `${diskCacheName}-${diskCacheConfig}`
   }
 }
 
-const repositoryCacheConfig = yaml.parse(core.getInput('repository-cache'))
+const repositoryCacheConfig = core.getInput('repository-cache')
-const repositoryCacheEnabled = repositoryCacheConfig !== false
+const repositoryCacheEnabled = repositoryCacheConfig !== 'false'
 let repositoryCacheFiles = [
   `${moduleRoot}/MODULE.bazel`,
   `${moduleRoot}/WORKSPACE.bazel`,
@@ -72,7 +64,7 @@ let repositoryCacheFiles = [
 ]
 if (repositoryCacheEnabled) {
   bazelrc.push(`common --repository_cache=${bazelRepository}`)
-  if (repositoryCacheConfig !== true) {
+  if (repositoryCacheConfig !== 'true') {
     repositoryCacheFiles = Array(repositoryCacheConfig).flat()
   }
 }
@@ -87,7 +79,6 @@ if (googleCredentials.length > 0 && !googleCredentialsSaved) {
   core.saveState('google-credentials-path', googleCredentialsPath)
 }
 
-const externalCacheConfig = yaml.parse(core.getInput('external-cache'))
 const bazelExternal = core.toPosixPath(`${bazelOutputBase}/external`)
 const externalCache = {}
 if (externalCacheConfig) {
@@ -138,7 +129,6 @@ core.exportVariable('BAZELISK_GITHUB_TOKEN', token)
 
 module.exports = {
   baseCacheKey,
-  cacheSave,
   bazeliskCache: {
     enabled: core.getBooleanInput('bazelisk-cache'),
     files: [`${moduleRoot}/.bazelversion`],

index.js

@@ -96,11 +96,7 @@ async function downloadBazelisk() {
 
   core.debug('Adding to the cache...');
   fs.chmodSync(downloadPath, '755');
-  let bazel_name = "bazel";
+  const cachePath = await tc.cacheFile(downloadPath, 'bazel', 'bazelisk', version)
-  if (platform == 'windows') {
-    bazel_name = `${bazel_name}.exe`
-  }
-  const cachePath = await tc.cacheFile(downloadPath, bazel_name, 'bazelisk', version)
   core.debug(`Successfully cached bazelisk to ${cachePath}`)
 
   return cachePath
@@ -133,16 +129,14 @@ async function restoreExternalCaches(cacheConfig) {
   // Now restore all external caches defined in manifest
   if (fs.existsSync(path)) {
     const manifest = fs.readFileSync(path, { encoding: 'utf8' })
-    const restorePromises = manifest.split('\n').filter(s => s)
+    for (const name of manifest.split('\n').filter(s => s)) {
-      .map(name => {
+      await restoreCache({
-        return restoreCache({
         enabled: cacheConfig[name]?.enabled ?? cacheConfig.default.enabled,
         files: cacheConfig[name]?.files || cacheConfig.default.files,
         name: cacheConfig.default.name(name),
         paths: cacheConfig.default.paths(name)
-        });
+      })
-      });
+    }
-    await Promise.all(restorePromises);
   }
 }
 
@@ -152,10 +146,9 @@ async function restoreCache(cacheConfig) {
   }
 
   const delay = Math.random() * 1000 // timeout <= 1 sec to reduce 429 errors
-  await setTimeout(delay)
+  await setTimeout(delay, async function () {
-
     core.startGroup(`Restore cache for ${cacheConfig.name}`)
-  try {
+
     const hash = await glob.hashFiles(cacheConfig.files.join('\n'))
     const name = cacheConfig.name
     const paths = cacheConfig.paths
@@ -178,11 +171,9 @@ async function restoreCache(cacheConfig) {
     } else {
       core.info(`Failed to restore ${name} cache`)
     }
-  } catch (err) {
+
-    core.warning(`Failed to restore ${name} cache with error: ${err}`)
-  } finally {
     core.endGroup()
-  }
+  }())
 }
 
 run()

package.json

@@ -1,10 +1,10 @@
 {
   "name": "setup-bazel",
-  "version": "0.18.0",
+  "version": "0.14.0",
   "description": "Install and configure Bazel for GitHub Actions",
   "main": "index.js",
   "engines": {
-    "node": "24.x"
+    "node": "22.x"
   },
   "scripts": {
     "build": "ncc build index.js -s -o dist/main && ncc build post.js -s -o dist/post",
@@ -13,11 +13,11 @@
   "author": "Alex Rodionov <p0deje@gmail.com>",
   "license": "MIT",
   "dependencies": {
-    "@actions/cache": "^6.0.0",
+    "@actions/cache": "^4.0.0",
-    "@actions/core": "^3.0.0",
+    "@actions/core": "^1.10.1",
-    "@actions/github": "^9.0.0",
+    "@actions/github": "^6.0.0",
-    "@actions/glob": "^0.6.0",
+    "@actions/glob": "^0.5.0",
-    "@actions/tool-cache": "^4.0.0",
+    "@actions/tool-cache": "^2.0.1",
     "@vercel/ncc": "^0.38.0",
     "yaml": "^2.2.1"
   }

post.js

@@ -13,11 +13,6 @@ async function run() {
 }
 
 async function saveCaches() {
-  if (!config.cacheSave) {
-    core.info('Cache saving is disabled (cache-save: false)')
-    return
-  }
-
   await saveCache(config.bazeliskCache)
   await saveCache(config.diskCache)
   await saveCache(config.repositoryCache)