@yemi33/minions 0.1.279 → 0.1.281

package/CHANGELOG.md

@@ -1,6 +1,14 @@
 # Changelog
 
-## 0.1.279 (2026-04-03)
+## 0.1.281 (2026-04-03)
+
+### Other
+- refactor: remove duplicate thinking indicator — single source via updateStreamDiv
+
+## 0.1.280 (2026-04-03)
+
+### Features
+-  thinking indicator shows progressive phases with elapsed timer
 
 ### Fixes
 -  doc chat elapsed time no longer overlaps copy button

package/dashboard/js/command-center.js

@@ -153,17 +153,6 @@ async function _ccDoSend(message, skipUserMsg) {
   var existingQueueEl = document.getElementById('cc-queue-indicator');
   if (existingQueueEl) existingQueueEl.remove();
 
-  // Show thinking indicator with timer + queue count
-  const queueCount = _ccQueue.length;
-  const queueBadge = queueCount > 0 ? ' <span style="font-size:9px;background:var(--surface);padding:1px 5px;border-radius:8px;border:1px solid var(--border)">+' + queueCount + ' queued</span>' : '';
-  const thinking = document.createElement('div');
-  thinking.id = 'cc-thinking';
-  thinking.style.cssText = 'padding:8px 12px;border-radius:8px;font-size:11px;color:var(--muted);align-self:flex-start;display:flex;align-items:center;gap:8px';
-  thinking.innerHTML = '<span class="dot-pulse" style="display:inline-flex;gap:3px"><span style="width:4px;height:4px;background:var(--blue);border-radius:50%;animation:dotPulse 1.2s infinite"></span><span style="width:4px;height:4px;background:var(--blue);border-radius:50%;animation:dotPulse 1.2s infinite;animation-delay:0.2s"></span><span style="width:4px;height:4px;background:var(--blue);border-radius:50%;animation:dotPulse 1.2s infinite;animation-delay:0.4s"></span></span> <span id="cc-thinking-text">Thinking...</span> <span id="cc-thinking-time" style="font-size:10px;color:var(--border)"></span>' + queueBadge;
-  document.getElementById('cc-messages').appendChild(thinking);
-  const ccMsgs = document.getElementById('cc-messages');
-  ccMsgs.scrollTop = ccMsgs.scrollHeight;
-
   const ccStartTime = Date.now();
   const phases = [
     [0, 'Thinking...'],
@@ -175,18 +164,6 @@ async function _ccDoSend(message, skipUserMsg) {
     [180000, 'Still going (this is unusually long)...'],
     [300000, 'Timing out soon...'],
   ];
-  const ccTimer = setInterval(() => {
-    const elapsed = Date.now() - ccStartTime;
-    const secs = Math.floor(elapsed / 1000);
-    const timeEl = document.getElementById('cc-thinking-time');
-    const textEl = document.getElementById('cc-thinking-text');
-    if (timeEl) timeEl.textContent = secs + 's';
-    if (textEl) {
-      for (let i = phases.length - 1; i >= 0; i--) {
-        if (elapsed >= phases[i][0]) { textEl.textContent = phases[i][1]; break; }
-      }
-    }
-  }, 500);
 
   try {
     // Stream response via SSE — shows text as it arrives
@@ -197,8 +174,6 @@ async function _ccDoSend(message, skipUserMsg) {
     });
 
     if (!res.ok) {
-      clearInterval(ccTimer);
-      thinking.remove();
       const errText = await res.text();
       ccAddMessage('assistant', '<span style="color:var(--red)">' + escHtml(errText || 'CC error') + '</span>' +
         (errText.includes('busy') ? ' <button onclick="ccNewSession()" style="margin-top:4px;padding:3px 10px;background:var(--surface2);border:1px solid var(--border);border-radius:4px;color:var(--blue);cursor:pointer;font-size:10px">Reset CC</button>' : ''));
@@ -206,8 +181,6 @@ async function _ccDoSend(message, skipUserMsg) {
     }
 
     // Use a temporary streaming div for live updates, then replace with ccAddMessage on completion
-    clearInterval(ccTimer);
-    try { thinking.remove(); } catch { /* may already be removed */ }
     // Add a temporary placeholder via ccAddMessage so it gets proper styling
     ccAddMessage('assistant', '<span style="color:var(--muted);font-size:11px">Thinking...</span>', true);
     const msgs = document.getElementById('cc-messages');
@@ -215,6 +188,15 @@ async function _ccDoSend(message, skipUserMsg) {
     let streamedText = '';
     let toolsUsed = [];
     const dotPulse = '<span style="display:inline-flex;gap:3px;margin-left:6px;vertical-align:middle"><span style="width:4px;height:4px;background:var(--blue);border-radius:50%;animation:dotPulse 1.2s infinite"></span><span style="width:4px;height:4px;background:var(--blue);border-radius:50%;animation:dotPulse 1.2s infinite;animation-delay:0.2s"></span><span style="width:4px;height:4px;background:var(--blue);border-radius:50%;animation:dotPulse 1.2s infinite;animation-delay:0.4s"></span></span>';
+    function _getThinkingPhase() {
+      var elapsed = Date.now() - ccStartTime;
+      var label = 'Thinking...';
+      for (var pi = phases.length - 1; pi >= 0; pi--) {
+        if (elapsed >= phases[pi][0]) { label = phases[pi][1]; break; }
+      }
+      var secs = Math.floor(elapsed / 1000);
+      return label + ' <span style="font-size:9px;color:var(--border)">' + secs + 's</span>';
+    }
     function updateStreamDiv() {
       var html = '';
       if (toolsUsed.length > 0) {
@@ -227,13 +209,16 @@ async function _ccDoSend(message, skipUserMsg) {
       if (streamedText) {
         html += renderMd(streamedText);
       }
-      html += '<div style="margin-top:' + (streamedText ? '6px' : '0') + '"><span style="color:var(--muted);font-size:11px">Thinking...' + dotPulse + '</span></div>';
+      html += '<div style="margin-top:' + (streamedText ? '6px' : '0') + '"><span style="color:var(--muted);font-size:11px">' + _getThinkingPhase() + dotPulse + '</span></div>';
       streamDiv.innerHTML = html;
       // Re-append queue indicators so they stay below the streaming content
       if (_ccQueue.length > 0) _renderQueueIndicator();
       if (msgs.scrollHeight - msgs.scrollTop - msgs.clientHeight < 150) msgs.scrollTop = msgs.scrollHeight;
     }
 
+    // Periodically update thinking phase text + timer
+    const phaseTimer = setInterval(updateStreamDiv, 1000);
+
     const reader = res.body.getReader();
     const decoder = new TextDecoder();
     let buf = '';
@@ -257,7 +242,7 @@ async function _ccDoSend(message, skipUserMsg) {
             if (msgs.scrollHeight - msgs.scrollTop - msgs.clientHeight < 150) msgs.scrollTop = msgs.scrollHeight;
           } else if (evt.type === 'done') {
             // Replace streaming div with a proper ccAddMessage
-            streamDiv.remove();
+            clearInterval(phaseTimer); streamDiv.remove();
             // placeholder was added with skipSave=true — nothing to pop
             const ccElapsed = Math.round((Date.now() - ccStartTime) / 1000);
             const rendered = renderMd(evt.text || streamedText || '');
@@ -267,7 +252,7 @@ async function _ccDoSend(message, skipUserMsg) {
               for (const action of evt.actions) { await ccExecuteAction(action); }
             }
           } else if (evt.type === 'error') {
-            streamDiv.remove();
+            clearInterval(phaseTimer); streamDiv.remove();
             // placeholder was skipSave — no pop needed
             ccAddMessage('assistant', '<span style="color:var(--red)">' + escHtml(evt.error) + '</span>');
           }
@@ -281,7 +266,7 @@ async function _ccDoSend(message, skipUserMsg) {
         try {
           const evt = JSON.parse(line.slice(6));
           if (evt.type === 'done') {
-            streamDiv.remove();
+            clearInterval(phaseTimer); streamDiv.remove();
             // placeholder was skipSave — no pop needed
             const ccElapsed = Math.round((Date.now() - ccStartTime) / 1000);
             const rendered = renderMd(evt.text || streamedText || '');
@@ -299,21 +284,20 @@ async function _ccDoSend(message, skipUserMsg) {
     }
     // If stream ended without a 'done' event, finalize with whatever we have
     if (streamDiv.parentNode) {
-      streamDiv.remove();
+      clearInterval(phaseTimer); streamDiv.remove();
       if (streamedText) {
         const ccElapsed = Math.round((Date.now() - ccStartTime) / 1000);
         ccAddMessage('assistant', renderMd(streamedText) + '<div style="font-size:9px;color:var(--muted);margin-top:6px;display:flex;justify-content:flex-end;padding-right:30px">' + ccElapsed + 's</div>');
       }
     }
   } catch (e) {
-    clearInterval(ccTimer);
-    try { thinking.remove(); } catch { /* may already be removed */ }
     const retryId = 'cc-retry-' + Date.now();
     ccAddMessage('assistant', '<span style="color:var(--red)">Error: ' + escHtml(e.message) + '</span>' +
       '<button id="' + retryId + '" onclick="ccRetryLast()" style="margin-top:6px;padding:4px 12px;background:var(--surface2);border:1px solid var(--border);border-radius:4px;color:var(--blue);cursor:pointer;font-size:11px">Retry</button>');
   } finally {
     _ccSending = false;
     _ccAbortController = null;
+    try { clearInterval(phaseTimer); } catch { /* may not be defined if error before reader */ }
     try { localStorage.removeItem('cc-sending'); } catch {}
     // Show notification badge on CC button if drawer is closed
     if (!_ccOpen) showNotifBadge(document.getElementById('cc-toggle-btn'));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.279",
+  "version": "0.1.281",
   "description": "Multi-agent AI dev team that runs from ~/.minions/ — five autonomous agents share a single engine, dashboard, and knowledge base",
   "bin": {
     "minions": "bin/minions.js"