@hanna84/mcp-writing 1.7.0 → 1.8.1

package/CHANGELOG.md

@@ -4,11 +4,31 @@ 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).
 
+#### [v1.8.1](https://github.com/hannasdev/mcp-writing.git
+/compare/v1.8.0...v1.8.1)
+
+- Fix inferProjectAndUniverse misidentifying two-segment universe paths under projects/ [`#51`](https://github.com/hannasdev/mcp-writing.git
+/pull/51)
+
+#### [v1.8.0](https://github.com/hannasdev/mcp-writing.git
+/compare/v1.7.0...v1.8.0)
+
+> 19 April 2026
+
+- feat(search): index metadata keywords in scenes FTS and preserve legacy index data [`#50`](https://github.com/hannasdev/mcp-writing.git
+/pull/50)
+- Release 1.8.0 [`5e3c4fd`](https://github.com/hannasdev/mcp-writing.git
+/commit/5e3c4fd6fa4caade6540d3228bfa52ee5f42c923)
+
 #### [v1.7.0](https://github.com/hannasdev/mcp-writing.git
 /compare/v1.6.3...v1.7.0)
 
+> 19 April 2026
+
 - feat: configurable OpenClaw runtime identity and ownership guardrails [`#49`](https://github.com/hannasdev/mcp-writing.git
 /pull/49)
+- Release 1.7.0 [`f859eb5`](https://github.com/hannasdev/mcp-writing.git
+/commit/f859eb57bcea412591635185e113f2c6467b0f59)
 
 #### [v1.6.3](https://github.com/hannasdev/mcp-writing.git
 /compare/v1.6.2...v1.6.3)

package/README.md

@@ -315,7 +315,7 @@ Outcome: you get AI speed with explicit approval and recoverable history for eve
 | `list_places` | All places |
 | `get_place_sheet` | Full place metadata, tags, associated characters, notes, and support notes |
 | `create_place_sheet` | Create a canonical place sheet folder and sidecar |
-| `search_metadata` | Full-text search across scene titles and loglines |
+| `search_metadata` | Full-text search across scene titles, loglines, and metadata keywords (tags/characters/places/versions) |
 | `list_threads` | All subplot threads for a project |
 | `get_thread_arc` | Scenes belonging to a thread, with per-thread beat |
 | `upsert_thread_link` | Create/update a thread and link it to a scene |

package/db.js

@@ -116,12 +116,43 @@ export const SCHEMA = `
   );
 
   CREATE VIRTUAL TABLE IF NOT EXISTS scenes_fts USING fts5(
-    scene_id, project_id, logline, title
+    scene_id, project_id, logline, title, keywords
   );
 `;
 
 export function openDb(dbPath) {
   const db = new DatabaseSync(dbPath);
   db.exec(SCHEMA);
+
+  // Rebuild legacy FTS table if it predates keyword indexing.
+  // Preserve existing indexed rows so metadata search remains available
+  // even before the next sync pass repopulates from source files.
+  const ftsSql = db.prepare(`
+    SELECT sql
+    FROM sqlite_master
+    WHERE type = 'table' AND name = 'scenes_fts'
+  `).get()?.sql;
+  if (typeof ftsSql === "string" && !ftsSql.toLowerCase().includes("keywords")) {
+    db.exec(`BEGIN IMMEDIATE;`);
+    try {
+      db.exec(`
+        CREATE VIRTUAL TABLE scenes_fts_migrating USING fts5(
+          scene_id, project_id, logline, title, keywords
+        );
+      `);
+      db.exec(`
+        INSERT INTO scenes_fts_migrating (scene_id, project_id, logline, title, keywords)
+        SELECT scene_id, project_id, logline, title, ''
+        FROM scenes_fts;
+      `);
+      db.exec(`DROP TABLE scenes_fts;`);
+      db.exec(`ALTER TABLE scenes_fts_migrating RENAME TO scenes_fts;`);
+      db.exec(`COMMIT;`);
+    } catch (err) {
+      db.exec(`ROLLBACK;`);
+      throw err;
+    }
+  }
+
   return db;
 }

package/index.js

@@ -946,7 +946,7 @@ function createMcpServer() {
   // ---- search_metadata -----------------------------------------------------
   s.tool(
     "search_metadata",
-    "Full-text search across scene titles and loglines (synopsis/logline text fields). Use this when you don't know the exact scene_id or chapter but want to find scenes by topic, theme, or keywords in the description. Not a prose search — use get_scene_prose to read actual text. Supports pagination via page/page_size and auto-paginates large result sets with total_count.",
+    "Full-text search across scene titles, loglines (synopsis/logline text fields), and metadata keywords (tags/characters/places/versions). Use this when you don't know the exact scene_id or chapter but want to find scenes by topic, theme, or metadata keyword. Not a prose search — use get_scene_prose to read actual text. Supports pagination via page/page_size and auto-paginates large result sets with total_count.",
     {
       query: z.string().describe("Search terms (e.g. 'hospital' or 'Sebastian feeding'). FTS5 syntax supported."),
       page: z.number().int().min(1).optional().describe("Optional page number for paginated responses (1-based)."),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hanna84/mcp-writing",
-  "version": "1.7.0",
+  "version": "1.8.1",
   "description": "MCP service for AI-assisted reasoning and editing on long-form fiction projects",
   "type": "module",
   "main": "index.js",

package/sync.js

@@ -96,17 +96,66 @@ export function normalizeSceneMetaForPath(syncDir, filePath, meta = {}) {
   };
 }
 
+// Structural directory names that are never project slugs under projects/<id>/.
+const PROJECT_STRUCTURAL_DIRS = new Set(["world", "scenes", "misc", "fragments", "feedback", "draft"]);
+
+// Cache universe project root existence checks during sync scans.
+const UNIVERSE_PROJECT_ROOT_CACHE = new Map();
+
+// Returns true for known structural path segments directly under a project root
+// (named dirs like "world", "scenes", and part-N / chapter-N path segments).
+function isProjectStructuralDir(name) {
+  const normalized = String(name ?? "").toLowerCase();
+  return PROJECT_STRUCTURAL_DIRS.has(normalized)
+    || /^part-\d+$/.test(normalized)
+    || /^chapter-\d+$/.test(normalized);
+}
+
+function isBookSlug(name) {
+  return /^book-[a-z0-9][a-z0-9-]*$/i.test(String(name ?? ""));
+}
+
+function hasUniverseProjectRoot(syncDir, universeId, projectSlug) {
+  const key = `${syncDir}::${universeId}/${projectSlug}`;
+  if (UNIVERSE_PROJECT_ROOT_CACHE.has(key)) {
+    return UNIVERSE_PROJECT_ROOT_CACHE.get(key);
+  }
+
+  const exists = fs.existsSync(path.join(syncDir, "universes", universeId, projectSlug));
+  UNIVERSE_PROJECT_ROOT_CACHE.set(key, exists);
+  return exists;
+}
+
 export function inferProjectAndUniverse(syncDir, filePath) {
   const rel = path.relative(syncDir, filePath);
   const parts = rel.split(path.sep);
 
   if (parts[0] === "universes" && parts.length >= 3) {
+    // Case-sensitive "world" intentionally matches isWorldFile() which also uses lowercase.
     if (parts[2] === "world") {
       return { universe_id: parts[1], project_id: null };
     }
     return { universe_id: parts[1], project_id: `${parts[1]}/${parts[2]}` };
   }
   if (parts[0] === "projects" && parts.length >= 2) {
+    // Detect accidental two-segment layout: projects/<universe>/<book>/...
+    // This occurs when a universe-scoped project_id (e.g. "universe-1/book-1-the-lamb")
+    // is written under projects/ instead of universes/.
+    // Detection is deliberately conservative to avoid mis-classifying valid nested
+    // project layouts (e.g. projects/my-novel/notes/...). All three conditions must hold:
+    //   1. parts[2] matches a book-* slug pattern (book-1, book-one, book-1-the-lamb, …)
+    //   2. parts[3] is a known structural directory (scenes, world, part-N, chapter-N, …)
+    //   3. A matching universes/<universe>/<book> directory exists on disk
+    if (
+      parts.length >= 4
+      && parts[2] !== undefined
+      && parts[3] !== undefined
+      && isBookSlug(parts[2])
+      && isProjectStructuralDir(parts[3])
+      && hasUniverseProjectRoot(syncDir, parts[1], parts[2])
+    ) {
+      return { universe_id: parts[1], project_id: `${parts[1]}/${parts[2]}` };
+    }
     return { universe_id: null, project_id: parts[1] };
   }
   return { universe_id: null, project_id: parts[0] ?? "default" };
@@ -515,14 +564,34 @@ export function indexSceneFile(db, syncDir, file, meta, prose) {
     );
   }
 
-  db.prepare(`INSERT OR REPLACE INTO scenes_fts (scene_id, project_id, logline, title) VALUES (?, ?, ?, ?)`).run(
-    meta.scene_id, project_id, meta.logline ?? meta.synopsis ?? "", meta.title ?? ""
+  const keywordTokens = [
+    ...(meta.tags ?? []),
+    ...(meta.characters ?? []),
+    ...(meta.places ?? []),
+    ...(meta.versions ?? []),
+  ]
+    .filter(Boolean)
+    .map(String)
+    .map(s => s.trim())
+    .filter(Boolean)
+    .join(" ");
+
+  db.prepare(`INSERT OR REPLACE INTO scenes_fts (scene_id, project_id, logline, title, keywords) VALUES (?, ?, ?, ?, ?)`).run(
+    meta.scene_id,
+    project_id,
+    meta.logline ?? meta.synopsis ?? "",
+    meta.title ?? "",
+    keywordTokens,
   );
 
   return { isStale };
 }
 
 export function syncAll(db, syncDir, { quiet = false, writable = false } = {}) {
+  // Reset per-run inference cache so filesystem changes between sync calls
+  // (for example after imports or path repairs) are reflected immediately.
+  UNIVERSE_PROJECT_ROOT_CACHE.clear();
+
   const files = walkFiles(syncDir);
   let indexed = 0;
   let staleMarked = 0;