@bli-cockpit/cli 0.2.54 → 0.2.56

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

@@ -6,886 +6,19 @@
  *
  * Split out of commands/local-args.ts (BLI-3578) alongside its sibling
  * local-args-collector.ts, which parses the commands that act on THIS machine.
- * The split follows the one the runtime already makes — these are the surfaces
- * that open a Tower session (`tower-command.ts`) — and every parser moved
- * verbatim: same flags, same defaults, same refusal sentences. The three
- * constants a parser publishes (`SCOUT_MIN_PREFIX_LENGTH`,
- * `SLACK_WORKSPACE_KEYS`, `WORKBOOK_MIN_WIDTH`) and the `NotesAction` type are
- * re-exported from `./local-args.js`, the address they have always had.
- */
-import { optionalEmail, optionalNonEmpty, optionalPositiveInteger, optionalUrl, parseNamedArgs, } from "./local-arg-values.js";
-export function parseJarvisArgs(args) {
-    const values = parseNamedArgs(args, {
-        allowedFlags: [
-            "--home",
-            "--dashboard-url",
-            "--prompt",
-            "--as",
-            "--thread",
-            "--model",
-            "--image",
-            "--file",
-            "--no-stream",
-            // BLI-3484: which day's page this turn is about.
-            "--date",
-            // BLI-3458: reading back what was already said, rather than saying
-            // something new. Neither takes a turn or reaches the model.
-            "--threads",
-            "--history",
-            // BLI-3560: reading back what a turn DID — the step tree of one turn.
-            "--trace",
-            "--limit",
-            "--json",
-        ],
-        valueFlags: [
-            "--home",
-            "--dashboard-url",
-            "--prompt",
-            "--as",
-            "--thread",
-            "--model",
-            "--trace",
-            "--image",
-            "--file",
-            "--date",
-            "--limit",
-        ],
-    });
-    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.");
-    }
-    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.");
-    }
-    // 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"));
-    const file = optionalNonEmpty(values.flags.get("--file"));
-    if (image && file) {
-        throw new Error("jarvis accepts either --image or --file, not both — they are the same flag.");
-    }
-    // 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
-    // question.
-    const threads = values.booleans.has("--threads");
-    const history = values.booleans.has("--history");
-    if (threads && history) {
-        throw new Error("jarvis --threads lists every thread; --history replays one. Pass one, not both.");
-    }
-    if ((threads || history) && (flaggedPrompt || positionalPrompt)) {
-        throw new Error("jarvis --threads and --history read back what was already said; they do not take a question.");
-    }
-    if ((threads || history) && image) {
-        throw new Error("jarvis --threads and --history do not take an attachment.");
-    }
-    // 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
-    // and bind no page at all.
-    const date = optionalNonEmpty(values.flags.get("--date"));
-    if ((threads || history) && date) {
-        throw new Error("jarvis --threads and --history replay what was said; they bind no page.");
-    }
-    // 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
-    // is worse than a refusal.
-    const trace = optionalNonEmpty(values.flags.get("--trace"));
-    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)) {
-        throw new Error("jarvis --trace shows a turn that already ran; it does not take a question.");
-    }
-    if (trace && image) {
-        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"),
-    };
-}
-/** The shortest prefix `cockpit scout start` will resolve. Below this, ids collide. */
-export const SCOUT_MIN_PREFIX_LENGTH = 6;
-/**
- * `cockpit scout [--days <n>]` reads the board; `cockpit scout start|dismiss|undo
- * <id>` moves one card. Positional shape modelled on `autostart`: an optional
- * action word first, then what it acts on.
- */
-export function parseScoutArgs(args) {
-    const values = parseNamedArgs(args, {
-        allowedFlags: ["--home", "--dashboard-url", "--days", "--json"],
-        valueFlags: ["--home", "--dashboard-url", "--days"],
-    });
-    if (values.positionals.length > 2) {
-        throw new Error("scout accepts at most an action (start|dismiss|undo) and one experiment id.");
-    }
-    const [rawAction, rawRef] = values.positionals;
-    const base = {
-        homeDir: optionalNonEmpty(values.flags.get("--home")),
-        dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
-        days: optionalPositiveInteger(values.flags.get("--days"), "--days"),
-        json: values.booleans.has("--json"),
-    };
-    if (!rawAction)
-        return { kind: "scout", action: "board", ...base };
-    if (rawAction !== "start" && rawAction !== "dismiss" && rawAction !== "undo") {
-        throw new Error("scout action must be start, dismiss, or undo.");
-    }
-    const experimentRef = optionalNonEmpty(rawRef);
-    if (!experimentRef) {
-        throw new Error(`scout ${rawAction} needs an experiment id from the board.`);
-    }
-    if (experimentRef.length < SCOUT_MIN_PREFIX_LENGTH) {
-        throw new Error(`scout ${rawAction} needs a full experiment id or a prefix of at least ${SCOUT_MIN_PREFIX_LENGTH} characters.`);
-    }
-    return { kind: "scout", action: rawAction, experimentRef, ...base };
-}
-/**
- * `cockpit ops status [--job <id>] [--skips]` and
- * `cockpit ops recompile --person <p> [--dry-run]` (BLI-3462).
- *
- * There is deliberately **no `--cadence`** on `recompile`. The compile path
- * behind it (`writePageAgain` → `publishPage`) takes no cadence and writes a
- * daily; accepting the flag and dropping it is the silent breakage BLI-2490
- * forbids, and threading cadence through that path is its own change. Ask for a
- * 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",
-            "--person",
-            "--dry-run",
-            "--json",
-        ],
-        valueFlags: ["--home", "--dashboard-url", "--job", "--person"],
-    });
-    if (values.positionals.length > 1) {
-        throw new Error("ops accepts one action: status or recompile.");
-    }
-    const rawAction = values.positionals[0] ?? "status";
-    if (rawAction !== "status" && rawAction !== "recompile") {
-        throw new Error("ops action must be status or recompile.");
-    }
-    const base = {
-        homeDir: optionalNonEmpty(values.flags.get("--home")),
-        dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
-        json: values.booleans.has("--json"),
-    };
-    if (rawAction === "status") {
-        return {
-            kind: "ops",
-            action: "status",
-            job: optionalNonEmpty(values.flags.get("--job")),
-            skips: values.booleans.has("--skips"),
-            ...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.");
-    }
-    return {
-        kind: "ops",
-        action: "recompile",
-        person,
-        dryRun: values.booleans.has("--dry-run"),
-        ...base,
-    };
-}
-/** The workspaces Cockpit collects. A typo is refused here rather than searched for. */
-export const SLACK_WORKSPACE_KEYS = ["bli", "blue_pearl"];
-/**
- * `cockpit slack coverage [--workspace <key>] [--stale-only]` and
- * `cockpit slack read [--person|--channel|--query|--since|--until|--limit]`
- * (BLI-3462).
- */
-export function parseSlackArgs(args) {
-    const values = parseNamedArgs(args, {
-        allowedFlags: [
-            "--home",
-            "--dashboard-url",
-            "--workspace",
-            "--stale-only",
-            "--person",
-            "--channel",
-            "--query",
-            "--since",
-            "--until",
-            "--limit",
-            "--json",
-        ],
-        valueFlags: [
-            "--home",
-            "--dashboard-url",
-            "--workspace",
-            "--person",
-            "--channel",
-            "--query",
-            "--since",
-            "--until",
-            "--limit",
-        ],
-    });
-    if (values.positionals.length > 1) {
-        throw new Error("slack accepts one action: coverage or read.");
-    }
-    const rawAction = values.positionals[0] ?? "coverage";
-    if (rawAction !== "coverage" && rawAction !== "read") {
-        throw new Error("slack action must be coverage or read.");
-    }
-    const base = {
-        homeDir: optionalNonEmpty(values.flags.get("--home")),
-        dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
-        json: values.booleans.has("--json"),
-    };
-    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"),
-            ...base,
-        };
-    }
-    const person = optionalNonEmpty(values.flags.get("--person"));
-    const channel = optionalNonEmpty(values.flags.get("--channel"));
-    const query = optionalNonEmpty(values.flags.get("--query"));
-    if (!person && !channel && !query) {
-        throw new Error("slack read needs at least one of --person, --channel or --query. An unfiltered dump of every " +
-            "message is not something this command offers.");
-    }
-    return {
-        kind: "slack",
-        action: "read",
-        person,
-        channel,
-        query,
-        since: optionalNonEmpty(values.flags.get("--since")),
-        until: optionalNonEmpty(values.flags.get("--until")),
-        limit: optionalPositiveInteger(values.flags.get("--limit"), "--limit"),
-        ...base,
-    };
-}
-/** The narrowest width worth wrapping to; below it every line is one word. */
-export const WORKBOOK_MIN_WIDTH = 20;
-/**
- * `cockpit workbook` with nothing lists the library, one positional lists a
- * project's shelf, two read a document.
- */
-export function parseWorkbookArgs(args) {
-    const values = parseNamedArgs(args, {
-        allowedFlags: [
-            "--home",
-            "--dashboard-url",
-            "--section",
-            "--width",
-            "--markdown",
-            "--json",
-        ],
-        valueFlags: ["--home", "--dashboard-url", "--section", "--width"],
-    });
-    if (values.positionals.length > 2) {
-        throw new Error("workbook accepts at most a project and a document.");
-    }
-    const project = optionalNonEmpty(values.positionals[0]);
-    const doc = optionalNonEmpty(values.positionals[1]);
-    const section = optionalNonEmpty(values.flags.get("--section"));
-    const markdown = values.booleans.has("--markdown");
-    if (!doc && (section || markdown)) {
-        throw new Error("workbook --section and --markdown need a project and a document, e.g. `cockpit workbook tower workbook`.");
-    }
-    const width = optionalPositiveInteger(values.flags.get("--width"), "--width");
-    if (width !== undefined && width < WORKBOOK_MIN_WIDTH) {
-        throw new Error(`workbook --width must be at least ${WORKBOOK_MIN_WIDTH}.`);
-    }
-    return {
-        kind: "workbook",
-        project,
-        doc,
-        section,
-        markdown,
-        width,
-        homeDir: optionalNonEmpty(values.flags.get("--home")),
-        dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
-        json: values.booleans.has("--json"),
-    };
-}
-/**
- * `cockpit brief` (BLI-3458) — the TODAY page, in the terminal.
- *
- * `--for` is the terminal's `?p=`, and it follows the same rule the website
- * does: the dashboard resolves it against the roster and the database (or the
- * route's own mirror of the database's rule) decides whether the page opens.
- * Nothing is decided here.
+ * This file itself became a table of contents in BLI-3636, split by COMMAND
+ * FAMILY into three siblings, each moved verbatim (same flags, same defaults,
+ * same refusal sentences):
  *
- * `--date` (BLI-3484) is the terminal's `?d=`, and the same sentence applies to
- * it twice over: what counts as a date, where that person's day starts, and
- * whether anything was compiled for it are all the dashboard's answer. A local
- * copy of "when does Tuesday begin" would be a second answer to a question this
- * product already answers per person, per zone.
- */
-export function parseBriefArgs(args) {
-    const values = parseNamedArgs(args, {
-        allowedFlags: [
-            "--home",
-            "--dashboard-url",
-            "--for",
-            "--as",
-            "--who",
-            "--version",
-            "--date",
-            "--delta",
-            "--against",
-            "--days",
-            "--tldr",
-            "--full",
-            "--versions",
-            "--claims",
-            "--render",
-            "--reason",
-            "--wait",
-            "--no-wait",
-            "--json",
-        ],
-        valueFlags: [
-            "--home",
-            "--dashboard-url",
-            "--for",
-            "--as",
-            "--who",
-            "--version",
-            "--date",
-            "--against",
-            "--days",
-            "--reason",
-        ],
-    });
-    // Bare `cockpit brief` reads the page, which is what somebody typing it almost
-    // always wants — the same shape `cockpit notes` and `cockpit scout` have.
-    // `status` (BLI-3462) answers why a brief was or was not delivered.
-    const first = values.positionals[0];
-    const action = (first === undefined ? "read" : first);
-    if (!["read", "edit", "rewrite", "history", "status"].includes(action)) {
-        throw new Error(`Unknown brief command: ${first}. Try edit, rewrite, history or status, or nothing to read it.`);
-    }
-    if (values.positionals.length > (first === undefined ? 0 : 1)) {
-        throw new Error(`brief ${action} does not take "${values.positionals[1]}".`);
-    }
-    const tldr = values.booleans.has("--tldr");
-    const full = values.booleans.has("--full");
-    if (tldr && full) {
-        throw new Error("brief accepts either --tldr or --full, not both.");
-    }
-    const wait = values.booleans.has("--wait");
-    const noWait = values.booleans.has("--no-wait");
-    if (wait && noWait) {
-        throw new Error("brief rewrite accepts either --wait or --no-wait, not both.");
-    }
-    const reason = optionalNonEmpty(values.flags.get("--reason"));
-    if (reason && reason.length > 280) {
-        // The server's own ceiling (`REASON_MAX`). Said here so the person is told
-        // before the page is opened rather than after they have finished editing.
-        throw new Error("A reason is limited to 280 characters.");
-    }
-    if (action !== "edit" && reason) {
-        throw new Error("--reason belongs to `cockpit brief edit`.");
-    }
-    if (action !== "rewrite" && (wait || noWait)) {
-        throw new Error("--wait and --no-wait belong to `cockpit brief rewrite`.");
-    }
-    // ── Reading a past day (BLI-3484) ────────────────────────────────────────
-    const date = optionalNonEmpty(values.flags.get("--date"));
-    const against = optionalNonEmpty(values.flags.get("--against"));
-    const delta = values.booleans.has("--delta");
-    const days = optionalPositiveInteger(values.flags.get("--days"), "--days");
-    if (date && action !== "read") {
-        throw new Error("--date belongs to `cockpit brief` on its own — it reads one day's page.");
-    }
-    if (delta && action !== "read") {
-        throw new Error("--delta belongs to `cockpit brief` on its own.");
-    }
-    if (against && !delta) {
-        // Said rather than silently ignored: somebody who typed `--against` asked
-        // for a comparison, and running the plain read would answer a different
-        // question without saying so.
-        throw new Error("--against needs --delta: it names the day to compare against.");
-    }
-    if (days !== undefined && action !== "history") {
-        throw new Error("--days belongs to `cockpit brief history`.");
-    }
-    if (action === "history" && (values.booleans.has("--tldr") || values.booleans.has("--claims"))) {
-        throw new Error("brief history lists days; --tldr and --claims belong to reading a page.");
-    }
-    // `--as` is accepted as an alias so the two conversational commands read the
-    // same way; `cockpit jarvis --as <person>` has meant this since BLI-3380.
-    // `--who` is the third spelling, and it exists because `cockpit brief status
-    // --who <person>` is the terminal echo of `jarvis:dm --who`.
-    const forPerson = optionalNonEmpty(values.flags.get("--for"));
-    const asPerson = optionalNonEmpty(values.flags.get("--as"));
-    const whoPerson = optionalNonEmpty(values.flags.get("--who"));
-    const named = [forPerson, asPerson, whoPerson].filter(Boolean);
-    if (new Set(named).size > 1) {
-        throw new Error("brief --for, --as and --who must name the same person.");
-    }
-    const render = values.booleans.has("--render");
-    if (render && action !== "status") {
-        throw new Error("brief --render belongs to `cockpit brief status`; the page is printed by default.");
-    }
-    return {
-        kind: "brief",
-        action,
-        render,
-        homeDir: optionalNonEmpty(values.flags.get("--home")),
-        dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
-        subject: forPerson ?? asPerson ?? whoPerson,
-        version: optionalNonEmpty(values.flags.get("--version")),
-        ...(date ? { date } : {}),
-        ...(delta ? { delta: true } : {}),
-        ...(against ? { against } : {}),
-        ...(days !== undefined ? { days } : {}),
-        tldr,
-        versions: values.booleans.has("--versions"),
-        claims: values.booleans.has("--claims"),
-        ...(reason ? { reason } : {}),
-        // Waiting is the default; only an explicit `--no-wait` turns it off. A
-        // person who typed `rewrite` wants the page, not a receipt.
-        ...(noWait ? { wait: false } : {}),
-        json: values.booleans.has("--json"),
-    };
-}
-/**
- * `cockpit correct` (BLI-3458) — say that one line on the page is wrong.
- *
- * `--claim` is required and comes from `cockpit brief --claims`, which prints
- * the id beside every line it names. A correction with no claim would be a note
- * about the page in general, and the panel has a sentinel for that; the
- * terminal does not offer it, because a person who has just read
- * `[claimId] the sentence` has the id in front of them.
- *
- * `--text` may be omitted when something is piped in — `git log | cockpit
- * correct --claim x` — and the command reads stdin instead. Absent both, the
- * command refuses rather than filing an empty correction.
- */
-export function parseCorrectArgs(args) {
-    const values = parseNamedArgs(args, {
-        allowedFlags: [
-            "--home",
-            "--dashboard-url",
-            "--claim",
-            "--text",
-            "--for",
-            "--as",
-            "--version",
-            "--supersedes",
-            "--json",
-        ],
-        valueFlags: [
-            "--home",
-            "--dashboard-url",
-            "--claim",
-            "--text",
-            "--for",
-            "--as",
-            "--version",
-            "--supersedes",
-        ],
-    });
-    const claimId = optionalNonEmpty(values.flags.get("--claim"));
-    if (!claimId) {
-        throw new Error("correct needs --claim <claimId>. Run `cockpit brief --claims` to see the id beside every line.");
-    }
-    const flaggedText = optionalNonEmpty(values.flags.get("--text"));
-    const positionalText = optionalNonEmpty(values.positionals.join(" "));
-    if (flaggedText && positionalText) {
-        throw new Error("correct accepts either --text or positional text, not both.");
-    }
-    const text = flaggedText ?? positionalText;
-    if (text && text.length > 4000) {
-        throw new Error("A correction is limited to 4000 characters.");
-    }
-    const forPerson = optionalNonEmpty(values.flags.get("--for"));
-    const asPerson = optionalNonEmpty(values.flags.get("--as"));
-    if (forPerson && asPerson && forPerson !== asPerson) {
-        throw new Error("correct --for and --as must name the same person.");
-    }
-    return {
-        kind: "correct",
-        homeDir: optionalNonEmpty(values.flags.get("--home")),
-        dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
-        claimId,
-        text,
-        subject: forPerson ?? asPerson,
-        version: optionalNonEmpty(values.flags.get("--version")),
-        supersedes: optionalNonEmpty(values.flags.get("--supersedes")),
-        json: values.booleans.has("--json"),
-    };
-}
-const NOTES_ACTIONS = new Set([
-    "list",
-    "show",
-    "shelf",
-    "shelves",
-    "upload",
-    "paste",
-    "share",
-    "unshare",
-    "move",
-]);
-/** Actions whose first positional is the note it acts on. */
-const NOTES_ACTIONS_NEEDING_A_NOTE = new Set([
-    "show",
-    "share",
-    "unshare",
-    "move",
-]);
-export function parseNotesArgs(args) {
-    const values = parseNamedArgs(args, {
-        allowedFlags: [
-            "--home",
-            "--dashboard-url",
-            "--file",
-            "--name",
-            "--exclude",
-            "--to",
-            "--clear-shelf",
-            "--series",
-            "--kind",
-            "--since",
-            "--until",
-            "--limit",
-            "--yes",
-            "--json",
-        ],
-        valueFlags: [
-            "--home",
-            "--dashboard-url",
-            "--file",
-            "--name",
-            "--exclude",
-            "--to",
-            "--series",
-            "--kind",
-            "--since",
-            "--until",
-            "--limit",
-        ],
-    });
-    // Bare `cockpit notes` is the list, which is the thing a person typing it
-    // almost always wants — same reasoning as bare `cockpit` running convergence.
-    const first = values.positionals[0];
-    const action = (first === undefined ? "list" : first);
-    if (!NOTES_ACTIONS.has(action)) {
-        throw new Error(`Unknown notes command: ${first}. Try list, show, shelf, shelves, upload, paste, share, unshare, or move.`);
-    }
-    const rest = values.positionals.slice(first === undefined ? 0 : 1);
-    const json = values.booleans.has("--json");
-    // `--json` implies `--yes`: a machine-readable run has nobody to prompt, and
-    // failing it for want of a confirmation flag it cannot see is a worse
-    // surprise than an explicit share it already asked for by name.
-    const yes = values.booleans.has("--yes") || json;
-    let noteId;
-    if (NOTES_ACTIONS_NEEDING_A_NOTE.has(action)) {
-        noteId = optionalNonEmpty(rest[0]);
-        if (!noteId)
-            throw new Error(`notes ${action} needs a note id.`);
-        if (rest.length > 1) {
-            throw new Error(`notes ${action} takes one note id, not ${rest.length}.`);
-        }
-    }
-    let paths;
-    if (action === "upload") {
-        // Explicit paths only. No globbing happens here — a shell that expands
-        // `*.md` hands us the names it found, and a shell that does not would have
-        // this command silently uploading a file literally called `*.md`.
-        paths = rest.filter((value) => value.trim() !== "");
-        if (paths.length === 0)
-            throw new Error("notes upload needs at least one file path.");
-    }
-    else if (action !== "paste" && !NOTES_ACTIONS_NEEDING_A_NOTE.has(action) && rest.length > 0) {
-        throw new Error(`notes ${action} does not take "${rest[0]}".`);
-    }
-    const to = optionalNonEmpty(values.flags.get("--to"));
-    const clearShelf = values.booleans.has("--clear-shelf");
-    if (action === "move") {
-        if (to && clearShelf) {
-            throw new Error("notes move accepts either --to or --clear-shelf, not both.");
-        }
-        if (!to && !clearShelf) {
-            throw new Error('notes move needs --to "<shelf>" or --clear-shelf.');
-        }
-    }
-    const limit = optionalPositiveInteger(values.flags.get("--limit"), "--limit");
-    return {
-        kind: "notes",
-        action,
-        homeDir: optionalNonEmpty(values.flags.get("--home")),
-        dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
-        ...(noteId ? { noteId } : {}),
-        ...(paths ? { paths } : {}),
-        filePath: optionalNonEmpty(values.flags.get("--file")),
-        name: optionalNonEmpty(values.flags.get("--name")),
-        exclude: optionalNonEmpty(values.flags.get("--exclude")),
-        ...(to ? { to } : {}),
-        ...(clearShelf ? { clearShelf } : {}),
-        series: optionalNonEmpty(values.flags.get("--series")),
-        meetingKind: optionalNonEmpty(values.flags.get("--kind")),
-        since: optionalNonEmpty(values.flags.get("--since")),
-        until: optionalNonEmpty(values.flags.get("--until")),
-        ...(limit === undefined ? {} : { limit }),
-        yes,
-        json,
-    };
-}
-const SETTINGS_SECTIONS = ["personal", "switches", "models", "env", "cli-floor"];
-/**
- * `cockpit settings [section] [verb] [args]` (BLI-3461).
+ *   local-args-tower-chat.ts    jarvis, correct   — talking to JARVIS directly
+ *   local-args-tower-pages.ts   brief, notes,     — reading a page instead of
+ *                                workbook            asking a question
+ *   local-args-tower-admin.ts   scout, ops, slack, — managing the account or
+ *                                settings, team,      watching the pipeline
+ *                                model
  *
- * The flag set is the UNION across sections and the combinations are checked
- * afterwards, per section — the same shape `parseDoctorArgs` uses. A flag that
- * belongs to another section is refused by name rather than silently ignored,
- * because a dropped `--project` on an env write is a change a person believes
- * they made.
+ * Every name this module has ever exported is still importable from here.
  */
-export function parseSettingsArgs(args) {
-    const values = parseNamedArgs(args, {
-        allowedFlags: [
-            "--home",
-            "--dashboard-url",
-            "--json",
-            "--chat-model",
-            "--brief-model",
-            "--chat",
-            "--memory",
-            "--project",
-            "--file",
-            "--id",
-            "--content-stdin",
-            "--yes",
-        ],
-        valueFlags: [
-            "--home",
-            "--dashboard-url",
-            "--chat-model",
-            "--brief-model",
-            "--chat",
-            "--memory",
-            "--project",
-            "--file",
-            "--id",
-        ],
-    });
-    const [rawSection, ...rest] = values.positionals;
-    const common = {
-        homeDir: optionalNonEmpty(values.flags.get("--home")),
-        dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
-        yes: values.booleans.has("--yes"),
-        json: values.booleans.has("--json"),
-    };
-    if (!rawSection) {
-        if (rest.length > 0)
-            throw new Error("settings does not accept that argument.");
-        return { kind: "settings", section: "overview", action: "show", ...common };
-    }
-    if (!SETTINGS_SECTIONS.includes(rawSection)) {
-        throw new Error(`settings section must be one of ${SETTINGS_SECTIONS.join(", ")} — or nothing, for all of them.`);
-    }
-    const section = rawSection;
-    if (section === "personal") {
-        if (rest.length > 0)
-            throw new Error("settings personal does not accept positional arguments.");
-        const chatModel = optionalNonEmpty(values.flags.get("--chat-model"));
-        const briefModel = optionalNonEmpty(values.flags.get("--brief-model"));
-        return {
-            kind: "settings",
-            section,
-            action: chatModel || briefModel ? "set" : "show",
-            chatModel,
-            briefModel,
-            ...common,
-        };
-    }
-    if (section === "switches") {
-        if (rest.length === 0) {
-            return { kind: "settings", section, action: "show", ...common };
-        }
-        if (rest[0] !== "set") {
-            throw new Error("settings switches takes no verb, or `set <key> <value>`.");
-        }
-        const [, key, value, ...extra] = rest;
-        if (!key || !value || extra.length > 0) {
-            throw new Error("settings switches set needs exactly a key and a value.");
-        }
-        return { kind: "settings", section, action: "set", switchKey: key, switchValue: value, ...common };
-    }
-    /**
-     * BLI-3557: `settings cli-floor` shows the fleet forced-update floor,
-     * `settings cli-floor <version>` raises it. There is no `set` verb because
-     * there is nothing else to do to a floor, and no flag for the version because
-     * a bare positional is how a release says it: `cockpit settings cli-floor
-     * 0.2.48`. The server refuses a LOWER version; the terminal does not
-     * second-guess that decision locally.
-     */
-    if (section === "cli-floor") {
-        if (rest.length === 0) {
-            return { kind: "settings", section, action: "show", ...common };
-        }
-        const [version, ...extra] = rest;
-        if (!version || extra.length > 0) {
-            throw new Error("settings cli-floor takes no argument, or one version: `cli-floor 0.2.48`.");
-        }
-        return { kind: "settings", section, action: "set", floorVersion: version, ...common };
-    }
-    if (section === "models") {
-        if (rest.length === 0) {
-            return { kind: "settings", section, action: "show", ...common };
-        }
-        if (rest[0] !== "set" || rest.length > 1) {
-            throw new Error("settings models takes no verb, or `set --chat <key>` / `set --memory <id>`.");
-        }
-        const orgChatModel = optionalNonEmpty(values.flags.get("--chat"));
-        const orgMemoryModel = optionalNonEmpty(values.flags.get("--memory"));
-        if (!orgChatModel && !orgMemoryModel) {
-            throw new Error("settings models set needs --chat <key>, --memory <id>, or both.");
-        }
-        return { kind: "settings", section, action: "set", orgChatModel, orgMemoryModel, ...common };
-    }
-    // env
-    const verb = rest[0] ?? "list";
-    if (rest.length > 1)
-        throw new Error("settings env takes one verb: list, set, or delete.");
-    if (verb === "list") {
-        return { kind: "settings", section, action: "list", ...common };
-    }
-    if (verb === "set") {
-        const envProject = optionalNonEmpty(values.flags.get("--project"));
-        const envFile = optionalNonEmpty(values.flags.get("--file"));
-        if (!envProject || !envFile) {
-            throw new Error("settings env set needs --project <project> and --file <file name>.");
-        }
-        if (!values.booleans.has("--content-stdin")) {
-            // Deliberate: there is no `--content <value>` flag and never will be. A
-            // secret on a command line lands in shell history and in every process
-            // listing on the machine.
-            throw new Error("settings env set reads the file contents from stdin: add --content-stdin and pipe the file in.");
-        }
-        return {
-            kind: "settings",
-            section,
-            action: "set",
-            envProject,
-            envFile,
-            contentStdin: true,
-            ...common,
-        };
-    }
-    if (verb === "delete") {
-        const envId = optionalNonEmpty(values.flags.get("--id"));
-        if (!envId)
-            throw new Error("settings env delete needs --id <uuid>.");
-        return { kind: "settings", section, action: "delete", envId, ...common };
-    }
-    throw new Error("settings env takes one verb: list, set, or delete.");
-}
-/** `cockpit team [members|invite <email>|role <userId>]` (BLI-3461). */
-export function parseTeamArgs(args) {
-    const values = parseNamedArgs(args, {
-        allowedFlags: ["--home", "--dashboard-url", "--json", "--role", "--team-id", "--yes"],
-        valueFlags: ["--home", "--dashboard-url", "--role", "--team-id"],
-    });
-    const common = {
-        homeDir: optionalNonEmpty(values.flags.get("--home")),
-        dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
-        teamId: optionalNonEmpty(values.flags.get("--team-id")),
-        yes: values.booleans.has("--yes"),
-        json: values.booleans.has("--json"),
-    };
-    const [verb, subject, ...extra] = values.positionals;
-    if (extra.length > 0)
-        throw new Error("team does not accept that many arguments.");
-    if (!verb || verb === "members") {
-        if (subject)
-            throw new Error("team members does not accept positional arguments.");
-        return { kind: "team", action: "members", ...common };
-    }
-    if (verb === "invite") {
-        if (!subject)
-            throw new Error("team invite needs an email address.");
-        const email = optionalEmail(subject);
-        const role = optionalNonEmpty(values.flags.get("--role"));
-        if (!role)
-            throw new Error("team invite needs --role <role>.");
-        return { kind: "team", action: "invite", email, role, ...common };
-    }
-    if (verb === "role") {
-        if (!subject)
-            throw new Error("team role needs the user id whose role is changing.");
-        const role = optionalNonEmpty(values.flags.get("--role"));
-        if (!role)
-            throw new Error("team role needs --role <role>.");
-        return { kind: "team", action: "role", targetUserId: subject, role, ...common };
-    }
-    throw new Error("team takes one verb: members, invite, or role.");
-}
-/** `cockpit model [show|set <key>]` — the alias surface for the personal chat model. */
-export function parseModelArgs(args) {
-    const values = parseNamedArgs(args, {
-        allowedFlags: ["--home", "--dashboard-url", "--json"],
-        valueFlags: ["--home", "--dashboard-url"],
-    });
-    const [verb, key, ...extra] = values.positionals;
-    if (extra.length > 0)
-        throw new Error("model does not accept that many arguments.");
-    const common = {
-        homeDir: optionalNonEmpty(values.flags.get("--home")),
-        dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
-        json: values.booleans.has("--json"),
-    };
-    if (!verb || verb === "show") {
-        if (key)
-            throw new Error("model show does not accept positional arguments.");
-        return { kind: "model", action: "show", ...common };
-    }
-    if (verb === "set") {
-        if (!key)
-            throw new Error("model set needs a model key, e.g. openai:gpt-5.6-terra.");
-        return { kind: "model", action: "set", modelKey: key, ...common };
-    }
-    throw new Error("model takes one verb: show or set <key>.");
-}
+export { parseJarvisArgs, parseCorrectArgs } from "./local-args-tower-chat.js";
+export { parseBriefArgs, WORKBOOK_MIN_WIDTH, parseWorkbookArgs, parseNotesArgs, } from "./local-args-tower-pages.js";
+export { SCOUT_MIN_PREFIX_LENGTH, parseScoutArgs, parseOpsArgs, SLACK_WORKSPACE_KEYS, parseSlackArgs, parseSettingsArgs, parseTeamArgs, parseModelArgs, } from "./local-args-tower-admin.js";