@agent-plan/mcp 0.2.19-next.21 → 0.2.19-next.23

package/dist/index.d.ts.map

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

package/dist/index.js

@@ -4,7 +4,7 @@ import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js"
 import * as z from "zod/v4";
 import { join } from "node:path";
 import { pathToFileURL } from "node:url";
-import { PlanStore, ExportService, withFeatureLock, needsMotivation, findPhaseByRef, findTaskByRef, buildRecap, migrateToGlobalSequence, addChecklistItem, removeChecklistItem, toggleChecklistItem, buildPhaseContextBlock, recommendNextTask } from "@agent-plan/core";
+import { PlanStore, ExportService, withFeatureLock, needsMotivation, findPhaseByRef, findTaskByRef, buildRecap, migrateToGlobalSequence, addChecklistItem, removeChecklistItem, toggleChecklistItem, buildPhaseContextBlock, checkExplicitTaskStart, recommendNextTask } from "@agent-plan/core";
 import { serve } from "@agent-plan/server";
 import { createChecklistItemId, createFeatureId, createPhaseId, createShortId, createTaskId, clampSlug, normalizeSlug, formatPhaseRef, formatFeatureRef, validateResolvedTarget } from "@agent-plan/core/naming";
 const STATUS_VALUES = ["planned", "in-progress", "done", "blocked", "canceled", "rejected", "deferred", "waiting"];
@@ -649,7 +649,18 @@ server.registerTool("planner-phase-show", {
     const requirementsBlock = reqCount > 0
         ? `\n\nLinked requirements:\n${linkedRequirements.map((requirement) => `- ${requirement.title} (${requirement.status})`).join("\n")}`
         : "";
-    return text(full ? `${summary}\n\n${phase.description || ""}${requirementsBlock}` : summary, { linkedRequirements });
+    return text(full ? `${summary}\n\n${phase.description || ""}${requirementsBlock}` : summary, {
+        phase: {
+            ref: formatPhaseRef(phase.number, featureNumberOfPhase(phase, features)),
+            shortId: phase.shortId,
+            title: phase.title,
+            summary: phase.summary,
+            status: phase.status,
+            taskCount: phase.tasks.length,
+            ...(full ? { description: phase.description } : {}),
+        },
+        linkedRequirements,
+    });
 });
 server.registerTool("planner-phase-discuss", {
     description: "Persist phase discovery fields and mark phase planned.",
@@ -1086,15 +1097,11 @@ server.registerTool("planner-task-start", {
     if (found.task.status === "done")
         return text(`Task ${found.task.id} is done. Use planner-task-update to reopen.`);
     const project = await st.loadProject();
-    const selection = recommendNextTask(features, await st.loadAllPhases(), project.workDeviations);
-    if (!selection.candidate || selection.candidate.task.id !== found.task.id) {
-        const recommended = selection.candidate ? ` Start ${selection.candidate.task.id} instead, or record a user-approved planner-task-deviation first.` : "";
-        return text(`Task start denied: ${selection.reason}.${recommended}`, {
-            kind: selection.kind,
-            reason: selection.reason,
-            recommendedTaskId: selection.candidate?.task.id,
-        });
-    }
+    const phases = await st.loadAllPhases();
+    const eligibility = checkExplicitTaskStart(features, phases, found.task.id, project.workDeviations);
+    if (!eligibility.eligible)
+        return text(`Task start denied: ${eligibility.reason}`, { reason: eligibility.reason });
+    const selection = recommendNextTask(features, phases, project.workDeviations);
     // Assemble the mandatory parent context before changing task lifecycle state.
     const parentFeature = found.phase.featureId ? features.find((f) => f.id === found.phase.featureId) : undefined;
     const [phaseWithRequirements, featureRequirements] = await Promise.all([
@@ -1105,6 +1112,13 @@ server.registerTool("planner-task-start", {
     // A pending handoff is context, not a lock: task start RETAINS it. It is
     // archived only when the phase completes (auto) or the user explicitly
     // clears it.
+    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 !== found.task.id
+            ? selection.kind === "active"
+                ? `\n\n⚠️ Active-task advisory: ${taskCompositeRef(selection.candidate.task, selection.candidate.phase, features)} is already in progress. Explicit task request honored.`
+                : `\n\n⚠️ Priority advisory: ${taskCompositeRef(selection.candidate.task, selection.candidate.phase, features)} is the automatic recommendation. Explicit task request honored.`
+            : "";
     const timestamp = nowISO();
     let updatedTask;
     await st.updatePhase(found.phase.id, (phase) => {
@@ -1129,11 +1143,11 @@ server.registerTool("planner-task-start", {
         return phase;
     });
     await st.syncTaskStatusRollup(found.phase.id);
-    if (selection.deviation?.temporaryTaskId === found.task.id && selection.deviation.state === "approved") {
-        await st.setWorkDeviationState(selection.deviation.id, "active", timestamp);
-    }
+    const approvedDeviation = project.workDeviations.find((deviation) => deviation.temporaryTaskId === found.task.id && deviation.state === "approved");
+    if (approvedDeviation)
+        await st.setWorkDeviationState(approvedDeviation.id, "active", timestamp);
     const t = updatedTask ?? found.task;
-    return writeAndSummarize(st, `✅ Task started: ${taskCompositeRef(t, found.phase, features)} — ${t.title} (in-progress)${t.shortId ? ` · ${t.shortId}` : ""}${phaseContext}`);
+    return writeAndSummarize(st, `✅ Task started: ${taskCompositeRef(t, found.phase, features)} — ${t.title} (in-progress)${t.shortId ? ` · ${t.shortId}` : ""}${phaseContext}${advisory}`);
 });
 server.registerTool("planner-task-complete", {
     description: "Set a task to done. Fails if checklist is incomplete unless force=true.",

package/package.json

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