@yemi33/minions 0.1.1937 → 0.1.1939

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/render-utils.js

@@ -60,7 +60,9 @@ function formatToolSummary(name, input) {
     }
     default: {
       var keys = Object.keys(inp);
-      if (keys.length === 0) return escHtml(name) + '()';
+      // No-input case (e.g. ACP chips that carry their full label in `name`
+      // like "Fetch /api/status") — render the name verbatim, no parens.
+      if (keys.length === 0) return escHtml(name);
       var firstKey = keys[0];
       var firstVal = String(inp[firstKey] || '');
       if (firstVal.length > 40) firstVal = firstVal.slice(0, 37) + '...';

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/distribution.md

@@ -1,6 +1,37 @@
 # Distribution & Publishing
 
-Minions is distributed as an npm package (`@yemi33/minions`) from a sanitized package boundary.
+Minions is distributed as an npm package (`@yemi33/minions`) from a sanitized package boundary. The source lives across three GitHub remotes, with a strict one-way port direction (see [Three-Remote Topology](#three-remote-topology)).
+
+## Three-Remote Topology
+
+| Remote (local name on `D:/squad`) | URL | Role |
+|-----------------------------------|-----|------|
+| `origin` | `https://github.com/yemi33/minions` | **Authoring source of truth.** All normal code changes (engine, dashboard, playbooks, agents, etc.) land here first. Triggers npm publish + downstream mirrors. |
+| `emu` | `https://github.com/yemishin_microsoft/minions` | Microsoft-identity mirror of `yemi33/minions`. Automated via `.github/workflows/mirror-to-emu.yml`. |
+| `opg` | `https://github.com/opg-microsoft/minions` | **Compliant store** under the `opg-microsoft` org. Automated mirror to a sync branch via `.github/workflows/mirror-to-opg.yml`; `main` is reconciled by a maintainer PR because it carries enterprise/compliance-only commits (see below). |
+
+### Port direction
+
+**Strictly one-way: `yemi33` → `emu`, `yemi33` → `opg`.**
+
+- **Do NOT port from `emu` or `opg` back into `yemi33`.** Their `main` branches may carry org-specific compliance content (`.github/policies/*`, `.github/acl/*`, `.github/compliance/*`, JIT access policy, internal access lists, MS-org issue templates) that has no business in the personal-fork authoring source.
+- **Direct pushes to `opg-microsoft/minions` are allowed for enterprise/compliance-specific changes only.** Examples: editing `.github/policies/jit.yml`, updating `.github/acl/access.yml`, modifying `.github/compliance/inventory.yml`. These changes stay on `opg` and are not backported.
+- Other teams (Microsoft infra, compliance review) may also push to `opg-microsoft/minions` directly. Treat its `main` as potentially ahead of `yemi33`/`emu` on policy files.
+
+### Why opg can't use the same direct-mirror pattern as emu
+
+`mirror-to-emu.yml` does `git push emu HEAD:master` — direct overwrite, because `emu/master` is meant to be a 1:1 reflection of `yemi33/master`. `opg-microsoft/minions` is different: its `main` carries enterprise-only commits that don't exist on `yemi33/master`, so a force-push to `opg/main` would destroy them. Instead, `mirror-to-opg.yml` force-pushes `yemi33/master` to `opg/sync/yemi33-master` (a dedicated bot-owned branch), and a maintainer opens or updates a PR from `sync/yemi33-master` → `main` for review. The first such PR uses `--allow-unrelated-histories` to bridge the initial divergence; subsequent ones merge incrementally.
+
+### Required secrets on `yemi33/minions`
+
+| Secret | Purpose | Workflow |
+|--------|---------|----------|
+| `NPM_TOKEN` | npm publish | `publish.yml` |
+| `EMU_PUSH_TOKEN` | Push to `yemishin_microsoft/minions` | `mirror-to-emu.yml` |
+| `EMU_PACKAGES_TOKEN` | Publish to `@yemishin_microsoft` internal registry | `publish-internal-github-packages.yml` |
+| `OPG_PUSH_TOKEN` | Push to `opg-microsoft/minions` sync branch | `mirror-to-opg.yml` |
+
+Each mirror workflow gracefully skips with a warning if its token secret is unset, so unconfigured downstreams never block the primary publish path.
 
 ## Distribution Boundary
 

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/cc-worker-pool.js

@@ -252,9 +252,7 @@ class Worker {
           try { this.inflight.onChunk(text); } catch { /* swallow */ }
         }
       } else if (update.sessionUpdate === 'tool_call' && this.inflight.onToolUse) {
-        // ACP `tool_call` (pending) → Claude-style {name, input} via
-        // _mapAcpToolCallToToolUse so the dashboard's formatToolSummary
-        // formatters (Bash → "$ <cmd>", etc.) work unchanged.
+        // ACP `tool_call` (pending) → chip label via _mapAcpToolCallToToolUse.
         const mapped = _mapAcpToolCallToToolUse(update);
         if (mapped) {
           try { this.inflight.onToolUse(mapped.name, mapped.input, update.toolCallId); }
@@ -447,50 +445,40 @@ function _extractChunkText(content) {
 // (Bash → "$ <cmd>", Read → "Reading <path>", etc.). Unknown kinds fall back
 // to ACP's human-readable `title` with the raw input attached, which renders
 // through the default `<title>(<key>: <val>)` formatter.
+// Map an ACP `tool_call` notification to a chip label. Strategy:
+//
+//   1. Detect well-known shapes by rawInput field presence and route through
+//      formatToolSummary's Claude-tool formatters (Bash → "$ <cmd>",
+//      Grep → "Searching `pat` in path", Read → "Reading <path>"). Preserves
+//      the granular detail you actually want when reading the chip list.
+//
+//   2. Fall back to ACP's human-curated `title` for anything that doesn't
+//      match. Title is always populated and immune to field-name drift, so
+//      the empty-`$` failure mode (kind:execute with no `command`) becomes
+//      a clean "Find work item W-..." chip instead of a broken stub.
+//
+// Kind-based routing is intentionally NOT used — ACP overloads `kind:read`
+// for both file-view and grep, and `kind:execute` sometimes arrives without
+// a `command`. Field-detection on rawInput is more reliable.
 function _mapAcpToolCallToToolUse(update) {
   if (!update || update.sessionUpdate !== 'tool_call') return null;
   const rawInput = (update.rawInput && typeof update.rawInput === 'object') ? update.rawInput : {};
-  const kind = String(update.kind || '').toLowerCase();
-  const title = update.title || '';
-
-  // Field-name normalization: Copilot ACP and Claude tool_use use different
-  // keys for the same concept (Copilot `path`, Claude `file_path`). Normalize
-  // here so the dashboard's formatToolSummary — written against Claude's
-  // names — produces the same chip text on both runtimes.
-  const filePath = rawInput.file_path || rawInput.path || rawInput.filePath || '';
-
-  // Pattern (Grep) — Copilot uses `paths` (plural) for the search scope where
-  // Claude's Grep takes `path`.
-  if (typeof rawInput.pattern === 'string') {
-    return {
-      name: 'Grep',
-      input: { pattern: rawInput.pattern, path: rawInput.paths || rawInput.path || '.' },
-    };
-  }
+  const title = update.title || update.kind || 'Tool';
 
-  switch (kind) {
-    case 'execute':
-      return { name: 'Bash', input: rawInput };
-    case 'read':
-      // ACP overloads `read` for both file-view and grep — the pattern check
-      // above already handled grep, so this branch is the file-view case.
-      return { name: 'Read', input: { file_path: filePath } };
-    case 'edit':
-      return { name: 'Edit', input: { file_path: filePath } };
-    case 'search':
-      // Pattern check above handled the grep case; arriving here means glob.
-      return { name: 'Glob', input: rawInput };
-    case 'fetch':
-      return { name: 'WebFetch', input: rawInput };
-    case 'think':
-      return { name: title || 'Think', input: rawInput };
-    default:
-      // Unknown kind — use ACP's human-readable title as the chip label and
-      // drop rawInput so formatToolSummary's default branch shows just the
-      // title (avoids `<title>(<key>: <val>)` clutter when the input shape
-      // is unfamiliar).
-      return { name: title || kind || 'Tool', input: {} };
+  if (typeof rawInput.command === 'string' && rawInput.command) {
+    return { name: 'Bash', input: { command: rawInput.command } };
+  }
+  if (typeof rawInput.pattern === 'string' && rawInput.pattern) {
+    return { name: 'Grep', input: { pattern: rawInput.pattern, path: rawInput.paths || rawInput.path || '.' } };
+  }
+  if (typeof rawInput.path === 'string' && rawInput.path) {
+    const isEdit = String(update.kind || '').toLowerCase() === 'edit';
+    return { name: isEdit ? 'Edit' : 'Read', input: { file_path: rawInput.path } };
+  }
+  if (typeof rawInput.url === 'string' && rawInput.url) {
+    return { name: 'WebFetch', input: { url: rawInput.url } };
   }
+  return { name: title, input: {} };
 }
 
 // ── Public API ────────────────────────────────────────────────────────────

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) {