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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@in-the-loop-labs/pair-review",
-  "version": "3.0.5",
+  "version": "3.0.6",
   "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.5",
+  "version": "3.0.6",
   "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.5",
+  "version": "3.0.6",
   "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/src/git/base-branch.js

@@ -171,46 +171,43 @@ function tryDefaultBranch(repoPath, currentBranch, deps) {
 }
 
 /**
- * Synchronously detect the default branch for a repository.
+ * Synchronously detect the default branch for a repository using only
+ * local refs (no network I/O).
  *
- * Uses the same logic as tryDefaultBranch but returns just the branch name
- * (or null). Suitable for call sites that need a quick, synchronous answer
- * without the full detectBaseBranch priority chain.
+ * Priority:
+ *   1. `git symbolic-ref refs/remotes/origin/HEAD` — reads the local ref
+ *      that `git clone` sets automatically.
+ *   2. Check whether `refs/heads/main` or `refs/heads/master` exist locally.
  *
- * @param {string} repoPath - Absolute path to the repository
+ * @param {string} localPath - Absolute path to the repository
  * @param {Object} [_deps] - Dependency overrides for testing
  * @returns {string|null} Default branch name, or null if it cannot be determined
  */
-function getDefaultBranch(repoPath, _deps) {
+function getDefaultBranch(localPath, _deps) {
+  if (!localPath) return null;
   const deps = { ...defaults, ..._deps };
 
-  // Try `git remote show origin`
+  // Try symbolic-ref (set by git clone)
   try {
-    const output = deps.execSync('git remote show origin', {
-      cwd: repoPath,
+    const ref = deps.execSync('git symbolic-ref refs/remotes/origin/HEAD', {
+      cwd: localPath,
       encoding: 'utf8',
       stdio: ['pipe', 'pipe', 'pipe'],
-      timeout: 5000
-    });
-
-    const match = output.match(/HEAD branch:\s*(.+)/);
-    if (match) {
-      const branch = match[1].trim();
-      if (branch && branch !== '(unknown)') {
-        return branch;
-      }
-    }
+    }).trim();
+    // ref looks like "refs/remotes/origin/main"
+    const branch = ref.replace(/^refs\/remotes\/origin\//, '');
+    if (branch && branch !== ref) return branch;
   } catch {
-    // No remote or network issue — try local refs
+    // origin/HEAD not set — fall through to local check
   }
 
   // Fallback: check if main or master exists locally
   for (const candidate of ['main', 'master']) {
     try {
-      deps.execSync(`git rev-parse --verify ${candidate}`, {
-        cwd: repoPath,
+      deps.execSync(`git rev-parse --verify refs/heads/${candidate}`, {
+        cwd: localPath,
         encoding: 'utf8',
-        stdio: ['pipe', 'pipe', 'pipe']
+        stdio: ['pipe', 'pipe', 'pipe'],
       });
       return candidate;
     } catch {

package/src/routes/local.js

@@ -31,6 +31,9 @@ const { getGeneratedFilePatterns } = require('../git/gitattributes');
 const { getShaAbbrevLength } = require('../git/sha-abbrev');
 const { validateCouncilConfig, normalizeCouncilConfig } = require('./councils');
 const { TIERS, TIER_ALIASES, VALID_TIERS, resolveTier } = require('../ai/prompts/config');
+const { getProviderClass, createProvider } = require('../ai/provider');
+const { getDefaultBranch } = require('../git/base-branch');
+const { CommentRepository } = require('../database');
 const {
   activeAnalyses,
   localReviewDiffs,
@@ -79,9 +82,8 @@ function isBranchAvailable(branchName, scopeStart, localPath) {
   if (includesBranch(scopeStart)) return true;
   if (!branchName || branchName === 'HEAD' || branchName === 'unknown') return false;
 
-  const { getDefaultBranch } = require('../git/base-branch');
-  const defaultBranch = localPath ? getDefaultBranch(localPath) : null;
-  // If detection fails, fall back to checking main/master
+  // Detect the default branch using only local refs (no network).
+  const defaultBranch = getDefaultBranch(localPath);
   if (defaultBranch) {
     return branchName !== defaultBranch;
   }
@@ -561,19 +563,14 @@ router.get('/api/local/:reviewId', async (req, res) => {
     const baseBranch = review.local_base_branch || null;
 
     // When scope does NOT include branch, check for branch detection info
-    // Frontend uses this to suggest expanding scope to include branch
+    // Frontend uses this to suggest expanding scope to include branch.
+    // Only use already-cached results here — never block the response on
+    // GitHub API calls.  Background detection (after res.json) will populate
+    // the cache for subsequent requests.
     let branchInfo = null;
     const cachedDiff = getLocalReviewDiff(reviewId);
     if (!includesBranch(scopeStart) && cachedDiff?.branchInfo) {
       branchInfo = cachedDiff.branchInfo;
-    } else if (!includesBranch(scopeStart) && !cachedDiff && review.local_path) {
-      // No cache (web UI started session) — run detection on-demand
-      const config = req.app.get('config') || {};
-      branchInfo = await detectAndBuildBranchInfo(review.local_path, branchName, {
-        repository: repositoryName,
-        githubToken: getGitHubToken(config),
-        enableGraphite: config.enable_graphite === true
-      });
     }
 
     // Check repo settings for auto_branch_review preference