fix: address Greptile review — null env, stale cache, deps, DB filter

P1: list_deploy_candidates crashed with AttributeError when a task had
    no environment. Guard with `if env else ""` (same pattern used in
    list_recent_runs_for_model).

P2: list_deploy_candidates loaded all project tasks and filtered in
    Python. Added model_configs__has_key=model_name to the queryset so
    the DB does the heavy lifting.

P2: Recent-runs popup showed stale data after a Quick Deploy because
    fetchedFor wasn't cleared. Now reset to null after successful
    dispatch so the next dropdown open refetches.

P2: getRunTriggerScope was defined inside the Runhistory component,
    triggering react-hooks/exhaustive-deps warnings and unnecessary
    re-creation. Moved to module scope — it has no dependency on
    component state.

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

Author: abhizipstack

backend/backend/core/scheduler/views.py

@@ -766,28 +766,29 @@ def list_deploy_candidates(request, project_id, model_name):
     """
     tasks = UserTaskDetails.objects.select_related("environment", "project").filter(
         project__project_uuid=project_id,
+        model_configs__has_key=model_name,
     )
 
     candidates = []
     for task in tasks:
         model_configs = task.model_configs or {}
         cfg = model_configs.get(model_name)
-        if not cfg:
-            continue
-        if not cfg.get("enabled", True):
+        if not cfg or not cfg.get("enabled", True):
             continue
         enabled_model_count = sum(
             1
             for m_cfg in model_configs.values()
             if m_cfg.get("enabled", True)
         )
+        env = task.environment
         candidates.append({
             "user_task_id": task.id,
             "task_name": task.task_name,
-            "environment_id": str(task.environment.environment_id),
-            "environment_name": getattr(
-                task.environment, "environment_name", ""
-            ) or getattr(task.environment, "name", ""),
+            "environment_id": str(env.environment_id) if env else "",
+            "environment_name": (
+                getattr(env, "environment_name", "")
+                or getattr(env, "name", "")
+            ) if env else "",
             "status": task.status,
             "prev_run_status": task.prev_run_status,
             "task_run_time": task.task_run_time.isoformat() if task.task_run_time else None,

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

@@ -1803,6 +1803,7 @@ function NoCodeModel({ nodeData }) {
             : `"${currentModelName}" is running on "${envName}" via job "${jobName}". Check Run History for progress.`,
       });
       setRefreshModels(true);
+      setRecentRunsState((prev) => ({ ...prev, fetchedFor: null }));
       setQuickDeployModal((prev) => ({
         ...prev,
         open: false,

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

@@ -61,6 +61,16 @@ const STATUS_OPTIONS = [
   { label: "Revoked", value: "REVOKED" },
 ];
 
+const getRunTriggerScope = (row) => {
+  const kw = row?.kwargs || {};
+  const legacyQuick = kw.source === "quick_deploy";
+  const models = kw.models_override || [];
+  const trigger = kw.trigger || (legacyQuick ? "manual" : "scheduled");
+  const scope =
+    kw.scope || (models.length > 0 || legacyQuick ? "model" : "job");
+  return { trigger, scope, models };
+};
+
 const Runhistory = () => {
   const axios = useAxiosPrivate();
   const {
@@ -83,17 +93,6 @@ const Runhistory = () => {
     scope: "",
   });
 
-  /* Derive trigger/scope from a run row with back-compat for legacy
-   * rows that only carried kwargs.source === "quick_deploy". */
-  const getRunTriggerScope = (row) => {
-    const kw = row?.kwargs || {};
-    const legacyQuick = kw.source === "quick_deploy";
-    const models = kw.models_override || [];
-    const trigger = kw.trigger || (legacyQuick ? "manual" : "scheduled");
-    const scope =
-      kw.scope || (models.length > 0 || legacyQuick ? "model" : "job");
-    return { trigger, scope, models };
-  };
   const [envInfo, setEnvInfo] = useState({
     env_type: "",
     job_name: "",