@bli-cockpit/cli 0.2.161 → 0.2.163

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

@@ -0,0 +1,48 @@
+import { optionalNonEmpty, optionalUrl, parseNamedArgs } from "./local-arg-values.js";
+export const RENAME_SCOPES = ["page", "everywhere", "undo"];
+export function parseRenameArgs(args) {
+    const values = parseNamedArgs(args, {
+        allowedFlags: ["--home", "--dashboard-url", "--scope", "--wrong", "--right", "--json"],
+        valueFlags: ["--home", "--dashboard-url", "--scope", "--wrong", "--right"],
+    });
+    const positionals = [...values.positionals];
+    const typed = positionals.shift() ?? "list";
+    if (typed !== "list" && typed !== "resolve") {
+        throw new Error(`Unknown rename command: ${typed}. Try list, or resolve <id> --scope page|everywhere|undo.`);
+    }
+    const id = positionals.shift();
+    // An OLD rule has no open choice and so no id: `--wrong X --right Y --scope
+    // undo` names it by its pair and removes the caller's own row.
+    const wrong = optionalNonEmpty(values.flags.get("--wrong"));
+    const right = optionalNonEmpty(values.flags.get("--right"));
+    const byPair = typed === "resolve" && !id && wrong !== undefined && right !== undefined;
+    if (typed === "resolve" && !id && !byPair) {
+        throw new Error("rename resolve needs the id from `cockpit rename list`, or --wrong and --right with --scope undo.");
+    }
+    if ((wrong !== undefined || right !== undefined) && (typed !== "resolve" || id || wrong === undefined || right === undefined)) {
+        throw new Error("--wrong and --right go together, on `cockpit rename resolve --scope undo`, instead of an id.");
+    }
+    if (positionals.length > 0 || (typed === "list" && id)) {
+        throw new Error(`rename ${typed} does not take "${typed === "list" ? id : positionals[0]}".`);
+    }
+    const scope = optionalNonEmpty(values.flags.get("--scope"));
+    if (typed === "resolve" && (!scope || !RENAME_SCOPES.includes(scope))) {
+        throw new Error(`rename resolve needs --scope ${RENAME_SCOPES.join("|")}.`);
+    }
+    if (byPair && scope !== "undo") {
+        throw new Error("--wrong and --right name a rule that already exists, so they take --scope undo only.");
+    }
+    if (typed === "list" && scope) {
+        throw new Error("--scope belongs to `cockpit rename resolve`.");
+    }
+    return {
+        kind: "rename",
+        action: typed,
+        ...(id ? { id } : {}),
+        ...(scope ? { scope: scope } : {}),
+        ...(byPair ? { wrongForm: wrong, canonicalForm: right } : {}),
+        homeDir: optionalNonEmpty(values.flags.get("--home")),
+        dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")),
+        json: values.booleans.has("--json"),
+    };
+}

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

@@ -37,4 +37,5 @@ export { parseModelsArgs } from "./local-args-tower-models.js";
 export { parseMailArgs } from "./local-args-tower-mail.js";
 export { parseCalArgs } from "./local-args-tower-cal.js";
 export { parseUsageArgs } from "./local-args-tower-usage.js";
-export { parseCareersArgs } from './local-args-tower-careers.js';
+export { parseCareersArgs } from './local-args-tower-careers.js';
+export { parseRenameArgs } from './local-args-tower-rename.js';

package/dist/commands/local-args.js

@@ -18,7 +18,7 @@
 import { parseRootsArgs } from "./roots.js";
 import { SURFACE_ALIASES } from "./local-help.js";
 import { parseAgentRulesArgs, parseAnalyzeArgs, parseAutostartArgs, parseBackfillArgs, parseCleanArgs, parseDoctorArgs, parseInstallArgs, parseLoginArgs, parseMemoryArgs, parseLogoutArgs, parseOnboardArgs, parseReleaseArgs, parseServeArgs, parseSessionsArgs, parseSourcesArgs, parseStartArgs, parseStatusArgs, parseSyncArgs, parseUpdateArgs, } from "./local-args-collector.js";
-import { parseBriefArgs, parseCorrectArgs, parseDocsArgs, parseIssueArgs, parseJarvisArgs, parseMailArgs, parseCalArgs, parseModelArgs, parseMsgArgs, parseNotesArgs, parseOpsArgs, parseProjectArgs, parseModelsArgs, parseScoutArgs, parseSearchArgs, parseSettingsArgs, parseSlackArgs, parseTeamArgs, parseWorkbookArgs, parseUsageArgs, parseCareersArgs, } from "./local-args-tower.js";
+import { parseBriefArgs, parseCorrectArgs, parseDocsArgs, parseIssueArgs, parseJarvisArgs, parseMailArgs, parseCalArgs, parseModelArgs, parseMsgArgs, parseNotesArgs, parseOpsArgs, parseProjectArgs, parseModelsArgs, parseScoutArgs, parseSearchArgs, parseSettingsArgs, parseSlackArgs, parseTeamArgs, parseWorkbookArgs, parseUsageArgs, parseCareersArgs, parseRenameArgs, } 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
@@ -127,6 +127,8 @@ export function parseLocalArgs(argv) {
             return parseReleaseArgs(argv.slice(1));
         case "careers":
             return parseCareersArgs(argv.slice(1));
+        case "rename":
+            return parseRenameArgs(argv.slice(1));
         case "usage":
             return parseUsageArgs(argv.slice(1));
         default:

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

@@ -22,6 +22,17 @@ 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 = [
+    [
+        "rename",
+        [
+            "Usage: cockpit rename [list|resolve <id> --scope <page|everywhere|undo>] [--dashboard-url <url>] [--json]",
+            "",
+            "A correction JARVIS cannot classify is fixed on ONE page and waits for your call: just this page, or everywhere from now on. A bare `cockpit rename` lists what is waiting.",
+            "resolve <id> --scope page — keep it a one-page fix. everywhere — write the rule and fix every other page. undo — take back a rule JARVIS applied on its own (yours only, unless you are a super admin).",
+            "An OLD rule has no open choice: `cockpit rename resolve --wrong <form> --right <form> --scope undo` removes your own row for that pair (a super admin may remove anyone's).",
+            "A fact (a status, a date, a number) is never made a rule, whatever is asked. The same door answers the panel's two buttons, a spoken answer, `cockpit jarvis` and the rename_* MCP tools.",
+        ],
+    ],
     [
         "careers",
         [

package/dist/commands/local-help.js

@@ -75,6 +75,7 @@ export const rootCommandNames = new Set([
     "search",
     "release",
     "careers",
+    "rename",
     "usage",
     ...SURFACE_ALIASES.keys(),
 ]);
@@ -128,6 +129,7 @@ export function localCommandHelp(command) {
         `  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>|send-takehome <id>|decide <id> --decision <passed_on|archived|reopen>|followup <id> [--record-sent-at <iso>]|interview <id> <--at <iso>|--clear>|grade <id> [--now]|resume <id> [--out <path>]|takehome show <role>|takehome set <role> --body-file <path>|settings show|settings set --enabled <true|false> --followup-enabled <true|false> --followup-after-days <n> --takehome-window-hours <n>] [--role <slug>] [--min-score <n>] [--since <date>] [--stage <s>] [--force] [--json]",
+        "  cockpit rename [list|resolve <id> --scope <page|everywhere|undo>] [--dashboard-url <url>] [--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/local.js

@@ -46,6 +46,7 @@ import { runMail } from "./mail.js";
 import { runProject } from "./project.js";
 import { runSearch } from "./search.js";
 import { runCareers } from "./careers.js";
+import { runRename } from "./rename-scope.js";
 import { runUsage } from "./usage.js";
 import { parseLocalArgs } from "./local-args.js";
 // `./local.js` is the published entry point for this command surface: the
@@ -234,6 +235,8 @@ export async function runLocalCockpitCli(argv, io = defaultIo()) {
                 return await runRelease(command, io);
             case "careers":
                 return await runCareers(command, io);
+            case "rename":
+                return await runRename(command, io);
             case "usage":
                 return await runUsage(command, io);
         }

package/dist/commands/public-root.js

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

package/dist/commands/rename-scope.js

@@ -0,0 +1,57 @@
+/**
+ * `cockpit rename list` / `cockpit rename resolve <id> --scope ...` (BLI-12378).
+ *
+ * The terminal twin of the panel's two buttons, over the same door
+ * (`/api/jarvis/rename-scope`) with the same device token. The door's reply
+ * sentence is printed as it is: it is the one JARVIS says everywhere.
+ *
+ * Metadata only in the logs: the ids and counts, never the words being renamed.
+ */
+import { askAgentDoor, emitAgentDoor, failAgentDoor, openAgentDoor } from "./agent-door.js";
+import { writeLine } from "./cli-io.js";
+const TAG = "[rename cli]";
+const DEADLINE_MS = 120_000;
+export async function runRename(command, io) {
+    const door = await openAgentDoor("rename", command, io);
+    const listing = command.action === "list";
+    const answer = await askAgentDoor(door, {
+        path: "/api/jarvis/rename-scope",
+        method: listing ? "GET" : "POST",
+        label: `rename ${command.action}`,
+        timeoutMs: DEADLINE_MS,
+        ...(listing
+            ? {}
+            : {
+                body: command.id
+                    ? { pendingId: command.id, choice: command.scope }
+                    : { wrongForm: command.wrongForm, canonicalForm: command.canonicalForm, choice: command.scope },
+            }),
+    });
+    if (!answer.ok)
+        return failAgentDoor(door, TAG, answer.reason, answer.detail);
+    const body = answer.body;
+    // An HTTP 200 with no recognisable shape is a dashboard that has not shipped
+    // this door, not an empty list.
+    if (listing ? !Array.isArray(body.pending) : typeof body.reply !== "string") {
+        return failAgentDoor(door, TAG, "rename_answer_unreadable", "Tower answered 200 with no rename receipt in it. The usual cause is a dashboard that has not shipped /api/jarvis/rename-scope yet; nothing was changed.");
+    }
+    writeLine(door.io.stderr, `${TAG} ${command.action} ${JSON.stringify({ open: body.pending?.length ?? null, ok: body.ok ?? null, scope: command.scope ?? null })}`);
+    if (door.json)
+        return emitAgentDoor(door, answer.body);
+    if (listing) {
+        const open = body.pending ?? [];
+        if (open.length === 0) {
+            writeLine(door.io.stdout, "No correction is waiting on you. Nothing was made a rule without a yes.");
+            return 0;
+        }
+        for (const row of open) {
+            const what = row.kind === "undo" ? "applied everywhere, can be undone" : "this page only, waiting for your call";
+            writeLine(door.io.stdout, `${row.id}  “${row.wrongForm}” -> ${row.canonicalForm}  (${what})`);
+        }
+        writeLine(door.io.stdout, "");
+        writeLine(door.io.stdout, "Answer one with: cockpit rename resolve <id> --scope page|everywhere|undo");
+        return 0;
+    }
+    writeLine(door.io.stdout, body.reply);
+    return body.ok === false ? 1 : 0;
+}

package/dist/commands/slack.js

@@ -226,6 +226,13 @@ function writeReadReply(command, io, payload, read, messages) {
     const styled = colorEnabled(io);
     const summary = typeof read["summary"] === "string" ? read["summary"] : "";
     const header = summary.split("\n")[0] ?? "";
+    // BLI-12381: an unknown, uncovered or empty read leads with its reason
+    // label on stdout (not only the stderr receipt), so a zero never reads as
+    // "nobody said anything".
+    const status = typeof read["status"] === "string" ? read["status"] : null;
+    if (status && (status !== "ok" || messages.length === 0)) {
+        writeLine(io.stdout, `slack read: status=${status} reason=${String(read["reason"] ?? "?")}`);
+    }
     if (header)
         writeLine(io.stdout, header);
     if (command.format === "compact") {
@@ -243,7 +250,8 @@ function writeReadReply(command, io, payload, read, messages) {
             }
         }
     }
-    if (typeof read["note"] === "string" && read["note"]) {
+    // A refusal's summary IS its note (tools-slack.ts `boundary`); print it once.
+    if (typeof read["note"] === "string" && read["note"] && read["note"] !== header) {
         writeLine(io.stdout, "");
         writeLine(io.stdout, dim(read["note"], styled));
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bli-cockpit/cli",
-  "version": "0.2.161",
+  "version": "0.2.163",
   "private": false,
   "type": "module",
   "bin": {
@@ -28,7 +28,7 @@
   },
   "dependencies": {
     "@bli-cockpit/memory-mcp": "0.1.45",
-    "@bli-cockpit/mcp": "0.1.79",
+    "@bli-cockpit/mcp": "0.1.81",
     "@bli-cockpit/telemetry-core": "0.1.57"
   },
   "repository": {