@ryanfw/prompt-orchestration-pipeline 0.10.0 → 0.11.0

package/src/ui/client/adapters/job-adapter.js

@@ -75,6 +75,8 @@ function normalizeTasks(rawTasks) {
           : undefined,
         // Preserve tokenUsage if present
         ...(t && t.tokenUsage ? { tokenUsage: t.tokenUsage } : {}),
+        // Preserve error object if present
+        ...(t && t.error ? { error: t.error } : {}),
       };
       tasks[name] = taskObj;
     });
@@ -106,11 +108,28 @@ function normalizeTasks(rawTasks) {
         ...(typeof t?.failedStage === "string" && t.failedStage.length > 0
           ? { failedStage: t.failedStage }
           : {}),
+        // Prefer new files.* schema, fallback to legacy artifacts
+        files:
+          t && t.files
+            ? {
+                artifacts: Array.isArray(t.files.artifacts)
+                  ? t.files.artifacts.slice()
+                  : [],
+                logs: Array.isArray(t.files.logs) ? t.files.logs.slice() : [],
+                tmp: Array.isArray(t.files.tmp) ? t.files.tmp.slice() : [],
+              }
+            : {
+                artifacts: [],
+                logs: [],
+                tmp: [],
+              },
         artifacts: Array.isArray(t && t.artifacts)
           ? t.artifacts.slice()
           : undefined,
         // Preserve tokenUsage if present
         ...(t && t.tokenUsage ? { tokenUsage: t.tokenUsage } : {}),
+        // Preserve error object if present
+        ...(t && t.error ? { error: t.error } : {}),
       };
     });
     return { tasks, warnings };
@@ -152,7 +171,7 @@ export function adaptJobSummary(apiJob) {
   // Demo-only: read canonical fields strictly
   const id = apiJob.jobId;
   const name = apiJob.title || "";
-  const rawTasks = apiJob.tasks;
+  const rawTasks = apiJob.tasks || apiJob.tasksStatus; // Handle both formats for backward compatibility
   const location = apiJob.location;
 
   // Job-level stage metadata
@@ -221,7 +240,7 @@ export function adaptJobDetail(apiDetail) {
   // Demo-only: read canonical fields strictly
   const id = apiDetail.jobId;
   const name = apiDetail.title || "";
-  const rawTasks = apiDetail.tasks;
+  const rawTasks = apiDetail.tasks || apiDetail.tasksStatus; // Handle both formats for backward compatibility
   const location = apiDetail.location;
 
   // Job-level stage metadata

package/src/ui/client/hooks/useJobDetailWithUpdates.js

@@ -347,6 +347,7 @@ export function useJobDetailWithUpdates(jobId) {
               newEs.addEventListener("job:removed", onJobRemoved);
               newEs.addEventListener("status:changed", onStatusChanged);
               newEs.addEventListener("state:change", onStateChange);
+              newEs.addEventListener("task:updated", onTaskUpdated);
               newEs.addEventListener("error", onError);
 
               esRef.current = newEs;
@@ -373,6 +374,94 @@ export function useJobDetailWithUpdates(jobId) {
             return;
           }
 
+          // Handle task:updated events with task-level merge logic
+          if (type === "task:updated") {
+            const p = payload || {};
+            const { jobId: eventJobId, taskId, task } = p;
+
+            // Filter by jobId
+            if (eventJobId && eventJobId !== jobId) {
+              return;
+            }
+
+            // Validate required fields
+            if (!taskId || !task) {
+              return;
+            }
+
+            startTransition(() => {
+              setData((prev) => {
+                // If no previous data or tasks, return unchanged
+                if (!prev || !prev.tasks) {
+                  return prev;
+                }
+
+                const prevTask = prev.tasks[taskId];
+
+                // Compare observable fields to determine if update is needed
+                const fieldsToCompare = [
+                  "state",
+                  "currentStage",
+                  "failedStage",
+                  "startedAt",
+                  "endedAt",
+                  "attempts",
+                  "executionTimeMs",
+                  "error",
+                ];
+
+                let hasChanged = false;
+
+                // If task doesn't exist yet, always treat as changed
+                if (!prevTask) {
+                  hasChanged = true;
+                } else {
+                  // Compare observable fields to determine if update is needed
+                  for (const field of fieldsToCompare) {
+                    if (prevTask[field] !== task[field]) {
+                      hasChanged = true;
+                      break;
+                    }
+                  }
+
+                  // Also compare tokenUsage and files arrays by reference
+                  if (
+                    prevTask.tokenUsage !== task.tokenUsage ||
+                    prevTask.files !== task.files
+                  ) {
+                    hasChanged = true;
+                  }
+                }
+
+                if (!hasChanged) {
+                  return prev; // No change, preserve identity
+                }
+
+                // Create new tasks map with updated task
+                const nextTasks = { ...prev.tasks, [taskId]: task };
+
+                // Recompute job-level summary fields
+                const taskCount = Object.keys(nextTasks).length;
+                const doneCount = Object.values(nextTasks).filter(
+                  (t) => t.state === "done"
+                ).length;
+                const progress =
+                  taskCount > 0 ? (doneCount / taskCount) * 100 : 0;
+
+                // Return new job object with updated tasks and summary
+                return {
+                  ...prev,
+                  tasks: nextTasks,
+                  doneCount,
+                  taskCount,
+                  progress,
+                  lastUpdated: new Date().toISOString(),
+                };
+              });
+            });
+            return; // Skip generic handling for task:updated
+          }
+
           // Path-matching state:change → schedule debounced refetch
           if (type === "state:change") {
             const d = (payload && (payload.data || payload)) || {};
@@ -415,6 +504,7 @@ export function useJobDetailWithUpdates(jobId) {
       const onStatusChanged = (evt) =>
         handleIncomingEvent("status:changed", evt);
       const onStateChange = (evt) => handleIncomingEvent("state:change", evt);
+      const onTaskUpdated = (evt) => handleIncomingEvent("task:updated", evt);
 
       es.addEventListener("open", onOpen);
       es.addEventListener("job:updated", onJobUpdated);
@@ -422,6 +512,7 @@ export function useJobDetailWithUpdates(jobId) {
       es.addEventListener("job:removed", onJobRemoved);
       es.addEventListener("status:changed", onStatusChanged);
       es.addEventListener("state:change", onStateChange);
+      es.addEventListener("task:updated", onTaskUpdated);
       es.addEventListener("error", onError);
 
       // Set connection status from readyState when possible
@@ -439,6 +530,7 @@ export function useJobDetailWithUpdates(jobId) {
           es.removeEventListener("job:removed", onJobRemoved);
           es.removeEventListener("status:changed", onStatusChanged);
           es.removeEventListener("state:change", onStateChange);
+          es.removeEventListener("task:updated", onTaskUpdated);
           es.removeEventListener("error", onError);
           es.close();
         } catch (err) {

package/src/ui/dist/assets/{index-DqkbzXZ1.js → index-DeDzq-Kk.js}

@@ -20023,7 +20023,9 @@ function normalizeTasks(rawTasks) {
         },
         artifacts: Array.isArray(t2 && t2.artifacts) ? t2.artifacts.slice() : void 0,
         // Preserve tokenUsage if present
-        ...t2 && t2.tokenUsage ? { tokenUsage: t2.tokenUsage } : {}
+        ...t2 && t2.tokenUsage ? { tokenUsage: t2.tokenUsage } : {},
+        // Preserve error object if present
+        ...t2 && t2.error ? { error: t2.error } : {}
       };
       tasks[name] = taskObj;
     });
@@ -20045,9 +20047,21 @@ function normalizeTasks(rawTasks) {
         // Preserve stage metadata for DAG visualization
         ...typeof t2?.currentStage === "string" && t2.currentStage.length > 0 ? { currentStage: t2.currentStage } : {},
         ...typeof t2?.failedStage === "string" && t2.failedStage.length > 0 ? { failedStage: t2.failedStage } : {},
+        // Prefer new files.* schema, fallback to legacy artifacts
+        files: t2 && t2.files ? {
+          artifacts: Array.isArray(t2.files.artifacts) ? t2.files.artifacts.slice() : [],
+          logs: Array.isArray(t2.files.logs) ? t2.files.logs.slice() : [],
+          tmp: Array.isArray(t2.files.tmp) ? t2.files.tmp.slice() : []
+        } : {
+          artifacts: [],
+          logs: [],
+          tmp: []
+        },
         artifacts: Array.isArray(t2 && t2.artifacts) ? t2.artifacts.slice() : void 0,
         // Preserve tokenUsage if present
-        ...t2 && t2.tokenUsage ? { tokenUsage: t2.tokenUsage } : {}
+        ...t2 && t2.tokenUsage ? { tokenUsage: t2.tokenUsage } : {},
+        // Preserve error object if present
+        ...t2 && t2.error ? { error: t2.error } : {}
       };
     });
     return { tasks, warnings };
@@ -20068,7 +20082,7 @@ function computeJobSummaryStats(tasks) {
 function adaptJobSummary(apiJob) {
   const id = apiJob.jobId;
   const name = apiJob.title || "";
-  const rawTasks = apiJob.tasks;
+  const rawTasks = apiJob.tasks || apiJob.tasksStatus;
   const location = apiJob.location;
   const current = apiJob.current;
   const currentStage = apiJob.currentStage;
@@ -20112,7 +20126,7 @@ function adaptJobSummary(apiJob) {
 function adaptJobDetail(apiDetail) {
   const id = apiDetail.jobId;
   const name = apiDetail.title || "";
-  const rawTasks = apiDetail.tasks;
+  const rawTasks = apiDetail.tasks || apiDetail.tasksStatus;
   const location = apiDetail.location;
   const current = apiDetail.current;
   const currentStage = apiDetail.currentStage;
@@ -20930,7 +20944,7 @@ function UploadSeed({ onUploadSuccess }) {
             " ",
             "or drag and drop"
           ] }),
-          /* @__PURE__ */ jsxRuntimeExports.jsx("p", { className: "text-xs text-gray-500", children: "JSON files only" })
+          /* @__PURE__ */ jsxRuntimeExports.jsx("p", { className: "text-xs text-gray-500", children: "JSON or zip files only" })
         ] })
       }
     ),
@@ -20939,7 +20953,7 @@ function UploadSeed({ onUploadSuccess }) {
       {
         ref: fileInputRef,
         type: "file",
-        accept: ".json",
+        accept: ".json,.zip",
         className: "hidden",
         onChange: handleFileChange,
         "data-testid": "file-input"
@@ -21706,6 +21720,7 @@ function TaskDetailSidebar({
   jobId,
   taskId,
   taskBody,
+  taskError,
   filesByTypeForItem = () => ({ artifacts: [], logs: [], tmp: [] }),
   task,
   onClose,
@@ -21715,6 +21730,7 @@ function TaskDetailSidebar({
   const [filePaneType, setFilePaneType] = reactExports.useState("artifacts");
   const [filePaneOpen, setFilePaneOpen] = reactExports.useState(false);
   const [filePaneFilename, setFilePaneFilename] = reactExports.useState(null);
+  const [showStack, setShowStack] = reactExports.useState(false);
   const closeButtonRef = reactExports.useRef(null);
   const getHeaderClasses2 = (status2) => {
     switch (status2) {
@@ -21794,7 +21810,29 @@ function TaskDetailSidebar({
           }
         ),
         /* @__PURE__ */ jsxRuntimeExports.jsxs("div", { className: "p-6 space-y-8 overflow-y-auto h-full", children: [
-          status === TaskState.FAILED && taskBody && /* @__PURE__ */ jsxRuntimeExports.jsx("section", { "aria-label": "Error", children: /* @__PURE__ */ jsxRuntimeExports.jsx(n$2, { role: "alert", "aria-live": "assertive", children: /* @__PURE__ */ jsxRuntimeExports.jsx(u$1, { className: "whitespace-pre-wrap break-words", children: taskBody }) }) }),
+          status === TaskState.FAILED && (taskError?.message || taskBody) && /* @__PURE__ */ jsxRuntimeExports.jsxs("section", { "aria-label": "Error", children: [
+            /* @__PURE__ */ jsxRuntimeExports.jsx(n$2, { role: "alert", "aria-live": "assertive", children: /* @__PURE__ */ jsxRuntimeExports.jsx(u$1, { className: "whitespace-pre-wrap break-words", children: taskError?.message || taskBody }) }),
+            taskError?.stack && /* @__PURE__ */ jsxRuntimeExports.jsxs("div", { className: "mt-3", children: [
+              /* @__PURE__ */ jsxRuntimeExports.jsx(
+                "button",
+                {
+                  onClick: () => setShowStack(!showStack),
+                  className: "text-sm text-blue-600 hover:text-blue-800 underline",
+                  "aria-expanded": showStack,
+                  "aria-controls": "error-stack",
+                  children: showStack ? "Hide stack" : "Show stack"
+                }
+              ),
+              showStack && /* @__PURE__ */ jsxRuntimeExports.jsx(
+                "pre",
+                {
+                  id: "error-stack",
+                  className: "mt-2 p-2 bg-gray-50 border rounded text-xs font-mono max-h-64 overflow-auto whitespace-pre-wrap",
+                  children: taskError.stack
+                }
+              )
+            ] })
+          ] }),
           /* @__PURE__ */ jsxRuntimeExports.jsx("section", { className: "mt-6", children: /* @__PURE__ */ jsxRuntimeExports.jsxs("div", { className: "flex items-center justify-between mb-4", children: [
             /* @__PURE__ */ jsxRuntimeExports.jsx("h3", { className: "text-base font-semibold text-gray-900", children: "Files" }),
             /* @__PURE__ */ jsxRuntimeExports.jsx("div", { className: "flex items-center space-x-2", children: /* @__PURE__ */ jsxRuntimeExports.jsxs("div", { className: "flex rounded-lg border border-gray-200 bg-gray-50 p-1", children: [
@@ -22187,6 +22225,9 @@ const getHeaderClasses = (status) => {
 const canShowRestart = (status) => {
   return status === TaskState.FAILED || status === TaskState.DONE;
 };
+const areEqualTaskCardProps = (prevProps, nextProps) => {
+  return prevProps.item === nextProps.item && prevProps.idx === nextProps.idx && prevProps.status === nextProps.status && prevProps.isActive === nextProps.isActive && prevProps.canRestart === nextProps.canRestart && prevProps.isSubmitting === nextProps.isSubmitting && prevProps.disabledReason === nextProps.disabledReason && prevProps.onClick === nextProps.onClick && prevProps.onKeyDown === nextProps.onKeyDown && prevProps.handleRestartClick === nextProps.handleRestartClick;
+};
 const TaskCard = reactExports.memo(function TaskCard2({
   item,
   idx,
@@ -22195,7 +22236,7 @@ const TaskCard = reactExports.memo(function TaskCard2({
   isActive,
   canRestart,
   isSubmitting,
-  getRestartDisabledReason,
+  disabledReason,
   onClick,
   onKeyDown,
   handleRestartClick
@@ -22293,7 +22334,7 @@ const TaskCard = reactExports.memo(function TaskCard2({
               onClick: (e2) => handleRestartClick(e2, item.id),
               disabled: !canRestart || isSubmitting,
               className: "text-xs cursor-pointer disabled:cursor-not-allowed",
-              title: !canRestart ? getRestartDisabledReason() : `Restart job from ${item.id}`,
+              title: !canRestart ? disabledReason : `Restart job from ${item.id}`,
               children: "Restart"
             }
           ) })
@@ -22301,14 +22342,15 @@ const TaskCard = reactExports.memo(function TaskCard2({
       ]
     }
   );
-});
+}, areEqualTaskCardProps);
 function DAGGrid({
   items,
   cols = 3,
   cardClass = "",
   activeIndex = 0,
   jobId,
-  filesByTypeForItem = () => createEmptyTaskFiles()
+  filesByTypeForItem = () => createEmptyTaskFiles(),
+  taskById = {}
 }) {
   const overlayRef = reactExports.useRef(null);
   const gridRef = reactExports.useRef(null);
@@ -22458,7 +22500,7 @@ function DAGGrid({
         cancelAnimationFrame(rafRef.current);
       }
     };
-  }, [items, effectiveCols, visualOrder]);
+  }, [items.length, effectiveCols, visualOrder]);
   const getStatus = (index2) => {
     const item = items[index2];
     const s2 = item?.status;
@@ -22653,6 +22695,7 @@ function DAGGrid({
           const status = getStatus(idx);
           const isActive = idx === activeIndex;
           const canRestart = isRestartEnabled();
+          const restartDisabledReason = getRestartDisabledReason();
           return /* @__PURE__ */ jsxRuntimeExports.jsx(
             TaskCard,
             {
@@ -22662,7 +22705,7 @@ function DAGGrid({
               isActive,
               canRestart,
               isSubmitting,
-              getRestartDisabledReason,
+              disabledReason: restartDisabledReason,
               onClick: () => {
                 setOpenIdx(idx);
               },
@@ -22689,6 +22732,7 @@ function DAGGrid({
         jobId,
         taskId: items[openIdx]?.id || `task-${openIdx}`,
         taskBody: items[openIdx]?.body || null,
+        taskError: taskById[items[openIdx]?.id]?.error || null,
         filesByTypeForItem,
         task: items[openIdx],
         taskIndex: openIdx,
@@ -22874,6 +22918,12 @@ function JobDetail({ job, pipeline }) {
       }
       return item;
     });
+    const allReused = newItems.every(
+      (item, index2) => item === prevItems[index2]
+    );
+    if (allReused && prevItems.length === newItems.length) {
+      return prevItems;
+    }
     prevDagItemsRef.current = newItems;
     return newItems;
   }, [stableDagItems]);
@@ -22894,7 +22944,8 @@ function JobDetail({ job, pipeline }) {
       items: dagItems,
       activeIndex,
       jobId: job.id,
-      filesByTypeForItem
+      filesByTypeForItem,
+      taskById
     }
   ) });
 }
@@ -23153,6 +23204,7 @@ function useJobDetailWithUpdates(jobId) {
               newEs.addEventListener("job:removed", onJobRemoved);
               newEs.addEventListener("status:changed", onStatusChanged);
               newEs.addEventListener("state:change", onStateChange);
+              newEs.addEventListener("task:updated", onTaskUpdated);
               newEs.addEventListener("error", onError);
               esRef.current = newEs;
             } catch (err) {
@@ -23172,6 +23224,66 @@ function useJobDetailWithUpdates(jobId) {
             eventQueue.current = (eventQueue.current || []).concat(eventObj);
             return;
           }
+          if (type === "task:updated") {
+            const p2 = payload || {};
+            const { jobId: eventJobId, taskId, task } = p2;
+            if (eventJobId && eventJobId !== jobId) {
+              return;
+            }
+            if (!taskId || !task) {
+              return;
+            }
+            startTransition(() => {
+              setData((prev) => {
+                if (!prev || !prev.tasks) {
+                  return prev;
+                }
+                const prevTask = prev.tasks[taskId];
+                const fieldsToCompare = [
+                  "state",
+                  "currentStage",
+                  "failedStage",
+                  "startedAt",
+                  "endedAt",
+                  "attempts",
+                  "executionTimeMs",
+                  "error"
+                ];
+                let hasChanged = false;
+                if (!prevTask) {
+                  hasChanged = true;
+                } else {
+                  for (const field of fieldsToCompare) {
+                    if (prevTask[field] !== task[field]) {
+                      hasChanged = true;
+                      break;
+                    }
+                  }
+                  if (prevTask.tokenUsage !== task.tokenUsage || prevTask.files !== task.files) {
+                    hasChanged = true;
+                  }
+                }
+                if (!hasChanged) {
+                  return prev;
+                }
+                const nextTasks = { ...prev.tasks, [taskId]: task };
+                const taskCount = Object.keys(nextTasks).length;
+                const doneCount = Object.values(nextTasks).filter(
+                  (t2) => t2.state === "done"
+                ).length;
+                const progress = taskCount > 0 ? doneCount / taskCount * 100 : 0;
+                return {
+                  ...prev,
+                  tasks: nextTasks,
+                  doneCount,
+                  taskCount,
+                  progress,
+                  lastUpdated: (/* @__PURE__ */ new Date()).toISOString()
+                };
+              });
+            });
+            return;
+          }
           if (type === "state:change") {
             const d2 = payload && (payload.data || payload) || {};
             if (typeof d2.path === "string" && matchesJobTasksStatusPath(d2.path, jobId)) {
@@ -23204,12 +23316,14 @@ function useJobDetailWithUpdates(jobId) {
       const onJobRemoved = (evt) => handleIncomingEvent("job:removed", evt);
       const onStatusChanged = (evt) => handleIncomingEvent("status:changed", evt);
       const onStateChange = (evt) => handleIncomingEvent("state:change", evt);
+      const onTaskUpdated = (evt) => handleIncomingEvent("task:updated", evt);
       es.addEventListener("open", onOpen);
       es.addEventListener("job:updated", onJobUpdated);
       es.addEventListener("job:created", onJobCreated);
       es.addEventListener("job:removed", onJobRemoved);
       es.addEventListener("status:changed", onStatusChanged);
       es.addEventListener("state:change", onStateChange);
+      es.addEventListener("task:updated", onTaskUpdated);
       es.addEventListener("error", onError);
       if (es.readyState === 1 && mountedRef.current) {
         setConnectionStatus("connected");
@@ -23224,6 +23338,7 @@ function useJobDetailWithUpdates(jobId) {
           es.removeEventListener("job:removed", onJobRemoved);
           es.removeEventListener("status:changed", onStatusChanged);
           es.removeEventListener("state:change", onStateChange);
+          es.removeEventListener("task:updated", onTaskUpdated);
           es.removeEventListener("error", onError);
           es.close();
         } catch (err) {