@in-the-loop-labs/pair-review 3.3.3 → 3.3.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@in-the-loop-labs/pair-review",
-  "version": "3.3.3",
+  "version": "3.3.5",
   "description": "Your AI-powered code review partner - Close the feedback loop with AI coding agents",
   "main": "src/server.js",
   "bin": {

package/plugin/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "pair-review",
-  "version": "3.3.3",
+  "version": "3.3.5",
   "description": "pair-review app integration — Open PRs and local changes in the pair-review web UI, run server-side AI analysis, and address review feedback. Requires the pair-review MCP server.",
   "author": {
     "name": "in-the-loop-labs",

package/plugin-code-critic/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "code-critic",
-  "version": "3.3.3",
+  "version": "3.3.5",
   "description": "AI-powered code review analysis — Run three-level AI analysis and implement-review-fix loops directly in your coding agent. Works standalone, no server required.",
   "author": {
     "name": "in-the-loop-labs",

package/public/css/pr.css

@@ -1075,6 +1075,25 @@
   font-style: italic;
 }
 
+.file-viewed-icon-wrapper {
+  display: inline-flex;
+  align-items: center;
+  flex-shrink: 0;
+  margin-right: 4px;
+}
+
+.file-viewed-icon {
+  color: var(--color-fg-muted, #8b949e);
+  vertical-align: middle;
+  flex-shrink: 0;
+}
+
+.file-item.viewed .file-name,
+.file-item.viewed.context-file-item .file-name {
+  color: var(--color-fg-muted, #8b949e);
+  font-style: italic;
+}
+
 /* Tree view styles */
 .tree-item {
   user-select: none;
@@ -8003,11 +8022,15 @@ body.resizing * {
   padding: 0 2px;
 }
 
-.analysis-history-level.level-on {
+.analysis-history-level.level-success {
   color: var(--color-success, #22c55e);
 }
 
-.analysis-history-level.level-off {
+.analysis-history-level.level-failed {
+  color: var(--color-danger, #d1242f);
+}
+
+.analysis-history-level.level-skipped {
   color: var(--color-text-muted);
   opacity: 0.5;
 }

package/public/js/components/TimeoutSelect.js

@@ -39,6 +39,10 @@ class TimeoutSelect {
     { value: '1800000', label: '30m' },
     { value: '2700000', label: '45m' },
     { value: '3600000', label: '60m' },
+    { value: '4500000', label: '75m' },
+    { value: '5400000', label: '90m' },
+    { value: '6300000', label: '105m' },
+    { value: '7200000', label: '120m' },
   ];
 
   /**

package/public/js/local.js

@@ -1508,12 +1508,13 @@ class LocalManager {
         fileCount: sortedFiles.length
       });
 
-      // Update file list sidebar
-      manager.updateFileList(sortedFiles);
-
       // Load viewed state before rendering so files can start collapsed
+      // and so the sidebar viewed indicator renders on first paint
       await manager.loadViewedState();
 
+      // Update file list sidebar
+      manager.updateFileList(sortedFiles);
+
       // Render diff
       manager.renderDiff({ changed_files: sortedFiles });
 

package/public/js/modules/analysis-history.js

@@ -502,7 +502,7 @@ class AnalysisHistoryManager {
     `;
 
     // Level indicators in preview
-    if (run.levels_config) {
+    if (run.levels_config || run.level_outcomes) {
       html += `
       <div class="analysis-preview-row">
         <span class="analysis-preview-label">Levels</span>
@@ -883,31 +883,56 @@ class AnalysisHistoryManager {
   }
 
   /**
-   * Render level indicators (L1/L2/L3) based on levels_config.
-   * @param {Object} run - Analysis run object with optional levels_config
+   * Render level indicators (L1/L2/L3/C) with tri-state outcomes
+   * (success / failed / skipped).
+   *
+   * Prefers persisted `level_outcomes` when available. Falls back to
+   * `levels_config` for legacy runs — in which case enabled levels render
+   * as success, disabled as skipped, and the consolidation slot is omitted
+   * (we have no historical data for it).
+   *
+   * @param {Object} run - Analysis run object
    * @returns {string} HTML string for level indicators
    */
   renderLevelIndicators(run) {
-    const levelsConfig = run.levels_config;
-    if (!levelsConfig) return '';
-
-    // levels_config can be:
-    // - An array like [1, 2] (voice-centric: enabled levels)
-    // - An object like { level1: true, level2: true, level3: false } (advanced)
-    const levels = [1, 2, 3];
-    const indicators = levels.map(level => {
-      let enabled;
-      if (Array.isArray(levelsConfig)) {
-        enabled = levelsConfig.includes(level);
-      } else {
-        const key = `level${level}`;
-        enabled = levelsConfig[key] !== false;
+    const outcomes = run.level_outcomes;
+    const config = run.levels_config;
+
+    const slots = [];
+    const addSlot = (label, outcome) => {
+      if (outcome) slots.push({ label, outcome });
+    };
+
+    if (outcomes) {
+      addSlot('L1', outcomes.level1);
+      addSlot('L2', outcomes.level2);
+      addSlot('L3', outcomes.level3);
+      addSlot('C', outcomes.consolidation);
+    } else if (config) {
+      // Legacy fallback: derive from config only. No failure state, no C slot.
+      // levels_config can be an array (voice-centric: enabled levels) or an
+      // object (advanced: per-level boolean).
+      for (const level of [1, 2, 3]) {
+        const enabled = Array.isArray(config)
+          ? config.includes(level)
+          : config[`level${level}`] !== false;
+        addSlot(`L${level}`, enabled ? 'success' : 'skipped');
       }
-      const cls = enabled ? 'level-on' : 'level-off';
-      const icon = enabled ? '\u2713' : '\u2717';
-      return `<span class="analysis-history-level ${cls}">L${level}${icon}</span>`;
-    });
-    return `<span class="analysis-history-levels">${indicators.join('')}</span>`;
+    } else {
+      return '';
+    }
+
+    const icon = { success: '\u2713', failed: '\u2717', skipped: '\u00B7' };
+    const cls = {
+      success: 'level-success',
+      failed: 'level-failed',
+      skipped: 'level-skipped'
+    };
+
+    const html = slots
+      .map(s => `<span class="analysis-history-level ${cls[s.outcome] || ''}">${s.label}${icon[s.outcome] || ''}</span>`)
+      .join('');
+    return `<span class="analysis-history-levels">${html}</span>`;
   }
 
   /**

package/public/js/pr.js

@@ -799,12 +799,13 @@ class PRManager {
           window.aiPanel.setFileOrder(this.canonicalFileOrder);
         }
 
-        // Update sidebar with file list
-        this.updateFileList(sortedFiles);
-
         // Load viewed state before rendering so files can start collapsed
+        // and so the sidebar viewed indicator renders on first paint
         await this.loadViewedState();
 
+        // Update sidebar with file list
+        this.updateFileList(sortedFiles);
+
         // Render diff using the existing renderDiff method
         this.renderDiff({ changed_files: sortedFiles });
 
@@ -2083,10 +2084,60 @@ class PRManager {
       }
     }
 
+    // Update sidebar file row to reflect viewed state
+    this.updateFileItemViewedState(filePath, isViewed);
+
     // Persist viewed state
     this.saveViewedState();
   }
 
+  /**
+   * Build the eye-slash icon wrapper element used to mark a sidebar
+   * file row as viewed. Shared by the initial render and in-place updates
+   * so the markup and attributes stay in sync.
+   * @returns {HTMLSpanElement}
+   */
+  _createViewedIcon() {
+    const viewedIcon = document.createElement('span');
+    viewedIcon.className = 'file-viewed-icon-wrapper';
+    viewedIcon.title = 'Marked as viewed';
+    viewedIcon.setAttribute('aria-label', 'Marked as viewed');
+    viewedIcon.innerHTML = '<svg class="file-viewed-icon" viewBox="0 0 16 16" width="14" height="14" fill="currentColor" aria-hidden="true"><path fill-rule="evenodd" d="M1.22 1.22a.75.75 0 0 1 1.06 0l12.5 12.5a.75.75 0 1 1-1.06 1.06l-1.82-1.82A7.44 7.44 0 0 1 8 14c-2.12 0-3.88-.81-5.26-1.94A13.13 13.13 0 0 1 .75 9.44a7.34 7.34 0 0 1-.51-.66.77.77 0 0 1 0-.84 12.52 12.52 0 0 1 .55-.72c.28-.34.66-.79 1.13-1.26L1.22 2.28a.75.75 0 0 1 0-1.06ZM4.5 5.56 6 7.06a2.5 2.5 0 0 0 2.94 2.94l1.22 1.22a4 4 0 0 1-5.66-5.66ZM8 3.5a4 4 0 0 1 3.98 4.46l3.04 3.04.1-.12c.36-.44.65-.87.87-1.22a.77.77 0 0 0 0-.84 13.13 13.13 0 0 0-2-2.62A7.44 7.44 0 0 0 8 2a7.4 7.4 0 0 0-2.3.36L7.1 3.78c.3-.18.62-.28.9-.28Z"/></svg>';
+    return viewedIcon;
+  }
+
+  /**
+   * Update the sidebar file row to reflect the viewed state.
+   * Adds/removes the .viewed class and injects/removes the eye-slash icon
+   * without re-rendering the whole file list.
+   * @param {string} filePath - Path of the file
+   * @param {boolean} isViewed - Whether the file is now viewed
+   */
+  updateFileItemViewedState(filePath, isViewed) {
+    const items = document.querySelectorAll('.file-item');
+    let item = null;
+    for (const candidate of items) {
+      if (candidate.dataset.path === filePath) {
+        item = candidate;
+        break;
+      }
+    }
+    if (!item) return;
+
+    const existingIcon = item.querySelector('.file-viewed-icon-wrapper');
+
+    if (isViewed) {
+      item.classList.add('viewed');
+      if (!existingIcon) {
+        const viewedIcon = this._createViewedIcon();
+        item.insertBefore(viewedIcon, item.firstChild);
+      }
+    } else {
+      item.classList.remove('viewed');
+      if (existingIcon) existingIcon.remove();
+    }
+  }
+
   /**
    * Save viewed files state to storage
    * Persists per PR for later retrieval
@@ -2357,7 +2408,9 @@ class PRManager {
           type: 'context',
           oldNumber: lineNumber,
           newNumber: lineNumber + lineOffset,  // Apply offset for correct right-side line number
-          content: content || ''
+          // Expanded rows should follow the same contract as parsed diff context
+          // lines so DiffRenderer strips the synthetic diff marker, not real indent.
+          content: ' ' + (content || '')
         };
 
         const lineRow = this.renderDiffLine(fragment, lineData, fileName, null);
@@ -2469,7 +2522,9 @@ class PRManager {
           type: 'context',
           oldNumber: lineNumber,
           newNumber: lineNumber + lineOffset,  // Apply offset for correct right-side line number
-          content: content || ''
+          // Expanded rows should follow the same contract as parsed diff context
+          // lines so DiffRenderer strips the synthetic diff marker, not real indent.
+          content: ' ' + (content || '')
         };
 
         const lineRow = this.renderDiffLine(fragment, lineData, fileName, null);
@@ -4191,6 +4246,11 @@ class PRManager {
 
     if (file.generated) item.classList.add('generated');
     if (file.contextFile) item.classList.add('context-file-item');
+    if (this.viewedFiles && this.viewedFiles.has(file.fullPath)) {
+      item.classList.add('viewed');
+      const viewedIcon = this._createViewedIcon();
+      item.insertBefore(viewedIcon, item.firstChild);
+    }
     if (file.renamed && file.renamedFrom) {
       item.title = `Renamed from: ${file.renamedFrom}`;
       const renameIcon = document.createElement('span');

package/src/ai/analyzer.js

@@ -550,13 +550,22 @@ class Analyzer {
           throw new CancellationError('Analysis was cancelled');
         }
 
+        // Build per-level outcome record for persistence and UI
+        const levelOutcomes = {
+          level1: levelResults.level1.status,
+          level2: levelResults.level2.status,
+          level3: levelResults.level3.status,
+          consolidation: 'success'
+        };
+
         // Update analysis_run record with completion data
         try {
           await analysisRunRepo.update(runId, {
             status: 'completed',
             summary: orchestrationResult.summary,
             totalSuggestions: finalSuggestions.length,
-            filesAnalyzed: validFiles.length
+            filesAnalyzed: validFiles.length,
+            levelOutcomes
           });
           logger.info(`${logPrefix}Updated analysis_run record to completed: ${finalSuggestions.length} suggestions, ${validFiles.length} files`);
         } catch (updateError) {
@@ -569,6 +578,7 @@ class Analyzer {
           runId,
           suggestions: finalSuggestions,
           levelResults,
+          levelOutcomes,
           summary: orchestrationResult.summary
         };
 
@@ -603,6 +613,14 @@ class Analyzer {
           throw new CancellationError('Analysis was cancelled');
         }
 
+        // Build per-level outcome record (consolidation failed, fallback path used)
+        const levelOutcomes = {
+          level1: levelResults.level1.status,
+          level2: levelResults.level2.status,
+          level3: levelResults.level3.status,
+          consolidation: 'failed'
+        };
+
         // Update analysis_run record with completion data (even though orchestration failed)
         const fallbackSummary = `Analysis complete (consolidation failed): ${finalFallbackSuggestions.length} suggestions`;
         try {
@@ -610,7 +628,8 @@ class Analyzer {
             status: 'completed',
             summary: fallbackSummary,
             totalSuggestions: finalFallbackSuggestions.length,
-            filesAnalyzed: validFiles.length
+            filesAnalyzed: validFiles.length,
+            levelOutcomes
           });
           logger.info(`${logPrefix}Updated analysis_run record to completed (fallback): ${finalFallbackSuggestions.length} suggestions`);
         } catch (updateError) {
@@ -621,6 +640,7 @@ class Analyzer {
           runId,
           suggestions: finalFallbackSuggestions,
           levelResults,
+          levelOutcomes,
           summary: fallbackSummary,
           orchestrationFailed: true
         };
@@ -2973,7 +2993,8 @@ File-level suggestions should NOT have a line number. They apply to the entire f
             status: 'completed',
             summary: result.summary,
             totalSuggestions: finalSuggestions.length,
-            filesAnalyzed: validFiles.length
+            filesAnalyzed: validFiles.length,
+            levelOutcomes: { consolidation: 'skipped' }
           });
         } catch (err) {
           logger.warn(`[ReviewerCouncil] Failed to update parent run: ${err.message}`);
@@ -2982,7 +3003,8 @@ File-level suggestions should NOT have a line number. They apply to the entire f
         return {
           runId: parentRunId,
           suggestions: finalSuggestions,
-          summary: result.summary || `Review council complete: ${finalSuggestions.length} suggestions`
+          summary: result.summary || `Review council complete: ${finalSuggestions.length} suggestions`,
+          levelOutcomes: { consolidation: 'skipped' }
         };
       }
 
@@ -3012,7 +3034,8 @@ File-level suggestions should NOT have a line number. They apply to the entire f
       return {
         runId: parentRunId,
         suggestions: result.suggestions,
-        summary: result.summary || `Review council complete: ${result.suggestions?.length || 0} suggestions`
+        summary: result.summary || `Review council complete: ${result.suggestions?.length || 0} suggestions`,
+        levelOutcomes: result.levelOutcomes
       };
     }
 
@@ -3085,7 +3108,8 @@ File-level suggestions should NOT have a line number. They apply to the entire f
             await analysisRunRepo.update(childRunId, {
               status: 'completed',
               summary: result.summary,
-              totalSuggestions: validatedSuggestions.length
+              totalSuggestions: validatedSuggestions.length,
+              levelOutcomes: { consolidation: 'skipped' }
             });
           } catch (err) {
             logger.warn(`[ReviewerCouncil] Failed to update child run ${childRunId}: ${err.message}`);
@@ -3217,7 +3241,8 @@ File-level suggestions should NOT have a line number. They apply to the entire f
           status: 'completed',
           summary: singleResult.summary,
           totalSuggestions: finalSuggestions.length,
-          filesAnalyzed: validFiles.length
+          filesAnalyzed: validFiles.length,
+          levelOutcomes: { consolidation: 'skipped' }
         });
       } catch (err) {
         logger.warn(`[ReviewerCouncil] Failed to update parent run: ${err.message}`);
@@ -3226,7 +3251,8 @@ File-level suggestions should NOT have a line number. They apply to the entire f
       return {
         runId: parentRunId,
         suggestions: finalSuggestions,
-        summary: singleResult.summary || `Review council complete: ${finalSuggestions.length} suggestions`
+        summary: singleResult.summary || `Review council complete: ${finalSuggestions.length} suggestions`,
+        levelOutcomes: { consolidation: 'skipped' }
       };
     }
 
@@ -3295,7 +3321,8 @@ File-level suggestions should NOT have a line number. They apply to the entire f
           status: 'completed',
           summary,
           totalSuggestions: finalSuggestions.length,
-          filesAnalyzed: validFiles.length
+          filesAnalyzed: validFiles.length,
+          levelOutcomes: { consolidation: 'success' }
         });
       } catch (err) {
         logger.warn(`[ReviewerCouncil] Failed to update parent run: ${err.message}`);
@@ -3306,7 +3333,8 @@ File-level suggestions should NOT have a line number. They apply to the entire f
       return {
         runId: parentRunId,
         suggestions: finalSuggestions,
-        summary
+        summary,
+        levelOutcomes: { consolidation: 'success' }
       };
     } catch (error) {
       logger.error(`[ReviewerCouncil] Cross-reviewer consolidation failed: ${error.message}`);
@@ -3324,7 +3352,8 @@ File-level suggestions should NOT have a line number. They apply to the entire f
           status: 'completed',
           summary: fallbackSummary,
           totalSuggestions: fallbackSuggestions.length,
-          filesAnalyzed: validFiles.length
+          filesAnalyzed: validFiles.length,
+          levelOutcomes: { consolidation: 'failed' }
         });
       } catch (err) {
         logger.warn(`[ReviewerCouncil] Failed to update parent run: ${err.message}`);
@@ -3334,7 +3363,8 @@ File-level suggestions should NOT have a line number. They apply to the entire f
         runId: parentRunId,
         suggestions: fallbackSuggestions,
         summary: fallbackSummary,
-        orchestrationFailed: true
+        orchestrationFailed: true,
+        levelOutcomes: { consolidation: 'failed' }
       };
     }
   }
@@ -3523,7 +3553,8 @@ File-level suggestions should NOT have a line number. They apply to the entire f
       return {
         runId,
         suggestions: finalSuggestions,
-        summary: bestVoiceSummary || `Council analysis complete: ${finalSuggestions.length} suggestions from single reviewer`
+        summary: bestVoiceSummary || `Council analysis complete: ${finalSuggestions.length} suggestions from single reviewer`,
+        levelOutcomes: { consolidation: 'skipped' }
       };
     }
 
@@ -3636,7 +3667,8 @@ File-level suggestions should NOT have a line number. They apply to the entire f
       return {
         runId,
         suggestions: finalSuggestions,
-        summary: orchestrationResult.summary
+        summary: orchestrationResult.summary,
+        levelOutcomes: { consolidation: 'success' }
       };
     } catch (error) {
       logger.error(`[Council] Cross-level consolidation failed: ${error.message}`);
@@ -3657,7 +3689,8 @@ File-level suggestions should NOT have a line number. They apply to the entire f
         runId,
         suggestions: finalFallback,
         summary: `Council analysis complete (consolidation failed): ${finalFallback.length} suggestions`,
-        orchestrationFailed: true
+        orchestrationFailed: true,
+        levelOutcomes: { consolidation: 'failed' }
       };
     }
   }

package/src/ai/claude-provider.js

@@ -20,6 +20,30 @@ const BIN_DIR = path.join(__dirname, '..', '..', 'bin');
  * Claude model definitions with tier mappings
  */
 const CLAUDE_MODELS = [
+  {
+    id: 'opus-4.7-xhigh',
+    cli_model: 'claude-opus-4-7',
+    env: { CLAUDE_CODE_EFFORT_LEVEL: 'xhigh' },
+    name: 'Opus 4.7 xhigh',
+    tier: 'thorough',
+    tagline: 'Latest Gen',
+    description: 'Opus 4.7 (latest) with extra-high effort',
+    badge: 'Latest',
+    badgeClass: 'badge-power'
+  },
+  {
+    id: 'opus',
+    aliases: ['opus-4.6-high'],
+    cli_model: 'claude-opus-4-6',
+    env: { CLAUDE_CODE_EFFORT_LEVEL: 'high' },
+    name: 'Opus 4.6 High',
+    tier: 'thorough',
+    tagline: 'Maximum Depth',
+    description: 'Opus 4.6 with high effort — deepest analysis',
+    badge: 'Most Thorough',
+    badgeClass: 'badge-power',
+    default: true
+  },
   {
     id: 'haiku',
     name: 'Haiku 4.6',
@@ -41,7 +65,7 @@ const CLAUDE_MODELS = [
   },
   {
     id: 'opus-4.6-low',
-    cli_model: 'opus',
+    cli_model: 'claude-opus-4-6',
     env: { CLAUDE_CODE_EFFORT_LEVEL: 'low' },
     name: 'Opus 4.6 Low',
     tier: 'balanced',
@@ -52,7 +76,7 @@ const CLAUDE_MODELS = [
   },
   {
     id: 'opus-4.6-medium',
-    cli_model: 'opus',
+    cli_model: 'claude-opus-4-6',
     env: { CLAUDE_CODE_EFFORT_LEVEL: 'medium' },
     name: 'Opus 4.6 Medium',
     tier: 'balanced',
@@ -61,21 +85,9 @@ const CLAUDE_MODELS = [
     badge: 'Thorough',
     badgeClass: 'badge-power'
   },
-  {
-    id: 'opus',
-    aliases: ['opus-4.6-high'],
-    env: { CLAUDE_CODE_EFFORT_LEVEL: 'high' },
-    name: 'Opus 4.6 High',
-    tier: 'thorough',
-    tagline: 'Maximum Depth',
-    description: 'Opus 4.6 with high effort — deepest analysis',
-    badge: 'Most Thorough',
-    badgeClass: 'badge-power',
-    default: true
-  },
   {
     id: 'opus-4.6-1m',
-    cli_model: 'opus[1m]',
+    cli_model: 'claude-opus-4-6[1m]',
     name: 'Opus 4.6 1M',
     tier: 'balanced',
     tagline: 'Extended Context',

package/src/database.js

@@ -21,7 +21,7 @@ function getDbPath() {
 /**
  * Current schema version - increment this when adding new migrations
  */
-const CURRENT_SCHEMA_VERSION = 43;
+const CURRENT_SCHEMA_VERSION = 44;
 
 /**
  * Database schema SQL statements
@@ -184,6 +184,7 @@ const SCHEMA_SQL = {
       parent_run_id TEXT,
       config_type TEXT DEFAULT 'single',
       levels_config TEXT,
+      level_outcomes TEXT,
       scope_start TEXT,
       scope_end TEXT,
       FOREIGN KEY (review_id) REFERENCES reviews(id) ON DELETE CASCADE
@@ -1866,6 +1867,25 @@ const MIGRATIONS = {
     };
     addColumnIfNotExists('repo_settings', 'load_skills', 'INTEGER');
     console.log('Migration to schema version 43 complete');
+  },
+
+  44: (db) => {
+    console.log('Running migration to schema version 44: Add level_outcomes to analysis_runs...');
+    const hasLevelOutcomes = columnExists(db, 'analysis_runs', 'level_outcomes');
+    if (!hasLevelOutcomes) {
+      try {
+        db.prepare(`ALTER TABLE analysis_runs ADD COLUMN level_outcomes TEXT`).run();
+        console.log('  Added level_outcomes column to analysis_runs');
+      } catch (error) {
+        if (!error.message.includes('duplicate column name')) {
+          throw error;
+        }
+        console.log('  Column level_outcomes already exists (race condition)');
+      }
+    } else {
+      console.log('  Column level_outcomes already exists');
+    }
+    console.log('Migration to schema version 44 complete');
   }
 };
 
@@ -4348,6 +4368,11 @@ class AnalysisRunRepository {
       params.push(updates.diff);
     }
 
+    if (updates.levelOutcomes !== undefined) {
+      setClauses.push('level_outcomes = ?');
+      params.push(updates.levelOutcomes === null ? null : JSON.stringify(updates.levelOutcomes));
+    }
+
     if (setClauses.length === 0) {
       return false;
     }
@@ -4380,7 +4405,7 @@ class AnalysisRunRepository {
     const columns = [
       'id', 'review_id', 'provider', 'model', 'tier', 'custom_instructions', 'global_instructions', 'repo_instructions', 'request_instructions',
       'head_sha', 'summary', 'status', 'total_suggestions', 'files_analyzed', 'started_at', 'completed_at',
-      'parent_run_id', 'config_type', 'levels_config'
+      'parent_run_id', 'config_type', 'levels_config', 'level_outcomes'
     ];
     if (includeDiff) {
       columns.splice(columns.indexOf('head_sha') + 1, 0, 'diff'); // Insert diff after head_sha
@@ -4408,7 +4433,7 @@ class AnalysisRunRepository {
     const columns = [
       'id', 'review_id', 'provider', 'model', 'tier', 'custom_instructions', 'global_instructions', 'repo_instructions', 'request_instructions',
       'head_sha', 'summary', 'status', 'total_suggestions', 'files_analyzed', 'started_at', 'completed_at',
-      'parent_run_id', 'config_type', 'levels_config'
+      'parent_run_id', 'config_type', 'levels_config', 'level_outcomes'
     ];
     if (includeDiff) {
       columns.splice(columns.indexOf('head_sha') + 1, 0, 'diff'); // Insert diff after head_sha
@@ -4446,7 +4471,7 @@ class AnalysisRunRepository {
     const columns = [
       'id', 'review_id', 'provider', 'model', 'tier', 'custom_instructions', 'global_instructions', 'repo_instructions', 'request_instructions',
       'head_sha', 'summary', 'status', 'total_suggestions', 'files_analyzed', 'started_at', 'completed_at',
-      'parent_run_id', 'config_type', 'levels_config'
+      'parent_run_id', 'config_type', 'levels_config', 'level_outcomes'
     ];
     if (includeDiff) {
       columns.splice(columns.indexOf('head_sha') + 1, 0, 'diff'); // Insert diff after head_sha
@@ -4473,7 +4498,7 @@ class AnalysisRunRepository {
     return query(this.db, `
       SELECT id, review_id, provider, model, tier, custom_instructions, global_instructions, repo_instructions, request_instructions,
              head_sha, summary, status, total_suggestions, files_analyzed, started_at, completed_at,
-             parent_run_id, config_type, levels_config
+             parent_run_id, config_type, levels_config, level_outcomes
       FROM analysis_runs
       WHERE parent_run_id = ?
       ORDER BY started_at ASC
@@ -4971,5 +4996,6 @@ module.exports = {
   generateWorktreeId,
   migrateExistingWorktrees,
   // Exported for testing only
-  _MIGRATIONS: MIGRATIONS
+  _MIGRATIONS: MIGRATIONS,
+  MIGRATIONS
 };