@bli-cockpit/cli 0.2.199 → 0.2.200

package/dist/commands/doctor-report.js

@@ -164,6 +164,7 @@ export function repairReceiptLines(repairs) {
 }
 export function doctorNeedsPerson(rows, receipt) {
     const defaults = {
+        "slack-connected": "cockpit slack connect",
         "cli-latest": "npm i -g @bli-cockpit/cli && cockpit doctor",
         "authed": "cockpit doctor",
         "roots-ok": "cockpit doctor",

package/dist/commands/doctor-slack.js

@@ -0,0 +1,25 @@
+import { DEFAULT_DASHBOARD_URL } from "../local-state.js";
+import { asRecord, callTower, openTower } from "./tower-command.js";
+export async function checkSlackConnected(context) {
+    const message = "Slack connected for private-channel coverage";
+    try {
+        const tower = await openTower("doctor", context.command, context.io);
+        const response = await callTower(tower, { path: "/api/slack/connections", label: "doctor-slack", timeoutMs: 5_000 });
+        if (!response.ok)
+            return { id: "slack-connected", status: "needs_fix", code: response.reason, message: `${message}: could not read connection state.`, nextAction: `${tower.dashboardUrl}/settings/personal` };
+        const connections = asRecord(response.body)["connections"];
+        if (!connections || connections.length !== 2)
+            throw new Error("connection_state_incomplete");
+        const missing = connections.filter(row => !row["connected"]);
+        const links = missing.map(row => new URL(String(row["connectPath"]), tower.dashboardUrl).toString());
+        console.error("[doctor slack] checked", JSON.stringify({ missing: missing.length, reason: missing.length ? "slack_connect_required" : "ok" }));
+        return { id: "slack-connected", status: missing.length ? "needs_fix" : "ok", code: missing.length ? "slack_connect_required" : "slack_connected",
+            message: missing.length ? `${message}: connect ${missing.map(row => row["workspace"]).join(", ")} at ${links.join(" or ")}` : message,
+            ...(missing.length ? { nextAction: links.join(" ") } : {}),
+        };
+    }
+    catch {
+        console.error("[doctor slack] refused", JSON.stringify({ reason: "connection_read_failed" }));
+        return { id: "slack-connected", status: "skipped", code: "connection_read_failed", message: `${message}: unavailable until this machine is signed in.`, nextAction: `${context.command.dashboardUrl ?? DEFAULT_DASHBOARD_URL}/settings/personal` };
+    }
+}

package/dist/commands/doctor.js

@@ -1,3 +1,4 @@
+import { checkSlackConnected } from "./doctor-slack.js";
 import { checkUsageTelemetry, fixUsageTelemetry } from "./doctor-usage-telemetry.js";
 import { LOCAL_COLLECTOR_VERSION } from "../local-state.js";
 import { isSemverBelow } from "../scheduled-self-update.js";
@@ -153,6 +154,7 @@ export function doctorInvariantTable() {
 function doctorInvariants() {
     return [
         { id: "cli-latest", check: checkCliLatest, fix: fixCliLatest },
+        { id: "slack-connected", check: context => context.deps.checkSlackConnected ? context.deps.checkSlackConnected(context) : Promise.resolve({ id: "slack-connected", status: "skipped", code: "not_probed", message: "Slack connection check not provided." }) },
         {
             id: "authed",
             check: (context) => context.deps.readAuth(context),
@@ -248,6 +250,7 @@ function defaultDoctorDeps(hooks) {
         readRootsCoverWork,
         resolveAndSaveRoots: (context) => hooks.resolveAndSaveRoots(context.command, context.io),
         checkSingleInstall: checkSingleInstallState,
+        checkSlackConnected,
         checkAutostart: checkAutostartState,
         fixAutostart: fixAutostartState,
         checkUsageTelemetry,

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

@@ -193,11 +193,11 @@ export function parseSlackArgs(args) {
         ],
     });
     if (values.positionals.length > 1) {
-        throw new Error("slack accepts one action: coverage or read.");
+        throw new Error("slack accepts one action: coverage, read, connect or disconnect.");
     }
     const rawAction = values.positionals[0] ?? "coverage";
-    if (rawAction !== "coverage" && rawAction !== "read") {
-        throw new Error("slack action must be coverage or read.");
+    if (!["coverage", "read", "connect", "disconnect"].includes(rawAction)) {
+        throw new Error("slack action must be coverage, read, connect or disconnect.");
     }
     const base = {
         homeDir: optionalNonEmpty(values.flags.get("--home")),
@@ -210,6 +210,14 @@ export function parseSlackArgs(args) {
     if (values.booleans.has("--list") && rawAction !== "coverage") {
         throw new Error("--list belongs to `cockpit slack coverage`.");
     }
+    if (rawAction === "connect" || rawAction === "disconnect") {
+        const workspace = optionalNonEmpty(values.flags.get("--workspace"));
+        if (workspace && !SLACK_WORKSPACE_KEYS.includes(workspace))
+            throw new Error("Unknown Slack workspace.");
+        if (rawAction === "disconnect" && !workspace)
+            throw new Error("slack disconnect needs --workspace bli|blue_pearl.");
+        return { kind: "slack", action: rawAction, workspace, ...base };
+    }
     if (rawAction === "coverage") {
         // `--workspace` here is a Slack workspace key, NOT the collection-root
         // `--workspace <path>` every setup command takes. Named at the point of use

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

@@ -262,7 +262,7 @@ export const TOWER_COMMAND_HELP = [
     [
         "slack",
         [
-            "Usage: cockpit slack [coverage [--workspace bli|blue_pearl] [--stale-only] [--list] | read [filters] [--format compact|full]] [--json]",
+            "Usage: cockpit slack [connect [--workspace bli|blue_pearl] | disconnect --workspace bli|blue_pearl | coverage [--workspace bli|blue_pearl] [--stale-only] [--list] | read [filters] [--format compact|full]] [--json]",
             "",
             "  cockpit slack coverage",
             "      What Cockpit's Slack bot can see: how many channels it is in, what each",

package/dist/commands/local-help.js

@@ -111,7 +111,7 @@ export function localCommandHelp(command) {
         "  cockpit models [list|show <provider:model>|compare <provider:model> <provider:model> [...]] [--highlight] [--dashboard-url <url>] [--json]",
         "  cockpit scout [board|start|dismiss|undo <experiment-id>] [--days <n>] [--dashboard-url <url>] [--json]",
         "  cockpit ops [status [--job <id>] [--skips] [--memory] [--models] | recompile --person <email|name|id> [--dry-run]] [--dashboard-url <url>] [--json]",
-        "  cockpit slack [coverage [--workspace bli|blue_pearl] [--stale-only] [--list] | read [--person <p>] [--channel <c>] [--query <text> [--word]] [--since <YYYY-MM-DD>] [--until <YYYY-MM-DD>] [--limit <n>] [--format compact|full]] [--json]",
+        "  cockpit slack [connect [--workspace bli|blue_pearl] | disconnect --workspace bli|blue_pearl | coverage [--workspace bli|blue_pearl] [--stale-only] [--list] | read [--person <p>] [--channel <c>] [--query <text> [--word]] [--since <YYYY-MM-DD>] [--until <YYYY-MM-DD>] [--limit <n>] [--format compact|full]] [--json]",
         "  cockpit settings [personal [--chat-model <key>] [--brief-model <key>] | switches [set <key> <value>] | models [set --chat <key>] [--memory <id>] | cli-floor [<version>] | env list|set --project <p> --file <f> --content-stdin|delete --id <uuid> [--yes]] [--json]",
         "  cockpit team [members | invite <email> --role <role> [--team-id <uuid>] | role <userId> --role <role> [--yes] | device [list] | device revoke <id|name> --reason <label> [--note \"<text>\"] [--yes]] [--json]",
         "  cockpit workbook [<project> [<doc>]] [--section <id>] [--markdown] [--width <n>] [--dashboard-url <url>] [--json]",

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.199");
+    writeLine(io?.stdout ?? process.stdout, "0.2.200");
     return 0;
   }
 

package/dist/commands/slack.js

@@ -24,6 +24,24 @@ import { colorEnabled, dim, writeLine } from "./cli-io.js";
 import { asRecord, callTower, openTower, writeCommandFailure } from "./tower-command.js";
 export async function runSlack(command, io) {
     const tower = await openTower("slack", command, io);
+    if (command.action === "connect" || command.action === "disconnect") {
+        const result = await callTower(tower, { path: `/api/slack/connections${command.action === "disconnect" ? `?workspace=${command.workspace}` : ""}`,
+            method: command.action === "disconnect" ? "DELETE" : "GET", label: "slack-connection" });
+        if (!result.ok)
+            return writeCommandFailure(io, command.json, result);
+        writeLine(io.stderr, `[slack cli] connection answered ${JSON.stringify({ action: command.action, workspace: command.workspace ?? "all", reason: "ok" })}`);
+        const body = asRecord(result.body);
+        if (command.json)
+            writeLine(io.stdout, JSON.stringify(body));
+        else if (command.action === "disconnect")
+            writeLine(io.stdout, `Disconnected Slack for ${command.workspace}.`);
+        else
+            for (const connection of (body["connections"] ?? [])) {
+                if (!command.workspace || command.workspace === connection["workspace"])
+                    writeLine(io.stdout, `${connection["workspace"]}: ${connection["connected"] ? "connected" : "Connect Slack"} ${new URL(String(connection["connectPath"]), tower.dashboardUrl).toString()}`);
+            }
+        return 0;
+    }
     return command.action === "coverage"
         ? runCoverageAction(command, io, tower)
         : runReadAction(command, io, tower);
@@ -86,6 +104,12 @@ function coverageLines(coverage, workspaces, staleOnly, list, paint) {
     for (const workspace of workspaces) {
         writeWorkspaceCoverageLines(lines, workspace, staleAfter, staleOnly, list, paint);
     }
+    for (const entry of (coverage["privateCoverage"] ?? [])) {
+        lines.push(`${entry["workspace"]}: ${entry["botMemberCount"]}/${entry["privateChannelsKnown"]} known private channels have the bot; ${entry["peopleConnected"]} people connected; as of ${entry["asOf"] ?? "no scan yet"}`);
+        lines.push(`  Not connected: ${(entry["peopleNotConnected"] ?? []).join(", ") || "none"}`);
+    }
+    if (coverage["privateCoverageError"])
+        lines.push(`Private coverage unavailable: ${coverage["privateCoverageError"]}`);
     writeCoverageFooter(lines, coverage, paint);
     return lines;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bli-cockpit/cli",
-  "version": "0.2.199",
+  "version": "0.2.200",
   "private": false,
   "type": "module",
   "bin": {
@@ -28,7 +28,7 @@
   },
   "dependencies": {
     "@bli-cockpit/memory-mcp": "0.1.60",
-    "@bli-cockpit/mcp": "0.1.111",
+    "@bli-cockpit/mcp": "0.1.112",
     "@bli-cockpit/telemetry-core": "0.1.69"
   },
   "repository": {