@hanna84/mcp-writing 3.1.4 → 3.2.1

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).
 
+#### [v3.2.1](https://github.com/hannasdev/mcp-writing.git
+/compare/v3.2.0...v3.2.1)
+
+- docs(skills): add review comment resolution workflow [`#172`](https://github.com/hannasdev/mcp-writing.git
+/pull/172)
+
+#### [v3.2.0](https://github.com/hannasdev/mcp-writing.git
+/compare/v3.1.4...v3.2.0)
+
+> 3 May 2026
+
+- feat(styleguide): publish AI boot files during skill setup [`#171`](https://github.com/hannasdev/mcp-writing.git
+/pull/171)
+- Release 3.2.0 [`ffd8715`](https://github.com/hannasdev/mcp-writing.git
+/commit/ffd87159a538ed93ee195becac04ab414e1663ca)
+
 #### [v3.1.4](https://github.com/hannasdev/mcp-writing.git
 /compare/v3.1.3...v3.1.4)
 
+> 3 May 2026
+
 - docs(prd): extract onboarding into shared framework PRD [`#170`](https://github.com/hannasdev/mcp-writing.git
 /pull/170)
+- Release 3.1.4 [`5ed3458`](https://github.com/hannasdev/mcp-writing.git
+/commit/5ed3458f354f89b1e6555924dd2e1c907270cfdc)
 
 #### [v3.1.3](https://github.com/hannasdev/mcp-writing.git
 /compare/v3.1.2...v3.1.3)

package/package.json

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

package/src/tools/styleguide.js

@@ -24,6 +24,155 @@ import {
 } from "../styleguide/prose-styleguide-skill.js";
 import { validateProjectId } from "../sync/importer.js";
 
+const CLAUDE_BOOT_BASENAME = "CLAUDE.md";
+const COPILOT_BOOT_RELATIVE_PATH = ".github/copilot-instructions.md";
+const PROSE_STYLEGUIDE_IMPORT_LINE = "@skills/prose-styleguide/SKILL.md";
+const COPILOT_STYLEGUIDE_MARKER_START = "<!-- MCP-WRITING:PROSE-STYLEGUIDE START -->";
+const COPILOT_STYLEGUIDE_MARKER_END = "<!-- MCP-WRITING:PROSE-STYLEGUIDE END -->";
+
+function buildSafeMarkdownFence(text) {
+  const runs = text.match(/`+/g) ?? [];
+  const longest = runs.reduce((max, run) => Math.max(max, run.length), 0);
+  return "`".repeat(Math.max(3, longest + 1));
+}
+
+function hasStandaloneClaudeImportLine(content) {
+  const lines = content.split(/\r?\n/);
+  let inFence = false;
+  let fenceChar = "";
+  let fenceLength = 0;
+
+  for (const line of lines) {
+    const trimmed = line.trim();
+    const fenceMatch = trimmed.match(/^(`{3,}|~{3,})/);
+
+    if (fenceMatch) {
+      const nextFenceChar = fenceMatch[1][0];
+      const nextFenceLength = fenceMatch[1].length;
+
+      if (!inFence) {
+        inFence = true;
+        fenceChar = nextFenceChar;
+        fenceLength = nextFenceLength;
+        continue;
+      }
+
+      if (nextFenceChar === fenceChar && nextFenceLength >= fenceLength) {
+        inFence = false;
+        fenceChar = "";
+        fenceLength = 0;
+      }
+      continue;
+    }
+
+    if (!inFence && trimmed === PROSE_STYLEGUIDE_IMPORT_LINE) {
+      return true;
+    }
+  }
+
+  return false;
+}
+
+function upsertClaudeBootFile({ syncDir, overwrite = false }) {
+  const targetPath = path.join(syncDir, CLAUDE_BOOT_BASENAME);
+  const exists = fs.existsSync(targetPath);
+
+  if (exists && !overwrite) {
+    const existing = fs.readFileSync(targetPath, "utf8");
+    if (hasStandaloneClaudeImportLine(existing)) {
+      return { path: path.resolve(targetPath), status: "unchanged" };
+    }
+
+    const suffix = existing.endsWith("\n") ? "\n" : "\n\n";
+    fs.writeFileSync(targetPath, `${existing}${suffix}${PROSE_STYLEGUIDE_IMPORT_LINE}\n`, "utf8");
+    return { path: path.resolve(targetPath), status: "updated" };
+  }
+
+  const content = [
+    "# Writing Assistant Boot File",
+    "",
+    "Load the prose styleguide by default:",
+    "",
+    PROSE_STYLEGUIDE_IMPORT_LINE,
+    "",
+  ].join("\n");
+
+  fs.mkdirSync(path.dirname(targetPath), { recursive: true });
+  fs.writeFileSync(targetPath, content, "utf8");
+  return { path: path.resolve(targetPath), status: exists ? "overwritten" : "created" };
+}
+
+function renderCopilotStyleguideBlock({ skillMarkdown }) {
+  const trimmedSkillMarkdown = skillMarkdown.trim();
+  const fence = buildSafeMarkdownFence(trimmedSkillMarkdown);
+
+  return [
+    COPILOT_STYLEGUIDE_MARKER_START,
+    "## Prose Styleguide",
+    "",
+    "This section is generated by mcp-writing setup. `skills/prose-styleguide/SKILL.md` is the canonical file.",
+    "",
+    "Since Copilot does not support imports, this is an inline snapshot:",
+    "",
+    `${fence}markdown`,
+    trimmedSkillMarkdown,
+    fence,
+    COPILOT_STYLEGUIDE_MARKER_END,
+    "",
+  ].join("\n");
+}
+
+function upsertCopilotBootFile({ syncDir, skillMarkdown, overwrite = false }) {
+  const targetPath = path.join(syncDir, COPILOT_BOOT_RELATIVE_PATH);
+  const exists = fs.existsSync(targetPath);
+  const styleguideBlock = renderCopilotStyleguideBlock({ skillMarkdown });
+
+  if (!exists || overwrite) {
+    const content = [
+      "# Copilot Instructions",
+      "",
+      styleguideBlock,
+    ].join("\n");
+    fs.mkdirSync(path.dirname(targetPath), { recursive: true });
+    fs.writeFileSync(targetPath, content, "utf8");
+    return { path: path.resolve(targetPath), status: exists ? "overwritten" : "created" };
+  }
+
+  const existing = fs.readFileSync(targetPath, "utf8");
+  const markerRegex = new RegExp(
+    `${COPILOT_STYLEGUIDE_MARKER_START}[\\s\\S]*?${COPILOT_STYLEGUIDE_MARKER_END}`,
+    "m"
+  );
+
+  const next = markerRegex.test(existing)
+    ? existing.replace(markerRegex, styleguideBlock.trimEnd())
+    : `${existing}${existing.endsWith("\n") ? "\n" : "\n\n"}${styleguideBlock}`;
+
+  fs.writeFileSync(targetPath, next, "utf8");
+  return { path: path.resolve(targetPath), status: markerRegex.test(existing) ? "updated" : "appended" };
+}
+
+function validateParentDirectory(targetPath, label) {
+  const parent = path.dirname(targetPath);
+  if (fs.existsSync(parent)) {
+    const parentStat = fs.statSync(parent);
+    if (!parentStat.isDirectory()) {
+      return {
+        ok: false,
+        error: {
+          code: "INVALID_TARGET_PARENT",
+          message: `${label} parent path must be a directory: ${path.resolve(parent)}`,
+          details: {
+            target_path: path.resolve(targetPath),
+            parent_path: path.resolve(parent),
+          },
+        },
+      };
+    }
+  }
+  return { ok: true };
+}
+
 export function registerStyleguideTools(s, {
   db,
   SYNC_DIR,
@@ -603,19 +752,32 @@ export function registerStyleguideTools(s, {
 
   s.tool(
     "setup_prose_styleguide_skill",
-    "Generate skills/prose-styleguide/SKILL.md from the resolved prose styleguide config and universal craft rules.",
+    "Generate skills/prose-styleguide/SKILL.md from the resolved prose styleguide config and universal craft rules. Optionally publish AI boot files (CLAUDE.md and .github/copilot-instructions.md) when using sync-root config scope.",
     {
-      project_id: z.string().optional().describe("Optional project ID for scoped config resolution (e.g. 'the-lamb' or 'universe-1/book-1')."),
+      project_id: z.string().optional().describe("Project-scoped skill generation is unsupported because this tool writes a shared sync-root skills/prose-styleguide/SKILL.md file."),
       overwrite: z.boolean().optional().describe("If true, replaces an existing skills/prose-styleguide/SKILL.md file."),
+      publish_boot_files: z.boolean().optional().describe("If true, also upserts CLAUDE.md and .github/copilot-instructions.md at sync root. Defaults to true."),
+      boot_files_overwrite: z.boolean().optional().describe("If true, rewrites existing boot files instead of in-place updates."),
     },
-    async ({ project_id, overwrite = false }) => {
+    async ({ project_id, overwrite = false, publish_boot_files, boot_files_overwrite = false }) => {
       if (project_id !== undefined) {
         const projectIdCheck = validateProjectId(project_id);
         if (!projectIdCheck.ok) {
           return errorResponse("INVALID_PROJECT_ID", projectIdCheck.reason, { project_id });
         }
+
+        return errorResponse(
+          "PROJECT_SCOPED_SKILL_UNSUPPORTED",
+          "setup_prose_styleguide_skill writes a shared sync-root skills/prose-styleguide/SKILL.md file and cannot run with project_id.",
+          {
+            project_id,
+            next_step: "Run setup_prose_styleguide_skill without project_id for workspace-wide skill setup.",
+          }
+        );
       }
 
+      const shouldPublishBootFiles = publish_boot_files ?? true;
+
       if (!SYNC_DIR_WRITABLE) {
         return errorResponse(
           "SYNC_DIR_NOT_WRITABLE",
@@ -672,8 +834,139 @@ export function registerStyleguideTools(s, {
         return errorResponse(generated.error.code, generated.error.message);
       }
 
-      fs.mkdirSync(path.dirname(skillPath), { recursive: true });
-      fs.writeFileSync(skillPath, generated.markdown, "utf8");
+      const skillPathCheck = validateParentDirectory(skillPath, "Skill file");
+      if (!skillPathCheck.ok) {
+        return errorResponse(
+          skillPathCheck.error.code,
+          skillPathCheck.error.message,
+          skillPathCheck.error.details
+        );
+      }
+
+      let bootFiles = [];
+      const claudePath = path.join(SYNC_DIR, CLAUDE_BOOT_BASENAME);
+      const copilotPath = path.join(SYNC_DIR, COPILOT_BOOT_RELATIVE_PATH);
+
+      if (shouldPublishBootFiles) {
+        if (!isPathCandidateInsideSyncDir(claudePath) || !isPathCandidateInsideSyncDir(copilotPath)) {
+          return errorResponse(
+            "INVALID_BOOT_FILE_PATH",
+            "Resolved AI boot file paths must be inside WRITING_SYNC_DIR.",
+            {
+              claude_path: path.resolve(claudePath),
+              copilot_path: path.resolve(copilotPath),
+              sync_dir: SYNC_DIR_ABS,
+            }
+          );
+        }
+
+        const claudePathCheck = validateParentDirectory(claudePath, "CLAUDE boot file");
+        if (!claudePathCheck.ok) {
+          return errorResponse(
+            claudePathCheck.error.code,
+            claudePathCheck.error.message,
+            claudePathCheck.error.details
+          );
+        }
+        const copilotPathCheck = validateParentDirectory(copilotPath, "Copilot boot file");
+        if (!copilotPathCheck.ok) {
+          return errorResponse(
+            copilotPathCheck.error.code,
+            copilotPathCheck.error.message,
+            copilotPathCheck.error.details
+          );
+        }
+
+      }
+
+      const mutationTargets = shouldPublishBootFiles
+        ? [skillPath, claudePath, copilotPath]
+        : [skillPath];
+      const backups = mutationTargets.map((targetPath) => {
+        if (!fs.existsSync(targetPath)) {
+          return { targetPath, existed: false, content: null };
+        }
+        return {
+          targetPath,
+          existed: true,
+          content: fs.readFileSync(targetPath, "utf8"),
+        };
+      });
+
+      function rollbackMutations() {
+        for (let i = backups.length - 1; i >= 0; i -= 1) {
+          const backup = backups[i];
+          if (backup.existed) {
+            fs.mkdirSync(path.dirname(backup.targetPath), { recursive: true });
+            fs.writeFileSync(backup.targetPath, backup.content, "utf8");
+            continue;
+          }
+          fs.rmSync(backup.targetPath, { force: true });
+        }
+      }
+
+      let failedStep = "";
+
+      try {
+        if (shouldPublishBootFiles) {
+          failedStep = "claude_boot_file";
+          const claudeResult = upsertClaudeBootFile({
+            syncDir: SYNC_DIR,
+            overwrite: boot_files_overwrite,
+          });
+
+          failedStep = "copilot_boot_file";
+          const copilotResult = upsertCopilotBootFile({
+            syncDir: SYNC_DIR,
+            skillMarkdown: generated.markdown,
+            overwrite: boot_files_overwrite,
+          });
+          bootFiles = [
+            { type: "claude", ...claudeResult },
+            { type: "copilot", ...copilotResult },
+          ];
+        }
+
+        failedStep = "skill_file";
+        fs.mkdirSync(path.dirname(skillPath), { recursive: true });
+        fs.writeFileSync(skillPath, generated.markdown, "utf8");
+      } catch (error) {
+        try {
+          rollbackMutations();
+        } catch (rollbackError) {
+          return errorResponse(
+            "SETUP_WRITE_ROLLBACK_FAILED",
+            "Failed to publish prose styleguide setup files and rollback was not clean.",
+            {
+              failed_step: failedStep,
+              reason: String(error?.message ?? error),
+              rollback_reason: String(rollbackError?.message ?? rollbackError),
+              target_path: path.resolve(skillPath),
+            }
+          );
+        }
+
+        if (failedStep === "skill_file") {
+          return errorResponse(
+            "STYLEGUIDE_SKILL_WRITE_FAILED",
+            "Failed to write skills/prose-styleguide/SKILL.md.",
+            {
+              failed_step: failedStep,
+              reason: String(error?.message ?? error),
+              target_path: path.resolve(skillPath),
+            }
+          );
+        }
+
+        return errorResponse(
+          "BOOT_FILE_WRITE_FAILED",
+          "Failed to publish AI boot files while setting up prose styleguide skill.",
+          {
+            failed_step: failedStep,
+            reason: String(error?.message ?? error),
+          }
+        );
+      }
 
       return jsonResponse({
         ok: true,
@@ -681,6 +974,8 @@ export function registerStyleguideTools(s, {
         project_id: project_id ?? null,
         injected_rules: generated.injected_rules,
         source_count: resolved.sources.length,
+        boot_files: bootFiles,
+        warnings: [],
       });
     }
   );

package/src/workflows/workflow-catalogue.js

@@ -101,6 +101,7 @@ export const WORKFLOW_CATALOGUE = [
       { tool: "bootstrap_prose_styleguide_config", note: "Detect dominant conventions. Confirm suggestions with the user before applying." },
       { tool: "setup_prose_styleguide_config", note: "Only if ALL context.styleguide_exists fields are false — a config at any scope is sufficient. Create at project_root scope (requires project_id and language e.g. 'english_us'), or sync_root if no project_id is known." },
       { tool: "update_prose_styleguide_config", note: "Apply the fields accepted from bootstrap suggestions." },
+      { tool: "setup_prose_styleguide_skill", note: "Run only for sync-root setup. This tool writes shared skills/prose-styleguide/SKILL.md (and optional boot files), so do not run it after project_root setup." },
     ],
   },
   {