ci: no-op heavy jobs on version-bump PRs so matrix checks report (PER-9560)

Author: AkashBrowserStack

.github/workflows/windows.yml

@@ -14,42 +14,51 @@ jobs:
     name: Detect version-only changes
     runs-on: ubuntu-latest
     outputs:
-      version_only: ${{ steps.filter.outputs.version_only }}
+      skip_heavy: ${{ steps.decide.outputs.skip_heavy }}
     steps:
-      - name: Check the PR changes only version files
-        id: filter
-        if: github.event_name == 'pull_request'
+      - name: Decide whether to skip the heavy test suites
+        id: decide
         env:
           GH_TOKEN: ${{ github.token }}
+          EVENT: ${{ github.event_name }}
+          HEAD_REF: ${{ github.head_ref }}
+          AUTHOR: ${{ github.event.pull_request.user.login }}
           PR: ${{ github.event.pull_request.number }}
           REPO: ${{ github.repository }}
         run: |
-          if ! files=$(gh api "repos/$REPO/pulls/$PR/files" --paginate --jq '.[].filename'); then
-            echo "Could not list PR files — running full CI to be safe."
-            echo "version_only=false" >> "$GITHUB_OUTPUT"
-            exit 0
-          fi
-          printf 'Changed files:\n%s\n' "$files"
-          # version_only=true only if EVERY changed file is lerna.json or a top-level
-          # packages/<pkg>/package.json (exactly what version-bump.yml commits).
-          others=$(printf '%s\n' "$files" | grep -vE '^(lerna\.json|packages/[^/]+/package\.json)$' || true)
-          if [ -n "$files" ] && [ -z "$others" ]; then
-            echo "version_only=true" >> "$GITHUB_OUTPUT"
-          else
-            echo "version_only=false" >> "$GITHUB_OUTPUT"
+          # Skip the heavy matrices only for automated version-bump PRs: a
+          # github-actions[bot] PR on a release/* branch whose diff is confined to
+          # lerna.json + packages/<pkg>/package.json (exactly what version-bump.yml
+          # commits). The jobs below still RUN (so the matrix expands and every
+          # required "Test @percy/<pkg>" check is created/reported) — only their
+          # work steps no-op. Fail-safe: anything unexpected -> skip_heavy=false ->
+          # full CI runs (PER-9560).
+          skip=false
+          if [ "$EVENT" = "pull_request" ] && [ "$AUTHOR" = "github-actions[bot]" ] && printf '%s' "$HEAD_REF" | grep -q '^release/'; then
+            if files=$(gh api "repos/$REPO/pulls/$PR/files" --paginate --jq '.[].filename'); then
+              printf 'Changed files:\n%s\n' "$files"
+              others=$(printf '%s\n' "$files" | grep -vE '^(lerna\.json|packages/[^/]+/package\.json)$' || true)
+              if [ -n "$files" ] && [ -z "$others" ]; then skip=true; fi
+            else
+              echo "Could not list PR files — running full CI to be safe."
+            fi
           fi
+          echo "skip_heavy=$skip" >> "$GITHUB_OUTPUT"
+          echo "skip_heavy=$skip"
 
   build:
     name: Build
     needs: [changes]
-    if: ${{ !(startsWith(github.head_ref, 'release/') && github.event.pull_request.user.login == 'github-actions[bot]' && needs.changes.outputs.version_only == 'true') }}
     runs-on: windows-latest
     steps:
       - uses: actions/checkout@v5
+        if: needs.changes.outputs.skip_heavy != 'true'
       - uses: actions/setup-node@v3
+        if: needs.changes.outputs.skip_heavy != 'true'
         with:
           node-version: 14
       - uses: actions/cache@v3
+        if: needs.changes.outputs.skip_heavy != 'true'
         with:
           path: |
             node_modules
@@ -62,17 +71,19 @@ jobs:
           restore-keys: >
             ${{ runner.os }}/node-14/
             ${{ hashFiles('.github/.cache-key') }}/
-      - run: yarn
-      - run: yarn build
+      - if: needs.changes.outputs.skip_heavy != 'true'
+        run: yarn
+      - if: needs.changes.outputs.skip_heavy != 'true'
+        run: yarn build
       - uses: actions/upload-artifact@v4
+        if: needs.changes.outputs.skip_heavy != 'true'
         with:
           name: dist
           path: packages/*/dist
 
   test:
     name: Test ${{ matrix.package }}
     needs: [build, changes]
-    if: ${{ !(startsWith(github.head_ref, 'release/') && github.event.pull_request.user.login == 'github-actions[bot]' && needs.changes.outputs.version_only == 'true') }}
     strategy:
       fail-fast: false
       matrix:
@@ -102,10 +113,13 @@ jobs:
       CLI_TEST_FAILURES_FILE: ${{ github.workspace }}/.cli-test-failures.json
     steps:
       - uses: actions/checkout@v5
+        if: needs.changes.outputs.skip_heavy != 'true'
       - uses: actions/setup-node@v3
+        if: needs.changes.outputs.skip_heavy != 'true'
         with:
           node-version: 14
       - uses: actions/cache@v3
+        if: needs.changes.outputs.skip_heavy != 'true'
         with:
           path: |
             node_modules
@@ -119,17 +133,22 @@ jobs:
             ${{ runner.os }}/node-14/
             ${{ hashFiles('.github/.cache-key') }}/
       - uses: actions/download-artifact@v5
+        if: needs.changes.outputs.skip_heavy != 'true'
         with:
           name: dist
           path: packages
-      - run: yarn
-      # First attempt runs the full suite and records any failed specs.
+      - if: needs.changes.outputs.skip_heavy != 'true'
+        run: yarn
+      # First attempt runs the full suite and records any failed specs. Skipped
+      # (with every other step) on version-bump PRs — see the `changes` job.
       - name: Run tests
         continue-on-error: true
         id: retry0
+        if: needs.changes.outputs.skip_heavy != 'true'
         run: yarn workspace ${{ matrix.package }} test --colors
       # Retries re-run ONLY the specs that failed in the previous attempt
       # (CLI_TEST_ONLY_FAILED), so a flaky spec costs seconds, not ~60 min.
+      # (These auto-skip when retry0 is skipped, so they need no extra guard.)
       - name: Run tests Retry (1/4)
         continue-on-error: true
         id: retry1