Implement robust logging with flexi_logger and update CI to verify logs

.gitea/workflows/smoke-tests.yml

@@ -179,37 +179,24 @@ jobs:
             exit 1
           fi
           
-          MS_LOG=$(ls -t logs/mudserver_*.log 2>/dev/null | head -n 1)
-          CB_LOG=$(ls -t logs/combat_*.log 2>/dev/null | head -n 1)
-          
           FAILED=0
           
-          if [ -z "$MS_LOG" ]; then
+          echo "Checking mudserver logs..."
-            echo "Error: no mudserver log files found"
+          grep -q "World '.*': .* rooms" logs/mudserver_*.log || { echo "Failed: World loading log missing"; FAILED=1; }
-            FAILED=1
+          grep -q "MUD server listening on" logs/mudserver_*.log || { echo "Failed: Listen log missing"; FAILED=1; }
-          else
+          grep -q "New character created: smoketest" logs/mudserver_*.log || { echo "Failed: smoketest creation log missing"; FAILED=1; }
-            echo "Checking mudserver log: $MS_LOG"
+          grep -q "Admin action: registration setting updated: '.*'" logs/mudserver_*.log || { echo "Failed: Admin action log missing"; FAILED=1; }
-            grep -q "World '.*': .* rooms" "$MS_LOG" || { echo "Failed: World loading log missing"; FAILED=1; }
-            grep -q "MUD server listening on" "$MS_LOG" || { echo "Failed: Listen log missing"; FAILED=1; }
-            grep -q "New character created: smoketest" "$MS_LOG" || { echo "Failed: smoketest creation log missing"; FAILED=1; }
-            grep -q "Admin action: registration setting updated: '.*'" "$MS_LOG" || { echo "Failed: Admin action log missing"; FAILED=1; }
-          fi
           
-          if [ -z "$CB_LOG" ]; then
+          echo "Checking combat logs..."
-            echo "Error: no combat log files found"
+          grep -q "Combat: Player 'smoketest' (ID .*) engaged NPC 'Shadowy Thief'" logs/combat_*.log || { echo "Failed: Combat engagement log missing"; FAILED=1; }
-            FAILED=1
+          grep -q "Combat: Player 'smoketest' (ID .*) killed NPC 'Shadowy Thief'" logs/combat_*.log || { echo "Failed: NPC kill log missing"; FAILED=1; }
-          else
-            echo "Checking combat log: $CB_LOG"
-            grep -q "Combat: Player 'smoketest' (ID .*) engaged NPC 'Shadowy Thief'" "$CB_LOG" || { echo "Failed: Combat engagement log missing"; FAILED=1; }
-            grep -q "Combat: Player 'smoketest' (ID .*) killed NPC 'Shadowy Thief'" "$CB_LOG" || { echo "Failed: NPC kill log missing"; FAILED=1; }
-          fi
           
           if [ $FAILED -ne 0 ]; then
             echo "--- LOG VERIFICATION FAILED ---"
             echo "--- MUDSERVER LOG CONTENTS ---"
-            [ -n "$MS_LOG" ] && cat "$MS_LOG"
+            cat logs/mudserver_*.log
             echo "--- COMBAT LOG CONTENTS ---"
-            [ -n "$CB_LOG" ] && cat "$CB_LOG"
+            cat logs/combat_*.log
             exit 1
           fi
           echo "Logging verification passed."

src/main.rs

@@ -92,6 +92,7 @@ async fn main() {
             Naming::Numbers,
             Cleanup::KeepLogFiles(7),
         )
+        .append()
         .write_mode(WriteMode::Direct)
         .try_build()
         .unwrap();
@@ -99,6 +100,7 @@ async fn main() {
     Logger::try_with_str(&log_level)
         .unwrap()
         .log_to_file(FileSpec::default().directory(&log_dir).basename("mudserver"))
+        .append()
         .duplicate_to_stderr(Duplicate::All)
         .rotate(
             Criterion::Size(10_000_000), // 10 MB