@in-the-loop-labs/pair-review 3.0.6 → 3.1.0

package/public/js/local.js

@@ -73,7 +73,15 @@ class LocalManager {
     const autoAnalyze = new URLSearchParams(window.location.search).get('analyze');
     if (autoAnalyze === 'true' && !window.prManager.isAnalyzing) {
       try {
-        await this.startLocalAnalysis(null, {});
+        // Fetch repo settings so we honour the repository's default provider/council
+        const manager = window.prManager;
+        const [repoSettings, reviewSettings] = await Promise.all([
+          manager.fetchRepoSettings().catch(() => null),
+          manager.fetchLastReviewSettings().catch(() => ({ custom_instructions: '', last_council_id: null }))
+        ]);
+        const config = await manager._buildDefaultAnalysisConfig(repoSettings, reviewSettings);
+
+        await this.startLocalAnalysis(null, config);
       } finally {
         const cleanUrl = new URL(window.location);
         cleanUrl.searchParams.delete('analyze');
@@ -260,10 +268,11 @@ class LocalManager {
           ? manager._stalenessPromise
           : self._fetchLocalStaleness();
         manager._stalenessPromise = null; // consume it
-        const [staleResult, repoSettings, reviewSettings] = await Promise.all([
+        const [staleResult, repoSettings, reviewSettings, appConfig] = await Promise.all([
           staleCheckWithTimeout,
           manager.fetchRepoSettings().catch(() => null),
-          manager.fetchLastReviewSettings().catch(() => ({ custom_instructions: '', last_council_id: null }))
+          manager.fetchLastReviewSettings().catch(() => ({ custom_instructions: '', last_council_id: null })),
+          fetch('/api/config').then(r => r.ok ? r.json() : {}).catch(() => ({}))
         ]);
         console.debug(`[Analyze] parallel-fetch (stale+settings): ${Math.round(performance.now() - _tParallel0)}ms`);
 
@@ -329,7 +338,9 @@ class LocalManager {
           repoInstructions: repoSettings?.default_instructions || '',
           lastInstructions: lastInstructions,
           lastCouncilId,
-          defaultCouncilId: repoSettings?.default_council_id || null
+          defaultCouncilId: repoSettings?.default_council_id || null,
+          hasPr: false,
+          hasGithubToken: Boolean(appConfig.has_github_token)
         });
 
         if (!config) {
@@ -376,7 +387,8 @@ class LocalManager {
               null,
               {
                 configType: data.status?.isCouncil ? (data.status.configType || 'advanced') : 'single',
-                enabledLevels: data.status?.enabledLevels || [1, 2, 3]
+                enabledLevels: data.status?.enabledLevels || [1, 2, 3],
+                noLevels: data.status?.noLevels || false
               }
             );
           }
@@ -480,6 +492,60 @@ class LocalManager {
       });
     };
 
+    // Base branch override for stack-aware diff in local mode
+    manager.currentBaseOverride = null;
+
+    // Render the base branch selector dropdown for stacked branches.
+    // When a local review has stack_data with 3+ entries, the user can pick
+    // which ancestor branch to diff against.
+    // Render the base branch selector when a Graphite stack has multiple ancestors.
+    // When shown, the selector replaces the static base branch text in the toolbar.
+    manager.renderBaseBranchSelector = function(pr) {
+      const selectorWrap = document.getElementById('base-branch-selector-wrap');
+      const sel = document.getElementById('base-branch-select');
+      const staticBase = document.getElementById('toolbar-base-branch-static');
+      if (!selectorWrap || !sel) return;
+
+      // Hide selector if no stack data or fewer than 3 entries (need at least 2 ancestors to switch between)
+      if (!pr.stack_data || pr.stack_data.length < 3) {
+        selectorWrap.setAttribute('hidden', '');
+        if (staticBase) staticBase.removeAttribute('hidden');
+        return;
+      }
+
+      // Ancestors = all stack entries except the last (current branch)
+      const ancestors = pr.stack_data.slice(0, -1);
+
+      // Build options using createElement for XSS safety
+      sel.innerHTML = '';
+      for (const entry of ancestors) {
+        const option = document.createElement('option');
+        option.value = entry.branch;
+        option.textContent = entry.prNumber ? `${entry.branch} (#${entry.prNumber})` : entry.branch;
+        if (entry.branch === pr.base_branch) {
+          option.selected = true;
+        }
+        sel.appendChild(option);
+      }
+
+      // Show selector, hide static text
+      selectorWrap.removeAttribute('hidden');
+      if (staticBase) staticBase.setAttribute('hidden', '');
+
+      // Wire up change listener (idempotent via data-listener-added pattern)
+      if (!sel.hasAttribute('data-listener-added')) {
+        sel.setAttribute('data-listener-added', 'true');
+        sel.addEventListener('change', async () => {
+          manager.currentBaseOverride = sel.value;
+          // If selection matches the original base, clear the override
+          if (sel.value === manager.currentPR.base_branch) {
+            manager.currentBaseOverride = null;
+          }
+          await self.loadLocalDiff();
+        });
+      }
+    };
+
     console.log('PRManager patched for local mode');
   }
 
@@ -509,7 +575,8 @@ class LocalManager {
           councilId: config.councilId || undefined,
           councilConfig: config.councilConfig || undefined,
           configType: config.configType || 'advanced',
-          customInstructions: config.customInstructions || null
+          customInstructions: config.customInstructions || null,
+          excludePrevious: config.excludePrevious || undefined
         };
       } else {
         analyzeUrl = `/api/local/${this.reviewId}/analyses`;
@@ -519,7 +586,8 @@ class LocalManager {
           tier: config.tier || 'balanced',
           customInstructions: config.customInstructions || null,
           enabledLevels: config.enabledLevels || [1, 2, 3],
-          skipLevel3: config.skipLevel3 || false
+          skipLevel3: config.skipLevel3 || false,
+          excludePrevious: config.excludePrevious || undefined
         };
       }
 
@@ -556,7 +624,8 @@ class LocalManager {
           config.isCouncil ? config.councilName : null,
           {
             configType: config.isCouncil ? (config.configType || 'advanced') : 'single',
-            enabledLevels: config.enabledLevels || [1, 2, 3]
+            enabledLevels: config.enabledLevels || [1, 2, 3],
+            noLevels: config.noLevels || false
           }
         );
       }
@@ -647,7 +716,7 @@ class LocalManager {
     try {
       // Show loading state
       refreshBtn.disabled = true;
-      refreshBtn.classList.add('btn-loading');
+      refreshBtn.classList.add('refreshing');
 
       const response = await fetch(`/api/local/${this.reviewId}/refresh`, {
         method: 'POST'
@@ -698,7 +767,7 @@ class LocalManager {
       // Reset button state
       if (refreshBtn) {
         refreshBtn.disabled = false;
-        refreshBtn.classList.remove('btn-loading');
+        refreshBtn.classList.remove('refreshing');
       }
     }
   }
@@ -799,6 +868,13 @@ class LocalManager {
       );
     }
 
+    // Reset base branch override before reloading diff so the fetch uses the default base
+    manager.currentBaseOverride = null;
+    const baseSel = document.getElementById('base-branch-select');
+    if (baseSel && manager.currentPR?.base_branch) {
+      baseSel.value = manager.currentPR.base_branch;
+    }
+
     // Reload the diff display
     await this.loadLocalDiff();
 
@@ -947,7 +1023,8 @@ class LocalManager {
         head_sha: reviewData.localHeadSha,
         shaAbbrevLength: reviewData.shaAbbrevLength || 7,
         reviewType: 'local',
-        localPath: reviewData.localPath
+        localPath: reviewData.localPath,
+        stack_data: reviewData.stackData || null
       };
 
       // Re-initialize DiffOptionsDropdown with scope options
@@ -1131,33 +1208,61 @@ class LocalManager {
       pathText.title = fullPath;
     }
 
-    // Update branch name (header badge)
+    // Update branch name in header badge
     const branchText = document.getElementById('local-branch-text');
     if (branchText) {
       branchText.textContent = reviewData.branch || 'unknown';
     }
 
+    // Wire up header branch copy button
+    const branchCopy = document.getElementById('local-branch-copy');
+    if (branchCopy && !branchCopy.hasAttribute('data-listener-added')) {
+      branchCopy.setAttribute('data-listener-added', 'true');
+      branchCopy.addEventListener('click', async (e) => {
+        e.stopPropagation();
+        const branch = branchText ? branchText.textContent : '';
+        if (!branch || branch === '--' || branch === 'unknown') return;
+        try {
+          await navigator.clipboard.writeText(branch);
+          branchCopy.classList.add('copied');
+          setTimeout(() => branchCopy.classList.remove('copied'), 2000);
+        } catch (err) {
+          console.error('Failed to copy branch name:', err);
+        }
+      });
+    }
+
     // Set descriptive tab title
     if (window.tabTitle && reviewData.branch) {
       window.tabTitle.setBase(reviewData.branch);
     }
 
-    // Show base branch badge when branch is in scope
+    // Show base branch in toolbar when branch is in scope
     const LS = window.LocalScope;
     const scopeStart = this.scopeStart || (LS ? LS.DEFAULT_SCOPE.start : 'unstaged');
     const scopeEnd = this.scopeEnd || (LS ? LS.DEFAULT_SCOPE.end : 'untracked');
     const hasBranch = LS ? LS.scopeIncludes(scopeStart, scopeEnd, 'branch') : false;
 
+    // Toolbar base branch display (static text, selector is wired separately)
+    const toolbarBaseWrap = document.getElementById('toolbar-base-branch-wrap');
+    const toolbarBaseStatic = document.getElementById('toolbar-base-branch-static');
+    const toolbarBaseText = document.getElementById('toolbar-base-branch-text');
+    if (hasBranch && reviewData.baseBranch) {
+      if (toolbarBaseText) toolbarBaseText.textContent = reviewData.baseBranch;
+      if (toolbarBaseWrap) toolbarBaseWrap.removeAttribute('hidden');
+    } else {
+      if (toolbarBaseWrap) toolbarBaseWrap.setAttribute('hidden', '');
+    }
+
+    // Hide header branch display — toolbar now shows branch info
     const branchVs = document.getElementById('local-branch-vs');
     const baseBranchEl = document.getElementById('local-base-branch');
     const baseBranchText = document.getElementById('local-base-branch-text');
-    if (hasBranch && reviewData.baseBranch) {
-      if (branchVs) branchVs.style.display = '';
-      if (baseBranchEl) baseBranchEl.style.display = '';
-      if (baseBranchText) baseBranchText.textContent = reviewData.baseBranch;
-    } else {
-      if (branchVs) branchVs.style.display = 'none';
-      if (baseBranchEl) baseBranchEl.style.display = 'none';
+    if (branchVs) branchVs.style.display = 'none';
+    if (baseBranchEl) baseBranchEl.style.display = 'none';
+    // Keep baseBranchText updated for data purposes even though header is hidden
+    if (baseBranchText && reviewData.baseBranch) {
+      baseBranchText.textContent = reviewData.baseBranch;
     }
 
     // Update refresh button tooltip based on scope
@@ -1167,29 +1272,6 @@ class LocalManager {
       refreshBtn.title = `Refresh diff (${scopeLabel})`;
     }
 
-    // Update branch name (toolbar) and wire up copy button
-    const branchName = document.getElementById('pr-branch-name');
-    if (branchName) {
-      branchName.textContent = reviewData.branch || 'unknown';
-    }
-
-    const branchCopy = document.getElementById('pr-branch-copy');
-    if (branchCopy && !branchCopy.hasAttribute('data-listener-added')) {
-      branchCopy.setAttribute('data-listener-added', 'true');
-      branchCopy.addEventListener('click', async (e) => {
-        e.stopPropagation();
-        const branch = branchName ? branchName.textContent : '';
-        if (!branch || branch === '--' || branch === 'unknown') return;
-        try {
-          await navigator.clipboard.writeText(branch);
-          branchCopy.classList.add('copied');
-          setTimeout(() => branchCopy.classList.remove('copied'), 2000);
-        } catch (err) {
-          console.error('Failed to copy branch name:', err);
-        }
-      });
-    }
-
     // Update commit SHA and wire up copy button
     const commitSha = document.getElementById('pr-commit-sha');
     if (commitSha && reviewData.localHeadSha) {
@@ -1255,6 +1337,12 @@ class LocalManager {
         settingsLink.style.display = 'none';
       }
     }
+
+    // Render base branch selector for stacked branches
+    const manager = window.prManager;
+    if (manager?.renderBaseBranchSelector) {
+      manager.renderBaseBranchSelector(manager.currentPR);
+    }
   }
 
   /**
@@ -1301,10 +1389,11 @@ class LocalManager {
     const manager = window.prManager;
 
     try {
-      let diffUrl = `/api/local/${this.reviewId}/diff`;
-      if (manager.hideWhitespace) {
-        diffUrl += '?w=1';
-      }
+      const params = new URLSearchParams();
+      if (manager.hideWhitespace) params.set('w', '1');
+      if (manager.currentBaseOverride) params.set('base', manager.currentBaseOverride);
+      const queryString = params.toString();
+      const diffUrl = `/api/local/${this.reviewId}/diff${queryString ? '?' + queryString : ''}`;
       const response = await fetch(diffUrl);
 
       if (!response.ok) {
@@ -1475,6 +1564,11 @@ class LocalManager {
         : `Local Changes - ${manager.currentPR.head_branch}`;
     }
 
+    // Reset base branch override on scope change (base branch context may differ)
+    if (manager) {
+      manager.currentBaseOverride = null;
+    }
+
     // Update header and reload diff
     this.updateLocalHeader(this.localData);
     await this.loadLocalDiff();

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

@@ -482,6 +482,7 @@ class SuggestionManager {
             ${suggestion.type === 'praise'
               ? `<span class="praise-badge" title="Nice Work"><svg viewBox="0 0 16 16"><path d="M8 .25a.75.75 0 01.673.418l1.882 3.815 4.21.612a.75.75 0 01.416 1.279l-3.046 2.97.719 4.192a.75.75 0 01-1.088.791L8 12.347l-3.766 1.98a.75.75 0 01-1.088-.79l.72-4.194L.818 6.374a.75.75 0 01.416-1.28l4.21-.611L7.327.668A.75.75 0 018 .25z"/></svg>Nice Work</span>`
               : `<span class="ai-suggestion-badge" data-type="${suggestion.type}" title="${this.getTypeDescription(suggestion.type)}"><svg viewBox="0 0 16 16" fill="currentColor" width="12" height="12"><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 Suggestion</span>`}
+            ${suggestion.severity ? `<span class="severity-badge severity-${suggestion.severity}">${escapeHtml(suggestion.severity.toUpperCase())}</span>` : ''}
             ${categoryLabel ? `<span class="ai-suggestion-category">${escapeHtml(categoryLabel)}</span>` : ''}
             <span class="ai-title">${escapeHtml(suggestion.title || '')}</span>
           </div>
@@ -497,7 +498,7 @@ class SuggestionManager {
           ${suggestion.type === 'praise'
             ? `<span class="praise-badge" title="Nice Work"><svg viewBox="0 0 16 16"><path d="M8 .25a.75.75 0 01.673.418l1.882 3.815 4.21.612a.75.75 0 01.416 1.279l-3.046 2.97.719 4.192a.75.75 0 01-1.088.791L8 12.347l-3.766 1.98a.75.75 0 01-1.088-.79l.72-4.194L.818 6.374a.75.75 0 01.416-1.28l4.21-.611L7.327.668A.75.75 0 018 .25z"/></svg>Nice Work</span>`
             : `<span class="ai-suggestion-badge collapsed" data-type="${suggestion.type}" title="${this.getTypeDescription(suggestion.type)}"><svg viewBox="0 0 16 16" fill="currentColor" width="10" height="10"><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 Suggestion</span>`}
-          <span class="collapsed-text">${isAdopted ? 'Suggestion adopted' : 'Hidden AI suggestion'}</span>
+          ${suggestion.severity ? `<span class="severity-badge severity-${suggestion.severity}">${escapeHtml(suggestion.severity.toUpperCase())}</span>` : ''}
           <span class="collapsed-title">${escapeHtml(suggestion.title || '')}</span>
           <div class="ai-suggestion-header-right">
             ${suggestion.reasoning && suggestion.reasoning.length > 0 ? `

package/public/js/pr.js

@@ -398,6 +398,51 @@ class PRManager {
     }
   }
 
+  /**
+   * Build analysis config from repo defaults (no modal interaction).
+   * Used by auto-analyze (--ai) to honour the repository's default provider/council.
+   * When the default is a council, fetches the council config from the server so the
+   * progress modal can render the voice/level layout.
+   * @param {Object|null} repoSettings - Repo settings from fetchRepoSettings()
+   * @param {Object} reviewSettings - Review settings from fetchLastReviewSettings()
+   * @returns {Promise<Object>} Config object suitable for startAnalysis / startLocalAnalysis
+   */
+  async _buildDefaultAnalysisConfig(repoSettings, reviewSettings) {
+    const defaultTab = repoSettings?.default_tab || 'single';
+    const councilId = repoSettings?.default_council_id || reviewSettings?.last_council_id || null;
+
+    if ((defaultTab === 'council' || defaultTab === 'advanced') && councilId) {
+      // Fetch the full council config so the progress modal can render correctly
+      let councilConfig = null;
+      let councilName = null;
+      try {
+        const resp = await fetch(`/api/councils/${councilId}`);
+        if (resp.ok) {
+          const data = await resp.json();
+          councilConfig = data.council?.config || null;
+          councilName = data.council?.name || null;
+        }
+      } catch (e) {
+        console.warn('Failed to fetch council config for auto-analyze:', e);
+      }
+
+      return {
+        isCouncil: true,
+        councilId,
+        councilConfig,
+        councilName,
+        configType: defaultTab,
+        customInstructions: null
+      };
+    }
+
+    return {
+      provider: repoSettings?.default_provider || 'claude',
+      model: repoSettings?.default_model || 'opus',
+      customInstructions: null
+    };
+  }
+
   /**
    * 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).
@@ -428,7 +473,14 @@ class PRManager {
           }
         }
 
-        await this.startAnalysis(owner, repo, prNumber, null, {});
+        // Fetch repo settings so we honour the repository's default provider/council
+        const [repoSettings, reviewSettings] = await Promise.all([
+          this.fetchRepoSettings().catch(() => null),
+          this.fetchLastReviewSettings().catch(() => ({ custom_instructions: '', last_council_id: null }))
+        ]);
+        const config = await this._buildDefaultAnalysisConfig(repoSettings, reviewSettings);
+
+        await this.startAnalysis(owner, repo, prNumber, null, config);
       } finally {
         this._autoAnalyzeRequested = false;
         const cleanUrl = new URL(window.location);
@@ -667,10 +719,9 @@ class PRManager {
    */
   async loadAndDisplayFiles(owner, repo, number) {
     try {
-      let diffUrl = `/api/pr/${owner}/${repo}/${number}/diff`;
-      if (this.hideWhitespace) {
-        diffUrl += '?w=1';
-      }
+      const diffUrl = this.hideWhitespace
+        ? `/api/pr/${owner}/${repo}/${number}/diff?w=1`
+        : `/api/pr/${owner}/${repo}/${number}/diff`;
       const response = await fetch(diffUrl);
 
       if (response.ok) {
@@ -4112,7 +4163,8 @@ class PRManager {
             null,
             {
               configType: data.status?.isCouncil ? (data.status.configType || 'advanced') : 'single',
-              enabledLevels: data.status?.enabledLevels || [1, 2, 3]
+              enabledLevels: data.status?.enabledLevels || [1, 2, 3],
+              noLevels: data.status?.noLevels || false
             }
           );
         }
@@ -4223,10 +4275,11 @@ class PRManager {
         : this._fetchStaleness(owner, repo, number);
       this._stalenessPromise = null; // consume it
 
-      const [staleResult, repoSettings, reviewSettings] = await Promise.all([
+      const [staleResult, repoSettings, reviewSettings, appConfig] = await Promise.all([
         staleCheckWithTimeout,
         this.fetchRepoSettings(),
-        this.fetchLastReviewSettings()
+        this.fetchLastReviewSettings(),
+        fetch('/api/config').then(r => r.ok ? r.json() : {}).catch(() => ({}))
       ]);
       console.debug(`[Analyze] parallel-fetch (stale+settings): ${Math.round(performance.now() - _tParallel0)}ms`);
 
@@ -4300,7 +4353,9 @@ class PRManager {
         repoInstructions: repoSettings?.default_instructions || '',
         lastInstructions: lastInstructions,
         lastCouncilId,
-        defaultCouncilId: repoSettings?.default_council_id || null
+        defaultCouncilId: repoSettings?.default_council_id || null,
+        hasPr: true,
+        hasGithubToken: Boolean(appConfig.has_github_token)
       });
 
       // If user cancelled, do nothing
@@ -4368,7 +4423,8 @@ class PRManager {
           councilId: config.councilId || undefined,
           councilConfig: config.councilConfig || undefined,
           configType: config.configType || 'advanced',
-          customInstructions: config.customInstructions || null
+          customInstructions: config.customInstructions || null,
+          excludePrevious: config.excludePrevious || undefined
         };
       } else {
         analyzeUrl = `/api/pr/${owner}/${repo}/${number}/analyses`;
@@ -4378,7 +4434,8 @@ class PRManager {
           tier: config.tier || 'balanced',
           customInstructions: config.customInstructions || null,
           enabledLevels: config.enabledLevels || [1, 2, 3],
-          skipLevel3: config.skipLevel3 || false
+          skipLevel3: config.skipLevel3 || false,
+          excludePrevious: config.excludePrevious || undefined
         };
       }
 
@@ -4419,7 +4476,8 @@ class PRManager {
           config.isCouncil ? config.councilName : null,
           {
             configType: config.isCouncil ? (config.configType || 'advanced') : 'single',
-            enabledLevels: config.enabledLevels || [1, 2, 3]
+            enabledLevels: config.enabledLevels || [1, 2, 3],
+            noLevels: config.noLevels || false
           }
         );
       }
@@ -4669,6 +4727,7 @@ class PRManager {
 
         this._hideStaleBadge();
         this._stalenessPromise = null;
+
         console.log('PR refreshed successfully');
       }
     } catch (error) {

package/public/js/repo-settings.js

@@ -635,12 +635,12 @@ class RepoSettingsPage {
       return;
     }
 
-    const tierIcon = window.getTierIcon ? window.getTierIcon(model.tier) : '';
+    const modelIcon = model.icon || (window.getTierIcon ? window.getTierIcon(model.tier) : '');
 
     container.innerHTML = `
       <div class="model-card selected settings-model-card-static" data-tier="${this.escapeHtml(model.tier || '')}">
         <div class="model-badge ${this.escapeHtml(model.badgeClass || '')}">${this.escapeHtml(model.badge || '')}</div>
-        <div class="model-icon">${tierIcon}</div>
+        <div class="model-icon">${modelIcon}</div>
         <div class="model-info">
           <span class="model-name">${this.escapeHtml(model.name)}</span>
           <span class="model-tagline">${this.escapeHtml(model.tagline || '')}</span>

package/public/local.html

@@ -20,6 +20,7 @@
             window.__pairReview.chatProvider = config.chat_provider || 'pi';
             window.__pairReview.chatProviders = chatProviders;
             window.__pairReview.chatSpinner = config.chat_spinner || 'dots';
+            window.__pairReview.chatEnterToSend = config.chat_enter_to_send !== false;
             document.documentElement.setAttribute('data-chat', state);
             const shortcutsState = config.chat_enable_shortcuts === false ? 'disabled' : 'enabled';
             document.documentElement.setAttribute('data-chat-shortcuts', shortcutsState);
@@ -40,6 +41,7 @@
     <!-- PR Display Styles -->
     <link rel="stylesheet" href="/css/pr.css">
     <link rel="stylesheet" href="/css/ai-summary-modal.css">
+    <link rel="stylesheet" href="/css/analysis-config.css">
     <link rel="stylesheet" href="/css/styles.css">
 
     <!-- Highlight.js for syntax highlighting -->
@@ -128,6 +130,14 @@
             text-overflow: ellipsis;
         }
 
+        .local-branch-badge .toolbar-copy-btn {
+            opacity: 0;
+        }
+
+        .local-branch-badge:hover .toolbar-copy-btn {
+            opacity: 1;
+        }
+
         .local-branch-vs {
             flex-shrink: 0;
         }
@@ -288,6 +298,12 @@
                                 <path d="M11.75 2.5a.75.75 0 100 1.5.75.75 0 000-1.5zm-2.25.75a2.25 2.25 0 113 2.122V6A2.5 2.5 0 0110 8.5H6a1 1 0 00-1 1v1.128a2.251 2.251 0 11-1.5 0V5.372a2.25 2.25 0 111.5 0v1.836A2.492 2.492 0 016 7h4a1 1 0 001-1v-.628A2.25 2.25 0 019.5 3.25zM4.25 12a.75.75 0 100 1.5.75.75 0 000-1.5zM3.5 3.25a.75.75 0 111.5 0 .75.75 0 01-1.5 0z"/>
                             </svg>
                             <span id="local-branch-text">--</span>
+                            <button type="button" class="local-branch-copy toolbar-copy-btn" id="local-branch-copy" title="Copy branch name" aria-label="Copy branch name">
+                                <svg viewBox="0 0 16 16" fill="currentColor" width="10" height="10">
+                                    <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>
+                                    <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"></path>
+                                </svg>
+                            </button>
                         </span>
                         <span class="local-branch-vs" id="local-branch-vs" style="display: none; font-size: 11px; color: var(--color-text-tertiary); margin: 0 4px;">vs</span>
                         <span class="local-branch-badge" id="local-base-branch" style="display: none;">
@@ -309,9 +325,13 @@
             <div class="header-right">
                 <div class="header-icon-group">
                     <button class="btn btn-icon" id="local-refresh-btn" title="Refresh diff from directory">
-                        <svg viewBox="0 0 16 16" fill="currentColor" width="16" height="16">
+                        <svg class="refresh-icon" viewBox="0 0 16 16" fill="currentColor" width="16" height="16">
                             <path d="M1.705 8.005a.75.75 0 0 1 .834.656 5.5 5.5 0 0 0 9.592 2.97l-1.204-1.204a.25.25 0 0 1 .177-.427h3.646a.25.25 0 0 1 .25.25v3.646a.25.25 0 0 1-.427.177l-1.38-1.38A7.002 7.002 0 0 1 1.05 8.84a.75.75 0 0 1 .656-.834ZM8 2.5a5.487 5.487 0 0 0-4.131 1.869l1.204 1.204A.25.25 0 0 1 4.896 6H1.25A.25.25 0 0 1 1 5.75V2.104a.25.25 0 0 1 .427-.177l1.38 1.38A7.002 7.002 0 0 1 14.95 7.16a.75.75 0 1 1-1.49.178A5.5 5.5 0 0 0 8 2.5Z"/>
                         </svg>
+                        <svg class="spinner-icon" viewBox="0 0 16 16" fill="currentColor" width="16" height="16">
+                            <path d="M8 0a8 8 0 1 1 0 16A8 8 0 0 1 8 0ZM1.5 8a6.5 6.5 0 1 0 13 0 6.5 6.5 0 0 0-13 0Z" opacity="0.25"/>
+                            <path d="M8 0a8 8 0 0 1 8 8h-2A6 6 0 0 0 8 2V0Z"/>
+                        </svg>
                     </button>
                     <button class="btn btn-icon" id="theme-toggle" title="Toggle theme">
                         <svg class="theme-icon-light" viewBox="0 0 16 16" fill="currentColor" width="16" height="16">
@@ -362,17 +382,18 @@
                         </svg>
                     </button>
                     <div class="toolbar-meta" id="toolbar-meta">
-                        <span class="toolbar-branch" id="pr-branch">
-                            <svg viewBox="0 0 16 16" fill="currentColor" width="12" height="12">
-                                <path d="M11.75 2.5a.75.75 0 100 1.5.75.75 0 000-1.5zm-2.25.75a2.25 2.25 0 113 2.122V6A2.5 2.5 0 0110 8.5H6a1 1 0 00-1 1v1.128a2.251 2.251 0 11-1.5 0V5.372a2.25 2.25 0 111.5 0v1.836A2.492 2.492 0 016 7h4a1 1 0 001-1v-.628A2.25 2.25 0 019.5 3.25zM4.25 12a.75.75 0 100 1.5.75.75 0 000-1.5zM3.5 3.25a.75.75 0 111.5 0 .75.75 0 01-1.5 0z"/>
-                            </svg>
-                            <span id="pr-branch-name">--</span>
-                            <button type="button" class="toolbar-branch-copy toolbar-copy-btn" id="pr-branch-copy" title="Copy branch name" aria-label="Copy branch name">
-                                <svg viewBox="0 0 16 16" fill="currentColor" width="10" height="10">
-                                    <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>
-                                    <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"></path>
+                        <span id="toolbar-base-branch-wrap" class="toolbar-base-branch-wrap" hidden>
+                            <span class="base-branch-vs">base:</span>
+                            <span id="toolbar-base-branch-static" class="toolbar-base-branch-static">
+                                <svg viewBox="0 0 16 16" fill="currentColor" width="12" height="12">
+                                    <path d="M11.75 2.5a.75.75 0 100 1.5.75.75 0 000-1.5zm-2.25.75a2.25 2.25 0 113 2.122V6A2.5 2.5 0 0110 8.5H6a1 1 0 00-1 1v1.128a2.251 2.251 0 11-1.5 0V5.372a2.25 2.25 0 111.5 0v1.836A2.492 2.492 0 016 7h4a1 1 0 001-1v-.628A2.25 2.25 0 019.5 3.25zM4.25 12a.75.75 0 100 1.5.75.75 0 000-1.5zM3.5 3.25a.75.75 0 111.5 0 .75.75 0 01-1.5 0z"/>
                                 </svg>
-                            </button>
+                                <span id="toolbar-base-branch-text"></span>
+                            </span>
+                            <span id="base-branch-selector-wrap" class="base-branch-selector-wrap" hidden>
+                                <label for="base-branch-select" class="sr-only">Compare against</label>
+                                <select id="base-branch-select" class="base-branch-select" title="Change base branch for diff"></select>
+                            </span>
                         </span>
                         <span class="toolbar-separator"></span>
                         <span class="toolbar-stat toolbar-stat-additions" id="pr-additions">+0</span>

package/public/pr.html

@@ -28,6 +28,7 @@
             window.__pairReview.share = config.share || null;
             window.__pairReview.enableGraphite = config.enable_graphite === true;
             window.__pairReview.chatSpinner = config.chat_spinner || 'dots';
+            window.__pairReview.chatEnterToSend = config.chat_enter_to_send !== false;
             document.documentElement.setAttribute('data-chat', state);
             const shortcutsState = config.chat_enable_shortcuts === false ? 'disabled' : 'enabled';
             document.documentElement.setAttribute('data-chat-shortcuts', shortcutsState);
@@ -48,6 +49,7 @@
     <!-- PR Display Styles -->
     <link rel="stylesheet" href="/css/pr.css">
     <link rel="stylesheet" href="/css/ai-summary-modal.css">
+    <link rel="stylesheet" href="/css/analysis-config.css">
     <link rel="stylesheet" href="/css/styles.css">
 
     <!-- Highlight.js for syntax highlighting -->