@agent-plan/pi-adapter 0.2.20 → 0.2.22

package/README.md

@@ -10,10 +10,22 @@ pi install npm:@agent-plan/pi-adapter
 
 Then restart Pi in a project directory to enable planning.
 
+Check the version actually loaded by Pi:
+
+```text
+/planner version
+```
+
+From a shell, inspect the installed package directly:
+
+```bash
+npm --prefix "$HOME/.pi/agent/npm" list @agent-plan/pi-adapter --depth=0
+```
+
 ## Commands
 
 Once installed, use `/planner init` to initialize planning in a project,
-then `/planner feature add`, `/planner phase add`, `/planner task add`,
+then `/planner version`, `/planner feature add`, `/planner phase add`, `/planner task add`,
 and the lifecycle commands `/planner task start`/`/planner task complete`.
 
 See the [root README](https://github.com/ovidius72/agent-planner#readme) for the full command reference.

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;AAgzBtF,MAAM,CAAC,OAAO,UAAU,eAAe,CAAC,EAAE,EAAE,YAAY,GAAG,IAAI,CAshH9D"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,KAAK,EAAE,YAAY,EAAoB,MAAM,iCAAiC,CAAC;AAozBtF,MAAM,CAAC,OAAO,UAAU,eAAe,CAAC,EAAE,EAAE,YAAY,GAAG,IAAI,CA4wH9D"}

package/dist/index.js

@@ -9,7 +9,7 @@
  *  - inject context into prompts
  */
 import { Type } from "typebox";
-import { ExportService, PlanStore, setWriteBusyHook, setWriteNotifyHook, withFeatureLock, needsMotivation, findPhaseByRef, findTaskByRef, buildRecap, addChecklistItem, removeChecklistItem, toggleChecklistItem, buildPhaseContextBlock, checkExplicitTaskStart, recommendNextTask } from "@agent-plan/core";
+import { ExportService, PlanStore, setWriteBusyHook, setWriteNotifyHook, withFeatureLock, needsMotivation, findPhaseByRef, findTaskByRef, buildRecap, addChecklistItem, removeChecklistItem, toggleChecklistItem, buildPhaseContextBlock, checkExplicitTaskStart, recommendNextTask, packageVersionFromModule, resolvedPackageVersion } from "@agent-plan/core";
 import { createChecklistItemId, createFeatureId, createPhaseId, createTaskId, clampSlug, normalizeSlug, formatPhaseRef, formatFeatureRef, featureNumberOfPhase, validateResolvedTarget } from "@agent-plan/core/naming";
 import { join, dirname } from "node:path";
 import { homedir } from "node:os";
@@ -21,6 +21,9 @@ import { serve } from "@agent-plan/server/serve";
 // ─── State (module-level) ───────────────────────────────────────────────
 const PI_CONFIG_DIR_NAME = ".pi";
 const PLAN_DIR_NAME = ".planner";
+const PI_ADAPTER_PACKAGE = packageVersionFromModule(import.meta.url, "@agent-plan/pi-adapter");
+const CORE_PACKAGE = resolvedPackageVersion("@agent-plan/core", import.meta.url);
+const SERVER_PACKAGE = resolvedPackageVersion("@agent-plan/server", import.meta.url);
 let capturedPi = null;
 let store = null;
 let server = null;
@@ -104,6 +107,7 @@ const healedStatusRoots = new Set();
 const PLANNER_COMMAND_COMPLETIONS = [
     { value: "init", label: "init", description: "Initialize planner in this project" },
     { value: "show", label: "show", description: "Show planner overview" },
+    { value: "version", label: "version", description: "Show loaded Agent Plan package versions" },
     { value: "repair", label: "repair", description: "Repair planner integrity" },
     { value: "cleanup-orphans", label: "cleanup-orphans", description: "List and remove orphan phase files" },
     { value: "project discuss", label: "project discuss", description: "Run project discovery" },
@@ -1041,6 +1045,7 @@ export default function planPiExtension(pi) {
         const PLANNER_MENU_ACTIONS = [
             "init",
             "show",
+            "version",
             "repair",
             "cleanup-orphans",
             "project discuss",
@@ -1076,7 +1081,7 @@ export default function planPiExtension(pi) {
             "load",
             "stop",
         ];
-        const SUB_HELP = "Available: init, show, repair, cleanup-orphans, project, feature, phase, task, discuss, handoff, web, export, export-full, bypass, clear-bypass, load, stop\n" +
+        const SUB_HELP = "Available: init, show, version, repair, cleanup-orphans, project, feature, phase, task, discuss, handoff, web, export, export-full, bypass, clear-bypass, load, stop\n" +
             "Try: /planner <TAB>  |  /planner feature list  |  /planner task start  |  /planner cleanup-orphans\n" +
             "Handoff actions: /planner handoff list | show P00x | write P00x | clear P00x | prepare (prepare proposes a target and asks for confirmation)";
         if (!a) {
@@ -1088,6 +1093,15 @@ export default function planPiExtension(pi) {
             await runPlanner(action, ctx);
             return;
         }
+        if (a === "version") {
+            ctx.ui.notify([
+                "Agent Plan runtime versions",
+                `${PI_ADAPTER_PACKAGE.name}: ${PI_ADAPTER_PACKAGE.version}`,
+                `${CORE_PACKAGE.name}: ${CORE_PACKAGE.version}`,
+                `${SERVER_PACKAGE.name}: ${SERVER_PACKAGE.version}`,
+            ].join("\n"), "info");
+            return;
+        }
         const st = ensureStore(ctx);
         // ── init ──
         if (a === "init") {
@@ -1957,6 +1971,7 @@ export default function planPiExtension(pi) {
                     acceptedDecisions: [],
                     checklist: [], subtasks: [],
                     dependsOn: [],
+                    pauseSnapshot: null, pauseHistory: [],
                     startedAt: "", completedAt: "",
                     createdAt: now, updatedAt: now,
                 };
@@ -3877,6 +3892,9 @@ export default function planPiExtension(pi) {
             const priority = await st.nextPriority("task", phaseId);
             const now = nowISO();
             const initialStatus = params.status ?? "planned";
+            if (initialStatus === "paused") {
+                return { content: [{ type: "text", text: "A task cannot be created paused without an in-progress checkpoint. Create it planned, then use task_pause/task_switch." }], details: {} };
+            }
             const task = {
                 id: taskId, phaseId, number: identity.number, shortId: identity.shortId, priority, shortName,
                 title: params.title,
@@ -3892,6 +3910,8 @@ export default function planPiExtension(pi) {
                     : [],
                 subtasks: [],
                 dependsOn: [],
+                pauseSnapshot: null,
+                pauseHistory: [],
                 startedAt: initialStatus === "in-progress" || initialStatus === "done" ? now : "",
                 completedAt: initialStatus === "done" ? now : "",
                 createdAt: now, updatedAt: now,
@@ -4085,7 +4105,12 @@ export default function planPiExtension(pi) {
             if (temporary.task.status !== "planned")
                 return { content: [{ type: "text", text: `Temporary task is not startable: ${temporary.task.status}.` }], details: {} };
             const timestamp = nowISO();
-            const record = { id: crypto.randomUUID(), recommendedTaskId: selected.candidate?.task.id ?? resume.task.id, temporaryTaskId: temporary.task.id, resumeTaskId: resume.task.id, reason: params.reason, requestedBy: "user", approvedBy: "user", state: "approved", createdAt: timestamp, activatedAt: "", resolvedAt: "" };
+            const record = {
+                id: crypto.randomUUID(), recommendedTaskId: selected.candidate?.task.id ?? resume.task.id,
+                temporaryTaskId: temporary.task.id, resumeTaskId: resume.task.id, reason: params.reason, snapshot: null,
+                requestedBy: "user", approvedBy: "user", state: "approved",
+                createdAt: timestamp, activatedAt: "", resumeRequiredAt: "", resolvedAt: "", resumedAt: "",
+            };
             await st.addWorkDeviation(record);
             await st.writeGenerated();
             const tempRef = `${formatPhaseRef(temporary.phase.number, featureNumberOfPhase(temporary.phase, features.features))}/T${String(temporary.task.number).padStart(3, "0")}`;
@@ -4093,10 +4118,198 @@ export default function planPiExtension(pi) {
             return { content: [{ type: "text", text: `✅ Approved deviation: ${tempRef} temporarily overrides ${resumeRef}. Resume target retained.` }], details: { deviation: record } };
         },
     });
+    pi.registerTool({
+        name: "task_pause",
+        label: "Task Pause",
+        description: "Pause an in-progress task with a mandatory durable checkpoint: why work stopped, what was underway, the exact resume location, and how to continue.",
+        parameters: Type.Object({
+            taskId: Type.String({ description: "Task ref to pause" }),
+            reason: Type.String({ description: "Why work is being paused" }),
+            what_was_being_done: Type.String({ description: "Concrete work underway at the checkpoint" }),
+            resume_location: Type.String({ description: "Exact file, symbol, command, or location from which to continue" }),
+            how_to_resume: Type.String({ description: "Actionable instructions for resuming" }),
+            paused_by: Type.Optional(Type.String({ description: "Agent/session identifier" })),
+        }),
+        async execute(_id, params, _signal, _onUpdate, ctx) {
+            const st = await requirePlan(ctx);
+            if (!st)
+                return { content: [{ type: "text", text: "No .planner/ found." }], details: {} };
+            const features = (await st.loadFeatures()).features;
+            const found = findTaskByRef(await st.loadAllPhases(), features, params.taskId.trim());
+            if (!found)
+                return { content: [{ type: "text", text: `Task not found: ${params.taskId}` }], details: {} };
+            if (found.task.status !== "in-progress")
+                return { content: [{ type: "text", text: `Task pause denied: task is ${found.task.status}, not in-progress.` }], details: found.task };
+            const snapshot = {
+                id: crypto.randomUUID(), reason: params.reason.trim(), whatWasBeingDone: params.what_was_being_done.trim(),
+                resumeLocation: params.resume_location.trim(), howToResume: params.how_to_resume.trim(), relatedTaskId: "",
+                pausedAt: nowISO(), pausedBy: params.paused_by?.trim() ?? "",
+            };
+            const paused = await st.pauseTask(found.phase.id, found.task.id, snapshot);
+            await st.syncTaskStatusRollup(found.phase.id);
+            const activeDeviation = (await st.loadProject()).workDeviations
+                .filter((deviation) => (deviation.state === "approved" || deviation.state === "active")
+                && deviation.temporaryTaskId === found.task.id)
+                .sort((left, right) => right.createdAt.localeCompare(left.createdAt))[0];
+            if (activeDeviation)
+                await st.setWorkDeviationState(activeDeviation.id, "resume-required", snapshot.pausedAt);
+            const resume = activeDeviation
+                ? findTaskByRef(await st.loadAllPhases(), features, activeDeviation.resumeTaskId)
+                : undefined;
+            await st.writeGenerated();
+            const reference = `${formatPhaseRef(found.phase.number, featureNumberOfPhase(found.phase, features))}/T${String(paused.number).padStart(3, "0")}`;
+            const resumeRef = resume
+                ? `${formatPhaseRef(resume.phase.number, featureNumberOfPhase(resume.phase, features))}/T${String(resume.task.number).padStart(3, "0")}`
+                : "";
+            return {
+                content: [{ type: "text", text: [
+                            `⏸️ Task paused: ${reference} — ${paused.title}`,
+                            `Why: ${snapshot.reason}`,
+                            `Checkpoint: ${snapshot.whatWasBeingDone}`,
+                            `Resume from: ${snapshot.resumeLocation}`,
+                            `How to resume: ${snapshot.howToResume}`,
+                            resume ? `↩️ RESUME REQUIRED: ${resumeRef} — ${resume.task.title}` : "",
+                        ].filter(Boolean).join("\n") }],
+                details: { task: paused, snapshot, ...(resume ? { resumeRequired: { taskId: resume.task.id } } : {}) },
+            };
+        },
+    });
+    pi.registerTool({
+        name: "task_switch",
+        label: "Task Switch",
+        description: "Switch from active work to a temporary task, even outside priority order. Atomically snapshots/pauses the source, starts the target, and pushes a durable LIFO return target.",
+        parameters: Type.Object({
+            from_task: Type.String({ description: "Currently in-progress task ref" }),
+            to_task: Type.String({ description: "Temporary task ref to start" }),
+            reason: Type.String({ description: "Why this temporary switch is necessary" }),
+            what_was_being_done: Type.String({ description: "Concrete work underway in the source task" }),
+            resume_location: Type.String({ description: "Exact source resume location" }),
+            how_to_resume: Type.String({ description: "Actionable source resume instructions" }),
+            switched_by: Type.Optional(Type.String({ description: "Agent/session identifier" })),
+        }),
+        async execute(_id, params, _signal, _onUpdate, ctx) {
+            const st = await requirePlan(ctx);
+            if (!st)
+                return { content: [{ type: "text", text: "No .planner/ found." }], details: {} };
+            const [featuresDoc, phases, project] = await Promise.all([st.loadFeatures(), st.loadAllPhases(), st.loadProject()]);
+            const features = featuresDoc.features;
+            const source = findTaskByRef(phases, features, params.from_task.trim());
+            const target = findTaskByRef(phases, features, params.to_task.trim());
+            if (!source)
+                return { content: [{ type: "text", text: `Task not found: ${params.from_task}` }], details: {} };
+            if (!target)
+                return { content: [{ type: "text", text: `Task not found: ${params.to_task}` }], details: {} };
+            if (source.task.id === target.task.id)
+                return { content: [{ type: "text", text: "Source and temporary task must differ." }], details: {} };
+            if (source.task.status !== "in-progress" && source.task.status !== "paused") {
+                return { content: [{ type: "text", text: `Task switch denied: source is ${source.task.status}; only in-progress or pending-resume paused work can be switched.` }], details: {} };
+            }
+            const eligibility = checkExplicitTaskStart(features, phases, target.task.id, project.workDeviations);
+            if (!eligibility.eligible)
+                return { content: [{ type: "text", text: `Task switch denied: ${eligibility.reason}` }], details: eligibility };
+            const timestamp = nowISO();
+            const snapshot = {
+                id: crypto.randomUUID(), reason: params.reason.trim(), whatWasBeingDone: params.what_was_being_done.trim(),
+                resumeLocation: params.resume_location.trim(), howToResume: params.how_to_resume.trim(),
+                relatedTaskId: target.task.id, pausedAt: timestamp, pausedBy: params.switched_by?.trim() ?? "",
+            };
+            const selection = recommendNextTask(features, phases, project.workDeviations);
+            const record = {
+                id: crypto.randomUUID(), recommendedTaskId: selection.candidate?.task.id ?? source.task.id,
+                temporaryTaskId: target.task.id, resumeTaskId: source.task.id, reason: snapshot.reason, snapshot,
+                requestedBy: "agent", approvedBy: params.switched_by?.trim() || "explicit task_switch",
+                state: "active", createdAt: timestamp, activatedAt: timestamp,
+                resumeRequiredAt: "", resolvedAt: "", resumedAt: "",
+            };
+            const sourceWasActive = source.task.status === "in-progress";
+            let sourceCheckpointed = false;
+            let targetStarted = false;
+            try {
+                if (sourceWasActive) {
+                    await st.pauseTask(source.phase.id, source.task.id, snapshot);
+                }
+                else {
+                    await st.updatePhase(source.phase.id, (phase) => {
+                        const task = phase.tasks.find((entry) => entry.id === source.task.id);
+                        if (!task)
+                            throw new Error(`Paused source task disappeared: ${params.from_task}`);
+                        task.pauseSnapshot = snapshot;
+                        task.pauseHistory = [...task.pauseHistory, snapshot];
+                        task.updatedAt = timestamp;
+                        return phase;
+                    });
+                }
+                sourceCheckpointed = true;
+                if (target.task.status === "paused") {
+                    await st.resumeTask(target.phase.id, target.task.id, timestamp);
+                }
+                else {
+                    await st.updatePhase(target.phase.id, (phase) => {
+                        const task = phase.tasks.find((entry) => entry.id === target.task.id);
+                        if (!task)
+                            throw new Error(`Temporary task disappeared: ${params.to_task}`);
+                        const previousStatus = task.status;
+                        applyTaskLifecycleDates(task, "in-progress", timestamp);
+                        task.statusLog = [...task.statusLog, {
+                                id: createChecklistItemId(task.id, task.statusLog.length + 1, `${previousStatus}-in-progress`),
+                                date: timestamp, fromStatus: previousStatus, toStatus: "in-progress",
+                                title: `${previousStatus} → in-progress`, description: `Temporary switch from ${source.task.title}.`,
+                            }];
+                        task.updatedAt = timestamp;
+                        phase.updatedAt = timestamp;
+                        return phase;
+                    });
+                }
+                targetStarted = true;
+                await st.addWorkDeviation(record);
+                if (target.task.status === "paused") {
+                    const resumedDeviation = project.workDeviations
+                        .filter((deviation) => deviation.resumeTaskId === target.task.id
+                        && (deviation.state === "resume-required" || deviation.state === "resolved"))
+                        .sort((left, right) => right.createdAt.localeCompare(left.createdAt))[0];
+                    if (resumedDeviation)
+                        await st.setWorkDeviationState(resumedDeviation.id, "resumed", timestamp);
+                }
+            }
+            catch (error) {
+                if (targetStarted) {
+                    await st.updatePhase(target.phase.id, (phase) => {
+                        phase.tasks = phase.tasks.map((task) => task.id === target.task.id ? target.task : task);
+                        return phase;
+                    }).catch(() => { });
+                }
+                if (sourceCheckpointed) {
+                    if (sourceWasActive)
+                        await st.resumeTask(source.phase.id, source.task.id, nowISO()).catch(() => { });
+                    else
+                        await st.updatePhase(source.phase.id, (phase) => {
+                            phase.tasks = phase.tasks.map((task) => task.id === source.task.id ? source.task : task);
+                            return phase;
+                        }).catch(() => { });
+                }
+                throw error;
+            }
+            await st.syncTaskStatusRollup(source.phase.id);
+            if (target.phase.id !== source.phase.id)
+                await st.syncTaskStatusRollup(target.phase.id);
+            await st.writeGenerated();
+            const sourceRef = `${formatPhaseRef(source.phase.number, featureNumberOfPhase(source.phase, features))}/T${String(source.task.number).padStart(3, "0")}`;
+            const targetRef = `${formatPhaseRef(target.phase.number, featureNumberOfPhase(target.phase, features))}/T${String(target.task.number).padStart(3, "0")}`;
+            return {
+                content: [{ type: "text", text: [
+                            `🔀 Task switched: ${sourceRef} → ${targetRef}`,
+                            `Paused snapshot: ${snapshot.whatWasBeingDone}`,
+                            `Return target: ${sourceRef} from ${snapshot.resumeLocation}`,
+                            "Normal priority was deliberately overridden; completing the temporary task will emit RESUME REQUIRED.",
+                        ].join("\n") }],
+                details: { deviation: record, snapshot, resumeTaskId: source.task.id, temporaryTaskId: target.task.id },
+            };
+        },
+    });
     pi.registerTool({
         name: "task_start",
         label: "Task Start",
-        description: "Set a task to in-progress. BEFORE calling, read feature_get(full=true) for the parent feature and its linked requirements, then phase_get(full=true) for the parent phase and its linked requirements, then task_get(full=true). Continue an already in-progress task; otherwise select ready work by feature → phase → task priority (lower number first), respecting dependencies and blocked/waiting states. Sets startedAt automatically and returns the ordered feature/phase requirements briefing.",
+        description: "Set a task to in-progress or resume paused work. BEFORE calling, read feature_get(full=true), phase_get(full=true), then task_get(full=true). A different active task must first be suspended through task_switch. Otherwise follow active/pending-resume/feature → phase → task priority guidance.",
         parameters: Type.Object({
             taskId: Type.String({ description: "Task ref: F00x/P00x/T00x, bare T00x (global), 5-char shortId, UUID, or title to start" }),
         }),
@@ -4119,6 +4332,17 @@ export default function planPiExtension(pi) {
             if (!eligibility.eligible)
                 return { content: [{ type: "text", text: `Task start denied: ${eligibility.reason}` }], details: eligibility };
             const selection = recommendNextTask(features, phases, project.workDeviations);
+            if (selection.kind === "conflict") {
+                return { content: [{ type: "text", text: `${selection.reason} Pause/reconcile active tasks before starting another task.` }], details: selection };
+            }
+            if (selection.kind === "active" && selection.candidate?.task.id !== task.id) {
+                const activeRef = `${formatPhaseRef(selection.candidate.phase.number, featureNumberOfPhase(selection.candidate.phase, features))}/T${String(selection.candidate.task.number).padStart(3, "0")}`;
+                return { content: [{ type: "text", text: `Task start denied: ${activeRef} is active. Use task_switch to snapshot and pause it before starting another task.` }], details: selection };
+            }
+            if (selection.kind === "resume" && selection.candidate?.task.id !== task.id) {
+                const resumeRef = `${formatPhaseRef(selection.candidate.phase.number, featureNumberOfPhase(selection.candidate.phase, features))}/T${String(selection.candidate.task.number).padStart(3, "0")}`;
+                return { content: [{ type: "text", text: `Task start denied: ${resumeRef} is pending resume. Resume it now, or use task_switch from that paused task with a new checkpoint and reason.` }], details: selection };
+            }
             // Assemble the mandatory parent context before changing task lifecycle state.
             // The output order is feature + its requirements, then phase + its requirements.
             const parentFeature = found.phase.featureId ? features.find((f) => f.id === found.phase.featureId) : undefined;
@@ -4127,29 +4351,42 @@ export default function planPiExtension(pi) {
                 parentFeature ? st.linkedRequirementsForFeature(parentFeature.id) : Promise.resolve([]),
             ]);
             const phaseContext = buildPhaseContextBlock(phaseWithRequirements, parentFeature, phaseWithRequirements.linkedRequirements ?? [], featureRequirements);
-            const advisory = selection.kind === "conflict"
-                ? `\n\n⚠️ ${selection.reason} Explicit task request honored; reconcile active work before relying on automatic recommendations.`
-                : selection.candidate && selection.candidate.task.id !== task.id
-                    ? selection.kind === "active"
-                        ? `\n\n⚠️ Active-task advisory: ${formatPhaseRef(selection.candidate.phase.number, featureNumberOfPhase(selection.candidate.phase, features))}/T${String(selection.candidate.task.number).padStart(3, "0")} is already in progress. Explicit task request honored.`
-                        : `\n\n⚠️ Priority advisory: ${formatPhaseRef(selection.candidate.phase.number, featureNumberOfPhase(selection.candidate.phase, features))}/T${String(selection.candidate.task.number).padStart(3, "0")} is the automatic recommendation. Explicit task request honored.`
-                    : "";
+            const advisory = selection.candidate && selection.candidate.task.id !== task.id
+                ? `\n\n⚠️ ${selection.kind === "resume" ? "Resume" : "Priority"} advisory: ${formatPhaseRef(selection.candidate.phase.number, featureNumberOfPhase(selection.candidate.phase, features))}/T${String(selection.candidate.task.number).padStart(3, "0")} is the automatic recommendation. Explicit task request honored.`
+                : "";
             const now = nowISO();
             let startedTask;
-            await st.updatePhase(found.phase.id, (phase) => {
-                const t = phase.tasks.find((x) => x.id === found.task.id);
-                if (!t)
+            if (task.status === "paused") {
+                startedTask = await st.resumeTask(found.phase.id, found.task.id, now);
+            }
+            else {
+                await st.updatePhase(found.phase.id, (phase) => {
+                    const t = phase.tasks.find((x) => x.id === found.task.id);
+                    if (!t)
+                        return phase;
+                    const previousStatus = t.status;
+                    applyTaskLifecycleDates(t, "in-progress", now);
+                    t.statusLog = [...t.statusLog, {
+                            id: createChecklistItemId(t.id, t.statusLog.length + 1, `${previousStatus}-in-progress`),
+                            date: now, fromStatus: previousStatus, toStatus: "in-progress",
+                            title: `${previousStatus} → in-progress`, description: "",
+                        }];
+                    t.updatedAt = now;
+                    phase.updatedAt = now;
+                    startedTask = t;
                     return phase;
-                applyTaskLifecycleDates(t, "in-progress", now);
-                t.updatedAt = now;
-                phase.updatedAt = now;
-                startedTask = t;
-                return phase;
-            });
+                });
+            }
             await st.syncTaskStatusRollup(found.phase.id);
             const approvedDeviation = project.workDeviations.find((deviation) => deviation.temporaryTaskId === task.id && deviation.state === "approved");
             if (approvedDeviation)
                 await st.setWorkDeviationState(approvedDeviation.id, "active", now);
+            const resumedDeviation = project.workDeviations
+                .filter((deviation) => deviation.resumeTaskId === task.id
+                && (deviation.state === "resume-required" || deviation.state === "resolved"))
+                .sort((left, right) => right.createdAt.localeCompare(left.createdAt))[0];
+            if (resumedDeviation)
+                await st.setWorkDeviationState(resumedDeviation.id, "resumed", now);
             await st.writeGenerated();
             if (!startedTask)
                 return { content: [{ type: "text", text: `Task not found: ${params.taskId}` }], details: {} };
@@ -4176,6 +4413,8 @@ export default function planPiExtension(pi) {
             const task = found.task;
             if (task.status === "done")
                 return { content: [{ type: "text", text: `Task ${task.id} is already done.` }], details: task };
+            if (task.status === "paused")
+                return { content: [{ type: "text", text: "Task completion denied: resume paused work with task_start before completing it, or cancel it explicitly with motivation." }], details: task };
             const unchecked = task.checklist.filter((item) => !item.checked);
             if (unchecked.length > 0 && !params.force) {
                 return {
@@ -4189,7 +4428,13 @@ export default function planPiExtension(pi) {
                 const t = phase.tasks.find((x) => x.id === found.task.id);
                 if (!t)
                     return phase;
+                const previousStatus = t.status;
                 applyTaskLifecycleDates(t, "done", now);
+                t.statusLog = [...t.statusLog, {
+                        id: createChecklistItemId(t.id, t.statusLog.length + 1, `${previousStatus}-done`),
+                        date: now, fromStatus: previousStatus, toStatus: "done",
+                        title: `${previousStatus} → done`, description: "",
+                    }];
                 if (params.description_update) {
                     const sep = t.description ? "\n\n---\n**Completion summary:**\n" : "**Completion summary:**\n";
                     t.description = t.description + sep + params.description_update;
@@ -4204,11 +4449,33 @@ export default function planPiExtension(pi) {
                 .filter((deviation) => (deviation.state === "approved" || deviation.state === "active") && deviation.temporaryTaskId === task.id)
                 .sort((left, right) => right.createdAt.localeCompare(left.createdAt))[0];
             if (completedDeviation)
-                await st.setWorkDeviationState(completedDeviation.id, "resolved", now);
+                await st.setWorkDeviationState(completedDeviation.id, "resume-required", now);
             await st.writeGenerated();
             if (!completedTask)
                 return { content: [{ type: "text", text: `Task not found: ${params.taskId}` }], details: {} };
-            return { content: [{ type: "text", text: `✅ Task completed: ${completedTask.id} — ${completedTask.title} (done)${clearedRef ? ` — phase handoff archived (${clearedRef})` : ""}` }], details: completedTask };
+            let resumeNotice = "";
+            let resumeRequired;
+            if (completedDeviation) {
+                const resume = findTaskByRef(await st.loadAllPhases(), features, completedDeviation.resumeTaskId);
+                if (resume) {
+                    const snapshot = resume.task.pauseSnapshot ?? completedDeviation.snapshot;
+                    const resumeRef = `${formatPhaseRef(resume.phase.number, featureNumberOfPhase(resume.phase, features))}/T${String(resume.task.number).padStart(3, "0")}`;
+                    resumeNotice = [
+                        "",
+                        `↩️ RESUME REQUIRED: ${resumeRef} — ${resume.task.title}`,
+                        snapshot ? `Paused because: ${snapshot.reason}` : "Return to the preserved task before selecting new priority work.",
+                        snapshot ? `Resume from: ${snapshot.resumeLocation}` : "",
+                        snapshot ? `How to resume: ${snapshot.howToResume}` : "",
+                        `Next action: task_start ${resumeRef}`,
+                    ].filter(Boolean).join("\n");
+                    resumeRequired = { taskId: resume.task.id, phaseId: resume.phase.id, snapshot };
+                }
+            }
+            const completedRef = `${formatPhaseRef(found.phase.number, featureNumberOfPhase(found.phase, features))}/T${String(completedTask.number).padStart(3, "0")}`;
+            return {
+                content: [{ type: "text", text: `✅ Task completed: ${completedRef} — ${completedTask.title} (done)${clearedRef ? ` — phase handoff archived (${clearedRef})` : ""}${resumeNotice}` }],
+                details: resumeRequired ? { task: completedTask, resumeRequired } : completedTask,
+            };
         },
     });
     pi.registerTool({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agent-plan/pi-adapter",
-  "version": "0.2.20",
+  "version": "0.2.22",
   "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.20",
-    "@agent-plan/server": "0.2.20"
+    "@agent-plan/core": "0.2.22",
+    "@agent-plan/server": "0.2.22"
   },
   "devDependencies": {
     "@earendil-works/pi-coding-agent": "^0.74.0",