@in-the-loop-labs/pair-review 3.0.6 → 3.1.0

package/public/js/components/AnalysisConfigModal.js

@@ -275,6 +275,28 @@ class AnalysisConfigModal {
             </div>
           </section>
 
+          <!-- Exclude Previous Findings -->
+          <details class="config-section exclude-previous-section">
+            <summary class="section-title">
+              Exclude Previous Findings
+              <span class="section-hint">(optional)</span>
+            </summary>
+            <div class="exclude-previous-options">
+              <label class="remember-toggle">
+                <input type="checkbox" id="exclude-github-comments" />
+                <span class="toggle-switch"></span>
+                <span class="toggle-label">GitHub PR review comments</span>
+              </label>
+              <p class="option-hint">Skip issues already noted in inline PR review comments</p>
+              <label class="remember-toggle">
+                <input type="checkbox" id="exclude-pr-feedback" />
+                <span class="toggle-switch"></span>
+                <span class="toggle-label">Existing pair-review feedback</span>
+              </label>
+              <p class="option-hint">Skip issues from previous AI suggestions and reviewer comments</p>
+            </div>
+          </details>
+
           <!-- Focus Presets - Hidden for now, may reintroduce later -->
           <section class="config-section" style="display: none;">
             <h4 class="section-title">
@@ -499,7 +521,7 @@ class AnalysisConfigModal {
     container.innerHTML = this.models.map(model => `
       <button class="model-card ${model.id === this.selectedModel ? 'selected' : ''}" data-model="${model.id}" data-tier="${model.tier}">
         <div class="model-badge ${model.badgeClass || ''}">${model.badge || ''}</div>
-        <div class="model-icon">${this.getModelIcon(model.tier)}</div>
+        <div class="model-icon">${model.icon || this.getModelIcon(model.tier)}</div>
         <div class="model-info">
           <span class="model-name">${model.name}</span>
           <span class="model-tagline">${model.tagline || ''}</span>
@@ -549,6 +571,47 @@ class AnalysisConfigModal {
     // Re-render model cards (handles its own event listeners)
     this.renderModelCards();
 
+    // Toggle level toggles visibility based on provider's capabilities
+    const provider = this.providers[providerId];
+    const supportsLevels = provider.capabilities?.review_levels !== false;
+    const supportsCustomInstructions = provider.capabilities?.custom_instructions !== false;
+    const levelToggles = this.modal.querySelector('.single-level-toggles');
+    const skipLevel3Info = this.modal.querySelector('#skip-level3-info');
+    const existingLevelsNote = this.modal.querySelector('.executable-provider-levels-note');
+    const customInstructionsSection = this.modal.querySelector('#custom-instructions')?.closest('.config-section');
+
+    if (!supportsLevels) {
+      if (levelToggles) levelToggles.style.display = 'none';
+      if (skipLevel3Info) skipLevel3Info.style.display = 'none';
+      if (!existingLevelsNote) {
+        const note = document.createElement('div');
+        note.className = 'executable-provider-note executable-provider-levels-note';
+        note.textContent = 'This provider runs its own analysis pipeline';
+        levelToggles?.parentNode?.appendChild(note);
+      }
+    } else {
+      if (levelToggles) levelToggles.style.display = '';
+      if (skipLevel3Info) skipLevel3Info.style.display = '';
+      if (existingLevelsNote) existingLevelsNote.remove();
+    }
+
+    // Toggle custom instructions visibility based on provider's capabilities
+    const instructionsContainer = customInstructionsSection?.querySelector('.instructions-container');
+    const existingInstructionsNote = customInstructionsSection?.querySelector('.executable-provider-instructions-note');
+
+    if (!supportsCustomInstructions) {
+      if (instructionsContainer) instructionsContainer.style.display = 'none';
+      if (!existingInstructionsNote && customInstructionsSection) {
+        const note = document.createElement('div');
+        note.className = 'executable-provider-note executable-provider-instructions-note';
+        note.textContent = 'This provider runs its own analysis pipeline and does not accept custom instructions.';
+        customInstructionsSection.appendChild(note);
+      }
+    } else {
+      if (instructionsContainer) instructionsContainer.style.display = '';
+      if (existingInstructionsNote) existingInstructionsNote.remove();
+    }
+
     // Update selection state for the selected model
     if (this.selectedModel) {
       this.selectModel(this.selectedModel);
@@ -711,7 +774,8 @@ class AnalysisConfigModal {
         councilName: selectedCouncil?.name || null,
         councilConfig: councilConfig,
         customInstructions: this.modal.querySelector('#vc-custom-instructions')?.value?.trim() || '',
-        repoInstructions: this.repoInstructions
+        repoInstructions: this.repoInstructions,
+        excludePrevious: this._getAndSaveExcludePrevious()
       };
 
       if (this.onSubmit) this.onSubmit(config);
@@ -736,7 +800,8 @@ class AnalysisConfigModal {
         councilName: selectedCouncil?.name || null,
         councilConfig: councilConfig,
         customInstructions: this.modal.querySelector('#council-custom-instructions')?.value?.trim() || '',
-        repoInstructions: this.repoInstructions
+        repoInstructions: this.repoInstructions,
+        excludePrevious: this._getAndSaveExcludePrevious()
       };
 
       if (this.onSubmit) this.onSubmit(config);
@@ -748,16 +813,21 @@ class AnalysisConfigModal {
     const selectedModelCard = this.modal.querySelector('.model-card.selected');
     const tier = selectedModelCard?.dataset?.tier || 'balanced';
 
+    const selectedProvider = this.providers[this.selectedProvider];
+    const noLevels = selectedProvider?.capabilities?.review_levels === false;
+
     const config = {
       provider: this.selectedProvider,
       model: this.selectedModel,
       tier: tier,
       instructions: this.buildInstructions(),
-      customInstructions: this.modal.querySelector('#custom-instructions')?.value?.trim() || '',
+      customInstructions: selectedProvider?.capabilities?.custom_instructions === false ? '' : (this.modal.querySelector('#custom-instructions')?.value?.trim() || ''),
       presets: Array.from(this.selectedPresets),
       repoInstructions: this.repoInstructions,
       enabledLevels: [...this.enabledLevels],
-      skipLevel3: !this.enabledLevels.includes(3)
+      skipLevel3: !this.enabledLevels.includes(3),
+      noLevels,
+      excludePrevious: this._getAndSaveExcludePrevious()
     };
 
     if (this.onSubmit) this.onSubmit(config);
@@ -895,6 +965,9 @@ class AnalysisConfigModal {
       }
     }
 
+    // Restore exclude-previous checkbox state from localStorage
+    this._restoreExcludePrevious(options);
+
     // Remove loading state and reveal content
     this._showLoading(false);
 
@@ -953,6 +1026,10 @@ class AnalysisConfigModal {
         <button class="analysis-tab" data-tab="advanced">Advanced</button>
       `;
 
+      // Hoist the exclude-previous section out of the single-model panel
+      // so it remains visible across all tabs.
+      const excludePreviousSection = singlePanel.querySelector('.exclude-previous-section');
+
       // Assemble
       modalBody.innerHTML = '';
       modalBody.appendChild(tabBar);
@@ -960,6 +1037,12 @@ class AnalysisConfigModal {
       modalBody.appendChild(councilPanel);
       modalBody.appendChild(advancedPanel);
 
+      // Place the exclude-previous section after all tab panels,
+      // just before the modal footer, so it's visible on every tab.
+      if (excludePreviousSection) {
+        modalBody.appendChild(excludePreviousSection);
+      }
+
       // Tab click listeners
       tabBar.querySelectorAll('.analysis-tab').forEach(tab => {
         tab.addEventListener('click', () => {
@@ -1212,6 +1295,73 @@ class AnalysisConfigModal {
     }, 200);
   }
 
+  /**
+   * Restore exclude-previous checkbox state from localStorage.
+   * Disables the GitHub checkbox when there is no PR or no GitHub token.
+   * @param {Object} options - show() options
+   * @private
+   */
+  _restoreExcludePrevious(options) {
+    const githubCb = this.modal.querySelector('#exclude-github-comments');
+    const feedbackCb = this.modal.querySelector('#exclude-pr-feedback');
+    if (!githubCb || !feedbackCb) return;
+
+    // Restore saved state
+    let saved = { github: false, feedback: false };
+    try {
+      const raw = localStorage.getItem('pair-review-exclude-previous');
+      if (raw) saved = JSON.parse(raw);
+    } catch (_) { /* ignore parse errors */ }
+
+    githubCb.checked = !!saved.github;
+    feedbackCb.checked = !!saved.feedback;
+
+    // Disable the GitHub checkbox when it cannot be used
+    const canUseGithub = options.hasPr !== false && options.hasGithubToken !== false;
+    githubCb.disabled = !canUseGithub;
+    if (!canUseGithub) {
+      const label = githubCb.closest('.remember-toggle');
+      if (label) label.title = 'Not available — requires a PR with a GitHub token';
+    } else {
+      const label = githubCb.closest('.remember-toggle');
+      if (label) label.title = '';
+    }
+  }
+
+  /**
+   * Read current exclude-previous checkbox state, save to localStorage,
+   * and return the config object.
+   * @returns {{ github: boolean, feedback: boolean }}
+   * @private
+   */
+  _getAndSaveExcludePrevious() {
+    const githubCb = this.modal.querySelector('#exclude-github-comments');
+    const feedbackCb = this.modal.querySelector('#exclude-pr-feedback');
+
+    // When the GitHub checkbox is disabled, preserve the existing localStorage
+    // value for `github` so the user's preference survives contexts where the
+    // option is unavailable.
+    let preservedGithub = false;
+    if (githubCb?.disabled) {
+      try {
+        const raw = localStorage.getItem('pair-review-exclude-previous');
+        if (raw) {
+          const prev = JSON.parse(raw);
+          preservedGithub = !!prev.github;
+        }
+      } catch (_) { /* ignore parse errors */ }
+    }
+
+    const state = {
+      github: githubCb?.disabled ? preservedGithub : !!githubCb?.checked,
+      feedback: !!feedbackCb?.checked
+    };
+    try {
+      localStorage.setItem('pair-review-exclude-previous', JSON.stringify(state));
+    } catch (_) { /* ignore storage errors */ }
+    return state;
+  }
+
   /**
    * Cleanup event listeners and pending timeouts
    */

package/public/js/components/ChatPanel.js

@@ -43,6 +43,7 @@ class ChatPanel {
     this._sessionWarm = false; // true once the session has been used in this page load
     this._activeProvider = window.__pairReview?.chatProvider || 'pi';
     this._chatProviders = window.__pairReview?.chatProviders || [];
+    this._enterToSend = window.__pairReview?.chatEnterToSend ?? true;
 
     this._render();
     this._bindEvents();
@@ -141,7 +142,7 @@ class ChatPanel {
         <div class="chat-panel__input-area">
           <textarea class="chat-panel__input" placeholder="Ask about this review..." rows="1"></textarea>
           <div class="chat-panel__input-footer">
-            <span class="chat-panel__input-hint">${typeof navigator !== 'undefined' && navigator.platform?.includes('Mac') ? '\u2318' : 'Ctrl'}+Enter to send</span>
+            <span class="chat-panel__input-hint" title="Configure with chat.enter_to_send">${this._enterToSend ? 'Enter to send, Shift+Enter for newline' : `${typeof navigator !== 'undefined' && navigator.platform?.includes('Mac') ? '\u2318' : 'Ctrl'}+Enter to send`}</span>
             <div class="chat-panel__input-actions">
               <button class="chat-panel__send-btn" title="Send" disabled>
                 <svg viewBox="0 0 16 16" fill="currentColor" width="14" height="14">
@@ -250,10 +251,26 @@ class ChatPanel {
 
     // Keyboard shortcuts
     this.inputEl.addEventListener('keydown', (e) => {
-      if (e.key === 'Enter' && (e.metaKey || e.ctrlKey)) {
-        e.preventDefault();
-        if (this.inputEl.value.trim() && !this.isStreaming) {
-          this.sendMessage();
+      if (e.key === 'Enter') {
+        // Ignore Enter during IME composition (e.g. CJK input) so the
+        // composition-confirming keystroke is not swallowed.
+        if (e.isComposing) return;
+
+        if (this._enterToSend) {
+          // Enter sends, Shift+Enter inserts newline
+          if (e.shiftKey) return; // let browser insert newline
+          e.preventDefault();
+          if (this.inputEl.value.trim() && !this.isStreaming) {
+            this.sendMessage();
+          }
+        } else {
+          // Cmd+Enter / Ctrl+Enter sends, plain Enter inserts newline
+          if (e.metaKey || e.ctrlKey) {
+            e.preventDefault();
+            if (this.inputEl.value.trim() && !this.isStreaming) {
+              this.sendMessage();
+            }
+          }
         }
       }
     });

package/public/js/components/CouncilProgressModal.js

@@ -23,6 +23,11 @@ class CouncilProgressModal {
     // Track per-voice completion state
     this._voiceStates = {};
 
+    // Track which voice keys are executable providers.
+    // Populated from progress events (levels.exec.voices) and persisted
+    // across show()/hide() cycles so reopened modals render correctly.
+    this._executableVoices = new Set();
+
     this._createModal();
     this._setupEventListeners();
   }
@@ -54,9 +59,12 @@ class CouncilProgressModal {
     // Detect rendering mode
     const configType = options.configType || (councilConfig ? 'advanced' : 'single');
     this._renderMode = configType;
+    this._noLevels = options.noLevels || false;
 
     // Rebuild DOM based on mode
-    if (configType === 'single') {
+    if (configType === 'single' && this._noLevels) {
+      this._rebuildBodyNoLevels();
+    } else if (configType === 'single') {
       const enabledLevels = options.enabledLevels || [1, 2, 3];
       this._rebuildBodySingleModel(enabledLevels);
     } else if (configType === 'council') {
@@ -239,11 +247,34 @@ class CouncilProgressModal {
    * @param {Object} status
    */
   updateProgress(status) {
+    // No-levels mode: update the single row based on overall status
+    if (this._noLevels) {
+      this._updateNoLevelsProgress(status);
+      // Terminal states
+      if (status.status === 'completed') {
+        this._handleCompletion(status);
+      } else if (status.status === 'failed') {
+        this._handleFailure(status);
+      } else if (status.status === 'cancelled') {
+        this._handleCancellation(status);
+      }
+      return;
+    }
+
     if (!status.levels || typeof status.levels !== 'object') {
       console.warn('Council progress: invalid status structure', status);
       return;
     }
 
+    // Track executable voice keys from exec level data so that reopened
+    // modals can render executable reviewers correctly without relying on
+    // the config modal's provider cache (which may be empty on reopen).
+    if (status.levels.exec?.voices) {
+      for (const voiceKey of Object.keys(status.levels.exec.voices)) {
+        this._executableVoices.add(voiceKey);
+      }
+    }
+
     if (this._renderMode === 'single') {
       // Single-model: update level headers directly
       for (let level = 1; level <= 3; level++) {
@@ -292,6 +323,63 @@ class CouncilProgressModal {
     }
   }
 
+  // ---------------------------------------------------------------------------
+  // No-levels progress
+  // ---------------------------------------------------------------------------
+
+  /**
+   * Update the single row for no-levels providers.
+   * Maps overall analysis status to the single "Running analysis..." row.
+   */
+  _updateNoLevelsProgress(status) {
+    const header = this.modal.querySelector('.council-level-header[data-level="analysis"]');
+    if (!header) return;
+
+    const iconEl = header.querySelector('.council-level-icon');
+    const statusEl = header.querySelector('.council-level-status');
+    if (!iconEl || !statusEl) return;
+
+    // Derive state from overall status or from any level status
+    let state = 'pending';
+    if (status.status === 'running' || status.status === 'in_progress') {
+      state = 'running';
+    } else if (status.status === 'completed') {
+      state = 'completed';
+    } else if (status.status === 'failed') {
+      state = 'failed';
+    } else if (status.status === 'cancelled') {
+      state = 'cancelled';
+    } else if (status.levels) {
+      // Fall back to checking individual levels
+      const levelValues = Object.values(status.levels);
+      if (levelValues.some(l => l.status === 'running')) state = 'running';
+      else if (levelValues.some(l => l.status === 'completed')) state = 'running';
+    }
+
+    this._renderState(iconEl, statusEl, state, 'council-level');
+
+    // Show stream event text in the snippet element
+    const levelEl = header.closest('.council-level');
+    const snippetEl = levelEl?.querySelector('.council-level-snippet');
+    if (snippetEl) {
+      // Check for stream events in any level
+      let streamText = null;
+      if (status.levels) {
+        for (const lvl of Object.values(status.levels)) {
+          if (lvl.streamEvent?.text) {
+            streamText = lvl.streamEvent.text;
+          }
+        }
+      }
+      if (state === 'running' && streamText) {
+        snippetEl.textContent = streamText;
+        snippetEl.style.display = 'block';
+      } else if (state !== 'running') {
+        snippetEl.style.display = 'none';
+      }
+    }
+  }
+
   // ---------------------------------------------------------------------------
   // Per-voice progress
   // ---------------------------------------------------------------------------
@@ -518,6 +606,55 @@ class CouncilProgressModal {
       }
     }
 
+    // Handle executable voice updates (level 'exec'):
+    // Executable voices emit progress with level: 'exec' instead of numeric 1/2/3.
+    // The single "Running analysis..." row uses data-vc-level="exec".
+    const execLevel = status.levels?.exec;
+    if (execLevel) {
+      if (execLevel.voices) {
+        // Check if any exec voice was rendered with native L1/L2/L3 structure
+        // instead of the single exec row. This happens when the modal was opened
+        // before _executableVoices was populated. If so, re-render the full body.
+        let needsRerender = false;
+        for (const voiceId of Object.keys(execLevel.voices)) {
+          const execEl = this.modal.querySelector(`[data-vc-voice="${voiceId}"][data-vc-level="exec"]`);
+          if (!execEl) {
+            // Voice exists in exec progress but has no exec DOM element — was likely
+            // rendered with native structure. Re-render to fix.
+            needsRerender = true;
+            break;
+          }
+        }
+        if (needsRerender && this.councilConfig) {
+          this._rebuildBodyVoiceCentric(this.councilConfig);
+          // Re-apply all progress to the fresh DOM
+          this._updateVoiceCentric(status);
+          return;
+        }
+
+        for (const [voiceId, vStatus] of Object.entries(execLevel.voices)) {
+          this._setVoiceCentricLevelState(voiceId, 'exec', vStatus.status || 'running', vStatus);
+        }
+      } else if (execLevel.status && execLevel.status !== 'pending') {
+        // Aggregate terminal update without per-voice breakdown (completion/cancellation
+        // paths may collapse exec progress to just { status, progress }). Propagate the
+        // status to every rendered exec row, mirroring the pattern used for numeric levels
+        // with shared terminal state in _updateVoiceFromLevelStatus.
+        const terminalState = execLevel.status;
+        const execEls = this.modal.querySelectorAll('[data-vc-level="exec"]');
+        execEls.forEach(el => {
+          const voiceKey = el.dataset.vcVoice;
+          const stateKey = `${voiceKey}:exec`;
+          if (this._voiceStates[stateKey] !== 'completed' && this._voiceStates[stateKey] !== 'failed') {
+            this._setVoiceCentricLevelState(voiceKey, 'exec', terminalState, execLevel);
+          }
+        });
+      }
+      if (execLevel.streamEvent?.text && execLevel.voiceId) {
+        this._setVoiceCentricStreamText(execLevel.voiceId, 'exec', execLevel.streamEvent.text);
+      }
+    }
+
     // Handle per-voice orchestration updates (level 4):
     // In voice-centric mode, each reviewer has a consolidation child at data-vc-level="4".
     // The backend tracks per-voice orchestration state in levels[4].voices, including
@@ -1073,6 +1210,26 @@ class CouncilProgressModal {
       }
     }
 
+    // Executable voices don't require enabled levels — add any not already captured
+    // (handles all-executable councils where no levels are enabled).
+    // Use _executableVoices (populated from progress events) as primary source,
+    // falling back to the config modal's provider cache. This ensures executable
+    // voices render correctly even when the modal is reopened for an already-running
+    // analysis where the config modal cache may be empty.
+    if (config.voices) {
+      const providersInfo = window.analysisConfigModal?.providers || {};
+      for (const voice of config.voices) {
+        const isExec = providersInfo[voice.provider]?.isExecutable || false;
+        if (isExec) {
+          const sig = `${voice.provider}|${voice.model}|${voice.tier || 'balanced'}|${voice.customInstructions || ''}`;
+          if (!seenSignatures.has(sig)) {
+            seenSignatures.add(sig);
+            uniqueVoices.push(voice);
+          }
+        }
+      }
+    }
+
     // Build voiceMap: voiceKey -> { voice, levels } using deduplicated array indices
     const voiceMap = new Map();
     uniqueVoices.forEach((voice, idx) => {
@@ -1080,11 +1237,33 @@ class CouncilProgressModal {
       voiceMap.set(voiceKey, { voice, levels: enabledLevelNums });
     });
 
+    // Also add any executable voices known from progress events but not yet
+    // in the voiceMap (covers the case where config modal providers cache is
+    // empty but we've already seen exec-level progress data).
+    for (const execVoiceKey of this._executableVoices) {
+      if (!voiceMap.has(execVoiceKey)) {
+        // Find the matching voice in config.voices by key pattern
+        const configVoices = config.voices || [];
+        const matchingVoice = configVoices.find((v, idx) => {
+          const candidateKey = `${v.provider}-${v.model}${idx > 0 ? `-${idx}` : ''}`;
+          return candidateKey === execVoiceKey;
+        });
+        if (matchingVoice) {
+          voiceMap.set(execVoiceKey, { voice: matchingVoice, levels: enabledLevelNums });
+        }
+      }
+    }
+
     let html = '<div class="council-progress-tree">';
 
-    // Build a parent row for each unique voice
+    // Build a parent row for each unique voice.
+    // Determine executable status from: (1) _executableVoices set (populated from
+    // progress events), or (2) config modal provider cache as fallback.
+    const providersMap = window.analysisConfigModal?.providers || {};
+
     for (const [voiceKey, { voice, levels }] of voiceMap) {
-      const label = this._formatVoiceLabel(voice);
+      const isExecutable = this._executableVoices.has(voiceKey) || (providersMap[voice.provider]?.isExecutable || false);
+      const label = this._formatVoiceLabel(voice, { isExecutable });
 
       html += `
         <div class="council-level" data-voice-key="${voiceKey}">
@@ -1096,31 +1275,46 @@ class CouncilProgressModal {
           <div class="council-level-children">
       `;
 
-      // Level children (orchestration row is always last, added separately below)
-      levels.forEach((levelNum) => {
-        const connectorClass = 'connector-mid';
+      if (isExecutable) {
+        // Executable voices: single "Running analysis..." row instead of L1/L2/L3
         html += `
-          <div class="council-voice ${connectorClass}" data-vc-voice="${voiceKey}" data-vc-level="${levelNum}">
-            <span class="council-voice-connector ${connectorClass}"></span>
-            <span class="council-voice-icon running"><span class="council-spinner"></span></span>
-            <span class="council-voice-label">Level ${levelNum} \u2014 ${levelNames[levelNum]}</span>
-            <span class="council-voice-status running">Running...</span>
+          <div class="council-voice connector-last" data-vc-voice="${voiceKey}" data-vc-level="exec">
+            <span class="council-voice-connector connector-last"></span>
+            <span class="council-voice-icon pending">\u25CB</span>
+            <span class="council-voice-label">Running analysis\u2026</span>
+            <span class="council-voice-status pending">Pending</span>
             <div class="council-voice-detail">
               <div class="council-voice-snippet" style="display: none;"></div>
             </div>
           </div>
         `;
-      });
+      } else {
+        // Native voices: L1/L2/L3 children + orchestration
+        levels.forEach((levelNum) => {
+          const connectorClass = 'connector-mid';
+          html += `
+            <div class="council-voice ${connectorClass}" data-vc-voice="${voiceKey}" data-vc-level="${levelNum}">
+              <span class="council-voice-connector ${connectorClass}"></span>
+              <span class="council-voice-icon running"><span class="council-spinner"></span></span>
+              <span class="council-voice-label">Level ${levelNum} \u2014 ${levelNames[levelNum]}</span>
+              <span class="council-voice-status running">Running...</span>
+              <div class="council-voice-detail">
+                <div class="council-voice-snippet" style="display: none;"></div>
+              </div>
+            </div>
+          `;
+        });
 
-      // Orchestration child (always last)
-      html += `
-          <div class="council-voice connector-last" data-vc-voice="${voiceKey}" data-vc-level="4">
-            <span class="council-voice-connector connector-last"></span>
-            <span class="council-voice-icon pending">\u25CB</span>
-            <span class="council-voice-label">Consolidation</span>
-            <span class="council-voice-status pending">Pending</span>
-          </div>
-      `;
+        // Orchestration child (always last)
+        html += `
+            <div class="council-voice connector-last" data-vc-voice="${voiceKey}" data-vc-level="4">
+              <span class="council-voice-connector connector-last"></span>
+              <span class="council-voice-icon pending">\u25CB</span>
+              <span class="council-voice-label">Consolidation</span>
+              <span class="council-voice-status pending">Pending</span>
+            </div>
+        `;
+      }
 
       html += `
           </div>
@@ -1205,6 +1399,28 @@ class CouncilProgressModal {
     body.innerHTML = html;
   }
 
+  /**
+   * Rebuild the modal body for no-levels providers (e.g., executable providers).
+   * Shows a single "Running analysis..." entry instead of the L1/L2/L3 breakdown.
+   */
+  _rebuildBodyNoLevels() {
+    const body = this.modal.querySelector('.council-progress-body');
+    if (!body) return;
+
+    body.innerHTML = `
+      <div class="council-progress-tree">
+        <div class="council-level" data-level="analysis">
+          <div class="council-level-header" data-level="analysis">
+            <span class="council-level-icon pending">\u25CB</span>
+            <span class="council-level-title">Running analysis\u2026</span>
+            <span class="council-level-status pending">Pending</span>
+          </div>
+          <div class="council-level-snippet" style="display: none;"></div>
+        </div>
+      </div>
+    `;
+  }
+
   _resetFooter() {
     const bgBtn = this.modal.querySelector('.council-bg-btn');
     const cancelBtn = this.modal.querySelector('.council-cancel-btn');
@@ -1404,9 +1620,10 @@ class CouncilProgressModal {
    * Example: { provider: 'claude', model: 'sonnet-4-5', tier: 'balanced' }
    *       -> "Claude sonnet-4-5 (Balanced)"
    */
-  _formatVoiceLabel(voice) {
+  _formatVoiceLabel(voice, { isExecutable = false } = {}) {
     const provider = this._capitalize(voice.provider || 'unknown');
     const model = voice.model || 'default';
+    if (isExecutable) return `${provider} ${model}`;
     const tier = this._capitalize(voice.tier || 'balanced');
     return `${provider} ${model} (${tier})`;
   }

package/public/js/components/TimeoutSelect.js

@@ -37,6 +37,8 @@ class TimeoutSelect {
     { value: '600000', label: '10m', selected: true },
     { value: '900000', label: '15m' },
     { value: '1800000', label: '30m' },
+    { value: '2700000', label: '45m' },
+    { value: '3600000', label: '60m' },
   ];
 
   /**