@in-the-loop-labs/pair-review 3.3.1 → 3.3.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@in-the-loop-labs/pair-review",
-  "version": "3.3.1",
+  "version": "3.3.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.3.1",
+  "version": "3.3.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.3.1",
+  "version": "3.3.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/src/database.js

@@ -2586,14 +2586,14 @@ class WorktreePoolRepository {
   }
 
   /**
-   * Find all pool worktrees for background fetch (excludes 'switching' status).
+   * Find all pool worktrees for background fetch (excludes transient statuses).
    * Ordered by last_fetched_at ASC NULLS FIRST (coldest first).
    */
   async findAllForFetch(repository) {
     return await query(this.db, `
       SELECT id, path, last_fetched_at, status
       FROM worktree_pool
-      WHERE repository = ? COLLATE NOCASE AND status != 'switching'
+      WHERE repository = ? COLLATE NOCASE AND status NOT IN ('switching', 'creating')
       ORDER BY last_fetched_at ASC NULLS FIRST
     `, [repository]);
   }

package/src/git/worktree.js

@@ -66,8 +66,9 @@ class GitWorktreeManager {
   /**
    * Resolve which git remote points to the given repository URLs.
    * Compares normalized URLs against all configured remotes. If no match is
-   * found, adds (or updates) a managed `pair-review-base` remote so that
-   * subsequent fetches target the correct repository (e.g. the base repo of a fork PR).
+   * found, falls back to an existing non-managed remote instead of mutating the
+   * repository's git config. This preserves proxy/mirror setups where the
+   * canonical fetch URL may differ from GitHub's clone URL.
    *
    * @param {Object} git - simple-git instance
    * @param {string} cloneUrl - HTTPS clone URL of the target repository
@@ -81,9 +82,7 @@ class GitWorktreeManager {
     const remoteOutput = await git.raw(['remote', '-v']);
 
     if (!remoteOutput || !remoteOutput.trim()) {
-      console.log(`No remotes found, adding ${MANAGED_REMOTE} remote for ${cloneUrl}`);
-      await git.addRemote(MANAGED_REMOTE, cloneUrl);
-      return MANAGED_REMOTE;
+      throw new Error(`No remotes configured — cannot resolve base repository for ${cloneUrl}`);
     }
 
     // Parse remote output into { name: url } map (fetch URLs only)
@@ -115,9 +114,16 @@ class GitWorktreeManager {
 
     const normalizedCloneUrl = normalizeUrl(cloneUrl);
     const normalizedSshUrl = sshUrl ? normalizeUrl(sshUrl) : '';
+    const remoteNames = Object.keys(remotes);
+    const fallbackRemote = remotes.origin
+      ? 'origin'
+      : remoteNames.find((name) => name !== MANAGED_REMOTE) || 'origin';
 
-    // Check each remote for a match
+    // Check each non-managed remote for a direct URL match
     for (const [name, url] of Object.entries(remotes)) {
+      if (name === MANAGED_REMOTE) {
+        continue;
+      }
       const normalizedRemoteUrl = normalizeUrl(url);
       if (normalizedRemoteUrl === normalizedCloneUrl ||
           (normalizedSshUrl && normalizedRemoteUrl === normalizedSshUrl)) {
@@ -126,16 +132,13 @@ class GitWorktreeManager {
       }
     }
 
-    // No match found — add or update the managed remote
-    if (remotes[MANAGED_REMOTE]) {
-      console.log(`Updating ${MANAGED_REMOTE} remote URL to ${cloneUrl}`);
-      await git.raw(['remote', 'set-url', MANAGED_REMOTE, cloneUrl]);
-    } else {
-      console.log(`Adding ${MANAGED_REMOTE} remote for ${cloneUrl}`);
-      await git.addRemote(MANAGED_REMOTE, cloneUrl);
-    }
-
-    return MANAGED_REMOTE;
+    console.warn(
+      `No configured remote matched ${cloneUrl}; using existing remote '${fallbackRemote}' without modifying git config`
+    );
+    // NOTE: For fork PRs this may return a remote that does not point to the
+    // base repository. Callers rely on fetchPRHead's SHA-fallback path and
+    // tolerant base-branch fetching to handle this without mutating git config.
+    return fallbackRemote;
   }
 
   /**

package/src/main.js

@@ -1201,6 +1201,11 @@ function startPoolBackgroundFetches(db, config) {
               if (elapsed < intervalMs) continue;
             }
 
+            if (!fs.existsSync(entry.path)) {
+              logger.warn(`Background fetch skipped for ${entry.id}: directory no longer exists (${entry.path})`);
+              continue;
+            }
+
             logger.info(`Background fetch starting for ${repoName} pool worktree ${entry.id}`);
             try {
               const git = simpleGit(entry.path, { timeout: { block: 300000 } });