@bli-cockpit/cli 0.2.57 → 0.2.59

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

@@ -0,0 +1,178 @@
+/**
+ * `cockpit issue` and `cockpit project` argument parsing (BLI-3716) — the
+ * Work surface's half of `local-args-tower.ts`, a sibling of
+ * `local-args-tower-docs-msg.ts` for the same reason that file is a sibling
+ * of the pages family: issues are their own noun with their own vocabulary
+ * (a state, an assignee, a project) and folding them into another family's
+ * parser would stretch that family's doc comment past the truth.
+ *
+ * Two commands, because a person types `cockpit project list`, not
+ * `cockpit issue projects` — the nouns are what a person says.
+ *
+ * Bodies never travel on argv: `create --title` names the issue, but the
+ * DESCRIPTION and every comment come from stdin or `--file`, the same
+ * discipline `cockpit docs` and `cockpit notes paste` keep (BLI-3706, and
+ * the sourcing-secrets verdict before it — argv is world-readable on a
+ * shared machine and lands in shell history).
+ */
+import { optionalNonEmpty, optionalPositiveInteger, optionalUrl, parseNamedArgs, } from "./local-arg-values.js";
+const ISSUE_ACTIONS = new Set([
+    "list",
+    "show",
+    "create",
+    "update",
+    "move",
+    "comment",
+    "history",
+]);
+/** Every verb except `list` and `create` names ONE issue first. */
+const ISSUE_ACTIONS_NEEDING_AN_ISSUE = new Set([
+    "show",
+    "update",
+    "move",
+    "comment",
+    "history",
+]);
+/**
+ * The closed state vocabulary, copied verbatim from
+ * `apps/dashboard/src/lib/work/api-doors.ts` `WORK_ISSUE_STATES` (which is
+ * itself the migration's CHECK constraint). Duplicated rather than imported
+ * because no dependency edge exists from the collector to the dashboard;
+ * `issue.ts`'s tests pin the list so a drift shows up here rather than as a
+ * 400 on a fleet machine.
+ */
+export const ISSUE_STATES = ["backlog", "todo", "in_progress", "in_review", "done", "canceled"];
+const UUID_PATTERN = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;
+export function parseIssueArgs(args) {
+    const values = parseNamedArgs(args, {
+        allowedFlags: [
+            "--home",
+            "--dashboard-url",
+            "--state",
+            "--assignee",
+            "--project",
+            "--limit",
+            "--title",
+            "--priority",
+            "--parent",
+            "--file",
+            "--body-stdin",
+            "--json",
+        ],
+        valueFlags: [
+            "--home",
+            "--dashboard-url",
+            "--state",
+            "--assignee",
+            "--project",
+            "--limit",
+            "--title",
+            "--priority",
+            "--parent",
+            "--file",
+        ],
+    });
+    const first = values.positionals[0];
+    const action = (first === undefined ? "list" : first);
+    if (!ISSUE_ACTIONS.has(action)) {
+        throw new Error(`Unknown issue command: ${first}. Try list, show, create, update, move, comment, or history.`);
+    }
+    const rest = values.positionals.slice(first === undefined ? 0 : 1);
+    let issueRef;
+    let moveState;
+    if (ISSUE_ACTIONS_NEEDING_AN_ISSUE.has(action)) {
+        issueRef = optionalNonEmpty(rest[0]);
+        if (!issueRef)
+            throw new Error(`issue ${action} needs an issue id or identifier, e.g. BLI-3654.`);
+        if (action === "move") {
+            moveState = optionalNonEmpty(rest[1]);
+            if (!moveState) {
+                throw new Error(`issue move needs a state: ${ISSUE_STATES.join(", ")}.`);
+            }
+            if (rest.length > 2)
+                throw new Error(`issue move takes an issue and one state, not ${rest.length} words.`);
+        }
+        else if (rest.length > 1) {
+            throw new Error(`issue ${action} takes one issue reference, not ${rest.length}.`);
+        }
+    }
+    else if (rest.length > 0) {
+        throw new Error(`issue ${action} does not take "${rest[0]}".`);
+    }
+    if (moveState && !ISSUE_STATES.includes(moveState)) {
+        throw new Error(`issue move state must be one of: ${ISSUE_STATES.join(", ")}.`);
+    }
+    const stateFilter = optionalNonEmpty(values.flags.get("--state"));
+    if (stateFilter !== undefined && !ISSUE_STATES.includes(stateFilter)) {
+        throw new Error(`--state must be one of: ${ISSUE_STATES.join(", ")}.`);
+    }
+    if (stateFilter !== undefined && action !== "list") {
+        throw new Error("--state filters `cockpit issue list`; to move an issue use `cockpit issue move <id> <state>`.");
+    }
+    const title = optionalNonEmpty(values.flags.get("--title"));
+    if (action === "create" && !title)
+        throw new Error("issue create needs --title.");
+    // Not `optionalPositiveInteger`: 0 is a legitimate priority ("none", the
+    // column default), and that helper refuses it as non-positive.
+    const priorityRaw = optionalNonEmpty(values.flags.get("--priority"));
+    let priority;
+    if (priorityRaw !== undefined) {
+        priority = Number(priorityRaw);
+        if (!Number.isInteger(priority) || priority < 0 || priority > 4) {
+            throw new Error("--priority must be 0-4 (0 none, 1 urgent, 2 high, 3 medium, 4 low).");
+        }
+    }
+    // An assignee is `me`, `unassigned`, or a person's uuid. A name or an
+    // email is refused HERE rather than travelling to Postgres as a malformed
+    // uuid, which would come back as a 500 that names nothing useful.
+    const assignee = optionalNonEmpty(values.flags.get("--assignee"));
+    if (assignee !== undefined && assignee !== "me" && assignee !== "unassigned" && !UUID_PATTERN.test(assignee)) {
+        throw new Error('--assignee must be "me", "unassigned", or a person\'s uuid (see `cockpit team`).');
+    }
+    if (assignee === "unassigned" && action !== "list") {
+        throw new Error('--assignee unassigned filters `cockpit issue list`; to clear an assignee pass --assignee "" is not supported yet.');
+    }
+    const limit = optionalPositiveInteger(values.flags.get("--limit"), "--limit");
+    if (limit !== undefined && action !== "list" && action !== "history") {
+        throw new Error("--limit belongs to `cockpit issue list` or `cockpit issue history`.");
+    }
+    return {
+        kind: "issue",
+        action,
+        homeDir: optionalNonEmpty(values.flags.get("--home")),
+        dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
+        ...(issueRef ? { issueRef } : {}),
+        ...(moveState ? { moveState } : {}),
+        ...(stateFilter ? { stateFilter } : {}),
+        ...(assignee ? { assignee } : {}),
+        project: optionalNonEmpty(values.flags.get("--project")),
+        ...(limit === undefined ? {} : { limit }),
+        title,
+        ...(priority === undefined ? {} : { priority }),
+        parentRef: optionalNonEmpty(values.flags.get("--parent")),
+        filePath: optionalNonEmpty(values.flags.get("--file")),
+        bodyStdin: values.booleans.has("--body-stdin"),
+        json: values.booleans.has("--json"),
+    };
+}
+export function parseProjectArgs(args) {
+    const values = parseNamedArgs(args, {
+        allowedFlags: ["--home", "--dashboard-url", "--archived", "--json"],
+        valueFlags: ["--home", "--dashboard-url"],
+    });
+    const first = values.positionals[0];
+    const action = (first === undefined ? "list" : first);
+    if (action !== "list")
+        throw new Error(`Unknown project command: ${first}. The only verb is list.`);
+    const rest = values.positionals.slice(first === undefined ? 0 : 1);
+    if (rest.length > 0)
+        throw new Error(`project list does not take "${rest[0]}".`);
+    return {
+        kind: "project",
+        action,
+        homeDir: optionalNonEmpty(values.flags.get("--home")),
+        dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
+        includeArchived: values.booleans.has("--archived"),
+        json: values.booleans.has("--json"),
+    };
+}

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

@@ -18,10 +18,16 @@
  *                                model
  *   local-args-tower-docs-msg.ts docs, msg          — the document library and
  *                                                      channels/messages (BLI-3706)
+ *   local-args-tower-work.ts    issue, project     — the issue tracker
+ *                                                      (BLI-3716)
+ *   local-args-tower-search.ts   search            — one bar over all five
+ *                                                      corpora (BLI-3728)
  *
  * Every name this module has ever exported is still importable from here.
  */
 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";
-export { parseDocsArgs, parseMsgArgs, } from "./local-args-tower-docs-msg.js";
+export { parseDocsArgs, parseMsgArgs, } from "./local-args-tower-docs-msg.js";
+export { ISSUE_STATES, parseIssueArgs, parseProjectArgs, } from "./local-args-tower-work.js";
+export { SEARCH_KINDS, parseSearchArgs } from "./local-args-tower-search.js";

package/dist/commands/local-args.js

@@ -1,23 +1,32 @@
+/**
+ * What a person may type, and what each subcommand means once typed.
+ *
+ * Two things live here and nothing else: `LocalCommand`, the shape every
+ * command runner is handed, and `parseLocalArgs`, the table that picks which
+ * decision table reads the rest of `argv`. The decision tables themselves are
+ * next door, one file per half of the product (BLI-3578):
+ *
+ *   local-arg-values.ts      argv into named flags, then one question per value
+ *   local-args-collector.ts  the commands that act on THIS machine
+ *   local-args-tower.ts      the commands that talk to Tower
+ *
+ * Split out of commands/local.ts originally so the command runners read as a
+ * table of contents and the parser (pure, independently testable) can change at
+ * its own rate. Behavior-preserving extraction throughout: functions moved
+ * verbatim, no logic change.
+ */
 import { parseAgentRulesArgs, parseAnalyzeArgs, parseAutostartArgs, parseBackfillArgs, parseCleanArgs, parseDoctorArgs, parseInstallArgs, parseLoginArgs, parseMemoryArgs, parseLogoutArgs, parseOnboardArgs, parseReleaseArgs, parseServeArgs, parseSessionsArgs, parseStartArgs, parseStatusArgs, parseSyncArgs, parseUpdateArgs, } from "./local-args-collector.js";
-import { parseBriefArgs, parseCorrectArgs, parseDocsArgs, parseJarvisArgs, parseModelArgs, parseMsgArgs, parseNotesArgs, parseOpsArgs, parseScoutArgs, parseSettingsArgs, parseSlackArgs, parseTeamArgs, parseWorkbookArgs, } from "./local-args-tower.js";
+import { parseBriefArgs, parseCorrectArgs, parseDocsArgs, parseIssueArgs, parseJarvisArgs, parseModelArgs, parseMsgArgs, parseNotesArgs, parseOpsArgs, parseProjectArgs, parseScoutArgs, parseSearchArgs, parseSettingsArgs, parseSlackArgs, parseTeamArgs, parseWorkbookArgs, } from "./local-args-tower.js";
 // `normalizeUrl` has always been part of this module's surface — `local.ts` and
 // `local-auth.ts` import it from here — so it stays exported from this address
 // even though it now lives next door. The same goes for the four names the
 // Tower parsers publish.
 export { normalizeUrl } from "./local-arg-values.js";
-export { SCOUT_MIN_PREFIX_LENGTH, SLACK_WORKSPACE_KEYS, WORKBOOK_MIN_WIDTH, } from "./local-args-tower.js";
-// The six human "set my machine up" doors. They are one thing wearing six
-// hats, so they all run the convergence command — but they keep accepting the
-// flags they always accepted, because DMs, runbooks and AGENTS.md rules across
-// the fleet still spell them out.
-export const DOCTOR_SETUP_ALIASES = [
-    "install",
-    "onboard",
-    "login",
-    "pair",
-    "update",
-    "upgrade",
-];
+// BLI-3728: the command SHAPES moved to a sibling when this file crossed the
+// 700-line ceiling. Re-exported here because ~40 modules and every sibling
+// parser import them from this path.
+export { DOCTOR_SETUP_ALIASES, } from "./local-command-shapes.js";
+export { SCOUT_MIN_PREFIX_LENGTH, SLACK_WORKSPACE_KEYS, WORKBOOK_MIN_WIDTH, ISSUE_STATES, } from "./local-args-tower.js";
 export function parseLocalArgs(argv) {
     const command = argv[0];
     switch (command) {
@@ -88,6 +97,12 @@ export function parseLocalArgs(argv) {
             return parseDocsArgs(argv.slice(1));
         case "msg":
             return parseMsgArgs(argv.slice(1));
+        case "issue":
+            return parseIssueArgs(argv.slice(1));
+        case "project":
+            return parseProjectArgs(argv.slice(1));
+        case "search":
+            return parseSearchArgs(argv.slice(1));
         case "release":
             return parseReleaseArgs(argv.slice(1));
         default:

package/dist/commands/local-command-shapes.js

@@ -0,0 +1,12 @@
+// The six human "set my machine up" doors. They are one thing wearing six
+// hats, so they all run the convergence command — but they keep accepting the
+// flags they always accepted, because DMs, runbooks and AGENTS.md rules across
+// the fleet still spell them out.
+export const DOCTOR_SETUP_ALIASES = [
+    "install",
+    "onboard",
+    "login",
+    "pair",
+    "update",
+    "upgrade",
+];