npm - openwriter - Versions diffs - 0.9.1 → 0.9.3 - Mend

openwriter 0.9.1 → 0.9.3

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-BZqt_dqW.css → index-ZQ1BICWp.css} +1 -1
package/dist/client/assets/index-wRyjoTwK.js +211 -0
package/dist/client/index.html +2 -2
package/dist/plugins/x-api/dist/index.js +23 -2
package/dist/server/image-upload.js +35 -0
package/dist/server/index.js +2 -2
package/dist/server/marks.js +9 -2
package/dist/server/state.js +2 -1
package/dist/server/workspace-routes.js +29 -1
package/dist/server/workspaces.js +45 -0
package/package.json +1 -1
package/skill/SKILL.md +21 -12
package/dist/client/assets/index-BYx7Ckkf.js +0 -210

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-BYx7Ckkf.js"></script>
-    <link rel="stylesheet" crossorigin href="/assets/index-BZqt_dqW.css">
+    <script type="module" crossorigin src="/assets/index-wRyjoTwK.js"></script>
+    <link rel="stylesheet" crossorigin href="/assets/index-ZQ1BICWp.css">
   </head>
   <body>
     <div id="root"></div>

package/dist/plugins/x-api/dist/index.js CHANGED Viewed

@@ -6,6 +6,7 @@
 import { Client, OAuth1 } from '@xdevplatform/xdk';
 import { join, extname } from 'path';
 import { readFileSync, existsSync } from 'fs';
+import sharp from 'sharp';
 import twitter from 'twitter-text';
 const { parseTweet } = twitter;
 function createXClient(config) {
@@ -196,9 +197,19 @@ const plugin = {
                     res.status(400).json({ success: false, error: 'X API credentials not configured' });
                     return;
                 }
-                const mediaBase64 = readFileSync(filePath).toString('base64');
+                let fileBuffer = readFileSync(filePath);
+                let uploadType = mediaType;
+                const origSize = fileBuffer.length;
+                // Compress large images or PNGs to JPEG to stay under X API limits
+                if (fileBuffer.length > 3 * 1024 * 1024 || ext === '.png') {
+                    fileBuffer = Buffer.from(await sharp(fileBuffer).jpeg({ quality: 85 }).toBuffer());
+                    uploadType = 'image/jpeg';
+                    console.log(`[X Plugin] Compressed ${filename}: ${(origSize / 1024 / 1024).toFixed(2)}MB → ${(fileBuffer.length / 1024 / 1024).toFixed(2)}MB`);
+                }
+                console.log(`[X Plugin] Uploading ${filename}: ${(fileBuffer.length / 1024 / 1024).toFixed(2)}MB, type: ${uploadType}`);
+                const mediaBase64 = fileBuffer.toString('base64');
                 const uploadResult = await client.media.upload({
-                    body: { media: mediaBase64, mediaCategory: 'tweet_image', mediaType },
+                    body: { media: mediaBase64, mediaCategory: 'tweet_image', mediaType: uploadType },
                 });
                 const mediaId = uploadResult?.data?.id
                     || uploadResult?.media_id_string;
@@ -211,6 +222,16 @@ const plugin = {
             }
             catch (err) {
                 console.error('[X Plugin] Media upload failed:', err.message);
+                if (err.response) {
+                    try {
+                        const body = await err.response.text();
+                        console.error('[X Plugin] X API response:', err.response.status, body);
+                    }
+                    catch { /* ignore */ }
+                }
+                if (err.data)
+                    console.error('[X Plugin] Error data:', JSON.stringify(err.data));
+                console.error('[X Plugin] Full error:', JSON.stringify(err, Object.getOwnPropertyNames(err)));
                 res.status(500).json({ success: false, error: err.message });
             }
         });

package/dist/server/image-upload.js CHANGED Viewed

@@ -54,5 +54,40 @@ export function createImageRouter() {
         const src = `/_images/${req.file.filename}`;
         res.json({ src });
     });
+    // Download external URL and save locally
+    router.post('/api/download-image', async (req, res) => {
+        const { url } = req.body;
+        if (!url || typeof url !== 'string') {
+            res.status(400).json({ error: 'No URL provided' });
+            return;
+        }
+        try {
+            const response = await fetch(url);
+            if (!response.ok) {
+                res.status(400).json({ error: 'Failed to fetch image' });
+                return;
+            }
+            const contentType = response.headers.get('content-type') || 'image/png';
+            if (!contentType.startsWith('image/')) {
+                res.status(400).json({ error: 'URL is not an image' });
+                return;
+            }
+            const ext = contentType.includes('jpeg') || contentType.includes('jpg') ? '.jpg'
+                : contentType.includes('gif') ? '.gif'
+                    : contentType.includes('webp') ? '.webp'
+                        : '.png';
+            ensureImagesDir();
+            const filename = `${randomUUID().slice(0, 8)}${ext}`;
+            const filePath = join(getImagesDir(), filename);
+            const buffer = Buffer.from(await response.arrayBuffer());
+            const { writeFileSync } = await import('fs');
+            writeFileSync(filePath, buffer);
+            const src = `/_images/${filename}`;
+            res.json({ src });
+        }
+        catch {
+            res.status(500).json({ error: 'Download failed' });
+        }
+    });
     return router;
 }

package/dist/server/index.js CHANGED Viewed

@@ -460,12 +460,12 @@ export async function startHttpServer(options = {}) {
     // Agent marks
     app.post('/api/marks', (req, res) => {
         try {
-            const { filename, text, note, nodeId } = req.body;
+            const { filename, text, note, nodeId, nodeIds } = req.body;
             if (!filename || !text || !nodeId) {
                 res.status(400).json({ error: 'filename, text, and nodeId are required' });
                 return;
             }
-            const mark = addMark(filename, text, note || '', nodeId);
+            const mark = addMark(filename, text, note || '', nodeId, nodeIds);
             broadcastMarksChanged(filename);
             res.json({ success: true, mark });
         }

package/dist/server/marks.js CHANGED Viewed

@@ -39,13 +39,14 @@ function writeMarkFile(filename, data) {
     }
     writeFileSync(path, JSON.stringify(data, null, 2));
 }
-export function addMark(filename, text, note, nodeId) {
+export function addMark(filename, text, note, nodeId, nodeIds) {
     const data = readMarkFile(filename);
     const mark = {
         id: randomUUID().slice(0, 8),
         text,
         note,
         nodeId,
+        ...(nodeIds && nodeIds.length > 1 ? { nodeIds } : {}),
         createdAt: new Date().toISOString(),
     };
     data.marks.push(mark);
@@ -150,7 +151,13 @@ export function pruneStaleMarks(filename, validNodeIds) {
         return 0;
     const validSet = new Set(validNodeIds);
     const before = data.marks.length;
-    data.marks = data.marks.filter((m) => validSet.has(m.nodeId));
+    data.marks = data.marks.filter((m) => {
+        // Multi-node mark: keep if ANY nodeId is still valid
+        if (m.nodeIds && m.nodeIds.length > 0) {
+            return m.nodeIds.some((id) => validSet.has(id));
+        }
+        return validSet.has(m.nodeId);
+    });
     const pruned = before - data.marks.length;
     if (pruned > 0)
         writeMarkFile(filename, data);

package/dist/server/state.js CHANGED Viewed

@@ -680,7 +680,8 @@ function applyChangesToDoc(doc, changes) {
             // Tweet thread: hard-delete paragraphs + adjacent HR immediately.
             // Tweet compose view can't handle pending deletes near HRs — hard-delete and resync.
             const delNode = found.parent[found.index];
-            if (delNode.type === 'paragraph' && state.metadata?.tweetContext) {
+            const hardDeleteTypes = ['paragraph', 'image', 'imageLoading'];
+            if (hardDeleteTypes.includes(delNode.type) && state.metadata?.tweetContext) {
                 const idx = found.index;
                 if (idx > 0 && found.parent[idx - 1].type === 'horizontalRule') {
                     found.parent.splice(idx, 1);

package/dist/server/workspace-routes.js CHANGED Viewed

@@ -3,7 +3,7 @@
  * Mounted in index.ts to keep the main file lean.
  */
 import { Router } from 'express';
-import { listWorkspaces, getWorkspace, createWorkspace, deleteWorkspace, reorderWorkspaces, addDoc, removeDoc, moveDoc, reorderDoc, addContainerToWorkspace, removeContainer, renameContainer, renameWorkspace, reorderContainer, } from './workspaces.js';
+import { listWorkspaces, getWorkspace, createWorkspace, deleteWorkspace, reorderWorkspaces, addDoc, removeDoc, moveDoc, reorderDoc, addContainerToWorkspace, removeContainer, renameContainer, renameWorkspace, reorderContainer, crossMoveContainer, promoteContainerToWorkspace, } from './workspaces.js';
 export function createWorkspaceRouter(b) {
     const router = Router();
     router.get('/api/workspaces', (_req, res) => {
@@ -36,6 +36,20 @@ export function createWorkspaceRouter(b) {
             res.status(400).json({ error: err.message });
         }
     });
+    // Promote container to standalone workspace
+    router.post('/api/workspaces/promote-container', (req, res) => {
+        try {
+            const { sourceWorkspace, containerId, afterWorkspaceFilename } = req.body;
+            if (!sourceWorkspace || !containerId)
+                return res.status(400).json({ error: 'sourceWorkspace and containerId required' });
+            const result = promoteContainerToWorkspace(sourceWorkspace, containerId, afterWorkspaceFilename ?? null);
+            b.broadcastWorkspacesChanged();
+            res.json(result);
+        }
+        catch (err) {
+            res.status(400).json({ error: err.message });
+        }
+    });
     router.get('/api/workspaces/:filename', (req, res) => {
         try {
             res.json(getWorkspace(req.params.filename));
@@ -149,6 +163,20 @@ export function createWorkspaceRouter(b) {
             res.status(400).json({ error: err.message });
         }
     });
+    // Cross-workspace container move
+    router.post('/api/workspaces/:targetFilename/containers/:containerId/cross-move', (req, res) => {
+        try {
+            const { sourceWorkspace } = req.body;
+            if (!sourceWorkspace)
+                return res.status(400).json({ error: 'sourceWorkspace required' });
+            const ws = crossMoveContainer(sourceWorkspace, req.params.targetFilename, req.params.containerId, req.body.targetContainerId ?? null, req.body.afterIdentifier ?? null);
+            b.broadcastWorkspacesChanged();
+            res.json(ws);
+        }
+        catch (err) {
+            res.status(400).json({ error: err.message });
+        }
+    });
     // Cross-workspace move (from one workspace to another)
     router.post('/api/workspaces/:targetFilename/docs/:docFile/cross-move', (req, res) => {
         try {

package/dist/server/workspaces.js CHANGED Viewed

@@ -252,6 +252,51 @@ export function moveContainer(wsFile, containerId, targetContainerId, afterIdent
     writeWorkspace(wsFile, ws);
     return ws;
 }
+export function crossMoveContainer(sourceWsFile, targetWsFile, containerId, targetContainerId, afterIdentifier) {
+    const sourceWs = getWorkspace(sourceWsFile);
+    const removed = removeNode(sourceWs.root, containerId);
+    if (removed.type !== 'container')
+        throw new Error(`Node "${containerId}" is not a container`);
+    writeWorkspace(sourceWsFile, sourceWs);
+    const targetWs = getWorkspace(targetWsFile);
+    // Insert into target — reuse moveNode-style logic
+    const target = targetContainerId === null
+        ? targetWs.root
+        : (() => { const f = findContainer(targetWs.root, targetContainerId); if (!f)
+            throw new Error('Target container not found'); return f.node.items; })();
+    if (afterIdentifier === null) {
+        target.unshift(removed);
+    }
+    else {
+        const afterIdx = target.findIndex((n) => (n.type === 'doc' && n.file === afterIdentifier) || (n.type === 'container' && n.id === afterIdentifier));
+        if (afterIdx === -1)
+            target.push(removed);
+        else
+            target.splice(afterIdx + 1, 0, removed);
+    }
+    writeWorkspace(targetWsFile, targetWs);
+    return targetWs;
+}
+export function promoteContainerToWorkspace(sourceWsFile, containerId, afterWorkspaceFilename) {
+    const sourceWs = getWorkspace(sourceWsFile);
+    const removed = removeNode(sourceWs.root, containerId);
+    if (removed.type !== 'container')
+        throw new Error(`Node "${containerId}" is not a container`);
+    writeWorkspace(sourceWsFile, sourceWs);
+    // Create new workspace with container's children as root
+    const slug = sanitizeFilename(removed.name).toLowerCase().replace(/\s+/g, '-');
+    const filename = `${slug}-${randomUUID().slice(0, 8)}.json`;
+    const workspace = { version: 2, title: removed.name, root: removed.items };
+    writeWorkspace(filename, workspace);
+    // Append to order then reposition
+    const order = readOrder();
+    order.push(filename);
+    writeOrder(order);
+    if (afterWorkspaceFilename !== null || order.length > 1) {
+        reorderWorkspaceAfter(filename, afterWorkspaceFilename);
+    }
+    return { filename, title: removed.name, docCount: countDocs(removed.items) };
+}
 export function reorderWorkspaceAfter(filename, afterFilename) {
     ensureWorkspacesDir();
     const order = readOrder();

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "openwriter",
-  "version": "0.9.1",
+  "version": "0.9.3",
   "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.4.2"
+  version: "0.4.5"
   repository: https://github.com/travsteward/openwriter
 license: MIT
 ---
@@ -402,29 +402,38 @@ Threads are single documents with `horizontalRule` nodes separating each tweet.
 **Do NOT use `populate_document` for threads.** Use `create_document` with `content_type: "tweet"` + `empty: true`, then `write_to_pad` with `horizontalRule` JSON nodes between tweets. The `content_type` flag sets `tweetContext` metadata automatically.
-**Critical: `horizontalRule` separators MUST use TipTap JSON `{ type: "horizontalRule" }`.** Markdown `---` does NOT create proper HR nodes — the thread will render as a single tweet. Tweet text content can be plain markdown strings.
+**THREE RULES for thread HRs:**
+1. **`horizontalRule` separators MUST use TipTap JSON `{ type: "horizontalRule" }`.** Markdown `---` does NOT create proper HR nodes.
+2. **Each HR must be its own change.** Do NOT use content arrays `[{type: "horizontalRule"}, {type: "paragraph", ...}]` — this silently drops the HR.
+3. **Send the ENTIRE thread in ONE `write_to_pad` call.** Do NOT split across multiple calls. Multiple calls create race conditions — if the user accepts changes between calls, pending HRs can be dropped. One call = atomic = no race conditions.
 ```
 1. create_document({ title: "Thread title", content_type: "tweet", empty: true })
 2. write_to_pad({ docId: "<docId>", changes: [
-     { operation: "insert", afterNodeId: "end", content: "Tweet 1 text" },
+     { operation: "insert", afterNodeId: "end", content: "Tweet 1 paragraph 1" },
+     { operation: "insert", afterNodeId: "end", content: "Tweet 1 paragraph 2" },
      { operation: "insert", afterNodeId: "end", content: { type: "horizontalRule" } },
-     { operation: "insert", afterNodeId: "end", content: "Tweet 2 text" },
+     { operation: "insert", afterNodeId: "end", content: "Tweet 2 paragraph 1" },
+     { operation: "insert", afterNodeId: "end", content: "Tweet 2 paragraph 2" },
      { operation: "insert", afterNodeId: "end", content: { type: "horizontalRule" } },
-     { operation: "insert", afterNodeId: "end", content: "Tweet 3 text" }
+     { operation: "insert", afterNodeId: "end", content: "Tweet 3 paragraph 1" }
    ]})
 ```
+**For long threads (many tweets):** still send in ONE call. The changes array can hold dozens of items. Atomicity matters more than streaming feel for threads — a half-built thread with missing HRs is worse than waiting for the full thread to arrive.
 ### Inserting New Tweets into Existing Threads
-**Insert HR + paragraph as ONE change with a content array.** Two separate calls will fail — the browser resyncs on HR insertion and overwrites the second call.
+**Mid-thread insertion is unreliable.** `afterNodeId: "end"` always means document end, not after your last insert. Inserting after specific node IDs mid-document has edge cases with pending changes and image nodes.
-```
-write_to_pad({ docId: "...", changes: [
-  { operation: "insert", afterNodeId: "<last-node-of-previous-tweet>",
-    content: [{ type: "horizontalRule" }, { type: "paragraph", content: [{ type: "text", text: "New tweet" }] }] }
-]})
-```
+**Preferred approach: rebuild the full thread.** Delete the document and recreate with all tweets in one atomic `write_to_pad` call. This is the only pattern that reliably produces correct thread structure.
+**If you must insert mid-thread:** use a single `write_to_pad` call with the HR and all content targeting the same `afterNodeId` (the last node of the preceding tweet). Content inserts in reverse order when sharing an afterNodeId, so list changes in reverse. This is fragile — prefer full rebuild.
+**Do NOT delete empty paragraphs after images.** Images create empty `<p>` nodes after them. These look like junk but HRs (thread separators) are dependent on them. Deleting the empty paragraph kills the HR too, merging two tweets into one. Leave them alone.
+**NEVER bulk-delete text nodes in a thread that contains images.** Image nodes survive text deletion and become orphans — stranded in the wrong position with no surrounding content. The user must then manually delete every orphan image from the browser. This is catastrophic. If you need to reorder tweets, move text around the existing images, or delete the entire document and start fresh (which properly removes everything including images).
 ### Paragraph Spacing in Tweets