@straiffi/archon 1.7.8 → 1.7.10

package/dist/server/index.js

@@ -28,8 +28,8 @@ import { emitMobileRealtime, getMobileAccessStatus, regenerateMobilePairingChall
 import { validateDesktopMobileAccessRequest } from './lib/mobileAccessSecurity.js';
 import { refreshChatSessionUsageSnapshot, refreshSpecSessionUsageSnapshot, stopAllBuildAgents, stopAllReviewAgents, stopBuildAgent, stopReviewAgent } from './lib/agent.js';
 import { runLightweightPrompt } from './lib/lightweightPrompt.js';
-import { startBuildChain, stopBuildChain } from './lib/buildChains.js';
-import { activateQueuedBuildSequenceTicket, compensateBuildSequenceAcceptance, enqueueBundledBuild, getDependencyOrderedBundleTicketIds, hasActiveSequenceForBundle, removeTicketFromBuildSequence, stopBuildSequence, } from './lib/buildSequences.js';
+import { advanceBuildChain, startBuildChain, stopBuildChain } from './lib/buildChains.js';
+import { activateQueuedBuildSequenceTicket, compensateBuildSequenceAcceptance, enqueueBundledBuild, getDependencyOrderedBundleTicketIds, getBuildSequenceIdForTicket, hasActiveSequenceForBundle, removeTicketFromBuildSequence, stopBuildSequence, } from './lib/buildSequences.js';
 import { prepareTicketForBuild } from './lib/buildFlow.js';
 import { getBundlePlanMutationBlockerById, shouldBlockPlanForBundleMutationById, } from './lib/bundleActivity.js';
 import { getDiscoveredModels, normalizeModelId, normalizeTicketModelId } from './lib/models.js';
@@ -47,12 +47,15 @@ import { exportProjectMemoryArchive, importProjectMemoryArchive } from './lib/pr
 import { buildReviewFindingsResponse, computeDiffSignature, getLatestReviewRunForContext, listReviewFindings } from './lib/reviewFindings.js';
 import { formatBuildOpenQuestionAnswersPrompt, getLatestBuildOpenQuestionBatchStatus, hasActiveBuildOpenQuestionBatch, listBuildOpenQuestionsForBundle, loadBuildOpenQuestionsByBundleIds, } from './lib/buildOpenQuestions.js';
 import { acceptBuildOpenQuestionResolution } from './lib/buildOpenQuestionOperations.js';
-import { completeAutomationProgression, createAutomationReviewFix, finishAutomationReviewFix, getAutomationProgression, recoverLegacyAutomationProgression, recoverStoppedSideEffectProgression, runAutomationProgression, stopAutomationProgression, waitAutomationProgression } from './lib/automationProgression.js';
+import { completeAutomationProgression, createAutomationReviewFix, ensureAutomationProgression, finishAutomationReviewFix, getAutomationProgression, recoverLegacyAutomationProgression, recoverStoppedSideEffectProgression, runAutomationProgression, stopAutomationProgression, waitAutomationProgression } from './lib/automationProgression.js';
 import { reconcileAutomationStage } from './lib/automationReconciliation.js';
+import { runAutomationReviewGateOrPause } from './lib/automationReviewGate.js';
 import { pausedRunBelongsToProgression, resolveAutomationResumeDecision } from './lib/automationResumeDecision.js';
 import { getAutomationStageDetails, } from './lib/automationStageDetails.js';
 import { buildAutomationFixInstruction, clearAutomationPause, countBlockingFindings, depthAtLeast, getBundleAutomationState, isAutomationDepth, recordAutomationPause, resolveAutomationConfig, upsertBundleAutomationState, } from './lib/ticketAutomation.js';
-import { resolveAutomationCheckGateResume, runAutomationCommitChain } from './lib/ticketAutomationCommit.js';
+import { resolveAutomationCheckGateResume, runAutomationCommitChain, runAutomationCommitOperation } from './lib/ticketAutomationCommit.js';
+import { canMutateBuildTicketCommitCheckpoint, resolveBuildTicketCommitCheckpointLineage } from './lib/buildTicketCommit.js';
+import { advanceSuccessfulBuildTicketAfterCheckpoint, compensateBuildSequenceTicketActivation } from './lib/buildCompletionContinuation.js';
 import { getDiscoveredSkills, normalizeSkillNames } from './lib/skills.js';
 import { clearAllPreviewProxies } from './lib/previewProxy.js';
 import { createExecutionRun, addExecutionRunLink, addExecutionRunLinks, getExecutionRun, hasActiveBuildExecutionRunForTicket, getGitHubActionsExecutionDetails, getTestExecutionDetails, GitHubActionsDispatchConflictError, listExecutionRunLinks, listExecutionRuns, listLifecycleEvents, listProjectLifecycleEvents, toCanonicalLifecycleEvent, transitionExecutionRun, withExecutionRunTransaction, } from './lib/executionRuns.js';
@@ -75,14 +78,15 @@ import { PlanningReferenceError, resolveExternalPlanningDescription } from './li
 import { refreshIfNeeded as refreshAssignedIssues, clearCachedAssignedIssues } from './lib/integrations/assignedIssuesCache.js';
 import { buildCommitMessagePrompt, normalizeGeneratedCommitMessage } from './lib/commitMessage.js';
 import { BUNDLE_PULL_REQUEST_SYNC_TTL_MS, getBundlePullRequestRecord, hasOpenBundlePullRequest, serializeBundlePullRequest, shouldSyncBundlePullRequest, syncTrackedBundlePullRequestIfNeeded, upsertBundlePullRequest, upsertBundlePullRequestFromGitHub, } from './lib/bundlePullRequests.js';
-import { countInboundProjectLinks, getEffectiveProject, getProjectCheckDefinitions, getProjectById, getProjectTestCommands, getProjectTicketStats, hasProjectRunIde, listInterruptedProjectDeletions, listProjects, releaseInterruptedProjectDeletionClaims, replaceProjectLinks, serializeProject, updateProjectActiveTarget, validateProjectPayload, } from './lib/projects.js';
+import { countInboundProjectLinks, getEffectiveProject, getProjectCheckDefinitions, getProjectById, getProjectTestCommands, getProjectTicketStats, hasProjectRunIde, listInterruptedProjectDeletions, listProjects, releaseInterruptedProjectDeletionClaims, replaceProjectLinks, serializeProject, updateProjectActiveTarget, validateProjectRequiredCheckPolicyReferences, validateProjectPayload, } from './lib/projects.js';
 import { getActiveRunStatus, getActiveRunStatusForProject, getPreviewStatusForProject, hasRunConfig, isRunning, openIde, openTicketIdeInWorkspace, resolveProjectRunContextKey, runSetupCommandsInWorkspace, runTicketInWorkspace, stopAllTickets, stopTicket } from './lib/run.js';
 import { clearAllTestSessions, createTestSession, deleteTestSession, discoverTestsForSession, doesTestFileExistForSession, getTestSession, isTestSelectionSupported, startTestSessionRun, stopTestSessionRun } from './lib/testSessions.js';
 import { closeCheckRunnerAdmissions, getCheckBatchAttempts, openCheckRunnerAdmissions, rerunCheckBatchAndSchedule, shutdownCheckRunner, startCheckBatch, stopCheckBatch } from './lib/checkRunner.js';
 import { findActiveCheckBatchForTarget, getCheckAttemptDetails, getCheckBatchDetails, hasActiveProjectCheckBatch, isCheckAttemptCurrent, resolveCheckBatchRerunEligibility } from './lib/checkRuns.js';
-import { getCheckDefinitionFingerprint } from './lib/checkDefinitions.js';
+import { getCheckDefinitionFingerprint, validateRequiredCheckPolicySnapshotSize } from './lib/checkDefinitions.js';
 import { getCheckGateEvaluation } from './lib/checkGateEvaluations.js';
 import { captureCheckWorkspaceIdentity } from './lib/checkWorkspaceIdentity.js';
+import { canonicalizeBundleRequiredCheckPolicyOverride, parseBundleRequiredCheckPolicyOverride, resolveEffectiveRequiredCheckPolicyMembership, } from './lib/requiredCheckPolicy.js';
 import { getBundleBuildBlockerResponse, shouldStartBundleBuildNow, transitionTicketState, } from './lib/ticketWorkflowOperations.js';
 import { submitPlanTicketFollowUp, submitTicketFollowUp, } from './lib/ticketFollowUpOperations.js';
 import { closePendingTicketPlanQuestionSet, listPendingTicketPlanQuestionSetMessageIds, resolveTicketPlanQuestionSet, restoreClosedTicketPlanQuestionSets, } from './lib/planQuestionOperations.js';
@@ -1460,6 +1464,10 @@ const serializeBundleRow = (projectId, bundle, openQuestions = bundle ? loadBuil
     const hasPreparedWorktree = bundle.kind === 'project_root'
         ? true
         : Boolean(project && resolveExistingWorktreePath(bundle.branch, project));
+    const parsedOverride = bundle.kind === 'project_root'
+        ? { value: null }
+        : parseBundleRequiredCheckPolicyOverride(bundle.required_check_policy_override);
+    const effectivePolicy = resolveEffectiveRequiredCheckPolicyMembership(project, bundle);
     return {
         ...bundle,
         has_prepared_worktree: hasPreparedWorktree,
@@ -1467,6 +1475,15 @@ const serializeBundleRow = (projectId, bundle, openQuestions = bundle ? loadBuil
             ? serializeBundlePullRequest(getBundlePullRequestRecord(bundle.id, projectId))
             : null,
         automation_config: resolveAutomationConfig(bundle, project),
+        required_check_policy_override: 'error' in parsedOverride ? null : parsedOverride.value,
+        required_check_policy: 'error' in effectivePolicy
+            ? undefined
+            : {
+                source: effectivePolicy.source,
+                required_check_definition_ids: effectivePolicy.required_check_definition_ids,
+                pre_review_check_definition_ids: effectivePolicy.pre_review_check_definition_ids,
+            },
+        required_check_policy_error: 'error' in parsedOverride ? parsedOverride.error : ('error' in effectivePolicy ? effectivePolicy.error : null),
         open_questions: openQuestions,
         build_open_question_batch_status: getLatestBuildOpenQuestionBatchStatus(projectId, bundle.id),
     };
@@ -3741,11 +3758,14 @@ app.patch('/bundles/:id', (req, res) => {
         return res.status(404).json({ error: 'Bundle not found' });
     }
     const hasNameUpdate = hasOwn(req.body, 'name');
+    const hasRequiredCheckPolicyOverrideUpdate = hasOwn(req.body, 'required_check_policy_override');
     if (isProjectRootBundle(bundle)) {
         return res.status(409).json({
             error: hasNameUpdate
                 ? 'Project root bundles cannot be renamed'
-                : 'Project root bundle automation cannot be overridden',
+                : hasRequiredCheckPolicyOverrideUpdate
+                    ? 'Project root bundles cannot override required-check policy'
+                    : 'Project root bundle automation cannot be overridden',
         });
     }
     let nextName = bundle.name;
@@ -3818,6 +3838,39 @@ app.patch('/bundles/:id', (req, res) => {
             return res.status(400).json({ error: 'automation_pause_on_open_questions must be a boolean' });
         }
     }
+    let requiredCheckPolicyOverride;
+    let requiredCheckPolicyOverrideValue;
+    if (hasRequiredCheckPolicyOverrideUpdate) {
+        const value = req.body.required_check_policy_override;
+        if (value == null) {
+            requiredCheckPolicyOverride = null;
+            requiredCheckPolicyOverrideValue = null;
+        }
+        else {
+            const result = canonicalizeBundleRequiredCheckPolicyOverride(value);
+            if ('error' in result) {
+                return res.status(400).json({ error: result.error });
+            }
+            requiredCheckPolicyOverrideValue = result.value;
+            requiredCheckPolicyOverride = result.value ? JSON.stringify(result.value) : null;
+        }
+    }
+    if (requiredCheckPolicyOverrideValue) {
+        const definitions = getProjectCheckDefinitions(result.project);
+        const definitionIds = new Set(definitions.map(definition => definition.id));
+        const missingId = [
+            ...requiredCheckPolicyOverrideValue.required_check_definition_ids,
+            ...requiredCheckPolicyOverrideValue.pre_review_check_definition_ids,
+        ].find(id => !definitionIds.has(id));
+        if (missingId) {
+            return res.status(400).json({ error: `required_check_policy_override references missing check definition ${missingId}` });
+        }
+        const requiredDefinitions = definitions.filter(definition => requiredCheckPolicyOverrideValue.required_check_definition_ids.includes(definition.id));
+        const policySnapshotSizeError = validateRequiredCheckPolicySnapshotSize(requiredCheckPolicyOverrideValue.required_check_definition_ids, requiredDefinitions);
+        if (policySnapshotSizeError) {
+            return res.status(400).json({ error: policySnapshotSizeError });
+        }
+    }
     let updated = null;
     try {
         updated = db.transaction(() => {
@@ -3835,6 +3888,7 @@ app.patch('/bundles/:id', (req, res) => {
                 automationDepth,
                 automationReviewFixEnabled,
                 automationPauseOnOpenQuestions,
+                requiredCheckPolicyOverride,
             });
         })();
     }
@@ -3938,6 +3992,18 @@ app.post('/bundles/:id/build-open-questions/resolve', (req, res) => {
             io.emit('ticket:updated', ticket);
         }
         if (reviewResult && !reviewResult.ok) {
+            if (reviewResult.code === 'bundle_build_incomplete') {
+                completeAutomationProgression({
+                    projectId: result.project.id,
+                    bundleId: bundle.id,
+                    progressionRunId: accepted.progressionRunId,
+                    broadcaster: io,
+                });
+                for (const ticket of listDetailedBundleTickets(result.project.id, bundle.id)) {
+                    io.emit('ticket:updated', ticket);
+                }
+                return res.json(serializeBundleDetails(result.project, bundle));
+            }
             waitAutomationProgression({
                 projectId: result.project.id,
                 bundleId: bundle.id,
@@ -3951,8 +4017,8 @@ app.post('/bundles/:id/build-open-questions/resolve', (req, res) => {
             return res.status(reviewResult.status).json({ error: reviewResult.error });
         }
         return res.json(serializeBundleDetails(result.project, bundle, {
-            review_owner_ticket_id: reviewResult?.value?.id,
-            target_ticket_id: reviewResult?.value?.id,
+            review_owner_ticket_id: reviewResult?.ok ? reviewResult.value?.id : undefined,
+            target_ticket_id: reviewResult?.ok ? reviewResult.value?.id : undefined,
         }));
     }
     for (const ticket of listDetailedBundleTickets(result.project.id, bundle.id)) {
@@ -4135,6 +4201,183 @@ app.post('/bundles/:id/resume-automation', async (req, res) => {
     if (!revalidateDecision()) {
         return res.status(409).json({ error: 'Bundle automation state changed before Resume could start' });
     }
+    const checkpointLineage = resolveBuildTicketCommitCheckpointLineage({
+        projectId: result.project.id,
+        bundleId: bundle.id,
+        progressionRunId,
+    });
+    if (checkpointLineage) {
+        const owner = getTicket(decision.owner_ticket_id);
+        if (!owner || owner.id !== checkpointLineage.ticket.id || owner.agent_status !== 'done') {
+            return res.status(409).json({ error: 'Checkpoint progression owner is no longer the active Build ticket' });
+        }
+        let retryOfRunId;
+        let commitAlreadyComplete = false;
+        const stageRunId = bundleState.automation_paused_stage_run_id;
+        if (stageRunId) {
+            const stage = getExecutionRun(result.project.id, stageRunId);
+            if (!stage
+                || !['automation_commit', 'check_batch'].includes(stage.kind)
+                || !pausedRunBelongsToProgression(result.project.id, bundle.id, progressionRunId, stageRunId)) {
+                return res.status(409).json({ error: 'The paused checkpoint stage does not belong to the active progression' });
+            }
+            if (stage.kind === 'automation_commit') {
+                let reconciled;
+                try {
+                    reconciled = await reconcileAutomationStage({
+                        projectId: result.project.id,
+                        bundleId: bundle.id,
+                        progressionRunId,
+                        stageRunId,
+                    });
+                }
+                catch (error) {
+                    return res.status(409).json({ error: error instanceof Error ? error.message : String(error) });
+                }
+                if (reconciled.domainStatus === 'ambiguous') {
+                    return res.status(409).json({ error: 'The paused checkpoint commit is ambiguous and cannot be resumed safely.' });
+                }
+                if (reconciled.status === 'completed' && reconciled.domainStatus === 'reconciled_completed') {
+                    commitAlreadyComplete = true;
+                }
+                else if (reconciled.status === 'interrupted' && reconciled.domainStatus === 'reconciled_absent') {
+                    retryOfRunId = stage.id;
+                }
+                else {
+                    return res.status(409).json({ error: 'The paused checkpoint commit has no safely resumable outcome.' });
+                }
+            }
+        }
+        try {
+            if (!revalidateDecision()) {
+                return res.status(409).json({ error: 'Bundle automation state changed before Resume could start' });
+            }
+            runAutomationProgression({
+                projectId: result.project.id,
+                bundleId: bundle.id,
+                progressionRunId,
+                broadcaster: io,
+            });
+            if (commitAlreadyComplete) {
+                completeAutomationProgression({
+                    projectId: result.project.id,
+                    bundleId: bundle.id,
+                    progressionRunId,
+                    broadcaster: io,
+                });
+            }
+            else {
+                await runAutomationCommitOperation({
+                    ticket: owner,
+                    io,
+                    progressionRunId,
+                    retryOfRunId,
+                    mutationLabel: 'checkpoint commit',
+                    canCommitLive: canMutateBuildTicketCommitCheckpoint,
+                    shouldCompleteProgression: () => true,
+                    failureMode: 'throw',
+                    requiredProgressionPurpose: 'build_ticket_checkpoint',
+                });
+            }
+        }
+        catch (error) {
+            return res.status(409).json({ error: error instanceof Error ? error.message : String(error) });
+        }
+        const workflowContext = validateWorkflowContext(checkpointLineage.progression.metadata);
+        const continuation = advanceSuccessfulBuildTicketAfterCheckpoint({
+            ticket: owner,
+            emitUpdatedTickets,
+        });
+        if (continuation.nextBuild) {
+            const nextBuild = continuation.nextBuild;
+            const nextSequenceId = continuation.nextSequenceId;
+            const scheduling = startBuild(nextBuild, io, {
+                initiation: {
+                    initiator: 'automation',
+                    causalParentRunId: checkpointLineage.parent.id,
+                    workflowContext,
+                },
+                sequenceId: nextSequenceId,
+                accept: () => {
+                    const transition = prepareTicketForBuild(nextBuild.id, {
+                        ignoredBlockerTicketIds: continuation.ignoredBlockerTicketIds,
+                    });
+                    if (!transition.ok) {
+                        throw new Error(transition.error);
+                    }
+                    if (!nextSequenceId) {
+                        throw new Error('Build sequence no longer exists');
+                    }
+                    activateQueuedBuildSequenceTicket(nextSequenceId, transition.ticket.id);
+                    return {
+                        ticket: transition.ticket,
+                        sequenceId: nextSequenceId,
+                        compensate: () => compensateBuildSequenceTicketActivation(nextSequenceId, transition.ticket.id),
+                    };
+                },
+            });
+            if (!scheduling.accepted) {
+                const stoppedSequence = stopBuildSequence(nextBuild.id);
+                emitUpdatedTickets(stoppedSequence.affectedTicketIds);
+            }
+        }
+        else if (!continuation.nextTicketId && owner.project_id && owner.worktree_bundle_id) {
+            const resumedOwnerForReview = getTicket(owner.id) ?? owner;
+            const liveProject = getEffectiveProject(resumedOwnerForReview) ?? result.project;
+            const automationConfig = resolveAutomationConfig(resumedOwnerForReview.bundle ?? null, liveProject);
+            const autoReviewEnabled = depthAtLeast(automationConfig.depth, 'auto_review');
+            const latestBundleState = getBundleAutomationState(owner.project_id, owner.worktree_bundle_id);
+            const openQuestions = listBuildOpenQuestionsForBundle(owner.project_id, owner.worktree_bundle_id);
+            const shouldPauseForOpenQuestions = autoReviewEnabled
+                && automationConfig.pauseOnOpenQuestions
+                && openQuestions.length > 0;
+            if (shouldPauseForOpenQuestions) {
+                const reviewProgression = ensureAutomationProgression({
+                    projectId: owner.project_id,
+                    bundleId: owner.worktree_bundle_id,
+                    ticketId: owner.id,
+                    ticketTitle: owner.title,
+                    parentRunId: checkpointLineage.parent.id,
+                    workflowContext,
+                    broadcaster: io,
+                });
+                waitAutomationProgression({
+                    projectId: owner.project_id,
+                    bundleId: owner.worktree_bundle_id,
+                    progressionRunId: reviewProgression.runId,
+                    pauseCode: 'open_questions',
+                    pauseReason: `${openQuestions.length} unresolved Build open question${openQuestions.length === 1 ? '' : 's'}`,
+                    reasonCode: 'precondition_failed',
+                    pausedStageRunId: checkpointLineage.parent.id,
+                    broadcaster: io,
+                });
+            }
+            else if (autoReviewEnabled && (!latestBundleState?.automation_paused_reason || latestBundleState.automation_paused_code === 'open_questions')) {
+                const reviewProgression = ensureAutomationProgression({
+                    projectId: owner.project_id,
+                    bundleId: owner.worktree_bundle_id,
+                    ticketId: owner.id,
+                    ticketTitle: owner.title,
+                    parentRunId: checkpointLineage.parent.id,
+                    workflowContext,
+                    broadcaster: io,
+                });
+                await runAutomationReviewGateOrPause({
+                    projectId: owner.project_id,
+                    bundleId: owner.worktree_bundle_id,
+                    progressionRunId: reviewProgression.runId,
+                    broadcaster: io,
+                    pausedStageRunId: checkpointLineage.parent.id,
+                    initiation: { workflowContext },
+                });
+            }
+        }
+        const resumedOwner = getTicket(owner.id);
+        if (resumedOwner) {
+            io.emit('ticket:updated', resumedOwner);
+        }
+        return res.json(serializeBundleDetails(result.project, bundle));
+    }
     let sideEffectResume = null;
     if (bundleState.automation_paused_stage_run_id) {
         const stageRunId = bundleState.automation_paused_stage_run_id;
@@ -4392,17 +4635,43 @@ app.post('/bundles/:id/resume-automation', async (req, res) => {
             return res.status(409).json({ error: 'The selected Resume outcome no longer matches the bundle state' });
         }
     }
-    else if (owner.state === 'build' && owner.agent_status === 'done' && decision.outcome === 'start_review') {
+    else if (owner.state === 'build' && owner.agent_status === 'done' && (decision.outcome === 'start_review' || decision.outcome === 'rerun_review_checks')) {
         if (!revalidateDecision()) {
             return res.status(409).json({ error: 'Bundle automation state changed before Resume could start' });
         }
-        const reviewResult = transitionBundleToReview(result.project.id, bundle.id, io, {
-            automatic: true,
-            initiation: { causalParentRunId: progressionRunId },
-            accept: acceptResumedReview,
-        });
-        if (!reviewResult.ok) {
-            return res.status(reviewResult.status).json({ error: reviewResult.error });
+        const progression = getAutomationProgression(result.project.id, bundle.id);
+        if (!progression || progression.id !== progressionRunId) {
+            return res.status(409).json({ error: 'Automation progression ownership is invalid' });
+        }
+        const workflowContext = validateWorkflowContext(progression.metadata);
+        let reviewStartFailureStatus = null;
+        let reviewStartFailureError = null;
+        try {
+            runAutomationProgression({
+                projectId: result.project.id,
+                bundleId: bundle.id,
+                progressionRunId,
+                broadcaster: io,
+            });
+            await runAutomationReviewGateOrPause({
+                projectId: result.project.id,
+                bundleId: bundle.id,
+                progressionRunId,
+                broadcaster: io,
+                pausedStageRunId: bundleState.automation_paused_stage_run_id,
+                initiation: { workflowContext },
+                onReviewStartFailure: failure => {
+                    reviewStartFailureStatus = failure.status;
+                    reviewStartFailureError = failure.error;
+                    restoreWaitingProgression();
+                },
+            });
+        }
+        catch (error) {
+            return res.status(409).json({ error: error instanceof Error ? error.message : String(error) });
+        }
+        if (reviewStartFailureStatus !== null && reviewStartFailureError !== null) {
+            return res.status(reviewStartFailureStatus).json({ error: reviewStartFailureError });
         }
     }
     else {
@@ -4462,6 +4731,11 @@ app.post('/bundles/:id/dismiss-automation-pause', async (req, res) => {
     catch (error) {
         return res.status(409).json({ error: error instanceof Error ? error.message : String(error) });
     }
+    const checkpointLineage = resolveBuildTicketCommitCheckpointLineage({
+        projectId: result.project.id,
+        bundleId: bundle.id,
+        progressionRunId,
+    });
     if (bundleState.automation_paused_stage_run_id) {
         const stage = getExecutionRun(result.project.id, bundleState.automation_paused_stage_run_id);
         if (!stage
@@ -4497,6 +4771,10 @@ app.post('/bundles/:id/dismiss-automation-pause', async (req, res) => {
         broadcaster: io,
         stopLocalWorker: worker => (worker.kind === 'review' ? stopReviewAgent(worker.ticket_id) : stopBuildAgent(worker.ticket_id)),
     });
+    if (checkpointLineage) {
+        const stoppedSequence = stopBuildSequence(checkpointLineage.ticket.id);
+        emitUpdatedTickets(stoppedSequence.affectedTicketIds);
+    }
     for (const ticket of listDetailedBundleTickets(result.project.id, bundle.id)) {
         io.emit('ticket:updated', ticket);
     }
@@ -5581,7 +5859,7 @@ app.post('/projects', (req, res) => {
         return res.status(400).json({ error: validation.error });
     }
     const projectCount = Number(db.prepare('SELECT COUNT(*) AS count FROM projects').get()?.count ?? 0);
-    const { name, repo_path, linked_project_ids, run_setup, run_services, check_definitions, required_check_definition_ids, test_commands, run_ide, preview_service_name, preview_path, preview_capability_mode, helper_model, helper_variant, commit_message_rules, auto_park_stale_tickets, memory_enabled, scan_include_globs_json, scan_exclude_globs_json, worktree_sync, spec_aware_code_review_enabled, automation_depth, automation_review_fix_enabled, automation_pause_on_open_questions, automation_review_severity, automation_review_max_iterations, } = validation.values;
+    const { name, repo_path, linked_project_ids, run_setup, run_services, check_definitions, required_check_definition_ids, pre_review_check_definition_ids, test_commands, run_ide, preview_service_name, preview_path, preview_capability_mode, helper_model, helper_variant, commit_message_rules, commit_after_each_ticket, auto_park_stale_tickets, memory_enabled, scan_include_globs_json, scan_exclude_globs_json, worktree_sync, spec_aware_code_review_enabled, automation_depth, automation_review_fix_enabled, automation_pause_on_open_questions, automation_review_severity, automation_review_max_iterations, } = validation.values;
     if (!repo_path) {
         return res.status(400).json({ error: 'repo_path must be a non-empty string' });
     }
@@ -5603,6 +5881,7 @@ app.post('/projects', (req, res) => {
       run_services,
       check_definitions,
       required_check_definition_ids,
+      pre_review_check_definition_ids,
       test_commands,
       run_ide,
       preview_service_name,
@@ -5611,6 +5890,7 @@ app.post('/projects', (req, res) => {
       helper_model,
       helper_variant,
       commit_message_rules,
+      commit_after_each_ticket,
       auto_park_stale_tickets,
       memory_enabled,
       scan_include_globs_json,
@@ -5623,8 +5903,8 @@ app.post('/projects', (req, res) => {
       automation_review_severity,
       automation_review_max_iterations
     )
-    VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
-  `).run(id, name, normalizedRepoPath, run_setup, run_services, check_definitions, required_check_definition_ids ?? '[]', test_commands, run_ide ?? null, preview_service_name ?? null, preview_path ?? null, preview_capability_mode ?? null, helper_model ?? null, helper_variant ?? null, commit_message_rules ?? null, auto_park_stale_tickets ?? 0, memory_enabled ?? 0, scan_include_globs_json ?? null, scan_exclude_globs_json ?? null, worktree_sync ?? null, spec_aware_code_review_enabled ?? 0, automation_depth ?? 'manual', automation_review_fix_enabled ?? 0, automation_pause_on_open_questions ?? 0, automation_review_severity ?? 'medium', automation_review_max_iterations ?? 3);
+    VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
+  `).run(id, name, normalizedRepoPath, run_setup, run_services, check_definitions, required_check_definition_ids ?? '[]', pre_review_check_definition_ids ?? '[]', test_commands, run_ide ?? null, preview_service_name ?? null, preview_path ?? null, preview_capability_mode ?? null, helper_model ?? null, helper_variant ?? null, commit_message_rules ?? null, commit_after_each_ticket ?? 0, auto_park_stale_tickets ?? 0, memory_enabled ?? 0, scan_include_globs_json ?? null, scan_exclude_globs_json ?? null, worktree_sync ?? null, spec_aware_code_review_enabled ?? 0, automation_depth ?? 'manual', automation_review_fix_enabled ?? 0, automation_pause_on_open_questions ?? 0, automation_review_severity ?? 'medium', automation_review_max_iterations ?? 3);
     replaceProjectLinks(id, linked_project_ids ?? []);
     ensureProjectRootBundle(id);
     if (projectCount === 0) {
@@ -6790,8 +7070,8 @@ app.patch('/projects/:id', (req, res) => {
             });
         }
     }
-    const { name, repo_path, linked_project_ids, run_setup, run_services, check_definitions, required_check_definition_ids, test_commands, run_ide, preview_service_name, preview_path, preview_capability_mode, helper_model, helper_variant, commit_message_rules, auto_park_stale_tickets, memory_enabled, scan_include_globs_json, scan_exclude_globs_json, worktree_sync, spec_aware_code_review_enabled, automation_depth, automation_review_fix_enabled, automation_pause_on_open_questions, automation_review_severity, automation_review_max_iterations, } = validation.values;
-    db.prepare(`
+    const { name, repo_path, linked_project_ids, run_setup, run_services, check_definitions, required_check_definition_ids, pre_review_check_definition_ids, test_commands, run_ide, preview_service_name, preview_path, preview_capability_mode, helper_model, helper_variant, commit_message_rules, commit_after_each_ticket, auto_park_stale_tickets, memory_enabled, scan_include_globs_json, scan_exclude_globs_json, worktree_sync, spec_aware_code_review_enabled, automation_depth, automation_review_fix_enabled, automation_pause_on_open_questions, automation_review_severity, automation_review_max_iterations, } = validation.values;
+    const updateProject = db.prepare(`
     UPDATE projects SET
       name = ?,
       repo_path = ?,
@@ -6799,6 +7079,7 @@ app.patch('/projects/:id', (req, res) => {
       run_services = ?,
       check_definitions = ?,
       required_check_definition_ids = ?,
+      pre_review_check_definition_ids = ?,
       test_commands = ?,
       run_ide = ?,
       preview_service_name = ?,
@@ -6807,6 +7088,7 @@ app.patch('/projects/:id', (req, res) => {
       helper_model = ?,
       helper_variant = ?,
       commit_message_rules = ?,
+      commit_after_each_ticket = ?,
       auto_park_stale_tickets = ?,
       memory_enabled = ?,
       scan_include_globs_json = ?,
@@ -6820,9 +7102,32 @@ app.patch('/projects/:id', (req, res) => {
       automation_review_max_iterations = ?,
       updated_at = CURRENT_TIMESTAMP
     WHERE id = ?
-  `).run(name ?? project.name, repo_path ?? project.repo_path, hasOwn(validation.values, 'run_setup') ? run_setup : project.run_setup, hasOwn(validation.values, 'run_services') ? run_services : project.run_services, hasOwn(validation.values, 'check_definitions') ? check_definitions : project.check_definitions, hasOwn(validation.values, 'required_check_definition_ids') ? required_check_definition_ids : project.required_check_definition_ids, hasOwn(validation.values, 'test_commands') ? test_commands : project.test_commands, hasOwn(validation.values, 'run_ide') ? run_ide : project.run_ide, hasOwn(validation.values, 'preview_service_name') ? preview_service_name : project.preview_service_name, hasOwn(validation.values, 'preview_path') ? preview_path : project.preview_path, hasOwn(validation.values, 'preview_capability_mode') ? preview_capability_mode : project.preview_capability_mode, hasOwn(validation.values, 'helper_model') ? helper_model : project.helper_model, hasOwn(validation.values, 'helper_variant') ? helper_variant : project.helper_variant, hasOwn(validation.values, 'commit_message_rules') ? commit_message_rules : project.commit_message_rules, hasOwn(validation.values, 'auto_park_stale_tickets') ? auto_park_stale_tickets : project.auto_park_stale_tickets, hasOwn(validation.values, 'memory_enabled') ? memory_enabled : project.memory_enabled, hasOwn(validation.values, 'scan_include_globs_json') ? scan_include_globs_json : project.scan_include_globs_json, hasOwn(validation.values, 'scan_exclude_globs_json') ? scan_exclude_globs_json : project.scan_exclude_globs_json, hasOwn(validation.values, 'worktree_sync') ? worktree_sync : project.worktree_sync, hasOwn(validation.values, 'spec_aware_code_review_enabled') ? spec_aware_code_review_enabled : project.spec_aware_code_review_enabled, hasOwn(validation.values, 'automation_depth') ? automation_depth : project.automation_depth, hasOwn(validation.values, 'automation_review_fix_enabled') ? automation_review_fix_enabled : project.automation_review_fix_enabled, hasOwn(validation.values, 'automation_pause_on_open_questions') ? automation_pause_on_open_questions : project.automation_pause_on_open_questions, hasOwn(validation.values, 'automation_review_severity') ? automation_review_severity : project.automation_review_severity, hasOwn(validation.values, 'automation_review_max_iterations') ? automation_review_max_iterations : project.automation_review_max_iterations, req.params.id);
-    if (hasOwn(validation.values, 'linked_project_ids')) {
-        replaceProjectLinks(project.id, linked_project_ids ?? []);
+  `);
+    const nextCheckDefinitions = hasOwn(validation.values, 'check_definitions') ? check_definitions ?? null : project.check_definitions;
+    const nextRequiredCheckDefinitionIds = hasOwn(validation.values, 'required_check_definition_ids')
+        ? required_check_definition_ids ?? '[]'
+        : project.required_check_definition_ids;
+    const nextPreReviewCheckDefinitionIds = hasOwn(validation.values, 'pre_review_check_definition_ids')
+        ? pre_review_check_definition_ids ?? '[]'
+        : project.pre_review_check_definition_ids;
+    const updateResult = db.transaction(() => {
+        const policyReferences = validateProjectRequiredCheckPolicyReferences({
+            projectId: project.id,
+            checkDefinitionsJson: nextCheckDefinitions,
+            requiredCheckDefinitionIdsJson: nextRequiredCheckDefinitionIds,
+            preReviewCheckDefinitionIdsJson: nextPreReviewCheckDefinitionIds,
+        });
+        if ('error' in policyReferences) {
+            return policyReferences;
+        }
+        updateProject.run(name ?? project.name, repo_path ?? project.repo_path, hasOwn(validation.values, 'run_setup') ? run_setup : project.run_setup, hasOwn(validation.values, 'run_services') ? run_services : project.run_services, nextCheckDefinitions, nextRequiredCheckDefinitionIds, nextPreReviewCheckDefinitionIds, hasOwn(validation.values, 'test_commands') ? test_commands : project.test_commands, hasOwn(validation.values, 'run_ide') ? run_ide : project.run_ide, hasOwn(validation.values, 'preview_service_name') ? preview_service_name : project.preview_service_name, hasOwn(validation.values, 'preview_path') ? preview_path : project.preview_path, hasOwn(validation.values, 'preview_capability_mode') ? preview_capability_mode : project.preview_capability_mode, hasOwn(validation.values, 'helper_model') ? helper_model : project.helper_model, hasOwn(validation.values, 'helper_variant') ? helper_variant : project.helper_variant, hasOwn(validation.values, 'commit_message_rules') ? commit_message_rules : project.commit_message_rules, hasOwn(validation.values, 'commit_after_each_ticket') ? commit_after_each_ticket : project.commit_after_each_ticket, hasOwn(validation.values, 'auto_park_stale_tickets') ? auto_park_stale_tickets : project.auto_park_stale_tickets, hasOwn(validation.values, 'memory_enabled') ? memory_enabled : project.memory_enabled, hasOwn(validation.values, 'scan_include_globs_json') ? scan_include_globs_json : project.scan_include_globs_json, hasOwn(validation.values, 'scan_exclude_globs_json') ? scan_exclude_globs_json : project.scan_exclude_globs_json, hasOwn(validation.values, 'worktree_sync') ? worktree_sync : project.worktree_sync, hasOwn(validation.values, 'spec_aware_code_review_enabled') ? spec_aware_code_review_enabled : project.spec_aware_code_review_enabled, hasOwn(validation.values, 'automation_depth') ? automation_depth : project.automation_depth, hasOwn(validation.values, 'automation_review_fix_enabled') ? automation_review_fix_enabled : project.automation_review_fix_enabled, hasOwn(validation.values, 'automation_pause_on_open_questions') ? automation_pause_on_open_questions : project.automation_pause_on_open_questions, hasOwn(validation.values, 'automation_review_severity') ? automation_review_severity : project.automation_review_severity, hasOwn(validation.values, 'automation_review_max_iterations') ? automation_review_max_iterations : project.automation_review_max_iterations, req.params.id);
+        if (hasOwn(validation.values, 'linked_project_ids')) {
+            replaceProjectLinks(project.id, linked_project_ids ?? []);
+        }
+        return { ok: true };
+    })();
+    if ('error' in updateResult) {
+        return res.status(400).json({ error: updateResult.error });
     }
     const updated = db.prepare('SELECT * FROM projects WHERE id = ?').get(req.params.id);
     res.json(serializeProject(updated ?? null));