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

package/public/js/components/CouncilProgressModal.js

@@ -21,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();
@@ -131,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);
     }
@@ -148,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
   }
 
   // ---------------------------------------------------------------------------
@@ -174,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);
 
@@ -228,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);

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);
       }