fix: disable periodic openclaw agents list subprocess
Spawning a full openclaw CLI process every 30s to list agents is too heavy — each invocation loads all plugins (~16s) and hangs until killed. Return empty array for now until a lighter mechanism is available.
This commit is contained in:
@@ -1,8 +1,3 @@
|
|||||||
import { execFile } from 'child_process';
|
|
||||||
import { promisify } from 'util';
|
|
||||||
|
|
||||||
const execFileAsync = promisify(execFile);
|
|
||||||
|
|
||||||
export interface OpenClawAgentInfo {
|
export interface OpenClawAgentInfo {
|
||||||
name: string;
|
name: string;
|
||||||
isDefault?: boolean;
|
isDefault?: boolean;
|
||||||
@@ -14,70 +9,38 @@ export interface OpenClawAgentInfo {
|
|||||||
routing?: string;
|
routing?: string;
|
||||||
}
|
}
|
||||||
|
|
||||||
export async function listOpenClawAgents(logger?: { debug?: (...args: any[]) => void; warn?: (...args: any[]) => void }): Promise<OpenClawAgentInfo[]> {
|
export async function listOpenClawAgents(_logger?: { debug?: (...args: any[]) => void; warn?: (...args: any[]) => void }): Promise<OpenClawAgentInfo[]> {
|
||||||
try {
|
return [];
|
||||||
const { stdout } = await execFileAsync('openclaw', ['agents', 'list'], {
|
|
||||||
timeout: 30000,
|
|
||||||
maxBuffer: 1024 * 1024,
|
|
||||||
});
|
|
||||||
return parseOpenClawAgents(stdout);
|
|
||||||
} catch (err) {
|
|
||||||
logger?.warn?.('Failed to run `openclaw agents list`', err);
|
|
||||||
return [];
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
export function parseOpenClawAgents(text: string): OpenClawAgentInfo[] {
|
export function parseOpenClawAgents(text: string): OpenClawAgentInfo[] {
|
||||||
const lines = text.split(/\r?\n/);
|
const lines = text.split(/\r?\n/);
|
||||||
const out: OpenClawAgentInfo[] = [];
|
const out: OpenClawAgentInfo[] = [];
|
||||||
let current: OpenClawAgentInfo | null = null;
|
let current: OpenClawAgentInfo | null = null;
|
||||||
|
const push = () => { if (current) out.push(current); current = null; };
|
||||||
const push = () => {
|
|
||||||
if (current) out.push(current);
|
|
||||||
current = null;
|
|
||||||
};
|
|
||||||
|
|
||||||
for (const raw of lines) {
|
for (const raw of lines) {
|
||||||
const line = raw.trimEnd();
|
const line = raw.trimEnd();
|
||||||
if (!line.trim() || line.startsWith('Agents:') || line.startsWith('Routing rules map') || line.startsWith('Channel status reflects')) continue;
|
if (!line.trim() || line.startsWith("Agents:") || line.startsWith("Routing rules map") || line.startsWith("Channel status reflects")) continue;
|
||||||
if (line.startsWith('- ')) {
|
if (line.startsWith("- ")) {
|
||||||
push();
|
push();
|
||||||
const m = line.match(/^-\s+(.+?)(?:\s+\((default)\))?$/);
|
const m = line.match(/^-\s+(.+?)(?:\s+\((default)\))?$/);
|
||||||
current = {
|
current = { name: m?.[1] || line.slice(2).trim(), isDefault: m?.[2] === "default" };
|
||||||
name: m?.[1] || line.slice(2).trim(),
|
|
||||||
isDefault: m?.[2] === 'default',
|
|
||||||
};
|
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
if (!current) continue;
|
if (!current) continue;
|
||||||
const trimmed = line.trim();
|
const trimmed = line.trim();
|
||||||
const idx = trimmed.indexOf(':');
|
const idx = trimmed.indexOf(":");
|
||||||
if (idx === -1) continue;
|
if (idx === -1) continue;
|
||||||
const key = trimmed.slice(0, idx).trim();
|
const key = trimmed.slice(0, idx).trim();
|
||||||
const value = trimmed.slice(idx + 1).trim();
|
const value = trimmed.slice(idx + 1).trim();
|
||||||
switch (key) {
|
switch (key) {
|
||||||
case 'Identity':
|
case "Identity": current.identity = value; break;
|
||||||
current.identity = value;
|
case "Workspace": current.workspace = value; break;
|
||||||
break;
|
case "Agent dir": current.agentDir = value; break;
|
||||||
case 'Workspace':
|
case "Model": current.model = value; break;
|
||||||
current.workspace = value;
|
case "Routing rules": { const n = Number(value); current.routingRules = Number.isFinite(n) ? n : undefined; break; }
|
||||||
break;
|
case "Routing": current.routing = value; break;
|
||||||
case 'Agent dir':
|
default: break;
|
||||||
current.agentDir = value;
|
|
||||||
break;
|
|
||||||
case 'Model':
|
|
||||||
current.model = value;
|
|
||||||
break;
|
|
||||||
case 'Routing rules': {
|
|
||||||
const n = Number(value);
|
|
||||||
current.routingRules = Number.isFinite(n) ? n : undefined;
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
case 'Routing':
|
|
||||||
current.routing = value;
|
|
||||||
break;
|
|
||||||
default:
|
|
||||||
break;
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
push();
|
push();
|
||||||
|
|||||||
Reference in New Issue
Block a user