@bli-cockpit/cli 0.2.237 → 0.2.239

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

@@ -65,6 +65,27 @@ export function parseScoutArgs(args) {
     }
     return { kind: "scout", action: rawAction, experimentRef, ...base };
 }
+const OPS_ALLOWED_FLAGS = [
+    "--home",
+    "--dashboard-url",
+    "--job",
+    "--skips",
+    "--memory",
+    "--memory-days",
+    // BLI-3798: the coverage half on its own, with the bucket table.
+    "--coverage",
+    // BLI-3912: one line per model — latency, tokens/s, today's spend, last
+    // probe verdict. A SECOND door like --memory, asked for only when named.
+    "--models",
+    "--turns",
+    "--tool-router",
+    // BLI-12507: the MCP's `people: true`. Per-device and per-person rows in --json.
+    "--people",
+    "--person",
+    "--dry-run",
+    "--json",
+];
+const OPS_VALUE_FLAGS = ["--home", "--dashboard-url", "--job", "--person", "--memory-days"];
 /**
  * `cockpit ops status [--job <id>] [--skips] [--memory [--memory-days N]]` and
  * `cockpit ops recompile --person <p> [--dry-run]` (BLI-3462).
@@ -76,29 +97,7 @@ export function parseScoutArgs(args) {
  * weekly or a monthly with the compile script, which does support it.
  */
 export function parseOpsArgs(args) {
-    const values = parseNamedArgs(args, {
-        allowedFlags: [
-            "--home",
-            "--dashboard-url",
-            "--job",
-            "--skips",
-            "--memory",
-            "--memory-days",
-            // BLI-3798: the coverage half on its own, with the bucket table.
-            "--coverage",
-            // BLI-3912: one line per model — latency, tokens/s, today's spend, last
-            // probe verdict. A SECOND door like --memory, asked for only when named.
-            "--models",
-            "--turns",
-            "--tool-router",
-            // BLI-12507: the MCP's `people: true`. Per-device and per-person rows in --json.
-            "--people",
-            "--person",
-            "--dry-run",
-            "--json",
-        ],
-        valueFlags: ["--home", "--dashboard-url", "--job", "--person", "--memory-days"],
-    });
+    const values = parseNamedArgs(args, { allowedFlags: OPS_ALLOWED_FLAGS, valueFlags: OPS_VALUE_FLAGS });
     if (values.positionals.length > 1) {
         throw new Error("ops accepts one action: status or recompile.");
     }
@@ -111,36 +110,39 @@ export function parseOpsArgs(args) {
         dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
         json: values.booleans.has("--json"),
     };
-    if (rawAction === "status") {
-        // BLI-3729: `--memory-days N` implies `--memory`, because asking for a
-        // window and getting nothing back is the silent breakage BLI-2490 forbids.
-        const memoryDays = optionalNonEmpty(values.flags.get("--memory-days"));
-        if (memoryDays !== undefined && !/^[0-9]{1,2}$/u.test(memoryDays)) {
-            throw new Error("ops --memory-days takes a whole number of days, 1 to 30.");
-        }
-        const coverageOnly = values.booleans.has("--coverage");
-        const job = optionalNonEmpty(values.flags.get("--job"));
-        if (coverageOnly && job !== undefined && job !== "collection-coverage") {
-            // Refused rather than silently preferring one: `--coverage --job
-            // daily-brief` is somebody asking two different questions in one
-            // command, and answering the wrong one is how a board loses trust.
-            throw new Error("ops --coverage IS --job collection-coverage. Pass one or the other, not a --job naming a different pipeline.");
-        }
-        return {
-            kind: "ops",
-            action: "status",
-            job: coverageOnly ? "collection-coverage" : job,
-            coverage: coverageOnly,
-            skips: values.booleans.has("--skips"),
-            memory: values.booleans.has("--memory") || memoryDays !== undefined,
-            models: values.booleans.has("--models"),
-            turns: values.booleans.has("--turns"),
-            toolRouter: values.booleans.has("--tool-router"),
-            people: values.booleans.has("--people"),
-            ...(memoryDays === undefined ? {} : { memoryDays: Number(memoryDays) }),
-            ...base,
-        };
+    return rawAction === "status" ? parseOpsStatus(values, base) : parseOpsRecompile(values, base);
+}
+function parseOpsStatus(values, base) {
+    // BLI-3729: `--memory-days N` implies `--memory`, because asking for a
+    // window and getting nothing back is the silent breakage BLI-2490 forbids.
+    const memoryDays = optionalNonEmpty(values.flags.get("--memory-days"));
+    if (memoryDays !== undefined && !/^[0-9]{1,2}$/u.test(memoryDays)) {
+        throw new Error("ops --memory-days takes a whole number of days, 1 to 30.");
+    }
+    const coverageOnly = values.booleans.has("--coverage");
+    const job = optionalNonEmpty(values.flags.get("--job"));
+    if (coverageOnly && job !== undefined && job !== "collection-coverage") {
+        // Refused rather than silently preferring one: `--coverage --job
+        // daily-brief` is somebody asking two different questions in one
+        // command, and answering the wrong one is how a board loses trust.
+        throw new Error("ops --coverage IS --job collection-coverage. Pass one or the other, not a --job naming a different pipeline.");
     }
+    return {
+        kind: "ops",
+        action: "status",
+        job: coverageOnly ? "collection-coverage" : job,
+        coverage: coverageOnly,
+        skips: values.booleans.has("--skips"),
+        memory: values.booleans.has("--memory") || memoryDays !== undefined,
+        models: values.booleans.has("--models"),
+        turns: values.booleans.has("--turns"),
+        toolRouter: values.booleans.has("--tool-router"),
+        people: values.booleans.has("--people"),
+        ...(memoryDays === undefined ? {} : { memoryDays: Number(memoryDays) }),
+        ...base,
+    };
+}
+function parseOpsRecompile(values, base) {
     const person = optionalNonEmpty(values.flags.get("--person"));
     if (!person) {
         throw new Error("ops recompile needs --person <email|name|id>. Nothing is recompiled by default, on purpose.");
@@ -161,37 +163,36 @@ export const SLACK_WORKSPACE_KEYS = ["bli", "blue_pearl"];
  * (BLI-3462).
  */
 const SLACK_READ_FORMATS = ["compact", "full"];
+const SLACK_ALLOWED_FLAGS = [
+    "--home",
+    "--dashboard-url",
+    "--workspace",
+    "--stale-only",
+    "--list",
+    "--person",
+    "--channel",
+    "--query",
+    "--word",
+    "--since",
+    "--until",
+    "--limit",
+    "--format",
+    "--json",
+];
+const SLACK_VALUE_FLAGS = [
+    "--home",
+    "--dashboard-url",
+    "--workspace",
+    "--person",
+    "--channel",
+    "--query",
+    "--since",
+    "--until",
+    "--limit",
+    "--format",
+];
 export function parseSlackArgs(args) {
-    const values = parseNamedArgs(args, {
-        allowedFlags: [
-            "--home",
-            "--dashboard-url",
-            "--workspace",
-            "--stale-only",
-            "--list",
-            "--person",
-            "--channel",
-            "--query",
-            "--word",
-            "--since",
-            "--until",
-            "--limit",
-            "--format",
-            "--json",
-        ],
-        valueFlags: [
-            "--home",
-            "--dashboard-url",
-            "--workspace",
-            "--person",
-            "--channel",
-            "--query",
-            "--since",
-            "--until",
-            "--limit",
-            "--format",
-        ],
-    });
+    const values = parseNamedArgs(args, { allowedFlags: SLACK_ALLOWED_FLAGS, valueFlags: SLACK_VALUE_FLAGS });
     if (values.positionals.length > 1) {
         throw new Error("slack accepts one action: coverage, read, connect or disconnect.");
     }
@@ -210,32 +211,39 @@ export function parseSlackArgs(args) {
     if (values.booleans.has("--list") && rawAction !== "coverage") {
         throw new Error("--list belongs to `cockpit slack coverage`.");
     }
-    if (rawAction === "connect" || rawAction === "disconnect") {
-        const workspace = optionalNonEmpty(values.flags.get("--workspace"));
-        if (workspace && !SLACK_WORKSPACE_KEYS.includes(workspace))
-            throw new Error("Unknown Slack workspace.");
-        if (rawAction === "disconnect" && !workspace)
-            throw new Error("slack disconnect needs --workspace bli|blue_pearl.");
-        return { kind: "slack", action: rawAction, workspace, ...base };
-    }
-    if (rawAction === "coverage") {
-        // `--workspace` here is a Slack workspace key, NOT the collection-root
-        // `--workspace <path>` every setup command takes. Named at the point of use
-        // so the two cannot be confused silently.
-        const workspace = optionalNonEmpty(values.flags.get("--workspace"));
-        if (workspace && !SLACK_WORKSPACE_KEYS.includes(workspace)) {
-            throw new Error(`slack --workspace must be one of ${SLACK_WORKSPACE_KEYS.join(", ")}; got ${workspace}.`);
-        }
-        return {
-            kind: "slack",
-            action: "coverage",
-            workspace,
-            staleOnly: values.booleans.has("--stale-only"),
-            // BLI-10450: counts alone could not answer "is there a Kaiben channel?".
-            list: values.booleans.has("--list"),
-            ...base,
-        };
+    if (rawAction === "connect" || rawAction === "disconnect")
+        return parseSlackConnection(values, base, rawAction);
+    if (rawAction === "coverage")
+        return parseSlackCoverage(values, base);
+    return parseSlackRead(values, base);
+}
+function parseSlackConnection(values, base, action) {
+    const workspace = optionalNonEmpty(values.flags.get("--workspace"));
+    if (workspace && !SLACK_WORKSPACE_KEYS.includes(workspace))
+        throw new Error("Unknown Slack workspace.");
+    if (action === "disconnect" && !workspace)
+        throw new Error("slack disconnect needs --workspace bli|blue_pearl.");
+    return { kind: "slack", action, workspace, ...base };
+}
+function parseSlackCoverage(values, base) {
+    // `--workspace` here is a Slack workspace key, NOT the collection-root
+    // `--workspace <path>` every setup command takes. Named at the point of use
+    // so the two cannot be confused silently.
+    const workspace = optionalNonEmpty(values.flags.get("--workspace"));
+    if (workspace && !SLACK_WORKSPACE_KEYS.includes(workspace)) {
+        throw new Error(`slack --workspace must be one of ${SLACK_WORKSPACE_KEYS.join(", ")}; got ${workspace}.`);
     }
+    return {
+        kind: "slack",
+        action: "coverage",
+        workspace,
+        staleOnly: values.booleans.has("--stale-only"),
+        // BLI-10450: counts alone could not answer "is there a Kaiben channel?".
+        list: values.booleans.has("--list"),
+        ...base,
+    };
+}
+function parseSlackRead(values, base) {
     const person = optionalNonEmpty(values.flags.get("--person"));
     const channel = optionalNonEmpty(values.flags.get("--channel"));
     const query = optionalNonEmpty(values.flags.get("--query"));
@@ -271,20 +279,19 @@ export function parseSlackArgs(args) {
  * `cockpit team [members|invite <email>|role <userId>|device list|device
  * revoke <id|name>]` (BLI-3461; `device` added BLI-3559).
  */
+const TEAM_ALLOWED_FLAGS = [
+    "--home",
+    "--dashboard-url",
+    "--json",
+    "--role",
+    "--team-id",
+    "--yes",
+    "--reason",
+    "--note",
+];
+const TEAM_VALUE_FLAGS = ["--home", "--dashboard-url", "--role", "--team-id", "--reason", "--note"];
 export function parseTeamArgs(args) {
-    const values = parseNamedArgs(args, {
-        allowedFlags: [
-            "--home",
-            "--dashboard-url",
-            "--json",
-            "--role",
-            "--team-id",
-            "--yes",
-            "--reason",
-            "--note",
-        ],
-        valueFlags: ["--home", "--dashboard-url", "--role", "--team-id", "--reason", "--note"],
-    });
+    const values = parseNamedArgs(args, { allowedFlags: TEAM_ALLOWED_FLAGS, valueFlags: TEAM_VALUE_FLAGS });
     const common = {
         homeDir: optionalNonEmpty(values.flags.get("--home")),
         dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
@@ -293,31 +300,15 @@ export function parseTeamArgs(args) {
         json: values.booleans.has("--json"),
     };
     const [verb, ...rest] = values.positionals;
-    if (verb === "github-login") {
-        const [operation, personRef, githubLogin, ...extra] = rest;
-        if (!["add", "remove", "list"].includes(operation ?? "") || !personRef || extra.length ||
-            (operation === "list" ? githubLogin !== undefined : !githubLogin)) {
-            throw new Error("team github-login takes add|remove <person> <login>, or list <person>.");
-        }
-        return { kind: "team", action: `github-login-${operation}`, personRef, githubLogin, ...common };
-    }
-    if (verb === "timezone") {
-        const [operation, personRef, zone, ...extra] = rest;
-        if (!["set", "list"].includes(operation ?? "") || extra.length ||
-            (operation === "list" ? personRef !== undefined : !personRef || !zone)) {
-            throw new Error("team timezone takes set <person> <iana-zone> (or the word unknown), or list.");
-        }
-        return { kind: "team", action: operation === "set" ? "timezone-set" : "timezone-list", ...(personRef ? { personRef } : {}), ...(zone ? { zone } : {}), ...common };
-    }
+    if (verb === "github-login")
+        return parseTeamGithubLoginArgs(rest, common);
+    if (verb === "timezone")
+        return parseTeamTimezoneArgs(rest, common);
     if (verb === "device") {
         return parseTeamDeviceArgs(rest, values.flags, common);
     }
-    if (verb === "grant" || verb === "revoke") {
-        const [person, section, level = "see", ...extra] = rest;
-        if (!person || section !== "careers" || !["see", "act"].includes(level) || extra.length)
-            throw new Error("team grant|revoke <person uuid> careers [see|act]");
-        return { kind: "team", action: verb, targetUserId: person, section: "careers", level: level, ...common };
-    }
+    if (verb === "grant" || verb === "revoke")
+        return parseTeamGrantArgs(verb, rest, common);
     const [subject, ...extra] = rest;
     if (extra.length > 0)
         throw new Error("team does not accept that many arguments.");
@@ -345,6 +336,28 @@ export function parseTeamArgs(args) {
     }
     throw new Error("team takes one verb: members, invite, role, grant, revoke, or device.");
 }
+function parseTeamGithubLoginArgs(rest, common) {
+    const [operation, personRef, githubLogin, ...extra] = rest;
+    if (!["add", "remove", "list"].includes(operation ?? "") || !personRef || extra.length ||
+        (operation === "list" ? githubLogin !== undefined : !githubLogin)) {
+        throw new Error("team github-login takes add|remove <person> <login>, or list <person>.");
+    }
+    return { kind: "team", action: `github-login-${operation}`, personRef, githubLogin, ...common };
+}
+function parseTeamTimezoneArgs(rest, common) {
+    const [operation, personRef, zone, ...extra] = rest;
+    if (!["set", "list"].includes(operation ?? "") || extra.length ||
+        (operation === "list" ? personRef !== undefined : !personRef || !zone)) {
+        throw new Error("team timezone takes set <person> <iana-zone> (or the word unknown), or list.");
+    }
+    return { kind: "team", action: operation === "set" ? "timezone-set" : "timezone-list", ...(personRef ? { personRef } : {}), ...(zone ? { zone } : {}), ...common };
+}
+function parseTeamGrantArgs(verb, rest, common) {
+    const [person, section, level = "see", ...extra] = rest;
+    if (!person || section !== "careers" || !["see", "act"].includes(level) || extra.length)
+        throw new Error("team grant|revoke <person uuid> careers [see|act]");
+    return { kind: "team", action: verb, targetUserId: person, section: "careers", level: level, ...common };
+}
 /**
  * `cockpit team device [list] [--json]` or
  * `cockpit team device revoke <id|name> --reason <label> [--note "<text>"]`

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

@@ -20,7 +20,7 @@
  */
 import { optionalNonEmpty, optionalPositiveInteger, optionalUrl, parseNamedArgs, } from "./local-arg-values.js";
 const CAL_ACTIONS = new Set([
-    "conflicts", "calendar-change",
+    "event", "export", "duplicate", "batch", "undo", "conflicts", "calendar-change",
     "today",
     "week",
     "next",
@@ -36,58 +36,67 @@ const CAL_ACTIONS = new Set([
     "delete",
 ]);
 /** Every verb that names ONE thing first — a calendar id, or (delete) an event id. */
-const CAL_ACTIONS_NEEDING_A_SUBJECT = new Set(["calendar-change", "sync", "detach", "share", "delete", "change", "cancel"]);
+const CAL_ACTIONS_NEEDING_A_SUBJECT = new Set(["event", "export", "duplicate", "calendar-change", "sync", "detach", "share", "delete", "change", "cancel"]);
 /** `sync-status [<calendar id>]` — every calendar, or one's. Ticket 10730. */
-const CAL_ACTIONS_WITH_OPTIONAL_SUBJECT = new Set(["sync-status"]);
+const CAL_ACTIONS_WITH_OPTIONAL_SUBJECT = new Set(["sync-status", "undo"]);
+const CAL_ALLOWED_FLAGS = [
+    "--home",
+    "--dashboard-url",
+    "--color",
+    "--calendar",
+    "--account",
+    "--tz",
+    "--occurrence-start",
+    "--offset",
+    "--hours",
+    "--from",
+    "--to",
+    "--limit",
+    "--all",
+    "--name",
+    "--title",
+    "--at",
+    "--until",
+    "--location",
+    "--attendee",
+    "--all-day",
+    "--timed",
+    "--keep-invitees",
+    "--meet",
+    "--org-visible",
+    "--private",
+    "--full",
+    "--yes",
+    "--json",
+    "--idempotency-key",
+    "--expected-version",
+    "--reason",
+];
+const CAL_VALUE_FLAGS = [
+    "--home",
+    "--dashboard-url",
+    "--color",
+    "--calendar",
+    "--account",
+    "--tz",
+    "--occurrence-start",
+    "--offset",
+    "--hours",
+    "--from",
+    "--to",
+    "--limit",
+    "--name",
+    "--title",
+    "--at",
+    "--until",
+    "--location",
+    "--attendee",
+    "--idempotency-key",
+    "--expected-version",
+    "--reason",
+];
 export function parseCalArgs(args) {
-    const values = parseNamedArgs(args, {
-        allowedFlags: [
-            "--home",
-            "--dashboard-url",
-            "--color",
-            "--calendar",
-            "--account",
-            "--tz",
-            "--offset",
-            "--hours",
-            "--from",
-            "--to",
-            "--limit",
-            "--all",
-            "--name",
-            "--title",
-            "--at",
-            "--until",
-            "--location",
-            "--attendee",
-            "--all-day",
-            "--meet",
-            "--org-visible",
-            "--private",
-            "--full",
-            "--yes",
-            "--json",
-        ],
-        valueFlags: [
-            "--home",
-            "--dashboard-url",
-            "--color",
-            "--calendar",
-            "--account",
-            "--tz",
-            "--offset",
-            "--hours",
-            "--from",
-            "--to",
-            "--limit",
-            "--name",
-            "--title",
-            "--at",
-            "--until",
-            "--location",
-            "--attendee",
-        ],
-    });
+    const values = parseNamedArgs(args, { allowedFlags: CAL_ALLOWED_FLAGS, valueFlags: CAL_VALUE_FLAGS });
     const { action, rest } = readCalAction(values);
     const { subject, query } = readCalSubjectOrQuery(action, rest);
     const { title, startsAt, endsAt } = readCalCreateDetails(action, values);
@@ -103,7 +112,7 @@ function readCalAction(values) {
     const first = values.positionals[0];
     const action = (first === undefined ? "today" : first);
     if (!CAL_ACTIONS.has(action)) {
-        throw new Error(`Unknown cal command: ${first}. Try today, week, conflicts, calendar-change, next, find, calendars, add-ical, create, share, detach, sync, sync-status, or delete.`);
+        throw new Error(`Unknown cal command: ${first}. Try today, week, conflicts, calendar-change, next, find, calendars, add-ical, create, duplicate, batch, undo, share, detach, sync, sync-status, or delete.`);
     }
     return { action, rest: values.positionals.slice(first === undefined ? 0 : 1) };
 }
@@ -113,8 +122,8 @@ function readCalSubjectOrQuery(action, rest) {
     if (CAL_ACTIONS_NEEDING_A_SUBJECT.has(action)) {
         subject = optionalNonEmpty(rest[0]);
         if (!subject) {
-            throw new Error(action === "delete"
-                ? "cal delete needs an event id — take one from a window verb's output (today/week/next/find)."
+            throw new Error(action === "delete" || action === "duplicate"
+                ? `cal ${action} needs an event id — take one from a window verb's output (today/week/next/find).`
                 : `cal ${action} needs a calendar id — take one from \`cockpit cal calendars\`.`);
         }
         if (rest.length > 1)
@@ -173,6 +182,7 @@ function readCalOffset(values) {
 function buildCalCommand(values, action, subject, query, title, startsAt, endsAt, offset, limit) {
     return {
         kind: "cal",
+        occurrenceStart: optionalNonEmpty(values.flags.get("--occurrence-start")),
         color: optionalNonEmpty(values.flags.get("--color")),
         action,
         homeDir: optionalNonEmpty(values.flags.get("--home")),
@@ -199,6 +209,11 @@ function buildCalCommand(values, action, subject, query, title, startsAt, endsAt
         location: optionalNonEmpty(values.flags.get("--location")),
         attendees: splitAddresses(values.flags.get("--attendee")),
         allDay: values.booleans.has("--all-day"),
+        timed: values.booleans.has("--timed"),
+        keepInvitees: values.booleans.has("--keep-invitees"),
+        idempotencyKey: optionalNonEmpty(values.flags.get("--idempotency-key")),
+        expectedVersion: optionalNonEmpty(values.flags.get("--expected-version")),
+        reason: optionalNonEmpty(values.flags.get("--reason")),
         meet: values.booleans.has("--meet"),
         orgVisible: values.booleans.has("--org-visible"),
         makePrivate: values.booleans.has("--private"),