From 1d96c4d534b3687d90e1ed8383b3167c30d257f4 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Mon, 9 Feb 2026 06:39:27 +0000
Subject: [PATCH] fix(swarm): restore getSwarmNodes original error behavior

getSwarmNodes was changed to return [] on error, but the existing
SwarmMonitorCard checks `if (!nodes)` to detect failures. Since ![]
is false, the error state was silently skipped, breaking the Overview
tab for users without Docker Swarm initialized.

Reverted to return undefined on error (original behavior) so the
existing Overview tab error handling continues to work. The Containers
tab already handles nodes === undefined explicitly.
---
 packages/server/src/services/docker.ts | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/packages/server/src/services/docker.ts b/packages/server/src/services/docker.ts
index ef36ec91c..2fec21ac3 100644
--- a/packages/server/src/services/docker.ts
+++ b/packages/server/src/services/docker.ts
@@ -371,11 +371,7 @@ export const getSwarmNodes = async (serverId?: string) => {
 
 		if (stderr) {
 			console.error(`Error: ${stderr}`);
-			return [];
-		}
-
-		if (!stdout.trim()) {
-			return [];
+			return;
 		}
 
 		const nodesArray = stdout
@@ -385,7 +381,6 @@ export const getSwarmNodes = async (serverId?: string) => {
 		return nodesArray;
 	} catch (error) {
 		console.error("getSwarmNodes error:", error);
-		return [];
 	}
 };