@agent-plan/mcp 0.2.19-next.2 → 0.2.19-next.4

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":";AAm5BA,wBAAsB,gBAAgB,IAAI,OAAO,CAAC,IAAI,CAAC,CAItD"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":";AA45BA,wBAAsB,gBAAgB,IAAI,OAAO,CAAC,IAAI,CAAC,CAItD"}

package/dist/index.js

@@ -6,7 +6,7 @@ import { join } from "node:path";
 import { pathToFileURL } from "node:url";
 import { PlanStore, ExportService, withFeatureLock, needsMotivation, findPhaseByRef, buildRecap } from "@agent-plan/core";
 import { serve } from "@agent-plan/server";
-import { createChecklistItemId, createFeatureId, createPhaseId, createShortId, createTaskId, clampSlug, normalizeSlug, formatPhaseRef } from "@agent-plan/core/naming";
+import { createChecklistItemId, createFeatureId, createPhaseId, createShortId, createTaskId, clampSlug, normalizeSlug, formatPhaseRef, formatFeatureRef } from "@agent-plan/core/naming";
 const STATUS_VALUES = ["planned", "in-progress", "done", "blocked", "canceled", "rejected", "deferred", "waiting"];
 const PHASE_STATUS_VALUES = ["draft", "discovery", ...STATUS_VALUES];
 function text(textValue, structuredContent) {
@@ -189,7 +189,7 @@ server.registerTool("planner-init", {
     if (goal !== undefined)
         project.goal = goal.trim();
     await st.saveProject(project);
-    return writeAndSummarize(st, `.planner/ initialized for "${project.name}"`, { project });
+    return writeAndSummarize(st, `.planner/ initialized for "${project.name}"`);
 });
 server.registerTool("planner-show", {
     description: "Show the current planner overview.",
@@ -227,7 +227,7 @@ server.registerTool("planner-project-language", {
     if (chatLanguage !== undefined)
         project.chatLanguage = chatLanguage.trim();
     await st.saveProject(project);
-    return writeAndSummarize(st, `Saved language preferences: content=${project.contentLanguage || "(unset)"}, chat=${project.chatLanguage || "(unset)"}`, { project });
+    return writeAndSummarize(st, `Saved language preferences: content=${project.contentLanguage || "(unset)"}, chat=${project.chatLanguage || "(unset)"}`);
 });
 server.registerTool("planner-project-discuss", {
     description: "Record/update project-level discovery fields. This is the MCP equivalent of grouped project discuss without Pi UI prompts.",
@@ -258,7 +258,7 @@ server.registerTool("planner-project-discuss", {
     if (params.decisions !== undefined)
         project.decisions = params.decisions.map((entry) => entry.trim()).filter(Boolean);
     await st.saveProject(project);
-    return writeAndSummarize(st, `Project discussed/updated: ${project.name}`, { project });
+    return writeAndSummarize(st, `Project discussed/updated: ${project.name}`);
 });
 server.registerTool("planner-feature-list", {
     description: "List features with phase/task counts.",
@@ -312,7 +312,7 @@ server.registerTool("planner-feature-add", {
         doc.features.push(feature);
         return doc;
     });
-    return writeAndSummarize(st, `Feature created: ${feature.id} — ${feature.name}`, { feature });
+    return writeAndSummarize(st, `✅ Feature created: ${formatFeatureRef(feature.number)} — ${feature.name}${feature.shortId ? ` · ${feature.shortId}` : ""}`);
 });
 server.registerTool("planner-feature-show", {
     description: "Show a feature by id or name.",
@@ -369,7 +369,7 @@ server.registerTool("planner-feature-update", {
         return doc;
     });
     const result = updated.features.find((entry) => entry.id === feature.id);
-    return writeAndSummarize(st, `Feature updated: ${result.id} — ${result.name}`, { feature: result });
+    return writeAndSummarize(st, `✅ Feature updated: ${formatFeatureRef(result.number)} — ${result.name}${result.shortId ? ` · ${result.shortId}` : ""}`);
 });
 server.registerTool("planner-feature-delete", {
     description: "Delete a feature. By default, phases are unlinked rather than deleted.",
@@ -452,6 +452,8 @@ server.registerTool("planner-phase-add", {
             updatedAt: timestamp,
             handoff: "",
             handoffUpdatedAt: "",
+            handoffReadAt: "",
+            handoffHistory: [],
         };
         await st.savePhase(phase);
         if (feature) {
@@ -466,7 +468,7 @@ server.registerTool("planner-phase-add", {
     });
     if (!phase)
         return text("Phase creation failed.");
-    return writeAndSummarize(st, `Phase created: ${phase.id} — ${phase.title}`, { phase });
+    return writeAndSummarize(st, `✅ Phase created: ${formatPhaseRef(phase.number, feature?.number)} — ${phase.title}${phase.shortId ? ` · ${phase.shortId}` : ""}`);
 });
 server.registerTool("planner-phase-show", {
     description: "Show a phase by id or name.",
@@ -492,7 +494,8 @@ server.registerTool("planner-phase-discuss", {
     },
 }, async ({ phase: ref, ...updates }) => {
     const st = await requireStore();
-    const found = findPhaseByRef(await st.loadAllPhases(), (await st.loadFeatures()).features, ref);
+    const features = (await st.loadFeatures()).features;
+    const found = findPhaseByRef(await st.loadAllPhases(), features, ref);
     if (!found)
         return text(`Phase not found: ${ref}`);
     const phase = await st.updatePhase(found.id, (entry) => {
@@ -517,7 +520,7 @@ server.registerTool("planner-phase-discuss", {
         entry.updatedAt = nowISO();
         return entry;
     });
-    return writeAndSummarize(st, `Phase discussed/planned: ${phase.id} — ${phase.title}`, { phase });
+    return writeAndSummarize(st, `✅ Phase discussed/planned: ${formatPhaseRef(found.number, featureNumberOfPhase(found, features))} — ${phase.title}${phase.shortId ? ` · ${phase.shortId}` : ""}`);
 });
 server.registerTool("planner-phase-update", {
     description: "Update phase fields.",
@@ -531,7 +534,8 @@ server.registerTool("planner-phase-update", {
     },
 }, async ({ phase: ref, ...updates }) => {
     const st = await requireStore();
-    const found = findPhaseByRef(await st.loadAllPhases(), (await st.loadFeatures()).features, ref);
+    const features = (await st.loadFeatures()).features;
+    const found = findPhaseByRef(await st.loadAllPhases(), features, ref);
     if (!found)
         return text(`Phase not found: ${ref}`);
     const phase = await st.updatePhase(found.id, (entry) => {
@@ -548,7 +552,7 @@ server.registerTool("planner-phase-update", {
         entry.updatedAt = nowISO();
         return entry;
     });
-    return writeAndSummarize(st, `Phase updated: ${phase.id} — ${phase.title}`, { phase });
+    return writeAndSummarize(st, `✅ Phase updated: ${formatPhaseRef(found.number, featureNumberOfPhase(found, features))} — ${phase.title}${phase.shortId ? ` · ${phase.shortId}` : ""}`);
 });
 server.registerTool("planner-phase-delete", {
     description: "Delete a phase and unlink it from features.",
@@ -733,16 +737,22 @@ server.registerTool("planner-task-start", {
     inputSchema: { task: z.string().min(1) },
 }, async ({ task: ref }) => {
     const st = await requireStore();
-    // Hygiene notice (non-blocking): surface an existing handoff but never block
-    // task_start — the handoff is a captured-context artifact, not a lock (e.g. a
-    // small modification after writing a handoff should start without forcing
-    // deletion, which would lose context).
-    const handoffNotice = (await st.listHandoffs()).length > 0
-        ? "ℹ️  One or more phases have a pending handoff — if relevant to the task you're starting, read it with planner-handoff-show <ref>, then clear with planner-handoff-clear. Proceeding with task start.\n"
-        : "";
     const found = findTaskByRef(await st.loadAllPhases(), ref);
     if (!found)
         return text(`Task not found: ${ref}`);
+    // Auto-archive the handoff on THIS task's phase (reason: task-started) — the
+    // agent is now actively working, so the captured context is consumed. Other
+    // phases' handoffs are left untouched (informational, non-blocking).
+    const allHandoffs = await st.listHandoffs();
+    let handoffNotice = "";
+    if ((found.phase.handoff ?? "") !== "") {
+        await st.clearPhaseHandoff(found.phase.id, "task-started").catch(() => { });
+        handoffNotice = "📦 Archived handoff for this phase (task started) — recover via planner-handoff-list + .planner/handoff-archive/.\n";
+    }
+    const _otherHandoffs = allHandoffs.filter((h) => h.phaseId !== found.phase.id);
+    if (_otherHandoffs.length > 0) {
+        handoffNotice += `ℹ️  ${_otherHandoffs.length} other phase handoff(s) pending — review with planner-handoff-list if relevant.\n`;
+    }
     const timestamp = nowISO();
     let updatedTask;
     await st.updatePhase(found.phase.id, (phase) => {
@@ -870,8 +880,8 @@ server.registerTool("planner-handoff-clear", {
     const r = await resolvePhaseForHandoff(st, phaseRef);
     if (!r.ok)
         return text(`❌ ${r.error}`);
-    await st.clearPhaseHandoff(r.phase.id);
-    return text(`✅ Cleared handoff on ${r.compositeRef}. (handoffUpdatedAt preserved as audit)`, { phaseRef: r.compositeRef, phaseId: r.phase.id });
+    await st.clearPhaseHandoff(r.phase.id, "manual");
+    return text(`✅ Cleared handoff on ${r.compositeRef} (handoffUpdatedAt preserved as audit; archived to .planner/handoff-archive/).`, { phaseRef: r.compositeRef, phaseId: r.phase.id });
 });
 server.registerTool("planner-web", {
     description: "Manage the planner web dashboard (start/status/stop) from MCP stdio. Runs an in-process HTTP+WS server bound to LAN; it lives as long as this MCP process. plan-root comes from AGENT_PLAN_ROOT or cwd()/.planner.",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agent-plan/mcp",
-  "version": "0.2.19-next.2",
+  "version": "0.2.19-next.4",
   "private": false,
   "description": "MCP server for Agent Plan, exposing planner-* tools over stdio.",
   "license": "MIT",
@@ -41,8 +41,8 @@
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.17.0",
     "zod": "^3.24.0",
-    "@agent-plan/core": "0.2.19-next.2",
-    "@agent-plan/server": "0.2.19-next.2"
+    "@agent-plan/core": "0.2.19-next.4",
+    "@agent-plan/server": "0.2.19-next.4"
   },
   "scripts": {
     "build": "tsc -p tsconfig.json",