From 8bb0508dedcf317276e40419401310e94e020379 Mon Sep 17 00:00:00 2001
From: Nathan Hangen <nhangen@gmail.com>
Date: Sun, 10 May 2026 21:32:35 -0400
Subject: [PATCH] fix(mcp): log metrics init failure at error level with
 err.code + port

Operators who opt in to metrics need an actionable signal when init
fails (EADDRINUSE is the common case). Bump warn -> error and capture
err.code, address/port, and stack.
---
 gitnexus/src/cli/mcp.ts   | 10 ++++++++--
 gitnexus/src/cli/serve.ts | 10 ++++++++--
 2 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/gitnexus/src/cli/mcp.ts b/gitnexus/src/cli/mcp.ts
index e6edebdf1..741feb6f7 100644
--- a/gitnexus/src/cli/mcp.ts
+++ b/gitnexus/src/cli/mcp.ts
@@ -64,8 +64,14 @@ export const mcpCommand = async () => {
       );
     }
   } catch (err: any) {
-    logger.warn(
-      { err: err?.message ?? String(err) },
+    logger.error(
+      {
+        err: err?.message ?? String(err),
+        code: err?.code,
+        port: err?.port ?? process.env['GITNEXUS_OTEL_METRICS_PORT'],
+        host: err?.address ?? process.env['GITNEXUS_OTEL_METRICS_HOST'],
+        stack: err?.stack,
+      },
       'GitNexus: failed to start metrics endpoint — continuing without metrics',
     );
   }
diff --git a/gitnexus/src/cli/serve.ts b/gitnexus/src/cli/serve.ts
index 9f7e2c341..87d000c23 100644
--- a/gitnexus/src/cli/serve.ts
+++ b/gitnexus/src/cli/serve.ts
@@ -42,8 +42,14 @@ export const serveCommand = async (options?: { port?: string; host?: string }) =
         );
       }
     } catch (metricsErr: any) {
-      logger.warn(
-        { err: metricsErr?.message ?? String(metricsErr) },
+      logger.error(
+        {
+          err: metricsErr?.message ?? String(metricsErr),
+          code: metricsErr?.code,
+          port: metricsErr?.port ?? process.env['GITNEXUS_OTEL_METRICS_PORT'],
+          host: metricsErr?.address ?? process.env['GITNEXUS_OTEL_METRICS_HOST'],
+          stack: metricsErr?.stack,
+        },
         '[gitnexus serve] failed to start metrics endpoint — continuing without metrics',
       );
     }