@bridge_gpt/mcp-server 0.2.67 → 0.2.69

package/build/conductor/epic-implementer-cli.js

@@ -31,6 +31,8 @@ import { HOOK_RUNTIME_PROBE_ACTION, HOOK_RUNTIME_PROBE_MARKER } from "./hook-run
 import { runFileScopeGuardCli } from "./file-scope-guard.js";
 import { runPostCommitHookProducer, runReferenceTransactionHookProducer } from "./git-producer.js";
 import { buildConductorDoctorReport, formatConductorDoctorReport } from "./doctor.js";
+import { createCommandDriftInspector } from "./command-drift-production.js";
+import { writeJsonDocument } from "./json-output.js";
 import { resolveSupervisorConfig } from "./supervisor-config.js";
 import { VERSION } from "../version.generated.js";
 // `runSupervisor` is imported LAZILY inside runSuperviseCommand: it transitively
@@ -80,6 +82,7 @@ export function getConductorUsage() {
         "  git-hook post-commit Run the post-commit producer (invoked by the installed hook)",
         "  git-hook reference-transaction --phase <p> --stdin-file <f>",
         "                       Run the reference-transaction producer (invoked by the hook)",
+        "  likely-tests --base <sha>  Local advisory test references for committed changes.",
         "  file-scope-guard     Warn-only: compare the branch diff against the declared",
         "                       touched-file set (always exits 0; never blocks a PR)",
         "  stop-run             Stop an epic run: block dispatch and cancel queued work",
@@ -94,6 +97,7 @@ export function getConductorUsage() {
         "  scope-reparse --epic-run-id <id> --scope-id <scope>",
         "                       Re-drive the run's OWN index scope through retry-reparse",
         "                       and schedule its parse. Refused if the scope is not the run's.",
+        "  escalate --epic-run-id <id> --ticket-key <key> --cause <cause> --action-needed <text>",
         "  dispatch-remediation --epic-run-id <id> --ticket-key <key> --context-file <path>",
         `                       Dispatch a remediation job carrying driver context (UTF-8,`,
         `                       at most DRIVER_CONTEXT_MAX_BYTES = ${DRIVER_CONTEXT_MAX_BYTES} bytes; never truncated).`,
@@ -109,7 +113,8 @@ export function getConductorUsage() {
         "  resume-run --epic-run-id <id> [--clear-quarantine] [--json]",
         "                       Resume a paused run (a quarantined one only with --clear-quarantine);",
         "                       activation is re-checked server-side. Active/no-control is a no-op.",
-        "  continue-child --epic-run-id <id> --ticket-key <key> [--reason <text>] [--json]",
+        "  continue-child --epic-run-id <id> --ticket-key <key> [--reason <text> --worker-exit-proof exited|fenced] [--json]",
+        "  grant-continuation --epic-run-id <id> --ticket-key <key> --reason <text> [--extra-continuations 1] [--extra-seconds <seconds>] [--json]",
         "                       Continue a parked child's CURRENT episode WITHOUT a reset: same attempt,",
         "                       retained work, only the allowance left. At most one dispatch per episode.",
         "  continuation-receipt --epic-run-id <id> --ticket-key <key> [--request-id <id>] [--json]",
@@ -179,6 +184,9 @@ export function getConductorUsage() {
         "  --no-deny-probe      Accepted for parity with `doctor`; inert here — `readiness`",
         "                       never spawns the deny-enforcement probe, and reports that",
         "                       check as an explicit skip naming `epic-implementer doctor`",
+        "  --epic-run-id <id>   The epic run being attached; when the live plane is bound",
+        "                       to it, the plane port and existing-plane rows pass with",
+        "                       reason own_plane instead of failing",
         "  --help               Print the readiness usage message",
         "  Note: advisory only. It blocks nothing and changes no exit code; `drive-epic`",
         "  remains the sole route-selection gate and server-side admission the sole refusal.",
@@ -303,6 +311,7 @@ const VALID_COMMANDS = new Set([
     "install-git-hooks",
     "git-hook",
     "file-scope-guard",
+    "likely-tests",
     // BAPI-872: CLI-only operator recovery verbs — never registered as MCP tools.
     "stop-run",
     "abandon-run",
@@ -310,6 +319,7 @@ const VALID_COMMANDS = new Set([
     "adopt-current-head-and-unpark",
     // BAPI-1154: CLI-only run-addressed repair verbs — never registered as MCP tools.
     "scope-reparse",
+    "escalate",
     "dispatch-remediation",
     "merge-child",
     // BAPI-1176: CLI-only run-addressed repair verbs — never registered as MCP tools.
@@ -318,6 +328,7 @@ const VALID_COMMANDS = new Set([
     "resume-run",
     // BAPI-1300: CLI-only parked-child continuation pair — never registered as MCP tools.
     "continue-child",
+    "grant-continuation",
     "continuation-receipt",
     // BAPI-1326: CLI-only allowance-preserving CI resume — never an MCP tool.
     "resume-ci",
@@ -543,11 +554,9 @@ export async function runEmitEventCommand(argv, deps = {}) {
         throw error;
     }
     if (parsed.json) {
-        console.log(JSON.stringify(result));
-    }
-    else {
-        console.log(JSON.stringify(result, null, 2));
+        return (await writeJsonDocument(JSON.stringify(result))) ? 0 : 1;
     }
+    console.log(JSON.stringify(result, null, 2));
     return 0;
 }
 // ---------------------------------------------------------------------------
@@ -649,7 +658,8 @@ export async function runSendMessageCommand(argv, deps = {}) {
     }
     const result = await sendWorkerMessage(parsed.input);
     if (parsed.json) {
-        console.log(JSON.stringify(result));
+        if (!(await writeJsonDocument(JSON.stringify(result))))
+            return 1;
     }
     else {
         console.log([
@@ -702,8 +712,7 @@ export async function runCheckMessagesCommand(argv) {
     }
     const result = await checkWorkerMessages(parsed.input);
     if (parsed.json) {
-        console.log(JSON.stringify(result));
-        return 0;
+        return (await writeJsonDocument(JSON.stringify(result))) ? 0 : 1;
     }
     const lines = [`Acknowledged ${result.acked_count} of ${result.count} message(s).`];
     for (const message of result.messages) {
@@ -728,10 +737,16 @@ export async function runDoctorCommand(argv, deps = {}) {
         return 0;
     }
     // A caller may inject deps (e.g. a fake deny inspector in tests) to stay hermetic.
-    const report = await buildConductorDoctorReport(bools.has("--no-deny-probe") ? { ...deps, skipDenyProbe: true } : deps);
+    // BAPI-1392: the warn-only committed-command drift row is read-only; a caller's
+    // injected inspector wins so tests stay hermetic.
+    const doctorDeps = {
+        inspectCommandDrift: createCommandDriftInspector({ env: deps.env ?? process.env, cwd: process.cwd() }),
+        ...deps,
+    };
+    const report = await buildConductorDoctorReport(bools.has("--no-deny-probe") ? { ...doctorDeps, skipDenyProbe: true } : doctorDeps);
     if (bools.has("--json")) {
-        console.log(JSON.stringify({ ...report.ledger, git_hooks: report.git_hooks, mcp_profile: report.mcp_profile, native_ledger: report.native_ledger, deny_enforcement: report.deny_enforcement }));
-        return 0;
+        const written = await writeJsonDocument(JSON.stringify({ ...report.ledger, git_hooks: report.git_hooks, mcp_profile: report.mcp_profile, native_ledger: report.native_ledger, deny_enforcement: report.deny_enforcement, ...(report.command_drift ? { command_drift: report.command_drift } : {}) }));
+        return written ? 0 : 1;
     }
     console.log(formatConductorDoctorReport(report));
     return 0;
@@ -1103,7 +1118,7 @@ export async function runApprovePlanCommand(argv) {
     if (!accessResult.ok) {
         const envelope = { error: "CREDENTIALS_UNAVAILABLE", status: 401, message: accessResult.error };
         if (parsed.json) {
-            console.log(JSON.stringify({ ok: false, kind: "unauthorized", error: envelope.message }));
+            await writeJsonDocument(JSON.stringify({ ok: false, kind: "unauthorized", error: envelope.message }));
         }
         else {
             console.error(`Error: ${accessResult.error}`);
@@ -1121,7 +1136,8 @@ export async function runApprovePlanCommand(argv) {
                 `the current approved version and that the blob has been stored first.`);
         }
         if (parsed.json) {
-            console.log(JSON.stringify({ ok: true, plan_hash: result.plan_hash }));
+            if (!(await writeJsonDocument(JSON.stringify({ ok: true, plan_hash: result.plan_hash }))))
+                return 1;
         }
         else {
             console.log(`Plan approved atomically. ` +
@@ -1136,7 +1152,7 @@ export async function runApprovePlanCommand(argv) {
         // rather than distinguishing 5xx (2) from other failures.
         const envelope = toConductorErrorEnvelope(error);
         if (parsed.json) {
-            console.log(JSON.stringify(envelope));
+            await writeJsonDocument(JSON.stringify(envelope));
         }
         else {
             console.error(`Error: ${formatConductorErrorLine(envelope)}`);
@@ -1190,7 +1206,7 @@ export async function runEpicStatusCommand(argv) {
     const accessResult = await resolveConductorBridgeApiAccess();
     if (!accessResult.ok) {
         if (parsed.json) {
-            console.log(JSON.stringify({ ok: false, kind: "unauthorized", error: accessResult.error }));
+            await writeJsonDocument(JSON.stringify({ ok: false, kind: "unauthorized", error: accessResult.error }));
         }
         else {
             console.error(`Error: ${accessResult.error}`);
@@ -1200,8 +1216,7 @@ export async function runEpicStatusCommand(argv) {
     try {
         const state = await fetchEpicRunState(accessResult.access, parsed.epicKey);
         if (parsed.json) {
-            console.log(JSON.stringify(state));
-            return 0;
+            return (await writeJsonDocument(JSON.stringify(state))) ? 0 : 1;
         }
         // Human-readable summary: structured status fields only, no raw payloads.
         const lines = [];
@@ -1231,7 +1246,9 @@ export async function runEpicStatusCommand(argv) {
     catch (error) {
         if (error instanceof BridgeApiError && error.status === 404) {
             if (parsed.json) {
-                console.log(JSON.stringify({ epic_key: parsed.epicKey, status: "unknown", state: null }));
+                const written = await writeJsonDocument(JSON.stringify({ epic_key: parsed.epicKey, status: "unknown", state: null }));
+                if (!written)
+                    return 1;
             }
             else {
                 console.log("No such epic found.");
@@ -1242,7 +1259,7 @@ export async function runEpicStatusCommand(argv) {
         // rather than distinguishing 5xx (2) from other failures.
         const envelope = toConductorErrorEnvelope(error);
         if (parsed.json) {
-            console.log(JSON.stringify(envelope));
+            await writeJsonDocument(JSON.stringify(envelope));
         }
         else {
             console.error(`Error: ${formatConductorErrorLine(envelope)}`);
@@ -1329,8 +1346,7 @@ export async function runPurgeCommand(argv) {
     }
     const result = await purgeConductorLedger();
     if (bools.has("--json")) {
-        console.log(JSON.stringify(result));
-        return 0;
+        return (await writeJsonDocument(JSON.stringify(result))) ? 0 : 1;
     }
     console.log([
         `Conductor ledger purged (existed: ${result.existed}).`,
@@ -1340,6 +1356,32 @@ export async function runPurgeCommand(argv) {
     ].join("\n"));
     return 0;
 }
+export async function runLikelyTestsCommand(argv, deps = {}) {
+    const { resolveLikelyTests, isFullCommitSha, validTestReference, DISCOVERY_NOTES } = await import("../executor/likely-tests.js");
+    const stdout = deps.stdout ?? ((text) => { process.stdout.write(text); });
+    const stderr = deps.stderr ?? ((text) => { process.stderr.write(text); });
+    let base;
+    if (argv.length === 2 && argv[0] === "--base")
+        base = argv[1];
+    if (argv.length === 1 && argv[0].startsWith("--base="))
+        base = argv[0].slice(7);
+    if (!base || !isFullCommitSha(base)) {
+        stderr(DISCOVERY_NOTES.invalid_arguments + "\n");
+        return 0;
+    }
+    try {
+        const result = await (deps.resolve ?? resolveLikelyTests)({ cwd: deps.cwd ?? process.cwd(), comparisonSha: base });
+        for (const reference of result.references)
+            if (validTestReference(reference))
+                stdout(reference + "\n");
+        for (const diagnostic of result.diagnostics)
+            stderr(DISCOVERY_NOTES[diagnostic] + "\n");
+    }
+    catch {
+        stderr(DISCOVERY_NOTES.git_unavailable + "\n");
+    }
+    return 0;
+}
 /**
  * Dispatch the epic-implementer CLI. Returns (asynchronously) a process exit
  * code (non-zero for parser, validation, and storage failures). The dispatcher
@@ -1351,7 +1393,9 @@ export async function runPurgeCommand(argv) {
  * profile group this dispatcher sits alongside remain unrenamed until S5; this
  * S4 slice renames only the public entry point and CLI-facing text.
  */
-export async function runEpicImplementerCli(argv) {
+export async function runEpicImplementerCli(argv, likelyTestsDeps) {
+    if (argv[0] === "likely-tests")
+        return runLikelyTestsCommand(argv.slice(1), likelyTestsDeps);
     const parsed = parseConductorArgs(argv);
     if (parsed.kind === "help") {
         console.log(getConductorUsage());
@@ -1400,6 +1444,8 @@ export async function runEpicImplementerCli(argv) {
                 // Also dispatched before any MCP construction or store open: it only
                 // imports the native binding, so it can answer on a host with no ledger.
                 return await runHookRuntimeProbeCommand();
+            case "likely-tests":
+                return runLikelyTestsCommand(parsed.argv);
             case "file-scope-guard":
                 // BAPI-507 (N-2): warn-only worker file-scope guard. Always exits 0.
                 return runFileScopeGuardCli();
@@ -1427,6 +1473,10 @@ export async function runEpicImplementerCli(argv) {
                 const { runScopeReparseCommand } = await import("./recovery-cli.js");
                 return await runScopeReparseCommand(parsed.argv);
             }
+            case "escalate": {
+                const { runEscalateCommand } = await import("./recovery-cli.js");
+                return await runEscalateCommand(parsed.argv);
+            }
             case "dispatch-remediation": {
                 const { runDispatchRemediationCommand } = await import("./recovery-cli.js");
                 return await runDispatchRemediationCommand(parsed.argv);
@@ -1453,6 +1503,10 @@ export async function runEpicImplementerCli(argv) {
                 const { runContinueChildCommand } = await import("./recovery-cli.js");
                 return await runContinueChildCommand(parsed.argv);
             }
+            case "grant-continuation": {
+                const { runGrantContinuationCommand } = await import("./recovery-cli.js");
+                return await runGrantContinuationCommand(parsed.argv);
+            }
             case "continuation-receipt": {
                 const { runContinuationReceiptCommand } = await import("./recovery-cli.js");
                 return await runContinuationReceiptCommand(parsed.argv);