fix: address reviewer feedback — null config, auto-expand, dedup helper

P1: enabled_model_count crashed with AttributeError when a model in
    model_configs had a null/non-dict value. Added isinstance(m_cfg,
    dict) guard before calling .get().

P2: FAILURE rows no longer auto-expanded after the earlier "collapse
    by default" change. Restored auto-expand for FAILURE rows only,
    keyed on backUpData (fresh loads) so filter changes don't reset
    user-expanded rows.

P2: Replaced the local formatRelativeTime in no-code-model.jsx with
    the shared getRelativeTime from common/helpers.js which already
    handles both past and future dates.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: abhizipstack

backend/backend/core/scheduler/views.py

@@ -778,7 +778,7 @@ def list_deploy_candidates(request, project_id, model_name):
         enabled_model_count = sum(
             1
             for m_cfg in model_configs.values()
-            if m_cfg.get("enabled", True)
+            if isinstance(m_cfg, dict) and m_cfg.get("enabled", True)
         )
         env = task.environment
         candidates.append({

frontend/src/ide/editor/no-code-model/no-code-model.jsx

@@ -90,6 +90,7 @@ import {
   addIdToObjects,
   checkPermission,
   getBaseUrl,
+  getRelativeTime,
   removeIdFromObjects,
   removeUnwantedKeys,
 } from "../../../common/helpers.js";
@@ -258,7 +259,6 @@ function NoCodeModel({ nodeData }) {
     runTask,
     listRecentRunsForModel,
   } = useJobService();
-  const { token } = theme.useToken();
 
   const [quickDeployModal, setQuickDeployModal] = useState({
     open: false,
@@ -1672,19 +1672,6 @@ function NoCodeModel({ nodeData }) {
     RETRY: "orange",
   };
 
-  const formatRelativeTime = (iso) => {
-    if (!iso) return "—";
-    const diff = Date.now() - new Date(iso).getTime();
-    const mins = Math.floor(diff / 60000);
-    if (mins < 1) return "just now";
-    if (mins < 60) return `${mins} min ago`;
-    const hrs = Math.floor(mins / 60);
-    if (hrs < 24) return `${hrs} hr ago`;
-    const days = Math.floor(hrs / 24);
-    if (days < 7) return `${days} day${days > 1 ? "s" : ""} ago`;
-    return new Date(iso).toLocaleDateString();
-  };
-
   const fetchRecentRuns = async () => {
     const name = nodeData?.node?.title;
     if (!name) return;
@@ -1757,7 +1744,7 @@ function NoCodeModel({ nodeData }) {
                 {run.environment_name ? ` · ${run.environment_name}` : ""}
               </Typography.Text>
               <Typography.Text type="secondary" style={{ fontSize: 11 }}>
-                {formatRelativeTime(run.start_time)}
+                {getRelativeTime(run.start_time)}
               </Typography.Text>
             </div>
           );

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

@@ -191,10 +191,13 @@ const Runhistory = () => {
     backUpData,
   ]);
 
-  /* ─── collapse all rows when the underlying data changes ─── */
+  /* ─── auto-expand failed rows on fresh data load ─── */
   useEffect(() => {
-    setExpandedRowKeys([]);
-  }, [JobHistoryData]);
+    const failedIds = (backUpData || [])
+      .filter((r) => r.status === "FAILURE" && r.error_message)
+      .map((r) => r.id);
+    setExpandedRowKeys(failedIds);
+  }, [backUpData]);
 
   /* ─── handlers ─── */
   const handleJobChange = useCallback((value) => {