@wrongstack/webui-server 0.292.1 → 0.295.0

package/dist/server/handlers.js

@@ -1,3 +1,16 @@
+// src/server/handlers/worklist-handlers.ts
+import {
+  addPlanItem,
+  emptyPlan,
+  getPlanTemplate,
+  loadPlan,
+  loadTasks,
+  mutatePlan,
+  mutateTasks,
+  savePlan,
+  setPlanItemStatus
+} from "@wrongstack/core/storage";
+
 // src/server/ws-payload-validation.ts
 import { FORBIDDEN_PROTO_KEYS } from "@wrongstack/core/utils";
 function isRecord(value) {
@@ -18,185 +31,202 @@ function validatePlanTemplateUsePayload(payload) {
 }
 
 // src/server/handlers/worklist-handlers.ts
-function sendResult(ws, ctx, ok, message) {
-  ctx.send(ws, { type: ok ? "ok" : "error", message });
+function sendResult(ctx, ws, success, message) {
+  ctx.send(ws, { type: "key.operation_result", payload: { success, message } });
+}
+function currentSessionId(ctx) {
+  return ctx.context.session?.id ?? "";
 }
 function sessionPayload(ctx, payload) {
   const provided = payload["sessionId"];
-  const fallback = ctx.context.session?.id ?? "";
-  const sessionId = typeof provided === "string" && provided.length > 0 ? provided : fallback;
+  const sessionId = typeof provided === "string" && provided.length > 0 ? provided : currentSessionId(ctx);
   return { ...payload, sessionId };
 }
+function planPathOf(ctx) {
+  const value = ctx.context.meta["plan.path"];
+  return typeof value === "string" && value ? value : void 0;
+}
+function taskPathOf(ctx) {
+  const value = ctx.context.meta["task.path"];
+  return typeof value === "string" && value ? value : void 0;
+}
 function handleTodosGet(ctx, ws) {
-  ctx.send(ws, { type: "todos.updated", payload: sessionPayload(ctx, { todos: ctx.context.todos }) });
+  ctx.send(ws, {
+    type: "todos.updated",
+    payload: sessionPayload(ctx, { todos: [...ctx.context.todos] })
+  });
 }
 function handleTodosClear(ctx, ws) {
   ctx.replaceTodos?.([]);
-  ctx.broadcast({ type: "todos.cleared", payload: sessionPayload(ctx, {}) });
-  sendResult(ws, ctx, true, "Todo board cleared.");
+  sendResult(ctx, ws, true, "Todos cleared");
+  ctx.broadcast({ type: "todos.updated", payload: sessionPayload(ctx, { todos: [] }) });
 }
 function handleTodosRemove(ctx, ws, payload) {
-  if (!payload || payload.id === void 0 && payload.index === void 0) {
-    sendResult(ws, ctx, false, "todos.remove requires id or index.");
+  if (!payload) {
+    sendResult(ctx, ws, false, "Missing id or index");
     return;
   }
-  const next = payload.id !== void 0 ? ctx.context.todos.filter((t) => t.id !== payload.id) : ctx.context.todos.filter((_, i) => i !== payload.index);
+  const todos = ctx.context.todos;
+  let targetIndex = -1;
+  if (typeof payload.id === "string") {
+    targetIndex = todos.findIndex((todo) => todo.id === payload.id);
+  } else if (typeof payload.index === "number" && payload.index > 0) {
+    targetIndex = payload.index - 1;
+  }
+  const removed = todos[targetIndex];
+  if (targetIndex < 0 || !removed) {
+    sendResult(ctx, ws, false, "Todo not found");
+    return;
+  }
+  const next = [...todos.slice(0, targetIndex), ...todos.slice(targetIndex + 1)];
   ctx.replaceTodos?.(next);
+  sendResult(ctx, ws, true, `Removed: ${removed.content}`);
   ctx.broadcast({ type: "todos.updated", payload: sessionPayload(ctx, { todos: next }) });
-  sendResult(ws, ctx, true, "Todo item removed.");
 }
 function handleTodoUpdate(ctx, ws, payload) {
-  const todo = ctx.context.todos.find((t) => t.id === payload.id);
-  if (!todo) {
-    sendResult(ws, ctx, false, `No todo with id "${payload.id}".`);
+  const index = ctx.context.todos.findIndex((todo) => todo.id === payload.id);
+  const existing = ctx.context.todos[index];
+  if (index === -1 || !existing) {
+    sendResult(ctx, ws, false, "Todo not found");
     return;
   }
-  const next = ctx.context.todos.map(
-    (t) => t.id === payload.id ? { ...t, ...payload.status !== void 0 && { status: payload.status }, ...payload.activeForm !== void 0 && { activeForm: payload.activeForm } } : t
-  );
+  const next = [...ctx.context.todos];
+  next[index] = {
+    ...existing,
+    status: payload.status ?? existing.status,
+    activeForm: payload.activeForm !== void 0 ? payload.activeForm : existing.activeForm
+  };
   ctx.replaceTodos?.(next);
+  sendResult(ctx, ws, true, `Todo "${existing.content}" updated`);
   ctx.broadcast({ type: "todos.updated", payload: sessionPayload(ctx, { todos: next }) });
-  sendResult(ws, ctx, true, `Todo "${todo.content}" updated.`);
 }
 async function handleTasksGet(ctx, ws) {
-  const taskPath = ctx.context.meta["task.path"];
-  if (typeof taskPath === "string" && taskPath) {
-    try {
-      const { loadTasks } = await import("@wrongstack/core");
-      const file = await loadTasks(taskPath);
-      ctx.send(ws, { type: "tasks.updated", payload: sessionPayload(ctx, { tasks: file?.tasks ?? [] }) });
-    } catch {
-      ctx.send(ws, { type: "tasks.updated", payload: sessionPayload(ctx, { tasks: [] }) });
-    }
-  } else {
+  const taskPath = taskPathOf(ctx);
+  if (!taskPath) {
     ctx.send(ws, {
       type: "tasks.updated",
       payload: sessionPayload(ctx, { tasks: [], error: "Task storage not configured." })
     });
+    return;
+  }
+  try {
+    const file = await loadTasks(taskPath);
+    ctx.send(ws, {
+      type: "tasks.updated",
+      payload: sessionPayload(ctx, { tasks: file?.tasks ?? [] })
+    });
+  } catch {
+    ctx.send(ws, { type: "tasks.updated", payload: sessionPayload(ctx, { tasks: [] }) });
   }
 }
 async function handleTaskUpdate(ctx, ws, payload) {
-  const taskPath = ctx.context.meta["task.path"];
-  if (typeof taskPath !== "string" || !taskPath) {
-    sendResult(ws, ctx, false, "Task storage is not configured for this session.");
+  const taskPath = taskPathOf(ctx);
+  if (!taskPath) {
+    sendResult(ctx, ws, false, "Task storage not configured.");
     return;
   }
   try {
-    const { loadTasks, saveTasks } = await import("@wrongstack/core");
-    const file = await loadTasks(taskPath);
-    if (!file) {
-      sendResult(ws, ctx, false, "No task file found.");
-      return;
-    }
-    const idx = file.tasks.findIndex((t) => t.id === payload.id);
-    if (idx === -1) {
-      sendResult(ws, ctx, false, `Task "${payload.id}" not found.`);
-      return;
-    }
-    const existing = file.tasks[idx];
-    if (!existing) {
-      sendResult(ws, ctx, false, `Task "${payload.id}" not found.`);
-      return;
-    }
-    file.tasks[idx] = { ...existing, status: payload.status };
-    await saveTasks(taskPath, file);
-    ctx.broadcast({ type: "tasks.updated", payload: sessionPayload(ctx, { tasks: file.tasks }) });
-    sendResult(ws, ctx, true, `Task "${payload.id}" marked ${payload.status}.`);
-  } catch (err) {
-    sendResult(ws, ctx, false, String(err));
+    const file = await mutateTasks(taskPath, currentSessionId(ctx), async (tasks) => {
+      const task = tasks.tasks.find((candidate) => candidate.id === payload.id);
+      if (!task) return tasks;
+      task.status = payload.status;
+      task.updatedAt = (/* @__PURE__ */ new Date()).toISOString();
+      return tasks;
+    });
+    sendResult(ctx, ws, true, `Task status updated to "${payload.status}".`);
+    ctx.broadcast({
+      type: "tasks.updated",
+      payload: sessionPayload(ctx, { tasks: file.tasks })
+    });
+  } catch (error) {
+    sendResult(ctx, ws, false, error instanceof Error ? error.message : String(error));
   }
 }
 async function handlePlanGet(ctx, ws) {
-  const planPath = ctx.context.meta["plan.path"];
-  const sessionId = ctx.context.session?.id ?? "";
-  if (typeof planPath === "string" && planPath) {
-    try {
-      const { loadPlan } = await import("@wrongstack/core");
-      const plan = await loadPlan(planPath);
-      ctx.send(ws, {
-        type: "plan.updated",
-        payload: sessionPayload(ctx, {
-          plan: plan ?? {
-            version: 1,
-            sessionId,
-            updatedAt: (/* @__PURE__ */ new Date()).toISOString(),
-            items: []
-          }
-        })
-      });
-    } catch {
-      ctx.send(ws, {
-        type: "plan.updated",
-        payload: sessionPayload(ctx, {
-          plan: {
-            version: 1,
-            sessionId,
-            updatedAt: (/* @__PURE__ */ new Date()).toISOString(),
-            items: []
-          }
-        })
-      });
-    }
-  } else {
+  const planPath = planPathOf(ctx);
+  const emptySnapshot = () => ({
+    version: 1,
+    sessionId: currentSessionId(ctx),
+    updatedAt: (/* @__PURE__ */ new Date()).toISOString(),
+    items: []
+  });
+  if (!planPath) {
+    ctx.send(ws, {
+      type: "plan.updated",
+      payload: sessionPayload(ctx, {
+        plan: null,
+        error: "Plan storage is not configured for this session."
+      })
+    });
+    return;
+  }
+  try {
+    const plan = await loadPlan(planPath);
     ctx.send(ws, {
       type: "plan.updated",
-      payload: sessionPayload(ctx, { plan: null, error: "Plan storage is not configured for this session." })
+      payload: sessionPayload(ctx, { plan: plan ?? emptySnapshot() })
+    });
+  } catch {
+    ctx.send(ws, {
+      type: "plan.updated",
+      payload: sessionPayload(ctx, { plan: emptySnapshot() })
     });
   }
 }
 async function handlePlanTemplateUse(ctx, ws, template) {
-  const planPath = ctx.context.meta["plan.path"];
-  const sessionId = ctx.context.session?.id ?? "";
-  if (typeof planPath !== "string" || !planPath) {
-    sendResult(ws, ctx, false, "Plan storage is not configured for this session.");
+  const planPath = planPathOf(ctx);
+  if (!planPath) {
+    sendResult(ctx, ws, false, "Plan storage is not configured for this session.");
     return;
   }
   try {
-    const { getPlanTemplate, loadPlan, savePlan, emptyPlan, addPlanItem } = await import("@wrongstack/core");
-    const tpl = getPlanTemplate(template);
-    if (!tpl) {
-      sendResult(ws, ctx, false, `Unknown template "${template}".`);
+    const templateDefinition = getPlanTemplate(template);
+    if (!templateDefinition) {
+      sendResult(ctx, ws, false, `Unknown template "${template}".`);
       return;
     }
-    let plan = await loadPlan(planPath) ?? emptyPlan(sessionId);
-    for (const item of tpl.items) {
+    let plan = await loadPlan(planPath) ?? emptyPlan(currentSessionId(ctx));
+    for (const item of templateDefinition.items) {
       ({ plan } = addPlanItem(plan, item.title, item.details));
     }
     await savePlan(planPath, plan);
-    sendResult(ws, ctx, true, `Applied template "${tpl.name}" \u2014 ${tpl.items.length} items added.`);
+    sendResult(
+      ctx,
+      ws,
+      true,
+      `Applied template "${templateDefinition.name}" \u2014 ${templateDefinition.items.length} items added.`
+    );
     ctx.broadcast({ type: "plan.updated", payload: sessionPayload(ctx, { plan }) });
-  } catch (err) {
-    sendResult(ws, ctx, false, String(err));
+  } catch (error) {
+    sendResult(ctx, ws, false, error instanceof Error ? error.message : String(error));
   }
 }
 async function handlePlanItemUpdate(ctx, ws, payload) {
-  const planPath = ctx.context.meta["plan.path"];
-  const sessionId = ctx.context.session?.id ?? "";
-  if (typeof planPath !== "string" || !planPath) {
-    sendResult(ws, ctx, false, "Plan storage is not configured for this session.");
+  const planPath = planPathOf(ctx);
+  if (!planPath) {
+    sendResult(ctx, ws, false, "Plan storage is not configured for this session.");
     return;
   }
   try {
-    const { mutatePlan, setPlanItemStatus } = await import("@wrongstack/core");
     let changed = false;
-    const plan = await mutatePlan(planPath, sessionId, async (p) => {
-      const before = p.updatedAt;
-      const updated = setPlanItemStatus(p, payload.target, payload.status);
-      changed = updated.updatedAt !== before;
-      return updated;
+    const plan = await mutatePlan(planPath, currentSessionId(ctx), async (currentPlan) => {
+      const before = currentPlan.updatedAt;
+      const next = setPlanItemStatus(currentPlan, payload.target, payload.status);
+      changed = next.updatedAt !== before;
+      return next;
     });
     if (!changed) {
-      sendResult(ws, ctx, false, `No plan item matched "${payload.target}".`);
+      sendResult(ctx, ws, false, `No plan item matched "${payload.target}".`);
       return;
     }
-    sendResult(ws, ctx, true, `Plan item status updated to "${payload.status}".`);
+    sendResult(ctx, ws, true, `Plan item status updated to "${payload.status}".`);
     ctx.broadcast({ type: "plan.updated", payload: sessionPayload(ctx, { plan }) });
-  } catch (err) {
-    sendResult(ws, ctx, false, String(err));
+  } catch (error) {
+    sendResult(ctx, ws, false, error instanceof Error ? error.message : String(error));
   }
 }
-async function handleWorklistMessage(ctx, ws, msg) {
-  switch (msg.type) {
+async function handleWorklistMessage(ctx, ws, message) {
+  switch (message.type) {
     case "todos.get":
       handleTodosGet(ctx, ws);
       return;
@@ -204,13 +234,13 @@ async function handleWorklistMessage(ctx, ws, msg) {
       handleTodosClear(ctx, ws);
       return;
     case "todos.remove":
-      handleTodosRemove(ctx, ws, msg.payload);
+      handleTodosRemove(ctx, ws, message.payload);
       return;
     case "todo.update":
       handleTodoUpdate(
         ctx,
         ws,
-        msg.payload
+        message.payload
       );
       return;
     case "tasks.get":
@@ -220,16 +250,16 @@ async function handleWorklistMessage(ctx, ws, msg) {
       await handleTaskUpdate(
         ctx,
         ws,
-        msg.payload
+        message.payload
       );
       return;
     case "plan.get":
       await handlePlanGet(ctx, ws);
       return;
     case "plan.template_use": {
-      const parsed = validatePlanTemplateUsePayload(msg.payload);
+      const parsed = validatePlanTemplateUsePayload(message.payload);
       if (!parsed.ok) {
-        sendResult(ws, ctx, false, parsed.message);
+        sendResult(ctx, ws, false, parsed.message);
         return;
       }
       await handlePlanTemplateUse(ctx, ws, parsed.value.template);
@@ -239,9 +269,8 @@ async function handleWorklistMessage(ctx, ws, msg) {
       await handlePlanItemUpdate(
         ctx,
         ws,
-        msg.payload
+        message.payload
       );
-      return;
   }
 }
 export {