app/client/components/status-dot.tsx

@@ -40,7 +40,7 @@ export const StatusDot = ({ variant, label, animated }: StatusDotProps) => {
 
 	return (
 		<Tooltip>
-			<TooltipTrigger aria-label={label}>
+			<TooltipTrigger>
 				<span className="relative flex size-3 mx-auto">
 					{statusMapping?.animated && (
 						<span

app/client/modules/backups/components/backup-card.tsx

@@ -17,8 +17,7 @@ export const BackupCard = ({ schedule }: { schedule: BackupSchedule }) => {
 						</div>
 						<BackupStatusDot
 							enabled={schedule.enabled}
-							hasError={schedule.lastBackupStatus === "error"}
+							hasError={!!schedule.lastBackupError}
-							hasWarning={schedule.lastBackupStatus === "warning"}
 							isInProgress={schedule.lastBackupStatus === "in_progress"}
 						/>
 					</div>

app/client/modules/backups/components/backup-status-dot.tsx

@@ -3,15 +3,13 @@ import { StatusDot } from "~/client/components/status-dot";
 export const BackupStatusDot = ({
 	enabled,
 	hasError,
-	hasWarning,
 	isInProgress,
 }: {
 	enabled: boolean;
 	hasError?: boolean;
-	hasWarning?: boolean;
 	isInProgress?: boolean;
 }) => {
-	let variant: "success" | "neutral" | "error" | "warning" | "info";
+	let variant: "success" | "neutral" | "error" | "info";
 	let label: string;
 
 	if (isInProgress) {
@@ -20,9 +18,6 @@ export const BackupStatusDot = ({
 	} else if (hasError) {
 		variant = "error";
 		label = "Error";
-	} else if (hasWarning) {
-		variant = "warning";
-		label = "Warning";
 	} else if (enabled) {
 		variant = "success";
 		label = "Active";

app/client/modules/backups/components/schedule-summary.tsx

@@ -20,8 +20,6 @@ import { toast } from "sonner";
 import { handleRepositoryError } from "~/client/lib/errors";
 import { formatShortDateTime, formatTimeAgo } from "~/client/lib/datetime";
 import { Link } from "@tanstack/react-router";
-import { Collapsible, CollapsibleContent, CollapsibleTrigger } from "~/client/components/ui/collapsible";
-import { cn } from "~/client/lib/utils";
 
 type Props = {
 	schedule: BackupSchedule;
@@ -200,41 +198,22 @@ export const ScheduleSummary = (props: Props) => {
 						</p>
 					</div>
 
-					{(schedule.lastBackupStatus === "warning" || schedule.lastBackupStatus === "error") && (
+					{schedule.lastBackupStatus === "warning" && (
 						<div className="@medium:col-span-2 @wide:col-span-4">
-							<Collapsible
+							<p className="text-xs uppercase text-muted-foreground">Warning Details</p>
-								className={cn("border border-border/50 rounded-lg overflow-hidden", {
+							<p className="font-mono text-sm text-yellow-600 whitespace-pre-wrap wrap-break-word">
-									"border-yellow-500/20 bg-yellow-500/5": schedule.lastBackupStatus === "warning",
-									"border-red-500/20 bg-red-500/5": schedule.lastBackupStatus === "error",
-								})}
-							>
-								<CollapsibleTrigger
-									className={cn("w-full justify-start p-3 hover:bg-muted/50 transition-colors", {
-										"hover:bg-yellow-500/10": schedule.lastBackupStatus === "warning",
-										"hover:bg-red-500/10": schedule.lastBackupStatus === "error",
-									})}
-								>
-									<span>{schedule.lastBackupStatus === "warning" ? "Warning details" : "Error details"}</span>
-								</CollapsibleTrigger>
-								<CollapsibleContent
-									className={cn("border-t border-border/50 bg-muted/30", {
-										"border-yellow-500/20 bg-yellow-500/8": schedule.lastBackupStatus === "warning",
-										"border-red-500/20 bg-red-500/8": schedule.lastBackupStatus === "error",
-									})}
-								>
-									<div className="p-3">
-										<p
-											className={cn("font-mono text-sm whitespace-pre-wrap wrap-break-word", {
-												"text-yellow-600": schedule.lastBackupStatus === "warning",
-												"text-red-600": schedule.lastBackupStatus === "error",
-											})}
-										>
 								{schedule.lastBackupError ??
-												"No additional details available. check your container logs for more information."}
+									"Last backup completed with warnings. Check your container logs for more details."}
 							</p>
 						</div>
-								</CollapsibleContent>
+					)}
-							</Collapsible>
+
+					{schedule.lastBackupError && schedule.lastBackupStatus === "error" && (
+						<div className="@medium:col-span-2 @wide:col-span-4">
+							<p className="text-xs uppercase text-muted-foreground">Error details</p>
+							<p className="font-mono text-sm text-red-600 whitespace-pre-wrap wrap-break-word">
+								{schedule.lastBackupError}
+							</p>
 						</div>
 					)}
 				</CardContent>

app/routes/api.$.ts

@@ -13,35 +13,26 @@ type NodeRuntimeRequest = Request & {
 	};
 };
 
-type RequestInitWithDuplex = RequestInit & {
+export const prepareApiRequest = (request: Request, timeoutMs: number) => {
-	duplex?: "half";
+	const nodeRequest = request as NodeRuntimeRequest;
-};
+	nodeRequest.runtime?.node?.res?.setTimeout(timeoutMs);
-
-export const prepareApiRequest = (request: NodeRuntimeRequest, timeoutMs: number) => {
-	request.runtime?.node?.res?.setTimeout(timeoutMs);
 
 	if (config.trustProxy && request.headers.has("x-forwarded-for")) {
 		return request.clone();
 	}
 
-	const remoteAddress = request.ip;
+	const remoteAddress = nodeRequest.ip;
-	const headers = new Headers(request.headers);
-
 	if (remoteAddress) {
+		const headers = new Headers(request.headers);
 		headers.set("x-forwarded-for", remoteAddress);
-	} else {
+
-		headers.delete("x-forwarded-for");
+		return new Request(request, { headers });
 	}
 
-	const init: RequestInitWithDuplex = {
+	const headers = new Headers(request.headers);
-		method: request.method,
+	headers.delete("x-forwarded-for");
-		headers,
-		body: request.body,
-		signal: request.signal,
-		duplex: request.body ? "half" : undefined,
-	};
 
-	return new Request(request.url, init);
+	return new Request(request, { headers });
 };
 
 const handle = ({ request }: { request: Request }) =>

app/server/modules/backups/__tests__/backups.execution.test.ts

@@ -10,7 +10,6 @@ import { generateBackupOutput } from "~/test/helpers/restic";
 import { TEST_ORG_ID } from "~/test/helpers/organization";
 import * as context from "~/server/core/request-context";
 import * as spawnModule from "@zerobyte/core/node";
-import type { SafeSpawnParams } from "@zerobyte/core/node";
 import { restic } from "~/server/core/restic";
 import { NotFoundError, BadRequestError } from "http-errors-enhanced";
 import { scheduleQueries } from "../backups.queries";
@@ -18,9 +17,7 @@ import { fromAny } from "@total-typescript/shoehorn";
 import { repositoriesService } from "~/server/modules/repositories/repositories.service";
 
 const setup = () => {
-	const resticBackupMock = mock((_: SafeSpawnParams) =>
+	const resticBackupMock = mock(() => Promise.resolve({ exitCode: 0, summary: generateBackupOutput(), error: "" }));
-		Promise.resolve({ exitCode: 0, summary: generateBackupOutput(), error: "" }),
-	);
 	const resticForgetMock = mock(() => Promise.resolve({ success: true, data: null }));
 	const resticCopyMock = mock(() => Promise.resolve({ success: true, output: "" }));
 	const refreshStatsMock = mock(() =>
@@ -151,32 +148,6 @@ describe("backup execution - validation failures", () => {
 });
 
 describe("stop backup", () => {
-	test("should keep restic warning details when backup completes with read errors", async () => {
-		const { resticBackupMock } = setup();
-		const volume = await createTestVolume();
-		const repository = await createTestRepository();
-		const schedule = await createTestBackupSchedule({
-			volumeId: volume.id,
-			repositoryId: repository.id,
-		});
-
-		resticBackupMock.mockImplementationOnce((params: SafeSpawnParams) => {
-			params.onStderr?.("error: open /mnt/data/private.db: permission denied");
-
-			return Promise.resolve({
-				exitCode: 3,
-				summary: generateBackupOutput(),
-				error: "Warning: at least one source file could not be read",
-			});
-		});
-
-		await backupsExecutionService.executeBackup(schedule.id);
-
-		const updatedSchedule = await backupsService.getScheduleById(schedule.id);
-		expect(updatedSchedule.lastBackupStatus).toBe("warning");
-		expect(updatedSchedule.lastBackupError).toBe("error: open /mnt/data/private.db: permission denied");
-	});
-
 	test("should stop a running backup", async () => {
 		// arrange
 		const { resticBackupMock } = setup();

app/server/modules/backups/backups.execution.ts

@@ -149,7 +149,6 @@ const finalizeSuccessfulBackup = async (
 	scheduleId: number,
 	exitCode: number,
 	result: ResticBackupOutputDto | null,
-	warningDetails: string | null,
 ) => {
 	const finalStatus = exitCode === 0 ? "success" : "warning";
 
@@ -175,7 +174,7 @@ const finalizeSuccessfulBackup = async (
 	await scheduleQueries.updateStatus(scheduleId, ctx.organizationId, {
 		lastBackupAt: Date.now(),
 		lastBackupStatus: finalStatus,
-		lastBackupError: finalStatus === "warning" ? warningDetails : null,
+		lastBackupError: null,
 		nextBackupAt,
 	});
 
@@ -286,13 +285,7 @@ const executeBackup = async (scheduleId: number, manual = false): Promise<void>
 
 	try {
 		const backupResult = await runBackupOperation(ctx, abortController.signal);
-		await finalizeSuccessfulBackup(
+		await finalizeSuccessfulBackup(ctx, scheduleId, backupResult.exitCode, backupResult.result);
-			ctx,
-			scheduleId,
-			backupResult.exitCode,
-			backupResult.result,
-			backupResult.warningDetails,
-		);
 	} catch (error) {
 		await handleBackupFailure(scheduleId, ctx.organizationId, error, ctx);
 	} finally {

packages/core/src/restic/commands/backup.ts

@@ -101,7 +101,6 @@ export const backup = async (
 	const logData = throttle((data: string) => {
 		logger.info(data.trim());
 	}, 5000);
-	const stderrLines: string[] = [];
 
 	const streamProgress = throttle((data: string) => {
 		if (options.onProgress) {
@@ -135,13 +134,6 @@ export const backup = async (
 				streamProgress(data);
 			}
 		},
-		onStderr: (error) => {
-			const line = error.trim();
-			if (line.length > 0) {
-				stderrLines.push(line);
-				logger.error(`restic stderr: ${line}`);
-			}
-		},
 	});
 
 	if (includeFile) {
@@ -154,7 +146,7 @@ export const backup = async (
 
 	if (options.signal?.aborted) {
 		logger.warn("Restic backup was aborted by signal.");
-		return { result: null, exitCode: res.exitCode, warningDetails: null };
+		return { result: null, exitCode: res.exitCode };
 	}
 
 	if (res.exitCode === 3) {
@@ -182,16 +174,8 @@ export const backup = async (
 
 	if (!result.success) {
 		logger.error(`Restic backup output validation failed: ${result.error.message}`);
-		return {
+		return { result: null, exitCode: res.exitCode };
-			result: null,
-			exitCode: res.exitCode,
-			warningDetails: stderrLines.length > 0 ? stderrLines.join("\n") : null,
-		};
 	}
 
-	return {
+	return { result: result.data, exitCode: res.exitCode };
-		result: result.data,
-		exitCode: res.exitCode,
-		warningDetails: stderrLines.length > 0 ? stderrLines.join("\n") : null,
-	};
 };