@bli-cockpit/cli 0.2.141 → 0.2.142

package/dist/commands/docs.js

@@ -49,6 +49,8 @@ export async function runDocs(command, io) {
             return createDoc(command, door);
         case "update":
             return updateDoc(command, door);
+        case "delete":
+            return deleteDocs(command, door);
         case "link":
             return linkDoc(command, door);
         case "sync-status":
@@ -234,6 +236,75 @@ async function updateDoc(command, door) {
     writeLine(door.io.stdout, `Updated ${String(document["title"] ?? "")} (${String(document["id"] ?? resolved.id)}).`);
     return 0;
 }
+/**
+ * BLI‑10555: soft-deletes one or more documents — `DELETE
+ * /api/docs/documents/[id]`, the same door the browser's own delete calls.
+ * Recoverable (only a super_admin clearing `deleted_at` brings it back, no
+ * self-service undo) and never cascades: a deleted document's children keep
+ * their own `parent_id` and stay exactly where they were.
+ *
+ * Mirrors `readDoc`'s multi-ref shape (BLI-10450): one shared `docs list`
+ * resolves every ref, then one `DELETE` per document — one bad ref never
+ * loses the rest of the batch.
+ */
+async function deleteDocs(command, door) {
+    const refs = command.docRefs && command.docRefs.length > 0 ? command.docRefs : [command.docRef ?? ""];
+    const resolved = await resolveDocIds(door, refs);
+    if (resolved.status === "list_failed")
+        return failAgentDoor(door, TAG, resolved.reason, resolved.detail);
+    const results = await Promise.all(refs.map((ref) => deleteOneDocument(door, ref, resolved.ids.get(ref))));
+    const failed = results.filter((result) => !result.ok);
+    writeLine(door.io.stderr, `${TAG} delete ${JSON.stringify({ requested: refs.length, deleted: results.length - failed.length, refused: failed.length })}`);
+    if (refs.length === 1) {
+        const [result] = results;
+        if (!result || !result.ok) {
+            return failAgentDoor(door, TAG, result?.reason ?? "document_not_found_or_unreadable", result?.detail ?? "Not found.");
+        }
+        if (door.json)
+            return emitAgentDoor(door, { ok: true, document_id: result.id });
+        writeLine(door.io.stdout, `Deleted ${result.id}. Recoverable by a super_admin; its children stay in place.`);
+        return 0;
+    }
+    if (door.json) {
+        return emitAgentDoor(door, {
+            ok: failed.length === 0,
+            documents: results.map((result) => result.ok
+                ? { ref: result.ref, ok: true, document_id: result.id }
+                : { ref: result.ref, ok: false, error: result.reason, detail: result.detail }),
+        }, failed.length === 0 ? 0 : 1);
+    }
+    results.forEach((result) => {
+        if (result.ok) {
+            writeLine(door.io.stdout, `Deleted ${result.ref} (${result.id}).`);
+        }
+        else {
+            writeLine(door.io.stderr, `${TAG} refused ${JSON.stringify({ ref: result.ref, reason: result.reason })}`);
+            writeLine(door.io.stdout, `${result.ref}: ${result.detail}`);
+        }
+    });
+    writeLine(door.io.stdout, "");
+    writeLine(door.io.stdout, `${results.length - failed.length}/${results.length} document(s) deleted. Recoverable by a super_admin; children stay in place.`);
+    return failed.length === 0 ? 0 : 1;
+}
+async function deleteOneDocument(door, ref, id) {
+    if (!id) {
+        return {
+            ref,
+            ok: false,
+            reason: "document_not_found_or_unreadable",
+            detail: `That document does not exist, or you cannot delete it: ${ref}`,
+        };
+    }
+    const answer = await askAgentDoor(door, {
+        path: `/api/docs/documents/${encodeURIComponent(id)}`,
+        method: "DELETE",
+        label: "docs delete",
+        timeoutMs: WRITE_DEADLINE_MS,
+    });
+    if (!answer.ok)
+        return { ref, ok: false, reason: answer.reason, detail: answer.detail };
+    return { ref, ok: true, id };
+}
 async function updateDocument(command, door, documentId, bodyMarkdown) {
     return askAgentDoor(door, {
         path: `/api/docs/documents/${encodeURIComponent(documentId)}`,

package/dist/commands/local-args-tower-docs-msg.js

@@ -7,10 +7,12 @@
  * added both without stretching either existing family's own doc comment.
  */
 import { optionalNonEmpty, optionalPositiveInteger, optionalUrl, parseNamedArgs, } from "./local-arg-values.js";
-const DOCS_ACTIONS = new Set(["list", "tree", "read", "create", "update", "link", "sync-status"]);
+const DOCS_ACTIONS = new Set(["list", "tree", "read", "create", "update", "delete", "link", "sync-status"]);
 // BLI-10450: `read` moved to its own branch in `parseDocsDocumentReference` —
 // it takes SEVERAL references now, so it no longer shares the "exactly one"
-// rule `update`/`link` still enforce.
+// rule `update`/`link` still enforce. BLI‑10555: `delete` takes several too
+// (the same door, called once per ref), so it joined `read`'s branch rather
+// than this set.
 const DOCS_ACTIONS_NEEDING_A_DOC = new Set(["update", "link"]);
 export function parseDocsArgs(args) {
     const values = parseNamedArgs(args, {
@@ -52,6 +54,11 @@ export function parseDocsArgs(args) {
     const { visibility, clearParent, parentId, allowEmpty, title, query, wholeWord, limit, bodies, section, offset } = parseDocsFlags(action, values);
     if (action === "read")
         validateDocsReadTargets(docRefs, parentId, bodies);
+    // BLI‑10555: `delete` needs no `--parent`/`--bodies` shape, only at least
+    // one reference — `read`'s emptier validation does not apply.
+    if (action === "delete" && (!docRefs || docRefs.length === 0)) {
+        throw new Error("docs delete needs one or more document ids/slugs.");
+    }
     return buildDocsCommand(values, action, docRef, docRefs, visibility, clearParent, parentId, allowEmpty, title, query, wholeWord, limit, bodies, section, offset, link);
 }
 /**
@@ -104,7 +111,7 @@ function parseDocsAction(positionals) {
     const first = positionals[0];
     const action = (first === undefined ? "list" : first);
     if (!DOCS_ACTIONS.has(action)) {
-        throw new Error(`Unknown docs command: ${first}. Try list, tree, read, create, update, link, or sync-status.`);
+        throw new Error(`Unknown docs command: ${first}. Try list, tree, read, create, update, delete, link, or sync-status.`);
     }
     return { action, rest: positionals.slice(first === undefined ? 0 : 1) };
 }
@@ -112,8 +119,10 @@ function parseDocsDocumentReference(action, rest) {
     // BLI-10450: `read` alone takes several references in one call — an agent
     // reading a folder's worth of documents used to pay a resolve-then-read
     // pair PER document. Emptiness is validated later, once `--parent`/
-    // `--bodies` are known (`validateDocsReadTargets`).
-    if (action === "read") {
+    // `--bodies` are known (`validateDocsReadTargets`). BLI‑10555: `delete`
+    // shares this shape — several refs, one shared resolve, one DELETE each —
+    // its own emptiness check runs right after this function returns.
+    if (action === "read" || action === "delete") {
         const refs = rest.map((token) => optionalNonEmpty(token)).filter((token) => Boolean(token));
         return { docRef: refs[0], docRefs: refs.length > 0 ? refs : undefined };
     }

package/dist/commands/local-help-commands-tower.js

@@ -42,7 +42,7 @@ export const TOWER_COMMAND_HELP = [
     [
         "docs",
         [
-            "Usage: cockpit docs [list|tree|read <id|slug> [<id|slug>...]|read --parent <id> --bodies|create|update <id>|link <id|slug> <notion-page>|sync-status [<id|slug>]] [flags]",
+            "Usage: cockpit docs [list|tree|read <id|slug> [<id|slug>...]|read --parent <id> --bodies|create|update <id>|delete <id|slug> [<id|slug>...]|link <id|slug> <notion-page>|sync-status [<id|slug>]] [flags]",
             "",
             "The Tower document library, typed. Bare `cockpit docs` lists every document you may read.",
             "list [--parent <id>|root] [--query <text> [--word]] [--limit <n>] — every document you may see: id, visibility, body size, slug, title. Metadata only — a list never carries a body, so it stays cheap enough for an agent to call (--json was 604 KB for 155 documents before BLI-3737, and is now a few tens of KB). --json shape: { documents: [{ id, slug, title, visibility, parent_id, source, updated_at, body_chars }] }.",
@@ -59,6 +59,7 @@ export const TOWER_COMMAND_HELP = [
             "create --title \"<title>\" [--parent <id>] [--visibility org|private] [--file <path>|--body-stdin] — the body comes from --file, or stdin (`cat body.md | cockpit docs create --title \"...\"`); neither means an empty body, matching the browser's own default. --json shape: { document }.",
             "update <id|slug> [--title \"<t>\"] [--visibility org|private] [--parent <id>|--clear-parent] [--file <path>|--body-stdin] [--allow-empty] — only the fields you pass change; a `--parent` change IS a move, there is no separate move verb. --json shape: { document }.",
             "  --allow-empty clears the page on purpose. A body that would empty a document that holds text is refused (refused_empty_body) unless you say so, because far more often it is a surface that lost the content than a person who meant it.",
+            "delete <id|slug> [<id|slug>...] — soft-deletes one or more documents (BLI-10555): the browser's own delete, sets deleted_at, never a real SQL DELETE. Recoverable — only a super_admin clearing deleted_at brings it back, there is no self-service undo. Deleting a parent does NOT delete its children; they keep their own parent_id and stay exactly where they were. Several refs resolve against ONE shared list call, then one DELETE each — one bad ref never loses the rest. --json shape: one ref answers { ok, document_id }; several answer { documents: [{ ref, ok, document_id } | { ref, ok: false, error, detail }] }.",
             "link <id|slug> <notion-url|page-id> — ties the document to a Notion page so the two-way sync moves it both ways. The page is the truth from then on: the next sync pulls it over this copy and keeps the copy as a version. `link <id|slug> --clear` unties it. Refusals: invalid_notion_page, notion_page_already_linked, document_not_writable.",
             "sync-status [<id|slug>] — every linked document's Notion sync state, or one's: synced (the body is what both sides last agreed on), tower_changed (edited here since; the next write-back pushes it), never_synced (linked, no baseline yet). Tower's half only: a Notion edit shows up at the next pull, not here. Metadata only, never a body.",
             "A document body is never accepted on the command line — --file (safest on Windows) or a pipe only, same discipline as `cockpit notes paste`.",

package/dist/commands/local-help.js

@@ -117,7 +117,7 @@ export function localCommandHelp(command) {
         "  cockpit agent-rules [install|uninstall|status] [--host codex|claude|all] [--workspace <path>] [--json]",
         "  cockpit memory [search \"<question>\"|save|update <id>|forget <id>|profile|install|status|log <win|loss|noise> --store <bli|supermemory|both> \"<reason>\"] [--container <tag>] [--limit <n>] [--no-recent] [--custom-id <id>] [--extract] [--content-stdin] [--reason <label>] [--file <path>] [--dashboard-url <url>] [--dry-run] [--json]",
         "  cockpit clean [--dry-run] [--all-committed] [--reconcile] [--dashboard-url <url>] [--json]",
-        "  cockpit docs [list|tree|read <id|slug>|create --title <t>|update <id>] [--parent <id>|root|--clear-parent] [--query <text> [--word]] [--limit <n>] [--visibility org|private] [--file <path>|--body-stdin] [--allow-empty] [--dashboard-url <url>] [--json]",
+        "  cockpit docs [list|tree|read <id|slug>|create --title <t>|update <id>|delete <id|slug>] [--parent <id>|root|--clear-parent] [--query <text> [--word]] [--limit <n>] [--visibility org|private] [--file <path>|--body-stdin] [--allow-empty] [--dashboard-url <url>] [--json]",
         "  cockpit msg [channels|create <name>|dm <email>|read <channel>|send <channel>|thread <id> --channel <channel>] [--private] [--members a@x,b@y] [--description <text>] [--thread <id>] [--limit <n>] [--dashboard-url <url>] [--json]",
         "  cockpit issue [list|show <BLI-id>|create --title <t>|update <BLI-id>|move <BLI-id> <state>|comment <BLI-id>|history <BLI-id>] [--state <s>] [--assignee me|unassigned|<uuid>] [--project <name|id>] [--limit <n>] [--priority 0-4] [--parent <BLI-id>] [--file <path>|--body-stdin] [--dashboard-url <url>] [--json]",
         "  cockpit mail [accounts|add-imap --address <a>|inbox|read <thread>|search \"<words>\"|send --account <id> --to <a> --subject <s>|attachment <id> --out <path>|sync <account>|detach <account>] [--account <id>] [--limit <n>] [--unread] [--label <l>] [--file <path>] [--dashboard-url <url>] [--json]",

package/dist/commands/public-root.js

@@ -21,7 +21,7 @@ export async function runCockpitCli(argv, io) {
   }
 
   if (command === "--version" || command === "-V" || command === "version") {
-    writeLine(io?.stdout ?? process.stdout, "0.2.141");
+    writeLine(io?.stdout ?? process.stdout, "0.2.142");
     return 0;
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bli-cockpit/cli",
-  "version": "0.2.141",
+  "version": "0.2.142",
   "private": false,
   "type": "module",
   "bin": {
@@ -28,7 +28,7 @@
   },
   "dependencies": {
     "@bli-cockpit/memory-mcp": "0.1.36",
-    "@bli-cockpit/mcp": "0.1.66",
+    "@bli-cockpit/mcp": "0.1.67",
     "@bli-cockpit/telemetry-core": "0.1.48"
   }
 }