fix: address Greptile review — compute next_run_time + memoize STATUS_META

P1: next_run_time was exposed in the API but never written, so the
    "Next Run" column was always empty. Added _compute_next_run_time
    that uses celery's remaining_estimate on the PeriodicTask's
    schedule to derive the next run from the last run time. Falls back
    to task.next_run_time if it happens to be set directly.

P2: STATUS_META was rebuilt on every render inside the component body.
    Wrapped in useMemo with [token] as the dependency so it only
    recomputes when the antd theme changes.

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

Author: abhizipstack

backend/backend/core/scheduler/views.py

@@ -20,6 +20,20 @@
 logger = logging.getLogger(__name__)
 
 
+
+def _compute_next_run_time(periodic, last_run_at):
+    """Derive the next run time from a PeriodicTask's schedule."""
+    if not periodic or not periodic.enabled:
+        return None
+    try:
+        schedule = periodic.schedule
+        reference = last_run_at or periodic.last_run_at or timezone.now()
+        remaining = schedule.remaining_estimate(reference)
+        return reference + remaining
+    except Exception:
+        return None
+
+
 def _is_valid_project_id(project_id):
     """Check if project_id is a real UUID (not a placeholder like '_all' or 'all')."""
     try:
@@ -164,7 +178,9 @@ def _serialize_task(task):
         "task_status": task.status,
         "task_run_time": task.task_run_time,
         "task_completion_time": task.task_completion_time,
-        "next_run_time": task.next_run_time,
+        "next_run_time": task.next_run_time or _compute_next_run_time(
+            periodic, task.task_run_time
+        ),
         "task_type": task_type,
         "description": task.description,
         "environment": {

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

@@ -312,44 +312,47 @@ const Runhistory = () => {
     return { scope, models };
   };
 
-  const STATUS_META = {
-    SUCCESS: {
-      icon: <CheckCircleFilled />,
-      label: "Succeeded",
-      color: token.colorSuccess,
-      bg: token.colorSuccessBg,
-    },
-    FAILURE: {
-      icon: <CloseCircleFilled />,
-      label: "Failed",
-      color: token.colorError,
-      bg: token.colorErrorBg,
-    },
-    STARTED: {
-      icon: <SyncOutlined spin />,
-      label: "Running",
-      color: token.colorInfo,
-      bg: token.colorInfoBg,
-    },
-    RETRY: {
-      icon: <SyncOutlined spin />,
-      label: "Retrying",
-      color: token.colorWarning,
-      bg: token.colorWarningBg,
-    },
-    REVOKED: {
-      icon: <ClockCircleOutlined />,
-      label: "Revoked",
-      color: token.colorTextSecondary,
-      bg: token.colorFillQuaternary,
-    },
-    PENDING: {
-      icon: <ClockCircleOutlined />,
-      label: "Pending",
-      color: token.colorTextSecondary,
-      bg: token.colorFillQuaternary,
-    },
-  };
+  const STATUS_META = useMemo(
+    () => ({
+      SUCCESS: {
+        icon: <CheckCircleFilled />,
+        label: "Succeeded",
+        color: token.colorSuccess,
+        bg: token.colorSuccessBg,
+      },
+      FAILURE: {
+        icon: <CloseCircleFilled />,
+        label: "Failed",
+        color: token.colorError,
+        bg: token.colorErrorBg,
+      },
+      STARTED: {
+        icon: <SyncOutlined spin />,
+        label: "Running",
+        color: token.colorInfo,
+        bg: token.colorInfoBg,
+      },
+      RETRY: {
+        icon: <SyncOutlined spin />,
+        label: "Retrying",
+        color: token.colorWarning,
+        bg: token.colorWarningBg,
+      },
+      REVOKED: {
+        icon: <ClockCircleOutlined />,
+        label: "Revoked",
+        color: token.colorTextSecondary,
+        bg: token.colorFillQuaternary,
+      },
+      PENDING: {
+        icon: <ClockCircleOutlined />,
+        label: "Pending",
+        color: token.colorTextSecondary,
+        bg: token.colorFillQuaternary,
+      },
+    }),
+    [token]
+  );
 
   /* ─── empty text ─── */
   const emptyDescription = useMemo(() => {