@yemi33/minions 0.1.2185 → 0.1.2187

package/README.md

@@ -334,7 +334,7 @@ For GitHub repos, install and authenticate the [GitHub CLI](https://cli.github.c
 
 ### Azure DevOps Users
 
-For the best experience with ADO repos, install the [Azure CLI](https://learn.microsoft.com/en-us/cli/azure/install-azure-cli) with the Azure DevOps extension. Agents should use the `az` CLI first for Azure DevOps operations such as PR creation, PR lookup, comments, reviewers, work items, and pipelines. Use the Azure DevOps MCP fallback only when `az` is unavailable in the environment or insufficient for a specific action.
+For the best experience with ADO repos, install the [Azure CLI](https://learn.microsoft.com/en-us/cli/azure/install-azure-cli) with the Azure DevOps extension. Agents should use the `az` CLI first for Azure DevOps operations such as PR creation, PR lookup, comments, reviewers, work items, and pipelines. If `az` is unavailable or insufficient for a specific action (e.g. PR comment threads on an older extension), agents fall back to the Azure DevOps REST API (`_apis/git/...`) directly, using a token from `az account get-access-token` — no MCP server required.
 
 ```bash
 # Install Azure CLI
@@ -348,8 +348,6 @@ az login
 az devops configure --defaults organization=https://dev.azure.com/YOUR_ORG project=YOUR_PROJECT
 ```
 
-Optionally add the [Azure DevOps MCP server](https://github.com/microsoft/azure-devops-mcp) to your Claude Code settings (`~/.claude.json`) as a fallback. Agents inherit it on next spawn — no sync step needed.
-
 ## Work Items
 
 All work items use the shared `playbooks/work-item.md` template, which provides consistent branch naming, worktree workflow, PR creation steps, and status tracking.
@@ -676,7 +674,7 @@ To move to a new machine: `npm install -g @yemi33/minions && minions init --forc
     watches.js           watch-actions.js
     # Repo-host integrations
     github.js            issues.js
-    ado.js               ado-token.js          ado-mcp-wrapper.js
+    ado.js               ado-token.js
     ado-status.js        check-status.js
     # Runtime state (generated, gitignored)
     control.json         <- running/paused/stopped

package/dashboard.js

@@ -3838,7 +3838,9 @@ function buildCCStateRefresh() {
   if (_refreshCache && now - _refreshCacheTs < REFRESH_TTL) return _refreshCache;
 
   const ts = new Date().toISOString().slice(0, 16);
-  const agents = getAgents().map(a => `- ${a.name}: ${a.status}`).join('\n');
+  // Agent roster carries the current task too — without it a resumed session
+  // sees only that an agent is "busy", never *what* it is busy with.
+  const agents = getAgents().map(a => `- ${a.name}: ${a.status}${a.currentTask ? ' — ' + a.currentTask.slice(0, 50) : ''}`).join('\n');
   const projects = PROJECTS.map(p => `- ${p.name} (${p.repo || p.localPath})`).join('\n');
 
   // MCP servers — just names + source for orientation
@@ -3852,6 +3854,31 @@ function buildCCStateRefresh() {
     }
   } catch { /* optional */ }
 
+  // Live operational slice — the "what's going on right now" that resumed
+  // sessions previously never saw (W-mpeap2ug only refreshed projects/MCP/
+  // agent-status). Without this, a non-expiring CC/doc-chat session goes stale
+  // on the active dispatch queue, pending depth, and PR/WI counts the moment
+  // the opening preamble scrolls out of the model's recency window. Capped so
+  // the refresh stays under the 2KB ceiling the unit tests enforce.
+  let activitySection = '';
+  try {
+    const dq = getDispatchQueue();
+    const activeArr = dq.active || [];
+    const maxActive = 10;
+    const activeLines = activeArr.slice(0, maxActive)
+      .map(d => `- ${d.agentName || d.agent}: ${(d.task || '').slice(0, 50)}`).join('\n');
+    const overflow = activeArr.length > maxActive ? `\n…and ${activeArr.length - maxActive} more` : '';
+    const active = activeLines ? activeLines + overflow : '(none)';
+    const pending = (dq.pending || []).length;
+    const prCount = getPullRequests().length;
+    const wiCount = getWorkItems().length;
+    activitySection = `
+
+**Active dispatch:**
+${active}
+Queue: ${pending} pending | PRs: ${prCount} | Work items: ${wiCount}`;
+  } catch { /* optional — never let a refresh failure break the turn */ }
+
   const result = `### State Refresh (${ts})
 
 **Projects:** ${PROJECTS.length}
@@ -3860,7 +3887,7 @@ ${projects || '(none)'}
 **MCP Tools:** ${mcpLine}
 
 **Agents:**
-${agents || '(none)'}`;
+${agents || '(none)'}${activitySection}`;
 
   _refreshCache = result;
   _refreshCacheTs = now;

package/docs/distribution.md

@@ -60,7 +60,7 @@ These files are excluded from published packages:
 Controlled by the `files` field in `package.json`:
 - `bin/minions.js` — CLI entry point
 - `engine.js`, `dashboard.js`, `dashboard/` (fragments), `minions.js` — core scripts
-- `engine/spawn-agent.js`, `engine/ado-mcp-wrapper.js` — engine helpers
+- `engine/spawn-agent.js` — engine helper
 - `agents/*/charter.md` — agent role definitions
 - `playbooks/*.md` — task templates
 - `config.template.json` — starter config

package/docs/onboarding.md

@@ -15,7 +15,7 @@ You only need to clone if you intend to modify Minions itself. End users normall
 ```bash
 git clone https://github.com/yemi33/minions.git ~/minions-dev
 cd ~/minions-dev
-npm install     # installs dev tooling (Playwright) + the lone runtime dep (@azure-devops/mcp); engine is otherwise built on Node built-ins
+npm install     # installs dev tooling (Playwright, ESLint) only; the engine itself has zero runtime deps (Node built-ins)
 ```
 
 You should now have:

package/engine/playbook.js

@@ -40,7 +40,7 @@ function getPrCreateInstructions(project) {
       `- Use --head to specify your feature branch name\n` +
       `- Include a meaningful --title and body file describing the changes`;
   }
-  // Default: Azure DevOps — prefer `az` CLI first, ADO MCP only as fallback
+  // Default: Azure DevOps — prefer `az` CLI first, ADO REST API as fallback
   const adoOrg = project?.adoOrg || '';
   const adoProject = project?.adoProject || '';
   const repoName = project?.repoName || '';
@@ -50,7 +50,7 @@ function getPrCreateInstructions(project) {
     `- Then: \`az repos pr create --repository "${repoName}" --source-branch <your-branch> --target-branch ${mainBranch} --title "PR title" --description @<body-file.md>\`\n` +
     `- Use \`@<file>\` syntax for \`--description\` so Markdown, quotes, and newlines pass safely\n` +
     `- Always set --target-branch to \`${mainBranch}\` (the main branch)\n\n` +
-    `If \`az\` is unavailable or insufficient for this operation, fall back to \`mcp__azure-ado__repo_create_pull_request\` with repositoryId \`${repoId}\`. Do not use \`gh\` for Azure DevOps repositories.`;
+    `If \`az\` is unavailable, fall back to the ADO REST API: get a token with \`az account get-access-token --resource 499b84ac-1321-427f-aa17-267ca6975798 --query accessToken -o tsv\` and \`POST .../_apis/git/repositories/${repoId}/pullrequests?api-version=7.1\` (Bearer auth). Do not use \`gh\` for Azure DevOps repositories.`;
 }
 
 function getPrCommentInstructions(project) {
@@ -72,12 +72,12 @@ function getPrCommentInstructions(project) {
       "```\n" +
       `Then run: \`gh pr comment <number> --body-file <body-file.md> --repo ${org}/${repo}\`. Without the marker, the engine cannot tell your post from a real human comment and will queue redundant fix-dispatches.`;
   }
-  // Azure DevOps — prefer `az` CLI first, ADO MCP only as fallback
+  // Azure DevOps — prefer `az` CLI first, ADO REST API as fallback
   const repoName = project?.repoName || '';
   return `For Azure DevOps, use the \`az\` CLI first to post a comment on the PR:\n` +
     `- Write the Markdown comment to a temporary file, then run: \`az repos pr comment create --pull-request-id <number> --content @<body-file.md>\` (substitute your project's repo \`${repoName}\` if not using \`az devops configure\` defaults)\n` +
     `- Use \`@<file>\` syntax for \`--content\` so Markdown, quotes, and newlines pass safely\n\n` +
-    `If \`az repos pr comment\` is unavailable or insufficient (e.g. older az-devops extension, thread/status semantics needed), fall back to \`mcp__azure-ado__repo_create_pull_request_thread\` with repositoryId \`${repoId}\`. Do not use \`gh\` for Azure DevOps repositories.`;
+    `If \`az repos pr comment\` is unavailable or insufficient (e.g. older az-devops extension, thread/status semantics needed), fall back to the ADO REST API: get a token with \`az account get-access-token --resource 499b84ac-1321-427f-aa17-267ca6975798 --query accessToken -o tsv\` and \`POST .../_apis/git/repositories/${repoId}/pullrequests/<number>/threads?api-version=7.1\` with body \`{"comments":[{"content":"..."}],"status":"active"}\` (Bearer auth). Do not use \`gh\` for Azure DevOps repositories.`;
 }
 
 function getPrFetchInstructions(project) {
@@ -95,13 +95,13 @@ function getPrFetchInstructions(project) {
       `- Or use \`gh pr checkout <number> --repo ${org}/${repo}\` to fetch and checkout in one step\n` +
       `- The base branch is \`${mainBranch}\``;
   }
-  // Azure DevOps — prefer `az` CLI first, ADO MCP only as fallback
+  // Azure DevOps — prefer `az` CLI first, ADO REST API as fallback
   const mainBranch = project?.localPath ? shared.resolveMainBranch(project.localPath, project.mainBranch) : (project?.mainBranch || 'main');
   return `For Azure DevOps, use the \`az\` CLI first to fetch PR status:\n` +
     `- \`az repos pr show --id <number>\` returns PR state, mergeStatus, source/target branches, vote summary, and policy/build evaluations\n` +
     `- For the local branch: \`git fetch origin <branch-name>\` then inspect via \`git show\`/\`git diff\` (do NOT checkout in your main working tree)\n` +
     `- The base branch is \`${mainBranch}\`\n\n` +
-    `If \`az\` is unavailable or insufficient, fall back to \`mcp__azure-ado__repo_get_pull_request_by_id\`. Do not use \`gh\` for Azure DevOps repositories.`;
+    `If \`az\` is unavailable, fall back to the ADO REST API: get a token with \`az account get-access-token --resource 499b84ac-1321-427f-aa17-267ca6975798 --query accessToken -o tsv\` and \`GET .../_apis/git/repositories/<repoId>/pullrequests/<number>?api-version=7.1\` (Bearer auth). Do not use \`gh\` for Azure DevOps repositories.`;
 }
 
 function getPrVoteInstructions(project) {
@@ -123,11 +123,11 @@ function getPrVoteInstructions(project) {
       "```\n" +
       `Then run: \`gh pr review <number> --comment --body-file <verdict.md> --repo ${org}/${repo}\`. Do NOT use \`--approve\` or \`--request-changes\` flags — they will fail.`;
   }
-  // Azure DevOps — prefer `az` CLI first, ADO MCP only as fallback
+  // Azure DevOps — prefer `az` CLI first, ADO REST API as fallback
   return `For Azure DevOps, use the \`az\` CLI first to set your reviewer vote:\n` +
     `- \`az repos pr set-vote --id <number> --vote {approve | approve-with-suggestions | reject | reset | wait-for-author}\`\n` +
     `- Pair the vote with \`az repos pr comment create --pull-request-id <number> --content @<verdict.md>\` so the verdict body is recorded as a thread comment\n\n` +
-    `If \`az\` is unavailable or insufficient, fall back to \`mcp__azure-ado__repo_update_pull_request_reviewers\` with repositoryId \`${repoId}\` (vote integers: 10=approve, 5=approve-with-suggestions, 0=no-vote, -5=wait-for-author, -10=reject). Do not use \`gh\` for Azure DevOps repositories.`;
+    `If \`az\` is unavailable, fall back to the ADO REST API: get a token with \`az account get-access-token --resource 499b84ac-1321-427f-aa17-267ca6975798 --query accessToken -o tsv\` and \`PUT .../_apis/git/repositories/${repoId}/pullrequests/<number>/reviewers/<reviewerId>?api-version=7.1\` with body \`{"vote":10}\` (10=approve, 5=approve-with-suggestions, 0=no-vote, -5=wait-for-author, -10=reject; Bearer auth). Do not use \`gh\` for Azure DevOps repositories.`;
 }
 
 function getRepoHostLabel(project) {
@@ -139,7 +139,7 @@ function getRepoHostLabel(project) {
 function getRepoHostToolRule(project) {
   const host = getRepoHost(project);
   if (host === 'github') return 'Use GitHub MCP tools or `gh` CLI for PR operations';
-  return 'For Azure DevOps, use the `az` CLI first for PR operations (e.g. `az repos pr create`, `az repos pr show`, `az repos pr comment`, `az repos pr set-vote`); use ADO MCP tools (`mcp__azure-ado__*`) only as a fallback when `az` is unavailable or insufficient. Do not use `gh` for Azure DevOps repositories.';
+  return 'For Azure DevOps, use the `az` CLI for PR operations (e.g. `az repos pr create`, `az repos pr show`, `az repos pr comment`, `az repos pr set-vote`); if `az` is unavailable or insufficient, fall back to the ADO REST API (`_apis/git/...`) with a token from `az account get-access-token --resource 499b84ac-1321-427f-aa17-267ca6975798`. Do not use `gh` for Azure DevOps repositories.';
 }
 
 // ─── Task Context Resolution ────────────────────────────────────────────────

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.2185",
+  "version": "0.1.2187",
   "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"
@@ -67,9 +67,6 @@
   "engines": {
     "node": ">=22.5"
   },
-  "dependencies": {
-    "@azure-devops/mcp": "2.7.0"
-  },
   "devDependencies": {
     "@playwright/test": "^1.58.2",
     "eslint": "^9.39.4",

package/playbooks/shared-rules.md

@@ -274,5 +274,5 @@ Output is JSON with the same fields. Exit 0 on success, 1 if not found.
 
 For Azure DevOps repo operations, use the `az` CLI first. Prefer commands such as `az repos pr create`, `az repos pr show`, `az repos pr list`, `az repos pr comment`, `az repos pr reviewer`, `az boards work-item`, and `az pipelines` after setting defaults with `az devops configure`.
 
-Use ADO MCP fallback tools (`mcp__azure-ado__*`) only when `az` is unavailable in the environment or insufficient for a specific operation. Do not choose MCP first just because it exists, and do not use `gh` for Azure DevOps repositories.
+If `az` is unavailable or insufficient for a specific operation (e.g. PR comment threads on an older az-devops extension), fall back to the ADO REST API directly: acquire a token with `az account get-access-token --resource 499b84ac-1321-427f-aa17-267ca6975798 --query accessToken -o tsv` and call the `_apis/git/...` endpoints with Bearer auth. Do not use `gh` for Azure DevOps repositories.
 {{/ado_shared_rules}}

package/engine/ado-mcp-wrapper.js

@@ -1,63 +0,0 @@
-#!/usr/bin/env node
-/**
- * Wrapper for @azure-devops/mcp that fetches an ADO token via the shared
- * az-first provider chain and sets AZURE_DEVOPS_EXT_PAT before launching the
- * MCP server.
- *
- * P-b3f8e1a5: @azure-devops/mcp is pinned in package.json and resolved from
- * local node_modules. We spawn process.execPath against the resolved bin file
- * instead of going through npx/npx.cmd. This (a) eliminates the per-cold-start
- * network fetch that ran with AZURE_DEVOPS_EXT_PAT in env, and (b) avoids the
- * Windows .cmd shim chain that crashes the runtime under spawn.
- */
-const { spawn } = require('child_process');
-const fs = require('fs');
-const path = require('path');
-const { acquireAdoTokenSync } = require('./ado-token');
-
-const PKG_NAME = '@azure-devops/mcp';
-const BIN_NAME = 'mcp-server-azuredevops';
-
-function resolveMcpBin() {
-  const pkgJsonPath = require.resolve(`${PKG_NAME}/package.json`);
-  const pkgJson = JSON.parse(fs.readFileSync(pkgJsonPath, 'utf8'));
-  const binField = pkgJson.bin;
-  const relBin = typeof binField === 'string' ? binField : binField && binField[BIN_NAME];
-  if (!relBin) {
-    throw new Error(`${PKG_NAME} package.json is missing bin entry "${BIN_NAME}"`);
-  }
-  return path.resolve(path.dirname(pkgJsonPath), relBin);
-}
-
-let token;
-try {
-  token = acquireAdoTokenSync().token;
-} catch (e) {
-  process.stderr.write('ado-mcp-wrapper: ADO auth failed: ' + e.message + '\n');
-  process.stderr.write('ado-mcp-wrapper: Run "az login" or refresh azureauth manually, then retry\n');
-  process.exit(1);
-}
-
-let binPath;
-try {
-  binPath = resolveMcpBin();
-} catch (e) {
-  process.stderr.write('ado-mcp-wrapper: failed to resolve ' + PKG_NAME + ': ' + e.message + '\n');
-  process.stderr.write('ado-mcp-wrapper: run "npm install" in the minions checkout to restore the pinned dependency\n');
-  process.exit(1);
-}
-
-const args = process.argv.slice(2);
-const child = spawn(process.execPath, [binPath, ...args], {
-  stdio: 'inherit',
-  env: { ...process.env, AZURE_DEVOPS_EXT_PAT: token, AZURE_DEVOPS_EXT_AZURE_RM_PAT: token },
-  windowsHide: true,
-  shell: false,
-});
-
-child.on('exit', (code) => process.exit(code || 0));
-child.on('error', (err) => {
-  process.stderr.write('ado-mcp-wrapper: ' + err.message + '\n');
-  process.exit(1);
-});
-