From d687988ba84e70f4c8cbc9731c170f5f985a5d1d Mon Sep 17 00:00:00 2001
From: Jannis Braun <151788261+TheZwiss@users.noreply.github.com>
Date: Mon, 23 Mar 2026 01:53:25 +0100
Subject: [PATCH] fix: use explicit counter for danglingFilesOnDisk in
 referencedFiles calc

Matches the orphanedFiles counting pattern and avoids a subtle
discrepancy if two dangling records shared the same filename.
---
 packages/server/src/utils/storageJanitor.ts | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/packages/server/src/utils/storageJanitor.ts b/packages/server/src/utils/storageJanitor.ts
index cc2a1a0a..54cc663f 100644
--- a/packages/server/src/utils/storageJanitor.ts
+++ b/packages/server/src/utils/storageJanitor.ts
@@ -189,6 +189,7 @@ export function getStorageStats(): StorageStats {
   let referencedSize = 0;
   let orphanedFiles = 0;
   let orphanedSize = 0;
+  let danglingFilesOnDisk = 0;
   const breakdownMap = new Map<string, { count: number; size: number }>();
 
   for (const file of diskFiles) {
@@ -201,7 +202,7 @@ export function getStorageStats(): StorageStats {
 
     if (danglingFilenames.has(file.filename)) {
       // File is on disk but its attachment record points to a deleted message
-      // — counted as dangling, not referenced
+      danglingFilesOnDisk++;
     } else if (referenced.has(file.filename)) {
       referencedSize += file.size;
     } else {
@@ -224,7 +225,7 @@ export function getStorageStats(): StorageStats {
   return {
     totalFiles: diskFiles.length,
     totalSize,
-    referencedFiles: diskFiles.length - orphanedFiles - danglingFilenames.size,
+    referencedFiles: diskFiles.length - orphanedFiles - danglingFilesOnDisk,
     referencedSize,
     orphanedFiles,
     orphanedSize,