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

package/public/js/components/CouncilProgressModal.js

@@ -7,8 +7,7 @@
  *   - Voice participants as child rows under each level
  *   - Consolidation section at the bottom
  *
- * Replaces ProgressModal when a council analysis is running.
- * The existing ProgressModal remains for single-model analysis.
+ * Handles both council and single-model analysis modes.
  */
 class CouncilProgressModal {
   constructor() {
@@ -22,9 +21,6 @@ class CouncilProgressModal {
 
     // Track per-voice completion state
     this._voiceStates = {};
-    // Track SSE endpoint mode
-    this._useLocalEndpoint = false;
-    this._localReviewId = null;
 
     this._createModal();
     this._setupEventListeners();
@@ -132,7 +128,7 @@ class CouncilProgressModal {
     }
 
     try {
-      await fetch(`/api/analyze/cancel/${this.currentAnalysisId}`, { method: 'POST' });
+      await fetch(`/api/analyses/${this.currentAnalysisId}/cancel`, { method: 'POST' });
     } catch (error) {
       console.warn('Cancel not available on server:', error.message);
     }
@@ -149,20 +145,21 @@ class CouncilProgressModal {
   }
 
   /**
-   * Configure for local mode SSE endpoint.
-   * @param {string|number} reviewId
+   * Configure for local mode.
+   * Retained as a public method since callers invoke it, but the SSE endpoint
+   * is now unified so no per-mode state is needed.
    */
-  setLocalMode(reviewId) {
-    this._useLocalEndpoint = true;
-    this._localReviewId = reviewId;
+  setLocalMode(_reviewId) {
+    // no-op: SSE endpoint is unified
   }
 
   /**
-   * Configure for PR mode SSE endpoint (default).
+   * Configure for PR mode (default).
+   * Retained as a public method since callers invoke it, but the SSE endpoint
+   * is now unified so no per-mode state is needed.
    */
   setPRMode() {
-    this._useLocalEndpoint = false;
-    this._localReviewId = null;
+    // no-op: SSE endpoint is unified
   }
 
   // ---------------------------------------------------------------------------
@@ -175,9 +172,7 @@ class CouncilProgressModal {
     }
     if (!this.currentAnalysisId) return;
 
-    const sseUrl = this._useLocalEndpoint
-      ? `/api/local/${this._localReviewId}/ai-suggestions/status`
-      : `/api/pr/${this.currentAnalysisId}/ai-suggestions/status`;
+    const sseUrl = `/api/analyses/${this.currentAnalysisId}/progress`;
 
     this.eventSource = new EventSource(sseUrl);
 
@@ -229,7 +224,7 @@ class CouncilProgressModal {
     this.statusCheckInterval = setInterval(async () => {
       if (!this.currentAnalysisId) return;
       try {
-        const response = await fetch(`/api/analyze/status/${this.currentAnalysisId}`);
+        const response = await fetch(`/api/analyses/${this.currentAnalysisId}/status`);
         if (!response.ok) throw new Error('Failed to fetch status');
         const status = await response.json();
         this.updateProgress(status);
@@ -673,6 +668,17 @@ class CouncilProgressModal {
 
     // Derive the parent header state from children
     this._refreshConsolidationHeader(iconEl, statusEl, state);
+
+    // Show stream event text in the snippet element (mirrors _updateSingleModelLevel logic)
+    const snippetEl = section.querySelector('.council-level-snippet');
+    if (snippetEl) {
+      if (state === 'running' && level4Status.streamEvent?.text) {
+        snippetEl.textContent = level4Status.streamEvent.text;
+        snippetEl.style.display = 'block';
+      } else if (state !== 'running') {
+        snippetEl.style.display = 'none';
+      }
+    }
   }
 
   /**
@@ -1128,6 +1134,7 @@ class CouncilProgressModal {
           <span class="council-level-title">Cross-Reviewer Consolidation</span>
           <span class="council-level-status pending">Pending</span>
         </div>
+        <div class="council-level-snippet" style="display: none;"></div>
       </div>
     `;
 
@@ -1188,6 +1195,7 @@ class CouncilProgressModal {
           <span class="council-level-title">Consolidation</span>
           <span class="council-level-status pending">Pending</span>
         </div>
+        <div class="council-level-snippet" style="display: none;"></div>
       </div>
     `;
 
@@ -1327,6 +1335,7 @@ class CouncilProgressModal {
             <span class="council-level-title">Consolidation</span>
             <span class="council-level-status pending">Pending</span>
           </div>
+          <div class="council-level-snippet" style="display: none;"></div>
         </div>
       `;
     }
@@ -1338,6 +1347,7 @@ class CouncilProgressModal {
           <span class="council-level-title">Consolidation</span>
           <span class="council-level-status pending">Pending</span>
         </div>
+        <div class="council-level-snippet" style="display: none;"></div>
         <div class="council-level-children">
     `;
 

package/public/js/components/KeyboardShortcuts.js

@@ -142,6 +142,7 @@ class KeyboardShortcuts {
     const groups = {
       navigation: { title: 'Navigation', shortcuts: [] },
       comments: { title: 'Comments', shortcuts: [] },
+      panels: { title: 'Panels', shortcuts: [] },
       general: { title: 'General', shortcuts: [] }
     };
 
@@ -154,6 +155,8 @@ class KeyboardShortcuts {
         groups.navigation.shortcuts.push(item);
       } else if (keys[0] === 'c') {
         groups.comments.shortcuts.push(item);
+      } else if (keys[0] === 'p') {
+        groups.panels.shortcuts.push(item);
       } else {
         groups.general.shortcuts.push(item);
       }