@better-openclaw/core 1.0.18 → 1.0.20

package/src/deployers/coolify.ts

@@ -139,29 +139,28 @@ export class CoolifyDeployer implements PaasDeployer {
 	}
 
 	async deploy(input: DeployInput): Promise<DeployResult> {
-		const steps: DeployStep[] = [
-			{ step: "Discover server", status: "pending" },
-			{ step: "Create project", status: "pending" },
-			{ step: "Create compose service", status: "pending" },
-			{ step: "Set environment variables", status: "pending" },
-			{ step: "Trigger deployment", status: "pending" },
-		];
+		const step1: DeployStep = { step: "Discover server", status: "pending" };
+		const step2: DeployStep = { step: "Create project", status: "pending" };
+		const step3: DeployStep = { step: "Create compose service", status: "pending" };
+		const step4: DeployStep = { step: "Set environment variables", status: "pending" };
+		const step5: DeployStep = { step: "Trigger deployment", status: "pending" };
+		const steps: DeployStep[] = [step1, step2, step3, step4, step5];
 
 		const result: DeployResult = { success: false, steps };
 
 		try {
 			// Step 1: Discover default server
-			steps[0].status = "running";
+			step1.status = "running";
 			const servers = await coolifyFetch<CoolifyServer[]>(input.target, "/servers");
 			if (!servers || servers.length === 0) {
 				throw new Error("No servers found in Coolify instance");
 			}
-			const server = servers[0];
-			steps[0].status = "done";
-			steps[0].detail = `Server: ${server.name} (${server.ip})`;
+			const server = servers[0] as CoolifyServer;
+			step1.status = "done";
+			step1.detail = `Server: ${server.name} (${server.ip})`;
 
 			// Step 2: Create project
-			steps[1].status = "running";
+			step2.status = "running";
 			const project = await coolifyFetch<CoolifyProject>(input.target, "/projects", {
 				method: "POST",
 				body: {
@@ -170,8 +169,8 @@ export class CoolifyDeployer implements PaasDeployer {
 				},
 			});
 			result.projectId = project.uuid;
-			steps[1].status = "done";
-			steps[1].detail = `Project: ${project.uuid}`;
+			step2.status = "done";
+			step2.detail = `Project: ${project.uuid}`;
 
 			// Get the default environment
 			const projectDetail = await coolifyFetch<CoolifyProject>(
@@ -185,7 +184,7 @@ export class CoolifyDeployer implements PaasDeployer {
 			}
 
 			// Step 3: Create compose service with docker_compose_raw
-			steps[2].status = "running";
+			step3.status = "running";
 			const service = await coolifyFetch<CoolifyService>(input.target, "/services", {
 				method: "POST",
 				body: {
@@ -200,11 +199,11 @@ export class CoolifyDeployer implements PaasDeployer {
 				},
 			});
 			result.composeId = service.uuid;
-			steps[2].status = "done";
-			steps[2].detail = `Service: ${service.uuid}`;
+			step3.status = "done";
+			step3.detail = `Service: ${service.uuid}`;
 
 			// Step 4: Set environment variables
-			steps[3].status = "running";
+			step4.status = "running";
 			const envVars = parseEnvContent(input.envContent);
 			if (envVars.length > 0) {
 				await coolifyFetch(input.target, `/services/${service.uuid}/envs`, {
@@ -212,26 +211,26 @@ export class CoolifyDeployer implements PaasDeployer {
 					body: envVars,
 				});
 			}
-			steps[3].status = "done";
-			steps[3].detail = `${envVars.length} variables set`;
+			step4.status = "done";
+			step4.detail = `${envVars.length} variables set`;
 
 			// Step 5: Trigger deployment
-			steps[4].status = "running";
+			step5.status = "running";
 			const deployments = await coolifyFetch<{ deployments: CoolifyDeployment[] }>(
 				input.target,
 				`/deploy?uuid=${service.uuid}&force=true`,
 			);
-			steps[4].status = "done";
-			steps[4].detail = deployments?.deployments?.[0]?.deployment_uuid ?? "Deployment triggered";
+			step5.status = "done";
+			step5.detail = deployments?.deployments?.[0]?.deployment_uuid ?? "Deployment triggered";
 
 			result.success = true;
 			const base = input.target.instanceUrl.replace(/\/+$/, "");
 			result.dashboardUrl = `${base}/project/${project.uuid}`;
 		} catch (err) {
-			const failedIdx = steps.findIndex((s) => s.status === "running");
-			if (failedIdx >= 0) {
-				steps[failedIdx].status = "error";
-				steps[failedIdx].detail = err instanceof Error ? err.message : String(err);
+			const failedStep = steps.find((s) => s.status === "running");
+			if (failedStep) {
+				failedStep.status = "error";
+				failedStep.detail = err instanceof Error ? err.message : String(err);
 			}
 			result.error = err instanceof Error ? err.message : String(err);
 		}

package/src/deployers/dokploy.ts

@@ -87,18 +87,17 @@ export class DokployDeployer implements PaasDeployer {
 	}
 
 	async deploy(input: DeployInput): Promise<DeployResult> {
-		const steps: DeployStep[] = [
-			{ step: "Create project", status: "pending" },
-			{ step: "Create compose stack", status: "pending" },
-			{ step: "Set environment variables", status: "pending" },
-			{ step: "Trigger deployment", status: "pending" },
-		];
+		const step1: DeployStep = { step: "Create project", status: "pending" };
+		const step2: DeployStep = { step: "Create compose stack", status: "pending" };
+		const step3: DeployStep = { step: "Set environment variables", status: "pending" };
+		const step4: DeployStep = { step: "Trigger deployment", status: "pending" };
+		const steps: DeployStep[] = [step1, step2, step3, step4];
 
 		const result: DeployResult = { success: false, steps };
 
 		try {
 			// Step 1: Create project
-			steps[0].status = "running";
+			step1.status = "running";
 			const project = await dokployFetch<DokployProject>(input.target, "project.create", {
 				method: "POST",
 				body: {
@@ -107,8 +106,8 @@ export class DokployDeployer implements PaasDeployer {
 				},
 			});
 			result.projectId = project.projectId;
-			steps[0].status = "done";
-			steps[0].detail = `Project ID: ${project.projectId}`;
+			step1.status = "done";
+			step1.detail = `Project ID: ${project.projectId}`;
 
 			// Get the default environment ID
 			const projectDetail = await dokployFetch<DokployProject>(
@@ -121,7 +120,7 @@ export class DokployDeployer implements PaasDeployer {
 			}
 
 			// Step 2: Create compose stack
-			steps[1].status = "running";
+			step2.status = "running";
 			const compose = await dokployFetch<DokployCompose>(input.target, "compose.create", {
 				method: "POST",
 				body: {
@@ -131,11 +130,11 @@ export class DokployDeployer implements PaasDeployer {
 				},
 			});
 			result.composeId = compose.composeId;
-			steps[1].status = "done";
-			steps[1].detail = `Compose ID: ${compose.composeId}`;
+			step2.status = "done";
+			step2.detail = `Compose ID: ${compose.composeId}`;
 
 			// Step 3: Set environment variables
-			steps[2].status = "running";
+			step3.status = "running";
 			await dokployFetch(input.target, "compose.update", {
 				method: "POST",
 				body: {
@@ -143,10 +142,10 @@ export class DokployDeployer implements PaasDeployer {
 					env: input.envContent,
 				},
 			});
-			steps[2].status = "done";
+			step3.status = "done";
 
 			// Step 4: Trigger deployment
-			steps[3].status = "running";
+			step4.status = "running";
 			await dokployFetch(input.target, "compose.deploy", {
 				method: "POST",
 				body: {
@@ -155,16 +154,16 @@ export class DokployDeployer implements PaasDeployer {
 					description: input.description ?? "Deployed via OpenClaw web builder",
 				},
 			});
-			steps[3].status = "done";
+			step4.status = "done";
 
 			result.success = true;
 			const base = input.target.instanceUrl.replace(/\/+$/, "");
 			result.dashboardUrl = `${base}/dashboard/project/${project.projectId}`;
 		} catch (err) {
-			const failedIdx = steps.findIndex((s) => s.status === "running");
-			if (failedIdx >= 0) {
-				steps[failedIdx].status = "error";
-				steps[failedIdx].detail = err instanceof Error ? err.message : String(err);
+			const failedStep = steps.find((s) => s.status === "running");
+			if (failedStep) {
+				failedStep.status = "error";
+				failedStep.detail = err instanceof Error ? err.message : String(err);
 			}
 			result.error = err instanceof Error ? err.message : String(err);
 		}

package/src/port-scanner.ts

@@ -63,7 +63,7 @@ async function isPortAvailable(port: number): Promise<boolean> {
 		const canBind = await new Promise<boolean>((resolve) => {
 			const server = net.createServer();
 
-			server.once("error", (err: NodeJS.ErrnoException) => {
+			server.once("error", (_err: NodeJS.ErrnoException) => {
 				// EACCES / EADDRINUSE / EPERM = port is reserved or in use
 				resolve(false);
 			});

package/src/services/definitions/usesend.ts

@@ -36,28 +36,28 @@ export const usesendDefinition: ServiceDefinition = {
 		},
 		{
 			key: "AWS_ACCESS_KEY",
-			defaultValue: "",
+			defaultValue: "your-aws-access-key",
 			secret: true,
 			description: "AWS access key for SES/SNS",
 			required: true,
 		},
 		{
 			key: "AWS_SECRET_KEY",
-			defaultValue: "",
+			defaultValue: "your-aws-secret-key",
 			secret: true,
 			description: "AWS secret key for SES/SNS",
 			required: true,
 		},
 		{
 			key: "GITHUB_ID",
-			defaultValue: "",
+			defaultValue: "your-github-oauth-client-id",
 			secret: false,
 			description: "GitHub OAuth app client ID",
 			required: true,
 		},
 		{
 			key: "GITHUB_SECRET",
-			defaultValue: "",
+			defaultValue: "your-github-oauth-client-secret",
 			secret: true,
 			description: "GitHub OAuth app client secret",
 			required: true,