@yemi33/minions 0.1.1937 → 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/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
@@ -5039,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 }); }
@@ -5209,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 }); }
   }
@@ -6273,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 {
@@ -6759,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 {
@@ -7100,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: {
@@ -7388,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) {
@@ -7551,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)
 
@@ -8322,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) {

package/engine/cli.js

@@ -731,19 +731,6 @@ const commands = {
       }
     }, 1000);
 
-    // Teams inbox poll timer — process incoming Teams messages through CC
-    const teams = require('./teams');
-    const teamsInboxInterval = config.teams?.inboxPollInterval ?? shared.ENGINE_DEFAULTS.teams.inboxPollInterval;
-    const teamsInboxTimer = teams.isTeamsEnabled() ? setInterval(() => {
-      try {
-        const ctrl = getControl();
-        if (ctrl.state !== 'running') return;
-        teams.processTeamsInbox().catch(err => {
-          shared.log('warn', `Teams inbox poll error: ${err.message}`);
-        });
-      } catch {}
-    }, teamsInboxInterval) : null;
-
     console.log(`Tick interval: ${interval / 1000}s | Max concurrent: ${config.engine?.maxConcurrent || 5}`);
     console.log('Press Ctrl+C to stop');
 
@@ -804,7 +791,6 @@ const commands = {
       console.log(`\n${signal} received — initiating graceful shutdown...`);
       clearInterval(tickTimer);
       clearInterval(fastPollTimer);
-      if (teamsInboxTimer) clearInterval(teamsInboxTimer);
       for (const f of _watchedFiles) { try { fs.unwatchFile(f); } catch { /* cleanup */ } }
       const stoppingAt = e.ts();
       const stoppingWrite = markControlStoppingForOwner(controlOwner, stoppingAt);

package/engine/github.js

@@ -646,14 +646,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 {}
-        }
       }
     }
 
@@ -717,15 +709,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) {

package/engine/lifecycle.js

@@ -377,26 +377,11 @@ function checkPlanCompletion(meta, config) {
       }
     }
 
-    if (createdAny) {
-      try {
-        const teams = require('./teams');
-        teams.teamsNotifyPlanEvent({ name: plan.plan_summary || planFile, file: planFile }, 'verify-created').catch(() => {});
-      } catch {}
-    }
     if (reopenedAny) log('info', `Plan ${planFile}: re-opened verify WI(s) for modified plan`);
   }
 
   // Archive deferred until verify completes
 
-  // Teams notification for plan completion — non-blocking
-  try {
-    const teams = require('./teams');
-    teams.teamsNotifyPlanEvent({
-      name: plan.plan_summary || planFile, file: planFile, project: plan.project,
-      doneCount: doneItems.length, totalCount: planFeatureIds.size,
-    }, 'plan-completed').catch(() => {});
-  } catch {}
-
   log('info', `PRD ${planFile} completed: ${doneItems.length} done, ${failedItems.length} failed, runtime ${runtimeMin}m`);
   return true;
 }
@@ -2271,14 +2256,6 @@ async function handlePostMerge(pr, project, config, newStatus) {
     });
   }
 
-  // Teams PR lifecycle notification — non-blocking
-  try {
-    const teams = require('./teams');
-    const prEvent = newStatus === PR_STATUS.MERGED ? 'pr-merged' : 'pr-abandoned';
-    const prFilePath = project ? projectPrPath(project) : null;
-    teams.teamsNotifyPrEvent(pr, prEvent, project, prFilePath).catch(() => {});
-  } catch {}
-
   log('info', `Post-merge hooks completed for ${pr.id}`);
 }
 
@@ -3719,12 +3696,6 @@ async function runPostCompletionHooks(dispatchItem, agentId, code, stdout, confi
   const metricsResult = isAutoRetry ? 'retry' : finalResult;
   updateMetrics(agentId, dispatchItem, metricsResult, taskUsage, prsCreatedCount, model);
 
-  // Teams notification — non-blocking
-  try {
-    const teams = require('./teams');
-    teams.teamsNotifyCompletion(dispatchItem, finalResult, agentId).catch(() => {});
-  } catch {}
-
   return { resultSummary, taskUsage, autoRecovered, structuredCompletion, completionContractFailure, agentReportedFailure, agentRetryable };
 }
 

package/engine/preflight.js

@@ -580,25 +580,6 @@ function doctor(minionsHome) {
     } else {
       runtimeResults.push({ name: 'Agents configured', ok: false, message: 'no agents in config.json — re-run: minions init (see docs/auto-discovery.md)' });
     }
-
-    // Check Teams integration — supports client secret OR certificate auth
-    const teams = config.teams;
-    if (teams && teams.enabled === true) {
-      const hasSecret = !!teams.appId && !!teams.appPassword;
-      const hasCert = !!teams.appId && !!teams.certPath && !!teams.privateKeyPath && !!teams.tenantId;
-      if (!hasSecret && !hasCert) {
-        const missing = [
-          !teams.appId && 'appId',
-          !teams.appPassword && !teams.certPath && 'appPassword or certPath+privateKeyPath+tenantId',
-        ].filter(Boolean).join(', ');
-        runtimeResults.push({ name: 'Teams integration', ok: 'warn', message: `enabled but missing: ${missing} — see docs/teams-setup.md` });
-      } else {
-        const authMode = hasCert ? 'certificate' : 'client secret';
-        runtimeResults.push({ name: 'Teams integration', ok: true, message: `configured (${authMode})` });
-      }
-    } else {
-      runtimeResults.push({ name: 'Teams integration', ok: 'warn', message: 'disabled — see docs/teams-setup.md' });
-    }
   } catch {
     runtimeResults.push({ name: 'Config', ok: false, message: `missing or invalid — run: minions init (see docs/distribution.md)` });
   }

package/engine/shared.js

@@ -1147,19 +1147,6 @@ const ENGINE_DEFAULTS = {
   // knows which subkeys to flag as deprecated. Do not consume `claude.*` in new code — use the runtime
   // adapter system (engine/runtimes/) and the resolveAgent*/resolveCc* helpers instead.
   _deprecatedConfigClaudeFields: ['binary', 'outputFormat', 'allowedTools', 'maxTurns', 'effort', 'budgetCap'],
-  // Teams integration — config.teams shape: { enabled, appId, appPassword, certPath, privateKeyPath, tenantId, notifyEvents, ccMirror, inboxPollInterval }
-  // Auth modes: (1) appId + appPassword (client secret), or (2) appId + certPath + privateKeyPath + tenantId (certificate)
-  teams: {
-    enabled: false,
-    appId: '',
-    appPassword: '',
-    certPath: '',          // PEM certificate file path (certificate auth)
-    privateKeyPath: '',    // PEM private key file path (certificate auth)
-    tenantId: '',          // Azure AD tenant ID (required for certificate auth)
-    notifyEvents: ['pr-merged', 'agent-completed', 'plan-completed', 'agent-failed'],
-    ccMirror: true,
-    inboxPollInterval: 15000,
-  },
 };
 
 // ─── Runtime Fleet Resolution (P-3b8e5f1d) ──────────────────────────────────

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.1937",
+  "version": "0.1.1938",
   "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"
@@ -63,9 +63,6 @@
   "devDependencies": {
     "@playwright/test": "^1.58.2"
   },
-  "dependencies": {
-    "botbuilder": "4.23.3"
-  },
   "publishConfig": {
     "access": "public"
   }