@bli-cockpit/cli 0.2.236 → 0.2.238

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

@@ -20,6 +20,7 @@
  */
 import { optionalNonEmpty, optionalPositiveInteger, optionalUrl, parseNamedArgs, } from "./local-arg-values.js";
 const CAL_ACTIONS = new Set([
+    "event", "export", "conflicts", "calendar-change",
     "today",
     "week",
     "next",
@@ -35,56 +36,59 @@ const CAL_ACTIONS = new Set([
     "delete",
 ]);
 /** Every verb that names ONE thing first — a calendar id, or (delete) an event id. */
-const CAL_ACTIONS_NEEDING_A_SUBJECT = new Set(["sync", "detach", "share", "delete", "change", "cancel"]);
+const CAL_ACTIONS_NEEDING_A_SUBJECT = new Set(["event", "export", "calendar-change", "sync", "detach", "share", "delete", "change", "cancel"]);
 /** `sync-status [<calendar id>]` — every calendar, or one's. Ticket 10730. */
 const CAL_ACTIONS_WITH_OPTIONAL_SUBJECT = new Set(["sync-status"]);
+const CAL_ALLOWED_FLAGS = [
+    "--home",
+    "--dashboard-url",
+    "--color",
+    "--calendar",
+    "--account",
+    "--tz",
+    "--occurrence-start",
+    "--offset",
+    "--hours",
+    "--from",
+    "--to",
+    "--limit",
+    "--all",
+    "--name",
+    "--title",
+    "--at",
+    "--until",
+    "--location",
+    "--attendee",
+    "--all-day",
+    "--meet",
+    "--org-visible",
+    "--private",
+    "--full",
+    "--yes",
+    "--json",
+];
+const CAL_VALUE_FLAGS = [
+    "--home",
+    "--dashboard-url",
+    "--color",
+    "--calendar",
+    "--account",
+    "--tz",
+    "--occurrence-start",
+    "--offset",
+    "--hours",
+    "--from",
+    "--to",
+    "--limit",
+    "--name",
+    "--title",
+    "--at",
+    "--until",
+    "--location",
+    "--attendee",
+];
 export function parseCalArgs(args) {
-    const values = parseNamedArgs(args, {
-        allowedFlags: [
-            "--home",
-            "--dashboard-url",
-            "--calendar",
-            "--account",
-            "--tz",
-            "--offset",
-            "--hours",
-            "--from",
-            "--to",
-            "--limit",
-            "--all",
-            "--name",
-            "--title",
-            "--at",
-            "--until",
-            "--location",
-            "--attendee",
-            "--all-day",
-            "--meet",
-            "--org-visible",
-            "--private",
-            "--full",
-            "--yes",
-            "--json",
-        ],
-        valueFlags: [
-            "--home",
-            "--dashboard-url",
-            "--calendar",
-            "--account",
-            "--tz",
-            "--offset",
-            "--hours",
-            "--from",
-            "--to",
-            "--limit",
-            "--name",
-            "--title",
-            "--at",
-            "--until",
-            "--location",
-            "--attendee",
-        ],
-    });
+    const values = parseNamedArgs(args, { allowedFlags: CAL_ALLOWED_FLAGS, valueFlags: CAL_VALUE_FLAGS });
     const { action, rest } = readCalAction(values);
     const { subject, query } = readCalSubjectOrQuery(action, rest);
     const { title, startsAt, endsAt } = readCalCreateDetails(action, values);
@@ -100,7 +104,7 @@ function readCalAction(values) {
     const first = values.positionals[0];
     const action = (first === undefined ? "today" : first);
     if (!CAL_ACTIONS.has(action)) {
-        throw new Error(`Unknown cal command: ${first}. Try today, week, next, find, calendars, add-ical, create, share, detach, sync, sync-status, or delete.`);
+        throw new Error(`Unknown cal command: ${first}. Try today, week, conflicts, calendar-change, next, find, calendars, add-ical, create, share, detach, sync, sync-status, or delete.`);
     }
     return { action, rest: values.positionals.slice(first === undefined ? 0 : 1) };
 }
@@ -170,6 +174,8 @@ function readCalOffset(values) {
 function buildCalCommand(values, action, subject, query, title, startsAt, endsAt, offset, limit) {
     return {
         kind: "cal",
+        occurrenceStart: optionalNonEmpty(values.flags.get("--occurrence-start")),
+        color: optionalNonEmpty(values.flags.get("--color")),
         action,
         homeDir: optionalNonEmpty(values.flags.get("--home")),
         dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),

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

@@ -9,11 +9,56 @@ const NEEDS_ROLE = ['takehome-show', 'takehome-set'];
 export function parseCareersArgs(args) {
     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', '--booking-link', '--grader-model', '--out', '--record-sent-at', '--at', '--followup-enabled', '--followup-after-days', '--takehome-window-hours'];
     const values = parseNamedArgs(args, { allowedFlags: [...valueFlags, '--json', '--now', '--force', '--clear'], 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 { verb, target } = readCareersVerb([...values.positionals]);
+    const rawScore = values.flags.get('--min-score');
+    const minScore = rawScore === undefined ? undefined : Number(rawScore);
+    if (minScore !== undefined && (!Number.isFinite(minScore) || minScore < 0 || minScore > 100))
+        throw new Error('--min-score must be 0 to 100.');
+    const since = optionalNonEmpty(values.flags.get('--since'));
+    if (since && !Number.isFinite(Date.parse(since)))
+        throw new Error('--since must be a date.');
+    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 = trueOrFalseFlag(values.flags.get('--enabled'), '--enabled');
+    const recordSentAt = readRecordSentAt(values, verb);
+    const { interviewAt, clear } = readInterviewFlags(values, verb);
+    const followupEnabledRaw = trueOrFalseFlag(values.flags.get('--followup-enabled'), '--followup-enabled');
+    const followupAfterDays = wholeFlag(values.flags.get('--followup-after-days'), '--followup-after-days', 1, 60);
+    const takehomeWindowHours = wholeFlag(values.flags.get('--takehome-window-hours'), '--takehome-window-hours', 1, 720);
+    const { outPath, force } = readResumeFileFlags(values, verb);
+    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')),
+        bookingLink: httpsFlag(values.flags.get('--booking-link')),
+        graderModel: optionalNonEmpty(values.flags.get('--grader-model')),
+        now: values.booleans.has('--now'),
+        outPath, force,
+        interviewAt: interviewAt ? new Date(interviewAt).toISOString() : undefined, clear: clear || undefined,
+        recordSentAt: recordSentAt ? new Date(recordSentAt).toISOString() : undefined,
+        followupEnabled: followupEnabledRaw === undefined ? undefined : followupEnabledRaw === 'true',
+        followupAfterDays, takehomeWindowHours,
+        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'),
+    };
+}
+/**
+ * `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.
+ */
+function readCareersVerb(positionals) {
     const first = positionals[0] ?? 'list';
     const paired = first === 'takehome' || first === 'settings';
     const typed = paired
@@ -32,32 +77,28 @@ export function parseCareersArgs(args) {
         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);
-    if (minScore !== undefined && (!Number.isFinite(minScore) || minScore < 0 || minScore > 100))
-        throw new Error('--min-score must be 0 to 100.');
-    const since = optionalNonEmpty(values.flags.get('--since'));
-    if (since && !Number.isFinite(Date.parse(since)))
-        throw new Error('--since must be a date.');
-    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.');
-    // BLI-12183: the second-chance follow-up. `--record-sent-at` records one sent
-    // outside Tower and sends nothing; the three settings flags are whole numbers
-    // and a boolean, refused here as well as on the door.
+    return { typed, verb, target };
+}
+function trueOrFalseFlag(raw, flag) {
+    const value = optionalNonEmpty(raw);
+    if (value !== undefined && value !== 'true' && value !== 'false')
+        throw new Error(`${flag} must be true or false.`);
+    return value;
+}
+// BLI-12183: the second-chance follow-up. `--record-sent-at` records one sent
+// outside Tower and sends nothing; the three settings flags are whole numbers
+// and a boolean, refused here as well as on the door.
+function readRecordSentAt(values, verb) {
     const recordSentAt = optionalNonEmpty(values.flags.get('--record-sent-at'));
     if (recordSentAt !== undefined && verb !== 'followup')
         throw new Error('--record-sent-at belongs to `cockpit careers followup <id>`.');
     if (recordSentAt !== undefined && !Number.isFinite(Date.parse(recordSentAt)))
         throw new Error('--record-sent-at must be an ISO date-time, e.g. 2026-10-03T08:00:00Z.');
-    // BLI-12198: an interview Edward booked himself. Exactly one of `--at <iso>`
-    // (book or reschedule) or `--clear`, refused here as well as on the door.
+    return recordSentAt;
+}
+// BLI-12198: an interview Edward booked himself. Exactly one of `--at <iso>`
+// (book or reschedule) or `--clear`, refused here as well as on the door.
+function readInterviewFlags(values, verb) {
     const interviewAt = optionalNonEmpty(values.flags.get('--at'));
     const clear = values.booleans.has('--clear');
     if (interviewAt !== undefined && verb !== 'interview')
@@ -71,13 +112,11 @@ export function parseCareersArgs(args) {
             throw new Error('--at must be an ISO date-time with an offset, e.g. 2026-10-05T10:00:00-07:00.');
         }
     }
-    const followupEnabledRaw = optionalNonEmpty(values.flags.get('--followup-enabled'));
-    if (followupEnabledRaw !== undefined && followupEnabledRaw !== 'true' && followupEnabledRaw !== 'false')
-        throw new Error('--followup-enabled must be true or false.');
-    const followupAfterDays = wholeFlag(values.flags.get('--followup-after-days'), '--followup-after-days', 1, 60);
-    const takehomeWindowHours = wholeFlag(values.flags.get('--takehome-window-hours'), '--takehome-window-hours', 1, 720);
-    // The resume is the only thing this noun writes to disk, so both file flags
-    // belong to it alone and say so rather than being accepted and ignored.
+    return { interviewAt, clear };
+}
+// The resume is the only thing this noun writes to disk, so both file flags
+// belong to it alone and say so rather than being accepted and ignored.
+function readResumeFileFlags(values, verb) {
     const outPath = optionalNonEmpty(values.flags.get('--out'));
     const force = values.booleans.has('--force');
     if (outPath !== undefined && verb !== 'resume')
@@ -86,27 +125,7 @@ export function parseCareersArgs(args) {
         throw new Error('--force belongs to `cockpit careers resume <id> --out <path>`.');
     if (force && outPath === undefined)
         throw new Error('--force only means something beside --out <path>: with no file to write there is nothing to overwrite.');
-    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')),
-        bookingLink: httpsFlag(values.flags.get('--booking-link')),
-        graderModel: optionalNonEmpty(values.flags.get('--grader-model')),
-        now: values.booleans.has('--now'),
-        outPath, force,
-        interviewAt: interviewAt ? new Date(interviewAt).toISOString() : undefined, clear: clear || undefined,
-        recordSentAt: recordSentAt ? new Date(recordSentAt).toISOString() : undefined,
-        followupEnabled: followupEnabledRaw === undefined ? undefined : followupEnabledRaw === 'true',
-        followupAfterDays, takehomeWindowHours,
-        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'),
-    };
+    return { outPath, force };
 }
 /**
  * BLI-4461: a booking link goes into an email a stranger clicks, so the

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

@@ -15,44 +15,43 @@ const DOCS_ACTIONS = new Set(["list", "tree", "read", "create", "update", "delet
 // (the same door, called once per ref), so it joined `read`'s branch rather
 // than this set.
 const DOCS_ACTIONS_NEEDING_A_DOC = new Set(["update", "link", "export"]);
+const DOCS_ALLOWED_FLAGS = [
+    "--home",
+    "--dashboard-url",
+    "--title",
+    "--parent",
+    "--clear-parent",
+    "--allow-empty",
+    "--visibility",
+    "--file",
+    "--body-stdin",
+    "--query",
+    "--word",
+    "--limit",
+    "--clear",
+    "--bodies",
+    "--section",
+    "--offset",
+    "--format",
+    "--out",
+    "--json",
+];
+const DOCS_VALUE_FLAGS = [
+    "--home",
+    "--dashboard-url",
+    "--title",
+    "--parent",
+    "--visibility",
+    "--file",
+    "--query",
+    "--limit",
+    "--section",
+    "--offset",
+    "--format",
+    "--out",
+];
 export function parseDocsArgs(args) {
-    const values = parseNamedArgs(args, {
-        allowedFlags: [
-            "--home",
-            "--dashboard-url",
-            "--title",
-            "--parent",
-            "--clear-parent",
-            "--allow-empty",
-            "--visibility",
-            "--file",
-            "--body-stdin",
-            "--query",
-            "--word",
-            "--limit",
-            "--clear",
-            "--bodies",
-            "--section",
-            "--offset",
-            "--format",
-            "--out",
-            "--json",
-        ],
-        valueFlags: [
-            "--home",
-            "--dashboard-url",
-            "--title",
-            "--parent",
-            "--visibility",
-            "--file",
-            "--query",
-            "--limit",
-            "--section",
-            "--offset",
-            "--format",
-            "--out",
-        ],
-    });
+    const values = parseNamedArgs(args, { allowedFlags: DOCS_ALLOWED_FLAGS, valueFlags: DOCS_VALUE_FLAGS });
     const { action, rest } = parseDocsAction(values.positionals);
     const link = parseDocsLink(action, rest, values.booleans.has("--clear"));
     const { docRef, docRefs } = parseDocsDocumentReference(action, link.rest);
@@ -150,6 +149,10 @@ function parseDocsDocumentReference(action, rest) {
     return { docRef: undefined, docRefs: undefined };
 }
 function parseDocsFlags(action, values) {
+    return { ...parseDocsWriteFlags(action, values), ...parseDocsReadFlags(action, values) };
+}
+/** The flags that decide where a document lives and what it is called. */
+function parseDocsWriteFlags(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".');
@@ -174,6 +177,10 @@ function parseDocsFlags(action, values) {
     if (action === "create" && !title) {
         throw new Error("docs create needs --title.");
     }
+    return { visibility, clearParent, parentId, allowEmpty, title };
+}
+/** The flags that narrow or slice what a list or read returns. */
+function parseDocsReadFlags(action, values) {
     // BLI-3737: narrowing belongs to `list` only. `--parent` keeps its create/
     // update meaning elsewhere (which parent to file the document under); on
     // `list` the same word names which parent to list UNDER, plus the literal
@@ -205,16 +212,19 @@ function parseDocsFlags(action, values) {
     const section = optionalNonEmpty(values.flags.get("--section"));
     if (section && action !== "read")
         throw new Error("--section belongs to `cockpit docs read`.");
-    const offsetRaw = optionalNonEmpty(values.flags.get("--offset"));
-    let offset;
-    if (offsetRaw !== undefined) {
-        offset = Number(offsetRaw);
-        if (!Number.isInteger(offset) || offset < 0)
-            throw new Error("--offset must be a whole number 0 or greater.");
-    }
+    const offset = parseDocsOffset(values.flags.get("--offset"));
     if (offset !== undefined && action !== "read")
         throw new Error("--offset belongs to `cockpit docs read`.");
-    return { visibility, clearParent, parentId, allowEmpty, title, query, wholeWord, limit, bodies, section, offset };
+    return { query, wholeWord, limit, bodies, section, offset };
+}
+function parseDocsOffset(raw) {
+    const offsetRaw = optionalNonEmpty(raw);
+    if (offsetRaw === undefined)
+        return undefined;
+    const offset = Number(offsetRaw);
+    if (!Number.isInteger(offset) || offset < 0)
+        throw new Error("--offset must be a whole number 0 or greater.");
+    return offset;
 }
 function buildDocsCommand(values, action, docRef, docRefs, visibility, clearParent, parentId, allowEmpty, title, query, wholeWord, limit, bodies, section, offset, link) {
     return {
@@ -243,146 +253,4 @@ function buildDocsCommand(values, action, docRef, docRefs, visibility, clearPare
         json: values.booleans.has("--json"),
     };
 }
-const MSG_ACTIONS = new Set(["channels", "read", "send", "thread", "create", "dm"]);
-const MSG_ACTIONS_NEEDING_A_CHANNEL = new Set(["read", "send"]);
-/**
- * `--members a@x.test,b@y.test` — addresses only, split on commas, never
- * fuzzy (BLI-3749). The door resolves each one exactly, so a typo comes back
- * as `person_not_found` naming the address rather than as a stranger added
- * to a private channel.
- */
-function parseMemberEmails(raw) {
-    const value = optionalNonEmpty(raw);
-    if (!value)
-        return undefined;
-    const emails = value
-        .split(",")
-        .map((part) => part.trim())
-        .filter((part) => part !== "");
-    if (emails.length === 0)
-        throw new Error("msg create --members needs at least one email address.");
-    const notAnAddress = emails.find((email) => !email.includes("@"));
-    if (notAnAddress) {
-        throw new Error(`msg create --members takes email addresses; "${notAnAddress}" is not one.`);
-    }
-    return emails;
-}
-export function parseMsgArgs(args) {
-    const values = parseNamedArgs(args, {
-        allowedFlags: [
-            "--home",
-            "--dashboard-url",
-            "--channel",
-            "--thread",
-            "--limit",
-            "--private",
-            "--members",
-            "--description",
-            "--json",
-        ],
-        valueFlags: ["--home", "--dashboard-url", "--channel", "--thread", "--limit", "--members", "--description"],
-    });
-    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.`);
-    }
-    return { action, rest: positionals.slice(first === undefined ? 0 : 1) };
-}
-function parseMsgCreateTarget(action, rest) {
-    let channelName;
-    let dmEmail;
-    if (action === "create") {
-        channelName = optionalNonEmpty(rest[0]);
-        if (!channelName)
-            throw new Error("msg create needs a channel name, e.g. `cockpit msg create general`.");
-        if (rest.length > 1)
-            throw new Error(`msg create takes one channel name, not ${rest.length}.`);
-        // `#general` is how a person says it and how `read`/`send` accept it, so
-        // the leading # is dropped here rather than becoming part of the name.
-        if (channelName.startsWith("#"))
-            channelName = channelName.slice(1);
-        if (channelName === "")
-            throw new Error("msg create needs a channel name, e.g. `cockpit msg create general`.");
-    }
-    else if (action === "dm") {
-        dmEmail = optionalNonEmpty(rest[0]);
-        if (!dmEmail)
-            throw new Error("msg dm needs the person's email address, e.g. `cockpit msg dm ada@example.com`.");
-        if (rest.length > 1)
-            throw new Error(`msg dm takes one email address, not ${rest.length}.`);
-        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`.");
-    }
-    const memberEmails = parseMemberEmails(values.flags.get("--members"));
-    if (memberEmails && action !== "create") {
-        throw new Error("--members belongs to `cockpit msg create`.");
-    }
-    const description = optionalNonEmpty(values.flags.get("--description"));
-    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]);
-        if (!channelRef)
-            throw new Error(`msg ${action} needs a channel id or name.`);
-        if (rest.length > 1)
-            throw new Error(`msg ${action} takes one channel, not ${rest.length}.`);
-    }
-    else if (action === "thread") {
-        // `thread <id> --channel <channel>` — the channel is a flag here because
-        // the positional names the THREAD, not the channel (`--channel` below).
-        if (rest.length > 1)
-            throw new Error(`msg thread takes one thread id, not ${rest.length}.`);
-    }
-    else if (action !== "create" && action !== "dm" && rest.length > 0) {
-        throw new Error(`msg ${action} does not take "${rest[0]}".`);
-    }
-    const threadFlag = optionalNonEmpty(values.flags.get("--thread"));
-    const threadId = action === "thread" ? optionalNonEmpty(rest[0]) : threadFlag;
-    if (action === "thread" && !threadId)
-        throw new Error("msg thread needs a thread id.");
-    const channelFlag = optionalNonEmpty(values.flags.get("--channel"));
-    if (action === "thread")
-        channelRef = channelFlag;
-    const limit = optionalPositiveInteger(values.flags.get("--limit"), "--limit");
-    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,
-        homeDir: optionalNonEmpty(values.flags.get("--home")),
-        dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
-        ...(channelRef ? { channelRef } : {}),
-        ...(channelName ? { channelName } : {}),
-        ...(dmEmail ? { dmEmail } : {}),
-        ...(memberEmails ? { memberEmails } : {}),
-        ...(description ? { description } : {}),
-        isPrivate,
-        ...(threadId ? { threadId } : {}),
-        ...(limit === undefined ? {} : { limit }),
-        json: values.booleans.has("--json"),
-    };
-}
+export { parseMsgArgs } from "./local-args-tower-msg.js";

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

@@ -33,26 +33,8 @@ export function parseGithubArgs(args) {
         throw new Error(`github ${action} takes a repo and a pull request number only.`);
     const pullNumber = numberText === undefined ? undefined : optionalPositiveInteger(numberText, "pull request number");
     const runId = optionalPositiveInteger(values.flags.get("--run"), "--run");
-    if (action === "rerun") {
-        if ((pullNumber === undefined) === (runId === undefined)) {
-            throw new Error("github rerun needs exactly one of a pull request number or --run <id>.");
-        }
-    }
-    else if (pullNumber === undefined) {
-        throw new Error(`github ${action} needs a pull request number.`);
-    }
-    const only = (flag, allowed) => {
-        if (values.flags.has(flag) && !allowed.includes(action)) {
-            throw new Error(`${flag} belongs to \`cockpit github ${allowed.join("|")}\`.`);
-        }
-    };
-    only("--claim", ["pr"]);
-    only("--method", ["merge"]);
-    only("--title", ["retitle"]);
-    only("--run", ["rerun"]);
-    if (values.flags.has("--confirm") && action === "pr") {
-        throw new Error("--confirm belongs to merge, retitle and rerun; pr only reads.");
-    }
+    requireTargetFor(action, pullNumber, runId);
+    refuseFlagsFromOtherActions(values, action);
     const title = optionalNonEmpty(values.flags.get("--title"));
     if (action === "retitle" && !title)
         throw new Error("github retitle needs --title \"<new title>\".");
@@ -78,4 +60,28 @@ export function parseGithubArgs(args) {
         dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
         json: values.booleans.has("--json"),
     };
+}
+function requireTargetFor(action, pullNumber, runId) {
+    if (action === "rerun") {
+        if ((pullNumber === undefined) === (runId === undefined)) {
+            throw new Error("github rerun needs exactly one of a pull request number or --run <id>.");
+        }
+    }
+    else if (pullNumber === undefined) {
+        throw new Error(`github ${action} needs a pull request number.`);
+    }
+}
+function refuseFlagsFromOtherActions(values, action) {
+    const only = (flag, allowed) => {
+        if (values.flags.has(flag) && !allowed.includes(action)) {
+            throw new Error(`${flag} belongs to \`cockpit github ${allowed.join("|")}\`.`);
+        }
+    };
+    only("--claim", ["pr"]);
+    only("--method", ["merge"]);
+    only("--title", ["retitle"]);
+    only("--run", ["rerun"]);
+    if (values.flags.has("--confirm") && action === "pr") {
+        throw new Error("--confirm belongs to merge, retitle and rerun; pr only reads.");
+    }
 }