@hanna84/mcp-writing 2.12.11 → 2.12.13

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.13](https://github.com/hannasdev/mcp-writing.git
+/compare/v2.12.12...v2.12.13)
+
+- refactor(ci): add legacy root import guardrail [`#128`](https://github.com/hannasdev/mcp-writing.git
+/pull/128)
+
+#### [v2.12.12](https://github.com/hannasdev/mcp-writing.git
+/compare/v2.12.11...v2.12.12)
+
+> 29 April 2026
+
+- refactor(root): remove compatibility shims after src migration [`#127`](https://github.com/hannasdev/mcp-writing.git
+/pull/127)
+- Release 2.12.12 [`ec309e6`](https://github.com/hannasdev/mcp-writing.git
+/commit/ec309e654549ed0120e079618811952831249338)
+
 #### [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)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hanna84/mcp-writing",
-  "version": "2.12.11",
+  "version": "2.12.13",
   "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"
@@ -82,6 +61,7 @@
     "setup:openclaw-env": "sh 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",

package/scripts/check-legacy-root-imports.mjs

@@ -0,0 +1,103 @@
+import fs from "node:fs";
+import path from "node:path";
+import { fileURLToPath } from "node:url";
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
+const ROOT = path.resolve(__dirname, "..");
+
+const SCAN_PATHS = ["src", "scripts", "test", "index.js"];
+const SOURCE_EXTENSIONS = new Set([".js", ".mjs"]);
+const LEGACY_ROOT_MODULES = new Set([
+  "async-jobs.js",
+  "async-progress.js",
+  "db.js",
+  "git.js",
+  "helpers.js",
+  "importer.js",
+  "metadata-lint.js",
+  "prose-styleguide.js",
+  "prose-styleguide-drift.js",
+  "prose-styleguide-skill.js",
+  "review-bundles.js",
+  "review-bundles-planner.js",
+  "review-bundles-renderer.js",
+  "review-bundles-writer.js",
+  "runtime-diagnostics.js",
+  "scene-character-batch.js",
+  "scene-character-normalization.js",
+  "scrivener-direct.js",
+  "sync.js",
+  "workflow-catalogue.js",
+  "world-entity-templates.js",
+]);
+
+const findings = [];
+
+function isSourceFile(filePath) {
+  return SOURCE_EXTENSIONS.has(path.extname(filePath));
+}
+
+function toPosix(filePath) {
+  return filePath.replace(/\\/g, "/");
+}
+
+function lineNumberForIndex(content, index) {
+  let line = 1;
+  for (let i = 0; i < index; i += 1) {
+    if (content[i] === "\n") line += 1;
+  }
+  return line;
+}
+
+function hasLegacyRootImport(specifier, importerPath) {
+  if (!specifier || !specifier.startsWith(".")) return false;
+  const resolved = path.resolve(path.dirname(importerPath), specifier);
+  const relativeToRoot = toPosix(path.relative(ROOT, resolved));
+  if (relativeToRoot.includes("/")) return false;
+  return LEGACY_ROOT_MODULES.has(relativeToRoot);
+}
+
+function scanFile(absolutePath) {
+  const source = fs.readFileSync(absolutePath, "utf8");
+  const importRegex =
+    /(?:\bfrom\s*["']([^"']+)["'])|(?:^\s*import\s*["']([^"']+)["'])|(?:\bimport\s*\(\s*["']([^"']+)["']\s*\))/gm;
+
+  for (const match of source.matchAll(importRegex)) {
+    const specifier = match[1] ?? match[2] ?? match[3] ?? null;
+    if (!hasLegacyRootImport(specifier, absolutePath)) continue;
+    findings.push({
+      file: path.relative(ROOT, absolutePath),
+      line: lineNumberForIndex(source, match.index ?? 0),
+      specifier,
+    });
+  }
+}
+
+function walk(absolutePath) {
+  const stat = fs.statSync(absolutePath);
+  if (stat.isDirectory()) {
+    for (const entry of fs.readdirSync(absolutePath, { withFileTypes: true })) {
+      walk(path.join(absolutePath, entry.name));
+    }
+    return;
+  }
+  if (isSourceFile(absolutePath)) scanFile(absolutePath);
+}
+
+for (const relativePath of SCAN_PATHS) {
+  const absolutePath = path.join(ROOT, relativePath);
+  if (fs.existsSync(absolutePath)) {
+    walk(absolutePath);
+  }
+}
+
+if (findings.length > 0) {
+  console.error("Found legacy root-module imports that should use src/** paths:");
+  for (const finding of findings) {
+    console.error(`- ${toPosix(finding.file)}:${finding.line} -> ${finding.specifier}`);
+  }
+  process.exit(1);
+}
+
+console.log("No legacy root-module imports found.");

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/helpers.js

@@ -1 +0,0 @@
-export * from "./src/core/helpers.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";