drafted 1.19.11 → 1.19.16

package/mcp/server.mjs

@@ -198,6 +198,17 @@ function scrubLocalPathMentions(description) {
 // inside the factory so each HTTP request gets its own isolated server.
 // Stdio mode uses the `mcpServer` singleton (built once at module load).
 
+// What an fs(rm) on a project path actually targets. A layer or lane path carries
+// no filename, so before this predicate existed it fell through to the "no file
+// path = archive the project" branch and archived the WHOLE project — returning a
+// normal-looking success. That is how "Ai For Coaches Talk" was lost to an
+// fs(rm) on /slides/deck. Only the bare project path may archive.
+export function rmScope(layer, lane, filename) {
+  if (filename) return 'file';
+  if (layer || lane) return 'directory';
+  return 'project';
+}
+
 // ── Org-ambiguity policy (the one decision core) ─────────────────
 // The org guard inside the factory plumbs session/HTTP state into this
 // side-effect-free predicate, which IS the policy (DRAFT-36 "one rule"). Top-level
@@ -3615,7 +3626,19 @@ tool('fs', 'Navigate Drafted like a local filesystem. An org is the top folder:
           // __archived folder. Frame/lane rm still deletes those items, but a
           // whole project is never hard-deleted by an agent — the web UI's
           // Archive bin is where a human permanently deletes.
-          if (!filePath) {
+          // A layer or lane path has no filename, so it used to fall through to
+          // the archive branch and archive the WHOLE project — with a
+          // success-shaped response. /slides/deck and /slides are one keystroke
+          // from the project path; only the bare project path may archive.
+          const scope = rmScope(layer, lane, filename);
+          if (scope === 'directory') {
+            return err(new Error(
+              `rm on a ${lane ? 'lane' : 'layer'} path deletes nothing — Drafted has no directory delete. ` +
+              `Remove the frames individually: /projects/<project>/${layer}${lane ? '/' + lane : ''}/<file>. ` +
+              `To archive the whole project, address the project itself: /projects/<project>`
+            ));
+          }
+          if (scope === 'project') {
             const pid = getState().projectId;
             if (!pid) return err(new Error('could not resolve project id for archive'));
             const result = await api('PATCH', `/api/project/${pid}`, { folder: '__archived' }, orgHeader);

package/mcp/test-org-guards.mjs

@@ -5,7 +5,7 @@ import assert from 'node:assert/strict';
 import { mkdtempSync } from 'node:fs';
 import { join } from 'node:path';
 import { tmpdir } from 'node:os';
-import { projectlessMutationNeedsOrg, boundOrgRejected, receiptOrg, splitOrgScope, stripUrlOrigin } from './server.mjs';
+import { projectlessMutationNeedsOrg, boundOrgRejected, receiptOrg, splitOrgScope, stripUrlOrigin, rmScope } from './server.mjs';
 import { loadPersistedProject, savePersistedProject } from './active-project-store.mjs';
 
 // One rule governs create AND fork (a fork is a create). A write proceeds when its
@@ -168,6 +168,16 @@ assert.equal(stripUrlOrigin('/o/acme/wiki/x'), '/o/acme/wiki/x', 'plain paths pa
 assert.equal(stripUrlOrigin('not a url'), 'not a url', 'non-URL input passes through');
 assert.equal(stripUrlOrigin('/f/00000000-0000-0000-0000-000000000000'), '/f/00000000-0000-0000-0000-000000000000', '/f/ frame links pass through');
 
+// fs(rm) scope: only a BARE project path may archive a project. A lane path
+// (/slides/deck) and a layer path (/slides) both parse with filename=null and
+// used to fall through to the archive branch, silently archiving the whole
+// project with a success-shaped response.
+assert.equal(rmScope('slides', 'deck', '12.html'), 'file', 'a full frame path removes that frame');
+assert.equal(rmScope('slides', null, '12.html'), 'file', 'a layer-root file path removes that frame');
+assert.equal(rmScope('slides', 'deck', null), 'directory', 'a LANE path must never archive the project');
+assert.equal(rmScope('slides', null, null), 'directory', 'a LAYER path must never archive the project');
+assert.equal(rmScope(null, null, null), 'project', 'only the bare project path archives the project');
+
 console.log('org-guard policy OK');
 // Importing server.mjs builds the stdio MCP singleton, which opens a WS reconnect
 // loop that keeps the event loop alive. Assertions are done — exit deterministically.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "drafted",
-  "version": "1.19.11",
+  "version": "1.19.16",
   "description": "Drafted — visual thinking surface for humans and AI agents. Renders HTML, markdown, images, and code as frames on a zoomable canvas, with MCP tools for AI agents and real-time sync for humans.",
   "type": "module",
   "files": [