@straiffi/archon 1.7.4 → 1.7.6

package/dist/server/index.js

@@ -10,7 +10,7 @@ import { adjectives, animals, uniqueNamesGenerator } from 'unique-names-generato
 import db from './db.js';
 import { deleteChatSession, getChatSession, listChatSessions, updateChatSessionMode } from './lib/chats.js';
 import { deleteSpecSession, getCreatedTicketIdsForSnapshot, getSpecSession, getSpecSnapshot, listSpecSessions } from './lib/specs.js';
-import { createChatAttachment, createSpecAttachment, getAttachmentById, deleteChatAttachment, 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 { 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, resolveChatMode, resolveChatTool, stopProjectChatSessionResponse, submitProjectChatSessionMessage, } from './lib/chatOperations.js';
 import { createProjectSpecSession, deleteProjectSpecSession, stopProjectSpecSessionResponse, submitProjectSpecSessionMessage, updateProjectSpecSessionMode, } from './lib/specOperations.js';
 import { buildProjectSpecReviewRevisionDraft, getProjectSpecReviewFindings, startProjectSpecReview, stopProjectSpecReview, } from './lib/specReviewOperations.js';
@@ -49,15 +49,18 @@ import { formatBuildOpenQuestionAnswersPrompt, getLatestBuildOpenQuestionBatchSt
 import { acceptBuildOpenQuestionResolution } from './lib/buildOpenQuestionOperations.js';
 import { completeAutomationProgression, createAutomationReviewFix, finishAutomationReviewFix, getAutomationProgression, recoverLegacyAutomationProgression, recoverStoppedSideEffectProgression, runAutomationProgression, stopAutomationProgression, waitAutomationProgression } from './lib/automationProgression.js';
 import { reconcileAutomationStage } from './lib/automationReconciliation.js';
+import { pausedRunBelongsToProgression, resolveAutomationResumeDecision } from './lib/automationResumeDecision.js';
 import { getAutomationStageDetails, } from './lib/automationStageDetails.js';
-import { clearAutomationPause, countBlockingFindings, depthAtLeast, getBundleAutomationState, isAutomationDepth, recordAutomationPause, resolveAutomationConfig, upsertBundleAutomationState, } from './lib/ticketAutomation.js';
+import { buildAutomationFixInstruction, clearAutomationPause, countBlockingFindings, depthAtLeast, getBundleAutomationState, isAutomationDepth, recordAutomationPause, resolveAutomationConfig, upsertBundleAutomationState, } from './lib/ticketAutomation.js';
 import { runAutomationCommitChain } from './lib/ticketAutomationCommit.js';
 import { getDiscoveredSkills, normalizeSkillNames } from './lib/skills.js';
 import { clearAllPreviewProxies } from './lib/previewProxy.js';
 import { createExecutionRun, addExecutionRunLink, addExecutionRunLinks, getExecutionRun, hasActiveBuildExecutionRunForTicket, getGitHubActionsExecutionDetails, getTestExecutionDetails, GitHubActionsDispatchConflictError, listExecutionRunLinks, listExecutionRuns, listLifecycleEvents, listProjectLifecycleEvents, toCanonicalLifecycleEvent, transitionExecutionRun, withExecutionRunTransaction, } from './lib/executionRuns.js';
 import { dispatchExecutionRetry, getExecutionRetryEligibility, resolveExecutionRetry } from './lib/executionRetry.js';
 import { listExecutionTimeline } from './lib/executionTimeline.js';
+import { loadExecutionArtifactSummariesForRuns, openExecutionArtifactForRetrieval, } from './lib/executionArtifacts.js';
 import { cleanupAllExecutionEvidence, EXECUTION_EVIDENCE_CLEANUP_INTERVAL_MS, resolveExecutionEvidenceRetentionDays, } from './lib/executionRunRetention.js';
+import { ExecutionArtifactCleanupError, reconcileExecutionArtifactRegistrationIntents, reconcileFinalizedExecutionArtifacts, reconcileProjectExecutionArtifacts, resolveArtifactCleanupIntervalMs, } from './lib/executionArtifactRetention.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';
@@ -72,7 +75,7 @@ import { PlanningReferenceError, resolveExternalPlanningDescription } from './li
 import { refreshIfNeeded as refreshAssignedIssues, clearCachedAssignedIssues } from './lib/integrations/assignedIssuesCache.js';
 import { buildCommitMessagePrompt, normalizeGeneratedCommitMessage } from './lib/commitMessage.js';
 import { BUNDLE_PULL_REQUEST_SYNC_TTL_MS, getBundlePullRequestRecord, hasOpenBundlePullRequest, serializeBundlePullRequest, shouldSyncBundlePullRequest, syncTrackedBundlePullRequestIfNeeded, upsertBundlePullRequest, upsertBundlePullRequestFromGitHub, } from './lib/bundlePullRequests.js';
-import { countInboundProjectLinks, getEffectiveProject, getProjectById, getProjectTicketStats, hasProjectRunIde, listProjects, replaceProjectLinks, serializeProject, updateProjectActiveTarget, validateProjectPayload, } from './lib/projects.js';
+import { countInboundProjectLinks, getEffectiveProject, getProjectById, getProjectTicketStats, hasProjectRunIde, listInterruptedProjectDeletions, listProjects, releaseInterruptedProjectDeletionClaims, 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';
 import { getBundleBuildBlockerResponse, shouldStartBundleBuildNow, transitionTicketState, } from './lib/ticketWorkflowOperations.js';
@@ -94,7 +97,30 @@ import { exportSpecToDocs, checkSpecExportCollision } from './lib/specExport.js'
 import { createTicketDependency, deleteTicketDependenciesForTicket, deleteTicketDependency, getTicketDependency, isTicketRunSetupReady, listTicketDependencies, validateCreateTicketDependency, } from './lib/ticketDependencies.js';
 import { startBuild } from './workers/build.js';
 import { expandPlanningTicketOutline, generatePlanningTicketOutline, generatePlanText, prepareLinkedPlanningResearch, startPlan, stopAllPlanAgents, } from './workers/plan.js';
-import { EXECUTION_RUN_KINDS, validateWorkflowContext } from './lib/types.js';
+import { EXECUTION_RUN_KINDS, MAX_ARTIFACTS_PER_RUN, validateWorkflowContext } from './lib/types.js';
+class ProjectDeletionError extends Error {
+    status;
+    constructor(status, message) {
+        super(message);
+        this.name = 'ProjectDeletionError';
+        this.status = status;
+    }
+}
+let executionArtifactMaintenanceTail = Promise.resolve();
+const runExclusiveExecutionArtifactMaintenance = async (task) => {
+    const previous = executionArtifactMaintenanceTail;
+    let release = () => undefined;
+    executionArtifactMaintenanceTail = new Promise(resolve => {
+        release = resolve;
+    });
+    await previous;
+    try {
+        return await task();
+    }
+    finally {
+        release();
+    }
+};
 const getErrorMessage = (error) => {
     return error instanceof Error ? error.message : String(error);
 };
@@ -112,6 +138,9 @@ const requireExistingProject = (projectId) => {
     if (!project) {
         return { status: 404, error: 'Not found' };
     }
+    if (project.deletion_started_at) {
+        return { status: 409, error: 'Project deletion is in progress' };
+    }
     return { project };
 };
 const requireProjectMemoryProject = (projectId) => {
@@ -261,6 +290,26 @@ const serializeGitHubActionsExecutionDetail = (details) => {
         jobs_count: details.jobs_count,
     };
 };
+const buildExecutionRunDetailResponse = (project, run) => {
+    const retryEligibility = getExecutionRetryEligibility(project.id, run.id);
+    const details = run.kind === 'test'
+        ? serializeTestExecutionDetail(getTestExecutionDetails(project.id, run.id), retryEligibility)
+        : 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 {
+        run,
+        links: listExecutionRunLinks(project.id, run.id),
+        events: listLifecycleEvents(project.id, run.id).map(toCanonicalLifecycleEvent),
+        details,
+        agent_runs: listAgentRunsForExecutionRun(run.id),
+        ...artifacts,
+    };
+};
 const formatWorktreeDeleteError = (target, branch, error) => {
     return `Unable to delete the ${target} worktree for branch "${branch}": ${getErrorMessage(error)}`;
 };
@@ -2400,6 +2449,17 @@ app.post('/projects/:id/chat-sessions/:sessionId/attachments', (req, res) => {
         if (error) {
             return res.status(400).json({ error: getChatAttachmentUploadErrorMessage(error) });
         }
+        const currentProject = getProjectById(req.params.id);
+        if (!currentProject) {
+            return res.status(404).json({ error: 'Not found' });
+        }
+        const currentChatSession = getProjectChatSession(currentProject.id, req.params.sessionId);
+        if (!currentChatSession) {
+            return res.status(404).json({ error: 'Not found' });
+        }
+        if (currentChatSession.agent_status === 'running') {
+            return res.status(409).json({ error: 'Wait for the current chat response to finish before uploading images.' });
+        }
         const files = (req.files ?? []);
         if (files.length === 0) {
             return res.status(400).json({ error: 'At least one image is required.' });
@@ -2417,11 +2477,11 @@ app.post('/projects/:id/chat-sessions/:sessionId/attachments', (req, res) => {
         try {
             const attachments = files.map(file => {
                 const attachment = createChatAttachment({
-                    chatSessionId: chatSession.id,
+                    chatSessionId: currentChatSession.id,
                     originalName: file.originalname,
                     mimeType: file.mimetype,
                     buffer: file.buffer,
-                    projectId: project.id,
+                    projectId: currentProject.id,
                 });
                 createdAttachmentIds.push(attachment.id);
                 return attachment;
@@ -2692,6 +2752,17 @@ app.post('/projects/:id/spec-sessions/:sessionId/attachments', (req, res) => {
         if (error) {
             return res.status(400).json({ error: getSpecAttachmentUploadErrorMessage(error) });
         }
+        const currentProject = getProjectById(req.params.id);
+        if (!currentProject) {
+            return res.status(404).json({ error: 'Not found' });
+        }
+        const currentSpecSession = getProjectSpecSession(currentProject.id, req.params.sessionId);
+        if (!currentSpecSession) {
+            return res.status(404).json({ error: 'Not found' });
+        }
+        if (currentSpecSession.agent_status === 'running') {
+            return res.status(409).json({ error: 'Wait for the current spec response to finish before uploading images.' });
+        }
         const files = (req.files ?? []);
         if (files.length === 0) {
             return res.status(400).json({ error: 'At least one image is required.' });
@@ -2709,11 +2780,11 @@ app.post('/projects/:id/spec-sessions/:sessionId/attachments', (req, res) => {
         try {
             const attachments = files.map(file => {
                 const attachment = createSpecAttachment({
-                    specSessionId: specSession.id,
+                    specSessionId: currentSpecSession.id,
                     originalName: file.originalname,
                     mimeType: file.mimetype,
                     buffer: file.buffer,
-                    projectId: project.id,
+                    projectId: currentProject.id,
                 });
                 createdAttachmentIds.push(attachment.id);
                 return attachment;
@@ -3884,25 +3955,6 @@ app.post('/bundles/:id/follow-up', (req, res) => {
 });
 const AUTOMATION_SIDE_EFFECT_KINDS = new Set(['automation_commit', 'automation_push', 'automation_pull_request']);
 const AUTOMATION_WORKER_PAUSE_KINDS = new Set(['build', 'review', 'build_follow_up', 'automation_review_fix']);
-const pausedRunBelongsToProgression = (projectId, bundleId, progressionRunId, pausedRunId) => {
-    const pausedRun = getExecutionRun(projectId, pausedRunId);
-    const progression = getExecutionRun(projectId, progressionRunId);
-    if (!pausedRun || !progression || progression.kind !== 'automation_progression') {
-        return false;
-    }
-    const ownsBundle = listExecutionRunLinks(projectId, pausedRun.id).some(link => (link.entity_type === 'bundle'
-        && link.entity_id === bundleId
-        && (link.relationship === 'primary' || link.relationship === 'context')));
-    if (!ownsBundle) {
-        return false;
-    }
-    if (pausedRun.parent_run_id === progressionRunId) {
-        return true;
-    }
-    const parent = pausedRun.parent_run_id ? getExecutionRun(projectId, pausedRun.parent_run_id) : null;
-    return parent?.kind === 'automation_review_fix' && parent.parent_run_id === progressionRunId
-        || progression.parent_run_id === pausedRun.id;
-};
 app.post('/bundles/:id/resume-automation', async (req, res) => {
     const result = getRequiredProject(req);
     if ('error' in result) {
@@ -3916,15 +3968,38 @@ 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' });
     }
-    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 resolveCurrentDecision = () => {
+        const currentState = getBundleAutomationState(result.project.id, bundle.id);
+        if (!currentState?.automation_paused_reason) {
+            return null;
+        }
+        return resolveAutomationResumeDecision({
+            projectId: result.project.id,
+            bundleId: bundle.id,
+            state: currentState,
+            config: resolveAutomationConfig(getBundle(bundle.id, result.project.id), getProjectById(result.project.id)),
+            tickets: listDetailedBundleTickets(result.project.id, bundle.id),
+            openQuestionCount: listBuildOpenQuestionsForBundle(result.project.id, bundle.id).length,
+            questionResolutionActive: hasActiveBuildOpenQuestionBatch(result.project.id, bundle.id),
+        });
+    };
+    const decision = resolveCurrentDecision();
+    if (!decision || decision.status !== 'enabled') {
+        const error = decision?.status === 'hidden' && decision.reason === 'question_resolution_active'
+            ? 'Build open question resolution is still in progress'
+            : decision?.status === 'hidden' && decision.reason === 'resolve_open_questions'
+                ? 'Resolve or dismiss all Build open questions before resuming automation'
+                : decision?.status === 'disabled' && decision.reason === 'invalid_stage_lineage'
+                    ? 'The paused automation stage does not belong to the active progression'
+                    : 'Bundle automation cannot be resumed in its current state';
+        return res.status(409).json({ error, automation_resume_decision: decision });
+    }
+    const revalidateDecision = () => {
+        const current = resolveCurrentDecision();
+        return current?.status === 'enabled'
+            && current.outcome === decision.outcome
+            && current.owner_ticket_id === decision.owner_ticket_id;
+    };
     let progressionRunId = bundleState.automation_progression_run_id;
     if (!progressionRunId) {
         try {
@@ -3952,12 +4027,13 @@ app.post('/bundles/:id/resume-automation', async (req, res) => {
     catch (error) {
         return res.status(409).json({ error: error instanceof Error ? error.message : String(error) });
     }
+    if (!revalidateDecision()) {
+        return res.status(409).json({ error: 'Bundle automation state changed before Resume could start' });
+    }
     let sideEffectResume = null;
-    let pausedRunKind = null;
     if (bundleState.automation_paused_stage_run_id) {
         const stageRunId = bundleState.automation_paused_stage_run_id;
         const stage = getExecutionRun(result.project.id, stageRunId);
-        pausedRunKind = stage?.kind ?? null;
         if (!stage
             || (!AUTOMATION_SIDE_EFFECT_KINDS.has(stage.kind) && !AUTOMATION_WORKER_PAUSE_KINDS.has(stage.kind))
             || !pausedRunBelongsToProgression(result.project.id, bundle.id, progressionRunId, stageRunId)) {
@@ -4021,33 +4097,7 @@ app.post('/bundles/:id/resume-automation', async (req, res) => {
             }
         }
     }
-    if (liveAutomationConfig.depth === 'manual') {
-        completeAutomationProgression({
-            projectId: result.project.id,
-            bundleId: bundle.id,
-            progressionRunId,
-            domainStatus: 'manual_boundary_reached',
-            broadcaster: io,
-        });
-        for (const ticket of listDetailedBundleTickets(result.project.id, bundle.id)) {
-            io.emit('ticket:updated', ticket);
-        }
-        return res.json(serializeBundleDetails(result.project, bundle));
-    }
-    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]
-            ?? (bundleState.automation_paused_code === 'review' && (pausedRunKind === 'build' || pausedRunKind === 'build_follow_up')
-                ? listBundleSliceTickets(result.project.id, bundle.id, 'build')
-                    .find(ticket => ticket.agent_status === 'done') ?? null
-                : null)
-            ?? null;
-    if (!reviewOwnerSource) {
-        return res.status(409).json({ error: 'No review ticket to resume from' });
-    }
-    const owner = getTicket(reviewOwnerSource.id);
+    const owner = getTicket(decision.owner_ticket_id);
     if (!owner) {
         return res.status(409).json({ error: 'No review ticket to resume from' });
     }
@@ -4081,6 +4131,9 @@ app.post('/bundles/:id/resume-automation', async (req, res) => {
         };
     };
     if (sideEffectResume) {
+        if (!decision.outcome.startsWith('reconcile_') || !revalidateDecision()) {
+            return res.status(409).json({ error: 'Bundle automation state changed before Resume could start' });
+        }
         try {
             runAutomationProgression({
                 projectId: result.project.id,
@@ -4099,27 +4152,17 @@ app.post('/bundles/:id/resume-automation', async (req, res) => {
         }
         return res.json(serializeBundleDetails(result.project, bundle));
     }
-    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) {
+        if (decision.outcome === 'start_review_fix') {
             let reviewFixRunId = null;
             let followUpResult;
             const previousIteration = bundleState.review_iteration_count ?? 0;
             const previousFixLoopActive = bundleState.fix_loop_active;
             try {
+                if (!revalidateDecision()) {
+                    return res.status(409).json({ error: 'Bundle automation state changed before Resume could start' });
+                }
                 const progression = getAutomationProgression(result.project.id, bundle.id);
                 if (!progression || progression.id !== progressionRunId) {
                     return res.status(409).json({ error: 'Automation progression ownership is invalid' });
@@ -4149,10 +4192,11 @@ app.post('/bundles/:id/resume-automation', async (req, res) => {
                     ticketId: owner.id,
                     requestedProjectId: result.project.id,
                     payload: {
-                        instruction: `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.`,
+                        instruction: buildAutomationFixInstruction(automationConfig.reviewSeverity),
                         mode: 'build',
                     },
                     broadcaster: io,
+                    persistTriggerMessage: false,
                     initiation: {
                         initiator: 'automation',
                         causalParentRunId: reviewFix.runId,
@@ -4206,8 +4250,11 @@ app.post('/bundles/:id/resume-automation', async (req, res) => {
                 return res.status(followUpResult.status).json({ error: followUpResult.error });
             }
         }
-        else if (depthAtLeast(automationConfig.depth, 'auto_commit')) {
+        else if (decision.outcome === 'continue_commit_chain') {
             try {
+                if (!revalidateDecision()) {
+                    return res.status(409).json({ error: 'Bundle automation state changed before Resume could start' });
+                }
                 runAutomationProgression({
                     projectId: result.project.id,
                     bundleId: bundle.id,
@@ -4220,7 +4267,10 @@ app.post('/bundles/:id/resume-automation', async (req, res) => {
                 return res.status(409).json({ error: error instanceof Error ? error.message : String(error) });
             }
         }
-        else {
+        else if (decision.outcome === 'rerun_review') {
+            if (!revalidateDecision()) {
+                return res.status(409).json({ error: 'Bundle automation state changed before Resume could start' });
+            }
             const reviewResult = rerunTicketReviewOperation(owner, io, {
                 automatic: true,
                 initiation: { causalParentRunId: progressionRunId },
@@ -4230,8 +4280,14 @@ app.post('/bundles/:id/resume-automation', async (req, res) => {
                 return res.status(reviewResult.status).json({ error: reviewResult.error });
             }
         }
+        else {
+            return res.status(409).json({ error: 'The selected Resume outcome no longer matches the bundle state' });
+        }
     }
-    else if (owner.state === 'build' && owner.agent_status === 'done') {
+    else if (owner.state === 'build' && owner.agent_status === 'done' && decision.outcome === 'start_review') {
+        if (!revalidateDecision()) {
+            return res.status(409).json({ error: 'Bundle automation state changed before Resume could start' });
+        }
         const reviewResult = transitionBundleToReview(result.project.id, bundle.id, io, {
             automatic: true,
             initiation: { causalParentRunId: progressionRunId },
@@ -6659,32 +6715,114 @@ app.patch('/projects/:id', (req, res) => {
     const updated = db.prepare('SELECT * FROM projects WHERE id = ?').get(req.params.id);
     res.json(serializeProject(updated ?? null));
 });
-app.delete('/projects/:id', (req, res) => {
-    const project = getProjectById(req.params.id);
+const assertProjectDeletionEligible = (projectId, claimId) => {
+    const project = db.prepare('SELECT * FROM projects WHERE id = ?').get(projectId);
     if (!project) {
-        return res.status(404).json({ error: 'Not found' });
+        throw new ProjectDeletionError(404, 'Not found');
     }
-    const projectCount = Number(db.prepare('SELECT COUNT(*) AS count FROM projects').get()?.count ?? 0);
+    if (claimId ? project.deletion_claim_id !== claimId : Boolean(project.deletion_started_at)) {
+        throw new ProjectDeletionError(409, 'Project deletion is already in progress');
+    }
+    const projectCount = Number(db.prepare(`
+    SELECT COUNT(*) AS count FROM projects
+    WHERE deletion_started_at IS NULL OR id = ?
+  `).get(projectId)?.count ?? 0);
     if (projectCount <= 1) {
-        return res.status(409).json({ error: 'Cannot delete the last remaining project' });
+        throw new ProjectDeletionError(409, 'Cannot delete the last remaining project');
     }
-    if (countInboundProjectLinks(project.id) > 0) {
-        return res.status(409).json({ error: 'Cannot delete a project that is linked from another project' });
+    if (countInboundProjectLinks(projectId) > 0) {
+        throw new ProjectDeletionError(409, 'Cannot delete a project that is linked from another project');
     }
-    const stats = getProjectTicketStats(project.id);
+    const stats = getProjectTicketStats(projectId);
     if (stats.ticketCount > 0 || stats.worktreeCount > 0 || stats.bundleCount > 0) {
-        return res.status(409).json({
-            error: 'Cannot delete a project with existing tickets, bundles, or worktrees',
-        });
+        throw new ProjectDeletionError(409, 'Cannot delete a project with existing tickets, bundles, or worktrees');
     }
-    db.transaction(() => {
+};
+const completeClaimedProjectDeletion = async (projectId, claimId) => {
+    await runExclusiveExecutionArtifactMaintenance(() => reconcileProjectExecutionArtifacts(projectId, {
+        emitter: io,
+        beginCleanup: () => {
+            const marked = db.prepare(`
+        UPDATE projects
+        SET deletion_cleanup_started_at = COALESCE(deletion_cleanup_started_at, CURRENT_TIMESTAMP)
+        WHERE id = ? AND deletion_claim_id = ?
+      `).run(projectId, claimId);
+            if (marked.changes !== 1) {
+                throw new ProjectDeletionError(409, 'Project deletion ownership was lost');
+            }
+        },
+        finalize: () => {
+            db.transaction(() => {
+                assertProjectDeletionEligible(projectId, claimId);
+                if (db.prepare('SELECT 1 FROM execution_artifact_registration_intents WHERE project_id = ? LIMIT 1').get(projectId)) {
+                    throw new ExecutionArtifactCleanupError('active_registration', 'Cannot delete project while an artifact registration is active');
+                }
+                if (db.prepare("SELECT 1 FROM execution_artifacts WHERE project_id = ? AND availability <> 'deleted' LIMIT 1").get(projectId)) {
+                    throw new ExecutionArtifactCleanupError('cleanup_failed', 'Cannot delete project because artifact file cleanup failed');
+                }
+                db.prepare("DELETE FROM execution_artifacts WHERE project_id = ? AND availability = 'deleted'").run(projectId);
+                deleteProjectAttachments(projectId);
+                db.prepare(`
+          UPDATE execution_runs
+          SET parent_run_id = NULL, retry_of_run_id = NULL
+          WHERE project_id = ?
+        `).run(projectId);
+                const deleted = db.prepare('DELETE FROM projects WHERE id = ? AND deletion_claim_id = ?').run(projectId, claimId);
+                if (deleted.changes !== 1) {
+                    throw new ProjectDeletionError(409, 'Project deletion ownership was lost');
+                }
+            })();
+        },
+    }));
+};
+export const resumeInterruptedProjectDeletions = async () => {
+    for (const project of listInterruptedProjectDeletions()) {
+        try {
+            await completeClaimedProjectDeletion(project.id, project.deletion_claim_id);
+        }
+        catch (error) {
+            console.error(`Failed to resume deletion for project ${project.id}`, error);
+        }
+    }
+};
+app.delete('/projects/:id', async (req, res) => {
+    const claimId = randomUUID();
+    try {
+        db.transaction(() => {
+            assertProjectDeletionEligible(req.params.id);
+            const claimed = db.prepare(`
+        UPDATE projects
+        SET deletion_started_at = CURRENT_TIMESTAMP, deletion_claim_id = ?
+        WHERE id = ? AND deletion_started_at IS NULL
+      `).run(claimId, req.params.id);
+            if (claimed.changes !== 1) {
+                throw new ProjectDeletionError(409, 'Project deletion is already in progress');
+            }
+        })();
+    }
+    catch (error) {
+        if (error instanceof ProjectDeletionError) {
+            return res.status(error.status).json({ error: error.message });
+        }
+        throw error;
+    }
+    try {
+        await completeClaimedProjectDeletion(req.params.id, claimId);
+    }
+    catch (error) {
         db.prepare(`
-      UPDATE execution_runs
-      SET parent_run_id = NULL, retry_of_run_id = NULL
-      WHERE project_id = ?
-    `).run(req.params.id);
-        db.prepare('DELETE FROM projects WHERE id = ?').run(req.params.id);
-    })();
+      UPDATE projects
+      SET deletion_started_at = NULL, deletion_claim_id = NULL, deletion_cleanup_started_at = NULL
+      WHERE id = ? AND deletion_claim_id = ? AND deletion_cleanup_started_at IS NULL
+    `).run(req.params.id, claimId);
+        if (error instanceof ExecutionArtifactCleanupError) {
+            return res.status(409).json({ error: error.message });
+        }
+        if (error instanceof ProjectDeletionError) {
+            return res.status(error.status).json({ error: error.message });
+        }
+        throw error;
+    }
     res.status(204).end();
 });
 app.post('/tickets', (req, res) => {
@@ -7612,22 +7750,7 @@ app.get('/projects/:id/execution-timeline/:runId', (req, res) => {
     if (scope.scopeEntity && !links.some(link => link.entity_type === scope.scopeEntity?.type && link.entity_id === scope.scopeEntity.id)) {
         return res.status(404).json({ error: 'Execution run not found' });
     }
-    const testDetails = run.kind === 'test' ? getTestExecutionDetails(project.id, run.id) : null;
-    const retryEligibility = getExecutionRetryEligibility(project.id, run.id);
-    const details = run.kind === 'test'
-        ? serializeTestExecutionDetail(testDetails, retryEligibility)
-        : 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 };
-    return res.json({
-        run,
-        links,
-        events: listLifecycleEvents(project.id, run.id).map(toCanonicalLifecycleEvent),
-        details,
-        agent_runs: listAgentRunsForExecutionRun(run.id),
-    });
+    return res.json(buildExecutionRunDetailResponse(project, run));
 });
 app.get('/projects/:id/execution-runs/:runId', (req, res) => {
     const project = getProjectById(req.params.id);
@@ -7638,19 +7761,44 @@ app.get('/projects/:id/execution-runs/:runId', (req, res) => {
     if (!run) {
         return res.status(404).json({ error: 'Execution run not found' });
     }
-    const testDetails = run.kind === 'test' ? getTestExecutionDetails(project.id, run.id) : null;
-    const retryEligibility = getExecutionRetryEligibility(project.id, run.id);
-    const details = run.kind === 'test'
-        ? serializeTestExecutionDetail(testDetails, retryEligibility)
-        : 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 links = listExecutionRunLinks(project.id, run.id);
-    const agentRuns = listAgentRunsForExecutionRun(run.id);
-    const events = listLifecycleEvents(project.id, run.id).map(toCanonicalLifecycleEvent);
-    return res.json({ run, links, events, details, agent_runs: agentRuns });
+    return res.json(buildExecutionRunDetailResponse(project, run));
+});
+app.get('/projects/:id/execution-runs/:runId/artifacts/:artifactId', async (req, res) => {
+    const project = getProjectById(req.params.id);
+    if (!project || !getExecutionRun(project.id, req.params.runId)) {
+        return res.status(404).json({ error: 'Not found' });
+    }
+    const retrieval = await openExecutionArtifactForRetrieval(project.id, req.params.runId, req.params.artifactId, io);
+    if (retrieval.kind === 'not_found') {
+        return res.status(404).json({ error: 'Not found' });
+    }
+    if (retrieval.kind === 'unavailable') {
+        return res.status(410).json({ error: 'Artifact unavailable' });
+    }
+    if (retrieval.kind === 'external') {
+        res.setHeader('Referrer-Policy', 'no-referrer');
+        res.setHeader('Location', retrieval.url);
+        return res.status(303).end();
+    }
+    res.setHeader('Content-Type', retrieval.mediaType);
+    res.setHeader('Content-Length', retrieval.byteSize);
+    res.setHeader('Content-Disposition', `attachment; filename="${retrieval.filename}"`);
+    res.setHeader('X-Content-Type-Options', 'nosniff');
+    const stream = retrieval.handle.createReadStream({ autoClose: true });
+    stream.on('error', () => {
+        console.error('Execution artifact stream failed', {
+            project_id: project.id,
+            execution_run_id: req.params.runId,
+            artifact_id: req.params.artifactId,
+        });
+        res.destroy();
+    });
+    res.on('close', () => {
+        if (!stream.destroyed) {
+            stream.destroy();
+        }
+    });
+    return stream.pipe(res);
 });
 app.post('/projects/:id/execution-runs/:runId/retry', async (req, res) => {
     const project = getProjectById(req.params.id);
@@ -7991,12 +8139,21 @@ if (clientBuild) {
         });
     });
 }
+app.use((error, _req, res, next) => {
+    if (error instanceof Error && error.message.includes('Project is being deleted')) {
+        res.status(409).json({ error: 'Project deletion is in progress' });
+        return;
+    }
+    next(error);
+});
 io.on('connection', (socket) => {
     console.log('client connected:', socket.id);
     registerTerminalSocketHandlers(io, socket);
 });
 const DEFAULT_PORT = 3001;
 let executionEvidenceCleanupTimer = null;
+let executionArtifactCleanupTimer = null;
+let executionArtifactCleanupPromise = null;
 let startServerPromise = null;
 let shutdownServerPromise = null;
 let shutdownRequested = false;
@@ -8023,6 +8180,41 @@ const startExecutionEvidenceCleanupTimer = () => {
     executionEvidenceCleanupTimer = setInterval(runExecutionEvidenceCleanup, EXECUTION_EVIDENCE_CLEANUP_INTERVAL_MS);
     executionEvidenceCleanupTimer.unref();
 };
+export const runExecutionArtifactCleanup = (mode) => {
+    if (executionArtifactCleanupPromise) {
+        return executionArtifactCleanupPromise;
+    }
+    executionArtifactCleanupPromise = (async () => {
+        if (mode === 'periodic') {
+            await resumeInterruptedProjectDeletions();
+        }
+        await runExclusiveExecutionArtifactMaintenance(async () => {
+            const intents = await reconcileExecutionArtifactRegistrationIntents({ mode });
+            const artifacts = await reconcileFinalizedExecutionArtifacts({ emitter: io });
+            const failures = intents.failed.length + artifacts.failed.length;
+            if (failures > 0) {
+                console.error(`Artifact reconciliation retained ${failures} item(s) for retry`);
+            }
+            if (intents.deleted + artifacts.deleted > 0) {
+                console.log(`Reconciled ${intents.deleted} artifact registration intent(s) and expired ${artifacts.deleted} artifact(s)`);
+            }
+        });
+    })().catch(error => {
+        console.error('Failed to reconcile execution artifacts', error);
+    }).finally(() => {
+        executionArtifactCleanupPromise = null;
+    });
+    return executionArtifactCleanupPromise;
+};
+const startExecutionArtifactCleanupTimer = () => {
+    if (executionArtifactCleanupTimer) {
+        return;
+    }
+    executionArtifactCleanupTimer = setInterval(() => {
+        void runExecutionArtifactCleanup('periodic');
+    }, resolveArtifactCleanupIntervalMs());
+    executionArtifactCleanupTimer.unref();
+};
 const resolveStartResult = (host) => {
     const address = httpServer.address();
     const { displayHost } = resolveDesktopServerHost(host);
@@ -8050,8 +8242,12 @@ export const startServer = ({ port = DEFAULT_PORT, host } = {}) => {
         return startServerPromise;
     }
     startServerPromise = (async () => {
+        releaseInterruptedProjectDeletionClaims();
+        await resumeInterruptedProjectDeletions();
         await reconcileStartupExecutions({ getGitHubConfig: getGitHubConnectionConfig });
         throwIfShutdownRequested();
+        await runExecutionArtifactCleanup('startup');
+        throwIfShutdownRequested();
         startGitHubActionsPolling(io);
         try {
             throwIfShutdownRequested();
@@ -8077,6 +8273,7 @@ export const startServer = ({ port = DEFAULT_PORT, host } = {}) => {
         runExecutionEvidenceCleanup();
         throwIfShutdownRequested();
         startExecutionEvidenceCleanupTimer();
+        startExecutionArtifactCleanupTimer();
         return resolveStartResult(host);
     })().finally(() => {
         startServerPromise = null;
@@ -8098,6 +8295,11 @@ export const shutdownServer = async (signal) => {
             clearInterval(executionEvidenceCleanupTimer);
             executionEvidenceCleanupTimer = null;
         }
+        if (executionArtifactCleanupTimer) {
+            clearInterval(executionArtifactCleanupTimer);
+            executionArtifactCleanupTimer = null;
+        }
+        await executionArtifactCleanupPromise;
         await stopMobileAccess();
         await clearAllTestSessions(io);
         await clearAllGitHubActionsSessions();