datadog-mcp 1.1.1 → 2.0.1

package/README.md

@@ -1,7 +1,7 @@
 # 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/release.yml)
+[![CI/Release](https://github.com/tantiope/datadog-mcp-server/actions/workflows/release.yml/badge.svg)](https://github.com/tantiope/datadog-mcp-server/actions/workflows/release.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)
 [![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)
@@ -207,7 +207,7 @@ When running with `--transport=http`:
 
 ### Compact Mode (Logs)
 
-Use `compact: true` when searching logs to reduce payload size. Strips custom attributes and keeps only essential fields:
+Use `compact: true` when searching logs to reduce token usage. Strips custom attributes and keeps only essential fields:
 
 ```
 logs({ action: "search", status: "error", compact: true })

package/dist/index.js

@@ -345,13 +345,20 @@ function buildMonitorUrl(monitorId, site = "datadoghq.com") {
   const base = getAppBaseUrl(site);
   return `${base}/monitors/${monitorId}`;
 }
-function buildMonitorsListUrl(query, site = "datadoghq.com") {
+function buildMonitorsListUrl(options, site = "datadoghq.com") {
   const base = getAppBaseUrl(site);
-  if (query) {
-    const params = new URLSearchParams({ query });
-    return `${base}/monitors/manage?${params.toString()}`;
+  const params = new URLSearchParams();
+  if (options?.name) {
+    params.set("query", options.name);
   }
-  return `${base}/monitors/manage`;
+  if (options?.tags && options.tags.length > 0) {
+    params.set("tags", options.tags.join(","));
+  }
+  if (options?.groupStates && options.groupStates.length > 0) {
+    params.set("group_states", options.groupStates.join(","));
+  }
+  const queryString = params.toString();
+  return queryString ? `${base}/monitors/manage?${queryString}` : `${base}/monitors/manage`;
 }
 function buildRumUrl(query, fromSec, toSec, site = "datadoghq.com") {
   const base = getAppBaseUrl(site);
@@ -392,9 +399,10 @@ var InputSchema = {
   message: z2.string().optional().describe("Mute message (for mute action)"),
   end: z2.number().optional().describe("Mute end timestamp (for mute action)")
 };
-function formatMonitor(m) {
+function formatMonitor(m, site = "datadoghq.com") {
+  const monitorId = m.id ?? 0;
   return {
-    id: m.id ?? 0,
+    id: monitorId,
     name: m.name ?? "",
     type: String(m.type ?? "unknown"),
     status: String(m.overallState ?? "unknown"),
@@ -402,7 +410,8 @@ function formatMonitor(m) {
     tags: m.tags ?? [],
     query: m.query ?? "",
     created: m.created ? new Date(m.created).toISOString() : "",
-    modified: m.modified ? new Date(m.modified).toISOString() : ""
+    modified: m.modified ? new Date(m.modified).toISOString() : "",
+    url: buildMonitorUrl(monitorId, site)
   };
 }
 async function listMonitors(api, params, limits, site) {
@@ -412,7 +421,7 @@ async function listMonitors(api, params, limits, site) {
     tags: params.tags?.join(","),
     groupStates: params.groupStates?.join(",")
   });
-  const monitors = response.slice(0, effectiveLimit).map(formatMonitor);
+  const monitors = response.slice(0, effectiveLimit).map((m) => formatMonitor(m, site));
   const statusCounts = {
     total: response.length,
     alert: response.filter((m) => m.overallState === "Alert").length,
@@ -423,7 +432,10 @@ async function listMonitors(api, params, limits, site) {
   return {
     monitors,
     summary: statusCounts,
-    datadog_url: buildMonitorsListUrl(params.name, site)
+    datadog_url: buildMonitorsListUrl(
+      { name: params.name, tags: params.tags, groupStates: params.groupStates },
+      site
+    )
   };
 }
 async function getMonitor(api, id, site) {
@@ -433,7 +445,7 @@ async function getMonitor(api, id, site) {
   }
   const monitor = await api.getMonitor({ monitorId });
   return {
-    monitor: formatMonitor(monitor),
+    monitor: formatMonitor(monitor, site),
     datadog_url: buildMonitorUrl(monitorId, site)
   };
 }
@@ -444,7 +456,8 @@ async function searchMonitors(api, query, limits, site) {
     name: m.name ?? "",
     status: String(m.status ?? "unknown"),
     type: m.type ?? "",
-    tags: m.tags ?? []
+    tags: m.tags ?? [],
+    url: buildMonitorUrl(m.id ?? 0, site)
   }));
   return {
     monitors,
@@ -453,7 +466,7 @@ async function searchMonitors(api, query, limits, site) {
       pageCount: response.metadata?.pageCount ?? 1,
       page: response.metadata?.page ?? 0
     },
-    datadog_url: buildMonitorsListUrl(query, site)
+    datadog_url: buildMonitorsListUrl({ name: query }, site)
   };
 }
 function normalizeMonitorConfig(config, isUpdate = false) {
@@ -507,21 +520,21 @@ function normalizeMonitorConfig(config, isUpdate = false) {
   }
   return normalized;
 }
-async function createMonitor(api, config) {
+async function createMonitor(api, config, site = "datadoghq.com") {
   const body = normalizeMonitorConfig(config);
   const monitor = await api.createMonitor({ body });
   return {
     success: true,
-    monitor: formatMonitor(monitor)
+    monitor: formatMonitor(monitor, site)
   };
 }
-async function updateMonitor(api, id, config) {
+async function updateMonitor(api, id, config, site = "datadoghq.com") {
   const monitorId = Number.parseInt(id, 10);
   const body = normalizeMonitorConfig(config, true);
   const monitor = await api.updateMonitor({ monitorId, body });
   return {
     success: true,
-    monitor: formatMonitor(monitor)
+    monitor: formatMonitor(monitor, site)
   };
 }
 async function deleteMonitor(api, id) {
@@ -582,12 +595,12 @@ TIP: For alert HISTORY (which monitors triggered), use the events tool with tags
           }
           case "create": {
             const monitorConfig = requireParam(config, "config", "create");
-            return toolResult(await createMonitor(api, monitorConfig));
+            return toolResult(await createMonitor(api, monitorConfig, site));
           }
           case "update": {
             const monitorId = requireParam(id, "id", "update");
             const updateConfig = requireParam(config, "config", "update");
-            return toolResult(await updateMonitor(api, monitorId, updateConfig));
+            return toolResult(await updateMonitor(api, monitorId, updateConfig, site));
           }
           case "delete": {
             const monitorId = requireParam(id, "id", "delete");