@in-the-loop-labs/pair-review 3.9.0 → 4.0.0

package/README.md

@@ -65,7 +65,7 @@ pair-review is a local web application for keeping humans in the loop with AI co
 - **Local-First**: All data and processing happens on your machine - no cloud dependencies
 - **GitHub-Familiar UI**: Interface feels instantly familiar to GitHub users
 - **Human-in-the-Loop**: AI suggests, you decide
-- **Multiple AI Providers**: Support for Claude, Gemini, Codex, Copilot, OpenCode, Cursor, and Pi. Use your existing subscription!
+- **Multiple AI Providers**: Support for Claude, Antigravity, Codex, Copilot, OpenCode, Cursor, and Pi. Use your existing subscription!
 - **Progressive**: Start simple with manual review, add AI analysis when you need it
 
 ## Workflows
@@ -437,7 +437,7 @@ pair-review supports several environment variables for customizing behavior:
 |----------|-------------|---------|
 | `GITHUB_TOKEN` | GitHub Personal Access Token (takes precedence over config file) | - |
 | `PAIR_REVIEW_CLAUDE_CMD` | Custom command to invoke Claude CLI | `claude` |
-| `PAIR_REVIEW_GEMINI_CMD` | Custom command to invoke Gemini CLI | `gemini` |
+| `PAIR_REVIEW_ANTIGRAVITY_CMD` | Custom command to invoke Antigravity CLI | `agy` |
 | `PAIR_REVIEW_CODEX_CMD` | Custom command to invoke Codex CLI | `codex` |
 | `PAIR_REVIEW_COPILOT_CMD` | Custom command to invoke Copilot CLI | `copilot` |
 | `PAIR_REVIEW_OPENCODE_CMD` | Custom command to invoke OpenCode CLI | `opencode` |
@@ -466,8 +466,8 @@ PAIR_REVIEW_CLAUDE_CMD="/usr/local/bin/claude" pair-review 123
 # Use a wrapper command (supports multi-word commands)
 PAIR_REVIEW_CLAUDE_CMD="devx claude" pair-review 123
 
-# Use Gemini with npx
-PAIR_REVIEW_GEMINI_CMD="npx gemini" pair-review --local
+# Use a custom path for the Antigravity CLI
+PAIR_REVIEW_ANTIGRAVITY_CMD="/usr/local/bin/agy" pair-review --local
 
 # Force a specific model for this review
 PAIR_REVIEW_MODEL="opus" pair-review 123
@@ -491,7 +491,7 @@ Create a Personal Access Token (PAT) with these scopes:
 pair-review integrates with AI providers via their CLI tools:
 
 - **Claude**: Uses Claude Code CLI
-- **Gemini**: Uses Gemini CLI
+- **Antigravity**: Uses the Antigravity CLI (`agy`), the successor to the Gemini CLI. Install it with `curl -fsSL https://antigravity.google/cli/install.sh | bash` (macOS/Linux; it is not an npm package) — see the [Antigravity docs](https://antigravity.google/docs). Antigravity is an analysis-only provider: it runs as an agentic reviewer in non-interactive print mode and has **no chat/ACP mode**.
 - **Codex**: Uses Codex CLI
 - **GitHub Copilot**: Uses Copilot CLI
 - **OpenCode**: Uses OpenCode CLI (requires model configuration)
@@ -502,7 +502,7 @@ You can select your preferred provider and model in the repository settings UI.
 
 #### Built-in vs. Configurable Providers
 
-Most providers (Claude, Gemini, Codex, Copilot) come with built-in model definitions. **OpenCode and Pi are different** - they have no built-in models and require you to configure which models to use.
+Most providers (Claude, Antigravity, Codex, Copilot) come with built-in model definitions. **OpenCode and Pi are different** - they have no built-in models and require you to configure which models to use.
 
 #### Configuring Custom Models
 
@@ -648,7 +648,7 @@ The key difference from Review Council is consolidation order: Advanced consolid
 
 - **Large changesets with Opus 4.6**: Use review council instructions to tell the model to use a Team — it will spawn sub-agents to divide the work
 - **Multi-model parallel review**: Give each model a specific focus area (security, performance, correctness) via per-model instructions and get combined results
-- **Cross-model perspectives**: Run the same analysis across different providers (e.g., Claude, Gemini, GPT) — where models agree signals high-confidence findings, but unique outliers from a single model can be equally valuable, catching issue types that only that model excels at spotting
+- **Cross-model perspectives**: Run the same analysis across different providers (e.g., Claude, Antigravity, GPT) — where models agree signals high-confidence findings, but unique outliers from a single model can be equally valuable, catching issue types that only that model excels at spotting
 
 ### Chat
 
@@ -683,7 +683,7 @@ Configure your preferred models in `providers.pi.models` — see [AI Provider Co
 }
 ```
 
-Available chat provider IDs: `pi`, `claude`, `codex`, `copilot-acp`, `gemini-acp`, `opencode-acp`, `cursor-acp`. Each supports `command`, `args` (replaces defaults), `extra_args` (appends), and `env` overrides. Codex chat also supports `sandbox`: use `workspace-write` by default, or `read-only` for discussion-only sessions.
+Available chat provider IDs: `pi`, `claude`, `codex`, `copilot-acp`, `opencode-acp`, `cursor-acp`. Each supports `command`, `args` (replaces defaults), `extra_args` (appends), and `env` overrides. Codex chat also supports `sandbox`: use `workspace-write` by default, or `read-only` for discussion-only sessions. (Antigravity is an analysis-only provider — it has no ACP mode, so there is no Antigravity chat provider.)
 
 **Keyboard shortcut:** Press `p` then `c` to toggle the chat panel.
 
@@ -959,8 +959,8 @@ These commands update your MCP configuration in `~/.claude/settings.json` (user-
 
 ### Prerequisites
 
-- Node.js 20.0.0 or higher
-- [pnpm](https://pnpm.io/) 10.x
+- Node.js 22.0.0 or higher
+- [pnpm](https://pnpm.io/) 11.x
 - Git
 
 ### Running Locally

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@in-the-loop-labs/pair-review",
-  "version": "3.9.0",
+  "version": "4.0.0",
   "description": "Your AI-powered code review partner - Close the feedback loop with AI coding agents",
   "main": "src/server.js",
   "bin": {
@@ -19,8 +19,9 @@
     "LICENSE"
   ],
   "engines": {
-    "node": ">=20.0.0"
+    "node": ">=22.0.0"
   },
+  "packageManager": "pnpm@11.9.0",
   "scripts": {
     "start": "node src/server.js",
     "dev": "node bin/pair-review.js",
@@ -47,7 +48,7 @@
     "cursor",
     "copilot",
     "codex",
-    "gemini"
+    "antigravity"
   ],
   "author": "Tim Perkins",
   "license": "Apache-2.0",
@@ -63,7 +64,7 @@
     "@agentclientprotocol/sdk": "^0.14.1",
     "@modelcontextprotocol/sdk": "^1.25.3",
     "@octokit/rest": "^19.0.11",
-    "better-sqlite3": "^11.8.1",
+    "better-sqlite3": "^12.11.1",
     "express": "^4.18.2",
     "glob": "^13.0.6",
     "markdown-it": "^13.0.2",
@@ -83,10 +84,5 @@
     "jsdom": "^29.0.1",
     "supertest": "^7.1.4",
     "vitest": "^4.0.16"
-  },
-  "pnpm": {
-    "onlyBuiltDependencies": [
-      "better-sqlite3"
-    ]
   }
 }

package/plugin/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "pair-review",
-  "version": "3.9.0",
+  "version": "4.0.0",
   "description": "pair-review app integration — Open PRs and local changes in the pair-review web UI, run server-side AI analysis, and address review feedback. Requires the pair-review MCP server.",
   "author": {
     "name": "in-the-loop-labs",

package/plugin-code-critic/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "code-critic",
-  "version": "3.9.0",
+  "version": "4.0.0",
   "description": "AI-powered code review analysis — Run three-level AI analysis and implement-review-fix loops directly in your coding agent. Works standalone, no server required.",
   "author": {
     "name": "in-the-loop-labs",

package/public/css/pr.css

@@ -6939,66 +6939,6 @@ body:not([data-theme="dark"]) .theme-icon-light {
   border-color: var(--color-border-secondary);
 }
 
-/* ============================================
-   Analysis Progress Dots
-   ============================================ */
-
-.analysis-progress-dots {
-  display: flex;
-  align-items: center;
-  gap: 6px;
-  margin-right: 8px;
-}
-
-.progress-dot {
-  width: 8px;
-  height: 8px;
-  border-radius: 50%;
-  border: 1.5px solid var(--color-accent-ai);
-  background: transparent;
-  transition: all 0.3s ease;
-}
-
-.progress-dot.active {
-  animation: pulse-dot 1.2s ease-in-out infinite;
-}
-
-.progress-dot.completed {
-  background: var(--color-accent-ai);
-  border-color: var(--color-accent-ai);
-}
-
-.progress-dot.error {
-  background: #ef4444;
-  border-color: #ef4444;
-}
-
-@keyframes pulse-dot {
-  0%, 100% {
-    transform: scale(1);
-    opacity: 1;
-  }
-  50% {
-    transform: scale(1.3);
-    opacity: 0.9;
-  }
-}
-
-/* Dark theme for progress dots */
-[data-theme="dark"] .progress-dot {
-  border-color: var(--color-accent-ai-dark);
-}
-
-[data-theme="dark"] .progress-dot.completed {
-  background: var(--color-accent-ai-dark);
-  border-color: var(--color-accent-ai-dark);
-}
-
-[data-theme="dark"] .progress-dot.error {
-  background: #f87171;
-  border-color: #f87171;
-}
-
 /* Magical Analyze Button - Amber AI CTA */
 #analyze-btn {
   background: linear-gradient(135deg, #fef3c7 0%, #fde68a 100%);
@@ -12959,6 +12899,18 @@ body.resizing * {
   animation: pulse-dot 1.2s ease-in-out infinite;
 }
 
+/* Pulse used by the running/setting-up stack-progress status icons above. */
+@keyframes pulse-dot {
+  0%, 100% {
+    transform: scale(1);
+    opacity: 1;
+  }
+  50% {
+    transform: scale(1.3);
+    opacity: 0.9;
+  }
+}
+
 .status-pending .stack-progress-status-icon {
   color: var(--color-text-muted, #d1d9e0);
 }

package/public/js/components/AdvancedConfigTab.js

@@ -170,7 +170,7 @@ class AdvancedConfigTab {
   /**
    * Set default orchestration provider/model for new councils.
    * Falls back to 'claude'/'sonnet' if not provided.
-   * @param {string} provider - Default provider ID (e.g., 'claude', 'gemini')
+   * @param {string} provider - Default provider ID (e.g., 'claude', 'antigravity')
    * @param {string} model - Default model ID (e.g., 'sonnet', 'opus')
    */
   setDefaultOrchestration(provider, model) {

package/public/js/components/AnalysisConfigModal.js

@@ -627,7 +627,7 @@ class AnalysisConfigModal {
   selectModel(modelId) {
     // Guard: never store a model that doesn't belong to the current provider.
     // Callers can pass a model resolved from a different scope (e.g. 'opus'
-    // while the provider is 'gemini'); fall back to the provider's default so
+    // while the provider is 'antigravity'); fall back to the provider's default so
     // the UI always shows a selected card and submissions carry a valid pair.
     if (modelId && this.models.length && !this.models.some(m => m.id === modelId)) {
       const fallback = this.models.find(m => m.default) || this.models[0];
@@ -952,7 +952,7 @@ class AnalysisConfigModal {
     }
     // Only honour currentModel when it belongs to the selected provider.
     // selectProvider() has already chosen a valid model for the provider (via
-    // tier matching); applying a foreign model here (e.g. 'opus' under 'gemini')
+    // tier matching); applying a foreign model here (e.g. 'opus' under 'antigravity')
     // would leave no model card selected and submit an invalid pair.
     if (options.currentModel && this.models.some(m => m.id === options.currentModel)) {
       this.selectModel(options.currentModel);

package/public/js/components/CouncilProgressModal.js

@@ -304,16 +304,6 @@ class CouncilProgressModal {
       }
     }
 
-    // Update toolbar progress dots
-    const manager = window.prManager || window.localManager;
-    if (manager?.updateProgressDot) {
-      for (let level = 1; level <= 4; level++) {
-        if (status.levels[level]) {
-          manager.updateProgressDot(level, status.levels[level].status);
-        }
-      }
-    }
-
     // Terminal states
     if (status.status === 'completed') {
       this._handleCompletion(status);

package/public/js/local.js

@@ -346,7 +346,7 @@ class LocalManager {
         const lastCouncilId = reviewSettings.last_council_id;
 
         // Resolve provider and model as a MATCHED pair so the council/advanced tabs
-        // are never seeded with a cross-provider model (e.g. gemini + opus), which
+        // are never seeded with a cross-provider model (e.g. antigravity + opus), which
         // would blank the model <select> and be rejected by the backend.
         const providersInfo = await manager._getProvidersInfo();
         const { provider: currentProvider, model: currentModel } = window.resolveProviderModelPair([
@@ -1893,55 +1893,6 @@ class LocalManager {
     };
     document.addEventListener('keydown', keyHandler);
   }
-
-  /**
-   * Get the progress dots container element
-   * @returns {HTMLElement|null}
-   */
-  getProgressDotsContainer() {
-    return document.getElementById('analysis-progress-dots');
-  }
-
-  /**
-   * Update a specific progress dot during analysis
-   * Maps level numbers to phase names:
-   * - Level 4 -> orchestration (finalization)
-   * - Level 1 -> level1
-   * - Level 2 -> level2
-   * - Level 3 -> level3
-   * @param {number} level - The level number (1, 2, 3, or 4)
-   * @param {string} status - The status ('running', 'completed', 'failed')
-   */
-  updateProgressDot(level, status) {
-    const container = this.getProgressDotsContainer();
-    if (!container) return;
-
-    // Map levels to dot phases
-    const phaseMap = {
-      4: 'orchestration', // Orchestration/finalization is level 4 in progress modal
-      1: 'level1',
-      2: 'level2',
-      3: 'level3'
-    };
-
-    const phase = phaseMap[level];
-    if (!phase) return;
-
-    const dot = container.querySelector(`[data-phase="${phase}"]`);
-    if (!dot) return;
-
-    // Remove existing states
-    dot.classList.remove('active', 'completed', 'error');
-
-    // Apply new state
-    if (status === 'running') {
-      dot.classList.add('active');
-    } else if (status === 'completed') {
-      dot.classList.add('completed');
-    } else if (status === 'failed') {
-      dot.classList.add('error');
-    }
-  }
 }
 
 // Initialize LocalManager when in local mode

package/public/js/pr.js

@@ -704,7 +704,7 @@ class PRManager {
     // Resolve provider and model as a MATCHED pair. Resolving each half
     // independently (repo || app || hardcoded) can mix a provider from one
     // scope with a model from another, yielding an invalid pair (e.g.
-    // gemini/opus) that startAnalysis would forward to the backend as-is.
+    // antigravity/opus) that startAnalysis would forward to the backend as-is.
     const providers = providersInfo || await this._getProvidersInfo();
     const { provider, model } = window.resolveProviderModelPair([
       { provider: repoSettings?.default_provider, model: repoSettings?.default_model },
@@ -3053,7 +3053,7 @@ class PRManager {
       ]);
 
       // Resolve provider and model as a MATCHED pair so the council/advanced tabs
-      // are never seeded with a cross-provider model (e.g. gemini + opus), which
+      // are never seeded with a cross-provider model (e.g. antigravity + opus), which
       // would blank the model <select> and be rejected by the backend.
       const providersInfo = await this._getProvidersInfo();
       const { provider: currentProvider, model: currentModel } = window.resolveProviderModelPair([
@@ -6588,9 +6588,6 @@ class PRManager {
     } else {
       btn.innerHTML = '<span class="analyzing-icon">✨</span> Analyzing...';
     }
-
-    // Show progress dots
-    this.showProgressDots();
   }
 
   /**
@@ -6615,9 +6612,6 @@ class PRManager {
     }
     btn.disabled = true;
 
-    // Complete all progress dots (they'll be hidden when button resets)
-    this.completeAllProgressDots();
-
     // Revert to normal after 2 seconds
     setTimeout(() => this.resetButton(), 2000);
   }
@@ -6645,114 +6639,6 @@ class PRManager {
     } else {
       btn.innerHTML = 'Analyze with AI';
     }
-
-    // Hide progress dots when resetting
-    this.hideProgressDots();
-  }
-
-  // ============================================
-  // Progress Dots Controller
-  // ============================================
-
-  /**
-   * Get the progress dots container
-   * @returns {HTMLElement|null}
-   */
-  getProgressDotsContainer() {
-    return document.getElementById('analysis-progress-dots');
-  }
-
-  /**
-   * Show progress dots (called when analysis starts)
-   */
-  showProgressDots() {
-    const container = this.getProgressDotsContainer();
-    if (!container) return;
-
-    container.style.display = 'flex';
-
-    // Reset all dots to initial state
-    const dots = container.querySelectorAll('.progress-dot');
-    dots.forEach(dot => {
-      dot.classList.remove('active', 'completed', 'error');
-    });
-
-    // Set first dot (orchestration) as active
-    const firstDot = container.querySelector('[data-phase="orchestration"]');
-    if (firstDot) {
-      firstDot.classList.add('active');
-    }
-  }
-
-  /**
-   * Hide progress dots (called when analysis completes or is cancelled)
-   */
-  hideProgressDots() {
-    const container = this.getProgressDotsContainer();
-    if (!container) return;
-
-    container.style.display = 'none';
-
-    // Reset all dots
-    const dots = container.querySelectorAll('.progress-dot');
-    dots.forEach(dot => {
-      dot.classList.remove('active', 'completed', 'error');
-    });
-  }
-
-  /**
-   * Update progress dots based on level status
-   * Maps level numbers to phases:
-   * - Level 4 (orchestration/finalization) -> orchestration
-   * - Level 1 -> level1
-   * - Level 2 -> level2
-   * - Level 3 -> level3
-   * @param {number} level - The level number (1, 2, 3, or 4)
-   * @param {string} status - The status ('running', 'completed', 'failed')
-   */
-  updateProgressDot(level, status) {
-    const container = this.getProgressDotsContainer();
-    if (!container) return;
-
-    // Map levels to dot phases
-    const phaseMap = {
-      4: 'orchestration', // Orchestration/finalization is level 4 in progress modal
-      1: 'level1',
-      2: 'level2',
-      3: 'level3'
-    };
-
-    const phase = phaseMap[level];
-    if (!phase) return;
-
-    const dot = container.querySelector(`[data-phase="${phase}"]`);
-    if (!dot) return;
-
-    // Remove existing states
-    dot.classList.remove('active', 'completed', 'error');
-
-    // Apply new state
-    if (status === 'running') {
-      dot.classList.add('active');
-    } else if (status === 'completed') {
-      dot.classList.add('completed');
-    } else if (status === 'failed') {
-      dot.classList.add('error');
-    }
-  }
-
-  /**
-   * Complete all progress dots (called briefly before hiding)
-   */
-  completeAllProgressDots() {
-    const container = this.getProgressDotsContainer();
-    if (!container) return;
-
-    const dots = container.querySelectorAll('.progress-dot');
-    dots.forEach(dot => {
-      dot.classList.remove('active', 'error');
-      dot.classList.add('completed');
-    });
   }
 
   /**
@@ -6960,7 +6846,7 @@ class PRManager {
       const lastCouncilId = reviewSettings.last_council_id;
 
       // Resolve provider and model as a MATCHED pair so the council/advanced tabs
-      // are never seeded with a cross-provider model (e.g. gemini + opus), which
+      // are never seeded with a cross-provider model (e.g. antigravity + opus), which
       // would blank the model <select> and be rejected by the backend.
       const providersInfo = await this._getProvidersInfo();
       const { provider: currentProvider, model: currentModel } = window.resolveProviderModelPair([

package/public/js/utils/provider-model.js

@@ -6,7 +6,7 @@
  * settings, then app config) and historically each half was resolved
  * independently with `repo || app || hardcoded`. When a scope overrides only
  * ONE half, the halves can come from different scopes and produce an invalid
- * pair such as `gemini` + `opus` (an Anthropic model). The modal then shows no
+ * pair such as `antigravity` + `opus` (an Anthropic model). The modal then shows no
  * selected model, and the non-modal auto-analyze path posts the broken pair
  * straight to the backend.
  *

package/public/local.html

@@ -435,13 +435,6 @@
                     </div>
                     <div class="diff-stats" id="diff-stats"></div>
                     <div class="toolbar-actions">
-                        <!-- Progress Dots (visible during analysis) -->
-                        <div class="analysis-progress-dots" id="analysis-progress-dots" style="display: none;">
-                            <span class="progress-dot" data-phase="level1" title="Level 1: Diff"></span>
-                            <span class="progress-dot" data-phase="level2" title="Level 2: File"></span>
-                            <span class="progress-dot" data-phase="level3" title="Level 3: Codebase"></span>
-                            <span class="progress-dot" data-phase="orchestration" title="Consolidation"></span>
-                        </div>
                         <button class="btn btn-sm btn-secondary" id="analyze-btn" title="Analyze with AI">
                             <svg class="analyze-icon" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" width="14" height="14">
                                 <path d="M9.813 15.904L9 18.75l-.813-2.846a4.5 4.5 0 00-3.09-3.09L2.25 12l2.846-.813a4.5 4.5 0 003.09-3.09L9 5.25l.813 2.846a4.5 4.5 0 003.09 3.09L15.75 12l-2.846.813a4.5 4.5 0 00-3.09 3.09z"/>

package/public/pr.html

@@ -231,13 +231,6 @@
                     </div>
                     <div class="diff-stats" id="diff-stats"></div>
                     <div class="toolbar-actions">
-                        <!-- Progress Dots (visible during analysis) -->
-                        <div class="analysis-progress-dots" id="analysis-progress-dots" style="display: none;">
-                            <span class="progress-dot" data-phase="level1" title="Level 1: Diff"></span>
-                            <span class="progress-dot" data-phase="level2" title="Level 2: File"></span>
-                            <span class="progress-dot" data-phase="level3" title="Level 3: Codebase"></span>
-                            <span class="progress-dot" data-phase="orchestration" title="Consolidation"></span>
-                        </div>
                         <button class="btn btn-sm btn-secondary" id="analyze-btn" title="Analyze with AI">
                             <svg class="analyze-icon" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" width="14" height="14">
                                 <path d="M9.813 15.904L9 18.75l-.813-2.846a4.5 4.5 0 00-3.09-3.09L2.25 12l2.846-.813a4.5 4.5 0 003.09-3.09L9 5.25l.813 2.846a4.5 4.5 0 003.09 3.09L15.75 12l-2.846.813a4.5 4.5 0 00-3.09 3.09z"/>

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"`