@ccdd/project 5.1.1 → 6.0.0

package/dist/src/broker/index.js

@@ -10,6 +10,10 @@ import { setTimeout as delay } from 'node:timers/promises';
 import { prepareReviewRequests } from '../requester/index.js';
 import { readWorkspaceConfig } from './config.js';
 import { createGraphDefinition } from './graph.js';
+import { localAdmission } from './admission.js';
+import { readChanges } from './changes.js';
+import { initializeRecords, records } from './storage.js';
+import { createReadiness } from './readiness.js';
 import { createStatusPolling } from './polling.js';
 import { ownerAlive, ownProcessIdentity } from './ownership.js';
 import { coalescingEligibility, findCoalescibleRequest } from './coalescing.js';
@@ -86,7 +90,7 @@ function prepareStateDirectory(repoPath, stateDir) {
     return fs.realpathSync(canonical);
 }
 /** Durable broker operations. Merely opening the store never starts a worker. */
-export function createBroker({ detail, repoPath, stateDir, repoId = 'demo', coalescingGraceMs = 15_000, maxConcurrentExecutors = 4, identityConcurrency = DEFAULT_IDENTITY_CONCURRENCY, executors, workspaceIntegrity = 'content', workspaceAdapter = { prepareWorkspace, reopenWorkspace } }) {
+export function createBroker({ detail, repoPath, stateDir, repoId = 'demo', coalescingGraceMs = 15_000, maxConcurrentExecutors = 4, admission = localAdmission(maxConcurrentExecutors), identityConcurrency = DEFAULT_IDENTITY_CONCURRENCY, executors, workspaceIntegrity = 'content', workspaceAdapter = { prepareWorkspace, reopenWorkspace } }) {
     positiveConcurrency(maxConcurrentExecutors, 'maxConcurrentExecutors');
     positiveConcurrency(identityConcurrency, 'identityConcurrency');
     if (!Number.isSafeInteger(coalescingGraceMs) || coalescingGraceMs < 0 || coalescingGraceMs > 300_000)
@@ -121,10 +125,12 @@ export function createBroker({ detail, repoPath, stateDir, repoId = 'demo', coal
       CREATE TABLE IF NOT EXISTS runs (id TEXT PRIMARY KEY, created_at TEXT NOT NULL, status TEXT NOT NULL, data TEXT NOT NULL);
       CREATE TABLE IF NOT EXISTS requests (id TEXT PRIMARY KEY, run_id TEXT NOT NULL REFERENCES runs(id), ordinal INTEGER NOT NULL, status TEXT NOT NULL, data TEXT NOT NULL);
       CREATE INDEX IF NOT EXISTS requests_run ON requests(run_id, ordinal);
-      CREATE INDEX IF NOT EXISTS requests_validation_input ON requests(json_extract(data, '$.validationInput.key'));
+      CREATE INDEX IF NOT EXISTS requests_validation_input ON requests(json_extract(data, '$.criticId'),json_extract(data, '$.inputKey'));
+      CREATE INDEX IF NOT EXISTS requests_ready ON requests(run_id,status,ordinal);
       CREATE TABLE IF NOT EXISTS events (id INTEGER PRIMARY KEY AUTOINCREMENT, run_id TEXT NOT NULL REFERENCES runs(id), request_id TEXT, created_at TEXT NOT NULL, type TEXT NOT NULL, message TEXT NOT NULL, data TEXT);
       CREATE TABLE IF NOT EXISTS metadata (key TEXT PRIMARY KEY, value TEXT NOT NULL);
       CREATE TABLE IF NOT EXISTS run_owners (run_id TEXT PRIMARY KEY REFERENCES runs(id), pid INTEGER NOT NULL, process_identity TEXT, token TEXT NOT NULL, claimed_at TEXT NOT NULL);`);
+        initializeRecords(db);
         const previousRepo = db.prepare('SELECT value FROM metadata WHERE key = ?').get('registered-repo');
         const identity = JSON.stringify({ repoPath, repoId });
         if (previousRepo && previousRepo.value !== identity)
@@ -144,53 +150,36 @@ export function createBroker({ detail, repoPath, stateDir, repoId = 'demo', coal
     const active = new Map();
     const listeners = new Set();
     const ensureOpen = () => { if (closed || closing)
-        throw new Error('Broker is closed.'); sweepSubmissionDeadlines(); };
+        throw new Error('Broker is closed.'); };
     const requireExecutors = () => {
         if (!executors || typeof executors.canExecute !== 'function' || typeof executors.execute !== 'function')
             throw new Error('Review submission and execution require an executor registry.');
         return executors;
     };
+    const store = records(db);
+    const clearReadCaches = () => store.clear();
     const transaction = (callback) => {
         db.exec('BEGIN IMMEDIATE');
         try {
-            const result = callback();
+            const value = callback();
             db.exec('COMMIT');
-            return result;
+            return value;
         }
         catch (error) {
-            db.exec('ROLLBACK');
+            try {
+                db.exec('ROLLBACK');
+            }
+            finally {
+                store.clear();
+            }
             throw error;
         }
     };
-    const requestData = (id) => {
-        const row = db.prepare('SELECT data FROM requests WHERE id = ?').get(id);
-        return row ? parseStored(row.data) : null;
-    };
-    // Run definitions never change after submission. Keep one parsed definition,
-    // while re-reading mutable lifecycle/claim state on every operation (including
-    // writes from other Broker processes). Do not hydrate a whole project per Critic.
-    let runDefinition;
-    const runData = (id) => {
-        if (runDefinition?.id !== id) {
-            const row = db.prepare('SELECT data FROM runs WHERE id = ?').get(id);
-            if (!row)
-                return null;
-            const run = parseStored(row.data);
-            if (!run.project)
-                return run;
-            runDefinition = { id, graph: run.graph, snapshot: run.project.snapshot, templates: run.project.templates };
-            return run;
-        }
-        const row = db.prepare("SELECT json_remove(data, '$.graph', '$.project.snapshot', '$.project.templates') AS data FROM runs WHERE id = ?").get(id);
-        if (!row)
-            return null;
-        const run = parseStored(row.data);
-        return { ...run, graph: runDefinition.graph, ...(run.project ? { project: { ...run.project, snapshot: runDefinition.snapshot, templates: runDefinition.templates } } : {}) };
-    };
+    const requestData = (id) => store.request(id);
+    const runData = (id) => store.run(id);
     const ownerData = (id) => db.prepare('SELECT * FROM run_owners WHERE run_id = ?').get(id);
-    const runRequests = (id) => db.prepare('SELECT data FROM requests WHERE run_id = ? ORDER BY ordinal').all(id).map(row => parseStored(row.data));
-    const sharedRequests = (run) => (run.project?.coalescedRequestIds ?? []).map(id => required(requestData(id), 'Shared Request'));
-    const attemptsFor = (run) => [...runRequests(run.id), ...sharedRequests(run)];
+    const runRequests = (id) => db.prepare('SELECT id FROM requests WHERE run_id=? ORDER BY ordinal').all(id).map(row => required(requestData(String(row.id)), 'Request'));
+    const sharedRequests = (run) => db.prepare("SELECT q.data FROM shared_members m JOIN requests q ON q.id=m.request_id WHERE m.run_id=? AND m.source_run_id!=?").all(run.id, run.id).map(row => parseStored(row.data));
     const submissionDeadlines = new Map();
     function sweepSubmissionDeadlines() {
         for (const id of submissionDeadlines.keys()) {
@@ -203,12 +192,22 @@ export function createBroker({ detail, repoPath, stateDir, repoId = 'demo', coal
         }
     }
     const coalescing = { reconcile: reconcileWithin, deadlines: submissionDeadlines };
-    const shareable = (request) => coalescingEligibility(db, request, coalescing) !== null;
-    const saveRequest = (request) => db.prepare('UPDATE requests SET status = ?, data = ? WHERE id = ?').run(request.status, JSON.stringify(request), request.id);
-    const saveRun = (run) => {
-        db.prepare('UPDATE runs SET status = ?, data = ? WHERE id = ?').run(run.status, JSON.stringify(run), run.id);
-        if (terminal.has(run.status))
-            submissionDeadlines.delete(run.id);
+    const requestHeader = (id) => { const row = db.prepare('SELECT data FROM requests WHERE id=?').get(id); return row ? parseStored(row.data) : null; };
+    const saveHeader = (packed) => {
+        db.prepare('UPDATE requests SET status=?,data=? WHERE id=?').run(packed.status, JSON.stringify(packed), packed.id);
+        readiness.transition(packed);
+    };
+    const saveRequest = (request) => {
+        const packed = required(requestHeader(request.id), 'Request');
+        for (const key of ['status', 'startedAt', 'completedAt', 'claimedBy', 'claimedAt', 'tryClaim', 'preparationAttempt', 'claimAttemptId', 'notifiedAt', 'error', 'errorCode', 'blockedReason']) {
+            if (request[key] === undefined)
+                delete packed[key];
+            else
+                packed[key] = request[key];
+        }
+        packed.resultRef = request.result ? store.put(request.result) : null;
+        packed.semanticRef = request.result ? store.put(semanticResult(request.result)) : null;
+        saveHeader(packed);
     };
     const appendEvent = (runId, requestId, type, message, data = null) => {
         db.prepare('INSERT INTO events(run_id,request_id,created_at,type,message,data) VALUES (?,?,?,?,?,?)').run(runId, requestId, now(), type, message.slice(0, 4000), data === null ? null : JSON.stringify(data));
@@ -231,112 +230,47 @@ export function createBroker({ detail, repoPath, stateDir, repoId = 'demo', coal
                 throw new Error('An in-place review requires its monitoring worker to remain alive.');
         },
     });
-    function refreshReadinessWithin(runId) {
-        sweepSubmissionDeadlines();
-        const run = required(runData(runId), 'Run'), requests = runRequests(runId);
-        if (run.project?.version === 3) {
-            if (terminal.has(run.status))
-                return;
-            for (const shared of sharedRequests(run))
-                reconcileWithin(shared.runId);
-            // Called in a write transaction: expire only unowned queued sources, then
-            // recheck all matching owned/leased requests before creating our own ticket.
-            const retained = sharedRequests(run).filter(request => terminal.has(request.status) || shareable(request));
-            if (retained.length !== (run.project.coalescedRequestIds ?? []).length) {
-                run.project.coalescedRequestIds = retained.map(request => request.id);
-                saveRun(run);
-                appendEvent(runId, null, 'request.uncoalesced', 'Unowned source submission lease expired; replanning this Run.');
-            }
-            brokerTestHooks.onPlan?.();
-            const plan = planProject(run.project.snapshot, readEvidence(db), { selection: run.project.selection, recursive: run.project.recursive, force: run.project.force, runId, coalescedRequestIds: run.project.coalescedRequestIds, attempts: [...requests, ...sharedRequests(run)] });
-            for (const item of plan.items.filter(item => item.action === 'EXECUTE')) {
-                if (!run.project.force) {
-                    const shared = findCoalescibleRequest(db, item.id, item.input.key, coalescing)?.request;
-                    if (shared) {
-                        run.project.coalescedRequestIds = [...new Set([...(run.project.coalescedRequestIds ?? []), shared.id])];
-                        saveRun(run);
-                        appendEvent(runId, shared.id, 'request.coalesced', 'Waiting for an identical active review in another Run.', { sourceRunId: shared.runId });
-                        continue;
-                    }
-                }
-                const envelope = run.project.templates.find(template => template.criticId === item.id);
-                if (!envelope)
-                    throw codedError(`Missing prepared Critic ${item.id}.`, 'REVIEW_GRAPH_INVALID');
-                const request = { ...copy(envelope), validationInput: copy(item.input), id: randomUUID(), runId,
-                    workspace: run.workspace, worktreePath: run.workspace.path, status: 'QUEUED', createdAt: now(),
-                    startedAt: null, completedAt: null, claimedBy: null, claimedAt: null, notifiedAt: null, result: null, error: null, blockedReason: null };
-                db.prepare('INSERT INTO requests(id,run_id,ordinal,status,data) VALUES (?,?,?,?,?)').run(request.id, runId, requests.length, request.status, JSON.stringify(request));
-                requests.push(request);
-                appendEvent(runId, request.id, 'request.queued', 'Pull validation found an executable Critic requiring an actual review.');
-            }
-            return;
-        }
-        throw new Error('Invalid stored Run format; submit a new validation request.');
-    }
-    const updateRunStatus = (runId) => {
-        const run = required(runData(runId), 'Run');
-        const states = attemptsFor(run).map(request => request.status);
-        if (run.project?.version === 3) {
-            if (terminal.has(run.status))
-                return;
-            brokerTestHooks.onPlan?.();
-            const plan = planProject(run.project.snapshot, readEvidence(db), { selection: run.project.selection, recursive: run.project.recursive, force: run.project.force, runId, coalescedRequestIds: run.project.coalescedRequestIds, attempts: attemptsFor(run) });
-            const status = states.includes('RUNNING') ? 'RUNNING' : states.includes('QUEUED') ? 'QUEUED' : states.includes('WAITING_HUMAN') ? 'WAITING_HUMAN' :
-                states.includes('ERROR') ? 'ERROR' : states.includes('RED') || plan.critics.some(critic => critic.status === 'RED') ? 'RED' : plan.satisfied ? 'GREEN' : 'INCOMPLETE';
-            if (run.status !== status) {
-                run.status = status;
-                if (terminal.has(status)) {
-                    run.completedAt = now();
-                    run.project.evidenceRequestIds = [...new Set(plan.critics.flatMap(c => c.result ? [c.result.requestId] : []))];
-                }
-                saveRun(run);
-                appendEvent(runId, null, 'run.status', `Run ${status}`, { status });
-            }
-            return;
-        }
-        throw new Error('Invalid stored Run format.');
-    };
+    const readiness = createReadiness(db, store, coalescing, appendEvent);
+    const refreshReadinessWithin = (id) => readiness.refresh(id);
+    const updateRunStatus = (id) => readiness.updateRun(id);
     function finishWithin(requestId, outcome, expectedStates = ['RUNNING', 'WAITING_HUMAN']) {
         const { result, error } = outcome, hasError = Object.hasOwn(outcome, 'error');
-        const request = requestData(requestId);
-        if (!request || !expectedStates.includes(request.status) || terminal.has(required(runData(request.runId), 'Run').status))
+        const request = requestHeader(requestId);
+        if (!request || !expectedStates.includes(request.status) || terminal.has(required(polling.runStatus(request.runId), 'Run')))
             return false;
         request.status = hasError ? 'ERROR' : required(result, 'review result').verdict;
-        request.result = result ?? null;
+        request.resultRef = result ? store.put(result) : null;
+        request.semanticRef = result ? store.put(semanticResult(result)) : null;
         request.error = hasError ? errorText(error) : null;
         request.errorCode = errorCode(error) ?? null;
         request.completedAt = now();
-        saveRequest(request);
+        saveHeader(request);
         appendEvent(request.runId, request.id, hasError ? 'request.error' : 'request.completed', hasError ? required(request.error, 'error message') : `Review ${required(result, 'review result').verdict}.`, { status: request.status, ...(request.errorCode ? { code: request.errorCode } : {}) });
-        refreshReadinessWithin(request.runId);
-        updateRunStatus(request.runId);
         return true;
     }
     function failWithin(runId, error) {
-        const run = runData(runId);
+        const run = readiness.header(runId);
         if (!run || terminal.has(run.status))
             return;
-        for (const request of runRequests(runId)) {
-            if (terminal.has(request.status))
-                continue;
+        // Cancellation is proportional to affected requests, never definitions.
+        submissionDeadlines.delete(runId);
+        run.status = 'ERROR';
+        run.error = errorText(error);
+        run.completedAt = now();
+        db.prepare('UPDATE runs SET status=?,data=? WHERE id=?').run(run.status, JSON.stringify(run), runId);
+        for (const row of db.prepare("SELECT data FROM requests WHERE run_id=? AND status IN ('QUEUED','RUNNING','WAITING_HUMAN','WAIT_DEPENDENCY','BLOCKED')").all(runId)) {
+            const request = parseStored(row.data);
             request.status = 'ERROR';
-            request.result = null;
+            request.resultRef = null;
+            request.semanticRef = null;
             request.error = errorText(error);
             request.errorCode = errorCode(error) ?? null;
             request.completedAt = now();
             request.blockedReason = null;
-            saveRequest(request);
+            db.prepare('UPDATE requests SET status=?,data=? WHERE id=?').run('ERROR', JSON.stringify(request), request.id);
+            readiness.transition(request);
             appendEvent(runId, request.id, 'request.error', request.error, { status: request.status, ...(request.errorCode ? { code: request.errorCode } : {}) });
         }
-        if (run.project?.version === 3) {
-            brokerTestHooks.onPlan?.();
-            const plan = planProject(run.project.snapshot, readEvidence(db), { selection: run.project.selection, recursive: run.project.recursive, force: run.project.force, runId, coalescedRequestIds: run.project.coalescedRequestIds, attempts: attemptsFor(run) });
-            run.project.evidenceRequestIds = [...new Set(plan.critics.flatMap(c => c.result ? [c.result.requestId] : []))];
-        }
-        run.status = 'ERROR';
-        run.error = errorText(error);
-        run.completedAt = now();
-        saveRun(run);
         appendEvent(runId, null, 'run.error', run.error);
     }
     function reconcileWithin(runId) {
@@ -359,14 +293,17 @@ export function createBroker({ detail, repoPath, stateDir, repoId = 'demo', coal
     }
     function getRun(id) {
         ensureOpen();
-        if (!runData(id))
+        if (!polling.runStatus(id))
             return null;
         if (ownerData(id))
             reconcile(id);
         const run = required(runData(id), 'Run');
         const owner = ownerData(id);
+        if (owner || terminal.has(run.status))
+            submissionDeadlines.delete(id);
         const events = db.prepare('SELECT * FROM (SELECT * FROM events WHERE run_id = ? ORDER BY id DESC LIMIT 500) ORDER BY id').all(id).map(event => ({ id: Number(event.id), runId: String(event.run_id), requestId: event.request_id === null ? null : String(event.request_id), createdAt: String(event.created_at), type: String(event.type), message: String(event.message), ...(event.data ? { data: parseStored(event.data) } : {}) }));
-        return { ...copy(run), scope: run.scope ?? { kind: run.graph ? 'graph' : 'chain' }, owner: owner ? { pid: owner.pid, claimedAt: owner.claimed_at } : null, requests: runRequests(id), events };
+        const view = copy(run);
+        return { ...view, scope: view.scope ?? { kind: view.graph ? 'graph' : 'chain' }, owner: owner ? { pid: owner.pid, claimedAt: owner.claimed_at } : null, requests: runRequests(id), events };
     }
     function failOwned(runId, token, error) {
         transaction(() => { if (ownerData(runId)?.token === token)
@@ -374,120 +311,140 @@ export function createBroker({ detail, repoPath, stateDir, repoId = 'demo', coal
         changed();
     }
     async function executeOne(requestId, workspace, token, signal) {
-        const initial = required(requestData(requestId), 'Request');
+        const initial = required(requestHeader(requestId), 'Request');
         const runId = initial.runId;
-        transaction(() => {
-            const request = required(requestData(requestId), 'Request');
-            if (ownerData(runId)?.token !== token || request.status !== 'QUEUED')
-                throw codedError('Review ownership changed before execution.', 'RUN_OWNERSHIP_LOST');
-            request.status = 'RUNNING';
-            request.startedAt = now();
-            request.blockedReason = null;
-            saveRequest(request);
-            appendEvent(runId, request.id, 'request.started', `${request.title}: reviewing the supplied workspace.`, { snapshotHash: workspace.descriptor.hash });
-            updateRunStatus(runId);
-        });
-        changed();
+        let lease;
         try {
+            if (initial.profile.kind !== 'human')
+                lease = await admission.acquire({ requestId, runId, kind: initial.profile.kind, provider: initial.profile.provider, model: initial.profile.model }, { signal, waiting(reason) {
+                        transaction(() => { const current = required(requestHeader(requestId), 'Request'); if (current.status !== 'QUEUED')
+                            return; current.blockedReason = reason.slice(0, 2000); saveHeader(current); });
+                        changed();
+                    } });
             signal.throwIfAborted();
-            await workspace.assertUnchanged();
-            const request = required(requestData(requestId), 'Request');
-            const runDir = path.join(stateDir, 'runs', runId, request.id);
-            fs.mkdirSync(runDir, { recursive: true, mode: 0o700 });
-            if (required(runData(runId), 'Run').project) {
-                const capability = await requireExecutors().canExecute(copy(request));
-                if (!capability?.ok)
-                    throw codedError(capability?.reason ?? 'No compatible executor.', capability?.code ?? 'EXECUTOR_UNAVAILABLE');
-            }
-            if (request.profile.kind === 'human') {
-                if (typeof requireExecutors().notifyHuman !== 'function')
-                    throw new Error('Human execution requires a registered alarm method.');
-                transaction(() => {
-                    const current = required(requestData(requestId), 'Request');
-                    if (current.status !== 'RUNNING')
-                        throw codedError('Review was canceled before Human notification.', 'REVIEW_CANCELED');
-                    current.status = 'WAITING_HUMAN';
-                    saveRequest(current);
-                    appendEvent(runId, requestId, 'human.waiting', 'Human review is ready; registered alarm delivery is pending.');
-                    updateRunStatus(runId);
-                });
-                await requireExecutors().notifyHuman(copy(required(requestData(requestId), 'Request')), { signal });
+            transaction(() => {
+                const request = required(requestHeader(requestId), 'Request');
+                if (ownerData(runId)?.token !== token || request.status !== 'QUEUED')
+                    throw codedError('Review ownership changed before execution.', 'RUN_OWNERSHIP_LOST');
+                request.status = 'RUNNING';
+                request.startedAt = now();
+                request.blockedReason = null;
+                saveHeader(request);
+                appendEvent(runId, request.id, 'request.started', `${request.title}: reviewing the supplied workspace.`, { snapshotHash: workspace.descriptor.hash });
+                updateRunStatus(runId);
+            });
+            changed();
+            try {
                 signal.throwIfAborted();
                 await workspace.assertUnchanged();
-                transaction(() => {
-                    const current = required(requestData(requestId), 'Request');
-                    if (current.status !== 'WAITING_HUMAN')
-                        return;
-                    current.notifiedAt = now();
-                    saveRequest(current);
-                    appendEvent(runId, requestId, 'human.notified', 'Registered human alarm methods confirmed delivery.');
-                });
-                changed();
-                return;
-            }
-            const result = validateResult(await requireExecutors().execute(copy(request), {
-                worktreePath: workspace.descriptor.path, workspacePath: workspace.descriptor.path, runDir, signal,
-                onEvent(event) {
-                    if (closed || closing || signal.aborted || !event || requestData(requestId)?.status !== 'RUNNING' || !['executor.started', 'artifact.tools.ready', 'artifact.tool.called', 'artifact.tool.completed', 'executor.usage', 'executor.final.invalid', 'executor.final.repair', 'executor.telemetry.failed', 'executor.completed'].includes(event.type))
-                        return;
-                    if (event.type === 'executor.final.invalid' || event.type === 'executor.final.repair') {
-                        const safe = finalResultEventData(event);
-                        if (safe) {
-                            appendEvent(runId, requestId, event.type, event.type, safe);
-                            changed();
-                        }
-                        return;
-                    }
-                    const safe = {};
-                    for (const key of ['name', 'provider', 'model', 'kind', 'artifactId', 'path'])
-                        if (typeof event[key] === 'string')
-                            safe[key] = event[key].slice(0, 1000);
-                    if (event.type === 'artifact.tool.completed') {
-                        Object.assign(safe, toolResponseBytes(event));
-                        if (typeof event.startedAt === 'string' && /^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}\.\d{3}Z$/.test(event.startedAt) && Number.isFinite(Date.parse(event.startedAt)))
-                            safe.startedAt = event.startedAt;
-                        if (typeof event.durationMs === 'number' && Number.isFinite(event.durationMs) && event.durationMs >= 0)
-                            safe.durationMs = event.durationMs;
-                        if (event.outcome === 'success' || event.outcome === 'error')
-                            safe.outcome = event.outcome;
-                        if (typeof event.operation === 'string')
-                            safe.operation = event.operation.slice(0, 1000);
-                    }
-                    if (event.type === 'executor.usage') {
-                        const usage = tokenUsage(event.usage);
-                        if (!usage)
+                const request = required(requestData(requestId), 'Request');
+                const runDir = path.join(stateDir, 'runs', runId, request.id);
+                fs.mkdirSync(runDir, { recursive: true, mode: 0o700 });
+                if (required(readiness.header(runId), 'Run').project) {
+                    const capability = await requireExecutors().canExecute(copy(request));
+                    if (!capability?.ok)
+                        throw codedError(capability?.reason ?? 'No compatible executor.', capability?.code ?? 'EXECUTOR_UNAVAILABLE');
+                }
+                if (request.profile.kind === 'human') {
+                    if (typeof requireExecutors().notifyHuman !== 'function')
+                        throw new Error('Human execution requires a registered alarm method.');
+                    transaction(() => {
+                        const current = required(requestData(requestId), 'Request');
+                        if (current.status !== 'RUNNING')
+                            throw codedError('Review was canceled before Human notification.', 'REVIEW_CANCELED');
+                        current.status = 'WAITING_HUMAN';
+                        saveRequest(current);
+                        appendEvent(runId, requestId, 'human.waiting', 'Human review is ready; registered alarm delivery is pending.');
+                        updateRunStatus(runId);
+                    });
+                    await requireExecutors().notifyHuman(copy(required(requestData(requestId), 'Request')), { signal });
+                    signal.throwIfAborted();
+                    await workspace.assertUnchanged();
+                    transaction(() => {
+                        const current = required(requestData(requestId), 'Request');
+                        if (current.status !== 'WAITING_HUMAN')
                             return;
-                        safe.usage = usage;
-                    }
-                    if (event.isError === true)
-                        safe.isError = true;
-                    if (object(event.observation)) {
-                        // Persist the same bounded metadata as final results, even if the Provider later fails.
-                        const observed = storedObservation(event.observation, event.isError === true);
-                        if (observed)
-                            safe.observation = observed;
-                    }
-                    if (Array.isArray(event.tools))
-                        safe.tools = event.tools.slice(0, 32).map(tool => typeof tool === 'string' ? tool : tool?.name).filter(value => typeof value === 'string');
-                    appendEvent(runId, requestId, event.type, event.type === 'executor.telemetry.failed' ? 'Some optional execution diagnostics could not be recorded.' : String(event.message ?? event.type).slice(0, 2000), safe);
+                        current.notifiedAt = now();
+                        saveRequest(current);
+                        appendEvent(runId, requestId, 'human.notified', 'Registered human alarm methods confirmed delivery.');
+                    });
                     changed();
-                },
-            }));
-            validateFinalResult(semanticResult(result), request);
-            await workspace.assertUnchanged();
-            signal.throwIfAborted();
-            transaction(() => { if (ownerData(runId)?.token === token)
-                finishWithin(requestId, { result }); });
-            changed();
+                    return;
+                }
+                const result = validateResult(await requireExecutors().execute(copy(request), {
+                    worktreePath: workspace.descriptor.path, workspacePath: workspace.descriptor.path, runDir, signal,
+                    onEvent(event) {
+                        if (closed || closing || signal.aborted || !event || polling.requestStatus(requestId) !== 'RUNNING' || !['executor.started', 'artifact.tools.ready', 'artifact.tool.called', 'artifact.tool.completed', 'executor.usage', 'executor.final.invalid', 'executor.final.repair', 'executor.telemetry.failed', 'executor.completed'].includes(event.type))
+                            return;
+                        if (event.type === 'executor.final.invalid' || event.type === 'executor.final.repair') {
+                            const safe = finalResultEventData(event);
+                            if (safe) {
+                                appendEvent(runId, requestId, event.type, event.type, safe);
+                                changed();
+                            }
+                            return;
+                        }
+                        const safe = {};
+                        for (const key of ['name', 'provider', 'model', 'kind', 'artifactId', 'path'])
+                            if (typeof event[key] === 'string')
+                                safe[key] = event[key].slice(0, 1000);
+                        if (event.type === 'artifact.tool.completed') {
+                            Object.assign(safe, toolResponseBytes(event));
+                            if (typeof event.startedAt === 'string' && /^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}\.\d{3}Z$/.test(event.startedAt) && Number.isFinite(Date.parse(event.startedAt)))
+                                safe.startedAt = event.startedAt;
+                            if (typeof event.durationMs === 'number' && Number.isFinite(event.durationMs) && event.durationMs >= 0)
+                                safe.durationMs = event.durationMs;
+                            if (event.outcome === 'success' || event.outcome === 'error')
+                                safe.outcome = event.outcome;
+                            if (typeof event.operation === 'string')
+                                safe.operation = event.operation.slice(0, 1000);
+                        }
+                        if (event.type === 'executor.usage') {
+                            const usage = tokenUsage(event.usage);
+                            if (!usage)
+                                return;
+                            safe.usage = usage;
+                        }
+                        if (event.isError === true)
+                            safe.isError = true;
+                        if (object(event.observation)) {
+                            // Persist the same bounded metadata as final results, even if the Provider later fails.
+                            const observed = storedObservation(event.observation, event.isError === true);
+                            if (observed)
+                                safe.observation = observed;
+                        }
+                        if (Array.isArray(event.tools))
+                            safe.tools = event.tools.slice(0, 32).map(tool => typeof tool === 'string' ? tool : tool?.name).filter(value => typeof value === 'string');
+                        appendEvent(runId, requestId, event.type, event.type === 'executor.telemetry.failed' ? 'Some optional execution diagnostics could not be recorded.' : String(event.message ?? event.type).slice(0, 2000), safe);
+                        changed();
+                    },
+                }));
+                validateFinalResult(semanticResult(result), request);
+                await workspace.assertUnchanged();
+                signal.throwIfAborted();
+                transaction(() => { if (ownerData(runId)?.token === token)
+                    finishWithin(requestId, { result }); });
+                changed();
+            }
+            catch (error) {
+                const reason = signal.aborted ? signal.reason : error;
+                if (signal.aborted || fatalExecutionError(reason))
+                    throw reason;
+                transaction(() => { if (ownerData(runId)?.token === token)
+                    finishWithin(requestId, { error: reason }); });
+                changed();
+            }
         }
         catch (error) {
-            const reason = signal.aborted ? signal.reason : error;
-            if (signal.aborted || fatalExecutionError(reason))
-                throw reason;
+            if (signal.aborted || fatalExecutionError(error))
+                throw error;
             transaction(() => { if (ownerData(runId)?.token === token)
-                finishWithin(requestId, { error: reason }); });
+                finishWithin(requestId, { error }, ['QUEUED']); });
             changed();
         }
+        finally {
+            await lease?.release();
+        }
     }
     async function run(runId, { signal, onStarted } = {}) {
         ensureOpen();
@@ -496,7 +453,8 @@ export function createBroker({ detail, repoPath, stateDir, repoId = 'demo', coal
         let stored = null;
         const acquired = transaction(() => {
             reconcileWithin(runId);
-            stored = runData(runId);
+            const header = readiness.header(runId);
+            stored = header ? { ...header, workspace: store.get(header.workspaceRef) } : null;
             if (!stored)
                 throw new Error('Unknown Run.');
             if (stored.project?.version !== 3)
@@ -507,6 +465,7 @@ export function createBroker({ detail, repoPath, stateDir, repoId = 'demo', coal
                 throw new Error('Invalid Run: no workspace descriptor.');
             if (ownerData(runId))
                 throw codedError('Another process already owns this review Run.', 'RUN_ALREADY_OWNED');
+            submissionDeadlines.delete(runId);
             db.prepare('INSERT INTO run_owners(run_id,pid,process_identity,token,claimed_at) VALUES (?,?,?,?,?)').run(runId, process.pid, ownProcessIdentity, token, now());
             appendEvent(runId, null, 'worker.started', 'A request-scoped worker owns this Run.', { pid: process.pid });
             return true;
@@ -525,10 +484,9 @@ export function createBroker({ detail, repoPath, stateDir, repoId = 'demo', coal
             let plannedRevision = -1, leaseDeadline = Infinity;
             let pendingSources = [];
             const notified = new Set();
-            const kinds = new Map();
             try {
                 onStarted?.({ runId, pid: process.pid });
-                workspace = await workspaceAdapter.reopenWorkspace(ownedRun.workspace, { signal: executionSignal });
+                workspace = await workspaceAdapter.reopenWorkspace(copy(ownedRun.workspace), { signal: executionSignal });
                 const reviewSignal = AbortSignal.any([executionSignal, workspace.signal]);
                 poll = setInterval(() => {
                     if (ownerData(runId)?.token !== token)
@@ -549,16 +507,15 @@ export function createBroker({ detail, repoPath, stateDir, repoId = 'demo', coal
                         transaction(() => {
                             refreshReadinessWithin(runId);
                             updateRunStatus(runId);
-                            const pending = sharedRequests(required(runData(runId), 'Run')).filter(request => !terminal.has(request.status));
+                            const pending = sharedRequests({ id: runId }).filter(request => !terminal.has(request.status));
                             pendingSources = [...new Set(pending.map(request => request.runId))];
                             leaseDeadline = Math.min(Infinity, ...pendingSources.filter(id => !ownerData(id)).map(id => submissionDeadlines.get(id) ?? performance.now()));
                             plannedRevision = polling.revision();
                         });
-                    const requests = polling.requests(runId);
+                    const requests = db.prepare("SELECT id,status,json_extract(data,'$.profile.kind') AS kind FROM requests WHERE run_id=? AND status='QUEUED' ORDER BY ordinal LIMIT ?").all(runId, maxConcurrentExecutors + inFlight.size + 1);
                     let executing = [...inFlight.values()].filter(item => item.kind !== 'human').length;
-                    for (const row of requests.filter(request => request.status === 'QUEUED')) {
-                        const kind = kinds.get(row.id) ?? required(requestData(row.id), 'Request').profile.kind;
-                        kinds.set(row.id, kind);
+                    for (const row of requests.filter(request => request.status === 'QUEUED' && !inFlight.has(request.id))) {
+                        const kind = row.kind;
                         if (kind !== 'human' && executing >= maxConcurrentExecutors)
                             continue;
                         if (kind !== 'human')
@@ -580,7 +537,7 @@ export function createBroker({ detail, repoPath, stateDir, repoId = 'demo', coal
                         await delay(Math.max(1, Math.min(100, leaseDeadline - performance.now())), undefined, { signal: reviewSignal });
                         continue;
                     }
-                    const waiting = requests.filter(request => request.status === 'WAITING_HUMAN');
+                    const waiting = db.prepare("SELECT id FROM requests WHERE run_id=? AND status='WAITING_HUMAN' ORDER BY ordinal LIMIT 1").all(runId);
                     if (waiting.length) {
                         for (const request of waiting) {
                             if (notified.has(request.id))
@@ -628,9 +585,10 @@ export function createBroker({ detail, repoPath, stateDir, repoId = 'demo', coal
         return entry.promise;
     }
     const broker = {
-        async submitProject({ requesterId = 'cli', selection, recursive = false, force = false, signal, identityConcurrency: submissionIdentityConcurrency = identityConcurrency, ...removed }) {
+        async submitProject({ requesterId = 'cli', selection, recursive = false, force = false, ignoreGates = false, signal, identityConcurrency: submissionIdentityConcurrency = identityConcurrency, ...removed }) {
             ensureOpen();
             requireExecutors();
+            sweepSubmissionDeadlines();
             positiveConcurrency(submissionIdentityConcurrency, 'identityConcurrency');
             if ('mode' in removed)
                 throw new Error('Workspace modes are no longer supported; supply an unchanged workspace.');
@@ -647,14 +605,14 @@ export function createBroker({ detail, repoPath, stateDir, repoId = 'demo', coal
                     templates.push(...await prepareReviewRequests({ repoPath: workspace.descriptor.path, repoId, snapshotHash: workspace.descriptor.hash, criticId: id, preparedConfig: config }));
                 const id = randomUUID(), createdAt = now();
                 const record = { id, repoId, coalescingGraceMs, snapshotHash: workspace.descriptor.hash, workspace: workspace.descriptor, requesterId,
-                    scope: { kind: 'project' }, graph: createGraphDefinition(config), project: { version: 3, snapshot, selection, recursive, force, templates }, status: 'QUEUED', createdAt };
+                    scope: { kind: 'project' }, graph: createGraphDefinition(config), project: { version: 3, snapshot, selection, recursive, force, ignoreGates, templates }, status: 'QUEUED', createdAt };
                 await workspace.assertUnchanged();
                 workspace.signal.throwIfAborted();
                 transaction(() => {
-                    db.prepare('INSERT INTO runs(id,created_at,status,data) VALUES (?,?,?,?)').run(id, createdAt, record.status, JSON.stringify(record));
+                    db.prepare('INSERT INTO runs(id,created_at,status,data) VALUES (?,?,?,?)').run(id, createdAt, record.status, JSON.stringify(store.packRun(record)));
                     appendEvent(id, null, 'run.submitted', 'Project validation requested against fixed input.', { selection, recursive, force });
-                    refreshReadinessWithin(id);
-                    updateRunStatus(id);
+                    brokerTestHooks.onPlan?.();
+                    readiness.initialize(record);
                 });
                 changed();
                 return required(getRun(id), 'Run');
@@ -866,7 +824,7 @@ export function createBroker({ detail, repoPath, stateDir, repoId = 'demo', coal
             ensureOpen();
             transaction(() => {
                 reconcileWithin(runId);
-                if (!runData(runId))
+                if (!polling.runStatus(runId))
                     throw new Error('Unknown Run.');
                 if (ownerData(runId))
                     throw codedError('Another process already owns this review Run.', 'RUN_ALREADY_OWNED');
@@ -879,7 +837,7 @@ export function createBroker({ detail, repoPath, stateDir, repoId = 'demo', coal
             ensureOpen();
             const error = codedError('Review canceled by requester.', 'REVIEW_CANCELED');
             transaction(() => {
-                if (!runData(runId))
+                if (!polling.runStatus(runId))
                     throw new Error('Unknown Run.');
                 failWithin(runId, error);
             });
@@ -889,6 +847,46 @@ export function createBroker({ detail, repoPath, stateDir, repoId = 'demo', coal
             changed();
             return getRun(runId);
         },
+        retryRequest(requestId) {
+            ensureOpen();
+            transaction(() => {
+                const request = required(requestHeader(requestId), 'Request');
+                if (request.status !== 'ERROR')
+                    throw new Error('Only an operationally failed request can be retried. Submit changed input as a new Run.');
+                const run = required(readiness.header(request.runId), 'Run');
+                if (ownerData(run.id))
+                    throw codedError('Wait for the current worker to settle before retrying.', 'RUN_ALREADY_OWNED');
+                run.status = 'QUEUED';
+                delete run.completedAt;
+                delete run.error;
+                db.prepare('UPDATE runs SET status=?,data=? WHERE id=?').run(run.status, JSON.stringify(run), run.id);
+                const gate = db.prepare('SELECT unmet,red FROM gate_counts WHERE run_id=? AND critic_id=?').get(request.runId, request.criticId);
+                request.status = Number(gate?.unmet ?? 0) ? Number(gate?.red ?? 0) ? 'BLOCKED' : 'WAIT_DEPENDENCY' : 'QUEUED';
+                request.error = null;
+                request.errorCode = null;
+                request.resultRef = null;
+                request.semanticRef = null;
+                request.startedAt = null;
+                request.completedAt = null;
+                saveHeader(request);
+                appendEvent(run.id, request.id, 'request.retried', 'Retry requested against the same immutable input.');
+            });
+            changed();
+            return viewRequest(requestData(requestId));
+        },
+        changes(runId, options) {
+            ensureOpen();
+            db.exec('BEGIN');
+            try {
+                const page = readChanges(db, runId, stateDir, options);
+                db.exec('COMMIT');
+                return page;
+            }
+            catch (error) {
+                db.exec('ROLLBACK');
+                throw error;
+            }
+        },
         onChange(callback) { listeners.add(callback); return () => listeners.delete(callback); },
         async close() {
             if (closed)
@@ -902,7 +900,7 @@ export function createBroker({ detail, repoPath, stateDir, repoId = 'demo', coal
             for (const entry of owned)
                 entry.abort.abort(codedError('The review worker stopped before completion.', 'WORKER_STOPPED'));
             await Promise.allSettled(owned.map(entry => entry.promise));
-            runDefinition = undefined;
+            clearReadCaches();
             submissionDeadlines.clear();
             listeners.clear();
             db.close();
@@ -915,8 +913,9 @@ export function createBroker({ detail, repoPath, stateDir, repoId = 'demo', coal
         ...broker,
         submitProject: async (...args) => viewRun(await broker.submitProject(...args)),
         run: async (...args) => viewRun(await broker.run(...args)),
-        getRun: (id) => viewRun(broker.getRun(id)),
-        listRuns: () => broker.listRuns().map(value => viewRun(value)),
+        getRun: (id) => { ensureOpen(); if (ownerData(id))
+            reconcile(id); return resultView({ detail }, detail === 'full' ? broker.getRun(id) : null, () => storedRequesterRun(db, id, stateDir)); },
+        listRuns: () => { ensureOpen(); return db.prepare('SELECT id FROM runs ORDER BY created_at DESC,rowid DESC').all().map(row => { const id = String(row.id); return resultView({ detail }, detail === 'full' ? required(broker.getRun(id), 'Run') : null, () => required(storedRequesterRun(db, id, stateDir), 'Run')); }); },
         getRequest: (id) => viewRequest(broker.getRequest(id)),
         claimHuman: async (...args) => viewRequest(await broker.claimHuman(...args)),
         completeHuman: async (...args) => viewRequest(await broker.completeHuman(...args)),