@in-the-loop-labs/pair-review 3.1.2 → 3.1.4

package/public/js/modules/comment-minimizer.js

@@ -7,7 +7,10 @@
  * are injected on the right edge of each diff line that has comments, showing
  * a person icon (user comments) or sparkles icon (AI suggestions).
  *
- * Clicking an indicator toggles visibility of that line's comments only.
+ * File-level comments (.file-comment-card inside .file-comments-zone) are also
+ * hidden, with an indicator button injected into the file header bar.
+ *
+ * Clicking an indicator toggles visibility of that line's or file's comments.
  */
 
 class CommentMinimizer {
@@ -24,6 +27,8 @@ class CommentMinimizer {
     this._active = false;
     // Track which diff lines have been expanded by the user (Set of diff row elements)
     this._expandedLines = new Set();
+    // Track which file-comments-zones have been expanded (Set of zone elements)
+    this._expandedFiles = new Set();
   }
 
   /** @returns {boolean} Whether minimize mode is active */
@@ -38,6 +43,7 @@ class CommentMinimizer {
   setMinimized(minimized) {
     this._active = minimized;
     this._expandedLines.clear();
+    this._expandedFiles.clear();
 
     const diffContainer = document.getElementById('diff-container');
     if (!diffContainer) return;
@@ -50,6 +56,8 @@ class CommentMinimizer {
       this._removeAllIndicators();
       // Remove any per-line expansion overrides
       document.querySelectorAll('.comment-expanded').forEach(el => el.classList.remove('comment-expanded'));
+      // Remove any per-file expansion overrides
+      document.querySelectorAll('.file-comments-expanded').forEach(el => el.classList.remove('file-comments-expanded'));
     }
   }
 
@@ -103,10 +111,13 @@ class CommentMinimizer {
       lineMap.set(diffRow, entry);
     }
 
-    // Inject indicators
+    // Inject line-level indicators
     for (const [diffRow, info] of lineMap) {
       this._injectIndicator(diffRow, info);
     }
+
+    // Scan file-comments-zones and inject file-header indicators
+    this._refreshFileIndicators();
   }
 
   /**
@@ -266,7 +277,22 @@ class CommentMinimizer {
   expandForElement(element) {
     if (!this._active) return;
 
-    // Find the containing comment/suggestion row
+    // Check if this element is inside a file-comments-zone (file-level comment)
+    const zone = element.closest('.file-comments-zone');
+    if (zone) {
+      if (this._expandedFiles.has(zone)) return; // already expanded
+      this._expandedFiles.add(zone);
+      zone.classList.add('file-comments-expanded');
+      // Update the file-header indicator button
+      const wrapper = zone.closest('.d2h-file-wrapper');
+      const btn = wrapper?.querySelector('.d2h-file-header .file-comment-indicator');
+      if (btn) {
+        btn.classList.add('expanded');
+      }
+      return;
+    }
+
+    // Line-level: find the containing comment/suggestion row
     const commentRow = element.closest('.user-comment-row, .ai-suggestion-row') || element;
     if (!commentRow.classList.contains('user-comment-row') && !commentRow.classList.contains('ai-suggestion-row')) {
       return;
@@ -290,9 +316,130 @@ class CommentMinimizer {
     }
   }
 
-  /** Remove all indicator buttons from the DOM. */
+  // ---------------------------------------------------------------------------
+  // File-level comment indicators
+  // ---------------------------------------------------------------------------
+
+  /**
+   * Scan all file-comments-zones and inject indicator buttons into file headers.
+   */
+  _refreshFileIndicators() {
+    const zones = document.querySelectorAll('.file-comments-zone');
+    for (const zone of zones) {
+      const cards = zone.querySelectorAll('.file-comment-card');
+      if (cards.length === 0) continue;
+
+      // Count comment types
+      const info = { hasUser: false, hasAI: false, hasAdopted: false, userCount: 0, aiCount: 0, adoptedCount: 0 };
+      for (const card of cards) {
+        // Skip collapsed cards (adopted/dismissed originals remain in DOM)
+        if (card.classList.contains('collapsed')) continue;
+
+        if (card.classList.contains('ai-suggestion')) {
+          info.hasAI = true;
+          info.aiCount++;
+        } else if (card.classList.contains('user-comment')) {
+          if (card.classList.contains('adopted-comment')) {
+            info.hasAdopted = true;
+            info.adoptedCount++;
+          } else {
+            info.hasUser = true;
+            info.userCount++;
+          }
+        }
+      }
+
+      if (info.userCount + info.aiCount + info.adoptedCount === 0) continue;
+
+      // Find the file header — zone and header are siblings inside .d2h-file-wrapper
+      const wrapper = zone.closest('.d2h-file-wrapper');
+      const header = wrapper?.querySelector('.d2h-file-header');
+      if (!header) continue;
+
+      this._injectFileIndicator(header, zone, info);
+    }
+  }
+
+  /**
+   * Inject an indicator button into a file header, positioned before the comment button.
+   * @param {HTMLElement} header - The .d2h-file-header element
+   * @param {HTMLElement} zone - The .file-comments-zone element
+   * @param {Object} info - { hasUser, hasAI, hasAdopted, userCount, aiCount, adoptedCount }
+   */
+  _injectFileIndicator(header, zone, info) {
+    // Don't double-inject
+    if (header.querySelector('.file-comment-indicator')) return;
+
+    const btn = document.createElement('button');
+    btn.className = 'file-comment-indicator';
+    btn.type = 'button';
+
+    // Build icon — pick the dominant type icon
+    const icons = [];
+    if (info.hasUser) {
+      icons.push(`<span class="indicator-icon indicator-user">${CommentMinimizer.PERSON_ICON}</span>`);
+    }
+    if (info.hasAdopted) {
+      icons.push(`<span class="indicator-icon indicator-adopted">${CommentMinimizer.AI_COMMENT_ICON}</span>`);
+    }
+    if (info.hasAI) {
+      icons.push(`<span class="indicator-icon indicator-ai">${CommentMinimizer.SPARKLES_ICON}</span>`);
+    }
+
+    const total = info.userCount + info.adoptedCount + info.aiCount;
+    const countBadge = total > 1 ? `<span class="indicator-count">${total}</span>` : '';
+
+    btn.innerHTML = icons.join('') + countBadge;
+
+    const totalLabel = [];
+    if (info.userCount) totalLabel.push(`${info.userCount} file comment${info.userCount !== 1 ? 's' : ''}`);
+    if (info.adoptedCount) totalLabel.push(`${info.adoptedCount} adopted`);
+    if (info.aiCount) totalLabel.push(`${info.aiCount} suggestion${info.aiCount !== 1 ? 's' : ''}`);
+    btn.title = totalLabel.join(', ');
+
+    // Restore expanded state
+    if (this._expandedFiles.has(zone)) {
+      btn.classList.add('expanded');
+    }
+
+    btn.addEventListener('click', (e) => {
+      e.stopPropagation();
+      e.preventDefault();
+      this._toggleFileComments(zone, btn);
+    });
+
+    // Insert before the file-header-comment-btn if present, otherwise append
+    const commentBtn = header.querySelector('.file-header-comment-btn');
+    if (commentBtn) {
+      header.insertBefore(btn, commentBtn);
+    } else {
+      header.appendChild(btn);
+    }
+  }
+
+  /**
+   * Toggle visibility of file-level comments for a specific file.
+   * @param {HTMLElement} zone - The .file-comments-zone element
+   * @param {HTMLElement} btn - The indicator button
+   */
+  _toggleFileComments(zone, btn) {
+    const isExpanded = this._expandedFiles.has(zone);
+
+    if (isExpanded) {
+      this._expandedFiles.delete(zone);
+      btn.classList.remove('expanded');
+      zone.classList.remove('file-comments-expanded');
+    } else {
+      this._expandedFiles.add(zone);
+      btn.classList.add('expanded');
+      zone.classList.add('file-comments-expanded');
+    }
+  }
+
+  /** Remove all indicator buttons (both line-level and file-level) from the DOM. */
   _removeAllIndicators() {
     document.querySelectorAll('.comment-indicator').forEach(btn => btn.remove());
+    document.querySelectorAll('.file-comment-indicator').forEach(btn => btn.remove());
   }
 }
 

package/public/js/modules/file-comment-manager.js

@@ -17,12 +17,15 @@ class FileCommentManager {
       if (chatBtn && window.chatPanel) {
         e.stopPropagation();
         const suggestionCard = chatBtn.closest('.ai-suggestion');
-        const bodyText = suggestionCard?.dataset?.originalBody
-          ? JSON.parse(suggestionCard.dataset.originalBody) : '';
+        const bodyText = suggestionCard?.dataset?.formattedBody
+          ? JSON.parse(suggestionCard.dataset.formattedBody)
+          : suggestionCard?.dataset?.originalBody
+            ? JSON.parse(suggestionCard.dataset.originalBody) : '';
         window.chatPanel.open({
           reviewId: this.prManager?.currentPR?.id,
           suggestionId: chatBtn.dataset.suggestionId,
           suggestionContext: {
+            suggestionId: chatBtn.dataset.suggestionId || null,
             title: chatBtn.dataset.title || '',
             body: bodyText,
             type: suggestionCard?.querySelector('.ai-suggestion-badge')?.dataset?.type || '',
@@ -312,6 +315,16 @@ class FileCommentManager {
       // Update count badge
       this.updateCommentCount(zone);
 
+      // Refresh minimize-mode indicators so file-header counts stay current
+      if (this.prManager?.commentMinimizer) {
+        this.prManager.commentMinimizer.refreshIndicators();
+        // Auto-expand so the new comment stays visible in minimize mode
+        const newCard = zone.querySelector(`.file-comment-card[data-comment-id="${commentData.id}"]`);
+        if (newCard) {
+          this.prManager.commentMinimizer.expandForElement(newCard);
+        }
+      }
+
       // Notify AI Panel if available
       if (window.aiPanel?.addComment) {
         window.aiPanel.addComment(commentData);
@@ -322,6 +335,8 @@ class FileCommentManager {
         this.prManager.updateCommentCount();
       }
 
+      window.chatPanel?.queueUserActionHint(`[User Action: created comment ${result.commentId}]`);
+
     } catch (error) {
       console.error('Error saving file-level comment:', error);
       if (window.toast) {
@@ -432,6 +447,7 @@ class FileCommentManager {
     // Store original markdown body for adopt functionality via extractSuggestionData
     // Use JSON.stringify to preserve newlines and special characters (matches line-level suggestions)
     card.dataset.originalBody = JSON.stringify(suggestion.body || '');
+    card.dataset.formattedBody = JSON.stringify(suggestion.formattedBody || '');
 
     // Store target info on the card for reliable retrieval in getFileAndLineInfo
     // File-level suggestions don't have line numbers, just the file name
@@ -604,6 +620,16 @@ class FileCommentManager {
       this.displayUserComment(zone, commentData);
       this.updateCommentCount(zone);
 
+      // Refresh minimize-mode indicators so file-header counts stay current
+      if (this.prManager?.commentMinimizer) {
+        this.prManager.commentMinimizer.refreshIndicators();
+        // Auto-expand so the new comment stays visible in minimize mode
+        const newCard = zone.querySelector(`.file-comment-card[data-comment-id="${commentData.id}"]`);
+        if (newCard) {
+          this.prManager.commentMinimizer.expandForElement(newCard);
+        }
+      }
+
       // Update parent comment count for Preview button
       if (this.prManager?.updateCommentCount) {
         this.prManager.updateCommentCount();
@@ -619,6 +645,8 @@ class FileCommentManager {
         window.aiPanel.updateFindingStatus(suggestion.id, 'adopted');
       }
 
+      window.chatPanel?.queueUserActionHint(`[User Action: adopted suggestion ${suggestion.id}]`);
+
     } catch (error) {
       console.error('Error adopting suggestion:', error);
       if (window.toast) {
@@ -657,11 +685,18 @@ class FileCommentManager {
 
       this.updateCommentCount(zone);
 
+      // Refresh minimize-mode indicators so file-header counts stay current
+      if (this.prManager?.commentMinimizer) {
+        this.prManager.commentMinimizer.refreshIndicators();
+      }
+
       // Update finding status in AI Panel (mark suggestion as dismissed)
       if (window.aiPanel?.updateFindingStatus) {
         window.aiPanel.updateFindingStatus(suggestionId, 'dismissed');
       }
 
+      window.chatPanel?.queueUserActionHint(`[User Action: dismissed suggestion ${suggestionId}]`);
+
     } catch (error) {
       console.error('Error dismissing suggestion:', error);
       if (window.toast) {
@@ -703,6 +738,13 @@ class FileCommentManager {
       // Update comment count (for consistency with dismissAISuggestion)
       this.updateCommentCount(zone);
 
+      // Refresh minimize-mode indicators so file-header counts stay current
+      if (this.prManager?.commentMinimizer) {
+        this.prManager.commentMinimizer.refreshIndicators();
+      }
+
+      window.chatPanel?.queueUserActionHint(`[User Action: restored suggestion ${suggestionId}]`);
+
     } catch (error) {
       console.error('Error restoring suggestion:', error);
       if (window.toast) {
@@ -848,6 +890,16 @@ class FileCommentManager {
       this.displayUserComment(zone, commentData);
       this.updateCommentCount(zone);
 
+      // Refresh minimize-mode indicators so file-header counts stay current
+      if (this.prManager?.commentMinimizer) {
+        this.prManager.commentMinimizer.refreshIndicators();
+        // Auto-expand so the new comment stays visible in minimize mode
+        const newCard = zone.querySelector(`.file-comment-card[data-comment-id="${commentData.id}"]`);
+        if (newCard) {
+          this.prManager.commentMinimizer.expandForElement(newCard);
+        }
+      }
+
       // Update parent comment count for Preview button
       if (this.prManager?.updateCommentCount) {
         this.prManager.updateCommentCount();
@@ -863,6 +915,8 @@ class FileCommentManager {
         window.aiPanel.updateFindingStatus(suggestion.id, 'adopted');
       }
 
+      window.chatPanel?.queueUserActionHint(`[User Action: adopted suggestion ${suggestion.id}]`);
+
     } catch (error) {
       console.error('Error adopting suggestion with edit:', error);
       if (window.toast) {
@@ -1012,6 +1066,11 @@ class FileCommentManager {
 
       this.updateCommentCount(zone);
 
+      // Refresh minimize-mode indicators so file-header counts stay current
+      if (this.prManager?.commentMinimizer) {
+        this.prManager.commentMinimizer.refreshIndicators();
+      }
+
       // Update parent comment count
       if (this.prManager?.updateCommentCount) {
         this.prManager.updateCommentCount();
@@ -1029,6 +1088,11 @@ class FileCommentManager {
         window.aiPanel.updateFindingStatus(apiResult.dismissedSuggestionId, 'dismissed');
       }
 
+      if (apiResult.dismissedSuggestionId) {
+        window.chatPanel?.queueUserActionHint(`[User Action: dismissed suggestion ${apiResult.dismissedSuggestionId}]`);
+      }
+      window.chatPanel?.queueUserActionHint(`[User Action: dismissed comment ${commentId}]`);
+
     } catch (error) {
       console.error('Error deleting comment:', error);
       if (window.toast) {

package/public/js/modules/suggestion-manager.js

@@ -24,8 +24,9 @@ class SuggestionManager {
           reviewId: this.prManager?.currentPR?.id,
           suggestionId: chatBtn.dataset.suggestionId,
           suggestionContext: {
+            suggestionId: chatBtn.dataset.suggestionId || null,
             title: chatBtn.dataset.title || suggestionData.suggestionTitle || '',
-            body: suggestionData.suggestionText || '',
+            body: suggestionData.formattedBody || suggestionData.suggestionText || '',
             type: suggestionData.suggestionType || '',
             file: chatBtn.dataset.file || '',
             line_start: suggestionDiv?.dataset?.lineNumber ? parseInt(suggestionDiv.dataset.lineNumber) : null,

package/public/js/pr.js

@@ -2542,6 +2542,11 @@ class PRManager {
       if (window.toast) {
         window.toast.showSuccess('Comment dismissed');
       }
+
+      if (apiResult.dismissedSuggestionId) {
+        window.chatPanel?.queueUserActionHint(`[User Action: dismissed suggestion ${apiResult.dismissedSuggestionId}]`);
+      }
+      window.chatPanel?.queueUserActionHint(`[User Action: dismissed comment ${commentId}]`);
     } catch (error) {
       console.error('Error deleting comment:', error);
       if (window.toast) {
@@ -2570,6 +2575,8 @@ class PRManager {
       if (window.toast) {
         window.toast.showSuccess('Comment restored');
       }
+
+      window.chatPanel?.queueUserActionHint(`[User Action: restored comment ${commentId}]`);
     } catch (error) {
       console.error('Error restoring comment:', error);
       if (window.toast) {
@@ -3050,6 +3057,7 @@ class PRManager {
             });
             this.displayUserComment(newComment, suggestionRow);
             this._notifyAdoption(suggestionId, newComment);
+            window.chatPanel?.queueUserActionHint(`[User Action: adopted suggestion ${suggestionId}]`);
           } catch (error) {
             console.error('Error saving edited suggestion:', error);
             alert(`Failed to save suggestion: ${error.message}`);
@@ -3095,6 +3103,7 @@ class PRManager {
 
       this.displayUserComment(result.newComment, result.suggestionRow);
       this._notifyAdoption(suggestionId, result.newComment);
+      window.chatPanel?.queueUserActionHint(`[User Action: adopted suggestion ${suggestionId}]`);
     } catch (error) {
       console.error('Error adopting suggestion:', error);
       alert(`Failed to adopt suggestion: ${error.message}`);
@@ -3158,6 +3167,8 @@ class PRManager {
       if (this.commentMinimizer) {
         this.commentMinimizer.refreshIndicators();
       }
+
+      window.chatPanel?.queueUserActionHint(`[User Action: dismissed suggestion ${suggestionId}]`);
     } catch (error) {
       console.error('Error dismissing suggestion:', error);
       alert('Failed to dismiss suggestion');
@@ -3213,6 +3224,8 @@ class PRManager {
       if (this.commentMinimizer) {
         this.commentMinimizer.refreshIndicators();
       }
+
+      window.chatPanel?.queueUserActionHint(`[User Action: restored suggestion ${suggestionId}]`);
     } catch (error) {
       console.error('Error restoring suggestion:', error);
       alert('Failed to restore suggestion');

package/src/ai/analyzer.js

@@ -126,8 +126,8 @@ async function runExecutableVoice(voiceProvider, reviewId, worktreePath, prMetad
   const tmpDir = await fs.mkdtemp(path.join(os.tmpdir(), 'pair-review-exec-'));
   try {
     const executableContext = {
-      title: prMetadata.title || '',
-      description: prMetadata.description || '',
+      title: prMetadata.title || null,
+      description: prMetadata.description || null,
       cwd: worktreePath,
       outputDir: tmpDir,
       model: voiceProvider.resolvedModel !== undefined ? voiceProvider.resolvedModel : (voiceProvider.model || null),
@@ -176,11 +176,16 @@ async function runExecutableVoice(voiceProvider, reviewId, worktreePath, prMetad
     });
 
     if (!result?.success || !result?.data) {
+      if (progressCallback) {
+        progressCallback({ level: 'exec', status: 'failed', progress: 'External tool returned no data' });
+      }
       throw new Error(`${logPrefix || ''}Executable provider returned no data`);
     }
 
+    const suggestions = result.data.suggestions || [];
+
     return {
-      suggestions: result.data.suggestions || [],
+      suggestions,
       summary: result.data.summary || ''
     };
   } finally {
@@ -2948,6 +2953,10 @@ File-level suggestions should NOT have a line number. They apply to the entire f
         const finalSuggestions = this.validateAndFinalizeSuggestions(result.suggestions, fileLineCountMap, validFiles);
         await this.storeSuggestions(reviewId, parentRunId, finalSuggestions, null, validFiles);
 
+        if (progressCallback) {
+          progressCallback({ level: 'exec', status: 'completed', progress: `External tool complete: ${finalSuggestions.length} suggestions` });
+        }
+
         try {
           await analysisRunRepo.update(parentRunId, {
             status: 'completed',
@@ -3074,6 +3083,10 @@ File-level suggestions should NOT have a line number. They apply to the entire f
           // Store validated voice suggestions
           await commentRepo.bulkInsertAISuggestions(reviewId, childRunId, validatedSuggestions, null);
 
+          if (voiceProgressCallback) {
+            voiceProgressCallback({ level: 'exec', status: 'completed', progress: `External tool complete: ${validatedSuggestions.length} suggestions` });
+          }
+
           const validatedResult = { ...result, suggestions: validatedSuggestions };
           return { voiceKey, reviewerLabel, childRunId, result: validatedResult, provider: voice.provider, model: voice.model, isExecutable: true, customInstructions: voice.customInstructions || null };
         }
@@ -3121,6 +3134,11 @@ File-level suggestions should NOT have a line number. They apply to the entire f
         } catch (err) {
           logger.warn(`[ReviewerCouncil] Failed to update child run ${childRunId}: ${err.message}`);
         }
+        // Notify the progress dialog so the voice row stops spinning
+        if (isExecutable && voiceProgressCallback) {
+          const terminalStatus = error.isCancellation ? 'cancelled' : 'failed';
+          voiceProgressCallback({ level: 'exec', status: terminalStatus, progress: error.message || 'Voice failed' });
+        }
         throw error;
       }
     });

package/src/ai/claude-provider.js

@@ -22,23 +22,13 @@ const BIN_DIR = path.join(__dirname, '..', '..', 'bin');
 const CLAUDE_MODELS = [
   {
     id: 'haiku',
-    name: 'Haiku 4.5',
+    name: 'Haiku 4.6',
     tier: 'fast',
     tagline: 'Lightning Fast',
     description: 'Quick analysis for simple changes',
     badge: 'Fastest',
     badgeClass: 'badge-speed'
   },
-  {
-    id: 'sonnet-4.5',
-    cli_model: 'claude-sonnet-4.5',
-    name: 'Sonnet 4.5',
-    tier: 'balanced',
-    tagline: 'Previous Gen',
-    description: 'Sonnet 4.5 — previous generation balanced model',
-    badge: 'Previous Gen',
-    badgeClass: 'badge-balanced'
-  },
   {
     id: 'sonnet-4.6',
     cli_model: 'claude-sonnet-4-6',

package/src/ai/codex-provider.js

@@ -21,27 +21,29 @@ const BIN_DIR = path.join(__dirname, '..', '..', 'bin');
  * Codex model definitions with tier mappings
  *
  * Based on OpenAI Codex Models guide (developers.openai.com/codex/models)
- * - gpt-5.1-codex-mini: Smaller, cost-effective variant for quick scans
- * - gpt-5.2-codex: Advanced coding model for everyday reviews, good reasoning/cost balance
- * - gpt-5.3-codex: Capable agentic coding model with frontier performance and reasoning
- * - gpt-5.4: Latest generation with enhanced reasoning depth
+ * - gpt-5.4-nano: Cheapest model ($0.20/$1.25 per MTok), good for surface scans
+ * - gpt-5.4-mini: Fast with 400k context ($0.75/$4.50 per MTok)
+ * - gpt-5.4: Flagship model combining coding, reasoning, and agentic workflows
+ * - gpt-5.3-codex: Industry-leading coding model for complex engineering tasks
+ *
+ * Deprecated (April 2026): gpt-5.1-codex-mini, gpt-5.1-codex-max, gpt-5.1-codex
  */
 const CODEX_MODELS = [
   {
-    id: 'gpt-5.1-codex-mini',
-    name: 'GPT-5.1 Mini',
+    id: 'gpt-5.4-nano',
+    name: 'GPT-5.4 Nano',
     tier: 'fast',
-    tagline: 'Blazing Fast',
-    description: 'Quick, low-cost reviews for style issues, obvious bugs, and lint-level feedback.',
-    badge: 'Fastest',
+    tagline: 'Cheapest',
+    description: 'Ultra-low-cost surface scans for style issues, obvious bugs, and lint-level feedback.',
+    badge: 'Cheapest',
     badgeClass: 'badge-speed'
   },
   {
-    id: 'gpt-5.2-codex',
-    name: 'GPT-5.2 Codex',
+    id: 'gpt-5.4-mini',
+    name: 'GPT-5.4 Mini',
     tier: 'balanced',
     tagline: 'Best Balance',
-    description: 'Strong everyday reviewer—good reasoning and code understanding for PR-sized changes without top-tier cost.',
+    description: 'Fast reviews with 400k context—good balance of speed and capability for everyday PR review.',
     badge: 'Recommended',
     badgeClass: 'badge-recommended',
     default: true
@@ -51,7 +53,7 @@ const CODEX_MODELS = [
     name: 'GPT-5.3 Codex',
     tier: 'thorough',
     tagline: 'Deep Review',
-    description: 'Capable agentic coding model—combines frontier coding performance with strong reasoning for cross-file analysis.',
+    description: 'Industry-leading coding model—frontier performance with strong reasoning for cross-file analysis.',
     badge: 'Thorough',
     badgeClass: 'badge-power'
   },
@@ -60,7 +62,7 @@ const CODEX_MODELS = [
     name: 'GPT-5.4',
     tier: 'thorough',
     tagline: 'Latest Gen',
-    description: 'Latest generation model with enhanced reasoning depth for complex architectural reviews.',
+    description: 'Flagship model combining coding, reasoning, and agentic workflows for complex architectural reviews.',
     badge: 'Most Thorough',
     badgeClass: 'badge-power'
   }
@@ -76,7 +78,7 @@ class CodexProvider extends AIProvider {
    * @param {Object} configOverrides.env - Additional environment variables
    * @param {Object[]} configOverrides.models - Custom model definitions
    */
-  constructor(model = 'gpt-5.2-codex', configOverrides = {}) {
+  constructor(model = 'gpt-5.4-mini', configOverrides = {}) {
     super(model);
 
     // Command precedence: ENV > config > default
@@ -698,7 +700,7 @@ class CodexProvider extends AIProvider {
   }
 
   static getDefaultModel() {
-    return 'gpt-5.2-codex';
+    return 'gpt-5.4-mini';
   }
 
   static getInstallInstructions() {

package/src/ai/copilot-provider.js

@@ -21,21 +21,30 @@ const BIN_DIR = path.join(__dirname, '..', '..', 'bin');
  *
  * GitHub Copilot CLI supports multiple AI models including OpenAI,
  * Anthropic, and Google models via the --model flag.
- * Available models (as of Feb 2026): claude-haiku-4.5, claude-sonnet-4.6,
- * claude-sonnet-4.5, gpt-5.2-codex, gpt-5.3-codex,
+ * Available models (as of April 2026): claude-haiku-4.6, claude-sonnet-4.6,
+ * claude-sonnet-4.5, gpt-5.4, gpt-5.4-mini, gpt-5.3-codex,
  * claude-opus-4.5, claude-opus-4.6, claude-opus-4.6-fast.
  * Default is claude-sonnet-4.6.
  */
 const COPILOT_MODELS = [
   {
-    id: 'claude-haiku-4.5',
-    name: 'Claude Haiku 4.5',
+    id: 'claude-haiku-4.6',
+    name: 'Claude Haiku 4.6',
     tier: 'fast',
     tagline: 'Quick Scan',
     description: 'Rapid feedback for obvious issues, style checks, and simple logic errors',
     badge: 'Speedy',
     badgeClass: 'badge-speed'
   },
+  {
+    id: 'gpt-5.4-mini',
+    name: 'GPT-5.4 Mini',
+    tier: 'fast',
+    tagline: 'Fast & Cheap',
+    description: 'Low-cost fast reviews with solid reasoning—included at no premium cost',
+    badge: 'Fast',
+    badgeClass: 'badge-speed'
+  },
   {
     id: 'claude-sonnet-4.6',
     name: 'Claude Sonnet 4.6',
@@ -47,29 +56,20 @@ const COPILOT_MODELS = [
     default: true
   },
   {
-    id: 'claude-sonnet-4.5',
-    name: 'Claude Sonnet 4.5',
-    tier: 'balanced',
-    tagline: 'Previous Gen',
-    description: 'Previous generation Sonnet—strong code understanding with excellent quality-to-cost ratio',
-    badge: 'Previous Gen',
-    badgeClass: 'badge-balanced'
-  },
-  {
-    id: 'gpt-5.2-codex',
-    name: 'GPT-5.2 Codex',
-    tier: 'balanced',
-    tagline: 'Alternative View',
-    description: 'OpenAI code-specialized model—different perspective for cross-file analysis',
-    badge: 'Balanced',
-    badgeClass: 'badge-balanced'
+    id: 'gpt-5.4',
+    name: 'GPT-5.4',
+    tier: 'thorough',
+    tagline: 'Latest OpenAI',
+    description: 'Flagship OpenAI model combining coding, reasoning, and agentic workflows',
+    badge: 'Latest',
+    badgeClass: 'badge-power'
   },
   {
     id: 'gpt-5.3-codex',
     name: 'GPT-5.3 Codex',
     tier: 'thorough',
     tagline: 'Deep Code Analysis',
-    description: 'Most capable OpenAI coding model—frontier performance for complex multi-file reviews',
+    description: 'Industry-leading coding model—frontier performance for complex multi-file reviews',
     badge: 'Thorough',
     badgeClass: 'badge-power'
   },