@in-the-loop-labs/pair-review 2.3.3 → 2.4.1

package/.pi/skills/review-model-guidance/SKILL.md

@@ -78,7 +78,7 @@ Here are the higher-end models and their strengths for code review:
   files or functions.
 
 **Google**
-- **google/gemini-3-pro-preview**: Google's latest and most capable model (1M context).
+- **google/gemini-3.1-pro-preview**: Google's latest and most capable model (1M context).
   Strong at cross-file analysis and understanding large codebases holistically.
 - **google/gemini-2.5-pro with thinking**: Excellent at large-context analysis — can
   reason over many files simultaneously with its 1M token context window. Good for

package/.pi/skills/review-roulette/SKILL.md

@@ -30,7 +30,7 @@ Example reasoning models you might see (provider/model format):
 - `openai/gpt-5.2-pro`
 - `google/gemini-2.5-pro` (with thinking)
 - `google/gemini-2.5-flash` (with thinking)
-- `google/gemini-3-pro-preview`
+- `google/gemini-3.1-pro-preview`
 - `xai/grok-4`
 
 The exact list depends on which API keys are configured. Always check — do not

package/README.md

@@ -248,7 +248,7 @@ pair-review loads configuration from multiple files, merged in order of increasi
 | 4 | `.pair-review/config.json` | Project-specific configuration (can be checked in) |
 | 5 (highest) | `.pair-review/config.local.json` | Personal project overrides (gitignored) |
 
-Nested objects (like `chat`, `providers`, `monorepos`) are deep-merged across layers — you only need to specify the keys you want to override.
+Nested objects (like `chat`, `providers`, `chat_providers`, `monorepos`) are deep-merged across layers — you only need to specify the keys you want to override.
 
 **`config.local.json`** files are intended for personal overrides that should not be committed to version control. Add `config.local.json` to your `.gitignore`.
 
@@ -471,6 +471,20 @@ npm install -g @mariozechner/pi-coding-agent
 
 Configure your preferred models in `providers.pi.models` — see [AI Provider Configuration](#ai-provider-configuration) for details. Everything else in pair-review works without Pi installed.
 
+**Chat provider command overrides:** To customize CLI commands for chat providers (e.g., to use a wrapper script), use the `chat_providers` config key:
+
+```json
+{
+  "chat_providers": {
+    "claude": { "command": "devx", "args": ["claude", "--"] },
+    "codex": { "command": "devx", "args": ["codex", "--", "app-server"] },
+    "opencode-acp": { "command": "devx", "args": ["opencode", "acp"] }
+  }
+}
+```
+
+Available chat provider IDs: `pi`, `claude`, `codex`, `copilot-acp`, `gemini-acp`, `opencode-acp`, `cursor-acp`. Each supports `command`, `args` (replaces defaults), `extra_args` (appends), and `env` overrides.
+
 **Keyboard shortcut:** Press `p` then `c` to toggle the chat panel.
 
 ### Customization

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@in-the-loop-labs/pair-review",
-  "version": "2.3.3",
+  "version": "2.4.1",
   "description": "Your AI-powered code review partner - Close the feedback loop with AI coding agents",
   "main": "src/server.js",
   "bin": {
@@ -59,6 +59,7 @@
   },
   "homepage": "https://github.com/in-the-loop-labs/pair-review#readme",
   "dependencies": {
+    "@agentclientprotocol/sdk": "^0.14.1",
     "@modelcontextprotocol/sdk": "^1.25.3",
     "@octokit/rest": "^19.0.11",
     "better-sqlite3": "^11.8.1",

package/plugin/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "pair-review",
-  "version": "2.3.3",
+  "version": "2.4.1",
   "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/skills/review-requests/SKILL.md

@@ -35,7 +35,7 @@ Each result provides `number`, `title`, `html_url`, and `repo` (as `owner/repo`)
 
 ### Phase 3: Open each PR in pair-review with auto-analysis
 
-**Limit**: Open at most **5** PRs. If more than 5 are found, open only the first 5 (most
+**Limit**: Open at most **10** PRs. If more than 10 are found, open only the first 10 (most
 recently updated) and report how many were skipped.
 
 For each PR found (up to the limit), open it in the browser with the `?analyze=true` query parameter, which

package/plugin-code-critic/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "code-critic",
-  "version": "2.3.3",
+  "version": "2.4.1",
   "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

@@ -5374,6 +5374,16 @@ tr.line-range-start .d2h-code-line-ctn {
   color: #1f883d;
 }
 
+.menu-item-icon {
+  display: flex;
+  align-items: center;
+  margin-left: 8px;
+}
+
+.menu-item-icon svg {
+  /* vertical alignment handled by flex parent */
+}
+
 .menu-item-text {
   flex: 1;
 }
@@ -5866,6 +5876,145 @@ body::before {
   text-align: center;
 }
 
+.pr-title-wrapper {
+  display: flex;
+  align-items: center;
+  justify-content: center;
+  gap: 2px;
+  position: relative;
+}
+
+.header .pr-description-toggle,
+[data-theme="dark"] .header .pr-description-toggle {
+  flex-shrink: 0;
+  background: transparent;
+  border: none;
+  color: var(--color-text-tertiary, #6e7681);
+  opacity: 0.7;
+  transition: opacity 0.15s ease, color 0.15s ease;
+}
+
+.pr-description-toggle:hover {
+  opacity: 1;
+  color: var(--color-text-secondary, #8b949e);
+}
+
+.pr-description-toggle.active {
+  opacity: 1;
+  color: var(--color-accent, #2f81f7);
+}
+
+/* PR description popover */
+.pr-description-popover {
+  position: absolute;
+  top: calc(100% + 8px);
+  left: 50%;
+  transform: translateX(-50%);
+  z-index: 1000;
+  width: min(500px, 90vw);
+  background: var(--color-bg-primary, #0d1117);
+  border: 1px solid var(--color-border-primary, rgba(255, 255, 255, 0.1));
+  border-radius: 8px;
+  box-shadow: 0 8px 24px rgba(0, 0, 0, 0.4), 0 2px 8px rgba(0, 0, 0, 0.2);
+  text-align: left;
+}
+
+.pr-description-popover-arrow {
+  position: absolute;
+  top: -6px;
+  left: 50%;
+  margin-left: -6px;
+  width: 12px;
+  height: 12px;
+  background: var(--color-bg-primary, #0d1117);
+  border-top: 1px solid var(--color-border-primary, rgba(255, 255, 255, 0.1));
+  border-left: 1px solid var(--color-border-primary, rgba(255, 255, 255, 0.1));
+  transform: rotate(45deg);
+}
+
+.pr-description-popover-header {
+  display: flex;
+  align-items: center;
+  justify-content: space-between;
+  padding: 8px 12px 8px 16px;
+  border-bottom: 1px solid var(--color-border-secondary, rgba(255, 255, 255, 0.06));
+}
+
+.pr-description-popover-title {
+  font-size: 0.75rem;
+  font-weight: 600;
+  color: var(--color-text-primary, #e6edf3);
+  text-transform: uppercase;
+  letter-spacing: 0.05em;
+}
+
+.pr-description-popover-close {
+  display: flex;
+  align-items: center;
+  justify-content: center;
+  width: 24px;
+  height: 24px;
+  padding: 0;
+  border: none;
+  border-radius: 4px;
+  background: transparent;
+  color: var(--color-text-tertiary, #6e7681);
+  cursor: pointer;
+  transition: all 0.15s ease;
+}
+
+.pr-description-popover-close:hover {
+  color: var(--color-text-secondary, #8b949e);
+  background: var(--color-bg-tertiary, rgba(255, 255, 255, 0.04));
+}
+
+.pr-description-popover-content {
+  padding: 12px 16px;
+  font-size: 0.8125rem;
+  line-height: 1.6;
+  color: var(--color-text-secondary, #8b949e);
+  max-height: calc(60vh - 40px);
+  overflow-y: auto;
+}
+
+.pr-description-popover-content p {
+  margin: 0 0 8px;
+}
+
+.pr-description-popover-content p:last-child {
+  margin-bottom: 0;
+}
+
+.pr-description-popover-content ul,
+.pr-description-popover-content ol {
+  margin: 4px 0 8px;
+  padding-left: 20px;
+}
+
+.pr-description-popover-content li {
+  margin: 2px 0;
+}
+
+.pr-description-popover-content code {
+  padding: 2px 6px;
+  background: var(--color-bg-tertiary, rgba(255, 255, 255, 0.04));
+  border-radius: 3px;
+  font-size: 0.75rem;
+}
+
+.pr-description-popover-content pre {
+  margin: 8px 0;
+  padding: 8px 12px;
+  background: var(--color-bg-tertiary, rgba(255, 255, 255, 0.04));
+  border-radius: 6px;
+  overflow-x: auto;
+}
+
+.pr-description-popover-content pre code {
+  padding: 0;
+  background: none;
+}
+
 .header-center .pr-title {
   font-size: 1rem;
   font-weight: 500;
@@ -10906,17 +11055,17 @@ body.resizing * {
   background: linear-gradient(to bottom, var(--chat-subtle), transparent);
   flex-shrink: 0;
 }
-/* Session picker wrapper */
-.chat-panel__session-picker {
+/* Provider picker */
+.chat-panel__provider-picker {
   position: relative;
   flex: 1;
   min-width: 0;
 }
 
-.chat-panel__session-picker-btn {
+.chat-panel__provider-picker-btn {
   display: flex;
   align-items: center;
-  gap: 8px;
+  gap: 6px;
   font-size: 13px;
   font-weight: 600;
   color: var(--chat-text);
@@ -10929,11 +11078,11 @@ body.resizing * {
   max-width: 100%;
 }
 
-.chat-panel__session-picker-btn:hover {
+.chat-panel__provider-picker-btn:hover {
   background: var(--color-bg-tertiary, rgba(128, 128, 128, 0.1));
 }
 
-.chat-panel__session-picker-btn svg:first-child {
+.chat-panel__provider-picker-btn svg:first-child {
   color: var(--chat-primary);
   flex-shrink: 0;
 }
@@ -10944,18 +11093,18 @@ body.resizing * {
   white-space: nowrap;
 }
 
-.chat-panel__chevron-sep {
-  opacity: 0.4;
-  font-size: 13px;
-}
-
-.chat-panel__chevron {
+.chat-panel__provider-chevron {
   flex-shrink: 0;
   opacity: 0.5;
+  transition: transform 0.15s ease;
 }
 
-/* Session dropdown flyout */
-.chat-panel__session-dropdown {
+.chat-panel__provider-picker-btn--open .chat-panel__provider-chevron {
+  transform: rotate(180deg);
+}
+
+/* Provider dropdown */
+.chat-panel__provider-dropdown {
   position: absolute;
   top: calc(100% + 4px);
   left: 0;
@@ -10970,6 +11119,72 @@ body.resizing * {
   padding: 4px;
 }
 
+.chat-panel__provider-empty {
+  padding: 16px;
+  text-align: center;
+  color: var(--color-text-tertiary);
+  font-size: 12px;
+}
+
+.chat-panel__provider-item {
+  display: flex;
+  align-items: center;
+  justify-content: space-between;
+  width: 100%;
+  padding: 8px 12px;
+  border: none;
+  background: transparent;
+  text-align: left;
+  border-radius: 6px;
+  cursor: pointer;
+  transition: background 0.15s ease;
+  font-size: 13px;
+  color: var(--color-text-primary, #1f2328);
+}
+
+.chat-panel__provider-item:hover:not([disabled]) {
+  background: var(--color-bg-tertiary, rgba(128, 128, 128, 0.08));
+}
+
+.chat-panel__provider-item--active {
+  background: var(--color-bg-secondary, #f6f8fa);
+}
+
+.chat-panel__provider-item--unavailable {
+  opacity: 0.5;
+  cursor: default;
+}
+
+.chat-panel__provider-name {
+  overflow: hidden;
+  text-overflow: ellipsis;
+  white-space: nowrap;
+}
+
+.chat-panel__provider-check {
+  flex-shrink: 0;
+  color: var(--chat-primary, #0969da);
+}
+
+/* Session picker wrapper (dropdown container) */
+.chat-panel__session-picker {
+  position: relative;
+}
+
+/* Session dropdown flyout — uses position:fixed to escape chat-panel overflow:hidden */
+.chat-panel__session-dropdown {
+  position: fixed;
+  width: 320px;
+  background: var(--color-bg-primary, #fff);
+  border: 1px solid var(--color-border-primary, #d0d7de);
+  border-radius: 8px;
+  box-shadow: 0 8px 24px rgba(0, 0, 0, 0.12);
+  max-height: 300px;
+  overflow-y: auto;
+  z-index: 1000;
+  padding: 4px;
+}
+
 .chat-panel__session-empty {
   padding: 16px;
   text-align: center;
@@ -11025,6 +11240,24 @@ body.resizing * {
   color: var(--color-text-tertiary, #656d76);
 }
 
+.chat-panel__session-provider {
+  display: inline-block;
+  font-size: 10px;
+  font-weight: 500;
+  color: var(--color-text-secondary, #59636e);
+  background: var(--color-bg-tertiary, rgba(128, 128, 128, 0.08));
+  border-radius: 3px;
+  padding: 0 4px;
+  margin-right: 6px;
+  line-height: 16px;
+  vertical-align: baseline;
+}
+
+[data-theme="dark"] .chat-panel__session-provider {
+  color: var(--color-text-secondary, #8b949e);
+  background: rgba(128, 128, 128, 0.15);
+}
+
 /* Dark theme overrides */
 [data-theme="dark"] .chat-panel__session-dropdown {
   background: var(--color-bg-primary, #0d1117);
@@ -11040,6 +11273,46 @@ body.resizing * {
   color: var(--color-text-primary, #e6edf3);
 }
 
+/* Dark theme: provider dropdown */
+[data-theme="dark"] .chat-panel__provider-dropdown {
+  background: var(--color-bg-primary, #0d1117);
+  border-color: var(--color-border-primary, #30363d);
+  box-shadow: 0 8px 24px rgba(0, 0, 0, 0.4);
+}
+
+[data-theme="dark"] .chat-panel__provider-item {
+  color: var(--color-text-primary, #e6edf3);
+}
+
+[data-theme="dark"] .chat-panel__provider-item--active {
+  background: var(--color-bg-secondary, #161b22);
+}
+
+/* History button (icon-only, in actions) */
+.chat-panel__history-btn {
+  display: flex;
+  align-items: center;
+  justify-content: center;
+  width: 28px;
+  height: 28px;
+  border: none;
+  background: transparent;
+  color: var(--color-text-tertiary);
+  border-radius: 6px;
+  cursor: pointer;
+  transition: background 0.15s ease, color 0.15s ease;
+}
+
+.chat-panel__history-btn:hover {
+  background: var(--color-bg-tertiary);
+  color: var(--color-text-primary);
+}
+
+.chat-panel__history-btn--open {
+  background: var(--color-bg-tertiary);
+  color: var(--color-text-primary);
+}
+
 .chat-panel__actions {
   display: flex;
   align-items: center;

package/public/index.html

@@ -542,55 +542,6 @@
             to { transform: rotate(360deg); }
         }
 
-        /* Usage Info (shown when no reviews) */
-        .usage-info {
-            background-color: var(--color-bg-primary);
-            border: 1px solid var(--color-border-primary);
-            border-radius: var(--radius-lg);
-            padding: 24px;
-            text-align: left;
-            max-width: 600px;
-            width: 100%;
-            margin-bottom: 32px;
-        }
-
-        .usage-info h3 {
-            font-size: 16px;
-            font-weight: 600;
-            margin-bottom: 16px;
-            color: var(--color-text-primary);
-        }
-
-        .usage-info p {
-            color: var(--color-text-secondary);
-            margin-bottom: 12px;
-        }
-
-        .usage-info code {
-            background-color: var(--color-bg-secondary);
-            padding: 2px 6px;
-            border-radius: var(--radius-sm);
-            font-family: var(--font-mono);
-            font-size: 13px;
-        }
-
-        .usage-info pre {
-            background-color: var(--color-bg-tertiary);
-            color: var(--color-text-primary);
-            padding: 16px;
-            border-radius: var(--radius-md);
-            margin: 12px 0;
-            overflow-x: auto;
-            font-family: var(--font-mono);
-            font-size: 13px;
-            line-height: 1.5;
-        }
-
-        .usage-info pre code {
-            background: none;
-            padding: 0;
-        }
-
         /* Recent Reviews Section */
         .recent-reviews-section {
             width: 100%;
@@ -754,6 +705,28 @@
             color: var(--color-text-primary);
         }
 
+        .btn-github-link {
+            display: inline-flex;
+            align-items: center;
+            justify-content: center;
+            width: 32px;
+            height: 32px;
+            padding: 0;
+            background: transparent;
+            border: 1px solid transparent;
+            border-radius: var(--radius-md);
+            color: var(--color-text-tertiary);
+            cursor: pointer;
+            transition: all var(--transition-fast);
+            text-decoration: none;
+        }
+
+        .btn-github-link:hover {
+            background-color: var(--color-bg-secondary);
+            border-color: var(--color-border-primary);
+            color: var(--color-text-primary);
+        }
+
         .recent-reviews-empty {
             text-align: center;
             padding: 40px 20px;
@@ -870,6 +843,76 @@
             display: block;
         }
 
+        .tab-divider {
+            width: 1px;
+            height: 16px;
+            background: var(--color-border-secondary);
+            align-self: center;
+            margin: 0 4px;
+        }
+
+        .tab-pane-header {
+            display: flex;
+            justify-content: flex-end;
+            align-items: center;
+            gap: 8px;
+            margin-bottom: 12px;
+        }
+
+        .fetched-at-label {
+            font-size: 12px;
+            color: var(--color-fg-muted);
+        }
+
+        .btn-refresh {
+            display: inline-flex;
+            align-items: center;
+            justify-content: center;
+            width: 32px;
+            height: 32px;
+            padding: 0;
+            background: transparent;
+            border: 1px solid var(--color-border-primary);
+            border-radius: var(--radius-md);
+            color: var(--color-text-tertiary);
+            cursor: pointer;
+            transition: all var(--transition-fast);
+        }
+
+        .btn-refresh:hover {
+            background: var(--color-bg-secondary);
+            color: var(--color-text-primary);
+        }
+
+        .btn-refresh.refreshing {
+            pointer-events: none;
+            opacity: 0.6;
+        }
+
+        .btn-refresh.refreshing svg {
+            animation: spin 0.8s linear infinite;
+        }
+
+        .collection-pr-row {
+            cursor: pointer;
+        }
+
+        .collection-pr-row:hover {
+            background: var(--color-bg-secondary);
+        }
+
+        .collection-pr-number {
+            color: var(--ai-primary);
+            font-weight: 500;
+            font-family: var(--font-mono);
+            cursor: pointer;
+        }
+
+        .collection-pr-number:hover {
+            color: var(--ai-secondary);
+            text-decoration: underline;
+        }
+
         /* Local Reviews Table - tighter spacing */
         .recent-reviews-table.local-table th,
         .recent-reviews-table.local-table td {
@@ -1021,10 +1064,6 @@
                 width: 100%;
             }
 
-            .usage-info {
-                padding: 20px;
-            }
-
             .recent-reviews-table {
                 font-size: 13px;
             }
@@ -1073,6 +1112,9 @@
                     <div class="section-header" id="recent-reviews-header">
                         <div class="tab-bar" id="unified-tab-bar">
                             <button class="tab-btn active" data-tab="pr-tab" type="button">Pull Requests</button>
+                            <button class="tab-btn" data-tab="review-requests-tab" type="button">My Review Requests</button>
+                            <button class="tab-btn" data-tab="my-prs-tab" type="button">My PRs</button>
+                            <span class="tab-divider"></span>
                             <button class="tab-btn" data-tab="local-tab" type="button">Local Reviews</button>
                         </div>
                     </div>
@@ -1104,6 +1146,32 @@
                         </div>
                     </div>
 
+                    <!-- My Review Requests Tab -->
+                    <div class="tab-pane" id="review-requests-tab">
+                        <div class="tab-pane-header">
+                            <span class="fetched-at-label" id="review-requests-fetched-at"></span>
+                            <button class="btn-refresh" id="refresh-review-requests" title="Refresh from GitHub">
+                                <svg width="14" height="14" viewBox="0 0 16 16" fill="currentColor"><path d="M1.705 8.005a.75.75 0 0 1 .834.656 5.5 5.5 0 0 0 9.592 2.97l-1.204-1.204a.25.25 0 0 1 .177-.427h3.646a.25.25 0 0 1 .25.25v3.646a.25.25 0 0 1-.427.177l-1.38-1.38A7.002 7.002 0 0 1 1.05 8.84a.75.75 0 0 1 .656-.834ZM8 2.5a5.487 5.487 0 0 0-4.131 1.869l1.204 1.204A.25.25 0 0 1 4.896 6H1.25A.25.25 0 0 1 1 5.75V2.104a.25.25 0 0 1 .427-.177l1.38 1.38A7.002 7.002 0 0 1 14.95 7.16a.75.75 0 0 1-1.49.178A5.5 5.5 0 0 0 8 2.5Z"/></svg>
+                            </button>
+                        </div>
+                        <div id="review-requests-container" class="recent-reviews-loading">
+                            Loading...
+                        </div>
+                    </div>
+
+                    <!-- My PRs Tab -->
+                    <div class="tab-pane" id="my-prs-tab">
+                        <div class="tab-pane-header">
+                            <span class="fetched-at-label" id="my-prs-fetched-at"></span>
+                            <button class="btn-refresh" id="refresh-my-prs" title="Refresh from GitHub">
+                                <svg width="14" height="14" viewBox="0 0 16 16" fill="currentColor"><path d="M1.705 8.005a.75.75 0 0 1 .834.656 5.5 5.5 0 0 0 9.592 2.97l-1.204-1.204a.25.25 0 0 1 .177-.427h3.646a.25.25 0 0 1 .25.25v3.646a.25.25 0 0 1-.427.177l-1.38-1.38A7.002 7.002 0 0 1 1.05 8.84a.75.75 0 0 1 .656-.834ZM8 2.5a5.487 5.487 0 0 0-4.131 1.869l1.204 1.204A.25.25 0 0 1 4.896 6H1.25A.25.25 0 0 1 1 5.75V2.104a.25.25 0 0 1 .427-.177l1.38 1.38A7.002 7.002 0 0 1 14.95 7.16a.75.75 0 0 1-1.49.178A5.5 5.5 0 0 0 8 2.5Z"/></svg>
+                            </button>
+                        </div>
+                        <div id="my-prs-container" class="recent-reviews-loading">
+                            Loading...
+                        </div>
+                    </div>
+
                     <!-- Local Reviews Tab: Input + Listing -->
                     <div class="tab-pane" id="local-tab">
                         <div class="start-review-section">
@@ -1134,10 +1202,6 @@
                         </div>
                     </div>
                 </div>
-
-                <!-- Usage Info (shown when no reviews, hidden initially during loading) -->
-                <!-- Content is populated from help modal via JS to avoid duplication -->
-                <div class="usage-info loading-hidden" id="usage-info"></div>
             </div>
         </main>
     </div>

package/public/js/components/AIPanel.js

@@ -1432,7 +1432,8 @@ class AIPanel {
      */
     clearAllFindings() {
         this.findings = [];
-        this.comments = [];
+        // NOTE: Do NOT clear this.comments here. User comments are independent
+        // of AI analysis and must persist across analysis runs.
         this.currentIndex = -1; // Reset navigation
         this.updateSegmentCounts();
         this.renderFindings();