Fix starting/stopping servers

Author: alexeyr

.github/workflows/benchmark.yml

@@ -267,14 +267,17 @@ jobs:
           for i in {1..30}; do
             if curl -fsS http://localhost:3001 > /dev/null; then
               echo "✅ Server is ready and responding"
-              exit 0
+              break
             fi
             echo "  Attempt $i/30: Server not ready yet..."
             sleep 1
           done
 
-          echo "❌ ERROR: Server failed to start within 30 seconds"
-          exit 1
+          # Check if server is actually responding
+          if ! curl -fsS http://localhost:3001 > /dev/null; then
+            echo "❌ ERROR: Server failed to start within 30 seconds"
+            exit 1
+          fi
 
       # ============================================
       # STEP 5: RUN CORE BENCHMARKS
@@ -294,6 +297,18 @@ jobs:
 
           echo "✅ Benchmark suite completed successfully"
 
+      - name: Stop Core production server
+        if: env.APP_VERSION != 'pro_only' && always()
+        run: |
+          echo "🛑 Stopping Core production server..."
+          # Find and kill the Puma process on port 3001
+          if lsof -ti:3001 > /dev/null 2>&1; then
+            kill $(lsof -ti:3001) || true
+            echo "✅ Server stopped"
+          else
+            echo "ℹ️ No server running on port 3001"
+          fi
+
       - name: Validate Core benchmark results
         if: env.APP_VERSION != 'pro_only'
         run: |
@@ -418,14 +433,17 @@ jobs:
           for i in {1..30}; do
             if curl -fsS http://localhost:3001 > /dev/null; then
               echo "✅ Server is ready and responding"
-              exit 0
+              break
             fi
             echo "  Attempt $i/30: Server not ready yet..."
             sleep 1
           done
 
-          echo "❌ ERROR: Server failed to start within 30 seconds"
-          exit 1
+          # Check if server is actually responding
+          if ! curl -fsS http://localhost:3001 > /dev/null; then
+            echo "❌ ERROR: Server failed to start within 30 seconds"
+            exit 1
+          fi
 
       # ============================================
       # STEP 7: RUN PRO BENCHMARKS
@@ -445,6 +463,18 @@ jobs:
 
           echo "✅ Benchmark suite completed successfully"
 
+      - name: Stop Pro production server
+        if: env.APP_VERSION != 'core_only' && always()
+        run: |
+          echo "🛑 Stopping Pro production server..."
+          # Find and kill the Puma process on port 3001
+          if lsof -ti:3001 > /dev/null 2>&1; then
+            kill $(lsof -ti:3001) || true
+            echo "✅ Server stopped"
+          else
+            echo "ℹ️ No server running on port 3001"
+          fi
+
       - name: Validate Pro benchmark results
         if: env.APP_VERSION != 'core_only'
         run: |