@hanna84/mcp-writing 2.0.0 → 2.0.1

package/CHANGELOG.md

@@ -4,11 +4,21 @@ 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).
 
-#### [v2.0.0](https://github.com/hannasdev/mcp-writing.git
+#### [v2.0.1](https://github.com/hannasdev/mcp-writing.git
+/compare/v2.0.0...v2.0.1)
+
+- fix(package): include scene-character-batch in published files [`#79`](https://github.com/hannasdev/mcp-writing.git
+/pull/79)
+
+### [v2.0.0](https://github.com/hannasdev/mcp-writing.git
 /compare/v1.17.0...v2.0.0)
 
+> 25 April 2026
+
 - feat(review-bundles)!: add PDF export via pdfkit [`#78`](https://github.com/hannasdev/mcp-writing.git
 /pull/78)
+- Release 2.0.0 [`ddccb28`](https://github.com/hannasdev/mcp-writing.git
+/commit/ddccb2869c6787b0cb64897d067886d59f7cb6e6)
 
 #### [v1.17.0](https://github.com/hannasdev/mcp-writing.git
 /compare/v1.16.2...v1.17.0)

package/package.json

@@ -1,12 +1,13 @@
 {
   "name": "@hanna84/mcp-writing",
-  "version": "2.0.0",
+  "version": "2.0.1",
   "description": "MCP service for AI-assisted reasoning and editing on long-form fiction projects",
   "type": "module",
   "main": "index.js",
   "files": [
     "index.js",
     "async-progress.js",
+    "scene-character-batch.js",
     "scrivener-direct.js",
     "importer.js",
     "db.js",

package/scene-character-batch.js

@@ -0,0 +1,221 @@
+import fs from "node:fs";
+import matter from "gray-matter";
+import { normalizeSceneMetaForPath, readMeta, writeMeta } from "./sync.js";
+
+function escapeRegex(text) {
+  return text.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+}
+
+function normalizeCharacterRows(rows) {
+  const clean = rows
+    .filter(row => row?.character_id && row?.name)
+    .map(row => ({
+      character_id: row.character_id,
+      name: String(row.name).trim(),
+      tokens: [...new Set(String(row.name).toLowerCase().split(/\s+/).filter(Boolean))],
+    }))
+    .filter(row => row.name.length > 0);
+
+  const tokenMap = new Map();
+  for (const row of clean) {
+    for (const token of row.tokens) {
+      if (!token || token.length < 3) continue;
+      const ids = tokenMap.get(token) ?? [];
+      ids.push(row.character_id);
+      tokenMap.set(token, ids);
+    }
+  }
+
+  return { clean, tokenMap };
+}
+
+function inferCharactersFromProse(prose, characterRows) {
+  const { clean, tokenMap } = characterRows;
+  const inferred = new Set();
+  const ambiguous_tokens = [];
+
+  for (const row of clean) {
+    if (row.tokens.length > 1) {
+      const pattern = row.tokens.map(escapeRegex).join("\\s+");
+      const regex = new RegExp(`\\b${pattern}\\b`, "i");
+      if (regex.test(prose)) {
+        inferred.add(row.character_id);
+        continue;
+      }
+    }
+
+    for (const token of row.tokens) {
+      if (!token || token.length < 3) continue;
+      const tokenRegex = new RegExp(`\\b${escapeRegex(token)}\\b`, "i");
+      if (!tokenRegex.test(prose)) continue;
+
+      const tokenIds = tokenMap.get(token) ?? [];
+      if (tokenIds.length === 1) {
+        inferred.add(row.character_id);
+      } else if (!ambiguous_tokens.includes(token)) {
+        ambiguous_tokens.push(token);
+      }
+    }
+  }
+
+  return {
+    inferred_characters: [...inferred],
+    ambiguous_tokens,
+  };
+}
+
+function nextTurn() {
+  return new Promise(resolve => setImmediate(resolve));
+}
+
+function getInterSceneDelayMs() {
+  const raw = Number(process.env.MCP_WRITING_SCENE_CHARACTER_BATCH_DELAY_MS ?? 0);
+  return Number.isFinite(raw) && raw > 0 ? raw : 0;
+}
+
+function delay(ms) {
+  return new Promise(resolve => setTimeout(resolve, ms));
+}
+
+export async function runSceneCharacterBatch({ syncDir, args, onProgress, shouldCancel }) {
+  const {
+    project_id,
+    dry_run = true,
+    replace_mode = "merge",
+    include_match_details = false,
+    project_exists = true,
+    target_scenes = [],
+    character_rows = [],
+  } = args;
+
+  const targetScenes = Array.isArray(target_scenes) ? target_scenes : [];
+  const characterRows = Array.isArray(character_rows) ? character_rows : [];
+  const normalizedCharacterRows = normalizeCharacterRows(characterRows);
+
+  const results = [];
+  let processed_scenes = 0;
+  let scenes_changed = 0;
+  let failed_scenes = 0;
+  let links_added = 0;
+  let links_removed = 0;
+  const interSceneDelayMs = getInterSceneDelayMs();
+
+  const emitProgress = () => {
+    if (typeof onProgress !== "function") return;
+    onProgress({
+      total_scenes: targetScenes.length,
+      processed_scenes,
+      scenes_changed,
+      failed_scenes,
+    });
+  };
+
+  emitProgress();
+
+  for (const scene of targetScenes) {
+    await nextTurn();
+
+    if (typeof shouldCancel === "function" && shouldCancel()) {
+      break;
+    }
+
+    try {
+      const raw = fs.readFileSync(scene.file_path, "utf8");
+      const { content: prose } = matter(raw);
+      const { meta } = readMeta(scene.file_path, syncDir, { writable: !dry_run });
+
+      const before_characters = [...new Set((meta.characters ?? []).map(String).filter(Boolean))];
+      const inference = inferCharactersFromProse(prose, normalizedCharacterRows);
+      const inferred_characters = inference.inferred_characters;
+
+      const afterSet = new Set(before_characters);
+      if (replace_mode === "replace") {
+        afterSet.clear();
+      }
+      for (const characterId of inferred_characters) {
+        afterSet.add(characterId);
+      }
+
+      const after_characters = [...afterSet];
+      const beforeSet = new Set(before_characters);
+      const added = after_characters.filter(id => !beforeSet.has(id));
+      const afterSetLookup = new Set(after_characters);
+      const removed = before_characters.filter(id => !afterSetLookup.has(id));
+      const changed = added.length > 0 || removed.length > 0;
+
+      if (!dry_run && changed) {
+        const updatedMeta = normalizeSceneMetaForPath(syncDir, scene.file_path, {
+          ...meta,
+          characters: after_characters,
+        }).meta;
+
+        writeMeta(scene.file_path, updatedMeta);
+      }
+
+      scenes_changed += changed ? 1 : 0;
+      links_added += added.length;
+      links_removed += removed.length;
+
+      const hasInferredMatches = inferred_characters.length > 0;
+      const sceneStatus = changed
+        ? "changed"
+        : (!hasInferredMatches && inference.ambiguous_tokens.length > 0 ? "skipped_ambiguous" : "unchanged");
+
+      results.push({
+        scene_id: scene.scene_id,
+        file_path: scene.file_path,
+        before_characters,
+        inferred_characters,
+        after_characters,
+        added,
+        removed,
+        changed,
+        status: sceneStatus,
+        ...(include_match_details ? { match_details: { ambiguous_tokens: inference.ambiguous_tokens } } : {}),
+      });
+    } catch (error) {
+      failed_scenes += 1;
+      results.push({
+        scene_id: scene.scene_id,
+        file_path: scene.file_path,
+        before_characters: [],
+        inferred_characters: [],
+        after_characters: [],
+        added: [],
+        removed: [],
+        changed: false,
+        status: "failed",
+        error: error instanceof Error ? error.message : String(error),
+      });
+    } finally {
+      processed_scenes += 1;
+      emitProgress();
+      if (interSceneDelayMs > 0) {
+        await delay(interSceneDelayMs);
+      }
+    }
+  }
+
+  const warnings = [];
+  if (failed_scenes > 0) {
+    warnings.push("PARTIAL_SUCCESS: one or more scenes failed to process.");
+  }
+  if (!project_exists && targetScenes.length === 0) {
+    warnings.push(`PROJECT_NOT_FOUND_WARNING: project '${project_id}' was not found; nothing to process.`);
+  }
+
+  return {
+    ok: true,
+    cancelled: Boolean(typeof shouldCancel === "function" && shouldCancel() && processed_scenes < targetScenes.length),
+    project_id,
+    dry_run: Boolean(dry_run),
+    total_scenes: targetScenes.length,
+    processed_scenes,
+    scenes_changed,
+    failed_scenes,
+    links_added,
+    links_removed,
+    results,
+    ...(warnings.length > 0 ? { warning: warnings.join(" ") } : {}),
+  };
+}