@straiffi/archon 1.6.1 → 1.7.0

package/dist/server/index.js

@@ -18,7 +18,7 @@ import { resolveChatSessionTarget } from './lib/chatTargets.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, getProjectBranch, getProjectBranchSuggestions, getSelectedGitUnifiedDiff, hasKnownBranchRef, listUnpushedCommitsForCwd, pullGitBranch, pushCurrentBranch, rebaseGitBranchOntoBase, resolveExistingWorktreePath, resolveWorktreePath, summarizeTicketDiffFiles, switchProjectBranch, syncWorktreeFilesAsync, undoLatestUnpushedCommitForCwd } from './lib/git.js';
+import { abortGitOperation, branchExistsLocally, commitGitChanges, createWorktreeAsync, deleteWorktree, getBundleReviewDiffForCwd, getGitDiffData, getGitDiffFiles, getGitStatus, getGitUnifiedDiff, getLocalProjectBranch, getProjectBranch, getProjectBranchSuggestions, getSelectedGitUnifiedDiff, hasKnownBranchRef, listUnpushedCommitsForCwd, pullGitBranch, pushCurrentBranch, rebaseGitBranchOntoBase, resolveExistingWorktreePath, resolveWorktreePath, summarizeTicketDiffFiles, switchProjectBranch, syncWorktreeFilesAsync, undoLatestUnpushedCommitForCwd } 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';
@@ -41,10 +41,12 @@ import { listProjectMemoryEvents } from './lib/projectMemoryEvents.js';
 import { getProjectMemorySelectionForAgentRun, listProjectMemoryUsage } from './lib/projectMemorySelections.js';
 import { decodeProjectMemoryCursor } from './lib/projectMemoryCursor.js';
 import { getProjectFileSuggestions } from './lib/projectFileSuggestions.js';
+import { getProjectFileManifest, ProjectFileContentError, ProjectFileTargetUnavailableError, readProjectFileContent } from './lib/projectFiles.js';
 import { updateProjectTargetSelection } from './lib/projectTargets.js';
 import { exportProjectMemoryArchive, importProjectMemoryArchive } from './lib/projectMemoryTransfer.js';
 import { buildReviewFindingsResponse, computeDiffSignature, getLatestReviewRunForContext, listReviewFindings } from './lib/reviewFindings.js';
-import { clearAutomationPause, countBlockingFindings, depthAtLeast, getBundleAutomationState, isAutomationDepth, resolveAutomationConfig, } from './lib/ticketAutomation.js';
+import { createBuildOpenQuestionResolutionBatch, failBuildOpenQuestionBatch, formatBuildOpenQuestionAnswersPrompt, getLatestBuildOpenQuestionBatchStatus, hasActiveBuildOpenQuestionBatch, listAnsweredBuildOpenQuestionsForBatch, listBuildOpenQuestionsForBundle, loadBuildOpenQuestionsByBundleIds, setBuildOpenQuestionBatchStatus, } from './lib/buildOpenQuestions.js';
+import { clearAutomationPause, countBlockingFindings, depthAtLeast, getBundleAutomationState, isAutomationDepth, recordAutomationPause, resolveAutomationConfig, } from './lib/ticketAutomation.js';
 import { runAutomationCommitChain } from './lib/ticketAutomationCommit.js';
 import { getDiscoveredSkills, normalizeSkillNames } from './lib/skills.js';
 import { clearAllPreviewProxies } from './lib/previewProxy.js';
@@ -834,6 +836,87 @@ const resolveProjectFileSuggestionWorkspace = (project, options = {}) => {
         cacheKey: `target:${workspace.target_kind}:${workspace.target_bundle_id ?? 'repo_root'}:${workspace.cwd}`,
     };
 };
+const resolveWorkspaceFileWorkspace = (project, targetKind, bundleId, ticketId) => {
+    if (targetKind === 'repo_root') {
+        if (!existsSync(project.repo_path)) {
+            return { error: 'Repository target is unavailable', status: 409 };
+        }
+        return {
+            cwd: project.repo_path,
+            branch: getLocalProjectBranch(project),
+            project,
+            cacheKey: `workspace-files:${project.id}:repo_root:${project.repo_path}`,
+            target: { kind: 'repo_root', bundle_id: null, ticket_id: null },
+        };
+    }
+    if (targetKind === 'bundle') {
+        if (!bundleId) {
+            return { error: 'bundle_id is required for a bundle target', status: 400 };
+        }
+        const bundle = getBundle(bundleId, project.id);
+        if (!bundle || isProjectRootBundle(bundle)) {
+            return { error: 'Bundle target not found', status: 404 };
+        }
+        const cwd = resolveExistingWorktreePath(bundle.branch, project);
+        if (!cwd) {
+            return { error: 'Bundle target is unavailable', status: 409 };
+        }
+        return {
+            cwd,
+            branch: bundle.branch,
+            project,
+            cacheKey: `workspace-files:${project.id}:bundle:${bundle.id}:${cwd}`,
+            target: { kind: 'bundle', bundle_id: bundle.id, ticket_id: null },
+        };
+    }
+    if (!ticketId) {
+        return { error: 'ticket_id is required for a ticket target', status: 400 };
+    }
+    const ticket = getTicket(ticketId);
+    if (!ticket || ticket.project_id !== project.id) {
+        return { error: 'Ticket target not found', status: 404 };
+    }
+    const bundle = ticket.worktree_bundle_id ? getBundle(ticket.worktree_bundle_id, project.id) : null;
+    if (bundle && isProjectRootBundle(bundle)) {
+        if (!existsSync(project.repo_path)) {
+            return { error: 'Ticket target is unavailable', status: 409 };
+        }
+        return {
+            cwd: project.repo_path,
+            branch: getLocalProjectBranch(project),
+            project,
+            cacheKey: `workspace-files:${project.id}:ticket:${ticket.id}:${project.repo_path}`,
+            target: { kind: 'ticket', bundle_id: null, ticket_id: ticket.id },
+        };
+    }
+    const branch = resolveTicketBranch(ticket);
+    const cwd = branch ? resolveExistingWorktreePath(branch, project) : null;
+    if (!branch || !cwd) {
+        return { error: 'Ticket target is unavailable', status: 409 };
+    }
+    return {
+        cwd,
+        branch,
+        project,
+        cacheKey: `workspace-files:${project.id}:ticket:${ticket.id}:${cwd}`,
+        target: { kind: 'ticket', bundle_id: null, ticket_id: ticket.id },
+    };
+};
+const parseWorkspaceFileTarget = (req) => {
+    const targetKind = typeof req.query.target_kind === 'string' ? req.query.target_kind : '';
+    if (targetKind !== 'repo_root' && targetKind !== 'bundle' && targetKind !== 'ticket') {
+        return { error: 'target_kind must be repo_root, bundle, or ticket', status: 400 };
+    }
+    return {
+        targetKind,
+        bundleId: typeof req.query.bundle_id === 'string' ? req.query.bundle_id.trim() || null : null,
+        ticketId: typeof req.query.ticket_id === 'string' ? req.query.ticket_id.trim() || null : null,
+    };
+};
+const getWorkspaceFileTargetUnavailableMessage = (targetKind) => {
+    const targetName = targetKind === 'repo_root' ? 'Repository' : targetKind === 'bundle' ? 'Bundle' : 'Ticket';
+    return `${targetName} target is unavailable`;
+};
 const getProjectBranchStateForFooter = (project, options = {}) => {
     const branchState = getProjectBranch(project, {
         refreshRemote: options.refreshRemote,
@@ -1062,11 +1145,11 @@ const startBundlePullRequestDiscoveryIfNeeded = (project, bundle) => {
     })();
     bundlePullRequestDiscoveryInFlight.set(key, task);
 };
-const serializeBundleRow = (projectId, bundle) => {
+const serializeBundleRow = (projectId, bundle, openQuestions = bundle ? loadBuildOpenQuestionsByBundleIds(projectId, [bundle.id]).get(bundle.id) ?? [] : [], resolvedProject) => {
     if (!bundle) {
         return null;
     }
-    const project = getProjectById(projectId);
+    const project = resolvedProject ?? getProjectById(projectId);
     const hasPreparedWorktree = bundle.kind === 'project_root'
         ? true
         : Boolean(project && resolveExistingWorktreePath(bundle.branch, project));
@@ -1077,6 +1160,8 @@ const serializeBundleRow = (projectId, bundle) => {
             ? serializeBundlePullRequest(getBundlePullRequestRecord(bundle.id, projectId))
             : null,
         automation_config: resolveAutomationConfig(bundle, project),
+        open_questions: openQuestions,
+        build_open_question_batch_status: getLatestBuildOpenQuestionBatchStatus(projectId, bundle.id),
     };
 };
 const serializeGitHubPullRequestSummary = (pullRequest) => {
@@ -1256,11 +1341,11 @@ const resolveBundleRenameEligibility = (project, bundle, tickets = listBundleTic
     return { eligible: true, reason: null };
 };
 const serializeBundleDetails = (project, bundle, extras = {}) => {
-    const serializedBundle = serializeBundleRow(project.id, bundle);
+    const tickets = listDetailedBundleTickets(project.id, bundle.id);
+    const serializedBundle = serializeBundleRow(project.id, bundle, tickets.flatMap(ticket => ticket.build_open_questions ?? []), project);
     if (!serializedBundle) {
         throw new Error(`Failed to serialize bundle ${bundle.id}`);
     }
-    const tickets = listDetailedBundleTickets(project.id, bundle.id);
     const latestSessionTicket = resolveBundleLatestSessionTicket(tickets);
     const resolvedTargetTicket = extras.target_ticket_id
         ? tickets.find(ticket => ticket.id === extras.target_ticket_id) ?? null
@@ -1389,6 +1474,9 @@ const reviewBundleTickets = (projectId, bundleId) => {
     if (!bundle) {
         return { ok: false, status: 404, error: 'Bundle not found' };
     }
+    if (hasActiveBuildOpenQuestionBatch(projectId, bundle.id)) {
+        return { ok: false, status: 409, error: 'Build open question resolution is still in progress' };
+    }
     const result = transitionBundleToReview(projectId, bundle.id, io);
     if (!result.ok) {
         return { ok: false, status: result.status, error: result.error };
@@ -2630,11 +2718,12 @@ app.post('/projects/:id/spec-sessions/:sessionId/snapshots/:snapshotId/export',
         return res.status(404).json({ error: 'Not found' });
     }
     try {
+        const overwrite = req.body?.overwrite === true;
         const collision = checkSpecExportCollision({
             specSession,
             repoPath: project.repo_path,
         });
-        if (collision) {
+        if (collision && !overwrite) {
             return res.status(409).json({
                 error: 'A spec with this title has already been exported.',
                 conflicting_path: collision,
@@ -2841,7 +2930,8 @@ app.get('/bundles', async (req, res) => {
     timing.mark('ensure_project_root_bundle');
     const bundles = listBundles(result.project.id);
     timing.mark('list_bundles');
-    const serializedBundles = bundles.map(bundle => serializeBundleRow(result.project.id, bundle));
+    const openQuestionsByBundleId = loadBuildOpenQuestionsByBundleIds(result.project.id, bundles.map(bundle => bundle.id));
+    const serializedBundles = bundles.map(bundle => serializeBundleRow(result.project.id, bundle, openQuestionsByBundleId.get(bundle.id) ?? [], result.project));
     timing.mark('serialize_bundles');
     timing.end({ count: serializedBundles.length });
     res.json(serializedBundles);
@@ -3323,6 +3413,19 @@ app.patch('/bundles/:id', (req, res) => {
             return res.status(400).json({ error: 'automation_review_fix_enabled must be a boolean' });
         }
     }
+    let automationPauseOnOpenQuestions = bundle.automation_pause_on_open_questions ?? null;
+    if (hasOwn(req.body, 'automation_pause_on_open_questions')) {
+        const value = req.body.automation_pause_on_open_questions;
+        if (value == null) {
+            automationPauseOnOpenQuestions = null;
+        }
+        else if (typeof value === 'boolean') {
+            automationPauseOnOpenQuestions = value ? 1 : 0;
+        }
+        else {
+            return res.status(400).json({ error: 'automation_pause_on_open_questions must be a boolean' });
+        }
+    }
     let updated = null;
     try {
         updated = db.transaction(() => {
@@ -3339,6 +3442,7 @@ app.patch('/bundles/:id', (req, res) => {
                 projectId: result.project.id,
                 automationDepth,
                 automationReviewFixEnabled,
+                automationPauseOnOpenQuestions,
             });
         })();
     }
@@ -3379,6 +3483,99 @@ app.post('/bundles/:id/review', (req, res) => {
     }
     return res.json(reviewResult.details);
 });
+app.post('/bundles/:id/build-open-questions/resolve', (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 submitted = req.body?.resolutions;
+    if (!Array.isArray(submitted) || submitted.length === 0) {
+        return res.status(400).json({ error: 'resolutions must be a non-empty array' });
+    }
+    const resolutions = submitted.map((resolution) => {
+        if (!resolution || typeof resolution !== 'object') {
+            return null;
+        }
+        const value = resolution;
+        return {
+            questionId: typeof value.question_id === 'string' ? value.question_id : '',
+            answer: typeof value.answer === 'string' ? value.answer : undefined,
+            dismiss: value.dismiss === true,
+            dismissalReason: typeof value.dismissal_reason === 'string' ? value.dismissal_reason : null,
+            dismissedBy: 'user',
+        };
+    });
+    if (resolutions.some(resolution => resolution === null || !resolution.questionId)) {
+        return res.status(400).json({ error: 'Each resolution requires a question_id' });
+    }
+    const buildTickets = listBundleSliceTickets(result.project.id, bundle.id, 'build');
+    if (buildTickets.length === 0
+        || buildTickets.some(ticket => ticket.agent_status !== 'done' && ticket.agent_status !== 'stopped')) {
+        return res.status(409).json({ error: 'All build tickets in the bundle must be done or stopped before resolving questions' });
+    }
+    const created = createBuildOpenQuestionResolutionBatch({
+        projectId: result.project.id,
+        bundleId: bundle.id,
+        resolutions: resolutions.filter((resolution) => resolution !== null),
+    });
+    if (!created.ok) {
+        return res.status(409).json({
+            error: created.error,
+            ...(created.code ? { code: created.code } : {}),
+        });
+    }
+    const answeredQuestions = listAnsweredBuildOpenQuestionsForBatch(created.batch.id);
+    if (answeredQuestions.length === 0) {
+        clearAutomationPause(result.project.id, bundle.id);
+        const automationConfig = resolveAutomationConfig(bundle, result.project);
+        const reviewResult = depthAtLeast(automationConfig.depth, 'auto_review')
+            ? transitionBundleToReview(result.project.id, bundle.id, io, { automatic: true })
+            : null;
+        for (const ticket of listDetailedBundleTickets(result.project.id, bundle.id)) {
+            io.emit('ticket:updated', ticket);
+        }
+        if (reviewResult && !reviewResult.ok) {
+            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,
+        }));
+    }
+    const targetTicketSource = resolveBundleTargetTicket(buildTickets);
+    const targetTicket = targetTicketSource ? getTicket(targetTicketSource.id) : null;
+    if (!targetTicket) {
+        failBuildOpenQuestionBatch(created.batch.id, 'No completed Build ticket is available for the follow-up');
+        recordAutomationPause(result.project.id, bundle.id, 'open_questions', 'Build open question follow-up could not be started');
+        return res.status(409).json({ error: 'No completed Build ticket is available for the follow-up' });
+    }
+    const instruction = [
+        'Apply this complete batch of answered Build requirements in one bundle-scoped pass. Preserve the grouping by originating ticket and do not work on dismissed questions.',
+        formatBuildOpenQuestionAnswersPrompt(answeredQuestions),
+    ].join('\n\n');
+    setBuildOpenQuestionBatchStatus(created.batch.id, 'running');
+    const started = startFollowUp(targetTicket, io, instruction, 'build', {
+        buildOpenQuestionBatchId: created.batch.id,
+    });
+    if (!started) {
+        failBuildOpenQuestionBatch(created.batch.id, 'Build follow-up could not be started');
+        recordAutomationPause(result.project.id, bundle.id, 'open_questions', 'Build open question follow-up could not be started');
+        for (const ticket of listDetailedBundleTickets(result.project.id, bundle.id)) {
+            io.emit('ticket:updated', ticket);
+        }
+        return res.status(409).json({ error: 'Build follow-up could not be started' });
+    }
+    for (const ticket of listDetailedBundleTickets(result.project.id, bundle.id)) {
+        io.emit('ticket:updated', ticket);
+    }
+    return res.status(202).json(serializeBundleDetails(result.project, bundle, {
+        target_ticket_id: targetTicket.id,
+    }));
+});
 app.post('/bundles/:id/undo-review', (req, res) => {
     const result = getRequiredProject(req);
     if ('error' in result) {
@@ -3482,10 +3679,21 @@ app.post('/bundles/:id/resume-automation', async (req, res) => {
     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;
+    const liveAutomationConfig = resolveAutomationConfig(bundle, result.project);
+    if (hasActiveBuildOpenQuestionBatch(result.project.id, bundle.id)) {
+        return res.status(409).json({ error: 'Build open question resolution is still in progress' });
+    }
+    if (bundleState.automation_paused_code === 'open_questions'
+        && liveAutomationConfig.pauseOnOpenQuestions
+        && listBuildOpenQuestionsForBundle(result.project.id, bundle.id).length > 0) {
+        return res.status(409).json({ error: 'Resolve or dismiss all Build open questions before resuming automation' });
+    }
+    const reviewOwnerSource = bundleState.automation_paused_code === 'open_questions'
+        ? listBundleSliceTickets(result.project.id, bundle.id, 'build')
+            .find(ticket => ticket.agent_status === 'done') ?? null
+        : 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' });
     }
@@ -3514,12 +3722,15 @@ app.post('/bundles/:id/resume-automation', async (req, res) => {
             && automationConfig.reviewFixEnabled
             && blockingCount > 0
             && iteration < automationConfig.reviewMaxIterations) {
+            clearAutomationPause(result.project.id, bundle.id);
             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')) {
+            clearAutomationPause(result.project.id, bundle.id);
             await runAutomationCommitChain(owner, project, owner.bundle ?? null, automationConfig, io);
         }
         else {
+            clearAutomationPause(result.project.id, bundle.id);
             startReview(owner, io);
         }
     }
@@ -3528,6 +3739,7 @@ app.post('/bundles/:id/resume-automation', async (req, res) => {
         if (!reviewResult.ok) {
             return res.status(reviewResult.status).json({ error: reviewResult.error });
         }
+        clearAutomationPause(result.project.id, bundle.id);
     }
     else {
         return res.status(409).json({ error: 'Ticket is not in a resumable state' });
@@ -3551,6 +3763,14 @@ app.post('/bundles/:id/dismiss-automation-pause', (req, res) => {
     if (!bundleState || !bundleState.automation_paused_reason) {
         return res.status(409).json({ error: 'Bundle automation is not paused' });
     }
+    if (hasActiveBuildOpenQuestionBatch(result.project.id, bundle.id)) {
+        return res.status(409).json({ error: 'Build open question resolution is still in progress' });
+    }
+    if (bundleState.automation_paused_code === 'open_questions'
+        && resolveAutomationConfig(bundle, result.project).pauseOnOpenQuestions
+        && listBuildOpenQuestionsForBundle(result.project.id, bundle.id).length > 0) {
+        return res.status(409).json({ error: 'Resolve or dismiss all Build open questions before dismissing this pause' });
+    }
     clearAutomationPause(result.project.id, bundle.id);
     for (const ticket of listDetailedBundleTickets(result.project.id, bundle.id)) {
         io.emit('ticket:updated', ticket);
@@ -4506,7 +4726,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, scan_include_globs_json, scan_exclude_globs_json, worktree_sync, spec_aware_code_review_enabled, automation_depth, automation_review_fix_enabled, automation_review_severity, automation_review_max_iterations, } = 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, 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' });
     }
@@ -4542,11 +4762,12 @@ app.post('/projects', (req, res) => {
       spec_aware_code_review_enabled,
       automation_depth,
       automation_review_fix_enabled,
+      automation_pause_on_open_questions,
       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, 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_review_severity ?? 'medium', automation_review_max_iterations ?? 3);
+    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, 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) {
@@ -5316,6 +5537,60 @@ app.get('/projects/:id/file-suggestions', (req, res) => {
         return res.status(500).json({ error: 'Unable to load project file suggestions' });
     }
 });
+app.get('/projects/:id/workspace-files', (req, res) => {
+    const project = getProjectById(req.params.id);
+    if (!project) {
+        return res.status(404).json({ error: 'Not found' });
+    }
+    const target = parseWorkspaceFileTarget(req);
+    if ('error' in target) {
+        return res.status(target.status).json({ error: target.error });
+    }
+    const workspace = resolveWorkspaceFileWorkspace(project, target.targetKind, target.bundleId, target.ticketId);
+    if ('error' in workspace) {
+        return res.status(workspace.status).json({ error: workspace.error });
+    }
+    try {
+        return res.json(getProjectFileManifest(workspace, { refresh: req.query.refresh === 'true' }));
+    }
+    catch (error) {
+        if (error instanceof ProjectFileTargetUnavailableError) {
+            return res.status(409).json({ error: getWorkspaceFileTargetUnavailableMessage(workspace.target.kind) });
+        }
+        console.error(`Failed to build workspace file manifest for project ${project.id}:`, error);
+        return res.status(500).json({ error: 'Unable to load workspace files' });
+    }
+});
+app.get('/projects/:id/workspace-files/content', (req, res) => {
+    const project = getProjectById(req.params.id);
+    if (!project) {
+        return res.status(404).json({ error: 'Not found' });
+    }
+    const target = parseWorkspaceFileTarget(req);
+    if ('error' in target) {
+        return res.status(target.status).json({ error: target.error });
+    }
+    if (typeof req.query.path !== 'string') {
+        return res.status(400).json({ error: 'path is required' });
+    }
+    const workspace = resolveWorkspaceFileWorkspace(project, target.targetKind, target.bundleId, target.ticketId);
+    if ('error' in workspace) {
+        return res.status(workspace.status).json({ error: workspace.error });
+    }
+    try {
+        return res.json(readProjectFileContent(workspace, req.query.path));
+    }
+    catch (error) {
+        if (error instanceof ProjectFileTargetUnavailableError) {
+            return res.status(409).json({ error: getWorkspaceFileTargetUnavailableMessage(workspace.target.kind) });
+        }
+        if (error instanceof ProjectFileContentError) {
+            return res.status(error.status).json({ error: error.message, reason: error.reason });
+        }
+        console.error(`Failed to read workspace file content for project ${project.id}:`, error);
+        return res.status(500).json({ error: 'Unable to read workspace file' });
+    }
+});
 app.post('/projects/:id/git-pull', async (req, res) => {
     const project = getProjectById(req.params.id);
     if (!project) {
@@ -5658,7 +5933,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, scan_include_globs_json, scan_exclude_globs_json, worktree_sync, spec_aware_code_review_enabled, automation_depth, automation_review_fix_enabled, automation_review_severity, automation_review_max_iterations, } = 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, 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(`
     UPDATE projects SET
       name = ?,
@@ -5681,11 +5956,12 @@ app.patch('/projects/:id', (req, res) => {
       spec_aware_code_review_enabled = ?,
       automation_depth = ?,
       automation_review_fix_enabled = ?,
+      automation_pause_on_open_questions = ?,
       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, '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_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);
+  `).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, '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 ?? []);
     }
@@ -6053,6 +6329,12 @@ app.post('/tickets/:id/retry', (req, res) => {
     if (ticket.agent_status !== 'error') {
         return res.status(400).json({ error: 'Ticket is not in error state' });
     }
+    if (ticket.state === 'review'
+        && ticket.project_id
+        && ticket.worktree_bundle_id
+        && hasActiveBuildOpenQuestionBatch(ticket.project_id, ticket.worktree_bundle_id)) {
+        return res.status(409).json({ error: 'Build open question resolution is still in progress' });
+    }
     db.prepare('UPDATE tickets SET agent_status = NULL, agent_log = NULL, agent_message_id = NULL, previous_agent_log = NULL, previous_agent_message_id = NULL, streaming_response = NULL, build_completed_at = NULL, auto_park_dismissed_at = NULL, updated_at = CURRENT_TIMESTAMP WHERE id = ?').run(req.params.id);
     const updated = getTicket(req.params.id);
     io.emit('ticket:updated', updated);