@in-the-loop-labs/pair-review 1.6.1 → 2.0.0

package/src/ai/gemini-provider.js

@@ -21,7 +21,8 @@ const BIN_DIR = path.join(__dirname, '..', '..', 'bin');
  */
 const GEMINI_MODELS = [
   {
-    id: 'gemini-3-flash-preview',
+    id: 'gemini-3-flash',
+    aliases: ['gemini-3-flash-preview'],
     name: '3.0 Flash',
     tier: 'fast',
     tagline: 'Rapid Sanity Check',
@@ -40,16 +41,28 @@ const GEMINI_MODELS = [
     default: true
   },
   {
-    id: 'gemini-3-pro-preview',
+    id: 'gemini-3-pro',
+    aliases: ['gemini-3-pro-preview'],
     name: '3.0 Pro',
     tier: 'thorough',
     tagline: 'Architectural Audit',
     description: 'Most intelligent Gemini model—advanced reasoning for deep architectural analysis',
     badge: 'Deep Dive',
     badgeClass: 'badge-power'
+  },
+  {
+    id: 'gemini-3.1-pro',
+    name: '3.1 Pro',
+    tier: 'thorough',
+    tagline: 'Latest & Greatest',
+    description: 'Newest Gemini model—cutting-edge reasoning for complex architectural reviews',
+    badge: 'Latest',
+    badgeClass: 'badge-power'
   }
 ];
 
+const DEFAULT_GEMINI_MODEL = 'gemini-2.5-pro';
+
 class GeminiProvider extends AIProvider {
   /**
    * @param {string} model - Model identifier
@@ -59,7 +72,7 @@ class GeminiProvider extends AIProvider {
    * @param {Object} configOverrides.env - Additional environment variables
    * @param {Object[]} configOverrides.models - Custom model definitions
    */
-  constructor(model = 'gemini-2.5-pro', configOverrides = {}) {
+  constructor(model = DEFAULT_GEMINI_MODEL, configOverrides = {}) {
     super(model);
 
     // Command precedence: ENV > config > default
@@ -690,7 +703,7 @@ class GeminiProvider extends AIProvider {
   }
 
   static getDefaultModel() {
-    return 'gemini-2.5-pro';
+    return DEFAULT_GEMINI_MODEL;
   }
 
   static getInstallInstructions() {

package/src/ai/prompts/config.js

@@ -4,7 +4,7 @@
  *
  * Defines tier mappings for the prompt system.
  * Note: Provider-specific model-to-tier mappings are defined in each provider's
- * getModels() method. Use getTierForModel() from src/ai/provider.js to query them.
+ * getModels() method. The tier is persisted on each analysis_runs record at creation time.
  */
 
 /**
@@ -21,6 +21,11 @@ const TIER_ALIASES = {
  */
 const TIERS = ['fast', 'balanced', 'thorough'];
 
+/**
+ * All accepted tier values (internal tiers + user-facing aliases)
+ */
+const VALID_TIERS = [...TIERS, ...Object.keys(TIER_ALIASES)];
+
 /**
  * Prompt types (analysis levels)
  */
@@ -44,6 +49,7 @@ function resolveTier(tierOrAlias) {
 module.exports = {
   TIER_ALIASES,
   TIERS,
+  VALID_TIERS,
   PROMPT_TYPES,
   resolveTier
 };

package/src/ai/provider-availability.js

@@ -2,7 +2,7 @@
 /**
  * Provider Availability Module
  *
- * Manages background checking of AI provider availability at server startup.
+ * Manages checking of AI provider availability at server startup.
  * Caches results and exposes them for the /api/providers endpoint.
  */
 

package/src/ai/provider.js

@@ -10,6 +10,7 @@ const path = require('path');
 const { spawn } = require('child_process');
 const logger = require('../utils/logger');
 const { extractJSON } = require('../utils/json-extractor');
+const { TIERS, TIER_ALIASES } = require('./prompts/config');
 
 // Directory containing bin scripts (git-diff-lines, etc.)
 const BIN_DIR = path.join(__dirname, '..', '..', 'bin');
@@ -345,23 +346,10 @@ function prettifyModelId(id) {
     .replace(/\b\w/g, c => c.toUpperCase());  // Capitalize each word
 }
 
-/**
- * Canonical model tiers
- */
-const CANONICAL_TIERS = ['fast', 'balanced', 'thorough'];
-
-/**
- * Tier aliases that map to canonical tiers
- */
-const TIER_ALIASES = {
-  free: 'fast',
-  premium: 'thorough'
-};
-
 /**
  * All valid tier values (canonical + aliases)
  */
-const VALID_TIERS = [...CANONICAL_TIERS, ...Object.keys(TIER_ALIASES)];
+const VALID_TIERS = [...TIERS, ...Object.keys(TIER_ALIASES)];
 
 /**
  * Normalize a tier to its canonical form
@@ -603,27 +591,6 @@ function createProvider(providerId, model = null) {
   return new ProviderClass(actualModel, { ...(overrides || {}), yolo: yoloMode });
 }
 
-/**
- * Get tier for a specific model from a provider
- * Queries the provider's model definitions (or config overrides) to find the tier
- * @param {string} providerId - Provider ID (e.g., 'claude', 'gemini')
- * @param {string} modelId - Model ID (e.g., 'sonnet', 'gemini-2.5-pro')
- * @returns {string|null} Tier name or null if provider or model not found
- */
-function getTierForModel(providerId, modelId) {
-  const ProviderClass = providerRegistry.get(providerId);
-  if (!ProviderClass) {
-    return null;
-  }
-
-  // Merge config models with built-in models
-  const overrides = providerConfigOverrides.get(providerId);
-  const models = mergeModels(ProviderClass.getModels(), overrides?.models);
-
-  const model = models.find(m => m.id === modelId);
-  return model?.tier || null;
-}
-
 /**
  * Test availability of a provider with timeout
  * @param {string} providerId - Provider ID
@@ -656,6 +623,27 @@ async function testProviderAvailability(providerId, timeout = 10000) {
   }
 }
 
+/**
+ * Get tier for a specific model from a provider
+ * Queries the provider's model definitions (or config overrides) to find the tier
+ * @param {string} providerId - Provider ID (e.g., 'claude', 'gemini')
+ * @param {string} modelId - Model ID (e.g., 'sonnet', 'gemini-2.5-pro')
+ * @returns {string|null} Tier name or null if provider or model not found
+ */
+function getTierForModel(providerId, modelId) {
+  const ProviderClass = providerRegistry.get(providerId);
+  if (!ProviderClass) {
+    return null;
+  }
+
+  // Merge config models with built-in models
+  const overrides = providerConfigOverrides.get(providerId);
+  const models = mergeModels(ProviderClass.getModels(), overrides?.models);
+
+  const model = models.find(m => m.id === modelId);
+  return model?.tier || null;
+}
+
 module.exports = {
   AIProvider,
   MODEL_TIERS,
@@ -665,12 +653,12 @@ module.exports = {
   getRegisteredProviderIds,
   getAllProvidersInfo,
   createProvider,
-  getTierForModel,
   testProviderAvailability,
   // Config override support
   applyConfigOverrides,
   getProviderConfigOverrides,
   inferModelDefaults,
   resolveDefaultModel,
-  prettifyModelId
+  prettifyModelId,
+  getTierForModel
 };

package/src/ai/stream-parser.js

@@ -3,7 +3,7 @@
  * Stream Parser - Side-channel parser for real-time AI streaming events
  *
  * Reads stdout data incrementally from provider processes and emits normalized
- * events for display in the ProgressModal. This is a read-only side channel;
+ * events for display in the progress modal. This is a read-only side channel;
  * the existing stdout buffering and final JSON extraction remain untouched.
  *
  * Normalized event shape:

package/src/chat/CONVENTIONS.md

@@ -0,0 +1,18 @@
+# Chat Module Conventions
+
+## Action handler pattern (ChatPanel.js)
+
+All action handlers (adopt, update, dismiss) follow identical structure:
+1. Guard: if streaming or no `_contextItemId`, return early
+2. Set `_pendingActionContext` with `{ type, itemId }`
+3. Set `inputEl.value` to clean, human-readable text (NO item IDs)
+4. Call `sendMessage()`
+
+See `_handleAdoptClick` as the canonical example. New action handlers must follow this pattern.
+
+## Action metadata separation
+
+Item IDs never appear in user-visible message text. They flow through:
+`_pendingActionContext` (frontend) → `actionContext` (API payload) → `[Action: ...]` hint (agent-facing message in session-manager.js)
+
+This keeps the chat transcript clean while giving the agent structured metadata it can parse reliably.