@in-the-loop-labs/pair-review 1.1.0 → 1.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@in-the-loop-labs/pair-review",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "description": "Your AI-powered code review partner - Close the feedback loop with AI coding agents",
   "main": "src/server.js",
   "bin": {

package/public/css/pr.css

@@ -5658,7 +5658,15 @@ body:not([data-theme="dark"]) .theme-icon-light {
   color: var(--color-text-tertiary);
   font-size: 0.75rem;
   max-width: var(--toolbar-branch-max-width);
-  overflow: hidden;
+  padding: 2px 6px;
+  background-color: var(--color-bg-tertiary);
+  border-radius: var(--radius-sm);
+  transition: all var(--transition-fast);
+}
+
+.toolbar-branch:hover {
+  background-color: var(--color-bg-secondary);
+  color: var(--color-text-secondary);
 }
 
 .toolbar-branch svg {
@@ -5672,6 +5680,36 @@ body:not([data-theme="dark"]) .theme-icon-light {
   text-overflow: ellipsis;
 }
 
+/* Shared copy button styles for toolbar (branch + commit) */
+.toolbar-copy-btn {
+  display: inline-flex;
+  align-items: center;
+  justify-content: center;
+  background: none;
+  border: none;
+  padding: 2px;
+  cursor: pointer;
+  color: var(--color-text-tertiary);
+  border-radius: var(--radius-xs);
+  transition: all var(--transition-fast);
+  opacity: 0;
+}
+
+.toolbar-branch:hover .toolbar-copy-btn,
+.toolbar-commit:hover .toolbar-copy-btn {
+  opacity: 1;
+}
+
+.toolbar-copy-btn:hover {
+  background-color: var(--color-bg-primary);
+  color: var(--color-accent-primary);
+}
+
+.toolbar-copy-btn.copied {
+  color: var(--color-success);
+  opacity: 1;
+}
+
 .toolbar-stat {
   font-family: var(--font-mono);
   font-size: 0.7rem;
@@ -5710,7 +5748,6 @@ body:not([data-theme="dark"]) .theme-icon-light {
   padding: 2px 6px;
   background-color: var(--color-bg-tertiary);
   border-radius: var(--radius-sm);
-  cursor: pointer;
   transition: all var(--transition-fast);
 }
 
@@ -5724,29 +5761,6 @@ body:not([data-theme="dark"]) .theme-icon-light {
   flex-shrink: 0;
 }
 
-.toolbar-commit-copy {
-  display: inline-flex;
-  align-items: center;
-  justify-content: center;
-  background: none;
-  border: none;
-  padding: 2px;
-  cursor: pointer;
-  color: var(--color-text-tertiary);
-  border-radius: var(--radius-xs);
-  transition: all var(--transition-fast);
-  opacity: 0;
-}
-
-.toolbar-commit:hover .toolbar-commit-copy {
-  opacity: 1;
-}
-
-.toolbar-commit-copy:hover {
-  background-color: var(--color-bg-primary);
-  color: var(--color-accent-primary);
-}
-
 /* Pending Draft Indicator in Toolbar */
 .pending-draft-indicator {
   display: inline-flex;

package/public/js/local.js

@@ -1329,19 +1329,59 @@ class LocalManager {
       pathText.title = fullPath;
     }
 
-    // Update branch name
+    // Update branch name (header badge)
     const branchText = document.getElementById('local-branch-text');
     if (branchText) {
       branchText.textContent = reviewData.branch || 'unknown';
     }
 
-    // Update commit SHA
+    // Update branch name (toolbar) and wire up copy button
+    const branchName = document.getElementById('pr-branch-name');
+    if (branchName) {
+      branchName.textContent = reviewData.branch || 'unknown';
+    }
+
+    const branchCopy = document.getElementById('pr-branch-copy');
+    if (branchCopy && !branchCopy.hasAttribute('data-listener-added')) {
+      branchCopy.setAttribute('data-listener-added', 'true');
+      branchCopy.addEventListener('click', async (e) => {
+        e.stopPropagation();
+        const branch = branchName ? branchName.textContent : '';
+        if (!branch || branch === '--' || branch === 'unknown') return;
+        try {
+          await navigator.clipboard.writeText(branch);
+          branchCopy.classList.add('copied');
+          setTimeout(() => branchCopy.classList.remove('copied'), 2000);
+        } catch (err) {
+          console.error('Failed to copy branch name:', err);
+        }
+      });
+    }
+
+    // Update commit SHA and wire up copy button
     const commitSha = document.getElementById('pr-commit-sha');
     if (commitSha && reviewData.localHeadSha) {
       commitSha.textContent = reviewData.localHeadSha.substring(0, 7);
       commitSha.dataset.fullSha = reviewData.localHeadSha;
     }
 
+    const commitCopy = document.getElementById('pr-commit-copy');
+    if (commitCopy && !commitCopy.hasAttribute('data-listener-added')) {
+      commitCopy.setAttribute('data-listener-added', 'true');
+      commitCopy.addEventListener('click', async (e) => {
+        e.stopPropagation();
+        const fullSha = commitSha ? commitSha.dataset.fullSha : '';
+        if (!fullSha) return;
+        try {
+          await navigator.clipboard.writeText(fullSha);
+          commitCopy.classList.add('copied');
+          setTimeout(() => commitCopy.classList.remove('copied'), 2000);
+        } catch (err) {
+          console.error('Failed to copy SHA:', err);
+        }
+      });
+    }
+
     // Update settings link visibility and href
     const settingsLink = document.getElementById('settings-link');
     if (settingsLink) {

package/public/js/modules/diff-renderer.js

@@ -150,6 +150,38 @@ class DiffRenderer {
     );
   }
 
+  /**
+   * Workaround for highlight.js Markdown grammar treating mid-word underscores
+   * as emphasis/strong markers.
+   *
+   * In Markdown, `_text_` is italic and `__text__` is bold, but only when the
+   * underscores are at word boundaries. highlight.js doesn't enforce this, so
+   * identifiers like `update_policy` get incorrectly highlighted:
+   *   update_policy -> update<span class="hljs-emphasis">_policy</span>
+   *   snake_case_var -> snake<span class="hljs-emphasis">_case_</span>var
+   *
+   * This method detects emphasis/strong spans that are preceded by a word
+   * character (letter, digit, or underscore) and strips the span wrapper,
+   * restoring the original text.
+   * @param {string} html - HTML output from highlight.js
+   * @returns {string} Corrected HTML
+   */
+  static fixMarkdownHighlighting(html) {
+    // Strip hljs-emphasis spans preceded by a word character (mid-word underscore)
+    // e.g., update<span class="hljs-emphasis">_policy</span> -> update_policy
+    html = html.replace(
+      /(\w)<span class="hljs-emphasis">(_[^<]*)<\/span>/g,
+      '$1$2'
+    );
+    // Strip hljs-strong spans preceded by a word character (mid-word double underscore)
+    // e.g., mid<span class="hljs-strong">__word__</span>bold -> mid__word__bold
+    html = html.replace(
+      /(\w)<span class="hljs-strong">(__[^<]*)<\/span>/g,
+      '$1$2'
+    );
+    return html;
+  }
+
   /**
    * Escape HTML characters
    * @param {string} text - Text to escape
@@ -304,6 +336,11 @@ class DiffRenderer {
           html = DiffRenderer.fixRubyHighlighting(html);
         }
 
+        // Workaround: highlight.js Markdown grammar treats mid-word underscores as emphasis
+        if (language === 'markdown') {
+          html = DiffRenderer.fixMarkdownHighlighting(html);
+        }
+
         contentCell.innerHTML = html;
       } catch (e) {
         // If highlighting fails, fall back to plain text

package/public/js/pr.js

@@ -520,12 +520,30 @@ class PRManager {
     // Update meta info - show only head branch, full info in tooltip
     const branchName = document.getElementById('pr-branch-name');
     const branchContainer = document.getElementById('pr-branch');
+    const branchCopy = document.getElementById('pr-branch-copy');
     if (branchName) {
       branchName.textContent = pr.head_branch;
       // Set tooltip with full branch info (base <- head, showing merge direction)
       if (branchContainer) {
         branchContainer.title = `${pr.base_branch} <- ${pr.head_branch}`;
       }
+
+      if (branchCopy && !branchCopy.hasAttribute('data-listener-added')) {
+        branchCopy.setAttribute('data-listener-added', 'true');
+        branchCopy.addEventListener('click', async (e) => {
+          e.stopPropagation();
+          const branch = branchName.textContent;
+          if (!branch || branch === '--') return;
+          try {
+            await navigator.clipboard.writeText(branch);
+            // Visual feedback
+            branchCopy.classList.add('copied');
+            setTimeout(() => branchCopy.classList.remove('copied'), 2000);
+          } catch (err) {
+            console.error('Failed to copy branch name:', err);
+          }
+        });
+      }
     }
 
     const additions = document.getElementById('pr-additions');

package/public/local.html

@@ -264,16 +264,29 @@
                         </svg>
                     </button>
                     <div class="toolbar-meta" id="toolbar-meta">
+                        <span class="toolbar-branch" id="pr-branch">
+                            <svg viewBox="0 0 16 16" fill="currentColor" width="12" height="12">
+                                <path d="M11.75 2.5a.75.75 0 100 1.5.75.75 0 000-1.5zm-2.25.75a2.25 2.25 0 113 2.122V6A2.5 2.5 0 0110 8.5H6a1 1 0 00-1 1v1.128a2.251 2.251 0 11-1.5 0V5.372a2.25 2.25 0 111.5 0v1.836A2.492 2.492 0 016 7h4a1 1 0 001-1v-.628A2.25 2.25 0 019.5 3.25zM4.25 12a.75.75 0 100 1.5.75.75 0 000-1.5zM3.5 3.25a.75.75 0 111.5 0 .75.75 0 01-1.5 0z"/>
+                            </svg>
+                            <span id="pr-branch-name">--</span>
+                            <button type="button" class="toolbar-branch-copy toolbar-copy-btn" id="pr-branch-copy" title="Copy branch name" aria-label="Copy branch name">
+                                <svg viewBox="0 0 16 16" fill="currentColor" width="10" height="10">
+                                    <path d="M0 6.75C0 5.784.784 5 1.75 5h1.5a.75.75 0 0 1 0 1.5h-1.5a.25.25 0 0 0-.25.25v7.5c0 .138.112.25.25.25h7.5a.25.25 0 0 0 .25-.25v-1.5a.75.75 0 0 1 1.5 0v1.5A1.75 1.75 0 0 1 9.25 16h-7.5A1.75 1.75 0 0 1 0 14.25Z"></path>
+                                    <path d="M5 1.75C5 .784 5.784 0 6.75 0h7.5C15.216 0 16 .784 16 1.75v7.5A1.75 1.75 0 0 1 14.25 11h-7.5A1.75 1.75 0 0 1 5 9.25Zm1.75-.25a.25.25 0 0 0-.25.25v7.5c0 .138.112.25.25.25h7.5a.25.25 0 0 0 .25-.25v-7.5a.25.25 0 0 0-.25-.25Z"></path>
+                                </svg>
+                            </button>
+                        </span>
+                        <span class="toolbar-separator"></span>
                         <span class="toolbar-stat toolbar-stat-additions" id="pr-additions">+0</span>
                         <span class="toolbar-stat toolbar-stat-deletions" id="pr-deletions">-0</span>
                         <span class="toolbar-files" id="pr-files-count">0 files</span>
                         <span class="toolbar-separator"></span>
-                        <span class="toolbar-commit" id="pr-commit" title="Click to copy full SHA">
+                        <span class="toolbar-commit" id="pr-commit" title="Commit SHA">
                             <svg viewBox="0 0 16 16" fill="currentColor" width="12" height="12">
                                 <path d="M11.93 8.5a4.002 4.002 0 0 1-7.86 0H.75a.75.75 0 0 1 0-1.5h3.32a4.002 4.002 0 0 1 7.86 0h3.32a.75.75 0 0 1 0 1.5Zm-1.43-.75a2.5 2.5 0 1 0-5 0 2.5 2.5 0 0 0 5 0Z"/>
                             </svg>
                             <span id="pr-commit-sha">HEAD</span>
-                            <button type="button" class="toolbar-commit-copy" id="pr-commit-copy" title="Copy full SHA" aria-label="Copy full SHA">
+                            <button type="button" class="toolbar-commit-copy toolbar-copy-btn" id="pr-commit-copy" title="Copy full SHA" aria-label="Copy full SHA">
                                 <svg viewBox="0 0 16 16" fill="currentColor" width="10" height="10">
                                     <path d="M0 6.75C0 5.784.784 5 1.75 5h1.5a.75.75 0 0 1 0 1.5h-1.5a.25.25 0 0 0-.25.25v7.5c0 .138.112.25.25.25h7.5a.25.25 0 0 0 .25-.25v-1.5a.75.75 0 0 1 1.5 0v1.5A1.75 1.75 0 0 1 9.25 16h-7.5A1.75 1.75 0 0 1 0 14.25Z"></path>
                                     <path d="M5 1.75C5 .784 5.784 0 6.75 0h7.5C15.216 0 16 .784 16 1.75v7.5A1.75 1.75 0 0 1 14.25 11h-7.5A1.75 1.75 0 0 1 5 9.25Zm1.75-.25a.25.25 0 0 0-.25.25v7.5c0 .138.112.25.25.25h7.5a.25.25 0 0 0 .25-.25v-7.5a.25.25 0 0 0-.25-.25Z"></path>

package/public/pr.html

@@ -147,18 +147,24 @@
                                 <path d="M11.75 2.5a.75.75 0 100 1.5.75.75 0 000-1.5zm-2.25.75a2.25 2.25 0 113 2.122V6A2.5 2.5 0 0110 8.5H6a1 1 0 00-1 1v1.128a2.251 2.251 0 11-1.5 0V5.372a2.25 2.25 0 111.5 0v1.836A2.492 2.492 0 016 7h4a1 1 0 001-1v-.628A2.25 2.25 0 019.5 3.25zM4.25 12a.75.75 0 100 1.5.75.75 0 000-1.5zM3.5 3.25a.75.75 0 111.5 0 .75.75 0 01-1.5 0z"/>
                             </svg>
                             <span id="pr-branch-name">--</span>
+                            <button type="button" class="toolbar-branch-copy toolbar-copy-btn" id="pr-branch-copy" title="Copy branch name" aria-label="Copy branch name">
+                                <svg viewBox="0 0 16 16" fill="currentColor" width="10" height="10">
+                                    <path d="M0 6.75C0 5.784.784 5 1.75 5h1.5a.75.75 0 0 1 0 1.5h-1.5a.25.25 0 0 0-.25.25v7.5c0 .138.112.25.25.25h7.5a.25.25 0 0 0 .25-.25v-1.5a.75.75 0 0 1 1.5 0v1.5A1.75 1.75 0 0 1 9.25 16h-7.5A1.75 1.75 0 0 1 0 14.25Z"></path>
+                                    <path d="M5 1.75C5 .784 5.784 0 6.75 0h7.5C15.216 0 16 .784 16 1.75v7.5A1.75 1.75 0 0 1 14.25 11h-7.5A1.75 1.75 0 0 1 5 9.25Zm1.75-.25a.25.25 0 0 0-.25.25v7.5c0 .138.112.25.25.25h7.5a.25.25 0 0 0 .25-.25v-7.5a.25.25 0 0 0-.25-.25Z"></path>
+                                </svg>
+                            </button>
                         </span>
                         <span class="toolbar-separator"></span>
                         <span class="toolbar-stat toolbar-stat-additions" id="pr-additions">+0</span>
                         <span class="toolbar-stat toolbar-stat-deletions" id="pr-deletions">-0</span>
                         <span class="toolbar-files" id="pr-files-count">0 files</span>
                         <span class="toolbar-separator"></span>
-                        <span class="toolbar-commit" id="pr-commit" title="Click to copy full SHA">
+                        <span class="toolbar-commit" id="pr-commit" title="Commit SHA">
                             <svg viewBox="0 0 16 16" fill="currentColor" width="12" height="12">
                                 <path d="M11.93 8.5a4.002 4.002 0 0 1-7.86 0H.75a.75.75 0 0 1 0-1.5h3.32a4.002 4.002 0 0 1 7.86 0h3.32a.75.75 0 0 1 0 1.5Zm-1.43-.75a2.5 2.5 0 1 0-5 0 2.5 2.5 0 0 0 5 0Z"/>
                             </svg>
                             <span id="pr-commit-sha">--</span>
-                            <button type="button" class="toolbar-commit-copy" id="pr-commit-copy" title="Copy full SHA" aria-label="Copy full SHA">
+                            <button type="button" class="toolbar-commit-copy toolbar-copy-btn" id="pr-commit-copy" title="Copy full SHA" aria-label="Copy full SHA">
                                 <svg viewBox="0 0 16 16" fill="currentColor" width="10" height="10">
                                     <path d="M0 6.75C0 5.784.784 5 1.75 5h1.5a.75.75 0 0 1 0 1.5h-1.5a.25.25 0 0 0-.25.25v7.5c0 .138.112.25.25.25h7.5a.25.25 0 0 0 .25-.25v-1.5a.75.75 0 0 1 1.5 0v1.5A1.75 1.75 0 0 1 9.25 16h-7.5A1.75 1.75 0 0 1 0 14.25Z"></path>
                                     <path d="M5 1.75C5 .784 5.784 0 6.75 0h7.5C15.216 0 16 .784 16 1.75v7.5A1.75 1.75 0 0 1 14.25 11h-7.5A1.75 1.75 0 0 1 5 9.25Zm1.75-.25a.25.25 0 0 0-.25.25v7.5c0 .138.112.25.25.25h7.5a.25.25 0 0 0 .25-.25v-7.5a.25.25 0 0 0-.25-.25Z"></path>