From 3a3c7a79684ec44d0910238535abbaefcbe5f76f Mon Sep 17 00:00:00 2001
From: Kfir Amar <kfira2002@gmail.com>
Date: Sun, 15 Mar 2026 01:51:59 +0200
Subject: [PATCH] fix(logs): map numeric pino levels

Normalize numeric pino levels correctly in the console log API so the logger transport fix does not misclassify info, warn, and error entries in file-backed logs.

Add a targeted regression test for numeric log entries.
---
 src/app/api/logs/console/route.ts      |  8 ++--
 tests/unit/console-log-levels.test.mjs | 54 ++++++++++++++++++++++++++
 2 files changed, 58 insertions(+), 4 deletions(-)
 create mode 100644 tests/unit/console-log-levels.test.mjs

diff --git a/src/app/api/logs/console/route.ts b/src/app/api/logs/console/route.ts
index cf12058825..3c7f5a4dbd 100644
--- a/src/app/api/logs/console/route.ts
+++ b/src/app/api/logs/console/route.ts
@@ -26,10 +26,10 @@ const LEVEL_ORDER: Record<string, number> = {
 // Map pino numeric levels to string levels
 const NUMERIC_LEVEL_MAP: Record<number, string> = {
   10: "trace",
-  20: "info",
-  30: "warn",
-  40: "error",
-  50: "fatal",
+  20: "debug",
+  30: "info",
+  40: "warn",
+  50: "error",
   60: "fatal",
 };
 
diff --git a/tests/unit/console-log-levels.test.mjs b/tests/unit/console-log-levels.test.mjs
new file mode 100644
index 0000000000..d268f7aba0
--- /dev/null
+++ b/tests/unit/console-log-levels.test.mjs
@@ -0,0 +1,54 @@
+import test from "node:test";
+import assert from "node:assert/strict";
+import fs from "node:fs";
+import os from "node:os";
+import path from "node:path";
+
+const TEST_LOG_DIR = fs.mkdtempSync(path.join(os.tmpdir(), "omniroute-console-log-levels-"));
+const TEST_LOG_PATH = path.join(TEST_LOG_DIR, "app.log");
+
+const originalLogFilePath = process.env.LOG_FILE_PATH;
+process.env.LOG_FILE_PATH = TEST_LOG_PATH;
+
+const route = await import("../../src/app/api/logs/console/route.ts");
+
+test.after(() => {
+  if (originalLogFilePath === undefined) {
+    delete process.env.LOG_FILE_PATH;
+  } else {
+    process.env.LOG_FILE_PATH = originalLogFilePath;
+  }
+  fs.rmSync(TEST_LOG_DIR, { recursive: true, force: true });
+});
+
+test("console log API normalizes numeric pino levels correctly", async () => {
+  fs.writeFileSync(
+    TEST_LOG_PATH,
+    [
+      JSON.stringify({
+        timestamp: new Date().toISOString(),
+        level: 30,
+        module: "probe",
+        msg: "info entry",
+      }),
+      JSON.stringify({
+        timestamp: new Date().toISOString(),
+        level: 40,
+        module: "probe",
+        msg: "warn entry",
+      }),
+    ].join("\n") + "\n",
+    "utf8"
+  );
+
+  const response = await route.GET(
+    new Request("http://localhost/api/logs/console?level=info&limit=10")
+  );
+  const body = await response.json();
+
+  assert.equal(response.status, 200);
+  assert.deepEqual(
+    body.map((entry) => entry.level),
+    ["info", "warn"]
+  );
+});