datadog-mcp 1.0.2 → 1.0.4

package/README.md

@@ -1,9 +1,9 @@
 # Datadog MCP Server
 
+[![Quality gate](https://sonarcloud.io/api/project_badges/quality_gate?project=TANTIOPE_datadog-mcp-server)](https://sonarcloud.io/summary/new_code?id=TANTIOPE_datadog-mcp-server)
 [![CI](https://github.com/tantiope/datadog-mcp-server/actions/workflows/ci.yml/badge.svg)](https://github.com/tantiope/datadog-mcp-server/actions/workflows/ci.yml)
 [![npm](https://img.shields.io/npm/v/datadog-mcp)](https://www.npmjs.com/package/datadog-mcp)
 [![License](https://img.shields.io/badge/License-Apache_2.0-blue.svg)](https://opensource.org/licenses/Apache-2.0)
-[![Quality Gate](https://sonarcloud.io/api/project_badges/measure?project=TANTIOPE_datadog-mcp-server&metric=alert_status)](https://sonarcloud.io/summary/new_code?id=TANTIOPE_datadog-mcp-server)
 [![Coverage](https://sonarcloud.io/api/project_badges/measure?project=TANTIOPE_datadog-mcp-server&metric=coverage)](https://sonarcloud.io/summary/new_code?id=TANTIOPE_datadog-mcp-server)
 
 > **DISCLAIMER**: This is a community-maintained project and is not officially affiliated with, endorsed by, or supported by Datadog, Inc. This MCP server utilizes the Datadog API but is developed independently.
@@ -80,6 +80,26 @@ DD_SITE=datadoghq.com  # Default. Use datadoghq.eu for EU, etc.
 }
 ```
 
+### Kubernetes
+
+**Use environment variables instead of container args:**
+
+```yaml
+env:
+  - name: DD_API_KEY
+    value: "your-api-key"
+  - name: DD_APP_KEY
+    value: "your-app-key"
+  - name: MCP_TRANSPORT
+    value: "http"
+  - name: MCP_PORT
+    value: "3000"
+  - name: MCP_HOST
+    value: "0.0.0.0"
+```
+
+> **Note:** Kubernetes `args:` replaces the entire Dockerfile CMD, causing Node.js to receive the flags instead of your application. Environment variables avoid this issue.
+
 ### HTTP Transport
 
 When running with `--transport=http`:

package/dist/index.js

@@ -35,7 +35,7 @@ var configSchema = z.object({
     transport: z.enum(["stdio", "http"]).default("stdio"),
     port: z.number().default(3e3),
     host: z.string().default("localhost")
-  }),
+  }).default({}),
   limits: z.object({
     maxResults: z.number().default(100),
     maxLogLines: z.number().default(100),
@@ -44,7 +44,7 @@ var configSchema = z.object({
     // Default limit for initial queries
     maxMetricDataPoints: z.number().default(1e3),
     defaultTimeRangeHours: z.number().default(24)
-  }),
+  }).default({}),
   features: z.object({
     readOnly: z.boolean().default(false),
     disabledTools: z.array(z.string()).default([])
@@ -52,6 +52,22 @@ var configSchema = z.object({
 });
 
 // src/config/index.ts
+function parseEqualsFormat(arg) {
+  if (!arg.includes("=")) return null;
+  const parts = arg.slice(2).split("=");
+  const key = parts[0];
+  const value = parts.slice(1).join("=");
+  return key && value !== void 0 ? [key, value] : null;
+}
+function parseSpacedFormat(arg, nextArg) {
+  if (nextArg && !nextArg.startsWith("--")) {
+    return [arg.slice(2), nextArg];
+  }
+  return null;
+}
+function parseBooleanFlag(arg) {
+  return arg.slice(2);
+}
 function parseArgs() {
   const strings = {};
   const booleans = /* @__PURE__ */ new Set();
@@ -60,23 +76,20 @@ function parseArgs() {
     const arg = argv[i];
     if (!arg) continue;
     if (arg.startsWith("--")) {
-      if (arg.includes("=")) {
-        const parts = arg.slice(2).split("=");
-        const key = parts[0];
-        const value = parts.slice(1).join("=");
-        if (key && value !== void 0) {
-          strings[key] = value;
-        }
-      } else {
-        const argName = arg.slice(2);
-        const nextArg = argv[i + 1];
-        if (nextArg && !nextArg.startsWith("--")) {
-          strings[argName] = nextArg;
-          i += 1;
-        } else {
-          booleans.add(argName);
-        }
+      const equalsResult = parseEqualsFormat(arg);
+      if (equalsResult) {
+        const [key, value] = equalsResult;
+        strings[key] = value;
+        continue;
       }
+      const spacedResult = parseSpacedFormat(arg, argv[i + 1]);
+      if (spacedResult) {
+        const [key, value] = spacedResult;
+        strings[key] = value;
+        i += 1;
+        continue;
+      }
+      booleans.add(parseBooleanFlag(arg));
     }
   }
   return { strings, booleans };
@@ -261,7 +274,7 @@ function formatResponse(data) {
   return [
     {
       type: "text",
-      text: JSON.stringify(data, null, 2)
+      text: JSON.stringify(data, null, 2) ?? "null"
     }
   ];
 }
@@ -1136,7 +1149,7 @@ TOKEN TIP: Use compact:true to reduce payload size (strips heavy fields) when qu
             );
           }
           case "aggregate": {
-            const aggregateQuery = requireParam(query, "query", "aggregate");
+            const aggregateQuery = query ?? "*";
             return toolResult(
               await aggregateLogs(
                 api,
@@ -1399,6 +1412,18 @@ function formatSpan(span) {
     tags
   };
 }
+function buildHttpStatusFilter(httpStatus) {
+  const status = httpStatus.toLowerCase();
+  if (status.endsWith("xx")) {
+    const base = Number.parseInt(status[0] ?? "0", 10) * 100;
+    return `@http.status_code:[${base} TO ${base + 99}]`;
+  }
+  if (status.startsWith(">=")) return `@http.status_code:>=${status.slice(2)}`;
+  if (status.startsWith(">")) return `@http.status_code:>${status.slice(1)}`;
+  if (status.startsWith("<=")) return `@http.status_code:<=${status.slice(2)}`;
+  if (status.startsWith("<")) return `@http.status_code:<${status.slice(1)}`;
+  return `@http.status_code:${httpStatus}`;
+}
 function buildTraceQuery(params) {
   const parts = [];
   if (params.query) {
@@ -1432,21 +1457,7 @@ function buildTraceQuery(params) {
     }
   }
   if (params.httpStatus) {
-    const status = params.httpStatus.toLowerCase();
-    if (status.endsWith("xx")) {
-      const base = Number.parseInt(status[0] ?? "0", 10) * 100;
-      parts.push(`@http.status_code:[${base} TO ${base + 99}]`);
-    } else if (status.startsWith(">=")) {
-      parts.push(`@http.status_code:>=${status.slice(2)}`);
-    } else if (status.startsWith(">")) {
-      parts.push(`@http.status_code:>${status.slice(1)}`);
-    } else if (status.startsWith("<=")) {
-      parts.push(`@http.status_code:<=${status.slice(2)}`);
-    } else if (status.startsWith("<")) {
-      parts.push(`@http.status_code:<${status.slice(1)}`);
-    } else {
-      parts.push(`@http.status_code:${params.httpStatus}`);
-    }
+    parts.push(buildHttpStatusFilter(params.httpStatus));
   }
   if (params.errorType) {
     const escaped = params.errorType.replace(/"/g, '\\"');
@@ -1751,7 +1762,7 @@ function extractTitleFromMessage(message) {
   if (!message) return "";
   const content = message.replace(/^%%%\s*\n?/, "").trim();
   const firstLine = content.split("\n")[0]?.trim() ?? "";
-  return firstLine.replace(/\s+!?\s*$/, "").trim();
+  return firstLine.replace(/\s*!?\s*$/, "").trim();
 }
 function extractMonitorIdFromMessage(message) {
   if (!message) return void 0;
@@ -2086,7 +2097,7 @@ async function timeseriesEventsV2(api, params, limits, site) {
   const fullQuery = buildEventQuery({
     query: params.query ?? "source:alert",
     sources: params.sources,
-    tags: params.tags ?? ["source:alert"]
+    tags: params.tags
   });
   const intervalMs = parseIntervalToMs(params.interval);
   const groupByFields = params.groupBy ?? ["monitor_name"];
@@ -2171,7 +2182,7 @@ async function incidentsEventsV2(api, params, limits, site) {
   const fullQuery = buildEventQuery({
     query: params.query ?? "source:alert",
     sources: params.sources,
-    tags: params.tags ?? ["source:alert"]
+    tags: params.tags
   });
   const dedupeWindowNs = parseDurationToNs(params.dedupeWindow ?? "5m");
   const dedupeWindowMs = dedupeWindowNs ? Math.floor(dedupeWindowNs / 1e6) : 3e5;
@@ -4804,8 +4815,9 @@ import { randomUUID } from "crypto";
 import { StreamableHTTPServerTransport } from "@modelcontextprotocol/sdk/server/streamableHttp.js";
 import { isInitializeRequest } from "@modelcontextprotocol/sdk/types.js";
 var transports = {};
-async function connectHttp(server, config) {
+function createExpressApp(server, config) {
   const app = express();
+  app.disable("x-powered-by");
   app.use(express.json());
   app.get("/health", (_req, res) => {
     res.json({ status: "ok", name: config.name, version: config.version });
@@ -4858,6 +4870,10 @@ async function connectHttp(server, config) {
       res.status(400).json({ error: "Invalid session" });
     }
   });
+  return app;
+}
+async function connectHttp(server, config) {
+  const app = createExpressApp(server, config);
   app.listen(config.port, config.host, () => {
     console.error(`[MCP] Datadog MCP server running on http://${config.host}:${config.port}/mcp`);
     console.error(`[MCP] Health check available at http://${config.host}:${config.port}/health`);