@agent-plan/pi-adapter 0.2.19-next.12 → 0.2.19-next.14

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,KAAK,EAAE,YAAY,EAAoB,MAAM,iCAAiC,CAAC;AAgvBtF,MAAM,CAAC,OAAO,UAAU,eAAe,CAAC,EAAE,EAAE,YAAY,GAAG,IAAI,CAw9F9D"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,KAAK,EAAE,YAAY,EAAoB,MAAM,iCAAiC,CAAC;AAmvBtF,MAAM,CAAC,OAAO,UAAU,eAAe,CAAC,EAAE,EAAE,YAAY,GAAG,IAAI,CAwmG9D"}

package/dist/index.js

@@ -9,7 +9,7 @@
  *  - inject context into prompts
  */
 import { Type } from "typebox";
-import { ExportService, PlanStore, setWriteBusyHook, setWriteNotifyHook, migrateToUuids, migrateToGlobalSequence, withFeatureLock, needsMotivation, findPhaseByRef, findTaskByRef, buildRecap } from "@agent-plan/core";
+import { ExportService, PlanStore, setWriteBusyHook, setWriteNotifyHook, migrateToUuids, migrateToGlobalSequence, withFeatureLock, needsMotivation, findPhaseByRef, findTaskByRef, buildRecap, addChecklistItem, removeChecklistItem, toggleChecklistItem } from "@agent-plan/core";
 import { createChecklistItemId, createFeatureId, createPhaseId, createShortId, createTaskId, clampSlug, normalizeSlug, formatPhaseRef, formatFeatureRef, featureNumberOfPhase } from "@agent-plan/core/naming";
 import { join, dirname } from "node:path";
 import { homedir } from "node:os";
@@ -124,6 +124,9 @@ const PLANNER_COMMAND_COMPLETIONS = [
     { value: "task delete", label: "task delete", description: "Delete a task" },
     { value: "task start", label: "task start", description: "Mark a task in-progress" },
     { value: "task complete", label: "task complete", description: "Mark a task done" },
+    { value: "task checklist-add", label: "task checklist-add", description: "Add a checklist step: /planner task checklist-add <T00x> <title>" },
+    { value: "task checklist-remove", label: "task checklist-remove", description: "Remove a step: /planner task checklist-remove <T00x> <C{n}|title>" },
+    { value: "task checklist-toggle", label: "task checklist-toggle", description: "Tick/untick a step: /planner task checklist-toggle <T00x> <C{n}|title> [on|off]" },
     { value: "handoff list", label: "handoff list", description: "List phases with a handoff" },
     { value: "handoff prepare", label: "handoff prepare", description: "Tell the agent to write the handoff (legacy file-based)" },
     { value: "handoff show", label: "handoff show [P00x]", description: "Show a phase handoff (default: in-progress phase)" },
@@ -1374,6 +1377,7 @@ export default function planPiExtension(pi) {
                     acceptedDecisions: [],
                     phaseIds: [],
                     dependsOn: [],
+                    statusLog: [],
                     createdAt: now,
                     updatedAt: now,
                 };
@@ -1553,6 +1557,7 @@ export default function planPiExtension(pi) {
                         handoff: "", handoffUpdatedAt: "",
                         handoffReadAt: "",
                         handoffHistory: [],
+                        statusLog: [],
                     };
                     await st.savePhase(phase);
                     await st.updateFeatures((doc) => {
@@ -1768,7 +1773,7 @@ export default function planPiExtension(pi) {
         // ═══════════════════════════════════════════════════════════════
         if (a === "task") {
             if (!b) {
-                ctx.ui.notify("task actions: add  |  show [id]  |  discuss [id|name]  |  delete  |  update  |  start [id]  |  complete [id]", "info");
+                ctx.ui.notify("task actions: add  |  show [id]  |  discuss [id|name]  |  delete  |  update  |  start [id]  |  complete [id]  |  checklist-add <T00x> <title>  |  checklist-remove <T00x> <C{n}|title>  |  checklist-toggle <T00x> <C{n}|title> [on|off]", "info");
                 return;
             }
             if (b === "add") {
@@ -1839,7 +1844,7 @@ export default function planPiExtension(pi) {
                 if (description?.trim())
                     task.description = description.trim();
                 if (checklistSeed?.trim()) {
-                    task.checklist = splitCsv(checklistSeed).map((itemTitle, index) => ({
+                    task.checklist = splitCsv(checklistSeed).map((itemTitle, index) => ({ number: index + 1,
                         id: createChecklistItemId(task.id, index + 1, itemTitle),
                         title: itemTitle,
                         checked: false,
@@ -2084,7 +2089,75 @@ export default function planPiExtension(pi) {
                 ctx.ui.notify(`✅ Task completed: ${formatPhaseRef(phase.number, featureNumberOfPhase(phase, (await st.loadFeatures()).features))}/T${String(task.number).padStart(3, "0")} — ${task.title} (done)${task.shortId ? ` · ${task.shortId}` : ""}`, "info");
                 return;
             }
-            ctx.ui.notify(`Unknown task action "${b}". Try: add, show, discuss, delete, update, start, complete`, "warning");
+            // checklist-add <T00x> <title>  /  checklist-remove <T00x> <C{n}|title>  /  checklist-toggle <T00x> <C{n}|title> [on|off]
+            if (b === "checklist-add" || b === "checklist-remove" || b === "checklist-toggle") {
+                const [taskRef, ...itemParts] = rest;
+                const itemText = itemParts.join(" ").trim();
+                if (!taskRef || !itemText) {
+                    ctx.ui.notify(`Usage: /planner task ${b} <T00x> <${b === "checklist-add" ? "title" : "C{n}|title"}>${b === "checklist-toggle" ? " [on|off]" : ""}`, "warning");
+                    return;
+                }
+                const resolved = findTaskByRef(await st.loadAllPhases(), (await st.loadFeatures()).features, taskRef.trim());
+                if (!resolved) {
+                    ctx.ui.notify(`Task not found: ${taskRef}`, "warning");
+                    return;
+                }
+                let checked;
+                if (b === "checklist-toggle") {
+                    const m = itemText.match(/\s+(on|off)$/i);
+                    if (m) {
+                        checked = m[1].toLowerCase() === "on";
+                    }
+                }
+                const selector = b === "checklist-toggle" ? itemText.replace(/\s+(on|off)$/i, "").trim() : itemText;
+                let msg = "";
+                await st.updatePhase(resolved.phase.id, (phase) => {
+                    const task = phase.tasks.find((t) => t.id === resolved.task.id);
+                    if (!task) {
+                        msg = `Task not found: ${taskRef}`;
+                        return phase;
+                    }
+                    const ck = task.checklist ?? [];
+                    if (b === "checklist-add") {
+                        const item = addChecklistItem(ck, task.id, selector);
+                        ck.push(item);
+                        task.checklist = ck;
+                        msg = `Added C${item.number} "${item.title}" (${ck.length} items)`;
+                    }
+                    else if (b === "checklist-remove") {
+                        if (ck.length === 0) {
+                            msg = `Task "${task.title}" has no checklist.`;
+                            return phase;
+                        }
+                        const removed = removeChecklistItem(ck, selector);
+                        if (!removed) {
+                            msg = `No checklist item matching "${selector}".`;
+                            return phase;
+                        }
+                        msg = `Removed C${removed.number} "${removed.title}" (${ck.length} items left)`;
+                    }
+                    else {
+                        if (ck.length === 0) {
+                            msg = `Task "${task.title}" has no checklist.`;
+                            return phase;
+                        }
+                        const target = toggleChecklistItem(ck, selector, checked);
+                        if (!target) {
+                            msg = `No checklist item matching "${selector}".`;
+                            return phase;
+                        }
+                        const doneCount = ck.filter((i) => i.checked).length;
+                        msg = `C${target.number} "${target.title}" → ${target.checked ? "done" : "open"} (${doneCount}/${ck.length} checked)`;
+                    }
+                    task.updatedAt = nowISO();
+                    phase.updatedAt = nowISO();
+                    return phase;
+                });
+                await st.writeGenerated();
+                ctx.ui.notify(`✅ ${msg}`, "info");
+                return;
+            }
+            ctx.ui.notify(`Unknown task action "${b}". Try: add, show, discuss, delete, update, start, complete, checklist-add, checklist-remove, checklist-toggle`, "warning");
             return;
         }
         // ── discuss (legacy alias) ──
@@ -2904,6 +2977,7 @@ export default function planPiExtension(pi) {
                 acceptedDecisions: [],
                 phaseIds: [],
                 dependsOn: [],
+                statusLog: [],
                 createdAt: now,
                 updatedAt: now,
             };
@@ -3119,6 +3193,7 @@ export default function planPiExtension(pi) {
                     handoff: "", handoffUpdatedAt: "",
                     handoffReadAt: "",
                     handoffHistory: [],
+                    statusLog: [],
                 };
                 await st.savePhase(phase);
                 // Atomic: serialize concurrent phase_create linking to the same feature.
@@ -3335,6 +3410,7 @@ export default function planPiExtension(pi) {
             description: Type.String({ description: "REQUIRED — execution context: code references (file:line), current state vs desired state, structs/traits to modify, concrete implementation steps, edge cases. Not a one-liner. Prefix with 'design-only' for pre-implementation design tasks.", minLength: 50 }),
             status: Type.Optional(Type.String({ description: "Initial status. Default: planned" })),
             shortName: Type.Optional(Type.String({ description: "Short slug for the task id. Auto-derived from title if omitted." })),
+            checklist: Type.Optional(Type.Array(Type.String(), { description: "Initial checklist items (plain strings). Seeded as C1, C2, … (unchecked)." })),
         }),
         async execute(_id, params, _signal, _onUpdate, ctx) {
             const st = await requirePlan(ctx);
@@ -3363,7 +3439,10 @@ export default function planPiExtension(pi) {
                 statusLog: [],
                 decisions: [],
                 acceptedDecisions: [],
-                checklist: [], subtasks: [],
+                checklist: params.checklist
+                    ? params.checklist.map((itemTitle, index) => ({ number: index + 1, id: createChecklistItemId(taskId, index + 1, itemTitle), title: itemTitle, checked: false }))
+                    : [],
+                subtasks: [],
                 dependsOn: [],
                 startedAt: initialStatus === "in-progress" || initialStatus === "done" ? now : "",
                 completedAt: initialStatus === "done" ? now : "",
@@ -3433,7 +3512,7 @@ export default function planPiExtension(pi) {
                     if (params.acceptedDecisions !== undefined)
                         task.acceptedDecisions = params.acceptedDecisions;
                     if (params.checklist !== undefined) {
-                        task.checklist = params.checklist.map((itemTitle, index) => ({
+                        task.checklist = params.checklist.map((itemTitle, index) => ({ number: index + 1,
                             id: createChecklistItemId(task.id, index + 1, itemTitle),
                             title: itemTitle,
                             checked: false,
@@ -3582,6 +3661,125 @@ export default function planPiExtension(pi) {
             return { content: [{ type: "text", text: `✅ Task completed: ${completedTask.id} — ${completedTask.title} (done)` }], details: completedTask };
         },
     });
+    pi.registerTool({
+        name: "task_checklist_toggle",
+        label: "Task Checklist Toggle",
+        description: "Tick/untick a task checklist item (a 'step') without rewriting the whole list. Accepts the item as C{n} (e.g. C2), item id, or title (case-insensitive, first match). Pass checked=true/false to set explicitly, or omit to toggle. Use this instead of writing DONE in step titles. Items are numbered C1/C2… per task.",
+        parameters: Type.Object({
+            taskId: Type.String({ description: "Task ref: F00x/P00x/T00x, bare T00x (global), 5-char shortId, UUID, or title" }),
+            item: Type.String({ description: "Checklist item selector: C{n} (e.g. C2), item id, or title (case-insensitive)" }),
+            checked: Type.Optional(Type.Boolean({ description: "Set explicitly: true=done, false=open. Omit to toggle." })),
+        }),
+        async execute(_id, params, _signal, _onUpdate, ctx) {
+            const st = await requirePlan(ctx);
+            if (!st)
+                return { content: [{ type: "text", text: "No .planner/ found." }], details: {} };
+            const found = findTaskByRef(await st.loadAllPhases(), (await st.loadFeatures()).features, params.taskId.trim());
+            if (!found)
+                return { content: [{ type: "text", text: `Task not found: ${params.taskId}` }], details: {} };
+            let result = "";
+            await st.updatePhase(found.phase.id, (phase) => {
+                const task = phase.tasks.find((t) => t.id === found.task.id);
+                if (!task) {
+                    result = `Task not found: ${params.taskId}`;
+                    return phase;
+                }
+                const ck = task.checklist ?? [];
+                if (ck.length === 0) {
+                    result = `Task "${found.task.title}" has no checklist.`;
+                    return phase;
+                }
+                const target = toggleChecklistItem(ck, params.item, params.checked);
+                if (!target) {
+                    result = `No checklist item matching "${params.item}".`;
+                    return phase;
+                }
+                task.updatedAt = nowISO();
+                phase.updatedAt = nowISO();
+                const doneCount = ck.filter((i) => i.checked).length;
+                result = `C${target.number} "${target.title}" → ${target.checked ? "done" : "open"} (${doneCount}/${ck.length} checked)`;
+                return phase;
+            });
+            await st.writeGenerated();
+            return { content: [{ type: "text", text: `✅ ${result}` }], details: {} };
+        },
+    });
+    pi.registerTool({
+        name: "task_checklist_add",
+        label: "Task Checklist Add",
+        description: "Add a single checklist item (a 'step') to a task without rewriting the list. The new item is appended as C{n} (next progressive number, stable id, unchecked). Use this to subdivide a task into smaller steps INSTEAD of spawning sub-tasks — the checklist keeps description, notes, statusLog and steps concentrated in one task (sub-tasks disperse context). Also use for granular adds instead of replacing the whole checklist via task_update.",
+        parameters: Type.Object({
+            taskId: Type.String({ description: "Task ref: F00x/P00x/T00x, bare T00x (global), 5-char shortId, UUID, or title" }),
+            title: Type.String({ description: "Checklist item text (a single step)" }),
+        }),
+        async execute(_id, params, _signal, _onUpdate, ctx) {
+            const st = await requirePlan(ctx);
+            if (!st)
+                return { content: [{ type: "text", text: "No .planner/ found." }], details: {} };
+            const found = findTaskByRef(await st.loadAllPhases(), (await st.loadFeatures()).features, params.taskId.trim());
+            if (!found)
+                return { content: [{ type: "text", text: `Task not found: ${params.taskId}` }], details: {} };
+            let result = "";
+            await st.updatePhase(found.phase.id, (phase) => {
+                const task = phase.tasks.find((t) => t.id === found.task.id);
+                if (!task) {
+                    result = `Task not found: ${params.taskId}`;
+                    return phase;
+                }
+                const ck = task.checklist ?? [];
+                const item = addChecklistItem(ck, task.id, params.title);
+                ck.push(item);
+                task.checklist = ck;
+                task.updatedAt = nowISO();
+                phase.updatedAt = nowISO();
+                result = `Added C${item.number} "${item.title}" (${ck.length} items)`;
+                return phase;
+            });
+            await st.writeGenerated();
+            return { content: [{ type: "text", text: `✅ ${result}` }], details: {} };
+        },
+    });
+    pi.registerTool({
+        name: "task_checklist_remove",
+        label: "Task Checklist Remove",
+        description: "Remove a single checklist item (a 'step') from a task by C{n} (e.g. C2), item id, or title (case-insensitive). Remaining items are renumbered C1..Cn for readability; their stable ids are preserved. Use this for granular removes instead of replacing the whole checklist via task_update.",
+        parameters: Type.Object({
+            taskId: Type.String({ description: "Task ref: F00x/P00x/T00x, bare T00x (global), 5-char shortId, UUID, or title" }),
+            item: Type.String({ description: "Checklist item selector: C{n} (e.g. C2), item id, or title (case-insensitive)" }),
+        }),
+        async execute(_id, params, _signal, _onUpdate, ctx) {
+            const st = await requirePlan(ctx);
+            if (!st)
+                return { content: [{ type: "text", text: "No .planner/ found." }], details: {} };
+            const found = findTaskByRef(await st.loadAllPhases(), (await st.loadFeatures()).features, params.taskId.trim());
+            if (!found)
+                return { content: [{ type: "text", text: `Task not found: ${params.taskId}` }], details: {} };
+            let result = "";
+            await st.updatePhase(found.phase.id, (phase) => {
+                const task = phase.tasks.find((t) => t.id === found.task.id);
+                if (!task) {
+                    result = `Task not found: ${params.taskId}`;
+                    return phase;
+                }
+                const ck = task.checklist ?? [];
+                if (ck.length === 0) {
+                    result = `Task "${found.task.title}" has no checklist.`;
+                    return phase;
+                }
+                const removed = removeChecklistItem(ck, params.item);
+                if (!removed) {
+                    result = `No checklist item matching "${params.item}".`;
+                    return phase;
+                }
+                task.updatedAt = nowISO();
+                phase.updatedAt = nowISO();
+                result = `Removed C${removed.number} "${removed.title}" (${ck.length} items left)`;
+                return phase;
+            });
+            await st.writeGenerated();
+            return { content: [{ type: "text", text: `✅ ${result}` }], details: {} };
+        },
+    });
     // ── Web lifecycle tools (agent-callable; parity with @agent-plan/mcp planner-web) ──
     // These let the agent manage the web dashboard directly, without relying
     // on the /planner slash command (which is not intercepted when the planner

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agent-plan/pi-adapter",
-  "version": "0.2.19-next.12",
+  "version": "0.2.19-next.14",
   "private": false,
   "description": "Pi agent extension for Agent Plan — structured project planning in .planner/",
   "keywords": [
@@ -28,8 +28,8 @@
   "dependencies": {
     "typebox": "^1.1.24",
     "zod": "^3.24.0",
-    "@agent-plan/core": "0.2.19-next.12",
-    "@agent-plan/server": "0.2.19-next.12"
+    "@agent-plan/core": "0.2.19-next.14",
+    "@agent-plan/server": "0.2.19-next.14"
   },
   "devDependencies": {
     "@earendil-works/pi-coding-agent": "^0.74.0",