@hanna84/mcp-writing 3.9.3 → 3.9.4

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.9.4](https://github.com/hannasdev/mcp-writing/compare/v3.9.3...v3.9.4)
+
+- refactor(structure): centralize scene structure patches [`#200`](https://github.com/hannasdev/mcp-writing/pull/200)
+
 #### [v3.9.3](https://github.com/hannasdev/mcp-writing/compare/v3.9.2...v3.9.3)
 
+> 17 May 2026
+
 - refactor(structure): extract target architecture M1 boundaries [`#199`](https://github.com/hannasdev/mcp-writing/pull/199)
+- Release 3.9.3 [`55c87ea`](https://github.com/hannasdev/mcp-writing/commit/55c87ea405841153765144d32c57803b6139a103)
 
 #### [v3.9.2](https://github.com/hannasdev/mcp-writing/compare/v3.9.1...v3.9.2)
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hanna84/mcp-writing",
-  "version": "3.9.3",
+  "version": "3.9.4",
   "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/structure/structure-inference.js

@@ -109,24 +109,35 @@ export function inferChapterStructureFromPath(syncDir, filePath, meta = {}) {
   };
 }
 
-export function normalizeSceneMetaForPath(syncDir, filePath, meta = {}) {
+export function buildSceneStructurePatch(syncDir, filePath, meta = {}, { chapter } = {}) {
   const derived = inferScenePositionFromPath(syncDir, filePath);
   const chapterStructure = inferChapterStructureFromPath(syncDir, filePath, meta);
-  const normalized = { ...meta };
+  const patch = {};
 
-  if (derived.part !== null) normalized.part = derived.part;
-  if (derived.chapter !== null) normalized.chapter = derived.chapter;
+  if (derived.part !== null) patch.part = derived.part;
+  if (derived.chapter !== null) patch.chapter = derived.chapter;
   if (chapterStructure.chapter?.chapter_id) {
-    normalized.chapter_id = chapterStructure.chapter.chapter_id;
-    normalized.chapter = chapterStructure.chapter.sort_index;
-    normalized.chapter_title = chapterStructure.chapter.title;
+    patch.chapter_id = chapterStructure.chapter.chapter_id;
+    patch.chapter = chapterStructure.chapter.sort_index;
+    patch.chapter_title = chapterStructure.chapter.title;
   }
   if (chapterStructure.role) {
-    normalized.scene_role = chapterStructure.role;
+    patch.scene_role = chapterStructure.role;
+  }
+  if (chapter !== undefined) {
+    if (chapter === null) {
+      patch.chapter_id = null;
+      patch.chapter = null;
+      patch.chapter_title = null;
+    } else {
+      patch.chapter_id = chapter.chapter_id;
+      patch.chapter = chapter.sort_index;
+      patch.chapter_title = chapter.title ?? null;
+    }
   }
 
   return {
-    meta: normalized,
+    patch,
     derived,
     chapterStructure,
     mismatches: {
@@ -135,3 +146,15 @@ export function normalizeSceneMetaForPath(syncDir, filePath, meta = {}) {
     },
   };
 }
+
+export function applySceneStructurePatch(syncDir, filePath, meta = {}, options = {}) {
+  const plan = buildSceneStructurePatch(syncDir, filePath, meta, options);
+  return {
+    ...plan,
+    meta: { ...meta, ...plan.patch },
+  };
+}
+
+export function normalizeSceneMetaForPath(syncDir, filePath, meta = {}) {
+  return applySceneStructurePatch(syncDir, filePath, meta);
+}

package/src/sync/sync.js

@@ -3,6 +3,8 @@ import path from "node:path";
 import matter from "gray-matter";
 import yaml from "js-yaml";
 import {
+  applySceneStructurePatch,
+  buildSceneStructurePatch,
   inferChapterStructureFromPath,
   inferScenePositionFromPath,
   normalizeSceneMetaForPath,
@@ -16,6 +18,8 @@ import { indexCanonicalEpigraph } from "../structure/epigraph-indexing.js";
 const { load: parseYaml, dump: stringifyYaml } = yaml;
 
 export {
+  applySceneStructurePatch,
+  buildSceneStructurePatch,
   inferChapterStructureFromPath,
   inferScenePositionFromPath,
   normalizeSceneMetaForPath,

package/src/tools/metadata.js

@@ -1,7 +1,7 @@
 import { z } from "zod";
 import fs from "node:fs";
 import matter from "gray-matter";
-import { readMeta, writeMeta, indexSceneFile, normalizeSceneMetaForPath } from "../sync/sync.js";
+import { readMeta, writeMeta, indexSceneFile, applySceneStructurePatch } from "../sync/sync.js";
 import { validateProjectId, validateUniverseId } from "../sync/importer.js";
 import { resolveValidatedChapterFilter } from "../core/chapter-resolution.js";
 import {
@@ -545,6 +545,7 @@ export function registerMetadataTools(s, {
       try {
         const { meta } = readMeta(scene.file_path, SYNC_DIR, { writable: true });
         const nextFields = { ...fields };
+        let chapter = undefined;
 
         if (fields.chapter_id === null && fields.chapter !== undefined) {
           return errorResponse(
@@ -559,8 +560,20 @@ export function registerMetadataTools(s, {
         }
 
         if (fields.chapter_id === null) {
-          nextFields.chapter = null;
-          nextFields.chapter_title = null;
+          const structurePlan = applySceneStructurePatch(SYNC_DIR, scene.file_path, meta);
+          if (structurePlan.derived.chapter !== null || structurePlan.chapterStructure.chapter?.chapter_id) {
+            return errorResponse(
+              "VALIDATION_ERROR",
+              "chapter_id cannot be cleared for a scene whose file path implies a chapter.",
+              {
+                project_id,
+                scene_id,
+                chapter_id: null,
+                path_chapter: structurePlan.chapterStructure.chapter?.chapter_id ?? structurePlan.derived.chapter,
+              }
+            );
+          }
+          chapter = null;
         } else if (fields.chapter_id !== undefined || fields.chapter !== undefined) {
           const resolvedChapterFilter = resolveValidatedChapterFilter(db, {
             projectId: project_id,
@@ -594,12 +607,10 @@ export function registerMetadataTools(s, {
             );
           }
 
-          nextFields.chapter_id = resolvedChapter.chapter_id;
-          nextFields.chapter = resolvedChapter.sort_index;
-          nextFields.chapter_title = resolvedChapter.title ?? null;
+          chapter = resolvedChapter;
         }
 
-        const updated = normalizeSceneMetaForPath(SYNC_DIR, scene.file_path, { ...meta, ...nextFields }).meta;
+        const updated = applySceneStructurePatch(SYNC_DIR, scene.file_path, { ...meta, ...nextFields }, { chapter }).meta;
         writeMeta(scene.file_path, updated);
 
         const { content: prose } = matter(fs.readFileSync(scene.file_path, "utf8"));