@in-the-loop-labs/pair-review 2.6.3 → 3.0.0

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

@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-3.0-or-later
+// Copyright 2026 Tim Perkins (tjwp) | SPDX-License-Identifier: Apache-2.0
 /**
  * Analysis History Manager
  *
@@ -20,11 +20,12 @@ class AnalysisHistoryManager {
    * @param {Function} options.onSelectionChange - Callback when a run is selected, receives (runId, run)
    * @param {string} options.containerPrefix - Prefix for DOM element IDs (default: 'analysis-context')
    */
-  constructor({ reviewId, mode, onSelectionChange, containerPrefix = 'analysis-context' }) {
+  constructor({ reviewId, mode, onSelectionChange, containerPrefix = 'analysis-context', shaAbbrevLength = 7 }) {
     this.reviewId = reviewId;
     this.mode = mode;
     this.onSelectionChange = onSelectionChange;
     this.containerPrefix = containerPrefix;
+    this.shaAbbrevLength = shaAbbrevLength;
 
     // State
     this.runs = [];
@@ -421,7 +422,7 @@ class AnalysisHistoryManager {
 
     // Format HEAD SHA - show abbreviated version with full SHA in title
     const headSha = run.head_sha;
-    const headShaDisplay = headSha ? headSha.substring(0, 7) : null;
+    const headShaDisplay = headSha ? headSha.substring(0, this.shaAbbrevLength) : null;
 
     // Format status
     const statusInfo = this.formatStatus(run.status);

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

@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-3.0-or-later
+// Copyright 2026 Tim Perkins (tjwp) | SPDX-License-Identifier: Apache-2.0
 /**
  * CommentManager - Comment UI handling
  * Handles comment forms, editing, saving, deletion, and display.
@@ -500,6 +500,11 @@ class CommentManager {
         this.prManager.updateCommentCount();
       }
 
+      // Refresh minimize-mode indicators so the new comment is reflected
+      if (window.prManager?.commentMinimizer) {
+        window.prManager.commentMinimizer.refreshIndicators();
+      }
+
     } catch (error) {
       console.error('Error saving comment:', error);
       alert('Failed to save comment');

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

@@ -0,0 +1,304 @@
+// Copyright 2026 Tim Perkins (tjwp) | SPDX-License-Identifier: Apache-2.0
+/**
+ * CommentMinimizer - Manages "minimize comments" mode for the diff view.
+ *
+ * When active, all inline comment rows (.user-comment-row) and AI suggestion
+ * rows (.ai-suggestion-row) are hidden via CSS class.  Small indicator buttons
+ * 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.
+ */
+
+class CommentMinimizer {
+  /** Person icon SVG (matches comment-manager.js octicon-person) */
+  static PERSON_ICON = `<svg viewBox="0 0 16 16" width="14" height="14" fill="currentColor"><path d="M10.561 8.073a6.005 6.005 0 0 1 3.432 5.142.75.75 0 1 1-1.498.07 4.5 4.5 0 0 0-8.99 0 .75.75 0 0 1-1.498-.07 6.004 6.004 0 0 1 3.431-5.142 3.999 3.999 0 1 1 5.123 0ZM10.5 5a2.5 2.5 0 1 0-5 0 2.5 2.5 0 0 0 5 0Z"/></svg>`;
+
+  /** Sparkles icon SVG (matches AI suggestion badge) */
+  static SPARKLES_ICON = `<svg viewBox="0 0 16 16" width="14" height="14" fill="currentColor"><path d="M9.6 2.279a.426.426 0 0 1 .8 0l.407 1.112a6.386 6.386 0 0 0 3.802 3.802l1.112.407a.426.426 0 0 1 0 .8l-1.112.407a6.386 6.386 0 0 0-3.802 3.802l-.407 1.112a.426.426 0 0 1-.8 0l-.407-1.112a6.386 6.386 0 0 0-3.802-3.802L4.279 8.4a.426.426 0 0 1 0-.8l1.112-.407a6.386 6.386 0 0 0 3.802-3.802L9.6 2.279Zm-4.267 8.837a.178.178 0 0 1 .334 0l.169.464a2.662 2.662 0 0 0 1.584 1.584l.464.169a.178.178 0 0 1 0 .334l-.464.169a2.662 2.662 0 0 0-1.584 1.584l-.169.464a.178.178 0 0 1-.334 0l-.169-.464a2.662 2.662 0 0 0-1.584-1.584l-.464-.169a.178.178 0 0 1 0-.334l.464-.169a2.662 2.662 0 0 0 1.584-1.584l.169-.464ZM2.8.14a.213.213 0 0 1 .4 0l.203.556a3.2 3.2 0 0 0 1.901 1.901l.556.203a.213.213 0 0 1 0 .4l-.556.203a3.2 3.2 0 0 0-1.901 1.901L3.2 5.86a.213.213 0 0 1-.4 0l-.203-.556A3.2 3.2 0 0 0 .696 3.403L.14 3.2a.213.213 0 0 1 0-.4l.556-.203A3.2 3.2 0 0 0 2.597.696L2.8.14Z"/></svg>`;
+
+  /** AI comment icon SVG — speech bubble with sparkles (for adopted AI suggestions) */
+  static AI_COMMENT_ICON = `<svg viewBox="0 0 16 16" width="14" height="14" fill="currentColor"><path d="M7.75 1a.75.75 0 0 1 0 1.5h-5a.25.25 0 0 0-.25.25v7.5c0 .138.112.25.25.25h2c.199 0 .39.079.53.22.141.14.22.331.22.53v2.19l2.72-2.72a.747.747 0 0 1 .53-.22h4.5a.25.25 0 0 0 .25-.25v-2a.75.75 0 0 1 1.5 0v2c0 .464-.184.909-.513 1.237A1.746 1.746 0 0 1 13.25 12H9.06l-2.573 2.573A1.457 1.457 0 0 1 4 13.543V12H2.75A1.75 1.75 0 0 1 1 10.25v-7.5C1 1.784 1.784 1 2.75 1h5Zm4.519-.837a.248.248 0 0 1 .466 0l.238.648a3.726 3.726 0 0 0 2.218 2.219l.649.238a.249.249 0 0 1 0 .467l-.649.238a3.725 3.725 0 0 0-2.218 2.218l-.238.649a.248.248 0 0 1-.466 0l-.239-.649a3.725 3.725 0 0 0-2.218-2.218l-.649-.238a.249.249 0 0 1 0-.467l.649-.238A3.726 3.726 0 0 0 12.03.811l.239-.648Z"/></svg>`;
+
+  constructor() {
+    this._active = false;
+    // Track which diff lines have been expanded by the user (Set of diff row elements)
+    this._expandedLines = new Set();
+  }
+
+  /** @returns {boolean} Whether minimize mode is active */
+  get active() {
+    return this._active;
+  }
+
+  /**
+   * Enable or disable minimize mode.
+   * @param {boolean} minimized
+   */
+  setMinimized(minimized) {
+    this._active = minimized;
+    this._expandedLines.clear();
+
+    const diffContainer = document.getElementById('diff-container');
+    if (!diffContainer) return;
+
+    if (minimized) {
+      diffContainer.classList.add('comments-minimized');
+      this.refreshIndicators();
+    } else {
+      diffContainer.classList.remove('comments-minimized');
+      this._removeAllIndicators();
+      // Remove any per-line expansion overrides
+      document.querySelectorAll('.comment-expanded').forEach(el => el.classList.remove('comment-expanded'));
+    }
+  }
+
+  /**
+   * Rebuild all indicator buttons on diff lines.
+   * Call this after comments or suggestions are added/removed/re-rendered.
+   */
+  refreshIndicators() {
+    if (!this._active) return;
+
+    this._removeAllIndicators();
+
+    // Find all comment and suggestion rows currently in the DOM
+    const commentRows = document.querySelectorAll('.user-comment-row');
+    const suggestionRows = document.querySelectorAll('.ai-suggestion-row');
+
+    // Build a map: diff row element → { hasUser, hasAI, hasAdopted, userCount, aiCount, adoptedCount }
+    const lineMap = new Map();
+
+    for (const row of commentRows) {
+      const diffRow = this._findDiffRowFor(row);
+      if (!diffRow) continue;
+      const entry = lineMap.get(diffRow) || { hasUser: false, hasAI: false, hasAdopted: false, userCount: 0, aiCount: 0, adoptedCount: 0 };
+      if (row.querySelector('.adopted-comment')) {
+        entry.hasAdopted = true;
+        entry.adoptedCount++;
+      } else {
+        entry.hasUser = true;
+        entry.userCount++;
+      }
+      lineMap.set(diffRow, entry);
+    }
+
+    for (const row of suggestionRows) {
+      const diffRow = this._findDiffRowFor(row);
+      if (!diffRow) continue;
+      const entry = lineMap.get(diffRow) || { hasUser: false, hasAI: false, hasAdopted: false, userCount: 0, aiCount: 0, adoptedCount: 0 };
+      // Count non-adopted suggestion divs only — adopted ones are already
+      // represented by the adopted comment row (avoid double-counting)
+      const allSuggestions = row.querySelectorAll('.ai-suggestion');
+      let activeCount = 0;
+      for (const s of allSuggestions) {
+        if (!s.dataset?.hiddenForAdoption) {
+          activeCount++;
+        }
+      }
+      if (activeCount > 0) {
+        entry.hasAI = true;
+        entry.aiCount += activeCount;
+      }
+      lineMap.set(diffRow, entry);
+    }
+
+    // Inject indicators
+    for (const [diffRow, info] of lineMap) {
+      this._injectIndicator(diffRow, info);
+    }
+  }
+
+  /**
+   * Walk backward from a comment/suggestion row to find its parent diff line.
+   * Skips other comment rows, suggestion rows, and context-expand rows.
+   * @param {HTMLElement} row
+   * @returns {HTMLElement|null}
+   */
+  _findDiffRowFor(row) {
+    let prev = row.previousElementSibling;
+    while (prev) {
+      if (
+        !prev.classList.contains('user-comment-row') &&
+        !prev.classList.contains('ai-suggestion-row') &&
+        !prev.classList.contains('comment-form-row') &&
+        !prev.classList.contains('context-expand-row')
+      ) {
+        return prev;
+      }
+      prev = prev.previousElementSibling;
+    }
+    return null;
+  }
+
+  /**
+   * Inject an indicator button into a diff line's code cell.
+   * @param {HTMLElement} diffRow - The diff table row
+   * @param {Object} info - { hasUser, hasAI, hasAdopted, userCount, aiCount, adoptedCount }
+   */
+  _injectIndicator(diffRow, info) {
+    const codeCell = diffRow.querySelector('.d2h-code-line-ctn');
+    if (!codeCell) return;
+
+    // Don't double-inject
+    if (codeCell.querySelector('.comment-indicator')) return;
+
+    const btn = document.createElement('button');
+    btn.className = 'comment-indicator';
+    btn.type = 'button';
+
+    // Build icon content — three types:
+    //   person (purple)   = user-originated comments
+    //   ai-comment (purple) = adopted AI suggestions
+    //   sparkles (amber)  = AI suggestions
+    const icons = [];
+    if (info.hasUser) {
+      icons.push(`<span class="indicator-icon indicator-user" title="${info.userCount} comment${info.userCount !== 1 ? 's' : ''}">${CommentMinimizer.PERSON_ICON}</span>`);
+    }
+    if (info.hasAdopted) {
+      icons.push(`<span class="indicator-icon indicator-adopted" title="${info.adoptedCount} adopted comment${info.adoptedCount !== 1 ? 's' : ''}">${CommentMinimizer.AI_COMMENT_ICON}</span>`);
+    }
+    if (info.hasAI) {
+      icons.push(`<span class="indicator-icon indicator-ai" title="${info.aiCount} suggestion${info.aiCount !== 1 ? 's' : ''}">${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} comment${info.userCount !== 1 ? 's' : ''}`);
+    if (info.adoptedCount) totalLabel.push(`${info.adoptedCount} adopted comment${info.adoptedCount !== 1 ? 's' : ''}`);
+    if (info.aiCount) totalLabel.push(`${info.aiCount} suggestion${info.aiCount !== 1 ? 's' : ''}`);
+    btn.title = totalLabel.join(', ');
+
+    // Check if this line was previously expanded
+    if (this._expandedLines.has(diffRow)) {
+      btn.classList.add('expanded');
+    }
+
+    // Click handler: toggle this line's comments
+    btn.addEventListener('click', (e) => {
+      e.stopPropagation();
+      e.preventDefault();
+      this._toggleLineComments(diffRow, btn);
+    });
+
+    // Make the code cell position:relative for absolute positioning of the indicator
+    codeCell.style.position = 'relative';
+    codeCell.appendChild(btn);
+  }
+
+  /**
+   * Toggle visibility of comment/suggestion rows for a specific diff line.
+   * @param {HTMLElement} diffRow
+   * @param {HTMLElement} btn - The indicator button
+   */
+  _toggleLineComments(diffRow, btn) {
+    const isExpanded = this._expandedLines.has(diffRow);
+
+    if (isExpanded) {
+      // Collapse: remove .comment-expanded from this line's rows
+      this._expandedLines.delete(diffRow);
+      btn.classList.remove('expanded');
+      this._getCommentRowsFor(diffRow).forEach(row => row.classList.remove('comment-expanded'));
+    } else {
+      // Expand: add .comment-expanded to this line's rows
+      this._expandedLines.add(diffRow);
+      btn.classList.add('expanded');
+      this._getCommentRowsFor(diffRow).forEach(row => row.classList.add('comment-expanded'));
+    }
+  }
+
+  /**
+   * Get all comment/suggestion rows that belong to a diff line.
+   * Walks forward from the diff row, collecting adjacent comment/suggestion rows.
+   * @param {HTMLElement} diffRow
+   * @returns {HTMLElement[]}
+   */
+  _getCommentRowsFor(diffRow) {
+    const rows = [];
+    let next = diffRow.nextElementSibling;
+    while (next) {
+      if (
+        next.classList.contains('user-comment-row') ||
+        next.classList.contains('ai-suggestion-row')
+      ) {
+        rows.push(next);
+      } else if (
+        next.classList.contains('comment-form-row') ||
+        next.classList.contains('context-expand-row')
+      ) {
+        // Skip these but keep looking
+        next = next.nextElementSibling;
+        continue;
+      } else {
+        // Hit another diff line — stop
+        break;
+      }
+      next = next.nextElementSibling;
+    }
+    return rows;
+  }
+
+  /**
+   * Find the parent diff row for a given comment/suggestion element.
+   * Public wrapper around _findDiffRowFor that first locates the containing
+   * comment/suggestion row from any child element.
+   * @param {HTMLElement} element - Any element inside (or equal to) a comment/suggestion row
+   * @returns {HTMLElement|null} The parent diff row, or null
+   */
+  findDiffRowFor(element) {
+    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 null;
+    }
+    return this._findDiffRowFor(commentRow);
+  }
+
+  // TODO: expose via API route so chat can programmatically expand findings when discussing them
+  /**
+   * Expand comments for a given element so it becomes visible when minimized.
+   * Call this before scrolling to a comment/suggestion row that may be hidden.
+   * @param {HTMLElement} element - The target comment/suggestion element (or row)
+   */
+  expandForElement(element) {
+    if (!this._active) return;
+
+    // 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;
+    }
+
+    // Find the parent diff row for this comment row
+    const diffRow = this._findDiffRowFor(commentRow);
+    if (!diffRow) return;
+
+    // Already expanded — nothing to do
+    if (this._expandedLines.has(diffRow)) return;
+
+    // Expand all comment rows for this diff line
+    this._expandedLines.add(diffRow);
+    this._getCommentRowsFor(diffRow).forEach(row => row.classList.add('comment-expanded'));
+
+    // Update the indicator button's expanded state
+    const btn = diffRow.querySelector('.d2h-code-line-ctn .comment-indicator');
+    if (btn) {
+      btn.classList.add('expanded');
+    }
+  }
+
+  /** Remove all indicator buttons from the DOM. */
+  _removeAllIndicators() {
+    document.querySelectorAll('.comment-indicator').forEach(btn => btn.remove());
+  }
+}
+
+window.CommentMinimizer = CommentMinimizer;
+
+// Export for testing
+if (typeof module !== 'undefined' && module.exports) {
+  module.exports = { CommentMinimizer };
+}

package/public/js/modules/diff-context.js

@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-3.0-or-later
+// Copyright 2026 Tim Perkins (tjwp) | SPDX-License-Identifier: Apache-2.0
 /**
  * DiffContext - Extract diff hunks for chat context enrichment.
  * Provides utilities to pull relevant unified diff sections for

package/public/js/modules/diff-renderer.js

@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-3.0-or-later
+// Copyright 2026 Tim Perkins (tjwp) | SPDX-License-Identifier: Apache-2.0
 /**
  * DiffRenderer - Diff parsing and line rendering
  * Handles rendering of diff content, syntax highlighting,

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

@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-3.0-or-later
+// Copyright 2026 Tim Perkins (tjwp) | SPDX-License-Identifier: Apache-2.0
 /**
  * FileCommentManager - File-level comment UI handling
  * Handles file-level comments zone rendering, forms, and interactions.

package/public/js/modules/file-list-merger.js

@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-3.0-or-later
+// Copyright 2026 Tim Perkins (tjwp) | SPDX-License-Identifier: Apache-2.0
 /**
  * File List Merger - Merges diff files with context files for sidebar display.
  *

package/public/js/modules/gap-coordinates.js

@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-3.0-or-later
+// Copyright 2026 Tim Perkins (tjwp) | SPDX-License-Identifier: Apache-2.0
 /**
  * Gap Coordinates - Coordinate system utilities for diff gap expansion
  *

package/public/js/modules/hunk-parser.js

@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-3.0-or-later
+// Copyright 2026 Tim Perkins (tjwp) | SPDX-License-Identifier: Apache-2.0
 /**
  * HunkParser - Hunk header parsing and gap context expansion
  * Handles parsing of unified diff hunk headers and expansion of collapsed sections

package/public/js/modules/line-tracker.js

@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-3.0-or-later
+// Copyright 2026 Tim Perkins (tjwp) | SPDX-License-Identifier: Apache-2.0
 /**
  * LineTracker - Line number mapping, range selection, and highlighting
  * Handles line number extraction, range selection for multi-line comments,

package/public/js/modules/panel-resizer.js

@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-3.0-or-later
+// Copyright 2026 Tim Perkins (tjwp) | SPDX-License-Identifier: Apache-2.0
 /**
  * Panel Resizer Module
  * Handles drag-to-resize functionality for the sidebar and AI panel

package/public/js/modules/storage-cleanup.js

@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-3.0-or-later
+// Copyright 2026 Tim Perkins (tjwp) | SPDX-License-Identifier: Apache-2.0
 /**
  * Storage Cleanup Module
  * Cleans up legacy localStorage keys from older versions of the app.

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

@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-3.0-or-later
+// Copyright 2026 Tim Perkins (tjwp) | SPDX-License-Identifier: Apache-2.0
 /**
  * SuggestionManager - AI suggestion handling
  * Handles display, adopt, dismiss, and restore of AI suggestions.

package/public/js/pr.js

@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-3.0-or-later
+// Copyright 2026 Tim Perkins (tjwp) | SPDX-License-Identifier: Apache-2.0
 /**
  * Pull Request UI Management
  * Main orchestrator that coordinates the extracted modules:
@@ -136,6 +136,8 @@ class PRManager {
     this.hideWhitespace = false;
     // Diff options dropdown (gear icon popover)
     this.diffOptionsDropdown = null;
+    // Comment minimizer — manages minimize mode indicators
+    this.commentMinimizer = window.CommentMinimizer ? new window.CommentMinimizer() : null;
     // Cached staleness check promise — shared between on-load and triggerAIAnalysis
     this._stalenessPromise = null;
     // Unique client ID for self-echo suppression on WebSocket review events.
@@ -192,6 +194,7 @@ class PRManager {
     if (diffOptionsBtn && window.DiffOptionsDropdown) {
       this.diffOptionsDropdown = new window.DiffOptionsDropdown(diffOptionsBtn, {
         onToggleWhitespace: (hide) => this.handleWhitespaceToggle(hide),
+        onToggleMinimize: (minimized) => this.handleMinimizeToggle(minimized),
       });
     }
 
@@ -461,6 +464,11 @@ class PRManager {
       // Render PR header with metadata
       this.renderPRHeader(prData);
 
+      // Set descriptive tab title
+      if (window.tabTitle) {
+        window.tabTitle.setBase(`PR #${number}`);
+      }
+
       // Fetch diff and file list from diff endpoint
       await this.loadAndDisplayFiles(owner, repo, number);
 
@@ -491,6 +499,7 @@ class PRManager {
         this.analysisHistoryManager = new window.AnalysisHistoryManager({
           reviewId: this.currentPR.id,
           mode: 'pr',
+          shaAbbrevLength: this.currentPR.shaAbbrevLength || 7,
           onSelectionChange: (runId, _run) => {
             this.selectedRunId = runId;
             this.loadAISuggestions(null, runId);
@@ -757,6 +766,18 @@ class PRManager {
     });
   }
 
+  /**
+   * Handle the minimize comments toggle from DiffOptionsDropdown.
+   * Toggles minimize mode which hides inline comments/suggestions and
+   * shows compact indicators on diff lines instead.
+   * @param {boolean} minimized - Whether to minimize comments
+   */
+  handleMinimizeToggle(minimized) {
+    if (this.commentMinimizer) {
+      this.commentMinimizer.setMinimized(minimized);
+    }
+  }
+
   /**
    * Parse unified diff to extract per-file patches
    * @param {string} diff - Full unified diff
@@ -879,7 +900,8 @@ class PRManager {
     const commitSha = document.getElementById('pr-commit-sha');
     const commitCopy = document.getElementById('pr-commit-copy');
     if (commitSha && pr.head_sha) {
-      commitSha.textContent = pr.head_sha.substring(0, 7);
+      const abbrevLen = pr.shaAbbrevLength || 7;
+      commitSha.textContent = pr.head_sha.substring(0, abbrevLen);
       // Store full SHA for copying (updates on refresh)
       commitSha.dataset.fullSha = pr.head_sha;
 
@@ -2453,6 +2475,11 @@ class PRManager {
         }
       }
 
+      // Refresh minimize-mode indicators so deleted comments no longer show
+      if (this.commentMinimizer) {
+        this.commentMinimizer.refreshIndicators();
+      }
+
       // Show success toast
       if (window.toast) {
         window.toast.showSuccess('Comment dismissed');
@@ -2673,6 +2700,11 @@ class PRManager {
       }
 
       this.updateCommentCount();
+
+      // Refresh minimize-mode indicators (no-op when minimize mode is off)
+      if (this.commentMinimizer) {
+        this.commentMinimizer.refreshIndicators();
+      }
     } catch (error) {
       console.error('Error loading user comments:', error);
     }
@@ -2753,7 +2785,11 @@ class PRManager {
   }
 
   async displayAISuggestions(suggestions) {
-    return this.suggestionManager.displayAISuggestions(suggestions);
+    await this.suggestionManager.displayAISuggestions(suggestions);
+    // Refresh minimize-mode indicators (no-op when minimize mode is off)
+    if (this.commentMinimizer) {
+      this.commentMinimizer.refreshIndicators();
+    }
   }
 
   createSuggestionRow(suggestions) {
@@ -2867,6 +2903,11 @@ class PRManager {
     }
 
     this.updateCommentCount();
+
+    // Refresh minimize-mode indicators so the adopted comment is reflected
+    if (this.commentMinimizer) {
+      this.commentMinimizer.refreshIndicators();
+    }
   }
 
   /**
@@ -2991,6 +3032,11 @@ class PRManager {
       if (window.aiPanel) {
         window.aiPanel.updateFindingStatus(suggestionId, 'dismissed');
       }
+
+      // Refresh minimize-mode indicators after suggestion state change
+      if (this.commentMinimizer) {
+        this.commentMinimizer.refreshIndicators();
+      }
     } catch (error) {
       console.error('Error dismissing suggestion:', error);
       alert('Failed to dismiss suggestion');
@@ -3041,6 +3087,11 @@ class PRManager {
       if (window.aiPanel) {
         window.aiPanel.updateFindingStatus(suggestionId, 'active');
       }
+
+      // Refresh minimize-mode indicators after suggestion state change
+      if (this.commentMinimizer) {
+        this.commentMinimizer.refreshIndicators();
+      }
     } catch (error) {
       console.error('Error restoring suggestion:', error);
       alert('Failed to restore suggestion');
@@ -4365,6 +4416,14 @@ class PRManager {
       const hasData = await this._hasActiveSessionData();
       if (hasData) {
         this._showStaleBadge('stale');
+        if (window.chatPanel) {
+          const abbrevLen = this.currentPR?.shaAbbrevLength || 7;
+          const oldSha = result.localHeadSha ? result.localHeadSha.substring(0, abbrevLen) : 'unknown';
+          const newSha = result.remoteHeadSha ? result.remoteHeadSha.substring(0, abbrevLen) : 'unknown';
+          window.chatPanel.queueDiffStateNotification(
+            `PR HEAD has changed (${oldSha} → ${newSha}). The diff has not been refreshed yet.`
+          );
+        }
       } else {
         // No user work to protect — refresh silently
         await this.refreshPR();
@@ -4427,8 +4486,9 @@ class PRManager {
   /**
    * Show the stale badge with an optional variant class.
    * @param {'stale'|'closed'|'merged'} type
+   * @param {string} [title] - Optional custom tooltip text. Falls back to type-specific defaults.
    */
-  _showStaleBadge(type) {
+  _showStaleBadge(type, title) {
     const badge = document.getElementById('stale-badge');
     if (!badge) return;
 
@@ -4439,14 +4499,14 @@ class PRManager {
     if (type === 'merged') {
       badge.classList.add('pr-merged');
       if (textEl) textEl.textContent = 'MERGED';
-      badge.title = 'This PR has been merged';
+      badge.title = title || 'This PR has been merged';
     } else if (type === 'closed') {
       badge.classList.add('pr-closed');
       if (textEl) textEl.textContent = 'CLOSED';
-      badge.title = 'This PR has been closed';
+      badge.title = title || 'This PR has been closed';
     } else {
       if (textEl) textEl.textContent = 'STALE';
-      badge.title = 'PR data is outdated';
+      badge.title = title || 'PR data is outdated';
     }
     badge.style.display = '';
   }
@@ -4482,6 +4542,8 @@ class PRManager {
       diffContainer.innerHTML = '<div class="loading">Refreshing pull request...</div>';
     }
 
+    const oldHeadSha = this.currentPR?.head_sha;
+
     try {
       // Call refresh API endpoint to fetch fresh data from GitHub
       const response = await fetch(`/api/pr/${owner}/${repo}/${number}/refresh`, {
@@ -4499,6 +4561,15 @@ class PRManager {
       if (data.success && data.data) {
         this.currentPR = data.data;
 
+        // Notify chat agent if HEAD SHA changed
+        const newHeadSha = data.data?.head_sha;
+        if (window.chatPanel && oldHeadSha && newHeadSha && oldHeadSha !== newHeadSha) {
+          const abbrevLen = this.currentPR?.shaAbbrevLength || 7;
+          window.chatPanel.queueDiffStateNotification(
+            `PR refreshed. HEAD changed: ${oldHeadSha.substring(0, abbrevLen)} → ${newHeadSha.substring(0, abbrevLen)}.`
+          );
+        }
+
         // Save scroll position and expanded state
         const scrollPosition = window.scrollY;
         const expandedFolders = new Set(this.expandedFolders);
@@ -5149,6 +5220,11 @@ if (typeof document !== 'undefined') {
       window.PanelResizer.init();
     }
 
+    // Initialize tab title manager
+    if (typeof TabTitle !== 'undefined') {
+      window.tabTitle = new TabTitle();
+    }
+
     prManager = new PRManager();
     // CRITICAL FIX: Make prManager available globally for component access
     window.prManager = prManager;

package/public/js/repo-settings.js

@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-3.0-or-later
+// Copyright 2026 Tim Perkins (tjwp) | SPDX-License-Identifier: Apache-2.0
 /**
  * Repository Settings Page JavaScript
  * Handles loading, saving, and managing repository AI settings

package/public/js/utils/category-emoji.js

@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-3.0-or-later
+// Copyright 2026 Tim Perkins (tjwp) | SPDX-License-Identifier: Apache-2.0
 /**
  * Canonical category-to-emoji mapping for AI suggestion types.
  * Used by SuggestionManager and FileCommentManager to format adopted comments.

package/public/js/utils/file-order.js

@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-3.0-or-later
+// Copyright 2026 Tim Perkins (tjwp) | SPDX-License-Identifier: Apache-2.0
 /**
  * File ordering utilities for consistent file display across components.
  *

package/public/js/utils/markdown.js

@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-3.0-or-later
+// Copyright 2026 Tim Perkins (tjwp) | SPDX-License-Identifier: Apache-2.0
 /**
  * Safe markdown renderer for comments
  * Uses markdown-it with security settings to prevent XSS