[autofix.ci] apply automated fixes

Add an informative tooltip explaining the cleanup behavior and linking
to Schedule Jobs docs for custom cleanup strategies.

Closes #3973

apps/dokploy/components/dashboard/application/environment/show-environment.tsx

@@ -56,17 +56,17 @@ export const ShowEnvironment = ({ id, type }: Props) => {
 	const [isEnvVisible, setIsEnvVisible] = useState(true);
 
 	const mutationMap = {
-		compose: () => api.compose.update.useMutation(),
+		compose: () => api.compose.saveEnvironment.useMutation(),
-		libsql: () => api.libsql.update.useMutation(),
+		libsql: () => api.libsql.saveEnvironment.useMutation(),
-		mariadb: () => api.mariadb.update.useMutation(),
+		mariadb: () => api.mariadb.saveEnvironment.useMutation(),
-		mongo: () => api.mongo.update.useMutation(),
+		mongo: () => api.mongo.saveEnvironment.useMutation(),
-		mysql: () => api.mysql.update.useMutation(),
+		mysql: () => api.mysql.saveEnvironment.useMutation(),
-		postgres: () => api.postgres.update.useMutation(),
+		postgres: () => api.postgres.saveEnvironment.useMutation(),
-		redis: () => api.redis.update.useMutation(),
+		redis: () => api.redis.saveEnvironment.useMutation(),
 	};
 	const { mutateAsync, isPending } = mutationMap[type]
 		? mutationMap[type]()
-		: api.mongo.update.useMutation();
+		: api.mongo.saveEnvironment.useMutation();
 
 	const form = useForm<EnvironmentSchema>({
 		defaultValues: {

apps/dokploy/components/dashboard/settings/servers/actions/toggle-docker-cleanup.tsx

@@ -1,7 +1,14 @@
-import { toast } from "sonner";
 import { Label } from "@/components/ui/label";
 import { Switch } from "@/components/ui/switch";
+import {
+	Tooltip,
+	TooltipContent,
+	TooltipProvider,
+	TooltipTrigger,
+} from "@/components/ui/tooltip";
 import { api } from "@/utils/api";
+import { HelpCircle } from "lucide-react";
+import { toast } from "sonner";
 
 interface Props {
 	serverId?: string;
@@ -52,7 +59,36 @@ export const ToggleDockerCleanup = ({ serverId }: Props) => {
 	return (
 		<div className="flex items-center gap-4">
 			<Switch checked={!!enabled} onCheckedChange={handleToggle} />
-			<Label className="text-primary">Daily Docker Cleanup</Label>
+			<TooltipProvider delayDuration={0}>
+				<Tooltip>
+					<TooltipTrigger asChild>
+						<Label className="text-primary flex items-center gap-1.5 cursor-pointer">
+							Daily Docker Cleanup
+							<HelpCircle className="size-4 text-muted-foreground" />
+						</Label>
+					</TooltipTrigger>
+					<TooltipContent side="top" className="max-w-sm">
+						<p>
+							Runs a full Docker cleanup daily, pruning stopped containers,
+							unused images, volumes, build cache, and system resources. This
+							may remove images built for Compose services that run on-demand
+							(backup runners, cron jobs, one-off tasks).
+						</p>
+						<p className="mt-1">
+							For custom cleanup strategies, use{" "}
+							<a
+								href="https://docs.dokploy.com/docs/core/schedule-jobs#example-1-automatic-docker-cleanup"
+								target="_blank"
+								rel="noopener noreferrer"
+								className="underline text-primary"
+							>
+								Schedule Jobs
+							</a>{" "}
+							on your web server or remote servers.
+						</p>
+					</TooltipContent>
+				</Tooltip>
+			</TooltipProvider>
 		</div>
 	);
 };

apps/dokploy/pages/api/deploy/compose/[refreshToken].ts

@@ -53,14 +53,9 @@ export default async function handler(
 
 		if (sourceType === "github") {
 			const branchName = extractBranchName(req.headers, req.body);
-			const normalizedCommits =
+			const normalizedCommits = req.body?.commits?.flatMap(
-				req.body?.commits
+				(commit: any) => commit.modified,
-					?.flatMap((commit: any) => [
+			);
-						...(commit.modified || []),
-						...(commit.added || []),
-						...(commit.removed || []),
-					])
-					.filter(Boolean) || [];
 
 			const shouldDeployPaths = shouldDeploy(
 				composeResult.watchPaths,
@@ -78,14 +73,9 @@ export default async function handler(
 			}
 		} else if (sourceType === "gitlab") {
 			const branchName = extractBranchName(req.headers, req.body);
-			const normalizedCommits =
+			const normalizedCommits = req.body?.commits?.flatMap(
-				req.body?.commits
+				(commit: any) => commit.modified,
-					?.flatMap((commit: any) => [
+			);
-						...(commit.modified || []),
-						...(commit.added || []),
-						...(commit.removed || []),
-					])
-					.filter(Boolean) || [];
 
 			const shouldDeployPaths = shouldDeploy(
 				composeResult.watchPaths,
@@ -134,32 +124,17 @@ export default async function handler(
 			let normalizedCommits: string[] = [];
 
 			if (provider === "github") {
-				normalizedCommits =
+				normalizedCommits = req.body?.commits?.flatMap(
-					req.body?.commits
+					(commit: any) => commit.modified,
-						?.flatMap((commit: any) => [
+				);
-							...(commit.modified || []),
-							...(commit.added || []),
-							...(commit.removed || []),
-						])
-						.filter(Boolean) || [];
 			} else if (provider === "gitlab") {
-				normalizedCommits =
+				normalizedCommits = req.body?.commits?.flatMap(
-					req.body?.commits
+					(commit: any) => commit.modified,
-						?.flatMap((commit: any) => [
+				);
-							...(commit.modified || []),
-							...(commit.added || []),
-							...(commit.removed || []),
-						])
-						.filter(Boolean) || [];
 			} else if (provider === "gitea") {
-				normalizedCommits =
+				normalizedCommits = req.body?.commits?.flatMap(
-					req.body?.commits
+					(commit: any) => commit.modified,
-						?.flatMap((commit: any) => [
+				);
-							...(commit.modified || []),
-							...(commit.added || []),
-							...(commit.removed || []),
-						])
-						.filter(Boolean) || [];
 			}
 
 			const shouldDeployPaths = shouldDeploy(
@@ -174,14 +149,9 @@ export default async function handler(
 		} else if (sourceType === "gitea") {
 			const branchName = extractBranchName(req.headers, req.body);
 
-			const normalizedCommits =
+			const normalizedCommits = req.body?.commits?.flatMap(
-				req.body?.commits
+				(commit: any) => commit.modified,
-					?.flatMap((commit: any) => [
+			);
-						...(commit.modified || []),
-						...(commit.added || []),
-						...(commit.removed || []),
-					])
-					.filter(Boolean) || [];
 
 			const shouldDeployPaths = shouldDeploy(
 				composeResult.watchPaths,

apps/dokploy/pages/api/deploy/github.ts

@@ -213,14 +213,9 @@ export default async function handler(
 			const deploymentTitle = extractCommitMessage(req.headers, req.body);
 			const deploymentHash = extractHash(req.headers, req.body);
 			const owner = githubBody?.repository?.owner?.name;
-			const normalizedCommits =
+			const normalizedCommits = githubBody?.commits?.flatMap(
-				githubBody?.commits
+				(commit: any) => commit.modified,
-					?.flatMap((commit: any) => [
+			);
-						...(commit.modified || []),
-						...(commit.added || []),
-						...(commit.removed || []),
-					])
-					.filter(Boolean) || [];
 
 			const apps = await db.query.applications.findMany({
 				where: and(

apps/dokploy/server/api/routers/compose.ts

@@ -61,6 +61,7 @@ import {
 	apiFindCompose,
 	apiRandomizeCompose,
 	apiRedeployCompose,
+	apiSaveEnvironmentVariablesCompose,
 	apiUpdateCompose,
 	compose as composeTable,
 	environments,
@@ -201,6 +202,31 @@ export const composeRouter = createTRPCRouter({
 			});
 			return updated;
 		}),
+	saveEnvironment: protectedProcedure
+		.input(apiSaveEnvironmentVariablesCompose)
+		.mutation(async ({ input, ctx }) => {
+			await checkServicePermissionAndAccess(ctx, input.composeId, {
+				envVars: ["write"],
+			});
+			const updated = await updateCompose(input.composeId, {
+				env: input.env,
+			});
+
+			if (!updated) {
+				throw new TRPCError({
+					code: "BAD_REQUEST",
+					message: "Error adding environment variables",
+				});
+			}
+
+			await audit(ctx, {
+				action: "update",
+				resourceType: "compose",
+				resourceId: input.composeId,
+				resourceName: updated?.name,
+			});
+			return true;
+		}),
 	delete: protectedProcedure
 		.input(apiDeleteCompose)
 		.mutation(async ({ input, ctx }) => {
@@ -290,7 +316,7 @@ export const composeRouter = createTRPCRouter({
 		.input(apiFetchServices)
 		.query(async ({ input, ctx }) => {
 			await checkServicePermissionAndAccess(ctx, input.composeId, {
-				service: ["create"],
+				service: ["read"],
 			});
 			return await loadServices(input.composeId, input.type);
 		}),

apps/dokploy/server/api/routers/notification.ts

@@ -677,7 +677,10 @@ export const notificationRouter = createTRPCRouter({
 			} catch (error) {
 				throw new TRPCError({
 					code: "BAD_REQUEST",
-					message: "Error testing the notification",
+					message:
+						error instanceof Error
+							? `Error testing the notification: ${error.message}`
+							: "Error testing the notification",
 					cause: error,
 				});
 			}

packages/server/src/db/schema/compose.ts

@@ -225,6 +225,13 @@ export const apiUpdateCompose = createSchema
 	})
 	.omit({ serverId: true });
 
+export const apiSaveEnvironmentVariablesCompose = createSchema
+	.pick({
+		composeId: true,
+		env: true,
+	})
+	.required();
+
 export const apiRandomizeCompose = createSchema
 	.pick({
 		composeId: true,

packages/server/src/services/compose.ts

@@ -251,15 +251,22 @@ export const deployCompose = async ({
 		} else {
 			await execAsync(commandWithLog);
 		}
-		command = "set -e;";
 		if (compose.sourceType !== "raw") {
+			command = "set -e;";
 			command += await generateApplyPatchesCommand({
 				id: compose.composeId,
 				type: "compose",
 				serverId: compose.serverId,
 			});
+			commandWithLog = `(${command}) >> ${deployment.logPath} 2>&1`;
+			if (compose.serverId) {
+				await execAsyncRemote(compose.serverId, commandWithLog);
+			} else {
+				await execAsync(commandWithLog);
+			}
 		}
 
+		command = "set -e;";
 		command += await getBuildComposeCommand(entity);
 		commandWithLog = `(${command}) >> ${deployment.logPath} 2>&1`;
 		if (compose.serverId) {
@@ -357,6 +364,23 @@ export const rebuildCompose = async ({
 		} else {
 			await execAsync(commandWithLog);
 		}
+
+		if (compose.sourceType !== "raw") {
+			command = "set -e;";
+			command += await generateApplyPatchesCommand({
+				id: compose.composeId,
+				type: "compose",
+				serverId: compose.serverId,
+			});
+			commandWithLog = `(${command}) >> ${deployment.logPath} 2>&1`;
+			if (compose.serverId) {
+				await execAsyncRemote(compose.serverId, commandWithLog);
+			} else {
+				await execAsync(commandWithLog);
+			}
+		}
+
+		command = "set -e;";
 		command += await getBuildComposeCommand(compose);
 		commandWithLog = `(${command}) >> ${deployment.logPath} 2>&1`;
 		if (compose.serverId) {

packages/server/src/utils/notifications/build-error.ts

@@ -240,14 +240,13 @@ export const sendBuildErrorNotifications = async ({
 									value: `\`\`\`${errorMessage}\`\`\``,
 									short: false,
 								},
-							],
-							actions: [
 								{
-									type: "button",
+									title: "Details",
-									text: "View Build Details",
+									value: `<${buildLink}|View Build Details>`,
-									url: buildLink,
+									short: false,
 								},
 							],
+							mrkdwn_in: ["fields"],
 						},
 					],
 				});

packages/server/src/utils/notifications/build-success.ts

@@ -256,14 +256,13 @@ export const sendBuildSuccessNotifications = async ({
 									value: date.toLocaleString(),
 									short: true,
 								},
-							],
-							actions: [
 								{
-									type: "button",
+									title: "Details",
-									text: "View Build Details",
+									value: `<${buildLink}|View Build Details>`,
-									url: buildLink,
+									short: false,
 								},
 							],
+							mrkdwn_in: ["fields"],
 						},
 					],
 				});