chore(logs): rotate daily and prune after 24h

src/logger.test.ts

@@ -7,7 +7,11 @@ import { afterEach, describe, expect, it, vi } from "vitest";
 
 import { setVerbose } from "./globals.js";
 import { logDebug, logError, logInfo, logSuccess, logWarn } from "./logger.js";
-import { resetLogger, setLoggerOverride } from "./logging.js";
+import {
+  DEFAULT_LOG_DIR,
+  resetLogger,
+  setLoggerOverride,
+} from "./logging.js";
 import type { RuntimeEnv } from "./runtime.js";
 
 describe("logger helpers", () => {
@@ -67,6 +71,28 @@ describe("logger helpers", () => {
     expect(content).toContain("warn-only");
     cleanup(logPath);
   });
+
+  it("uses daily rolling default log file and prunes old ones", () => {
+    resetLogger();
+    setLoggerOverride({}); // force defaults regardless of user config
+    const today = new Date().toISOString().slice(0, 10);
+    const todayPath = path.join(DEFAULT_LOG_DIR, `warelay-${today}.log`);
+
+    // create an old file to be pruned
+    const oldPath = path.join(DEFAULT_LOG_DIR, "warelay-2000-01-01.log");
+    fs.mkdirSync(DEFAULT_LOG_DIR, { recursive: true });
+    fs.writeFileSync(oldPath, "old");
+    fs.utimesSync(oldPath, new Date(0), new Date(0));
+    cleanup(todayPath);
+
+    logInfo("roll-me");
+
+    expect(fs.existsSync(todayPath)).toBe(true);
+    expect(fs.readFileSync(todayPath, "utf-8")).toContain("roll-me");
+    expect(fs.existsSync(oldPath)).toBe(false);
+
+    cleanup(todayPath);
+  });
 });
 
 function pathForTest() {

src/logging.ts

@@ -6,8 +6,12 @@ import pino, { type Bindings, type LevelWithSilent, type Logger } from "pino";
 import { loadConfig, type WarelayConfig } from "./config/config.js";
 import { isVerbose } from "./globals.js";
 
-const DEFAULT_LOG_DIR = path.join(os.tmpdir(), "warelay");
-export const DEFAULT_LOG_FILE = path.join(DEFAULT_LOG_DIR, "warelay.log");
+export const DEFAULT_LOG_DIR = path.join(os.tmpdir(), "warelay");
+export const DEFAULT_LOG_FILE = path.join(DEFAULT_LOG_DIR, "warelay.log"); // legacy single-file path
+
+const LOG_PREFIX = "warelay";
+const LOG_SUFFIX = ".log";
+const MAX_LOG_AGE_MS = 24 * 60 * 60 * 1000; // 24h
 
 const ALLOWED_LEVELS: readonly LevelWithSilent[] = [
   "silent",
@@ -46,7 +50,7 @@ function resolveSettings(): ResolvedSettings {
   const cfg: WarelayConfig["logging"] | undefined =
     overrideSettings ?? loadConfig().logging;
   const level = normalizeLevel(cfg?.level);
-  const file = cfg?.file ?? DEFAULT_LOG_FILE;
+  const file = cfg?.file ?? defaultRollingPathForToday();
   return { level, file };
 }
 
@@ -57,6 +61,10 @@ function settingsChanged(a: ResolvedSettings | null, b: ResolvedSettings) {
 
 function buildLogger(settings: ResolvedSettings): Logger {
   fs.mkdirSync(path.dirname(settings.file), { recursive: true });
+  // Clean up stale rolling logs when using a dated log filename.
+  if (isRollingPath(settings.file)) {
+    pruneOldRollingLogs(path.dirname(settings.file));
+  }
   const destination = pino.destination({
     dest: settings.file,
     mkdir: true,
@@ -104,3 +112,39 @@ export function resetLogger() {
   cachedSettings = null;
   overrideSettings = null;
 }
+
+function defaultRollingPathForToday(): string {
+  const today = new Date().toISOString().slice(0, 10); // YYYY-MM-DD
+  return path.join(DEFAULT_LOG_DIR, `${LOG_PREFIX}-${today}${LOG_SUFFIX}`);
+}
+
+function isRollingPath(file: string): boolean {
+  const base = path.basename(file);
+  return (
+    base.startsWith(`${LOG_PREFIX}-`) &&
+    base.endsWith(LOG_SUFFIX) &&
+    base.length === `${LOG_PREFIX}-YYYY-MM-DD${LOG_SUFFIX}`.length
+  );
+}
+
+function pruneOldRollingLogs(dir: string): void {
+  try {
+    const entries = fs.readdirSync(dir, { withFileTypes: true });
+    const cutoff = Date.now() - MAX_LOG_AGE_MS;
+    for (const entry of entries) {
+      if (!entry.isFile()) continue;
+      if (!entry.name.startsWith(`${LOG_PREFIX}-`) || !entry.name.endsWith(LOG_SUFFIX)) continue;
+      const fullPath = path.join(dir, entry.name);
+      try {
+        const stat = fs.statSync(fullPath);
+        if (stat.mtimeMs < cutoff) {
+          fs.rmSync(fullPath, { force: true });
+        }
+      } catch {
+        // ignore errors during pruning
+      }
+    }
+  } catch {
+    // ignore missing dir or read errors
+  }
+}