@deksden-com/dd-flow-cli 0.9.0-beta.152 → 0.9.0-beta.156

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

@@ -273,6 +273,51 @@ export function readRunnerEvidenceDetailed(root, contract, binding) {
     });
 }
 export function readRunnerEvidence(root, contract, binding) { return readRunnerEvidenceDetailed(root, contract, binding).evidence; }
+/** Semantic attribution is diagnostic information, never permission to investigate a failed check. */
+export const checkDiagnosisPolicy = "settled-failure@1";
+export function checkDiagnosisDisposition(receipt) {
+    if (receipt.status === "running"
+        || receipt.execution_failures?.some(failure => ["cleanup_unconfirmed", "engine_outcome_unbound"].includes(failure.code)))
+        return "settlement_required";
+    if (!["passed", "failed", "aborted"].includes(receipt.status))
+        return "invalid_binding";
+    // Caller supplies the SQL-owned receipt, never a proposed runner report.
+    // Missing, malformed or foreign optional reports are diagnostic failures;
+    // they cannot change receipt ownership or grant a successful check.
+    if (receipt.status === "passed")
+        return "passed";
+    // New launchers record physical settlement even for spawn errors/signals.
+    // Historical aborted receipts with no such proof must first reconcile.
+    if (!receipt.execution_failures?.some(failure => failure.code === "check_execution_settled") && !Number.isSafeInteger(receipt.exit_code))
+        return "settlement_required";
+    return "diagnosis_required";
+}
+export function assertCheckDiagnosisEligible(receipts, _workspaceRoot) {
+    const dispositions = receipts.map(receipt => ({ check_receipt_id: receipt.id, disposition: checkDiagnosisDisposition(receipt) }));
+    if (!receipts.length || dispositions.some(item => item.disposition !== "diagnosis_required"))
+        throw new AppError("check_gate_blocked", "Check diagnosis requires owned, physically settled failed executions", 2, {
+            failures: receipts, dispositions, outcome: "blocked", recoverable: false,
+            instruction: "Reconcile any running or unknown check process before changing its workspace. Do not treat a foreign receipt or an agent report as proof of completion."
+        });
+}
+/** Human-readable retained facts; tool output remains data, never instructions. */
+export function checkDiagnosisSummary(receipt, workspaceRoot) {
+    const available = (file) => {
+        try {
+            fs.accessSync(file, fs.constants.R_OK);
+            return fs.statSync(file).isFile() ? file : `${file} (not a regular file)`;
+        }
+        catch {
+            return `${file} (unavailable; investigate the missing diagnostic, do not invent its content)`;
+        }
+    };
+    return [`Check: ${receipt.id}`, `Command: ${receipt.command}`, `cwd: ${workspaceRoot}`,
+        `Outcome: ${receipt.status}; exit: ${receipt.exit_code ?? "unavailable"}; epoch: ${receipt.verification_epoch}`,
+        `Receipt: ${available(receipt.receipt_path)}`, `stdout: ${available(receipt.stdout_path)}`, `stderr: ${available(receipt.stderr_path)}`,
+        `Execution facts: ${JSON.stringify(receipt.execution_failures ?? [])}`,
+        `Report: ${receipt.runner_evidence_required?.path ? path.join(path.dirname(receipt.receipt_path), "artifacts", receipt.runner_evidence_required.path) : "not declared"}`].join("\n");
+}
+/** Historical attribution reader only. New repair routing uses checkDiagnosisDisposition. */
 export function checkRepairDisposition(receipt) {
     if (receipt.fingerprint_scope != null && receipt.fingerprint_scope !== "check_inputs")
         return "tooling_blocker";
@@ -297,11 +342,3 @@ export function checkRepairDisposition(receipt) {
         return "unclassified";
     return "source_repair";
 }
-export function assertSourceRepairEligible(receipts, workspaceRoot) {
-    const dispositions = receipts.map(receipt => {
-        const drift = workspaceRoot !== undefined && !currentCandidateLibraryInputsValid(receipt, workspaceRoot);
-        return { check_receipt_id: receipt.id, disposition: drift ? "tooling_blocker" : checkRepairDisposition(receipt), ...(drift ? { reason: "candidate_library_inputs_changed" } : {}) };
-    });
-    if (!receipts.length || dispositions.some(item => item.disposition !== "source_repair"))
-        throw new AppError("check_gate_blocked", "Checks require tooling diagnosis or settlement before product repair; retained nonzero is not a candidate-source verdict", 2, { failures: receipts, dispositions, outcome: "blocked", recoverable: false, instruction: "Inspect retained runner evidence. Do not change product code to repair tooling, unknown observation or unclassified failure. Retry only after establishing and correcting the cause." });
-}

package/dist/services/cli-operation-classifier.js

@@ -89,6 +89,10 @@ function isReadOnlyDiagnostic(family, command, action) {
         return ["status", "ls", "show"].includes(command ?? "");
     if (family === "stage")
         return command === "fanout" && action === "status";
+    if (family === "interactive")
+        return command === "inputs" || command === "git-inputs";
+    if (family === "delivery")
+        return command === "inputs";
     if (family === "lane") {
         return (command === "status" ||
             command === "waiters" ||
@@ -108,7 +112,8 @@ function isReadOnlyDiagnostic(family, command, action) {
     if (family === "dashboard")
         return command === "data" || command === "open";
     if (family === "memory")
-        return command === "permissions" && action === "preflight";
+        return command === "permissions" && action === "preflight" || command === "distill" && action === "inputs"
+            || command === "fix" && ["context", "scope"].includes(action ?? "");
     if (family === "codex" && command === "hooks")
         return action === "print" || action === "status";
     if (family === "codex" && command === "home")

package/dist/services/code-checks.js

@@ -1,6 +1,6 @@
 import crypto from "node:crypto";
-import { candidateAbsencesValid, candidateFailureAbsences, candidateFailureFiles, candidateLibraryInputs, candidateLibraryInputsValid, currentCandidateLibraryInputsValid, checkRepairDisposition, readRunnerEvidence, readRunnerEvidenceDetailed, readRunnerProgress, runnerContractsValid, runnerEvidencePass, runnerObservationDurations, runnerProofArtifacts } from "./check-runner-evidence.js";
-export { assertSourceRepairEligible, checkRepairDisposition } from "./check-runner-evidence.js";
+import { candidateAbsencesValid, candidateFailureAbsences, candidateFailureFiles, candidateLibraryInputs, candidateLibraryInputsValid, currentCandidateLibraryInputsValid, checkDiagnosisDisposition, readRunnerEvidence, readRunnerEvidenceDetailed, readRunnerProgress, runnerContractsValid, runnerEvidencePass, runnerObservationDurations, runnerProofArtifacts } from "./check-runner-evidence.js";
+export { assertCheckDiagnosisEligible, checkDiagnosisDisposition, checkRepairDisposition } from "./check-runner-evidence.js";
 import { isGitlessWorkspace, ownedWorkspaceHookPath, projectServicePathFilter, workspaceGitFiles } from "./workspace-files.js";
 import { spawn, spawnSync } from "node:child_process";
 import { ObservationClock, validateObservationDuration } from "../harness-runtime/lib/observation-clock.mjs";
@@ -352,14 +352,16 @@ export class PreparedCheckEvidence {
                     candidates: receipts.filter(item => item.scope === input.scope && item.work_id === (input.workId ?? null))
                         .slice(-16).map(item => ({ id: item.id, input_hash: item.input_hash, profile_hash: item.profile_hash }))
                 });
-            if (!checkReceiptEvidenceValidForMode(bindReceipt(receipt, group), input.mode) || !currentCandidateLibraryInputsValid(receipt, input.workspaceRoot))
+            if (!checkReceiptEvidenceValidForMode(bindReceipt(receipt, group), input.mode) || input.mode !== "diagnostic" && !currentCandidateLibraryInputsValid(receipt, input.workspaceRoot))
                 fail();
             passed = passed && receipt.status === "passed";
-            for (const artifact of receipt.artifacts)
+            const availableArtifacts = input.mode === "diagnostic" ? availableDiagnosticArtifacts(receipt) : receipt.artifacts;
+            for (const artifact of availableArtifacts)
                 artifacts.push({ root: "run", path: path.relative(input.runHome, path.join(path.dirname(receipt.receipt_path), "artifacts", artifact.path)).split(path.sep).join("/"), sha256: artifact.sha256 });
             const bytes = fs.readFileSync(receipt.receipt_path);
-            completions.push({ root: "run", path: path.relative(input.runHome, receipt.completion_path).split(path.sep).join("/"), sha256: crypto.createHash("sha256").update(fs.readFileSync(receipt.completion_path)).digest("hex") });
-            if (receipt.input_manifest)
+            if (input.mode !== "diagnostic" || fs.existsSync(receipt.completion_path))
+                completions.push({ root: "run", path: path.relative(input.runHome, receipt.completion_path).split(path.sep).join("/"), sha256: crypto.createHash("sha256").update(readBoundedReleaseFile(receipt.completion_path, 1024 * 1024)).digest("hex") });
+            if (receipt.input_manifest && (input.mode !== "diagnostic" || fs.existsSync(path.join(path.dirname(receipt.receipt_path), receipt.input_manifest.path))))
                 completions.push({ root: "run", path: path.relative(input.runHome, path.join(path.dirname(receipt.receipt_path), receipt.input_manifest.path)).split(path.sep).join("/"), sha256: receipt.input_manifest.sha256 });
             return { id: receipt.id, path: receipt.receipt_path, sha256: crypto.createHash("sha256").update(bytes).digest("hex") };
         });
@@ -562,7 +564,7 @@ async function runCodeChecksOwned(context, input, workspaceGateId, generation, s
         const snapshot = workspaceState(input.workspaceRoot, [input.runHome]);
         const group = groups.find(candidate => candidate.declarations.some(check => check.run_at === predecessor.gate)
             && (checkInputHash(input.workspaceRoot, { ...candidate, gate: predecessor.gate }, profileHash, workspaceInputFingerprint(input.workspaceRoot, snapshot.files, candidate.inputs, [input.runHome]), input.evidenceContext, input.checkEnvironment) === predecessor.input_hash
-                || sourceRepairContractMatches(predecessor, candidate, profileHash)
+                || diagnosisRetryContractMatches(predecessor, candidate, profileHash, checkInputHash(input.workspaceRoot, { ...candidate, gate: predecessor.gate }, profileHash, "", input.evidenceContext, input.checkEnvironment, false), checkInputHash(input.workspaceRoot, { ...candidate, gate: predecessor.gate }, profileHash, "", input.evidenceContext, input.checkEnvironment))
                 || input.retryReason === "check_inputs_changed" && staleRetryContractMatches(predecessor, candidate, profileHash, input)));
         if (!group || retries.has(group))
             throw new AppError("retry_check_invalid", "Retry must match exactly one current execution contract", 2, { check_receipt_id: receiptId });
@@ -579,7 +581,9 @@ async function runCodeChecksOwned(context, input, workspaceGateId, generation, s
         const inputFingerprint = inputFilesFingerprint(inputFiles);
         const inputHash = checkInputHash(input.workspaceRoot, group, profileHash, inputFingerprint, input.evidenceContext, input.checkEnvironment);
         const epoch = crypto.createHash("sha256").update(`${group.gate}\0${inputFingerprint}`).digest("hex");
-        const executionContractHash = checkInputHash(input.workspaceRoot, group, profileHash, "", input.evidenceContext, input.checkEnvironment);
+        // Retry preserves the declared check/wave, not the failed conditions being
+        // corrected. Observed environment still participates in inputHash/PASS reuse.
+        const executionContractHash = checkInputHash(input.workspaceRoot, group, profileHash, "", input.evidenceContext, input.checkEnvironment, false);
         const reservationInput = { ...input, retryReceiptIds: retries.has(group) ? [retries.get(group)] : [], generation, group, before, inputFiles, inputFingerprint, executionContractHash, profileHash, inputHash, epoch, startedAt };
         delete reservationInput.retryReceiptId;
         const decision = reserveOrReuseReceipt(context, reservationInput);
@@ -592,6 +596,7 @@ async function runCodeChecksOwned(context, input, workspaceGateId, generation, s
         const { id, evidenceDir, stdoutPath, stderrPath, receiptPath, completionPath } = reserved;
         let resources = {};
         let result = { exitCode: null, error: "check did not start", aborted: true };
+        let dispatched = false, physicallySettled = false;
         try {
             fs.mkdirSync(evidenceDir, { recursive: true });
             retainInputManifest(context, reserved);
@@ -624,6 +629,7 @@ async function runCodeChecksOwned(context, input, workspaceGateId, generation, s
                 const runnerEnvironment = group.runnerEvidence ? { DD_FLOW_CHECK_RUNNER_EVIDENCE_PATH: path.join(evidenceDir, group.runnerEvidence.path), DD_FLOW_CHECK_RUNNER_PROGRESS_PATH: path.join(evidenceDir, "runner-progress.jsonl"), DD_FLOW_CHECK_INVOCATION_ID: id, DD_FLOW_CHECK_ID: id, DD_FLOW_CHECK_INPUT_HASH: inputHash, DD_FLOW_CHECK_VERIFICATION_EPOCH: epoch } : {};
                 if (group.runnerEvidence)
                     fs.mkdirSync(path.dirname(path.join(evidenceDir, group.runnerEvidence.path)), { recursive: true });
+                dispatched = true;
                 result = await runCheck(group.command, input.workspaceRoot, { ...codeExecutionEnvironment(input.workspaceRoot, input.checkEnvironment), ...browserCheckCapabilityEnvironment(context, input.runId, input.runHome), DD_FLOW_EVIDENCE_DIR: evidenceDir, DD_FLOW_CHECK_COMPLETION_FILE: completionPath, ...runnerEnvironment, ...portEnvironment(resources) }, stdout, stderr, (elapsed) => input.progress?.(`check ${index + 1}/${groups.length} still running (${elapsed}s): ${group.command}`), (pid) => {
                     confirmedExpiry = confirmManagedProcess(context, { id: managed.id, leaseToken: managed.lease_token, pid, processGroupId: process.platform === "win32" ? null : pid }).lease_expires_at;
                     assertRunMutationAllowed(context, input.projectId, input.runId, generation);
@@ -638,6 +644,7 @@ async function runCodeChecksOwned(context, input, workspaceGateId, generation, s
             const current = managedProcessStatus(context).find(item => item.id === managed.id);
             if (processTreeIsAlive(current))
                 throw new AppError("check_in_progress", "The check shell exited but its process tree is still live; retain this receipt and its resources until settlement", 2, { process_id: managed.id, check_receipt_id: id });
+            physicallySettled = !result.cleanupUnconfirmed;
             allocation.release();
             if (!result.cleanupUnconfirmed)
                 finishManagedProcess(context, { id: managed.id, leaseToken: managed.lease_token, state: result.aborted || result.exitCode !== 0 || Boolean(result.error) ? "failed" : "stopped", reason: result.error });
@@ -664,6 +671,9 @@ async function runCodeChecksOwned(context, input, workspaceGateId, generation, s
         const physicalFailures = executionFailures({ fingerprintScope: "check_inputs", exitCode: result.exitCode, launcherError: result.error ?? undefined, cleanupUnconfirmed: result.cleanupUnconfirmed, missing: artifacts.missing, mutations: mutationPaths, invalidRunner: Boolean(group.runnerEvidence && !runnerEvidence) });
         if (result.failure)
             physicalFailures.push(result.failure);
+        // This marker is SQL-owned execution metadata, not a semantic runner verdict.
+        if (result.aborted && (!dispatched || physicallySettled))
+            physicalFailures.push({ code: "check_execution_settled", message: "No owned check process remains; investigate the retained launch or execution failure" });
         const candidateFiles = candidateFailureFiles(runnerEvidence, input.workspaceRoot);
         const candidateAbsences = candidateFailureAbsences(runnerEvidence, input.workspaceRoot);
         const libraryInputs = candidateLibraryInputs(runnerEvidence, input.workspaceRoot);
@@ -826,14 +836,9 @@ function reserveOrReuseReceipt(context, input) {
     const successor = retryIds.size === 1 ? receipts.find((receipt) => receipt.retry_of_receipt_id === [...retryIds][0] && receipt.input_hash === input.inputHash && receipt.profile_hash === input.profileHash) : null;
     const reusableSuccessor = successor?.status === "passed" && checkReceiptEvidenceValid(successor);
     const retryPredecessor = receipts.find(receipt => retryIds.has(receipt.id));
-    const sourceCorrection = Boolean(retryPredecessor && (sourceRepairContractMatches(retryPredecessor, input.group, input.profileHash)
+    const sourceCorrection = Boolean(retryPredecessor && (diagnosisRetryContractMatches(retryPredecessor, input.group, input.profileHash, input.executionContractHash, checkInputHash(input.workspaceRoot, input.group, input.profileHash, "", input.evidenceContext, input.checkEnvironment))
         || input.retryReason === "check_inputs_changed" && staleRetryContractMatches(retryPredecessor, input.group, input.profileHash, input)));
     const latestContract = receipts.filter(receipt => receipt.command === input.group.command && receipt.profile_hash === input.profileHash && receipt.gate === input.group.gate && receipt.scope === input.scope && receipt.work_id === (input.workId ?? null)).at(-1);
-    // Hash files before SQLite's writer reservation. Captured identities remain
-    // valid historical evidence, but cannot authorize reuse in a changed install.
-    const priorLibrariesCurrent = !prior || currentCandidateLibraryInputsValid(prior, input.workspaceRoot);
-    const latestLibrariesCurrent = !latestContract || currentCandidateLibraryInputsValid(latestContract, input.workspaceRoot);
-    const libraryDrift = (receipt) => { throw new AppError("check_evidence_stale", "Captured library inputs differ from the current candidate installation; request an explicit restored-condition retry", 2, { reason: "candidate_library_inputs_changed", check_receipt_id: receipt.id, effect: "no_effect" }); };
     return context.db.writeTransaction(() => {
         assertRunMutationAllowed(context, input.projectId, input.runId, input.generation);
         if (JSON.stringify(context.db.all(snapshotSql, [input.projectId, input.runId])) !== snapshot)
@@ -869,16 +874,10 @@ function reserveOrReuseReceipt(context, input) {
         // An unchanged failed contract needs a concrete restored-condition retry,
         // not repeated coordinator finish until it happens to become green.
         if (!retryId && latestContract?.fingerprint_scope === "check_inputs" && latestContract.status === "failed"
-            && latestContract.execution_failures?.some(failure => failure.code === "check_inputs_changed") && latestContract.execution_contract_hash === input.executionContractHash) {
-            if (!latestLibrariesCurrent)
-                libraryDrift(latestContract);
+            && latestContract.execution_failures?.some(failure => failure.code === "check_inputs_changed") && latestContract.execution_contract_hash === input.executionContractHash)
             return { receipt: latestContract };
-        }
-        if (prior && ["failed", "aborted"].includes(prior.status) && !retryId) {
-            if (!priorLibrariesCurrent)
-                libraryDrift(prior);
+        if (prior && ["failed", "aborted"].includes(prior.status) && !retryId)
             return { receipt: prior };
-        }
         if (prior?.status === "passed" && !retryIds.has(prior.id) && input.group.reusable && reusablePrior)
             return { receipt: prior };
         let ordinal = (context.db.get("SELECT COALESCE(MAX(reservation_sequence), 0) + 1 AS sequence FROM check_receipts WHERE project_id = ? AND run_id = ?", [input.projectId, input.runId])?.sequence ?? 1);
@@ -895,9 +894,12 @@ function reserveOrReuseReceipt(context, input) {
         return { reserved };
     }, "checks.reserveOrReuseReceipt");
 }
-function sourceRepairContractMatches(receipt, group, profileHash) {
-    return checkRepairDisposition(receipt) === "source_repair" && group.command === receipt.command && profileHash === receipt.profile_hash && isDeepStrictEqual(group.inputs, receipt.inputs)
-        && isDeepStrictEqual(derivedReceiptArtifacts(group, receipt).sort(), [...receipt.required_artifacts].sort()) && isDeepStrictEqual([...group.ports].sort(), Object.keys(receipt.resources.ports).sort());
+function diagnosisRetryContractMatches(receipt, group, profileHash, executionContractHash, legacyExecutionContractHash) {
+    return checkDiagnosisDisposition(receipt) === "diagnosis_required" && group.command === receipt.command && profileHash === receipt.profile_hash && isDeepStrictEqual(group.inputs, receipt.inputs)
+        && [executionContractHash, legacyExecutionContractHash].includes(receipt.execution_contract_hash ?? "")
+        && group.requiredArtifacts.every(artifact => receipt.required_artifacts.includes(artifact))
+        && isDeepStrictEqual(group.runnerEvidence ?? null, receipt.runner_evidence_required ?? null)
+        && isDeepStrictEqual([...group.ports].sort(), Object.keys(receipt.resources.ports).sort());
 }
 function staleCheckRetryEligible(receipt) {
     if (receipt.fingerprint_scope !== "check_inputs" || receipt.status !== "failed" || receipt.exit_code !== 0 || receipt.retry_reason === "check_inputs_changed"
@@ -908,7 +910,8 @@ function staleCheckRetryEligible(receipt) {
 }
 function staleRetryContractMatches(receipt, group, profileHash, input) {
     return group.reusable && staleCheckRetryEligible(receipt) && isDeepStrictEqual(group.runnerEvidence ?? null, receipt.runner_evidence_required ?? null)
-        && receipt.execution_contract_hash === checkInputHash(input.workspaceRoot, group, profileHash, "", input.evidenceContext, input.checkEnvironment);
+        && [checkInputHash(input.workspaceRoot, group, profileHash, "", input.evidenceContext, input.checkEnvironment, false),
+            checkInputHash(input.workspaceRoot, group, profileHash, "", input.evidenceContext, input.checkEnvironment)].includes(receipt.execution_contract_hash ?? "");
 }
 function receiptFor(input) { return { id: input.id, local_id: input.localId, declaration_id: input.group.declarations[0].id, check_refs: input.group.declarations.map(checkRef), gate: input.group.gate, scope: input.scope, work_id: input.workId ?? null, command: input.group.command, input_hash: input.inputHash, verification_epoch: input.epoch, retry_of_receipt_id: input.retryOf ?? null, retry_reason: input.retryReason ?? null, status: input.status, ...(input.status === "aborted" ? { abort_reason: input.result.error } : {}), exit_code: input.status === "running" ? null : input.result.exitCode, stdout_path: input.stdoutPath, stderr_path: input.stderrPath, receipt_path: input.receiptPath, completion_path: input.completionPath, required_artifacts: input.group.requiredArtifacts, inputs: input.group.inputs, fingerprint_scope: "check_inputs", input_manifest: input.inputManifest ?? null, execution_contract_hash: input.executionContractHash, workspace_fingerprint: input.before.fingerprint, before_fingerprint: input.inputFingerprint, after_fingerprint: input.status === "running" ? input.inputFingerprint : input.after.fingerprint, profile_hash: input.profileHash, mutation_paths: input.mutationPaths, artifacts: input.artifacts, resources: { ports: input.resources }, started_at: input.startedAt, finished_at: input.status === "running" ? input.startedAt : new Date().toISOString() }; }
 function retainInputManifest(context, reserved) {
@@ -966,8 +969,6 @@ export function validateFailedCheckRetry(context, input) {
     const declared = new Set(input.declarations.flatMap((check) => [check.id, checkRef(check)]));
     if (!receipt.check_refs.some((ref) => declared.has(ref)))
         throw new AppError("retry_check_invalid", "--retry-check does not belong to this stage gate", 2, { check_receipt_id: input.receiptId, check_refs: receipt.check_refs });
-    if (receipt.fingerprint_scope === "check_inputs" && receipt.execution_failures?.some(failure => failure.code === "check_inputs_changed") && input.reason !== "check_inputs_changed")
-        throw new AppError("retry_check_invalid", "Scoped drift uses its bounded stale transition, not an environment/source retry", 2, { check_receipt_id: receipt.id });
     if (input.reason === "check_inputs_changed" && (!staleCheckRetryEligible(receipt) || receipt.retry_reason === "check_inputs_changed"))
         throw new AppError("retry_check_invalid", "Scoped input retry requires a bound, safely completed stale predecessor and one unconsumed episode", 2, { check_receipt_id: receipt.id });
     return receipt;
@@ -1326,12 +1327,28 @@ function explicitInputFiles(root, inputs, ignoredRoots, sourceFiles) {
         visit(path.resolve(root, input), input, new Set());
     return [...result.entries()];
 }
-function checkInputHash(workspaceRoot, group, profileHash, inputFingerprint, evidenceContext, checkEnvironment) { return crypto.createHash("sha256").update(JSON.stringify({ workspace_root: path.resolve(workspaceRoot), gate: group.reusable ? null : group.gate, command: group.command, required_artifacts: group.requiredArtifacts, ports: group.ports, inputs: group.inputs, ...(group.runnerEvidence ? { runner_evidence: group.runnerEvidence } : {}), profile_hash: profileHash, environment: environmentFingerprint(workspaceRoot, group.environmentInputs, checkEnvironment), input_fingerprint: inputFingerprint, ...(group.environmentInputs === undefined ? {} : { environment_inputs: group.environmentInputs }), ...(evidenceContext === undefined ? {} : { evidence_context: evidenceContext }) })).digest("hex"); }
+function checkInputHash(workspaceRoot, group, profileHash, inputFingerprint, evidenceContext, checkEnvironment, observedEnvironment = true) { return crypto.createHash("sha256").update(JSON.stringify({ workspace_root: path.resolve(workspaceRoot), gate: group.reusable ? null : group.gate, command: group.command, required_artifacts: group.requiredArtifacts, ports: group.ports, inputs: group.inputs, ...(group.runnerEvidence ? { runner_evidence: group.runnerEvidence } : {}), profile_hash: profileHash, environment: observedEnvironment ? environmentFingerprint(workspaceRoot, group.environmentInputs, checkEnvironment) : null, input_fingerprint: inputFingerprint, ...(group.environmentInputs === undefined ? {} : { environment_inputs: group.environmentInputs }), ...(evidenceContext === undefined ? {} : { evidence_context: evidenceContext }) })).digest("hex"); }
 function derivedReceiptArtifacts(group, receipt) { return [...new Set([...group.requiredArtifacts, ...runnerProofArtifacts(receipt.runner_evidence), ...(receipt.required_artifacts.includes("engine-outcome.json") ? ["engine-outcome.json"] : [])])]; }
 function bindReceipt(receipt, group) { return { ...receipt, gate: group.gate, declaration_id: group.declarations[0].id, check_refs: group.declarations.map(checkRef), required_artifacts: derivedReceiptArtifacts(group, receipt), inputs: group.inputs }; }
 export function checkReceiptEvidenceValid(receipt) { return checkReceiptEvidenceValidForMode(receipt); }
+function availableDiagnosticArtifacts(receipt) {
+    const root = path.join(path.dirname(receipt.receipt_path), "artifacts");
+    return receipt.artifacts.flatMap(artifact => {
+        const relative = path.isAbsolute(artifact.path) ? path.relative(root, artifact.path) : artifact.path;
+        const observed = collectCheckArtifacts(root, [relative]);
+        return observed.items.filter(item => item.sha256 === artifact.sha256);
+    });
+}
 function checkReceiptEvidenceValidForMode(receipt, mode) {
     try {
+        if (mode === "diagnostic" && receipt.status !== "passed") {
+            if (checkDiagnosisDisposition(receipt) !== "diagnosis_required")
+                return false;
+            // SQL selects this execution. Only its identity must survive in the
+            // diagnostic packet; missing reports/logs cannot erase a failed command.
+            const stored = JSON.parse(readBoundedReleaseFile(receipt.receipt_path, 1024 * 1024).toString("utf8"));
+            return ["id", "scope", "work_id", "command", "input_hash", "verification_epoch", "status", "exit_code", "started_at", "finished_at"].every(field => (stored[field] ?? null) === (receipt[field] ?? null));
+        }
         const staleDiagnostic = mode === "diagnostic" && receipt.fingerprint_scope === "check_inputs" && receipt.status === "failed" && Number.isInteger(receipt.exit_code)
             && receipt.before_fingerprint !== receipt.after_fingerprint && receipt.mutation_paths.length > 0 && receipt.execution_failures?.some(failure => failure.code === "check_inputs_changed");
         if (receipt.fingerprint_scope != null && receipt.fingerprint_scope !== "check_inputs")

package/dist/services/codex-hook-delivery.js

@@ -19,7 +19,10 @@ export async function verifyCodexHookDelivery(env, payload) {
         throw new AppError("native_hook_timeout", "Native hook deadline expired before delivery verification", 1, { phase: "verification", effect: "no_effect", recoverable: false });
     const proof = await callDaemon(env.DD_FLOW_CODEX_STATE_DIR, "hook.verify", {
         daemonId: env.DD_FLOW_DAEMON_ID, rootSessionId: payload.session_id,
-        sessionId: payload.agent_id ?? payload.session_id, toolCallId: payload.tool_use_id, turnId: payload.turn_id
+        sessionId: payload.agent_id ?? payload.session_id, toolCallId: payload.tool_use_id, turnId: payload.turn_id,
+        // The daemon verifies the native delivery before retaining it; the source
+        // command cannot itself grant admission to the CLI.
+        nativePayload: payload, nativeCommand: command
     }, Math.min(5000, remaining));
     if (typeof proof.projectRoot !== "string"
         || typeof proof.ddFlowHome !== "string" || !env.DD_FLOW_HOME || path.resolve(env.DD_FLOW_HOME) !== path.resolve(proof.ddFlowHome)
@@ -27,6 +30,12 @@ export async function verifyCodexHookDelivery(env, payload) {
         || typeof proof.toolCallId !== "string" || !(proof.parentSessionId === null || typeof proof.parentSessionId === "string")) {
         throw new AppError("native_hook_unproven", "Native hook proof differs from the requested project/runtime", 1, { effect: "no_effect" });
     }
+    const binding = proof.nativeHookBinding;
+    const retained = binding && typeof binding === "object" && !Array.isArray(binding) ? binding : null;
     return { daemonId: proof.daemonId, rootSessionId: proof.rootSessionId, sessionId: proof.sessionId,
-        parentSessionId: proof.parentSessionId, toolCallId: proof.toolCallId, projectRoot: proof.projectRoot, command: command };
+        parentSessionId: proof.parentSessionId, toolCallId: proof.toolCallId, projectRoot: proof.projectRoot, command: command,
+        ...(retained?.schema_id === "dd-flow/hook-request@1" && retained.source_harness === "codex-desktop"
+            && retained.daemon_id === proof.daemonId && retained.root_provider_session_id === proof.rootSessionId
+            && retained.provider_session_id === proof.sessionId && retained.tool_call_id === proof.toolCallId
+            && retained.native_command === command && typeof retained.request_id === "string" ? { nativeHookBinding: retained } : {}) };
 }

package/dist/services/controller-child-repair.js

@@ -0,0 +1,95 @@
+import { AppError } from "../shared/errors.js";
+import { renderNativeChildFollowupInstruction, renderWaitInstructions } from "../harness-runtime/lib/delegation-instructions.mjs";
+import { nativeItemsSettled } from "../harness-runtime/lib/provider-overload-policy.mjs";
+import { parseLifecycleCommand } from "./lifecycle-command.js";
+import { storageSessionId } from "./session-identity.js";
+import { stageWorkIds } from "./stage-work-graph.js";
+import { appendEvent, digest } from "./run-controller-state.js";
+const blocked = (workId, reason, diagnostic = {}) => new AppError("child_work_repair_blocked", "Completed child has no safe automatic Work correction", 1, { work_id: workId, reason, reconciliation_required: true, effect: "no_effect", ...diagnostic });
+/** One durable same-child correction per Work, not a provider retry or a new worker. */
+export function completedChildRepair(context, row, input) {
+    const members = stageWorkIds(context, row.project_id, row.run_id, input.stage);
+    for (const child of input.children) {
+        if (child.status !== "completed")
+            continue;
+        const links = context.db.all(`SELECT w.work_id,w.status FROM work_sessions ws
+      JOIN works w ON w.work_id=ws.work_id JOIN sessions s ON s.session_id=ws.session_id AND s.project_id=w.project_id
+      WHERE w.project_id=? AND w.run_id=? AND ws.session_id=? AND s.parent_session_id=?`, [row.project_id, row.run_id, storageSessionId({ harness_id: input.harness, session_id: child.session_id }), storageSessionId({ harness_id: input.harness, session_id: input.parentId })])
+            .filter(link => members.has(link.work_id));
+        if (!links.some(link => link.status === "running"))
+            continue;
+        if (links.length !== 1 || child.parent_session_id !== input.parentId || child.provenance === "parent_mismatch"
+            || !child.terminal_turn_id || !nativeItemsSettled(child.native_turn_items))
+            throw blocked(links[0].work_id, "native_identity_unconfirmed");
+        const workId = links[0].work_id;
+        if (context.db.get("SELECT 1 FROM check_receipts WHERE project_id=? AND run_id=? AND work_id=? AND status='running' LIMIT 1", [row.project_id, row.run_id, workId]))
+            continue;
+        if (context.db.get("SELECT 1 FROM works WHERE parent_work_id=? AND status IN ('created','running','paused') LIMIT 1", [workId]))
+            throw blocked(workId, "descendants_unsettled");
+        const previous = context.db.get(`SELECT e.controller_id,e.data_json FROM run_controller_events e
+      JOIN run_controllers c ON c.controller_id=e.controller_id WHERE c.project_id=? AND c.run_id=?
+      AND e.event_type='work_repair_reserved' AND json_extract(e.data_json,'$.work_id')=? ORDER BY e.sequence LIMIT 1`, [row.project_id, row.run_id, workId]);
+        if (previous) {
+            const retained = JSON.parse(previous.data_json);
+            const operation = context.db.get("SELECT status,input_json FROM run_controller_operations WHERE operation_id=?", [retained.operation_id]);
+            const diagnostic = { operation_id: retained.operation_id, refusal_invocation_id: retained.refusal_invocation_id };
+            if (previous.controller_id !== row.controller_id || retained.generation !== row.generation || retained.terminal_turn_id !== child.terminal_turn_id
+                || retained.child_session_id !== child.session_id)
+                throw blocked(workId, "automatic_episode_exhausted", diagnostic);
+            if (operation?.status === "completed" || operation?.status === "failed")
+                throw blocked(workId, "followup_admission_unconfirmed", diagnostic);
+            // Same operation ID lets the adapter observe an uncertain dispatch. It
+            // may dispatch only if its own durable phase proves it never crossed out.
+            return retained;
+        }
+        const refusals = context.db.all("SELECT id,command,scope_json,identity_json,outcome_json FROM lifecycle_invocations WHERE status='settled' AND json_extract(outcome_json,'$.error.details.work_id')=? ORDER BY rowid DESC", [workId]);
+        const refusal = refusals.find(attempt => {
+            try {
+                const scope = JSON.parse(attempt.scope_json), identity = JSON.parse(attempt.identity_json ?? "null");
+                const parsed = parseLifecycleCommand(attempt.command);
+                return attempt.scope_json === context.env.DD_FLOW_INVOCATION_SCOPE && parsed.kind === "standalone" && parsed.invocation.operation === "work_finish" && scope.runId === row.run_id
+                    && scope.generation === row.generation && scope.rootSessionId === input.parentId
+                    && Array.isArray(identity) && identity[2] === child.session_id && identity[3] === input.parentId;
+            }
+            catch {
+                return false;
+            }
+        });
+        if (!refusal)
+            throw blocked(workId, "repair_refusal_missing");
+        const outcome = JSON.parse(refusal.outcome_json), error = outcome.error, details = error?.details;
+        const diagnostic = { refusal_invocation_id: refusal.id, primary_error: { code: error?.code, message: error?.message } };
+        if (details?.effect !== "no_effect" || details.recoverable !== true || typeof details.retry_command !== "string"
+            || typeof outcome.successor_invocation_id !== "string")
+            throw blocked(workId, "repair_successor_unproven", diagnostic);
+        const successor = context.db.get("SELECT scope_json,status FROM lifecycle_invocations WHERE id=?", [outcome.successor_invocation_id]);
+        if (successor?.status !== "issued" || successor.scope_json !== refusal.scope_json)
+            throw blocked(workId, "repair_successor_not_issued", diagnostic);
+        const instruction = renderNativeChildFollowupInstruction(input.harness, child, { purpose: "work_repair", context: JSON.stringify({ work_id: workId, code: error.code, message: error.message, retry_command: details.retry_command }) });
+        if (!instruction)
+            throw blocked(workId, "same_child_repair_unsupported", diagnostic);
+        return { work_id: workId, generation: row.generation, child_session_id: child.session_id, terminal_turn_id: child.terminal_turn_id, refusal_invocation_id: refusal.id,
+            successor_invocation_id: outcome.successor_invocation_id, operation_id: `${row.controller_id}:work-repair:${digest([row.run_id, workId])}`,
+            prompt: `Continue only the existing child for this unfinished Work. This is one bounded correction, not a new fanout wave.\n${instruction}\n\nA follow-up submission acknowledgement or the child's previous completed Turn is not completion of this correction. Wait for this same child's follow-up; never resend the message to poll it.\n${renderWaitInstructions({ harness: input.harness, stage: input.stage })}` };
+    }
+    return null;
+}
+export function reserveChildRepair(context, row, repair, assertOwner) {
+    context.db.writeTransaction(() => {
+        assertOwner();
+        if (repair.generation !== row.generation)
+            throw blocked(repair.work_id, "generation_changed_before_followup");
+        const work = context.db.get("SELECT status FROM works WHERE project_id=? AND run_id=? AND work_id=?", [row.project_id, row.run_id, repair.work_id]);
+        if (work?.status !== "running")
+            throw blocked(repair.work_id, "work_changed_before_followup");
+        const successor = context.db.get("SELECT status FROM lifecycle_invocations WHERE id=?", [repair.successor_invocation_id]);
+        if (successor?.status !== "issued")
+            throw blocked(repair.work_id, "successor_changed_before_followup");
+        const reserved = context.db.get(`SELECT e.controller_id,e.data_json FROM run_controller_events e JOIN run_controllers c ON c.controller_id=e.controller_id
+      WHERE c.project_id=? AND c.run_id=? AND e.event_type='work_repair_reserved' AND json_extract(e.data_json,'$.work_id')=?`, [row.project_id, row.run_id, repair.work_id]);
+        if (reserved && (reserved.controller_id !== row.controller_id || reserved.data_json !== JSON.stringify(repair)))
+            throw blocked(repair.work_id, "episode_reserved_by_another_dispatch");
+        if (!reserved)
+            appendEvent(context, row.controller_id, "work_repair_reserved", repair);
+    });
+}

package/dist/services/controller-execution.js

@@ -5,6 +5,7 @@ import { resolveExecutionPolicy, resolveStageTransition, stageExecutionCwd } fro
 import { readFlowRunExecutionBundle } from "./runs.js";
 import { hybridDigest } from "./hybrid-contracts.js";
 import { isDiagnosticFlow } from "../domain/diagnostic-flows.js";
+import { isDeliveryFlow } from "../domain/delivery-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.
  */
@@ -44,7 +45,9 @@ export function genericControllerRouting(execution, workspaceRoot) {
  * 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) || isDiagnosticFlow(run.flow_kind)) {
+    const boundDistill = run.flow_kind === "mb-distill" && JSON.parse(run.index_json).execution_bundle?.flow_id === "mb-distill";
+    const boundDelivery = isDeliveryFlow(run.flow_kind) && JSON.parse(run.index_json).execution_bundle?.flow_id === run.flow_kind;
+    if (manifest?.schema_id === "dd-flow/run-controller-manifest@2" || ["mb-init", "mb-upgrade", "mb-fix", "review-fix", "interactive"].includes(run.flow_kind) || isDiagnosticFlow(run.flow_kind) || boundDistill || boundDelivery) {
         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);

package/dist/services/controller-fanout.js

@@ -12,7 +12,8 @@ import { requireProjectByRoot } from "./projects.js";
 import { stageAttempt, stageWorkIds } from "./stage-work-graph.js";
 import { runtimeCommand } from "./runtime-command.js";
 import { promptDeliveryInstruction } from "./prompt-delivery.js";
-import { currentStageFinishCommand } from "./lifecycle-invocations.js";
+import { currentStageFinishAuthority } from "./lifecycle-invocations.js";
+import { acceptedChildWork } from "./controller-result-authority.js";
 function semanticCompletionPrompt(stage, completed, command) {
     const task = stage === "plan-review"
         ? "Classify the reviewer findings, apply every accepted correction in the current PLAN-REVIEW Work, and write the authoritative review decision."
@@ -71,11 +72,17 @@ export async function nextControllerFanout(context, input) {
         throw new AppError("stage_graph_compatibility_required", "The retained graph cannot be dispatched by this controller; use the RUN's pinned engine", 1, { stage: input.stage, run_id: input.runId, graph: fanout });
     if (children.length) {
         const reconciled = await reconcileVnextFanout(context, { ...input, observations });
-        const issues = reconciled.issues.filter(issue => !["child_check_in_progress", "child_capacity_pending"].includes(issue.code));
+        // A known settled child's Work blocker does not consume an independent
+        // ready branch. It remains running, so this cannot manufacture Stage success.
+        const issues = reconciled.issues.filter(issue => !["child_check_in_progress", "child_capacity_pending"].includes(issue.code)
+            && !(issue.code === "execution_ended_without_work_result" && issue.work_id && input.repairBlockedWorks?.includes(issue.work_id)));
         if (issues.length)
             throw new AppError("fanout_reconciliation_required", "Native child outcome requires reconciliation, not replay", 1, { issues });
-        if (children.some(child => child.status === "running" || child.status === "unknown"))
-            return { kind: "wait", reason: "native_children_unsettled" };
+        const unsettled = children.filter(child => child.status === "running" || child.status === "unknown");
+        if (unsettled.length) {
+            const project = requireProjectByRoot(context, input.projectRoot);
+            return { kind: "wait", reason: unsettled.every(child => acceptedChildWork(context, { project_id: project.id, run_id: input.runId }, input.harness, input.sessionId, child.session_id)) ? "accepted_work_native_cleanup" : "native_children_unsettled" };
+        }
         fanout = getVnextFanoutStatus(context, input).orchestration;
     }
     if (input.dispatchFence && !fanout.works.failed)
@@ -155,7 +162,8 @@ export async function nextControllerFanout(context, input) {
             const work = choice.work;
             const row = context.db.get("SELECT payload_json,result_schema FROM works WHERE work_id=?", [work.work_id]);
             const payload = JSON.parse(row?.payload_json ?? "null");
-            if (payload?.kind === "memory-review" && row?.result_schema === "dd-flow/memory-review-work-result@1") {
+            if (payload?.kind === "memory-review" && row?.result_schema === "dd-flow/memory-review-work-result@1"
+                || payload?.kind === "interactive-review" && row?.result_schema === "dd-flow/interactive-work-result@1") {
                 const copy = prepareMemoryNativeReviewCopy(context, work.work_id, true);
                 return { ...work, start_command: `cd '${copy.root.replaceAll("'", "'\\''")}' && ${workStartCommand(context, assertWorkLaunchReady(context, work.work_id))}` };
             }
@@ -171,7 +179,7 @@ export async function nextControllerFanout(context, input) {
             if (!work.work_id || !work.start_command)
                 throw new AppError("fanout_contract_invalid", "Ready Work omits its trusted start command", 1);
         const totalWorks = fanout.works.created + fanout.works.running + fanout.works.completed + fanout.works.failed + fanout.works.cancelled;
-        return prompt("dispatch_work", renderDelegationInstructions({ harness: input.harness, stage: input.stage, capacity, works: wave, totalWorks, remainingWorks: fanout.works.created - wave.length }));
+        return { ...prompt("dispatch_work", renderDelegationInstructions({ harness: input.harness, stage: input.stage, capacity, works: wave, totalWorks, remainingWorks: fanout.works.created - wave.length })), work_ids: wave.map(work => work.work_id) };
     }
     if (fanout.works.running)
         return { kind: "wait", reason: "work_receipts_pending" };
@@ -209,10 +217,12 @@ export async function nextControllerFanout(context, input) {
             ].join("\n\n"));
         }
     }
-    const finish = currentStageFinishCommand(context, input.stage);
+    const finish = currentStageFinishAuthority(context, input.stage);
     if (!finish)
-        throw new AppError("controller_finish_authority_missing", "Settled child Work has no current issued Stage finish; reconcile lifecycle authority", 1, { stage: input.stage });
-    return prompt("semantic_completion", semanticCompletionPrompt(input.stage, fanout.works.completed, finish));
+        throw new AppError("controller_finish_authority_missing", "Settled child Work has no current Stage finish authority", 1, { stage: input.stage, reconciliation_required: true });
+    if (finish.status !== "issued" || !finish.command)
+        throw new AppError("controller_finish_authority_pending", "Observe the original Stage finish before another productive dispatch", 1, { stage: input.stage, invocation_id: finish.id, status: finish.status, reconciliation_required: true });
+    return prompt("semantic_completion", semanticCompletionPrompt(input.stage, fanout.works.completed, finish.command));
 }
 /** Shared pinned runtime normalization; preserve structured errors at the TS boundary. */
 export function controllerNativeChildren(receipt, rootSessionId) {

package/dist/services/controller-memory-worker.js

@@ -6,7 +6,8 @@ import { launchExternalWork } from "./external-work-launch.js";
 import { requireProjectByRoot } from "./projects.js";
 import { assertControllerLeaseAllowed } from "./run-controller-process.js";
 import { assertManagedProcessLease } from "./managed-processes.js";
-import { isDiagnosticFlow } from "../domain/diagnostic-flows.js";
+import { isSourceSnapshotFlow } from "../domain/source-snapshot-flows.js";
+import { isDeliveryFlow } from "../domain/delivery-flows.js";
 if (!parentPort)
     throw new AppError("controller_memory_task_invalid", "Memory worker requires its controller thread", 1);
 const port = parentPort, cancellation = new AbortController();
@@ -18,9 +19,10 @@ let reply;
 try {
     context = createContext(env, "prepared_existing");
     const project = requireProjectByRoot(context, task.input.projectRoot);
-    const run = context.db.get("SELECT flow_kind FROM runs WHERE project_id=? AND id=?", [project.id, task.input.runId]);
-    if (!run || !["mb-init", "mb-upgrade"].includes(run.flow_kind) && !isDiagnosticFlow(run.flow_kind))
-        throw new AppError("controller_memory_task_invalid", "Worker requires the selected memory or diagnostic RUN", 1, { effect: "no_effect" });
+    const run = context.db.get("SELECT flow_kind,index_json FROM runs WHERE project_id=? AND id=?", [project.id, task.input.runId]);
+    const delivery = run && isDeliveryFlow(run.flow_kind) && JSON.parse(run.index_json).execution_bundle?.flow_id === run.flow_kind;
+    if (!run || !["mb-init", "mb-upgrade", "mb-fix", "interactive"].includes(run.flow_kind) && !isSourceSnapshotFlow(run.flow_kind) && !delivery)
+        throw new AppError("controller_memory_task_invalid", "Worker requires the selected memory, interactive, source-snapshot or bound delivery RUN", 1, { effect: "no_effect" });
     if (task.operation === "external") {
         const work = context.db.get("SELECT project_id,run_id FROM works WHERE work_id=?", [task.input.workId]);
         if (!work || work.project_id !== project.id || work.run_id !== task.input.runId)