@hanna84/mcp-writing 2.12.10 → 2.12.12

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.12](https://github.com/hannasdev/mcp-writing.git
+/compare/v2.12.11...v2.12.12)
+
+- refactor(root): remove compatibility shims after src migration [`#127`](https://github.com/hannasdev/mcp-writing.git
+/pull/127)
+
+#### [v2.12.11](https://github.com/hannasdev/mcp-writing.git
+/compare/v2.12.10...v2.12.11)
+
+> 29 April 2026
+
+- refactor(core): move helpers under src/core [`#126`](https://github.com/hannasdev/mcp-writing.git
+/pull/126)
+- Release 2.12.11 [`dff03db`](https://github.com/hannasdev/mcp-writing.git
+/commit/dff03db488274a1753e1b6bc49fb48173ab61115)
+
 #### [v2.12.10](https://github.com/hannasdev/mcp-writing.git
 /compare/v2.12.9...v2.12.10)
 
+> 29 April 2026
+
 - refactor(scripts): move async job runner under src/scripts [`#125`](https://github.com/hannasdev/mcp-writing.git
 /pull/125)
+- Release 2.12.10 [`fa20160`](https://github.com/hannasdev/mcp-writing.git
+/commit/fa20160423211dd6ab7416a82fdf3365b69ae6d1)
 
 #### [v2.12.9](https://github.com/hannasdev/mcp-writing.git
 /compare/v2.12.8...v2.12.9)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hanna84/mcp-writing",
-  "version": "2.12.10",
+  "version": "2.12.12",
   "description": "MCP service for AI-assisted reasoning and editing on long-form fiction projects",
   "homepage": "https://hannasdev.github.io/mcp-writing/",
   "type": "module",
@@ -8,27 +8,27 @@
   "exports": {
     ".": "./index.js",
     "./index.js": "./index.js",
-    "./async-jobs.js": "./async-jobs.js",
-    "./async-progress.js": "./async-progress.js",
-    "./helpers.js": "./helpers.js",
-    "./scene-character-batch.js": "./scene-character-batch.js",
-    "./scrivener-direct.js": "./scrivener-direct.js",
-    "./importer.js": "./importer.js",
-    "./db.js": "./db.js",
-    "./sync.js": "./sync.js",
-    "./git.js": "./git.js",
-    "./world-entity-templates.js": "./world-entity-templates.js",
-    "./workflow-catalogue.js": "./workflow-catalogue.js",
-    "./runtime-diagnostics.js": "./runtime-diagnostics.js",
-    "./metadata-lint.js": "./metadata-lint.js",
-    "./scene-character-normalization.js": "./scene-character-normalization.js",
-    "./review-bundles.js": "./review-bundles.js",
-    "./review-bundles-planner.js": "./review-bundles-planner.js",
-    "./review-bundles-renderer.js": "./review-bundles-renderer.js",
-    "./review-bundles-writer.js": "./review-bundles-writer.js",
-    "./prose-styleguide.js": "./prose-styleguide.js",
-    "./prose-styleguide-drift.js": "./prose-styleguide-drift.js",
-    "./prose-styleguide-skill.js": "./prose-styleguide-skill.js",
+    "./async-jobs.js": "./src/runtime/async-jobs.js",
+    "./async-progress.js": "./src/runtime/async-progress.js",
+    "./helpers.js": "./src/core/helpers.js",
+    "./scene-character-batch.js": "./src/sync/scene-character-batch.js",
+    "./scrivener-direct.js": "./src/sync/scrivener-direct.js",
+    "./importer.js": "./src/sync/importer.js",
+    "./db.js": "./src/core/db.js",
+    "./sync.js": "./src/sync/sync.js",
+    "./git.js": "./src/core/git.js",
+    "./world-entity-templates.js": "./src/world/world-entity-templates.js",
+    "./workflow-catalogue.js": "./src/workflows/workflow-catalogue.js",
+    "./runtime-diagnostics.js": "./src/runtime/runtime-diagnostics.js",
+    "./metadata-lint.js": "./src/sync/metadata-lint.js",
+    "./scene-character-normalization.js": "./src/sync/scene-character-normalization.js",
+    "./review-bundles.js": "./src/review-bundles/review-bundles.js",
+    "./review-bundles-planner.js": "./src/review-bundles/review-bundles-planner.js",
+    "./review-bundles-renderer.js": "./src/review-bundles/review-bundles-renderer.js",
+    "./review-bundles-writer.js": "./src/review-bundles/review-bundles-writer.js",
+    "./prose-styleguide.js": "./src/styleguide/prose-styleguide.js",
+    "./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/*",
     "./src/*": "./src/*",
@@ -42,27 +42,6 @@
     "bin/",
     "index.js",
     "src/",
-    "async-jobs.js",
-    "async-progress.js",
-    "helpers.js",
-    "scene-character-batch.js",
-    "scrivener-direct.js",
-    "importer.js",
-    "db.js",
-    "sync.js",
-    "git.js",
-    "world-entity-templates.js",
-    "workflow-catalogue.js",
-    "runtime-diagnostics.js",
-    "metadata-lint.js",
-    "scene-character-normalization.js",
-    "review-bundles.js",
-    "review-bundles-planner.js",
-    "review-bundles-renderer.js",
-    "review-bundles-writer.js",
-    "prose-styleguide.js",
-    "prose-styleguide-drift.js",
-    "prose-styleguide-skill.js",
     "scripts/",
     "README.md",
     "CHANGELOG.md"

package/{helpers.js → src/core/helpers.js}

@@ -2,14 +2,21 @@ import fs from "node:fs";
 import path from "node:path";
 import matter from "gray-matter";
 import yaml from "js-yaml";
-import { sidecarPath, syncAll } from "./src/sync/sync.js";
+import { sidecarPath, syncAll } from "../sync/sync.js";
 import {
   slugifyEntityName,
   renderCharacterSheetTemplate,
   renderPlaceSheetTemplate,
   renderCharacterArcTemplate,
-} from "./src/world/world-entity-templates.js";
-import { ReviewBundlePlanError } from "./src/review-bundles/review-bundles.js";
+} from "../world/world-entity-templates.js";
+
+function createCoreValidationError(code, message, details) {
+  const error = new Error(message);
+  error.name = "CoreValidationError";
+  error.code = code;
+  error.details = details;
+  return error;
+}
 
 export function deriveLoglineFromProse(prose) {
   const compact = prose.replace(/\s+/g, " ").trim();
@@ -197,7 +204,7 @@ export function createHelpers({ syncDir, syncDirReal, syncDirAbs, db, syncDirWri
     while (!fs.existsSync(existingAncestor)) {
       const parentDir = path.dirname(existingAncestor);
       if (parentDir === existingAncestor) {
-        throw new ReviewBundlePlanError(
+        throw createCoreValidationError(
           "INVALID_OUTPUT_DIR",
           "output_dir must be inside WRITING_SYNC_DIR.",
           { output_dir: resolvedOutputDir, sync_dir: syncDirAbs }
@@ -210,10 +217,14 @@ export function createHelpers({ syncDir, syncDirReal, syncDirAbs, db, syncDirWri
     try {
       realExistingAncestor = fs.realpathSync.native(existingAncestor);
     } catch (err) {
-      throw new ReviewBundlePlanError(
+      throw createCoreValidationError(
         "INVALID_OUTPUT_DIR",
         "output_dir ancestor could not be resolved: path may be inaccessible.",
-        { output_dir: outputDir, existing_ancestor: existingAncestor, cause: err.message }
+        {
+          output_dir: outputDir,
+          existing_ancestor: existingAncestor,
+          cause: err instanceof Error ? err.message : String(err),
+        }
       );
     }
     const relativeFromAncestor = path.relative(existingAncestor, resolvedOutputDir);
@@ -221,7 +232,7 @@ export function createHelpers({ syncDir, syncDirReal, syncDirAbs, db, syncDirWri
 
     const relativeToSyncDir = path.relative(syncDirReal, resolvedOutputDir);
     if (relativeToSyncDir.startsWith("..") || path.isAbsolute(relativeToSyncDir)) {
-      throw new ReviewBundlePlanError(
+      throw createCoreValidationError(
         "INVALID_OUTPUT_DIR",
         "output_dir must be inside WRITING_SYNC_DIR.",
         { output_dir: resolvedOutputDir, sync_dir: syncDirAbs }

package/src/index.js

@@ -17,7 +17,7 @@ import {
   readSupportingNotesForEntity,
   readEntityMetadata,
   resolveBatchTargetScenes,
-} from "../helpers.js";
+} from "./core/helpers.js";
 import { STYLEGUIDE_CONFIG_BASENAME } from "./styleguide/prose-styleguide.js";
 import { registerSyncTools } from "./tools/sync.js";
 import { registerSearchTools } from "./tools/search.js";

package/src/tools/review-bundles.js

@@ -79,6 +79,14 @@ export function registerReviewBundleTools(s, {
         if (error instanceof ReviewBundlePlanError) {
           return errorResponse(error.code, error.message, error.details);
         }
+        if (
+          error &&
+          typeof error === "object" &&
+          error.name === "CoreValidationError" &&
+          typeof error.code === "string"
+        ) {
+          return errorResponse(error.code, error.message ?? "Request failed.", error.details);
+        }
         return errorResponse(
           "PREVIEW_FAILED",
           error instanceof Error ? error.message : "Failed to generate review bundle preview."
@@ -197,6 +205,14 @@ export function registerReviewBundleTools(s, {
         if (error instanceof ReviewBundlePlanError) {
           return errorResponse(error.code, error.message, error.details);
         }
+        if (
+          error &&
+          typeof error === "object" &&
+          error.name === "CoreValidationError" &&
+          typeof error.code === "string"
+        ) {
+          return errorResponse(error.code, error.message ?? "Request failed.", error.details);
+        }
         return errorResponse(
           "CREATE_BUNDLE_FAILED",
           error instanceof Error ? error.message : "Failed to create review bundle artifacts."

package/async-jobs.js

@@ -1 +0,0 @@
-export * from "./src/runtime/async-jobs.js";

package/async-progress.js

@@ -1 +0,0 @@
-export * from "./src/runtime/async-progress.js";

package/db.js

@@ -1 +0,0 @@
-export * from "./src/core/db.js";

package/git.js

@@ -1 +0,0 @@
-export * from "./src/core/git.js";

package/importer.js

@@ -1 +0,0 @@
-export * from "./src/sync/importer.js";

package/metadata-lint.js

@@ -1 +0,0 @@
-export * from "./src/sync/metadata-lint.js";

package/prose-styleguide-drift.js

@@ -1 +0,0 @@
-export * from "./src/styleguide/prose-styleguide-drift.js";

package/prose-styleguide-skill.js

@@ -1 +0,0 @@
-export * from "./src/styleguide/prose-styleguide-skill.js";

package/prose-styleguide.js

@@ -1 +0,0 @@
-export * from "./src/styleguide/prose-styleguide.js";

package/review-bundles-planner.js

@@ -1 +0,0 @@
-export * from "./src/review-bundles/review-bundles-planner.js";

package/review-bundles-renderer.js

@@ -1 +0,0 @@
-export * from "./src/review-bundles/review-bundles-renderer.js";

package/review-bundles-writer.js

@@ -1 +0,0 @@
-export * from "./src/review-bundles/review-bundles-writer.js";

package/review-bundles.js

@@ -1 +0,0 @@
-export * from "./src/review-bundles/review-bundles.js";

package/runtime-diagnostics.js

@@ -1 +0,0 @@
-export * from "./src/runtime/runtime-diagnostics.js";

package/scene-character-batch.js

@@ -1 +0,0 @@
-export * from "./src/sync/scene-character-batch.js";

package/scene-character-normalization.js

@@ -1 +0,0 @@
-export * from "./src/sync/scene-character-normalization.js";

package/scrivener-direct.js

@@ -1 +0,0 @@
-export * from "./src/sync/scrivener-direct.js";

package/sync.js

@@ -1 +0,0 @@
-export * from "./src/sync/sync.js";

package/workflow-catalogue.js

@@ -1 +0,0 @@
-export * from "./src/workflows/workflow-catalogue.js";

package/world-entity-templates.js

@@ -1 +0,0 @@
-export * from "./src/world/world-entity-templates.js";