@hanna84/mcp-writing 3.5.1 → 3.5.3

package/CHANGELOG.md

@@ -4,9 +4,23 @@ 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.5.3](https://github.com/hannasdev/mcp-writing/compare/v3.5.2...v3.5.3)
+
+- fix(review-bundles): keep epigraph prose without epigraph titles in beta exports [`#185`](https://github.com/hannasdev/mcp-writing/pull/185)
+
+#### [v3.5.2](https://github.com/hannasdev/mcp-writing/compare/v3.5.1...v3.5.2)
+
+> 8 May 2026
+
+- Improve beta prose flow and centered scene dateline [`#184`](https://github.com/hannasdev/mcp-writing/pull/184)
+- Release 3.5.2 [`5fb224a`](https://github.com/hannasdev/mcp-writing/commit/5fb224af5a66111c9b8d3d32e17acbb908fa5f97)
+
 #### [v3.5.1](https://github.com/hannasdev/mcp-writing/compare/v3.5.0...v3.5.1)
 
+> 8 May 2026
+
 - Fix beta accountability follow-ups and test coverage [`#183`](https://github.com/hannasdev/mcp-writing/pull/183)
+- Release 3.5.1 [`28d77f7`](https://github.com/hannasdev/mcp-writing/commit/28d77f755708dc8e987caa69dfcb6f3c5408b916)
 
 #### [v3.5.0](https://github.com/hannasdev/mcp-writing/compare/v3.4.4...v3.5.0)
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hanna84/mcp-writing",
-  "version": "3.5.1",
+  "version": "3.5.3",
   "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/review-bundles/review-bundles-renderer.js

@@ -57,7 +57,7 @@ function renderBetaFeedbackFormMarkdown({ projectId, recipientName, generatedAt
   ].join("\n") + "\n";
 }
 
-function loadBundleSceneRows(dbHandle, projectId, sceneIds) {
+function loadBundleSceneRowsWithTags(dbHandle, projectId, sceneIds) {
   if (!Array.isArray(sceneIds) || sceneIds.length === 0) return [];
   const rows = [];
   // 900 is safely below SQLite's per-query bound of 999 host parameters
@@ -74,6 +74,7 @@ function loadBundleSceneRows(dbHandle, projectId, sceneIds) {
         title,
         part,
         chapter,
+        chapter_title,
         timeline_position,
         logline,
         pov,
@@ -85,12 +86,36 @@ function loadBundleSceneRows(dbHandle, projectId, sceneIds) {
     rows.push(...chunkRows);
   }
 
-  const rowMap = new Map(rows.map(row => [row.scene_id, row]));
   const orderedRows = [];
   const missingSceneIds = [];
 
+  // Load tags in batches to avoid N+1 queries for large bundles.
+  const tagsBySceneId = new Map(rows.map(row => [row.scene_id, []]));
+  const tagChunkSize = 900;
+  for (let offset = 0; offset < rows.length; offset += tagChunkSize) {
+    const chunk = rows.slice(offset, offset + tagChunkSize);
+    const chunkSceneIds = chunk.map(row => row.scene_id);
+    if (chunkSceneIds.length === 0) continue;
+    const placeholders = chunkSceneIds.map(() => "?").join(",");
+    const tagRows = dbHandle.prepare(`
+      SELECT scene_id, tag
+      FROM scene_tags
+      WHERE project_id = ? AND scene_id IN (${placeholders})
+    `).all(projectId, ...chunkSceneIds);
+    for (const tagRow of tagRows) {
+      const tags = tagsBySceneId.get(tagRow.scene_id);
+      if (tags) tags.push(tagRow.tag);
+    }
+  }
+
+  const rowsWithTags = rows.map(row => ({
+    ...row,
+    tags: tagsBySceneId.get(row.scene_id) ?? [],
+  }));
+  const rowMapWithTags = new Map(rowsWithTags.map(row => [row.scene_id, row]));
+
   for (const sceneId of sceneIds) {
-    const row = rowMap.get(sceneId);
+    const row = rowMapWithTags.get(sceneId);
     if (row) {
       orderedRows.push(row);
     } else {
@@ -233,20 +258,149 @@ function readProse(filePath, { syncDir } = {}) {
   }
 }
 
+function normalizeHardWrappedProse(rawProse) {
+  const prose = String(rawProse ?? "").replace(/\r\n?/g, "\n").trim();
+  if (!prose) return "";
+  const paragraphs = prose
+    .split(/\n\s*\n/g)
+    .map(paragraph => paragraph.replace(/\s*\n\s*/g, " ").trim())
+    .filter(Boolean);
+  return paragraphs.join("\n\n");
+}
+
+function extractSceneDateline(prose) {
+  const normalized = String(prose ?? "").replace(/\r\n?/g, "\n").trim();
+  if (!normalized) {
+    return { dateline: null, body: "" };
+  }
+
+  const lines = normalized
+    .split("\n")
+    .map(line => line.trim())
+    .filter(Boolean);
+  if (lines.length === 0) {
+    return { dateline: null, body: "" };
+  }
+
+  const firstParagraph = lines[0];
+  const dashMatch = firstParagraph.match(/^(.+?)\s*[–-]\s*(.+)$/);
+  const left = dashMatch?.[1]?.trim() ?? "";
+  const right = dashMatch?.[2]?.trim() ?? "";
+  const totalWords = firstParagraph.split(/\s+/).filter(Boolean).length;
+  const looksLikeDateline = (
+    firstParagraph.length >= 6
+    && firstParagraph.length <= 90
+    && Boolean(dashMatch)
+    && left.length >= 2
+    && right.length >= 2
+    && totalWords <= 14
+    && !/[!?]/.test(firstParagraph)
+    && !/[“”"']/.test(firstParagraph)
+  );
+
+  if (!looksLikeDateline) {
+    return { dateline: null, body: normalized };
+  }
+
+  return {
+    dateline: firstParagraph,
+    body: lines.slice(1).join("\n"),
+  };
+}
+
+function normalizeBetaProseFlow(prose) {
+  const normalized = String(prose ?? "").replace(/\r\n?/g, "\n").trim();
+  if (!normalized) return "";
+  const paragraphs = normalized
+    .split(/\n\s*\n/g)
+    .map(paragraph => paragraph
+      .split("\n")
+      .map(line => line.trim())
+      .filter(Boolean)
+      .join("\n"))
+    .filter(Boolean);
+  // For beta exports, convert paragraph blocks into regular line breaks so the
+  // reading flow stays continuous without large section gaps.
+  return paragraphs.join("\n");
+}
+
+function normalizeBetaTypography(prose) {
+  return String(prose ?? "")
+    .replace(/(^|\s)--(\s|$)/g, "$1—$2");
+}
+
+function renderProseWithInlineEmphasis(doc, prose, {
+  bodyFont,
+  italicFont,
+  fontSize,
+  width,
+  align = "left",
+  lineGap = 0,
+  paragraphGap = 0,
+  blankLineMoveDown = 0.15,
+}) {
+  const lines = String(prose ?? "").split("\n");
+  for (const line of lines) {
+    if (line.length === 0) {
+      doc.moveDown(blankLineMoveDown);
+      continue;
+    }
+
+    if (line.trim() === "***") {
+      doc.moveDown(0.5);
+      doc.fontSize(fontSize).font(bodyFont);
+      doc.text("***", { align: "center", width, lineGap, paragraphGap: 0 });
+      doc.moveDown(0.5);
+      continue;
+    }
+
+    const segments = line.split(/(\*[^*\n]+\*)/g).filter(Boolean);
+
+    for (let index = 0; index < segments.length; index += 1) {
+      const segment = segments[index];
+      const isItalic = /^\*[^*\n]+\*$/.test(segment);
+      const text = isItalic ? segment.slice(1, -1) : segment;
+      if (!text) continue;
+      doc.fontSize(fontSize).font(isItalic ? italicFont : bodyFont);
+      doc.text(text, {
+        align,
+        width,
+        lineGap,
+        paragraphGap,
+        continued: index < segments.length - 1,
+      });
+    }
+  }
+  doc.font(bodyFont).fontSize(fontSize);
+}
+
 function renderSceneBlock(scene, options) {
   const {
     profile,
     includeSceneIds,
     includeMetadataSidebar,
     includeParagraphAnchors,
+    showChapterHeading,
   } = options;
 
-  const title = scene.title || scene.scene_id;
-  const sceneHeading = includeSceneIds
-    ? `## ${escapeMarkdown(title)} (${escapeMarkdown(scene.scene_id)})`
-    : `## ${escapeMarkdown(title)}`;
+  const isBetaProfile = profile === "beta_reader_personalized";
+  const isEpigraph = isBetaProfile && scene.tags?.includes("epigraph");
+
+  const parts = [];
 
-  const parts = [sceneHeading];
+  // Render chapter heading only when the caller detects a chapter transition.
+  if (isBetaProfile && scene.chapter_title && showChapterHeading) {
+    parts.push(`## ${escapeMarkdown(scene.chapter_title)}`);
+  }
+
+  // Only render heading if not an epigraph
+  if (!isEpigraph) {
+    const title = scene.title || scene.scene_id;
+    const sceneHeading = includeSceneIds
+      ? `## ${escapeMarkdown(title)} (${escapeMarkdown(scene.scene_id)})`
+      : `## ${escapeMarkdown(title)}`;
+    parts.push(sceneHeading);
+  }
 
   if (profile === "outline_discussion") {
     const summaryParts = [];
@@ -278,11 +432,11 @@ function renderSceneBlock(scene, options) {
 
   const prose = scene.prose ?? "";
   if (!includeParagraphAnchors || prose.length === 0) {
-    parts.push(prose);
+    parts.push(normalizeHardWrappedProse(prose));
     return parts.join("\n\n");
   }
 
-  const paragraphs = prose
+  const paragraphs = normalizeHardWrappedProse(prose)
     .split(/\n\s*\n/g)
     .map(p => p.trim())
     .filter(Boolean);
@@ -362,7 +516,7 @@ export function renderReviewBundleMarkdown(dbHandle, plan, { generatedAt, syncDi
   const syncDir = syncDirOpt ?? process.env.WRITING_SYNC_DIR ?? null;
 
   const sceneIds = plan.ordering.map(row => row.scene_id);
-  const rows = loadBundleSceneRows(dbHandle, plan.resolved_scope.project_id, sceneIds);
+  const rows = loadBundleSceneRowsWithTags(dbHandle, plan.resolved_scope.project_id, sceneIds);
   const sections = [];
   const recipientName = plan.resolved_scope?.options?.recipient_name;
   const recipientDisplayName = normalizeRecipientDisplayName(recipientName);
@@ -391,7 +545,8 @@ export function renderReviewBundleMarkdown(dbHandle, plan, { generatedAt, syncDi
     );
   }
 
-  for (const scene of rows) {
+  for (let sceneIndex = 0; sceneIndex < rows.length; sceneIndex += 1) {
+    const scene = rows[sceneIndex];
     let prose = "";
     if (profile === "editor_detailed" || profile === "beta_reader_personalized") {
       const resolved = readProse(scene.file_path, { syncDir });
@@ -409,11 +564,16 @@ export function renderReviewBundleMarkdown(dbHandle, plan, { generatedAt, syncDi
       prose = resolved;
     }
     const withProse = { ...scene, prose };
+    const prevScene = sceneIndex > 0 ? rows[sceneIndex - 1] : null;
+    const showChapterHeading = isBetaProfile
+      && Boolean(scene.chapter_title)
+      && (!prevScene || prevScene.chapter !== scene.chapter);
     sections.push(renderSceneBlock(withProse, {
       profile,
       includeSceneIds,
       includeMetadataSidebar,
       includeParagraphAnchors,
+      showChapterHeading,
     }));
   }
 
@@ -433,15 +593,16 @@ export function renderReviewBundlePdfWithMetadata(dbHandle, plan, { generatedAt,
   const syncDir = syncDirOpt ?? process.env.WRITING_SYNC_DIR ?? null;
   const isBetaProfile = profile === "beta_reader_personalized";
   const proseFontSize = isBetaProfile ? 8 : 10;
-  const proseLineGap = isBetaProfile ? 3.2 : 3;
+  const proseLineGap = isBetaProfile ? 1.6 : 3;
   const bodyFont = profile === "beta_reader_personalized" ? "Times-Roman" : "Helvetica";
   const coverHeadingFont = profile === "beta_reader_personalized" ? "Times-Bold" : "Helvetica-Bold";
   // Beta scene headings intentionally use body font (non-bold) per product direction.
   const sceneHeadingFont = isBetaProfile ? bodyFont : coverHeadingFont;
-  const metaFont = profile === "beta_reader_personalized" ? "Times-Italic" : "Helvetica-Oblique";
+  const italicFont = profile === "beta_reader_personalized" ? "Times-Italic" : "Helvetica-Oblique";
+  const metaFont = italicFont;
 
   const sceneIds = plan.ordering.map(row => row.scene_id);
-  const rows = loadBundleSceneRows(dbHandle, plan.resolved_scope.project_id, sceneIds);
+  const rows = loadBundleSceneRowsWithTags(dbHandle, plan.resolved_scope.project_id, sceneIds);
   const recipientName = plan.resolved_scope?.options?.recipient_name;
   const recipientDisplayName = normalizeRecipientDisplayName(recipientName);
   const footerRecipientDisplayName = sanitizeFooterRecipientDisplayName(recipientDisplayName);
@@ -494,7 +655,7 @@ export function renderReviewBundlePdfWithMetadata(dbHandle, plan, { generatedAt,
     doc.restore();
     // Restore prose style so auto-flowed text keeps consistent typography
     // on pages added during long text rendering.
-    doc.font(bodyFont).fontSize(proseFontSize);
+    doc.font(bodyFont).fontSize(proseFontSize).fillColor("#000000");
     doc.x = previousX;
     doc.y = previousY;
   };
@@ -530,8 +691,9 @@ export function renderReviewBundlePdfWithMetadata(dbHandle, plan, { generatedAt,
 
     try {
       doc.addPage();
-      doc.fontSize(24).font(coverHeadingFont).text(`Review Bundle: ${plan.resolved_scope.project_id}`, { align: "left" });
-      doc.moveDown(0.5);
+      const coverLabel = `Review Bundle: ${plan.resolved_scope.project_id}`;
+      doc.fontSize(isBetaProfile ? 11 : 24).font(coverHeadingFont).text(coverLabel, { align: "left" });
+      doc.moveDown(isBetaProfile ? 0.2 : 0.5);
       doc.fontSize(11).font(bodyFont);
       if (profile !== "beta_reader_personalized") {
         doc.text(`Profile: ${profile}`, { align: "left" });
@@ -556,19 +718,31 @@ export function renderReviewBundlePdfWithMetadata(dbHandle, plan, { generatedAt,
         doc.moveDown();
       }
 
-      for (const scene of rows) {
+      for (let sceneIndex = 0; sceneIndex < rows.length; sceneIndex += 1) {
+        const scene = rows[sceneIndex];
+        const prevScene = sceneIndex > 0 ? rows[sceneIndex - 1] : null;
         if (isBetaProfile) {
           // Give chapter titles generous vertical breathing room for a
           // print-like opening feel before prose begins.
           doc.moveDown(2.0);
         }
-        doc.fontSize(isBetaProfile ? 13 : 14).font(sceneHeadingFont);
-        let heading = scene.title || scene.scene_id;
-        if (includeSceneIds) {
-          heading += ` [${scene.scene_id}]`;
+        if (isBetaProfile && scene.chapter_title && (!prevScene || prevScene.chapter !== scene.chapter)) {
+          doc.fontSize(16).font(coverHeadingFont);
+          doc.text(scene.chapter_title, { align: "center" });
+          doc.moveDown(1.0);
+        }
+
+        // Skip title rendering for epigraphs in beta profile
+        const isEpigraph = isBetaProfile && scene.tags?.includes("epigraph");
+        if (!isEpigraph) {
+          doc.fontSize(isBetaProfile ? 13 : 14).font(sceneHeadingFont);
+          let heading = scene.title || scene.scene_id;
+          if (includeSceneIds) {
+            heading += ` [${scene.scene_id}]`;
+          }
+          doc.text(heading, { align: isBetaProfile ? "center" : "left" });
+          doc.moveDown(isBetaProfile ? 1.6 : 0.2);
         }
-        doc.text(heading, { align: isBetaProfile ? "center" : "left" });
-        doc.moveDown(isBetaProfile ? 1.6 : 0.2);
 
         const metaParts = [];
         if (profile !== "beta_reader_personalized") {
@@ -604,14 +778,35 @@ export function renderReviewBundlePdfWithMetadata(dbHandle, plan, { generatedAt,
               }
             );
           }
-          prose = resolved;
+          let sceneDateline = null;
+          if (isBetaProfile) {
+            prose = normalizeBetaProseFlow(resolved);
+            const extracted = extractSceneDateline(prose);
+            sceneDateline = extracted.dateline ? normalizeBetaTypography(extracted.dateline) : null;
+            prose = normalizeBetaTypography(extracted.body);
+          } else {
+            prose = normalizeHardWrappedProse(resolved);
+          }
+
+          if (sceneDateline) {
+            doc.fontSize(10).font(metaFont);
+            doc.text(sceneDateline, {
+              align: "center",
+              width: doc.page.width - doc.page.margins.left - doc.page.margins.right,
+            });
+            doc.moveDown(1.0);
+          }
 
           const textWidth = doc.page.width - doc.page.margins.left - doc.page.margins.right;
-          doc.fontSize(proseFontSize).font(bodyFont);
-          doc.text(prose, {
+          renderProseWithInlineEmphasis(doc, prose, {
+            bodyFont,
+            italicFont,
+            fontSize: proseFontSize,
             align: "left",
             width: textWidth,
             lineGap: proseLineGap,
+            paragraphGap: 0,
+            blankLineMoveDown: isBetaProfile ? 0.15 : 0.65,
           });
         }
 
@@ -619,7 +814,7 @@ export function renderReviewBundlePdfWithMetadata(dbHandle, plan, { generatedAt,
         // Add page break between scenes only for prose-including profiles where
         // clear scene separation matters. For outline_discussion, let content flow.
         const includesProse = profile === "editor_detailed" || profile === "beta_reader_personalized";
-        if (includesProse && scene !== rows[rows.length - 1]) {
+        if (includesProse && sceneIndex < rows.length - 1) {
           doc.addPage();
         }
       }
@@ -642,4 +837,5 @@ export {
   buildPageFingerprintToken,
   buildFingerprintSeed,
   buildFingerprintSeedHash,
+  extractSceneDateline,
 };

package/src/review-bundles/review-bundles.js

@@ -13,6 +13,7 @@ export {
   buildPageFingerprintToken,
   buildFingerprintSeed,
   buildFingerprintSeedHash,
+  extractSceneDateline,
 } from "./review-bundles-renderer.js";
 
 export { createReviewBundleArtifacts } from "./review-bundles-writer.js";

package/src/sync/importer.js

@@ -358,12 +358,7 @@ export function importScrivenerSync({
       continue;
     }
 
-    if (isEpigraph(rawTitle)) {
-      logger(`  SKIP  (epigraph) ${filename}`);
-      skipped++;
-      continue;
-    }
-
+    const isEpigraphScene = isEpigraph(rawTitle);
     const title = cleanTitle(rawTitle);
     const existingScene = existingScenes.get(String(binderId)) ?? null;
     const sceneId = existingScene?.meta?.scene_id ?? makeSceneId(binderId, title);
@@ -375,6 +370,10 @@ export function importScrivenerSync({
     const destFile = path.join(targetDir, `${seq.toString().padStart(3, "0")} ${rawTitle} [${binderId}].${ext}`);
     const sidecar = destFile.replace(/\.(txt|md)$/, ".meta.yaml");
 
+    const existingTags = Array.isArray(existingScene?.meta?.tags)
+      ? existingScene.meta.tags
+      : [];
+
     const meta = {
       ...(existingScene?.meta ?? {}),
       scene_id: sceneId,
@@ -383,6 +382,7 @@ export function importScrivenerSync({
       title,
       timeline_position: seq,
       ...(beatCarry ? { save_the_cat_beat: beatCarry } : {}),
+      ...(isEpigraphScene ? { tags: [...existingTags, "epigraph"].filter((v, i, a) => a.indexOf(v) === i) } : {}),
     };
 
     if (!beatCarry && existingScene?.meta && Object.hasOwn(existingScene.meta, "save_the_cat_beat")) {
@@ -424,7 +424,7 @@ export function importScrivenerSync({
   logger("");
   logger(`${"-".repeat(50)}`);
   logger(`Created:  ${created} sidecars${dryRun ? " (dry run)" : ""}`);
-  logger(`Skipped:  ${skipped} (empty / epigraph / pattern)`);
+  logger(`Skipped:  ${skipped} (empty / pattern)`);
   if (existing) logger(`Existing: ${existing} already had sidecars`);
   logger(`Beat markers seen: ${beatMarkersSeen}`);