From a6a0f7a373b039997ba8f1294edd30aa4cc39170 Mon Sep 17 00:00:00 2001
From: Anish Sarkar <104695310+AnishSarkar22@users.noreply.github.com>
Date: Wed, 20 May 2026 20:03:51 +0530
Subject: [PATCH] chore: streamline GitHub Actions workflow by removing change
 detection job and simplifying test conditions

---
 .github/workflows/backend-tests.yml | 49 +++++------------------------
 1 file changed, 8 insertions(+), 41 deletions(-)

diff --git a/.github/workflows/backend-tests.yml b/.github/workflows/backend-tests.yml
index ec8a2e7f6..7916dcf50 100644
--- a/.github/workflows/backend-tests.yml
+++ b/.github/workflows/backend-tests.yml
@@ -4,36 +4,19 @@ on:
   pull_request:
     branches: [main, dev]
     types: [opened, synchronize, reopened, ready_for_review]
+    paths:
+      - 'surfsense_backend/**'
+      - '.github/workflows/backend-tests.yml'
 
 concurrency:
   group: ${{ github.workflow }}-${{ github.ref }}
   cancel-in-progress: true
 
 jobs:
-  changes:
-    name: Changes
-    runs-on: ubuntu-latest
-    outputs:
-      backend: ${{ steps.filter.outputs.backend }}
-
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v6
-
-      - name: Check changed files
-        id: filter
-        uses: dorny/paths-filter@v4
-        with:
-          filters: |
-            backend:
-              - 'surfsense_backend/**'
-              - '.github/workflows/backend-tests.yml'
-
   unit-tests:
     name: Unit Tests
     runs-on: ubuntu-latest
-    needs: changes
-    if: ${{ github.event.pull_request.draft == false && needs.changes.outputs.backend == 'true' }}
+    if: github.event.pull_request.draft == false
     env:
       EMBEDDING_MODEL: sentence-transformers/all-MiniLM-L6-v2
 
@@ -76,8 +59,7 @@ jobs:
   integration-tests:
     name: Integration Tests
     runs-on: ubuntu-latest
-    needs: changes
-    if: ${{ github.event.pull_request.draft == false && needs.changes.outputs.backend == 'true' }}
+    if: github.event.pull_request.draft == false
     env:
       EMBEDDING_MODEL: sentence-transformers/all-MiniLM-L6-v2
 
@@ -139,29 +121,14 @@ jobs:
   test-gate:
     name: Test Gate
     runs-on: ubuntu-latest
-    needs: [changes, unit-tests, integration-tests]
+    needs: [unit-tests, integration-tests]
     if: always()
 
     steps:
       - name: Check all test jobs
         run: |
-          if [[ "${{ needs.changes.result }}" == "failure" || "${{ needs.changes.result }}" == "cancelled" ]]; then
-            echo "Backend change detection failed"
-            exit 1
-          fi
-
-          if [[ "${{ github.event.pull_request.draft }}" == "true" ]]; then
-            echo "Draft PR; backend tests skipped"
-            exit 0
-          fi
-
-          if [[ "${{ needs.changes.outputs.backend }}" != "true" ]]; then
-            echo "No backend changes detected; backend tests skipped"
-            exit 0
-          fi
-
-          if [[ "${{ needs.unit-tests.result }}" != "success" ||
-                "${{ needs.integration-tests.result }}" != "success" ]]; then
+          if [[ "${{ needs.unit-tests.result }}" == "failure" ||
+                "${{ needs.integration-tests.result }}" == "failure" ]]; then
             echo "Backend tests failed"
             exit 1
           else