metheus-governance-mcp-cli 0.2.289 → 0.2.290

package/README.md

@@ -379,7 +379,7 @@ Behavior:
 - `bot room-audit` now suggests executable runner routes for the managed server+local intersection. Room visibility is still reported, but a bot no longer has to appear in the admin list before route automation can prepare it.
 - `bot room-audit` defaults to `monitor` only unless you pass `--role` or `--roles`.
 - `bot room-audit --apply true` writes missing suggested routes into `~/.metheus/bot-runner.json` and disables overlapping enabled routes in the same project/provider/destination/bot scope that are outside the selected role set.
-- `runner project up` is the one-command direct CLI path for Telegram project operations: it runs the same room audit, applies the selected role routes, and starts detached polling by default unless you explicitly pass `--start false`.
+- `runner project up` is the direct CLI preparation path for Telegram project operations: it runs the same room audit and applies the selected role routes. Use `runner start-detached` for persistent polling, or let the TUI call `runner.project_up` to bootstrap detached polling in one step.
 - `runner project up` can be narrowed with `--bot-name`, `--bot-id`, `--role`, or `--roles <csv>` when you do not want every suggested role route for that room.
 - `bot remove` without flags starts a guided numbered flow: provider -> bot entry -> confirm removal.
 - Telegram stores one bot file per entry under `~/.metheus/telegram-bots/<ServerBotName>.env` with generic fields:
@@ -421,7 +421,6 @@ metheus-governance-mcp-cli bot remove --provider telegram --bot-name <server_bot
 metheus-governance-mcp-cli bot verify --provider telegram --bot-name <server_bot_name> --json true
 metheus-governance-mcp-cli bot room-audit --provider telegram --project-id <project_uuid> --destination-label <room_label> --json true
 metheus-governance-mcp-cli bot room-audit --provider telegram --project-id <project_uuid> --destination-label <room_label> --apply true --json true
-metheus-governance-mcp-cli runner project up --project-id <project_uuid> --provider telegram --destination-label <room_label>
 metheus-governance-mcp-cli runner project up --project-id <project_uuid> --provider telegram --destination-label <room_label> --start false
 metheus-governance-mcp-cli runner start-detached --project-id <project_uuid> --provider telegram --destination-label <room_label>
 metheus-governance-mcp-cli runner project up --project-id <project_uuid> --provider telegram --destination-label <room_label> --bot-name <server_bot_name> --roles monitor,review --start false
@@ -556,7 +555,7 @@ metheus-governance-mcp-cli runner start --route-name telegram-monitor --concurre
 Route management:
 - `runner route add` creates one executable route by selecting the project, provider, role, server bot, and project chat destination in order.
 - `runner route add` now auto-uses the suggested route name, `5000` ms poll interval, and `enabled=true` unless you pass explicit flags or edit the route later.
-- `runner project up` is the shortest practical route bootstrap path for Telegram: it audits one project destination, writes any missing suggested routes, and starts detached polling by default unless you explicitly pass `--start false`.
+- `runner project up` is the shortest practical direct CLI preparation path for Telegram: it audits one project destination and writes any missing suggested routes. Start persistent polling separately with `runner start-detached`, or use the TUI/local tool path that bootstraps detached polling for you.
 - if room visibility probe fails but one or more enabled routes already exist for that project destination, `runner project up` now treats the probe failure as a warning and can still start those existing routes.
 - `runner project up --dry-run-delivery true` lets the started runner validate route execution without sending a real provider message.
 - In public Telegram bot conversations, the stored route role is treated as a hint only. Live room context, the current human request, and recent bot replies take priority over the stored route role hint when the local AI client decides how to answer.
@@ -569,7 +568,6 @@ Route management:
 Recommended operational path:
 
 ```bash
-metheus-governance-mcp-cli runner project up --project-id <project_uuid> --provider telegram --destination-label <room_label>
 metheus-governance-mcp-cli runner project up --project-id <project_uuid> --provider telegram --destination-label <room_label> --start false
 metheus-governance-mcp-cli runner start-detached --project-id <project_uuid> --provider telegram --destination-label <room_label>
 metheus-governance-mcp-cli runner project up --project-id <project_uuid> --provider telegram --destination-label <room_label> --bot-name <server_bot_name> --roles monitor,review --start false

package/cli.mjs

@@ -12306,7 +12306,7 @@ function resolveRunnerProjectUpExecutionPolicy(flags = {}) {
     ? boolFromRaw(startDetachedRaw, true)
     : (explicitDetachedAliasRequested
       ? boolFromRaw(flags.detached, false)
-      : !explicitStartRequested);
+      : false);
   const shouldStartRunner = startRequested || startDetachedRequested;
   return {
     applyRequested,
@@ -12323,6 +12323,24 @@ function buildRunnerStartDetachedCommand(flags = {}) {
   })].join(" ");
 }
 
+function sleepSyncMilliseconds(durationMs) {
+  const numericDurationMs = Math.max(0, intFromRaw(durationMs, 0) || 0);
+  if (numericDurationMs <= 0) return;
+  Atomics.wait(new Int32Array(new SharedArrayBuffer(4)), 0, 0, numericDurationMs);
+}
+
+function ensureDetachedRunnerBootstrapped(childPID, logFilePath = "") {
+  sleepSyncMilliseconds(1200);
+  if (isProcessAlive(childPID)) {
+    return;
+  }
+  const normalizedLogFilePath = String(logFilePath || "").trim();
+  if (normalizedLogFilePath && fs.existsSync(normalizedLogFilePath)) {
+    throw new Error(`detached runner exited during bootstrap; inspect ${normalizedLogFilePath}`);
+  }
+  throw new Error("detached runner exited during bootstrap before the polling loop became stable");
+}
+
 function buildRunnerProjectUpNextSteps({
   applyRequested,
   shouldStartRunner,
@@ -12354,7 +12372,7 @@ async function buildRunnerProjectUpResult(flags = {}) {
   const ui = createPrompter();
   try {
     if (shouldRenderPromptChrome(flags)) {
-      ui.setFlow("RUNNER PROJECT UP", "Audit one project destination, create missing runner routes, and start detached polling by default unless --start false is passed");
+      ui.setFlow("RUNNER PROJECT UP", "Audit one project destination, create missing runner routes, and optionally start polling when explicitly requested");
     }
     const provider = String(flags.provider || "").trim()
       ? normalizeBotProvider(flags.provider)
@@ -12452,7 +12470,7 @@ async function buildRunnerProjectUpResult(flags = {}) {
       route_apply_changed: Boolean(applyResult.changed),
       route_config_file: String(applyResult.filePath || auditPayload.routeSuggestionConfigFilePath || "-").trim() || "-",
       enabled_routes_for_selection: matchingRoutes.map((route) => normalizeRunnerRoute(route).name).filter(Boolean),
-      start_requested: shouldStartRunner,
+      start_requested: startRequested,
       start_detached_requested: startDetachedRequested,
       next_steps: [],
       applied_routes: ensureArray(applyResult.appliedRoutes).map((item) => safeObject(item)),
@@ -12848,12 +12866,13 @@ async function launchDetachedRunnerProcess(flags, routes, sourceCommand) {
     if (launched.status !== 0) {
       throw new Error(String(launched.stderr || launched.stdout || "failed to launch detached runner").trim());
     }
-    const childPID = intFromRawAllowZero(String(launched.stdout || "").trim().split(/\r?\n/).pop(), 0);
-    if (childPID <= 0) {
-      throw new Error("detached runner launch did not return a child process id");
-    }
-    return buildRunnerDetachedLaunchRecord(childPID, routes, startFlags, sourceCommand, detachedLogFilePath);
-  }
+    const childPID = intFromRawAllowZero(String(launched.stdout || "").trim().split(/\r?\n/).pop(), 0);
+    if (childPID <= 0) {
+      throw new Error("detached runner launch did not return a child process id");
+    }
+    ensureDetachedRunnerBootstrapped(childPID, detachedLogFilePath);
+    return buildRunnerDetachedLaunchRecord(childPID, routes, startFlags, sourceCommand, detachedLogFilePath);
+  }
   const launchTempDir = fs.mkdtempSync(path.join(os.tmpdir(), "metheus-runner-launch-"));
   const scriptPath = path.join(launchTempDir, "start-runner.sh");
   const pidFilePath = path.join(launchTempDir, "runner.pid");
@@ -12877,13 +12896,14 @@ async function launchDetachedRunnerProcess(flags, routes, sourceCommand) {
       });
       if (launched.error) {
         throw launched.error;
-      }
-      if (launched.status !== 0) {
-        throw new Error(String(launched.stderr || launched.stdout || "failed to launch detached runner in Terminal.app").trim());
-      }
-      const childPID = await waitForRunnerPIDFile(pidFilePath);
-      return buildRunnerDetachedLaunchRecord(childPID, routes, startFlags, sourceCommand, detachedLogFilePath);
-    }
+      }
+      if (launched.status !== 0) {
+        throw new Error(String(launched.stderr || launched.stdout || "failed to launch detached runner in Terminal.app").trim());
+      }
+      const childPID = await waitForRunnerPIDFile(pidFilePath);
+      ensureDetachedRunnerBootstrapped(childPID, detachedLogFilePath);
+      return buildRunnerDetachedLaunchRecord(childPID, routes, startFlags, sourceCommand, detachedLogFilePath);
+    }
     const terminalLauncher = resolveLinuxDetachedTerminalLauncher(scriptPath);
     if (!terminalLauncher) {
       throw new Error("no supported Linux terminal emulator was found (tried x-terminal-emulator, gnome-terminal, konsole, mate-terminal, tilix, alacritty, xterm). Install one of those terminals so runner.start_detached can open an independent window");
@@ -12894,12 +12914,13 @@ async function launchDetachedRunnerProcess(flags, routes, sourceCommand) {
       stdio: "ignore",
     });
     child.unref();
-    if (!child.pid) {
-      throw new Error("detached terminal launcher did not return a process id");
-    }
-    const runnerPID = await waitForRunnerPIDFile(pidFilePath);
-    return buildRunnerDetachedLaunchRecord(runnerPID, routes, startFlags, sourceCommand, detachedLogFilePath);
-  } finally {
+    if (!child.pid) {
+      throw new Error("detached terminal launcher did not return a process id");
+    }
+    const runnerPID = await waitForRunnerPIDFile(pidFilePath);
+    ensureDetachedRunnerBootstrapped(runnerPID, detachedLogFilePath);
+    return buildRunnerDetachedLaunchRecord(runnerPID, routes, startFlags, sourceCommand, detachedLogFilePath);
+  } finally {
     try {
       fs.rmSync(launchTempDir, { recursive: true, force: true });
     } catch {}
@@ -19334,15 +19355,15 @@ TELEGRAM_BOT_REVIEW_TOKEN=review-token
   try {
     const projectUpPolicy = resolveRunnerProjectUpExecutionPolicy({});
     push(
-      "runner_project_up_default_policy_prefers_detached_start",
+      "runner_project_up_default_policy_keeps_prepare_only_for_direct_cli",
       projectUpPolicy.applyRequested === true
         && projectUpPolicy.startRequested === false
-        && projectUpPolicy.startDetachedRequested === true
-        && projectUpPolicy.shouldStartRunner === true,
+        && projectUpPolicy.startDetachedRequested === false
+        && projectUpPolicy.shouldStartRunner === false,
       `apply=${projectUpPolicy.applyRequested} start=${projectUpPolicy.startRequested} detached=${projectUpPolicy.startDetachedRequested} shouldStart=${projectUpPolicy.shouldStartRunner}`,
     );
   } catch (err) {
-    push("runner_project_up_default_policy_prefers_detached_start", false, String(err?.message || err));
+    push("runner_project_up_default_policy_keeps_prepare_only_for_direct_cli", false, String(err?.message || err));
   }
 
   try {

package/package.json

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