fix: zero rows now shows 0 instead of — (null)

Backend: add _sum_or_none() helper — returns None only when ALL model
values are None, returns 0 when sum is legitimately zero.

Frontend: replace `|| null` with `.some(m => m.field != null)` check.
A zero-row incremental run now correctly shows "0" instead of "—".

Author: wicky-zipstack

backend/backend/core/scheduler/celery_tasks.py

@@ -30,6 +30,15 @@
 
 logger = logging.getLogger(__name__)
 
+
+def _sum_or_none(results, attr):
+    """Sum an attribute across results. Return None only if ALL values are None,
+    return 0 if the sum is legitimately zero (not all missing)."""
+    values = [getattr(r, attr, None) for r in results]
+    if all(v is None for v in values):
+        return None
+    return sum(v or 0 for v in values)
+
 # Default max duration before a job is considered stuck (1 hour)
 DEFAULT_STUCK_JOB_THRESHOLD_SECONDS = 3600
 
@@ -373,18 +382,10 @@ def _clean_name(raw):
                 "total": len(user_results),
                 "passed": sum(1 for r in user_results if r.end_status == "OK"),
                 "failed": sum(1 for r in user_results if r.end_status == "FAIL"),
-                "rows_processed": sum(
-                    getattr(r, "rows_affected", 0) or 0 for r in user_results
-                ) or None,
-                "rows_added": sum(
-                    getattr(r, "rows_inserted", 0) or 0 for r in user_results
-                ) or None,
-                "rows_modified": sum(
-                    getattr(r, "rows_updated", 0) or 0 for r in user_results
-                ) or None,
-                "rows_deleted": sum(
-                    getattr(r, "rows_deleted", 0) or 0 for r in user_results
-                ) or None,
+                "rows_processed": _sum_or_none(user_results, "rows_affected"),
+                "rows_added": _sum_or_none(user_results, "rows_inserted"),
+                "rows_modified": _sum_or_none(user_results, "rows_updated"),
+                "rows_deleted": _sum_or_none(user_results, "rows_deleted"),
             }
         except Exception:
             _clear_base_result()

frontend/src/ide/run-history/Runhistory.jsx

@@ -487,12 +487,15 @@ const Runhistory = () => {
           const nonEph = (r.result?.models || []).filter(
             (m) => m.type !== "ephemeral"
           );
-          const added =
-            nonEph.reduce((s, m) => s + (m.rows_inserted || 0), 0) || null;
-          const modified =
-            nonEph.reduce((s, m) => s + (m.rows_updated || 0), 0) || null;
-          const deleted =
-            nonEph.reduce((s, m) => s + (m.rows_deleted || 0), 0) || null;
+          const added = nonEph.some((m) => m.rows_inserted != null)
+            ? nonEph.reduce((s, m) => s + (m.rows_inserted || 0), 0)
+            : null;
+          const modified = nonEph.some((m) => m.rows_updated != null)
+            ? nonEph.reduce((s, m) => s + (m.rows_updated || 0), 0)
+            : null;
+          const deleted = nonEph.some((m) => m.rows_deleted != null)
+            ? nonEph.reduce((s, m) => s + (m.rows_deleted || 0), 0)
+            : null;
           if (added === null && modified === null && deleted === null)
             return <Text type="secondary">—</Text>;
           return (
@@ -636,14 +639,18 @@ const Runhistory = () => {
     const errorModelName = failedModels.length > 0 ? failedModels[0] : null;
 
     // Aggregate row-level changes — exclude ephemeral
-    const totalRowsProcessed =
-      models.reduce((sum, m) => sum + (m.rows_affected || 0), 0) || null;
-    const totalAdded =
-      models.reduce((sum, m) => sum + (m.rows_inserted || 0), 0) || null;
-    const totalModified =
-      models.reduce((sum, m) => sum + (m.rows_updated || 0), 0) || null;
-    const totalDeleted =
-      models.reduce((sum, m) => sum + (m.rows_deleted || 0), 0) || null;
+    const totalRowsProcessed = models.some((m) => m.rows_affected != null)
+      ? models.reduce((sum, m) => sum + (m.rows_affected || 0), 0)
+      : null;
+    const totalAdded = models.some((m) => m.rows_inserted != null)
+      ? models.reduce((sum, m) => sum + (m.rows_inserted || 0), 0)
+      : null;
+    const totalModified = models.some((m) => m.rows_updated != null)
+      ? models.reduce((sum, m) => sum + (m.rows_updated || 0), 0)
+      : null;
+    const totalDeleted = models.some((m) => m.rows_deleted != null)
+      ? models.reduce((sum, m) => sum + (m.rows_deleted || 0), 0)
+      : null;
 
     // Parse error into message + stack
     const errorLines = (run.error_message || "").split("\n");