metheus-governance-mcp-cli 0.2.65 → 0.2.66

package/README.md

@@ -227,6 +227,9 @@ Behavior:
 - `bot setup` asks for `Telegram / Slack / KakaoTalk` first, then prompts with numbered actions.
 - `bot add` without flags starts a guided question flow: provider -> server bot -> local bot key -> token -> verify -> role/AI binding -> default bot choice.
 - `bot edit` without flags starts a guided numbered flow: provider -> bot entry -> guided edit -> step-by-step field choices.
+- `bot set-default` without flags starts a guided numbered flow: provider -> bot entry -> confirm default change.
+- `bot verify` without flags starts a guided numbered flow: provider -> bot entry -> output format.
+- `bot remove` without flags starts a guided numbered flow: provider -> bot entry -> confirm removal.
 - Telegram supports named local bot entries with:
   - `SERVER_BOT_ID`
   - `USERNAME`

package/lib/bot-commands.mjs

@@ -182,6 +182,26 @@ async function promptYesNo(ui, promptText, defaultValue = true) {
   }
 }
 
+async function promptConfirmChoice(
+  ui,
+  title,
+  {
+    confirmLabel = "Confirm",
+    confirmDescription = "",
+    cancelLabel = "Cancel",
+    cancelDescription = "",
+    defaultValue = "confirm",
+  } = {},
+) {
+  const options = [
+    { value: "confirm", label: confirmLabel, description: confirmDescription },
+    { value: "cancel", label: cancelLabel, description: cancelDescription },
+  ];
+  const defaultIndex = defaultValue === "cancel" ? 1 : 0;
+  const selected = await promptChoice(ui, title, options, { defaultIndex });
+  return selected?.value === "confirm";
+}
+
 function formatChoiceLabel(option) {
   return `${String(option.label || option.value || "").trim()}${option.description ? ` - ${option.description}` : ""}`;
 }
@@ -1262,7 +1282,13 @@ async function removeTelegramBot(ui, deps) {
   const parsed = { ...state.parsed };
   const selected = await chooseTelegramEntry(ui, parsed, deps, "Select Telegram bot entry to remove");
   if (!selected) return;
-  if (!await promptYesNo(ui, `Remove Telegram bot "${selected.key}"?`, false)) {
+  if (!await promptConfirmChoice(ui, `Remove Telegram bot "${selected.key}"?`, {
+    confirmLabel: "Remove bot",
+    confirmDescription: "delete this local Telegram bot entry",
+    cancelLabel: "Cancel",
+    cancelDescription: "keep the current local bot entry",
+    defaultValue: "cancel",
+  })) {
     process.stdout.write("Cancelled.\n");
     return;
   }
@@ -1338,7 +1364,23 @@ async function verifyProviderEntry(ui, provider, flags, deps) {
       }
     }
   }
-  if (boolFromRaw(flags.json, false)) {
+  const interactiveJsonChoice = (
+    !boolFromRaw(flags.json, false)
+    && !boolFromRaw(flags["non-interactive"] ?? flags.yes, false)
+  )
+    ? await promptChoice(
+      ui,
+      "Verification output format",
+      [
+        { value: "text", label: "Text output (Recommended)", description: "human-readable summary" },
+        { value: "json", label: "JSON output", description: "machine-readable verification payload" },
+      ],
+      { defaultIndex: 0 },
+    )
+    : null;
+  const outputAsJson = boolFromRaw(flags.json, false) || interactiveJsonChoice?.value === "json";
+
+  if (outputAsJson) {
     process.stdout.write(
       `${JSON.stringify({
         ok: overallOK,
@@ -1594,9 +1636,7 @@ async function runBotVerify(ui, flags, deps) {
 }
 
 async function runBotSetDefault(ui, flags, deps) {
-  const provider = String(flags.provider || "").trim()
-    ? requireDependency(deps, "normalizeBotProvider")(flags.provider)
-    : "telegram";
+  const provider = await selectProvider(ui, flags.provider, deps);
   if (provider !== "telegram") {
     throw new Error("bot set-default currently supports only --provider telegram");
   }
@@ -1609,6 +1649,16 @@ async function runBotSetDefault(ui, flags, deps) {
   if (!selected) {
     throw new Error("Telegram bot selector is required for set-default");
   }
+  if (!nonInteractive && !await promptConfirmChoice(ui, `Set "${selected.key}" as TELEGRAM_DEFAULT_BOT_KEY?`, {
+    confirmLabel: "Set default bot",
+    confirmDescription: "make this the default local Telegram bot entry",
+    cancelLabel: "Cancel",
+    cancelDescription: "leave the current default unchanged",
+    defaultValue: "confirm",
+  })) {
+    process.stdout.write("Cancelled.\n");
+    return;
+  }
   parsed.TELEGRAM_DEFAULT_BOT_KEY = selected.key;
   const filePath = writeProviderEnvState("telegram", parsed, deps);
   process.stdout.write(`Set TELEGRAM_DEFAULT_BOT_KEY=${selected.key} in ${filePath}\n`);

package/lib/selftest-bot-commands.mjs

@@ -45,6 +45,45 @@ function readJSON(rawText) {
   return JSON.parse(String(rawText || "").trim() || "{}");
 }
 
+function readTrailingJSON(rawText) {
+  const text = String(rawText || "");
+  const start = text.indexOf("{");
+  if (start < 0) {
+    return readJSON(text);
+  }
+  let depth = 0;
+  let inString = false;
+  let escaping = false;
+  for (let index = start; index < text.length; index += 1) {
+    const char = text[index];
+    if (inString) {
+      if (escaping) {
+        escaping = false;
+      } else if (char === "\\") {
+        escaping = true;
+      } else if (char === "\"") {
+        inString = false;
+      }
+      continue;
+    }
+    if (char === "\"") {
+      inString = true;
+      continue;
+    }
+    if (char === "{") {
+      depth += 1;
+      continue;
+    }
+    if (char === "}") {
+      depth -= 1;
+      if (depth === 0) {
+        return readJSON(text.slice(start, index + 1));
+      }
+    }
+  }
+  return readJSON(text.slice(start));
+}
+
 function createMockServer() {
   const serverBots = [
     {
@@ -338,6 +377,50 @@ export async function runSelftestBotCommands(push, deps) {
       `client=${String(editedState.TELEGRAM_BOT_MAIN_TEST_AI_CLIENT || "")} model=${String(editedState.TELEGRAM_BOT_MAIN_TEST_AI_MODEL || "")}`,
     );
 
+    await runCLI({
+      cliPath,
+      args: ["bot", "set-default"],
+      env: {
+        ...env,
+        METHEUS_SCRIPTED_PROMPT_ANSWERS: JSON.stringify([
+          "1", // provider: telegram
+          "1", // bot entry: main_test
+          "1", // confirm set default
+        ]),
+      },
+    });
+    const guidedDefaultState = parseSimpleEnvText(fs.readFileSync(telegramEnvPath, "utf8"));
+    push(
+      "bot_set_default_guided_selects_entry",
+      String(guidedDefaultState.TELEGRAM_DEFAULT_BOT_KEY || "") === "main_test",
+      `default=${String(guidedDefaultState.TELEGRAM_DEFAULT_BOT_KEY || "")}`,
+    );
+
+    const guidedVerifyResult = await runCLI({
+      cliPath,
+      args: [
+        "bot", "verify",
+        "--base-url", baseURL,
+        "--timeout-seconds", "5",
+      ],
+      env: {
+        ...env,
+        METHEUS_SCRIPTED_PROMPT_ANSWERS: JSON.stringify([
+          "1", // provider: telegram
+          "1", // bot entry: main_test
+          "2", // output format: json
+        ]),
+      },
+    });
+    const guidedVerifyPayload = readTrailingJSON(guidedVerifyResult.stdout);
+    push(
+      "bot_verify_guided_can_emit_json",
+      guidedVerifyPayload.ok === true
+        && safeObject(guidedVerifyPayload.serverBinding).ok === true
+        && String(guidedVerifyPayload.client || "") === "claude",
+      `verify=${String(guidedVerifyPayload.ok)} server=${String(safeObject(guidedVerifyPayload.serverBinding).detail || "")}`,
+    );
+
     await runCLI({
       cliPath,
       args: [
@@ -435,16 +518,49 @@ export async function runSelftestBotCommands(push, deps) {
       `client=${String(aliasAddState.TELEGRAM_BOT_EXPLICIT_TEST_AI_CLIENT || "")} model=${String(aliasAddState.TELEGRAM_BOT_EXPLICIT_TEST_AI_MODEL || "")}`,
     );
 
+    const guidedRemoveBeforeList = await runCLI({
+      cliPath,
+      args: [
+        "bot", "list",
+        "--provider", "telegram",
+        "--json", "true",
+      ],
+      env,
+    });
+    const guidedRemoveBeforePayload = ensureArray(readJSON(guidedRemoveBeforeList.stdout));
+    const guidedRemoveBeforeEntry = safeObject(guidedRemoveBeforePayload[0]);
+    const guidedRemoveEntries = ensureArray(guidedRemoveBeforeEntry.entries);
+    const explicitEntryIndex = guidedRemoveEntries.findIndex((entry) => String(safeObject(entry).key || "") === "explicit_test");
+
     await runCLI({
+      cliPath,
+      args: ["bot", "remove"],
+      env: {
+        ...env,
+        METHEUS_SCRIPTED_PROMPT_ANSWERS: JSON.stringify([
+          "1", // provider: telegram
+          String(explicitEntryIndex + 1), // bot entry: explicit_test
+          "1", // confirm remove
+        ]),
+      },
+    });
+    const guidedRemoveList = await runCLI({
       cliPath,
       args: [
-        "bot", "remove",
+        "bot", "list",
         "--provider", "telegram",
-        "--bot-key", "explicit_test",
-        "--non-interactive", "true",
+        "--json", "true",
       ],
       env,
     });
+    const guidedRemovePayload = ensureArray(readJSON(guidedRemoveList.stdout));
+    const guidedRemoveEntry = safeObject(guidedRemovePayload[0]);
+    push(
+      "bot_remove_guided_deletes_selected_entry",
+      explicitEntryIndex >= 0
+        && ensureArray(guidedRemoveEntry.entries).length === 0,
+      `entries=${String(ensureArray(guidedRemoveEntry.entries).length)} selected=${String(explicitEntryIndex + 1)}`,
+    );
 
     const migratedEnv = parseSimpleEnvText(fs.readFileSync(telegramEnvPath, "utf8"));
     migratedEnv.TELEGRAM_BOT_TOKEN = "legacy-selftest-token";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "metheus-governance-mcp-cli",
-  "version": "0.2.65",
+  "version": "0.2.66",
   "description": "Metheus Governance MCP CLI (setup + stdio proxy)",
   "type": "module",
   "files": [