@in-the-loop-labs/pair-review 2.4.2 → 2.4.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@in-the-loop-labs/pair-review",
-  "version": "2.4.2",
+  "version": "2.4.3",
   "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": "2.4.2",
+  "version": "2.4.3",
   "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": "2.4.2",
+  "version": "2.4.3",
   "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

@@ -12277,9 +12277,12 @@ body.resizing * {
   font-size: 14px;
 }
 
-/* Override flex: 1 so the file name doesn't push the badge away */
+/* Don't grow (keeps badge next to text), shrink for long paths, clip from the left so the filename stays visible */
 .context-file-header .d2h-file-name {
-  flex: none;
+  flex: 0 1 auto;
+  direction: rtl;
+  unicode-bidi: plaintext;
+  text-overflow: ellipsis;
 }
 
 /* Push viewed checkbox (and everything after it) to the right */

package/public/js/pr.js

@@ -4480,7 +4480,16 @@ class PRManager {
     const tbody = document.createElement('tbody');
     tbody.className = 'd2h-diff-tbody context-chunk';
     tbody.dataset.contextId = contextFile.id;
-    tbody.dataset.lineStart = contextFile.line_start;
+
+    // Compute effective display range, shifting for end-of-file
+    const clampedEnd = Math.min(contextFile.line_end, data.lines.length);
+    const intendedSize = contextFile.line_end - contextFile.line_start + 1;
+    let effectiveStart = contextFile.line_start;
+    const actualSize = clampedEnd - effectiveStart + 1;
+    if (actualSize < intendedSize && effectiveStart > 1) {
+      effectiveStart = Math.max(1, effectiveStart - (intendedSize - actualSize));
+    }
+    tbody.dataset.lineStart = effectiveStart;
 
     // Chunk header row with range label and per-chunk dismiss button
     const headerRow = document.createElement('tr');
@@ -4493,8 +4502,7 @@ class PRManager {
     contentTd.colSpan = 3;
     const rangeLabel = document.createElement('span');
     rangeLabel.className = 'context-range-label';
-    const lineEnd = Math.min(contextFile.line_end, data.lines.length);
-    rangeLabel.textContent = `Lines ${contextFile.line_start}\u2013${lineEnd}`;
+    rangeLabel.textContent = `Lines ${effectiveStart}\u2013${clampedEnd}`;
     contentTd.appendChild(rangeLabel);
     const chunkDismiss = document.createElement('button');
     chunkDismiss.className = 'context-chunk-dismiss';
@@ -4508,25 +4516,22 @@ class PRManager {
     headerRow.appendChild(contentTd);
     tbody.appendChild(headerRow);
 
-    const lineStart = contextFile.line_start;
-    const clampedEnd = Math.min(contextFile.line_end, data.lines.length);
-
     // Add expand-up gap row if there are lines above the context range
-    if (lineStart > 1) {
-      const gapAboveSize = lineStart - 1;
+    if (effectiveStart > 1) {
+      const gapAboveSize = effectiveStart - 1;
       const gapAbove = window.HunkParser.createGapRowElement(
         contextFile.file,
-        1,              // startLine (old coords)
-        lineStart - 1,  // endLine (old coords)
+        1,                  // startLine (old coords)
+        effectiveStart - 1, // endLine (old coords)
         gapAboveSize,
         'above',
         this.expandGapContext.bind(this),
-        1               // startLineNew (same as old for context files — no diff offset)
+        1                   // startLineNew (same as old for context files — no diff offset)
       );
       tbody.appendChild(gapAbove);
     }
 
-    for (let i = lineStart; i <= clampedEnd; i++) {
+    for (let i = effectiveStart; i <= clampedEnd; i++) {
       const lineData = {
         type: 'context',
         oldNumber: i,
@@ -4874,8 +4879,9 @@ class PRManager {
       lineStartVal = 1;
       lineEndVal = 100;
     } else if (lineEnd == null) {
-      lineStartVal = lineStart;
-      lineEndVal = lineStart + 49;
+      // Center a ~21-line window around the target line (±10 lines)
+      lineStartVal = Math.max(1, lineStart - 10);
+      lineEndVal = lineStartVal + 20;
     } else {
       lineStartVal = lineStart;
       lineEndVal = Math.min(lineEnd, lineStart + 499);

package/src/chat/claude-code-bridge.js

@@ -465,6 +465,14 @@ class ClaudeCodeBridge extends EventEmitter {
             toolName: name,
             status: 'start',
           });
+        } else if (event.content_block && event.content_block.type === 'text') {
+          // When a new text block starts and we already have accumulated text
+          // from a previous block, inject paragraph separation so the markdown
+          // renderer doesn't smash the blocks together (e.g., "diff.The").
+          if (this._accumulatedText) {
+            this._accumulatedText += '\n\n';
+            this.emit('delta', { text: '\n\n' });
+          }
         }
         break;