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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@in-the-loop-labs/pair-review",
-  "version": "3.2.0",
+  "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.0",
+  "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.0",
+  "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/public/css/pr.css

@@ -12085,6 +12085,25 @@ body.resizing * {
   padding: 0;
 }
 
+.chat-panel__message--assistant .chat-panel__bubble table {
+  border-collapse: collapse;
+  font-size: 12px;
+  margin: 8px 0;
+  overflow-x: auto;
+  display: block;
+}
+
+.chat-panel__message--assistant .chat-panel__bubble th,
+.chat-panel__message--assistant .chat-panel__bubble td {
+  border: 1px solid var(--color-border-primary);
+  padding: 4px 8px;
+}
+
+.chat-panel__message--assistant .chat-panel__bubble th {
+  background: var(--color-bg-tertiary);
+  font-weight: 600;
+}
+
 .chat-panel__message--assistant .chat-panel__bubble ul,
 .chat-panel__message--assistant .chat-panel__bubble ol {
   margin: 4px 0;
@@ -13287,7 +13306,7 @@ body.resizing * {
   box-shadow: 0 4px 16px var(--color-shadow, rgba(0, 0, 0, 0.12));
   z-index: 100;
   padding: 6px 0;
-  max-height: 420px;
+  max-height: min(calc(100vh - 120px), 80vh);
   overflow-y: auto;
 }
 

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 || {};

package/src/ai/pi-provider.js

@@ -130,6 +130,8 @@ class PiProvider extends AIProvider {
    * @param {string[]} configOverrides.extra_args - Additional CLI arguments
    * @param {Object} configOverrides.env - Additional environment variables
    * @param {Object[]} configOverrides.models - Custom model definitions
+   * @param {boolean} [configOverrides.load_skills=true] - When false, adds --no-skills to suppress skill auto-discovery
+   * @param {boolean} [configOverrides.app_extensions=true] - When false, omits the -e task extension flag
    */
   constructor(model, configOverrides = {}) {
     super(model || 'default');
@@ -197,19 +199,25 @@ class PiProvider extends AIProvider {
     // The task extension is loaded to give the model a subagent tool for delegating
     // work to isolated subprocesses, preserving the main context window.
     // --no-prompt-templates: prompt templates can't be triggered in -p mode, so suppress
-    // them to avoid wasting context. Skills and extensions are left enabled so the
-    // subprocess has access to the user's configured environment. To disable them,
-    // add --no-skills or --no-extensions to extra_args in provider/model config.
+    // them to avoid wasting context.
+    // load_skills (default true): when false, adds --no-skills to suppress auto-discovery.
+    //   Explicit --skill args from built-in models (e.g. multi-model) still load.
+    // app_extensions (default true): when false, omits the -e task extension flag.
+    //   Useful when auto-discovery already loads the extension (e.g. developing pair-review).
+    const loadSkills = configOverrides.load_skills !== false;
+    const appExtensions = configOverrides.app_extensions !== false;
     const sessionArgs = process.env.PAIR_REVIEW_PI_SESSION ? [] : ['--no-session'];
+    const extensionArgs = appExtensions ? ['-e', TASK_EXTENSION_DIR] : [];
+    const skillArgs = loadSkills ? [] : ['--no-skills'];
     let baseArgs;
     if (configOverrides.yolo) {
       baseArgs = ['-p', '--mode', 'json', ...cliModelArgs, ...sessionArgs,
         '--no-prompt-templates',
-        '-e', TASK_EXTENSION_DIR];
+        ...extensionArgs, ...skillArgs];
     } else {
       baseArgs = ['-p', '--mode', 'json', ...cliModelArgs, '--tools', 'read,bash,grep,find,ls', ...sessionArgs,
         '--no-prompt-templates',
-        '-e', TASK_EXTENSION_DIR];
+        ...extensionArgs, ...skillArgs];
     }
     const builtInArgs = builtIn?.extra_args || [];
     const providerArgs = configOverrides.extra_args || [];

package/src/ai/provider.js

@@ -532,6 +532,8 @@ function applyConfigOverrides(config) {
         installInstructions: providerConfig.installInstructions,
         extra_args: providerConfig.extra_args,
         env: providerConfig.env,
+        load_skills: providerConfig.load_skills,
+        app_extensions: providerConfig.app_extensions,
         models: AliasClass.getModels() !== BaseClass.getModels() ? AliasClass.getModels() : null
       });
       logger.debug(`Registered aliased provider: ${providerId} (base: ${providerConfig.type})`);
@@ -557,6 +559,8 @@ function applyConfigOverrides(config) {
       installInstructions: providerConfig.installInstructions,
       extra_args: providerConfig.extra_args,
       env: providerConfig.env,
+      load_skills: providerConfig.load_skills,
+      app_extensions: providerConfig.app_extensions,
       models: processedModels
     });
   }

package/src/chat/chat-providers.js

@@ -121,6 +121,8 @@ function getChatProvider(id) {
     if (overrides.extra_args && Array.isArray(overrides.extra_args)) {
       provider.args = [...provider.args, ...overrides.extra_args];
     }
+    if (overrides.load_skills !== undefined) provider.load_skills = overrides.load_skills;
+    if (overrides.app_extensions !== undefined) provider.app_extensions = overrides.app_extensions;
     if (provider.command.includes(' ')) {
       provider.useShell = true;
     }
@@ -142,6 +144,8 @@ function getChatProvider(id) {
   if (overrides.extra_args && Array.isArray(overrides.extra_args)) {
     merged.args = [...(merged.args || []), ...overrides.extra_args];
   }
+  if (overrides.load_skills !== undefined) merged.load_skills = overrides.load_skills;
+  if (overrides.app_extensions !== undefined) merged.app_extensions = overrides.app_extensions;
   // For multi-word commands (e.g. "devx claude"), use shell mode
   if (merged.command && merged.command.includes(' ')) {
     merged.useShell = true;

package/src/chat/pi-bridge.js

@@ -37,6 +37,7 @@ class PiBridge extends EventEmitter {
    * @param {string[]} [options.extensions] - Array of extension directory paths to load via -e
    * @param {string[]} [options.extraArgs] - Extra CLI args to append (e.g., from config extra_args)
    * @param {string} [options.sessionPath] - Path to a session file for resumption
+   * @param {boolean} [options.loadSkills] - When false, adds --no-skills to suppress auto-discovery (default: true)
    */
   constructor(options = {}) {
     super();
@@ -52,6 +53,7 @@ class PiBridge extends EventEmitter {
     this.extensions = options.extensions || [];
     this.extraArgs = options.extraArgs || [];
     this.sessionPath = options.sessionPath || null;
+    this.loadSkills = options.loadSkills !== false;
 
     this._process = null;
     this._readline = null;
@@ -290,6 +292,12 @@ class PiBridge extends EventEmitter {
       args.push('-e', ext);
     }
 
+    // Suppress skill auto-discovery when loadSkills is false.
+    // Explicit --skill entries above still load.
+    if (!this.loadSkills) {
+      args.push('--no-skills');
+    }
+
     // Append extra args from provider config (e.g., extra_args in chat_providers).
     // These go last so they can override earlier flags if needed.
     if (this.extraArgs.length > 0) {

package/src/chat/session-manager.js

@@ -575,6 +575,9 @@ class ChatSessionManager {
     // 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).
+    // app_extensions (default true): when false, omit pair-review's task extension.
+    // load_skills (default true): when false, suppress Pi's skill auto-discovery.
+    const appExtensions = def?.app_extensions !== false;
     return new PiBridge({
       ...options,
       provider: def?.provider || null,
@@ -584,7 +587,8 @@ class ChatSessionManager {
       env: def?.env,
       useShell: def?.useShell,
       tools: CHAT_TOOLS,
-      extensions: [taskExtensionDir],
+      extensions: appExtensions ? [taskExtensionDir] : [],
+      loadSkills: def?.load_skills,
     });
   }