diff options
| author | Agando <roduccomm@gmail.com> | 2026-10-23 18:22:30 +0200 |
|---|---|---|
| committer | Agando <roduccomm@gmail.com> | 2026-10-23 18:22:30 +0200 |
| commit | 2bc11203b73fe2eb210ce3dd643773e625609c25 (patch) | |
| tree | 60222e83add1a2fd05fa705527ff7dc4ce1efea4 /modules/core/module-orchestrator.js | |
| parent | f3bbb69b25d1b1619f7bd9d5861480af7e9e834f (diff) | |
feat(newos4): agent status says never-scheduled vs waiting; `guides` command (v4.6.63)
Packet 6 and 7.
6 — four agents showed `sleeping` and nothing distinguished "waiting for work" from
"never handed a task". Added tasksAssigned/lastTaskAt to the agent record (counted
where a task is actually assigned), and the status line now reads
idle — NEVER SCHEDULED (0 tasks since 2026-10-23 16:22)
idle — waiting for work (3 task(s), last 2026-10-23 16:22)
parked — NEVER SCHEDULED ...
plus "Dispatch: 0/3 agent(s) have ever been given a task" and a warning that
spawning is not dispatching.
7 — /guides/ was reachable by nobody. New `guides` command lists it newest-first
with sizes and the preview/cat hints; help.html gained a Guides nav entry and
section with links to the published guides.
Verified live on v4.6.63: `guides` → 2 entries after seeding two files;
orchestrator-status shows 3x NEVER SCHEDULED on a fresh session (the packet's own
observation), and "waiting for work (3 task(s) ...)" once a counter is set, with
"Dispatch: 1/3".
Diffstat (limited to 'modules/core/module-orchestrator.js')
| -rw-r--r-- | modules/core/module-orchestrator.js | 38 |
1 files changed, 34 insertions, 4 deletions
diff --git a/modules/core/module-orchestrator.js b/modules/core/module-orchestrator.js index ad9f020..d03f7eb 100644 --- a/modules/core/module-orchestrator.js +++ b/modules/core/module-orchestrator.js @@ -369,7 +369,11 @@ class OrchestratorModule { restartPolicy: this._RESTART_POLICY, failureCount: 0, restartCount: 0, - nextRestartAt: null + nextRestartAt: null, + // v4.6.63: "sleeping" alone could not tell an agent WAITING for work from one + // that was never handed a task (packet §6: 4 registered, 0 executing). + tasksAssigned: 0, + lastTaskAt: null }; this._agents[id] = agent; // Create VFS agent directory with status @@ -1196,6 +1200,8 @@ class OrchestratorModule { agent.currentTask = task; agent.leaseId = lease?.id; agent.lastHeartbeat = Date.now(); + agent.tasksAssigned = (agent.tasksAssigned || 0) + 1; + agent.lastTaskAt = new Date().toISOString(); // Send A2A request via inbox this.sendA2AMessage([agent.id, JSON.stringify({ @@ -1414,12 +1420,36 @@ class OrchestratorModule { this.terminal.print(` Default restart: ${this._RESTART_POLICY}`, 'info'); this.terminal.print(` Default agent model: ${this._AGENT_DEFAULT_MODEL || 'none (uses main AI model)'}`, 'info'); this.terminal.print(` A2A Messages: ${a2aTotal} (${a2aPending} pending)`, 'info'); - Object.values(this._agents).forEach(a => { - const task = a.currentTask ? ` (task: ${a.currentTask.substring(0, 40)})` : ''; + // v4.6.63: say what the state MEANS. A parked agent with 0 assignments and a parked + // agent between tasks looked identical from outside (packet §6). + const agentsArr = Object.values(this._agents); + const everScheduled = agentsArr.filter(a => (a.tasksAssigned || 0) > 0).length; + const describe = (a) => { + const n = a.tasksAssigned || 0; + const since = a.createdAt ? String(a.createdAt).slice(0, 16).replace('T', ' ') : '?'; + const last = a.lastTaskAt ? String(a.lastTaskAt).slice(0, 16).replace('T', ' ') : null; + if (a.status === 'busy') { + return 'busy' + (a.currentTask ? ' — ' + String(a.currentTask).substring(0, 40) : ''); + } + if (a.status === 'failed' || a.status === 'restarting') { + return a.status + ` — ${a.failureCount || 0} failure(s), ${n} task(s) ever`; + } + if (n === 0) { + return (a.status === 'sleeping' ? 'parked' : a.status) + + ` — NEVER SCHEDULED (0 tasks since ${since})`; + } + return (a.status === 'sleeping' ? 'parked' : a.status) + + ` — waiting for work (${n} task(s), last ${last || '?'})`; + }; + agentsArr.forEach(a => { const modelInfo = a.model ? ` [model: ${a.model}]` : ''; const backoff = a.nextRestartAt && a.nextRestartAt > Date.now() ? ` [restart in ${Math.round((a.nextRestartAt - Date.now()) / 1000)}s]` : ''; - this.terminal.print(` ${a.id} [${a.role}] — ${a.status}${modelInfo}${task}${backoff}`, 'info'); + this.terminal.print(` ${a.id} [${a.role}] — ${describe(a)}${modelInfo}${backoff}`, 'info'); }); + this.terminal.print(` Dispatch: ${everScheduled}/${total} agent(s) have ever been given a task`, everScheduled === 0 && total > 0 ? 'warning' : 'info'); + if (everScheduled === 0 && total > 0) { + this.terminal.print(' ⚠️ No agent has ever been scheduled — spawning is not dispatching. Use orchestrate <task> or orchestrator-spawn <role> <task>.', 'warning'); + } } listAgents() { |