@miraland-labs/conduit-bridge 0.16.135 → 0.16.138

package/dist/ensure-test-evidence.js

@@ -210,7 +210,7 @@ export async function captureVerificationFailure(input) {
         // shown to be the same bug.
         const text = boundedFailureDetail(command, result.stdout.trim() ? result.stdout.trim().split("\n") : [], result.stderr.trim() ? result.stderr.trim().split("\n") : [], result.code, DIAGNOSIS_DETAIL_MAX_CHARS);
         if (text.trim())
-            return text;
+            return { command, detail: text };
     }
     // Every bounded command passed (or none could start): agent failed for another reason.
     return null;
@@ -456,6 +456,18 @@ export function detectGateRanNothing(command, stdout, stderr, code) {
     }
     return null;
 }
+/**
+ * A declared gate that ran and failed. Carries the command as a field so the verification signal
+ * can name the gate without reading it back out of this error's text.
+ */
+export class VerificationFailedError extends Error {
+    command;
+    constructor(command, detail) {
+        super(`Agent report: Verification failed (${command}): ${detail}`);
+        this.name = "VerificationFailedError";
+        this.command = command;
+    }
+}
 /**
  * A declared gate that ran no tests, or that only printed its own usage text: a misconfigured gate,
  * not a failed one. Carries the typed fields `execution.ts` needs to emit the T1 signal — the
@@ -520,7 +532,7 @@ export async function ensureTestEvidence(input) {
                 elided: detail.includes("omitted"),
                 ends_with_exit: detail.trimEnd().endsWith(`exit ${result.code}`),
             }));
-            throw new Error(`Agent report: Verification failed (${command}): ${detail}`);
+            throw new VerificationFailedError(command, detail);
         }
         // Empty exit-0 is not proof — a no-op recipe (`@true`) must not clear required_evidence: test.
         if (stdoutLines.length === 0 && stderrLines.length === 0) {

package/dist/execution.js

@@ -26,7 +26,7 @@ import { ensureLandCommit } from "./ensure-land-commit.js";
 import { AGENT_NO_LAND_COMMIT_PREFIX, AgentNoLandCommitError, agentNoLandCommitMessage, requiresLandCommit } from "./land-contract.js";
 import { agentClaimsRepositoryWork, claimsVsGitMismatch, isLandTreeEmpty, reportClaimsRepositoryWork, } from "./land-git-reality.js";
 const execFileAsync = promisify(execFile);
-import { boundedTail, captureVerificationFailure, detectGateRanNothing, ensureTestEvidence, needsTestEvidence, runBoundedVerificationCommand, verificationEvidenceCommands, VerificationGateRanNothingError } from "./ensure-test-evidence.js";
+import { boundedTail, captureVerificationFailure, detectGateRanNothing, ensureTestEvidence, needsTestEvidence, runBoundedVerificationCommand, verificationEvidenceCommands, VerificationFailedError, VerificationGateRanNothingError } from "./ensure-test-evidence.js";
 import { ensureChangeEvidence } from "./ensure-change-evidence.js";
 import { ensureDemonstrationEvidence } from "./ensure-demonstration-evidence.js";
 import { ensureNormativeEvidence } from "./ensure-normative-evidence.js";
@@ -471,11 +471,6 @@ export function parseDiagnosticRepairBrief(text) {
     }
     return parsed.data;
 }
-/**
- * The gate command inside the witness line Bridge itself wrote. Mirrored as
- * VERIFICATION_GATE_PATTERN in src/conductor/repair.ts.
- */
-const VERIFICATION_GATE_PATTERN = /Verification failed \(([^)]+)\)/;
 /** Prompt for Option C: IDE-depth understanding in the retained tree, with write authority absent. */
 export function buildDiagnosticPrompt(input) {
     const pkg = input.workPackage;
@@ -488,7 +483,7 @@ export function buildDiagnosticPrompt(input) {
     // Bridge re-runs the gate itself after the next authoring run. Without this rule the diagnosis
     // reads the gate as the agent's own command and briefs the agent to run a different one, which
     // cannot change what Bridge runs: the briefed run then fails on identical output.
-    const gate = VERIFICATION_GATE_PATTERN.exec(input.failureOutput)?.[1]?.trim();
+    const gate = input.gateCommand;
     return [
         "You are Conductor's read-only diagnosis lane. Diagnose the exact failed authoring run in the current retained worktree.",
         "Inspect the actual code, git diff, nearby callers, and relevant tests. Run only the supplied project verification commands when useful.",
@@ -1413,6 +1408,7 @@ async function runAttempt(client, config, driver, workspace, brief, taskId, watc
             spec,
             workPackage,
             failureOutput: diagnosticFailure,
+            gateCommand: workPackage.diagnostic_context?.gate_command ?? null,
             verificationCommands: attemptBrief?.verification ?? [],
         })
         : buildAssignmentPrompt({
@@ -1484,7 +1480,7 @@ async function runAttempt(client, config, driver, workspace, brief, taskId, watc
     // gate that moved instead of showing two hex strings.
     await client.updateAttempt(taskId, { executionFacts, verificationCommands });
     await client.attemptRequest(taskId, "progress", {
-        phase: diagnosis ? "inspecting" : "changing",
+        phase: diagnosis ? "inspecting" : "agent_running",
         execution_facts: executionFacts,
         message: resuming
             ? `Resuming ${driver.name} after Bridge restart with proven session and worktree.`
@@ -1728,15 +1724,22 @@ async function runAttempt(client, config, driver, workspace, brief, taskId, watc
             // transient retry — the one verdict that would run the capped work again at the same price.
             const turnCapReached = result.signal?.kind === "turn_cap";
             const recoverVerification = grants.includes("test_run") && !subscriptionExhausted(agentMessage) && !turnCapReached;
-            if (recoverVerification)
+            if (recoverVerification) {
                 watchdog.phase("verification");
-            const verificationDetail = recoverVerification
+                await client.attemptRequest(taskId, "progress", {
+                    phase: "verifying",
+                    message: "Bridge is running the project's verification commands to read the failure.",
+                    idempotency_key: `bridge:progress:${active.attemptId}:verify-failure`,
+                });
+            }
+            const capturedVerification = recoverVerification
                 ? await captureVerificationFailure({
                     workspace: attemptWorkspace,
                     verificationCommands: attemptBrief?.verification ?? [],
                     acceptance: spec.acceptance ?? [],
                 })
                 : null;
+            const verificationDetail = capturedVerification?.detail ?? null;
             // The vendor named the model as the cause, so the model is what the retry must change. The
             // model rides in the prefix: the control plane cannot read Bridge's selection any other way.
             const diffStat = verificationDetail && attemptWorkspace && startCommit
@@ -1771,8 +1774,8 @@ async function runAttempt(client, config, driver, workspace, brief, taskId, watc
                 : subscriptionExhausted(agentMessage)
                     ? { phase: "agent", witness: "vendor", kind: "quota", vendor_message: agentMessage.slice(0, 2_000) }
                     : result.signal
-                        ? verificationDetail
-                            ? { ...result.signal, phase: "verification", witness: "bridge", stderr_tail: undefined, stdout_tail: undefined, ...(diffStat ? { diff_stat: diffStat } : {}) }
+                        ? capturedVerification
+                            ? { ...result.signal, phase: "verification", witness: "bridge", kind: "verification_failed", command: capturedVerification.command, stderr_tail: undefined, stdout_tail: undefined, ...(diffStat ? { diff_stat: diffStat } : {}) }
                             : result.signal
                         : undefined;
             const response = await queueTerminal(client, taskId, { action: "fail", body: {
@@ -2144,6 +2147,13 @@ async function runAttempt(client, config, driver, workspace, brief, taskId, watc
                 ? await changedPathsSince(attemptWorkspace, deliveryBaseCommit)
                 : null;
             // Mechanical test path: agent often forgets verbatim make test / npm test output.
+            if (grants.includes("test_run")) {
+                await client.attemptRequest(taskId, "progress", {
+                    phase: "verifying",
+                    message: "Bridge is running the project's verification commands.",
+                    idempotency_key: `bridge:progress:${active.attemptId}:verify`,
+                });
+            }
             report = await ensureTestEvidence({
                 workspace: attemptWorkspace,
                 report,
@@ -2221,7 +2231,8 @@ async function runAttempt(client, config, driver, workspace, brief, taskId, watc
             // the same verify-class pre-delivery failure as the driver-failed path above: preserve the
             // exact tree and send the witness marker that arms Conductor diagnosis. Treating it as a
             // delivery-envelope defect would stop before Invariant 23 ever ran.
-            const verificationFailed = /^Agent report: Verification failed \(/.test(message);
+            // The command is the field on the throw, never a parse of this message.
+            const verificationFailed = error instanceof VerificationFailedError;
             const diffStat = verificationFailed && attemptWorkspace
                 ? await diffStatSince(attemptWorkspace, deliveryBaseCommit)
                     .then((stat) => stat?.slice(0, 8_000) ?? null)
@@ -2244,6 +2255,7 @@ async function runAttempt(client, config, driver, workspace, brief, taskId, watc
                             phase: "verification",
                             witness: "bridge",
                             kind: "verification_failed",
+                            command: error.command,
                             ...(diffStat ? { diff_stat: diffStat } : {}),
                         } } : {}),
                     // A rework's own PR may already be open (ensureDeliveryPullRequest ran before this

package/dist/work-package-schema.js

@@ -56,6 +56,8 @@ export const workPackageSchema = z.object({
     diagnostic_context: z.object({
         source_attempt_id: z.string().uuid(),
         failure_output: z.string().min(1).max(12_000),
+        /** The gate Bridge re-runs, from the failed attempt's verification signal; null names no gate. */
+        gate_command: z.string().min(1).max(2_000).nullable(),
     }).strict().optional(),
     /** Working/Delivery language for Bridge assignment prose. */
     working_language: z.enum(["en", "zh"]).optional(),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@miraland-labs/conduit-bridge",
-  "version": "0.16.135",
+  "version": "0.16.138",
   "description": "Conduit Bridge CLI \u2014 join, connect, disconnect, multi-driver lanes, and run Claude Code / Codex / Cursor / OpenCode / Pi / Kiro / Antigravity / Grok Build agents for a Conduit organization",
   "type": "module",
   "bin": {