@hanna84/mcp-writing 2.12.6 → 2.12.8

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).
 
+#### [v2.12.8](https://github.com/hannasdev/mcp-writing.git
+/compare/v2.12.7...v2.12.8)
+
+- refactor(domains): move review, styleguide, and workflow modules into src [`#123`](https://github.com/hannasdev/mcp-writing.git
+/pull/123)
+
+#### [v2.12.7](https://github.com/hannasdev/mcp-writing.git
+/compare/v2.12.6...v2.12.7)
+
+> 28 April 2026
+
+- refactor(core): move db and git modules under src/core [`#122`](https://github.com/hannasdev/mcp-writing.git
+/pull/122)
+- Release 2.12.7 [`4644e20`](https://github.com/hannasdev/mcp-writing.git
+/commit/4644e20a0b63f508e7a9578c600596dd546b4653)
+
 #### [v2.12.6](https://github.com/hannasdev/mcp-writing.git
 /compare/v2.12.5...v2.12.6)
 
+> 28 April 2026
+
 - refactor: move sync and world modules under src domains [`#120`](https://github.com/hannasdev/mcp-writing.git
 /pull/120)
+- Release 2.12.6 [`e20c399`](https://github.com/hannasdev/mcp-writing.git
+/commit/e20c399a56933cef11171b733fbc008f1e4e9fa8)
 
 #### [v2.12.5](https://github.com/hannasdev/mcp-writing.git
 /compare/v2.12.4...v2.12.5)

package/db.js

@@ -1,268 +1 @@
-import { DatabaseSync } from "node:sqlite";
-
-export const SCHEMA = `
-  CREATE TABLE IF NOT EXISTS universes (
-    universe_id TEXT PRIMARY KEY,
-    name        TEXT NOT NULL
-  );
-
-  CREATE TABLE IF NOT EXISTS projects (
-    project_id  TEXT PRIMARY KEY,
-    universe_id TEXT REFERENCES universes(universe_id),
-    name        TEXT NOT NULL
-  );
-
-  CREATE TABLE IF NOT EXISTS scenes (
-    scene_id          TEXT NOT NULL,
-    project_id        TEXT NOT NULL REFERENCES projects(project_id),
-    title             TEXT,
-    part              INTEGER,
-    chapter           INTEGER,
-    chapter_title     TEXT,
-    pov               TEXT,
-    logline           TEXT,
-    scene_change      TEXT,
-    causality         INTEGER,
-    stakes            INTEGER,
-    scene_functions   TEXT,
-    save_the_cat_beat TEXT,
-    timeline_position INTEGER,
-    story_time        TEXT,
-    word_count        INTEGER,
-    file_path         TEXT NOT NULL,
-    prose_checksum    TEXT,
-    metadata_stale    INTEGER NOT NULL DEFAULT 0,
-    updated_at        TEXT NOT NULL,
-    PRIMARY KEY (scene_id, project_id)
-  );
-
-  CREATE TABLE IF NOT EXISTS scene_characters (
-    scene_id     TEXT NOT NULL,
-    character_id TEXT NOT NULL,
-    PRIMARY KEY (scene_id, character_id)
-  );
-
-  CREATE TABLE IF NOT EXISTS scene_places (
-    scene_id TEXT NOT NULL,
-    place_id TEXT NOT NULL,
-    PRIMARY KEY (scene_id, place_id)
-  );
-
-  CREATE TABLE IF NOT EXISTS scene_tags (
-    scene_id TEXT NOT NULL,
-    tag      TEXT NOT NULL,
-    PRIMARY KEY (scene_id, tag)
-  );
-
-  CREATE TABLE IF NOT EXISTS scene_threads (
-    scene_id  TEXT NOT NULL,
-    thread_id TEXT NOT NULL,
-    beat      TEXT,
-    PRIMARY KEY (scene_id, thread_id)
-  );
-
-  CREATE TABLE IF NOT EXISTS characters (
-    character_id     TEXT NOT NULL PRIMARY KEY,
-    project_id       TEXT,
-    universe_id      TEXT,
-    name             TEXT NOT NULL,
-    role             TEXT,
-    arc_summary      TEXT,
-    first_appearance TEXT,
-    file_path        TEXT
-  );
-
-  CREATE TABLE IF NOT EXISTS character_traits (
-    character_id TEXT NOT NULL,
-    trait        TEXT NOT NULL,
-    PRIMARY KEY (character_id, trait)
-  );
-
-  CREATE TABLE IF NOT EXISTS character_relationships (
-    from_character    TEXT NOT NULL,
-    to_character      TEXT NOT NULL,
-    relationship_type TEXT NOT NULL,
-    strength          TEXT,
-    scene_id          TEXT,
-    note              TEXT
-  );
-
-  CREATE TABLE IF NOT EXISTS places (
-    place_id    TEXT NOT NULL PRIMARY KEY,
-    project_id  TEXT,
-    universe_id TEXT,
-    name        TEXT NOT NULL,
-    file_path   TEXT
-  );
-
-  CREATE TABLE IF NOT EXISTS threads (
-    thread_id  TEXT NOT NULL PRIMARY KEY,
-    project_id TEXT NOT NULL,
-    name       TEXT NOT NULL,
-    status     TEXT NOT NULL DEFAULT 'active'
-  );
-
-  CREATE TABLE IF NOT EXISTS reference_docs (
-    doc_id      TEXT NOT NULL PRIMARY KEY,
-    project_id  TEXT,
-    universe_id TEXT,
-    title       TEXT NOT NULL,
-    file_path   TEXT NOT NULL
-  );
-
-  CREATE TABLE IF NOT EXISTS reference_doc_tags (
-    doc_id TEXT NOT NULL,
-    tag    TEXT NOT NULL,
-    PRIMARY KEY (doc_id, tag)
-  );
-
-  CREATE VIRTUAL TABLE IF NOT EXISTS scenes_fts USING fts5(
-    scene_id, project_id, logline, title, keywords
-  );
-
-  CREATE TABLE IF NOT EXISTS schema_version (
-    id      INTEGER PRIMARY KEY CHECK (id = 1),
-    version INTEGER NOT NULL
-  );
-
-  CREATE TABLE IF NOT EXISTS async_jobs (
-    job_id      TEXT NOT NULL PRIMARY KEY,
-    kind        TEXT NOT NULL,
-    status      TEXT NOT NULL,
-    created_at  TEXT NOT NULL,
-    started_at  TEXT,
-    finished_at TEXT,
-    error       TEXT,
-    result_json TEXT
-  );
-`;
-
-// Each function is applied exactly once, in order, when version < its index+1.
-// Each migration runs inside a transaction with the version bump — crash-safe.
-// Migrations must be idempotent (guard against already-applied state).
-// Never edit existing entries — add new ones at the end.
-const MIGRATIONS = [
-  // 1: add chapter_title column to scenes
-  (db) => {
-    const sceneColumns = db.prepare(`PRAGMA table_info(scenes)`).all();
-    if (!sceneColumns.some(c => c.name === "chapter_title")) {
-      db.exec(`ALTER TABLE scenes ADD COLUMN chapter_title TEXT;`);
-    }
-  },
-  // 2: rebuild FTS table to include keywords column (preserve existing rows)
-  (db) => {
-    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(`
-        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;`);
-    }
-  },
-];
-
-// The version every database should reach after openDb. Not the current DB value —
-// query schema_version directly if you need the live version of a specific database.
-export const CURRENT_SCHEMA_VERSION = MIGRATIONS.length;
-
-function applyMigrations(db) {
-  db.prepare(`INSERT OR IGNORE INTO schema_version (id, version) VALUES (1, 0)`).run();
-  for (;;) {
-    db.exec(`BEGIN IMMEDIATE;`);
-    try {
-      const { version } = db.prepare(`SELECT version FROM schema_version WHERE id = 1`).get();
-      if (version >= MIGRATIONS.length) {
-        db.exec(`COMMIT;`);
-        break;
-      }
-      MIGRATIONS[version](db);
-      // WHERE version = ? ensures the bump is monotonic: a concurrent opener
-      // that advanced the version first will cause this UPDATE to match 0 rows,
-      // which is safe — the migration is already applied.
-      db.prepare(`UPDATE schema_version SET version = ? WHERE id = 1 AND version = ?`).run(version + 1, version);
-      db.exec(`COMMIT;`);
-    } catch (err) {
-      db.exec(`ROLLBACK;`);
-      throw err;
-    }
-  }
-}
-
-export function openDb(dbPath) {
-  const db = new DatabaseSync(dbPath);
-  db.exec(SCHEMA);
-  applyMigrations(db);
-  return db;
-}
-
-export function checkpointJobCreate(db, job) {
-  db.prepare(`
-    INSERT OR IGNORE INTO async_jobs (job_id, kind, status, created_at, started_at)
-    VALUES (?, ?, ?, ?, ?)
-  `).run(job.id, job.kind, job.status, job.createdAt, job.startedAt ?? null);
-}
-
-export function checkpointJobFinish(db, job) {
-  // UPSERT so a terminal state is always recorded even if checkpointJobCreate
-  // was skipped due to a best-effort failure.
-  db.prepare(`
-    INSERT INTO async_jobs
-      (job_id, kind, status, created_at, started_at, finished_at, error, result_json)
-    VALUES (?, ?, ?, ?, ?, ?, ?, ?)
-    ON CONFLICT(job_id) DO UPDATE SET
-      status      = excluded.status,
-      finished_at = excluded.finished_at,
-      error       = excluded.error,
-      result_json = excluded.result_json
-  `).run(
-    job.id,
-    job.kind,
-    job.status,
-    job.createdAt,
-    job.startedAt ?? null,
-    job.finishedAt ?? null,
-    job.error ?? null,
-    job.result != null ? JSON.stringify(job.result) : null
-  );
-}
-
-export function pruneJobCheckpoints(db, ttlMs) {
-  const cutoff = new Date(Date.now() - ttlMs).toISOString();
-  db.prepare(`
-    DELETE FROM async_jobs WHERE finished_at IS NOT NULL AND finished_at < ?
-  `).run(cutoff);
-}
-
-export function loadStalledJobs(db) {
-  // 'cancelling' included defensively; in practice only 'running' rows exist
-  // since we never write a 'cancelling' checkpoint between create and finish.
-  return db.prepare(`
-    SELECT job_id, kind, status, created_at, started_at
-    FROM async_jobs WHERE status IN ('running', 'cancelling')
-  `).all().map(row => ({
-    id: row.job_id,
-    kind: row.kind,
-    status: row.status,
-    createdAt: row.created_at,
-    startedAt: row.started_at ?? null,
-    finishedAt: null,
-    error: null,
-    result: null,
-    progress: null,
-    child: null,
-    onComplete: null,
-    tmpDir: null,
-    requestPath: null,
-    resultPath: null,
-  }));
-}
+export * from "./src/core/db.js";

package/git.js

@@ -1,209 +1 @@
-import { execFileSync, execSync } from "node:child_process";
-import fs from "node:fs";
-import path from "node:path";
-
-/**
- * Check if a directory is itself the root of a git repository (not just inside one).
- * This prevents mcp-writing's own .git from being used for prose snapshots when
- * WRITING_SYNC_DIR is a subdirectory of the code repo.
- */
-export function isGitRepository(dirPath) {
-  try {
-    const gitRoot = execSync("git rev-parse --show-toplevel", {
-      cwd: dirPath,
-      stdio: "pipe",
-      encoding: "utf8",
-    }).trim();
-    return fs.realpathSync(gitRoot) === fs.realpathSync(dirPath);
-  } catch {
-    return false;
-  }
-}
-
-/**
- * Initialize a git repository in a directory
- */
-export function initGitRepository(dirPath) {
-  try {
-    execSync("git init", { cwd: dirPath, stdio: "pipe" });
-    // Set a dummy user config for commits if not already set
-    try {
-      execSync("git config user.email", { cwd: dirPath, stdio: "pipe" });
-    } catch {
-      execSync("git config user.email writing-mcp@local", { cwd: dirPath, stdio: "pipe" });
-      execSync("git config user.name writing-mcp", { cwd: dirPath, stdio: "pipe" });
-    }
-    return true;
-  } catch (err) {
-    throw new Error(`Failed to initialize git repository: ${err.message}`, { cause: err });
-  }
-}
-
-/**
- * Check if git is available on PATH
- */
-export function isGitAvailable() {
-  try {
-    execSync("git --version", { stdio: "pipe" });
-    return true;
-  } catch {
-    return false;
-  }
-}
-
-/**
- * Create a git commit for one or more scene-related files
- * Returns { commit_hash: string, commit_message: string }
- */
-export function createSnapshot(dirPath, filePath, sceneId, instruction, options = {}) {
-  try {
-    const { messagePrefix = "pre-edit snapshot" } = options;
-    const inputPaths = Array.isArray(filePath) ? filePath : [filePath];
-    const relPaths = [...new Set(inputPaths
-      .filter(Boolean)
-      .map(p => path.relative(dirPath, p)))];
-    if (!relPaths.length) {
-      throw new Error("No file paths provided for snapshot");
-    }
-    // Use -A so removed/renamed paths are staged as part of relocation snapshots.
-    execFileSync("git", ["add", "-A", "--", ...relPaths], { cwd: dirPath, stdio: "pipe" });
-
-    const commitMessage = messagePrefix
-      ? `${messagePrefix}: ${sceneId} — ${instruction}`
-      : String(instruction);
-
-    execFileSync("git", ["commit", "-m", commitMessage], {
-      cwd: dirPath,
-      stdio: "pipe",
-    });
-    const commitHash = execFileSync("git", ["rev-parse", "HEAD"], {
-      cwd: dirPath,
-      stdio: "pipe",
-      encoding: "utf8",
-    }).trim();
-
-    return {
-      commit_hash: commitHash,
-      commit_message: commitMessage,
-    };
-  } catch (err) {
-    // Check if nothing changed (no error, just no commit)
-    const stderr = err?.stderr ? String(err.stderr) : "";
-    const stdout = err?.stdout ? String(err.stdout) : "";
-    const text = `${stderr}\n${stdout}\n${err?.message ?? ""}`;
-    if (text.includes("nothing to commit") || err.status === 1) {
-      return {
-        commit_hash: null,
-        commit_message: null,
-        reason: "no changes to commit",
-      };
-    }
-    throw new Error(`Failed to create snapshot: ${err.message}`, { cause: err });
-  }
-}
-
-/**
- * List git commits for a file, with timestamps and messages
- * Returns array of { commit_hash, timestamp, message }
- */
-export function listSnapshots(dirPath, filePath) {
-  try {
-    const relPath = path.relative(dirPath, filePath);
-    const output = execFileSync(
-      "git",
-      ["log", "--pretty=format:%h|%ai|%s", "--", relPath],
-      {
-        cwd: dirPath,
-        stdio: "pipe",
-        encoding: "utf8",
-      }
-    );
-
-    if (!output) return [];
-
-    return output
-      .split("\n")
-      .filter(Boolean)
-      .map((line) => {
-        const [hash, timestamp, ...messageParts] = line.split("|");
-        return {
-          commit_hash: hash.trim(),
-          timestamp: timestamp.trim(),
-          message: messageParts.join("|").trim(),
-        };
-      });
-  } catch (err) {
-    // If there are no commits yet, return empty array
-    if (err.message.includes("your current branch") || err.status === 128) {
-      return [];
-    }
-    throw new Error(`Failed to list snapshots: ${err.message}`, { cause: err });
-  }
-}
-
-/**
- * Get prose content from a specific git commit
- * If commit is null, returns current working tree version
- */
-export function getSceneProseAtCommit(dirPath, filePath, commitHash) {
-  try {
-    const relPath = path.relative(dirPath, filePath);
-
-    if (!commitHash) {
-      // Return current working tree version
-      return fs.readFileSync(filePath, "utf8");
-    }
-
-    // Get version from git
-    const content = execFileSync("git", ["show", `${commitHash}:${relPath}`], {
-      cwd: dirPath,
-      stdio: "pipe",
-      encoding: "utf8",
-    });
-
-    return content;
-  } catch (err) {
-    // ENOENT from execFileSync means the git binary was not found on PATH — not a missing file in the commit.
-    if (err.code === "ENOENT") {
-      throw new Error("git executable not found; ensure git is installed and on PATH", { cause: err });
-    }
-    // git exit 128 with "path ... does not exist in" or "bad object" indicates missing path/commit.
-    const stderr = err?.stderr ? String(err.stderr) : "";
-    if (err.status === 128 && (stderr.includes("does not exist in") || stderr.includes("bad object"))) {
-      throw new Error(`File not found in commit ${commitHash}`, { cause: err });
-    }
-    throw new Error(`Failed to retrieve scene prose: ${err.message}`, { cause: err });
-  }
-}
-
-/**
- * Check if working tree is clean (no uncommitted changes)
- */
-export function isWorkingTreeClean(dirPath) {
-  try {
-    const output = execSync("git status --porcelain", {
-      cwd: dirPath,
-      stdio: "pipe",
-      encoding: "utf8",
-    });
-    return output.trim() === "";
-  } catch {
-    return false;
-  }
-}
-
-/**
- * Get the HEAD commit hash
- */
-export function getHeadCommitHash(dirPath) {
-  try {
-    const hash = execSync("git rev-parse HEAD", {
-      cwd: dirPath,
-      stdio: "pipe",
-      encoding: "utf8",
-    }).trim();
-    return hash;
-  } catch {
-    return null;
-  }
-}
+export * from "./src/core/git.js";

package/helpers.js

@@ -9,7 +9,7 @@ import {
   renderPlaceSheetTemplate,
   renderCharacterArcTemplate,
 } from "./src/world/world-entity-templates.js";
-import { ReviewBundlePlanError } from "./review-bundles.js";
+import { ReviewBundlePlanError } from "./src/review-bundles/review-bundles.js";
 
 export function deriveLoglineFromProse(prose) {
   const compact = prose.replace(/\s+/g, " ").trim();

package/package.json

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

package/prose-styleguide-drift.js

@@ -1,125 +1 @@
-function detectQuotationStyle(prose) {
-  const counts = {
-    double: (prose.match(/"[^"]{2,}"/g) ?? []).length,
-    guillemets: (prose.match(/«[^»]{2,}»/g) ?? []).length,
-    low9: (prose.match(/„[^“]{2,}“/g) ?? []).length,
-    corner_brackets: (prose.match(/「[^」]{2,}」/g) ?? []).length,
-    dialogue_dash_en: (prose.match(/^\s*–\s/mg) ?? []).length,
-    dialogue_dash_em: (prose.match(/^\s*—\s/mg) ?? []).length,
-    single: (prose.match(/'[^'\n]{2,}'/g) ?? []).length,
-  };
-
-  let best = null;
-  let bestCount = 0;
-  for (const [style, count] of Object.entries(counts)) {
-    if (count > bestCount) {
-      best = style;
-      bestCount = count;
-    }
-  }
-  return bestCount > 0 ? best : null;
-}
-
-function detectEmDashSpacing(prose) {
-  const spaced = (prose.match(/\s—\s/g) ?? []).length;
-  const closed = (prose.match(/\S—\S/g) ?? []).length;
-  if (spaced === 0 && closed === 0) return null;
-  return spaced >= closed ? "spaced" : "closed";
-}
-
-function detectSpellingVariant(prose) {
-  const lower = prose.toLowerCase();
-  const ukSignals = ["colour", "realise", "centre", "honour", "travelling"];
-  const usSignals = ["color", "realize", "center", "honor", "traveling"];
-
-  const countHits = (signals) => signals.reduce((sum, word) => {
-    const re = new RegExp(`\\b${word}\\b`, "g");
-    return sum + (lower.match(re) ?? []).length;
-  }, 0);
-
-  const uk = countHits(ukSignals);
-  const us = countHits(usSignals);
-  if (uk === 0 && us === 0) return null;
-  return uk >= us ? "uk" : "us";
-}
-
-function detectTenseHint(prose) {
-  const lower = prose.toLowerCase();
-  const past = (lower.match(/\b(was|were|had|did)\b/g) ?? []).length;
-  const present = (lower.match(/\b(is|are|has|do)\b/g) ?? []).length;
-  if (past === 0 && present === 0) return null;
-  return present >= past ? "present" : "past";
-}
-
-function mostCommonValue(values) {
-  const counts = new Map();
-  for (const value of values) {
-    if (!value) continue;
-    counts.set(value, (counts.get(value) ?? 0) + 1);
-  }
-  if (counts.size === 0) return null;
-
-  let bestValue = null;
-  let bestCount = 0;
-  for (const [value, count] of counts.entries()) {
-    if (count > bestCount) {
-      bestValue = value;
-      bestCount = count;
-    }
-  }
-  return { value: bestValue, count: bestCount, total: values.filter(Boolean).length };
-}
-
-export function detectStyleguideSignals(prose) {
-  return {
-    quotation_style: detectQuotationStyle(prose),
-    em_dash_spacing: detectEmDashSpacing(prose),
-    spelling: detectSpellingVariant(prose),
-    tense: detectTenseHint(prose),
-  };
-}
-
-export function analyzeSceneStyleguideDrift({ prose, resolvedConfig }) {
-  const observed = detectStyleguideSignals(prose);
-  const drift = [];
-
-  for (const field of ["quotation_style", "em_dash_spacing", "spelling", "tense"]) {
-    const declared = resolvedConfig?.[field];
-    const seen = observed[field];
-    if (!declared || !seen) continue;
-    if (declared !== seen) {
-      drift.push({ field, declared, observed: seen });
-    }
-  }
-
-  return { observed, drift };
-}
-
-export function suggestStyleguideUpdatesFromScenes({
-  sceneAnalyses,
-  resolvedConfig,
-  minAgreement = 0.6,
-  minEvidence = 3,
-}) {
-  const suggestions = {};
-
-  for (const field of ["quotation_style", "em_dash_spacing", "spelling", "tense"]) {
-    const values = sceneAnalyses.map((scene) => scene.observed?.[field] ?? null);
-    const common = mostCommonValue(values);
-    if (!common) continue;
-    if (common.total < minEvidence) continue;
-
-    const agreement = common.total > 0 ? common.count / common.total : 0;
-    const fieldThreshold = field === "tense" ? Math.max(minAgreement, 0.75) : minAgreement;
-    if (agreement < fieldThreshold) continue;
-    if (resolvedConfig?.[field] === common.value) continue;
-
-    suggestions[field] = {
-      suggested_value: common.value,
-      agreement,
-      based_on_scenes: common.total,
-    };
-  }
-
-  return suggestions;
-}
+export * from "./src/styleguide/prose-styleguide-drift.js";