@agent-plan/server 0.2.19-next.2 → 0.2.19-next.21

package/dist/serve.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"serve.d.ts","sourceRoot":"","sources":["../src/serve.ts"],"names":[],"mappings":"AAWA,OAAO,EAAE,KAAK,EAAE,MAAM,aAAa,CAAC;AAiHpC,MAAM,WAAW,kBAAkB;IACjC,GAAG,EAAE,MAAM,CAAC;IACZ,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,IAAI,CAAC,EAAE,OAAO,CAAC;IACf,IAAI,CAAC,EAAE,OAAO,CAAC;IACf,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,GAAG,CAAC,EAAE,OAAO,CAAC;CACf;AAED,MAAM,WAAW,cAAc;IAC7B,IAAI,EAAE,OAAO,GAAG,KAAK,CAAC;IACtB,QAAQ,EAAE,MAAM,CAAC;IACjB,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;CAC7B;AAED,MAAM,WAAW,QAAQ;IACvB,SAAS,CAAC,EAAE,OAAO,CAAC,MAAM,CAAC,QAAQ,GAAG,MAAM,GAAG,QAAQ,GAAG,QAAQ,EAAE,kBAAkB,CAAC,CAAC,CAAC;IACzF,MAAM,CAAC,EAAE,cAAc,GAAG,SAAS,CAAC;CACrC;AAsuBD,MAAM,WAAW,YAAY;IAC3B,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,MAAM,CAAC;IACjB,IAAI,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC1B,SAAS,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC/B,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,QAAQ,CAAC,EAAE,QAAQ,GAAG,SAAS,CAAC;IAChC,MAAM,CAAC,EAAE,CAAC,MAAM,OAAO,CAAC,GAAG,SAAS,CAAC;CACtC;AAED,MAAM,WAAW,WAAW;IAC1B,GAAG,EAAE,MAAM,CAAC;IACZ,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC5B,IAAI,EAAE,OAAO,GAAG,KAAK,CAAC;IACtB,QAAQ,EAAE,MAAM,CAAC;IACjB,GAAG,EAAE,KAAK,CAAC;IACX,KAAK,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;CAC5B;AAED,wBAAsB,KAAK,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO,CAAC,WAAW,CAAC,CAkFvE"}
+{"version":3,"file":"serve.d.ts","sourceRoot":"","sources":["../src/serve.ts"],"names":[],"mappings":"AAWA,OAAO,EAAE,KAAK,EAAE,MAAM,aAAa,CAAC;AAuIpC,MAAM,WAAW,kBAAkB;IACjC,GAAG,EAAE,MAAM,CAAC;IACZ,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,IAAI,CAAC,EAAE,OAAO,CAAC;IACf,IAAI,CAAC,EAAE,OAAO,CAAC;IACf,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,GAAG,CAAC,EAAE,OAAO,CAAC;CACf;AAED,MAAM,WAAW,cAAc;IAC7B,IAAI,EAAE,OAAO,GAAG,KAAK,CAAC;IACtB,QAAQ,EAAE,MAAM,CAAC;IACjB,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;CAC7B;AAED,MAAM,WAAW,QAAQ;IACvB,SAAS,CAAC,EAAE,OAAO,CAAC,MAAM,CAAC,QAAQ,GAAG,MAAM,GAAG,QAAQ,GAAG,QAAQ,EAAE,kBAAkB,CAAC,CAAC,CAAC;IACzF,MAAM,CAAC,EAAE,cAAc,GAAG,SAAS,CAAC;CACrC;AAo0BD,MAAM,WAAW,YAAY;IAC3B,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,MAAM,CAAC;IACjB,IAAI,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC1B,SAAS,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC/B,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,QAAQ,CAAC,EAAE,QAAQ,GAAG,SAAS,CAAC;IAChC,MAAM,CAAC,EAAE,CAAC,MAAM,OAAO,CAAC,GAAG,SAAS,CAAC;CACtC;AAED,MAAM,WAAW,WAAW;IAC1B,GAAG,EAAE,MAAM,CAAC;IACZ,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC5B,IAAI,EAAE,OAAO,GAAG,KAAK,CAAC;IACtB,QAAQ,EAAE,MAAM,CAAC;IACjB,GAAG,EAAE,KAAK,CAAC;IACX,KAAK,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;CAC5B;AAED,wBAAsB,KAAK,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO,CAAC,WAAW,CAAC,CAkFvE"}

package/dist/serve.js

@@ -6,7 +6,7 @@ import { fileURLToPath } from "node:url";
 import { createAdaptorServer } from "@hono/node-server";
 import { Hono } from "hono";
 import { cors } from "hono/cors";
-import { ExportService, PlanStore, PlanStoreError, createFeatureId, createPhaseId, createShortId, createTaskId, normalizeSlug, withFeatureLock, needsMotivation } from "@agent-plan/core";
+import { ExportService, PlanStore, PlanStoreError, createFeatureId, createPhaseId, createChecklistItemId, createShortId, createTaskId, findPhaseByRef, normalizeSlug, withFeatureLock, needsMotivation } from "@agent-plan/core";
 import { WsHub } from "./ws-hub.js";
 // ─── Watcher ────────────────────────────────────────────────────────────
 let watcherAbort = null;
@@ -14,6 +14,22 @@ let watcherHubRef = { current: null };
 function nowISO() {
     return new Date().toISOString();
 }
+/** Resolve requirement phase refs before persistence so requirements never dangle. */
+async function resolveRequirementPhaseIds(store, linkedPhaseIds) {
+    if (!Array.isArray(linkedPhaseIds) || linkedPhaseIds.length === 0) {
+        return { error: "linkedPhaseIds must contain at least one phase" };
+    }
+    const refs = linkedPhaseIds.map((entry) => typeof entry === "string" ? entry.trim() : "");
+    if (refs.some((ref) => !ref)) {
+        return { error: "linkedPhaseIds must contain non-empty phase references" };
+    }
+    const [phases, featuresDocument] = await Promise.all([store.loadAllPhases(), store.loadFeatures()]);
+    const resolved = refs.map((ref) => findPhaseByRef(phases, featuresDocument.features, ref));
+    const missing = resolved.findIndex((phase) => !phase);
+    if (missing !== -1)
+        return { error: `linked phase not found: ${refs[missing]}` };
+    return { linkedPhaseIds: [...new Set(resolved.map((phase) => phase.id))] };
+}
 function nextTaskNumber(phase) {
     const numbers = phase.tasks.map((task) => task.number || 0).filter((n) => Number.isFinite(n));
     return (numbers.length > 0 ? Math.max(...numbers) : 0) + 1;
@@ -168,21 +184,29 @@ function createApiApp(store, hubRef, apiPrefix = "", uiConfig, isBusy) {
     app.get(route("/requirements"), async (c) => c.json(await store.loadRequirements()));
     app.post(route("/requirements"), async (c) => {
         const body = await c.req.json();
-        const reqs = await store.updateRequirements((doc) => { doc.requirements.push(body); return doc; });
+        const links = await resolveRequirementPhaseIds(store, body.linkedPhaseIds);
+        if ("error" in links)
+            return c.json({ error: links.error }, 400);
+        const requirement = { ...body, linkedPhaseIds: links.linkedPhaseIds };
+        const reqs = await store.updateRequirements((doc) => { doc.requirements.push(requirement); return doc; });
         await store.writeGenerated();
         hub()?.broadcast({ type: "requirements-updated", data: reqs });
         hub()?.broadcast({ type: "plan-rendered", data: {} });
-        return c.json(body, 201);
+        return c.json(requirement, 201);
     });
     app.put(route("/requirements/:id"), async (c) => {
         const id = c.req.param("id");
         const body = await c.req.json();
+        const links = await resolveRequirementPhaseIds(store, body.linkedPhaseIds);
+        if ("error" in links)
+            return c.json({ error: links.error }, 400);
+        const requirement = { ...body, linkedPhaseIds: links.linkedPhaseIds };
         let found = false;
         const reqs = await store.updateRequirements((doc) => {
             const idx = doc.requirements.findIndex((r) => r.id === id);
             if (idx === -1)
                 return doc;
-            doc.requirements[idx] = body;
+            doc.requirements[idx] = requirement;
             found = true;
             return doc;
         });
@@ -191,7 +215,22 @@ function createApiApp(store, hubRef, apiPrefix = "", uiConfig, isBusy) {
         await store.writeGenerated();
         hub()?.broadcast({ type: "requirements-updated", data: reqs });
         hub()?.broadcast({ type: "plan-rendered", data: {} });
-        return c.json(body);
+        return c.json(requirement);
+    });
+    app.delete(route("/requirements/:id"), async (c) => {
+        const id = c.req.param("id");
+        let found = false;
+        const reqs = await store.updateRequirements((doc) => {
+            const next = doc.requirements.filter((requirement) => requirement.id !== id);
+            found = next.length !== doc.requirements.length;
+            return found ? { requirements: next } : doc;
+        });
+        if (!found)
+            return c.json({ error: "not found" }, 404);
+        await store.writeGenerated();
+        hub()?.broadcast({ type: "requirements-updated", data: reqs });
+        hub()?.broadcast({ type: "plan-rendered", data: {} });
+        return c.json({ deleted: id });
     });
     // ── Features ─────────────────────────────────────────────────────
     app.get(route("/features"), async (c) => c.json((await store.loadFeatures()).features));
@@ -222,6 +261,7 @@ function createApiApp(store, hubRef, apiPrefix = "", uiConfig, isBusy) {
             workRemaining: "",
             acceptedDecisions: [],
             phaseIds: [], dependsOn: [],
+            statusLog: [],
             createdAt: now,
             updatedAt: now,
         };
@@ -280,7 +320,7 @@ function createApiApp(store, hubRef, apiPrefix = "", uiConfig, isBusy) {
     });
     // ── Phases ───────────────────────────────────────────────────────
     app.get(route("/phases"), async (c) => {
-        const allPhases = await store.loadAllPhases();
+        const allPhases = await store.loadAllPhasesWithRequirements();
         const featureId = c.req.query("featureId");
         if (featureId)
             return c.json(allPhases.filter((p) => p.featureId === featureId));
@@ -344,6 +384,9 @@ function createApiApp(store, hubRef, apiPrefix = "", uiConfig, isBusy) {
                 updatedAt: now,
                 handoff: "",
                 handoffUpdatedAt: "",
+                handoffReadAt: "",
+                handoffHistory: [],
+                statusLog: [],
             };
             await store.savePhase(phase);
             // Link to feature if featureId provided
@@ -371,7 +414,7 @@ function createApiApp(store, hubRef, apiPrefix = "", uiConfig, isBusy) {
         if (!id)
             return c.json({ error: "id required" }, 400);
         try {
-            return c.json(await store.loadPhase(id));
+            return c.json(await store.loadPhaseWithRequirements(id));
         }
         catch {
             return c.json({ error: "phase not found" }, 404);
@@ -444,8 +487,11 @@ function createApiApp(store, hubRef, apiPrefix = "", uiConfig, isBusy) {
         const initialStatus = body.status ?? "planned";
         const shortId = createShortId(await store.assignedShortIds());
         const priority = await store.nextPriority("task", phase.id);
+        const taskId = createTaskId();
+        const checklistItems = (body.checklist ?? []).map((s) => s.trim()).filter((s) => s.length > 0)
+            .map((item, index) => ({ id: createChecklistItemId(taskId, index + 1, item), number: index + 1, title: item, checked: false }));
         const task = {
-            id: createTaskId(),
+            id: taskId,
             phaseId: phase.id,
             number: taskNumber,
             shortId,
@@ -458,7 +504,7 @@ function createApiApp(store, hubRef, apiPrefix = "", uiConfig, isBusy) {
             statusLog: [],
             decisions: [],
             acceptedDecisions: [],
-            checklist: [],
+            checklist: checklistItems,
             subtasks: [],
             dependsOn: [],
             startedAt: initialStatus === "in-progress" || initialStatus === "done" ? now : "",
@@ -642,20 +688,52 @@ function createApiApp(store, hubRef, apiPrefix = "", uiConfig, isBusy) {
     });
     // ── Reorder (priority) ───────────────────────────────────────────
     app.post(route("/reorder"), async (c) => {
+        // Midpoint-insert reorder: only the moved item's priority changes (when
+        // the gap between its new neighbours allows it). When the gap is exhausted
+        // (≤1, e.g. all-zero defaults or tight spacing), the segment is reindexed
+        // with GAP. Keeps priority as int (no schema migration) and minimises the
+        // set of changed items per drag.
         const body = await c.req.json();
-        const { kind, ids } = body;
-        if (!Array.isArray(ids) || ids.length === 0)
-            return c.json({ error: "ids required" }, 400);
-        const gap = 10;
+        const { kind, movedId, beforeId, afterId } = body;
+        if (!movedId)
+            return c.json({ error: "movedId required" }, 400);
+        const GAP = 10;
+        // Reindex a list in-place: reconstruct the desired order (moved removed +
+        // reinserted between beforeId and afterId) then assign (i+1)*GAP.
+        const reindexList = (all) => {
+            const sorted = [...all].sort((a, b) => a.priority - b.priority || a.number - b.number);
+            const moved = sorted.find((x) => x.id === movedId);
+            const withoutMoved = sorted.filter((x) => x.id !== movedId);
+            let insertIdx;
+            if (beforeId) {
+                const bi = withoutMoved.findIndex((x) => x.id === beforeId);
+                insertIdx = bi === -1 ? withoutMoved.length : bi + 1;
+            }
+            else {
+                insertIdx = 0;
+            }
+            if (moved)
+                withoutMoved.splice(insertIdx, 0, moved);
+            withoutMoved.forEach((item, i) => { item.priority = (i + 1) * GAP; });
+        };
         if (kind === "feature") {
             // Priority-only: suspend status rollup so reordering never flips a
             // partially-done feature to in-progress (priority ≠ status).
             await store.runBatch(async () => {
                 await store.updateFeatures((doc) => {
-                    for (const [i, id] of ids.entries()) {
-                        const f = doc.features.find((x) => x.id === id);
-                        if (f)
-                            f.priority = (i + 1) * gap;
+                    const moved = doc.features.find((x) => x.id === movedId);
+                    if (!moved)
+                        return doc;
+                    const before = beforeId ? doc.features.find((x) => x.id === beforeId) : null;
+                    const after = afterId ? doc.features.find((x) => x.id === afterId) : null;
+                    const beforeP = before ? before.priority : 0;
+                    const maxP = doc.features.reduce((m, f) => Math.max(m, f.priority), 0);
+                    const afterP = after ? after.priority : (maxP + GAP);
+                    if (afterP - beforeP > 1) {
+                        moved.priority = Math.floor((beforeP + afterP) / 2);
+                    }
+                    else {
+                        reindexList(doc.features);
                     }
                     doc.features.sort((a, b) => a.priority - b.priority || a.number - b.number);
                     return doc;
@@ -665,10 +743,36 @@ function createApiApp(store, hubRef, apiPrefix = "", uiConfig, isBusy) {
         }
         else if (kind === "phase") {
             const phases = await store.loadAllPhases();
+            const moved = phases.find((p) => p.id === movedId);
+            if (!moved)
+                return c.json({ error: "moved phase not found" }, 404);
+            const siblings = phases.filter((p) => p.featureId === moved.featureId);
+            const before = beforeId ? siblings.find((p) => p.id === beforeId) : null;
+            const after = afterId ? siblings.find((p) => p.id === afterId) : null;
+            const beforeP = before ? before.priority : 0;
+            const maxP = siblings.reduce((m, p) => Math.max(m, p.priority), 0);
+            const afterP = after ? after.priority : (maxP + GAP);
             await store.runBatch(async () => {
-                for (const [i, id] of ids.entries()) {
-                    if (phases.some((x) => x.id === id)) {
-                        await store.updatePhase(id, (entry) => { entry.priority = (i + 1) * gap; return entry; });
+                if (afterP - beforeP > 1) {
+                    await store.updatePhase(movedId, (entry) => { entry.priority = Math.floor((beforeP + afterP) / 2); return entry; });
+                }
+                else {
+                    // Reindex all siblings (each phase is a separate file)
+                    const sorted = [...siblings].sort((a, b) => a.priority - b.priority || a.number - b.number);
+                    const withoutMoved = sorted.filter((x) => x.id !== movedId);
+                    let insertIdx;
+                    if (beforeId) {
+                        const bi = withoutMoved.findIndex((x) => x.id === beforeId);
+                        insertIdx = bi === -1 ? withoutMoved.length : bi + 1;
+                    }
+                    else {
+                        insertIdx = 0;
+                    }
+                    const movedEntry = sorted.find((x) => x.id === movedId);
+                    if (movedEntry)
+                        withoutMoved.splice(insertIdx, 0, movedEntry);
+                    for (const [i, p] of withoutMoved.entries()) {
+                        await store.updatePhase(p.id, (entry) => { entry.priority = (i + 1) * GAP; return entry; });
                     }
                 }
             });
@@ -676,14 +780,23 @@ function createApiApp(store, hubRef, apiPrefix = "", uiConfig, isBusy) {
         }
         else if (kind === "task") {
             const allPhases = await store.loadAllPhases();
-            const host = allPhases.find((p) => p.tasks.some((t) => ids.includes(t.id)));
+            const host = allPhases.find((p) => p.tasks.some((t) => t.id === movedId));
             if (host) {
                 await store.runBatch(async () => {
                     await store.updatePhase(host.id, (phase) => {
-                        for (const [i, id] of ids.entries()) {
-                            const t = phase.tasks.find((x) => x.id === id);
-                            if (t)
-                                t.priority = (i + 1) * gap;
+                        const moved = phase.tasks.find((x) => x.id === movedId);
+                        if (!moved)
+                            return phase;
+                        const before = beforeId ? phase.tasks.find((x) => x.id === beforeId) : null;
+                        const after = afterId ? phase.tasks.find((x) => x.id === afterId) : null;
+                        const beforeP = before ? before.priority : 0;
+                        const maxP = phase.tasks.reduce((m, t) => Math.max(m, t.priority), 0);
+                        const afterP = after ? after.priority : (maxP + GAP);
+                        if (afterP - beforeP > 1) {
+                            moved.priority = Math.floor((beforeP + afterP) / 2);
+                        }
+                        else {
+                            reindexList(phase.tasks);
                         }
                         phase.tasks.sort((a, b) => a.priority - b.priority || a.number - b.number);
                         phase.taskIds = phase.tasks.map((t) => t.id);
@@ -698,7 +811,7 @@ function createApiApp(store, hubRef, apiPrefix = "", uiConfig, isBusy) {
         }
         await store.writeGenerated();
         hub()?.broadcast({ type: "plan-rendered", data: {} });
-        return c.json({ ok: true, kind, count: ids.length });
+        return c.json({ ok: true, kind, movedId });
     });
     // ── Render ───────────────────────────────────────────────────────
     app.post(route("/render"), async (c) => {
@@ -711,6 +824,10 @@ function createApiApp(store, hubRef, apiPrefix = "", uiConfig, isBusy) {
         const list = await store.listHandoffs();
         return c.json({ handoffs: list });
     });
+    app.get(route("/handoffs/archive"), async (c) => {
+        const archived = await store.listArchivedHandoffs();
+        return c.json({ archived });
+    });
     app.get(route("/phases/:id/handoff"), async (c) => {
         const id = c.req.param("id");
         if (!id)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agent-plan/server",
-  "version": "0.2.19-next.2",
+  "version": "0.2.19-next.21",
   "private": false,
   "description": "Local HTTP/WebSocket server for Agent Plan dashboards and planner APIs.",
   "license": "MIT",
@@ -41,7 +41,7 @@
     "@hono/node-server": "^1.14.0",
     "hono": "^4.7.0",
     "ws": "^8.18.0",
-    "@agent-plan/core": "0.2.19-next.2"
+    "@agent-plan/core": "0.2.19-next.21"
   },
   "devDependencies": {
     "@types/ws": "^8.18.0"