@in-the-loop-labs/pair-review 3.1.0 → 3.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@in-the-loop-labs/pair-review",
-  "version": "3.1.0",
+  "version": "3.1.2",
   "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.1.0",
+  "version": "3.1.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.1.0",
+  "version": "3.1.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/analysis-config.css

@@ -81,3 +81,23 @@
 .option-hint:last-child {
   margin-bottom: 0;
 }
+
+/* Section disabled by provider capability —
+   pointer-events on the section blocks interaction; opacity is applied
+   only to the options area so the note remains clearly readable and
+   the already-disabled GitHub toggle doesn't double-dim. */
+.exclude-previous-section.exclude-previous-disabled {
+  pointer-events: none;
+}
+
+.exclude-previous-section.exclude-previous-disabled > summary {
+  opacity: 0.5;
+}
+
+.exclude-previous-section.exclude-previous-disabled .exclude-previous-options {
+  opacity: 0.5;
+}
+
+.exclude-previous-section .executable-provider-note {
+  margin: 8px 14px;
+}

package/public/js/components/AdvancedConfigTab.js

@@ -705,10 +705,13 @@ class AdvancedConfigTab {
 
   _populateProviderDropdown(select) {
     const currentValue = select.value;
+    const isConsolidation = select.dataset.target === 'orchestration';
     select.innerHTML = '';
     const providerIds = Object.keys(this.providers).filter(id => {
       const p = this.providers[id];
-      return !p.availability || p.availability.available;
+      if (p.availability && !p.availability.available) return false;
+      if (isConsolidation && p.capabilities?.consolidation === false) return false;
+      return true;
     }).sort((a, b) => (this.providers[a].name || a).localeCompare(this.providers[b].name || b));
 
     for (const id of providerIds) {

package/public/js/components/AnalysisConfigModal.js

@@ -616,6 +616,9 @@ class AnalysisConfigModal {
     if (this.selectedModel) {
       this.selectModel(this.selectedModel);
     }
+
+    // Update exclude-previous section based on provider's exclude_previous capability
+    this._updateExcludePreviousState();
   }
 
   /**
@@ -827,7 +830,9 @@ class AnalysisConfigModal {
       enabledLevels: [...this.enabledLevels],
       skipLevel3: !this.enabledLevels.includes(3),
       noLevels,
-      excludePrevious: this._getAndSaveExcludePrevious()
+      excludePrevious: selectedProvider?.capabilities?.exclude_previous === false
+        ? undefined
+        : this._getAndSaveExcludePrevious()
     };
 
     if (this.onSubmit) this.onSubmit(config);
@@ -1151,6 +1156,9 @@ class AnalysisConfigModal {
         dirtyHintContainer.style.display = 'none';
       }
     }
+
+    // Re-evaluate exclude-previous section based on new tab context
+    this._updateExcludePreviousState();
   }
 
   /**
@@ -1362,6 +1370,49 @@ class AnalysisConfigModal {
     return state;
   }
 
+  /**
+   * Update the exclude-previous section's enabled/disabled state based on
+   * the active tab and the selected provider's capabilities.
+   *
+   * Council/Advanced tabs: always enabled (orchestration handles dedup).
+   * Single Model tab: disabled when provider has exclude_previous === false.
+   *
+   * This only toggles the visual state of the entire section — individual
+   * checkbox checked/disabled state is never modified, so persisted
+   * localStorage values remain untouched.
+   * @private
+   */
+  _updateExcludePreviousState() {
+    const section = this.modal.querySelector('.exclude-previous-section');
+    if (!section) return;
+
+    const provider = this.providers[this.selectedProvider];
+    const disabledByCapability = this.activeTab === 'single'
+      && provider?.capabilities?.exclude_previous === false;
+
+    const existingNote = section.querySelector('.executable-provider-exclude-note');
+
+    if (disabledByCapability) {
+      section.classList.add('exclude-previous-disabled');
+      section.setAttribute('open', '');  // Ensure note is visible even if section was collapsed
+
+      if (!existingNote) {
+        const note = document.createElement('div');
+        note.className = 'executable-provider-note executable-provider-exclude-note';
+        note.textContent = 'This provider does not support excluding previous findings.';
+        const options = section.querySelector('.exclude-previous-options');
+        if (options) {
+          section.insertBefore(note, options);
+        } else {
+          section.appendChild(note);
+        }
+      }
+    } else {
+      section.classList.remove('exclude-previous-disabled');
+      if (existingNote) existingNote.remove();
+    }
+  }
+
   /**
    * Cleanup event listeners and pending timeouts
    */

package/public/js/components/CouncilProgressModal.js

@@ -1621,7 +1621,8 @@ class CouncilProgressModal {
    *       -> "Claude sonnet-4-5 (Balanced)"
    */
   _formatVoiceLabel(voice, { isExecutable = false } = {}) {
-    const provider = this._capitalize(voice.provider || 'unknown');
+    const providersMap = window.analysisConfigModal?.providers || {};
+    const provider = providersMap[voice.provider]?.name || this._capitalize(voice.provider || 'unknown');
     const model = voice.model || 'default';
     if (isExecutable) return `${provider} ${model}`;
     const tier = this._capitalize(voice.tier || 'balanced');

package/public/js/components/VoiceCentricConfigTab.js

@@ -770,10 +770,13 @@ class VoiceCentricConfigTab {
 
   _populateProviderDropdown(select) {
     const currentValue = select.value;
+    const isConsolidation = select.dataset.target === 'orchestration';
     select.innerHTML = '';
     const providerIds = Object.keys(this.providers).filter(id => {
       const p = this.providers[id];
-      return !p.availability || p.availability.available;
+      if (p.availability && !p.availability.available) return false;
+      if (isConsolidation && p.capabilities?.consolidation === false) return false;
+      return true;
     }).sort((a, b) => (this.providers[a].name || a).localeCompare(this.providers[b].name || b));
 
     for (const id of providerIds) {

package/public/js/local.js

@@ -1526,6 +1526,23 @@ class LocalManager {
     }
   }
 
+  /**
+   * Build a notification string describing a scope change for the chat agent.
+   * @param {string} prefix - Leading message (e.g. "Diff scope changed to X.")
+   * @param {{ description: string, diffCommand: string, excludes: string, includesUntracked: boolean }|null} hints - Scope git hints
+   * @returns {string} Formatted notification text
+   */
+  _buildScopeNotification(prefix, hints) {
+    const parts = [prefix];
+    if (hints) {
+      parts.push(`Scope: ${hints.description}`);
+      parts.push(`Diff command: \`${hints.diffCommand}\``);
+      if (hints.excludes) parts.push(hints.excludes);
+      if (hints.includesUntracked) parts.push('Untracked files are included. List them with: `git ls-files --others --exclude-standard`');
+    }
+    return parts.join('\n');
+  }
+
   /**
    * Apply the result of a scope-change POST to local state, UI, and diff.
    * Shared by _handleScopeChange and showBranchReviewDialog.handleConfirm.
@@ -1617,9 +1634,11 @@ class LocalManager {
       // Notify chat agent about scope change
       if (window.chatPanel) {
         const label = LS ? LS.scopeLabel(scopeStart, scopeEnd) : `${scopeStart}\u2013${scopeEnd}`;
-        window.chatPanel.queueDiffStateNotification(
-          `Diff scope changed to ${label}. The set of reviewed files has changed.`
+        const hints = LS ? LS.scopeGitHints(scopeStart, scopeEnd, this.localData?.baseBranch) : null;
+        const notification = this._buildScopeNotification(
+          `Diff scope changed to ${label}. The set of reviewed files has changed.`, hints
         );
+        window.chatPanel.queueDiffStateNotification(notification);
       }
 
       if (window.toast) {
@@ -1752,9 +1771,11 @@ class LocalManager {
 
         if (window.chatPanel) {
           const label = LS ? LS.scopeLabel('branch', newEnd) : 'branch';
-          window.chatPanel.queueDiffStateNotification(
-            `Diff scope changed to ${label} via branch review. The set of reviewed files has changed.`
+          const hints = LS ? LS.scopeGitHints('branch', newEnd, branchInfo.baseBranch) : null;
+          const notification = self._buildScopeNotification(
+            `Diff scope changed to ${label} via branch review. The set of reviewed files has changed.`, hints
           );
+          window.chatPanel.queueDiffStateNotification(notification);
         }
 
         if (window.toast) {

package/src/ai/executable-provider.js

@@ -86,6 +86,8 @@ ${rawOutput}`;
  * @param {Object} config.capabilities - Provider capabilities overrides
  * @param {boolean} config.capabilities.review_levels - Whether the tool supports L1/L2/L3 analysis
  * @param {boolean} config.capabilities.custom_instructions - Whether the tool supports custom instructions
+ * @param {boolean} config.capabilities.exclude_previous - Whether the tool supports excluding previous findings
+ * @param {boolean} config.capabilities.consolidation - Whether the tool can be used for consolidation
  * @param {Object} config.context_args - Maps context keys to CLI flags
  * @param {string} config.output_glob - Glob pattern to find result file
  * @param {string} config.mapping_instructions - Tool-specific mapping instructions for LLM
@@ -525,7 +527,9 @@ function createExecutableProviderClass(id, config) {
   const caps = config.capabilities || {};
   ExecProvider.capabilities = {
     review_levels: caps.review_levels !== undefined ? caps.review_levels : false,
-    custom_instructions: caps.custom_instructions !== undefined ? caps.custom_instructions : false
+    custom_instructions: caps.custom_instructions !== undefined ? caps.custom_instructions : false,
+    exclude_previous: caps.exclude_previous !== undefined ? caps.exclude_previous : false,
+    consolidation: caps.consolidation !== undefined ? caps.consolidation : false
   };
 
   return ExecProvider;

package/src/ai/provider.js

@@ -621,7 +621,9 @@ function getAllProvidersInfo() {
     // Build capabilities: executable providers define their own, others get defaults
     const capabilities = ProviderClass.capabilities || {
       review_levels: true,
-      custom_instructions: true
+      custom_instructions: true,
+      exclude_previous: true,
+      consolidation: true
     };
 
     providers.push({

package/src/chat/prompt-builder.js

@@ -8,6 +8,7 @@
  */
 
 const logger = require('../utils/logger');
+const { scopeGitHints, DEFAULT_SCOPE } = require('../local-scope');
 
 /**
  * Build a lean system prompt for chat sessions.
@@ -140,9 +141,27 @@ function buildReviewContext(review, prData) {
     lines.push(`This is a local code review for: ${name}`);
     lines.push('');
     lines.push('## Viewing Code Changes');
-    lines.push('The changes under review are **unstaged and untracked local changes**. Staged changes (`git diff --no-ext-diff --cached`) are treated as already reviewed.');
-    lines.push('To see the diff under review: `git diff --no-ext-diff`');
-    lines.push('Do NOT use `git diff HEAD~1` or `git log` — those show committed history, not the changes under review.');
+
+    const scopeStart = review.local_scope_start || DEFAULT_SCOPE.start;
+    const scopeEnd = review.local_scope_end || DEFAULT_SCOPE.end;
+    const baseBranch = review.local_base_branch || null;
+    const hints = scopeGitHints(scopeStart, scopeEnd, baseBranch);
+
+    if (hints) {
+      lines.push(`The current diff scope is **${hints.label}**: ${hints.description}`);
+      lines.push(`To see the diff under review: \`${hints.diffCommand}\``);
+      if (hints.includesUntracked) {
+        lines.push('Untracked files are included. List them with: `git ls-files --others --exclude-standard`');
+      }
+      if (hints.excludes) {
+        lines.push(hints.excludes);
+      }
+    } else {
+      lines.push('The changes under review are **unstaged and untracked local changes**. Staged changes (`git diff --no-ext-diff --cached`) are treated as already reviewed.');
+      lines.push('To see the diff under review: `git diff --no-ext-diff`');
+      lines.push('Do NOT use `git diff HEAD~1` or `git log` — those show committed history, not the changes under review.');
+    }
+    lines.push('The diff scope can change during this session. If you receive a "[Diff State Update]" notification, follow the updated diff command it provides.');
   } else {
     const parts = [];
     if (review.repository) {

package/src/github/parser.js

@@ -30,7 +30,7 @@ class PRArgumentParser {
     // Check if input is a PR number
     const prNumber = parseInt(input);
     if (isNaN(prNumber) || prNumber <= 0) {
-      throw new Error('Invalid input format. Expected: PR number, GitHub URL (https://github.com/owner/repo/pull/number), Graphite URL (https://app.graphite.com/github/pr/owner/repo/number), or pair-review:// URL');
+      throw new Error('Invalid input format. Expected: PR number, GitHub URL (https://github.com/owner/repo/pull/number), Graphite URL (https://app.graphite.com/github/pr/owner/repo/number or https://app.graphite.com/github/owner/repo/pull/number), or pair-review:// URL');
     }
 
     // Parse repository from current directory's git remote
@@ -112,14 +112,18 @@ class PRArgumentParser {
    * @returns {Object} Parsed information { owner, repo, number }
    */
   parseGraphiteURL(url) {
-    // Match Graphite PR URL pattern: https://app.graphite.{dev|com}/github/pr/owner/repo/number[/optional-title]
-    const match = url.match(/^https:\/\/app\.graphite\.(?:dev|com)\/github\/pr\/([^\/]+)\/([^\/]+)\/(\d+)(?:\/.*)?$/);
+    // Match Graphite PR URL patterns:
+    //   https://app.graphite.{dev|com}/github/pr/owner/repo/number[/optional-title]
+    //   https://app.graphite.{dev|com}/github/owner/repo/pull/number[/optional-title]
+    const match = url.match(/^https:\/\/app\.graphite\.(?:dev|com)\/github\/(?:pr\/([^\/]+)\/([^\/]+)\/(\d+)|([^\/]+)\/([^\/]+)\/pull\/(\d+))(?:\/[^?]*)?(?:\?.*)?$/);
 
     if (!match) {
-      throw new Error('Invalid Graphite URL format. Expected: https://app.graphite.com/github/pr/owner/repo/number');
+      throw new Error('Invalid Graphite URL format. Expected: https://app.graphite.com/github/pr/owner/repo/number or https://app.graphite.com/github/owner/repo/pull/number');
     }
 
-    const [, owner, repo, numberStr] = match;
+    const owner = match[1] || match[4];
+    const repo = match[2] || match[5];
+    const numberStr = match[3] || match[6];
     return this._createPRInfo(owner, repo, numberStr, 'Graphite');
   }
 
@@ -156,7 +160,7 @@ class PRArgumentParser {
         ? 'https://github.com/owner/repo/pull/number'
         : source === 'pair-review://'
           ? 'pair-review://pr/owner/repo/number'
-          : 'https://app.graphite.com/github/pr/owner/repo/number';
+          : 'https://app.graphite.com/github/pr/owner/repo/number or https://app.graphite.com/github/owner/repo/pull/number';
       throw new Error(`Invalid ${source} URL format. Expected: ${exampleUrl}`);
     }
 

package/src/local-scope.js

@@ -39,6 +39,88 @@ function scopeLabel(start, end) {
   return `${label(start)}\u2013${label(end)}`;
 }
 
+/**
+ * Return git command hints for a scope range.
+ * @param {string} start - Scope start
+ * @param {string} end - Scope end
+ * @param {string} [baseBranch] - Base branch name (e.g. 'main'); used in merge-base commands
+ * @returns {{ label: string, description: string, diffCommand: string, excludes: string, includesUntracked: boolean }|null}
+ */
+function scopeGitHints(start, end, baseBranch) {
+  if (!isValidScope(start, end)) return null;
+
+  const mb = baseBranch
+    ? '$(git merge-base ' + baseBranch + ' HEAD)'
+    : '<merge-base>';
+  const incUntracked = scopeIncludes(start, end, 'untracked');
+  const label = scopeLabel(start, end);
+
+  const key = start + '-' + end;
+  const hints = {
+    'branch-branch': {
+      description: 'Committed changes on this branch since the merge-base.',
+      diffCommand: 'git diff --no-ext-diff ' + mb + '..HEAD',
+      excludes: 'Staged, unstaged, and untracked changes are NOT included in the review.'
+    },
+    'branch-staged': {
+      description: 'Committed changes plus staged changes, relative to the merge-base.',
+      diffCommand: 'git diff --no-ext-diff --cached ' + mb,
+      excludes: 'Unstaged and untracked changes are NOT included in the review.'
+    },
+    'branch-unstaged': {
+      description: 'All tracked changes (committed, staged, and unstaged) relative to the merge-base.',
+      diffCommand: 'git diff --no-ext-diff ' + mb,
+      excludes: 'Untracked files are NOT included in the review.'
+    },
+    'branch-untracked': {
+      description: 'All changes (committed, staged, unstaged, and untracked) relative to the merge-base.',
+      diffCommand: 'git diff --no-ext-diff ' + mb,
+      excludes: ''
+    },
+    'staged-staged': {
+      description: 'Only staged changes (added to the index but not yet committed).',
+      diffCommand: 'git diff --no-ext-diff --cached',
+      excludes: 'Unstaged, untracked, and committed branch changes are NOT included in the review.'
+    },
+    'staged-unstaged': {
+      description: 'Staged and unstaged changes relative to HEAD.',
+      diffCommand: 'git diff --no-ext-diff HEAD',
+      excludes: 'Untracked files are NOT included in the review.'
+    },
+    'staged-untracked': {
+      description: 'Staged, unstaged, and untracked changes relative to HEAD.',
+      diffCommand: 'git diff --no-ext-diff HEAD',
+      excludes: ''
+    },
+    'unstaged-unstaged': {
+      description: 'Only unstaged working tree changes (not staged, not committed).',
+      diffCommand: 'git diff --no-ext-diff',
+      excludes: 'Staged changes (`git diff --no-ext-diff --cached`) are treated as already reviewed. Untracked files are NOT included in the review.'
+    },
+    'unstaged-untracked': {
+      description: 'Unstaged and untracked local changes.',
+      diffCommand: 'git diff --no-ext-diff',
+      excludes: 'Staged changes (`git diff --no-ext-diff --cached`) are treated as already reviewed.'
+    },
+    'untracked-untracked': {
+      description: 'Only untracked files (new files not yet added to git).',
+      diffCommand: 'git ls-files --others --exclude-standard',
+      excludes: 'Tracked file changes (staged, unstaged, committed) are NOT included in the review.'
+    }
+  };
+
+  const entry = hints[key];
+  if (!entry) return null;
+
+  return {
+    label: label,
+    description: entry.description,
+    diffCommand: entry.diffCommand,
+    excludes: entry.excludes,
+    includesUntracked: incUntracked && key !== 'untracked-untracked'
+  };
+}
+
 const LocalScope = {
   STOPS,
   DEFAULT_SCOPE,
@@ -47,6 +129,7 @@ const LocalScope = {
   includesBranch,
   fromLegacyMode,
   scopeLabel,
+  scopeGitHints,
 };
 
 if (typeof window !== 'undefined') {