@bli-cockpit/cli 0.2.237 → 0.2.238

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

@@ -20,7 +20,7 @@
  */
 import { optionalNonEmpty, optionalPositiveInteger, optionalUrl, parseNamedArgs, } from "./local-arg-values.js";
 const CAL_ACTIONS = new Set([
-    "conflicts", "calendar-change",
+    "event", "export", "conflicts", "calendar-change",
     "today",
     "week",
     "next",
@@ -36,58 +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(["calendar-change", "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",
-            "--color",
-            "--calendar",
-            "--account",
-            "--tz",
-            "--offset",
-            "--hours",
-            "--from",
-            "--to",
-            "--limit",
-            "--all",
-            "--name",
-            "--title",
-            "--at",
-            "--until",
-            "--location",
-            "--attendee",
-            "--all-day",
-            "--meet",
-            "--org-visible",
-            "--private",
-            "--full",
-            "--yes",
-            "--json",
-        ],
-        valueFlags: [
-            "--home",
-            "--dashboard-url",
-            "--color",
-            "--calendar",
-            "--account",
-            "--tz",
-            "--offset",
-            "--hours",
-            "--from",
-            "--to",
-            "--limit",
-            "--name",
-            "--title",
-            "--at",
-            "--until",
-            "--location",
-            "--attendee",
-        ],
-    });
+    const values = parseNamedArgs(args, { allowedFlags: CAL_ALLOWED_FLAGS, valueFlags: CAL_VALUE_FLAGS });
     const { action, rest } = readCalAction(values);
     const { subject, query } = readCalSubjectOrQuery(action, rest);
     const { title, startsAt, endsAt } = readCalCreateDetails(action, values);
@@ -173,6 +174,7 @@ function readCalOffset(values) {
 function buildCalCommand(values, action, subject, query, title, startsAt, endsAt, offset, limit) {
     return {
         kind: "cal",
+        occurrenceStart: optionalNonEmpty(values.flags.get("--occurrence-start")),
         color: optionalNonEmpty(values.flags.get("--color")),
         action,
         homeDir: optionalNonEmpty(values.flags.get("--home")),

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 {

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.");
+    }
 }

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

@@ -37,27 +37,39 @@ export function parseHitsArgs(args) {
     }
     if (given("--weeks") && action !== "timeline")
         throw new Error("--weeks belongs to hits timeline.");
-    if (action === "timeline") {
-        if (given("--scope") || given("--version"))
-            throw new Error("--scope and --version belong to hits approve.");
-        const weeks = optionalPositiveInteger(values.flags.get("--weeks"), "--weeks");
-        if (weeks !== undefined && weeks > 26)
-            throw new Error("--weeks must be 26 or fewer.");
-        return { ...common, action, ...(weeks !== undefined ? { weeks } : {}) };
-    }
-    if (action === "coach") {
-        if (given("--scope") || given("--version"))
-            throw new Error("--scope and --version belong to hits approve.");
-        return { ...common, action };
-    }
-    if (action === "approve") {
-        const scope = optionalNonEmpty(values.flags.get("--scope"));
-        if (!scope || !SCOPE.test(scope))
-            throw new Error("hits approve wants --scope team, agents or project:<id> (for example project:pearl-portal).");
-        return { ...common, action, scope, ...versionOf(values.flags.get("--version")) };
-    }
+    const parsed = { values, given, common };
+    if (action === "timeline")
+        return parseHitsTimeline(parsed);
+    if (action === "coach")
+        return parseHitsCoach(parsed);
+    if (action === "approve")
+        return parseHitsApprove(parsed);
+    return parseHitsRead(parsed);
+}
+function refuseApproveFlags({ given }) {
     if (given("--scope") || given("--version"))
         throw new Error("--scope and --version belong to hits approve.");
+}
+function parseHitsTimeline(parsed) {
+    refuseApproveFlags(parsed);
+    const weeks = optionalPositiveInteger(parsed.values.flags.get("--weeks"), "--weeks");
+    if (weeks !== undefined && weeks > 26)
+        throw new Error("--weeks must be 26 or fewer.");
+    return { ...parsed.common, action: "timeline", ...(weeks !== undefined ? { weeks } : {}) };
+}
+function parseHitsCoach(parsed) {
+    refuseApproveFlags(parsed);
+    return { ...parsed.common, action: "coach" };
+}
+function parseHitsApprove({ values, common }) {
+    const scope = optionalNonEmpty(values.flags.get("--scope"));
+    if (!scope || !SCOPE.test(scope))
+        throw new Error("hits approve wants --scope team, agents or project:<id> (for example project:pearl-portal).");
+    return { ...common, action: "approve", scope, ...versionOf(values.flags.get("--version")) };
+}
+function parseHitsRead(parsed) {
+    const { values, given, common } = parsed;
+    refuseApproveFlags(parsed);
     const whoFlags = ["--mine", "--person", "--agent", "--project"].filter(given);
     if (whoFlags.length > 1)
         throw new Error(`Pick one of --mine, --person, --agent or --project (got ${whoFlags.join(" and ")}).`);
@@ -77,7 +89,7 @@ export function parseHitsArgs(args) {
         throw new Error("--learning is read for one person: use --mine or --person <name>.");
     return {
         ...common,
-        action,
+        action: "read",
         ...(who ? { who } : {}),
         ...(person ? { person } : {}),
         ...(project ? { project } : {}),