@bli-cockpit/cli 0.2.236 → 0.2.238

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

@@ -1,7 +1,7 @@
 /**
- * The three commands that read a page instead of asking a question: the daily
- * page (`cockpit brief`), meeting notes (`cockpit notes`), and the document
- * library (`cockpit workbook`). Split out of local-args-tower.ts (BLI-3636).
+ * The commands that read a page instead of asking a question: the daily
+ * page (`cockpit brief`) and the document library (`cockpit workbook`); meeting
+ * notes (`cockpit notes`) live in `local-args-tower-notes.ts`. Split out of local-args-tower.ts (BLI-3636).
  *
  * Every parser moved verbatim: same flags, same defaults, same refusal
  * sentences. The three names `local-args.ts` re-exports from this family
@@ -9,83 +9,109 @@
  * `./local-args-tower.js`, the address they have always had.
  */
 import { optionalNonEmpty, optionalPositiveInteger, optionalUrl, parseNamedArgs, } from "./local-arg-values.js";
-/**
- * `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.
- *
- * `--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.
- */
+const BRIEF_ACTIONS = ["read", "edit", "rewrite", "history", "status", "done", "undone"];
+const BRIEF_ALLOWED_FLAGS = [
+    "--home",
+    "--dashboard-url",
+    "--for",
+    "--as",
+    "--who",
+    "--version",
+    "--date",
+    "--delta",
+    "--against",
+    "--days",
+    "--tldr",
+    "--full",
+    "--versions",
+    "--claims",
+    "--render",
+    "--missed",
+    "--reason",
+    "--wait",
+    "--no-wait",
+    "--json",
+];
+const BRIEF_VALUE_FLAGS = [
+    "--home",
+    "--dashboard-url",
+    "--for",
+    "--as",
+    "--who",
+    "--version",
+    "--date",
+    "--against",
+    "--days",
+    "--reason",
+];
 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",
-            "--missed",
-            "--reason",
-            "--wait",
-            "--no-wait",
-            "--json",
-        ],
-        valueFlags: [
-            "--home",
-            "--dashboard-url",
-            "--for",
-            "--as",
-            "--who",
-            "--version",
-            "--date",
-            "--against",
-            "--days",
-            "--reason",
-        ],
-    });
+    const values = parseNamedArgs(args, { allowedFlags: BRIEF_ALLOWED_FLAGS, valueFlags: BRIEF_VALUE_FLAGS });
+    const action = readBriefAction(values.positionals[0]);
+    const item = readBriefItem(action, values.positionals);
+    const tldr = readBriefTldr(values);
+    const { reason, noWait } = readBriefRewriteAndReasonFlags(values, action);
+    const { date, against, delta, days } = readBriefPastDayFlags(values, action);
+    const people = readBriefPeople(values);
+    const { render, missed } = readBriefStatusFlags(values, action, people.named);
+    return {
+        kind: "brief",
+        action,
+        ...(item ? { item } : {}),
+        render,
+        missed,
+        homeDir: optionalNonEmpty(values.flags.get("--home")),
+        dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
+        subject: people.forPerson ?? people.asPerson ?? people.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"),
+    };
+}
+function readBriefAction(first) {
     // 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", "done", "undone"].includes(action)) {
+    if (!BRIEF_ACTIONS.includes(action)) {
         throw new Error(`Unknown brief command: ${first}. Try done, undone, edit, rewrite, history or status, or nothing to read it.`);
     }
+    return action;
+}
+/** The item `done`/`undone` tick off, and a refusal for any positional the verb does not take. */
+function readBriefItem(action, positionals) {
     // BLI-3605: `done` and `undone` are the only brief verbs that take a
     // positional, and they REQUIRE one, "which item" is the whole command, and
     // a bare `cockpit brief done` that guessed at the top row would tick off
     // somebody's work for them.
     const ticksAnItem = action === "done" || action === "undone";
-    const item = ticksAnItem ? values.positionals[1] : undefined;
+    const item = ticksAnItem ? positionals[1] : undefined;
     if (ticksAnItem && !item) {
         throw new Error(`brief ${action} needs the item: a ticket like BLI-3511, or a claim id from \`cockpit brief --claims\`.`);
     }
-    const positionalsAllowed = first === undefined ? 0 : ticksAnItem ? 2 : 1;
-    if (values.positionals.length > positionalsAllowed) {
-        throw new Error(`brief ${action} does not take "${values.positionals[positionalsAllowed]}".`);
+    const positionalsAllowed = positionals[0] === undefined ? 0 : ticksAnItem ? 2 : 1;
+    if (positionals.length > positionalsAllowed) {
+        throw new Error(`brief ${action} does not take "${positionals[positionalsAllowed]}".`);
     }
+    return item;
+}
+function readBriefTldr(values) {
     const tldr = values.booleans.has("--tldr");
-    const full = values.booleans.has("--full");
-    if (tldr && full) {
+    if (tldr && values.booleans.has("--full")) {
         throw new Error("brief accepts either --tldr or --full, not both.");
     }
+    return tldr;
+}
+function readBriefRewriteAndReasonFlags(values, action) {
     const wait = values.booleans.has("--wait");
     const noWait = values.booleans.has("--no-wait");
     if (wait && noWait) {
@@ -103,7 +129,10 @@ export function parseBriefArgs(args) {
     if (action !== "rewrite" && (wait || noWait)) {
         throw new Error("--wait and --no-wait belong to `cockpit brief rewrite`.");
     }
-    // ── Reading a past day (BLI-3484) ────────────────────────────────────────
+    return { reason, noWait };
+}
+/** Reading a past day (BLI-3484). */
+function readBriefPastDayFlags(values, action) {
     const date = optionalNonEmpty(values.flags.get("--date"));
     const against = optionalNonEmpty(values.flags.get("--against"));
     const delta = values.booleans.has("--delta");
@@ -126,6 +155,9 @@ export function parseBriefArgs(args) {
     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.");
     }
+    return { date, against, delta, days };
+}
+function readBriefPeople(values) {
     // `--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
@@ -137,6 +169,9 @@ export function parseBriefArgs(args) {
     if (new Set(named).size > 1) {
         throw new Error("brief --for, --as and --who must name the same person.");
     }
+    return { forPerson, asPerson, whoPerson, named };
+}
+function readBriefStatusFlags(values, action, named) {
     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.");
@@ -148,323 +183,10 @@ export function parseBriefArgs(args) {
     if (missed && named.length > 0) {
         throw new Error("brief status --missed answers for everybody; it does not take --for, --as or --who.");
     }
-    return {
-        kind: "brief",
-        action,
-        ...(item ? { item } : {}),
-        render,
-        missed,
-        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"),
-    };
-}
-/**
- * The notetakers `cockpit notes connect` will actually seal a key for
- * (BLI-4391). It mirrors `CONNECTABLE_NOTES_PROVIDERS` in the dashboard, and
- * the door is still the authority — this list only saves a round trip and a
- * sealed key for a provider with no reader.
- *
- * `otter` and `granola` are deliberately absent: Otter's public API is
- * Enterprise-only and Granola still runs on its own server-side cron.
- */
-const CONNECTABLE_PROVIDERS = ["fellow", "circleback", "wisprflow"];
-/**
- * BLI-12926. Providers with no API key at all: the person signs in through
- * their browser and `connect` runs that sign-in itself. `--key-stdin` is
- * optional for these, and carries a pre-built sign-in bundle when given.
- */
-const PROVIDERS_SIGNED_IN_BY_BROWSER = ["wisprflow"];
-/** The provider whose API is per workspace, so its account needs a subdomain. */
-const PROVIDERS_NEEDING_A_WORKSPACE = ["fellow"];
-const NOTES_ACTIONS = new Set([
-    "folders", "mkdir", "rmdir", "rename",
-    "list",
-    "show",
-    "shelf",
-    "shelves",
-    "upload",
-    "paste",
-    "share",
-    "unshare",
-    "summarize",
-    "move",
-    "place",
-    // BLI-4383: the notetaker a person connects, and the meetings it records.
-    "accounts",
-    "connect",
-    "detach",
-    "sync",
-    "sync-status",
-    // BLI-12926: the alternates of a meeting several notetakers recorded.
-    "recordings",
-    "recording",
-    // BLI-4394: arm that notetaker's webhook with the vendor's signing secret.
-    "webhook",
-    // Ticket 10718: the Granola cron's own last-several-runs health.
-    "granola-status",
-    "transcription-stats",
-    "bot-upcoming",
-]);
-/** Actions whose first positional is the note it acts on. */
-const NOTES_ACTIONS_NEEDING_A_NOTE = new Set([
-    "show",
-    "share",
-    "unshare",
-    "summarize",
-    "move",
-    "place",
-    // BLI-12926: `recordings` names the note, `recording` names the recording.
-    "recordings",
-    "recording",
-]);
-/**
- * BLI-4383. Actions whose first positional is a NOTETAKER ACCOUNT id, not a
- * note id. `sync` takes one too, and is handled beside these rather than in
- * this set because its id is checked at the door instead of here.
- */
-const NOTES_ACTIONS_NEEDING_AN_ACCOUNT = new Set(["detach", "webhook"]);
-export function parseNotesArgs(args) {
-    const values = parseNamedArgs(args, {
-        allowedFlags: [
-            "--wait",
-            "--home",
-            "--dashboard-url",
-            "--file",
-            "--name",
-            "--exclude",
-            "--to",
-            "--folder",
-            "--clear-shelf",
-            "--apply",
-            "--series",
-            "--kind",
-            "--since",
-            "--until",
-            "--limit",
-            "--month",
-            "--offset",
-            "--yes",
-            "--json",
-            "--tree",
-            // BLI-4383. `--key-stdin` is a FLAG and not a value: the key itself is
-            // read from stdin and never from argv.
-            "--provider",
-            "--key-stdin",
-            // BLI-4391. Fellow's API is per WORKSPACE, and a subdomain is a label,
-            // not a secret, so this one carries a value while the key never does.
-            "--workspace",
-            // BLI-4394. `--secret-stdin` is a FLAG and not a value, for the same
-            // reason `--key-stdin` is: the signing secret never touches argv.
-            "--secret-stdin",
-            "--show-webhook",
-            // BLI-18006. `sync <id> --repair-truncated` re-reads Wispr Flow notes whose
-            // transcript was stored cut short. A FLAG, not a verb: it is the same sync
-            // door with a different request.
-            "--repair-truncated",
-        ],
-        valueFlags: [
-            "--home",
-            "--dashboard-url",
-            "--file",
-            "--name",
-            "--exclude",
-            "--to",
-            "--folder",
-            "--series",
-            "--kind",
-            "--since",
-            "--until",
-            "--limit",
-            "--month",
-            "--offset",
-            "--provider",
-            "--workspace",
-            "--show-webhook",
-        ],
-    });
-    // 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, summarize, move, place, accounts, connect, detach, sync, sync-status, recordings, recording, webhook, granola-status, or transcription-stats.`);
-    }
-    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 accountId;
-    if (NOTES_ACTIONS_NEEDING_AN_ACCOUNT.has(action) || action === "sync" || action === "sync-status") {
-        accountId = optionalNonEmpty(rest[0]);
-        if (!accountId && NOTES_ACTIONS_NEEDING_AN_ACCOUNT.has(action)) {
-            throw new Error(`notes ${action} needs a notetaker account id.`);
-        }
-        if (rest.length > 1) {
-            throw new Error(`notes ${action} takes one account id, not ${rest.length}.`);
-        }
-    }
-    // BLI-4394. `--show-webhook <id>` is a flag on `accounts` rather than a verb
-    // of its own, because "what is this one's webhook URL?" is the question a
-    // person asks WHILE looking at the list.
-    const showWebhook = optionalNonEmpty(values.flags.get("--show-webhook"));
-    if (showWebhook !== undefined) {
-        if (action !== "accounts") {
-            throw new Error("--show-webhook belongs to `cockpit notes accounts --show-webhook <account-id>`.");
-        }
-        accountId = showWebhook;
-    }
-    if (action === "webhook") {
-        if (!values.booleans.has("--secret-stdin")) {
-            // Naming the flag is the point: it says out loud that the signing secret
-            // is not an argument, so nobody goes looking for a `--secret` that will
-            // never exist.
-            throw new Error('notes webhook reads the signing secret from stdin. Say so: printf "%s" "<secret>" | cockpit notes webhook <account-id> --secret-stdin');
-        }
-    }
-    else if (values.booleans.has("--secret-stdin")) {
-        throw new Error("--secret-stdin belongs to `cockpit notes webhook <account-id> --secret-stdin`.");
-    }
-    const provider = optionalNonEmpty(values.flags.get("--provider"));
-    const workspace = optionalNonEmpty(values.flags.get("--workspace"));
-    if (action === "connect") {
-        if (!provider) {
-            throw new Error(`notes connect needs --provider ${CONNECTABLE_PROVIDERS.join("|")}.`);
-        }
-        if (!CONNECTABLE_PROVIDERS.includes(provider)) {
-            throw new Error(`notes connect does not know the provider "${provider}". Tower reads ${CONNECTABLE_PROVIDERS.join(" and ")}.`);
-        }
-        if (PROVIDERS_NEEDING_A_WORKSPACE.includes(provider) && !workspace) {
-            throw new Error(`notes connect --provider ${provider} needs --workspace <subdomain>: its API lives at https://<subdomain>.fellow.app, so a key alone cannot address anything.`);
-        }
-        if (workspace !== undefined && !/^[a-z0-9-]{1,63}$/.test(workspace)) {
-            throw new Error("--workspace is a workspace subdomain: lowercase letters, digits and hyphens, 1 to 63 characters.");
-        }
-        if (!values.booleans.has("--key-stdin") && !PROVIDERS_SIGNED_IN_BY_BROWSER.includes(provider)) {
-            // Naming the flag is the point: it says out loud that the key is not an
-            // argument, so nobody goes looking for a `--key` that will never exist.
-            throw new Error('notes connect reads the key from stdin. Say so: printf "%s" "<key>" | cockpit notes connect --provider fellow --workspace <subdomain> --key-stdin');
-        }
-        if (rest.length > 0)
-            throw new Error(`notes connect does not take "${rest[0]}".`);
-    }
-    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 === "mkdir" || action === "rmdir" || action === "rename") {
-        if (rest.length !== 1 || !rest[0]?.trim())
-            throw new Error(`notes ${action} needs one folder path.`);
-    }
-    else if (action !== "paste"
-        && action !== "connect"
-        && action !== "sync"
-        && action !== "sync-status"
-        && !NOTES_ACTIONS_NEEDING_A_NOTE.has(action)
-        && !NOTES_ACTIONS_NEEDING_AN_ACCOUNT.has(action)
-        && rest.length > 0) {
-        throw new Error(`notes ${action} does not take "${rest[0]}".`);
-    }
-    if (action === "rename" && !optionalNonEmpty(values.flags.get("--name")))
-        throw new Error("notes rename needs --name <name>.");
-    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 && !values.flags.get("--folder")) {
-            throw new Error('notes move needs --to "<shelf>", --clear-shelf, or --folder "<path>".');
-        }
-    }
-    const repairTruncated = values.booleans.has("--repair-truncated");
-    if (repairTruncated && action !== "sync") {
-        throw new Error("--repair-truncated belongs to `cockpit notes sync <account-id> --repair-truncated`.");
-    }
-    const limit = optionalPositiveInteger(values.flags.get("--limit"), "--limit");
-    const month = optionalNonEmpty(values.flags.get("--month"));
-    if (month !== undefined) {
-        if (action !== "transcription-stats") {
-            throw new Error("--month belongs to `cockpit notes transcription-stats --month YYYY-MM`.");
-        }
-        if (!/^\d{4}-(0[1-9]|1[0-2])$/.test(month)) {
-            throw new Error("--month is a calendar month like 2026-10.");
-        }
-    }
-    // BLI-18006: a long note is read in parts, like `cockpit docs read --offset`.
-    const offsetRaw = optionalNonEmpty(values.flags.get("--offset"));
-    let offset;
-    if (offsetRaw !== undefined) {
-        if (action !== "show")
-            throw new Error("--offset belongs to `cockpit notes show <id> --offset <characters> --limit <characters>`.");
-        offset = Number(offsetRaw);
-        if (!Number.isInteger(offset) || offset < 0)
-            throw new Error("--offset must be a whole number 0 or greater.");
-    }
-    return {
-        kind: "notes",
-        ...(values.booleans.has("--wait") ? { wait: true } : {}),
-        action,
-        ...(values.flags.has("--folder") ? { folder: values.flags.get("--folder") } : {}),
-        ...(action === "mkdir" || action === "rmdir" || action === "rename" ? { folder: rest[0] } : {}),
-        ...(values.booleans.has("--tree") ? { tree: true } : {}),
-        homeDir: optionalNonEmpty(values.flags.get("--home")),
-        dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
-        ...(noteId ? { noteId } : {}),
-        ...(accountId ? { accountId } : {}),
-        ...(provider ? { provider } : {}),
-        ...(workspace ? { workspace } : {}),
-        ...(showWebhook ? { showWebhook: true } : {}),
-        ...(action === "connect" && values.booleans.has("--key-stdin") ? { keyStdin: true } : {}),
-        ...(paths ? { paths } : {}),
-        filePath: optionalNonEmpty(values.flags.get("--file")),
-        name: optionalNonEmpty(values.flags.get("--name")),
-        exclude: optionalNonEmpty(values.flags.get("--exclude")),
-        ...(to ? { to } : {}),
-        ...(clearShelf ? { clearShelf } : {}),
-        // BLI-4058. `place` says where a note belongs; only `--apply` moves it.
-        ...(values.booleans.has("--apply") ? { apply: true } : {}),
-        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 }),
-        ...(month ? { month } : {}),
-        ...(offset === undefined ? {} : { offset }),
-        ...(repairTruncated ? { repairTruncated: true } : {}),
-        yes,
-        json,
-    };
+    return { render, missed };
 }
+// Meeting notes have their own argument table; these names keep the address they have always had.
+export { parseNotesArgs } from "./local-args-tower-notes.js";
 /** The narrowest width worth wrapping to; below it every line is one word. */
 export const WORKBOOK_MIN_WIDTH = 20;
 /**

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

@@ -13,37 +13,36 @@
  */
 import { optionalNonEmpty, optionalUrl, parseNamedArgs } from "./local-arg-values.js";
 const SETTINGS_SECTIONS = ["personal", "switches", "models", "env", "cli-floor"];
+const SETTINGS_ALLOWED_FLAGS = [
+    "--home",
+    "--dashboard-url",
+    "--json",
+    "--chat-model",
+    "--brief-model",
+    "--chat",
+    "--memory",
+    "--note-reader",
+    "--project",
+    "--file",
+    "--id",
+    "--content-stdin",
+    "--yes",
+];
+const SETTINGS_VALUE_FLAGS = [
+    "--home",
+    "--dashboard-url",
+    "--chat-model",
+    "--brief-model",
+    "--chat",
+    "--memory",
+    "--note-reader",
+    "--project",
+    "--file",
+    "--id",
+];
 /** `cockpit settings [section] [verb] [args]`: the section picker. */
 export function parseSettingsArgs(args) {
-    const values = parseNamedArgs(args, {
-        allowedFlags: [
-            "--home",
-            "--dashboard-url",
-            "--json",
-            "--chat-model",
-            "--brief-model",
-            "--chat",
-            "--memory",
-            "--note-reader",
-            "--project",
-            "--file",
-            "--id",
-            "--content-stdin",
-            "--yes",
-        ],
-        valueFlags: [
-            "--home",
-            "--dashboard-url",
-            "--chat-model",
-            "--brief-model",
-            "--chat",
-            "--memory",
-            "--note-reader",
-            "--project",
-            "--file",
-            "--id",
-        ],
-    });
+    const values = parseNamedArgs(args, { allowedFlags: SETTINGS_ALLOWED_FLAGS, valueFlags: SETTINGS_VALUE_FLAGS });
     const [rawSection, ...rest] = values.positionals;
     const common = {
         homeDir: optionalNonEmpty(values.flags.get("--home")),