From 6ab5c549c1672189f928cea3cc4381e1e9399e79 Mon Sep 17 00:00:00 2001
From: Nico <47644445+nicotsx@users.noreply.github.com>
Date: Fri, 12 Dec 2025 17:55:37 +0100
Subject: [PATCH] fix(doctor): fail gracefully in case of unexpected error
 (#132)

---
 app/client/modules/repositories/tabs/info.tsx    |  8 ++++----
 .../modules/repositories/repositories.service.ts | 16 ++++++++++++----
 2 files changed, 16 insertions(+), 8 deletions(-)

diff --git a/app/client/modules/repositories/tabs/info.tsx b/app/client/modules/repositories/tabs/info.tsx
index bd198ea8..ac56cbd7 100644
--- a/app/client/modules/repositories/tabs/info.tsx
+++ b/app/client/modules/repositories/tabs/info.tsx
@@ -87,7 +87,7 @@ export const RepositoryInfoTabContent = ({ repository }: Props) => {
 								<p className="text-sm text-muted-foreground">Unique identifier for the repository.</p>
 							</div>
 							<div className="space-y-2">
-								<Label htmlFor="compressionMode">Compression Mode</Label>
+								<Label htmlFor="compressionMode">Compression mode</Label>
 								<Select value={compressionMode} onValueChange={(val) => setCompressionMode(val as CompressionMode)}>
 									<SelectTrigger id="compressionMode">
 										<SelectValue placeholder="Select compression mode" />
@@ -115,11 +115,11 @@ export const RepositoryInfoTabContent = ({ repository }: Props) => {
 								<p className="mt-1 text-sm">{repository.status || "unknown"}</p>
 							</div>
 							<div>
-								<div className="text-sm font-medium text-muted-foreground">Created At</div>
-								<p className="mt-1 text-sm">{new Date(repository.createdAt * 1000).toLocaleString()}</p>
+								<div className="text-sm font-medium text-muted-foreground">Created at</div>
+								<p className="mt-1 text-sm">{new Date(repository.createdAt).toLocaleString()}</p>
 							</div>
 							<div>
-								<div className="text-sm font-medium text-muted-foreground">Last Checked</div>
+								<div className="text-sm font-medium text-muted-foreground">Last checked</div>
 								<p className="mt-1 text-sm">
 									{repository.lastChecked ? new Date(repository.lastChecked).toLocaleString() : "Never"}
 								</p>
diff --git a/app/server/modules/repositories/repositories.service.ts b/app/server/modules/repositories/repositories.service.ts
index ea23a161..a06d8d58 100644
--- a/app/server/modules/repositories/repositories.service.ts
+++ b/app/server/modules/repositories/repositories.service.ts
@@ -360,23 +360,31 @@ const doctorRepository = async (name: string) => {
 				error: recheckResult.error,
 			});
 		}
+	} catch (error) {
+		steps.push({
+			step: "unexpected_error",
+			success: false,
+			output: null,
+			error: toMessage(error),
+		});
 	} finally {
 		releaseLock();
 	}
 
-	const allSuccessful = steps.every((s) => s.success);
+	const doctorSucceeded = steps.every((step) => step.success);
+	const doctorError = steps.find((step) => step.error)?.error ?? null;
 
 	await db
 		.update(repositoriesTable)
 		.set({
-			status: allSuccessful ? "healthy" : "error",
+			status: doctorSucceeded ? "healthy" : "error",
 			lastChecked: Date.now(),
-			lastError: allSuccessful ? null : steps.find((s) => !s.success)?.error,
+			lastError: doctorError,
 		})
 		.where(eq(repositoriesTable.id, repository.id));
 
 	return {
-		success: allSuccessful,
+		success: doctorSucceeded,
 		steps,
 	};
 };