@in-the-loop-labs/pair-review 3.2.2 → 3.3.0

package/public/js/components/AIPanel.js

@@ -793,38 +793,7 @@ class AIPanel {
         this.findingsList.querySelectorAll('.quick-action-chat').forEach(btn => {
             btn.addEventListener('click', (e) => {
                 e.stopPropagation(); // Prevent triggering item click
-                if (!window.chatPanel) return;
-
-                const findingId = btn.dataset.findingId ? parseInt(btn.dataset.findingId, 10) : null;
-                const commentId = btn.dataset.commentId ? parseInt(btn.dataset.commentId, 10) : null;
-                const file = btn.dataset.findingFile || '';
-                const title = btn.dataset.findingTitle || '';
-
-                // Build context from the finding data
-                let suggestionContext = { title, file };
-
-                if (findingId && this.findings) {
-                    const finding = this.findings.find(f => f.id === findingId);
-                    if (finding) {
-                        suggestionContext = {
-                            suggestionId: findingId ? String(findingId) : null,
-                            title: finding.title || title,
-                            body: finding.formattedBody || finding.body || '',
-                            type: finding.type || '',
-                            file: finding.file || file,
-                            line_start: finding.line_start || null,
-                            line_end: finding.line_end || null,
-                            side: 'RIGHT',
-                            reasoning: null
-                        };
-                    }
-                }
-
-                window.chatPanel.open({
-                    reviewId: window.prManager?.currentPR?.id,
-                    suggestionId: findingId ? String(findingId) : (commentId ? String(commentId) : undefined),
-                    suggestionContext
-                });
+                this.openQuickActionChat(btn);
             });
         });
 
@@ -887,6 +856,79 @@ class AIPanel {
         }
     }
 
+    /**
+     * Handle chat button clicks from review panel quick actions.
+     * Suggestions use suggestionContext; comments use commentContext.
+     * @param {HTMLButtonElement} btn - The clicked chat button
+     */
+    openQuickActionChat(btn) {
+        if (!window.chatPanel) return;
+
+        const findingId = btn.dataset.findingId ? parseInt(btn.dataset.findingId, 10) : null;
+        const commentId = btn.dataset.commentId ? parseInt(btn.dataset.commentId, 10) : null;
+        const reviewId = window.prManager?.currentPR?.id;
+
+        const buildCommentContext = (comment, fallbackDataset = {}) => ({
+            commentId: comment?.id ? String(comment.id) : String(commentId),
+            body: comment?.body || '',
+            file: comment?.file || fallbackDataset.commentFile || '',
+            line_start: comment?.line_start ?? (fallbackDataset.commentLineStart ? parseInt(fallbackDataset.commentLineStart, 10) : null),
+            line_end: comment?.line_end ?? (fallbackDataset.commentLineEnd ? parseInt(fallbackDataset.commentLineEnd, 10) : null),
+            parentId: comment?.parent_id ?? (fallbackDataset.commentParentId ? parseInt(fallbackDataset.commentParentId, 10) : null),
+            source: 'user',
+            isFileLevel: comment?.is_file_level === 1 || comment?.is_file_level === true
+        });
+
+        if (commentId) {
+            const comment = this.comments?.find(c => c.id === commentId);
+
+            window.chatPanel.open({
+                reviewId,
+                commentContext: buildCommentContext(comment, btn.dataset)
+            });
+            return;
+        }
+
+        const file = btn.dataset.findingFile || '';
+        const title = btn.dataset.findingTitle || '';
+        let suggestionContext = { title, file };
+
+        if (findingId && this.findings) {
+            const finding = this.findings.find(f => f.id === findingId);
+            if (finding) {
+                if (finding.status === 'adopted') {
+                    const adoptedComment = this.comments?.find(c => c.parent_id === findingId && c.status !== 'inactive')
+                        || this.comments?.find(c => c.parent_id === findingId);
+                    if (adoptedComment) {
+                        window.chatPanel.open({
+                            reviewId,
+                            commentContext: buildCommentContext(adoptedComment)
+                        });
+                        return;
+                    }
+                }
+
+                suggestionContext = {
+                    suggestionId: String(findingId),
+                    title: finding.title || title,
+                    body: finding.formattedBody || finding.body || '',
+                    type: finding.type || '',
+                    file: finding.file || file,
+                    line_start: finding.line_start ?? null,
+                    line_end: finding.line_end ?? null,
+                    side: 'RIGHT',
+                    reasoning: null
+                };
+            }
+        }
+
+        window.chatPanel.open({
+            reviewId,
+            suggestionId: findingId ? String(findingId) : undefined,
+            suggestionContext
+        });
+    }
+
     onFindingClick(item) {
         const itemId = item.dataset.id;
         const itemType = item.dataset.itemType;
@@ -1204,9 +1246,9 @@ class AIPanel {
         `;
         }
 
-        // Chat button for active and dismissed findings (upper-right corner)
+        // Chat button for all findings when chat is available
         let chatAction = '';
-        if (finding.status !== 'adopted' && document.documentElement.getAttribute('data-chat') === 'available') {
+        if (document.documentElement.getAttribute('data-chat') === 'available') {
             chatAction = `
             <div class="finding-chat-action">
                 <button class="quick-action-btn quick-action-chat" data-finding-id="${finding.id}" data-finding-file="${finding.file || ''}" data-finding-title="${this.escapeHtml(title)}" title="Chat" aria-label="Chat about suggestion">
@@ -1279,12 +1321,12 @@ class AIPanel {
             `;
         }
 
-        // Chat button for active AI-originated comments
+        // Chat button for active comments
         let chatAction = '';
-        if (!isDismissed && comment.parent_id && document.documentElement.getAttribute('data-chat') === 'available') {
+        if (!isDismissed && document.documentElement.getAttribute('data-chat') === 'available') {
             chatAction = `
             <div class="finding-chat-action">
-                <button class="quick-action-btn quick-action-chat" data-comment-id="${comment.id}" data-finding-file="${comment.file || ''}" data-finding-title="${this.escapeHtml(title)}" title="Chat" aria-label="Chat about comment">
+                <button class="quick-action-btn quick-action-chat" data-comment-id="${comment.id}" data-comment-file="${this.escapeHtml(comment.file || '')}" data-comment-line-start="${comment.line_start ?? ''}" data-comment-line-end="${comment.line_end ?? ''}" data-comment-parent-id="${comment.parent_id || ''}" title="Chat" aria-label="Chat about comment">
                     <svg viewBox="0 0 16 16" fill="currentColor" width="12" height="12"><path d="M1.75 1h8.5c.966 0 1.75.784 1.75 1.75v5.5A1.75 1.75 0 0 1 10.25 10H7.061l-2.574 2.573A1.458 1.458 0 0 1 2 11.543V10h-.25A1.75 1.75 0 0 1 0 8.25v-5.5C0 1.784.784 1 1.75 1ZM1.5 2.75v5.5c0 .138.112.25.25.25h1a.75.75 0 0 1 .75.75v2.19l2.72-2.72a.749.749 0 0 1 .53-.22h3.5a.25.25 0 0 0 .25-.25v-5.5a.25.25 0 0 0-.25-.25h-8.5a.25.25 0 0 0-.25.25Zm13 2a.25.25 0 0 0-.25-.25h-.5a.75.75 0 0 1 0-1.5h.5c.966 0 1.75.784 1.75 1.75v5.5A1.75 1.75 0 0 1 14.25 12H14v1.543a1.458 1.458 0 0 1-2.487 1.03L9.22 12.28a.749.749 0 0 1 .326-1.275.749.749 0 0 1 .734.215l2.22 2.22v-2.19a.75.75 0 0 1 .75-.75h1a.25.25 0 0 0 .25-.25Z"/></svg>
                 </button>
             </div>

package/public/js/components/DiffOptionsDropdown.js

@@ -44,11 +44,13 @@ class DiffOptionsDropdown {
    * @param {{start:string,end:string}} [callbacks.initialScope]
    * @param {boolean} [callbacks.branchAvailable]
    */
-  constructor(buttonElement, { onToggleWhitespace, onToggleMinimize, onScopeChange, initialScope, branchAvailable }) {
+  constructor(buttonElement, { onToggleWhitespace, onToggleMinimize, onScopeChange, initialScope, branchAvailable, worktreePath }) {
     this._btn = buttonElement;
     this._onToggleWhitespace = onToggleWhitespace;
     this._onToggleMinimize = onToggleMinimize || (() => {});
     this._onScopeChange = onScopeChange || null;
+    this._worktreePath = worktreePath || null;
+    this._worktreeName = null;
 
     this._popoverEl = null;
     this._checkbox = null;
@@ -169,6 +171,18 @@ class DiffOptionsDropdown {
     this._updateScopeUI();
   }
 
+  /** Set the worktree display name (relative path from base dir). */
+  set worktreeName(value) {
+    this._worktreeName = value || null;
+    this._updateWorktreeRow();
+  }
+
+  /** Set the worktree path (updates UI if popover already exists). */
+  set worktreePath(value) {
+    this._worktreePath = value || null;
+    this._updateWorktreeRow();
+  }
+
   /** Clear the scope status indicator (call after scope change completes). */
   clearScopeStatus() {
     if (this._scopeStatusEl) {
@@ -231,12 +245,18 @@ class DiffOptionsDropdown {
     const minCheckbox = minLabel.querySelector('input');
     popover.appendChild(minLabel);
 
+    // Worktree path row (placeholder — populated by _updateWorktreeRow)
+    this._worktreeRowEl = null;
+
     document.body.appendChild(popover);
 
     this._popoverEl = popover;
     this._checkbox = wsCheckbox;
     this._minimizeCheckbox = minCheckbox;
 
+    // Render worktree row if path is already known
+    this._updateWorktreeRow();
+
     // Respond to checkbox changes
     wsCheckbox.addEventListener('change', () => {
       this._hideWhitespace = wsCheckbox.checked;
@@ -281,6 +301,69 @@ class DiffOptionsDropdown {
     return label;
   }
 
+  /**
+   * Render or update the worktree path row at the bottom of the popover.
+   */
+  _updateWorktreeRow() {
+    if (!this._popoverEl) return;
+
+    // Remove existing row if any
+    if (this._worktreeRowEl) {
+      this._worktreeRowEl.remove();
+      this._worktreeRowEl = null;
+    }
+    if (!this._worktreePath) return;
+
+    const displayName = this._worktreeName || this._worktreePath.split('/').pop();
+
+    // Divider
+    const divider = document.createElement('div');
+    divider.style.height = '1px';
+    divider.style.background = 'var(--color-border-primary, #d0d7de)';
+    divider.style.margin = '0 20px';
+
+    // Row container
+    const row = document.createElement('div');
+    row.style.display = 'flex';
+    row.style.alignItems = 'center';
+    row.style.gap = '6px';
+    row.style.padding = '8px 12px';
+    row.style.fontSize = '0.8125rem';
+    row.style.color = 'var(--color-fg-muted, #656d76)';
+
+    const text = document.createElement('span');
+    text.textContent = `Worktree: ${displayName}`;
+    text.style.overflow = 'hidden';
+    text.style.textOverflow = 'ellipsis';
+    text.style.whiteSpace = 'nowrap';
+    text.style.minWidth = '0';
+
+    const copyBtn = document.createElement('button');
+    copyBtn.title = 'Copy full path';
+    copyBtn.style.cssText = 'background:none;border:none;cursor:pointer;padding:2px;color:inherit;display:flex;flex-shrink:0;';
+    copyBtn.innerHTML = '<svg width="14" height="14" viewBox="0 0 16 16" fill="currentColor"><path d="M0 6.75C0 5.784.784 5 1.75 5h1.5a.75.75 0 0 1 0 1.5h-1.5a.25.25 0 0 0-.25.25v7.5c0 .138.112.25.25.25h7.5a.25.25 0 0 0 .25-.25v-1.5a.75.75 0 0 1 1.5 0v1.5A1.75 1.75 0 0 1 9.25 16h-7.5A1.75 1.75 0 0 1 0 14.25Z"/><path d="M5 1.75C5 .784 5.784 0 6.75 0h7.5C15.216 0 16 .784 16 1.75v7.5A1.75 1.75 0 0 1 14.25 11h-7.5A1.75 1.75 0 0 1 5 9.25Zm1.75-.25a.25.25 0 0 0-.25.25v7.5c0 .138.112.25.25.25h7.5a.25.25 0 0 0 .25-.25v-7.5a.25.25 0 0 0-.25-.25Z"/></svg>';
+
+    const fullPath = this._worktreePath;
+    copyBtn.addEventListener('click', (e) => {
+      e.stopPropagation();
+      navigator.clipboard.writeText(fullPath).then(() => {
+        const orig = text.textContent;
+        text.textContent = 'Copied!';
+        setTimeout(() => { text.textContent = orig; }, 1500);
+      });
+    });
+
+    row.appendChild(text);
+    row.appendChild(copyBtn);
+
+    // Use a container so we can remove both elements via one reference
+    const container = document.createElement('div');
+    container.appendChild(divider);
+    container.appendChild(row);
+    this._popoverEl.appendChild(container);
+    this._worktreeRowEl = container;
+  }
+
   _renderScopeSelector(popover) {
     const LS = this._localScope;
 

package/public/js/index.js

@@ -141,11 +141,12 @@
    */
   function setFormLoading(tab, loading, text) {
     const ids = tab === 'pr'
-      ? { input: 'pr-url-input', btn: 'start-review-btn', loadingEl: 'start-review-loading-pr', loadingText: 'start-review-loading-text-pr', errorEl: 'start-review-error-pr', btnLabel: 'Start Review' }
-      : { input: 'local-path-input', btn: 'start-local-btn', loadingEl: 'start-review-loading-local', loadingText: 'start-review-loading-text-local', errorEl: 'start-review-error-local', btnLabel: 'Review Local' };
+      ? { input: 'pr-url-input', btn: 'start-review-btn', analyzeBtn: 'analyze-review-btn', loadingEl: 'start-review-loading-pr', loadingText: 'start-review-loading-text-pr', errorEl: 'start-review-error-pr', btnLabel: 'Open' }
+      : { input: 'local-path-input', btn: 'start-local-btn', analyzeBtn: 'analyze-local-btn', loadingEl: 'start-review-loading-local', loadingText: 'start-review-loading-text-local', errorEl: 'start-review-error-local', btnLabel: 'Open' };
 
     const inputEl = document.getElementById(ids.input);
     const btnEl = document.getElementById(ids.btn);
+    const analyzeBtnEl = document.getElementById(ids.analyzeBtn);
     const loadingEl = document.getElementById(ids.loadingEl);
     const loadingTextEl = document.getElementById(ids.loadingText);
     const errorEl = document.getElementById(ids.errorEl);
@@ -153,12 +154,14 @@
     if (loading) {
       if (inputEl) inputEl.disabled = true;
       if (btnEl) { btnEl.disabled = true; btnEl.textContent = 'Starting...'; }
+      if (analyzeBtnEl) analyzeBtnEl.disabled = true;
       if (loadingEl) loadingEl.classList.add('visible');
       if (loadingTextEl && text) loadingTextEl.textContent = text;
       if (errorEl) errorEl.classList.remove('visible', 'info');
     } else {
       if (inputEl) inputEl.disabled = false;
       if (btnEl) { btnEl.disabled = false; btnEl.textContent = ids.btnLabel; }
+      if (analyzeBtnEl) analyzeBtnEl.disabled = false;
       if (loadingEl) loadingEl.classList.remove('visible');
     }
   }
@@ -688,8 +691,10 @@
    * Navigates to the setup page which shows step-by-step progress,
    * matching the flow used when reviews are started from the MCP/CLI.
    * @param {Event} event - Form submit event
+   * @param {Object} [options] - Optional settings
+   * @param {boolean} [options.analyze=false] - When true, append analyze=true to the navigation URL
    */
-  async function handleStartLocal(event) {
+  async function handleStartLocal(event, { analyze = false } = {}) {
     event.preventDefault();
 
     const input = document.getElementById('local-path-input');
@@ -707,7 +712,9 @@
 
     // Navigate to the setup page which shows step-by-step progress
     // The /local?path= route serves setup.html which handles the full setup flow
-    window.location.href = '/local?path=' + encodeURIComponent(pathValue);
+    let href = '/local?path=' + encodeURIComponent(pathValue);
+    if (analyze) href += '&analyze=true';
+    window.location.href = href;
   }
 
   // ─── Browse Directory ──────────────────────────────────────────────────────
@@ -1080,7 +1087,7 @@
    * directly for PRs that already exist in the database.
    * @param {Event} event - Form submit event
    */
-  async function handleStartReview(event) {
+  async function handleStartReview(event, { analyze = false } = {}) {
     event.preventDefault();
 
     const input = document.getElementById('pr-url-input');
@@ -1111,7 +1118,9 @@
 
     // Navigate to the PR route which serves setup.html (with step-by-step progress)
     // for new PRs, or pr.html directly for PRs already in the database
-    window.location.href = '/pr/' + encodeURIComponent(parsed.owner) + '/' + encodeURIComponent(parsed.repo) + '/' + encodeURIComponent(parsed.prNumber);
+    let href = '/pr/' + encodeURIComponent(parsed.owner) + '/' + encodeURIComponent(parsed.repo) + '/' + encodeURIComponent(parsed.prNumber);
+    if (analyze) href += '?analyze=true';
+    window.location.href = href;
   }
 
   // ─── Config & Command Examples ──────────────────────────────────────────────
@@ -1870,6 +1879,22 @@
       browseBtn.addEventListener('click', handleBrowseLocal);
     }
 
+    // Set up PR Analyze button handler
+    const analyzeReviewBtn = document.getElementById('analyze-review-btn');
+    if (analyzeReviewBtn) {
+      analyzeReviewBtn.addEventListener('click', function (event) {
+        handleStartReview(event, { analyze: true });
+      });
+    }
+
+    // Set up Local Analyze button handler
+    const analyzeLocalBtn = document.getElementById('analyze-local-btn');
+    if (analyzeLocalBtn) {
+      analyzeLocalBtn.addEventListener('click', function (event) {
+        handleStartLocal(event, { analyze: true });
+      });
+    }
+
     // Note: No explicit Enter keypress handlers are needed here.
     // Both inputs are inside <form> elements, so pressing Enter
     // natively triggers form submission.

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

@@ -205,9 +205,11 @@ class AnalysisHistoryManager {
   /**
    * Load analysis runs from the API (initial load only)
    *
-   * This method is intended for initial page load and always selects the latest run.
-   * For refreshing after a new analysis completes, use refresh() instead, which has
-   * logic to optionally preserve the user's current selection.
+   * This method is intended for initial page load. On first load it selects the
+   * latest run; on subsequent calls it preserves the current selection when that
+   * run still exists. For refreshing after a new analysis completes, use
+   * refresh(), which has logic to optionally preserve the user's current
+   * selection while surfacing a new run indicator.
    *
    * @returns {Promise<Array>} The loaded runs
    */
@@ -227,11 +229,13 @@ class AnalysisHistoryManager {
       return [];
     }
 
-    // Always select the latest run (first in the list since they're ordered by date DESC)
-    // This ensures that after a new analysis completes, its results are displayed
     const latestRun = this.runs[0];
-    const shouldTriggerCallback = !this.selectedRunId || String(this.selectedRunId) !== String(latestRun.id);
-    await this.selectRun(latestRun.id, shouldTriggerCallback);
+    const currentRun = this.selectedRunId
+      ? this.runs.find(run => String(run.id) === String(this.selectedRunId))
+      : null;
+    const runToSelect = currentRun || latestRun;
+    const shouldTriggerCallback = String(this.selectedRunId) !== String(runToSelect.id);
+    await this.selectRun(runToSelect.id, shouldTriggerCallback);
 
     // Render the dropdown (after selecting so the selected state is correct)
     this.renderDropdown(this.runs);

package/public/js/pr.js

@@ -506,6 +506,22 @@ class PRManager {
     }
   }
 
+  /**
+   * Sync worktree name/path to the diff options dropdown.
+   * Clears both fields when worktree_path is absent.
+   * @param {Object} prData - PR data object from the API
+   */
+  _syncWorktreeDropdown(prData) {
+    if (!this.diffOptionsDropdown) return;
+    if (prData.worktree_path) {
+      this.diffOptionsDropdown.worktreeName = prData.worktree_name || null;
+      this.diffOptionsDropdown.worktreePath = prData.worktree_path;
+    } else {
+      this.diffOptionsDropdown.worktreeName = null;
+      this.diffOptionsDropdown.worktreePath = null;
+    }
+  }
+
   /**
    * Load PR data from the API
    * @param {string} owner - Repository owner
@@ -529,6 +545,9 @@ class PRManager {
       const prData = responseData.data || responseData;
       this.currentPR = prData;
 
+      // Update diff options dropdown with worktree path and display name
+      this._syncWorktreeDropdown(prData);
+
       // Render PR header with metadata
       this.renderPRHeader(prData);
 
@@ -5129,6 +5148,9 @@ class PRManager {
           );
         }
 
+        // Sync worktree label to dropdown (may have changed after refresh)
+        this._syncWorktreeDropdown(data.data);
+
         // Save scroll position and expanded state
         const scrollPosition = window.scrollY;
         const expandedFolders = new Set(this.expandedFolders);