@miraland-labs/conduit-bridge 0.14.8 → 0.14.9

package/dist/execution.js

@@ -7,8 +7,11 @@ import { DRIVERS, agentReportTemplate, buildAssignmentPrompt, evidenceKinds, ext
 import { assertClassFloor } from "./execution-class.js";
 import { pickDriverForClaim, resolveDriverFuel, supportsReadOnlyDiagnosis } from "./drivers.js";
 import { attemptWorktreePath, createAttemptWorktree, proveResumeWorktree, quarantineAttemptWorktree, removeAttemptWorktree, } from "./attempt-worktree.js";
+import { execFile } from "node:child_process";
+import { promisify } from "node:util";
 import { buildWorkspaceBrief, ensureCommitAvailable, normalizeRepositoryUrl, resolveAttemptStartCommit } from "./brief.js";
 import { ensureDeliveryPullRequest } from "./ensure-pull-request.js";
+const execFileAsync = promisify(execFile);
 import { captureVerificationFailure, ensureTestEvidence } from "./ensure-test-evidence.js";
 /** Keep a Mac awake only while an assignment is active; display sleep remains allowed. */
 export function startIdleSleepGuard(options = {}) {
@@ -969,7 +972,12 @@ async function runClaimedAssignment(client, config, driver, workspace, brief, ta
                 grants,
                 verificationCommands: attemptBrief?.verification ?? [],
             });
-            validateDeliveryReport(report, spec, grants);
+            // Ask git what changed before trusting what the report says changed.
+            const startCommit = executionContract.requested_base_commit && executionContract.claimed_head
+                ? await resolveAttemptStartCommit(workspace, executionContract.requested_base_commit, executionContract.claimed_head)
+                : null;
+            const actualPaths = startCommit ? await changedPathsSince(attemptWorkspace, startCommit) : null;
+            validateDeliveryReport(report, spec, grants, actualPaths ?? undefined);
         }
         catch (error) {
             const message = error instanceof Error ? error.message : "Agent delivery report was invalid";
@@ -1198,13 +1206,13 @@ async function prepareDelivery(client, attemptId, taskId, report) {
         },
     };
 }
-export function validateDeliveryReport(report, spec, grants = []) {
+export function validateDeliveryReport(report, spec, grants = [], actualPaths) {
     // Research findings live in the report. Do not demand a published pack URL for work_role=research
     // even when a planner mis-labeled the package as deliverable=artifact.
     const artifactDelivery = spec.deliverable === "artifact" && spec.work_role !== "research";
     // Artifact packs are not git landings — do not demand head_commit when the planner put
     // `.conduit/artifacts/**` in change_scope as a write hint (path-4 canary incident).
-    validateChangeScope(report, spec.change_scope ?? [], { requireHeadCommit: !artifactDelivery });
+    validateChangeScope(report, spec.change_scope ?? [], { requireHeadCommit: !artifactDelivery, actualPaths });
     if (artifactDelivery) {
         const published = report.evidence.some((item) => {
             if (!["preview", "research", "documentation"].includes(item.kind))
@@ -1271,6 +1279,45 @@ function referenceKind(kind) {
         return "document";
     return "other";
 }
+/**
+ * What this attempt actually changed, according to git rather than to the agent's prose.
+ *
+ * The scope check used to compare approved paths against strings the agent wrote about itself, so a
+ * run that edited an approved file and named it "tools.ts" failed the contract and spent an attempt
+ * on a path format. Git knows the truth, and Bridge holds the worktree: this is both stricter (a
+ * change cannot be omitted from the report to escape the check) and more forgiving (how the agent
+ * phrases a path stops mattering).
+ *
+ * Null when git cannot answer — an unknown must not fail a delivery, so the caller falls back to the
+ * report exactly as before.
+ */
+export async function changedPathsSince(worktree, startCommit) {
+    const run = async (args) => {
+        try {
+            const { stdout } = await execFileAsync("git", ["-C", worktree, ...args], { timeout: 30_000, maxBuffer: 8_000_000 });
+            return stdout;
+        }
+        catch {
+            return null;
+        }
+    };
+    const committed = await run(["diff", "--name-only", `${startCommit}..HEAD`]);
+    if (committed === null)
+        return null;
+    const paths = new Set(committed.split("\n").map((line) => line.trim()).filter(Boolean));
+    // Work the agent left uncommitted still lands in the delivery for lands=false packages.
+    const pending = await run(["status", "--porcelain"]);
+    for (const line of (pending ?? "").split("\n")) {
+        const entry = line.slice(3).trim();
+        if (!entry)
+            continue;
+        // A rename reports "old -> new"; the new path is the one that exists.
+        const path = entry.includes(" -> ") ? entry.slice(entry.lastIndexOf(" -> ") + 4).trim() : entry;
+        if (path)
+            paths.add(path.replace(/^"|"$/g, ""));
+    }
+    return [...paths];
+}
 export function validateChangeScope(report, changeScope, options = {}) {
     if (!changeScope.length)
         return;
@@ -1278,9 +1325,11 @@ export function validateChangeScope(report, changeScope, options = {}) {
     if (requireHeadCommit && !report.head_commit) {
         throw new Error("Repository changes require a delivered head commit");
     }
-    if (!report.changes.length)
+    // Git's answer when we have it; the agent's description only when we do not.
+    const paths = options.actualPaths
+        ?? report.changes.map((change) => change.split(/\s+[—-]\s+/, 1)[0].trim().replaceAll("\\", "/"));
+    if (!paths.length)
         return;
-    const paths = report.changes.map((change) => change.split(/\s+[—-]\s+/, 1)[0].trim().replaceAll("\\", "/"));
     const outside = paths.filter((path) => {
         if (!path || path.startsWith("/") || path.split("/").includes(".."))
             return true;

package/package.json

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