@in-the-loop-labs/pair-review 3.9.1 → 4.1.0

package/public/local.html

@@ -591,6 +591,7 @@
     <!-- Shared storage-key and provider/model helpers -->
     <script src="/js/utils/storage-keys.js"></script>
     <script src="/js/utils/provider-model.js"></script>
+    <script src="/js/utils/analyze-params.js"></script>
 
     <!-- Category emoji mapping -->
     <script src="/js/utils/category-emoji.js"></script>

package/public/pr.html

@@ -394,6 +394,7 @@
     <!-- Shared storage-key and provider/model helpers -->
     <script src="/js/utils/storage-keys.js"></script>
     <script src="/js/utils/provider-model.js"></script>
+    <script src="/js/utils/analyze-params.js"></script>
 
     <!-- Category emoji mapping -->
     <script src="/js/utils/category-emoji.js"></script>

package/public/setup.html

@@ -530,6 +530,7 @@
     <script src="/js/ws-client.js"></script>
     <script src="/js/components/UpdateBanner.js"></script>
     <script src="/js/utils/notification-sounds.js"></script>
+    <script src="/js/utils/analyze-params.js"></script>
 
     <script>
         // Copyright 2026 Tim Perkins (tjwp) | SPDX-License-Identifier: Apache-2.0
@@ -611,6 +612,14 @@
                     context.path = params.get('path');
                 }
             }
+            // Forward a delegated --scope/--base selection (the CLI carries them
+            // on the /local?path=... URL when delegating to a running server).
+            // The server re-validates; this only relays the request.
+            if (mode === 'local') {
+                const scopeParams = new URLSearchParams(window.location.search);
+                if (scopeParams.get('scope')) context.scope = scopeParams.get('scope');
+                if (scopeParams.get('base')) context.base = scopeParams.get('base');
+            }
 
             const steps = mode === 'local' ? LOCAL_STEPS : PR_STEPS;
             const stepStates = {};
@@ -751,7 +760,23 @@
                     var fetchOptions = { method: 'POST' };
                     if (mode === 'local') {
                         fetchOptions.headers = { 'Content-Type': 'application/json' };
-                        fetchOptions.body = JSON.stringify({ path: context.path });
+                        var localBody = { path: context.path };
+                        if (context.scope) localBody.scope = context.scope;
+                        if (context.base) localBody.base = context.base;
+                        fetchOptions.body = JSON.stringify(localBody);
+                    } else if (mode === 'pr') {
+                        // A `host` query param (set when opening a PR from a dashboard
+                        // collection row or a pasted URL) tells setup which system the
+                        // PR lives on, so the fetch binds there directly instead of
+                        // probing. Sentinel: the literal "github" means github.com and
+                        // maps to body host null; any other value is an api_host URL
+                        // string (alt host) — an api_host is always a URL, so "github"
+                        // is unambiguous. Absent = unknown; the server derives the host.
+                        var setupHost = new URLSearchParams(window.location.search).get('host');
+                        if (setupHost) {
+                            fetchOptions.headers = { 'Content-Type': 'application/json' };
+                            fetchOptions.body = JSON.stringify({ host: setupHost === 'github' ? null : setupHost });
+                        }
                     }
 
                     var response = await fetch(postUrl, fetchOptions);
@@ -764,16 +789,13 @@
 
                     var data = await response.json();
 
-                    // If the review already exists, redirect immediately
+                    // If the review already exists, redirect immediately.
+                    // Relay the full auto-analyze intent bundle (analyze /
+                    // analysisConfigId / council / provider / model) so a delegated
+                    // override or CLI council survives this hop to the review page.
                     if (data.existing && data.reviewUrl) {
                         var targetUrl = new URL(data.reviewUrl, window.location.origin);
-                        var qs = new URLSearchParams(window.location.search);
-                        if (qs.get('analyze') === 'true') targetUrl.searchParams.set('analyze', qs.get('analyze'));
-                        if (qs.get('analysisConfigId')) targetUrl.searchParams.set('analysisConfigId', qs.get('analysisConfigId'));
-                        // Forward the CLI-supplied council selection (PR & local cold-start
-                        // and delegated paths route through here); the review page keys
-                        // council auto-analysis on this param.
-                        if (qs.get('council')) targetUrl.searchParams.set('council', qs.get('council'));
+                        window.carryAnalyzeParams(window.location.search, targetUrl);
                         window.location.href = targetUrl.toString();
                         return;
                     }
@@ -832,13 +854,10 @@
                             setTimeout(function() {
                                 if (msg.reviewUrl) {
                                     var targetUrl = new URL(msg.reviewUrl, window.location.origin);
-                                    var qs = new URLSearchParams(window.location.search);
-                                    if (qs.get('analyze') === 'true') targetUrl.searchParams.set('analyze', qs.get('analyze'));
-                                    if (qs.get('analysisConfigId')) targetUrl.searchParams.set('analysisConfigId', qs.get('analysisConfigId'));
-                                    // Forward the CLI-supplied council selection (PR & local cold-start
-                                    // and delegated paths route through here); the review page keys
-                                    // council auto-analysis on this param.
-                                    if (qs.get('council')) targetUrl.searchParams.set('council', qs.get('council'));
+                                    // Relay the full auto-analyze intent bundle (analyze /
+                                    // analysisConfigId / council / provider / model) so a
+                                    // delegated override or CLI council survives the hop.
+                                    window.carryAnalyzeParams(window.location.search, targetUrl);
                                     window.location.href = targetUrl.toString();
                                 }
                             }, 400);

package/src/ai/analyzer.js

@@ -342,8 +342,8 @@ A suggestion is a duplicate if it targets the same file and overlapping lines an
 class Analyzer {
   /**
    * @param {Object} database - Database instance
-   * @param {string} model - Model to use (e.g., 'opus', 'gemini-2.5-pro')
-   * @param {string} provider - Provider ID (e.g., 'claude', 'gemini'). Defaults to 'claude'.
+   * @param {string} model - Model to use (e.g., 'opus', 'gemini-3.1-pro-low')
+   * @param {string} provider - Provider ID (e.g., 'claude', 'antigravity'). Defaults to 'claude'.
    * @param {Object} providerOverrides - Per-call config overrides passed to createProvider (optional)
    * @param {Object|null} providerOverridesMap - Per-provider overrides map for council mode (provider ID → overrides)
    */
@@ -860,8 +860,8 @@ class Analyzer {
    *
    * Uses the bare command name since BIN_DIR is added to PATH in all
    * providers.  Using the absolute path causes issues with AI providers
-   * that pattern-match allowed tools (e.g., Gemini only matches
-   * 'git-diff-lines', not the full path).
+   * that pattern-match allowed shell commands by their bare command name
+   * (matching 'git-diff-lines', not the full path).
    *
    * @param {string|null} worktreePath - Path to the git worktree
    * @returns {string} e.g. `git-diff-lines` or `git-diff-lines --cwd "/path"`