npm - openwriter - Versions diffs - 0.28.1 → 0.29.0 - Mend

openwriter 0.28.1 → 0.29.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (13) hide show

package/dist/client/assets/index-Gdw1m46J.css +1 -0
package/dist/client/assets/index-SAoCjUU-.js +214 -0
package/dist/client/index.html +2 -2
package/dist/server/documents.js +16 -4
package/dist/server/index.js +25 -2
package/dist/server/mcp.js +13 -4
package/dist/server/pending-overlay.js +46 -0
package/dist/server/prompt-debug.js +68 -0
package/dist/server/state.js +74 -10
package/package.json +1 -1
package/skill/SKILL.md +3 -1
package/dist/client/assets/index-3no79ry9.css +0 -1
package/dist/client/assets/index-DzHT4klX.js +0 -214

package/dist/client/index.html CHANGED Viewed

@@ -10,8 +10,8 @@
     <link rel="preconnect" href="https://fonts.googleapis.com" />
     <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin />
     <link href="https://fonts.googleapis.com/css2?family=Charter:ital,wght@0,400;0,700;1,400&family=Crimson+Pro:ital,wght@0,300;0,400;0,600;0,700;1,400&family=DM+Sans:ital,wght@0,400;0,500;0,600;0,700;1,400&family=DM+Serif+Display&family=IBM+Plex+Mono:wght@400;500;600&family=IBM+Plex+Sans:wght@400;500;600&family=Inter:wght@400;500;600;700&family=Libre+Baskerville:ital,wght@0,400;0,700;1,400&family=Literata:ital,opsz,wght@0,7..72,400;0,7..72,600;0,7..72,700;1,7..72,400&family=Newsreader:ital,opsz,wght@0,6..72,400;0,6..72,600;1,6..72,400&family=Playfair+Display:wght@400;600;700;900&family=Source+Serif+4:ital,opsz,wght@0,8..60,400;0,8..60,600;0,8..60,700;1,8..60,400&family=Space+Grotesk:wght@400;500;600;700&family=Space+Mono:wght@400;700&display=swap" rel="stylesheet" />
-    <script type="module" crossorigin src="/assets/index-DzHT4klX.js"></script>
-    <link rel="stylesheet" crossorigin href="/assets/index-3no79ry9.css">
+    <script type="module" crossorigin src="/assets/index-SAoCjUU-.js"></script>
+    <link rel="stylesheet" crossorigin href="/assets/index-Gdw1m46J.css">
   </head>
   <body>
     <div id="root"></div>

package/dist/server/documents.js CHANGED Viewed

@@ -1108,7 +1108,7 @@ export function openFile(fullPath) {
     const filename = isExternalDoc(canonPath) ? canonPath : baseName;
     return { document: getDocument(), title: getTitle(), filename };
 }
-export function duplicateDocument(filename) {
+export function duplicateDocument(filename, variant) {
     // Cancel any pending debounced save, then save current doc immediately
     cancelDebouncedSave();
     save();
@@ -1118,17 +1118,29 @@ export function duplicateDocument(filename) {
     }
     const raw = readFileSync(sourcePath, 'utf-8');
     const parsed = markdownToTiptap(raw);
+    // Title suffix: variants read as "(Tweet)" / "(Blog)", plain copies as "(Copy)".
+    const suffix = variant?.variantType
+        ? variant.variantType.charAt(0).toUpperCase() + variant.variantType.slice(1)
+        : 'Copy';
     // Generate deduplicated title
-    let newTitle = `${parsed.title} (Copy)`;
+    let newTitle = `${parsed.title} (${suffix})`;
     let filePath = filePathForTitle(newTitle);
     if (existsSync(filePath)) {
         let counter = 2;
-        while (existsSync(filePathForTitle(`${parsed.title} (Copy ${counter})`)))
+        while (existsSync(filePathForTitle(`${parsed.title} (${suffix} ${counter})`)))
             counter++;
-        newTitle = `${parsed.title} (Copy ${counter})`;
+        newTitle = `${parsed.title} (${suffix} ${counter})`;
         filePath = filePathForTitle(newTitle);
     }
     const metadata = { ...parsed.metadata, title: newTitle, docId: generateNodeId() };
+    // Variant relationship — set AFTER the spread so it overrides any inherited
+    // masterDocId/variantType from the source doc. masterDocId points at the
+    // source (the master); variantType labels this copy's intended format.
+    // adr: docs/variants.md
+    if (variant?.masterDocId)
+        metadata.masterDocId = variant.masterDocId;
+    if (variant?.variantType)
+        metadata.variantType = variant.variantType;
     setActiveDocument(parsed.document, newTitle, filePath, false, undefined, metadata);
     const { markdown } = tiptapToMarkdownChecked(parsed.document, newTitle, metadata);
     ensureDataDir();

package/dist/server/index.js CHANGED Viewed

@@ -15,6 +15,7 @@ import { save, cancelDebouncedSave, load, getDocument, getTitle, getFilePath, ge
 import { syncPostHistory } from './post-sync.js';
 import { enrollManualPostForAutoplug } from './autoplug-enroll.js';
 import { listDocuments, switchDocument, createDocument, deleteDocument, duplicateDocument, reloadDocument, updateDocumentTitle, openFile, reorderDocs, searchDocuments, listArchivedDocuments, archiveDocument, unarchiveDocument, getActiveFilename, batchResolve, listPendingSorts } from './documents.js';
+import { writePromptDebug, isPromptDebugEnabled } from './prompt-debug.js';
 import { createWorkspaceRouter } from './workspace-routes.js';
 import { createLinkRouter } from './link-routes.js';
 import { createTweetRouter } from './tweet-routes.js';
@@ -224,6 +225,28 @@ export async function startHttpServer(options = {}) {
             res.status(500).json({ error: err.message });
         }
     });
+    // Prompt debug inspector: write the realized AV prompt to a timestamped .md doc
+    // for hand review. Off by default — gated by OW_PROMPT_DEBUG (see docs/prompt-debug.md).
+    // When off, no-ops so the client POST stays harmless.
+    app.post('/api/prompt-debug', (req, res) => {
+        try {
+            if (!isPromptDebugEnabled()) {
+                res.json({ success: false, skipped: true });
+                return;
+            }
+            const { action, debug, metadata } = req.body;
+            if (!debug) {
+                res.status(400).json({ error: 'debug payload is required' });
+                return;
+            }
+            const filename = writePromptDebug(action, debug, metadata);
+            broadcastDocumentsChanged();
+            res.json({ success: true, filename });
+        }
+        catch (err) {
+            res.status(500).json({ error: err.message });
+        }
+    });
     // Toggle auto-accept on a workspace or container. Inherits to every doc inside.
     // Body: { wsFile, containerId?, enabled }. Omit containerId to target the
     // whole workspace; pass it to target a specific container.
@@ -601,12 +624,12 @@ export async function startHttpServer(options = {}) {
     });
     app.post('/api/documents/duplicate', (req, res) => {
         try {
-            const { filename } = req.body;
+            const { filename, masterDocId, variantType } = req.body;
             if (!filename) {
                 res.status(400).json({ error: 'filename is required' });
                 return;
             }
-            const result = duplicateDocument(filename);
+            const result = duplicateDocument(filename, (masterDocId || variantType) ? { masterDocId, variantType } : undefined);
             broadcastDocumentSwitched(result.document, result.title, result.filename);
             broadcastDocumentsChanged();
             res.json(result);

package/dist/server/mcp.js CHANGED Viewed

@@ -497,8 +497,10 @@ export const TOOL_REGISTRY = [
             url: z.string().optional().describe('Tweet URL — REQUIRED for content_type "reply" or "quote" (e.g. "https://x.com/user/status/123"). Sets tweetContext.url automatically. Ignored for other content types.'),
             afterId: z.string().optional().describe('Place the new doc immediately after this docId (8-char hex) or containerId inside its parent. Omit to append to the bottom of the parent (the default — matches ascending-order convention: newest at bottom). Requires workspace.'),
             status: z.enum(['canonical', 'draft']).optional().describe('Agent-owned lifecycle. "canonical" = committed to spine / load-bearing for the workspace (use for Beats docs that have locked, Research Notes, Master References). "draft" = working / not load-bearing yet / scratch (DUMP docs, first-pass beats). Defaults to "draft" when omitted. Change later via set_metadata({ status: ... }) on lifecycle transitions. v0.19.0.'),
+            masterDocId: z.string().optional().describe('Make this doc a VARIANT of another doc. Pass the master doc\'s docId (8-char hex). The variant nests under its master in the sidebar (expandable tree). Use when creating a derivative — e.g. a tweet thread from a blog post. Pair with variantType. See docs/variants.md.'),
+            variantType: z.string().optional().describe('Label for what kind of variant this is (e.g. "tweet", "blog", "linkedin"). Shows as a badge in the sidebar. Only meaningful alongside masterDocId.'),
         },
-        handler: async ({ title, path, workspace, container, empty, content_type, url, afterId, status }) => {
+        handler: async ({ title, path, workspace, container, empty, content_type, url, afterId, status, masterDocId, variantType }) => {
             // Require url for reply/quote
             if ((content_type === 'reply' || content_type === 'quote') && !url) {
                 return { content: [{ type: 'text', text: `Error: content_type "${content_type}" requires a url parameter (e.g. "https://x.com/user/status/123").` }] };
@@ -532,13 +534,20 @@ export const TOOL_REGISTRY = [
             // canonical via set_metadata({ status: "canonical" }) on lifecycle
             // transitions. See brief 2026-05-21-simplify-enrichment-schema-three-fields.
             const statusMeta = { status: status ?? 'draft' };
+            // Variant relationship (optional) — lands on the first disk write so the
+            // doc nests under its master in the sidebar immediately. See docs/variants.md.
+            const variantMeta = {};
+            if (masterDocId)
+                variantMeta.masterDocId = masterDocId;
+            if (variantType)
+                variantMeta.variantType = variantType;
             try {
                 if (empty) {
                     // Immediate switch — no spinner, no populate_document needed
                     const result = createDocument(title, undefined, path);
                     setAgentLock(result.filename);
-                    // Apply status + type-specific metadata in one merge
-                    const initMeta = { ...statusMeta };
+                    // Apply status + variant + type-specific metadata in one merge
+                    const initMeta = { ...statusMeta, ...variantMeta };
                     if (content_type) {
                         const typeMeta = resolveTypeMeta(content_type, url);
                         if (typeMeta)
@@ -578,7 +587,7 @@ export const TOOL_REGISTRY = [
                 // Merge status with any content-type metadata so it lands on the first
                 // disk write.
                 const typeMeta = content_type ? resolveTypeMeta(content_type, url) : undefined;
-                const initialMeta = { ...statusMeta, ...(typeMeta || {}) };
+                const initialMeta = { ...statusMeta, ...variantMeta, ...(typeMeta || {}) };
                 const result = createDocumentFile(title, path, initialMeta);
                 let wsInfo = '';
                 if (wsTarget) {

package/dist/server/pending-overlay.js CHANGED Viewed

@@ -859,6 +859,52 @@ function sameContent(a, b) {
     const bClean = sanitizeNodeForBaseline(b);
     return JSON.stringify(aClean) === JSON.stringify(bClean);
 }
+/**
+ * Repair canonical rewrite nodes that still hold the rewrite's NEW content.
+ *
+ * `splitMergedDoc` / `stripPendingFromDoc` can only revert a rewrite via the
+ * node's own `pendingOriginalContent` attr. When a merged doc arrives from an
+ * untrusted source — a browser doc-update (`syncBrowserDocUpdate`) — whose
+ * rewrite node dropped or staled that attr, the revert silently fails and the
+ * rewrite TEXT lands in canonical. The next `applyOverlayPure` then compares
+ * canonical-as-rewrite to the (still-correct) overlay baseline and falsely
+ * flags `pendingStaleBaseline` (the amber dotted-underline indicator).
+ *
+ * The server's overlay entry retains the authoritative `originalBaseline`, so
+ * re-assert it here — but ONLY when the canonical node currently equals the
+ * entry's `newContent`. That condition means "the revert failed." If canonical
+ * holds anything else (a genuine out-of-band edit), it is left untouched so
+ * real stale-baseline drift is still surfaced for review.
+ *
+ * adr: adr/pending-overlay-model.md · adr: adr/tweet-paragraph-convention.md
+ */
+export function reconcileCanonicalToBaselines(canonical, entries) {
+    const byId = new Map();
+    for (const e of entries) {
+        if (e.status === 'rewrite' && e.originalBaseline?.content && e.newContent?.content) {
+            byId.set(e.nodeId, e);
+        }
+    }
+    if (byId.size === 0)
+        return;
+    const key = (content) => JSON.stringify(sanitizeNodeForBaseline({ content }));
+    function walk(nodes) {
+        if (!Array.isArray(nodes))
+            return;
+        for (const n of nodes) {
+            const e = n?.attrs?.id ? byId.get(n.attrs.id) : undefined;
+            if (e && key(n.content) === key(e.newContent.content)) {
+                // Canonical is holding the rewrite text — the split couldn't revert it.
+                // Restore the authoritative baseline.
+                n.content = JSON.parse(JSON.stringify(e.originalBaseline.content));
+            }
+            else if (n.content) {
+                walk(n.content);
+            }
+        }
+    }
+    walk(canonical?.content || []);
+}
 function sanitizeNodeForBaseline(node) {
     // Strip volatile fields (ids, pending attrs) for content comparison.
     const cloned = JSON.parse(JSON.stringify(node));

package/dist/server/prompt-debug.js ADDED Viewed

@@ -0,0 +1,68 @@
+/**
+ * File: prompt-debug.ts
+ * Purpose: Write AV prompt debug data to timestamped .md files for inspection.
+ *   Each enhance creates a new `_prompt-<action>-<ts>.md` doc in the data dir,
+ *   visible in the OpenWriter sidebar — so the exact system+user prompt that was
+ *   sent to the AI can be reviewed by hand.
+ * Control: gated by isPromptDebugEnabled() (OW_PROMPT_DEBUG env). Off by default.
+ *   See docs/prompt-debug.md for the on/off switch.
+ */
+import { getDataDir, ensureDataDir, atomicWriteFileSync } from './helpers.js';
+import { join } from 'path';
+/** True when the prompt-debug inspector is switched on. Read lazily so the
+ *  process picks up OW_PROMPT_DEBUG without code changes. */
+export function isPromptDebugEnabled() {
+    const v = process.env.OW_PROMPT_DEBUG;
+    return v === '1' || v === 'true';
+}
+/**
+ * Write prompt debug info to a timestamped markdown file.
+ * Returns the filename created.
+ */
+export function writePromptDebug(action, debug, metadata) {
+    ensureDataDir();
+    const now = new Date();
+    const ts = now.toISOString().replace(/[:.]/g, '-').slice(0, 19);
+    const filename = `_prompt-${action || 'debug'}-${ts}.md`;
+    const filePath = join(getDataDir(), filename);
+    const timeStr = now.toLocaleTimeString('en-US', { hour12: true, hour: '2-digit', minute: '2-digit', second: '2-digit' });
+    let md = `---\ntitle: "Prompt Debug: ${action} @ ${timeStr}"\n---\n\n`;
+    // Metadata summary
+    if (metadata) {
+        md += `## Metadata\n\n`;
+        md += `| Key | Value |\n|-----|-------|\n`;
+        if (metadata.action)
+            md += `| Action | ${metadata.action} |\n`;
+        if (metadata.profileUsed)
+            md += `| Profile | ${metadata.profileUsed} |\n`;
+        if (metadata.nodesIn != null)
+            md += `| Nodes In | ${metadata.nodesIn} |\n`;
+        if (metadata.nodesOut != null)
+            md += `| Nodes Out | ${metadata.nodesOut} |\n`;
+        if (metadata.ragExamples != null)
+            md += `| RAG Examples | ${metadata.ragExamples} |\n`;
+        if (metadata.ragTotalWords != null)
+            md += `| RAG Total Words | ${metadata.ragTotalWords} |\n`;
+        if (metadata.processingTimeMs != null)
+            md += `| Processing Time | ${metadata.processingTimeMs}ms |\n`;
+        if (metadata.estimatedCost != null)
+            md += `| Estimated Cost | $${metadata.estimatedCost.toFixed(4)} |\n`;
+        md += `\n`;
+    }
+    // System prompt
+    md += `## System Prompt\n\n`;
+    md += debug.systemPrompt + '\n\n';
+    // User prompt
+    md += `---\n\n## User Prompt\n\n`;
+    md += debug.userPrompt + '\n\n';
+    // RAG examples
+    if (debug.ragExamples && debug.ragExamples.length > 0) {
+        md += `---\n\n## RAG Examples (${debug.ragExamples.length})\n\n`;
+        for (const ex of debug.ragExamples) {
+            md += `### ${ex.anchor} (${ex.wordCount} words)\n\n`;
+            md += ex.context + '\n\n';
+        }
+    }
+    atomicWriteFileSync(filePath, md);
+    return filename;
+}

package/dist/server/state.js CHANGED Viewed

@@ -9,14 +9,14 @@ import matter from 'gray-matter';
 import { tiptapToMarkdown, tiptapToMarkdownChecked, tiptapToBody, markdownToTiptap } from './markdown.js';
 import { applyTextEditsToNode } from './text-edit.js';
 import { getDataDir, TEMP_PREFIX, ensureDataDir, filePathForTitle, tempFilePath, generateNodeId, LEAF_BLOCK_TYPES, resolveDocPath, isExternalDoc, atomicWriteFileSync, canonicalizePath, canonicalizeIdentifier } from './helpers.js';
-import { snapshotIfNeeded, ensureDocId } from './versions.js';
+import { snapshotIfNeeded, ensureDocId, forceSnapshot } from './versions.js';
 import { syncReferencesFromProse, invalidateBacklinksCache, writeFrontmatter } from './backlinks.js';
 import { isAutoAcceptInheritedForDoc } from './workspaces.js';
 import { matchNodes } from './node-matcher.js';
 import { tiptapToBlocks, applyIdsToTiptap } from './node-blocks.js';
 import { anyLegacyRaw } from './node-fingerprint.js';
 import { markdownToNodes, resolvePreviousNodes, resolveGraveyard } from './markdown-parse.js';
-import { extractOverlay, applyOverlayPure, splitMergedDoc, saveOverlay, loadOverlay, loadDocFromDisk, deleteOverlay, repairOverlaysOnStartup, diagLog } from './pending-overlay.js';
+import { extractOverlay, applyOverlayPure, splitMergedDoc, reconcileCanonicalToBaselines, saveOverlay, loadOverlay, loadDocFromDisk, deleteOverlay, repairOverlaysOnStartup, diagLog } from './pending-overlay.js';
 import { loadPendingMetadata, savePendingMetadata } from './pending-metadata.js';
 import { harvestSentenceHashes, harvestCharCount, isEnrichmentStale } from './enrichment.js';
 import { clearActivityBuffer } from './activity-log.js';
@@ -145,6 +145,42 @@ function setPrimaryFromMerged(merged) {
     state.canonical = canonical;
     setOverlayFromEntries(overlayEntries);
 }
+/**
+ * Browser-write body-fidelity invariant. adr: adr/browser-write-fidelity.md
+ *
+ * A browser editor surface can mount empty, or parse the canonical body
+ * through a NARROWER schema (the X-article / tweet compose extensions),
+ * then autosave that lossy view back — collapsing a populated body to
+ * empty/near-empty on disk. This is the "autosave-clobber" class: silent
+ * data loss the user never authored.
+ *
+ * The invariant lives at the BROWSER-WRITE BOUNDARY — every function that
+ * replaces canonical from a browser-sent doc (`updateDocument`,
+ * `syncBrowserDocUpdate`, `saveDocToFile`). It deliberately does NOT live
+ * at the disk chokepoint (`writeToDisk`): restore_version, MCP edits, and
+ * agent `applyChanges` mutate canonical directly and are TRUSTED to shrink
+ * a doc intentionally. Recovery-restores GROW the doc (incoming > current)
+ * and so pass freely here too.
+ *
+ * A replacement that collapses a substantial body (>5 nodes) to under 30%
+ * of its node count is a view artifact, not an edit — refuse it.
+ */
+export function wouldCollapseBody(current, incoming) {
+    const currentNodes = current?.content?.length ?? 0;
+    const incomingNodes = incoming?.content?.length ?? 0;
+    return currentNodes > 5 && incomingNodes < currentNodes * 0.3;
+}
+/**
+ * Checkpoint-then-refuse helper for the active doc. Forces a version
+ * snapshot of the current ON-DISK body — still the good content, since the
+ * clobbering write is being refused — so it is recoverable under a labeled
+ * version even if no prior autosave snapshot happened to retain it.
+ */
+function checkpointActiveBody() {
+    const docId = getDocId();
+    if (docId && state.filePath)
+        forceSnapshot(docId, state.filePath);
+}
 /**
  * Sync routing for a stale-version browser doc-update. The browser's
  * submission was captured at server version `browserVersion`; the server
@@ -168,6 +204,15 @@ function setPrimaryFromMerged(merged) {
  * adr: adr/pending-overlay-model.md
  */
 export function syncBrowserDocUpdate(browserDoc, browserVersion) {
+    // Browser-write fidelity: a STALE-version browser doc-update is the same
+    // autosave-clobber class as the current-version path — and this path
+    // previously bypassed the guard, writing the empty surface straight to
+    // canonical. Refuse + checkpoint here too. adr: adr/browser-write-fidelity.md
+    if (wouldCollapseBody(state.document, browserDoc)) {
+        checkpointActiveBody();
+        console.error(`[State] REFUSED body-collapse in syncBrowserDocUpdate: ${browserDoc?.content?.length ?? 0} nodes would replace ${state.document?.content?.length ?? 0} nodes (checkpointed, write refused)`);
+        return { preservedServerEntries: 0 };
+    }
     const { canonical: browserCanonical, overlayEntries: browserOverlay } = splitMergedDoc(browserDoc);
     // Identify server overlay entries to preserve: those added after browser's baseline.
     const preserved = [];
@@ -188,7 +233,14 @@ export function syncBrowserDocUpdate(browserDoc, browserVersion) {
     }
     // Apply: browser's canonical view + merged overlay.
     state.canonical = browserCanonical;
-    setOverlayFromEntries(Array.from(merged.values()));
+    // The browser-derived canonical may still hold a rewrite's NEW text when the
+    // browser dropped that node's pendingOriginalContent (stripPendingFromDoc
+    // couldn't revert it). Re-assert the authoritative baseline from the merged
+    // overlay so the next applyOverlayPure doesn't falsely flag pendingStaleBaseline.
+    // adr: adr/pending-overlay-model.md
+    const mergedEntries = Array.from(merged.values());
+    reconcileCanonicalToBaselines(state.canonical, mergedEntries);
+    setOverlayFromEntries(mergedEntries);
     return { preservedServerEntries: preserved.length };
 }
 const listeners = new Set();
@@ -846,13 +898,14 @@ export function getStatus() {
 // SETTERS
 // ============================================================================
 export function updateDocument(doc) {
-    // Safety: reject dramatically smaller documents (same logic as destructive save check).
-    // Prevents stale browser tabs from overwriting the correct in-memory state with
-    // corrupted content (e.g. tweet compose view sending 4-node doc vs 40-node original).
-    const currentNodes = state.document?.content?.length ?? 0;
-    const incomingNodes = doc?.content?.length ?? 0;
-    if (currentNodes > 5 && incomingNodes < currentNodes * 0.3) {
-        console.error(`[State] BLOCKED destructive updateDocument: ${incomingNodes} nodes would replace ${currentNodes} nodes`);
+    // Browser-write fidelity: refuse a clobbering body-collapse. A compose
+    // surface that mounted empty (wrong view) or parsed the body through a
+    // narrower schema can otherwise overwrite a populated body with near-
+    // nothing. Checkpoint the good on-disk body first, then refuse.
+    // adr: adr/browser-write-fidelity.md
+    if (wouldCollapseBody(state.document, doc)) {
+        checkpointActiveBody();
+        console.error(`[State] REFUSED body-collapse in updateDocument: ${doc?.content?.length ?? 0} nodes would replace ${state.document?.content?.length ?? 0} nodes (checkpointed, write refused)`);
         return;
     }
     // Trust the browser-sent doc as authoritative. The WebSocket handler's
@@ -2706,6 +2759,17 @@ export function saveDocToFile(filename, doc) {
     try {
         const raw = readFileSync(targetPath, 'utf-8');
         const parsed = markdownToTiptap(raw);
+        // Browser-write fidelity: this routes a doc-update to a NON-active file
+        // (the server switched away mid-flight). Refuse a clobbering collapse
+        // against the target's own on-disk body; checkpoint it first.
+        // adr: adr/browser-write-fidelity.md
+        if (wouldCollapseBody(parsed.document, doc)) {
+            const refuseDocId = (parsed.metadata && typeof parsed.metadata.docId === 'string') ? parsed.metadata.docId : '';
+            if (refuseDocId)
+                forceSnapshot(refuseDocId, targetPath);
+            console.error(`[State] REFUSED body-collapse in saveDocToFile(${filename}): ${doc?.content?.length ?? 0} nodes would replace ${parsed.document?.content?.length ?? 0} nodes (checkpointed, write refused)`);
+            return;
+        }
         // Transfer pending attrs from on-disk version to the incoming doc
         if (hasPendingChanges(parsed.document)) {
             transferPendingAttrs(parsed.document, doc);

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "openwriter",
-  "version": "0.28.1",
+  "version": "0.29.0",
   "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",

package/skill/SKILL.md CHANGED Viewed

@@ -16,7 +16,7 @@ description: |
   Requires: OpenWriter MCP server configured. Browser UI at localhost:5050.
 metadata:
   author: travsteward
-  version: "0.16.3"
+  version: "0.16.4"
   repository: https://github.com/travsteward/openwriter
 license: MIT
 ---
@@ -346,6 +346,8 @@ This eliminates the need for separate `create_workspace`, `create_container`, an
 ### Batched Creation (multiple docs at once)
+**Variants** — when repurposing a doc into another format (a thread off a blog post, a LinkedIn cut of a newsletter), pass `create_document({ masterDocId, variantType })` so the new doc nests under its master in the sidebar instead of floating off as a disconnected doc. Users do the same via right-click → "Create variant".
 When creating **two or more documents together** — a tweet thread saved as separate docs, a series of blog drafts, newsletter variants, a workspace populated with several files — use `declare_writes` instead of looping `create_document`. It's one tool call, registers all sidebar spinners atomically, and survives app refreshes.
 ```