@deksden-com/dd-flow-cli 0.9.0-beta.145 → 0.9.0-beta.149

package/dist/services/check-runner-evidence.js

@@ -5,14 +5,24 @@ import path from "node:path";
 import { parse } from "shell-quote";
 import { AppError } from "../shared/errors.js";
 import { validateObservationDuration } from "../harness-runtime/lib/observation-clock.mjs";
+import { isDeepStrictEqual } from "node:util";
+import { fileURLToPath } from "node:url";
 function object(value) { return Boolean(value) && typeof value === "object" && !Array.isArray(value); }
 function boundedString(value, bytes) { return typeof value === "string" && Buffer.byteLength(value) <= bytes; }
 const validatedBrowserProofs = new WeakSet();
+const unsupportedObservationContracts = new WeakSet();
+const validatedNativeProofs = new WeakSet();
+const validatedChildren = new WeakMap();
+const validatedReports = new WeakMap();
+const hash = (value) => typeof value === "string" && /^[a-f0-9]{64}$/.test(value);
+const absoluteFile = (value) => typeof value === "string" && value.length > 0 && value.length <= 4096 && !value.includes("\0") && path.isAbsolute(value);
+export function candidateAbsencesValid(value) { return Array.isArray(value) && value.length <= 64 && value.every(item => object(item) && absoluteFile(item.file) && item.path_state === "missing" && object(item.declaration) && absoluteFile(item.declaration.file) && hash(item.declaration.sha256)); }
 function relativeProofPath(value) { return typeof value === "string" && value.length > 0 && Buffer.byteLength(value) <= 4096 && !path.isAbsolute(value) && !value.includes("\\") && value.split("/").every(part => part !== "" && part !== "." && part !== ".."); }
 function proofArtifact(value) { return object(value) && relativeProofPath(value.relative_path) && typeof value.sha256 === "string" && /^[a-f0-9]{64}$/.test(value.sha256); }
 function browserProofShape(proof) {
     const attempt = proof.attempt, step = proof.step;
-    return proof.native_code === "PLAYWRIGHT_ACTION_INACTIVITY" && object(attempt) && boundedString(attempt.invocation_id, 4096) && boundedString(attempt.test_id, 4096) && Boolean(attempt.test_id)
+    return (proof.observation_contract === undefined || proof.observation_contract === "dd-tasks/browser-observation@2" && object(proof.target) && [proof.target.page_id, proof.target.context_id].every(value => boundedString(value, 256) && Boolean(value)) && Number.isSafeInteger(proof.window_epoch) && Number(proof.window_epoch) > 0)
+        && proof.native_code === "PLAYWRIGHT_ACTION_INACTIVITY" && object(attempt) && boundedString(attempt.invocation_id, 4096) && boundedString(attempt.test_id, 4096) && Boolean(attempt.test_id)
         && [attempt.retry, attempt.repeat_each_index, attempt.worker_index].every(value => Number.isSafeInteger(value) && value >= 0)
         && object(step) && boundedString(step.id, 4096) && Boolean(step.id) && ["pw:api", "expect"].includes(String(step.category)) && step.file === proof.file && Number.isSafeInteger(step.line) && step.line > 0
         && proofArtifact(proof.worker_snapshot) && proofArtifact(proof.native_result);
@@ -29,6 +39,21 @@ export function runnerFailureValid(value) {
         return false;
     if (value.repair_evidence !== undefined) {
         const proof = value.repair_evidence;
+        if (object(proof) && proof.rule === "candidate_test_runtime_error")
+            return ["TypeError", "ReferenceError"].includes(String(proof.exception_type))
+                && absoluteFile(proof.file) && hash(proof.file_sha256) && ["test", "setup", "collection"].includes(String(proof.phase))
+                && [proof.line, proof.column].every(value => Number.isSafeInteger(value) && Number(value) > 0) && proofArtifact(proof.native_artifact);
+        if (object(proof) && proof.rule === "candidate_migration_sql")
+            return proof.native_code === "42601" && absoluteFile(proof.file) && hash(proof.file_sha256)
+                && boundedString(proof.operation_id, 4096) && Boolean(proof.operation_id) && proofArtifact(proof.native_artifact) && object(proof.world)
+                && boundedString(proof.world.invocation_id, 4096) && typeof proof.world.world_id === "string" && /^[a-f0-9]{32}$/.test(proof.world.world_id)
+                && boundedString(proof.world.role, 128) && Boolean(proof.world.role) && hash(proof.world.migrations_sha256) && proof.world.phase === "migrating" && proof.world.setup_completed === false;
+        if (object(proof) && proof.rule === "candidate_tool_diagnostic")
+            return absoluteFile(proof.file) && proofArtifact(proof.native_artifact)
+                && object(proof.tool) && { biome: "2.5.6", typescript: "5.9.3", docs: "1", "value-absence": "1" }[String(proof.tool.name)] === proof.tool.version
+                && object(proof.step) && boundedString(proof.step.id, 256) && Boolean(proof.step.id) && Number.isSafeInteger(proof.step.ordinal) && proof.step.ordinal >= 0 && hash(proof.step.command_sha256)
+                && boundedString(proof.diagnostic_code, 256) && (proof.tool.name === "biome" ? ["format", "assist/source/organizeImports"].includes(String(proof.diagnostic_code)) || /^lint\/[\w/-]+$/.test(String(proof.diagnostic_code)) : proof.tool.name === "typescript" ? /^TS\d+$/.test(String(proof.diagnostic_code)) : proof.tool.name === "docs" ? ["required_path_missing", "docs_contract"].includes(String(proof.diagnostic_code)) : proof.diagnostic_code === "value_absence")
+                && (proof.path_state === "present" && hash(proof.file_sha256) || proof.path_state === "missing" && proof.tool.name === "docs" && proof.diagnostic_code === "required_path_missing" && proof.file_sha256 === null && object(proof.declaration) && absoluteFile(proof.declaration.file) && hash(proof.declaration.sha256));
         if (!object(proof) || !boundedString(proof.rule, 128) || !boundedString(proof.native_code, 128) || !boundedString(proof.file, 4096) || !object(proof.world))
             return false;
         const world = proof.world;
@@ -45,23 +70,132 @@ function candidateFile(root, file) {
     if (!root || !path.isAbsolute(file))
         return false;
     try {
-        const relative = path.relative(fs.realpathSync(root), fs.realpathSync(file));
-        return relative !== "" && relative !== ".." && !relative.startsWith(`..${path.sep}`) && !path.isAbsolute(relative) && fs.statSync(file).isFile();
+        const realRoot = fs.realpathSync(root), relative = path.relative(realRoot, fs.realpathSync(file));
+        if (!relative || relative === ".." || relative.startsWith(`..${path.sep}`) || path.isAbsolute(relative))
+            return false;
+        let current = file;
+        while (fs.realpathSync(current) !== realRoot) {
+            if (fs.lstatSync(current).isSymbolicLink())
+                return false;
+            const parent = path.dirname(current);
+            if (parent === current)
+                return false;
+            current = parent;
+        }
+        return fs.statSync(file).isFile();
     }
     catch {
         return false;
     }
 }
 export function candidateFailureFiles(evidence, root) {
-    return [...new Set((evidence?.failures ?? []).flatMap(failure => failure.file && candidateFile(root, failure.file) ? [failure.file] : []))];
+    return [...new Set((evidence?.failures ?? []).flatMap(failure => {
+            const child = validatedChildren.get(failure);
+            if (child)
+                return candidateFailureFiles({ ...child.evidence, failures: [child.failure] }, root);
+            const proof = failure.repair_evidence;
+            const checksum = proof && "file_sha256" in proof ? proof.file_sha256 : undefined;
+            try {
+                return failure.file && candidateFile(root, failure.file) && (checksum === undefined || checksum === crypto.createHash("sha256").update(fs.readFileSync(failure.file)).digest("hex")) ? [failure.file] : [];
+            }
+            catch {
+                return [];
+            }
+        }))];
+}
+function absentCandidate(root, file) {
+    if (!root || !absoluteFile(file))
+        return false;
+    try {
+        let current = path.resolve(file);
+        try {
+            fs.lstatSync(current);
+            return false;
+        }
+        catch (error) {
+            if (error.code !== "ENOENT")
+                return false;
+        }
+        for (;;) {
+            try {
+                if (fs.lstatSync(current).isSymbolicLink())
+                    return false;
+                break;
+            }
+            catch (error) {
+                if (error.code !== "ENOENT")
+                    return false;
+                const parent = path.dirname(current);
+                if (parent === current)
+                    return false;
+                current = parent;
+            }
+        }
+        const realRoot = fs.realpathSync(root), relative = path.relative(realRoot, fs.realpathSync(current));
+        if (relative === ".." || relative.startsWith(`..${path.sep}`) || path.isAbsolute(relative))
+            return false;
+        while (fs.realpathSync(current) !== realRoot) {
+            if (fs.lstatSync(current).isSymbolicLink())
+                return false;
+            current = path.dirname(current);
+        }
+        return true;
+    }
+    catch {
+        return false;
+    }
 }
-function candidateFailure(failure, evidence, root, capturedFiles) {
-    const owned = (file) => capturedFiles === undefined ? candidateFile(root, file) : capturedFiles.includes(file);
+export function candidateFailureAbsences(evidence, root) {
+    return (evidence?.failures ?? []).flatMap(failure => {
+        const child = validatedChildren.get(failure);
+        if (child)
+            return candidateFailureAbsences({ ...child.evidence, failures: [child.failure] }, root);
+        try {
+            const proof = failure.repair_evidence;
+            if (proof?.rule !== "candidate_tool_diagnostic" || proof.path_state !== "missing" || !proof.declaration || !validatedNativeProofs.has(proof)
+                || !absentCandidate(root, proof.file) || !candidateFile(root, proof.declaration.file) || crypto.createHash("sha256").update(fs.readFileSync(proof.declaration.file)).digest("hex") !== proof.declaration.sha256)
+                return [];
+            return [{ file: proof.file, declaration: proof.declaration, path_state: "missing" }];
+        }
+        catch {
+            return [];
+        }
+    });
+}
+function candidateFailure(failure, evidence, root, capturedFiles, capturedAbsences) {
+    const child = validatedChildren.get(failure);
+    if (child)
+        return candidateFailure(child.failure, child.evidence, root, capturedFiles, capturedAbsences);
+    const owned = (file) => {
+        if (capturedFiles !== undefined)
+            return capturedFiles.includes(file);
+        if (!candidateFile(root, file))
+            return false;
+        const proof = failure.repair_evidence;
+        try {
+            return !proof || !("file_sha256" in proof) || proof.file_sha256 === crypto.createHash("sha256").update(fs.readFileSync(file)).digest("hex");
+        }
+        catch {
+            return false;
+        }
+    };
     if (failure.origin === "tooling" || failure.origin === "unknown")
         return false;
     if (failure.category === "assertion")
         return failure.origin === undefined || failure.origin === "candidate" && failure.file !== undefined && owned(failure.file);
     const proof = failure.repair_evidence;
+    if (proof?.rule === "candidate_test_runtime_error")
+        return validatedNativeProofs.has(proof) && failure.category === "unclassified" && failure.origin === "candidate"
+            && failure.phase === proof.phase && failure.native_code === proof.exception_type && failure.file === proof.file && owned(proof.file);
+    if (proof?.rule === "candidate_migration_sql" || proof?.rule === "candidate_tool_diagnostic") {
+        if (!validatedNativeProofs.has(proof) || failure.category !== "unclassified" || failure.origin !== "candidate" || failure.file !== proof.file || !["setup", "collection", "test"].includes(failure.phase ?? ""))
+            return false;
+        if (proof.rule === "candidate_migration_sql")
+            return failure.phase === "setup" && failure.native_code === "42601" && proof.world.invocation_id === evidence.binding.invocation_id && owned(proof.file);
+        if (failure.phase !== "collection" || failure.native_code !== proof.diagnostic_code)
+            return false;
+        return proof.path_state === "missing" ? Boolean(proof.declaration && (capturedAbsences ?? (root ? candidateFailureAbsences(evidence, root) : [])).some(item => item.file === proof.file && isDeepStrictEqual(item.declaration, proof.declaration))) : owned(proof.file);
+    }
     if (proof?.rule === "candidate_browser_action_inactivity")
         return validatedBrowserProofs.has(proof) && failure.category === "unclassified" && failure.phase === "test" && failure.origin === "candidate" && failure.native_code === proof.native_code && failure.file === proof.file && failure.test_id === proof.attempt.test_id && proof.attempt.invocation_id === evidence.binding.invocation_id && owned(proof.file);
     return failure.category === "unclassified" && failure.origin === "candidate" && ["test", "setup"].includes(failure.phase ?? "") && proof?.rule === "candidate_sql_integrity" && ["23502", "23503", "23505", "23514"].includes(proof.native_code) && failure.native_code === proof.native_code && failure.file === proof.file && proof.world.invocation_id === evidence.binding.invocation_id && owned(proof.file);
@@ -107,7 +241,7 @@ export function readRunnerProgress(file, invocationId, lastSequence = 0, now = D
             if (!value || typeof value !== "object" || Array.isArray(value) || value.schema_id !== "dd-flow/check-runner-progress@1" || value.invocation_id !== invocationId
                 || !Number.isSafeInteger(value.sequence) || value.sequence <= 0 || typeof value.at !== "string" || !Number.isFinite(Date.parse(value.at))
                 || !["startup", "execution"].includes(value.phase)
-                || !["setup_step_completed", "run_started", "module_collected", "test_started", "test_completed", "step_completed", "run_completed"].includes(value.event))
+                || !["setup_step_completed", "run_started", "module_collected", "test_started", "test_completed", "step_started", "step_completed", "run_completed"].includes(value.event))
                 throw new Error("Malformed official progress record");
             if (value.sequence <= lastSequence || Date.parse(value.at) > now)
                 continue;
@@ -164,6 +298,9 @@ function readProofArtifact(root, ref, limit = 256 * 1024) {
 function matches(actual, expected) { return object(actual) && Object.entries(expected).every(([key, value]) => actual[key] === value); }
 function validateBrowserArtifacts(root, proof, binding) {
     const snapshot = readProofArtifact(root, proof.worker_snapshot), native = readProofArtifact(root, proof.native_result);
+    if (proof.observation_contract !== undefined && (proof.observation_contract !== "dd-tasks/browser-observation@2" || snapshot.observation_contract !== proof.observation_contract || native.observation_contract !== proof.observation_contract
+        || !isDeepStrictEqual(snapshot.target, proof.target) || !isDeepStrictEqual(native.target, proof.target) || snapshot.window_epoch !== proof.window_epoch || native.window_epoch !== proof.window_epoch))
+        return false;
     if (!Array.isArray(native.errors) || native.errors.length < 1 || native.errors.length > 64 || native.errors.some(error => !boundedString(error, 4096) || !/^(?:Error: )?(?:[A-Za-z][\w.]*: )?Test timeout of [1-9]\d*ms exceeded\.(?:\nCall log:[\s\S]*)?$/.test(stripVTControlCharacters(String(error)))))
         return false;
     if (!matches(snapshot.binding, binding) || !matches(native.binding, binding) || !matches(snapshot.attempt, proof.attempt) || !matches(native.attempt, proof.attempt)
@@ -175,7 +312,13 @@ function validateBrowserArtifacts(root, proof, binding) {
     return Number.isSafeInteger(quiet) && quiet > 0 && [last, deadline, observed, health, ended].every(Number.isFinite) && Math.abs(deadline - last - quiet) <= 2 && observed >= last && observed <= deadline && health <= observed && health >= last && deadline - health <= Math.min(1000, quiet / 4) && ended >= deadline && ended <= Date.now() && ended - deadline <= 5000;
 }
 export function runnerProofArtifacts(evidence) {
-    return [...new Set((evidence?.failures ?? []).flatMap(failure => { const proof = failure.repair_evidence; return [...(failure.diagnostic_artifact ? [failure.diagnostic_artifact.relative_path] : []), ...(proof?.rule === "candidate_browser_action_inactivity" ? [proof.worker_snapshot.relative_path, proof.native_result.relative_path] : [])]; }))];
+    return [...new Set([...(evidence?.failures ?? []).flatMap(failure => {
+                const child = validatedChildren.get(failure);
+                if (child && failure.child_evidence)
+                    return [failure.child_evidence.report.relative_path, ...runnerProofArtifacts(child.evidence).map(file => path.posix.join(path.posix.dirname(failure.child_evidence.report.relative_path), file))];
+                const proof = failure.repair_evidence;
+                return [...(failure.diagnostic_artifact ? [failure.diagnostic_artifact.relative_path] : []), ...(proof?.rule === "candidate_browser_action_inactivity" ? [proof.worker_snapshot.relative_path, proof.native_result.relative_path] : proof && "native_artifact" in proof ? [proof.native_artifact.relative_path] : [])];
+            }), ...(evidence ? validatedReports.get(evidence) ?? [] : []).flatMap(child => [child.report.relative_path, ...runnerProofArtifacts(child.evidence).map(file => path.posix.join(path.posix.dirname(child.report.relative_path), file))])])];
 }
 export function runnerContractsValid(value) {
     if (value === undefined)
@@ -186,13 +329,15 @@ export function runnerContractsValid(value) {
         if (!raw || typeof raw !== "object" || Array.isArray(raw))
             return false;
         const contract = raw;
-        return alias.startsWith("@check/") && contract.schema_id === "dd-flow/check-runner-evidence@1" && typeof contract.path === "string" && contract.path.length > 0 && contract.path.split("/")[0] !== "engine-outcome.json" && !contract.path.includes("\\") && !path.isAbsolute(contract.path) && !contract.path.split("/").some(part => part === ".." || part === "" || part === ".");
+        return alias.startsWith("@check/") && ["dd-flow/check-runner-evidence@1", "dd-flow/check-runner-evidence@2"].includes(String(contract.schema_id)) && typeof contract.path === "string" && contract.path.length > 0 && contract.path.split("/")[0] !== "engine-outcome.json" && !contract.path.includes("\\") && !path.isAbsolute(contract.path) && !contract.path.split("/").some(part => part === ".." || part === "" || part === ".");
     });
 }
-export function readRunnerEvidence(root, contract, binding) {
+export function readRunnerEvidence(root, contract, binding, depth = 0, budget = { reports: 0 }) {
     try {
+        if (depth > 4 || ++budget.reports > 64)
+            return null;
         let current = root;
-        if (!fs.lstatSync(current).isDirectory())
+        if (!fs.lstatSync(current).isDirectory() || fs.lstatSync(current).isSymbolicLink())
             return null;
         for (const part of contract.path.split(/[\\/]/)) {
             current = path.join(current, part);
@@ -200,34 +345,141 @@ export function readRunnerEvidence(root, contract, binding) {
                 return null;
         }
         const evidence = JSON.parse(readBoundedProofFile(current, 1024 * 1024).toString("utf8"));
+        if (!["dd-flow/check-runner-evidence@1", "dd-flow/check-runner-evidence@2"].includes(contract.schema_id))
+            return null;
+        const partial = evidence.failure_projection;
+        if (partial !== undefined && (evidence.schema_id !== "dd-flow/check-runner-evidence@2" || !object(partial) || partial.state !== "partial" || !["count_bound", "byte_bound", "interrupt"].includes(partial.reason)
+            || ![partial.represented, partial.omitted].every(value => Number.isSafeInteger(value) && value >= 0)))
+            return null;
         if (evidence.schema_id !== contract.schema_id || !evidence.binding || Object.entries(binding).some(([key, value]) => evidence.binding[key] !== value)
             || typeof evidence.runner?.name !== "string" || !evidence.runner.name || typeof evidence.runner.version !== "string" || !evidence.runner.version || typeof evidence.completed !== "boolean" || !evidence.counts
             || ![evidence.counts.passed, evidence.counts.failed, evidence.counts.skipped].every(n => Number.isSafeInteger(n) && n >= 0)
             || !Array.isArray(evidence.failures) || evidence.failures.length > 64 || evidence.failures.some(f => !runnerFailureValid(f))
-            || evidence.counts.failed > evidence.failures.length || evidence.observation_unknown !== undefined && typeof evidence.observation_unknown !== "boolean")
+            || !partial && evidence.counts.failed > evidence.failures.length || evidence.observation_unknown !== undefined && typeof evidence.observation_unknown !== "boolean")
             return null;
+        const children = new Map();
+        const consumed = new Map();
+        if (evidence.steps !== undefined) {
+            if (!Array.isArray(evidence.steps) || evidence.steps.length > 64)
+                return null;
+            const ids = new Set(), invocations = new Set();
+            let stopped = false;
+            for (const [ordinal, step] of evidence.steps.entries()) {
+                if (!step || !boundedString(step.id, 256) || !step.id || ids.has(step.id) || step.ordinal !== ordinal || !hash(step.command_sha256) || !["passed", "failed", "unreached"].includes(step.state))
+                    return null;
+                ids.add(step.id);
+                if (step.state === "unreached") {
+                    if (!stopped || step.binding !== undefined || step.report !== undefined || step.exit_code !== undefined)
+                        return null;
+                    continue;
+                }
+                if (stopped)
+                    return null;
+                if (step.state === "failed")
+                    stopped = true;
+                if (!step.binding || !proofArtifact(step.report) || !Number.isSafeInteger(step.exit_code) || step.state === "passed" && step.exit_code !== 0 || step.state === "failed" && step.exit_code === 0 || invocations.has(step.binding.invocation_id) || step.binding.invocation_id === binding.invocation_id)
+                    return null;
+                if (step.binding.invocation_id !== `${binding.invocation_id}:step:${step.id}` || step.binding.check_id !== `${binding.check_id}/${step.id}` || step.binding.verification_epoch !== binding.verification_epoch || step.binding.input_hash !== crypto.createHash("sha256").update(JSON.stringify([binding.input_hash, step.command_sha256])).digest("hex"))
+                    return null;
+                if (crypto.createHash("sha256").update(step.binding.command).digest("hex") !== step.command_sha256)
+                    return null;
+                invocations.add(step.binding.invocation_id);
+                readProofArtifact(root, step.report, 1024 * 1024);
+                const child = readRunnerEvidence(path.join(root, path.dirname(step.report.relative_path)), { schema_id: contract.schema_id, path: path.basename(step.report.relative_path) }, step.binding, depth + 1, budget);
+                if (!child || step.state === "passed" && (child.counts.failed !== 0 || child.failures.length !== 0) || step.state === "failed" && !child.failures.length && child.completed && !child.observation_unknown)
+                    return null;
+                if (!child.completed || child.observation_unknown)
+                    stopped = true;
+                children.set(step.id, child);
+            }
+            for (const key of ["passed", "failed", "skipped"])
+                if (evidence.counts[key] !== [...children.values()].reduce((total, child) => total + child.counts[key], 0))
+                    return null;
+        }
         for (const failure of evidence.failures) {
+            if (failure.child_evidence) {
+                const ref = failure.child_evidence, step = evidence.steps?.find(item => item.id === ref.step_id), child = children.get(ref.step_id);
+                if (!step || !child || !isDeepStrictEqual(ref.binding, step.binding) || !isDeepStrictEqual(ref.report, step.report))
+                    return null;
+                const original = { ...failure };
+                delete original.child_evidence;
+                const used = consumed.get(child) ?? new Set();
+                consumed.set(child, used);
+                const index = child.failures.findIndex((item, index) => { const native = { ...item }; delete native.child_evidence; return !used.has(index) && isDeepStrictEqual(native, original); });
+                if (index < 0)
+                    return null;
+                used.add(index);
+                const matched = child.failures[index];
+                if (!matched.child_evidence && matched.repair_evidence?.rule === "candidate_tool_diagnostic" && !isDeepStrictEqual(matched.repair_evidence.step, { id: step.id, ordinal: step.ordinal, command_sha256: step.command_sha256 }))
+                    return null;
+                validatedChildren.set(failure, { failure: matched, evidence: child });
+                continue;
+            }
             if (failure.diagnostic_artifact && !matches(readProofArtifact(root, failure.diagnostic_artifact, 2 * 1024 * 1024).binding, binding))
                 return null;
             const proof = failure.repair_evidence;
+            if (proof?.rule === "candidate_migration_sql" || proof?.rule === "candidate_tool_diagnostic" || proof?.rule === "candidate_test_runtime_error") {
+                const native = readProofArtifact(root, proof.native_artifact, 2 * 1024 * 1024), facts = Object.fromEntries(Object.entries(proof).filter(([key]) => key !== "native_artifact"));
+                if (!isDeepStrictEqual(native.binding, binding) || Object.entries(facts).some(([key, value]) => !isDeepStrictEqual(native[key], value)))
+                    return null;
+                if (proof.rule === "candidate_test_runtime_error") {
+                    if (evidence.schema_id !== "dd-flow/check-runner-evidence@2" || native.executing_file !== proof.file || native.exception_name !== proof.exception_type
+                        || !isDeepStrictEqual(native.top_frame, { file: proof.file, line: proof.line, column: proof.column }) || !boundedString(native.stack, 16384) || !String(native.stack).startsWith(`${proof.exception_type}:`))
+                        return null;
+                    const firstFrame = String(native.stack).split("\n").find(line => /^\s*at\s/.test(line));
+                    const frame = firstFrame?.match(/^\s*at\s+(?:.*?\()?((?:file:\/\/)?\/[^\n]+):(\d+):(\d+)\)?$/);
+                    if (!frame || (frame[1].startsWith("file://") ? fileURLToPath(frame[1]) : frame[1]) !== proof.file || Number(frame[2]) !== proof.line || Number(frame[3]) !== proof.column)
+                        return null;
+                }
+                validatedNativeProofs.add(proof);
+            }
             if (proof?.rule === "candidate_browser_action_inactivity") {
-                if (!validateBrowserArtifacts(root, proof, binding))
+                if (proof.observation_contract === undefined && evidence.schema_id === "dd-flow/check-runner-evidence@1") {
+                    // Legacy diagnostics remain readable; they do not prove the new window/target guarantee.
+                    if (!validateBrowserArtifacts(root, proof, binding))
+                        return null;
+                    evidence.completed = false;
+                    unsupportedObservationContracts.add(evidence);
+                    continue;
+                }
+                if (proof.observation_contract !== "dd-tasks/browser-observation@2" || !validateBrowserArtifacts(root, proof, binding))
                     return null;
                 validatedBrowserProofs.add(proof);
             }
         }
+        const represented = [...consumed.values()].reduce((total, used) => total + used.size, 0), childFailures = [...children.values()].reduce((total, child) => total + child.failures.length, 0);
+        if (partial) {
+            if (children.size ? partial.represented !== represented || partial.omitted !== childFailures - represented : partial.represented !== evidence.failures.filter(failure => failure.code !== "report_truncated").length)
+                return null;
+            evidence.completed = false;
+        }
+        else if (represented !== childFailures)
+            return null;
+        // Valid but incomplete native observation is diagnostic evidence, not a
+        // malformed schema. Preserve the primary failure and references through
+        // every composite, including a parent that incorrectly claims completion.
+        if ([...children.values()].some(child => !child.completed || child.observation_unknown)) {
+            evidence.completed = false;
+            if ([...children.values()].some(child => unsupportedObservationContracts.has(child)))
+                unsupportedObservationContracts.add(evidence);
+            if (evidence.schema_id === "dd-flow/check-runner-evidence@1" && [...children.values()].some(child => !child.completed && !unsupportedObservationContracts.has(child)) || [...children.values()].some(child => child.observation_unknown))
+                evidence.observation_unknown = true;
+        }
+        validatedReports.set(evidence, (evidence.steps ?? []).flatMap(step => step.report && children.has(step.id) ? [{ report: step.report, evidence: children.get(step.id) }] : []));
         return evidence;
     }
     catch {
         return null;
     }
 }
-export function runnerEvidencePass(evidence) { return Boolean(evidence?.completed && !evidence.observation_unknown && evidence.counts.failed === 0 && evidence.failures.length === 0); }
+export function runnerEvidencePass(evidence) { return Boolean(evidence?.completed && !evidence.failure_projection && !evidence.observation_unknown && evidence.counts.failed === 0 && evidence.failures.length === 0); }
 export function checkRepairDisposition(receipt) {
     if (receipt.fingerprint_scope != null && receipt.fingerprint_scope !== "check_inputs")
         return "tooling_blocker";
     if (receipt.status === "running" || receipt.status === "aborted" || receipt.runner_evidence?.observation_unknown)
         return "settlement_required";
+    if (receipt.runner_evidence && unsupportedObservationContracts.has(receipt.runner_evidence) || receipt.runner_evidence?.failure_projection || receipt.runner_evidence?.schema_id === "dd-flow/check-runner-evidence@2" && !receipt.runner_evidence.completed)
+        return "tooling_blocker";
     if (receipt.status === "passed")
         return "passed";
     if (receipt.execution_failures?.length || receipt.mutation_paths?.length || receipt.before_fingerprint !== receipt.after_fingerprint || receipt.diagnostic_evidence_valid === false)
@@ -241,7 +493,7 @@ export function checkRepairDisposition(receipt) {
         return "tooling_blocker";
     if (!evidence.completed)
         return "unclassified";
-    if (evidence.failures.some(f => !runnerFailureValid(f) || !candidateFailure(f, evidence, receipt.candidate_root, receipt.candidate_files)))
+    if (evidence.failures.some(f => !runnerFailureValid(f) || !candidateFailure(f, evidence, receipt.candidate_root, receipt.candidate_files, receipt.candidate_absences)))
         return "unclassified";
     return "source_repair";
 }

package/dist/services/code-checks.js

@@ -1,5 +1,5 @@
 import crypto from "node:crypto";
-import { candidateFailureFiles, checkRepairDisposition, readRunnerEvidence, readRunnerProgress, runnerContractsValid, runnerEvidencePass, runnerObservationDurations, runnerProofArtifacts } from "./check-runner-evidence.js";
+import { candidateAbsencesValid, candidateFailureAbsences, candidateFailureFiles, checkRepairDisposition, readRunnerEvidence, readRunnerProgress, runnerContractsValid, runnerEvidencePass, runnerObservationDurations, runnerProofArtifacts } from "./check-runner-evidence.js";
 export { assertSourceRepairEligible, checkRepairDisposition } from "./check-runner-evidence.js";
 import { isGitlessWorkspace, ownedWorkspaceHookPath, projectServicePathFilter, workspaceGitFiles } from "./workspace-files.js";
 import { spawn, spawnSync } from "node:child_process";
@@ -16,6 +16,7 @@ import { confirmManagedProcess, finishManagedProcess, renewManagedProcess, manag
 import { admitRuntimeCheck, runtimeBudgetForRun } from "./runtime-budget.js";
 import { getResourceDatabase } from "../storage/database.js";
 import { readBoundedReleaseFile } from "./release-source.js";
+import { ownedCheckEnvironment, validateCheckEnvironment } from "./check-environment.js";
 const profileRelativePath = path.join(".memory-bank", "spec", "engineering", "code-check-profile.json");
 // Receipt reservation precedes child spawn so an interrupted CLI leaves an
 // auditable attempt. A second caller must not mistake that small handoff window
@@ -61,10 +62,7 @@ export function terminalCheckSettled(context, record) {
     }
 }
 export function codeExecutionEnvironment(workspaceRoot, checkEnvironment) {
-    if (checkEnvironment !== undefined && (!checkEnvironment || typeof checkEnvironment !== "object" || Array.isArray(checkEnvironment)
-        || Object.keys(checkEnvironment).length !== 1 || typeof checkEnvironment.CODEX_HOME !== "string"
-        || checkEnvironment.CODEX_HOME.length > 4096 || checkEnvironment.CODEX_HOME.includes("\0") || !path.isAbsolute(checkEnvironment.CODEX_HOME)))
-        throw new AppError("invalid_check_environment", "Owned check environment must name one absolute Codex home", 2, { effect: "no_effect" });
+    const selected = validateCheckEnvironment(checkEnvironment);
     const suffix = crypto.createHash("sha256").update(path.resolve(workspaceRoot)).digest("hex").slice(0, 12);
     // The managed CLI's Node must win over a provider app's PATH: a Rosetta
     // provider can otherwise run x64 checks against arm64-installed modules.
@@ -72,8 +70,7 @@ export function codeExecutionEnvironment(workspaceRoot, checkEnvironment) {
     delete environment.NODE_PATH;
     for (const key of ["DD_FLOW_CHECK_RUNNER_EVIDENCE_PATH", "DD_FLOW_CHECK_RUNNER_PROGRESS_PATH", "DD_FLOW_CHECK_INVOCATION_ID", "DD_FLOW_CHECK_ID", "DD_FLOW_CHECK_INPUT_HASH", "DD_FLOW_CHECK_VERIFICATION_EPOCH", "DD_FLOW_CHECK_COMMAND"])
         delete environment[key];
-    if (checkEnvironment)
-        environment.CODEX_HOME = checkEnvironment.CODEX_HOME;
+    Object.assign(environment, selected);
     return environment;
 }
 export function workspaceOperationId(workspaceRoot) { return `workspace:${crypto.createHash("sha256").update(path.resolve(workspaceRoot)).digest("hex")}`; }
@@ -302,6 +299,9 @@ export class PreparedCheckEvidence {
         Object.freeze(this);
     }
     static prepare(context, input) {
+        const checkEnvironment = ownedCheckEnvironment(context.env, input.checkEnvironment);
+        if (checkEnvironment)
+            input = { ...input, checkEnvironment };
         validateCheckEvidenceContext(input.evidenceContext);
         const fail = () => { throw new AppError("check_evidence_stale", "Current declared checks lack owned passed evidence", 2, { effect: "no_effect" }); };
         if (!input.checks.length || (input.scope === "work") !== Boolean(input.workId) || input.mode !== undefined && input.mode !== "diagnostic")
@@ -488,6 +488,9 @@ function checkEvidenceSqlSnapshot(context, input) {
     }));
 }
 export async function runCodeChecks(context, input) {
+    const checkEnvironment = ownedCheckEnvironment(context.env, input.checkEnvironment);
+    if (checkEnvironment)
+        input = { ...input, checkEnvironment };
     validateCheckEvidenceContext(input.evidenceContext);
     const generation = assertRunMutationAllowed(context, input.projectId, input.runId);
     let gate;
@@ -658,8 +661,9 @@ async function runCodeChecksOwned(context, input, workspaceGateId, generation, s
         if (result.failure)
             physicalFailures.push(result.failure);
         const candidateFiles = candidateFailureFiles(runnerEvidence, input.workspaceRoot);
+        const candidateAbsences = candidateFailureAbsences(runnerEvidence, input.workspaceRoot);
         if (group.runnerEvidence)
-            retainExecutionOutcome(evidenceDir, { id, command: group.command, input_hash: inputHash, verification_epoch: epoch, fingerprint_scope: "check_inputs", input_manifest: reserved.inputManifest ?? null, execution_contract_hash: executionContractHash }, input.workspaceRoot, physicalFailures, candidateFiles);
+            retainExecutionOutcome(evidenceDir, { id, command: group.command, input_hash: inputHash, verification_epoch: epoch, fingerprint_scope: "check_inputs", input_manifest: reserved.inputManifest ?? null, execution_contract_hash: executionContractHash }, input.workspaceRoot, physicalFailures, candidateFiles, candidateAbsences);
         const requiredArtifacts = [...new Set([...group.requiredArtifacts, ...runnerProofArtifacts(runnerEvidence), ...(group.runnerEvidence ? ["engine-outcome.json"] : [])])];
         artifacts = collectCheckArtifacts(evidenceDir, requiredArtifacts);
         const status = result.aborted ? "aborted" : result.exitCode === 0 && !result.error && artifacts.complete && mutationPaths.length === 0 && (!group.runnerEvidence || runnerEvidencePass(runnerEvidence)) ? "passed" : "failed";
@@ -667,6 +671,7 @@ async function runCodeChecksOwned(context, input, workspaceGateId, generation, s
         receipt.required_artifacts = requiredArtifacts;
         receipt.candidate_root = input.workspaceRoot;
         receipt.candidate_files = candidateFiles;
+        receipt.candidate_absences = candidateAbsences;
         receipt.execution_failures = physicalFailures;
         context.db.run("UPDATE check_receipts SET execution_failures_json = ? WHERE project_id = ? AND id = ? AND status = 'running'", [JSON.stringify(physicalFailures), input.projectId, id]);
         const completion = readCompletion(completionPath);
@@ -754,6 +759,7 @@ export function reconcileUnfinishedChecks(context, input) {
         if (scoped && !completionSnapshot)
             physicalFailures.push({ code: "completion_snapshot_unknown", message: "Detached check has no bound scoped completion snapshot; current inputs cannot prove inputs at command completion" });
         let candidateFiles = candidateFailureFiles(runnerEvidence, input.workspaceRoot);
+        let candidateAbsences = candidateFailureAbsences(runnerEvidence, input.workspaceRoot);
         if (receipt?.runner_evidence_required) {
             const evidenceDir = path.join(path.dirname(pendingReceipt.receipt_path), "artifacts");
             if (fs.existsSync(path.join(evidenceDir, "engine-outcome.json"))) {
@@ -763,13 +769,14 @@ export function reconcileUnfinishedChecks(context, input) {
                 if (proven.candidate_root) {
                     physicalFailures.push(...(proven.execution_failures ?? []));
                     candidateFiles = proven.candidate_files ?? candidateFiles;
+                    candidateAbsences = proven.candidate_absences ?? [];
                 }
                 else {
                     physicalFailures.push({ code: "engine_outcome_unbound", message: "Existing execution outcome is not bound to retained inventory; prior launcher/cleanup outcome is unknown" });
                     fs.renameSync(path.join(evidenceDir, "engine-outcome.json"), path.join(evidenceDir, `engine-outcome-unconfirmed-${crypto.randomUUID()}.json`));
                 }
             }
-            retainExecutionOutcome(evidenceDir, receipt, input.workspaceRoot, physicalFailures, candidateFiles);
+            retainExecutionOutcome(evidenceDir, receipt, input.workspaceRoot, physicalFailures, candidateFiles, candidateAbsences);
             receipt.required_artifacts = [...new Set([...(receipt.required_artifacts ?? []), ...runnerProofArtifacts(runnerEvidence), "engine-outcome.json"])];
             artifacts = collectCheckArtifacts(evidenceDir, receipt.required_artifacts);
         }
@@ -782,6 +789,7 @@ export function reconcileUnfinishedChecks(context, input) {
             const repaired = { ...receipt, ...(receipt.runner_evidence_required ? { runner_evidence: runnerEvidence ?? null } : {}), completion_path: receipt.completion_path || path.join(path.dirname(pendingReceipt.receipt_path), "completion.json"), required_artifacts: receipt.required_artifacts ?? [], ...(status === "aborted" ? { abort_reason: "completion_marker_missing" } : {}), status, exit_code: exitCode, after_fingerprint: afterFingerprint, mutation_paths: mutationPaths, artifacts: artifacts.items, finished_at: finishedAt };
             repaired.candidate_root = input.workspaceRoot;
             repaired.candidate_files = candidateFiles;
+            repaired.candidate_absences = candidateAbsences;
             repaired.execution_failures = physicalFailures;
             writeReceipt(pendingReceipt.receipt_path, repaired);
             if (status === "failed")
@@ -980,12 +988,12 @@ function resolvedReceiptArtifacts(receiptPath, artifacts, binding) {
     if (retainedOutcome.complete && retainedOutcome.items.every(item => artifacts.some(artifact => relocate(artifact.path) === item.path && artifact.sha256 === item.sha256))) {
         try {
             const value = JSON.parse(fs.readFileSync(outcomePath, "utf8"));
-            if (value.schema_id === "dd-flow/check-execution-outcome@1" && value.id === binding.id && value.command === binding.command && value.input_hash === binding.input_hash && value.verification_epoch === binding.verification_epoch && (value.fingerprint_scope ?? null) === (binding.fingerprint_scope ?? null) && (value.execution_contract_hash ?? null) === (binding.execution_contract_hash ?? null) && JSON.stringify(value.input_manifest ?? null) === JSON.stringify(binding.input_manifest ?? null) && typeof value.candidate_root === "string" && path.isAbsolute(value.candidate_root) && (value.candidate_files === undefined || Array.isArray(value.candidate_files) && value.candidate_files.length <= 64 && value.candidate_files.every((file) => typeof file === "string" && path.isAbsolute(file) && Buffer.byteLength(file) <= 4096)) && Array.isArray(value.execution_failures) && value.execution_failures.every((failure) => failure && typeof failure === "object" && typeof failure.code === "string" && typeof failure.message === "string"))
+            if (value.schema_id === "dd-flow/check-execution-outcome@1" && value.id === binding.id && value.command === binding.command && value.input_hash === binding.input_hash && value.verification_epoch === binding.verification_epoch && (value.fingerprint_scope ?? null) === (binding.fingerprint_scope ?? null) && (value.execution_contract_hash ?? null) === (binding.execution_contract_hash ?? null) && JSON.stringify(value.input_manifest ?? null) === JSON.stringify(binding.input_manifest ?? null) && typeof value.candidate_root === "string" && path.isAbsolute(value.candidate_root) && (value.candidate_files === undefined || Array.isArray(value.candidate_files) && value.candidate_files.length <= 64 && value.candidate_files.every((file) => typeof file === "string" && path.isAbsolute(file) && Buffer.byteLength(file) <= 4096)) && (value.candidate_absences === undefined || candidateAbsencesValid(value.candidate_absences)) && Array.isArray(value.execution_failures) && value.execution_failures.every((failure) => failure && typeof failure === "object" && typeof failure.code === "string" && typeof failure.message === "string"))
                 outcome = value;
         }
         catch { /* Missing physical proof cannot grant attribution. */ }
     }
-    return { completion_path: path.join(currentRoot, "completion.json"), required_artifacts: original.required_artifacts ?? [], inputs: original.inputs ?? [], artifacts: artifacts.map((artifact) => ({ ...artifact, path: relocate(artifact.path) })), ...(original.runner_evidence_required ? { runner_evidence_required: original.runner_evidence_required, runner_evidence: evidence } : {}), execution_failures: outcome.execution_failures ?? [], ...(outcome.candidate_root ? { candidate_root: outcome.candidate_root } : {}), ...(outcome.candidate_files ? { candidate_files: outcome.candidate_files } : {}) };
+    return { completion_path: path.join(currentRoot, "completion.json"), required_artifacts: original.required_artifacts ?? [], inputs: original.inputs ?? [], artifacts: artifacts.map((artifact) => ({ ...artifact, path: relocate(artifact.path) })), ...(original.runner_evidence_required ? { runner_evidence_required: original.runner_evidence_required, runner_evidence: evidence } : {}), execution_failures: outcome.execution_failures ?? [], ...(outcome.candidate_root ? { candidate_root: outcome.candidate_root } : {}), ...(outcome.candidate_files ? { candidate_files: outcome.candidate_files } : {}), ...(outcome.candidate_absences ? { candidate_absences: outcome.candidate_absences } : {}) };
 }
 function executionFailures(input) {
     const failures = [];
@@ -1003,11 +1011,11 @@ function executionFailures(input) {
         failures.push({ code: "invalid_runner_evidence", message: "Required runner evidence is unavailable or invalid" });
     return failures;
 }
-function retainExecutionOutcome(evidenceDir, binding, candidateRoot, failures, candidateFiles) {
+function retainExecutionOutcome(evidenceDir, binding, candidateRoot, failures, candidateFiles, candidateAbsences = []) {
     const temporary = path.join(evidenceDir, `.engine-outcome-${crypto.randomUUID()}.tmp`);
     if (!fs.lstatSync(evidenceDir).isDirectory() || fs.lstatSync(evidenceDir).isSymbolicLink())
         throw new AppError("check_evidence_stale", "Check evidence directory was replaced", 1);
-    fs.writeFileSync(temporary, JSON.stringify({ schema_id: "dd-flow/check-execution-outcome@1", id: binding.id, command: binding.command, input_hash: binding.input_hash, verification_epoch: binding.verification_epoch, fingerprint_scope: binding.fingerprint_scope ?? null, input_manifest: binding.input_manifest ?? null, execution_contract_hash: binding.execution_contract_hash ?? null, candidate_root: candidateRoot, candidate_files: candidateFiles, execution_failures: failures }), { flag: "wx", mode: 0o600 });
+    fs.writeFileSync(temporary, JSON.stringify({ schema_id: "dd-flow/check-execution-outcome@1", id: binding.id, command: binding.command, input_hash: binding.input_hash, verification_epoch: binding.verification_epoch, fingerprint_scope: binding.fingerprint_scope ?? null, input_manifest: binding.input_manifest ?? null, execution_contract_hash: binding.execution_contract_hash ?? null, candidate_root: candidateRoot, candidate_files: candidateFiles, candidate_absences: candidateAbsences, execution_failures: failures }), { flag: "wx", mode: 0o600 });
     fs.renameSync(temporary, path.join(evidenceDir, "engine-outcome.json"));
 }
 export function workspaceFingerprint(root) { return workspaceState(root).fingerprint; }
@@ -1162,7 +1170,7 @@ function environmentFingerprint(workspaceRoot, inputs, checkEnvironment) {
         "CODEX_PERMISSION_PROFILE", "CODEX_SESSION_ID", "CODEX_VERSION", "CODEX_TOOL_CALL_ID"]);
     return crypto.createHash("sha256").update(JSON.stringify(Object.entries(environment)
         .filter(([name, value]) => value !== undefined && !bookkeeping.has(name)
-        && (inputs === undefined || inputs.some(input => input.endsWith("*") ? name.startsWith(input.slice(0, -1)) : name === input)))
+        && (name === "PLAYWRIGHT_BROWSERS_PATH" || inputs === undefined || inputs.some(input => input.endsWith("*") ? name.startsWith(input.slice(0, -1)) : name === input)))
         .sort(([left], [right]) => left.localeCompare(right)))).digest("hex");
 }
 function acquireWorkspaceGate(context, input) {
@@ -1337,7 +1345,7 @@ function checkReceiptEvidenceValidForMode(receipt, mode) {
         // Receipt annotations are not execution authority. Compare the SQL-owned
         // failures with the independently hash/binding-checked engine sidecar.
         const hasEngineOutcome = receipt.required_artifacts.includes("engine-outcome.json") || receipt.artifacts.some(artifact => artifact.path === "engine-outcome.json" || artifact.path === path.join(path.dirname(receipt.receipt_path), "artifacts", "engine-outcome.json"));
-        if (hasEngineOutcome && !isDeepStrictEqual(retainedProof.execution_failures ?? [], receipt.execution_failures ?? []))
+        if (hasEngineOutcome && (!isDeepStrictEqual(retainedProof.execution_failures ?? [], receipt.execution_failures ?? []) || !isDeepStrictEqual(retainedProof.candidate_absences ?? [], receipt.candidate_absences ?? [])))
             return false;
         const root = path.join(path.dirname(receipt.receipt_path), "artifacts");
         if (stored.runner_evidence_required && mode !== "diagnostic" && !runnerEvidencePass(readRunnerEvidence(root, stored.runner_evidence_required, { invocation_id: receipt.id, check_id: receipt.id, command: receipt.command, input_hash: receipt.input_hash, verification_epoch: receipt.verification_epoch })))

package/dist/services/controller-execution.js

@@ -4,10 +4,11 @@ import { FrozenGenericExecution } from "./generic-execution.js";
 import { resolveExecutionPolicy, resolveStageTransition, stageExecutionCwd } from "./execution-policy.js";
 import { readFlowRunExecutionBundle } from "./runs.js";
 import { hybridDigest } from "./hybrid-contracts.js";
+import { isDiagnosticFlow } from "../domain/diagnostic-flows.js";
 /** Adapt generic routing to shared fanout APIs, not to SDLC settings. No alias,
  * operator file, merge policy or project default participates in this selection.
  */
-export function genericControllerRouting(execution) {
+export function genericControllerRouting(execution, workspaceRoot) {
     if (!(execution instanceof FrozenGenericExecution) || execution.settings.execution !== "managed") {
         throw new AppError("controller_execution_invalid", "Managed controller requires managed frozen execution", 1, { effect: "no_effect" });
     }
@@ -33,14 +34,17 @@ export function genericControllerRouting(execution) {
                 execution.stage(fromStage);
             return { ...resolveExecutionPolicy({ ddFlowHome: "", policy, profiles, stage }), session_mode: selected.session_mode };
         },
-        cwd(stage, _mergeTargetWorkspace, _currentRun) { execution.stage(stage); return path.resolve(snapshot.scope.workspace); }
+        cwd(stage, _mergeTargetWorkspace, currentRun) {
+            execution.stage(stage);
+            return path.resolve(currentRun?.workspace_root ?? workspaceRoot ?? snapshot.scope.workspace);
+        }
     };
 }
 /** Restore execution from RUN authority, never from a generic manifest copy.
  * Legacy SDLC manifests retain their original settings and routing behavior.
  */
 export function selectControllerExecution(context, run, manifest) {
-    if (manifest?.schema_id === "dd-flow/run-controller-manifest@2" || ["mb-init", "mb-upgrade"].includes(run.flow_kind)) {
+    if (manifest?.schema_id === "dd-flow/run-controller-manifest@2" || ["mb-init", "mb-upgrade"].includes(run.flow_kind) || isDiagnosticFlow(run.flow_kind)) {
         const bundle = readFlowRunExecutionBundle(context, { projectRoot: run.project_root, runId: run.id });
         if (!bundle || bundle.descriptor.descriptor.id !== run.flow_kind || bundle.descriptor.descriptor.id === "sdlc")
             throw new AppError("controller_execution_invalid", "Generic controller requires its retained non-SDLC bundle", 1);
@@ -51,7 +55,9 @@ export function selectControllerExecution(context, run, manifest) {
             || manifest.request?.project_root !== run.project_root || manifest.request?.run_id !== run.id)) {
             throw new AppError("controller_execution_invalid", "Controller execution binding differs from retained RUN authority", 1);
         }
-        return { ...genericControllerRouting(bundle.execution), bundle, binding, legacy: undefined };
+        // Bundle admission has already checked the current RUN workspace against
+        // its trusted relocation receipts; frozen scope remains historical input.
+        return { ...genericControllerRouting(bundle.execution, run.workspace_root), bundle, binding, legacy: undefined };
     }
     const index = JSON.parse(run.index_json);
     const legacy = manifest ? manifest.execution_profile : index.execution_profile;