fix: address reviewer feedback — next_run_time, null guards, stale closure

P1: _compute_next_run_time added remaining to reference (last_run_at)
    instead of now, giving wrong results for interval jobs. Fixed to
    use timezone.now() + remaining. Added logger.debug on failure.

P1: getTooltipText crashed when a job had no linked periodic task.
    Guarded with optional chaining on periodic_task_details.

P2: Auto-expand useEffect used JobHistoryData as dep, resetting
    user-expanded rows on every filter change. Switched to backUpData
    so it only fires on fresh data loads.

P2: handleJobChange useCallback was missing getRunHistoryList in its
    dependency array, risking stale closure. Added to deps.

Nit: Added RUNNING key to STATUS_META so backends that send RUNNING
     instead of STARTED render correctly.

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

Author: abhizipstack

backend/backend/core/scheduler/views.py

@@ -29,8 +29,9 @@ def _compute_next_run_time(periodic, last_run_at):
         schedule = periodic.schedule
         reference = last_run_at or periodic.last_run_at or timezone.now()
         remaining = schedule.remaining_estimate(reference)
-        return reference + remaining
+        return timezone.now() + remaining
     except Exception:
+        logger.debug("Failed to compute next_run_time for %s", periodic, exc_info=True)
         return None
 
 

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

@@ -140,10 +140,13 @@ const Runhistory = () => {
       const { page_items } = res.data.data;
       const scheduledObj = {};
       const jobIds = page_items.map((el) => {
-        scheduledObj[el.user_task_id] = getTooltipText(
-          el.periodic_task_details[el.task_type],
-          el.task_type
-        );
+        const taskDetails = el.periodic_task_details?.[el.task_type];
+        if (taskDetails) {
+          scheduledObj[el.user_task_id] = getTooltipText(
+            taskDetails,
+            el.task_type
+          );
+        }
         return { label: el.task_name, value: el.user_task_id };
       });
       setJobSchedule(scheduledObj);
@@ -180,13 +183,13 @@ const Runhistory = () => {
     }
   }, [filterQueries.status, backUpData]);
 
-  /* ─── auto-expand failed rows ─── */
+  /* ─── auto-expand failed rows on fresh data load (not on filter changes) ─── */
   useEffect(() => {
-    const failedIds = (JobHistoryData || [])
+    const failedIds = (backUpData || [])
       .filter((r) => r.status === "FAILURE" && r.error_message)
       .map((r) => r.id);
     setExpandedRowKeys(failedIds);
-  }, [JobHistoryData]);
+  }, [backUpData]);
 
   /* ─── handlers ─── */
   const handleJobChange = useCallback(
@@ -203,7 +206,7 @@ const Runhistory = () => {
         { replace: true }
       );
     },
-    [setSearchParams]
+    [setSearchParams, getRunHistoryList]
   );
 
   const handleStatusChange = useCallback((value) => {
@@ -332,6 +335,12 @@ const Runhistory = () => {
         color: token.colorInfo,
         bg: token.colorInfoBg,
       },
+      RUNNING: {
+        icon: <SyncOutlined spin />,
+        label: "Running",
+        color: token.colorInfo,
+        bg: token.colorInfoBg,
+      },
       RETRY: {
         icon: <SyncOutlined spin />,
         label: "Retrying",