From 405e6bfc297319e8c24c435a619a0ac010bcf59b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?V=C3=ADctor=20Falc=C3=B3n?= <victoor89@gmail.com>
Date: Sun, 26 Apr 2026 09:15:06 +0100
Subject: [PATCH] ci: skip build and deploy when no code changes (#335)

## Why

Pushes to `main` that only modify workflows, README, or docs were
triggering production deploys.

## What

- Adds a `changes` job using `dorny/paths-filter@v3` to detect whether
the push touched application code.
- Gates `build-image` and `deploy` on `needs.changes.outputs.code ==
'true'`.

## Code paths considered

`app/`, `bootstrap/`, `config/`, `database/`, `public/`, `resources/`,
`routes/`, `tests/`, `storage/`, `artisan`, `composer.{json,lock}`,
`package*.json`, `bun.lock*`, `vite.config.*`, `tsconfig*.json`,
`eslint.config.*`, `.prettierrc*`, `phpstan.neon*`, `phpunit.xml*`,
`pint.json`, `Dockerfile`, `docker/**`, `.dockerignore`, `.env.example`.

Anything else (e.g. `.github/**`, `*.md`, `docs/**`) is treated as
non-deployable.
---
 .github/workflows/ci.yml           | 47 +++++++++++++++++++++++++++---
 tests/Feature/SentryConfigTest.php |  2 +-
 2 files changed, 44 insertions(+), 5 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 484b4a9d..aae6a0b1 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -223,10 +223,49 @@ jobs:
           DB_USERNAME: root
           DB_PASSWORD: password
 
+  changes:
+    runs-on: ubuntu-latest
+    outputs:
+      code: ${{ steps.filter.outputs.code }}
+    steps:
+      - uses: actions/checkout@v4
+      - uses: dorny/paths-filter@v3
+        id: filter
+        with:
+          filters: |
+            code:
+              - 'app/**'
+              - 'bootstrap/**'
+              - 'config/**'
+              - 'database/**'
+              - 'public/**'
+              - 'resources/**'
+              - 'routes/**'
+              - 'tests/**'
+              - 'storage/**'
+              - 'artisan'
+              - 'composer.json'
+              - 'composer.lock'
+              - 'package.json'
+              - 'package-lock.json'
+              - 'bun.lock'
+              - 'bun.lockb'
+              - 'vite.config.*'
+              - 'tsconfig*.json'
+              - 'eslint.config.*'
+              - '.prettierrc*'
+              - 'phpstan.neon*'
+              - 'phpunit.xml*'
+              - 'pint.json'
+              - 'Dockerfile'
+              - 'docker/**'
+              - '.dockerignore'
+              - '.env.example'
+
   build-image:
     runs-on: ubuntu-latest
-    needs: [tests, linter, static-analysis, performance-tests]
-    if: (github.ref == 'refs/heads/main' && github.event_name == 'push') || github.event_name == 'workflow_dispatch'
+    needs: [tests, linter, static-analysis, performance-tests, changes]
+    if: ((github.ref == 'refs/heads/main' && github.event_name == 'push') || github.event_name == 'workflow_dispatch') && needs.changes.outputs.code == 'true'
     env:
       SENTRY_RELEASE: whisper-money@${{ github.sha }}
     permissions:
@@ -291,8 +330,8 @@ jobs:
 
   deploy:
     runs-on: ubuntu-latest
-    needs: [build-image]
-    if: github.ref == 'refs/heads/main' && github.event_name == 'push'
+    needs: [build-image, changes]
+    if: github.ref == 'refs/heads/main' && github.event_name == 'push' && needs.changes.outputs.code == 'true'
     env:
       SENTRY_RELEASE: whisper-money@${{ github.sha }}
     concurrency:
diff --git a/tests/Feature/SentryConfigTest.php b/tests/Feature/SentryConfigTest.php
index 8a898a47..2d69b4ec 100644
--- a/tests/Feature/SentryConfigTest.php
+++ b/tests/Feature/SentryConfigTest.php
@@ -8,6 +8,6 @@ it('waits for the image build before marking a sentry deploy', function () {
     $workflow = file_get_contents(base_path('.github/workflows/ci.yml'));
 
     expect($workflow)
-        ->toContain("deploy:\n    runs-on: ubuntu-latest\n    needs: [build-image]")
+        ->toContain("deploy:\n    runs-on: ubuntu-latest\n    needs: [build-image, changes]")
         ->toContain('run: sentry-cli releases deploys "$SENTRY_RELEASE" new -e production');
 });