@straiffi/archon 1.8.7 → 1.8.9

package/dist/server/index.js

@@ -12,6 +12,8 @@ import { pruneAllRepositoryGraphScans } from './lib/repositoryGraphRetention.js'
 import { registerRepositoryGraphRoutes } from './lib/repositoryGraphRoutes.js';
 import { deleteChatSession, getChatSession, listChatSessions } from './lib/chats.js';
 import { deleteSpecSession, getCreatedTicketIdsForSnapshot, getSpecSession, getSpecSnapshot, listSpecSessions } from './lib/specs.js';
+import { getSpecSnapshotRepositoryEvidenceFreshness } from './lib/specSnapshotEvidence.js';
+import { projectSpecSnapshotForActionPrompt } from './lib/specRepositoryEvidenceFormatting.js';
 import { createChatAttachment, createSpecAttachment, getAttachmentById, deleteChatAttachment, deleteProjectAttachments, deleteSpecSessionAttachment, getChatAttachmentForSession, getChatAttachmentLinkedMessageId, getSpecAttachmentForSession, getSpecAttachmentLinkedMessageId, MAX_CHAT_ATTACHMENTS_PER_TURN, MAX_CHAT_ATTACHMENT_SIZE_BYTES, MAX_SPEC_ATTACHMENTS_PER_TURN, MAX_SPEC_ATTACHMENT_SIZE_BYTES, readChatAttachmentContent, readSpecAttachmentContent, validateImageAttachment, } from './lib/attachments.js';
 import { createProjectChatSession, resolveChatTool, resolveProjectChatPlanQuestionSet, stopProjectChatSessionResponse, submitProjectChatSessionMessage, updateProjectChatSessionMode, } from './lib/chatOperations.js';
 import { createProjectSpecSession, deleteProjectSpecSession, resolveProjectSpecPlanQuestionSet, stopProjectSpecSessionResponse, submitProjectSpecSessionMessage, updateProjectSpecSessionMode, } from './lib/specOperations.js';
@@ -21,7 +23,7 @@ import { resolveSessionTargetForExecution } from './lib/sessionTargets.js';
 import { createChatMessage, getChatMessage, updateChatMessageContent } from './lib/chatMessages.js';
 import { attachChatTicketProposalsToMessageContent } from './lib/chatTicketProposals.js';
 import { createCorsOriginResolver, parseCorsOrigins } from './lib/cors.js';
-import { abortGitOperation, branchExistsLocally, commitGitChanges, createWorktreeAsync, deleteWorktree, getBundleReviewDiffForCwd, getGitDiffData, getGitDiffFiles, getGitStatus, getGitUnifiedDiff, getLocalProjectBranch, getProjectBranch, getProjectBranchSuggestions, getSelectedGitUnifiedDiff, getUnpushedCommitDiffForCwd, getUnpushedCommitSummaryForCwd, hasKnownBranchRef, listUnpushedCommitsForCwd, pullGitBranch, pushCurrentBranch, rebaseGitBranchOntoBase, resolveExistingWorktreePath, resolveWorktreePath, squashUnpushedCommitsForCwd, summarizeTicketDiffFiles, switchProjectBranch, syncWorktreeFilesAsync, undoLatestUnpushedCommitForCwd, UnpushedCommitDiffSelectionError } from './lib/git.js';
+import { abortGitOperation, branchExistsLocally, commitGitChanges, createWorktreeAsync, deleteWorktree, getBundleReviewDiffForCwd, getGitDiffDataAsync, getGitDiffFiles, getGitStatus, getGitStatusAsync, getGitUnifiedDiff, getGitUnifiedDiffAsync, getLocalProjectBranch, getProjectBranch, getProjectBranchSuggestions, getSelectedGitUnifiedDiff, getUnpushedCommitDiffForCwd, getUnpushedCommitSummaryForCwd, getUnpushedCommitSummaryForCwdAsync, hasKnownBranchRef, listUnpushedCommitsForCwd, pullGitBranch, pushCurrentBranch, rebaseGitBranchOntoBase, resolveExistingWorktreePath, resolveWorktreePath, squashUnpushedCommitsForCwd, summarizeTicketDiffFiles, switchProjectBranch, syncWorktreeFilesAsync, undoLatestUnpushedCommitForCwd, UnpushedCommitDiffSelectionError } from './lib/git.js';
 import config from './lib/config.js';
 import { autoConfigProject, ProjectAutoConfigError } from './lib/projectAutoConfig.js';
 import { analyzeRepoPath, createInitialCommit, ensureInitialCommitForWorktrees, initializeGitRepo, isInitialCommitRequiredError, } from './lib/projectRepos.js';
@@ -88,7 +90,8 @@ import { getActiveRunStatus, getActiveRunStatusForProject, getPreviewStatusForPr
 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 { CheckRepairError, getCheckRepairCandidateState, getCheckRepairDetails, getLatestCheckRepairForBatch, retryCheckRepair, startCheckRepair, stopCheckRepair } from './lib/checkRepair.js';
-import { findActiveCheckBatchForTarget, getCheckAttemptDetails, getCheckBatchDetails, hasActiveProjectCheckBatch, isCheckAttemptCurrent, resolveCheckBatchRerunEligibility } from './lib/checkRuns.js';
+import { getCheckAutoFixDetails, stopCheckAutoFix } from './lib/checkAutoFix.js';
+import { findActiveCheckAutoFixForTarget, findActiveCheckBatchForTarget, getCheckAttemptDetails, getCheckBatchDetails, hasActiveProjectCheckBatch, isCheckAttemptCurrent, resolveCheckBatchRerunEligibility } from './lib/checkRuns.js';
 import { getCheckAttemptDiagnosticResponse, getCheckBatchDiagnosticResponse } from './lib/checkDiagnostics.js';
 import { getCheckDefinitionFingerprint } from './lib/checkDefinitions.js';
 import { getCheckGateEvaluation, hydrateCheckGatePresentApplicability } from './lib/checkGateEvaluations.js';
@@ -113,7 +116,7 @@ import { parseIncomingExternalSource } from './lib/externalSource.js';
 import { materializeTicketsWithDependencies } from './lib/ticketBatch.js';
 import { IMPLEMENTATION_READY_DESCRIPTION_SECTION_HEADINGS } from './lib/ticketDescriptionContract.js';
 import { buildImplementationResourcesAppendix, buildSpecImplementationDraft, buildVisualizationAppendix } from './lib/specDraft.js';
-import { exportSpecToDocs, checkSpecExportCollision } from './lib/specExport.js';
+import { exportSpecToDocs, checkSpecExportCollision, SpecExportCollisionError } from './lib/specExport.js';
 import { createTicketDependency, deleteTicketDependenciesForTicket, deleteTicketDependency, getTicketDependency, isTicketBuildComplete, isTicketRunSetupReady, listTicketDependencies, validateCreateTicketDependency, } from './lib/ticketDependencies.js';
 import { startBuild } from './workers/build.js';
 import { expandPlanningTicketOutline, generatePlanningTicketOutline, generatePlanText, prepareLinkedPlanningResearch, startPlan, stopAllPlanAgents, } from './workers/plan.js';
@@ -425,11 +428,19 @@ const buildExecutionRunDetailResponse = async (project, run) => {
                         retry_eligibility: retryEligibility,
                         action_kind: null,
                     }
-                    : run.kind === 'github_actions'
-                        ? { ...serializeGitHubActionsExecutionDetail(getGitHubActionsExecutionDetails(project.id, run.id)), retry_eligibility: retryEligibility }
-                        : ['automation_commit', 'automation_push', 'automation_pull_request'].includes(run.kind)
-                            ? { kind: run.kind, available: true, automation_stage: getAutomationStageDetails(project.id, run.id), retry_eligibility: retryEligibility }
-                            : { kind: run.kind, available: true, retry_eligibility: retryEligibility };
+                    : run.kind === 'check_auto_fix'
+                        ? {
+                            kind: 'check_auto_fix',
+                            available: true,
+                            check_auto_fix: getCheckAutoFixDetails(project.id, run.id),
+                            retry_eligibility: retryEligibility,
+                            action_kind: null,
+                        }
+                        : run.kind === 'github_actions'
+                            ? { ...serializeGitHubActionsExecutionDetail(getGitHubActionsExecutionDetails(project.id, run.id)), retry_eligibility: retryEligibility }
+                            : ['automation_commit', 'automation_push', 'automation_pull_request'].includes(run.kind)
+                                ? { kind: run.kind, available: true, automation_stage: getAutomationStageDetails(project.id, run.id), retry_eligibility: retryEligibility }
+                                : { kind: run.kind, available: true, retry_eligibility: retryEligibility };
     const artifacts = loadExecutionArtifactSummariesForRuns(project.id, [run.id], MAX_ARTIFACTS_PER_RUN).get(run.id)
         ?? { artifacts: [], artifact_count: 0 };
     return {
@@ -490,7 +501,8 @@ const getBundlePlanBlockerError = () => {
     return 'AI planning is blocked because this bundle worktree is currently being modified.';
 };
 const hasActiveBundleCheckWork = (projectId, bundleId) => Boolean(getActiveBundleRepairBlocker(bundleId)
-    || findActiveCheckBatchForTarget(projectId, 'bundle', bundleId));
+    || findActiveCheckBatchForTarget(projectId, 'bundle', bundleId)
+    || findActiveCheckAutoFixForTarget(projectId, 'bundle', bundleId));
 const deleteTicketRecord = async (ticket) => {
     if (ticket.project_id
         && ticket.worktree_bundle_id
@@ -1446,8 +1458,8 @@ const bundlePullRequestDiscoveryErrorTimestamps = new Map();
 const bundlePullRequestSyncInFlight = new Map();
 const bundlePullRequestSyncErrorTimestamps = new Map();
 const getBundlePullRequestDiscoveryKey = (projectId, bundleId) => `${projectId}:${bundleId}`;
-const startPassiveGitHubWork = (label, callback) => {
-    setImmediate(() => {
+const startPassiveGitHubWork = (response, label, callback) => {
+    const start = () => setImmediate(() => {
         try {
             callback();
         }
@@ -1455,6 +1467,11 @@ const startPassiveGitHubWork = (label, callback) => {
             console.warn(`[integrations] github passive work failed ${label}`, error);
         }
     });
+    if (response.writableFinished) {
+        start();
+        return;
+    }
+    response.once('finish', start);
 };
 const isBundlePullRequestDiscoveryCoolingDown = (timestamps, key, cooldownMs) => {
     const lastAttemptAt = timestamps.get(key);
@@ -2019,7 +2036,7 @@ const buildBundleTickets = (projectId, bundleId, broadcaster) => {
         value: details,
     };
 };
-const getCurrentReviewWorkspaceSignature = (ticket) => {
+const getCurrentReviewWorkspaceSignature = async (ticket) => {
     if (!ticket) {
         return null;
     }
@@ -2027,9 +2044,9 @@ const getCurrentReviewWorkspaceSignature = (ticket) => {
     if ('error' in workspace) {
         return null;
     }
-    return getGitStatus(workspace.cwd, workspace.branch, workspace.project).workspace_signature ?? null;
+    return (await getGitStatusAsync(workspace.cwd, workspace.branch, workspace.project)).workspace_signature ?? null;
 };
-const getCurrentReviewDiffSignature = (ticket) => {
+const getCurrentReviewDiffSignature = async (ticket) => {
     if (!ticket) {
         return null;
     }
@@ -2037,9 +2054,9 @@ const getCurrentReviewDiffSignature = (ticket) => {
     if ('error' in workspace) {
         return null;
     }
-    return computeDiffSignature(getGitUnifiedDiff(workspace.cwd));
+    return computeDiffSignature(await getGitUnifiedDiffAsync(workspace.cwd));
 };
-const getBundleReviewFindings = (projectId, bundleId) => {
+const getBundleReviewFindings = async (projectId, bundleId) => {
     const bundle = getBundle(bundleId, projectId);
     if (!bundle) {
         return null;
@@ -2070,13 +2087,13 @@ const getBundleReviewFindings = (projectId, bundleId) => {
     return buildReviewFindingsResponse({
         context,
         isContextRunning: tickets.some(ticket => ticket.agent_status === 'running'),
-        currentWorkspaceSignature: getCurrentReviewWorkspaceSignature(representativeTicket),
+        currentWorkspaceSignature: await getCurrentReviewWorkspaceSignature(representativeTicket),
         currentDiffSignature: latestReviewRun && latestReviewRun.workspace_signature === null
-            ? getCurrentReviewDiffSignature(representativeTicket)
+            ? await getCurrentReviewDiffSignature(representativeTicket)
             : null,
     });
 };
-const getTicketReviewFindings = (ticketId) => {
+const getTicketReviewFindings = async (ticketId) => {
     const ticket = getTicketRouteContext(ticketId);
     if (!ticket) {
         return null;
@@ -2090,9 +2107,9 @@ const getTicketReviewFindings = (ticketId) => {
     return buildReviewFindingsResponse({
         context,
         isContextRunning: ticket.agent_status === 'running',
-        currentWorkspaceSignature: getCurrentReviewWorkspaceSignature(ticket),
+        currentWorkspaceSignature: await getCurrentReviewWorkspaceSignature(ticket),
         currentDiffSignature: latestReviewRun && latestReviewRun.workspace_signature === null
-            ? getCurrentReviewDiffSignature(ticket)
+            ? await getCurrentReviewDiffSignature(ticket)
             : null,
     });
 };
@@ -2733,13 +2750,14 @@ app.post('/projects/:id/chat-sessions/:sessionId/stop', async (req, res) => {
     const result = await stopProjectChatSessionResponse({
         projectId: req.params.id,
         chatSessionId: req.params.sessionId,
+        broadcaster: io,
     });
     if (!result.ok) {
         return res.status(result.status).json({ error: result.error });
     }
     return res.status(result.status).json(result.value);
 });
-app.get('/projects/:id/chat-sessions/:sessionId/diff', (req, res) => {
+app.get('/projects/:id/chat-sessions/:sessionId/diff', async (req, res) => {
     const project = getProjectById(req.params.id);
     if (!project) {
         return res.status(404).json({ error: 'Not found' });
@@ -2752,14 +2770,14 @@ app.get('/projects/:id/chat-sessions/:sessionId/diff', (req, res) => {
     if (!workspace.target_available || !workspace.cwd) {
         return res.status(409).json({ error: workspace.target_unavailable_reason ?? 'Chat target is unavailable.' });
     }
-    const diffData = getGitDiffData(workspace.cwd);
+    const diffData = await getGitDiffDataAsync(workspace.cwd);
     return res.json({
         stats: summarizeTicketDiffFiles(diffData.files),
         files: diffData.files,
         diff: diffData.diff,
     });
 });
-app.get('/projects/:id/chat-sessions/:sessionId/git-status', (req, res) => {
+app.get('/projects/:id/chat-sessions/:sessionId/git-status', async (req, res) => {
     const project = getProjectById(req.params.id);
     if (!project) {
         return res.status(404).json({ error: 'Not found' });
@@ -2772,7 +2790,7 @@ app.get('/projects/:id/chat-sessions/:sessionId/git-status', (req, res) => {
     if (!workspace.target_available || !workspace.cwd) {
         return res.status(409).json({ error: workspace.target_unavailable_reason ?? 'Chat target is unavailable.' });
     }
-    return res.json(getGitStatus(workspace.cwd, workspace.branch, project));
+    return res.json(await getGitStatusAsync(workspace.cwd, workspace.branch, project));
 });
 app.delete('/projects/:id/chat-sessions/:sessionId', (req, res) => {
     const project = getProjectById(req.params.id);
@@ -3073,7 +3091,7 @@ app.delete('/projects/:id/spec-sessions/:sessionId', (req, res) => {
     });
     return res.json({ ok: true });
 });
-app.get('/projects/:id/spec-sessions/:sessionId/snapshots/:snapshotId', (req, res) => {
+app.get('/projects/:id/spec-sessions/:sessionId/snapshots/:snapshotId', async (req, res) => {
     const project = getProjectById(req.params.id);
     if (!project) {
         return res.status(404).json({ error: 'Not found' });
@@ -3091,6 +3109,7 @@ app.get('/projects/:id/spec-sessions/:sessionId/snapshots/:snapshotId', (req, re
     }
     return res.json({
         ...snapshot,
+        repository_evidence_freshness: await getSpecSnapshotRepositoryEvidenceFreshness(project, snapshot.repository_evidence),
         created_ticket_ids: getCreatedTicketIdsForSnapshot(snapshot.id, project.id),
     });
 });
@@ -3190,7 +3209,7 @@ app.post('/projects/:id/spec-sessions/:sessionId/snapshots/:snapshotId/create-ti
         return res.status(500).json({ error: getErrorMessage(error) || 'Unable to create tickets from the spec snapshot.' });
     }
 });
-app.post('/projects/:id/spec-sessions/:sessionId/snapshots/:snapshotId/implement-in-chat', (req, res) => {
+app.post('/projects/:id/spec-sessions/:sessionId/snapshots/:snapshotId/implement-in-chat', async (req, res) => {
     const project = getProjectById(req.params.id);
     if (!project) {
         return res.status(404).json({ error: 'Not found' });
@@ -3207,14 +3226,20 @@ app.post('/projects/:id/spec-sessions/:sessionId/snapshots/:snapshotId/implement
     if (!targetResult.ok) {
         return res.status(409).json({ error: targetResult.error });
     }
-    const draftText = buildSpecImplementationDraft({ snapshot, specSession });
+    const repositoryEvidenceFreshness = await getSpecSnapshotRepositoryEvidenceFreshness(project, snapshot.repository_evidence, { uncached: true });
+    const promptSnapshot = projectSpecSnapshotForActionPrompt(snapshot, repositoryEvidenceFreshness);
+    const draftText = buildSpecImplementationDraft({
+        snapshot: promptSnapshot,
+        specSession,
+        repositoryEvidenceFreshness,
+    });
     return res.status(200).json({
         draft_text: draftText,
         spec_session_id: specSession.id,
         spec_snapshot_id: snapshot.id,
     });
 });
-app.post('/projects/:id/spec-sessions/:sessionId/snapshots/:snapshotId/export', (req, res) => {
+app.post('/projects/:id/spec-sessions/:sessionId/snapshots/:snapshotId/export', async (req, res) => {
     const project = getProjectById(req.params.id);
     if (!project) {
         return res.status(404).json({ error: 'Not found' });
@@ -3243,10 +3268,14 @@ app.post('/projects/:id/spec-sessions/:sessionId/snapshots/:snapshotId/export',
                 conflicting_path: collision,
             });
         }
+        const repositoryEvidenceFreshness = await getSpecSnapshotRepositoryEvidenceFreshness(project, snapshot.repository_evidence, { uncached: true });
+        const exportSnapshot = projectSpecSnapshotForActionPrompt(snapshot, repositoryEvidenceFreshness);
         const result = exportSpecToDocs({
-            snapshot,
+            snapshot: exportSnapshot,
             specSession,
             repoPath: targetResult.target.cwd,
+            repositoryEvidenceFreshness,
+            overwrite,
         });
         return res.status(201).json({
             path: result.filePath,
@@ -3254,6 +3283,12 @@ app.post('/projects/:id/spec-sessions/:sessionId/snapshots/:snapshotId/export',
         });
     }
     catch (error) {
+        if (error instanceof SpecExportCollisionError) {
+            return res.status(409).json({
+                error: error.message,
+                conflicting_path: error.filePath,
+            });
+        }
         return res.status(500).json({ error: getErrorMessage(error) || 'Unable to export the spec snapshot.' });
     }
 });
@@ -3353,14 +3388,14 @@ app.get('/tickets/:id', async (req, res) => {
     res.json(ticket);
     if (passivePullRequestContext) {
         const { project, bundle } = passivePullRequestContext;
-        startPassiveGitHubWork('ticket-open-pr-sync', () => {
+        startPassiveGitHubWork(res, 'ticket-open-pr-sync', () => {
             startTrackedBundlePullRequestSyncIfNeeded(project, bundle);
             startBundlePullRequestDiscoveryIfNeeded(project, bundle);
         });
     }
     return;
 });
-app.get('/tickets/:id/review-findings', (req, res) => {
+app.get('/tickets/:id/review-findings', async (req, res) => {
     const ticket = getTicketRouteContext(req.params.id);
     if (!ticket) {
         return res.status(404).json({ error: 'Not found' });
@@ -3368,7 +3403,7 @@ app.get('/tickets/:id/review-findings', (req, res) => {
     if (!matchesTicketProjectContext(ticket, req)) {
         return res.status(404).json({ error: 'Not found' });
     }
-    const response = getTicketReviewFindings(ticket.id);
+    const response = await getTicketReviewFindings(ticket.id);
     if (!response) {
         return res.status(404).json({ error: 'Not found' });
     }
@@ -3384,7 +3419,7 @@ app.get('/tickets/:id/agent-runs', (req, res) => {
     }
     return res.json({ runs: listAgentRunsForParent('ticket', ticket.id) });
 });
-app.get('/tickets/:id/diff', (req, res) => {
+app.get('/tickets/:id/diff', async (req, res) => {
     const ticket = getTicketRouteContext(req.params.id);
     if (!ticket) {
         return res.status(404).json({ error: 'Not found' });
@@ -3397,7 +3432,7 @@ app.get('/tickets/:id/diff', (req, res) => {
         const { status, error } = workspace;
         return res.status(status).json({ error });
     }
-    const diffData = getGitDiffData(workspace.cwd);
+    const diffData = await getGitDiffDataAsync(workspace.cwd);
     return res.json({
         stats: summarizeTicketDiffFiles(diffData.files),
         files: diffData.files,
@@ -3418,7 +3453,7 @@ app.get('/tickets/:id/committed-diff', (req, res) => {
     }
     return sendUnpushedCommitDiff(res, workspace, getCommitHashQuery(req));
 });
-app.get('/tickets/:id/git-status', (req, res) => {
+app.get('/tickets/:id/git-status', async (req, res) => {
     const ticket = getTicketRouteContext(req.params.id);
     if (!ticket) {
         return res.status(404).json({ error: 'Not found' });
@@ -3431,9 +3466,9 @@ app.get('/tickets/:id/git-status', (req, res) => {
         return res.status(workspace.status).json({ error: workspace.error });
     }
     const refreshRemote = req.query.refresh_remote === 'true';
-    return res.json(getGitStatus(workspace.cwd, workspace.branch, workspace.project, { refreshRemote }));
+    return res.json(await getGitStatusAsync(workspace.cwd, workspace.branch, workspace.project, { refreshRemote }));
 });
-app.get('/tickets/:id/git-commits', (req, res) => {
+app.get('/tickets/:id/git-commits', async (req, res) => {
     const ticket = getTicketRouteContext(req.params.id);
     if (!ticket) {
         return res.status(404).json({ error: 'Not found' });
@@ -3445,7 +3480,7 @@ app.get('/tickets/:id/git-commits', (req, res) => {
     if ('error' in workspace) {
         return res.status(workspace.status).json({ error: workspace.error });
     }
-    const summary = getUnpushedCommitSummaryForCwd(workspace.cwd, workspace.branch, workspace.project);
+    const summary = await getUnpushedCommitSummaryForCwdAsync(workspace.cwd, workspace.branch, workspace.project);
     return res.json({ commits: summary.commits, total_count: summary.totalCount });
 });
 app.get('/bundles', async (req, res) => {
@@ -3465,7 +3500,7 @@ app.get('/bundles', async (req, res) => {
     timing.mark('serialize_bundles');
     timing.end({ count: serializedBundles.length });
     res.json(serializedBundles);
-    startPassiveGitHubWork('project-bundle-pr-sync', () => {
+    startPassiveGitHubWork(res, 'project-bundle-pr-sync', () => {
         startProjectBundlePullRequestSyncIfNeeded(result.project, bundles);
     });
 });
@@ -3493,16 +3528,16 @@ app.get('/bundles/:id/conversation', (req, res) => {
     }
     const payload = serializeBundleConversationDetails(result.project, bundle);
     res.json(payload);
-    startPassiveGitHubWork('bundle-conversation-pr-discovery', () => {
+    startPassiveGitHubWork(res, 'bundle-conversation-pr-discovery', () => {
         startBundlePullRequestDiscoveryIfNeeded(result.project, bundle);
     });
 });
-app.get('/bundles/:id/review-findings', (req, res) => {
+app.get('/bundles/:id/review-findings', async (req, res) => {
     const result = getRequiredProject(req);
     if ('error' in result) {
         return res.status(400).json({ error: result.error });
     }
-    const response = getBundleReviewFindings(result.project.id, req.params.id);
+    const response = await getBundleReviewFindings(result.project.id, req.params.id);
     if (!response) {
         return res.status(404).json({ error: 'Bundle not found' });
     }
@@ -5933,7 +5968,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, pre_review_check_definition_ids, post_build_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 { name, repo_path, linked_project_ids, run_setup, run_services, check_definitions, required_check_definition_ids, pre_review_check_definition_ids, post_build_check_definition_ids, auto_fix_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' });
     }
@@ -5957,6 +5992,7 @@ app.post('/projects', (req, res) => {
       required_check_definition_ids,
       pre_review_check_definition_ids,
       post_build_check_definition_ids,
+      auto_fix_check_definition_ids,
       test_commands,
       run_ide,
       preview_service_name,
@@ -5978,8 +6014,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 ?? '[]', pre_review_check_definition_ids ?? '[]', post_build_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);
+    VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
+  `).run(id, name, normalizedRepoPath, run_setup, run_services, check_definitions, required_check_definition_ids ?? '[]', pre_review_check_definition_ids ?? '[]', post_build_check_definition_ids ?? '[]', auto_fix_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) {
@@ -6873,7 +6909,7 @@ app.post('/projects/:id/git-abort', async (req, res) => {
         return res.status(500).json({ error: 'Unable to determine project branch' });
     }
 });
-app.get('/projects/:id/diff', (req, res) => {
+app.get('/projects/:id/diff', async (req, res) => {
     const project = getProjectById(req.params.id);
     if (!project) {
         return res.status(404).json({ error: 'Not found' });
@@ -6882,7 +6918,7 @@ app.get('/projects/:id/diff', (req, res) => {
     if ('error' in workspace) {
         return res.status(workspace.status).json({ error: workspace.error });
     }
-    const diffData = getGitDiffData(workspace.cwd);
+    const diffData = await getGitDiffDataAsync(workspace.cwd);
     return res.json({
         stats: summarizeTicketDiffFiles(diffData.files),
         files: diffData.files,
@@ -6900,7 +6936,7 @@ app.get('/projects/:id/committed-diff', (req, res) => {
     }
     return sendUnpushedCommitDiff(res, workspace, getCommitHashQuery(req));
 });
-app.get('/projects/:id/git-status', (req, res) => {
+app.get('/projects/:id/git-status', async (req, res) => {
     const project = getProjectById(req.params.id);
     if (!project) {
         return res.status(404).json({ error: 'Not found' });
@@ -6910,9 +6946,9 @@ app.get('/projects/:id/git-status', (req, res) => {
         return res.status(workspace.status).json({ error: workspace.error });
     }
     const refreshRemote = req.query.refresh_remote === 'true';
-    return res.json(getGitStatus(workspace.cwd, workspace.branch, workspace.project, { refreshRemote }));
+    return res.json(await getGitStatusAsync(workspace.cwd, workspace.branch, workspace.project, { refreshRemote }));
 });
-app.get('/projects/:id/git-commits', (req, res) => {
+app.get('/projects/:id/git-commits', async (req, res) => {
     const project = getProjectById(req.params.id);
     if (!project) {
         return res.status(404).json({ error: 'Not found' });
@@ -6921,7 +6957,7 @@ app.get('/projects/:id/git-commits', (req, res) => {
     if ('error' in workspace) {
         return res.status(workspace.status).json({ error: workspace.error });
     }
-    const summary = getUnpushedCommitSummaryForCwd(workspace.cwd, workspace.branch, workspace.project);
+    const summary = await getUnpushedCommitSummaryForCwdAsync(workspace.cwd, workspace.branch, workspace.project);
     return res.json({ commits: summary.commits, total_count: summary.totalCount });
 });
 app.post('/projects/:id/git-commit', async (req, res) => {
@@ -7186,7 +7222,7 @@ app.patch('/projects/:id', (req, res) => {
             });
         }
     }
-    const { name, repo_path, linked_project_ids, run_setup, run_services, check_definitions, required_check_definition_ids, pre_review_check_definition_ids, post_build_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 { name, repo_path, linked_project_ids, run_setup, run_services, check_definitions, required_check_definition_ids, pre_review_check_definition_ids, post_build_check_definition_ids, auto_fix_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 = ?,
@@ -7197,6 +7233,7 @@ app.patch('/projects/:id', (req, res) => {
       required_check_definition_ids = ?,
       pre_review_check_definition_ids = ?,
       post_build_check_definition_ids = ?,
+      auto_fix_check_definition_ids = ?,
       test_commands = ?,
       run_ide = ?,
       preview_service_name = ?,
@@ -7230,6 +7267,9 @@ app.patch('/projects/:id', (req, res) => {
     const nextPostBuildCheckDefinitionIds = hasOwn(validation.values, 'post_build_check_definition_ids')
         ? post_build_check_definition_ids ?? '[]'
         : project.post_build_check_definition_ids ?? '[]';
+    const nextAutoFixCheckDefinitionIds = hasOwn(validation.values, 'auto_fix_check_definition_ids')
+        ? auto_fix_check_definition_ids ?? '[]'
+        : project.auto_fix_check_definition_ids;
     const updateResult = db.transaction(() => {
         const policyReferences = validateProjectRequiredCheckPolicyReferences({
             projectId: project.id,
@@ -7237,11 +7277,12 @@ app.patch('/projects/:id', (req, res) => {
             requiredCheckDefinitionIdsJson: nextRequiredCheckDefinitionIds,
             preReviewCheckDefinitionIdsJson: nextPreReviewCheckDefinitionIds,
             postBuildCheckDefinitionIdsJson: nextPostBuildCheckDefinitionIds,
+            autoFixCheckDefinitionIdsJson: nextAutoFixCheckDefinitionIds,
         });
         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, nextPostBuildCheckDefinitionIds, 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);
+        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, nextPostBuildCheckDefinitionIds, nextAutoFixCheckDefinitionIds, 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 ?? []);
         }
@@ -7278,6 +7319,13 @@ const assertProjectDeletionEligible = (projectId, claimId) => {
     if (hasActiveProjectCheckBatch(projectId)) {
         throw new ProjectDeletionError(409, 'Cannot delete a project while checks are active');
     }
+    if (db.prepare("SELECT 1 FROM sqlite_master WHERE type = 'table' AND name = 'check_auto_fix_loops'").get() && db.prepare(`
+    SELECT 1 FROM check_auto_fix_loops loop
+    JOIN execution_runs run ON run.id = loop.execution_run_id
+    WHERE run.project_id = ? AND loop.terminal_reason IS NULL LIMIT 1
+  `).get(projectId)) {
+        throw new ProjectDeletionError(409, 'Cannot delete a project while automatic check repair is active');
+    }
     if (!claimId && hasActiveProjectRepositoryGraphScan(projectId)) {
         throw new ProjectDeletionError(409, 'Cannot delete a project while a repository graph scan is active');
     }
@@ -7311,6 +7359,12 @@ const completeClaimedProjectDeletion = async (projectId, claimId) => {
           WHERE evaluation_id IN (SELECT id FROM check_gate_evaluations WHERE project_id = ?)
         `).run(projectId);
                 db.prepare('DELETE FROM check_gate_evaluations WHERE project_id = ?').run(projectId);
+                if (db.prepare("SELECT 1 FROM sqlite_master WHERE type = 'table' AND name = 'check_auto_fix_loops'").get()) {
+                    db.prepare(`
+            DELETE FROM check_auto_fix_loops
+            WHERE execution_run_id IN (SELECT id FROM execution_runs WHERE project_id = ?)
+          `).run(projectId);
+                }
                 if (db.prepare("SELECT 1 FROM sqlite_master WHERE type = 'table' AND name = 'check_repair_details'").get()) {
                     db.prepare(`
             DELETE FROM check_repair_details
@@ -8361,6 +8415,9 @@ app.post('/projects/:id/check-repairs/:repairRunId/stop', async (req, res) => {
         return res.status(404).json({ error: 'Not found' });
     }
     try {
+        if (getCheckAutoFixDetails(req.params.id, req.params.repairRunId)) {
+            return res.json(await stopCheckAutoFix(req.params.id, req.params.repairRunId, io));
+        }
         return res.json(await stopCheckRepair(req.params.id, req.params.repairRunId, io));
     }
     catch (error) {
@@ -8499,6 +8556,7 @@ app.get('/projects/:id/check-batches/:batchRunId/diagnostics', async (req, res)
         ...diagnostics,
         repair: await getCheckRepairCandidateState(project.id, req.params.batchRunId),
         latest_repair: getLatestCheckRepairForBatch(project.id, req.params.batchRunId),
+        auto_fix: getCheckAutoFixDetails(project.id, req.params.batchRunId),
     });
 });
 app.get('/projects/:id/check-attempts/:attemptRunId/diagnostics', (req, res) => {
@@ -8936,6 +8994,7 @@ app.post('/tickets/:id/stop-build', async (req, res) => {
     const result = await stopBuildTicketRun({
         ticketId: req.params.id,
         requestedProjectId: getRequestedProjectId(req),
+        broadcaster: io,
     });
     if (!result.ok) {
         return res.status(result.status).json({ error: result.error });