@in-the-loop-labs/pair-review 3.2.1 → 3.2.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@in-the-loop-labs/pair-review",
-  "version": "3.2.1",
+  "version": "3.2.2",
   "description": "Your AI-powered code review partner - Close the feedback loop with AI coding agents",
   "main": "src/server.js",
   "bin": {
@@ -81,5 +81,10 @@
     "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.2.1",
+  "version": "3.2.2",
   "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.2.1",
+  "version": "3.2.2",
   "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/ai/analyzer.js

@@ -96,7 +96,7 @@ function buildVoiceContext(voice, idx, instructions, progressCallback, db) {
   const voiceProvider = isExecutable ? createProvider(voice.provider, voice.model) : null;
 
   const voiceTier = voice.tier || 'balanced';
-  const voiceTimeout = voice.timeout || 600000;
+  const voiceTimeout = voice.timeout || ProviderClass?.defaultTimeout || 600000;
 
   // Wrap progress callback with voice-centric metadata
   const voiceProgressCallback = progressCallback ? (update) => {
@@ -319,7 +319,10 @@ class Analyzer {
     const runId = options.runId || uuidv4();
     const { analysisId, skipRunCreation, skipLevel3, reviewerNum, excludePrevious, serverPort } = options;
     const logPrefix = options.logPrefix || '';
-    const executionTimeout = options.timeout || 600000; // Default 10 minutes
+    // Respect provider-configured timeout (e.g. Pi's 15 min, executable providers)
+    const ProviderClass = getProviderClass(this.provider);
+    const providerTimeout = ProviderClass?.defaultTimeout;
+    const executionTimeout = options.timeout || providerTimeout || 600000; // Default 10 minutes
 
     // Resolve enabledLevels: prefer explicit option, fall back to skipLevel3 compat
     const enabledLevels = options.enabledLevels
@@ -3396,6 +3399,7 @@ File-level suggestions should NOT have a line number. They apply to the entire f
             : voice.customInstructions;
         }
 
+        const VoiceProviderClass = getProviderClass(voice.provider);
         voiceTasks.push({
           voiceId,
           reviewerLabel,
@@ -3404,7 +3408,7 @@ File-level suggestions should NOT have a line number. They apply to the entire f
           provider: voice.provider,
           model: voice.model,
           tier,
-          timeout: voice.timeout || 600000,
+          timeout: voice.timeout || VoiceProviderClass?.defaultTimeout || 600000,
           customInstructions: voiceInstructions,
           voiceCustomInstructions: voice.customInstructions || null
         });

package/src/ai/executable-provider.js

@@ -522,6 +522,11 @@ function createExecutableProviderClass(id, config) {
   ExecProvider.getDefaultModel = () => resolveDefaultModel(models) || models[0]?.id;
   ExecProvider.getInstallInstructions = () => config.installInstructions || `Install ${id}`;
 
+  // Surface configured timeout so the UI can pre-populate TimeoutSelect
+  if (config.timeout != null) {
+    ExecProvider.defaultTimeout = config.timeout;
+  }
+
   // Flags for the system
   ExecProvider.isExecutable = true;
   const caps = config.capabilities || {};