drafted 1.5.0 → 1.6.0

package/mcp/server.mjs

@@ -1545,8 +1545,8 @@ tool('batch', 'Batch operations on the ACTIVE PROJECT. Response includes "projec
 
 // ── Asset tools ──────────────────────────────────────────────────
 
-tool('asset', 'Manage supporting files (CSS, JS, images, fonts) in the ACTIVE PROJECT. Assets are referenced by frames via relative paths — e.g., if your HTML has <link href="css/styles.css">, upload with asset_path="css/styles.css". Assets are NOT frames — they don\'t appear on the canvas. `action=upload` to add/replace, `action=list` to browse.', {
-  action: z.enum(['upload', 'list']).describe('Operation to perform.'),
+tool('asset', 'Manage supporting files (CSS, JS, images, fonts) in the ACTIVE PROJECT. Assets are referenced by frames via relative paths — e.g., if your HTML has <link href="css/styles.css">, upload with asset_path="css/styles.css". Assets are NOT frames — they don\'t appear on the canvas. `action=upload` to add/replace, `action=list` to browse, `action=rm` to delete.', {
+  action: z.enum(['upload', 'list', 'rm']).describe('Operation to perform.'),
   asset_path: z.string().optional().describe('[upload] relative asset path (e.g. "css/styles.css"). Must match the path used in HTML references.'),
   file_path: z.string().optional().describe('[upload] absolute path to a local file. Mutually exclusive with content/base64.'),
   content: z.string().optional().describe('[upload] text content (for CSS/JS). Mutually exclusive with file_path/base64.'),
@@ -1589,6 +1589,14 @@ tool('asset', 'Manage supporting files (CSS, JS, images, fonts) in the ACTIVE PR
       const query = frame_id ? `?frameId=${frame_id}` : '';
       return ok(await api('GET', `/api/projects/${projectId}/assets${query}`));
     }
+    if (action === 'rm') {
+      const skillErr = await checkProjectSkills(projectId);
+      if (skillErr) return err(new Error(skillErr));
+      const { asset_path } = args;
+      if (!asset_path) throw new Error('asset_path is required for action=rm');
+      if (asset_path.includes('..')) throw new Error('asset_path must not contain ".."');
+      return ok(await api('DELETE', `/api/projects/${projectId}/assets/${asset_path}`));
+    }
     throw new Error(`Unknown asset action: ${action}`);
   } catch (error) { return err(error); }
 });
@@ -1960,20 +1968,18 @@ tool('wiki', 'Per-org wiki. Markdown pages with paths as hierarchy. You and othe
       }
 
       // ── read ────────────────────────────────────────────────────
+      // Returns content in hashline format (`lineNum:hash|content`) so the
+      // agent can produce hashline edit operations. Mirrors frame.read.
       case 'read': {
         const { path: readPath, lines: readLines } = args;
         if (!readPath) throw new Error('path required for action=read');
         const normalized = normalizeWikiPath(readPath);
-        const page = await api('GET', `/api/wiki/page?path=${encodeURIComponent(normalized)}`);
-        // Client-side line slice — server returns full content; we trim if `lines` was passed.
-        let outContent = page.content;
-        if (readLines && typeof outContent === 'string') {
-          const m = readLines.match(/^(\d+)-(\d+)$/);
-          if (!m) throw new Error(`lines must be "N-M" (e.g. "10-50"), got: ${readLines}`);
-          const [start, end] = [parseInt(m[1]), parseInt(m[2])];
-          const allLines = outContent.split('\n');
-          outContent = allLines.slice(Math.max(0, start - 1), end).join('\n');
+        const params = new URLSearchParams({ path: normalized, format: 'hashline' });
+        if (readLines) {
+          if (!/^\d+-\d+$/.test(readLines)) throw new Error(`lines must be "N-M" (e.g. "10-50"), got: ${readLines}`);
+          params.set('lines', readLines);
         }
+        const page = await api('GET', `/api/wiki/page?${params.toString()}`);
         // Get backlink count via search (approximate)
         let backlinkCount = 0;
         try {
@@ -1985,7 +1991,8 @@ tool('wiki', 'Per-org wiki. Markdown pages with paths as hierarchy. You and othe
           title: page.title,
           type: page.type,
           frontmatter: page.frontmatter,
-          content: outContent,
+          content: page.content,
+          totalLines: page.totalLines,
           lastEditedBy: page.updatedBy,
           lastEditedAt: page.updatedAt,
           backlinkCount,
@@ -2069,15 +2076,17 @@ tool('wiki', 'Per-org wiki. Markdown pages with paths as hierarchy. You and othe
       }
 
       // ── edit ────────────────────────────────────────────────────
+      // Hashlines come from `read` (which now formats content as
+      // `lineNum:hash|content`). The server applies the ops via the same
+      // hashline algorithm — no client-side re-hashing, no algorithm drift.
       case 'edit': {
         const { path: editPath, operations: editOps } = args;
         if (!editPath) throw new Error('path required for action=edit');
         if (!Array.isArray(editOps) || editOps.length === 0) throw new Error('operations (array) required for action=edit');
         const normalized = normalizeWikiPath(editPath);
         const page = await api('GET', `/api/wiki/page?path=${encodeURIComponent(normalized)}`);
-        const newContent = applyHashlineOps(page.content || '', editOps);
-        const result = await api('PATCH', `/api/wiki/pages/${page.id}`, { content: newContent });
-        return ok(withOrg({ path: result.path, id: result.id, updated: true }));
+        const result = await api('POST', `/api/wiki/pages/${page.id}/edit`, { operations: editOps });
+        return ok(withOrg({ path: result.path, id: result.id, updated: true, applied: result.applied }));
       }
 
       // ── mv ──────────────────────────────────────────────────────

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "drafted",
-  "version": "1.5.0",
+  "version": "1.6.0",
   "description": "Drafted CLI - Design preview server for Claude agents",
   "type": "module",
   "files": [