@miraland-labs/conduit-bridge 0.16.160 → 0.16.162

package/dist/ensure-land-commit.js

@@ -40,19 +40,19 @@ export async function ensureLandCommit(input) {
     const listPending = input.listUncommittedPaths ?? ((workspace) => uncommittedPaths(workspace));
     const commitPaths = input.commitPaths ?? commitScopedPaths;
     const ahead = await countAhead(input.workspace, base);
-    if (ahead !== null && ahead > 0) {
-        return { kind: "already_landed", headCommit: await readHead(input.workspace) };
-    }
     const pending = filterPathsInScope(await listPending(input.workspace), input.spec.change_scope ?? []);
-    if (!pending.length) {
-        throw new AgentNoLandCommitError(base);
+    if (pending.length) {
+        await commitPaths(input.workspace, pending);
+        const afterAhead = await countAhead(input.workspace, base);
+        if (afterAhead === 0) {
+            throw new AgentNoLandCommitError(base);
+        }
+        return { kind: "committed", headCommit: await readHead(input.workspace), paths: pending };
     }
-    await commitPaths(input.workspace, pending);
-    const afterAhead = await countAhead(input.workspace, base);
-    if (afterAhead === 0) {
-        throw new AgentNoLandCommitError(base);
+    if (ahead !== null && ahead > 0) {
+        return { kind: "already_landed", headCommit: await readHead(input.workspace) };
     }
-    return { kind: "committed", headCommit: await readHead(input.workspace), paths: pending };
+    throw new AgentNoLandCommitError(base);
 }
 async function currentBranch(workspace) {
     const { stdout } = await execFileAsync("git", ["-C", workspace, "branch", "--show-current"], {

package/dist/ensure-test-evidence.js

@@ -240,20 +240,33 @@ export function boundedDetailLines(lines, maxLines, identifiers = []) {
     // is proved by one line in the middle of a long suite, and a reviewer must not infer it from the
     // summary. So the lines that name an identifier the criteria name are kept too, in run order. The
     // cap keeps a broad identifier that matches every line from storing the whole run.
+    // A run of several suites prints one summary per suite, and only the last one sits in the tail:
+    // cargo's "test result: ok. 614 passed" for the cli binary says nothing about the core binary. A
+    // criterion settled by the totals (a test count delta) needs every summary, so they are kept too.
+    const summaries = middle.filter((line) => SUITE_SUMMARY.some((form) => form.test(line))).slice(0, NAMED_DETAIL_LINES_MAX);
     const named = identifiers.length === 0
         ? []
-        : [...new Set(middle.filter((line) => identifiers.some((id) => line.includes(id))))]
+        : [...new Set(middle.filter((line) => !summaries.includes(line) && identifiers.some((id) => line.includes(id))))]
             .slice(0, NAMED_DETAIL_LINES_MAX)
             .map((line) => (line.length > 500 ? `${line.slice(0, 500)}…` : line));
-    const dropped = middle.length - named.length;
+    const dropped = middle.length - summaries.length - named.length;
     return [
         ...lines.slice(0, head),
         `… ${dropped} line${dropped === 1 ? "" : "s"} omitted …`,
+        ...summaries,
         ...named,
         ...lines.slice(lines.length - tail),
     ];
 }
 const NAMED_DETAIL_LINES_MAX = 40;
+/** One suite's totals line: cargo, vitest, unittest, pytest, go test. */
+const SUITE_SUMMARY = [
+    /^test result: /,
+    /^\s*(Test Files|Tests)\s+\d/,
+    /^Ran \d+ tests?\b/,
+    /^=+ .*\b(passed|failed)\b.* in [\d.]+s =+$/,
+    /^(ok|FAIL)\s+\S+\s+[\d.]+s$/,
+];
 /**
  * The test and symbol names the acceptance criteria name: every backticked string, and every
  * snake_case, camelCase, dotted or `::` token of at least 8 characters. An ordinary English word has

package/dist/investigation.js

@@ -11,6 +11,7 @@ import { boundedTail, runBoundedVerificationCommand } from "./ensure-test-eviden
 import { createAttemptWorktree, removeAttemptWorktree } from "./attempt-worktree.js";
 import { buildWorkspaceBrief, ensureCommitAvailable, normalizeRepositoryUrl } from "./brief.js";
 import { headCommitOrNull } from "./git.js";
+import { ConduitRequestError } from "./client.js";
 import { learnDriverFuel, resolveAssignmentModel, changedPathsSince, runDeliveryVerificationGates, discardWorktreeWrites } from "./execution.js";
 import { DRIVERS, execute, extractAgentReportJsonText, parseJsonObjectCandidate } from "./driver.js";
 import { pickDriverForClaim, resolveDriverFuel, supportsReadOnlyDiagnosis } from "./drivers.js";
@@ -773,13 +774,24 @@ export async function executeNextInvestigation(client, config, workspace, brief,
     const fingerprintBefore = await workspaceFingerprint(workspace);
     let attemptWorkspace = null;
     const runId = investigationRunId(assignment.id);
-    // Renew inside the lease for as long as the driver runs; a renewal the control plane refuses
-    // means the run is already over for it, and the settle below reports that honestly.
+    // Renew inside the lease for as long as the driver runs. A terminal refusal (`invalid_lease`)
+    // means the control plane already took the run back — abort the driver and do not settle.
+    const runAbort = new AbortController();
+    let leaseTakenBack = false;
     const renewTimer = setInterval(() => {
         void client.request(`/runner/v1/investigations/${assignment.id}/renew`, {
             method: "POST",
             body: JSON.stringify({ lease_token: leaseToken }),
-        }).catch((error) => console.error(`Investigation ${assignment.id} lease renewal failed: ${redactSecrets(error instanceof Error ? error.message : "unknown")}`));
+        }).catch((error) => {
+            if (error instanceof ConduitRequestError && error.code === "invalid_lease") {
+                leaseTakenBack = true;
+                clearInterval(renewTimer);
+                runAbort.abort();
+                console.error(`Investigation ${assignment.id} stopped: lease is no longer valid`);
+                return;
+            }
+            console.error(`Investigation ${assignment.id} lease renewal failed: ${redactSecrets(error instanceof Error ? error.message : "unknown")}`);
+        });
     }, options.renewIntervalMs ?? INVESTIGATION_RENEW_INTERVAL_MS);
     renewTimer.unref?.();
     let heartbeatRunning = false;
@@ -853,7 +865,10 @@ export async function executeNextInvestigation(client, config, workspace, brief,
             fuel,
             timeoutMs: Math.min(timeoutMs ?? assignment.budget.max_duration_ms, assignment.budget.max_duration_ms),
             maxTurns: config.maxTurns,
+            signal: runAbort.signal,
         });
+        if (leaseTakenBack || runAbort.signal.aborted)
+            return true;
         await learnDriverFuel(config, driver.name, investigationFuel, result);
         if (result.status === "failed") {
             const message = result.error ?? "Investigation run failed";
@@ -927,6 +942,9 @@ export async function executeNextInvestigation(client, config, workspace, brief,
         console.log(`Investigation ${assignment.id} settled with a bounded brief.`);
     }
     catch (error) {
+        if (leaseTakenBack || runAbort.signal.aborted) {
+            return true;
+        }
         const message = error instanceof Error ? error.message : "investigation_failed";
         await settle(client, assignment.id, {
             lease_token: leaseToken,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@miraland-labs/conduit-bridge",
-  "version": "0.16.160",
+  "version": "0.16.162",
   "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": {