npm - openwriter - Versions diffs - 0.6.6 → 0.6.7 - Mend

openwriter 0.6.6 → 0.6.7

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 (10) hide show

package/dist/client/assets/{index-BuXVL8Bc.css → index-BDSezXwn.css} +1 -1
package/dist/client/assets/{index-CZrV9Ryb.js → index-DxAyyL3w.js} +45 -45
package/dist/client/index.html +2 -2
package/dist/server/documents.js +19 -6
package/dist/server/markdown-parse.js +3 -2
package/dist/server/markdown-serialize.js +4 -0
package/dist/server/markdown.js +1 -1
package/dist/server/mcp.js +70 -38
package/dist/server/state.js +155 -8
package/package.json +1 -1

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-CZrV9Ryb.js"></script>
-    <link rel="stylesheet" crossorigin href="/assets/index-BuXVL8Bc.css">
+    <script type="module" crossorigin src="/assets/index-DxAyyL3w.js"></script>
+    <link rel="stylesheet" crossorigin href="/assets/index-BDSezXwn.css">
   </head>
   <body>
     <div id="root"></div>

package/dist/server/documents.js CHANGED Viewed

@@ -119,7 +119,13 @@ export function listDocuments() {
             const stat = statSync(extPath);
             const raw = readFileSync(extPath, 'utf-8');
             const { data, content } = matter(raw);
-            const title = data.title || 'Untitled';
+            let title = data.title || 'Untitled';
+            // Title fallback: use filename stem for external files without a title
+            if (title === 'Untitled') {
+                const stem = extPath.split(/[/\\]/).pop()?.replace(/\.md$/i, '');
+                if (stem)
+                    title = stem;
+            }
             const trimmed = content.trim();
             const wordCount = trimmed ? trimmed.split(/\s+/).length : 0;
             files.push({
@@ -358,7 +364,7 @@ export function switchDocument(filename) {
     // Check cache first — preserves stable node IDs across switches
     const cached = getCachedDocument(targetPath);
     if (cached) {
-        setActiveDocument(cached.document, cached.title, targetPath, cached.isTemp, cached.lastModified, cached.metadata);
+        setActiveDocument(cached.document, cached.title, targetPath, cached.isTemp, cached.lastModified, cached.metadata, cached.originalFrontmatter);
         return { document: getDocument(), title: getTitle(), filename };
     }
     const raw = readFileSync(targetPath, 'utf-8');
@@ -367,7 +373,7 @@ export function switchDocument(filename) {
     // Ensure docId exists on loaded doc metadata (lazy migration)
     ensureDocId(parsed.metadata);
     const baseName = targetPath.split(/[/\\]/).pop() || '';
-    setActiveDocument(parsed.document, parsed.title, targetPath, baseName.startsWith(TEMP_PREFIX), mtime, parsed.metadata);
+    setActiveDocument(parsed.document, parsed.title, targetPath, baseName.startsWith(TEMP_PREFIX), mtime, parsed.metadata, parsed.rawFrontmatter);
     return { document: getDocument(), title: getTitle(), filename };
 }
 export function createDocument(title, content, path) {
@@ -486,7 +492,7 @@ export async function deleteDocument(filename) {
         throw new Error('Cannot delete the only document');
     }
     const isDeletingActive = targetPath === getFilePath();
-    if (existsSync(targetPath)) {
+    if (!isExternalDoc(filename) && existsSync(targetPath)) {
         await trash(targetPath);
     }
     if (isDeletingActive) {
@@ -552,7 +558,7 @@ export function openFile(fullPath) {
     // Check cache first — preserves stable node IDs
     const cached = getCachedDocument(fullPath);
     if (cached) {
-        setActiveDocument(cached.document, cached.title, fullPath, cached.isTemp, cached.lastModified, cached.metadata);
+        setActiveDocument(cached.document, cached.title, fullPath, cached.isTemp, cached.lastModified, cached.metadata, cached.originalFrontmatter);
         const filename = isExternalDoc(fullPath) ? fullPath : (fullPath.split(/[/\\]/).pop() || '');
         return { document: getDocument(), title: getTitle(), filename };
     }
@@ -560,8 +566,15 @@ export function openFile(fullPath) {
     const parsed = markdownToTiptap(raw);
     const mtime = new Date(statSync(fullPath).mtimeMs);
     ensureDocId(parsed.metadata);
+    // Title fallback: use filename stem instead of "Untitled" for files without a title
+    let title = parsed.title;
+    if (title === 'Untitled') {
+        const stem = fullPath.split(/[/\\]/).pop()?.replace(/\.md$/i, '');
+        if (stem)
+            title = stem;
+    }
     const baseName = fullPath.split(/[/\\]/).pop() || '';
-    setActiveDocument(parsed.document, parsed.title, fullPath, baseName.startsWith(TEMP_PREFIX), mtime, parsed.metadata);
+    setActiveDocument(parsed.document, title, fullPath, baseName.startsWith(TEMP_PREFIX), mtime, parsed.metadata, parsed.rawFrontmatter);
     // Use full path as filename for external docs, basename for getDataDir() docs
     const filename = isExternalDoc(fullPath) ? fullPath : baseName;
     return { document: getDocument(), title: getTitle(), filename };

package/dist/server/markdown-parse.js CHANGED Viewed

@@ -20,7 +20,8 @@ md.use(markdownItMark);
 md.use(markdownItSub);
 md.use(markdownItSup);
 export function markdownToTiptap(markdown) {
-    const { data, content } = matter(markdown);
+    const result = matter(markdown);
+    const { data, content } = result;
     const title = data.title || 'Untitled';
     const tokens = md.parse(content, {});
     const docContent = tokensToTiptap(tokens);
@@ -35,7 +36,7 @@ export function markdownToTiptap(markdown) {
     // Strip pending from returned metadata (consumed into node attrs)
     const metadata = { ...data };
     delete metadata.pending;
-    return { title, metadata, document: doc };
+    return { title, metadata, document: doc, rawFrontmatter: result.matter || null };
 }
 /**
  * Rehydrate pending state from frontmatter into leaf block node attrs.

package/dist/server/markdown-serialize.js CHANGED Viewed

@@ -82,6 +82,10 @@ export function tiptapToMarkdown(doc, title, metadata) {
     const body = nodesToMarkdown(doc.content || []);
     return frontmatter + body;
 }
+/** Convert TipTap document to markdown body only (no frontmatter). */
+export function tiptapToBody(doc) {
+    return nodesToMarkdown(doc.content || []);
+}
 function nodesToMarkdown(nodes) {
     let result = '';
     for (const node of nodes) {

package/dist/server/markdown.js CHANGED Viewed

@@ -2,5 +2,5 @@
  * Barrel re-export for markdown serialization and parsing.
  * All existing imports from './markdown.js' continue to work unchanged.
  */
-export { tiptapToMarkdown, nodeText, inlineToMarkdown } from './markdown-serialize.js';
+export { tiptapToMarkdown, tiptapToBody, nodeText, inlineToMarkdown } from './markdown-serialize.js';
 export { markdownToTiptap, markdownToNodes } from './markdown-parse.js';

package/dist/server/mcp.js CHANGED Viewed

@@ -9,7 +9,7 @@ import { randomUUID } from 'crypto';
 import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
 import { z } from 'zod';
-import { getDataDir, ensureDataDir, resolveDocPath } from './helpers.js';
+import { getDataDir, ensureDataDir, resolveDocPath, generateNodeId } from './helpers.js';
 import { getDocument, getWordCount, getPendingChangeCount, getTitle, getStatus, getNodesByIds, getMetadata, setMetadata, applyChanges, applyTextEdits, updateDocument, save, markAllNodesAsPending, setAgentLock, updatePendingCacheForActiveDoc, populateDocumentFile, applyChangesToFile, applyTextEditsToFile, getDocId, getFilePath, } from './state.js';
 import { listDocuments, switchDocument, createDocument, createDocumentFile, deleteDocument, openFile, getActiveFilename, updateDocumentTitle, promoteTempFile, archiveDocument, unarchiveDocument, resolveDocId } from './documents.js';
 import { broadcastDocumentSwitched, broadcastDocumentsChanged, broadcastWorkspacesChanged, broadcastTitleChanged, broadcastMetadataChanged, broadcastPendingDocsChanged, broadcastWritingStarted, broadcastWritingFinished } from './ws.js';
@@ -826,37 +826,66 @@ export const TOOL_REGISTRY = [
                 return { content: [{ type: 'text', text: 'Error: GEMINI_API_KEY environment variable is not set.' }] };
             }
             const filename = resolveDocId(docId);
-            // Generate image via Gemini
-            const { GoogleGenAI } = await import('@google/genai');
-            const ai = new GoogleGenAI({ apiKey });
-            const response = await ai.models.generateContent({
-                model: 'gemini-3.1-flash-image-preview',
-                contents: `Generate a ${aspect_ratio || '16:9'} aspect ratio image: ${prompt}`,
-                config: {
-                    responseModalities: ['IMAGE'],
-                },
-            });
-            const parts = response.candidates?.[0]?.content?.parts;
-            const imagePart = parts?.find((p) => p.inlineData);
-            if (!imagePart?.inlineData?.data) {
-                return { content: [{ type: 'text', text: 'Error: Gemini returned no image data.' }] };
-            }
-            // Save to ~/.openwriter/_images/
-            ensureDataDir();
-            const imagesDir = join(getDataDir(), '_images');
-            if (!existsSync(imagesDir))
-                mkdirSync(imagesDir, { recursive: true });
-            const imgFilename = `${randomUUID().slice(0, 8)}.png`;
-            const filePath = join(imagesDir, imgFilename);
-            writeFileSync(filePath, Buffer.from(imagePart.inlineData.data, 'base64'));
-            const src = `/_images/${imgFilename}`;
-            // Build image node and insert change
-            const imageNode = { type: 'image', attrs: { src, alt: alt || prompt } };
-            const change = { operation: 'insert', afterNodeId, content: [imageNode] };
             const targetIsNonActive = filename && filename !== getActiveFilename();
-            if (targetIsNonActive) {
-                const { lastNodeId } = applyChangesToFile(filename, [change]);
-                broadcastPendingDocsChanged();
+            // Phase 1: Insert imageLoading placeholder immediately (active doc only)
+            const loadingNodeId = generateNodeId();
+            if (!targetIsNonActive) {
+                const loadingChange = {
+                    operation: 'insert',
+                    afterNodeId,
+                    content: [{ type: 'imageLoading', attrs: { id: loadingNodeId } }],
+                };
+                applyChanges([loadingChange]);
+            }
+            try {
+                // Generate image via Gemini
+                const { GoogleGenAI } = await import('@google/genai');
+                const ai = new GoogleGenAI({ apiKey });
+                const response = await ai.models.generateContent({
+                    model: 'gemini-3.1-flash-image-preview',
+                    contents: `Generate a ${aspect_ratio || '16:9'} aspect ratio image: ${prompt}`,
+                    config: {
+                        responseModalities: ['IMAGE'],
+                    },
+                });
+                const parts = response.candidates?.[0]?.content?.parts;
+                const imagePart = parts?.find((p) => p.inlineData);
+                if (!imagePart?.inlineData?.data) {
+                    // Remove placeholder on failure
+                    if (!targetIsNonActive) {
+                        applyChanges([{ operation: 'delete', nodeId: loadingNodeId }]);
+                    }
+                    return { content: [{ type: 'text', text: 'Error: Gemini returned no image data.' }] };
+                }
+                // Save to ~/.openwriter/_images/
+                ensureDataDir();
+                const imagesDir = join(getDataDir(), '_images');
+                if (!existsSync(imagesDir))
+                    mkdirSync(imagesDir, { recursive: true });
+                const imgFilename = `${randomUUID().slice(0, 8)}.png`;
+                const filePath = join(imagesDir, imgFilename);
+                writeFileSync(filePath, Buffer.from(imagePart.inlineData.data, 'base64'));
+                const src = `/_images/${imgFilename}`;
+                // Phase 2: Replace placeholder with real image (or direct insert for non-active)
+                if (targetIsNonActive) {
+                    const imageNode = { type: 'image', attrs: { src, alt: alt || prompt } };
+                    const change = { operation: 'insert', afterNodeId, content: [imageNode] };
+                    const { lastNodeId } = applyChangesToFile(filename, [change]);
+                    broadcastPendingDocsChanged();
+                    return {
+                        content: [{
+                                type: 'text',
+                                text: JSON.stringify({ success: true, src, ...(lastNodeId ? { lastNodeId } : {}) }),
+                            }],
+                    };
+                }
+                const imageNode = { type: 'image', attrs: { src, alt: alt || prompt } };
+                const rewriteChange = {
+                    operation: 'rewrite',
+                    nodeId: loadingNodeId,
+                    content: [imageNode],
+                };
+                const { lastNodeId } = applyChanges([rewriteChange]);
                 return {
                     content: [{
                             type: 'text',
@@ -864,13 +893,16 @@ export const TOOL_REGISTRY = [
                         }],
                 };
             }
-            const { lastNodeId } = applyChanges([change]);
-            return {
-                content: [{
-                        type: 'text',
-                        text: JSON.stringify({ success: true, src, ...(lastNodeId ? { lastNodeId } : {}) }),
-                    }],
-            };
+            catch (err) {
+                // Remove placeholder on error
+                if (!targetIsNonActive) {
+                    try {
+                        applyChanges([{ operation: 'delete', nodeId: loadingNodeId }]);
+                    }
+                    catch { }
+                }
+                return { content: [{ type: 'text', text: `Error: ${err.message || 'Image generation failed.'}` }] };
+            }
         },
     },
     {

package/dist/server/state.js CHANGED Viewed

@@ -6,7 +6,7 @@
 import { existsSync, readFileSync, writeFileSync, unlinkSync, readdirSync, statSync, utimesSync } from 'fs';
 import { join } from 'path';
 import matter from 'gray-matter';
-import { tiptapToMarkdown, markdownToTiptap } from './markdown.js';
+import { tiptapToMarkdown, 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 } from './helpers.js';
 import { snapshotIfNeeded, ensureDocId } from './versions.js';
@@ -22,6 +22,7 @@ let state = {
     isTemp: true,
     lastModified: new Date(),
     docId: '',
+    originalFrontmatter: null,
 };
 const listeners = new Set();
 // ============================================================================
@@ -104,6 +105,105 @@ function extractText(nodes) {
     })
         .join('\n');
 }
+/**
+ * Compute linear text from a node's inline content array.
+ * Matches the frontend's mapTextOffsetToPos: text chars + hardBreak=1 char.
+ */
+function linearText(content) {
+    if (!content)
+        return '';
+    let out = '';
+    for (const child of content) {
+        if (child.type === 'text' && typeof child.text === 'string')
+            out += child.text;
+        else if (child.type === 'hardBreak')
+            out += '\n';
+    }
+    return out;
+}
+/**
+ * Tokenize text into words with character offsets.
+ */
+function tokenize(text) {
+    const words = [];
+    const re = /\S+/g;
+    let match;
+    while ((match = re.exec(text))) {
+        words.push({ word: match[0], start: match.index, end: match.index + match[0].length });
+    }
+    return words;
+}
+/**
+ * Compute sub-node selection range by word-level diff.
+ * Finds first and last differing words → decoration spans that range.
+ */
+function computePartialRange(origContent, newContent) {
+    const origText = linearText(origContent || []);
+    const newText = linearText(newContent || []);
+    if (!origText || !newText || origText === newText)
+        return null;
+    const origWords = tokenize(origText);
+    const newWords = tokenize(newText);
+    if (origWords.length === 0 || newWords.length === 0)
+        return null;
+    // First differing word from start
+    let firstDiff = 0;
+    while (firstDiff < origWords.length && firstDiff < newWords.length &&
+        origWords[firstDiff].word === newWords[firstDiff].word)
+        firstDiff++;
+    // All words identical (shouldn't happen since text differs, but guard)
+    if (firstDiff === origWords.length && firstDiff === newWords.length)
+        return null;
+    // Last differing word from end
+    let origEnd = origWords.length - 1;
+    let newEnd = newWords.length - 1;
+    while (origEnd >= firstDiff && newEnd >= firstDiff &&
+        origWords[origEnd].word === newWords[newEnd].word) {
+        origEnd--;
+        newEnd--;
+    }
+    // No common words at start or end → full rewrite, skip partial
+    if (firstDiff === 0 && origEnd === origWords.length - 1 && newEnd === newWords.length - 1)
+        return null;
+    // Raw character offsets from first/last changed words
+    let origFrom = firstDiff < origWords.length ? origWords[firstDiff].start : origText.length;
+    let origTo = origEnd >= firstDiff && origEnd < origWords.length ? origWords[origEnd].end : origFrom;
+    let newFrom = firstDiff < newWords.length ? newWords[firstDiff].start : newText.length;
+    let newTo = newEnd >= firstDiff && newEnd < newWords.length ? newWords[newEnd].end : newFrom;
+    // Snap start back to previous sentence boundary (after ". ")
+    const snapBack = (text, pos) => {
+        let i = pos - 1;
+        while (i > 0) {
+            if (text[i] === '.' && i + 1 < text.length && text[i + 1] === ' ')
+                return i + 2;
+            i--;
+        }
+        return 0; // No period found → start of text
+    };
+    // Snap end forward to next sentence boundary (the ". " or end of text)
+    const snapForward = (text, pos) => {
+        let i = pos;
+        while (i < text.length) {
+            if (text[i] === '.' && (i + 1 >= text.length || text[i + 1] === ' '))
+                return i + 1;
+            i++;
+        }
+        return text.length;
+    };
+    origFrom = snapBack(origText, origFrom);
+    origTo = snapForward(origText, origTo);
+    newFrom = snapBack(newText, newFrom);
+    newTo = snapForward(newText, newTo);
+    // If almost everything changed, full-node decoration
+    if ((origTo - origFrom + newTo - newFrom) >= (origText.length + newText.length) * 0.8)
+        return null;
+    return {
+        selectionFrom: newFrom,
+        selectionTo: newTo,
+        originalFrom: origFrom,
+        originalTo: origTo,
+    };
+}
 export function getWordCount() {
     const text = getPlainText();
     return text.trim() ? text.trim().split(/\s+/).length : 0;
@@ -419,6 +519,12 @@ function applyChangesToDoc(doc, changes) {
             const isEmptyNode = !originalText.trim();
             // Only store original on first rewrite (preserve baseline for reject)
             const existingOriginal = found.parent[found.index].attrs?.pendingOriginalContent;
+            // Detect partial change: if only a sub-range of the node text changed,
+            // attach selection range attrs so the frontend decorates only that part
+            let partialRange = null;
+            if (!isEmptyNode && contentArray.length === 1) {
+                partialRange = computePartialRange(originalNode.content || [], contentArray[0].content || []);
+            }
             // First node replaces the target (rewrite or insert if empty)
             const firstNode = {
                 ...contentArray[0],
@@ -427,6 +533,12 @@ function applyChangesToDoc(doc, changes) {
                     id: change.nodeId,
                     pendingStatus: isEmptyNode ? 'insert' : 'rewrite',
                     ...(isEmptyNode ? {} : { pendingOriginalContent: existingOriginal || originalNode }),
+                    ...(partialRange ? {
+                        pendingSelectionFrom: partialRange.selectionFrom,
+                        pendingSelectionTo: partialRange.selectionTo,
+                        pendingOriginalFrom: partialRange.originalFrom,
+                        pendingOriginalTo: partialRange.originalTo,
+                    } : {}),
                 },
             };
             // Additional nodes get inserted after as pending inserts
@@ -533,7 +645,7 @@ export function applyTextEdits(nodeId, edits) {
     return { success: true };
 }
 /** Set the active document state. Used by documents.ts for multi-doc operations. */
-export function setActiveDocument(doc, title, filePath, isTemp, lastModified, metadata) {
+export function setActiveDocument(doc, title, filePath, isTemp, lastModified, metadata, originalFrontmatter) {
     state.document = doc;
     state.title = title;
     state.metadata = metadata || { title };
@@ -541,6 +653,7 @@ export function setActiveDocument(doc, title, filePath, isTemp, lastModified, me
     state.isTemp = isTemp;
     state.lastModified = lastModified || new Date();
     state.docId = ensureDocId(state.metadata);
+    state.originalFrontmatter = originalFrontmatter ?? null;
 }
 // ============================================================================
 // PENDING DOCUMENT CACHE (avoids disk scans on every broadcast)
@@ -628,6 +741,7 @@ export function cacheActiveDocument() {
         lastModified: state.lastModified,
         docId: state.docId,
         fileMtime,
+        originalFrontmatter: state.originalFrontmatter,
     });
 }
 /** Get a cached document if the file hasn't been modified externally. Returns null on miss or stale. */
@@ -661,6 +775,8 @@ function updateCacheEntry(filePath, doc, title, metadata, isTemp, docId) {
         fileMtime = statSync(filePath).mtimeMs;
     }
     catch { /* best-effort */ }
+    // Preserve originalFrontmatter from existing cache entry (if any)
+    const existing = docCache.get(filePath);
     docCache.set(filePath, {
         document: structuredClone(doc),
         metadata: structuredClone(metadata),
@@ -669,6 +785,7 @@ function updateCacheEntry(filePath, doc, title, metadata, isTemp, docId) {
         lastModified: new Date(),
         docId,
         fileMtime,
+        originalFrontmatter: existing?.originalFrontmatter ?? null,
     });
 }
 /** Reset all in-memory caches. Called on profile switch. */
@@ -684,6 +801,7 @@ export function clearAllCaches() {
         isTemp: true,
         lastModified: new Date(),
         docId: '',
+        originalFrontmatter: null,
     };
 }
 // ============================================================================
@@ -772,7 +890,17 @@ export function getPendingDocInfo() {
 // ============================================================================
 function writeToDisk() {
     ensureDataDir();
-    const markdown = tiptapToMarkdown(state.document, state.title, state.metadata);
+    let markdown;
+    if (isExternalDoc(state.filePath)) {
+        // External files: preserve original frontmatter verbatim, no OpenWriter metadata injected
+        const body = tiptapToBody(state.document).replace(/(?:\s*<!-- -->\s*)+$/, '\n');
+        markdown = state.originalFrontmatter
+            ? `---\n${state.originalFrontmatter}\n---\n\n${body}`
+            : body;
+    }
+    else {
+        markdown = tiptapToMarkdown(state.document, state.title, state.metadata);
+    }
     if (existsSync(state.filePath)) {
         // Skip write if content is identical (prevents phantom git changes on doc switch)
         try {
@@ -1048,9 +1176,9 @@ export function addDocTag(filename, tag) {
         }
     }
     else {
-        // Non-active doc — read/write disk
+        // Non-active doc — read/write disk (skip external files: tags are OpenWriter metadata)
         const targetPath = resolveDocPath(filename);
-        if (!existsSync(targetPath))
+        if (isExternalDoc(targetPath) || !existsSync(targetPath))
             return;
         try {
             const raw = readFileSync(targetPath, 'utf-8');
@@ -1088,8 +1216,9 @@ export function removeDocTag(filename, tag) {
         }
     }
     else {
+        // Non-active doc — read/write disk (skip external files: tags are OpenWriter metadata)
         const targetPath = resolveDocPath(filename);
-        if (!existsSync(targetPath))
+        if (isExternalDoc(targetPath) || !existsSync(targetPath))
             return;
         try {
             const raw = readFileSync(targetPath, 'utf-8');
@@ -1127,7 +1256,16 @@ export function saveDocToFile(filename, doc) {
         if (hasPendingChanges(parsed.document)) {
             transferPendingAttrs(parsed.document, doc);
         }
-        const markdown = tiptapToMarkdown(doc, parsed.title, parsed.metadata);
+        let markdown;
+        if (isExternalDoc(targetPath)) {
+            const body = tiptapToBody(doc);
+            markdown = parsed.rawFrontmatter
+                ? `---\n${parsed.rawFrontmatter}\n---\n\n${body}`
+                : body;
+        }
+        else {
+            markdown = tiptapToMarkdown(doc, parsed.title, parsed.metadata);
+        }
         atomicWriteFileSync(targetPath, markdown);
     }
     catch { /* best-effort */ }
@@ -1161,7 +1299,16 @@ export function stripPendingAttrsFromFile(filename, clearAgentCreated) {
         if (clearAgentCreated && parsed.metadata.agentCreated) {
             delete parsed.metadata.agentCreated;
         }
-        const markdown = tiptapToMarkdown(parsed.document, parsed.title, parsed.metadata);
+        let markdown;
+        if (isExternalDoc(targetPath)) {
+            const body = tiptapToBody(parsed.document);
+            markdown = parsed.rawFrontmatter
+                ? `---\n${parsed.rawFrontmatter}\n---\n\n${body}`
+                : body;
+        }
+        else {
+            markdown = tiptapToMarkdown(parsed.document, parsed.title, parsed.metadata);
+        }
         atomicWriteFileSync(targetPath, markdown);
         removePendingCacheEntry(filename);
     }

package/package.json CHANGED Viewed

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