codecartographer-pi 0.10.0 → 0.12.0

package/dist/mcp-server/server.js

@@ -15,7 +15,7 @@ import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js"
 import { CallToolRequestSchema, ErrorCode, ListToolsRequestSchema, McpError, } from "@modelcontextprotocol/sdk/types.js";
 import { cp, mkdir, readFile, rm, writeFile } from "node:fs/promises";
 import { basename, isAbsolute, join } from "node:path";
-import { buildPhasePrompt, buildSkillPrompt, buildThreadLogEntry, buildValidationSummary, canonicalPath, closeoutFileName, createEmptyStatus, dateOnly, DEFAULT_PIPELINE_PATH, deriveSlug, discoverLibrary, ensureCloseoutStub, getNextEligiblePhase, getPipelineLabel, getWorkspaceState, isValidSlug, listEntries, listSkillNames, loadCodecartoConfig, loadYamlFile, normalizeForComparison, normalizeStatus, PACKAGE_VERSION, packagedWorkspaceDir, pathExists, publishEntry, reindex as libraryReindex, resolvePhase, resolvePipelineChoice, stringifySimpleYaml, uniqueStrings, updateStatusAtomically, validatePhaseOutput, } from "../core/index.js";
+import { buildPhasePrompt, buildSkillPrompt, buildValidationSummary, canonicalPath, completeValidatedPhase, createEmptyStatus, DEFAULT_PIPELINE_PATH, deriveSlug, discoverLibrary, getNextEligiblePhase, getPipelineLabel, getWorkspaceState, isValidSlug, listEntries, listSkillNames, loadCodecartoConfig, loadYamlFile, normalizeForComparison, PACKAGE_VERSION, packagedWorkspaceDir, pathExists, PhasePreflightError, publishEntry, reindex as libraryReindex, resolvePhase, resolvePipelineChoice, stringifySimpleYaml, validatePhaseOutput, } from "../core/index.js";
 // ---------- input helpers ----------
 async function validateCwd(cwd) {
     if (typeof cwd !== "string" || !cwd.trim()) {
@@ -46,6 +46,17 @@ function textResult(text, structured) {
         result.structuredContent = structured;
     return result;
 }
+async function buildMcpPhasePrompt(state, phase, forced) {
+    try {
+        return await buildPhasePrompt(state, phase, forced);
+    }
+    catch (error) {
+        if (error instanceof PhasePreflightError) {
+            throw new McpError(ErrorCode.InvalidRequest, error.message);
+        }
+        throw error;
+    }
+}
 // ---------- handlers ----------
 export async function handleInit(args) {
     const cwd = await validateCwd(args.cwd);
@@ -101,12 +112,16 @@ export async function handleStatus(args) {
     const completed = state.pipeline.phase_order.filter((id) => state.status.phases[id]?.status === "complete").length;
     const totalCarryForward = Object.values(state.status.phases).reduce((sum, phase) => sum + (phase.carry_forward?.length ?? 0), 0);
     const currentOpenQuestions = currentPhase === "complete" ? 0 : state.status.phases[currentPhase]?.open_questions.length ?? 0;
+    const terminalOpenQuestions = Object.values(state.status.phases).reduce((sum, phase) => sum + (phase.open_questions?.length ?? 0), 0);
+    const postPipelinePending = state.status.post_pipeline.filter((entry) => entry.status !== "resolved").length;
     const summary = [
         `Phase: ${currentPhase}`,
+        `Pipeline state: ${currentPhase === "complete" ? "complete" : "in progress"}`,
         `Pipeline: ${getPipelineLabel(state.status.pipeline)} (${state.status.pipeline})`,
         `Progress: ${completed}/${state.pipeline.phase_order.length} complete`,
-        `Open questions (current phase): ${currentOpenQuestions}`,
-        `Carry-forward (all phases): ${totalCarryForward}`,
+        `Open questions (terminal unresolved): ${terminalOpenQuestions}`,
+        `Carry-forward (pipeline phases): ${totalCarryForward}`,
+        `Post-pipeline work: ${postPipelinePending} pending`,
         `Next: ${state.status.next_actions[0] ?? (nextPhase ? `Begin ${nextPhase.id}` : "All phases complete.")}`,
     ].join("\n");
     return textResult(summary, {
@@ -116,7 +131,9 @@ export async function handleStatus(args) {
         completed,
         total: state.pipeline.phase_order.length,
         openQuestionsCurrentPhase: currentOpenQuestions,
+        openQuestionsTerminal: terminalOpenQuestions,
         carryForwardTotal: totalCarryForward,
+        postPipelinePending,
         nextActions: state.status.next_actions,
     });
 }
@@ -129,7 +146,7 @@ export async function handleNext(args) {
             complete: true,
         });
     }
-    const prompt = await buildPhasePrompt(state, phase, false);
+    const prompt = await buildMcpPhasePrompt(state, phase, false);
     return textResult(prompt, { phase: phase.id, forced: false });
 }
 export async function handlePhase(args) {
@@ -142,7 +159,7 @@ export async function handlePhase(args) {
     if (!phase) {
         throw new McpError(ErrorCode.InvalidParams, `Unknown phase: ${args.phase}`);
     }
-    const prompt = await buildPhasePrompt(state, phase, true);
+    const prompt = await buildMcpPhasePrompt(state, phase, true);
     return textResult(prompt, { phase: phase.id, forced: true });
 }
 export async function handleValidate(args) {
@@ -172,70 +189,9 @@ export async function handleComplete(args) {
     if (validation.overall === "FAIL" || validation.overall === "MISSING") {
         throw new McpError(ErrorCode.InvalidRequest, `Cannot complete ${validation.phaseId}: validation is ${validation.overall}.\n${buildValidationSummary(validation).join("\n")}`);
     }
-    const completionTimestamp = new Date().toISOString();
-    const updatedState = await updateStatusAtomically(cwd, (lockedState) => {
-        const phase = resolvePhase(lockedState, validation.phaseId);
-        if (!phase?.primary_output) {
-            throw new Error(`Phase ${validation.phaseId} is missing primary_output.`);
-        }
-        const nextStatus = normalizeStatus(lockedState.status, lockedState.pipeline, lockedState.status.pipeline, lockedState.cwd);
-        const existingPhase = nextStatus.phases[validation.phaseId] ?? {
-            status: "pending",
-            owner_notes: [],
-            outputs_present: [],
-            open_questions: [],
-            carry_forward: [],
-        };
-        const gapEntries = validation.rows
-            .filter((row) => row.result.toUpperCase().includes("PARTIAL"))
-            .map((row) => ({
-            kind: "needs-maintainer-decision",
-            description: row.criterion || "Partial validation gap",
-            deferred_reason: row.evidence || "Marked PARTIAL by validation",
-        }));
-        const mergedOpenQuestions = [...existingPhase.open_questions];
-        for (const candidate of gapEntries) {
-            const dupe = mergedOpenQuestions.some((entry) => entry.description === candidate.description && entry.deferred_reason === candidate.deferred_reason);
-            if (!dupe)
-                mergedOpenQuestions.push(candidate);
-        }
-        nextStatus.phases[validation.phaseId] = {
-            status: "complete",
-            owner_notes: uniqueStrings([
-                ...existingPhase.owner_notes,
-                `Completed via codecarto_complete on ${completionTimestamp}.`,
-                `Primary output: .codecarto/${validation.primaryOutput}`,
-                `Validation: ${validation.overall}`,
-            ]).slice(-3),
-            outputs_present: uniqueStrings([...existingPhase.outputs_present, validation.primaryOutput]),
-            open_questions: mergedOpenQuestions,
-            carry_forward: existingPhase.carry_forward ?? [],
-        };
-        nextStatus.last_updated = completionTimestamp;
-        const updatedWorkspaceState = {
-            ...lockedState,
-            status: nextStatus,
-        };
-        const nextEligible = getNextEligiblePhase(updatedWorkspaceState);
-        nextStatus.current_phase = nextEligible?.id ?? "complete";
-        nextStatus.next_actions = nextEligible
-            ? [`Begin ${nextEligible.id} phase by producing ${nextEligible.primary_output ?? `findings/${nextEligible.id}/`}`]
-            : ["All phases complete. Review findings, open questions, and downstream implementation notes."];
-        return {
-            state: { ...updatedWorkspaceState, status: nextStatus },
-            threadLogEntry: buildThreadLogEntry(validation.phaseId, validation, completionTimestamp),
-        };
+    const { updatedState, closeoutNotice } = await completeValidatedPhase(cwd, validation, "codecarto_complete").catch((error) => {
+        throw new McpError(ErrorCode.InvalidParams, error instanceof Error ? error.message : String(error));
     });
-    let closeoutNotice;
-    try {
-        const created = await ensureCloseoutStub(updatedState.workspaceDir, validation.phaseId, completionTimestamp);
-        if (created) {
-            closeoutNotice = `Closeout stub created: .codecarto/closeouts/${closeoutFileName(dateOnly(completionTimestamp), validation.phaseId)} (fill it in)`;
-        }
-    }
-    catch (error) {
-        closeoutNotice = `Closeout stub not created: ${error instanceof Error ? error.message : String(error)}`;
-    }
     const lines = [
         `Marked ${validation.phaseId} complete (validation: ${validation.overall}).`,
         `Next phase: ${updatedState.status.current_phase}`,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codecartographer-pi",
-  "version": "0.10.0",
+  "version": "0.12.0",
   "description": "CodeCartographer packaged for Pi as an extension-driven workflow wrapper.",
   "type": "module",
   "keywords": [
@@ -37,13 +37,14 @@
     "build": "tsc",
     "prepublishOnly": "npm run build",
     "test": "node --experimental-strip-types --disable-warning=ExperimentalWarning --test tests/*.test.mjs",
-    "smoke": "node scripts/smoke-mcp.mjs"
+    "smoke": "node scripts/smoke-mcp.mjs",
+    "demo:synthesis": "npm run build && node scripts/create-synthesis-demo.mjs"
   },
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.29.0"
   },
   "peerDependencies": {
-    "@earendil-works/pi-coding-agent": "~0.74.0",
+    "@earendil-works/pi-coding-agent": "^0.80.10",
     "@sinclair/typebox": "*"
   },
   "pi": {