Add maxTotalChatBackups config.yaml value

3849908fe1322436d6fa437eb0c48957d8772cf2

Cohee <18619528+Cohee1207@users.noreply.github.com>

default/config.yaml+2 -0 Viewed

 disableChatBackup: false
 # Number of backups to keep for each chat and settings file
 numberOfBackups: 50
+# Maximum number of chat backups to keep per user (starting from the most recent). Set to -1 to keep all backups.
+maxTotalChatBackups: 500
 # Interval in milliseconds to throttle chat backups per user
 chatBackupThrottleInterval: 10000
 # Allowed hosts for card downloads

src/endpoints/chats.js+7 -0 Viewed

         writeFileAtomicSync(backupFile, chat, 'utf-8');
         removeOldBackups(directory, `chat_${name}_`);
+        const maxTotalChatBackups = Number(getConfigValue('maxTotalChatBackups', 500));
+        if (isNaN(maxTotalChatBackups) || maxTotalChatBackups < 0) {
+            return;
+        }
+        removeOldBackups(directory, 'chat_', maxTotalChatBackups);
     } catch (err) {
         console.log(`Could not backup chat for ${name}`, err);
+    }

src/util.js+11 -3 Viewed

  * Remove old backups with the given prefix from a specified directory.
  * @param {string} directory The root directory to remove backups from.
  * @param {string} prefix File prefix to filter backups by.
+ * @param {number?} limit Maximum number of backups to keep. If null, the limit is determined by the `numberOfBackups` config value.
  */
-export function removeOldBackups(directory, prefix) {
+export function removeOldBackups(directory, prefix, limit = null) {
-    const MAX_BACKUPS = Number(getConfigValue('numberOfBackups', 50));
+    const MAX_BACKUPS = limit ?? Number(getConfigValue('numberOfBackups', 50));
     let files = fs.readdirSync(directory).filter(f => f.startsWith(prefix));
     if (files.length > MAX_BACKUPS) {
         files = files.map(f => path.join(directory, f));
         files.sort((a, b) => fs.statSync(a).mtimeMs - fs.statSync(b).mtimeMs);
-        fs.rmSync(files[0]);
+        while (files.length > MAX_BACKUPS) {
+            const oldest = files.shift();
+            if (!oldest) {
+                break;
+            }
+            fs.rmSync(oldest);
+        }
+    }
+}