Merge branch 'develop' into main

Author: drorganvidez

app/modules/generator/routes.py

@@ -92,6 +92,8 @@ def validate_step1_form(form):
         num_models = int(num_models_val)
         if num_models < 1:
             errors["num_models_val"] = "Number of models must be at least 1."
+        elif num_models > 1000:
+            errors["num_models_val"] = "Number of models cannot exceed 1000."
     except Exception:
         errors["num_models_val"] = "Number of models must be an integer."
 

app/modules/generator/templates/generator/step1.html

@@ -9,7 +9,7 @@
 
         {{ number_field('num_models_val', 'Number of models',
             'How many random feature models to generate in this batch.',
-            values, errors, default=5, min=1) }}
+            values, errors, default=5, min=1, max=1000) }}
 
         {{ number_field('seed', 'Seed',
             'Random seed used to make runs reproducible. Same seed + same parameters = same models.',

app/modules/generator/tests/test_validators.py

@@ -61,6 +61,16 @@ def test_step1_rejects_negative_seed():
     assert "seed" in errors
 
 
+def test_step1_rejects_num_models_over_cap():
+    errors, _ = validate_step1_form(MultiDict({"num_models_val": "1001", "seed": "1"}))
+    assert "num_models_val" in errors
+
+
+def test_step1_accepts_num_models_at_cap():
+    errors, _ = validate_step1_form(MultiDict({"num_models_val": "1000", "seed": "1"}))
+    assert "num_models_val" not in errors
+
+
 # ── step2 ─────────────────────────────────────────────────────────────────