Ayuriel/openclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(cli): align nodes list and status to show consistent paired node count (#50847)

Browse Source 
- Always call node.list API to get paired status
- Use node.list paired nodes as fallback when node.pair.list returns empty
- Fixes inconsistent display between 'nodes status' and 'nodes list' commands

Fixes #50847
This commit is contained in:
Yaohua-Leo 2026-03-20 20:06:30 +08:00
parent 4c60956d8e
commit 86c92cb4ce
1 changed files with 18 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
src/cli/nodes-cli/register.status.ts
View File

@ -304,30 +304,28 @@ export function registerNodesStatusCommands(nodes: Command) {
await runNodesCommand("list", async () => {
const connectedOnly = Boolean(opts.connected);
const sinceMs = parseSinceMs(opts.lastConnected, "Invalid --last-connected");
const result = await callGatewayCli("node.pair.list", opts, {});
const { pending, paired } = parsePairingList(result);
const nodeListResult = await callGatewayCli("node.list", opts, {});
const pairingListResult = await callGatewayCli("node.pair.list", opts, {});
const { pending, paired } = parsePairingList(pairingListResult);
const { heading, muted, warn } = getNodesTheme();
const tableWidth = getTerminalTableWidth();
const now = Date.now();
const hasFilters = connectedOnly || sinceMs !== undefined;
const pendingRows = hasFilters ? [] : pending;
const connectedById = hasFilters
? new Map(
parseNodeList(await callGatewayCli("node.list", opts, {})).map((node) => [
node.nodeId,
node,
]),
)
: null;
const liveNodesById = new Map(
parseNodeList(nodeListResult).map((node) => [node.nodeId, node]),
);
const pendingRows = connectedOnly || sinceMs !== undefined ? [] : pending;
const filteredPaired = paired.filter((node) => {
if (connectedOnly) {
const live = connectedById?.get(node.nodeId);
const live = liveNodesById.get(node.nodeId);
if (!live?.connected) {
return false;
}
}
if (sinceMs !== undefined) {
const live = connectedById?.get(node.nodeId);
const live = liveNodesById.get(node.nodeId);
const lastConnectedAtMs =
typeof node.lastConnectedAtMs === "number"
? node.lastConnectedAtMs
@ -343,8 +341,12 @@ export function registerNodesStatusCommands(nodes: Command) {
}
return true;
});
const liveNodes = parseNodeList(nodeListResult);
const livePairedNodes = liveNodes.filter((n) => n.paired);
const totalPairedCount = paired.length > 0 ? paired.length : livePairedNodes.length;
const filteredLabel =
hasFilters && filteredPaired.length !== paired.length ? ` (of ${paired.length})` : "";
filteredPaired.length !== totalPairedCount ? ` (of ${totalPairedCount})` : "";
defaultRuntime.log(
`Pending: ${pendingRows.length} · Paired: ${filteredPaired.length}${filteredLabel}`,
);
@ -370,7 +372,7 @@ export function registerNodesStatusCommands(nodes: Command) {
if (filteredPaired.length > 0) {
const pairedRows = filteredPaired.map((n) => {
const live = connectedById?.get(n.nodeId);
const live = liveNodesById.get(n.nodeId);
const lastConnectedAtMs =
typeof n.lastConnectedAtMs === "number"
? n.lastConnectedAtMs