From a61248aeb09be95e554a38044ccf1073cf59bd8a Mon Sep 17 00:00:00 2001
From: SukkaW <isukkaw@gmail.com>
Date: Wed, 16 Apr 2025 14:56:21 +0800
Subject: [PATCH] CI: improve cloudflare wrangler cache

---
 .github/workflows/main.yml | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
index f8dfc0b8..ba8b0c01 100644
--- a/.github/workflows/main.yml
+++ b/.github/workflows/main.yml
@@ -144,6 +144,10 @@ jobs:
     name: Deploy to Cloudflare Pages
     if: github.ref == 'refs/heads/master'
     runs-on: ubuntu-24.04-arm
+    # matrix is a tricky way to define a variable directly in the jon yaml
+    strategy:
+      matrix:
+        wranglerVersion: ['3.114.6']
     steps:
       - name: Get NPM cache directory path
         id: npm_cache_path
@@ -154,7 +158,7 @@ jobs:
           path: |
             ${{ steps.npm_cache_path.outputs.dir }}
             node_modules
-          key: ${{ runner.os }}-${{ runner.arch }}-deploy-to-cloudflare-npm
+          key: deploy-to-cloudflare-npm-${{ runner.os }}-${{ runner.arch }}-wrangler-${{ matrix.wranglerVersion }}
       - uses: actions/download-artifact@v4
         with:
           name: build-artifact-${{ github.sha }}-${{ github.run_number }}
@@ -165,13 +169,13 @@ jobs:
           apiToken: ${{ secrets.CLOUDFLARE_API_TOKEN }}
           accountId: ${{ secrets.CLOUDFLARE_ACCOUNT_ID }}
           command: pages deploy public --project-name=sukkaw-ruleset --commit-dirty=true --branch=main
-          wranglerVersion: '3.90.0'
+          wranglerVersion: ${{ matrix.wranglerVersion }}
       - uses: actions/cache/save@v4
         with:
           path: |
             ${{ steps.npm_cache_path.outputs.dir }}
             node_modules
-          key: ${{ runner.os }}-${{ runner.arch }}-deploy-to-cloudflare-npm
+          key: deploy-to-cloudflare-npm-${{ runner.os }}-${{ runner.arch }}-wrangler-${{ matrix.wranglerVersion }}
   deploy_to_github_gitlab:
     needs:
       - build