Refactor integration workflow skipping with job-level conditions.

Use a dedicated paths-filter job and gate each integration job with jobs.if so skipped checks stay clean and still satisfy required-check reporting.

Made-with: Cursor

Author: Cursor

.github/workflows/proxy_integration_tests_javascript.yml

@@ -10,46 +10,45 @@ on:
 
 permissions:
   contents: read
+  pull-requests: read
 
 jobs:
+  changes:
+    runs-on: ubuntu-latest
+    outputs:
+      relevant: ${{ steps.filter.outputs.relevant }}
+    steps:
+      - name: Detect JavaScript changes
+        id: filter
+        uses: dorny/paths-filter@de90cc6fb38fc0963ad72b210f1f284cd68cea36 # v3
+        with:
+          filters: |
+            relevant:
+              - "javascript/**"
+              - ".github/workflows/proxy_integration_tests_javascript.yml"
+
   integration:
+    needs: changes
+    if: needs.changes.outputs.relevant == 'true'
     runs-on: ubuntu-latest
 
     steps:
       - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2 (Node 24)
         with:
           persist-credentials: false
 
-      - name: Detect JavaScript changes
-        id: changes
-        run: |
-          set -euo pipefail
-          changed="false"
-          while IFS= read -r file; do
-            case "${file}" in
-              javascript/*|.github/workflows/proxy_integration_tests_javascript.yml)
-                changed="true"
-                break
-                ;;
-            esac
-          done < <(git diff --name-only "${{ github.event.pull_request.base.sha }}" "${{ github.sha }}")
-          echo "changed=${changed}" >> "${GITHUB_OUTPUT}"
-
       - name: Set up Node
-        if: steps.changes.outputs.changed == 'true'
         uses: actions/setup-node@53b83947a5a98c8d113130e565377fae1a50d02f # v6.3.0 (Node 24)
         with:
           node-version: "24"
           cache: npm
           cache-dependency-path: javascript/package-lock.json
 
       - name: Install dependencies
-        if: steps.changes.outputs.changed == 'true'
         working-directory: javascript
         run: npm ci
 
       - name: Require PROXY_URL Actions secret
-        if: steps.changes.outputs.changed == 'true'
         env:
           PROXY_URL: ${{ secrets.PROXY_URL }}
         run: |
@@ -59,12 +58,7 @@ jobs:
           fi
 
       - name: Run integration tests
-        if: steps.changes.outputs.changed == 'true'
         working-directory: javascript
         env:
           PROXY_URL: ${{ secrets.PROXY_URL }}
         run: npm test
-
-      - name: Skip when JavaScript files are unchanged
-        if: steps.changes.outputs.changed != 'true'
-        run: echo "No JavaScript integration test changes detected; skipping."

.github/workflows/proxy_integration_tests_php.yml

@@ -10,46 +10,45 @@ on:
 
 permissions:
   contents: read
+  pull-requests: read
 
 jobs:
+  changes:
+    runs-on: ubuntu-latest
+    outputs:
+      relevant: ${{ steps.filter.outputs.relevant }}
+    steps:
+      - name: Detect PHP changes
+        id: filter
+        uses: dorny/paths-filter@de90cc6fb38fc0963ad72b210f1f284cd68cea36 # v3
+        with:
+          filters: |
+            relevant:
+              - "php/**"
+              - ".github/workflows/proxy_integration_tests_php.yml"
+
   integration:
+    needs: changes
+    if: needs.changes.outputs.relevant == 'true'
     runs-on: ubuntu-latest
 
     steps:
       - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2 (Node 24)
         with:
           persist-credentials: false
 
-      - name: Detect PHP changes
-        id: changes
-        run: |
-          set -euo pipefail
-          changed="false"
-          while IFS= read -r file; do
-            case "${file}" in
-              php/*|.github/workflows/proxy_integration_tests_php.yml)
-                changed="true"
-                break
-                ;;
-            esac
-          done < <(git diff --name-only "${{ github.event.pull_request.base.sha }}" "${{ github.sha }}")
-          echo "changed=${changed}" >> "${GITHUB_OUTPUT}"
-
       - name: Set up PHP
-        if: steps.changes.outputs.changed == 'true'
         uses: shivammathur/setup-php@v2
         with:
           php-version: "8.3"
           extensions: curl
           coverage: none
 
       - name: Install Composer dependencies
-        if: steps.changes.outputs.changed == 'true'
         working-directory: php
         run: composer install --no-interaction --prefer-dist
 
       - name: Require PROXY_URL Actions secret
-        if: steps.changes.outputs.changed == 'true'
         env:
           PROXY_URL: ${{ secrets.PROXY_URL }}
         run: |
@@ -59,12 +58,7 @@ jobs:
           fi
 
       - name: Run integration tests
-        if: steps.changes.outputs.changed == 'true'
         working-directory: php
         env:
           PROXY_URL: ${{ secrets.PROXY_URL }}
         run: php run_tests.php
-
-      - name: Skip when PHP files are unchanged
-        if: steps.changes.outputs.changed != 'true'
-        run: echo "No PHP integration test changes detected; skipping."

.github/workflows/proxy_integration_tests_python.yml

@@ -10,50 +10,48 @@ on:
 
 permissions:
   contents: read
+  pull-requests: read
 
 jobs:
+  changes:
+    runs-on: ubuntu-latest
+    outputs:
+      relevant: ${{ steps.filter.outputs.relevant }}
+    steps:
+      - name: Detect Python changes
+        id: filter
+        uses: dorny/paths-filter@de90cc6fb38fc0963ad72b210f1f284cd68cea36 # v3
+        with:
+          filters: |
+            relevant:
+              - "python/**"
+              - ".github/workflows/proxy_integration_tests_python.yml"
+
   integration:
+    needs: changes
+    if: needs.changes.outputs.relevant == 'true'
     runs-on: ubuntu-latest
 
     steps:
       - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2 (Node 24)
         with:
           persist-credentials: false
 
-      - name: Detect Python changes
-        id: changes
-        run: |
-          set -euo pipefail
-          changed="false"
-          while IFS= read -r file; do
-            case "${file}" in
-              python/*|.github/workflows/proxy_integration_tests_python.yml)
-                changed="true"
-                break
-                ;;
-            esac
-          done < <(git diff --name-only "${{ github.event.pull_request.base.sha }}" "${{ github.sha }}")
-          echo "changed=${changed}" >> "${GITHUB_OUTPUT}"
-
       - name: Set up Python
-        if: steps.changes.outputs.changed == 'true'
         uses: actions/setup-python@a309ff8b426b58ec0e2a45f0f869d46889d02405 # v6.2.0 (Node 24)
         with:
           # Pin for reproducible dependency wheels (pycurl, etc.); adjust as needed.
           python-version: "3.12"
 
       - name: Install system dependencies (pycurl)
-        if: steps.changes.outputs.changed == 'true'
         run: sudo apt-get update && sudo apt-get install -y libcurl4-openssl-dev
 
       - name: Install example dependencies
-        if: steps.changes.outputs.changed == 'true'
         run: |
           python -m pip install --upgrade pip
           pip install -r python/requirements.txt
 
       - name: Require PROXY_URL Actions secret
-        if: steps.changes.outputs.changed == 'true'
         env:
           PROXY_URL: ${{ secrets.PROXY_URL }}
         run: |
@@ -63,12 +61,7 @@ jobs:
           fi
 
       - name: Run integration tests
-        if: steps.changes.outputs.changed == 'true'
         working-directory: python
         env:
           PROXY_URL: ${{ secrets.PROXY_URL }}
         run: python run_tests.py
-
-      - name: Skip when Python files are unchanged
-        if: steps.changes.outputs.changed != 'true'
-        run: echo "No Python integration test changes detected; skipping."

.github/workflows/proxy_integration_tests_ruby.yml

@@ -10,41 +10,41 @@ on:
 
 permissions:
   contents: read
+  pull-requests: read
 
 jobs:
+  changes:
+    runs-on: ubuntu-latest
+    outputs:
+      relevant: ${{ steps.filter.outputs.relevant }}
+    steps:
+      - name: Detect Ruby changes
+        id: filter
+        uses: dorny/paths-filter@de90cc6fb38fc0963ad72b210f1f284cd68cea36 # v3
+        with:
+          filters: |
+            relevant:
+              - "ruby/**"
+              - ".github/workflows/proxy_integration_tests_ruby.yml"
+
   integration:
+    needs: changes
+    if: needs.changes.outputs.relevant == 'true'
     runs-on: ubuntu-latest
 
     steps:
       - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2 (Node 24)
         with:
           persist-credentials: false
 
-      - name: Detect Ruby changes
-        id: changes
-        run: |
-          set -euo pipefail
-          changed="false"
-          while IFS= read -r file; do
-            case "${file}" in
-              ruby/*|.github/workflows/proxy_integration_tests_ruby.yml)
-                changed="true"
-                break
-                ;;
-            esac
-          done < <(git diff --name-only "${{ github.event.pull_request.base.sha }}" "${{ github.sha }}")
-          echo "changed=${changed}" >> "${GITHUB_OUTPUT}"
-
       - name: Set up Ruby
-        if: steps.changes.outputs.changed == 'true'
         uses: ruby/setup-ruby@3ff19f5e2baf30647122352b96108b1fbe250c64 # v1.299.0 (Node 24)
         with:
           ruby-version: "3.3"
           bundler-cache: true
           working-directory: ruby
 
       - name: Require PROXY_URL Actions secret
-        if: steps.changes.outputs.changed == 'true'
         env:
           PROXY_URL: ${{ secrets.PROXY_URL }}
         run: |
@@ -54,12 +54,7 @@ jobs:
           fi
 
       - name: Run integration tests
-        if: steps.changes.outputs.changed == 'true'
         working-directory: ruby
         env:
           PROXY_URL: ${{ secrets.PROXY_URL }}
         run: bundle exec ruby run_tests.rb
-
-      - name: Skip when Ruby files are unchanged
-        if: steps.changes.outputs.changed != 'true'
-        run: echo "No Ruby integration test changes detected; skipping."