@straiffi/archon 1.3.11 → 1.4.0

package/dist/server/index.js

@@ -40,17 +40,20 @@ import { getProjectContextResponse, startProjectContextScan } from './lib/projec
 import { getProjectFileSuggestions } from './lib/projectFileSuggestions.js';
 import { updateProjectTargetSelection } from './lib/projectTargets.js';
 import { exportProjectMemoryArchive, importProjectMemoryArchive } from './lib/projectMemoryTransfer.js';
-import { buildReviewFindingsResponse, computeDiffSignature, getLatestReviewRunForContext } from './lib/reviewFindings.js';
+import { buildReviewFindingsResponse, computeDiffSignature, getLatestReviewRunForContext, listReviewFindings } from './lib/reviewFindings.js';
+import { clearAutomationPause, countBlockingFindings, depthAtLeast, getBundleAutomationState, resolveAutomationConfig, } from './lib/ticketAutomation.js';
+import { runAutomationCommitChain } from './lib/ticketAutomationCommit.js';
 import { getDiscoveredSkills, normalizeSkillNames } from './lib/skills.js';
 import { clearAllPreviewProxies } from './lib/previewProxy.js';
 import { shutdownRealtimeServer } from './lib/shutdown.js';
 import { resolveClientBuildPaths, shouldServeClientApp, STATIC_CLIENT_DIRECTORY_OPTIONS, STATIC_CLIENT_INDEX_OPTIONS, } from './lib/staticClient.js';
 import { deleteIntegrationConnection, getGitHubConnectionConfig, getJiraConnectionConfig, listIntegrationConnections, upsertIntegrationConnection, } from './lib/integrations/index.js';
-import { createGitHubPullRequest, findOpenGitHubPullRequest, findOpenGitHubPullRequestByHeadBranch, getGitHubPullRequestComments, getGitHubPullRequestReviews, getGitHubPullRequest, GitHubApiError, hasGitHubRemoteBranch, resolveGitHubRepository, validateGitHubConnection, } from './lib/integrations/github.js';
+import { createGitHubPullRequest, createOrFindGitHubPullRequest, findOpenGitHubPullRequest, findOpenGitHubPullRequestByHeadBranch, getGitHubPullRequestComments, getGitHubPullRequestReviews, getGitHubPullRequest, GitHubApiError, hasGitHubRemoteBranch, resolveGitHubRepository, validateGitHubConnection, } from './lib/integrations/github.js';
 import { importJiraIssue, JiraApiError, validateJiraConnection } from './lib/integrations/jira.js';
 import { importGitHubIssue } from './lib/integrations/github-issues.js';
 import { PlanningReferenceError, resolveExternalPlanningDescription } from './lib/integrations/planning.js';
-import { getBundlePullRequestRecord, hasOpenBundlePullRequest, serializeBundlePullRequest, shouldSyncBundlePullRequest, upsertBundlePullRequest, } from './lib/bundlePullRequests.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, getProjectById, getProjectTicketStats, hasProjectRunIde, listProjects, replaceProjectLinks, serializeProject, updateProjectActiveTarget, 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, findProjectTestCommand, getTestSession, isTestSelectionSupported, startTestSessionRun, stopTestSessionRun } from './lib/testSessions.js';
@@ -58,6 +61,7 @@ import { getBundleBuildBlockerResponse, shouldStartBundleBuildNow, transitionTic
 import { submitPlanTicketFollowUp, submitTicketFollowUp, } from './lib/ticketFollowUpOperations.js';
 import { canUseBuildControls, formatActiveRunConflict, getTicketRunReadinessError, resumeBuildTicketRun, runProjectTarget, runTicketRun, stopBuildTicketRun, stopProjectTargetRun, stopTicketRun, } from './lib/ticketRunOperations.js';
 import { isTicketAgentSettingsPayload, updateTicketAgentSettings, } from './lib/ticketSettingsOperations.js';
+import { resolveBundleReviewOwner, transitionBundleToReview } from './lib/bundleReviewOperations.js';
 import { getActiveBundleStageChangeHead, undoTicketStage } from './lib/ticketUndo.js';
 import { closeAllTerminalSessions, createProjectTerminalSession, createTerminalSessionForWorkspace, destroyTerminalSessionById, registerTerminalSocketHandlers, } from './lib/terminal.js';
 import { countBundleTickets, createTicketRecord, createBundle, deleteBundle, ensureProjectRootBundle, autoParkTicketIfStale, autoParkSettledBundleTicket, isBundleSettledForAutoPark, clearAutoParkDismissalIfNeeded, assignTicketDoneOrder, assignTicketLaneOrder, getBundle, getBundleByName, getBundleByBranch, getBundleRepresentativeTicketContext, isProjectRootBundle, getTicket, isBundledTicket, listTicketsInRunContext, listBundles, listBoardTicketEnrichment, listTickets, resolveTicketBranch, resolveTicketTool, } from './lib/tickets.js';
@@ -279,7 +283,6 @@ const resolveRequestedSkills = (value) => normalizeSkillNames(value);
 const asTrimmedString = (value) => {
     return typeof value === 'string' ? value.trim() : '';
 };
-const COMMIT_MESSAGE_DIFF_CHAR_LIMIT = 12_000;
 const chatAttachmentUpload = multer({
     storage: multer.memoryStorage(),
     limits: {
@@ -316,36 +319,6 @@ const getSpecAttachmentUploadErrorMessage = (error) => {
     }
     return 'Unable to upload the selected images right now.';
 };
-const normalizeGeneratedCommitMessage = (value) => {
-    const firstLine = value
-        .replace(/\r/g, '\n')
-        .split('\n')
-        .map(line => line.trim())
-        .find(line => line.length > 0) ?? '';
-    return firstLine
-        .replace(/^[-*]\s+/, '')
-        .replace(/^[`"']+/, '')
-        .replace(/[`"']+$/, '')
-        .trim();
-};
-const buildCommitMessagePrompt = ({ commitMessageRules, selectedFiles, diff, }) => {
-    const diffExcerpt = diff.length > COMMIT_MESSAGE_DIFF_CHAR_LIMIT
-        ? `${diff.slice(0, COMMIT_MESSAGE_DIFF_CHAR_LIMIT)}\n\n[diff truncated for speed]`
-        : diff;
-    return [
-        'You generate git commit subjects quickly from a selected diff.',
-        'Prefer speed and a strong best-effort summary over perfect accuracy.',
-        'Return only the commit subject as a single line. No quotes. No markdown. No explanation.',
-        'Keep it concise and under 72 characters when practical.',
-        commitMessageRules
-            ? `Commit message rules:\n${commitMessageRules}`
-            : 'Commit message rules:\nUse a clear default git subject line.',
-        `Selected files:\n${selectedFiles.map(file => `- ${file}`).join('\n')}`,
-        diffExcerpt.trim().length > 0
-            ? `Selected diff:\n${diffExcerpt}`
-            : 'Selected diff:\n[no diff excerpt available, infer from file names only]',
-    ].join('\n\n');
-};
 const normalizeGeneratedPullRequestDescription = (value) => {
     const normalized = value
         .replace(/\r\n/g, '\n')
@@ -895,11 +868,6 @@ const listBundleConversationTickets = (projectId, bundleId) => {
         }))
         : [];
 };
-const moveBundleTicketToReview = (ticket) => {
-    const laneOrder = assignTicketLaneOrder({ ...ticket, state: 'review' });
-    db.prepare('UPDATE tickets SET state = ?, tool = ?, agent_status = NULL, streaming_response = NULL, lane_order = ?, updated_at = CURRENT_TIMESTAMP WHERE id = ?').run('review', resolveTicketTool(ticket, config.tool ?? 'opencode'), laneOrder ?? null, ticket.id);
-    return getTicket(ticket.id);
-};
 const restoreBundleReviewTicketToBuild = (ticket) => {
     const laneOrder = assignTicketLaneOrder({ ...ticket, state: 'build' });
     db.prepare('UPDATE tickets SET state = ?, agent_status = ?, streaming_response = NULL, lane_order = ?, updated_at = CURRENT_TIMESTAMP WHERE id = ?').run('build', 'done', laneOrder ?? null, ticket.id);
@@ -979,7 +947,6 @@ const getBundleReviewState = (projectId, tickets) => {
         can_re_review: reviewState.can_re_review,
     };
 };
-const BUNDLE_PULL_REQUEST_SYNC_TTL_MS = 60 * 1000;
 const BUNDLE_PULL_REQUEST_DISCOVERY_MISS_COOLDOWN_MS = 10 * 60 * 1000;
 const BUNDLE_PULL_REQUEST_DISCOVERY_ERROR_COOLDOWN_MS = 2 * 60 * 1000;
 const BUNDLE_PULL_REQUEST_SYNC_ERROR_COOLDOWN_MS = 2 * 60 * 1000;
@@ -1086,33 +1053,6 @@ const serializeGitHubPullRequestSummary = (pullRequest) => {
         last_synced_at: null,
     };
 };
-const createOrFindGitHubPullRequest = async (config, input) => {
-    const existingPullRequest = await findOpenGitHubPullRequest(config, {
-        headBranch: input.headBranch,
-        baseBranch: input.baseBranch,
-    });
-    return existingPullRequest ?? await createGitHubPullRequest(config, input);
-};
-const upsertBundlePullRequestFromGitHub = (projectId, bundleId, config, pullRequest) => {
-    return upsertBundlePullRequest({
-        bundleId,
-        projectId,
-        repoHost: config.host,
-        repoOwner: config.repo_owner,
-        repoName: config.repo_name,
-        number: pullRequest.number,
-        url: pullRequest.url,
-        title: pullRequest.title,
-        baseBranch: pullRequest.baseBranch,
-        headBranch: pullRequest.headBranch,
-        state: pullRequest.state,
-        isDraft: pullRequest.isDraft,
-        mergedAt: pullRequest.mergedAt,
-        closedAt: pullRequest.closedAt,
-        githubNodeId: pullRequest.nodeId,
-        lastSyncedAt: new Date().toISOString(),
-    });
-};
 const discoverOpenBundlePullRequest = async (project, bundle) => {
     const githubConfig = getGitHubConnectionConfig(project.id);
     const existingRecord = getBundlePullRequestRecord(bundle.id, project.id);
@@ -1129,30 +1069,6 @@ const discoverOpenBundlePullRequest = async (project, bundle) => {
     console.info(`[integrations] github pr discovered project=${project.id} bundle=${bundle.id} pr=${discoveredPullRequest.number}`);
     return upsertBundlePullRequestFromGitHub(project.id, bundle.id, githubConfig, discoveredPullRequest);
 };
-const syncTrackedBundlePullRequestIfNeeded = async (project, bundle, options = {}) => {
-    const force = options.force ?? false;
-    const rethrow = options.rethrow ?? false;
-    const githubConfig = getGitHubConnectionConfig(project.id);
-    const existingRecord = getBundlePullRequestRecord(bundle.id, project.id);
-    if (!githubConfig || !existingRecord) {
-        return existingRecord;
-    }
-    if (!force && !shouldSyncBundlePullRequest(existingRecord, BUNDLE_PULL_REQUEST_SYNC_TTL_MS)) {
-        return existingRecord;
-    }
-    console.info(`[integrations] github pr sync project=${project.id} bundle=${bundle.id} pr=${existingRecord.number}`);
-    try {
-        const nextPullRequest = await getGitHubPullRequest(githubConfig, existingRecord.number);
-        return upsertBundlePullRequestFromGitHub(project.id, bundle.id, githubConfig, nextPullRequest);
-    }
-    catch (error) {
-        console.warn(`[integrations] github pr sync failed project=${project.id} bundle=${bundle.id} pr=${existingRecord.number}`, error);
-        if (rethrow) {
-            throw error;
-        }
-        return existingRecord;
-    }
-};
 const shouldStartTrackedBundlePullRequestSync = (project, bundle, options = {}) => {
     const force = options.force ?? false;
     const existingRecord = getBundlePullRequestRecord(bundle.id, project.id);
@@ -1390,35 +1306,6 @@ const submitBundleFollowUpResponse = (projectId, bundleId, payload, broadcaster)
         }),
     };
 };
-const resolveBundleReviewOwner = (projectId, bundleId) => {
-    const tickets = listDetailedBundleTickets(projectId, bundleId);
-    const candidate = tickets.find(ticket => {
-        if (ticket.state !== 'review') {
-            return false;
-        }
-        const latestReviewHead = getActiveBundleStageChangeHead(ticket, 'review');
-        return latestReviewHead?.ticket_id === ticket.id;
-    });
-    if (candidate) {
-        return candidate;
-    }
-    return tickets.find(ticket => ticket.state === 'review' && ticket.undo?.stage === 'review') ?? null;
-};
-const getBundleReviewEligibilityError = (projectId, bundleId) => {
-    const buildTickets = listBundleSliceTickets(projectId, bundleId, 'build');
-    if (buildTickets.length === 0) {
-        const reviewTickets = listBundleSliceTickets(projectId, bundleId, 'review');
-        if (reviewTickets.some(ticket => ticket.can_re_review)) {
-            return null;
-        }
-        return 'No review tickets are available to re-run in this bundle';
-    }
-    const blockingTicket = buildTickets.find(ticket => ticket.agent_status !== 'done' && ticket.agent_status !== 'stopped');
-    if (blockingTicket) {
-        return 'All build tickets in the bundle must be done or stopped before starting review';
-    }
-    return null;
-};
 const rerunTicketReview = (ticketId) => {
     const ticket = getTicket(ticketId);
     if (!ticket) {
@@ -1445,53 +1332,18 @@ const reviewBundleTickets = (projectId, bundleId) => {
     if (!bundle) {
         return { ok: false, status: 404, error: 'Bundle not found' };
     }
-    const eligibilityError = getBundleReviewEligibilityError(projectId, bundle.id);
-    if (eligibilityError) {
-        return { ok: false, status: 409, error: eligibilityError };
-    }
-    const buildTickets = listBundleSliceTickets(projectId, bundle.id, 'build');
-    if (buildTickets.length === 0) {
-        const reviewOwnerSource = resolveBundleReviewOwner(projectId, bundle.id)
-            ?? listBundleSliceTickets(projectId, bundle.id, 'review').find(ticket => ticket.can_re_review)
-            ?? null;
-        if (!reviewOwnerSource || !reviewOwnerSource.can_re_review) {
-            return { ok: false, status: 409, error: 'No review tickets are available to re-run in this bundle' };
-        }
-        const reviewOwner = getTicket(reviewOwnerSource.id);
-        if (!reviewOwner) {
-            return { ok: false, status: 409, error: 'Unable to start review for this bundle' };
-        }
-        startReview(reviewOwner, io);
-        return {
-            ok: true,
-            bundle,
-            reviewOwner,
-            details: serializeBundleDetails(project, bundle, {
-                review_owner_ticket_id: reviewOwner.id,
-                target_ticket_id: reviewOwner.id,
-            }),
-        };
-    }
-    const reviewOwnerSource = buildTickets[0];
-    if (!reviewOwnerSource) {
-        return { ok: false, status: 409, error: 'Unable to start review for this bundle' };
-    }
-    const movedTickets = buildTickets
-        .map(moveBundleTicketToReview)
-        .filter((ticket) => ticket !== null);
-    emitUpdatedTickets(movedTickets.map(ticket => ticket.id));
-    const reviewOwner = getTicket(reviewOwnerSource.id);
-    if (!reviewOwner) {
-        return { ok: false, status: 409, error: 'Unable to start review for this bundle' };
+    const result = transitionBundleToReview(projectId, bundle.id, io);
+    if (!result.ok) {
+        return { ok: false, status: result.status, error: result.error };
     }
-    startReview(reviewOwner, io);
+    const reviewOwner = result.value;
     return {
         ok: true,
         bundle,
         reviewOwner,
         details: serializeBundleDetails(project, bundle, {
-            review_owner_ticket_id: reviewOwner.id,
-            target_ticket_id: reviewOwner.id,
+            review_owner_ticket_id: reviewOwner?.id,
+            target_ticket_id: reviewOwner?.id,
         }),
     };
 };
@@ -3432,6 +3284,94 @@ app.post('/bundles/:id/follow-up', (req, res) => {
     }
     return res.status(response.status).json(response.value);
 });
+app.post('/bundles/:id/resume-automation', async (req, res) => {
+    const result = getRequiredProject(req);
+    if ('error' in result) {
+        return res.status(400).json({ error: result.error });
+    }
+    const bundle = getBundle(req.params.id, result.project.id);
+    if (!bundle) {
+        return res.status(404).json({ error: 'Bundle not found' });
+    }
+    const bundleState = getBundleAutomationState(result.project.id, bundle.id);
+    if (!bundleState || !bundleState.automation_paused_reason) {
+        return res.status(409).json({ error: 'Bundle automation is not paused' });
+    }
+    clearAutomationPause(result.project.id, bundle.id);
+    const reviewOwnerSource = resolveBundleReviewOwner(result.project.id, bundle.id)
+        ?? listBundleSliceTickets(result.project.id, bundle.id, 'review')[0]
+        ?? null;
+    if (!reviewOwnerSource) {
+        return res.status(409).json({ error: 'No review ticket to resume from' });
+    }
+    const owner = getTicket(reviewOwnerSource.id);
+    if (!owner) {
+        return res.status(409).json({ error: 'No review ticket to resume from' });
+    }
+    const project = getEffectiveProject(owner);
+    if (!project) {
+        return res.status(400).json({ error: 'Ticket project no longer exists' });
+    }
+    const automationConfig = resolveAutomationConfig(owner, project);
+    if (owner.state === 'review' && owner.agent_status === 'running') {
+        return res.status(409).json({ error: 'Review already running' });
+    }
+    if (owner.state === 'review' && owner.agent_status !== 'running') {
+        const latestRun = getLatestReviewRunForContext({
+            ticketId: owner.id,
+            projectId: owner.project_id,
+            worktreeBundleId: owner.worktree_bundle_id,
+        });
+        const findings = latestRun ? listReviewFindings(latestRun.id) : [];
+        const blockingCount = countBlockingFindings(findings, automationConfig);
+        const iteration = bundleState.review_iteration_count ?? 0;
+        if (owner.can_re_review
+            && automationConfig.reviewFixEnabled
+            && blockingCount > 0
+            && iteration < automationConfig.reviewMaxIterations) {
+            startFollowUp(owner, io, `Fix the code review findings at or above "${automationConfig.reviewSeverity}" severity. Address each blocking finding directly and minimally. Do not introduce unrelated changes, reformatting, or refactors. Do not commit.`, 'build');
+        }
+        else if (depthAtLeast(automationConfig.depth, 'auto_commit')) {
+            await runAutomationCommitChain(owner, project, owner.bundle ?? null, automationConfig, io);
+        }
+        else {
+            startReview(owner, io);
+        }
+    }
+    else if (owner.state === 'build' && owner.agent_status === 'done') {
+        const reviewResult = reviewBundleTickets(result.project.id, bundle.id);
+        if (!reviewResult.ok) {
+            return res.status(reviewResult.status).json({ error: reviewResult.error });
+        }
+    }
+    else {
+        return res.status(409).json({ error: 'Ticket is not in a resumable state' });
+    }
+    const resumedOwner = getTicket(owner.id);
+    if (resumedOwner) {
+        io.emit('ticket:updated', resumedOwner);
+    }
+    return res.json(serializeBundleDetails(result.project, bundle));
+});
+app.post('/bundles/:id/dismiss-automation-pause', (req, res) => {
+    const result = getRequiredProject(req);
+    if ('error' in result) {
+        return res.status(400).json({ error: result.error });
+    }
+    const bundle = getBundle(req.params.id, result.project.id);
+    if (!bundle) {
+        return res.status(404).json({ error: 'Bundle not found' });
+    }
+    const bundleState = getBundleAutomationState(result.project.id, bundle.id);
+    if (!bundleState || !bundleState.automation_paused_reason) {
+        return res.status(409).json({ error: 'Bundle automation is not paused' });
+    }
+    clearAutomationPause(result.project.id, bundle.id);
+    for (const ticket of listDetailedBundleTickets(result.project.id, bundle.id)) {
+        io.emit('ticket:updated', ticket);
+    }
+    return res.json(serializeBundleDetails(result.project, bundle));
+});
 app.get('/tool', (_req, res) => {
     res.json({ tool: config.tool ?? 'opencode' });
 });
@@ -3920,7 +3860,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, 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, worktree_sync, } = validation.values;
+    const { name, repo_path, linked_project_ids, run_setup, run_services, 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, worktree_sync, automation_depth, automation_review_fix_enabled, 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' });
     }
@@ -3950,10 +3890,14 @@ app.post('/projects', (req, res) => {
       commit_message_rules,
       auto_park_stale_tickets,
       memory_enabled,
-      worktree_sync
+      worktree_sync,
+      automation_depth,
+      automation_review_fix_enabled,
+      automation_review_severity,
+      automation_review_max_iterations
     )
-    VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
-  `).run(id, name, normalizedRepoPath, run_setup, run_services, 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, worktree_sync ?? null);
+    VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
+  `).run(id, name, normalizedRepoPath, run_setup, run_services, 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, worktree_sync ?? null, automation_depth ?? 'manual', automation_review_fix_enabled ?? 0, automation_review_severity ?? 'medium', automation_review_max_iterations ?? 3);
     replaceProjectLinks(id, linked_project_ids ?? []);
     ensureProjectRootBundle(id);
     if (projectCount === 0) {
@@ -4928,7 +4872,7 @@ app.patch('/projects/:id', (req, res) => {
             });
         }
     }
-    const { name, repo_path, linked_project_ids, run_setup, run_services, 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, worktree_sync, } = validation.values;
+    const { name, repo_path, linked_project_ids, run_setup, run_services, 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, worktree_sync, automation_depth, automation_review_fix_enabled, automation_review_severity, automation_review_max_iterations, } = validation.values;
     db.prepare(`
     UPDATE projects SET
       name = ?,
@@ -4946,9 +4890,13 @@ app.patch('/projects/:id', (req, res) => {
       auto_park_stale_tickets = ?,
       memory_enabled = ?,
       worktree_sync = ?,
+      automation_depth = ?,
+      automation_review_fix_enabled = ?,
+      automation_review_severity = ?,
+      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, '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, 'worktree_sync') ? worktree_sync : project.worktree_sync, req.params.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, '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, 'worktree_sync') ? worktree_sync : project.worktree_sync, 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_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 ?? []);
     }