Address pr comments

Author: marcorosa

.github/workflows/docker.yml

@@ -70,7 +70,7 @@ jobs:
               BUILD_FRONTEND=true
               echo "✅ Frontend version changed: $PREVIOUS_FRONTEND_VERSION → $FRONTEND_VERSION"
             else
-              echo "❎ Frontend version unchanged: $FRONTEND_VERSION. Skip docker backend."
+              echo "❎ Frontend version unchanged: $FRONTEND_VERSION. Skip docker frontend."
             fi
           fi
 

.github/workflows/pr-bot.yml

@@ -22,3 +22,5 @@ jobs:
           aicore-service-key: ${{ secrets.AICORE_SERVICE_KEY }}
           model: anthropic--claude-4-sonnet
           exclude-files: package-lock.json, uv.lock
+          footer-text: |
+            Always critique what AI says. Do not let AI replace YOUR I.

frontend/generate-config.sh

@@ -3,6 +3,14 @@
 # Generate runtime configuration from environment variables
 # This script runs at container startup to create config.json from environment variables
 
+# Validate required environment variables
+required_vars="BACKEND_URL BACKEND_WS_URL API_URL"
+for var in $required_vars; do
+    if [ -z "$(eval echo \$$var)" ]; then
+        echo "Warning: $var is not set, using default value"
+    fi
+done
+
 # Set default values if environment variables are not provided
 BACKEND_URL=${BACKEND_URL:-"http://localhost:8080"}
 BACKEND_WS_URL=${BACKEND_WS_URL:-"ws://localhost:8080/agent"}

frontend/src/app/weight-dialog/weight-dialog.component.ts

@@ -38,6 +38,17 @@ export class WeightDialogComponent implements OnInit {
   }
 
   onSave() {
+    // Validate weights before sending
+    const invalidWeights = Object.entries(this.currentWeights)
+      .filter(([_, weight]) => weight < 0 || !Number.isInteger(weight));
+
+    if (invalidWeights.length > 0) {
+      this.snackBar.open('Please enter valid positive integers for all weights', '❌', {
+        duration: 3000
+      });
+      return;
+    }
+
     const headers = new HttpHeaders({
       'X-API-Key': this.apiKey
     });