openwriter 0.28.0 → 0.28.1

package/dist/server/markdown-parse.js

@@ -830,3 +830,45 @@ function splitParagraphOnDoubleBreaks(content, firstId) {
         content: g,
     }));
 }
+/**
+ * Heal fused double-`<br>` paragraphs across a node array — the structured/JSON
+ * sibling of the per-paragraph `splitParagraphOnDoubleBreaks` heal that runs on
+ * the markdown-string parse path. Agent writes and Author's Voice rewrites can
+ * arrive as TipTap JSON, which never passes through markdown-it, so a tweet-style
+ * paragraph carrying a run of 2+ `hardBreak`s would otherwise enter canonical as
+ * a single fused node the parser refuses to reproduce — breaking the serialize→
+ * reparse round-trip (sync-check FAIL), destabilizing the node-identity matcher,
+ * and corrupting pending decorations.
+ *
+ * Canonical form for every doc type is separate paragraph nodes; intra-paragraph
+ * SINGLE `<br>`s (legitimate soft line breaks) pass through untouched. Idempotent:
+ * already-split content (e.g. from the markdown path) is returned unchanged, with
+ * the original node and all its attrs preserved when no fusion is present.
+ *
+ * adr: adr/tweet-paragraph-convention.md
+ */
+export function splitFusedParagraphs(nodes) {
+    if (!Array.isArray(nodes))
+        return nodes;
+    const out = [];
+    for (const node of nodes) {
+        if (node?.type === 'paragraph' && Array.isArray(node.content)) {
+            const split = splitParagraphOnDoubleBreaks(node.content, node.attrs?.id || null);
+            if (split.length <= 1) {
+                // No fusion — keep the original node intact (preserves all attrs).
+                out.push(node);
+            }
+            else {
+                // Fusion healed: first chunk inherits the original node's attrs/id,
+                // the rest become fresh paragraph nodes.
+                split.forEach((p, idx) => {
+                    out.push(idx === 0 ? { ...node, content: p.content } : p);
+                });
+            }
+        }
+        else {
+            out.push(node);
+        }
+    }
+    return out;
+}

package/dist/server/markdown.js

@@ -11,7 +11,7 @@ import { tiptapToMarkdown } from './markdown-serialize.js';
 import { markdownToTiptap } from './markdown-parse.js';
 import { shapeOfTiptap, compareShapes, formatSyncReport, } from './node-sync-check.js';
 export { tiptapToMarkdown, tiptapToBody, nodeText, inlineToMarkdown } from './markdown-serialize.js';
-export { markdownToTiptap, markdownToNodes } from './markdown-parse.js';
+export { markdownToTiptap, markdownToNodes, splitFusedParagraphs } from './markdown-parse.js';
 export { shapeOfTiptap, computeShape, compareShapes, formatSyncReport, } from './node-sync-check.js';
 /**
  * Serialize TipTap to markdown, then re-parse and verify that the round-trip

package/dist/server/mcp.js

@@ -25,7 +25,7 @@ import { toCompactFormat, compactNodes, parseMarkdownContent } from './compact.j
 import matter from 'gray-matter';
 import { getUpdateInfo } from './update-check.js';
 import { listVersions, forceSnapshot, getVersionContent } from './versions.js';
-import { tiptapToMarkdown } from './markdown.js';
+import { tiptapToMarkdown, splitFusedParagraphs } from './markdown.js';
 import { loadDocFromDisk } from './pending-overlay.js';
 import { getComments, getCommentCount, getGlobalCommentSummary, resolveComments } from './comments.js';
 import { readTasks, addTask, updateTask, removeTask } from './tasks.js';
@@ -293,14 +293,19 @@ export const TOOL_REGISTRY = [
                 if (typeof resolved.content === 'string') {
                     resolved.content = parseMarkdownContent(resolved.content);
                 }
-                // Tweet docs used to collapse multi-paragraph content into a single
-                // paragraph with hardBreaks (mergeParagraphsToHardBreaks). That made
-                // every multi-paragraph write land as ONE pending-insert decoration
-                // for review, which destroys per-paragraph accept/reject. Each
-                // paragraph the agent writes is a distinct review unit; preserve
-                // that structure. Thread separation (multiple tweets in a thread)
-                // is signaled explicitly by horizontalRule nodes from the agent,
-                // not implied by paragraph count.
+                // Canonical form for every doc type (including X templates) is separate
+                // paragraph nodes — one node per review unit, intra-paragraph single
+                // <br>s preserved. Heal any fused double-<br> paragraph that arrives as
+                // TipTap JSON: the markdown-string path above already splits via
+                // markdown-it, but JSON content bypasses it, so an X-style fused node
+                // would otherwise enter canonical and break the serialize→reparse
+                // round-trip (sync-check FAIL), the node-identity matcher, and the
+                // pending decorations. Idempotent on already-split content.
+                // adr: adr/tweet-paragraph-convention.md
+                if (resolved.content != null && resolved.operation !== 'delete') {
+                    const asArray = Array.isArray(resolved.content) ? resolved.content : [resolved.content];
+                    resolved.content = splitFusedParagraphs(asArray);
+                }
                 return resolved;
             });
             // Auto-clean: if doc has only a single empty paragraph and first change is
@@ -621,12 +626,8 @@ export const TOOL_REGISTRY = [
             try {
                 let doc;
                 if (typeof content === 'string') {
-                    // Don't collapse multi-paragraph content to a single hardBreak-
-                    // separated paragraph for tweet docs. Each paragraph is a
-                    // distinct review unit (its own pending-insert decoration); the
-                    // user accepts/rejects per paragraph. Explicit thread structure
-                    // (multiple tweets in a thread) comes from horizontalRule nodes
-                    // in the agent's input, not from paragraph count.
+                    // Canonical form is separate paragraph nodes for every doc type
+                    // (including X templates) — each paragraph a distinct review unit.
                     doc = { type: 'doc', content: parseMarkdownContent(content) };
                 }
                 else if (content?.type === 'doc' && Array.isArray(content.content)) {
@@ -638,6 +639,10 @@ export const TOOL_REGISTRY = [
                         content: [{ type: 'text', text: 'Error: content must be a markdown string or TipTap JSON { type: "doc", content: [...] }' }],
                     };
                 }
+                // Heal fused double-<br> paragraphs that arrive as TipTap JSON (X-template
+                // content bypasses the markdown-string split). Idempotent on the string
+                // path above. adr: adr/tweet-paragraph-convention.md
+                doc.content = splitFusedParagraphs(doc.content);
                 // Non-active target: write directly to disk without disrupting the user's view
                 const targetIsNonActive = filename && filename !== getActiveFilename();
                 if (targetIsNonActive) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "openwriter",
-  "version": "0.28.0",
+  "version": "0.28.1",
   "description": "The open-source writing surface for AI agents. Markdown-native editor with pending change review — your agent writes, you accept or reject.",
   "type": "module",
   "license": "MIT",