@bli-cockpit/cli 0.2.121 → 0.2.122

package/dist/commands/careers.js

@@ -1,16 +1,88 @@
+import { readFile } from 'node:fs/promises';
 import { askAgentDoor, emitAgentDoor, failAgentDoor, openAgentDoor } from './agent-door.js';
+/**
+ * `cockpit careers` — application review (BLI-3706) and, since BLI-4460, the
+ * pipeline: invite, decide, the role's take-home and the four settings.
+ *
+ * Every verb goes through the SAME Tower door its MCP twin calls; nothing here
+ * decides policy. The take-home body arrives from a FILE rather than a flag,
+ * because an email body does not belong in an argument vector, a shell history
+ * or a process list.
+ */
 export async function runCareers(command, io) {
     const door = await openAgentDoor('careers', command, io);
-    const query = new URLSearchParams();
-    if (command.role)
-        query.set('role', command.role);
-    if (command.minScore !== undefined)
-        query.set('min_score', String(command.minScore));
-    if (command.since)
-        query.set('since', command.since);
-    const path = command.action === 'list' ? `/api/careers/applications?${query}` : `/api/careers/applications/${encodeURIComponent(command.id)}${command.action === 'rescreen' ? '/rescreen' : ''}`;
-    const answer = await askAgentDoor(door, { path, method: command.action === 'rescreen' ? 'POST' : 'GET', label: `careers ${command.action}`, timeoutMs: 60_000 });
+    let plan;
+    try {
+        plan = await planCareersRequest(command);
+    }
+    catch (error) {
+        return failAgentDoor(door, '[careers]', 'invalid_request', error instanceof Error ? error.message : 'Could not read that request.');
+    }
+    const answer = await askAgentDoor(door, { ...plan, label: `careers ${command.action}`, timeoutMs: 60_000 });
     if (!answer.ok)
         return failAgentDoor(door, '[careers]', answer.reason, answer.detail);
     return emitAgentDoor(door, answer.body);
+}
+async function planCareersRequest(command) {
+    const application = `/api/careers/applications/${encodeURIComponent(command.id ?? '')}`;
+    switch (command.action) {
+        case 'list': {
+            const query = new URLSearchParams();
+            if (command.role)
+                query.set('role', command.role);
+            if (command.minScore !== undefined)
+                query.set('min_score', String(command.minScore));
+            if (command.since)
+                query.set('since', command.since);
+            if (command.stage)
+                query.set('stage', command.stage);
+            return { path: `/api/careers/applications?${query}`, method: 'GET' };
+        }
+        case 'show': return { path: application, method: 'GET' };
+        case 'rescreen': return { path: `${application}/rescreen`, method: 'POST' };
+        case 'invite': return { path: `${application}/invite`, method: 'POST' };
+        case 'decide': return { path: `${application}/decide`, method: 'POST', body: { decision: command.decision } };
+        case 'takehome-show': return { path: takehomePath(command), method: 'GET' };
+        case 'takehome-set': return { path: takehomePath(command), method: 'PUT', body: await takehomeBody(command) };
+        case 'settings-show': return { path: '/api/careers/settings', method: 'GET' };
+        case 'settings-set': return { path: '/api/careers/settings', method: 'PUT', body: settingsBody(command) };
+    }
+}
+function takehomePath(command) {
+    return `/api/careers/takehomes/${encodeURIComponent(command.takehomeRole ?? '')}`;
+}
+async function takehomeBody(command) {
+    const body = {};
+    if (command.subject)
+        body.subject = command.subject;
+    if (command.link)
+        body.artifact_link = command.link;
+    if (command.passScore !== undefined)
+        body.pass_score = command.passScore;
+    if (command.bodyFile) {
+        const text = await readFile(command.bodyFile, 'utf8');
+        if (!text.includes('{first_name}') || !text.includes('{link}')) {
+            throw new Error('That body file is missing {first_name} or {link}, so nothing was sent to Tower.');
+        }
+        body.body_text = text;
+    }
+    if (Object.keys(body).length === 0) {
+        throw new Error('careers takehome set needs at least one of --subject, --body-file, --link or --pass-score.');
+    }
+    return body;
+}
+function settingsBody(command) {
+    const body = {};
+    if (command.autoInviteEnabled !== undefined)
+        body.auto_invite_enabled = command.autoInviteEnabled;
+    if (command.autoInviteMinScore !== undefined)
+        body.auto_invite_min_score = command.autoInviteMinScore;
+    if (command.attentionMinScore !== undefined)
+        body.attention_min_score = command.attentionMinScore;
+    if (command.inviteAccountId !== undefined)
+        body.invite_account_id = command.inviteAccountId;
+    if (Object.keys(body).length === 0) {
+        throw new Error('careers settings set needs at least one of --enabled, --auto-invite-min-score, --attention-min-score or --account.');
+    }
+    return body;
 }

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

@@ -1,13 +1,31 @@
 import { optionalNonEmpty, optionalUrl, parseNamedArgs } from './local-arg-values.js';
+const ACTIONS = ['list', 'show', 'rescreen', 'invite', 'decide', 'takehome-show', 'takehome-set', 'settings-show', 'settings-set'];
+const DECISIONS = ['passed_on', 'archived', 'reopen'];
+/** Verbs that name an application id, a role slug, or nothing at all. */
+const NEEDS_ID = ['show', 'rescreen', 'invite', 'decide'];
+const NEEDS_ROLE = ['takehome-show', 'takehome-set'];
 export function parseCareersArgs(args) {
-    const values = parseNamedArgs(args, { allowedFlags: ['--role', '--min-score', '--since', '--home', '--dashboard-url', '--json'], valueFlags: ['--role', '--min-score', '--since', '--home', '--dashboard-url'] });
-    const action = values.positionals[0] ?? 'list';
-    if (action !== 'list' && action !== 'show' && action !== 'rescreen')
-        throw new Error('careers takes list, show, or rescreen.');
-    const id = values.positionals[1];
-    if (action !== 'list' && !id)
-        throw new Error(`careers ${action} needs an application id.`);
-    if (values.positionals.length > (action === 'list' ? 1 : 2))
+    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'];
+    const values = parseNamedArgs(args, { allowedFlags: [...valueFlags, '--json'], 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 first = positionals[0] ?? 'list';
+    const paired = first === 'takehome' || first === 'settings';
+    const action = paired
+        ? `${first}-${positionals.splice(0, positionals[1] === undefined ? 1 : 2)[1] ?? 'show'}`
+        : (positionals.shift() ?? 'list');
+    if (!ACTIONS.includes(action))
+        throw new Error(`careers takes ${ACTIONS.join(', ')}.`);
+    const verb = action;
+    const target = positionals.shift();
+    if (NEEDS_ID.includes(verb) && !target)
+        throw new Error(`careers ${verb} needs an application id.`);
+    if (NEEDS_ROLE.includes(verb) && !target)
+        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);
@@ -16,5 +34,34 @@ export function parseCareersArgs(args) {
     const since = optionalNonEmpty(values.flags.get('--since'));
     if (since && !Number.isFinite(Date.parse(since)))
         throw new Error('--since must be a date.');
-    return { kind: 'careers', action, id, role: optionalNonEmpty(values.flags.get('--role')), minScore, since: since ? new Date(since).toISOString() : undefined, homeDir: optionalNonEmpty(values.flags.get('--home')), dashboardUrl: optionalUrl(values.flags.get('--dashboard-url')), json: values.booleans.has('--json') };
+    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.');
+    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')),
+        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'),
+    };
+}
+function numberFlag(raw, flag, min, max) {
+    if (raw === undefined)
+        return undefined;
+    const value = Number(raw);
+    if (!Number.isFinite(value) || value < min || value > max)
+        throw new Error(`${flag} must be ${min} to ${max}.`);
+    return value;
 }

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

@@ -22,7 +22,21 @@ const SEARCH_KIND_LIST = SEARCH_KINDS.join(",");
 const SEARCH_CORPORA_COUNT = SEARCH_KINDS.length;
 /** One entry per Tower noun, in the order `cockpit --help` lists them. */
 export const TOWER_COMMAND_HELP = [
-    ["careers", ["Usage: cockpit careers [list|show <id>|rescreen <id>] [--role <slug>] [--min-score <n>] [--since <date>] [--json]", "Super-admin application review. Lists at most 100 matches with total and has_more; use filters to narrow."]],
+    [
+        "careers",
+        [
+            "Usage: cockpit careers [list|show <id>|rescreen <id>|invite <id>|decide <id> --decision <d>|takehome show <role>|takehome set <role>|settings show|settings set] [flags]",
+            "",
+            "Super-admin application review and pipeline. Lists at most 100 matches with total and has_more; use filters to narrow.",
+            "list [--role <slug>] [--min-score <n>] [--since <date>] [--stage <screened|invited|submitted|graded|booking_sent|passed_on|archived>] — the board, newest first, with the two thresholds in force.",
+            "invite <id> — sends that role's take-home from the configured mailbox, whatever the screening scored. Only a row still at `screened`.",
+            "decide <id> --decision <passed_on|archived|reopen> — your own call on a row; `reopen` puts it back to `screened`.",
+            "takehome show <role> — the subject, body, link and the reviewer's rubric for that role.",
+            "takehome set <role> [--subject <s>] [--body-file <path>] [--link <url>] [--pass-score <0-10>] — the body comes from a FILE and must keep {first_name} and {link}.",
+            "settings show — the bar, the attention threshold, whether automatic sending is on, and which mailbox it goes out from.",
+            "settings set [--enabled true|false] [--auto-invite-min-score <0-100>] [--attention-min-score <0-100>] [--account <mailbox uuid>] — each takes effect on the next screening, no deploy.",
+        ],
+    ],
     [
         "docs",
         [

package/dist/commands/local-help.js

@@ -104,7 +104,7 @@ export function localCommandHelp(command) {
         "  cockpit project [list] [--archived] [--dashboard-url <url>] [--json]",
         `  cockpit search "<words>" [--kind ${SEARCH_KINDS.join(",")}] [--limit <n>] [--dashboard-url <url>] [--json]`,
         "  cockpit release [--dry-run] [--skip-checks] [--no-floor] [--tag <tag>] [--access <public|restricted>] [--otp <code>]",
-        "  cockpit careers [list|show <id>|rescreen <id>] [--role <slug>] [--min-score <n>] [--since <date>] [--json]",
+        "  cockpit careers [list|show <id>|rescreen <id>|invite <id>|decide <id> --decision <passed_on|archived|reopen>|takehome show <role>|takehome set <role> --body-file <path>|settings show|settings set --enabled <true|false>] [--role <slug>] [--min-score <n>] [--since <date>] [--stage <s>] [--json]",
         "  cockpit usage sessions [--task <BLI-NNNN> | --subject <name>] [--repo <label>] [--since <window>] [--json]",
         "  cockpit usage sessions --session <id> [--from-window <n>] [--json]",
         "  --by-topic groups work types; --by-subject groups open subjects from session summaries.",

package/dist/commands/public-root.js

@@ -15,7 +15,7 @@ export async function runCockpitCli(argv, io) {
   }
 
   if (command === "--version" || command === "-V" || command === "version") {
-    writeLine(io?.stdout ?? process.stdout, "0.2.121");
+    writeLine(io?.stdout ?? process.stdout, "0.2.122");
     return 0;
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bli-cockpit/cli",
-  "version": "0.2.121",
+  "version": "0.2.122",
   "private": false,
   "type": "module",
   "bin": {
@@ -28,7 +28,7 @@
   },
   "dependencies": {
     "@bli-cockpit/memory-mcp": "0.1.32",
-    "@bli-cockpit/mcp": "0.1.48",
+    "@bli-cockpit/mcp": "0.1.49",
     "@bli-cockpit/telemetry-core": "0.1.48"
   }
 }