@better-openclaw/core 1.0.30 → 1.0.31

package/dist/logger/index.mjs

@@ -1,5 +1,4 @@
 import { OperationsLogger, StepTracker } from "./logger.mjs";
 import { CallbackSink } from "./sinks/callback-sink.mjs";
 import { ConsoleSink } from "./sinks/console-sink.mjs";
-import { FileSink } from "./sinks/file-sink.mjs";
-export { CallbackSink, ConsoleSink, FileSink, OperationsLogger, StepTracker };
+export { CallbackSink, ConsoleSink, OperationsLogger, StepTracker };

package/dist/port-scanner.test.cjs

@@ -0,0 +1,155 @@
+const require_test_CTcmp4Su = require("./test.CTcmp4Su-BWSPM8ZQ.cjs");
+const require_port_scanner = require("./port-scanner.cjs");
+//#region src/port-scanner.test.ts
+/**
+* Tests for port scanner conflict detection and reassignment.
+*
+* We mock isPortAvailable via vi.mock to avoid actual port scanning
+* during tests. The focus is on the conflict detection logic.
+*/
+require_test_CTcmp4Su.vi.mock("node:net", () => {
+	const portsInUse = new Set([
+		80,
+		443,
+		3e3
+	]);
+	return { default: {
+		Socket: class MockSocket {
+			connect(port) {
+				setTimeout(() => {
+					if (portsInUse.has(port)) this.connectCb?.();
+					else this.errorCb?.();
+				}, 1);
+			}
+			once(event, cb) {
+				if (event === "connect") this.connectCb = cb;
+				if (event === "error") this.errorCb = cb;
+				if (event === "timeout") this.timeoutCb = cb;
+			}
+			setTimeout() {}
+			removeAllListeners() {}
+			destroy() {}
+		},
+		createServer() {
+			return {
+				once(event, cb) {
+					if (event === "error") this.errorCb = cb;
+				},
+				listen(_port, _host, cb) {
+					cb();
+				},
+				close(cb) {
+					cb();
+				}
+			};
+		}
+	} };
+});
+function makeService(id, ports) {
+	return {
+		id,
+		name: id,
+		description: `${id} service`,
+		icon: "",
+		category: "test",
+		image: `${id}:latest`,
+		ports,
+		volumes: [],
+		environment: [],
+		dependencies: [],
+		conflicts: [],
+		skills: [],
+		memoryMB: 256,
+		docsUrl: ""
+	};
+}
+require_test_CTcmp4Su.describe("formatPortConflicts", () => {
+	require_test_CTcmp4Su.it("returns empty array when no reassignments", () => {
+		require_test_CTcmp4Su.globalExpect(require_port_scanner.formatPortConflicts([makeService("redis", [{
+			host: 6379,
+			container: 6379,
+			exposed: true,
+			description: "Redis"
+		}])], /* @__PURE__ */ new Map())).toEqual([]);
+	});
+	require_test_CTcmp4Su.it("returns conflicts with suggested ports", () => {
+		const conflicts = require_port_scanner.formatPortConflicts([makeService("redis", [{
+			host: 6379,
+			container: 6379,
+			exposed: true,
+			description: "Redis"
+		}])], new Map([["redis", new Map([[6379, 7379]])]]));
+		require_test_CTcmp4Su.globalExpect(conflicts).toHaveLength(1);
+		require_test_CTcmp4Su.globalExpect(conflicts[0]).toEqual({
+			port: 6379,
+			serviceId: "redis",
+			description: "Redis",
+			suggestedPort: 7379
+		});
+	});
+	require_test_CTcmp4Su.it("handles multiple services with multiple port reassignments", () => {
+		const conflicts = require_port_scanner.formatPortConflicts([makeService("web", [{
+			host: 80,
+			container: 80,
+			exposed: true,
+			description: "HTTP"
+		}, {
+			host: 443,
+			container: 443,
+			exposed: true,
+			description: "HTTPS"
+		}]), makeService("api", [{
+			host: 8080,
+			container: 8080,
+			exposed: true,
+			description: "API"
+		}])], new Map([["web", new Map([[80, 1080], [443, 1443]])]]));
+		require_test_CTcmp4Su.globalExpect(conflicts).toHaveLength(2);
+		require_test_CTcmp4Su.globalExpect(conflicts[0].port).toBe(80);
+		require_test_CTcmp4Su.globalExpect(conflicts[0].suggestedPort).toBe(1080);
+		require_test_CTcmp4Su.globalExpect(conflicts[1].port).toBe(443);
+		require_test_CTcmp4Su.globalExpect(conflicts[1].suggestedPort).toBe(1443);
+	});
+	require_test_CTcmp4Su.it("uses service name as fallback when port has no description", () => {
+		require_test_CTcmp4Su.globalExpect(require_port_scanner.formatPortConflicts([makeService("myapp", [{
+			host: 9e3,
+			container: 9e3,
+			exposed: true,
+			description: ""
+		}])], new Map([["myapp", new Map([[9e3, 1e4]])]]))[0].description).toBe("myapp port");
+	});
+});
+require_test_CTcmp4Su.describe("scanPortConflicts", () => {
+	require_test_CTcmp4Su.it("returns empty map for services with no ports", async () => {
+		require_test_CTcmp4Su.globalExpect((await require_port_scanner.scanPortConflicts([makeService("redis", [])])).size).toBe(0);
+	});
+	require_test_CTcmp4Su.it("returns empty map for services with no exposed ports", async () => {
+		require_test_CTcmp4Su.globalExpect((await require_port_scanner.scanPortConflicts([makeService("redis", [{
+			host: 6379,
+			container: 6379,
+			exposed: false,
+			description: "Redis"
+		}])])).size).toBe(0);
+	});
+	require_test_CTcmp4Su.it("detects inter-service port conflicts", async () => {
+		const result = await require_port_scanner.scanPortConflicts([makeService("web1", [{
+			host: 8080,
+			container: 80,
+			exposed: true,
+			description: "HTTP"
+		}]), makeService("web2", [{
+			host: 8080,
+			container: 80,
+			exposed: true,
+			description: "HTTP"
+		}])]);
+		require_test_CTcmp4Su.globalExpect(result.has("web1")).toBe(false);
+		require_test_CTcmp4Su.globalExpect(result.has("web2")).toBe(true);
+		const web2Reassignments = result.get("web2");
+		require_test_CTcmp4Su.globalExpect(web2Reassignments.has(8080)).toBe(true);
+		require_test_CTcmp4Su.globalExpect(web2Reassignments.get(8080)).toBeGreaterThan(8080);
+	});
+});
+//#endregion
+
+//# sourceMappingURL=port-scanner.test.cjs.map

package/dist/port-scanner.test.cjs.map

@@ -0,0 +1 @@
+{"version":3,"file":"port-scanner.test.cjs","names":["vi","describe","formatPortConflicts","scanPortConflicts"],"sources":["../src/port-scanner.test.ts"],"sourcesContent":["import { describe, expect, it, vi } from \"vitest\";\nimport { formatPortConflicts, scanPortConflicts } from \"./port-scanner.js\";\nimport type { ServiceDefinition } from \"./types.js\";\n\n/**\n * Tests for port scanner conflict detection and reassignment.\n *\n * We mock isPortAvailable via vi.mock to avoid actual port scanning\n * during tests. The focus is on the conflict detection logic.\n */\n\n// Mock the internal isPortAvailable to control port availability\nvi.mock(\"node:net\", () => {\n\tconst portsInUse = new Set([80, 443, 3000]);\n\n\treturn {\n\t\tdefault: {\n\t\t\tSocket: class MockSocket {\n\t\t\t\tconnect(port: number) {\n\t\t\t\t\t// Simulate: ports in portsInUse are \"in use\"\n\t\t\t\t\tsetTimeout(() => {\n\t\t\t\t\t\tif (portsInUse.has(port)) {\n\t\t\t\t\t\t\t(this as any).connectCb?.();\n\t\t\t\t\t\t} else {\n\t\t\t\t\t\t\t(this as any).errorCb?.();\n\t\t\t\t\t\t}\n\t\t\t\t\t}, 1);\n\t\t\t\t}\n\t\t\t\tonce(event: string, cb: () => void) {\n\t\t\t\t\tif (event === \"connect\") (this as any).connectCb = cb;\n\t\t\t\t\tif (event === \"error\") (this as any).errorCb = cb;\n\t\t\t\t\tif (event === \"timeout\") (this as any).timeoutCb = cb;\n\t\t\t\t}\n\t\t\t\tsetTimeout() {}\n\t\t\t\tremoveAllListeners() {}\n\t\t\t\tdestroy() {}\n\t\t\t},\n\t\t\tcreateServer() {\n\t\t\t\treturn {\n\t\t\t\t\tonce(event: string, cb: (err?: Error) => void) {\n\t\t\t\t\t\tif (event === \"error\") (this as any).errorCb = cb;\n\t\t\t\t\t},\n\t\t\t\t\tlisten(_port: number, _host: string, cb: () => void) {\n\t\t\t\t\t\tcb();\n\t\t\t\t\t},\n\t\t\t\t\tclose(cb: () => void) {\n\t\t\t\t\t\tcb();\n\t\t\t\t\t},\n\t\t\t\t};\n\t\t\t},\n\t\t},\n\t};\n});\n\nfunction makeService(\n\tid: string,\n\tports: { host: number; container: number; exposed: boolean; description: string }[],\n): ServiceDefinition {\n\treturn {\n\t\tid,\n\t\tname: id,\n\t\tdescription: `${id} service`,\n\t\ticon: \"\",\n\t\tcategory: \"test\",\n\t\timage: `${id}:latest`,\n\t\tports,\n\t\tvolumes: [],\n\t\tenvironment: [],\n\t\tdependencies: [],\n\t\tconflicts: [],\n\t\tskills: [],\n\t\tmemoryMB: 256,\n\t\tdocsUrl: \"\",\n\t} as ServiceDefinition;\n}\n\ndescribe(\"formatPortConflicts\", () => {\n\tit(\"returns empty array when no reassignments\", () => {\n\t\tconst services = [\n\t\t\tmakeService(\"redis\", [{ host: 6379, container: 6379, exposed: true, description: \"Redis\" }]),\n\t\t];\n\t\tconst reassignments = new Map();\n\t\tconst conflicts = formatPortConflicts(services, reassignments);\n\t\texpect(conflicts).toEqual([]);\n\t});\n\n\tit(\"returns conflicts with suggested ports\", () => {\n\t\tconst services = [\n\t\t\tmakeService(\"redis\", [{ host: 6379, container: 6379, exposed: true, description: \"Redis\" }]),\n\t\t];\n\t\tconst reassignments = new Map([[\"redis\", new Map([[6379, 7379]])]]);\n\t\tconst conflicts = formatPortConflicts(services, reassignments);\n\t\texpect(conflicts).toHaveLength(1);\n\t\texpect(conflicts[0]).toEqual({\n\t\t\tport: 6379,\n\t\t\tserviceId: \"redis\",\n\t\t\tdescription: \"Redis\",\n\t\t\tsuggestedPort: 7379,\n\t\t});\n\t});\n\n\tit(\"handles multiple services with multiple port reassignments\", () => {\n\t\tconst services = [\n\t\t\tmakeService(\"web\", [\n\t\t\t\t{ host: 80, container: 80, exposed: true, description: \"HTTP\" },\n\t\t\t\t{ host: 443, container: 443, exposed: true, description: \"HTTPS\" },\n\t\t\t]),\n\t\t\tmakeService(\"api\", [{ host: 8080, container: 8080, exposed: true, description: \"API\" }]),\n\t\t];\n\t\tconst reassignments = new Map([\n\t\t\t[\n\t\t\t\t\"web\",\n\t\t\t\tnew Map([\n\t\t\t\t\t[80, 1080],\n\t\t\t\t\t[443, 1443],\n\t\t\t\t]),\n\t\t\t],\n\t\t]);\n\t\tconst conflicts = formatPortConflicts(services, reassignments);\n\t\texpect(conflicts).toHaveLength(2);\n\t\texpect(conflicts[0]!.port).toBe(80);\n\t\texpect(conflicts[0]!.suggestedPort).toBe(1080);\n\t\texpect(conflicts[1]!.port).toBe(443);\n\t\texpect(conflicts[1]!.suggestedPort).toBe(1443);\n\t});\n\n\tit(\"uses service name as fallback when port has no description\", () => {\n\t\tconst services = [\n\t\t\tmakeService(\"myapp\", [{ host: 9000, container: 9000, exposed: true, description: \"\" }]),\n\t\t];\n\t\tconst reassignments = new Map([[\"myapp\", new Map([[9000, 10000]])]]);\n\t\tconst conflicts = formatPortConflicts(services, reassignments);\n\t\texpect(conflicts[0]!.description).toBe(\"myapp port\");\n\t});\n});\n\ndescribe(\"scanPortConflicts\", () => {\n\tit(\"returns empty map for services with no ports\", async () => {\n\t\tconst services = [makeService(\"redis\", [])];\n\t\tconst result = await scanPortConflicts(services);\n\t\texpect(result.size).toBe(0);\n\t});\n\n\tit(\"returns empty map for services with no exposed ports\", async () => {\n\t\tconst services = [\n\t\t\tmakeService(\"redis\", [{ host: 6379, container: 6379, exposed: false, description: \"Redis\" }]),\n\t\t];\n\t\tconst result = await scanPortConflicts(services);\n\t\texpect(result.size).toBe(0);\n\t});\n\n\tit(\"detects inter-service port conflicts\", async () => {\n\t\t// Two services claiming the same host port\n\t\tconst services = [\n\t\t\tmakeService(\"web1\", [{ host: 8080, container: 80, exposed: true, description: \"HTTP\" }]),\n\t\t\tmakeService(\"web2\", [{ host: 8080, container: 80, exposed: true, description: \"HTTP\" }]),\n\t\t];\n\t\tconst result = await scanPortConflicts(services);\n\n\t\t// First service wins, second gets reassigned\n\t\texpect(result.has(\"web1\")).toBe(false); // web1 keeps its port\n\t\texpect(result.has(\"web2\")).toBe(true); // web2 gets reassigned\n\t\tconst web2Reassignments = result.get(\"web2\")!;\n\t\texpect(web2Reassignments.has(8080)).toBe(true);\n\t\texpect(web2Reassignments.get(8080)).toBeGreaterThan(8080);\n\t});\n});\n"],"mappings":";;;;;;;;;AAYAA,sBAAAA,GAAG,KAAK,kBAAkB;CACzB,MAAM,aAAa,IAAI,IAAI;EAAC;EAAI;EAAK;EAAK,CAAC;AAE3C,QAAO,EACN,SAAS;EACR,QAAQ,MAAM,WAAW;GACxB,QAAQ,MAAc;AAErB,qBAAiB;AAChB,SAAI,WAAW,IAAI,KAAK,CACtB,MAAa,aAAa;SAE1B,MAAa,WAAW;OAExB,EAAE;;GAEN,KAAK,OAAe,IAAgB;AACnC,QAAI,UAAU,UAAY,MAAa,YAAY;AACnD,QAAI,UAAU,QAAU,MAAa,UAAU;AAC/C,QAAI,UAAU,UAAY,MAAa,YAAY;;GAEpD,aAAa;GACb,qBAAqB;GACrB,UAAU;;EAEX,eAAe;AACd,UAAO;IACN,KAAK,OAAe,IAA2B;AAC9C,SAAI,UAAU,QAAU,MAAa,UAAU;;IAEhD,OAAO,OAAe,OAAe,IAAgB;AACpD,SAAI;;IAEL,MAAM,IAAgB;AACrB,SAAI;;IAEL;;EAEF,EACD;EACA;AAEF,SAAS,YACR,IACA,OACoB;AACpB,QAAO;EACN;EACA,MAAM;EACN,aAAa,GAAG,GAAG;EACnB,MAAM;EACN,UAAU;EACV,OAAO,GAAG,GAAG;EACb;EACA,SAAS,EAAE;EACX,aAAa,EAAE;EACf,cAAc,EAAE;EAChB,WAAW,EAAE;EACb,QAAQ,EAAE;EACV,UAAU;EACV,SAAS;EACT;;AAGFC,sBAAAA,SAAS,6BAA6B;AACrC,uBAAA,GAAG,mDAAmD;AAMrD,wBAAA,aADkBC,qBAAAA,oBAJD,CAChB,YAAY,SAAS,CAAC;GAAE,MAAM;GAAM,WAAW;GAAM,SAAS;GAAM,aAAa;GAAS,CAAC,CAAC,CAC5F,kBACqB,IAAI,KAAK,CAC+B,CAC7C,CAAC,QAAQ,EAAE,CAAC;GAC5B;AAEF,uBAAA,GAAG,gDAAgD;EAKlD,MAAM,YAAYA,qBAAAA,oBAJD,CAChB,YAAY,SAAS,CAAC;GAAE,MAAM;GAAM,WAAW;GAAM,SAAS;GAAM,aAAa;GAAS,CAAC,CAAC,CAC5F,EACqB,IAAI,IAAI,CAAC,CAAC,SAAS,IAAI,IAAI,CAAC,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CACL;AAC9D,wBAAA,aAAO,UAAU,CAAC,aAAa,EAAE;AACjC,wBAAA,aAAO,UAAU,GAAG,CAAC,QAAQ;GAC5B,MAAM;GACN,WAAW;GACX,aAAa;GACb,eAAe;GACf,CAAC;GACD;AAEF,uBAAA,GAAG,oEAAoE;EAiBtE,MAAM,YAAYA,qBAAAA,oBAhBD,CAChB,YAAY,OAAO,CAClB;GAAE,MAAM;GAAI,WAAW;GAAI,SAAS;GAAM,aAAa;GAAQ,EAC/D;GAAE,MAAM;GAAK,WAAW;GAAK,SAAS;GAAM,aAAa;GAAS,CAClE,CAAC,EACF,YAAY,OAAO,CAAC;GAAE,MAAM;GAAM,WAAW;GAAM,SAAS;GAAM,aAAa;GAAO,CAAC,CAAC,CACxF,EACqB,IAAI,IAAI,CAC7B,CACC,OACA,IAAI,IAAI,CACP,CAAC,IAAI,KAAK,EACV,CAAC,KAAK,KAAK,CACX,CAAC,CACF,CACD,CAAC,CAC4D;AAC9D,wBAAA,aAAO,UAAU,CAAC,aAAa,EAAE;AACjC,wBAAA,aAAO,UAAU,GAAI,KAAK,CAAC,KAAK,GAAG;AACnC,wBAAA,aAAO,UAAU,GAAI,cAAc,CAAC,KAAK,KAAK;AAC9C,wBAAA,aAAO,UAAU,GAAI,KAAK,CAAC,KAAK,IAAI;AACpC,wBAAA,aAAO,UAAU,GAAI,cAAc,CAAC,KAAK,KAAK;GAC7C;AAEF,uBAAA,GAAG,oEAAoE;AAMtE,wBAAA,aADkBA,qBAAAA,oBAJD,CAChB,YAAY,SAAS,CAAC;GAAE,MAAM;GAAM,WAAW;GAAM,SAAS;GAAM,aAAa;GAAI,CAAC,CAAC,CACvF,EACqB,IAAI,IAAI,CAAC,CAAC,SAAS,IAAI,IAAI,CAAC,CAAC,KAAM,IAAM,CAAC,CAAC,CAAC,CAAC,CAAC,CACN,CAC7C,GAAI,YAAY,CAAC,KAAK,aAAa;GACnD;EACD;AAEFD,sBAAAA,SAAS,2BAA2B;AACnC,uBAAA,GAAG,gDAAgD,YAAY;AAG9D,wBAAA,cADe,MAAME,qBAAAA,kBADJ,CAAC,YAAY,SAAS,EAAE,CAAC,CAAC,CACK,EAClC,KAAK,CAAC,KAAK,EAAE;GAC1B;AAEF,uBAAA,GAAG,wDAAwD,YAAY;AAKtE,wBAAA,cADe,MAAMA,qBAAAA,kBAHJ,CAChB,YAAY,SAAS,CAAC;GAAE,MAAM;GAAM,WAAW;GAAM,SAAS;GAAO,aAAa;GAAS,CAAC,CAAC,CAC7F,CAC+C,EAClC,KAAK,CAAC,KAAK,EAAE;GAC1B;AAEF,uBAAA,GAAG,wCAAwC,YAAY;EAMtD,MAAM,SAAS,MAAMA,qBAAAA,kBAJJ,CAChB,YAAY,QAAQ,CAAC;GAAE,MAAM;GAAM,WAAW;GAAI,SAAS;GAAM,aAAa;GAAQ,CAAC,CAAC,EACxF,YAAY,QAAQ,CAAC;GAAE,MAAM;GAAM,WAAW;GAAI,SAAS;GAAM,aAAa;GAAQ,CAAC,CAAC,CACxF,CAC+C;AAGhD,wBAAA,aAAO,OAAO,IAAI,OAAO,CAAC,CAAC,KAAK,MAAM;AACtC,wBAAA,aAAO,OAAO,IAAI,OAAO,CAAC,CAAC,KAAK,KAAK;EACrC,MAAM,oBAAoB,OAAO,IAAI,OAAO;AAC5C,wBAAA,aAAO,kBAAkB,IAAI,KAAK,CAAC,CAAC,KAAK,KAAK;AAC9C,wBAAA,aAAO,kBAAkB,IAAI,KAAK,CAAC,CAAC,gBAAgB,KAAK;GACxD;EACD"}

package/dist/port-scanner.test.d.cts

@@ -0,0 +1 @@
+export { };

package/dist/port-scanner.test.d.mts

@@ -0,0 +1 @@
+export { };

package/dist/port-scanner.test.mjs

@@ -0,0 +1,156 @@
+import { a as describe, n as vi, o as it, t as globalExpect } from "./test.CTcmp4Su-BRa7-bTj.mjs";
+import { formatPortConflicts, scanPortConflicts } from "./port-scanner.mjs";
+//#region src/port-scanner.test.ts
+/**
+* Tests for port scanner conflict detection and reassignment.
+*
+* We mock isPortAvailable via vi.mock to avoid actual port scanning
+* during tests. The focus is on the conflict detection logic.
+*/
+vi.mock("node:net", () => {
+	const portsInUse = new Set([
+		80,
+		443,
+		3e3
+	]);
+	return { default: {
+		Socket: class MockSocket {
+			connect(port) {
+				setTimeout(() => {
+					if (portsInUse.has(port)) this.connectCb?.();
+					else this.errorCb?.();
+				}, 1);
+			}
+			once(event, cb) {
+				if (event === "connect") this.connectCb = cb;
+				if (event === "error") this.errorCb = cb;
+				if (event === "timeout") this.timeoutCb = cb;
+			}
+			setTimeout() {}
+			removeAllListeners() {}
+			destroy() {}
+		},
+		createServer() {
+			return {
+				once(event, cb) {
+					if (event === "error") this.errorCb = cb;
+				},
+				listen(_port, _host, cb) {
+					cb();
+				},
+				close(cb) {
+					cb();
+				}
+			};
+		}
+	} };
+});
+function makeService(id, ports) {
+	return {
+		id,
+		name: id,
+		description: `${id} service`,
+		icon: "",
+		category: "test",
+		image: `${id}:latest`,
+		ports,
+		volumes: [],
+		environment: [],
+		dependencies: [],
+		conflicts: [],
+		skills: [],
+		memoryMB: 256,
+		docsUrl: ""
+	};
+}
+describe("formatPortConflicts", () => {
+	it("returns empty array when no reassignments", () => {
+		globalExpect(formatPortConflicts([makeService("redis", [{
+			host: 6379,
+			container: 6379,
+			exposed: true,
+			description: "Redis"
+		}])], /* @__PURE__ */ new Map())).toEqual([]);
+	});
+	it("returns conflicts with suggested ports", () => {
+		const conflicts = formatPortConflicts([makeService("redis", [{
+			host: 6379,
+			container: 6379,
+			exposed: true,
+			description: "Redis"
+		}])], new Map([["redis", new Map([[6379, 7379]])]]));
+		globalExpect(conflicts).toHaveLength(1);
+		globalExpect(conflicts[0]).toEqual({
+			port: 6379,
+			serviceId: "redis",
+			description: "Redis",
+			suggestedPort: 7379
+		});
+	});
+	it("handles multiple services with multiple port reassignments", () => {
+		const conflicts = formatPortConflicts([makeService("web", [{
+			host: 80,
+			container: 80,
+			exposed: true,
+			description: "HTTP"
+		}, {
+			host: 443,
+			container: 443,
+			exposed: true,
+			description: "HTTPS"
+		}]), makeService("api", [{
+			host: 8080,
+			container: 8080,
+			exposed: true,
+			description: "API"
+		}])], new Map([["web", new Map([[80, 1080], [443, 1443]])]]));
+		globalExpect(conflicts).toHaveLength(2);
+		globalExpect(conflicts[0].port).toBe(80);
+		globalExpect(conflicts[0].suggestedPort).toBe(1080);
+		globalExpect(conflicts[1].port).toBe(443);
+		globalExpect(conflicts[1].suggestedPort).toBe(1443);
+	});
+	it("uses service name as fallback when port has no description", () => {
+		globalExpect(formatPortConflicts([makeService("myapp", [{
+			host: 9e3,
+			container: 9e3,
+			exposed: true,
+			description: ""
+		}])], new Map([["myapp", new Map([[9e3, 1e4]])]]))[0].description).toBe("myapp port");
+	});
+});
+describe("scanPortConflicts", () => {
+	it("returns empty map for services with no ports", async () => {
+		globalExpect((await scanPortConflicts([makeService("redis", [])])).size).toBe(0);
+	});
+	it("returns empty map for services with no exposed ports", async () => {
+		globalExpect((await scanPortConflicts([makeService("redis", [{
+			host: 6379,
+			container: 6379,
+			exposed: false,
+			description: "Redis"
+		}])])).size).toBe(0);
+	});
+	it("detects inter-service port conflicts", async () => {
+		const result = await scanPortConflicts([makeService("web1", [{
+			host: 8080,
+			container: 80,
+			exposed: true,
+			description: "HTTP"
+		}]), makeService("web2", [{
+			host: 8080,
+			container: 80,
+			exposed: true,
+			description: "HTTP"
+		}])]);
+		globalExpect(result.has("web1")).toBe(false);
+		globalExpect(result.has("web2")).toBe(true);
+		const web2Reassignments = result.get("web2");
+		globalExpect(web2Reassignments.has(8080)).toBe(true);
+		globalExpect(web2Reassignments.get(8080)).toBeGreaterThan(8080);
+	});
+});
+//#endregion
+export {};
+
+//# sourceMappingURL=port-scanner.test.mjs.map

package/dist/port-scanner.test.mjs.map

@@ -0,0 +1 @@
+{"version":3,"file":"port-scanner.test.mjs","names":[],"sources":["../src/port-scanner.test.ts"],"sourcesContent":["import { describe, expect, it, vi } from \"vitest\";\nimport { formatPortConflicts, scanPortConflicts } from \"./port-scanner.js\";\nimport type { ServiceDefinition } from \"./types.js\";\n\n/**\n * Tests for port scanner conflict detection and reassignment.\n *\n * We mock isPortAvailable via vi.mock to avoid actual port scanning\n * during tests. The focus is on the conflict detection logic.\n */\n\n// Mock the internal isPortAvailable to control port availability\nvi.mock(\"node:net\", () => {\n\tconst portsInUse = new Set([80, 443, 3000]);\n\n\treturn {\n\t\tdefault: {\n\t\t\tSocket: class MockSocket {\n\t\t\t\tconnect(port: number) {\n\t\t\t\t\t// Simulate: ports in portsInUse are \"in use\"\n\t\t\t\t\tsetTimeout(() => {\n\t\t\t\t\t\tif (portsInUse.has(port)) {\n\t\t\t\t\t\t\t(this as any).connectCb?.();\n\t\t\t\t\t\t} else {\n\t\t\t\t\t\t\t(this as any).errorCb?.();\n\t\t\t\t\t\t}\n\t\t\t\t\t}, 1);\n\t\t\t\t}\n\t\t\t\tonce(event: string, cb: () => void) {\n\t\t\t\t\tif (event === \"connect\") (this as any).connectCb = cb;\n\t\t\t\t\tif (event === \"error\") (this as any).errorCb = cb;\n\t\t\t\t\tif (event === \"timeout\") (this as any).timeoutCb = cb;\n\t\t\t\t}\n\t\t\t\tsetTimeout() {}\n\t\t\t\tremoveAllListeners() {}\n\t\t\t\tdestroy() {}\n\t\t\t},\n\t\t\tcreateServer() {\n\t\t\t\treturn {\n\t\t\t\t\tonce(event: string, cb: (err?: Error) => void) {\n\t\t\t\t\t\tif (event === \"error\") (this as any).errorCb = cb;\n\t\t\t\t\t},\n\t\t\t\t\tlisten(_port: number, _host: string, cb: () => void) {\n\t\t\t\t\t\tcb();\n\t\t\t\t\t},\n\t\t\t\t\tclose(cb: () => void) {\n\t\t\t\t\t\tcb();\n\t\t\t\t\t},\n\t\t\t\t};\n\t\t\t},\n\t\t},\n\t};\n});\n\nfunction makeService(\n\tid: string,\n\tports: { host: number; container: number; exposed: boolean; description: string }[],\n): ServiceDefinition {\n\treturn {\n\t\tid,\n\t\tname: id,\n\t\tdescription: `${id} service`,\n\t\ticon: \"\",\n\t\tcategory: \"test\",\n\t\timage: `${id}:latest`,\n\t\tports,\n\t\tvolumes: [],\n\t\tenvironment: [],\n\t\tdependencies: [],\n\t\tconflicts: [],\n\t\tskills: [],\n\t\tmemoryMB: 256,\n\t\tdocsUrl: \"\",\n\t} as ServiceDefinition;\n}\n\ndescribe(\"formatPortConflicts\", () => {\n\tit(\"returns empty array when no reassignments\", () => {\n\t\tconst services = [\n\t\t\tmakeService(\"redis\", [{ host: 6379, container: 6379, exposed: true, description: \"Redis\" }]),\n\t\t];\n\t\tconst reassignments = new Map();\n\t\tconst conflicts = formatPortConflicts(services, reassignments);\n\t\texpect(conflicts).toEqual([]);\n\t});\n\n\tit(\"returns conflicts with suggested ports\", () => {\n\t\tconst services = [\n\t\t\tmakeService(\"redis\", [{ host: 6379, container: 6379, exposed: true, description: \"Redis\" }]),\n\t\t];\n\t\tconst reassignments = new Map([[\"redis\", new Map([[6379, 7379]])]]);\n\t\tconst conflicts = formatPortConflicts(services, reassignments);\n\t\texpect(conflicts).toHaveLength(1);\n\t\texpect(conflicts[0]).toEqual({\n\t\t\tport: 6379,\n\t\t\tserviceId: \"redis\",\n\t\t\tdescription: \"Redis\",\n\t\t\tsuggestedPort: 7379,\n\t\t});\n\t});\n\n\tit(\"handles multiple services with multiple port reassignments\", () => {\n\t\tconst services = [\n\t\t\tmakeService(\"web\", [\n\t\t\t\t{ host: 80, container: 80, exposed: true, description: \"HTTP\" },\n\t\t\t\t{ host: 443, container: 443, exposed: true, description: \"HTTPS\" },\n\t\t\t]),\n\t\t\tmakeService(\"api\", [{ host: 8080, container: 8080, exposed: true, description: \"API\" }]),\n\t\t];\n\t\tconst reassignments = new Map([\n\t\t\t[\n\t\t\t\t\"web\",\n\t\t\t\tnew Map([\n\t\t\t\t\t[80, 1080],\n\t\t\t\t\t[443, 1443],\n\t\t\t\t]),\n\t\t\t],\n\t\t]);\n\t\tconst conflicts = formatPortConflicts(services, reassignments);\n\t\texpect(conflicts).toHaveLength(2);\n\t\texpect(conflicts[0]!.port).toBe(80);\n\t\texpect(conflicts[0]!.suggestedPort).toBe(1080);\n\t\texpect(conflicts[1]!.port).toBe(443);\n\t\texpect(conflicts[1]!.suggestedPort).toBe(1443);\n\t});\n\n\tit(\"uses service name as fallback when port has no description\", () => {\n\t\tconst services = [\n\t\t\tmakeService(\"myapp\", [{ host: 9000, container: 9000, exposed: true, description: \"\" }]),\n\t\t];\n\t\tconst reassignments = new Map([[\"myapp\", new Map([[9000, 10000]])]]);\n\t\tconst conflicts = formatPortConflicts(services, reassignments);\n\t\texpect(conflicts[0]!.description).toBe(\"myapp port\");\n\t});\n});\n\ndescribe(\"scanPortConflicts\", () => {\n\tit(\"returns empty map for services with no ports\", async () => {\n\t\tconst services = [makeService(\"redis\", [])];\n\t\tconst result = await scanPortConflicts(services);\n\t\texpect(result.size).toBe(0);\n\t});\n\n\tit(\"returns empty map for services with no exposed ports\", async () => {\n\t\tconst services = [\n\t\t\tmakeService(\"redis\", [{ host: 6379, container: 6379, exposed: false, description: \"Redis\" }]),\n\t\t];\n\t\tconst result = await scanPortConflicts(services);\n\t\texpect(result.size).toBe(0);\n\t});\n\n\tit(\"detects inter-service port conflicts\", async () => {\n\t\t// Two services claiming the same host port\n\t\tconst services = [\n\t\t\tmakeService(\"web1\", [{ host: 8080, container: 80, exposed: true, description: \"HTTP\" }]),\n\t\t\tmakeService(\"web2\", [{ host: 8080, container: 80, exposed: true, description: \"HTTP\" }]),\n\t\t];\n\t\tconst result = await scanPortConflicts(services);\n\n\t\t// First service wins, second gets reassigned\n\t\texpect(result.has(\"web1\")).toBe(false); // web1 keeps its port\n\t\texpect(result.has(\"web2\")).toBe(true); // web2 gets reassigned\n\t\tconst web2Reassignments = result.get(\"web2\")!;\n\t\texpect(web2Reassignments.has(8080)).toBe(true);\n\t\texpect(web2Reassignments.get(8080)).toBeGreaterThan(8080);\n\t});\n});\n"],"mappings":";;;;;;;;;AAYA,GAAG,KAAK,kBAAkB;CACzB,MAAM,aAAa,IAAI,IAAI;EAAC;EAAI;EAAK;EAAK,CAAC;AAE3C,QAAO,EACN,SAAS;EACR,QAAQ,MAAM,WAAW;GACxB,QAAQ,MAAc;AAErB,qBAAiB;AAChB,SAAI,WAAW,IAAI,KAAK,CACtB,MAAa,aAAa;SAE1B,MAAa,WAAW;OAExB,EAAE;;GAEN,KAAK,OAAe,IAAgB;AACnC,QAAI,UAAU,UAAY,MAAa,YAAY;AACnD,QAAI,UAAU,QAAU,MAAa,UAAU;AAC/C,QAAI,UAAU,UAAY,MAAa,YAAY;;GAEpD,aAAa;GACb,qBAAqB;GACrB,UAAU;;EAEX,eAAe;AACd,UAAO;IACN,KAAK,OAAe,IAA2B;AAC9C,SAAI,UAAU,QAAU,MAAa,UAAU;;IAEhD,OAAO,OAAe,OAAe,IAAgB;AACpD,SAAI;;IAEL,MAAM,IAAgB;AACrB,SAAI;;IAEL;;EAEF,EACD;EACA;AAEF,SAAS,YACR,IACA,OACoB;AACpB,QAAO;EACN;EACA,MAAM;EACN,aAAa,GAAG,GAAG;EACnB,MAAM;EACN,UAAU;EACV,OAAO,GAAG,GAAG;EACb;EACA,SAAS,EAAE;EACX,aAAa,EAAE;EACf,cAAc,EAAE;EAChB,WAAW,EAAE;EACb,QAAQ,EAAE;EACV,UAAU;EACV,SAAS;EACT;;AAGF,SAAS,6BAA6B;AACrC,IAAG,mDAAmD;AAMrD,eADkB,oBAJD,CAChB,YAAY,SAAS,CAAC;GAAE,MAAM;GAAM,WAAW;GAAM,SAAS;GAAM,aAAa;GAAS,CAAC,CAAC,CAC5F,kBACqB,IAAI,KAAK,CAC+B,CAC7C,CAAC,QAAQ,EAAE,CAAC;GAC5B;AAEF,IAAG,gDAAgD;EAKlD,MAAM,YAAY,oBAJD,CAChB,YAAY,SAAS,CAAC;GAAE,MAAM;GAAM,WAAW;GAAM,SAAS;GAAM,aAAa;GAAS,CAAC,CAAC,CAC5F,EACqB,IAAI,IAAI,CAAC,CAAC,SAAS,IAAI,IAAI,CAAC,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CACL;AAC9D,eAAO,UAAU,CAAC,aAAa,EAAE;AACjC,eAAO,UAAU,GAAG,CAAC,QAAQ;GAC5B,MAAM;GACN,WAAW;GACX,aAAa;GACb,eAAe;GACf,CAAC;GACD;AAEF,IAAG,oEAAoE;EAiBtE,MAAM,YAAY,oBAhBD,CAChB,YAAY,OAAO,CAClB;GAAE,MAAM;GAAI,WAAW;GAAI,SAAS;GAAM,aAAa;GAAQ,EAC/D;GAAE,MAAM;GAAK,WAAW;GAAK,SAAS;GAAM,aAAa;GAAS,CAClE,CAAC,EACF,YAAY,OAAO,CAAC;GAAE,MAAM;GAAM,WAAW;GAAM,SAAS;GAAM,aAAa;GAAO,CAAC,CAAC,CACxF,EACqB,IAAI,IAAI,CAC7B,CACC,OACA,IAAI,IAAI,CACP,CAAC,IAAI,KAAK,EACV,CAAC,KAAK,KAAK,CACX,CAAC,CACF,CACD,CAAC,CAC4D;AAC9D,eAAO,UAAU,CAAC,aAAa,EAAE;AACjC,eAAO,UAAU,GAAI,KAAK,CAAC,KAAK,GAAG;AACnC,eAAO,UAAU,GAAI,cAAc,CAAC,KAAK,KAAK;AAC9C,eAAO,UAAU,GAAI,KAAK,CAAC,KAAK,IAAI;AACpC,eAAO,UAAU,GAAI,cAAc,CAAC,KAAK,KAAK;GAC7C;AAEF,IAAG,oEAAoE;AAMtE,eADkB,oBAJD,CAChB,YAAY,SAAS,CAAC;GAAE,MAAM;GAAM,WAAW;GAAM,SAAS;GAAM,aAAa;GAAI,CAAC,CAAC,CACvF,EACqB,IAAI,IAAI,CAAC,CAAC,SAAS,IAAI,IAAI,CAAC,CAAC,KAAM,IAAM,CAAC,CAAC,CAAC,CAAC,CAAC,CACN,CAC7C,GAAI,YAAY,CAAC,KAAK,aAAa;GACnD;EACD;AAEF,SAAS,2BAA2B;AACnC,IAAG,gDAAgD,YAAY;AAG9D,gBADe,MAAM,kBADJ,CAAC,YAAY,SAAS,EAAE,CAAC,CAAC,CACK,EAClC,KAAK,CAAC,KAAK,EAAE;GAC1B;AAEF,IAAG,wDAAwD,YAAY;AAKtE,gBADe,MAAM,kBAHJ,CAChB,YAAY,SAAS,CAAC;GAAE,MAAM;GAAM,WAAW;GAAM,SAAS;GAAO,aAAa;GAAS,CAAC,CAAC,CAC7F,CAC+C,EAClC,KAAK,CAAC,KAAK,EAAE;GAC1B;AAEF,IAAG,wCAAwC,YAAY;EAMtD,MAAM,SAAS,MAAM,kBAJJ,CAChB,YAAY,QAAQ,CAAC;GAAE,MAAM;GAAM,WAAW;GAAI,SAAS;GAAM,aAAa;GAAQ,CAAC,CAAC,EACxF,YAAY,QAAQ,CAAC;GAAE,MAAM;GAAM,WAAW;GAAI,SAAS;GAAM,aAAa;GAAQ,CAAC,CAAC,CACxF,CAC+C;AAGhD,eAAO,OAAO,IAAI,OAAO,CAAC,CAAC,KAAK,MAAM;AACtC,eAAO,OAAO,IAAI,OAAO,CAAC,CAAC,KAAK,KAAK;EACrC,MAAM,oBAAoB,OAAO,IAAI,OAAO;AAC5C,eAAO,kBAAkB,IAAI,KAAK,CAAC,CAAC,KAAK,KAAK;AAC9C,eAAO,kBAAkB,IAAI,KAAK,CAAC,CAAC,gBAAgB,KAAK;GACxD;EACD"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@better-openclaw/core",
-  "version": "1.0.30",
+  "version": "1.0.31",
   "private": false,
   "description": "Core logic for better-openclaw: schemas, service registry, resolver, composer, validators and generators",
   "author": "bidew.io <bachir@bidew.io>",

package/src/deployers/coolify.test.ts

@@ -0,0 +1,180 @@
+import { beforeEach, describe, expect, it, vi } from "vitest";
+
+/**
+ * Tests for Coolify deployer utility functions and deploy orchestration.
+ *
+ * We mock `fetch` to simulate Coolify API responses and test the
+ * multi-step deployment flow without making real HTTP requests.
+ */
+
+// Inline the utility functions for direct testing
+function hashString(str: string) {
+	let hash = 0;
+	for (let i = 0; i < str.length; i++) {
+		hash = (hash << 5) - hash + str.charCodeAt(i);
+		hash |= 0;
+	}
+	return hash;
+}
+
+function parseEnvContent(envContent?: string) {
+	if (!envContent) return [];
+	const result = [];
+	for (const line of envContent.split("\n")) {
+		const trimmed = line.trim();
+		if (!trimmed || trimmed.startsWith("#")) continue;
+		const idx = trimmed.indexOf("=");
+		if (idx <= 0) continue;
+		const key = trimmed.slice(0, idx);
+		const value = trimmed.slice(idx + 1);
+		result.push({ key, value, is_preview: false, is_build_time: false, is_literal: true });
+	}
+	return result;
+}
+
+describe("hashString", () => {
+	it("returns 0 for empty string", () => {
+		expect(hashString("")).toBe(0);
+	});
+
+	it("returns same hash for same input", () => {
+		expect(hashString("hello")).toBe(hashString("hello"));
+	});
+
+	it("returns different hash for different input", () => {
+		expect(hashString("hello")).not.toBe(hashString("world"));
+	});
+
+	it("detects compose changes", () => {
+		const v1 = "version: '3'\nservices:\n  redis:\n    image: redis:7";
+		const v2 = "version: '3'\nservices:\n  redis:\n    image: redis:8";
+		expect(hashString(v1)).not.toBe(hashString(v2));
+	});
+});
+
+describe("parseEnvContent", () => {
+	it("returns empty array for undefined input", () => {
+		expect(parseEnvContent(undefined)).toEqual([]);
+	});
+
+	it("returns empty array for empty string", () => {
+		expect(parseEnvContent("")).toEqual([]);
+	});
+
+	it("parses simple key=value pairs", () => {
+		const result = parseEnvContent("FOO=bar\nBAZ=qux");
+		expect(result).toHaveLength(2);
+		expect(result[0]).toEqual({
+			key: "FOO",
+			value: "bar",
+			is_preview: false,
+			is_build_time: false,
+			is_literal: true,
+		});
+	});
+
+	it("skips comments", () => {
+		const result = parseEnvContent("# This is a comment\nFOO=bar");
+		expect(result).toHaveLength(1);
+		expect(result[0]!.key).toBe("FOO");
+	});
+
+	it("skips blank lines", () => {
+		const result = parseEnvContent("FOO=bar\n\n\nBAZ=qux");
+		expect(result).toHaveLength(2);
+	});
+
+	it("handles values containing equals signs", () => {
+		const result = parseEnvContent(
+			"DATABASE_URL=postgres://user:pass@host:5432/db?sslmode=require",
+		);
+		expect(result).toHaveLength(1);
+		expect(result[0]!.key).toBe("DATABASE_URL");
+		expect(result[0]!.value).toBe("postgres://user:pass@host:5432/db?sslmode=require");
+	});
+
+	it("handles empty values", () => {
+		const result = parseEnvContent("EMPTY_VAR=");
+		expect(result).toHaveLength(1);
+		expect(result[0]!.key).toBe("EMPTY_VAR");
+		expect(result[0]!.value).toBe("");
+	});
+
+	it("skips lines without = sign", () => {
+		const result = parseEnvContent("INVALID_LINE\nVALID=yes");
+		expect(result).toHaveLength(1);
+		expect(result[0]!.key).toBe("VALID");
+	});
+
+	it("handles quoted values", () => {
+		const result = parseEnvContent('PASSWORD="my secret"');
+		expect(result[0]!.key).toBe("PASSWORD");
+		expect(result[0]!.value).toBe('"my secret"');
+	});
+});
+
+describe("CoolifyDeployer", () => {
+	beforeEach(() => {
+		vi.restoreAllMocks();
+	});
+
+	it("can be imported and instantiated", async () => {
+		const { CoolifyDeployer } = await import("./coolify.js");
+		const deployer = new CoolifyDeployer();
+		expect(deployer.name).toBe("Coolify");
+		expect(deployer.id).toBe("coolify");
+	});
+
+	it("testConnection returns error on network failure", async () => {
+		vi.stubGlobal("fetch", vi.fn().mockRejectedValue(new Error("Connection refused")));
+		const { CoolifyDeployer } = await import("./coolify.js");
+		const deployer = new CoolifyDeployer();
+		const result = await deployer.testConnection({
+			instanceUrl: "https://coolify.example.com",
+			apiKey: "test-key",
+		});
+		expect(result.ok).toBe(false);
+		expect(result.error).toContain("Connection refused");
+		vi.unstubAllGlobals();
+	});
+
+	it("testConnection returns ok on successful version response", async () => {
+		vi.stubGlobal(
+			"fetch",
+			vi.fn().mockResolvedValue({
+				ok: true,
+				text: () => Promise.resolve('"v4.0.0"'),
+			}),
+		);
+		const { CoolifyDeployer } = await import("./coolify.js");
+		const deployer = new CoolifyDeployer();
+		const result = await deployer.testConnection({
+			instanceUrl: "https://coolify.example.com",
+			apiKey: "test-key",
+		});
+		expect(result.ok).toBe(true);
+		vi.unstubAllGlobals();
+	});
+
+	it("deploy returns error when no servers available", async () => {
+		vi.stubGlobal(
+			"fetch",
+			vi.fn().mockResolvedValue({
+				ok: true,
+				text: () => Promise.resolve("[]"),
+			}),
+		);
+		const { CoolifyDeployer } = await import("./coolify.js");
+		const deployer = new CoolifyDeployer();
+		const result = await deployer.deploy({
+			target: { instanceUrl: "https://coolify.example.com", apiKey: "test-key" },
+			projectName: "test-project",
+			composeYaml: "version: '3'",
+			envContent: "",
+		});
+		expect(result.success).toBe(false);
+		expect(result.error).toContain("No Coolify servers");
+		expect(result.steps[0]!.status).toBe("error");
+		vi.unstubAllGlobals();
+	});
+});

package/src/deployers/dokploy.test.ts

@@ -0,0 +1,120 @@
+import { beforeEach, describe, expect, it, vi } from "vitest";
+
+describe("DokployDeployer", () => {
+	beforeEach(() => {
+		vi.restoreAllMocks();
+	});
+
+	it("can be imported and instantiated", async () => {
+		const { DokployDeployer } = await import("./dokploy.js");
+		const deployer = new DokployDeployer();
+		expect(deployer.name).toBe("Dokploy");
+		expect(deployer.id).toBe("dokploy");
+	});
+
+	it("testConnection returns error on network failure", async () => {
+		vi.stubGlobal("fetch", vi.fn().mockRejectedValue(new Error("ECONNREFUSED")));
+		const { DokployDeployer } = await import("./dokploy.js");
+		const deployer = new DokployDeployer();
+		const result = await deployer.testConnection({
+			instanceUrl: "https://dokploy.example.com",
+			apiKey: "test-key",
+		});
+		expect(result.ok).toBe(false);
+		expect(result.error).toContain("ECONNREFUSED");
+		vi.unstubAllGlobals();
+	});
+
+	it("testConnection returns ok when project list succeeds", async () => {
+		vi.stubGlobal(
+			"fetch",
+			vi.fn().mockResolvedValue({
+				ok: true,
+				text: () => Promise.resolve("[]"),
+			}),
+		);
+		const { DokployDeployer } = await import("./dokploy.js");
+		const deployer = new DokployDeployer();
+		const result = await deployer.testConnection({
+			instanceUrl: "https://dokploy.example.com",
+			apiKey: "test-key",
+		});
+		expect(result.ok).toBe(true);
+		vi.unstubAllGlobals();
+	});
+
+	it("listServers returns empty array on API failure", async () => {
+		vi.stubGlobal("fetch", vi.fn().mockRejectedValue(new Error("Not found")));
+		const { DokployDeployer } = await import("./dokploy.js");
+		const deployer = new DokployDeployer();
+		const servers = await deployer.listServers({
+			instanceUrl: "https://dokploy.example.com",
+			apiKey: "test-key",
+		});
+		expect(servers).toEqual([]);
+		vi.unstubAllGlobals();
+	});
+
+	it("listServers maps server fields correctly", async () => {
+		vi.stubGlobal(
+			"fetch",
+			vi.fn().mockResolvedValue({
+				ok: true,
+				text: () =>
+					Promise.resolve(
+						JSON.stringify([
+							{ serverId: "s1", name: "Production", ipAddress: "1.2.3.4" },
+							{ serverId: "s2", name: "Staging", ipAddress: "5.6.7.8" },
+						]),
+					),
+			}),
+		);
+		const { DokployDeployer } = await import("./dokploy.js");
+		const deployer = new DokployDeployer();
+		const servers = await deployer.listServers({
+			instanceUrl: "https://dokploy.example.com",
+			apiKey: "test-key",
+		});
+		expect(servers).toHaveLength(2);
+		expect(servers[0]).toEqual({ id: "s1", name: "Production", ip: "1.2.3.4" });
+		expect(servers[1]).toEqual({ id: "s2", name: "Staging", ip: "5.6.7.8" });
+		vi.unstubAllGlobals();
+	});
+
+	it("deploy sets error on running step when an API call fails", async () => {
+		let callCount = 0;
+		vi.stubGlobal(
+			"fetch",
+			vi.fn().mockImplementation(() => {
+				callCount++;
+				if (callCount === 1) {
+					// project.all succeeds
+					return Promise.resolve({
+						ok: true,
+						text: () => Promise.resolve("[]"),
+					});
+				}
+				// project.create fails
+				return Promise.resolve({
+					ok: false,
+					status: 500,
+					text: () => Promise.resolve('{"message":"Internal error"}'),
+				});
+			}),
+		);
+		const { DokployDeployer } = await import("./dokploy.js");
+		const deployer = new DokployDeployer();
+		const result = await deployer.deploy({
+			target: { instanceUrl: "https://dokploy.example.com", apiKey: "key" },
+			projectName: "test",
+			composeYaml: "version: '3'",
+			envContent: "",
+		});
+		expect(result.success).toBe(false);
+		expect(result.error).toBeDefined();
+		// At least one step should be in error state
+		const errorStep = result.steps.find((s) => s.status === "error");
+		expect(errorStep).toBeDefined();
+		vi.unstubAllGlobals();
+	});
+});