@in-the-loop-labs/pair-review 3.0.2 → 3.0.3

package/bin/pair-review.js

@@ -8,10 +8,13 @@ const pkg = require('../package.json');
 const args = process.argv.slice(2);
 const isMCP = args.includes('--mcp');
 
-// Check for updates and notify user (skip in MCP mode to avoid stdout pollution)
+// Check for updates and notify user (skip in MCP mode and when config suppresses it)
 if (!isMCP) {
-  const updateNotifier = require('update-notifier');
-  updateNotifier({ pkg }).notify();
+  const { shouldSkipUpdateNotifier } = require('../src/config');
+  if (!shouldSkipUpdateNotifier()) {
+    const updateNotifier = require('update-notifier');
+    updateNotifier({ pkg }).notify();
+  }
 }
 
 async function main() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@in-the-loop-labs/pair-review",
-  "version": "3.0.2",
+  "version": "3.0.3",
   "description": "Your AI-powered code review partner - Close the feedback loop with AI coding agents",
   "main": "src/server.js",
   "bin": {

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

@@ -1,6 +1,6 @@
 {
   "name": "pair-review",
-  "version": "3.0.2",
+  "version": "3.0.3",
   "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.0.2",
+  "version": "3.0.3",
   "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

@@ -1616,9 +1616,13 @@
 }
 
 /* Scrollable wrapper for diff tables — provides per-file horizontal scroll
-   now that .d2h-file-wrapper and .diff-container use overflow:visible for sticky headers */
+   now that .d2h-file-wrapper and .diff-container use overflow:visible for sticky headers.
+   overflow-y must be explicitly hidden: CSS spec coerces it to 'auto' when overflow-x
+   is non-visible, which creates an accidental vertical scroll trap that eats wheel events
+   at the boundary instead of propagating them to .diff-view. */
 .d2h-file-body {
   overflow-x: auto;
+  overflow-y: hidden;
 }
 
 .d2h-diff-table {
@@ -5964,6 +5968,7 @@ body::before {
 .header-center {
   flex: 1;
   min-width: 0;
+  overflow: hidden;
   text-align: center;
 }
 

package/public/local.html

@@ -112,10 +112,24 @@
             font-family: 'JetBrains Mono', monospace;
             font-size: 11px;
             color: var(--color-text-primary);
+            white-space: nowrap;
+            overflow: hidden;
+            text-overflow: ellipsis;
+            max-width: 260px;
         }
 
         .local-branch-badge svg {
             color: var(--color-text-tertiary);
+            flex-shrink: 0;
+        }
+
+        .local-branch-badge span {
+            overflow: hidden;
+            text-overflow: ellipsis;
+        }
+
+        .local-branch-vs {
+            flex-shrink: 0;
         }
 
         /* Dark mode improvements for header icon buttons */
@@ -139,18 +153,25 @@
             gap: 16px;
             font-size: 13px;
             color: var(--color-text-secondary);
+            min-width: 0;
+            overflow: hidden;
         }
 
         .local-header-info .info-item {
             display: flex;
             align-items: center;
             gap: 6px;
+            min-width: 0;
+            overflow: hidden;
         }
 
         .local-header-info .info-value {
             color: var(--color-text-primary);
             font-weight: 600;
             font-size: 14px;
+            white-space: nowrap;
+            overflow: hidden;
+            text-overflow: ellipsis;
         }
 
         /* Path display in toolbar-meta (with left truncation) */
@@ -193,11 +214,12 @@
             border-radius: 4px;
             border: 1px dashed var(--color-border-primary);
             transition: all 0.15s ease;
-            max-width: 360px;
+            max-width: 100%;
             overflow: hidden;
             text-overflow: ellipsis;
             white-space: nowrap;
             position: relative;
+            display: inline-block;
         }
 
         .local-review-name:hover {

package/src/ai/analyzer.js

@@ -13,6 +13,7 @@ const { normalizePath, pathExistsInList, resolveRenamedFile } = require('../util
 const { buildFileLineCountMap, validateSuggestionLineNumbers } = require('../utils/line-validation');
 const { getPromptBuilder } = require('./prompts');
 const { formatValidFiles } = require('./prompts/shared/valid-files');
+const { GIT_DIFF_FLAGS } = require('../git/diff-flags');
 const {
   buildAnalysisLineNumberGuidance,
   buildOrchestrationLineNumberGuidance: buildOrchestrationGuidance,
@@ -27,13 +28,7 @@ const { buildSparseCheckoutGuidance } = require('./prompts/sparse-checkout-guida
 /** Minimum total suggestion count across all voices before consolidation is applied */
 const COUNCIL_CONSOLIDATION_THRESHOLD = 8;
 
-/**
- * Common git diff flags used across all diff operations.
- * - --no-color: Disable color output (guards against color.diff=always in user config)
- * - --no-ext-diff: Disable external diff drivers
- * - --src-prefix/--dst-prefix: Ensure consistent a/ b/ prefixes (overrides user's diff.noprefix)
- */
-const GIT_DIFF_COMMON_FLAGS = '--no-color --no-ext-diff --src-prefix=a/ --dst-prefix=b/';
+// GIT_DIFF_FLAGS imported from ../git/diff-flags
 
 /**
  * Build a human-readable display label for a council voice/reviewer.
@@ -667,7 +662,7 @@ Do NOT create suggestions for any files not in this list. If you cannot find iss
   async getChangedFilesList(worktreePath, prMetadata) {
     try {
       const { stdout } = await execPromise(
-        `git diff --no-ext-diff ${prMetadata.base_sha}...${prMetadata.head_sha} --name-only`,
+        `git diff ${GIT_DIFF_FLAGS} ${prMetadata.base_sha}...${prMetadata.head_sha} --name-only`,
         { cwd: worktreePath }
       );
       return stdout.trim().split('\n').filter(f => f.length > 0);
@@ -691,7 +686,7 @@ Do NOT create suggestions for any files not in this list. If you cannot find iss
     try {
       // Get modified tracked files (unstaged)
       const { stdout: unstaged } = await execPromise(
-        'git diff --no-ext-diff --name-only',
+        `git diff ${GIT_DIFF_FLAGS} --name-only`,
         { cwd: localPath }
       );
 
@@ -708,7 +703,7 @@ Do NOT create suggestions for any files not in this list. If you cannot find iss
       // Include staged files when scope includes staged
       if (options.includeStaged) {
         const { stdout: staged } = await execPromise(
-          'git diff --no-ext-diff --cached --name-only',
+          `git diff ${GIT_DIFF_FLAGS} --cached --name-only`,
           { cwd: localPath }
         );
         const stagedFiles = staged.trim().split('\n').filter(f => f.length > 0);
@@ -997,10 +992,10 @@ ${prMetadata.description || '(No description provided)'}
     const isLocal = prMetadata.reviewType === 'local';
     if (isLocal) {
       // For local mode, diff against HEAD to see working directory changes
-      return suffix ? `git diff ${GIT_DIFF_COMMON_FLAGS} HEAD ${suffix}` : `git diff ${GIT_DIFF_COMMON_FLAGS} HEAD`;
+      return suffix ? `git diff ${GIT_DIFF_FLAGS} HEAD ${suffix}` : `git diff ${GIT_DIFF_FLAGS} HEAD`;
     }
     // For PR mode, diff between base and head commits
-    const baseCmd = `git diff ${GIT_DIFF_COMMON_FLAGS} ${prMetadata.base_sha}...${prMetadata.head_sha}`;
+    const baseCmd = `git diff ${GIT_DIFF_FLAGS} ${prMetadata.base_sha}...${prMetadata.head_sha}`;
     return suffix ? `${baseCmd} ${suffix}` : baseCmd;
   }
 

package/src/chat/chat-providers.js

@@ -117,6 +117,7 @@ function getChatProvider(id) {
       env: overrides.env || {},
     };
     if (overrides.model) provider.model = overrides.model;
+    if (overrides.provider) provider.provider = overrides.provider;
     if (overrides.extra_args && Array.isArray(overrides.extra_args)) {
       provider.args = [...provider.args, ...overrides.extra_args];
     }
@@ -132,6 +133,7 @@ function getChatProvider(id) {
   if (overrides.name || overrides.label) merged.name = overrides.name || overrides.label;
   if (overrides.command) merged.command = overrides.command;
   if (overrides.model) merged.model = overrides.model;
+  if (overrides.provider) merged.provider = overrides.provider;
   if (overrides.env) merged.env = { ...merged.env, ...overrides.env };
   if (overrides.args) {
     merged.args = overrides.args;

package/src/chat/pi-bridge.js

@@ -14,6 +14,7 @@ const { EventEmitter } = require('events');
 const { spawn } = require('child_process');
 const { createInterface } = require('readline');
 const logger = require('../utils/logger');
+const { quoteShellArgs } = require('../ai/provider');
 
 /**
  * Dialog methods in extension_ui_request that expect a response.
@@ -30,6 +31,8 @@ class PiBridge extends EventEmitter {
    * @param {string} [options.systemPrompt] - System prompt text
    * @param {string} [options.tools] - Comma-separated tool list (default: 'read,grep,find,ls')
    * @param {string} [options.piCommand] - Override Pi command (default: 'pi')
+   * @param {Object} [options.env] - Extra env vars for subprocess
+   * @param {boolean} [options.useShell] - Use shell mode for multi-word commands
    * @param {string[]} [options.skills] - Array of skill file paths to load via --skill
    * @param {string[]} [options.extensions] - Array of extension directory paths to load via -e
    * @param {string} [options.sessionPath] - Path to a session file for resumption
@@ -42,6 +45,8 @@ class PiBridge extends EventEmitter {
     this.systemPrompt = options.systemPrompt || null;
     this.tools = options.tools || 'read,grep,find,ls';
     this.piCommand = options.piCommand || process.env.PAIR_REVIEW_PI_CMD || 'pi';
+    this.env = options.env || {};
+    this.useShell = options.useShell || false;
     this.skills = options.skills || [];
     this.extensions = options.extensions || [];
     this.sessionPath = options.sessionPath || null;
@@ -69,15 +74,20 @@ class PiBridge extends EventEmitter {
 
     const args = this._buildArgs();
     const command = this.piCommand;
-    const spawnArgs = args;
+    const useShell = this.useShell;
 
-    logger.info(`[PiBridge] Starting Pi RPC: ${command} ${spawnArgs.join(' ')}`);
+    // For multi-word commands (e.g. "devx pi"), use shell mode.
+    const spawnCmd = useShell ? `${command} ${quoteShellArgs(args).join(' ')}` : command;
+    const spawnArgs = useShell ? [] : args;
+
+    logger.info(`[PiBridge] Starting Pi RPC: ${command} ${args.join(' ')}`);
 
     return new Promise((resolve, reject) => {
-      const proc = spawn(command, spawnArgs, {
+      const proc = spawn(spawnCmd, spawnArgs, {
         cwd: this.cwd,
-        env: { ...process.env },
-        stdio: ['pipe', 'pipe', 'pipe']
+        env: { ...process.env, ...this.env },
+        stdio: ['pipe', 'pipe', 'pipe'],
+        shell: useShell,
       });
 
       this._process = proc;

package/src/chat/session-manager.js

@@ -548,6 +548,7 @@ class ChatSessionManager {
       const providerDef = getChatProvider(provider);
       return new ClaudeCodeBridge({
         ...options,
+        model: options.model || providerDef?.model,
         claudeCommand: providerDef?.command,
         env: providerDef?.env,
         useShell: providerDef?.useShell,
@@ -564,8 +565,19 @@ class ChatSessionManager {
         useShell: providerDef?.useShell,
       });
     }
+    // Pi provider — resolve config overrides (command, model, env) from provider def.
+    // options.provider is the chat provider ID (e.g. "pi") — do NOT pass it to PiBridge,
+    // which would forward it as `--provider pi` to the Pi CLI.  The CLI's --provider flag
+    // expects a model provider ("google", "anthropic", etc.) and should only come from
+    // explicit user configuration (providerDef.provider).
+    const providerDef = getChatProvider(provider);
     return new PiBridge({
       ...options,
+      provider: providerDef?.provider || null,
+      model: options.model || providerDef?.model,
+      piCommand: providerDef?.command,
+      env: providerDef?.env,
+      useShell: providerDef?.useShell,
       tools: CHAT_TOOLS,
       extensions: [taskExtensionDir],
     });

package/src/config.js

@@ -37,7 +37,8 @@ const DEFAULT_CONFIG = {
   monorepos: {},  // Monorepo configurations: { "owner/repo": { path: "~/path/to/clone" } }
   assisted_by_url: "https://github.com/in-the-loop-labs/pair-review",  // URL for "Review assisted by" footer link
   hooks: {},  // Hook commands per event: { "review.started": { "my_hook": { "command": "..." } } }
-  enable_graphite: false  // When true, shows Graphite links alongside GitHub links
+  enable_graphite: false,  // When true, shows Graphite links alongside GitHub links
+  skip_update_notifier: false  // When true, suppresses the "update available" notification on exit
 };
 
 /**
@@ -495,6 +496,42 @@ function warnIfDevModeWithoutDbName(config) {
   }
 }
 
+/**
+ * Synchronously checks whether the update notifier should be skipped.
+ * Reads config files in the standard merge order (managed → global → global.local
+ * → project → project.local) and returns the resolved boolean value of
+ * `skip_update_notifier`. Designed for use in bin/pair-review.js which runs
+ * before the async main process.
+ *
+ * @returns {boolean} True if the update notifier should be suppressed
+ */
+function shouldSkipUpdateNotifier() {
+  const fsSync = require('fs');
+  const localDir = path.join(process.cwd(), '.pair-review');
+  // Keep in sync with the sources list in loadConfig()
+  const sources = [
+    MANAGED_CONFIG_FILE,
+    CONFIG_FILE,
+    CONFIG_LOCAL_FILE,
+    path.join(localDir, 'config.json'),
+    path.join(localDir, 'config.local.json'),
+  ];
+
+  let skip = false;
+  for (const filePath of sources) {
+    try {
+      const data = fsSync.readFileSync(filePath, 'utf8');
+      const parsed = JSON.parse(data);
+      if ('skip_update_notifier' in parsed) {
+        skip = Boolean(parsed.skip_update_notifier);
+      }
+    } catch {
+      // File missing or malformed — skip silently
+    }
+  }
+  return skip;
+}
+
 module.exports = {
   deepMerge,
   loadConfig,
@@ -515,6 +552,7 @@ module.exports = {
   resolveMonorepoOptions,
   resolveDbName,
   warnIfDevModeWithoutDbName,
+  shouldSkipUpdateNotifier,
   _resetTokenCache,
   DEFAULT_CHECKOUT_TIMEOUT_MS
 };

package/src/git/diff-flags.js

@@ -0,0 +1,43 @@
+// Copyright 2026 Tim Perkins (tjwp) | SPDX-License-Identifier: Apache-2.0
+
+/**
+ * Shared git diff flags used across all diff operations.
+ *
+ * Rationale for each flag:
+ * - --no-color: Disable color output for consistent parsing (overrides color.diff / color.ui)
+ * - --no-ext-diff: Disable external diff drivers (overrides diff.external)
+ * - --src-prefix=a/ --dst-prefix=b/: Ensure consistent a/ b/ prefixes (overrides diff.noprefix / diff.mnemonicPrefix)
+ * - --no-relative: Ensure paths are repo-root-relative (overrides diff.relative)
+ */
+
+/**
+ * String form for execSync / exec shell calls (e.g. `git diff ${GIT_DIFF_FLAGS} ...`).
+ */
+const GIT_DIFF_FLAGS = '--no-color --no-ext-diff --src-prefix=a/ --dst-prefix=b/ --no-relative';
+
+/**
+ * Array form for simple-git .diff() calls (full diff output including file content).
+ */
+const GIT_DIFF_FLAGS_ARRAY = [
+  '--no-color',
+  '--no-ext-diff',
+  '--src-prefix=a/',
+  '--dst-prefix=b/',
+  '--no-relative'
+];
+
+/**
+ * Array form for simple-git .diffSummary() calls.
+ * Omits --src-prefix/--dst-prefix since diffSummary doesn't output file content with prefixes.
+ */
+const GIT_DIFF_SUMMARY_FLAGS_ARRAY = [
+  '--no-color',
+  '--no-ext-diff',
+  '--no-relative'
+];
+
+module.exports = {
+  GIT_DIFF_FLAGS,
+  GIT_DIFF_FLAGS_ARRAY,
+  GIT_DIFF_SUMMARY_FLAGS_ARRAY
+};

package/src/git/worktree.js

@@ -7,6 +7,7 @@ const { getConfigDir, DEFAULT_CHECKOUT_TIMEOUT_MS } = require('../config');
 const { WorktreeRepository, generateWorktreeId } = require('../database');
 const { getGeneratedFilePatterns } = require('./gitattributes');
 const { normalizeRepository, resolveRenamedFile, resolveRenamedFileOld } = require('../utils/paths');
+const { GIT_DIFF_FLAGS_ARRAY, GIT_DIFF_SUMMARY_FLAGS_ARRAY } = require('./diff-flags');
 const { spawn, execSync } = require('child_process');
 
 /**
@@ -534,10 +535,12 @@ class GitWorktreeManager {
 
       // Generate diff between base SHA and head SHA (not branch names)
       // This ensures we compare the exact commits from the PR, even if the base branch has moved
+      // Defensive flags to normalize output regardless of user's git config
+      // (see src/git/diff-flags.js for rationale)
       const diff = await git.diff([
         `${prData.base_sha}...${prData.head_sha}`,
         '--unified=3',
-        '--no-ext-diff'
+        ...GIT_DIFF_FLAGS_ARRAY
       ]);
 
       return diff;
@@ -560,7 +563,10 @@ class GitWorktreeManager {
 
       // Get file changes with stats using base SHA and head SHA
       // This ensures we get the exact files changed in the PR, even if the base branch has moved
-      const diffSummary = await git.diffSummary([`${prData.base_sha}...${prData.head_sha}`, '--no-ext-diff']);
+      const diffSummary = await git.diffSummary([
+        `${prData.base_sha}...${prData.head_sha}`,
+        ...GIT_DIFF_SUMMARY_FLAGS_ARRAY
+      ]);
 
       // Parse .gitattributes to identify generated files
       const gitattributes = await getGeneratedFilePatterns(worktreePath);

package/src/local-review.js

@@ -15,7 +15,8 @@ const { initializeDatabase, ReviewRepository, RepoSettingsRepository } = require
 const { startServer } = require('./server');
 const { localReviewDiffs } = require('./routes/shared');
 const { getShaAbbrevLength } = require('./git/sha-abbrev');
-const open = (...args) => import('open').then(({ default: open }) => open(...args));
+const { GIT_DIFF_FLAGS } = require('./git/diff-flags');
+const open = (...args) => process.env.PAIR_REVIEW_NO_OPEN ? Promise.resolve() : import('open').then(({ default: open }) => open(...args));
 
 // Design note: This module uses execSync for git commands despite async function signatures.
 // For a local CLI tool, synchronous execution is acceptable and simplifies error handling.
@@ -33,13 +34,7 @@ const MAX_FILE_SIZE = 1024 * 1024;
  */
 const GIT_DIFF_HAS_DIFFERENCES = 1;
 
-/**
- * Common git diff flags used across all diff operations.
- * - --no-color: Disable color output for consistent parsing
- * - --no-ext-diff: Disable external diff drivers
- * - --src-prefix/--dst-prefix: Ensure consistent a/ b/ prefixes (overrides user's diff.noprefix)
- */
-const GIT_DIFF_COMMON_FLAGS = '--no-color --no-ext-diff --src-prefix=a/ --dst-prefix=b/';
+// GIT_DIFF_FLAGS imported from ./git/diff-flags
 
 /**
  * Find the main git repository root, resolving through worktrees.
@@ -409,7 +404,7 @@ function generateUntrackedDiffs(repoPath, untrackedFiles, wFlag) {
         const filePath = path.join(repoPath, untracked.file);
         let fileDiff;
         try {
-          fileDiff = execSync(`git diff --no-index ${GIT_DIFF_COMMON_FLAGS}${wFlag} -- /dev/null "${filePath}"`, {
+          fileDiff = execSync(`git diff --no-index ${GIT_DIFF_FLAGS}${wFlag} -- /dev/null "${filePath}"`, {
             cwd: repoPath,
             encoding: 'utf8',
             stdio: ['pipe', 'pipe', 'pipe'],
@@ -486,37 +481,37 @@ async function generateScopedDiff(repoPath, scopeStart, scopeEnd, baseBranch, op
   try {
     if (hasBranch && !hasStaged && !hasUnstaged) {
       // Branch only → committed changes since merge-base
-      diff = execSync(`git diff ${mergeBaseSha}..HEAD ${GIT_DIFF_COMMON_FLAGS} --unified=25${wFlag}`, {
+      diff = execSync(`git diff ${mergeBaseSha}..HEAD ${GIT_DIFF_FLAGS} --unified=25${wFlag}`, {
         cwd: repoPath, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'],
         maxBuffer: 50 * 1024 * 1024
       });
     } else if (hasBranch && hasStaged && !hasUnstaged) {
       // Branch–Staged → staged changes relative to merge-base
-      diff = execSync(`git diff --cached ${mergeBaseSha} ${GIT_DIFF_COMMON_FLAGS} --unified=25${wFlag}`, {
+      diff = execSync(`git diff --cached ${mergeBaseSha} ${GIT_DIFF_FLAGS} --unified=25${wFlag}`, {
         cwd: repoPath, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'],
         maxBuffer: 50 * 1024 * 1024
       });
     } else if (hasBranch && hasUnstaged) {
       // Branch–Unstaged (or Branch–Untracked) → working tree vs merge-base
-      diff = execSync(`git diff ${mergeBaseSha} ${GIT_DIFF_COMMON_FLAGS} --unified=25${wFlag}`, {
+      diff = execSync(`git diff ${mergeBaseSha} ${GIT_DIFF_FLAGS} --unified=25${wFlag}`, {
         cwd: repoPath, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'],
         maxBuffer: 50 * 1024 * 1024
       });
     } else if (hasStaged && !hasUnstaged) {
       // Staged only → cached changes
-      diff = execSync(`git diff --cached ${GIT_DIFF_COMMON_FLAGS} --unified=25${wFlag}`, {
+      diff = execSync(`git diff --cached ${GIT_DIFF_FLAGS} --unified=25${wFlag}`, {
         cwd: repoPath, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'],
         maxBuffer: 50 * 1024 * 1024
       });
     } else if (hasStaged && hasUnstaged) {
       // Staged–Unstaged (or Staged–Untracked) → all changes vs HEAD
-      diff = execSync(`git diff HEAD ${GIT_DIFF_COMMON_FLAGS} --unified=25${wFlag}`, {
+      diff = execSync(`git diff HEAD ${GIT_DIFF_FLAGS} --unified=25${wFlag}`, {
         cwd: repoPath, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'],
         maxBuffer: 50 * 1024 * 1024
       });
     } else if (hasUnstaged) {
       // Unstaged only or Unstaged–Untracked → working tree changes
-      diff = execSync(`git diff ${GIT_DIFF_COMMON_FLAGS} --unified=25${wFlag}`, {
+      diff = execSync(`git diff ${GIT_DIFF_FLAGS} --unified=25${wFlag}`, {
         cwd: repoPath, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'],
         maxBuffer: 50 * 1024 * 1024
       });
@@ -536,7 +531,7 @@ async function generateScopedDiff(repoPath, scopeStart, scopeEnd, baseBranch, op
   // Count staged/unstaged for stats when relevant
   if (hasStaged) {
     try {
-      const stagedDiff = execSync(`git diff --cached --stat --no-color --no-ext-diff`, {
+      const stagedDiff = execSync(`git diff --cached --stat ${GIT_DIFF_FLAGS}`, {
         cwd: repoPath, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe']
       });
       if (stagedDiff.trim()) {
@@ -546,7 +541,7 @@ async function generateScopedDiff(repoPath, scopeStart, scopeEnd, baseBranch, op
   }
   if (hasUnstaged) {
     try {
-      const unstagedDiff = execSync(`git diff --stat --no-color --no-ext-diff`, {
+      const unstagedDiff = execSync(`git diff --stat ${GIT_DIFF_FLAGS}`, {
         cwd: repoPath, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe']
       });
       if (unstagedDiff.trim()) {
@@ -598,7 +593,7 @@ async function computeScopedDigest(repoPath, scopeStart, scopeEnd) {
   // Staged in scope → cached diff content
   if (scopeIncludes(scopeStart, scopeEnd, 'staged')) {
     try {
-      const result = await execAsync(`git diff --cached ${GIT_DIFF_COMMON_FLAGS}`, {
+      const result = await execAsync(`git diff --cached ${GIT_DIFF_FLAGS}`, {
         cwd: repoPath, encoding: 'utf8', maxBuffer: 50 * 1024 * 1024
       });
       parts.push('STAGED:' + result.stdout);
@@ -610,7 +605,7 @@ async function computeScopedDigest(repoPath, scopeStart, scopeEnd) {
   // Unstaged in scope → working tree diff
   if (scopeIncludes(scopeStart, scopeEnd, 'unstaged')) {
     try {
-      const result = await execAsync(`git diff ${GIT_DIFF_COMMON_FLAGS}`, {
+      const result = await execAsync(`git diff ${GIT_DIFF_FLAGS}`, {
         cwd: repoPath, encoding: 'utf8', maxBuffer: 50 * 1024 * 1024
       });
       parts.push('UNSTAGED:' + result.stdout);
@@ -665,7 +660,7 @@ async function generateLocalDiff(repoPath, options = {}) {
   // Always count staged changes for CLI info message, even when staged is out of scope
   if (!result.stats.stagedChanges) {
     try {
-      const stagedStat = execSync('git diff --cached --stat --no-color --no-ext-diff', {
+      const stagedStat = execSync(`git diff --cached --stat ${GIT_DIFF_FLAGS}`, {
         cwd: repoPath, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe']
       });
       if (stagedStat.trim()) {
@@ -881,7 +876,10 @@ async function handleLocalReview(targetPath, flags = {}) {
     const port = await startServer(db);
 
     // Open browser to local review view
-    const url = `http://localhost:${port}/local/${sessionId}`;
+    let url = `http://localhost:${port}/local/${sessionId}`;
+    if (flags.ai) {
+      url += '?analyze=true';
+    }
     console.log(`\nOpening browser to: ${url}`);
     await open(url);
 

package/src/main.js

@@ -15,8 +15,9 @@ const { normalizeRepository, resolveRenamedFile, resolveRenamedFileOld } = requi
 const logger = require('./utils/logger');
 const simpleGit = require('simple-git');
 const { getGeneratedFilePatterns } = require('./git/gitattributes');
+const { GIT_DIFF_FLAGS_ARRAY, GIT_DIFF_SUMMARY_FLAGS_ARRAY } = require('./git/diff-flags');
 const { getEmoji: getCategoryEmoji } = require('./utils/category-emoji');
-const open = (...args) => import('open').then(({default: open}) => open(...args));
+const open = (...args) => process.env.PAIR_REVIEW_NO_OPEN ? Promise.resolve() : import('open').then(({default: open}) => open(...args));
 const { registerProtocolHandler, unregisterProtocolHandler } = require('./protocol-handler');
 
 let db = null;
@@ -705,10 +706,17 @@ async function performHeadlessReview(args, config, db, flags, options) {
         }
       }
 
-      diff = await git.diff([`${prData.base_sha}...${prData.head_sha}`, '--unified=3', '--no-ext-diff']);
+      diff = await git.diff([
+        `${prData.base_sha}...${prData.head_sha}`,
+        '--unified=3',
+        ...GIT_DIFF_FLAGS_ARRAY
+      ]);
 
       // Get changed files
-      const diffSummary = await git.diffSummary([`${prData.base_sha}...${prData.head_sha}`, '--no-ext-diff']);
+      const diffSummary = await git.diffSummary([
+        `${prData.base_sha}...${prData.head_sha}`,
+        ...GIT_DIFF_SUMMARY_FLAGS_ARRAY
+      ]);
       const gitattributes = await getGeneratedFilePatterns(worktreePath);
 
       changedFiles = diffSummary.files.map(file => {

package/src/routes/pr.js

@@ -32,6 +32,7 @@ const { broadcastReviewEvent } = require('../events/review-events');
 const { fireHooks, hasHooks } = require('../hooks/hook-runner');
 const { buildReviewStartedPayload, buildReviewLoadedPayload, buildAnalysisStartedPayload, buildAnalysisCompletedPayload, getCachedUser } = require('../hooks/payloads');
 const simpleGit = require('simple-git');
+const { GIT_DIFF_FLAGS_ARRAY, GIT_DIFF_SUMMARY_FLAGS_ARRAY } = require('../git/diff-flags');
 const {
   activeAnalyses,
   reviewToAnalysisId,
@@ -706,10 +707,19 @@ router.get('/api/pr/:owner/:repo/:number/diff', async (req, res) => {
 
         if (baseSha && headSha) {
           // Regenerate diff with -w flag to ignore whitespace changes
-          diffContent = await git.diff([`${baseSha}...${headSha}`, '--unified=3', '-w', '--no-ext-diff']);
+          diffContent = await git.diff([
+            `${baseSha}...${headSha}`,
+            '--unified=3',
+            '-w',
+            ...GIT_DIFF_FLAGS_ARRAY
+          ]);
 
           // Regenerate changed files stats with -w flag
-          const diffSummary = await git.diffSummary([`${baseSha}...${headSha}`, '-w', '--no-ext-diff']);
+          const diffSummary = await git.diffSummary([
+            `${baseSha}...${headSha}`,
+            '-w',
+            ...GIT_DIFF_SUMMARY_FLAGS_ARRAY
+          ]);
           const gitattributes = await getGeneratedFilePatterns(worktreePath);
           changedFiles = diffSummary.files.map(file => {
             const resolvedFile = resolveRenamedFile(file.file);

package/src/server.js

@@ -210,7 +210,7 @@ async function startServer(sharedDb = null) {
     
     // Bulk-open — opens multiple local URLs in the OS browser via the `open` package.
     // Bypasses popup blockers since the server shells out directly.
-    const openUrl = (...args) => import('open').then(({ default: open }) => open(...args));
+    const openUrl = (...args) => process.env.PAIR_REVIEW_NO_OPEN ? Promise.resolve() : import('open').then(({ default: open }) => open(...args));
     app.post('/api/bulk-open', async (req, res) => {
       try {
         const { urls } = req.body || {};

package/src/utils/diff-file-list.js

@@ -2,6 +2,7 @@
 const { promisify } = require('util');
 const { exec } = require('child_process');
 const { queryOne } = require('../database');
+const { GIT_DIFF_FLAGS } = require('../git/diff-flags');
 
 const execPromise = promisify(exec);
 
@@ -37,7 +38,7 @@ async function getDiffFileList(db, review) {
     try {
       const opts = { cwd: review.local_path };
       const [{ stdout: unstaged }, { stdout: untracked }] = await Promise.all([
-        execPromise('git diff --no-ext-diff --name-only', opts),
+        execPromise(`git diff ${GIT_DIFF_FLAGS} --name-only`, opts),
         execPromise('git ls-files --others --exclude-standard', opts),
       ]);
       const combined = `${unstaged}\n${untracked}`