@in-the-loop-labs/pair-review 3.6.0 → 3.7.1

package/public/js/components/ReviewModal.js

@@ -64,9 +64,9 @@ class ReviewModal {
               </div>
               <div class="pending-draft-notice-content">
                 <span class="pending-draft-notice-text">
-                  You have a pending draft review on GitHub with <strong id="pending-draft-count">0</strong> comments.
+                  You have a pending draft review on <span class="rm-host-name">GitHub</span> with <strong id="pending-draft-count">0</strong> comments.
                   Submitting here will add to or complete this review.
-                  <a href="#" id="pending-draft-link" target="_blank" rel="noopener noreferrer">Manage on GitHub</a>.
+                  <a href="#" id="pending-draft-link" target="_blank" rel="noopener noreferrer">Manage on <span class="rm-host-name">GitHub</span></a>.
                 </span>
               </div>
             </div>
@@ -120,7 +120,7 @@ class ReviewModal {
                   <input type="radio" name="review-event" value="DRAFT">
                   <div class="review-type-content">
                     <span class="review-type-label">Save as Draft</span>
-                    <span class="review-type-desc">Save your review as a draft on GitHub to finish later.</span>
+                    <span class="review-type-desc">Save your review as a draft on <span class="rm-host-name">GitHub</span> to finish later.</span>
                   </div>
                 </label>
               </div>
@@ -284,6 +284,11 @@ class ReviewModal {
     // Update AI summary link visibility
     this.updateAISummaryLink();
 
+    // Apply the configured remote-host display name + icon (resolves
+    // asynchronously after the modal HTML was built).
+    this.applyHostName();
+    this.applySubmitButtonIcon();
+
     // Update pending draft notice
     this.updatePendingDraftNotice();
   }
@@ -311,11 +316,17 @@ class ReviewModal {
         countElement.textContent = String(pendingDraft.comments_count || 0);
       }
 
-      // Update the link - hide if no github_url
+      // Update the link. Prefer the URL built from the repo's configured
+      // url_template (host-correct) over the server-reported github_url,
+      // which some alt-hosts return as a wrong-host github.com/issues URL.
       const linkElement = notice.querySelector('#pending-draft-link');
       if (linkElement) {
-        if (pendingDraft.github_url) {
-          linkElement.href = pendingDraft.github_url;
+        const templatedUrl = (typeof window !== 'undefined' && window.RepoLinks
+          && typeof window.RepoLinks.externalUrl === 'function')
+          ? window.RepoLinks.externalUrl() : null;
+        const manageUrl = templatedUrl || pendingDraft.github_url;
+        if (manageUrl) {
+          linkElement.href = manageUrl;
           linkElement.style.display = 'inline';
         } else {
           linkElement.style.display = 'none';
@@ -452,6 +463,8 @@ class ReviewModal {
       submitBtn.disabled = false;
       cancelBtn.style.display = 'inline-block';
       closeBtn.style.display = 'inline-block';
+      // innerHTML reset drops any host icon — re-apply it.
+      this.applySubmitButtonIcon();
     }
   }
 
@@ -531,7 +544,7 @@ class ReviewModal {
         const reviewUrl = result.reviewUrl || result.github_url;
         if (isDraft) {
           window.toast.showSuccess(
-            'Draft review submitted to GitHub successfully!',
+            `Draft review submitted to ${ReviewModal.escapeHtml(ReviewModal.hostName())} successfully!`,
             {
               duration: 5000
             }
@@ -541,7 +554,7 @@ class ReviewModal {
             'Review submitted successfully!',
             {
               link: reviewUrl,
-              linkText: 'View on GitHub',
+              linkText: `View on ${ReviewModal.escapeHtml(ReviewModal.hostName())}`,
               duration: 5000
             }
           );
@@ -586,11 +599,16 @@ class ReviewModal {
       }
 
       if (isDraft) {
-        // After 2 seconds, open GitHub PR page for drafts
-        setTimeout(() => {
-          const githubUrl = result.github_url || `https://github.com/${pr.owner}/${pr.repo}/pull/${pr.number}`;
-          window.open(githubUrl, '_blank');
-        }, 2000);
+        // After 2 seconds, open the PR page for drafts. Use the PR's canonical
+        // html_url (correct host + `/pull/`) rather than the review's html_url,
+        // which some alt-hosts return as a github.com `/issues/<n>` URL. Never
+        // assume github.com — see resolveDraftPrUrl.
+        const prUrl = ReviewModal.resolveDraftPrUrl(pr, result);
+        if (prUrl) {
+          setTimeout(() => {
+            window.open(prUrl, '_blank');
+          }, 2000);
+        }
       }
       
     } catch (error) {
@@ -676,6 +694,110 @@ class ReviewModal {
     localStorage.setItem(ASSISTED_BY_STORAGE_KEY, String(checkbox.checked));
   }
 
+  /**
+   * Display name of the remote code host, for user-facing text in place of
+   * the literal "GitHub". Reads the configured `links.external.name` via
+   * `window.RepoLinks.hostName()`, falling back to "GitHub".
+   *
+   * @returns {string}
+   */
+  static hostName() {
+    if (typeof window !== 'undefined' && window.RepoLinks
+        && typeof window.RepoLinks.hostName === 'function') {
+      return window.RepoLinks.hostName();
+    }
+    return 'GitHub';
+  }
+
+  /**
+   * Escape a string for safe interpolation into HTML. Used for the host
+   * name (user-supplied config) before it goes into the success toast,
+   * which renders its message/linkText via innerHTML.
+   *
+   * @param {string} text
+   * @returns {string}
+   */
+  static escapeHtml(text) {
+    return String(text == null ? '' : text)
+      .replace(/&/g, '&amp;')
+      .replace(/</g, '&lt;')
+      .replace(/>/g, '&gt;')
+      .replace(/"/g, '&quot;')
+      .replace(/'/g, '&#39;');
+  }
+
+  /**
+   * Resolve the URL to open in a new tab after a draft submit.
+   *
+   * Precedence:
+   *   1. The URL built from the repo's configured `links.external.url_template`
+   *      (`window.RepoLinks.externalUrl()`) — authoritative and host-correct.
+   *   2. The PR's canonical `html_url` (the host's own PR page).
+   *   3. The server-reported `github_url` as a last resort.
+   *
+   * Some alt-hosts return the pending-review `html_url` as a
+   * `github.com/.../issues/<n>` URL, which lands on the wrong host and page.
+   * We must never assume github.com, so there is no hardcoded fallback host:
+   * if none of the above yields a URL we open nothing.
+   *
+   * @param {{html_url?: string}|null|undefined} pr - current PR (from prManager)
+   * @param {{github_url?: string}|null|undefined} result - submit-review response
+   * @returns {string|null} URL to open, or null if none is available
+   */
+  static resolveDraftPrUrl(pr, result) {
+    if (typeof window !== 'undefined' && window.RepoLinks
+        && typeof window.RepoLinks.externalUrl === 'function') {
+      const templated = window.RepoLinks.externalUrl();
+      if (templated) return templated;
+    }
+    if (pr && pr.html_url) return pr.html_url;
+    if (result && result.github_url) return result.github_url;
+    return null;
+  }
+
+  /**
+   * Update host-name-dependent static text in the modal (the pending-draft
+   * notice and the "Save as Draft" description) to the configured host name.
+   * Called from `show()` because the name resolves asynchronously after the
+   * modal HTML is built. No-op when the modal isn't present.
+   */
+  applyHostName() {
+    if (!this.modal) return;
+    const name = ReviewModal.hostName();
+    const spans = this.modal.querySelectorAll('.rm-host-name');
+    spans.forEach((el) => { el.textContent = name; });
+  }
+
+  /**
+   * Prepend the configured external-host icon to the submit button, when an
+   * icon is configured for the repo. The icon is parsed via
+   * `window.RepoLinks.parseSvgIcon` (DOMParser + attribute stripping) and
+   * inserted as a DOM node — never via innerHTML. Idempotent: any previously
+   * inserted icon is removed first. No-op for plain github.com repos.
+   */
+  applySubmitButtonIcon() {
+    const submitBtn = this.modal?.querySelector('#submit-review-btn-modal');
+    if (!submitBtn) return;
+
+    const existing = submitBtn.querySelector?.('.submit-host-icon');
+    if (existing) existing.remove();
+
+    if (typeof window === 'undefined' || !window.RepoLinks
+        || typeof window.RepoLinks.externalIcon !== 'function'
+        || typeof window.RepoLinks.parseSvgIcon !== 'function') {
+      return;
+    }
+    const iconStr = window.RepoLinks.externalIcon();
+    if (!iconStr) return;
+    const svg = window.RepoLinks.parseSvgIcon(iconStr);
+    if (!svg) return;
+
+    svg.classList.add('submit-host-icon');
+    if (!svg.getAttribute('width')) svg.setAttribute('width', '16');
+    if (!svg.getAttribute('height')) svg.setAttribute('height', '16');
+    submitBtn.insertBefore(svg, submitBtn.firstChild);
+  }
+
 }
 
 // Initialize when DOM is ready if not already initialized

package/public/js/components/SuggestionNavigator.js

@@ -10,7 +10,10 @@ class SuggestionNavigator {
     this.isCollapsed = this.loadCollapsedState();
     this.element = null;
     this.collapseToggle = null;
-    
+    // Monotonic token so a fast Next/Prev that supersedes an in-flight
+    // goToSuggestion can tell the older call to bail after its await.
+    this._navGen = 0;
+
     this.init();
     this.bindEvents();
   }
@@ -236,7 +239,7 @@ class SuggestionNavigator {
   /**
    * Navigate to specific suggestion by index
    */
-  goToSuggestion(index) {
+  async goToSuggestion(index) {
     if (index < 0 || index >= this.suggestions.length) {
       return;
     }
@@ -244,10 +247,42 @@ class SuggestionNavigator {
     this.currentSuggestionIndex = index;
     this.updateCounter();
     this.updateNavigationButtons();
+    // The suggestion's row only exists once its file body has rendered
+    // (lazy bodies start empty), so render it before the highlight/scroll
+    // lookups below — otherwise both silently miss on the first attempt.
+    // A collapsed file is expanded first so the row is actually visible.
+    const myGen = ++this._navGen;
+    await this.ensureSuggestionVisible(this.suggestions[index]);
+    // A newer goToSuggestion ran while we awaited and moved
+    // this.currentSuggestionIndex — let it own the highlight/scroll.
+    if (myGen !== this._navGen) return;
     this.highlightCurrentSuggestion();
     this.scrollToSuggestion();
   }
 
+  /**
+   * Make sure a suggestion's file is expanded and its lazy diff body is
+   * rendered so the suggestion row exists in the DOM. Best effort: any
+   * failure falls through to the old lookup-miss behavior.
+   * @param {Object} suggestion
+   */
+  async ensureSuggestionVisible(suggestion) {
+    const file = suggestion?.file;
+    const pm = window.prManager;
+    if (!file || !pm) return;
+    try {
+      const wrapper = pm.findFileElement?.(file);
+      if (wrapper?.classList.contains('collapsed') && pm.toggleFileCollapse) {
+        // Renders the lazy body and removes `collapsed`.
+        await pm.toggleFileCollapse(wrapper.dataset.fileName || file);
+      } else if (pm.ensureFileBodyRendered) {
+        await pm.ensureFileBodyRendered(file);
+      }
+    } catch (err) {
+      console.warn('[SuggestionNavigator] could not prepare suggestion file', file, err);
+    }
+  }
+
   /**
    * Check if a suggestion should be skipped during navigation
    */
@@ -370,18 +405,22 @@ class SuggestionNavigator {
       
       if (suggestionEl) {
         const minimizer = window.prManager?.commentMinimizer;
+        let scrollTarget = suggestionEl;
         if (minimizer?.active) {
           // Expand file-level comments so the target becomes visible
           minimizer.expandForElement(suggestionEl);
           // Comments are minimized — scroll to the parent diff line instead
-          const diffRow = minimizer.findDiffRowFor(suggestionEl);
-          if (diffRow) {
-            diffRow.scrollIntoView({ behavior: 'smooth', block: 'center', inline: 'nearest' });
-          } else {
-            suggestionEl.scrollIntoView({ behavior: 'smooth', block: 'center', inline: 'nearest' });
-          }
+          scrollTarget = minimizer.findDiffRowFor(suggestionEl) || suggestionEl;
+        }
+        // Land the target at the top of the diff panel (scroll-margin-top in
+        // pr.css offsets it below the sticky toolbar + file header).
+        const options = { behavior: 'smooth', block: 'start', inline: 'nearest' };
+        // Stable variant re-corrects after lazy file bodies render
+        // mid-scroll and shift the layout. Fire-and-forget.
+        if (window.ScrollUtils?.scrollIntoViewStable) {
+          window.ScrollUtils.scrollIntoViewStable(scrollTarget, options);
         } else {
-          suggestionEl.scrollIntoView({ behavior: 'smooth', block: 'center', inline: 'nearest' });
+          scrollTarget.scrollIntoView(options);
         }
       }
     }
@@ -474,4 +513,10 @@ class SuggestionNavigator {
 }
 
 // Export for use
-window.SuggestionNavigator = SuggestionNavigator;
+if (typeof window !== 'undefined') {
+  window.SuggestionNavigator = SuggestionNavigator;
+}
+
+if (typeof module !== 'undefined' && module.exports) {
+  module.exports = SuggestionNavigator;
+}

package/public/js/components/VoiceCentricConfigTab.js

@@ -235,6 +235,37 @@ class VoiceCentricConfigTab {
    */
   setDefaultCouncilId(councilId) {
     this._pendingDefaultCouncilId = councilId;
+    // On a cached reopen the councils are already loaded, so loadCouncils() —
+    // and the _renderCouncilSelector() call that applies the pending default —
+    // will not run again (the modal instance is reused; see AnalysisConfigModal
+    // caching on window.analysisConfigModal). Apply it now so the saved/default
+    // council is restored instead of being silently dropped onto a blank
+    // "+ New Council" selection.
+    if (this._councilsLoaded && this._injected) {
+      this._renderCouncilSelector();
+    }
+  }
+
+  /**
+   * Reset selection and editor state for a fresh modal open.
+   *
+   * The AnalysisConfigModal (and therefore this tab) is reused across runs — and
+   * in the index/bulk flow, across different repositories. Without this reset a
+   * council selected in a previous run (or its pending default / dirty edits)
+   * would carry over and could be displayed or submitted for the next batch.
+   */
+  reset() {
+    this.selectedCouncilId = null;
+    this._pendingDefaultCouncilId = null;
+    this._isDirty = false;
+    if (!this._injected) return;
+    const selector = this.modal.querySelector('#vc-council-selector');
+    if (selector) {
+      selector.value = '';
+      selector.classList.add('new-council-selected');
+    }
+    this._applyConfigToUI(this._defaultConfig());
+    this._markClean();
   }
 
   /**
@@ -1494,3 +1525,8 @@ class VoiceCentricConfigTab {
 if (typeof window !== 'undefined') {
   window.VoiceCentricConfigTab = VoiceCentricConfigTab;
 }
+
+// Export for unit testing (Node/CommonJS environment)
+if (typeof module !== 'undefined' && module.exports) {
+  module.exports = { VoiceCentricConfigTab };
+}

package/public/js/index.js

@@ -90,6 +90,8 @@
 
   /** localStorage key for persisting the active tab */
   const TAB_STORAGE_KEY = 'pair-review-active-tab';
+  const BULK_ANALYSIS_TAB_STORAGE_KEY = 'pair-review-bulk-analysis-tab';
+  const BULK_ANALYSIS_INSTRUCTIONS_STORAGE_KEY = 'pair-review-bulk-analysis-instructions';
 
   /**
    * Format a relative time string from a date
@@ -133,6 +135,10 @@
     return div.innerHTML;
   }
 
+  // encodeBase64Utf8 / getRepoStorageKey live in public/js/utils/storage-keys.js
+  // (window.encodeBase64Utf8 / window.getRepoStorageKey), shared with pr.js so the
+  // per-repo keys this page writes stay byte-identical to those the PR page reads.
+
   const LOCAL_REVIEW_PATH_URL_ERROR = 'Local reviews require a filesystem path, not a URL. Pass GitHub or Graphite URLs as PR review inputs instead.';
 
   function isUrlLikeLocalReviewPath(value) {
@@ -1377,6 +1383,9 @@
         window.__pairReview.chatProvider = config.chat_provider || 'pi';
         const chatProviders = config.chat_providers || [];
         window.__pairReview.chatProviders = chatProviders;
+        window.__pairReview.defaultProvider = config.default_provider || 'claude';
+        window.__pairReview.defaultModel = config.default_model || 'opus';
+        window.__pairReview.hasGithubToken = Boolean(config.has_github_token);
         window.__pairReview.enableGraphite = config.enable_graphite === true;
         window.__pairReview.chatSpinner = config.chat_spinner || 'dots';
         window.__pairReview.chatEnterToSend = config.chat_enter_to_send !== false;
@@ -1404,6 +1413,7 @@
 
   /** Currently active SelectionMode instance (only one tab at a time) */
   var activeSelection = null;
+  var bulkAnalysisConfigModal = null;
 
   /**
    * SelectionMode manages checkbox-based selection for a single tab's table.
@@ -1877,6 +1887,40 @@
     });
   }
 
+  /**
+   * Read selected collection rows as PR descriptors.
+   * @param {Set} selectedIds - PR URLs (data-pr-url values)
+   * @param {string} tbodyId - tbody element ID
+   * @returns {Array<{owner: string, repo: string, number: string, prUrl: string}>}
+   */
+  function getSelectedCollectionRows(selectedIds, tbodyId) {
+    var tbody = document.getElementById(tbodyId);
+    if (!tbody) return [];
+
+    var rows = [];
+    var trs = tbody.querySelectorAll('tr[data-pr-url]');
+    for (var i = 0; i < trs.length; i++) {
+      var tr = trs[i];
+      var prUrl = tr.dataset.prUrl;
+      if (!selectedIds.has(prUrl)) continue;
+      if (tr.dataset.owner && tr.dataset.repo && tr.dataset.number) {
+        rows.push({
+          owner: tr.dataset.owner,
+          repo: tr.dataset.repo,
+          number: tr.dataset.number,
+          prUrl: prUrl
+        });
+      }
+    }
+    return rows;
+  }
+
+  function buildReviewUrlsFromRows(rows, query) {
+    return rows.map(function (row) {
+      return '/pr/' + encodeURIComponent(row.owner) + '/' + encodeURIComponent(row.repo) + '/' + row.number + (query || '');
+    });
+  }
+
   /**
    * Build pair-review URLs from selected collection rows.
    * @param {Set} selectedIds - PR URLs (data-pr-url values)
@@ -1885,20 +1929,116 @@
    * @returns {string[]} array of pair-review URLs
    */
   function buildReviewUrls(selectedIds, tbodyId, query) {
-    var tbody = document.getElementById(tbodyId);
-    if (!tbody) return [];
-    var urls = [];
-    selectedIds.forEach(function (prUrl) {
-      var row = tbody.querySelector('tr[data-pr-url="' + CSS.escape(prUrl) + '"]');
-      if (!row) return;
-      var owner = row.dataset.owner;
-      var repo = row.dataset.repo;
-      var number = row.dataset.number;
-      if (owner && repo && number) {
-        urls.push('/pr/' + encodeURIComponent(owner) + '/' + encodeURIComponent(repo) + '/' + number + (query || ''));
+    return buildReviewUrlsFromRows(getSelectedCollectionRows(selectedIds, tbodyId), query);
+  }
+
+  function getCommonRepository(rows) {
+    if (rows.length === 0) return null;
+    var first = rows[0];
+    var firstKey = (first.owner + '/' + first.repo).toLowerCase();
+    for (var i = 1; i < rows.length; i++) {
+      if ((rows[i].owner + '/' + rows[i].repo).toLowerCase() !== firstKey) {
+        return null;
       }
+    }
+    return { owner: first.owner, repo: first.repo };
+  }
+
+  async function fetchBulkRepoSettings(commonRepo) {
+    if (!commonRepo) return null;
+    try {
+      var response = await fetch('/api/repos/' + encodeURIComponent(commonRepo.owner) + '/' + encodeURIComponent(commonRepo.repo) + '/settings');
+      if (!response.ok) return null;
+      return await response.json();
+    } catch (error) {
+      console.warn('Failed to fetch repo settings for bulk analysis:', error);
+      return null;
+    }
+  }
+
+  function getBulkAnalysisStorageKeys(commonRepo) {
+    if (!commonRepo) {
+      return {
+        tab: BULK_ANALYSIS_TAB_STORAGE_KEY,
+        instructions: BULK_ANALYSIS_INSTRUCTIONS_STORAGE_KEY
+      };
+    }
+    return {
+      tab: window.getRepoStorageKey('pair-review-tab', commonRepo.owner, commonRepo.repo),
+      instructions: window.getRepoStorageKey('pair-review-instructions', commonRepo.owner, commonRepo.repo)
+    };
+  }
+
+  // Cache the /api/providers metadata so the bulk modal can resolve a coherent
+  // provider/model pair before showing (mirrors PRManager._getProvidersInfo).
+  function getBulkProvidersInfo() {
+    window.__pairReview = window.__pairReview || {};
+    if (!window.__pairReview._providersInfoPromise) {
+      window.__pairReview._providersInfoPromise = fetch('/api/providers')
+        .then(function (r) { return r.ok ? r.json() : {}; })
+        .then(function (d) { return Array.isArray(d.providers) ? d.providers : []; })
+        .catch(function () { return []; });
+    }
+    return window.__pairReview._providersInfoPromise;
+  }
+
+  async function showBulkAnalysisConfig(rows) {
+    if (!window.AnalysisConfigModal) return null;
+
+    if (!bulkAnalysisConfigModal) {
+      bulkAnalysisConfigModal = new window.AnalysisConfigModal();
+    }
+
+    var commonRepo = getCommonRepository(rows);
+    var storageKeys = getBulkAnalysisStorageKeys(commonRepo);
+    var repoSettings = await fetchBulkRepoSettings(commonRepo);
+    var rememberedTab = localStorage.getItem(storageKeys.tab);
+    var lastInstructions = localStorage.getItem(storageKeys.instructions) || '';
+
+    bulkAnalysisConfigModal.onTabChange = function (tabId) {
+      localStorage.setItem(storageKeys.tab, tabId);
+    };
+
+    var providersInfo = await getBulkProvidersInfo();
+    var resolvedPair = window.resolveProviderModelPair([
+      { provider: repoSettings?.default_provider, model: repoSettings?.default_model },
+      { provider: window.__pairReview?.defaultProvider, model: window.__pairReview?.defaultModel }
+    ], providersInfo);
+
+    var config = await bulkAnalysisConfigModal.show({
+      currentModel: resolvedPair.model,
+      currentProvider: resolvedPair.provider,
+      defaultTab: rememberedTab || repoSettings?.default_tab || 'single',
+      repoInstructions: commonRepo ? (repoSettings?.default_instructions || '') : '',
+      lastInstructions: lastInstructions,
+      defaultCouncilId: commonRepo ? (repoSettings?.default_council_id || null) : null,
+      hasPr: true,
+      hasGithubToken: window.__pairReview?.hasGithubToken !== false
+    });
+
+    if (!config) return null;
+
+    var submittedInstructions = config.customInstructions || '';
+    if (submittedInstructions) {
+      localStorage.setItem(storageKeys.instructions, submittedInstructions);
+    } else {
+      localStorage.removeItem(storageKeys.instructions);
+    }
+
+    return config;
+  }
+
+  async function storeBulkAnalysisConfig(config) {
+    var response = await fetch('/api/bulk-analysis-configs', {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ analysisConfig: config })
     });
-    return urls;
+    var data = await response.json().catch(function () { return {}; });
+    if (!response.ok || !data.id) {
+      throw new Error(data.error || 'Failed to save analysis settings');
+    }
+    return data.id;
   }
 
   /**
@@ -1924,10 +2064,29 @@
     bulkOpenUrls(urls);
   }
 
-  function handleBulkAnalyze(selectedIds, selectionInstance) {
-    var urls = buildReviewUrls(selectedIds, selectionInstance.config.tbodyId, '?analyze=true');
-    selectionInstance.exit();
-    bulkOpenUrls(urls);
+  async function handleBulkAnalyze(selectedIds, selectionInstance) {
+    var rows = getSelectedCollectionRows(selectedIds, selectionInstance.config.tbodyId);
+    if (rows.length === 0) return;
+
+    if (!window.AnalysisConfigModal) {
+      selectionInstance.exit();
+      bulkOpenUrls(buildReviewUrlsFromRows(rows, '?analyze=true'));
+      return;
+    }
+
+    try {
+      var config = await showBulkAnalysisConfig(rows);
+      if (!config) return;
+
+      var configId = await storeBulkAnalysisConfig(config);
+      var query = '?analyze=true&analysisConfigId=' + encodeURIComponent(configId);
+      var urls = buildReviewUrlsFromRows(rows, query);
+      selectionInstance.exit();
+      bulkOpenUrls(urls);
+    } catch (error) {
+      console.error('Bulk analyze error:', error);
+      if (window.toast) window.toast.error('Failed to start bulk analysis: ' + error.message);
+    }
   }
 
   // ─── Event Delegation ───────────────────────────────────────────────────────