@yemi33/minions 0.1.1936 → 0.1.1938

package/README.md

@@ -676,8 +676,6 @@ To move to a new machine: `npm install -g @yemi33/minions && minions init --forc
     github.js            issues.js
     ado.js               ado-token.js          ado-mcp-wrapper.js
     ado-status.js        check-status.js
-    # Notifications
-    teams.js             teams-cards.js
     # Runtime state (generated, gitignored)
     control.json         <- running/paused/stopped
     dispatch.json        <- pending/active/completed queue

package/dashboard/js/command-center.js

@@ -397,13 +397,7 @@ function ccSwitchTab(id) {
       var html = '';
       var tools = tab._toolsUsed || [];
       if (tools.length > 0) {
-        html += '<div style="margin-bottom:6px">';
-        tools.forEach(function(t) {
-          var name = typeof t === 'string' ? t : t.name;
-          var input = typeof t === 'string' ? {} : (t.input || {});
-          html += '<div style="color:var(--muted);font-size:10px;font-family:monospace"><span style="flex-shrink:0">&#9679;</span> ' + formatToolSummary(name, input) + '</div>';
-        });
-        html += '</div>';
+        html += '<div style="margin-bottom:6px">' + tools.map(renderToolChip).join('') + '</div>';
       }
       var text = tab._streamedText || '';
       if (text) html += renderMd(text);
@@ -779,13 +773,7 @@ async function _ccDoSend(message, skipUserMsg, forceTabId, intentMetadata) {
       }
       var html = '';
       if (toolsUsed.length > 0) {
-        html += '<div style="margin-bottom:6px">';
-        toolsUsed.forEach(function(t) {
-          var name = typeof t === 'string' ? t : t.name;
-          var input = typeof t === 'string' ? {} : (t.input || {});
-          html += '<div style="color:var(--muted);font-size:10px;font-family:monospace"><span style="flex-shrink:0">&#9679;</span> ' + formatToolSummary(name, input) + '</div>';
-        });
-        html += '</div>';
+        html += '<div style="margin-bottom:6px">' + toolsUsed.map(renderToolChip).join('') + '</div>';
       }
       if (streamedText) {
         html += renderMd(streamedText);
@@ -851,10 +839,19 @@ async function _ccDoSend(message, skipUserMsg, forceTabId, intentMetadata) {
       } else if (evt.type === 'heartbeat') {
         return;
       } else if (evt.type === 'tool') {
-        toolsUsed.push({ name: evt.name, input: evt.input || {} });
+        toolsUsed.push({ name: evt.name, input: evt.input || {}, id: evt.id || null, status: evt.id ? 'pending' : null });
         if (activeTab) activeTab._toolsUsed = toolsUsed.slice();
         updateStreamDiv();
         if (msgs.scrollHeight - msgs.scrollTop - msgs.clientHeight < 150) msgs.scrollTop = msgs.scrollHeight;
+      } else if (evt.type === 'tool-update') {
+        for (var ti = 0; ti < toolsUsed.length; ti++) {
+          if (toolsUsed[ti] && toolsUsed[ti].id === evt.id) {
+            toolsUsed[ti].status = evt.status;
+            break;
+          }
+        }
+        if (activeTab) activeTab._toolsUsed = toolsUsed.slice();
+        updateStreamDiv();
       } else if (evt.type === 'done') {
         terminalEventSeen = true;
         _cleanupStreamDiv();

package/dashboard/js/render-utils.js

@@ -1,6 +1,26 @@
 // dashboard/js/render-utils.js — Shared formatting helpers for agent output rendering
 // Depends on: escHtml() and renderMd() from utils.js (loaded before this file)
 
+// marker + color per tool-call status. ACP `tool_call_update` flips chips from
+// pending → completed/failed mid-stream so the user sees real progress; the
+// non-pool (Claude direct) path emits status=null and stays on the neutral dot.
+var _CC_TOOL_CHIP_STYLE = {
+  pending:   { marker: '●', color: 'var(--muted)' },
+  completed: { marker: '✓', color: 'var(--green, #4ade80)' },
+  failed:    { marker: '✕', color: 'var(--red, #ef4444)' },
+};
+
+function renderToolChip(t) {
+  var name = typeof t === 'string' ? t : t.name;
+  var input = typeof t === 'string' ? {} : (t.input || {});
+  var status = typeof t === 'string' ? null : (t.status || null);
+  var style = _CC_TOOL_CHIP_STYLE[status] || _CC_TOOL_CHIP_STYLE.pending;
+  return '<div style="color:' + style.color + ';font-size:10px;font-family:monospace">'
+    + '<span style="flex-shrink:0">' + style.marker + '</span> '
+    + formatToolSummary(name, input)
+    + '</div>';
+}
+
 /**
  * Returns a one-line human-readable summary for a Claude tool call.
  * @param {string} name - Tool name (e.g. 'Bash', 'Read', 'Edit')

package/dashboard/js/settings.js

@@ -45,7 +45,6 @@ async function openSettings() {
   const e = data.engine || {};
   const c = data.claude || {};
   const agents = data.agents || {};
-  const t = data.teams || {};
 
   // Per-agent override placeholders surface the inherited fleet defaults as
   // muted text — operators see exactly what each agent will resolve to without
@@ -242,28 +241,6 @@ async function openSettings() {
       '</details>' +
     '</div>' +
 
-    '<h3 style="font-size:13px;color:var(--blue);margin-bottom:8px">Teams Integration</h3>' +
-    '<div style="display:flex;flex-direction:column;gap:6px;margin-bottom:8px">' +
-      settingsToggle('Enable Teams', 'set-teams-enabled', !!t.enabled, 'Connect Minions to Microsoft Teams') +
-    '</div>' +
-    '<div style="display:grid;grid-template-columns:1fr 1fr;gap:8px;margin-bottom:8px">' +
-      settingsField('App ID', 'set-teams-appId', t.appId || '', '', 'Microsoft App ID from Azure Bot Configuration') +
-      settingsField('App Password', 'set-teams-appPassword', t.appPassword || '', '', 'Client secret (leave blank for certificate auth)') +
-    '</div>' +
-    '<div style="font-size:10px;color:var(--muted);margin-bottom:4px;font-weight:600">Certificate Auth (alternative to client secret)</div>' +
-    '<div style="display:grid;grid-template-columns:1fr 1fr;gap:8px;margin-bottom:8px">' +
-      settingsField('Certificate Path', 'set-teams-certPath', t.certPath || '', '', 'Path to PEM certificate file') +
-      settingsField('Private Key Path', 'set-teams-privateKeyPath', t.privateKeyPath || '', '', 'Path to PEM private key file') +
-      settingsField('Tenant ID', 'set-teams-tenantId', t.tenantId || '', '', 'Azure AD tenant ID (required for cert auth)') +
-    '</div>' +
-    '<div style="display:grid;grid-template-columns:1fr 1fr;gap:8px;margin-bottom:16px">' +
-      settingsField('Notify Events', 'set-teams-notifyEvents', (t.notifyEvents || []).join(', '), '', 'Comma-separated event types to notify') +
-      settingsField('Inbox Poll Interval', 'set-teams-inboxPollInterval', t.inboxPollInterval || 15000, 'ms', 'How often to check for Teams messages') +
-    '</div>' +
-    '<div style="display:flex;flex-direction:column;gap:6px;margin-bottom:16px">' +
-      settingsToggle('CC Mirror', 'set-teams-ccMirror', t.ccMirror !== false, 'Mirror Command Center responses to Teams') +
-    '</div>' +
-
     '<h3 style="font-size:13px;color:var(--blue);margin-bottom:8px">Claude CLI</h3>' +
     '<div style="display:grid;grid-template-columns:1fr;gap:8px;margin-bottom:8px">' +
       settingsField('Allowed Tools', 'set-allowedTools', c.allowedTools || '', '', 'Claude allow-list passed through for compatibility; runtime bypass flags are adapter-owned.') +
@@ -637,18 +614,6 @@ async function saveSettings() {
       allowedTools: document.getElementById('set-allowedTools').value,
     };
 
-    const teamsPayload = {
-      enabled: document.getElementById('set-teams-enabled').checked,
-      appId: document.getElementById('set-teams-appId').value,
-      appPassword: document.getElementById('set-teams-appPassword').value,
-      certPath: document.getElementById('set-teams-certPath').value,
-      privateKeyPath: document.getElementById('set-teams-privateKeyPath').value,
-      tenantId: document.getElementById('set-teams-tenantId').value,
-      notifyEvents: document.getElementById('set-teams-notifyEvents').value,
-      inboxPollInterval: document.getElementById('set-teams-inboxPollInterval').value,
-      ccMirror: document.getElementById('set-teams-ccMirror').checked,
-    };
-
     const agentsPayload = {};
     document.querySelectorAll('[data-agent][data-field]').forEach(function(el) {
       const id = el.dataset.agent;
@@ -671,7 +636,7 @@ async function saveSettings() {
     // Save config
     const res = await fetch('/api/settings', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ engine: enginePayload, claude: claudePayload, agents: agentsPayload, teams: teamsPayload, projects: projectsPayload })
+      body: JSON.stringify({ engine: enginePayload, claude: claudePayload, agents: agentsPayload, projects: projectsPayload })
     });
     const result = await res.json();
     if (!res.ok) throw new Error(result.error);

package/dashboard.js

@@ -21,7 +21,6 @@ const _dashboardVersion = {
 };
 const shared = require('./engine/shared');
 const queries = require('./engine/queries');
-const teams = require('./engine/teams');
 const ado = require('./engine/ado');
 const gh = require('./engine/github');
 const issues = require('./engine/issues');
@@ -121,10 +120,6 @@ function mergeSettingsConfigUpdate(current, candidate, body, patch = {}) {
       if (candidate.agents && candidate.agents[id]) current.agents[id] = candidate.agents[id];
     }
   }
-  if (body.teams) {
-    if (candidate.teams) current.teams = candidate.teams;
-    else delete current.teams;
-  }
   if (body.projects && Array.isArray(body.projects)) {
     if (!Array.isArray(current.projects)) current.projects = [];
     for (const update of body.projects) {
@@ -804,7 +799,6 @@ function _steeringDeliveryState(agentId) {
 }
 
 const PLANS_DIR = path.join(MINIONS_DIR, 'plans');
-const TEAMS_INBOX_PATH = path.join(ENGINE_DIR, 'teams-inbox.json');
 
 // Resolve a plan/PRD file path: .json files live in prd/, .md files in plans/
 // Validates that the file stays within the expected directory to prevent path traversal.
@@ -2018,7 +2012,6 @@ const CC_API_FALLBACK_METHODS = new Set(['GET', 'POST', 'DELETE']);
 const CC_API_FALLBACK_BLOCKED_PREFIXES = [
   '/api/command-center',
   '/api/doc-chat',
-  '/api/bot',
 ];
 
 // SoT for CC's runtime API index. Captured lazily on the first HTTP request
@@ -2498,7 +2491,7 @@ async function _preflightModelCheck({ runtime: cliOverride, model: modelOverride
  * document body. Always re-sending extraContext is correctness-safe; the
  * pool's warm-process saving is preserved regardless.
  */
-function _invokeDocChatViaPool({ prompt, model, effort, engineConfig, systemPrompt, sessionKey, freshSession, timeoutMs, onChunk, onToolUse }) {
+function _invokeDocChatViaPool({ prompt, model, effort, engineConfig, systemPrompt, sessionKey, freshSession, timeoutMs, onChunk, onToolUse, onToolUpdate }) {
   const oneShot = !!freshSession;
   const tabKey = oneShot
     ? 'doc-chat:fresh:' + shared.uid()
@@ -2576,9 +2569,14 @@ function _invokeDocChatViaPool({ prompt, model, effort, engineConfig, systemProm
           try { onChunk(accumulated); } catch { /* swallow */ }
         }
       },
-      onToolUse: (name, input) => {
+      onToolUse: (name, input, toolCallId) => {
         if (onToolUse) {
-          try { onToolUse(name, input || {}); } catch { /* swallow */ }
+          try { onToolUse(name, input || {}, toolCallId); } catch { /* swallow */ }
+        }
+      },
+      onToolUpdate: (toolCallId, status) => {
+        if (onToolUpdate) {
+          try { onToolUpdate(toolCallId, status); } catch { /* swallow */ }
         }
       },
       onDone: () => {
@@ -2762,7 +2760,7 @@ async function ccCall(message, { store = 'cc', sessionKey, extraContext, label =
   return result;
 }
 
-async function ccCallStreaming(message, { store = 'cc', sessionKey, extraContext, label = 'command-center', timeout = CC_CALL_TIMEOUT_MS, maxTurns, allowedTools = 'Bash,Read,Write,Edit,Glob,Grep,WebFetch,WebSearch', skipStatePreamble = false, skipPreflight = false, model, onAbortReady, onChunk, onToolUse, onRetry, systemPrompt = CC_STATIC_SYSTEM_PROMPT, transcript, turnId, freshSession = false } = {}) {
+async function ccCallStreaming(message, { store = 'cc', sessionKey, extraContext, label = 'command-center', timeout = CC_CALL_TIMEOUT_MS, maxTurns, allowedTools = 'Bash,Read,Write,Edit,Glob,Grep,WebFetch,WebSearch', skipStatePreamble = false, skipPreflight = false, model, onAbortReady, onChunk, onToolUse, onToolUpdate, onRetry, systemPrompt = CC_STATIC_SYSTEM_PROMPT, transcript, turnId, freshSession = false } = {}) {
   if (!maxTurns) maxTurns = CONFIG.engine?.ccMaxTurns || shared.ENGINE_DEFAULTS.ccMaxTurns;
   if (!model) model = CONFIG.engine?.ccModel || shared.ENGINE_DEFAULTS.ccModel;
   const ccEffort = CONFIG.engine?.ccEffort || shared.ENGINE_DEFAULTS.ccEffort;
@@ -2793,7 +2791,7 @@ async function ccCallStreaming(message, { store = 'cc', sessionKey, extraContext
     const p = _invokeDocChatViaPool({
       prompt: poolPrompt, sessionKey, model, effort: ccEffort,
       engineConfig: CONFIG.engine, systemPrompt,
-      onChunk, onToolUse,
+      onChunk, onToolUse, onToolUpdate,
       freshSession, timeoutMs: timeout,
     });
     if (onAbortReady) onAbortReady(p.abort);
@@ -5034,9 +5032,6 @@ const server = http.createServer(async (req, res) => {
         }
       }
 
-      // Teams notification for plan approval — non-blocking
-      try { teams.teamsNotifyPlanEvent({ name: plan.plan_summary || body.file, file: body.file }, 'plan-approved').catch(() => {}); } catch {}
-
       invalidateStatusCache();
       return jsonReply(res, 200, { ok: true, status: 'approved', resumedWorkItems: resumed, diffAwareUpdate: diffAwareQueued });
     } catch (e) { return jsonReply(res, 400, { error: e.message }); }
@@ -5204,9 +5199,6 @@ const server = http.createServer(async (req, res) => {
         return data;
       }, { defaultValue: {} });
 
-      // Teams notification for plan rejection — non-blocking
-      try { teams.teamsNotifyPlanEvent({ name: plan.plan_summary || body.file, file: body.file }, 'plan-rejected').catch(() => {}); } catch {}
-
       return jsonReply(res, 200, { ok: true, status: 'rejected' });
     } catch (e) { return jsonReply(res, 400, { error: e.message }); }
   }
@@ -6163,17 +6155,10 @@ What would you like to discuss or change? When you're happy, say "approve" and I
         const abort = ccInFlightAborts.get(tabId);
         if (abort) { try { abort(); } catch {} }
       }
-      // Sub-task C of W-mp2w003600196c51: when the worker pool is on, abort
-      // must also fire `session/cancel` on the persistent ACP process so the
-      // remote daemon stops generating into a torn-down session. The pool
-      // exposes cancellation via the SessionHandle returned from getSession;
-      // we don't keep that handle around here, so route through closeTab to
-      // both cancel inflight and tear down the worker (cheaper than tracking
-      // per-tab handles in dashboard state, and matches "tab close" semantics
-      // — if the user explicitly aborted, we don't owe them a warm process).
-      // Off when the flag is off so legacy SIGTERM-only behavior is preserved.
+      // Cancel the inflight ACP turn but keep the warm worker; closing the
+      // tab here would force the next turn to pay the ~7-8 s cold-spawn.
       if (shared.resolveCcUseWorkerPool(CONFIG.engine)) {
-        try { ccWorkerPool.closeTab(tabId); } catch { /* swallow */ }
+        try { ccWorkerPool.cancelInflight(tabId); } catch { /* swallow */ }
       }
       _clearCcLiveStream(tabId);
       _releaseCCTab(tabId);
@@ -6275,10 +6260,6 @@ What would you like to discuss or change? When you're happy, say "approve" and I
           sessionId: ccSession.sessionId,
           newSession: !wasResume,
         };
-        // Mirror user-facing text to Teams (skip Teams-originated turns).
-        if (!tabId.startsWith('teams-')) {
-          teams.teamsPostCCResponse(body.message, result.text).catch(() => {});
-        }
         if (sessionReset) replyBody.sessionReset = true;
         return jsonReply(res, 200, replyBody);
       } finally {
@@ -6333,9 +6314,10 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       },
       onToolUse: (name, input) => {
         _touchCcLiveStream(liveState);
-        toolUses.push({ name, input: input || {} });
-        liveState.tools.push({ name, input: input || {} });
-        if (liveState.writer) liveState.writer({ type: 'tool', name, input: _lightToolInput(input) });
+        const entry = { name, input: input || {}, id: null, status: null };
+        toolUses.push(entry);
+        liveState.tools.push(entry);
+        if (liveState.writer) liveState.writer({ type: 'tool', name, input: _lightToolInput(input), id: null });
       },
     });
   }
@@ -6431,14 +6413,26 @@ What would you like to discuss or change? When you're happy, say "approve" and I
           liveState.text = accumulated;
           if (liveState.writer) liveState.writer({ type: 'chunk', text: accumulated });
         },
-        onToolUse: (name, input) => {
+        onToolUse: (name, input, toolCallId) => {
           if (_tFirstTool == null) _tFirstTool = Date.now();
           _toolUseCount += 1;
           _touchCcLiveStream(liveState);
           const safeInput = input || {};
-          if (Array.isArray(toolUses)) toolUses.push({ name, input: safeInput });
-          if (Array.isArray(liveState.tools)) liveState.tools.push({ name, input: safeInput });
-          if (liveState.writer) liveState.writer({ type: 'tool', name, input: _lightToolInput(safeInput) });
+          const entry = { name, input: safeInput, id: toolCallId || null, status: toolCallId ? 'pending' : null };
+          if (Array.isArray(toolUses)) toolUses.push(entry);
+          if (Array.isArray(liveState.tools)) liveState.tools.push(entry);
+          if (liveState.writer) liveState.writer({ type: 'tool', name, input: _lightToolInput(safeInput), id: toolCallId || null });
+        },
+        onToolUpdate: (toolCallId, status) => {
+          _touchCcLiveStream(liveState);
+          // toolUses and liveState.tools hold the same entry references (both
+          // populated via onToolUse above), so mutating in one side reflects
+          // in the other. Patch once.
+          if (Array.isArray(toolUses)) {
+            const entry = toolUses.find((e) => e && e.id === toolCallId);
+            if (entry) entry.status = status;
+          }
+          if (liveState.writer) liveState.writer({ type: 'tool-update', id: toolCallId, status });
         },
         onDone: () => {
           _emitTimingLog(_lifecycle, _tSessionReady, Date.now(), 'done');
@@ -6748,12 +6742,6 @@ What would you like to discuss or change? When you're happy, say "approve" and I
         liveState.donePayload = donePayload;
         if (liveState.writer) liveState.writer(donePayload);
 
-        // Mirror CC response to Teams (non-blocking, skip Teams-originated)
-        const _streamTabId = body.tabId || 'default';
-        if (!_streamTabId.startsWith('teams-')) {
-          teams.teamsPostCCResponse(body.message, displayText).catch(() => {});
-        }
-
         if (liveState.endResponse) liveState.endResponse();
         _scheduleCcLiveCleanup(tabId);
       } finally {
@@ -7089,7 +7077,6 @@ What would you like to discuss or change? When you're happy, say "approve" and I
         engine,
         claude: settingsClaudeConfig(config),
         agents: config.agents || {},
-        teams: { ...shared.ENGINE_DEFAULTS.teams, ...(config.teams || {}) },
         projects: (config.projects || []).map(p => ({
           name: p.name,
           workSources: {
@@ -7377,22 +7364,6 @@ What would you like to discuss or change? When you're happy, say "approve" and I
         }
       }
 
-      if (body.teams) {
-        if (!config.teams) config.teams = {};
-        const tm = body.teams;
-        if (tm.enabled !== undefined) config.teams.enabled = !!tm.enabled;
-        for (const key of ['appId', 'appPassword', 'certPath', 'privateKeyPath', 'tenantId']) {
-          if (tm[key] !== undefined) config.teams[key] = String(tm[key] || '');
-        }
-        if (tm.notifyEvents !== undefined) {
-          config.teams.notifyEvents = Array.isArray(tm.notifyEvents) ? tm.notifyEvents : String(tm.notifyEvents || '').split(',').map(s => s.trim()).filter(Boolean);
-        }
-        if (tm.inboxPollInterval !== undefined) config.teams.inboxPollInterval = Math.max(5000, Number(tm.inboxPollInterval) || 15000);
-        if (tm.ccMirror !== undefined) config.teams.ccMirror = !!tm.ccMirror;
-        // Invalidate cached adapter so credential changes take effect
-        teams._resetAdapter();
-      }
-
       if (body.projects && Array.isArray(body.projects)) {
         if (!config.projects) config.projects = [];
         for (const update of body.projects) {
@@ -7540,93 +7511,6 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     }
   }
 
-  // ── Teams Bot Handler ─────────────────────────────────────────────────────
-
-  async function handleTeamsBot(req, res) {
-    if (!teams.isTeamsEnabled()) {
-      return jsonReply(res, 503, { error: 'Teams integration disabled' }, req);
-    }
-    const adapter = teams.createAdapter();
-    if (!adapter) {
-      return jsonReply(res, 503, { error: 'Teams adapter unavailable' }, req);
-    }
-    try {
-      await adapter.process(req, res, async (context) => {
-        const activity = context.activity;
-        const cfg = teams.getTeamsConfig();
-
-        // Save conversation reference on install/member events
-        if (activity.type === 'conversationUpdate' && activity.membersAdded?.length) {
-          const ref = context.activity.conversation?.id;
-          if (ref) {
-            const convRef = {
-              activityId: activity.id,
-              user: activity.from,
-              bot: activity.recipient,
-              conversation: activity.conversation,
-              channelId: activity.channelId,
-              locale: activity.locale,
-              serviceUrl: activity.serviceUrl,
-            };
-            teams.saveConversationRef(activity.conversation.id, convRef);
-            shared.log('info', `Teams conversationUpdate: saved ref for ${activity.conversation.id}`);
-          }
-        }
-
-        if (activity.type === 'installationUpdate') {
-          const convRef = {
-            activityId: activity.id,
-            user: activity.from,
-            bot: activity.recipient,
-            conversation: activity.conversation,
-            channelId: activity.channelId,
-            locale: activity.locale,
-            serviceUrl: activity.serviceUrl,
-          };
-          if (activity.conversation?.id) {
-            teams.saveConversationRef(activity.conversation.id, convRef);
-            shared.log('info', `Teams installationUpdate: saved ref for ${activity.conversation.id}`);
-          }
-        }
-
-        // Handle incoming messages
-        if (activity.type === 'message' && activity.text) {
-          // Filter bot's own echo messages
-          if (activity.from?.id === cfg.appId) return;
-
-          const msgId = `teams-${Date.now()}-${shared.uid()}`;
-          const convRef = {
-            activityId: activity.id,
-            user: activity.from,
-            bot: activity.recipient,
-            conversation: activity.conversation,
-            channelId: activity.channelId,
-            locale: activity.locale,
-            serviceUrl: activity.serviceUrl,
-          };
-          mutateJsonFileLocked(TEAMS_INBOX_PATH, (inbox) => {
-            if (!Array.isArray(inbox)) inbox = [];
-            inbox.push({
-              id: msgId,
-              text: activity.text,
-              from: activity.from?.name || activity.from?.id || 'unknown',
-              conversationRef: convRef,
-              receivedAt: new Date().toISOString(),
-              _processedAt: null,
-            });
-            return inbox;
-          }, { defaultValue: [] });
-          shared.log('info', `Teams message received from ${activity.from?.name || 'unknown'}: ${activity.text.slice(0, 80)}`);
-        }
-      });
-    } catch (err) {
-      shared.log('warn', `Teams bot handler error: ${err.message}`);
-      if (!res.headersSent) {
-        return jsonReply(res, 500, { error: 'Bot processing failed' }, req);
-      }
-    }
-  }
-
   // ── Route Registry ──────────────────────────────────────────────────────────
   // Order matters: specific routes before general ones (e.g., /api/plans/approve before /api/plans/:file)
 
@@ -8311,16 +8195,13 @@ What would you like to discuss or change? When you're happy, say "approve" and I
 
     // Settings
     { method: 'GET', path: '/api/settings', desc: 'Return current engine + claude + routing config', handler: handleSettingsRead },
-    { method: 'POST', path: '/api/settings', desc: 'Update engine + claude + agent + teams + projects config', params: 'engine?, claude?, agents?, teams?, projects?', handler: handleSettingsUpdate },
+    { method: 'POST', path: '/api/settings', desc: 'Update engine + claude + agent + projects config', params: 'engine?, claude?, agents?, projects?', handler: handleSettingsUpdate },
     { method: 'POST', path: '/api/settings/routing', desc: 'Update routing.md', params: 'content', handler: handleSettingsRouting },
     { method: 'POST', path: '/api/settings/reset', desc: 'Reset engine + claude + agent settings to defaults', handler: handleSettingsReset },
 
     // Feature flags (experimental / in-progress UX gates — see engine/features.js)
     { method: 'GET', path: '/api/features', desc: 'List registered feature flags with current enabled state', handler: handleFeaturesList },
     { method: 'POST', path: '/api/features/toggle', desc: 'Enable/disable a registered feature flag', params: 'id, enabled', handler: handleFeaturesToggle },
-
-    // Teams Bot Framework webhook
-    { method: 'POST', path: '/api/bot', desc: 'Bot Framework webhook for Teams integration', handler: handleTeamsBot },
   ];
 
   // ── Route Dispatcher ────────────────────────────────────────────────────────

package/docs/README.md

@@ -23,13 +23,11 @@ Architecture, design proposals, and lifecycle references for people working on t
 
 ## Operations
 
-Operational runbooks for engine operators, fleet maintainers, and Teams integrators.
+Operational runbooks for engine operators and fleet maintainers.
 
 - [auto-discovery.md](auto-discovery.md) — Auto-discovery and execution pipeline: the per-tick orchestration loop and the four work-discovery sources.
 - [engine-restart.md](engine-restart.md) — How agents survive an engine restart: state persistence, the 20-minute startup grace period, and orphan reattachment via PID files and `live-output.log`.
 - [human-vs-automated.md](human-vs-automated.md) — Quick reference table of which features humans start, run, decide, and recover, and the two human approval gates.
-- [teams-production.md](teams-production.md) — Three deployment options (Azure App Service, Container Apps, self-hosted VM) for migrating the Teams bot from a Dev Tunnel to a stable HTTPS endpoint.
-- [teams-setup.md](teams-setup.md) — End-to-end guide for connecting Minions to Microsoft Teams via Azure Bot Framework and a Dev Tunnel.
 
 ---
 

package/docs/deprecated.json

@@ -14,6 +14,30 @@
     "reason": "Read-side tolerance: cleanup sweep auto-migrates four obsolete work-item / PRD status strings ('in-pr', 'implemented', 'complete', 'needs-human-review') to the canonical 'done' / 'failed' values. The aliases are no longer written anywhere in the engine; the constants exist only to repair stale on-disk values from old engine versions.",
     "targetRemovalDate": null,
     "notes": "Keep indefinitely until telemetry / a sweep log shows zero migrations performed for 30 consecutive days across all known projects (work-items.json + prd/*.json). At that point the constants and both _migrateLegacyItem branches in engine/cleanup.js (definitions at :799-800; usage at :803-815 for work items and :880-887 for PRD missing_features) can be deleted. Total cost on disk today: 4 strings."
+  },
+  {
+    "id": "native-teams-integration",
+    "removedAt": "2026-05-14",
+    "reason": "Native Microsoft Teams Bot Framework integration removed end-to-end. The Teams MCP server (teams-* tools, configured in the CC client outside this repo) supersedes the in-repo Bot Framework path. Removal closes the dual-implementation gap and drops the botbuilder dependency.",
+    "removedLocations": [
+      "engine/teams.js",
+      "engine/teams-cards.js",
+      "engine/teams-state.json (runtime state)",
+      "engine/teams-inbox.json (runtime state, generated by the deleted /api/bot handler)",
+      "dashboard.js: POST /api/bot route + handleTeamsBot, TEAMS_INBOX_PATH constant, CC mirror hooks (teamsPostCCResponse), plan-approval/rejection Teams notifications, settings GET/POST teams block",
+      "dashboard/js/settings.js: Teams Integration settings UI + teamsPayload submit",
+      "engine/lifecycle.js: teamsNotifyCompletion, teamsNotifyPlanEvent (verify-created + plan-completed), teamsNotifyPrEvent (post-merge)",
+      "engine/github.js + engine/ado.js: teamsNotifyPrEvent on pr-approved and build-failed",
+      "engine/preflight.js: Teams integration doctor check",
+      "engine/cli.js: teamsInboxTimer + clearInterval on shutdown",
+      "engine/shared.js: ENGINE_DEFAULTS.teams block",
+      "package.json: botbuilder dependency (4.23.3)",
+      "docs/teams-setup.md, docs/teams-production.md",
+      "test/unit/auto-recovery.test.js: ~58 Teams test cases",
+      "test/unit/preflight-behavioral.test.js: 4 doctor Teams checks + teams field in the docs-link coverage scenario",
+      "README.md / CLAUDE.md / docs/README.md / TODO.md / docs/rfc-completion-json.md: prose references"
+    ],
+    "notes": "The Teams MCP (teams-* tools) lives outside this repo in the CC client config and is NOT affected. If Teams-style notifications are needed again, route them through the MCP layer or an external webhook watch action — do not re-introduce the Bot Framework SDK in-process."
   }
 ]
 

package/docs/rfc-completion-json.md

@@ -101,8 +101,8 @@ The agent must not write the file in pieces. Empty, truncated, or malformed JSON
 
   // ── Always required ──────────────────────────────────────────────────────
   "status": "done",                       // see §4.2
-  "summary": "Added /api/bot endpoint and wired Teams inbox.",  // ≤500 chars
-  "filesChanged": ["engine/teams.js", "dashboard.js"],          // optional, hint only
+  "summary": "Added /api/example endpoint and wired example integration.",  // ≤500 chars
+  "filesChanged": ["engine/example.js", "dashboard.js"],          // optional, hint only
 
   // ── PR control plane (replaces sites 1, 4 for fix/implement/verify) ─────
   "prs": [
@@ -271,7 +271,7 @@ These paths stay on stdout / live-output.log:
 
 1. **`engine/timeout.js` completion-via-output detection** (`timeout.js:189-219`). The signal is the engine-written `[process-exit]` sentinel, emitted even if the agent crashed before writing completion.json. Removing it would mean orphans that finished during process-handle loss are never reconciled.
 2. **Stale-orphan cleanup via `live-output.log` mtime** (`timeout.js:178`). Completion.json is written once at exit, so `live-output.log` remains the best indirect signal after the engine loses process tracking.
-3. **`parseStreamJsonOutput` for `resultSummary`** in `parseAgentOutput` (`lifecycle.js:1483`). This extracts the human-readable summary from the CLI's stream-json. Even after the flip, `completion.summary` is *also* extracted, but the stream-json text remains the canonical "what did the agent say last" — used in dashboards, agent history, Teams notifications. The two coexist: `completion.summary` is for routing decisions, the stream-json text is for display.
+3. **`parseStreamJsonOutput` for `resultSummary`** in `parseAgentOutput` (`lifecycle.js:1483`). This extracts the human-readable summary from the CLI's stream-json. Even after the flip, `completion.summary` is *also* extracted, but the stream-json text remains the canonical "what did the agent say last" — used in dashboards, agent history, and external notifications. The two coexist: `completion.summary` is for routing decisions, the stream-json text is for display.
 4. **Inbox-file skill scan** (`lifecycle.js:2013-2024`). Some agents write skills into their inbox findings file (a deliberate human-discoverable artifact). The completion file deprecates inline ` ```skill ` blocks in stdout, but the inbox file scan is opt-in and stays — it's a different surface (a real file the agent intentionally wrote, not regex-scraped from stdout).
 
 ### 5.5 Backward Compatibility
@@ -355,7 +355,7 @@ engine falls back to stdout parsing during the dual-mode period. After the
 flip date, missing/invalid completion.json marks your dispatch failed.
 
 Do NOT include sensitive data (tokens, API keys) — completion.json is read
-by the engine and may surface in dashboard views and Teams notifications.
+by the engine and may surface in dashboard views and external notifications.
 ```
 
 ### 7.2 Per-Playbook Removals

package/engine/ado.js

@@ -860,14 +860,6 @@ async function pollPrStatus(config) {
       pr.reviewStatus = newReviewStatus;
       updated = true;
       shared.trackReviewMetric(pr, newReviewStatus, config);
-      if (newReviewStatus === 'approved') {
-        // Teams notification for PR approval — non-blocking, edge-triggered (only on transition)
-        try {
-          const teams = require('./teams');
-          const prFilePath = shared.projectPrPath(project);
-          teams.teamsNotifyPrEvent(pr, 'pr-approved', project, prFilePath).catch(() => {});
-        } catch {}
-      }
     }
 
     if (newStatus !== PR_STATUS.ACTIVE) return updated;
@@ -976,15 +968,6 @@ async function pollPrStatus(config) {
           }
         }
         updated = true;
-
-        if (buildStatus === 'failing') {
-          // Teams notification for build failure — non-blocking
-          try {
-            const teams = require('./teams');
-            const prFilePath = shared.projectPrPath(project);
-            teams.teamsNotifyPrEvent(pr, 'build-failed', project, prFilePath).catch(() => {});
-          } catch {}
-        }
       }
       if (buildStatus === 'failing') {
         if (buildFailReason && pr.buildFailReason !== buildFailReason) {