agent-pool-mcp 1.7.2 → 1.7.3

package/README.md

@@ -183,12 +183,20 @@ See [parallel-work guide](examples/parallel-work.md) and built-in `orchestrator`
 
 ## MCP Ecosystem
 
-Best used together with [**project-graph-mcp**](https://www.npmjs.com/package/project-graph-mcp) — AST-based codebase analysis:
+Best used as part of [**mcp-agent-portal**](https://github.com/rnd-pro/mcp-agent-portal) — a unified MCP aggregator that bundles agent-pool, project-graph, and other servers behind a single config entry:
 
-| Layer | agent-pool-mcp | project-graph-mcp |
-|-------|---------------|-------------------|
-| **Primary IDE agent** | Delegates tasks, consults peer | Navigates codebase, runs analysis |
-| **Gemini CLI workers** | Executes delegated tasks | Available as MCP tool inside workers |
+```json
+{
+  "mcpServers": {
+    "agent-portal": {
+      "command": "npx",
+      "args": ["-y", "mcp-agent-portal"]
+    }
+  }
+}
+```
+
+Also works standalone alongside [**project-graph-mcp**](https://www.npmjs.com/package/project-graph-mcp) — AST-based codebase analysis:
 
 ```json
 {
@@ -214,6 +222,7 @@ Best used together with [**project-graph-mcp**](https://www.npmjs.com/package/pr
 - [examples/parallel-work.md](examples/parallel-work.md) — Delegation patterns and best practices
 
 ## Related Projects
+- [mcp-agent-portal](https://github.com/rnd-pro/mcp-agent-portal) — Unified MCP aggregator + web dashboard + AI agent runtime
 - [project-graph-mcp](https://github.com/rnd-pro/project-graph-mcp) — AST-based codebase analysis for AI agents
 - [Symbiote.js](https://github.com/symbiotejs/symbiote.js) — Isomorphic Reactive Web Components framework
 - [JSDA-Kit](https://github.com/rnd-pro/jsda-kit) — SSG/SSR toolkit for modern web applications

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-pool-mcp",
-  "version": "1.7.2",
+  "version": "1.7.3",
   "type": "module",
   "description": "MCP Server for multi-agent task delegation and orchestration via Gemini CLI",
   "main": "index.js",

package/src/runner/gemini-runner.js

@@ -10,7 +10,7 @@ import { spawn, execFile } from 'node:child_process';
 import { trackChild, killGroup, untrackChild } from './process-manager.js';
 import { getRunner, loadConfig } from './config.js';
 import { buildSshSpawn, parseRemotePid } from './ssh.js';
-import { setTaskPid, updateTaskResult, pushTaskEvent } from '../tools/results.js';
+import { setTaskPid, updateTaskResult, pushTaskEvent, pushTaskStderr } from '../tools/results.js';
 
 const DEFAULT_TIMEOUT_SEC = 600;
 const DEFAULT_APPROVAL_MODE = 'yolo';
@@ -160,6 +160,7 @@ export function runGeminiStreaming({ prompt, cwd, model, approvalMode, timeout,
 
     child.stderr.on('data', (chunk) => {
       stderrData += chunk.toString();
+      if (taskId) pushTaskStderr(taskId, chunk.toString());
     });
 
     child.on('close', (code) => {

package/src/tools/results.js

@@ -77,6 +77,8 @@ export function createTask(taskId, prompt, waitHint, approvalMode) {
     waitHint: waitHint ?? null,
     pid: null,
     liveEvents: [],
+    lastEventAt: null,
+    stderr: '',
   });
 }
 
@@ -90,6 +92,7 @@ export function pushTaskEvent(taskId, event) {
   const entry = taskStore.get(taskId);
   if (entry && entry.status === 'running') {
     entry.liveEvents.push(event);
+    entry.lastEventAt = Date.now();
     // Ring buffer: keep only the last MAX_LIVE_EVENTS
     if (entry.liveEvents.length > MAX_LIVE_EVENTS) {
       entry.liveEvents = entry.liveEvents.slice(-MAX_LIVE_EVENTS);
@@ -97,6 +100,23 @@ export function pushTaskEvent(taskId, event) {
   }
 }
 
+/**
+ * Append stderr data to a task (for diagnostics).
+ *
+ * @param {string} taskId
+ * @param {string} chunk - stderr chunk
+ */
+export function pushTaskStderr(taskId, chunk) {
+  const entry = taskStore.get(taskId);
+  if (entry && entry.status === 'running') {
+    entry.stderr += chunk;
+    // Keep only last 2KB of stderr
+    if (entry.stderr.length > 2048) {
+      entry.stderr = entry.stderr.slice(-2048);
+    }
+  }
+}
+
 /**
  * Associate a PID with a task (called after spawn).
  *
@@ -316,6 +336,68 @@ export function formatTaskResult(taskId) {
       progress = '\n\n⏳ *Cold start — Gemini CLI initialization takes ~15-20s*';
     }
 
+    // Time since last event — key diagnostic
+    let activityInfo = '';
+    if (entry.lastEventAt) {
+      const silentSec = ((Date.now() - entry.lastEventAt) / 1000).toFixed(0);
+      if (parseInt(silentSec) > 60) {
+        activityInfo = `\n⏱️ Last event ${silentSec}s ago — model may be thinking or rate-limited.`;
+      }
+    }
+
+    // Stderr diagnostics — parse rate limits, extract actionable info
+    let stderrInfo = '';
+    if (entry.stderr) {
+      const raw = entry.stderr;
+
+      // Count rate limit occurrences
+      const rateLimitCount = (raw.match(/429|Too Many Requests|RESOURCE_EXHAUSTED/gi) || []).length;
+
+      // Forward-compatible: extract retry delay if present in stderr.
+      // Currently Google SDK does NOT include retryDelayMs in console.error output,
+      // but if Gemini CLI or SDK adds it in the future, this parser will pick it up.
+      // Supported formats:
+      // retryDelay: '42s'  (Google API RetryInfo)
+      // retryDelayMs: 42000  (Gemini CLI error object)
+      // Retry-After: 30  (HTTP header)
+      let retrySeconds = null;
+      const msMatch = raw.match(/retryDelayMs[:"'\s]*(\d+)/i);
+      const secMatch = raw.match(/retryDelay[:"'\s]*'?(\d+)s'?/i);
+      const headerMatch = raw.match(/Retry-After[:"'\s]*(\d+)/i);
+      if (msMatch) retrySeconds = Math.ceil(parseInt(msMatch[1]) / 1000);
+      else if (secMatch) retrySeconds = parseInt(secMatch[1]);
+      else if (headerMatch) retrySeconds = parseInt(headerMatch[1]);
+
+      if (rateLimitCount > 0) {
+        const parts = [`⚡ Rate limited (429 × ${rateLimitCount})`];
+        if (retrySeconds) {
+          const resetAt = new Date(Date.now() + retrySeconds * 1000);
+          const resetTime = resetAt.toLocaleTimeString('en-US', { hour: '2-digit', minute: '2-digit', hour12: false });
+          parts.push(`resets ~${resetTime} (${retrySeconds}s)`);
+        }
+        stderrInfo = `\n${parts.join(' — ')}`;
+      } else {
+        // Show other errors (non-rate-limit)
+        const lines = raw.trim().split('\n')
+          .filter((l) => !l.includes('IDEClient') && !l.includes('YOLO mode') && !l.includes('Loaded cached') && l.trim())
+          .slice(-3);
+        if (lines.length > 0) {
+          stderrInfo = `\n📋 stderr: ${lines.join(' | ').substring(0, 200)}`;
+        }
+      }
+    }
+
+    // PID alive check
+    let pidStatus = '';
+    if (entry.pid) {
+      try {
+        process.kill(entry.pid, 0); // signal 0 = check alive
+        pidStatus = ' (alive)';
+      } catch {
+        pidStatus = ' (dead ⚠️)';
+      }
+    }
+
     // System load awareness during polling
     const load = getSystemLoad();
     const loadInfo = load.warning ? `\n\n${load.warning}` : '';
@@ -325,7 +407,7 @@ export function formatTaskResult(taskId) {
     return {
       content: [{
         type: 'text',
-        text: `⏳ Task is still running (${elapsed}s elapsed, ${entry.liveEvents.length} events).\n\n- **Prompt**: ${entry.prompt.substring(0, 100)}...\n- **Mode**: ${modeLabel}${progress}\n\n💡 **${hint}**${loadInfo}\n\nCheck again later with \`get_task_result\`.`,
+        text: `⏳ Task is still running (${elapsed}s elapsed, ${entry.liveEvents.length} events).\n\n- **Prompt**: ${entry.prompt.substring(0, 100)}...\n- **Mode**: ${modeLabel}\n- **PID**: ${entry.pid ?? 'unknown'}${pidStatus}${activityInfo}${stderrInfo}${progress}\n\n💡 **${hint}**${loadInfo}\n\nCheck again later with \`get_task_result\`.`,
       }],
     };
   }
@@ -371,6 +453,20 @@ export function formatTaskResult(taskId) {
     if (result.errors?.length > 0) {
       sections.push(`### Errors\n\n${result.errors.join('\n')}`);
     }
+    // Include stderr diagnostics (rate limits, etc.)
+    if (entry.stderr) {
+      const rateLimitCount = (entry.stderr.match(/429|Too Many Requests|RESOURCE_EXHAUSTED/gi) || []).length;
+      if (rateLimitCount > 0) {
+        sections.push(`### Cause: Rate Limit\n\n⚡ **${rateLimitCount} rate limit errors (429)** during execution. Task failed because API quota was exhausted.\n\n💡 Wait a few minutes for quota to reset, then retry.`);
+      } else {
+        const stderrLines = entry.stderr.trim().split('\n')
+          .filter((l) => !l.includes('IDEClient') && !l.includes('YOLO mode') && !l.includes('Loaded cached') && l.trim())
+          .slice(-5);
+        if (stderrLines.length > 0) {
+          sections.push(`### Stderr\n\n\`\`\`\n${stderrLines.join('\n')}\n\`\`\``);
+        }
+      }
+    }
     const errorSignal = (result.errors ?? []).join(' ');
     sections.push(`### Recovery\n\n${classifyError(errorSignal || `exit code ${result.exitCode}`)}`);
   }