@in-the-loop-labs/pair-review 3.9.0 → 3.9.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@in-the-loop-labs/pair-review",
-  "version": "3.9.0",
+  "version": "3.9.1",
   "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.9.0",
+  "version": "3.9.1",
   "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.9.0",
+  "version": "3.9.1",
   "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

@@ -6939,66 +6939,6 @@ body:not([data-theme="dark"]) .theme-icon-light {
   border-color: var(--color-border-secondary);
 }
 
-/* ============================================
-   Analysis Progress Dots
-   ============================================ */
-
-.analysis-progress-dots {
-  display: flex;
-  align-items: center;
-  gap: 6px;
-  margin-right: 8px;
-}
-
-.progress-dot {
-  width: 8px;
-  height: 8px;
-  border-radius: 50%;
-  border: 1.5px solid var(--color-accent-ai);
-  background: transparent;
-  transition: all 0.3s ease;
-}
-
-.progress-dot.active {
-  animation: pulse-dot 1.2s ease-in-out infinite;
-}
-
-.progress-dot.completed {
-  background: var(--color-accent-ai);
-  border-color: var(--color-accent-ai);
-}
-
-.progress-dot.error {
-  background: #ef4444;
-  border-color: #ef4444;
-}
-
-@keyframes pulse-dot {
-  0%, 100% {
-    transform: scale(1);
-    opacity: 1;
-  }
-  50% {
-    transform: scale(1.3);
-    opacity: 0.9;
-  }
-}
-
-/* Dark theme for progress dots */
-[data-theme="dark"] .progress-dot {
-  border-color: var(--color-accent-ai-dark);
-}
-
-[data-theme="dark"] .progress-dot.completed {
-  background: var(--color-accent-ai-dark);
-  border-color: var(--color-accent-ai-dark);
-}
-
-[data-theme="dark"] .progress-dot.error {
-  background: #f87171;
-  border-color: #f87171;
-}
-
 /* Magical Analyze Button - Amber AI CTA */
 #analyze-btn {
   background: linear-gradient(135deg, #fef3c7 0%, #fde68a 100%);
@@ -12959,6 +12899,18 @@ body.resizing * {
   animation: pulse-dot 1.2s ease-in-out infinite;
 }
 
+/* Pulse used by the running/setting-up stack-progress status icons above. */
+@keyframes pulse-dot {
+  0%, 100% {
+    transform: scale(1);
+    opacity: 1;
+  }
+  50% {
+    transform: scale(1.3);
+    opacity: 0.9;
+  }
+}
+
 .status-pending .stack-progress-status-icon {
   color: var(--color-text-muted, #d1d9e0);
 }

package/public/js/components/CouncilProgressModal.js

@@ -304,16 +304,6 @@ class CouncilProgressModal {
       }
     }
 
-    // Update toolbar progress dots
-    const manager = window.prManager || window.localManager;
-    if (manager?.updateProgressDot) {
-      for (let level = 1; level <= 4; level++) {
-        if (status.levels[level]) {
-          manager.updateProgressDot(level, status.levels[level].status);
-        }
-      }
-    }
-
     // Terminal states
     if (status.status === 'completed') {
       this._handleCompletion(status);

package/public/js/local.js

@@ -1893,55 +1893,6 @@ class LocalManager {
     };
     document.addEventListener('keydown', keyHandler);
   }
-
-  /**
-   * Get the progress dots container element
-   * @returns {HTMLElement|null}
-   */
-  getProgressDotsContainer() {
-    return document.getElementById('analysis-progress-dots');
-  }
-
-  /**
-   * Update a specific progress dot during analysis
-   * Maps level numbers to phase names:
-   * - Level 4 -> orchestration (finalization)
-   * - Level 1 -> level1
-   * - Level 2 -> level2
-   * - Level 3 -> level3
-   * @param {number} level - The level number (1, 2, 3, or 4)
-   * @param {string} status - The status ('running', 'completed', 'failed')
-   */
-  updateProgressDot(level, status) {
-    const container = this.getProgressDotsContainer();
-    if (!container) return;
-
-    // Map levels to dot phases
-    const phaseMap = {
-      4: 'orchestration', // Orchestration/finalization is level 4 in progress modal
-      1: 'level1',
-      2: 'level2',
-      3: 'level3'
-    };
-
-    const phase = phaseMap[level];
-    if (!phase) return;
-
-    const dot = container.querySelector(`[data-phase="${phase}"]`);
-    if (!dot) return;
-
-    // Remove existing states
-    dot.classList.remove('active', 'completed', 'error');
-
-    // Apply new state
-    if (status === 'running') {
-      dot.classList.add('active');
-    } else if (status === 'completed') {
-      dot.classList.add('completed');
-    } else if (status === 'failed') {
-      dot.classList.add('error');
-    }
-  }
 }
 
 // Initialize LocalManager when in local mode

package/public/js/pr.js

@@ -6588,9 +6588,6 @@ class PRManager {
     } else {
       btn.innerHTML = '<span class="analyzing-icon">✨</span> Analyzing...';
     }
-
-    // Show progress dots
-    this.showProgressDots();
   }
 
   /**
@@ -6615,9 +6612,6 @@ class PRManager {
     }
     btn.disabled = true;
 
-    // Complete all progress dots (they'll be hidden when button resets)
-    this.completeAllProgressDots();
-
     // Revert to normal after 2 seconds
     setTimeout(() => this.resetButton(), 2000);
   }
@@ -6645,114 +6639,6 @@ class PRManager {
     } else {
       btn.innerHTML = 'Analyze with AI';
     }
-
-    // Hide progress dots when resetting
-    this.hideProgressDots();
-  }
-
-  // ============================================
-  // Progress Dots Controller
-  // ============================================
-
-  /**
-   * Get the progress dots container
-   * @returns {HTMLElement|null}
-   */
-  getProgressDotsContainer() {
-    return document.getElementById('analysis-progress-dots');
-  }
-
-  /**
-   * Show progress dots (called when analysis starts)
-   */
-  showProgressDots() {
-    const container = this.getProgressDotsContainer();
-    if (!container) return;
-
-    container.style.display = 'flex';
-
-    // Reset all dots to initial state
-    const dots = container.querySelectorAll('.progress-dot');
-    dots.forEach(dot => {
-      dot.classList.remove('active', 'completed', 'error');
-    });
-
-    // Set first dot (orchestration) as active
-    const firstDot = container.querySelector('[data-phase="orchestration"]');
-    if (firstDot) {
-      firstDot.classList.add('active');
-    }
-  }
-
-  /**
-   * Hide progress dots (called when analysis completes or is cancelled)
-   */
-  hideProgressDots() {
-    const container = this.getProgressDotsContainer();
-    if (!container) return;
-
-    container.style.display = 'none';
-
-    // Reset all dots
-    const dots = container.querySelectorAll('.progress-dot');
-    dots.forEach(dot => {
-      dot.classList.remove('active', 'completed', 'error');
-    });
-  }
-
-  /**
-   * Update progress dots based on level status
-   * Maps level numbers to phases:
-   * - Level 4 (orchestration/finalization) -> orchestration
-   * - Level 1 -> level1
-   * - Level 2 -> level2
-   * - Level 3 -> level3
-   * @param {number} level - The level number (1, 2, 3, or 4)
-   * @param {string} status - The status ('running', 'completed', 'failed')
-   */
-  updateProgressDot(level, status) {
-    const container = this.getProgressDotsContainer();
-    if (!container) return;
-
-    // Map levels to dot phases
-    const phaseMap = {
-      4: 'orchestration', // Orchestration/finalization is level 4 in progress modal
-      1: 'level1',
-      2: 'level2',
-      3: 'level3'
-    };
-
-    const phase = phaseMap[level];
-    if (!phase) return;
-
-    const dot = container.querySelector(`[data-phase="${phase}"]`);
-    if (!dot) return;
-
-    // Remove existing states
-    dot.classList.remove('active', 'completed', 'error');
-
-    // Apply new state
-    if (status === 'running') {
-      dot.classList.add('active');
-    } else if (status === 'completed') {
-      dot.classList.add('completed');
-    } else if (status === 'failed') {
-      dot.classList.add('error');
-    }
-  }
-
-  /**
-   * Complete all progress dots (called briefly before hiding)
-   */
-  completeAllProgressDots() {
-    const container = this.getProgressDotsContainer();
-    if (!container) return;
-
-    const dots = container.querySelectorAll('.progress-dot');
-    dots.forEach(dot => {
-      dot.classList.remove('active', 'error');
-      dot.classList.add('completed');
-    });
   }
 
   /**

package/public/local.html

@@ -435,13 +435,6 @@
                     </div>
                     <div class="diff-stats" id="diff-stats"></div>
                     <div class="toolbar-actions">
-                        <!-- Progress Dots (visible during analysis) -->
-                        <div class="analysis-progress-dots" id="analysis-progress-dots" style="display: none;">
-                            <span class="progress-dot" data-phase="level1" title="Level 1: Diff"></span>
-                            <span class="progress-dot" data-phase="level2" title="Level 2: File"></span>
-                            <span class="progress-dot" data-phase="level3" title="Level 3: Codebase"></span>
-                            <span class="progress-dot" data-phase="orchestration" title="Consolidation"></span>
-                        </div>
                         <button class="btn btn-sm btn-secondary" id="analyze-btn" title="Analyze with AI">
                             <svg class="analyze-icon" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" width="14" height="14">
                                 <path d="M9.813 15.904L9 18.75l-.813-2.846a4.5 4.5 0 00-3.09-3.09L2.25 12l2.846-.813a4.5 4.5 0 003.09-3.09L9 5.25l.813 2.846a4.5 4.5 0 003.09 3.09L15.75 12l-2.846.813a4.5 4.5 0 00-3.09 3.09z"/>

package/public/pr.html

@@ -231,13 +231,6 @@
                     </div>
                     <div class="diff-stats" id="diff-stats"></div>
                     <div class="toolbar-actions">
-                        <!-- Progress Dots (visible during analysis) -->
-                        <div class="analysis-progress-dots" id="analysis-progress-dots" style="display: none;">
-                            <span class="progress-dot" data-phase="level1" title="Level 1: Diff"></span>
-                            <span class="progress-dot" data-phase="level2" title="Level 2: File"></span>
-                            <span class="progress-dot" data-phase="level3" title="Level 3: Codebase"></span>
-                            <span class="progress-dot" data-phase="orchestration" title="Consolidation"></span>
-                        </div>
                         <button class="btn btn-sm btn-secondary" id="analyze-btn" title="Analyze with AI">
                             <svg class="analyze-icon" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" width="14" height="14">
                                 <path d="M9.813 15.904L9 18.75l-.813-2.846a4.5 4.5 0 00-3.09-3.09L2.25 12l2.846-.813a4.5 4.5 0 003.09-3.09L9 5.25l.813 2.846a4.5 4.5 0 003.09 3.09L15.75 12l-2.846.813a4.5 4.5 0 00-3.09 3.09z"/>

package/src/ai/claude-provider.js

@@ -26,9 +26,9 @@ const BIN_DIR = path.join(__dirname, '..', '..', 'bin');
  * in the constructor's base args; individual models can override this via extra_args
  * (e.g., Haiku uses adaptive thinking for efficiency).
  *
- * Effort support by model (newest CLIs): Fable 5 and Opus 4.8 / 4.7 support
- * low|medium|high|xhigh|max; Opus 4.6 & Sonnet 4.6 support low|medium|high|max
- * (no xhigh); Haiku has no effort levels.
+ * Effort support by model (newest CLIs): Fable 5, Opus 4.8 / 4.7, and Sonnet 5
+ * support low|medium|high|xhigh|max; Opus 4.6 & Sonnet 4.6 support
+ * low|medium|high|max (no xhigh); Haiku has no effort levels.
  */
 const CLAUDE_MODELS = [
   // ── Thorough tier ───────────────────────────────────────────────────────
@@ -119,15 +119,37 @@ const CLAUDE_MODELS = [
     badgeClass: 'badge-power'
   },
   // ── Balanced tier ───────────────────────────────────────────────────────
+  {
+    id: 'sonnet-5-xhigh',
+    cli_model: 'claude-sonnet-5',
+    env: { CLAUDE_CODE_EFFORT_LEVEL: 'xhigh' },
+    name: 'Sonnet 5 XHigh',
+    tier: 'balanced',
+    tagline: 'Best Balance',
+    description: 'Sonnet 5 (newest) with extra-high effort — recommended for most reviews',
+    badge: 'Recommended',
+    badgeClass: 'badge-recommended'
+  },
+  {
+    id: 'sonnet-5-high',
+    cli_model: 'claude-sonnet-5',
+    env: { CLAUDE_CODE_EFFORT_LEVEL: 'high' },
+    name: 'Sonnet 5 High',
+    tier: 'balanced',
+    tagline: 'Faster',
+    description: 'Sonnet 5 with high effort — quicker than XHigh, still sharp',
+    badge: 'Quick & Sharp',
+    badgeClass: 'badge-speed'
+  },
   {
     id: 'sonnet-4.6',
     cli_model: 'claude-sonnet-4-6',
     name: 'Sonnet 4.6',
     tier: 'balanced',
-    tagline: 'Best Balance',
-    description: 'Recommended for most reviews',
-    badge: 'Standard',
-    badgeClass: 'badge-recommended'
+    tagline: 'Previous Gen',
+    description: 'Sonnet 4.6 — previous generation balanced model',
+    badge: 'Previous Gen',
+    badgeClass: 'badge-balanced'
   },
   {
     id: 'opus-4.6-1m',

package/src/main.js

@@ -196,7 +196,8 @@ OPTIONS:
                             Available models: opus, sonnet, haiku (Claude Code);
                             also: fable-5-xhigh, fable-5-high, opus-4.8-xhigh,
                                   opus-4.8-high, opus-4.7-xhigh, opus-4.7-high,
-                                  opus-4.6-high, opus-4.6-1m, sonnet-4.6
+                                  opus-4.6-high, opus-4.6-1m, sonnet-5-xhigh,
+                                  sonnet-5-high, sonnet-4.6
                             (opus is Opus 4.8 XHigh, the default)
                             or use provider-specific models with Gemini/Codex
     --council <handle>      Run analysis with a saved council (multi-voice). Handle is a