@hanna84/mcp-writing 2.12.16 → 2.12.17

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.12.17](https://github.com/hannasdev/mcp-writing.git
+/compare/v2.12.16...v2.12.17)
+
+- fix: restrict package files allowlist after src test move [`#140`](https://github.com/hannasdev/mcp-writing.git
+/pull/140)
+
 #### [v2.12.16](https://github.com/hannasdev/mcp-writing.git
 /compare/v2.12.15...v2.12.16)
 
+> 30 April 2026
+
 - refactor(src): move scripts and tests under src [`#139`](https://github.com/hannasdev/mcp-writing.git
 /pull/139)
+- Release 2.12.16 [`cd34af2`](https://github.com/hannasdev/mcp-writing.git
+/commit/cd34af28c4a06e7c44d78b9b9ef8f185bf063222)
 
 #### [v2.12.15](https://github.com/hannasdev/mcp-writing.git
 /compare/v2.12.14...v2.12.15)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hanna84/mcp-writing",
-  "version": "2.12.16",
+  "version": "2.12.17",
   "description": "MCP service for AI-assisted reasoning and editing on long-form fiction projects",
   "homepage": "https://hannasdev.github.io/mcp-writing/",
   "type": "module",
@@ -41,7 +41,16 @@
   "files": [
     "bin/",
     "index.js",
-    "src/",
+    "src/index.js",
+    "src/core/",
+    "src/review-bundles/",
+    "src/runtime/",
+    "src/scripts/",
+    "src/styleguide/",
+    "src/sync/",
+    "src/tools/",
+    "src/workflows/",
+    "src/world/",
     "README.md",
     "CHANGELOG.md"
   ],

package/src/test/helpers/db.js

@@ -1,47 +0,0 @@
-import { openDb } from "../../core/db.js";
-
-export function insertTestScene(db, {
-  sceneId,
-  projectId = "test-novel",
-  title = null,
-  part = null,
-  chapter = null,
-  timelinePosition = null,
-  metadataStale = 0,
-  wordCount = null,
-}) {
-  const now = new Date().toISOString();
-  db.prepare(`
-    INSERT INTO scenes (
-      scene_id,
-      project_id,
-      title,
-      part,
-      chapter,
-      timeline_position,
-      word_count,
-      file_path,
-      prose_checksum,
-      metadata_stale,
-      updated_at
-    ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
-  `).run(
-    sceneId,
-    projectId,
-    title,
-    part,
-    chapter,
-    timelinePosition,
-    wordCount,
-    `/tmp/${sceneId}.md`,
-    "deadbeef",
-    metadataStale,
-    now
-  );
-}
-
-export function setupReviewBundleTestDb() {
-  const db = openDb(":memory:");
-  db.prepare(`INSERT INTO projects (project_id, universe_id, name) VALUES (?, ?, ?)`).run("test-novel", null, "Test Novel");
-  return db;
-}

package/src/test/helpers/fixtures.js

@@ -1,380 +0,0 @@
-import fs from "node:fs";
-import path from "node:path";
-
-export function copyDirSync(src, dest) {
-  fs.mkdirSync(dest, { recursive: true });
-  for (const entry of fs.readdirSync(src, { withFileTypes: true })) {
-    const srcPath = path.join(src, entry.name);
-    const destPath = path.join(dest, entry.name);
-    if (entry.isDirectory()) copyDirSync(srcPath, destPath);
-    else fs.copyFileSync(srcPath, destPath);
-  }
-}
-
-export function writeFileSyncWithDirs(filePath, content) {
-  fs.mkdirSync(path.dirname(filePath), { recursive: true });
-  fs.writeFileSync(filePath, content, "utf8");
-}
-
-export function createTestSyncFixture(syncDir) {
-  writeFileSyncWithDirs(
-    path.join(syncDir, "projects", "test-novel", "part-1", "chapter-1", "sc-001.md"),
-    `---
-scene_id: sc-001
-title: The Return
-part: 1
-chapter: 1
-characters: [elena, marcus]
-places: [harbor-district]
-logline: Elena returns to the harbor district after three years away and runs into Marcus.
-save_the_cat: Opening Image
-pov: elena
-timeline_position: 1
-story_time: "Day 1, late afternoon"
-tags: [reunion, tension, harbor]
----
-
-The ferry docked at quarter past four, which meant Elena had seventeen minutes before the evening freight shift began and the harbor became impassable. She had timed it deliberately. She did not want to see anyone she knew.
-
-She was at the bottom of the gangway when she heard her name.
-
-Marcus was standing by the storage shed with a clipboard in one hand and an expression she recognized -- the particular look he got when he was pretending not to be surprised. He was very bad at pretending.
-
-"You could have called," he said.
-
-"I could have," she agreed, and kept walking.
-
-He fell into step beside her anyway, which was exactly what she had expected him to do.
-`
-  );
-
-  writeFileSyncWithDirs(
-    path.join(syncDir, "projects", "test-novel", "part-1", "chapter-1", "sc-001.meta.yaml"),
-    `scene_id: sc-001
-title: The Return
-part: 1
-chapter: 1
-characters:
-  - elena
-  - marcus
-places:
-  - harbor-district
-logline: >-
-  Elena returns to the harbor district after three years away and runs into
-  Marcus.
-save_the_cat: Opening Image
-pov: elena
-timeline_position: 1
-story_time: 'Day 1, late afternoon'
-tags:
-  - reunion
-  - tension
-  - harbor
-`
-  );
-
-  writeFileSyncWithDirs(
-    path.join(syncDir, "projects", "test-novel", "part-1", "chapter-1", "sc-002.md"),
-    `---
-scene_id: sc-002
-title: The Argument
-part: 1
-chapter: 1
-characters: [elena, marcus]
-places: [harbor-district]
-logline: Elena and Marcus argue about why she left; she deflects, he pushes back harder than before.
-save_the_cat: Theme Stated
-pov: elena
-timeline_position: 2
-story_time: "Day 1, evening"
-tags: [conflict, backstory, harbor]
----
-
-They ended up at the old bait shed because the wind had picked up and it was the nearest shelter. The shed smelled the same as it always had -- salt and something faintly chemical. Elena had spent half her childhood in this shed. She wished she were somewhere else.
-
-"You didn't call me," Marcus said. "You didn't write. Three years."
-
-"I was busy."
-
-"Everyone is busy. That's not an answer."
-
-She looked at the water instead of him. "It's the one I've got."
-
-He was quiet for a long time. When he spoke again, his voice had changed -- less patient, more tired. "I'm not angry you left, Elena. I'm angry you decided I wouldn't understand."
-
-She didn't have an answer for that either.
-`
-  );
-
-  writeFileSyncWithDirs(
-    path.join(syncDir, "projects", "test-novel", "part-1", "chapter-1", "sc-002.meta.yaml"),
-    `scene_id: sc-002
-title: The Argument
-part: 1
-chapter: 1
-characters:
-  - elena
-  - marcus
-places:
-  - harbor-district
-logline: >-
-  Elena and Marcus argue about why she left; she deflects, he pushes back harder
-  than before.
-save_the_cat: Theme Stated
-pov: elena
-timeline_position: 2
-story_time: 'Day 1, evening'
-tags:
-  - conflict
-  - backstory
-  - harbor
-  - Daniel Nystrom
-`
-  );
-
-  writeFileSyncWithDirs(
-    path.join(syncDir, "projects", "test-novel", "part-1", "chapter-2", "sc-003.md"),
-    `---
-scene_id: sc-003
-title: The Offer
-part: 1
-chapter: 2
-characters: [elena]
-places: [harbor-district]
-logline: Elena receives an envelope at her old address -- an offer she doesn't understand yet, but can't ignore.
-save_the_cat: Catalyst
-pov: elena
-timeline_position: 3
-story_time: "Day 2, morning"
-tags: [mystery, catalyst, solo]
----
-
-The envelope had been slipped under the door of the flat she no longer lived in. The landlord had kept it for her -- "figured you'd be back eventually," he said, in a tone that suggested he had not figured this at all.
-
-Her name was on the front in handwriting she didn't recognize. Inside was a single card with an address across town and a time: 9 p.m., two days from now.
-
-No name. No explanation.
-
-She turned the card over. On the back, in smaller writing: *You know what happened to your father. We do too.*
-
-Elena Voss sat down on the floor of the empty flat and stared at the card for a long time.
-`
-  );
-
-  writeFileSyncWithDirs(
-    path.join(syncDir, "projects", "test-novel", "part-1", "chapter-2", "sc-003.meta.yaml"),
-    `scene_id: sc-003
-title: The Offer
-part: 1
-chapter: 2
-characters:
-  - elena
-places:
-  - harbor-district
-logline: >-
-  Elena receives an envelope at her old address -- an offer she doesn't
-  understand yet, but can't ignore.
-save_the_cat: Catalyst
-pov: elena
-timeline_position: 3
-story_time: 'Day 2, morning'
-tags:
-  - mystery
-  - catalyst
-  - solo
-`
-  );
-
-  writeFileSyncWithDirs(
-    path.join(syncDir, "projects", "test-novel", "world", "characters", "elena.md"),
-    `---
-character_id: elena
-name: Elena Voss
-role: protagonist
-traits: [driven, guarded, perceptive, self-sabotaging]
-arc_summary: Learns to trust others without losing herself.
-first_appearance: sc-001
-tags: [main-cast]
----
-
-Elena grew up in the harbor district, the daughter of a dockworker who disappeared when she was twelve. She has spent most of her adult life building walls and calling it independence. Perceptive to a fault -- she sees through people quickly, which makes her both valuable and exhausting to be around.
-
-Her self-sabotaging streak shows up most clearly in relationships. When things get close, she finds a reason to leave first.
-`
-  );
-
-  writeFileSyncWithDirs(
-    path.join(syncDir, "projects", "test-novel", "world", "characters", "elena.meta.yaml"),
-    `character_id: elena
-name: Elena Voss
-role: protagonist
-traits:
-  - driven
-  - guarded
-  - perceptive
-  - self-sabotaging
-arc_summary: Learns to trust others without losing herself.
-first_appearance: sc-001
-tags:
-  - main-cast
-`
-  );
-
-  writeFileSyncWithDirs(
-    path.join(syncDir, "projects", "test-novel", "world", "characters", "marcus.md"),
-    `---
-character_id: marcus
-name: Marcus Hale
-role: supporting
-traits: [patient, idealistic, stubborn, warm]
-arc_summary: Has to decide whether loyalty to Elena is worth the cost to himself.
-first_appearance: sc-001
-tags: [main-cast]
----
-
-Marcus runs a small freight operation out of the harbor. He has known Elena since they were teenagers and is one of the few people she has never fully pushed away -- not for lack of trying on her part.
-
-He is patient in a way that sometimes reads as passive. He is not passive. He is waiting for the right moment, which he has been doing for approximately fifteen years.
-`
-  );
-
-  writeFileSyncWithDirs(
-    path.join(syncDir, "projects", "test-novel", "world", "characters", "marcus.meta.yaml"),
-    `character_id: marcus
-name: Marcus Hale
-role: supporting
-traits:
-  - patient
-  - idealistic
-  - stubborn
-  - warm
-arc_summary: Has to decide whether loyalty to Elena is worth the cost to himself.
-first_appearance: sc-001
-tags:
-  - main-cast
-`
-  );
-
-  writeFileSyncWithDirs(
-    path.join(syncDir, "projects", "test-novel", "world", "places", "harbor-district.md"),
-    `---
-place_id: harbor-district
-name: The Harbor District
-associated_characters: [elena, marcus]
-tags: [urban, working-class, recurring]
----
-
-The harbor district is loud and smells of brine and diesel. The buildings closest to the water are old enough to have survived two floods and a fire. Most of the businesses that used to operate here have moved inland; the ones that remain are either too stubborn or too poor to follow.
-
-It is the kind of place people are from, not the kind of place people choose.
-`
-  );
-
-  writeFileSyncWithDirs(
-    path.join(syncDir, "projects", "test-novel", "world", "places", "harbor-district.meta.yaml"),
-    `place_id: harbor-district
-name: The Harbor District
-associated_characters:
-  - elena
-  - marcus
-tags:
-  - urban
-  - working-class
-  - recurring
-`
-  );
-}
-
-export function createScrivenerDraftFixture(baseDir) {
-  const draftDir = path.join(baseDir, "Draft");
-  fs.mkdirSync(draftDir, { recursive: true });
-
-  fs.writeFileSync(
-    path.join(draftDir, "001 Scene Arrival [10].txt"),
-    "Elena arrives at the station and scans for familiar faces.\n",
-    "utf8"
-  );
-
-  fs.writeFileSync(path.join(draftDir, "002 -Setup- [11].txt"), "", "utf8");
-
-  fs.writeFileSync(
-    path.join(draftDir, "003 Epigraph [12].txt"),
-    "A city remembers what its people forget.\n",
-    "utf8"
-  );
-
-  fs.writeFileSync(
-    path.join(draftDir, "004 Scene Debate [13].txt"),
-    "Marcus challenges Elena's plan in the stairwell.\n",
-    "utf8"
-  );
-
-  fs.writeFileSync(path.join(draftDir, "005 Chapter Card [14].txt"), "", "utf8");
-  fs.writeFileSync(path.join(draftDir, "006 Notes.txt"), "Not in expected filename format.\n", "utf8");
-}
-
-export function createScrivenerProjectBundleFixture(baseDir) {
-  const scrivDir = path.join(baseDir, "Sebastian the Vampire.scriv");
-  const scrivxPath = path.join(scrivDir, "Sebastian the Vampire.scrivx");
-  fs.mkdirSync(path.join(scrivDir, "Files", "Data", "UUID-10"), { recursive: true });
-  fs.mkdirSync(path.join(scrivDir, "Files", "Data", "UUID-13"), { recursive: true });
-
-  fs.writeFileSync(
-    path.join(scrivDir, "Files", "Data", "UUID-10", "synopsis.txt"),
-    "Elena arrives at the station and scans for familiar faces.\n",
-    "utf8"
-  );
-  fs.writeFileSync(
-    path.join(scrivDir, "Files", "Data", "UUID-13", "synopsis.txt"),
-    "Marcus challenges Elena's plan in the stairwell.\n",
-    "utf8"
-  );
-
-  const xml = `<?xml version="1.0" encoding="UTF-8"?>
-<ScrivenerProject>
-  <ExternalSyncMap>
-    <SyncItem ID="UUID-10">10</SyncItem>
-    <SyncItem ID="UUID-13">13</SyncItem>
-  </ExternalSyncMap>
-  <Keywords>
-    <Keyword ID="kw-elena"><Title>Elena Voss</Title></Keyword>
-    <Keyword ID="kw-version"><Title>v1.1</Title></Keyword>
-  </Keywords>
-  <Binder>
-    <BinderItem Type="DraftFolder" UUID="draft-root">
-      <Children>
-        <BinderItem Type="Folder" UUID="part-1">
-          <Title>Part One</Title>
-          <Children>
-            <BinderItem Type="Folder" UUID="chapter-1">
-              <Title>Arrival</Title>
-              <Children>
-                <BinderItem Type="Text" UUID="UUID-10">
-                  <Keywords>
-                    <KeywordID>kw-elena</KeywordID>
-                    <KeywordID>kw-version</KeywordID>
-                  </Keywords>
-                  <MetaData>
-                    <MetaDataItem><FieldID>savethecat!</FieldID><Value>Setup</Value></MetaDataItem>
-                    <MetaDataItem><FieldID>causality</FieldID><Value>2</Value></MetaDataItem>
-                    <MetaDataItem><FieldID>f:character</FieldID><Value>Yes</Value></MetaDataItem>
-                  </MetaData>
-                </BinderItem>
-                <BinderItem Type="Text" UUID="UUID-13">
-                  <MetaData>
-                    <MetaDataItem><FieldID>stakes</FieldID><Value>3</Value></MetaDataItem>
-                  </MetaData>
-                </BinderItem>
-              </Children>
-            </BinderItem>
-          </Children>
-        </BinderItem>
-      </Children>
-    </BinderItem>
-  </Binder>
-</ScrivenerProject>`;
-
-  fs.mkdirSync(scrivDir, { recursive: true });
-  fs.writeFileSync(scrivxPath, xml, "utf8");
-  return scrivDir;
-}

package/src/test/helpers/server.js

@@ -1,137 +0,0 @@
-import { spawn } from "node:child_process";
-import { fileURLToPath } from "node:url";
-import path from "node:path";
-import fs from "node:fs";
-import os from "node:os";
-import { Client } from "@modelcontextprotocol/sdk/client/index.js";
-import { SSEClientTransport } from "@modelcontextprotocol/sdk/client/sse.js";
-import { createTestSyncFixture, copyDirSync } from "./fixtures.js";
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-const ROOT = path.resolve(__dirname, "../..");
-
-export function spawnServer(port, syncDir, extraEnv = {}) {
-  const proc = spawn(
-    process.execPath,
-    ["--experimental-sqlite", path.join(ROOT, "index.js")],
-    {
-      env: {
-        ...process.env,
-        WRITING_SYNC_DIR: syncDir,
-        DB_PATH: ":memory:",
-        HTTP_PORT: String(port),
-        ...extraEnv,
-      },
-      stdio: ["ignore", "ignore", "pipe"],
-    }
-  );
-  proc.on("error", (err) => {
-    throw new Error(`Failed to start server: ${err.message}`);
-  });
-  return proc;
-}
-
-export async function waitForServer(url, proc = null, retries = 20, delayMs = 200) {
-  let stderr = "";
-  if (proc?.stderr) {
-    proc.stderr.on("data", (chunk) => { stderr += chunk.toString("utf8"); });
-  }
-  for (let i = 0; i < retries; i++) {
-    try {
-      const res = await fetch(`${url}/healthz`);
-      if (res.ok) return;
-    } catch {}
-    await new Promise((r) => setTimeout(r, delayMs));
-  }
-  const hint = stderr.trim() ? `\nServer stderr:\n${stderr.trim()}` : "";
-  throw new Error(`Server at ${url} did not become ready${hint}`);
-}
-
-export async function waitForExit(proc, timeoutMs = 5000) {
-  return await new Promise((resolve, reject) => {
-    const timeout = setTimeout(
-      () => reject(new Error("Process did not exit in time")),
-      timeoutMs
-    );
-    proc.once("exit", (code, signal) => {
-      clearTimeout(timeout);
-      resolve({ code, signal });
-    });
-  });
-}
-
-export async function connectClient(url) {
-  const c = new Client({ name: "integration-test-client", version: "1.0.0" });
-  const transport = new SSEClientTransport(new URL(`${url}/sse`));
-  await c.connect(transport);
-  return c;
-}
-
-/**
- * Creates a self-contained server context for integration test files.
- * Each file gets its own read-only and writable server pair.
- *
- * Usage:
- *   const ctx = createTestContext(3079, 3078);
- *   before(() => ctx.setup());
- *   after(() => ctx.teardown());
- *   const callTool = (n, a) => ctx.callTool(n, a);
- */
-export function createTestContext(readPort, writePort, extraEnv = {}) {
-  let serverProc, writeServerProc, client, writeClient;
-  let readSyncDir, writeSyncDir;
-
-  const ctx = {
-    get readSyncDir() { return readSyncDir; },
-    get writeSyncDir() { return writeSyncDir; },
-    get client() { return client; },
-    get writeClient() { return writeClient; },
-
-    async setup() {
-      readSyncDir = fs.mkdtempSync(path.join(os.tmpdir(), "mcp-writing-read-"));
-      createTestSyncFixture(readSyncDir);
-      serverProc = spawnServer(readPort, readSyncDir, { DEFAULT_METADATA_PAGE_SIZE: "2", ...extraEnv });
-      await waitForServer(`http://localhost:${readPort}`, serverProc);
-      client = await connectClient(`http://localhost:${readPort}`);
-
-      writeSyncDir = fs.mkdtempSync(path.join(os.tmpdir(), "mcp-writing-write-"));
-      copyDirSync(readSyncDir, writeSyncDir);
-      writeServerProc = spawnServer(writePort, writeSyncDir, { DEFAULT_METADATA_PAGE_SIZE: "2", ...extraEnv });
-      await waitForServer(`http://localhost:${writePort}`, writeServerProc);
-      writeClient = await connectClient(`http://localhost:${writePort}`);
-    },
-
-    async teardown() {
-      if (client) try { await client.close(); } catch {}
-      if (writeClient) try { await writeClient.close(); } catch {}
-      if (serverProc) serverProc.kill();
-      if (writeServerProc) writeServerProc.kill();
-      if (readSyncDir) fs.rmSync(readSyncDir, { recursive: true, force: true });
-      if (writeSyncDir) fs.rmSync(writeSyncDir, { recursive: true, force: true });
-    },
-
-    async callTool(name, args = {}) {
-      const result = await client.callTool({ name, arguments: args });
-      return result.content?.[0]?.text ?? "";
-    },
-
-    async callWriteTool(name, args = {}) {
-      const result = await writeClient.callTool({ name, arguments: args });
-      return result.content?.[0]?.text ?? "";
-    },
-
-    async waitForAsyncJob(jobId, timeoutMs = 12000) {
-      const start = Date.now();
-      while (Date.now() - start < timeoutMs) {
-        const text = await ctx.callWriteTool("get_async_job_status", { job_id: jobId });
-        const parsed = JSON.parse(text);
-        const status = parsed.job?.status;
-        if (status === "completed" || status === "failed" || status === "cancelled") return parsed;
-        await new Promise((r) => setTimeout(r, 100));
-      }
-      throw new Error(`Timed out waiting for async job ${jobId}`);
-    },
-  };
-
-  return ctx;
-}