@bli-cockpit/cli 0.2.236 → 0.2.238

package/dist/commands/local-args-tower-hits.js

@@ -37,27 +37,39 @@ export function parseHitsArgs(args) {
     }
     if (given("--weeks") && action !== "timeline")
         throw new Error("--weeks belongs to hits timeline.");
-    if (action === "timeline") {
-        if (given("--scope") || given("--version"))
-            throw new Error("--scope and --version belong to hits approve.");
-        const weeks = optionalPositiveInteger(values.flags.get("--weeks"), "--weeks");
-        if (weeks !== undefined && weeks > 26)
-            throw new Error("--weeks must be 26 or fewer.");
-        return { ...common, action, ...(weeks !== undefined ? { weeks } : {}) };
-    }
-    if (action === "coach") {
-        if (given("--scope") || given("--version"))
-            throw new Error("--scope and --version belong to hits approve.");
-        return { ...common, action };
-    }
-    if (action === "approve") {
-        const scope = optionalNonEmpty(values.flags.get("--scope"));
-        if (!scope || !SCOPE.test(scope))
-            throw new Error("hits approve wants --scope team, agents or project:<id> (for example project:pearl-portal).");
-        return { ...common, action, scope, ...versionOf(values.flags.get("--version")) };
-    }
+    const parsed = { values, given, common };
+    if (action === "timeline")
+        return parseHitsTimeline(parsed);
+    if (action === "coach")
+        return parseHitsCoach(parsed);
+    if (action === "approve")
+        return parseHitsApprove(parsed);
+    return parseHitsRead(parsed);
+}
+function refuseApproveFlags({ given }) {
     if (given("--scope") || given("--version"))
         throw new Error("--scope and --version belong to hits approve.");
+}
+function parseHitsTimeline(parsed) {
+    refuseApproveFlags(parsed);
+    const weeks = optionalPositiveInteger(parsed.values.flags.get("--weeks"), "--weeks");
+    if (weeks !== undefined && weeks > 26)
+        throw new Error("--weeks must be 26 or fewer.");
+    return { ...parsed.common, action: "timeline", ...(weeks !== undefined ? { weeks } : {}) };
+}
+function parseHitsCoach(parsed) {
+    refuseApproveFlags(parsed);
+    return { ...parsed.common, action: "coach" };
+}
+function parseHitsApprove({ values, common }) {
+    const scope = optionalNonEmpty(values.flags.get("--scope"));
+    if (!scope || !SCOPE.test(scope))
+        throw new Error("hits approve wants --scope team, agents or project:<id> (for example project:pearl-portal).");
+    return { ...common, action: "approve", scope, ...versionOf(values.flags.get("--version")) };
+}
+function parseHitsRead(parsed) {
+    const { values, given, common } = parsed;
+    refuseApproveFlags(parsed);
     const whoFlags = ["--mine", "--person", "--agent", "--project"].filter(given);
     if (whoFlags.length > 1)
         throw new Error(`Pick one of --mine, --person, --agent or --project (got ${whoFlags.join(" and ")}).`);
@@ -77,7 +89,7 @@ export function parseHitsArgs(args) {
         throw new Error("--learning is read for one person: use --mine or --person <name>.");
     return {
         ...common,
-        action,
+        action: "read",
         ...(who ? { who } : {}),
         ...(person ? { person } : {}),
         ...(project ? { project } : {}),

package/dist/commands/local-args-tower-manager.js

@@ -3,6 +3,8 @@
  *
  *   cockpit manager clients
  *   cockpit manager client <client>
+ *   cockpit manager home <client>
+ *   cockpit manager summary <client> --version <v>|none [--health on_track|at_risk|blocked] [--headline <t>] [--bullet "YYYY-MM-DD: t"|"t"]... [--next <t>|none] [--blocker <t>|none]
  *   cockpit manager run <run-id>
  *   cockpit manager list <client> [--timeline] [--needs-review] [--status open|done|dropped|closed|all] [--owner <name> | --no-owner] [--overdue] [--kind <kind>] [--search <text>] [--archived] [--review dismissed|all]
  *   cockpit manager queue <client>
@@ -22,27 +24,36 @@
  */
 import { optionalNonEmpty, optionalUrl, parseNamedArgs } from "./local-arg-values.js";
 export const MANAGER_VERBS = [
-    "clients", "client", "run", "list", "queue", "show", "add", "edit", "done", "reopen", "link",
+    "clients", "client", "home", "summary", "run", "list", "queue", "show", "add", "edit", "done", "reopen", "link",
     "approve", "dismiss", "proposal", "archive", "unarchive",
 ];
 export const MANAGER_ACTION_STATUSES = ["open", "done", "dropped", "closed", "all"];
 export const MANAGER_TIMELINE_KINDS = ["meeting", "document", "milestone", "internal"];
+export const MANAGER_SUMMARY_HEALTHS = ["on_track", "at_risk", "blocked"];
+/** The verbs whose one positional word is a client (its id or its exact name). */
+const CLIENT_VERBS = new Set(["client", "home", "summary", "list", "queue", "add"]);
 const ID = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;
 const VERSION = /^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}/;
 const ONE_VERB = `manager takes one verb: ${MANAGER_VERBS.join(", ")}. Example: \`cockpit manager clients\` or \`cockpit manager list <client>\`.`;
 const OWN_FLAGS = [
     "--timeline", "--needs-review", "--status", "--owner", "--owners", "--no-owner", "--kind", "--search", "--archived", "--review",
     "--title", "--detail", "--due", "--date", "--meeting", "--none", "--version", "--request-id", "--restore", "--overdue", "--source-url", "--source-note",
+    "--health", "--headline", "--bullet", "--next", "--blocker",
 ];
 const COMMON_FLAGS = ["--home", "--dashboard-url", "--json", "--help"];
 const VALUE_FLAGS = [
     "--status", "--owner", "--owners", "--kind", "--search", "--review", "--title", "--detail", "--due", "--date", "--meeting",
     "--version", "--request-id", "--source-url", "--source-note", "--home", "--dashboard-url",
+    "--health", "--headline", "--bullet", "--next", "--blocker",
 ];
+/** `summary --bullet` is given once per bullet: the one value flag that may repeat. */
+const REPEATABLE_FLAGS = new Set(["--bullet"]);
 /** Which flags each verb owns, so a flag on the wrong verb is refused by name rather than ignored. */
 export const FLAGS_BY_VERB = {
     clients: [],
     client: [],
+    home: [],
+    summary: ["--version", "--health", "--headline", "--bullet", "--next", "--blocker"],
     run: [],
     list: ["--timeline", "--needs-review", "--status", "--owner", "--no-owner", "--overdue", "--kind", "--search", "--archived", "--review"],
     queue: [],
@@ -62,6 +73,8 @@ export const FLAGS_BY_VERB = {
 const POSITIONALS = {
     clients: [],
     client: ["client"],
+    home: ["client"],
+    summary: ["client"],
     run: ["run id"],
     list: ["client"],
     queue: ["client"],
@@ -80,6 +93,8 @@ const POSITIONALS = {
 /** A value flag given twice would silently keep only the last; say so instead. */
 function refuseRepeatedFlags(args) {
     for (const flag of VALUE_FLAGS) {
+        if (REPEATABLE_FLAGS.has(flag))
+            continue;
         const times = args.filter((arg) => arg === flag || arg.startsWith(`${flag}=`)).length;
         if (times > 1) {
             throw new Error(flag === "--owners"
@@ -113,12 +128,12 @@ export function parseManagerArgs(args) {
     if (rest.length !== wanted.length) {
         throw new Error(wanted.length === 0
             ? `\`cockpit manager ${verb}\` takes no positional arguments.`
-            : `\`cockpit manager ${verb}\` takes exactly: ${wanted.map((word) => `<${word}>`).join(" ")}${verb === "list" || verb === "queue" || verb === "add" || verb === "client" ? " (quote a client's name that has spaces)" : ""}.`);
+            : `\`cockpit manager ${verb}\` takes exactly: ${wanted.map((word) => `<${word}>`).join(" ")}${CLIENT_VERBS.has(verb) ? " (quote a client's name that has spaces)" : ""}.`);
     }
     const timeline = values.booleans.has("--timeline");
     const shape = { ...base, action: verb, timeline };
     const flag = (name) => values.flags.get(name);
-    if (verb === "list" || verb === "queue" || verb === "add" || verb === "client")
+    if (CLIENT_VERBS.has(verb))
         shape.client = rest[0];
     else if (verb === "run")
         shape.runId = readId(rest[0], "A run");
@@ -128,9 +143,11 @@ export function parseManagerArgs(args) {
         shape.proposalId = rest[1];
         shape.decision = readDecision(rest[2]);
     }
-    const needsVersion = !["clients", "client", "run", "list", "queue", "show", "add"].includes(verb);
+    const needsVersion = !["clients", "client", "home", "summary", "run", "list", "queue", "show", "add"].includes(verb);
     if (needsVersion)
         shape.version = readVersion(flag("--version"), verb);
+    if (verb === "summary")
+        readSummaryFields(values, shape);
     if (verb === "list")
         readListFilters(values, shape);
     if (verb === "add")
@@ -368,4 +385,68 @@ function readLink(values, shape) {
     if (!meeting && !none)
         throw new Error("`cockpit manager link` needs --meeting <timeline-id> to link, or --none to unlink.");
     shape.timelineItemId = meeting ? readId(meeting, "A timeline item") : null;
+}
+/** `summary`'s version: the card's `summary.updatedAt` as `home --json` gives it, or `none` when the client has no card yet. */
+function readSummaryVersion(raw) {
+    if (raw === undefined) {
+        throw new Error("`cockpit manager summary` needs --version: the card's summary.updatedAt from `cockpit manager home <client> --json`, or none when the client has no card yet. It keeps you from writing over a newer card.");
+    }
+    if (isNone(raw))
+        return null;
+    if (!VERSION.test(raw))
+        throw new Error("--version is the card's summary.updatedAt exactly as `cockpit manager home <client> --json` gives it, like 2026-10-09T17:03:11.123456+00:00, or none.");
+    return raw;
+}
+/** `--bullet "2026-10-14: Signed the SOW"` is a dated bullet; without a leading `YYYY-MM-DD: ` it is undated. */
+export function readBullet(raw) {
+    // A bullet pasted from a list keeps no "- " or "* " of its own.
+    const value = raw.trim().replace(/^[-*•]\s+/u, "");
+    const dated = /^(\d{4}-\d{2}-\d{2}):\s*(.*)$/s.exec(value);
+    const dateLike = dated ? null : /^(\d{1,4}[-/.]\d{1,2}[-/.]\d{1,4})\s*:/.exec(value);
+    if (dateLike)
+        throw new Error(`--bullet "${raw}" starts with ${dateLike[1]}, which is not a YYYY-MM-DD day. Write it like 2026-10-14, or leave the date out for an undated bullet.`);
+    const date = dated ? dated[1] : null;
+    const text = (dated ? dated[2] : value).trim();
+    if (date !== null && !isCalendarDate(date))
+        throw new Error(`--bullet "${raw}" starts with ${date}, which is not a real day. Write the date YYYY-MM-DD, like 2026-10-14, or leave it out for an undated bullet.`);
+    if (text === "")
+        throw new Error("--bullet needs words after its date. Example: --bullet \"2026-10-14: Signed the SOW\".");
+    return { date, text };
+}
+/** The card's fields. Counts, lengths and "blocked needs a blocker" are the server's rules, refused there by name. */
+function readSummaryFields(values, shape) {
+    shape.version = readSummaryVersion(values.flags.get("--version"));
+    let changes = 0;
+    const health = values.flags.get("--health");
+    if (health !== undefined) {
+        if (!MANAGER_SUMMARY_HEALTHS.includes(health))
+            throw new Error(`--health wants one of ${MANAGER_SUMMARY_HEALTHS.join(", ")}.`);
+        shape.health = health;
+        changes += 1;
+    }
+    const headline = values.flags.get("--headline");
+    if (headline !== undefined) {
+        if (headline.trim() === "")
+            throw new Error("--headline cannot be blank.");
+        shape.headline = headline.trim();
+        changes += 1;
+    }
+    const bullets = values.flagValues.get("--bullet");
+    if (bullets !== undefined && bullets.length > 0) {
+        shape.bullets = bullets.map(readBullet);
+        changes += 1;
+    }
+    const next = values.flags.get("--next");
+    if (next !== undefined) {
+        shape.next = isNone(next) ? null : next.trim();
+        changes += 1;
+    }
+    const blocker = values.flags.get("--blocker");
+    if (blocker !== undefined) {
+        shape.blocker = isNone(blocker) ? null : blocker.trim();
+        changes += 1;
+    }
+    if (changes === 0) {
+        throw new Error("`cockpit manager summary` has nothing to change. Give at least one of --health, --headline, --bullet (1 to 5 of them, replacing the list), --next, --blocker. A flag you leave out keeps its value; none clears --next or --blocker.");
+    }
 }

package/dist/commands/local-args-tower-memory.js

@@ -119,6 +119,40 @@ export function parseMemoryStoreArgs(action, args) {
     const values = parseNamedArgs(args, { allowedFlags: ALLOWED_FLAGS, valueFlags: VALUE_FLAGS });
     const rest = values.positionals.slice(1);
     const containers = values.flagValues.get("--container") ?? [];
+    rejectFlagsThatDoNotBelongTo(action, values, containers, rest);
+    const command = {
+        kind: "memory-store",
+        homeDir: optionalNonEmpty(values.flags.get("--home")),
+        dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
+        workspace: optionalNonEmpty(workRootFlagValue(values)),
+        containerTags: containers
+            .map((tag) => tag.trim())
+            .filter((tag) => tag.length > 0),
+        json: values.booleans.has("--json"),
+    };
+    if (action === "requeue")
+        return parseRequeueVerb(command, values, rest);
+    if (action === "profile")
+        return parseProfileVerb(command, rest);
+    if (action === "search")
+        return parseSearchVerb(command, values, rest);
+    if (action === "consolidation")
+        return { ...command, action };
+    if (action === "exam")
+        return parseExamVerb(command, values, rest);
+    if (action === "source")
+        return parseSourceVerb(command, values, rest);
+    if (action === "private" || action === "public" || action === "chain") {
+        return parseOneMemoryIdVerb(command, action, rest);
+    }
+    if (action === "save")
+        return parseSaveVerb(command, values, rest);
+    if (action === "update")
+        return parseUpdateVerb(command, values, rest);
+    return parseForgetVerb(command, values, rest);
+}
+/** Every flag is legal to the parser; this says which verb each one belongs to. */
+function rejectFlagsThatDoNotBelongTo(action, values, containers, rest) {
     if (action !== "search" && action !== "consolidation" && containers.length > 1) {
         throw new Error(`memory ${action} takes at most one --container.`);
     }
@@ -166,130 +200,118 @@ export function parseMemoryStoreArgs(action, args) {
     if (values.flags.has("--file") && action !== "save" && action !== "update") {
         throw new Error("--file only applies to memory save and memory update.");
     }
-    const command = {
-        kind: "memory-store",
-        action,
-        homeDir: optionalNonEmpty(values.flags.get("--home")),
-        dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
-        workspace: optionalNonEmpty(workRootFlagValue(values)),
-        containerTags: containers
-            .map((tag) => tag.trim())
-            .filter((tag) => tag.length > 0),
-        json: values.booleans.has("--json"),
-    };
-    if (action === "requeue") {
-        // A window is REQUIRED. There is no requeue-everything: applying re-runs
-        // one model call per receipt, so the ceiling on a wrong filter is a bill.
-        if (rest.length > 0) {
-            throw new Error("memory requeue takes no words; name the window with --since and --until.");
-        }
-        const since = readInstant(values.flags.get("--since"), "--since");
-        if (!since) {
-            throw new Error("memory requeue needs --since <iso>, e.g. `cockpit memory requeue --since 2026-09-21T04:10:00Z --dry-run`.");
-        }
-        const until = readInstant(values.flags.get("--until"), "--until");
-        if (until && until < since)
-            throw new Error("--until is before --since; nothing could match.");
-        const source = optionalNonEmpty(values.flags.get("--source"));
-        if (source && source !== "hook" && source !== "harvest") {
-            throw new Error("--source is hook or harvest — the two values a receipt can carry.");
-        }
-        if (values.booleans.has("--dry-run") && values.booleans.has("--apply")) {
-            throw new Error("--dry-run and --apply are opposites; name one.");
-        }
-        return {
-            ...command,
-            action,
-            since,
-            until,
-            ...(source ? { source: source } : {}),
-            // The safe question unless somebody typed otherwise.
-            dryRun: !values.booleans.has("--apply"),
-            reason: optionalNonEmpty(values.flags.get("--reason")),
-        };
-    }
-    if (action === "profile") {
-        // A pure read of ONE container's compiled page. No positional: there is
-        // nothing to ask it, and a stray word would look like a query this verb
-        // silently ignored.
-        if (rest.length > 0) {
-            throw new Error("memory profile takes no words; it reads this repository's compiled page. Name another space with --container.");
-        }
-        return { ...command, action };
+}
+function parseRequeueVerb(command, values, rest) {
+    // A window is REQUIRED. There is no requeue-everything: applying re-runs
+    // one model call per receipt, so the ceiling on a wrong filter is a bill.
+    if (rest.length > 0) {
+        throw new Error("memory requeue takes no words; name the window with --since and --until.");
     }
-    if (action === "search") {
-        // Every remaining positional joined, so `cockpit memory search what did we
-        // decide about pagination` needs no quotes — the same shape `cockpit
-        // search` and `cockpit cal find` already have.
-        const query = rest.join(" ").trim();
-        if (!query)
-            throw new Error("memory search needs a question, e.g. `cockpit memory search \"pagination rule\"`.");
-        // BLI-19246: both flags absent leaves the choice to the door (exact-detail questions reach the originals).
-        const includeOriginals = values.booleans.has("--include-originals") ? true : values.booleans.has("--no-originals") ? false : undefined;
-        // BLI-19255: the raw source behind the top hits; absent lets the door decide by the question.
-        const includeSource = values.booleans.has("--include-source") ? true : values.booleans.has("--no-source") ? false : undefined;
-        return {
-            ...command, action, query, limit: readLimit(values.flags.get("--limit")), includeRecent: !values.booleans.has("--no-recent"),
-            ...(includeOriginals === undefined ? {} : { includeOriginals }),
-            ...(includeSource === undefined ? {} : { includeSource }),
-        };
+    const since = readInstant(values.flags.get("--since"), "--since");
+    if (!since) {
+        throw new Error("memory requeue needs --since <iso>, e.g. `cockpit memory requeue --since 2026-09-21T04:10:00Z --dry-run`.");
     }
-    if (action === "consolidation") {
-        return { ...command, action };
+    const until = readInstant(values.flags.get("--until"), "--until");
+    if (until && until < since)
+        throw new Error("--until is before --since; nothing could match.");
+    const source = optionalNonEmpty(values.flags.get("--source"));
+    if (source && source !== "hook" && source !== "harvest") {
+        throw new Error("--source is hook or harvest — the two values a receipt can carry.");
     }
-    if (action === "exam") {
-        if (rest.length > 0)
-            throw new Error("memory exam takes no words: `cockpit memory exam [--night <YYYY-MM-DD>] [--misses] [--json]`.");
-        const night = optionalNonEmpty(values.flags.get("--night"));
-        if (night && !/^\d{4}-\d{2}-\d{2}$/.test(night))
-            throw new Error("--night must be a date like 2026-10-14.");
-        return { ...command, action, ...(night ? { night } : {}), misses: values.booleans.has("--misses") };
+    if (values.booleans.has("--dry-run") && values.booleans.has("--apply")) {
+        throw new Error("--dry-run and --apply are opposites; name one.");
     }
-    if (action === "source") {
-        // BLI-19255: one memory, named by the id `cockpit memory search` printed; `--query` steers which part of the source.
-        const memoryId = optionalNonEmpty(rest[0]);
-        if (!memoryId)
-            throw new Error("memory source needs the memory id `cockpit memory search` printed.");
-        if (rest.length > 1)
-            throw new Error("memory source takes one id.");
-        const rawChars = values.flags.get("--chars");
-        const chars = rawChars === undefined ? undefined : Number(rawChars);
-        if (chars !== undefined && (!Number.isInteger(chars) || chars < 200 || chars > 6000))
-            throw new Error("--chars is a whole number from 200 to 6000.");
-        return { ...command, action, memoryId, ...(values.flags.get("--query") ? { sourceQuery: values.flags.get("--query") } : {}), ...(chars === undefined ? {} : { sourceChars: chars }) };
+    return {
+        ...command,
+        action: "requeue",
+        since,
+        until,
+        ...(source ? { source: source } : {}),
+        // The safe question unless somebody typed otherwise.
+        dryRun: !values.booleans.has("--apply"),
+        reason: optionalNonEmpty(values.flags.get("--reason")),
+    };
+}
+function parseProfileVerb(command, rest) {
+    // A pure read of ONE container's compiled page. No positional: there is
+    // nothing to ask it, and a stray word would look like a query this verb
+    // silently ignored.
+    if (rest.length > 0) {
+        throw new Error("memory profile takes no words; it reads this repository's compiled page. Name another space with --container.");
     }
-    if (action === "private" || action === "public" || action === "chain") {
-        // BLI-12927: one memory, named by the id `cockpit memory search` printed.
-        const memoryId = optionalNonEmpty(rest[0]);
-        if (!memoryId) {
-            throw new Error(`memory ${action} needs the memory id \`cockpit memory search\` printed.`);
-        }
-        if (rest.length > 1)
-            throw new Error(`memory ${action} takes one id.`);
-        return { ...command, action, memoryId };
+    return { ...command, action: "profile" };
+}
+function parseSearchVerb(command, values, rest) {
+    // Every remaining positional joined, so `cockpit memory search what did we
+    // decide about pagination` needs no quotes — the same shape `cockpit
+    // search` and `cockpit cal find` already have.
+    const query = rest.join(" ").trim();
+    if (!query)
+        throw new Error("memory search needs a question, e.g. `cockpit memory search \"pagination rule\"`.");
+    // BLI-19246: both flags absent leaves the choice to the door (exact-detail questions reach the originals).
+    const includeOriginals = values.booleans.has("--include-originals") ? true : values.booleans.has("--no-originals") ? false : undefined;
+    // BLI-19255: the raw source behind the top hits; absent lets the door decide by the question.
+    const includeSource = values.booleans.has("--include-source") ? true : values.booleans.has("--no-source") ? false : undefined;
+    return {
+        ...command, action: "search", query, limit: readLimit(values.flags.get("--limit")), includeRecent: !values.booleans.has("--no-recent"),
+        ...(includeOriginals === undefined ? {} : { includeOriginals }),
+        ...(includeSource === undefined ? {} : { includeSource }),
+    };
+}
+function parseExamVerb(command, values, rest) {
+    if (rest.length > 0)
+        throw new Error("memory exam takes no words: `cockpit memory exam [--night <YYYY-MM-DD>] [--misses] [--json]`.");
+    const night = optionalNonEmpty(values.flags.get("--night"));
+    if (night && !/^\d{4}-\d{2}-\d{2}$/.test(night))
+        throw new Error("--night must be a date like 2026-10-14.");
+    return { ...command, action: "exam", ...(night ? { night } : {}), misses: values.booleans.has("--misses") };
+}
+function parseSourceVerb(command, values, rest) {
+    // BLI-19255: one memory, named by the id `cockpit memory search` printed; `--query` steers which part of the source.
+    const memoryId = optionalNonEmpty(rest[0]);
+    if (!memoryId)
+        throw new Error("memory source needs the memory id `cockpit memory search` printed.");
+    if (rest.length > 1)
+        throw new Error("memory source takes one id.");
+    const rawChars = values.flags.get("--chars");
+    const chars = rawChars === undefined ? undefined : Number(rawChars);
+    if (chars !== undefined && (!Number.isInteger(chars) || chars < 200 || chars > 6000))
+        throw new Error("--chars is a whole number from 200 to 6000.");
+    return { ...command, action: "source", memoryId, ...(values.flags.get("--query") ? { sourceQuery: values.flags.get("--query") } : {}), ...(chars === undefined ? {} : { sourceChars: chars }) };
+}
+function parseOneMemoryIdVerb(command, action, rest) {
+    // BLI-12927: one memory, named by the id `cockpit memory search` printed.
+    const memoryId = optionalNonEmpty(rest[0]);
+    if (!memoryId) {
+        throw new Error(`memory ${action} needs the memory id \`cockpit memory search\` printed.`);
     }
-    if (action === "save") {
-        if (rest.length > 0) {
-            throw new Error("memory save reads the memory from stdin or --file, never from the command line: `echo \"...\" | cockpit memory save`.");
-        }
-        return {
-            ...command,
-            action,
-            customId: optionalNonEmpty(values.flags.get("--custom-id")),
-            ...(values.flagValues.has("--alias") ? { aliases: values.flagValues.get("--alias") } : {}),
-            extract: values.booleans.has("--extract"),
-            filePath: optionalNonEmpty(values.flags.get("--file")),
-        };
+    if (rest.length > 1)
+        throw new Error(`memory ${action} takes one id.`);
+    return { ...command, action, memoryId };
+}
+function parseSaveVerb(command, values, rest) {
+    if (rest.length > 0) {
+        throw new Error("memory save reads the memory from stdin or --file, never from the command line: `echo \"...\" | cockpit memory save`.");
     }
-    if (action === "update") {
-        const memoryId = optionalNonEmpty(rest[0]);
-        if (!memoryId)
-            throw new Error("memory update needs the memory id `cockpit memory search` printed.");
-        if (rest.length > 1) {
-            throw new Error("memory update takes one id; the replacement text comes from stdin or --file, never from the command line.");
-        }
-        return { ...command, action, memoryId, filePath: optionalNonEmpty(values.flags.get("--file")) };
+    return {
+        ...command,
+        action: "save",
+        customId: optionalNonEmpty(values.flags.get("--custom-id")),
+        ...(values.flagValues.has("--alias") ? { aliases: values.flagValues.get("--alias") } : {}),
+        extract: values.booleans.has("--extract"),
+        filePath: optionalNonEmpty(values.flags.get("--file")),
+    };
+}
+function parseUpdateVerb(command, values, rest) {
+    const memoryId = optionalNonEmpty(rest[0]);
+    if (!memoryId)
+        throw new Error("memory update needs the memory id `cockpit memory search` printed.");
+    if (rest.length > 1) {
+        throw new Error("memory update takes one id; the replacement text comes from stdin or --file, never from the command line.");
     }
+    return { ...command, action: "update", memoryId, filePath: optionalNonEmpty(values.flags.get("--file")) };
+}
+function parseForgetVerb(command, values, rest) {
     const memoryId = optionalNonEmpty(rest[0]);
     if (rest.length > 1)
         throw new Error("memory forget takes one id, or --content-stdin with a --container.");
@@ -302,7 +324,7 @@ export function parseMemoryStoreArgs(action, args) {
     }
     return {
         ...command,
-        action,
+        action: "forget",
         memoryId,
         contentStdin,
         reason: optionalNonEmpty(values.flags.get("--reason")),