Retention policy for log files

Author: CapulusCodeNinja

nexus_config/Internal/LoggingConfiguration.cs

@@ -47,6 +47,7 @@ protected virtual void ConfigureNLogDynamically(Microsoft.Extensions.Logging.Log
         var nlogConfig = LogManager.Configuration ?? new NLog.Config.LoggingConfiguration();
 
         // Ensure main file target exists
+        var retentionDays = m_Settings.Get().Logging.RetentionDays;
         if (nlogConfig.FindTargetByName("mainFile") is not NLog.Targets.FileTarget)
         {
             var fileTarget = new NLog.Targets.FileTarget("mainFile")
@@ -55,7 +56,7 @@ protected virtual void ConfigureNLogDynamically(Microsoft.Extensions.Logging.Log
                 ArchiveFileName = Path.Combine(AppDomain.CurrentDomain.BaseDirectory, "archive", "mcp-nexus-${shortdate}-{##}.log"),
                 ArchiveEvery = NLog.Targets.FileArchivePeriod.Day,
                 ArchiveSuffixFormat = "{#}",
-                MaxArchiveFiles = 356,
+                MaxArchiveFiles = Math.Max(1, retentionDays),
                 KeepFileOpen = true,
                 AutoFlush = true,
                 CreateDirs = true,

nexus_config/Models/LoggingSettings.cs

@@ -9,4 +9,10 @@ public class LoggingSettings
     /// Gets or sets the log level.
     /// </summary>
     public string LogLevel { get; set; } = "Information";
+
+    /// <summary>
+    /// Gets or sets the maximum number of archived log files to retain.
+    /// Each archived file typically represents one day of logs.
+    /// </summary>
+    public int RetentionDays { get; set; } = 7;
 }

nexus_config/appsettings.Defaults.json

@@ -1,6 +1,7 @@
 {
   "Logging": {
-    "LogLevel": "Debug"
+    "LogLevel": "Debug",
+    "RetentionDays": 7
   },
   "McpNexus": {
     "Server": {

unittests/nexus_config_unittests/Models/SharedConfigurationTests.cs

@@ -316,10 +316,12 @@ public void LoggingSettings_CanBeModified()
         var settings = new LoggingSettings
         {
             LogLevel = "Debug",
+            RetentionDays = 30,
         };
 
         // Assert
         _ = settings.LogLevel.Should().Be("Debug");
+        _ = settings.RetentionDays.Should().Be(30);
     }
 
     /// <summary>