@bli-cockpit/cli 0.2.119 → 0.2.122

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

@@ -77,12 +77,25 @@ export function parseCalArgs(args) {
             "--attendee",
         ],
     });
+    const { action, rest } = readCalAction(values);
+    const { subject, query } = readCalSubjectOrQuery(action, rest);
+    const { title, startsAt, endsAt } = readCalCreateDetails(action, values);
+    validateCalSharing(action, values);
+    // Read in the order the flags have always been checked, so a caller who got
+    // two of them wrong is told about --limit first, as before.
+    const limit = optionalPositiveInteger(values.flags.get("--limit"), "--limit");
+    const offset = readCalOffset(values);
+    return buildCalCommand(values, action, subject, query, title, startsAt, endsAt, offset, limit);
+}
+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, next, find, calendars, add-ical, create, share, detach, or sync.`);
     }
-    const rest = values.positionals.slice(first === undefined ? 0 : 1);
+    return { action, rest: values.positionals.slice(first === undefined ? 0 : 1) };
+}
+function readCalSubjectOrQuery(action, rest) {
     let subject;
     let query;
     if (CAL_ACTIONS_NEEDING_A_SUBJECT.has(action)) {
@@ -100,6 +113,9 @@ export function parseCalArgs(args) {
     else if (rest.length > 0) {
         throw new Error(`cal ${action} does not take "${rest[0]}".`);
     }
+    return { subject, query };
+}
+function readCalCreateDetails(action, values) {
     const title = optionalNonEmpty(values.flags.get("--title"));
     const startsAt = optionalNonEmpty(values.flags.get("--at"));
     const endsAt = optionalNonEmpty(values.flags.get("--until"));
@@ -114,15 +130,24 @@ export function parseCalArgs(args) {
         if (!endsAt)
             throw new Error("cal create needs --until <when it ends>.");
     }
-    if (action === "share" && !values.booleans.has("--org-visible") && !values.booleans.has("--private")) {
+    return { title, startsAt, endsAt };
+}
+function validateCalSharing(action, values) {
+    if (action === "share" &&
+        !values.booleans.has("--org-visible") &&
+        !values.booleans.has("--private")) {
         throw new Error("cal share needs --org-visible (every member may read it) or --private (only you). Sharing is a decision, not a default.");
     }
-    const limit = optionalPositiveInteger(values.flags.get("--limit"), "--limit");
+}
+function readCalOffset(values) {
     const offsetRaw = optionalNonEmpty(values.flags.get("--offset"));
     const offset = offsetRaw === undefined ? undefined : Number.parseInt(offsetRaw, 10);
     if (offset !== undefined && !Number.isFinite(offset)) {
         throw new Error("--offset takes a whole number of days (today/next) or weeks (week).");
     }
+    return offset;
+}
+function buildCalCommand(values, action, subject, query, title, startsAt, endsAt, offset, limit) {
     return {
         kind: "cal",
         action,

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

@@ -1,13 +1,31 @@
 import { optionalNonEmpty, optionalUrl, parseNamedArgs } from './local-arg-values.js';
+const ACTIONS = ['list', 'show', 'rescreen', 'invite', 'decide', 'takehome-show', 'takehome-set', 'settings-show', 'settings-set'];
+const DECISIONS = ['passed_on', 'archived', 'reopen'];
+/** Verbs that name an application id, a role slug, or nothing at all. */
+const NEEDS_ID = ['show', 'rescreen', 'invite', 'decide'];
+const NEEDS_ROLE = ['takehome-show', 'takehome-set'];
 export function parseCareersArgs(args) {
-    const values = parseNamedArgs(args, { allowedFlags: ['--role', '--min-score', '--since', '--home', '--dashboard-url', '--json'], valueFlags: ['--role', '--min-score', '--since', '--home', '--dashboard-url'] });
-    const action = values.positionals[0] ?? 'list';
-    if (action !== 'list' && action !== 'show' && action !== 'rescreen')
-        throw new Error('careers takes list, show, or rescreen.');
-    const id = values.positionals[1];
-    if (action !== 'list' && !id)
-        throw new Error(`careers ${action} needs an application id.`);
-    if (values.positionals.length > (action === 'list' ? 1 : 2))
+    const valueFlags = ['--role', '--min-score', '--since', '--home', '--dashboard-url', '--decision', '--stage', '--subject', '--body-file', '--link', '--pass-score', '--account', '--auto-invite-min-score', '--attention-min-score', '--enabled'];
+    const values = parseNamedArgs(args, { allowedFlags: [...valueFlags, '--json'], valueFlags });
+    const positionals = [...values.positionals];
+    // `takehome show` and `settings set` are two words a person types and one
+    // action everything downstream reads.
+    // A bare `careers takehome` or `careers settings` reads as its `show`, the way
+    // `team device` reads as `team device list`: the noun alone is a question.
+    const first = positionals[0] ?? 'list';
+    const paired = first === 'takehome' || first === 'settings';
+    const action = paired
+        ? `${first}-${positionals.splice(0, positionals[1] === undefined ? 1 : 2)[1] ?? 'show'}`
+        : (positionals.shift() ?? 'list');
+    if (!ACTIONS.includes(action))
+        throw new Error(`careers takes ${ACTIONS.join(', ')}.`);
+    const verb = action;
+    const target = positionals.shift();
+    if (NEEDS_ID.includes(verb) && !target)
+        throw new Error(`careers ${verb} needs an application id.`);
+    if (NEEDS_ROLE.includes(verb) && !target)
+        throw new Error(`careers ${verb.replace('-', ' ')} needs a role slug.`);
+    if (positionals.length > 0)
         throw new Error('Unexpected careers argument.');
     const rawScore = values.flags.get('--min-score');
     const minScore = rawScore === undefined ? undefined : Number(rawScore);
@@ -16,5 +34,34 @@ export function parseCareersArgs(args) {
     const since = optionalNonEmpty(values.flags.get('--since'));
     if (since && !Number.isFinite(Date.parse(since)))
         throw new Error('--since must be a date.');
-    return { kind: 'careers', action, id, role: optionalNonEmpty(values.flags.get('--role')), minScore, since: since ? new Date(since).toISOString() : undefined, homeDir: optionalNonEmpty(values.flags.get('--home')), dashboardUrl: optionalUrl(values.flags.get('--dashboard-url')), json: values.booleans.has('--json') };
+    const decision = optionalNonEmpty(values.flags.get('--decision'));
+    if (verb === 'decide' && (!decision || !DECISIONS.includes(decision)))
+        throw new Error(`careers decide needs --decision ${DECISIONS.join('|')}.`);
+    const passScore = numberFlag(values.flags.get('--pass-score'), '--pass-score', 0, 10);
+    const autoInviteMinScore = numberFlag(values.flags.get('--auto-invite-min-score'), '--auto-invite-min-score', 0, 100);
+    const attentionMinScore = numberFlag(values.flags.get('--attention-min-score'), '--attention-min-score', 0, 100);
+    const enabledRaw = optionalNonEmpty(values.flags.get('--enabled'));
+    if (enabledRaw !== undefined && enabledRaw !== 'true' && enabledRaw !== 'false')
+        throw new Error('--enabled must be true or false.');
+    return {
+        kind: 'careers', action: verb,
+        id: NEEDS_ID.includes(verb) ? target : undefined,
+        takehomeRole: NEEDS_ROLE.includes(verb) ? target : undefined,
+        role: optionalNonEmpty(values.flags.get('--role')), minScore, since: since ? new Date(since).toISOString() : undefined,
+        stage: optionalNonEmpty(values.flags.get('--stage')), decision,
+        subject: optionalNonEmpty(values.flags.get('--subject')), bodyFile: optionalNonEmpty(values.flags.get('--body-file')),
+        link: optionalNonEmpty(values.flags.get('--link')), passScore,
+        inviteAccountId: optionalNonEmpty(values.flags.get('--account')),
+        autoInviteMinScore, attentionMinScore,
+        autoInviteEnabled: enabledRaw === undefined ? undefined : enabledRaw === 'true',
+        homeDir: optionalNonEmpty(values.flags.get('--home')), dashboardUrl: optionalUrl(values.flags.get('--dashboard-url')), json: values.booleans.has('--json'),
+    };
+}
+function numberFlag(raw, flag, min, max) {
+    if (raw === undefined)
+        return undefined;
+    const value = Number(raw);
+    if (!Number.isFinite(value) || value < min || value > max)
+        throw new Error(`${flag} must be ${min} to ${max}.`);
+    return value;
 }

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

@@ -10,7 +10,39 @@
  */
 import { optionalNonEmpty, optionalPositiveInteger, optionalUrl, parseNamedArgs, } from "./local-arg-values.js";
 export function parseJarvisArgs(args) {
-    const values = parseNamedArgs(args, {
+    const values = parseJarvisNamedArgs(args);
+    const prompt = parseJarvisPrompt(values);
+    const thread = parseJarvisThread(values);
+    const imagePath = parseJarvisAttachment(values);
+    const { threads, history } = parseJarvisReadback(values, prompt, imagePath);
+    const date = parseJarvisDate(values, threads, history);
+    const trace = parseJarvisTrace(values, threads, history, prompt, imagePath);
+    return {
+        kind: "jarvis",
+        homeDir: optionalNonEmpty(values.flags.get("--home")),
+        dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
+        prompt,
+        subject: optionalNonEmpty(values.flags.get("--as")),
+        thread,
+        threads,
+        history,
+        ...(trace ? { trace } : {}),
+        limit: optionalPositiveInteger(values.flags.get("--limit"), "--limit"),
+        // BLI-3381: no client-side allowlist — the dashboard forwards this key
+        // to the inference server's own allowlist and relays its refusal.
+        model: optionalNonEmpty(values.flags.get("--model")),
+        ...(date ? { date } : {}),
+        imagePath,
+        // BLI-3457: streaming is on unless a caller opts out. A dashboard that
+        // does not stream yet still answers plain JSON, so this flag is for
+        // callers that want the single-body shape on purpose, not a compat knob.
+        stream: !values.booleans.has("--no-stream"),
+        json: values.booleans.has("--json"),
+        showApprovalCode: values.booleans.has("--show-approval-code"),
+    };
+}
+function parseJarvisNamedArgs(args) {
+    return parseNamedArgs(args, {
         allowedFlags: [
             "--home",
             "--dashboard-url",
@@ -50,15 +82,23 @@ export function parseJarvisArgs(args) {
             "--limit",
         ],
     });
+}
+function parseJarvisPrompt(values) {
     const flaggedPrompt = optionalNonEmpty(values.flags.get("--prompt"));
     const positionalPrompt = optionalNonEmpty(values.positionals.join(" "));
     if (flaggedPrompt && positionalPrompt) {
         throw new Error("jarvis accepts either --prompt or positional text, not both.");
     }
+    return flaggedPrompt ?? positionalPrompt;
+}
+function parseJarvisThread(values) {
     const thread = optionalNonEmpty(values.flags.get("--thread")) ?? "main";
     if (!/^[A-Za-z0-9_-]{1,40}$/.test(thread)) {
         throw new Error("jarvis --thread must use 1 to 40 letters, numbers, underscores, or hyphens.");
     }
+    return thread;
+}
+function parseJarvisAttachment(values) {
     // BLI-3414: `--file` is a plain alias for `--image` — same flag, whichever
     // word a person reaches for first.
     const image = optionalNonEmpty(values.flags.get("--image"));
@@ -66,6 +106,9 @@ export function parseJarvisArgs(args) {
     if (image && file) {
         throw new Error("jarvis accepts either --image or --file, not both — they are the same flag.");
     }
+    return image ?? file;
+}
+function parseJarvisReadback(values, prompt, imagePath) {
     // BLI-3458. Reading history and asking a question are different acts, and a
     // command that quietly did one while you asked for the other would be worse
     // than a refusal — `--threads` with a question would silently drop the
@@ -75,12 +118,15 @@ export function parseJarvisArgs(args) {
     if (threads && history) {
         throw new Error("jarvis --threads lists every thread; --history replays one. Pass one, not both.");
     }
-    if ((threads || history) && (flaggedPrompt || positionalPrompt)) {
+    if ((threads || history) && prompt) {
         throw new Error("jarvis --threads and --history read back what was already said; they do not take a question.");
     }
-    if ((threads || history) && image) {
+    if ((threads || history) && imagePath) {
         throw new Error("jarvis --threads and --history do not take an attachment.");
     }
+    return { threads, history };
+}
+function parseJarvisDate(values, threads, history) {
     // BLI-3484. `--date` binds the page that was live on one of the subject's
     // days, so a turn can be about Sunday's page. Refused on the two reading
     // commands for the same reason an attachment is: they replay what was said
@@ -89,6 +135,9 @@ export function parseJarvisArgs(args) {
     if ((threads || history) && date) {
         throw new Error("jarvis --threads and --history replay what was said; they bind no page.");
     }
+    return date;
+}
+function parseJarvisTrace(values, threads, history, prompt, imagePath) {
     // BLI-3560. `--trace` reads back what a turn DID, the same class of act as
     // `--threads` and `--history`, and refused alongside them for the same
     // reason: a command that quietly asked a question while you asked to see one
@@ -97,38 +146,16 @@ export function parseJarvisArgs(args) {
     if (trace && (threads || history)) {
         throw new Error("jarvis --trace shows one turn's steps; --threads and --history replay what was said. Pass one.");
     }
-    if (trace && (flaggedPrompt || positionalPrompt)) {
+    if (trace && prompt) {
         throw new Error("jarvis --trace shows a turn that already ran; it does not take a question.");
     }
-    if (trace && image) {
+    if (trace && imagePath) {
         throw new Error("jarvis --trace does not take an attachment.");
     }
     if (trace !== undefined && !/^(last|[A-Za-z0-9_-]{1,64})$/.test(trace)) {
         throw new Error("jarvis --trace takes `last` or a trace id (up to 64 letters, numbers, underscores or hyphens).");
     }
-    return {
-        kind: "jarvis",
-        homeDir: optionalNonEmpty(values.flags.get("--home")),
-        dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
-        prompt: flaggedPrompt ?? positionalPrompt,
-        subject: optionalNonEmpty(values.flags.get("--as")),
-        thread,
-        threads,
-        history,
-        ...(trace ? { trace } : {}),
-        limit: optionalPositiveInteger(values.flags.get("--limit"), "--limit"),
-        // BLI-3381: no client-side allowlist — the dashboard forwards this key
-        // to the inference server's own allowlist and relays its refusal.
-        model: optionalNonEmpty(values.flags.get("--model")),
-        ...(date ? { date } : {}),
-        imagePath: image ?? file,
-        // BLI-3457: streaming is on unless a caller opts out. A dashboard that
-        // does not stream yet still answers plain JSON, so this flag is for
-        // callers that want the single-body shape on purpose, not a compat knob.
-        stream: !values.booleans.has("--no-stream"),
-        json: values.booleans.has("--json"),
-        showApprovalCode: values.booleans.has("--show-approval-code"),
-    };
+    return trace;
 }
 /**
  * `cockpit correct` (BLI-3458) — say that one line on the page is wrong.

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

@@ -27,23 +27,33 @@ export function parseDocsArgs(args) {
         ],
         valueFlags: ["--home", "--dashboard-url", "--title", "--parent", "--visibility", "--file", "--query", "--limit"],
     });
-    const first = values.positionals[0];
+    const { action, rest } = parseDocsAction(values.positionals);
+    const docRef = parseDocsDocumentReference(action, rest);
+    const { visibility, clearParent, parentId, allowEmpty, title, query, limit } = parseDocsFlags(action, values);
+    return buildDocsCommand(values, action, docRef, visibility, clearParent, parentId, allowEmpty, title, query, limit);
+}
+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, or update.`);
     }
-    const rest = values.positionals.slice(first === undefined ? 0 : 1);
-    let docRef;
+    return { action, rest: positionals.slice(first === undefined ? 0 : 1) };
+}
+function parseDocsDocumentReference(action, rest) {
     if (DOCS_ACTIONS_NEEDING_A_DOC.has(action)) {
-        docRef = optionalNonEmpty(rest[0]);
+        const docRef = optionalNonEmpty(rest[0]);
         if (!docRef)
             throw new Error(`docs ${action} needs a document id or slug.`);
         if (rest.length > 1)
             throw new Error(`docs ${action} takes one document reference, not ${rest.length}.`);
+        return docRef;
     }
-    else if (rest.length > 0) {
+    if (rest.length > 0)
         throw new Error(`docs ${action} does not take "${rest[0]}".`);
-    }
+    return undefined;
+}
+function parseDocsFlags(action, values) {
     const visibility = optionalNonEmpty(values.flags.get("--visibility"));
     if (visibility && visibility !== "org" && visibility !== "private") {
         throw new Error('docs --visibility must be "org" or "private".');
@@ -80,6 +90,9 @@ export function parseDocsArgs(args) {
     if (limit !== undefined && action !== "list") {
         throw new Error("--limit belongs to `cockpit docs list`.");
     }
+    return { visibility, clearParent, parentId, allowEmpty, title, query, limit };
+}
+function buildDocsCommand(values, action, docRef, visibility, clearParent, parentId, allowEmpty, title, query, limit) {
     return {
         kind: "docs",
         action,
@@ -137,12 +150,21 @@ export function parseMsgArgs(args) {
         ],
         valueFlags: ["--home", "--dashboard-url", "--channel", "--thread", "--limit", "--members", "--description"],
     });
-    const first = values.positionals[0];
+    const { action, rest } = parseMsgAction(values.positionals);
+    const { channelName, dmEmail } = parseMsgCreateTarget(action, rest);
+    const { isPrivate, memberEmails, description } = parseMsgCreateFlags(action, values);
+    const { channelRef, threadId, limit } = parseMsgReadTarget(action, rest, values);
+    return buildMsgCommand(values, action, channelRef, channelName, dmEmail, memberEmails, description, isPrivate, threadId, limit);
+}
+function parseMsgAction(positionals) {
+    const first = positionals[0];
     const action = (first === undefined ? "channels" : first);
     if (!MSG_ACTIONS.has(action)) {
         throw new Error(`Unknown msg command: ${first}. Try channels, read, send, thread, create, or dm.`);
     }
-    const rest = values.positionals.slice(first === undefined ? 0 : 1);
+    return { action, rest: positionals.slice(first === undefined ? 0 : 1) };
+}
+function parseMsgCreateTarget(action, rest) {
     let channelName;
     let dmEmail;
     if (action === "create") {
@@ -167,6 +189,9 @@ export function parseMsgArgs(args) {
         if (!dmEmail.includes("@"))
             throw new Error(`msg dm takes an email address; "${dmEmail}" is not one.`);
     }
+    return { channelName, dmEmail };
+}
+function parseMsgCreateFlags(action, values) {
     const isPrivate = values.booleans.has("--private");
     if (isPrivate && action !== "create") {
         throw new Error("--private belongs to `cockpit msg create`.");
@@ -179,6 +204,9 @@ export function parseMsgArgs(args) {
     if (description && action !== "create") {
         throw new Error("--description belongs to `cockpit msg create`.");
     }
+    return { isPrivate, memberEmails, description };
+}
+function parseMsgReadTarget(action, rest, values) {
     let channelRef;
     if (MSG_ACTIONS_NEEDING_A_CHANNEL.has(action)) {
         channelRef = optionalNonEmpty(rest[0]);
@@ -207,6 +235,9 @@ export function parseMsgArgs(args) {
     if (limit !== undefined && action !== "read" && action !== "thread") {
         throw new Error("--limit belongs to `cockpit msg read` or `cockpit msg thread`.");
     }
+    return { channelRef, threadId, limit };
+}
+function buildMsgCommand(values, action, channelRef, channelName, dmEmail, memberEmails, description, isPrivate, threadId, limit) {
     return {
         kind: "msg",
         action,

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

@@ -70,12 +70,23 @@ export function parseMailArgs(args) {
             "--out",
         ],
     });
+    const { action, rest } = readMailAction(values);
+    const { subject, query } = readMailSubjectOrQuery(action, rest);
+    const address = readImapAddress(action, values);
+    const { to, cc } = readMailRecipients(action, values);
+    const outPath = readAttachmentPath(action, values);
+    const limit = readMailLimit(action, values);
+    return buildMailCommand(values, action, subject, query, address, to, cc, outPath, limit);
+}
+function readMailAction(values) {
     const first = values.positionals[0];
     const action = (first === undefined ? "inbox" : first);
     if (!MAIL_ACTIONS.has(action)) {
         throw new Error(`Unknown mail command: ${first}. Try accounts, add-imap, detach, inbox, read, search, send, attachment, or sync.`);
     }
-    const rest = values.positionals.slice(first === undefined ? 0 : 1);
+    return { action, rest: values.positionals.slice(first === undefined ? 0 : 1) };
+}
+function readMailSubjectOrQuery(action, rest) {
     let subject;
     let query;
     if (MAIL_ACTIONS_NEEDING_A_SUBJECT.has(action)) {
@@ -96,10 +107,16 @@ export function parseMailArgs(args) {
     else if (rest.length > 0) {
         throw new Error(`mail ${action} does not take "${rest[0]}".`);
     }
+    return { subject, query };
+}
+function readImapAddress(action, values) {
     const address = optionalNonEmpty(values.flags.get("--address"));
     if (action === "add-imap" && !address) {
         throw new Error("mail add-imap needs --address, and reads the app password from stdin.");
     }
+    return address;
+}
+function readMailRecipients(action, values) {
     const to = splitAddresses(values.flags.get("--to"));
     const cc = splitAddresses(values.flags.get("--cc"));
     if (action === "send" && to.length === 0) {
@@ -108,14 +125,23 @@ export function parseMailArgs(args) {
     if (action === "send" && !optionalNonEmpty(values.flags.get("--account"))) {
         throw new Error("mail send needs --account <id>: with several mailboxes attached, which address this goes out from is yours to say. `cockpit mail accounts` lists them.");
     }
+    return { to, cc };
+}
+function readAttachmentPath(action, values) {
     const outPath = optionalNonEmpty(values.flags.get("--out"));
     if (action === "attachment" && !outPath) {
         throw new Error("mail attachment needs --out <path>: the file is written to disk, never to stdout.");
     }
+    return outPath;
+}
+function readMailLimit(action, values) {
     const limit = optionalPositiveInteger(values.flags.get("--limit"), "--limit");
     if (limit !== undefined && action !== "inbox" && action !== "search") {
         throw new Error("--limit belongs to `cockpit mail inbox` or `cockpit mail search`.");
     }
+    return limit;
+}
+function buildMailCommand(values, action, subject, query, address, to, cc, outPath, limit) {
     return {
         kind: "mail",
         action,

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

@@ -28,23 +28,7 @@ export function parseModelsArgs(args) {
     // `compare` is the one verb that takes several ids, so it is parsed before
     // the at-most-two rule the other two live under (BLI-3919).
     if (values.positionals[0] === "compare") {
-        const modelIds = values.positionals.slice(1).map((id) => id.trim()).filter((id) => id.length > 0);
-        const unique = [...new Set(modelIds)];
-        if (unique.length < COMPARE_MIN) {
-            throw new Error(`models compare needs at least ${COMPARE_MIN} model ids, e.g. ` +
-                "`models compare openai:gpt-5.6-luna openai:gpt-5.6-terra`.");
-        }
-        if (unique.length > COMPARE_MAX) {
-            throw new Error(`models compare holds at most ${COMPARE_MAX} models; ${unique.length} named. ` +
-                "A fifth column stops being readable.");
-        }
-        return {
-            kind: "models",
-            action: "compare",
-            modelIds: unique,
-            highlight: values.booleans.has("--highlight"),
-            ...base,
-        };
+        return parseModelComparison(values.positionals, values.booleans.has("--highlight"), base);
     }
     if (values.booleans.has("--highlight")) {
         throw new Error("--highlight only means something for `models compare`.");
@@ -69,4 +53,17 @@ export function parseModelsArgs(args) {
         throw new Error("models show needs a model id, e.g. `models show openai:gpt-5.6-terra`.");
     }
     return { kind: "models", action: "show", modelId, ...base };
+}
+function parseModelComparison(positionals, highlight, base) {
+    const modelIds = positionals.slice(1).map((id) => id.trim()).filter((id) => id.length > 0);
+    const unique = [...new Set(modelIds)];
+    if (unique.length < COMPARE_MIN) {
+        throw new Error(`models compare needs at least ${COMPARE_MIN} model ids, e.g. ` +
+            "`models compare openai:gpt-5.6-luna openai:gpt-5.6-terra`.");
+    }
+    if (unique.length > COMPARE_MAX) {
+        throw new Error(`models compare holds at most ${COMPARE_MAX} models; ${unique.length} named. ` +
+            "A fifth column stops being readable.");
+    }
+    return { kind: "models", action: "compare", modelIds: unique, highlight, ...base };
 }

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

@@ -72,12 +72,25 @@ export function parseIssueArgs(args) {
             "--file",
         ],
     });
+    const { action, rest } = readIssueAction(values);
+    const { issueRef, moveState } = readIssueReference(action, rest);
+    validateIssueState(moveState, "issue move state must be one of: ");
+    const stateFilter = readIssueStateFilter(action, values);
+    const title = readIssueTitle(action, values);
+    const priority = readIssuePriority(values);
+    const assignee = readIssueAssignee(action, values);
+    const limit = readIssueLimit(action, values);
+    return buildIssueCommand(values, action, issueRef, moveState, stateFilter, title, priority, assignee, limit);
+}
+function readIssueAction(values) {
     const first = values.positionals[0];
     const action = (first === undefined ? "list" : first);
     if (!ISSUE_ACTIONS.has(action)) {
         throw new Error(`Unknown issue command: ${first}. Try list, show, create, update, move, comment, or history.`);
     }
-    const rest = values.positionals.slice(first === undefined ? 0 : 1);
+    return { action, rest: values.positionals.slice(first === undefined ? 0 : 1) };
+}
+function readIssueReference(action, rest) {
     let issueRef;
     let moveState;
     if (ISSUE_ACTIONS_NEEDING_AN_ISSUE.has(action)) {
@@ -99,19 +112,28 @@ export function parseIssueArgs(args) {
     else if (rest.length > 0) {
         throw new Error(`issue ${action} does not take "${rest[0]}".`);
     }
-    if (moveState && !ISSUE_STATES.includes(moveState)) {
-        throw new Error(`issue move state must be one of: ${ISSUE_STATES.join(", ")}.`);
+    return { issueRef, moveState };
+}
+function validateIssueState(state, errorPrefix) {
+    if (state && !ISSUE_STATES.includes(state)) {
+        throw new Error(`${errorPrefix}${ISSUE_STATES.join(", ")}.`);
     }
+}
+function readIssueStateFilter(action, values) {
     const stateFilter = optionalNonEmpty(values.flags.get("--state"));
-    if (stateFilter !== undefined && !ISSUE_STATES.includes(stateFilter)) {
-        throw new Error(`--state must be one of: ${ISSUE_STATES.join(", ")}.`);
-    }
+    validateIssueState(stateFilter, "--state must be one of: ");
     if (stateFilter !== undefined && action !== "list") {
         throw new Error("--state filters `cockpit issue list`; to move an issue use `cockpit issue move <id> <state>`.");
     }
+    return stateFilter;
+}
+function readIssueTitle(action, values) {
     const title = optionalNonEmpty(values.flags.get("--title"));
     if (action === "create" && !title)
         throw new Error("issue create needs --title.");
+    return title;
+}
+function readIssuePriority(values) {
     // Not `optionalPositiveInteger`: 0 is a legitimate priority ("none", the
     // column default), and that helper refuses it as non-positive.
     const priorityRaw = optionalNonEmpty(values.flags.get("--priority"));
@@ -122,6 +144,9 @@ export function parseIssueArgs(args) {
             throw new Error("--priority must be 0-4 (0 none, 1 urgent, 2 high, 3 medium, 4 low).");
         }
     }
+    return priority;
+}
+function readIssueAssignee(action, values) {
     // An assignee is `me`, `unassigned`, or a person's uuid. A name or an
     // email is refused HERE rather than travelling to Postgres as a malformed
     // uuid, which would come back as a 500 that names nothing useful.
@@ -132,10 +157,16 @@ export function parseIssueArgs(args) {
     if (assignee === "unassigned" && action !== "list") {
         throw new Error('--assignee unassigned filters `cockpit issue list`; to clear an assignee pass --assignee "" is not supported yet.');
     }
+    return assignee;
+}
+function readIssueLimit(action, values) {
     const limit = optionalPositiveInteger(values.flags.get("--limit"), "--limit");
     if (limit !== undefined && action !== "list" && action !== "history") {
         throw new Error("--limit belongs to `cockpit issue list` or `cockpit issue history`.");
     }
+    return limit;
+}
+function buildIssueCommand(values, action, issueRef, moveState, stateFilter, title, priority, assignee, limit) {
     return {
         kind: "issue",
         action,

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

@@ -22,7 +22,21 @@ const SEARCH_KIND_LIST = SEARCH_KINDS.join(",");
 const SEARCH_CORPORA_COUNT = SEARCH_KINDS.length;
 /** One entry per Tower noun, in the order `cockpit --help` lists them. */
 export const TOWER_COMMAND_HELP = [
-    ["careers", ["Usage: cockpit careers [list|show <id>|rescreen <id>] [--role <slug>] [--min-score <n>] [--since <date>] [--json]", "Super-admin application review. Lists at most 100 matches with total and has_more; use filters to narrow."]],
+    [
+        "careers",
+        [
+            "Usage: cockpit careers [list|show <id>|rescreen <id>|invite <id>|decide <id> --decision <d>|takehome show <role>|takehome set <role>|settings show|settings set] [flags]",
+            "",
+            "Super-admin application review and pipeline. Lists at most 100 matches with total and has_more; use filters to narrow.",
+            "list [--role <slug>] [--min-score <n>] [--since <date>] [--stage <screened|invited|submitted|graded|booking_sent|passed_on|archived>] — the board, newest first, with the two thresholds in force.",
+            "invite <id> — sends that role's take-home from the configured mailbox, whatever the screening scored. Only a row still at `screened`.",
+            "decide <id> --decision <passed_on|archived|reopen> — your own call on a row; `reopen` puts it back to `screened`.",
+            "takehome show <role> — the subject, body, link and the reviewer's rubric for that role.",
+            "takehome set <role> [--subject <s>] [--body-file <path>] [--link <url>] [--pass-score <0-10>] — the body comes from a FILE and must keep {first_name} and {link}.",
+            "settings show — the bar, the attention threshold, whether automatic sending is on, and which mailbox it goes out from.",
+            "settings set [--enabled true|false] [--auto-invite-min-score <0-100>] [--attention-min-score <0-100>] [--account <mailbox uuid>] — each takes effect on the next screening, no deploy.",
+        ],
+    ],
     [
         "docs",
         [

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

@@ -327,12 +327,13 @@ export function localSubcommandHelp(command) {
         [
             "ops",
             [
-                "Usage: cockpit ops [status [--job <id>] [--coverage] [--skips] [--memory [--memory-days N]] [--models] | recompile --person <email|name|id> [--dry-run]] [--json]",
+                "Usage: cockpit ops [status [--job <id>] [--coverage] [--skips] [--memory [--memory-days N]] [--models] [--turns] [--tool-router] | recompile --person <email|name|id> [--dry-run]] [--json]",
                 "",
                 "  cockpit ops status",
                 "      One line per scheduled job: when it last produced something, and whether that",
                 "      is late FOR THAT JOB. A once-daily job quiet for 18 hours reads ok; a",
                 "      quarter-hourly one does not. Every line names the schedule it is judged against.",
+                "      --turns adds 30-day JARVIS turn aggregates; --tool-router adds the shadow-router agreement board.",
                 "      A job that is not ok also prints what its artifact does and does not prove —",
                 "      several only write a row when there is something new, so quiet can mean a quiet",
                 "      week rather than a broken cron.",

package/dist/commands/local-help.js

@@ -104,7 +104,7 @@ export function localCommandHelp(command) {
         "  cockpit project [list] [--archived] [--dashboard-url <url>] [--json]",
         `  cockpit search "<words>" [--kind ${SEARCH_KINDS.join(",")}] [--limit <n>] [--dashboard-url <url>] [--json]`,
         "  cockpit release [--dry-run] [--skip-checks] [--no-floor] [--tag <tag>] [--access <public|restricted>] [--otp <code>]",
-        "  cockpit careers [list|show <id>|rescreen <id>] [--role <slug>] [--min-score <n>] [--since <date>] [--json]",
+        "  cockpit careers [list|show <id>|rescreen <id>|invite <id>|decide <id> --decision <passed_on|archived|reopen>|takehome show <role>|takehome set <role> --body-file <path>|settings show|settings set --enabled <true|false>] [--role <slug>] [--min-score <n>] [--since <date>] [--stage <s>] [--json]",
         "  cockpit usage sessions [--task <BLI-NNNN> | --subject <name>] [--repo <label>] [--since <window>] [--json]",
         "  cockpit usage sessions --session <id> [--from-window <n>] [--json]",
         "  --by-topic groups work types; --by-subject groups open subjects from session summaries.",