@chatpanel/events 0.89.3 → 0.89.4

package/index.js

@@ -212,7 +212,7 @@ export { DEFAULT_MIN_CALLS, cardOverride, applyCard } from './model-candidates.j
 export { SCM_KINDS, validateConnection, normalizeConnection, parseRemote, connectionFor, branchFor, worktreeDirFor, credentialEnv, describeConnection, blankConnection, connectionFromForm } from './scm-connection.js';
 export { messagesFor, mergeTranscript, clipTranscript, clipMessage, newSteps, continuationNote, createControl, STEP_MAX_CHARS, TASK_TRANSCRIPT_MAX_CHARS } from './team-task.js';
 export { runTeam, resumeTeam, dryRunTeam, isModelUnavailable, TeamRunError, RUN_STATUSES } from './team-run.js';
-export { teamToolProvider, teamToolSpec, describeTeamForApproval, TEAM_TOOL_NAME } from './team-tool.js';
+export { teamToolProvider, teamToolSpec, teamToolTimeoutMs, describeTeamForApproval, TEAM_TOOL_NAME } from './team-tool.js';
 export { teamLine, teamLanes } from './team-trail.js';
 export { mcpDispatchProvider, MCP_TOOL_NAME } from './mcp-dispatch.js';
 export { createManifest, ManifestError, SOURCES } from './manifest.js';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chatpanel/events",
-  "version": "0.89.3",
+  "version": "0.89.4",
   "description": "The canonical ChatPanel event-log and capability contracts \u2014 typed durable facts, clock-free deterministic linearization, schema upcasting, and the invariants the replay harness asserts. Pure, dependency-free ESM shared by the ChatPanel extension, gateway and bridge.",
   "type": "module",
   "main": "index.js",

package/team-board.js

@@ -74,7 +74,9 @@ export function parseFindings(text, { role, taskId } = {}) {
 // ── threads, posts, asks ────────────────────────────────────────────────────────────────
 
 export const THREAD_KINDS = Object.freeze(['task', 'ask', 'discussion', 'proposal']);
-export const THREAD_STATUSES = Object.freeze(['open', 'waiting', 'resolved', 'approved', 'rejected']);
+// `failed`: the task behind the thread ended without an answer (every model on the roster
+// tried, or a hard error) — not `resolved`, which read as "done" on the board.
+export const THREAD_STATUSES = Object.freeze(['open', 'waiting', 'resolved', 'failed', 'approved', 'rejected']);
 export const POST_KINDS = Object.freeze(['finding', 'note', 'question', 'answer', 'draft', 'decision']);
 export const POST_STATUSES = Object.freeze(['open', 'proposed', 'approved', 'rejected']);
 export const ASK_TYPES = Object.freeze(['info', 'budget', 'permission', 'direction']);

package/team-run.js

@@ -58,7 +58,11 @@ export function isModelUnavailable(error) {
   // Also: a relayed agent that exited, a provider that closed the stream, and a turn that
   // came back with nothing — the model did not answer, and the next one might. A refusal,
   // a timeout the caller set, a bad request or a budget stop are not this.
-  return /model[_ ]not[_ ]found|not found|not deployed|inaccessible|does not exist|no such model|unknown model|unsupported model|not available|unavailable|no api key|not configured|"status":\s*(404|401|403|500|502|503)\b|\b(404|401|403|502|503)\b|exited \d+|returned no answer|did not answer|closed the connection|couldn't reach|could not reach|ECONNREFUSED|overloaded|capacity/i.test(m);
+  // A server that is not there: Node says ECONNREFUSED, a browser says only "Failed to
+  // fetch" (the extension reports it as "network error") — a local model killed mid-run
+  // arrived as the latter and ended the task on its first attempt with Claude Code sitting
+  // idle on the roster.
+  return /model[_ ]not[_ ]found|not found|not deployed|inaccessible|does not exist|no such model|unknown model|unsupported model|not available|unavailable|no api key|not configured|"status":\s*(404|401|403|500|502|503)\b|\b(404|401|403|502|503)\b|exited \d+|returned no answer|did not answer|closed the connection|couldn't reach|could not reach|ECONNREFUSED|ECONNRESET|ENOTFOUND|EHOSTUNREACH|socket hang up|fetch failed|failed to fetch|load failed|network ?error|overloaded|capacity/i.test(m);
 }
 
 export function dryRunTeam(team, request, { appoint = null } = {}) {
@@ -355,7 +359,13 @@ export async function runTeam({
       const findings = status === 'ok' ? parseFindings(text, { role: role.id, taskId: task.id }) : [];
       if (findings.length) { board.add(findings); for (const f of findings) say('task.finding', { taskId: task.id, role: role.id, finding: f }); }
       const thread = board.threadForTask(task.id);
-      if (thread && status !== 'waiting') board.setThreadStatus(thread.id, 'resolved');
+      // The thread says how the task ended. A failure is posted in it as well — a person reading
+      // the board sees "researcher failed: network error" where it happened, and what was tried.
+      if (thread && status === 'ok') board.setThreadStatus(thread.id, 'resolved');
+      else if (thread && status !== 'waiting') {
+        board.post({ threadId: thread.id, by: RUNNER, kind: 'note', text: `${role.id} ${status === 'over-budget' ? 'stopped at the budget' : 'failed'}${error ? `: ${String(error).slice(0, 300)}` : ''}${attempts.length > 1 ? ` (after ${attempts.length} models: ${attempts.map((a) => a.model).join(', ')})` : ''}.` });
+        board.setThreadStatus(thread.id, 'failed');
+      }
       const row = { id: task.id, role: task.role, title: task.title, status, text, error, usage, ms: now() - t0, findings, transcript: clipTranscript(transcript), attempts, ...(routed ? { routed } : {}), ...(scm ? { scm } : {}), ...(askedAndWaiting ? { waitingOn: askedAndWaiting } : {}) };
       tasksOut.push(row);
       say(status === 'ok' ? 'task.done' : 'task.failed', { taskId: task.id, role: task.role, status, error, ms: row.ms, findings: findings.length, ...(askedAndWaiting ? { threadId: askedAndWaiting } : {}) });

package/team-tool.js

@@ -26,10 +26,23 @@ function catalogue(teams) {
   return `Saved teams: ${list.map((t) => `${t.name} (${(t.roles || []).map((r) => r.id).join(', ')})${t.description ? ` — ${t.description}` : ''}`).join('; ')}.`;
 }
 
+/**
+ * How long one call of the tool may take: the longest budget among the teams plus the merge,
+ * never under two minutes. A relay between a CLI agent and this tool (the bridge's MCP
+ * server) times a call by this; without it a 300 s team hit the relay's 120 s default,
+ * Claude Code was told "tool call timed out" and ran the team AGAIN while the first run was
+ * still working.
+ */
+export function teamToolTimeoutMs(teams) {
+  const longest = Math.max(0, ...(teams || []).map((t) => Number(t?.budget?.ms) || 0));
+  return Math.max(120_000, (longest || 10 * 60_000) + 60_000);
+}
+
 export function teamToolSpec(teams) {
   return {
     name: TEAM_TOOL_NAME,
     annotations: { readOnlyHint: false, destructiveHint: false, idempotentHint: false },
+    timeoutMs: teamToolTimeoutMs(teams),
     description:
       `Saved agent teams — several roles working a request in parallel, merged into one answer. ${catalogue(teams)} `
       + 'Actions: {"action":"run","name":"<team>","request":"<what to do>"} runs one (streams; may take a while); '