@hanna84/mcp-writing 3.29.2 → 3.29.3

package/CHANGELOG.md

@@ -4,9 +4,16 @@ All notable changes to this project will be documented in this file. Dates are d
 
 Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
+#### [v3.29.3](https://github.com/hannasdev/mcp-writing/compare/v3.29.2...v3.29.3)
+
+- fix: tighten MCP response contracts [`#242`](https://github.com/hannasdev/mcp-writing/pull/242)
+
 #### [v3.29.2](https://github.com/hannasdev/mcp-writing/compare/v3.29.1...v3.29.2)
 
+> 11 June 2026
+
 - ci: skip PR template check for Dependabot [`#241`](https://github.com/hannasdev/mcp-writing/pull/241)
+- Release 3.29.2 [`a26b7cc`](https://github.com/hannasdev/mcp-writing/commit/a26b7cc3d66d250fd9f30bd6d9ddcad49187008c)
 
 #### [v3.29.1](https://github.com/hannasdev/mcp-writing/compare/v3.29.0...v3.29.1)
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hanna84/mcp-writing",
-  "version": "3.29.2",
+  "version": "3.29.3",
   "description": "MCP service for AI-assisted reasoning and editing on long-form fiction projects",
   "homepage": "https://hannasdev.github.io/mcp-writing/",
   "type": "module",

package/src/core/canonical-target-resolution.js

@@ -114,6 +114,7 @@ function buildResolutionFailure({ targetKind, input, projectId, universeId, cand
     input,
     project_id: projectId,
     ...(universeId !== undefined ? { universe_id: universeId } : {}),
+    suggestions_available: cappedCandidates.length > 0,
     candidate_matches: cappedCandidates,
     next_step: nextStepForTargetKind(targetKind),
   };

package/src/index.js

@@ -134,7 +134,10 @@ function paginateRows(rows, { page, pageSize, forcePagination = false }) {
 }
 
 function jsonResponse(payload) {
-  return { content: [{ type: "text", text: JSON.stringify(payload, null, 2) }] };
+  return {
+    content: [{ type: "text", text: JSON.stringify(payload, null, 2) }],
+    ...(payload?.ok === false ? { isError: true } : {}),
+  };
 }
 
 function errorResponse(code, message, details) {
@@ -612,6 +615,7 @@ function createMcpServer() {
     async () => {
       return jsonResponse({
         server_version: MCP_SERVER_VERSION,
+        transport: MCP_TRANSPORT === "stdio" ? "stdio" : "http",
         sync_dir: SYNC_DIR_ABS,
         db_path: DB_PATH_DISPLAY,
         db_migration_warnings: DB_STARTUP_WARNINGS,

package/src/sync/sync.js

@@ -1688,7 +1688,7 @@ export function observeOrphanedSidecars(syncDir, { indexedSceneIds = new Set() }
 }
 
 const WARNING_TYPE_LABELS = {
-  no_scene_id: "Skipped (no scene_id)",
+  no_scene_id: "Ignored non-manuscript files",
   duplicate_scene_id: "Duplicate scene_id",
   path_metadata_mismatch: "Path/metadata mismatch",
   chapter_structure: "Chapter structure",
@@ -1703,7 +1703,7 @@ const WARNING_TYPE_LABELS = {
 };
 
 const WARNING_PATTERNS = [
-  { type: "no_scene_id",            re: /^Skipped \(no scene_id\):/  },
+  { type: "no_scene_id",            re: /^Ignored non-manuscript file \(no scene_id\):/  },
   { type: "duplicate_scene_id",     re: /^Duplicate scene_id/        },
   { type: "path_metadata_mismatch", re: /^Path\/metadata mismatch/   },
   { type: "chapter_structure",      re: /^(Chapter structure warning|Epigraph requires explicit chapter linkage|Epigraph references unknown chapter_id|Scene references unknown chapter_id|Ambiguous chapter linkage|Epigraph identity conflict|Managed structure sync ignored|Managed structure sync preserved canonical epigraph)/ },
@@ -1718,6 +1718,7 @@ const WARNING_PATTERNS = [
 ];
 
 const MAX_WARNING_EXAMPLES = 5;
+const INFO_WARNING_TYPES = new Set(["no_scene_id"]);
 
 export function buildWarningSummary(warnings) {
   const summary = {};
@@ -1725,7 +1726,14 @@ export function buildWarningSummary(warnings) {
     const firstLine = w.split("\n")[0];
     const match = WARNING_PATTERNS.find(p => p.re.test(firstLine));
     const type = match?.type ?? "other";
-    if (!summary[type]) summary[type] = { count: 0, examples: [] };
+    if (!summary[type]) {
+      summary[type] = {
+        count: 0,
+        examples: [],
+        severity: INFO_WARNING_TYPES.has(type) ? "info" : "warning",
+        label: WARNING_TYPE_LABELS[type] ?? type,
+      };
+    }
     summary[type].count++;
     if (summary[type].examples.length < MAX_WARNING_EXAMPLES) {
       summary[type].examples.push(firstLine);
@@ -1799,7 +1807,7 @@ export function syncAll(db, syncDir, { quiet = false, writable = false, relation
 
       if (!meta.scene_id && !chapterStructure.isEpigraph) {
         skipped++;
-        if (!quiet) warnings.push(`Skipped (no scene_id): ${structureObservation.relativePath}`);
+        if (!quiet) warnings.push(`Ignored non-manuscript file (no scene_id): ${structureObservation.relativePath}`);
         continue;
       }
 
@@ -1921,12 +1929,13 @@ export function syncAll(db, syncDir, { quiet = false, writable = false, relation
       `[mcp-writing] Sync complete: ${indexed} scenes indexed, ${staleMarked} scenes marked stale` +
       (epigraphsIndexed ? `, ${epigraphsIndexed} epigraphs indexed, ${epigraphsStaleMarked} epigraphs marked stale` : "") +
       (sidecarsMigrated ? `, ${sidecarsMigrated} sidecars auto-generated` : "") +
-      (skipped ? `, ${skipped} files skipped` : "") + "\n"
+      (skipped ? `, ${skipped} non-manuscript file(s) ignored` : "") + "\n"
     );
     for (const [type, entry] of Object.entries(warningSummary)) {
       const count = entry.count;
       const label = WARNING_TYPE_LABELS[type] ?? type;
-      process.stderr.write(`[mcp-writing] WARNING: ${label}: ${count} file(s). First example: ${entry.examples[0]}\n`);
+      const level = entry.severity === "info" ? "INFO" : "WARNING";
+      process.stderr.write(`[mcp-writing] ${level}: ${label}: ${count} file(s). First example: ${entry.examples[0]}\n`);
     }
   }
   return {

package/src/tools/search.js

@@ -485,13 +485,15 @@ export function registerSearchTools(s, {
         pageSize: page_size,
         forcePagination: rows.length > DEFAULT_METADATA_PAGE_SIZE,
       });
+      const resolvedFrom = mergeResolvedFilters(resolvedFilters);
 
       const payload = paged.paginated
         ? {
             results: paged.rows,
             ...paged.meta,
             warning,
-            resolved_filters: mergeResolvedFilters(resolvedFilters),
+            resolved_filters: resolvedFrom,
+            resolved_from: resolvedFrom,
             next_step: staleCount > 0
               ? "Touch stale scenes as you work and run enrich_scene(scene_id, project_id) to recover metadata parity incrementally."
               : undefined,
@@ -500,7 +502,8 @@ export function registerSearchTools(s, {
             results: rows,
             total_count: rows.length,
             warning,
-            resolved_filters: mergeResolvedFilters(resolvedFilters),
+            resolved_filters: resolvedFrom,
+            resolved_from: resolvedFrom,
             next_step: staleCount > 0
               ? "Touch stale scenes as you work and run enrich_scene(scene_id, project_id) to recover metadata parity incrementally."
               : undefined,

package/src/tools/sync.js

@@ -60,14 +60,22 @@ export function registerSyncTools(s, {
     }
     if (result.sidecarsMigrated) parts.push(`${result.sidecarsMigrated} sidecar(s) auto-generated from frontmatter.`);
     if (result.skipped) {
-      parts.push(`${result.skipped} file(s) skipped (no scene_id).`);
-      parts.push(`Tip: for raw Scrivener Draft exports, run src/scripts/import.js first, then run sync again.`);
+      parts.push(`${result.skipped} non-manuscript file(s) ignored because they did not declare scene_id.`);
+      parts.push(`Tip: if these are raw Scrivener Draft scene exports, run src/scripts/import.js first, then run sync again.`);
     }
     const summary = result.warningSummary;
     const summaryEntries = Object.entries(summary);
     if (summaryEntries.length) {
-      const lines = summaryEntries.map(([type, entry]) => `- ${type}: ${entry.count} (e.g. ${entry.examples[0]})`);
-      parts.push(`\n⚠️ Warning summary:\n` + lines.join("\n"));
+      const infoEntries = summaryEntries.filter(([, entry]) => entry.severity === "info");
+      const warningEntries = summaryEntries.filter(([, entry]) => entry.severity !== "info");
+      if (infoEntries.length) {
+        const lines = infoEntries.map(([type, entry]) => `- ${entry.label ?? type}: ${entry.count} (e.g. ${entry.examples[0]})`);
+        parts.push(`\nIgnored file summary:\n` + lines.join("\n"));
+      }
+      if (warningEntries.length) {
+        const lines = warningEntries.map(([type, entry]) => `- ${entry.label ?? type}: ${entry.count} (e.g. ${entry.examples[0]})`);
+        parts.push(`\nWarning summary:\n` + lines.join("\n"));
+      }
     }
     return { content: [{ type: "text", text: parts.join(" ") }] };
   });