@bli-cockpit/cli 0.2.199 → 0.2.201

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-args-tower-memory.js

@@ -35,6 +35,7 @@ import { parseMemoryRulesArgs } from "./memory-rules.js";
 import { optionalNonEmpty, optionalUrl, parseNamedArgs, workRootFlagValue, } from "./local-arg-values.js";
 export const MEMORY_STORE_ACTIONS = new Set([
     "rules",
+    "knowledge",
     "search",
     "save",
     "update",
@@ -88,8 +89,8 @@ const VALUE_FLAGS = [
     "--source",
 ];
 export function parseMemoryStoreArgs(action, args) {
-    if (action === "rules")
-        return parseMemoryRulesArgs(args);
+    if (action === "rules" || action === "knowledge")
+        return parseMemoryRulesArgs(args, action);
     const values = parseNamedArgs(args, { allowedFlags: ALLOWED_FLAGS, valueFlags: VALUE_FLAGS });
     const rest = values.positionals.slice(1);
     const containers = values.flagValues.get("--container") ?? [];

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/memory-rules.js

@@ -1,15 +1,15 @@
 import { askAgentDoor, emitAgentDoor, failAgentDoor } from "./agent-door.js";
 import { writeLine } from "./cli-io.js";
 import { optionalNonEmpty, optionalUrl, parseNamedArgs } from "./local-arg-values.js";
-export function parseMemoryRulesArgs(args) {
-    const values = parseNamedArgs(args, { allowedFlags: ["--person", "--project", "--limit", "--json", "--home", "--dashboard-url"],
+export function parseMemoryRulesArgs(args, action = "rules") {
+    const values = parseNamedArgs(args, { allowedFlags: ["--person", "--project", "--limit", "--json", "--by-project", "--home", "--dashboard-url"],
         valueFlags: ["--person", "--project", "--limit", "--home", "--dashboard-url"] });
     if (values.positionals.slice(1).length)
-        throw new Error("memory rules takes no positionals; use --person and --project.");
+        throw new Error(`memory ${action} takes no positionals; use --person and --project.`);
     const raw = values.flags.get("--limit");
     if (raw !== undefined && (!/^\d+$/.test(raw) || Number(raw) < 1 || Number(raw) > 1000))
         throw new Error("--limit must be between 1 and 1000.");
-    return { kind: "memory-store", action: "rules", containerTags: [],
+    return { kind: "memory-store", action, byProject: values.booleans.has("--by-project"), containerTags: [],
         person: optionalNonEmpty(values.flags.get("--person")), project: optionalNonEmpty(values.flags.get("--project")),
         limit: raw === undefined ? undefined : Number(raw), json: values.booleans.has("--json"),
         homeDir: optionalNonEmpty(values.flags.get("--home")), dashboardUrl: optionalUrl(values.flags.get("--dashboard-url")) };
@@ -22,21 +22,32 @@ export async function runMemoryRules(command, door) {
         params.set("project", command.project);
     if (command.limit !== undefined)
         params.set("limit", String(command.limit));
-    const answer = await askAgentDoor(door, { path: `/api/memory/rules${params.size ? `?${params}` : ""}`,
-        method: "GET", label: "memory rules", timeoutMs: 60_000 });
+    const answer = await askAgentDoor(door, { path: `/api/memory/${command.action}${params.size ? `?${params}` : ""}`,
+        method: "GET", label: `memory ${command.action}`, timeoutMs: 60_000 });
     if (!answer.ok)
         return failAgentDoor(door, "[memory cli]", answer.reason, answer.detail);
     const body = answer.body;
-    writeLine(door.io.stderr, `[memory cli] rules ${JSON.stringify({ reason: body.capped ? "capped" : "exhausted", returned: body.returned, groups: body.groups.length })}`);
+    writeLine(door.io.stderr, `[memory cli] ${command.action} ${JSON.stringify({ reason: body.capped ? "capped" : "exhausted", returned: body.returned, groups: body.projectGroups?.length ?? body.groups?.length ?? 0 })}`);
     if (door.json)
         return emitAgentDoor(door, answer.body);
-    for (const group of body.groups) {
-        writeLine(door.io.stdout, group.owner);
-        for (const rule of group.rules)
-            writeLine(door.io.stdout, `  [${rule.created_at.slice(0, 10)}] ${rule.project ? `[${rule.project}] ` : ""}${rule.text}${group.owner === "unattributed" ? ` (space: ${rule.container})` : ""}`);
+    if (command.byProject || command.action === "knowledge") {
+        for (const group of body.projectGroups ?? []) {
+            writeLine(door.io.stdout, `${group.project} (${group.count}) : ${group.headline.split("\n")[0]}`);
+            for (const topic of group.topics) {
+                writeLine(door.io.stdout, `  ${topic.title}`);
+                for (const memory of topic.memories)
+                    writeLine(door.io.stdout, `    ${memory.text}${memory.repetition > 1 ? ` (said ${memory.repetition} times)` : ""}${memory.importance !== null ? ` [importance ${memory.importance}/100]` : ""}`);
+            }
+        }
     }
+    else
+        for (const group of body.groups) {
+            writeLine(door.io.stdout, group.owner);
+            for (const rule of group.rules)
+                writeLine(door.io.stdout, `  [${rule.created_at.slice(0, 10)}] ${rule.project ? `[${rule.project}] ` : ""}${rule.text}${group.owner === "unattributed" ? ` (space: ${rule.container})` : ""}`);
+        }
     if (!body.returned)
-        writeLine(door.io.stdout, "No ground rules matched.");
+        writeLine(door.io.stdout, `No ${command.action === "knowledge" ? "project knowledge" : "ground rules"} matched.`);
     if (body.capped)
         writeLine(door.io.stdout, "Result capped. Narrow by person or project, or increase --limit (max 1000).");
     return 0;

package/dist/commands/memory-store.js

@@ -65,6 +65,7 @@ const FORGET_MAX_CHARS = 10_000;
 export async function runMemoryStore(command, io) {
     const door = await openAgentDoor("memory", command, io);
     switch (command.action) {
+        case "knowledge":
         case "rules":
             return runMemoryRules(command, door);
         case "search":

package/dist/commands/memory-verb-list.js

@@ -26,7 +26,7 @@ export function runMemoryVerbList(io, json) {
             ok: true,
             noun: "memory",
             machine_verbs: ["install", "status"],
-            store_verbs: ["search", "save", "update", "forget", "private", "public", "profile", "consolidation", "requeue"],
+            store_verbs: ["rules", "knowledge", "search", "save", "update", "forget", "private", "public", "profile", "consolidation", "requeue"],
             log_verbs: ["log"],
         }));
         return 0;
@@ -44,7 +44,8 @@ export function runMemoryVerbList(io, json) {
         "      you can forget or update only your own memories (a super admin, any)",
         "  cockpit memory private <id> [--json]       only you and super admins can read this memory",
         "  cockpit memory public <id> [--json]        every member can read it again (how every memory starts)",
-        "  cockpit memory rules [--person <name>] [--project <name>] [--limit <n>] [--json]",
+        "  cockpit memory rules [--person <name>] [--project <name-or-slug>] [--by-project] [--limit <n>] [--json]",
+        "  cockpit memory knowledge [--person <name>] [--project <name-or-slug>] [--by-project] [--limit <n>] [--json]",
         "  cockpit memory profile [--container <tag>] [--json]",
         "      the compiled one-page briefing every session opens with; a read, never a compile",
         "  cockpit memory consolidation [--container <tag>] [--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.201");
     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.201",
   "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.113",
     "@bli-cockpit/telemetry-core": "0.1.69"
   },
   "repository": {