@in-the-loop-labs/pair-review 2.3.1 → 2.3.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@in-the-loop-labs/pair-review",
-  "version": "2.3.1",
+  "version": "2.3.2",
   "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": "2.3.1",
+  "version": "2.3.2",
   "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": "2.3.1",
+  "version": "2.3.2",
   "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/js/components/AIPanel.js

@@ -207,6 +207,45 @@ class AIPanel {
         }
     }
 
+    /**
+     * Get the localStorage key for the collapsed state (per-review)
+     * @returns {string|null} Storage key or null if no PR context
+     */
+    _getCollapsedStorageKey() {
+        if (!this.currentPRKey) return null;
+        return `pair-review-panel-collapsed_${this.currentPRKey}`;
+    }
+
+    /**
+     * Save the collapsed state to localStorage (per-review)
+     */
+    _saveCollapsedState() {
+        const key = this._getCollapsedStorageKey();
+        if (key) {
+            localStorage.setItem(key, this.isCollapsed ? 'true' : 'false');
+        }
+    }
+
+    /**
+     * Restore the collapsed state from localStorage, or collapse for new reviews.
+     * If the user had the panel expanded for this review, expand it.
+     * Otherwise (new review or previously collapsed), collapse it.
+     */
+    _restoreOrCollapsePanel() {
+        const key = this._getCollapsedStorageKey();
+        if (key) {
+            const stored = localStorage.getItem(key);
+            if (stored === 'false') {
+                this.expand();
+            } else {
+                // 'true' or no saved state (new review) → collapse
+                this.collapse();
+            }
+        } else {
+            this.collapse();
+        }
+    }
+
     /**
      * Get the localStorage key for the filter state (per-review)
      * @returns {string|null} Storage key or null if no PR context
@@ -275,6 +314,7 @@ class AIPanel {
         // Set CSS variable to 0 so width calculations don't reserve space
         document.documentElement.style.setProperty('--ai-panel-width', '0px');
         window.panelGroup?._onReviewVisibilityChanged(false);
+        this._saveCollapsedState();
     }
 
     expand() {
@@ -285,6 +325,7 @@ class AIPanel {
         // Restore CSS variable from saved width or default
         document.documentElement.style.setProperty('--ai-panel-width', `${this.getEffectivePanelWidth()}px`);
         window.panelGroup?._onReviewVisibilityChanged(true);
+        this._saveCollapsedState();
     }
 
     /**
@@ -296,6 +337,7 @@ class AIPanel {
      */
     setPR(owner, repo, number) {
         this.currentPRKey = `${owner}/${repo}#${number}`;
+        this._restoreOrCollapsePanel();
         this.restoreSegmentSelection();
         this.restoreFilterState();
     }
@@ -327,6 +369,10 @@ class AIPanel {
      */
     setAnalysisState(state) {
         this.analysisState = state;
+        // Auto-expand panel when analysis starts
+        if (state === 'loading' && this.isCollapsed) {
+            this.expand();
+        }
         // Re-render if currently showing empty state
         if (this.findings.length === 0 && this.selectedSegment === 'ai') {
             this.renderFindings();
@@ -1737,3 +1783,8 @@ class AIPanel {
 document.addEventListener('DOMContentLoaded', () => {
     window.aiPanel = new AIPanel();
 });
+
+// Export for testing
+if (typeof module !== 'undefined' && module.exports) {
+    module.exports = { AIPanel };
+}

package/public/js/components/PanelGroup.js

@@ -41,6 +41,7 @@ class PanelGroup {
     this._reviewVisible = !document.getElementById('ai-panel')?.classList.contains('collapsed');
     this._chatVisible = false;
     this._popoverVisible = false;
+    this._currentPRKey = null; // Set via setPR() for per-review chat persistence
 
     // Read persisted layout
     const savedLayout = localStorage.getItem(PanelGroup.STORAGE_KEY);
@@ -79,20 +80,17 @@ class PanelGroup {
     // Apply initial layout
     this._applyLayout(this._layout);
 
-    // Restore chat visibility from last session (only if chat is available)
-    const chatState = document.documentElement.getAttribute('data-chat');
-    if (chatState === 'available') {
-      this._restoreChatFromStorage();
-    } else {
-      // Chat not available yet — zero out CSS variable so max-width calcs are correct.
-      document.documentElement.style.setProperty('--chat-panel-width', '0px');
-    }
+    // Chat starts hidden; per-review state restored when setPR() is called.
+    document.documentElement.style.setProperty('--chat-panel-width', '0px');
 
     // Listen for late chat-state transitions (config fetch may complete after constructor)
     window.addEventListener('chat-state-changed', (e) => {
       const state = e.detail?.state;
       if (state === 'available') {
-        this._restoreChatFromStorage();
+        // Only restore if PR context is already set (setPR was called before chat became available)
+        if (this._currentPRKey) {
+          this._restoreChatFromStorage();
+        }
       } else if (state === 'unavailable' && this.chatToggleBtn) {
         this.chatToggleBtn.title = 'Install and configure Pi to enable chat';
       }
@@ -422,10 +420,34 @@ class PanelGroup {
   }
 
   /**
-   * Restore chat visibility from localStorage (called when chat becomes available)
+   * Set the current PR for per-review chat visibility persistence.
+   * Call this when a PR/review loads (after aiPanel.setPR).
+   * @param {string} prKey - e.g. "owner/repo#123" or "local/local#2"
+   */
+  setPR(prKey) {
+    this._currentPRKey = prKey;
+    // Now that we know the review context, restore chat state if chat is available
+    if (this._isChatAvailable()) {
+      this._restoreChatFromStorage();
+    }
+  }
+
+  /**
+   * Get the per-review localStorage key for chat visibility.
+   * @returns {string|null} Storage key or null if no PR context
+   */
+  _getChatVisibleStorageKey() {
+    if (!this._currentPRKey) return null;
+    return `${PanelGroup.CHAT_VISIBLE_KEY}_${this._currentPRKey}`;
+  }
+
+  /**
+   * Restore chat visibility from localStorage (per-review).
+   * For new reviews (no saved state), chat stays hidden.
    */
   _restoreChatFromStorage() {
-    const savedChatVisible = localStorage.getItem(PanelGroup.CHAT_VISIBLE_KEY);
+    const key = this._getChatVisibleStorageKey();
+    const savedChatVisible = key ? localStorage.getItem(key) : null;
     if (savedChatVisible === 'true') {
       this._chatVisible = true;
       this.chatPanel.open({ suppressFocus: true });
@@ -478,8 +500,11 @@ class PanelGroup {
       this.chatToggleBtn.classList.toggle('active', visible);
     }
 
-    // Persist chat visibility
-    localStorage.setItem(PanelGroup.CHAT_VISIBLE_KEY, visible ? 'true' : 'false');
+    // Persist chat visibility (per-review)
+    const chatKey = this._getChatVisibleStorageKey();
+    if (chatKey) {
+      localStorage.setItem(chatKey, visible ? 'true' : 'false');
+    }
 
     // Clear inline flex heights so the remaining panel fills the space
     if (this._layout.startsWith('v-')) {

package/public/js/local.js

@@ -374,6 +374,7 @@ class LocalManager {
         if (data.running && data.analysisId) {
           manager.currentAnalysisId = data.analysisId;
           manager.isAnalyzing = true;
+          window.aiPanel?.setAnalysisState('loading');
           manager.setButtonAnalyzing(data.analysisId);
 
           // Show the appropriate progress modal
@@ -791,10 +792,11 @@ class LocalManager {
         window.aiPanel = new window.AIPanel();
       }
 
-      // Set local context for AI Panel (restores filter state from localStorage)
+      // Set local context for AI Panel and Panel Group (restores per-review state from localStorage)
       if (window.aiPanel?.setPR) {
         window.aiPanel.setPR('local', reviewData.repository, this.reviewId);
       }
+      window.panelGroup?.setPR(`local/${reviewData.repository}#${this.reviewId}`);
 
       // Load saved comments using the restored filter state from AI Panel
       const includeDismissed = window.aiPanel?.showDismissedComments || false;

package/public/js/pr.js

@@ -368,6 +368,9 @@ class PRManager {
 
   /**
    * Auto-trigger analysis if ?analyze=true is present in the URL.
+   * Skips refresh if data was just loaded fresh by loadPR (to avoid redundant fetches).
+   * Otherwise, refreshes PR data first to ensure we analyze the latest code.
+   * If refresh fails, proceeds with existing data rather than failing entirely.
    * Cleans up the query parameter afterwards regardless of success or failure.
    * @param {string} owner - Repository owner
    * @param {string} repo - Repository name
@@ -378,6 +381,21 @@ class PRManager {
     if (autoAnalyze === 'true' && !this.isAnalyzing) {
       this._autoAnalyzeRequested = true;
       try {
+        // Skip refresh if we just loaded fresh data (loadPR sets _justLoaded = true).
+        // Otherwise, refresh to ensure we have the latest PR data in case the worktree
+        // already existed but the PR has new commits since last load.
+        if (this._justLoaded) {
+          this._justLoaded = false;
+        } else {
+          try {
+            await this.refreshPR();
+          } catch (e) {
+            // If refresh fails, proceed with existing data - this is intentional.
+            // We'd rather analyze stale data than fail entirely.
+            console.warn('Pre-analysis refresh failed, proceeding with existing data', e);
+          }
+        }
+
         await this.startAnalysis(owner, repo, prNumber, null, {});
       } finally {
         this._autoAnalyzeRequested = false;
@@ -426,11 +444,12 @@ class PRManager {
         window.aiPanel = new window.AIPanel();
       }
 
-      // Set PR context for AI Panel (for PR-specific localStorage keys)
-      // This restores the filter state from localStorage
+      // Set PR context for AI Panel and Panel Group (for per-review localStorage keys)
+      // This restores the filter state and chat visibility from localStorage
       if (window.aiPanel?.setPR) {
         window.aiPanel.setPR(owner, repo, number);
       }
+      window.panelGroup?.setPR(`${owner}/${repo}#${number}`);
 
       // Load saved comments using the restored filter state from AI Panel
       // If AI Panel has showDismissedComments=true (restored from localStorage), use that
@@ -470,6 +489,8 @@ class PRManager {
       this.showError(error.message);
     } finally {
       this.setLoading(false);
+      // Mark that we just loaded fresh data - used by _maybeAutoAnalyze to skip redundant refresh
+      this._justLoaded = true;
     }
   }
 

package/public/local.html

@@ -404,7 +404,7 @@
             <!-- Right Panel Group (AI Panel + Chat) -->
             <div class="right-panel-group" id="right-panel-group">
                 <!-- AI Analysis Panel (Right) -->
-                <aside class="ai-panel" id="ai-panel">
+                <aside class="ai-panel collapsed" id="ai-panel">
                     <div class="resize-handle resize-handle-left" data-panel="ai-panel"></div>
                     <div class="ai-panel-header">
                         <div class="ai-panel-title">

package/public/pr.html

@@ -227,7 +227,7 @@
             <!-- Right Panel Group (AI Panel + Chat) -->
             <div class="right-panel-group" id="right-panel-group">
                 <!-- AI Analysis Panel (Right) -->
-                <aside class="ai-panel" id="ai-panel">
+                <aside class="ai-panel collapsed" id="ai-panel">
                     <div class="resize-handle resize-handle-left" data-panel="ai-panel"></div>
                     <div class="ai-panel-header">
                         <div class="ai-panel-title">