@hanna84/mcp-writing 3.5.0 → 3.5.2

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.2](https://github.com/hannasdev/mcp-writing/compare/v3.5.1...v3.5.2)
+
+- Improve beta prose flow and centered scene dateline [`#184`](https://github.com/hannasdev/mcp-writing/pull/184)
+
+#### [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)
 
+> 8 May 2026
+
 - feat: add accountable beta-reader bundle fingerprinting [`#182`](https://github.com/hannasdev/mcp-writing/pull/182)
+- Release 3.5.0 [`6134dfc`](https://github.com/hannasdev/mcp-writing/commit/6134dfc17e998c118f5fdbb68f1a26e3ba673f05)
 
 #### [v3.4.4](https://github.com/hannasdev/mcp-writing/compare/v3.4.3...v3.4.4)
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hanna84/mcp-writing",
-  "version": "3.5.0",
+  "version": "3.5.2",
   "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-planner.js

@@ -335,16 +335,23 @@ export function buildReviewBundlePlan(dbHandle, {
     : undefined;
 
   const safeBundleName = slugifyBundleName(bundle_name || `${project_id}-${profile}`);
+  const normalizedSceneIds = Array.isArray(scene_ids)
+    ? Array.from(new Set(scene_ids.map(sceneId => String(sceneId)))).sort()
+    : undefined;
   const appliedFilters = {
     ...(part !== undefined ? { part } : {}),
     ...(chapter !== undefined ? { chapter } : {}),
     ...(Array.isArray(normalizedChapters) ? { chapters: normalizedChapters } : {}),
     ...(tag ? { tag } : {}),
-    ...(Array.isArray(scene_ids) ? { scene_ids } : {}),
+    ...(Array.isArray(normalizedSceneIds) ? { scene_ids: normalizedSceneIds } : {}),
   };
   const resolvedBetaAccountability = profile === "beta_reader_personalized"
     ? Boolean(beta_accountability ?? true)
     : false;
+  const isBetaProfile = profile === "beta_reader_personalized";
+  const resolvedIncludeSceneIds = isBetaProfile ? false : Boolean(include_scene_ids);
+  const resolvedIncludeMetadataSidebar = isBetaProfile ? false : Boolean(include_metadata_sidebar);
+  const resolvedIncludeParagraphAnchors = isBetaProfile ? false : Boolean(include_paragraph_anchors);
 
   return {
     ok: true,
@@ -353,9 +360,9 @@ export function buildReviewBundlePlan(dbHandle, {
       project_id,
       filters: appliedFilters,
       options: {
-        include_scene_ids: Boolean(include_scene_ids),
-        include_metadata_sidebar: Boolean(include_metadata_sidebar),
-        include_paragraph_anchors: Boolean(include_paragraph_anchors),
+        include_scene_ids: resolvedIncludeSceneIds,
+        include_metadata_sidebar: resolvedIncludeMetadataSidebar,
+        include_paragraph_anchors: resolvedIncludeParagraphAnchors,
         beta_accountability: resolvedBetaAccountability,
         ...(resolvedRecipientName ? { recipient_name: resolvedRecipientName } : {}),
       },

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

@@ -233,6 +233,122 @@ 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,
@@ -278,11 +394,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);
@@ -346,11 +462,16 @@ function buildPageFingerprintToken({ seedHash, pageNumber }) {
   return `BR-${digest}-P${String(pageNumber).padStart(3, "0")}`;
 }
 
+function sanitizeFooterRecipientDisplayName(recipientDisplayName) {
+  return String(recipientDisplayName ?? "").replaceAll("|", "/");
+}
+
 export function renderReviewBundleMarkdown(dbHandle, plan, { generatedAt, syncDir: syncDirOpt } = {}) {
   const profile = plan.profile;
-  const includeSceneIds = Boolean(plan.resolved_scope?.options?.include_scene_ids);
-  const includeMetadataSidebar = Boolean(plan.resolved_scope?.options?.include_metadata_sidebar);
-  const includeParagraphAnchors = Boolean(plan.resolved_scope?.options?.include_paragraph_anchors);
+  const isBetaProfile = profile === "beta_reader_personalized";
+  const includeSceneIds = isBetaProfile ? false : Boolean(plan.resolved_scope?.options?.include_scene_ids);
+  const includeMetadataSidebar = isBetaProfile ? false : Boolean(plan.resolved_scope?.options?.include_metadata_sidebar);
+  const includeParagraphAnchors = isBetaProfile ? false : Boolean(plan.resolved_scope?.options?.include_paragraph_anchors);
   // Prefer explicitly threaded syncDir; fall back to env.
   // No further fallback: if syncDir is null, resolveSceneFilePath returns null
   // and SCENE_PROSE_READ_FAILED is thrown, making misconfiguration explicit.
@@ -365,12 +486,13 @@ export function renderReviewBundleMarkdown(dbHandle, plan, { generatedAt, syncDi
   const headerLines = [
     `# Review Bundle: ${escapeMarkdown(plan.resolved_scope.project_id)}`,
     "",
-    `- Profile: ${profile}`,
+    ...(profile !== "beta_reader_personalized" ? [`- Profile: ${profile}`] : []),
     ...(profile === "beta_reader_personalized"
       ? [`- Recipient: ${escapeMarkdown(recipientDisplayName)}`]
       : []),
-    `- Generated at: ${generatedAt ?? new Date().toISOString()}`,
-    `- Scene count: ${plan.summary.scene_count}`,
+    ...(profile !== "beta_reader_personalized"
+      ? [`- Generated at: ${generatedAt ?? new Date().toISOString()}`, `- Scene count: ${plan.summary.scene_count}`]
+      : []),
   ];
   sections.push(headerLines.join("\n"));
 
@@ -421,13 +543,25 @@ export function renderReviewBundlePdf(dbHandle, plan, { generatedAt, syncDir: sy
 
 export function renderReviewBundlePdfWithMetadata(dbHandle, plan, { generatedAt, syncDir: syncDirOpt } = {}) {
   const profile = plan.profile;
-  const includeSceneIds = Boolean(plan.resolved_scope?.options?.include_scene_ids);
+  const includeSceneIds = profile === "beta_reader_personalized"
+    ? false
+    : Boolean(plan.resolved_scope?.options?.include_scene_ids);
   const syncDir = syncDirOpt ?? process.env.WRITING_SYNC_DIR ?? null;
+  const isBetaProfile = profile === "beta_reader_personalized";
+  const proseFontSize = isBetaProfile ? 8 : 10;
+  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 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 recipientName = plan.resolved_scope?.options?.recipient_name;
   const recipientDisplayName = normalizeRecipientDisplayName(recipientName);
+  const footerRecipientDisplayName = sanitizeFooterRecipientDisplayName(recipientDisplayName);
   const betaAccountabilityEnabled = profile === "beta_reader_personalized"
     && Boolean(plan.resolved_scope?.options?.beta_accountability);
   const effectiveGeneratedAt = generatedAt ?? new Date().toISOString();
@@ -441,7 +575,8 @@ export function renderReviewBundlePdfWithMetadata(dbHandle, plan, { generatedAt,
   const pdfOptions = profile === "beta_reader_personalized"
     ? {
         size: [432, 648], // 6x9in in PDF points
-        margins: { top: 64, right: 58, bottom: 72, left: 58 },
+        // Extra bottom margin reserves clear space above the accountability footer.
+        margins: { top: 64, right: 58, bottom: 96, left: 58 },
         autoFirstPage: false,
       }
     : {
@@ -455,23 +590,30 @@ export function renderReviewBundlePdfWithMetadata(dbHandle, plan, { generatedAt,
 
   const drawAccountabilityFooter = () => {
     if (!betaAccountabilityEnabled || !fingerprintSeedHash) return;
+    const previousX = doc.x;
+    const previousY = doc.y;
     pageNumber += 1;
     const token = buildPageFingerprintToken({
       seedHash: fingerprintSeedHash,
       pageNumber,
     });
     pageTokens.push({ page: pageNumber, token });
-    const footerY = doc.page.height - doc.page.margins.bottom - 12;
-    const footerWidth = doc.page.width - doc.page.margins.left - doc.page.margins.right;
-    const footerText = `For: ${recipientDisplayName} | Fingerprint: ${token} | Page ${pageNumber}`;
+    const footerY = doc.page.height - 42;
+    const footerText = `For: ${footerRecipientDisplayName} | Fingerprint: ${token}`;
+    const pageNumberText = String(pageNumber);
     doc.save();
-    doc.font("Helvetica").fontSize(8).fillColor("#555555");
-    doc.text(footerText, doc.page.margins.left, footerY, {
-      width: footerWidth,
-      align: "left",
-      lineBreak: false,
-    });
+    doc.font("Times-Roman").fontSize(8).fillColor("#555555");
+    // Draw footer in no-wrap mode to avoid layout flow side effects.
+    doc.text(footerText, doc.page.margins.left, footerY, { lineBreak: false });
+    const pageNumberWidth = doc.widthOfString(pageNumberText);
+    const pageNumberX = (doc.page.width - pageNumberWidth) / 2;
+    doc.text(pageNumberText, pageNumberX, doc.page.height - 24, { lineBreak: false });
     doc.restore();
+    // Restore prose style so auto-flowed text keeps consistent typography
+    // on pages added during long text rendering.
+    doc.font(bodyFont).fontSize(proseFontSize).fillColor("#000000");
+    doc.x = previousX;
+    doc.y = previousY;
   };
   doc.on("pageAdded", drawAccountabilityFooter);
 
@@ -505,22 +647,26 @@ export function renderReviewBundlePdfWithMetadata(dbHandle, plan, { generatedAt,
 
     try {
       doc.addPage();
-      doc.fontSize(24).font("Helvetica-Bold").text(`Review Bundle: ${plan.resolved_scope.project_id}`, { align: "left" });
-      doc.moveDown(0.5);
-      doc.fontSize(11).font("Helvetica");
-      doc.text(`Profile: ${profile}`, { align: "left" });
+      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" });
+      }
       if (profile === "beta_reader_personalized") {
         doc.text(`Recipient: ${recipientDisplayName}`, { align: "left" });
+      } else {
+        doc.text(`Generated: ${effectiveGeneratedAt}`, { align: "left" });
+        doc.text(`Scenes: ${plan.summary.scene_count}`, { align: "left" });
       }
-      doc.text(`Generated: ${effectiveGeneratedAt}`, { align: "left" });
-      doc.text(`Scenes: ${plan.summary.scene_count}`, { align: "left" });
       doc.moveDown();
 
       if (profile === "beta_reader_personalized") {
-        doc.fontSize(12).font("Helvetica-Bold").text("Usage Notice", { align: "left" });
+        doc.fontSize(12).font("Times-Bold").text("Usage Notice", { align: "left" });
         doc.moveDown(0.3);
         const noticeWidth = doc.page.width - doc.page.margins.left - doc.page.margins.right;
-        doc.fontSize(10).font("Helvetica");
+        doc.fontSize(10).font("Times-Roman");
         doc.text("This beta-reader draft is intended for private review and feedback. Please do not redistribute without explicit author permission.", {
           align: "left",
           width: noticeWidth,
@@ -529,22 +675,29 @@ export function renderReviewBundlePdfWithMetadata(dbHandle, plan, { generatedAt,
       }
 
       for (const scene of rows) {
-        doc.fontSize(14).font("Helvetica-Bold");
+        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}]`;
         }
-        doc.text(heading, { align: "left" });
-        doc.moveDown(0.2);
+        doc.text(heading, { align: isBetaProfile ? "center" : "left" });
+        doc.moveDown(isBetaProfile ? 1.6 : 0.2);
 
         const metaParts = [];
-        if (scene.pov) metaParts.push(`POV: ${scene.pov}`);
-        if (scene.save_the_cat_beat) metaParts.push(`Beat: ${scene.save_the_cat_beat}`);
+        if (profile !== "beta_reader_personalized") {
+          if (scene.pov) metaParts.push(`POV: ${scene.pov}`);
+          if (scene.save_the_cat_beat) metaParts.push(`Beat: ${scene.save_the_cat_beat}`);
+        }
         if (metaParts.length > 0) {
           const metaWidth = doc.page.width - doc.page.margins.left - doc.page.margins.right;
-          doc.fontSize(9).font("Helvetica-Oblique");
+          doc.fontSize(9).font(metaFont);
           doc.text(metaParts.join(" • "), { align: "left", width: metaWidth });
-          doc.font("Helvetica");
+          doc.font(bodyFont);
           doc.moveDown(0.2);
         }
 
@@ -569,14 +722,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(10).font("Helvetica");
-          doc.text(prose, {
+          renderProseWithInlineEmphasis(doc, prose, {
+            bodyFont,
+            italicFont,
+            fontSize: proseFontSize,
             align: "left",
             width: textWidth,
-            lineGap: profile === "beta_reader_personalized" ? 4.5 : 3,
+            lineGap: proseLineGap,
+            paragraphGap: 0,
+            blankLineMoveDown: isBetaProfile ? 0.15 : 0.65,
           });
         }
 
@@ -607,4 +781,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";