M8F-66: [Feature] Added tests to CI workflow and Updated PR notification

Author: vinaayakh-aot

.github/workflows/ci.yml

@@ -15,12 +15,12 @@ permissions:
 
 jobs:
   changes:
-    if: vars.AUTO_BUILD == 'true' || github.event_name == 'workflow_dispatch'
     runs-on: ubuntu-latest
     outputs:
       backend: ${{ steps.filter.outputs.backend }}
       frontend: ${{ steps.filter.outputs.frontend }}
-      extensions: ${{ steps.filter.outputs.extensions }}
+      extensions_backend: ${{ steps.filter.outputs.extensions_backend }}
+      extensions_frontend: ${{ steps.filter.outputs.extensions_frontend }}
       migrations: ${{ steps.filter.outputs.migrations }}
       docker_m8flow: ${{ steps.filter.outputs.docker_m8flow }}
     steps:
@@ -33,9 +33,11 @@ jobs:
               - 'spiffworkflow-backend/**'
             frontend:
               - 'spiffworkflow-frontend/**'
-            extensions:
+            extensions_backend:
               - 'extensions/m8flow-backend/**'
               - 'spiffworkflow-backend/**'
+            extensions_frontend:
+              - 'extensions/frontend/**'
             migrations:
               - 'spiffworkflow-backend/migrations/versions/*.py'
             docker_m8flow:
@@ -47,7 +49,7 @@ jobs:
   # -------------------------
   backend:
     needs: changes
-    if: ${{ (vars.AUTO_BUILD == 'true' || github.event_name == 'workflow_dispatch') && needs.changes.outputs.backend == 'true' }}
+    if: ${{ needs.changes.outputs.backend == 'true' }}
     runs-on: ubuntu-latest
     defaults:
       run:
@@ -56,7 +58,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-python@v5
         with:
-          python-version: '3.10'
+          python-version: '3.11'
       - name: Install uv
         run: pip install uv
       - name: Install dependencies
@@ -73,7 +75,7 @@ jobs:
   # -------------------------
   frontend:
     needs: changes
-    if: ${{ (vars.AUTO_BUILD == 'true' || github.event_name == 'workflow_dispatch') && needs.changes.outputs.frontend == 'true' }}
+    if: ${{ needs.changes.outputs.frontend == 'true' }}
     runs-on: ubuntu-latest
     defaults:
       run:
@@ -93,11 +95,11 @@ jobs:
         run: npm test
 
   # -------------------------
-  # Extensions CI
+  # Extensions Backend CI
   # -------------------------
-  extensions:
+  extensions-backend:
     needs: changes
-    if: ${{ (vars.AUTO_BUILD == 'true' || github.event_name == 'workflow_dispatch') && needs.changes.outputs.extensions == 'true' }}
+    if: ${{ needs.changes.outputs.extensions_backend == 'true' }}
     runs-on: ubuntu-latest
     defaults:
       run:
@@ -106,7 +108,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-python@v5
         with:
-          python-version: '3.10'
+          python-version: '3.11'
       - name: Install uv
         run: pip install uv
       - name: Install dependencies
@@ -116,13 +118,37 @@ jobs:
           export PYTHONPATH=$PYTHONPATH:$(pwd):$(pwd)/src:$(pwd)/../extensions/m8flow-backend/src
           uv run pytest ../extensions/m8flow-backend/tests
 
+  # -------------------------
+  # Extensions Frontend CI
+  # -------------------------
+  extensions-frontend:
+    needs: changes
+    if: ${{ needs.changes.outputs.extensions_frontend == 'true' }}
+    runs-on: ubuntu-latest
+    defaults:
+      run:
+        working-directory: extensions/frontend
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: '20'
+          cache: 'npm'
+          cache-dependency-path: extensions/frontend/package-lock.json
+      - name: Install dependencies
+        run: npm ci
+      - name: Typecheck
+        run: npm run typecheck
+      - name: Test
+        run: npm test
+
   # -------------------------
   # Security: CodeQL
   # -------------------------
   codeql:
     name: CodeQL Scan
     runs-on: ubuntu-latest
-    if: ${{ (vars.AUTO_BUILD == 'true' || github.event_name == 'workflow_dispatch') && github.event_name == 'pull_request' }}
+    if: ${{ github.event_name == 'pull_request' }}
     strategy:
       matrix:
         language: [ 'python', 'javascript' ]
@@ -141,7 +167,7 @@ jobs:
   trivy:
     name: Trivy Security Scan
     runs-on: ubuntu-latest
-    if: ${{ (vars.AUTO_BUILD == 'true' || github.event_name == 'workflow_dispatch') && github.event_name == 'pull_request' }}
+    if: ${{ github.event_name == 'pull_request' }}
     steps:
       - uses: actions/checkout@v4
       - uses: aquasecurity/trivy-action@0.20.0
@@ -158,7 +184,7 @@ jobs:
   migration-check:
     name: Migration Compatibility Check
     needs: changes
-    if: ${{ (vars.AUTO_BUILD == 'true' || github.event_name == 'workflow_dispatch') && needs.changes.outputs.migrations == 'true' && github.event_name == 'pull_request' }}
+    if: ${{ needs.changes.outputs.migrations == 'true' && github.event_name == 'pull_request' }}
     uses: ./.github/workflows/check-migrations.yml
     with:
       migrations_changed: true
@@ -170,7 +196,7 @@ jobs:
     name: Docker Build Dry Run
     needs: changes
     runs-on: ubuntu-latest
-    if: ${{ (vars.AUTO_BUILD == 'true' || github.event_name == 'workflow_dispatch') && (needs.changes.outputs.backend == 'true' || needs.changes.outputs.frontend == 'true' || needs.changes.outputs.extensions == 'true' || needs.changes.outputs.docker_m8flow == 'true') && github.event_name == 'pull_request' }}
+    if: ${{ (needs.changes.outputs.backend == 'true' || needs.changes.outputs.frontend == 'true' || needs.changes.outputs.extensions_backend == 'true' || needs.changes.outputs.docker_m8flow == 'true') && github.event_name == 'pull_request' }}
     steps:
       - uses: actions/checkout@v4
       - uses: docker/setup-buildx-action@v3

.github/workflows/pr-notification.yml

@@ -1,13 +1,14 @@
 name: Notify Google Chat on New Issue
 
 on:
-  pull_request:
+  pull_request_target:
     branches: [ main ]
     types: [ opened ]
 
 jobs:
   send-alert:
     runs-on: ubuntu-latest
+    if: github.event.pull_request.draft == false
     steps:
       - name: Send message to Google Chat
         env: