@agent-plan/server 0.2.19-next.0 → 0.2.19-next.10

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;AA8tBD,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,CA6EvE"}
+{"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;AAqyBD,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

@@ -344,6 +344,8 @@ function createApiApp(store, hubRef, apiPrefix = "", uiConfig, isBusy) {
                 updatedAt: now,
                 handoff: "",
                 handoffUpdatedAt: "",
+                handoffReadAt: "",
+                handoffHistory: [],
             };
             await store.savePhase(phase);
             // Link to feature if featureId provided
@@ -642,45 +644,120 @@ 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") {
-            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;
-                }
-                doc.features.sort((a, b) => a.priority - b.priority || a.number - b.number);
-                return doc;
+            // 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) => {
+                    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;
+                });
             });
             hub()?.broadcast({ type: "features-updated", data: { action: "reordered" } });
         }
         else if (kind === "phase") {
             const phases = await store.loadAllPhases();
-            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; });
+            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 () => {
+                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; });
+                    }
+                }
+            });
             hub()?.broadcast({ type: "phases-updated", data: { action: "reordered" } });
         }
         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.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;
-                    }
-                    phase.tasks.sort((a, b) => a.priority - b.priority || a.number - b.number);
-                    phase.taskIds = phase.tasks.map((t) => t.id);
-                    return phase;
+                await store.runBatch(async () => {
+                    await store.updatePhase(host.id, (phase) => {
+                        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);
+                        return phase;
+                    });
                 });
                 hub()?.broadcast({ type: "phases-updated", data: { action: "reordered", phaseId: host.id, featureId: host.featureId } });
             }
@@ -690,7 +767,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) => {
@@ -814,6 +891,11 @@ export async function serve(options) {
     }
     // Self-heal stale/legacy persisted statuses before the UI reads them.
     await store.syncStatuses();
+    // Backfill missing shortIds (globally-unique 5-char) + priorities so the
+    // Web UI always shows short IDs even before the adapter's first-turn init
+    // runs. Idempotent: only assigns MISSING shortIds/priorities, never
+    // overwrites existing ones.
+    await store.ensureShortIdsAndPriority().catch(() => null);
     // Shared mutable reference — routes see the hub after it's created
     const hubRef = { current: null };
     // Default to the bundled web UI when the caller doesn't pass staticDir. Pass

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agent-plan/server",
-  "version": "0.2.19-next.0",
+  "version": "0.2.19-next.10",
   "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.0"
+    "@agent-plan/core": "0.2.19-next.10"
   },
   "devDependencies": {
     "@types/ws": "^8.18.0"