[autofix.ci] apply automated fixes

commit.modified can be undefined causing micromatch to throw
"Expected input to be a string". Also includes added and removed
paths from commits so watch paths can match against all changed files.

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.saveEnvironment.useMutation(),
+		compose: () => api.compose.update.useMutation(),
-		libsql: () => api.libsql.saveEnvironment.useMutation(),
+		libsql: () => api.libsql.update.useMutation(),
-		mariadb: () => api.mariadb.saveEnvironment.useMutation(),
+		mariadb: () => api.mariadb.update.useMutation(),
-		mongo: () => api.mongo.saveEnvironment.useMutation(),
+		mongo: () => api.mongo.update.useMutation(),
-		mysql: () => api.mysql.saveEnvironment.useMutation(),
+		mysql: () => api.mysql.update.useMutation(),
-		postgres: () => api.postgres.saveEnvironment.useMutation(),
+		postgres: () => api.postgres.update.useMutation(),
-		redis: () => api.redis.saveEnvironment.useMutation(),
+		redis: () => api.redis.update.useMutation(),
 	};
 	const { mutateAsync, isPending } = mutationMap[type]
 		? mutationMap[type]()
-		: api.mongo.saveEnvironment.useMutation();
+		: api.mongo.update.useMutation();
 
 	const form = useForm<EnvironmentSchema>({
 		defaultValues: {

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

@@ -1,14 +1,7 @@
+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;
@@ -59,36 +52,7 @@ export const ToggleDockerCleanup = ({ serverId }: Props) => {
 	return (
 		<div className="flex items-center gap-4">
 			<Switch checked={!!enabled} onCheckedChange={handleToggle} />
-			<TooltipProvider delayDuration={0}>
+			<Label className="text-primary">Daily Docker Cleanup</Label>
-				<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,9 +53,14 @@ export default async function handler(
 
 		if (sourceType === "github") {
 			const branchName = extractBranchName(req.headers, req.body);
-			const normalizedCommits = req.body?.commits?.flatMap(
+			const normalizedCommits =
-				(commit: any) => commit.modified,
+				req.body?.commits
-			);
+					?.flatMap((commit: any) => [
+						...(commit.modified || []),
+						...(commit.added || []),
+						...(commit.removed || []),
+					])
+					.filter(Boolean) || [];
 
 			const shouldDeployPaths = shouldDeploy(
 				composeResult.watchPaths,
@@ -73,9 +78,14 @@ export default async function handler(
 			}
 		} else if (sourceType === "gitlab") {
 			const branchName = extractBranchName(req.headers, req.body);
-			const normalizedCommits = req.body?.commits?.flatMap(
+			const normalizedCommits =
-				(commit: any) => commit.modified,
+				req.body?.commits
-			);
+					?.flatMap((commit: any) => [
+						...(commit.modified || []),
+						...(commit.added || []),
+						...(commit.removed || []),
+					])
+					.filter(Boolean) || [];
 
 			const shouldDeployPaths = shouldDeploy(
 				composeResult.watchPaths,
@@ -124,17 +134,32 @@ export default async function handler(
 			let normalizedCommits: string[] = [];
 
 			if (provider === "github") {
-				normalizedCommits = req.body?.commits?.flatMap(
+				normalizedCommits =
-					(commit: any) => commit.modified,
+					req.body?.commits
-				);
+						?.flatMap((commit: any) => [
+							...(commit.modified || []),
+							...(commit.added || []),
+							...(commit.removed || []),
+						])
+						.filter(Boolean) || [];
 			} else if (provider === "gitlab") {
-				normalizedCommits = req.body?.commits?.flatMap(
+				normalizedCommits =
-					(commit: any) => commit.modified,
+					req.body?.commits
-				);
+						?.flatMap((commit: any) => [
+							...(commit.modified || []),
+							...(commit.added || []),
+							...(commit.removed || []),
+						])
+						.filter(Boolean) || [];
 			} else if (provider === "gitea") {
-				normalizedCommits = req.body?.commits?.flatMap(
+				normalizedCommits =
-					(commit: any) => commit.modified,
+					req.body?.commits
-				);
+						?.flatMap((commit: any) => [
+							...(commit.modified || []),
+							...(commit.added || []),
+							...(commit.removed || []),
+						])
+						.filter(Boolean) || [];
 			}
 
 			const shouldDeployPaths = shouldDeploy(
@@ -149,9 +174,14 @@ export default async function handler(
 		} else if (sourceType === "gitea") {
 			const branchName = extractBranchName(req.headers, req.body);
 
-			const normalizedCommits = req.body?.commits?.flatMap(
+			const normalizedCommits =
-				(commit: any) => commit.modified,
+				req.body?.commits
-			);
+					?.flatMap((commit: any) => [
+						...(commit.modified || []),
+						...(commit.added || []),
+						...(commit.removed || []),
+					])
+					.filter(Boolean) || [];
 
 			const shouldDeployPaths = shouldDeploy(
 				composeResult.watchPaths,

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

@@ -213,9 +213,14 @@ 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 = githubBody?.commits?.flatMap(
+			const normalizedCommits =
-				(commit: any) => commit.modified,
+				githubBody?.commits
-			);
+					?.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,7 +61,6 @@ import {
 	apiFindCompose,
 	apiRandomizeCompose,
 	apiRedeployCompose,
-	apiSaveEnvironmentVariablesCompose,
 	apiUpdateCompose,
 	compose as composeTable,
 	environments,
@@ -202,31 +201,6 @@ 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 }) => {
@@ -316,7 +290,7 @@ export const composeRouter = createTRPCRouter({
 		.input(apiFetchServices)
 		.query(async ({ input, ctx }) => {
 			await checkServicePermissionAndAccess(ctx, input.composeId, {
-				service: ["read"],
+				service: ["create"],
 			});
 			return await loadServices(input.composeId, input.type);
 		}),

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

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

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

@@ -225,13 +225,6 @@ 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,22 +251,15 @@ 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) {
@@ -364,23 +357,6 @@ 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,13 +240,14 @@ export const sendBuildErrorNotifications = async ({
 									value: `\`\`\`${errorMessage}\`\`\``,
 									short: false,
 								},
+							],
+							actions: [
 								{
-									title: "Details",
+									type: "button",
-									value: `<${buildLink}|View Build Details>`,
+									text: "View Build Details",
-									short: false,
+									url: buildLink,
 								},
 							],
-							mrkdwn_in: ["fields"],
 						},
 					],
 				});

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

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