codecartographer-pi 0.25.0 → 0.26.0

package/dist/mcp-server/server.js

@@ -17,7 +17,7 @@ import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js"
 import { CallToolRequestSchema, ErrorCode, ListToolsRequestSchema, McpError, } from "@modelcontextprotocol/sdk/types.js";
 import { mkdir, readFile, readdir, rename, writeFile } from "node:fs/promises";
 import { basename, isAbsolute, join } from "node:path";
-import { buildPhasePrompt, buildSkillPrompt, buildValidationSummary, BROADSIDE_DIR, broadsideDirFor, BROADSIDE_LENS_IDS, BROADSIDE_SKILL_NAME, BroadsideConfigError, defaultBroadsideConfig, backupWorkspaceState, canonicalPath, copyPackagedWorkspace, collectResultText, completeValidatedPhase, computePerPhaseTotals, computeTotals, createEmptyStatus, DEFAULT_PIPELINE_PATH, deriveSlug, discoverLibrary, describeScaffoldStaleness, detectProvenanceConflicts, estimateSubmitText, getLens, describeDanglingCarryForward, describeMissingCompletedOutputs, describeStuckPipeline, getPipelineLabel, getWorkspaceState, isValidSlug, isWithinPathResolved, listBatchModels, listEntries, listGuideTopics, loadBroadsideConfig, modelsText, readGuide, listMissingCompletedOutputs, listSkillNames, resolvePipelineOutcome, resolveSkillName, loadCodecartoConfig, loadUsage, loadYamlFile, normalizeForComparison, PACKAGE_VERSION, packagedWorkspaceDir, pathExists, PhasePreflightError, previewPublishVersion, publishEntry, reindex as libraryReindex, refreshScaffold, resolvePhase, resolvePipelineChoice, readBroadsideSkill, runBroadsideCollect, runBroadsideStatus, runBroadsideSubmit, runBroadsideVerify, verifyResultText, seedOrchestratorFiles, statusLineWriter, statusText, stringifySimpleYaml, switchPipeline, validatePhaseOutput, writeLibraryConfig, writeDashboard, } from "../core/index.js";
+import { buildPhasePrompt, buildSkillPrompt, buildValidationSummary, BROADSIDE_DIR, broadsideDirFor, BROADSIDE_LENS_IDS, BROADSIDE_SKILL_NAME, BroadsideConfigError, defaultBroadsideConfig, backupWorkspaceState, canonicalPath, copyPackagedWorkspace, collectResultText, completeValidatedPhase, computePerPhaseTotals, computeTotals, createEmptyStatus, DEFAULT_PIPELINE_PATH, deriveSlug, discoverLibrary, describeScaffoldStaleness, detectProvenanceConflicts, estimateSubmitText, getLens, describeDanglingCarryForward, describeMissingCompletedOutputs, describeStuckPipeline, getPipelineLabel, getWorkspaceState, isValidSlug, isWithinPathResolved, listBatchModels, listEntriesWithIndexState, listGuideTopics, loadBroadsideConfig, modelsText, readGuide, listMissingCompletedOutputs, listSkillNames, resolvePipelineOutcome, resolveSkillName, loadCodecartoConfig, loadUsage, loadYamlFile, normalizeForComparison, PACKAGE_VERSION, packagedWorkspaceDir, pathExists, PhasePreflightError, previewPublishVersion, publishEntry, reindex as libraryReindex, refreshScaffold, resolvePhase, resolvePipelineChoice, readBroadsideSkill, runBroadsideCollect, runBroadsideStatus, runBroadsideSubmit, runBroadsideVerify, verifyResultText, seedOrchestratorFiles, statusLineWriter, statusText, stringifySimpleYaml, switchPipeline, validatePhaseOutput, writeLibraryConfig, writeDashboard, } from "../core/index.js";
 import { applyAmendment } from "../core/amendment.js";
 import { appendUsageRun } from "../core/usage.js";
 import { initLibrary } from "../core/library.js";
@@ -545,7 +545,10 @@ export async function readSpecArg(args, allowedRoots) {
         if (!withinAllowed.some((result) => result)) {
             throw new McpError(ErrorCode.InvalidParams, `spec_path must be within the workspace (.codecarto/) or the configured library path. Got: ${args.spec_path}`);
         }
-        return readFile(args.spec_path, "utf8");
+        // The path that was checked is the path that is read: the original
+        // string could be re-pointed through a symlink between the check and
+        // the read; the canonical one cannot (#362).
+        return readFile(resolvedSpecPath, "utf8");
     }
     throw new McpError(ErrorCode.InvalidParams, "Either spec (inline content) or spec_path (absolute file path) is required");
 }
@@ -749,7 +752,7 @@ export async function handleLibraryList(args) {
         filter.slug = args.slug;
     if (typeof args.source_repo === "string" && args.source_repo !== "")
         filter.source_repo = args.source_repo;
-    const entries = await listEntries(libraryPath, filter);
+    const { entries, indexState } = await listEntriesWithIndexState(libraryPath, filter);
     // Computed over the listed entries only, read-only: one readdir per entry
     // plus one small metadata read per older version.
     const conflicts = await detectProvenanceConflicts(libraryPath, entries);
@@ -766,11 +769,17 @@ export async function handleLibraryList(args) {
             }),
             ...provenanceConflictLines(conflicts),
         ].join("\n");
-    return textResult(summary, {
+    // A list never writes the index (#357): say when it answered from the
+    // entry directories instead, so the caller can run a reindex.
+    const indexNote = indexState === "indexed"
+        ? ""
+        : `\nindex.yaml is ${indexState}; this listing was built from the entry directories. Run codecarto_library_reindex to rebuild it.`;
+    return textResult(`${summary}${indexNote}`, {
         libraryPath,
         libraryName: marker.name,
         namespaced: marker.namespaced,
         count: entries.length,
+        indexState,
         entries,
         provenance_conflicts: conflicts,
     });
@@ -996,6 +1005,7 @@ export async function handleRefreshScaffold(args) {
         written: result.written,
         scaffoldVersionBefore: result.scaffoldVersionBefore,
         scaffoldVersionAfter: result.scaffoldVersionAfter,
+        threadLogEntryAppended: result.threadLogEntryAppended,
     });
 }
 export async function handleAmend(args) {
@@ -1208,6 +1218,7 @@ export async function handleBroadside(args) {
             candidates: verified.candidates,
             totalCost: verified.totalCost,
             ...(verified.stoppedByCost && { stoppedByCost: true }),
+            redactedValues: verified.redactedValues,
             findings: verified.findings,
         });
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codecartographer-pi",
-  "version": "0.25.0",
+  "version": "0.26.0",
   "mcpName": "io.github.HuginnIndustries/codecartographer",
   "description": "Turn an unfamiliar codebase into a validated reimplementation spec, then synthesize confirmed specs and a product vision into a traceable plan.",
   "type": "module",