@hanna84/mcp-writing 2.12.15 → 2.12.17

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.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)
 
+> 29 April 2026
+
 - chore(ci): skip CI jobs on automated release commits [`#130`](https://github.com/hannasdev/mcp-writing.git
 /pull/130)
+- Release 2.12.15 [`392e0fd`](https://github.com/hannasdev/mcp-writing.git
+/commit/392e0fdf311d6873d95f63a73373ba79f33cb731)
 
 #### [v2.12.14](https://github.com/hannasdev/mcp-writing.git
 /compare/v2.12.13...v2.12.14)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hanna84/mcp-writing",
-  "version": "2.12.15",
+  "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",
@@ -30,7 +30,7 @@
     "./prose-styleguide-drift.js": "./src/styleguide/prose-styleguide-drift.js",
     "./prose-styleguide-skill.js": "./src/styleguide/prose-styleguide-skill.js",
     "./tools/*": "./src/tools/*",
-    "./scripts/*": "./scripts/*",
+    "./scripts/*": "./src/scripts/*",
     "./src/*": "./src/*",
     "./package.json": "./package.json"
   },
@@ -41,8 +41,16 @@
   "files": [
     "bin/",
     "index.js",
-    "src/",
-    "scripts/",
+    "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"
   ],
@@ -51,22 +59,22 @@
   },
   "scripts": {
     "start": "node --experimental-sqlite index.js",
-    "new:entity": "node scripts/new-world-entity.js",
-    "manual:realtest": "node scripts/manual-scrivener-realtest.mjs",
-    "manual:test": "node scripts/manual/test.mjs",
-    "manual:scenarios": "node scripts/manual/test-scenarios.mjs",
-    "manual:merge-beta-test": "node scripts/manual/run_mcp_test.js",
-    "manual:review-bundle": "node scripts/manual/run_create_review_bundle.js",
-    "normalize:scene-characters": "node --experimental-sqlite scripts/normalize-scene-characters.mjs",
-    "setup:openclaw-env": "sh scripts/setup-openclaw-env.sh",
+    "new:entity": "node src/scripts/new-world-entity.js",
+    "manual:realtest": "node src/scripts/manual-scrivener-realtest.mjs",
+    "manual:test": "node src/scripts/manual/test.mjs",
+    "manual:scenarios": "node src/scripts/manual/test-scenarios.mjs",
+    "manual:merge-beta-test": "node src/scripts/manual/run_mcp_test.js",
+    "manual:review-bundle": "node src/scripts/manual/run_create_review_bundle.js",
+    "normalize:scene-characters": "node --experimental-sqlite src/scripts/normalize-scene-characters.mjs",
+    "setup:openclaw-env": "sh src/scripts/setup-openclaw-env.sh",
     "release": "release-it",
-    "lint": "eslint *.js src/ scripts/",
-    "guard:legacy-root-imports": "node scripts/check-legacy-root-imports.mjs",
-    "docs": "node scripts/generate-tool-docs.mjs",
-    "lint:metadata": "node scripts/lint-metadata.mjs",
-    "sync:server-json-version": "node scripts/sync-server-json-version.mjs",
-    "test:unit": "node --experimental-sqlite --test test/unit/*.test.mjs",
-    "test:integration": "node --experimental-sqlite --test --test-concurrency=1 test/integration/*.test.mjs",
+    "lint": "eslint *.js src/index.js src/core src/review-bundles src/runtime src/scripts src/styleguide src/sync src/tools src/workflows src/world",
+    "guard:legacy-root-imports": "node src/scripts/check-legacy-root-imports.mjs",
+    "docs": "node src/scripts/generate-tool-docs.mjs",
+    "lint:metadata": "node src/scripts/lint-metadata.mjs",
+    "sync:server-json-version": "node src/scripts/sync-server-json-version.mjs",
+    "test:unit": "node --experimental-sqlite --test src/test/unit/*.test.mjs",
+    "test:integration": "node --experimental-sqlite --test --test-concurrency=1 src/test/integration/*.test.mjs",
     "test": "npm run test:unit && npm run test:integration"
   },
   "keywords": [

package/src/runtime/runtime-diagnostics.js

@@ -91,7 +91,7 @@ export function getRuntimeDiagnostics({
     recommendations.push("If git reports 'dubious ownership' for mounted repos, add: git config --system --add safe.directory /sync");
   }
 
-  recommendations.push("If indexing finds many files without scene_id, run scripts/import.js first for Scrivener Draft exports, then run sync.");
+  recommendations.push("If indexing finds many files without scene_id, run src/scripts/import.js first for Scrivener Draft exports, then run sync.");
 
   return { warnings, recommendations };
 }

package/src/scripts/async-job-runner.mjs

@@ -92,7 +92,7 @@ async function main() {
   const resultPath = process.argv[3];
 
   if (!requestPath || !resultPath) {
-    throw new Error("Usage: node scripts/async-job-runner.mjs <request.json> <result.json>");
+    throw new Error("Usage: node src/scripts/async-job-runner.mjs <request.json> <result.json>");
   }
 
   const request = JSON.parse(fs.readFileSync(requestPath, "utf8"));

package/{scripts → src/scripts}/check-legacy-root-imports.mjs

@@ -4,9 +4,9 @@ import { fileURLToPath } from "node:url";
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
-const ROOT = path.resolve(__dirname, "..");
+const ROOT = path.resolve(__dirname, "../..");
 
-const SCAN_PATHS = ["src", "scripts", "test", "index.js"];
+const SCAN_PATHS = ["src", "index.js"];
 const SOURCE_EXTENSIONS = new Set([".js", ".mjs"]);
 const LEGACY_ROOT_MODULES = new Set([
   "async-jobs.js",

package/{scripts → src/scripts}/generate-tool-docs.mjs

@@ -3,7 +3,7 @@
  * Generates docs/tools.md from tool definitions in the runtime entrypoint
  * (src/index.js when present, otherwise index.js) and src/tools/*.js.
  *
- * Run:  node scripts/generate-tool-docs.mjs
+ * Run:  node src/scripts/generate-tool-docs.mjs
  *   or: npm run docs
  *
  * The output is the single source of truth for the tool reference.
@@ -14,7 +14,7 @@ import { readFileSync, writeFileSync, mkdirSync, readdirSync } from 'node:fs';
 import path from 'node:path';
 import { fileURLToPath } from 'node:url';
 
-const ROOT = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..');
+const ROOT = path.resolve(path.dirname(fileURLToPath(import.meta.url)), "../..");
 const OUT  = path.join(ROOT, 'docs', 'tools.md');
 
 // Build a map from each registration function name to its module source.

package/{scripts → src/scripts}/import.js

@@ -3,14 +3,14 @@
  * Import a Scrivener External Folder Sync output into mcp-writing sidecar format.
  *
  * Usage:
- *   node scripts/import.js <scrivener-sync-dir> <mcp-sync-dir> [--project <id>] [--dry-run]
+ *   node src/scripts/import.js <scrivener-sync-dir> <mcp-sync-dir> [--project <id>] [--dry-run]
  */
 
 import path from "node:path";
-import { importScrivenerSync, validateProjectId } from "../src/sync/importer.js";
+import { importScrivenerSync, validateProjectId } from "../sync/importer.js";
 
 function printUsage() {
-  console.log("Usage: node scripts/import.js <scrivener-sync-dir> <mcp-sync-dir> [--project <id>] [--dry-run]");
+  console.log("Usage: node src/scripts/import.js <scrivener-sync-dir> <mcp-sync-dir> [--project <id>] [--dry-run]");
 }
 
 const args = process.argv.slice(2);

package/{scripts → src/scripts}/lint-metadata.mjs

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 import path from "node:path";
-import { lintMetadataInSyncDir } from "../src/sync/metadata-lint.js";
+import { lintMetadataInSyncDir } from "../sync/metadata-lint.js";
 
 function parseArgs(argv) {
   const args = { syncDir: process.env.WRITING_SYNC_DIR ?? "./sync" };

package/{scripts → src/scripts}/manual/run_create_review_bundle.js

@@ -7,14 +7,14 @@ import fs from "node:fs";
 async function main() {
   const [projectId, profile, outputDir, ...flags] = process.argv.slice(2);
   if (!projectId || !profile || !outputDir) {
-    console.error("Usage: WRITING_SYNC_DIR=/path/to/sync node scripts/manual/run_create_review_bundle.js <project_id> <profile> <output_dir> [--anchors] [--recipient <name>] [--bundle-name <name>] [--skip-preview] [--show-files]");
+    console.error("Usage: WRITING_SYNC_DIR=/path/to/sync node src/scripts/manual/run_create_review_bundle.js <project_id> <profile> <output_dir> [--anchors] [--recipient <name>] [--bundle-name <name>] [--skip-preview] [--show-files]");
     process.exit(1);
   }
 
   if (!process.env.WRITING_SYNC_DIR) {
     console.error(
       "WRITING_SYNC_DIR is required. Set it to the root of your sync directory.\n" +
-      "Usage: WRITING_SYNC_DIR=/path/to/sync node scripts/manual/run_create_review_bundle.js <project_id> <profile> <output_dir>"
+      "Usage: WRITING_SYNC_DIR=/path/to/sync node src/scripts/manual/run_create_review_bundle.js <project_id> <profile> <output_dir>"
     );
     process.exit(1);
   }

package/{scripts → src/scripts}/manual/run_mcp_and_review.js

@@ -17,7 +17,7 @@ async function main() {
 
   if (!projectId) {
     console.error(
-      "Usage: WRITING_SYNC_DIR=/path/to/sync node scripts/manual/run_mcp_and_review.js <projectId> [outputDir]"
+      "Usage: WRITING_SYNC_DIR=/path/to/sync node src/scripts/manual/run_mcp_and_review.js <projectId> [outputDir]"
     );
     process.exit(1);
   }
@@ -25,7 +25,7 @@ async function main() {
   if (!envWritingSyncDir) {
     console.error(
       "WRITING_SYNC_DIR is required. Set it to the root of your sync directory.\n" +
-      "Usage: WRITING_SYNC_DIR=/path/to/sync node scripts/manual/run_mcp_and_review.js <projectId> [outputDir]"
+      "Usage: WRITING_SYNC_DIR=/path/to/sync node src/scripts/manual/run_mcp_and_review.js <projectId> [outputDir]"
     );
     process.exit(1);
   }

package/{scripts → src/scripts}/manual/run_mcp_test.js

@@ -4,8 +4,8 @@ import path from "path";
 import process from "process";
 
 function usage() {
-  console.log("Usage: node scripts/manual/run_mcp_test.js <source_project_dir> [project_id] [sync_dir]");
-  console.log("Example: node scripts/manual/run_mcp_test.js ~/Novel.scriv demo-project ~/sync-root");
+  console.log("Usage: node src/scripts/manual/run_mcp_test.js <source_project_dir> [project_id] [sync_dir]");
+  console.log("Example: node src/scripts/manual/run_mcp_test.js ~/Novel.scriv demo-project ~/sync-root");
 }
 
 async function runCase(env, args) {

package/{scripts → src/scripts}/manual-scrivener-realtest.mjs

@@ -1,13 +1,13 @@
 import fs from "node:fs";
 import path from "node:path";
 import os from "node:os";
-import { importScrivenerSync, validateProjectId } from "../src/sync/importer.js";
-import { mergeScrivenerProjectMetadata } from "../src/sync/scrivener-direct.js";
+import { importScrivenerSync, validateProjectId } from "../sync/importer.js";
+import { mergeScrivenerProjectMetadata } from "../sync/scrivener-direct.js";
 
 function usage() {
   return [
     "Usage:",
-    "  node scripts/manual-scrivener-realtest.mjs \\",
+    "  node src/scripts/manual-scrivener-realtest.mjs \\",
     "    --source-dir <external-sync-dir> \\",
     "    --scriv-path <copied-project.scriv> \\",
     "    --project-id <project|universe/project> [options]",

package/{scripts → src/scripts}/manual-validation.mjs

@@ -137,7 +137,7 @@ async function runPhaseB() {
   // Import
   try {
     const importOutput = execSync(
-      `node scripts/import.js ./txt ${IMPORT_DIR} --project scrivener-export`,
+      `node src/scripts/import.js ./txt ${IMPORT_DIR} --project scrivener-export`,
       { cwd: ROOT, encoding: "utf8", stdio: ["pipe", "pipe", "pipe"] }
     );
     console.log("✓ Import completed");
@@ -151,7 +151,7 @@ async function runPhaseB() {
   // Lint
   try {
     const lintOutput = execSync(
-      `node scripts/lint-metadata.mjs --sync-dir ${IMPORT_DIR}`,
+      `node src/scripts/lint-metadata.mjs --sync-dir ${IMPORT_DIR}`,
       { cwd: ROOT, encoding: "utf8", stdio: ["pipe", "pipe", "pipe"] }
     );
     results.lintOutput = lintOutput;

package/{scripts → src/scripts}/merge-scrivx.js

@@ -3,7 +3,7 @@
  * Merge Scrivener project metadata into mcp-writing sidecar files.
  *
  * Usage:
- *   node scripts/merge-scrivx.js <path-to.scriv> <mcp-sync-dir> [options]
+ *   node src/scripts/merge-scrivx.js <path-to.scriv> <mcp-sync-dir> [options]
  *
  *   <path-to.scriv>   Path to the Scrivener .scriv bundle (the folder)
  *   <mcp-sync-dir>    The WRITING_SYNC_DIR root (e.g. ./sync)
@@ -27,15 +27,15 @@
  */
 
 import path from "node:path";
-import { validateProjectId } from "../src/sync/importer.js";
-import { mergeScrivenerProjectMetadata } from "../src/sync/scrivener-direct.js";
+import { validateProjectId } from "../sync/importer.js";
+import { mergeScrivenerProjectMetadata } from "../sync/scrivener-direct.js";
 
 // ---------------------------------------------------------------------------
 // Args
 // ---------------------------------------------------------------------------
 const args = process.argv.slice(2);
 if (args.length < 2 || args[0] === "--help") {
-  console.log("Usage: node scripts/merge-scrivx.js <path-to.scriv> <mcp-sync-dir> [--project <id>] [--dry-run] [--organize-by-chapters]");
+  console.log("Usage: node src/scripts/merge-scrivx.js <path-to.scriv> <mcp-sync-dir> [--project <id>] [--dry-run] [--organize-by-chapters]");
   process.exit(args[0] === "--help" ? 0 : 1);
 }
 

package/{scripts → src/scripts}/new-world-entity.js

@@ -6,7 +6,7 @@ import {
   renderCharacterSheetTemplate,
   renderPlaceSheetTemplate,
   slugifyEntityName,
-} from "../src/world/world-entity-templates.js";
+} from "../world/world-entity-templates.js";
 
 function parseArgs(argv) {
   const args = argv.slice(2);
@@ -39,11 +39,11 @@ function parseArgs(argv) {
 function usage() {
   return [
     "Usage:",
-    "  node scripts/new-world-entity.js --sync-dir <dir> --kind <character|place> --scope <project|universe> --name <display name> [--project <project-id>] [--universe <universe-id>] [--dry-run]",
+    "  node src/scripts/new-world-entity.js --sync-dir <dir> --kind <character|place> --scope <project|universe> --name <display name> [--project <project-id>] [--universe <universe-id>] [--dry-run]",
     "",
     "Examples:",
-    "  node scripts/new-world-entity.js --sync-dir ./writing --kind character --scope universe --universe universe-1 --name 'Mira Nystrom'",
-    "  node scripts/new-world-entity.js --sync-dir ./writing --kind place --scope project --project universe-1/book-1-the-lamb --name 'University Hospital'",
+    "  node src/scripts/new-world-entity.js --sync-dir ./writing --kind character --scope universe --universe universe-1 --name 'Mira Nystrom'",
+    "  node src/scripts/new-world-entity.js --sync-dir ./writing --kind place --scope project --project universe-1/book-1-the-lamb --name 'University Hospital'",
   ].join("\n");
 }
 

package/{scripts → src/scripts}/normalize-scene-characters.mjs

@@ -1,8 +1,8 @@
 #!/usr/bin/env node
 import path from "node:path";
-import { openDb } from "../src/core/db.js";
-import { buildCharacterNormalizationContext, normalizeSceneCharacters } from "../src/sync/scene-character-normalization.js";
-import { normalizeSceneMetaForPath, readMeta, syncAll, writeMeta } from "../src/sync/sync.js";
+import { openDb } from "../core/db.js";
+import { buildCharacterNormalizationContext, normalizeSceneCharacters } from "../sync/scene-character-normalization.js";
+import { normalizeSceneMetaForPath, readMeta, syncAll, writeMeta } from "../sync/sync.js";
 
 function readRequiredValue(argv, index, option) {
   const value = argv[index + 1];
@@ -54,7 +54,7 @@ function parseArgs(argv) {
 function usage() {
   return [
     "Usage:",
-    "  node --experimental-sqlite scripts/normalize-scene-characters.mjs [--sync-dir <dir>] [--project-id <id>] [--limit <n>] [--write] [--json]",
+    "  node --experimental-sqlite src/scripts/normalize-scene-characters.mjs [--sync-dir <dir>] [--project-id <id>] [--limit <n>] [--write] [--json]",
     "",
     "Options:",
     "  --sync-dir, -d   WRITING_SYNC_DIR root (default: env WRITING_SYNC_DIR or ./sync)",

package/{scripts → src/scripts}/profile-review-bundles.mjs

@@ -2,19 +2,19 @@
 
 /**
  * Profile review-bundle generation performance
- * Usage: node --experimental-sqlite scripts/profile-review-bundles.mjs
+ * Usage: node --experimental-sqlite src/scripts/profile-review-bundles.mjs
  */
 
 import fs from "node:fs";
 import path from "node:path";
 import os from "node:os";
-import { openDb } from "../src/core/db.js";
-import { syncAll } from "../src/sync/sync.js";
-import { isGitRepository, getHeadCommitHash } from "../src/core/git.js";
+import { openDb } from "../core/db.js";
+import { syncAll } from "../sync/sync.js";
+import { isGitRepository, getHeadCommitHash } from "../core/git.js";
 import {
   buildReviewBundlePlan,
   createReviewBundleArtifacts,
-} from "../src/review-bundles/review-bundles.js";
+} from "../review-bundles/review-bundles.js";
 
 const PROJECT_SYNC_DIR = process.env.WRITING_SYNC_DIR ?? process.argv[2] ?? null;
 const DB_PATH = process.env.DB_PATH ?? (PROJECT_SYNC_DIR ? path.join(PROJECT_SYNC_DIR, ".mcp", "writing.db") : null);
@@ -119,7 +119,7 @@ async function profileScenario(
 async function main() {
   if (!PROJECT_SYNC_DIR) {
     console.error("✗ WRITING_SYNC_DIR env var or a path argument is required.");
-    console.error("  Usage: WRITING_SYNC_DIR=/path/to/project node --experimental-sqlite scripts/profile-review-bundles.mjs");
+    console.error("  Usage: WRITING_SYNC_DIR=/path/to/project node --experimental-sqlite src/scripts/profile-review-bundles.mjs");
     process.exit(1);
   }
 

package/{scripts → src/scripts}/sync-server-json-version.mjs

@@ -4,7 +4,7 @@ import { fileURLToPath } from "node:url";
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
-const ROOT = path.resolve(__dirname, "..");
+const ROOT = path.resolve(__dirname, "../..");
 const packageJsonPath = path.join(ROOT, "package.json");
 const serverJsonPath = path.join(ROOT, "server.json");
 

package/src/tools/sync.js

@@ -31,7 +31,7 @@ export function registerSyncTools(s, {
     if (result.sidecarsMigrated) parts.push(`${result.sidecarsMigrated} sidecar(s) auto-generated from frontmatter.`);
     if (result.skipped) {
       parts.push(`${result.skipped} file(s) skipped (no scene_id).`);
-      parts.push(`Tip: for raw Scrivener Draft exports, run scripts/import.js first, then run sync again.`);
+      parts.push(`Tip: for raw Scrivener Draft exports, run src/scripts/import.js first, then run sync again.`);
     }
     const summary = result.warningSummary;
     const summaryEntries = Object.entries(summary);

package/scripts/async-job-runner.mjs

@@ -1 +0,0 @@
-import "../src/scripts/async-job-runner.mjs";