@nimbus-dev/connectors 0.2.0 → 0.2.2

package/connectors/miro/src/tools.ts

@@ -0,0 +1,65 @@
+import { z } from "zod";
+import { createJsonGetter, envAuthHeaders } from "../../../shared/env-json-api.ts";
+import { searchToolInputSchema } from "../../../shared/mcp-search-tool.ts";
+import { mcpJsonResult as jsonResult } from "../../../shared/mcp-tool-kit.ts";
+import type { ZodToolRegistrar } from "../../../shared/run-read-only-mcp-connector.ts";
+import { filterMiroBoards } from "./search-filter.ts";
+
+const BASE = "https://api.miro.com";
+
+const miroGet = createJsonGetter({
+  base: BASE,
+  label: "Miro",
+  headers: envAuthHeaders({ env: "MIRO_TOKEN" }),
+});
+
+function boardsFrom(root: unknown): unknown[] {
+  const data = (root as { data?: unknown } | null)?.data;
+  return Array.isArray(data) ? data : [];
+}
+
+/** Tool names exposed by this connector — for contract/introspection tests. */
+export const MIRO_TOOL_NAMES = ["miro_get", "miro_list", "miro_search"] as const;
+
+export function registerMiroTools(reg: ZodToolRegistrar): void {
+  reg(
+    "miro_list",
+    "List the authenticated user's Miro boards (GET /v2/boards?limit=N&cursor=<cursor>). Returns the { data: [...], cursor?, total, size } envelope — `data` holds the board objects, each shaped { id, name, description, createdAt, modifiedAt, owner: { name }, viewLink, ... }; `cursor` (when present) is the opaque token for the next page. `limit` (default 50, max 50) caps the page_size of the single first-page request.",
+    z.object({
+      limit: z.number().int().min(1).max(50).optional(),
+      cursor: z.string().optional(),
+    }),
+    async (p) => {
+      const params = new URLSearchParams({ limit: String(p.limit ?? 50) });
+      if (p.cursor !== undefined && p.cursor !== "") {
+        params.set("cursor", p.cursor);
+      }
+      return jsonResult(await miroGet(`/v2/boards?${params.toString()}`));
+    },
+  );
+
+  reg(
+    "miro_get",
+    "Fetch one Miro board by its id (GET /v2/boards/{boardId}). Returns the board object directly (NOT wrapped in `data`). Throws when no board with that id exists.",
+    z.object({
+      id: z.string().min(1),
+    }),
+    async (p) => {
+      return jsonResult(await miroGet(`/v2/boards/${encodeURIComponent(p.id)}`));
+    },
+  );
+
+  reg(
+    "miro_search",
+    "**Substring search over the FIRST PAGE only** (up to 50 most-recently-listed boards) of the authenticated user's Miro boards. This tool fetches GET /v2/boards?limit=50 once and matches the query locally (case-insensitive) against the board name, description, and owner name. **Boards beyond the first page are not searchable here — query the local Nimbus index instead for full coverage.** Returns a { matches: [...] } envelope.",
+    searchToolInputSchema(50),
+    async (p) => {
+      const root = await miroGet("/v2/boards?limit=50");
+      const matches = filterMiroBoards(boardsFrom(root), {
+        query: p.query,
+        limit: p.limit,
+      });
+      return jsonResult({ matches });
+    },
+  );
+}

package/connectors/mlflow/src/search-filter.ts

@@ -2,15 +2,11 @@ import {
   asRecord,
   makeQueryFilter,
   type SearchMatchOptions,
+  stringField,
 } from "../../../shared/search-filter.ts";
 
 export type MlflowSearchMatchOptions = SearchMatchOptions;
 
-function stringAt(row: Record<string, unknown>, key: string): string {
-  const v = row[key];
-  return typeof v === "string" ? v : "";
-}
-
 function tagsHaystack(row: Record<string, unknown>): string {
   const tags = row["tags"];
   if (!Array.isArray(tags)) {
@@ -22,7 +18,7 @@ function tagsHaystack(row: Record<string, unknown>): string {
     if (tag === undefined) {
       continue;
     }
-    parts.push(`${stringAt(tag, "key")}=${stringAt(tag, "value")}`);
+    parts.push(`${stringField(tag, "key")}=${stringField(tag, "value")}`);
   }
   return parts.join(" ");
 }
@@ -32,7 +28,7 @@ function fieldsOf(item: unknown): readonly string[] | null {
   if (row === undefined) {
     return null;
   }
-  return [stringAt(row, "name"), stringAt(row, "description"), tagsHaystack(row)];
+  return [stringField(row, "name"), stringField(row, "description"), tagsHaystack(row)];
 }
 
 export const filterMlflowModels = makeQueryFilter(fieldsOf);

package/connectors/mlflow/src/server.ts

@@ -1,5 +1,12 @@
 import { z } from "zod";
 import { type ConsentServer, createWriteToolRegistrar } from "../../../shared/consent-kit.ts";
+import {
+  createJsonGetter,
+  createJsonPoster,
+  envAuthHeaders,
+  type JsonApiConfig,
+  requiredBaseUrl,
+} from "../../../shared/env-json-api.ts";
 import { searchToolInputSchema } from "../../../shared/mcp-search-tool.ts";
 import { fetchWithTimeout, mcpJsonResult as jsonResult } from "../../../shared/mcp-tool-kit.ts";
 import {
@@ -8,47 +15,20 @@ import {
 } from "../../../shared/run-read-only-mcp-connector.ts";
 import { filterMlflowModels } from "./search-filter.ts";
 
-function trimTrailingSlash(s: string): string {
-  return s.endsWith("/") ? s.slice(0, -1) : s;
-}
-
-function apiBase(): string {
-  const v = process.env["MLFLOW_HOST"]?.trim();
-  if (v === undefined || v === "") {
-    throw new Error("MLFLOW_HOST is not set");
-  }
-  return trimTrailingSlash(v);
-}
-
-function authHeader(): Record<string, string> {
-  const k = process.env["MLFLOW_TOKEN"]?.trim();
-  if (k === undefined || k === "") {
-    throw new Error("MLFLOW_TOKEN is not set");
-  }
-  return { Authorization: `Bearer ${k}`, Accept: "application/json" };
-}
-
-async function mlflowGet(path: string): Promise<unknown> {
-  const res = await fetchWithTimeout(`${apiBase()}${path}`, { headers: authHeader() });
-  const text = await res.text();
-  if (!res.ok) {
-    throw new Error(`MLflow ${String(res.status)}: ${text.slice(0, 400)}`);
-  }
-  return JSON.parse(text) as unknown;
-}
+/**
+ * One config for reads and the mutating requests alike. `fetchWithTimeout`, not the global
+ * fetch: MLflow is a self-hosted tracking server, and one that stops answering must fail the
+ * tool call rather than hang it.
+ */
+const api: JsonApiConfig = {
+  base: () => requiredBaseUrl("MLFLOW_HOST"),
+  label: "MLflow",
+  headers: envAuthHeaders({ env: "MLFLOW_TOKEN" }),
+  fetch: fetchWithTimeout,
+};
 
-async function mlflowPost(path: string, body: unknown): Promise<unknown> {
-  const res = await fetchWithTimeout(`${apiBase()}${path}`, {
-    method: "POST",
-    headers: { ...authHeader(), "Content-Type": "application/json" },
-    body: JSON.stringify(body),
-  });
-  const text = await res.text();
-  if (!res.ok) {
-    throw new Error(`MLflow ${path} ${String(res.status)}: ${text.slice(0, 400)}`);
-  }
-  return text === "" ? {} : (JSON.parse(text) as unknown);
-}
+const mlflowGet = createJsonGetter(api);
+const mlflowPost = createJsonPoster(api);
 
 const TRANSITION_PATH = "/api/2.0/mlflow/model-versions/transition-stage";
 

package/connectors/monte-carlo/src/server.ts

@@ -1,11 +1,14 @@
 import { z } from "zod";
 import { type ConsentServer, createWriteToolRegistrar } from "../../../shared/consent-kit.ts";
+import { cursorListInputSchema } from "../../../shared/cursor-list-tool.ts";
+import { requiredEnv } from "../../../shared/env-json-api.ts";
 import { searchToolInputSchema } from "../../../shared/mcp-search-tool.ts";
 import { fetchWithTimeout, mcpJsonResult as jsonResult } from "../../../shared/mcp-tool-kit.ts";
 import {
   runReadOnlyMcpConnector,
   type ZodToolRegistrar,
 } from "../../../shared/run-read-only-mcp-connector.ts";
+import { asRecord } from "../../../shared/search-filter.ts";
 import { filterMonteCarloIncidents } from "./search-filter.ts";
 
 const GRAPHQL_URL = "https://api.getmontecarlo.com/graphql";
@@ -35,20 +38,6 @@ const GET_INCIDENTS_QUERY = `
   }
 `.trim();
 
-function requireEnv(name: string): string {
-  const v = process.env[name]?.trim();
-  if (v === undefined || v === "") {
-    throw new Error(`${name} is not set`);
-  }
-  return v;
-}
-
-function asObject(value: unknown): Record<string, unknown> | null {
-  return value !== null && typeof value === "object" && !Array.isArray(value)
-    ? (value as Record<string, unknown>)
-    : null;
-}
-
 /**
  * Single POST helper for the Monte Carlo GraphQL endpoint. Sends `{ query, variables }` with the
  * `x-mcd-id`/`x-mcd-token` key-pair auth headers, throws on a non-ok HTTP status or a non-empty
@@ -75,10 +64,10 @@ async function mcGraphql(
   if (!res.ok) {
     throw new Error(`Monte Carlo API error ${String(res.status)}: ${text.slice(0, 400)}`);
   }
-  const parsed = asObject(JSON.parse(text) as unknown) ?? {};
+  const parsed = asRecord(JSON.parse(text) as unknown) ?? {};
   const errors = parsed["errors"];
   if (Array.isArray(errors) && errors.length > 0) {
-    const first = asObject(errors[0])?.["message"];
+    const first = asRecord(errors[0])?.["message"];
     const message = typeof first === "string" ? first : JSON.stringify(errors[0]);
     throw new Error(`Monte Carlo GraphQL error: ${message}`);
   }
@@ -92,14 +81,14 @@ interface IncidentsPage {
 }
 
 function parseIncidentsPage(parsed: unknown): IncidentsPage {
-  const getIncidents = asObject(asObject(asObject(parsed)?.["data"])?.["getIncidents"]);
+  const getIncidents = asRecord(asRecord(asRecord(parsed)?.["data"])?.["getIncidents"]);
   const edges = Array.isArray(getIncidents?.["edges"]) ? (getIncidents["edges"] as unknown[]) : [];
   const incidents: unknown[] = [];
   for (const edge of edges) {
-    const node = asObject(edge)?.["node"];
+    const node = asRecord(edge)?.["node"];
     if (node !== undefined) incidents.push(node);
   }
-  const pageInfo = asObject(getIncidents?.["pageInfo"]);
+  const pageInfo = asRecord(getIncidents?.["pageInfo"]);
   const hasNextPage = pageInfo?.["hasNextPage"] === true;
   const endCursor = typeof pageInfo?.["endCursor"] === "string" ? pageInfo["endCursor"] : null;
   return { incidents, hasNextPage, endCursor };
@@ -151,13 +140,10 @@ export function registerMonteCarloTools(reg: ZodToolRegistrar, server: unknown):
   reg(
     "montecarlo_list",
     "List Monte Carlo data-quality incidents (relay GraphQL `getIncidents`). Paginated: `cursor` (the relay `after` token) + `limit` (default 200, max 500) → `{ items, nextCursor }`.",
-    z.object({
-      cursor: z.string().nullable().optional(),
-      limit: z.number().int().min(1).max(500).optional(),
-    }),
+    cursorListInputSchema(),
     async (p) => {
-      const apiId = requireEnv("MONTECARLO_API_ID");
-      const apiToken = requireEnv("MONTECARLO_API_TOKEN");
+      const apiId = requiredEnv("MONTECARLO_API_ID");
+      const apiToken = requiredEnv("MONTECARLO_API_TOKEN");
       const first = p.limit ?? 200;
       const after = p.cursor === undefined || p.cursor === "" ? null : p.cursor;
       const page = await fetchIncidentsPage(apiId, apiToken, first, after);
@@ -173,10 +159,10 @@ export function registerMonteCarloTools(reg: ZodToolRegistrar, server: unknown):
       id: z.string().min(1),
     }),
     async (p) => {
-      const apiId = requireEnv("MONTECARLO_API_ID");
-      const apiToken = requireEnv("MONTECARLO_API_TOKEN");
+      const apiId = requiredEnv("MONTECARLO_API_ID");
+      const apiToken = requiredEnv("MONTECARLO_API_TOKEN");
       const incidents = await fetchIncidents(apiId, apiToken);
-      const found = incidents.find((inc) => asObject(inc)?.["incidentId"] === p.id);
+      const found = incidents.find((inc) => asRecord(inc)?.["incidentId"] === p.id);
       if (found === undefined) {
         throw new Error(`Monte Carlo incident not found: ${p.id}`);
       }
@@ -189,46 +175,53 @@ export function registerMonteCarloTools(reg: ZodToolRegistrar, server: unknown):
     "Substring search across Monte Carlo incidents. Matches the query (case-insensitive) against incidentId, status, severity, and monitoredTable. Returns a `{ matches: [...] }` envelope.",
     searchToolInputSchema(200),
     async (p) => {
-      const apiId = requireEnv("MONTECARLO_API_ID");
-      const apiToken = requireEnv("MONTECARLO_API_TOKEN");
+      const apiId = requiredEnv("MONTECARLO_API_ID");
+      const apiToken = requiredEnv("MONTECARLO_API_TOKEN");
       const incidents = await fetchIncidents(apiId, apiToken);
       const matches = filterMonteCarloIncidents(incidents, { query: p.query, limit: p.limit });
       return jsonResult({ matches });
     },
   );
 
-  registerWriteTool(
+  /**
+   * Acknowledge and resolve are ONE mutation, `setIncidentFeedback`, with a different feedback
+   * value, so they share everything except their name, action type, description and that value.
+   */
+  function registerFeedbackTool(
+    name: string,
+    mutates: string,
+    description: string,
+    feedback: "ACKNOWLEDGED" | "RESOLVED",
+  ): void {
+    registerWriteTool(
+      name,
+      {
+        mutates,
+        recoverable: true,
+        scopeTargetOf: (p) => ({ kind: "incident", value: p.incidentId }),
+      },
+      description,
+      z.object({ incidentId: z.string().min(1) }),
+      async (p) => {
+        const apiId = requiredEnv("MONTECARLO_API_ID");
+        const apiToken = requiredEnv("MONTECARLO_API_TOKEN");
+        await setIncidentFeedback(apiId, apiToken, p.incidentId, feedback);
+        return jsonResult({ status: "ok", incidentId: p.incidentId });
+      },
+    );
+  }
+
+  registerFeedbackTool(
     "montecarlo_incident_acknowledge",
-    {
-      mutates: "montecarlo.incident.acknowledge",
-      recoverable: true,
-      scopeTargetOf: (p) => ({ kind: "incident", value: p.incidentId }),
-    },
+    "montecarlo.incident.acknowledge",
     "Acknowledge a Monte Carlo incident.",
-    z.object({ incidentId: z.string().min(1) }),
-    async (p) => {
-      const apiId = requireEnv("MONTECARLO_API_ID");
-      const apiToken = requireEnv("MONTECARLO_API_TOKEN");
-      await setIncidentFeedback(apiId, apiToken, p.incidentId, "ACKNOWLEDGED");
-      return jsonResult({ status: "ok", incidentId: p.incidentId });
-    },
+    "ACKNOWLEDGED",
   );
-
-  registerWriteTool(
+  registerFeedbackTool(
     "montecarlo_incident_resolve",
-    {
-      mutates: "montecarlo.incident.resolve",
-      recoverable: true,
-      scopeTargetOf: (p) => ({ kind: "incident", value: p.incidentId }),
-    },
+    "montecarlo.incident.resolve",
     "Resolve a Monte Carlo incident.",
-    z.object({ incidentId: z.string().min(1) }),
-    async (p) => {
-      const apiId = requireEnv("MONTECARLO_API_ID");
-      const apiToken = requireEnv("MONTECARLO_API_TOKEN");
-      await setIncidentFeedback(apiId, apiToken, p.incidentId, "RESOLVED");
-      return jsonResult({ status: "ok", incidentId: p.incidentId });
-    },
+    "RESOLVED",
   );
 }
 

package/connectors/netlify/src/search-filter.ts

@@ -1,21 +1,13 @@
 import {
   asObjectish,
   makeQueryFilter,
+  nestedString,
   type SearchMatchOptions,
   stringField,
 } from "../../../shared/search-filter.ts";
 
 export type NetlifySearchMatchOptions = SearchMatchOptions;
 
-function subStringField(row: Record<string, unknown>, key: string, sub: string): string {
-  const obj = row[key];
-  if (obj === null || typeof obj !== "object" || Array.isArray(obj)) {
-    return "";
-  }
-  const v = (obj as Record<string, unknown>)[sub];
-  return typeof v === "string" ? v : "";
-}
-
 function fieldsOf(item: unknown): readonly string[] | null {
   const row = asObjectish(item);
   if (row === undefined) {
@@ -26,11 +18,11 @@ function fieldsOf(item: unknown): readonly string[] | null {
     stringField(row, "name"),
     stringField(row, "url"),
     stringField(row, "ssl_url"),
-    subStringField(row, "build_settings", "repo_url"),
-    subStringField(row, "build_settings", "repo_branch"),
-    subStringField(row, "published_deploy", "state"),
-    subStringField(row, "published_deploy", "branch"),
-    subStringField(row, "published_deploy", "commit_ref"),
+    nestedString(row, ["build_settings", "repo_url"]),
+    nestedString(row, ["build_settings", "repo_branch"]),
+    nestedString(row, ["published_deploy", "state"]),
+    nestedString(row, ["published_deploy", "branch"]),
+    nestedString(row, ["published_deploy", "commit_ref"]),
   ];
 }
 

package/connectors/netlify/src/server.ts

@@ -1,64 +1,6 @@
-import { z } from "zod";
-import { matchesResult, searchToolInputSchema } from "../../../shared/mcp-search-tool.ts";
-import { mcpJsonResult as jsonResult } from "../../../shared/mcp-tool-kit.ts";
 import { runReadOnlyMcpConnector } from "../../../shared/run-read-only-mcp-connector.ts";
-import { filterNetlifySites } from "./search-filter.ts";
-
-const BASE = "https://api.netlify.com";
-
-function token(): string {
-  const t = process.env["NETLIFY_TOKEN"]?.trim();
-  if (t === undefined || t === "") {
-    throw new Error("NETLIFY_TOKEN is not set");
-  }
-  return t;
-}
-
-function authHeader(): Record<string, string> {
-  return { Authorization: `Bearer ${token()}`, Accept: "application/json" };
-}
-
-async function netlifyGet(path: string): Promise<unknown> {
-  const res = await fetch(`${BASE}${path}`, { headers: authHeader() });
-  const text = await res.text();
-  if (!res.ok) {
-    throw new Error(`Netlify ${String(res.status)}: ${text.slice(0, 400)}`);
-  }
-  return JSON.parse(text) as unknown;
-}
+import { registerNetlifyTools } from "./tools.ts";
 
 await runReadOnlyMcpConnector("nimbus-netlify", (reg) => {
-  reg(
-    "netlify_list",
-    "List Netlify sites (`GET /api/v1/sites`), capped at `per_page` (default 100). Returns a bare JSON array of site objects, each carrying the embedded `published_deploy` status and `build_settings`.",
-    z.object({
-      per_page: z.number().int().min(1).max(100).optional(),
-    }),
-    async (p) => {
-      const search = new URLSearchParams({ per_page: String(p.per_page ?? 100), page: "1" });
-      return jsonResult(await netlifyGet(`/api/v1/sites?${search.toString()}`));
-    },
-  );
-
-  reg(
-    "netlify_get",
-    "Fetch one Netlify site by its id (`GET /api/v1/sites/{siteId}`). Returns the site object directly. Throws when no match is found.",
-    z.object({
-      siteId: z.string().min(1),
-    }),
-    async (p) => {
-      return jsonResult(await netlifyGet(`/api/v1/sites/${encodeURIComponent(p.siteId)}`));
-    },
-  );
-
-  reg(
-    "netlify_search",
-    "Substring search across Netlify sites. Matches the query against id, name, url, ssl_url, the linked git repo + branch, and the published-deploy state / branch / commit ref (case-insensitive). Returns a `{ matches: [...] }` envelope.",
-    searchToolInputSchema(100),
-    async (p) => {
-      const search = new URLSearchParams({ per_page: "100", page: "1" });
-      const root = await netlifyGet(`/api/v1/sites?${search.toString()}`);
-      return matchesResult(root, filterNetlifySites, p);
-    },
-  );
+  registerNetlifyTools(reg);
 });

package/connectors/netlify/src/tools.ts

@@ -0,0 +1,53 @@
+import { z } from "zod";
+import { createJsonGetter, envAuthHeaders } from "../../../shared/env-json-api.ts";
+import { matchesResult, searchToolInputSchema } from "../../../shared/mcp-search-tool.ts";
+import { mcpJsonResult as jsonResult } from "../../../shared/mcp-tool-kit.ts";
+import type { ZodToolRegistrar } from "../../../shared/run-read-only-mcp-connector.ts";
+import { filterNetlifySites } from "./search-filter.ts";
+
+const BASE = "https://api.netlify.com";
+
+const netlifyGet = createJsonGetter({
+  base: BASE,
+  label: "Netlify",
+  headers: envAuthHeaders({ env: "NETLIFY_TOKEN" }),
+});
+
+/** Tool names exposed by this connector — for contract/introspection tests. */
+export const NETLIFY_TOOL_NAMES = ["netlify_get", "netlify_list", "netlify_search"] as const;
+
+export function registerNetlifyTools(reg: ZodToolRegistrar): void {
+  reg(
+    "netlify_list",
+    "List Netlify sites (`GET /api/v1/sites`), capped at `per_page` (default 100). Returns a bare JSON array of site objects, each carrying the embedded `published_deploy` status and `build_settings`.",
+    z.object({
+      per_page: z.number().int().min(1).max(100).optional(),
+    }),
+    async (p) => {
+      const search = new URLSearchParams({ per_page: String(p.per_page ?? 100), page: "1" });
+      return jsonResult(await netlifyGet(`/api/v1/sites?${search.toString()}`));
+    },
+  );
+
+  reg(
+    "netlify_get",
+    "Fetch one Netlify site by its id (`GET /api/v1/sites/{siteId}`). Returns the site object directly. Throws when no match is found.",
+    z.object({
+      siteId: z.string().min(1),
+    }),
+    async (p) => {
+      return jsonResult(await netlifyGet(`/api/v1/sites/${encodeURIComponent(p.siteId)}`));
+    },
+  );
+
+  reg(
+    "netlify_search",
+    "Substring search across Netlify sites. Matches the query against id, name, url, ssl_url, the linked git repo + branch, and the published-deploy state / branch / commit ref (case-insensitive). Returns a `{ matches: [...] }` envelope.",
+    searchToolInputSchema(100),
+    async (p) => {
+      const search = new URLSearchParams({ per_page: "100", page: "1" });
+      const root = await netlifyGet(`/api/v1/sites?${search.toString()}`);
+      return matchesResult(root, filterNetlifySites, p);
+    },
+  );
+}

package/connectors/newrelic/src/server.ts

@@ -1,48 +1,10 @@
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
-import { z } from "zod";
-
-import {
-  createRegisterSimpleTool,
-  createZodToolRegistrar,
-  mcpJsonResult as jsonResult,
-} from "../../../shared/mcp-tool-kit.ts";
-
-function apiKey(): string {
-  const k = process.env["NEW_RELIC_API_KEY"]?.trim();
-  if (k === undefined || k === "") {
-    throw new Error("NEW_RELIC_API_KEY is not set");
-  }
-  return k;
-}
-
-async function nrGet(path: string): Promise<unknown> {
-  const res = await fetch(`https://api.newrelic.com${path}`, {
-    headers: { "X-Api-Key": apiKey(), Accept: "application/json" },
-  });
-  const text = await res.text();
-  if (!res.ok) {
-    throw new Error(`New Relic ${String(res.status)}: ${text.slice(0, 400)}`);
-  }
-  return JSON.parse(text) as unknown;
-}
+import { registerNewrelicTools } from "./tools.ts";
 
 const mcp = new McpServer({ name: "nimbus-newrelic", version: "0.1.0" });
-const reg = createZodToolRegistrar(createRegisterSimpleTool(mcp));
-
-reg("newrelic_application_list", "List APM applications.", z.object({}), async () =>
-  jsonResult(await nrGet("/v2/applications.json")),
-);
 
-reg(
-  "newrelic_alert_violations",
-  "List recent alert violations.",
-  z.object({ only_open: z.boolean().optional() }),
-  async (p) => {
-    const only = p.only_open === true ? "true" : "false";
-    return jsonResult(await nrGet(`/v2/alerts_violations.json?only_open=${only}`));
-  },
-);
+registerNewrelicTools(mcp);
 
 const transport = new StdioServerTransport();
 await mcp.connect(transport);

package/connectors/newrelic/src/tools.ts

@@ -0,0 +1,37 @@
+import { z } from "zod";
+import { createJsonGetter, envAuthHeaders } from "../../../shared/env-json-api.ts";
+import {
+  createRegisterSimpleTool,
+  createZodToolRegistrar,
+  mcpJsonResult as jsonResult,
+} from "../../../shared/mcp-tool-kit.ts";
+
+const nrGet = createJsonGetter({
+  base: "https://api.newrelic.com",
+  label: "New Relic",
+  headers: envAuthHeaders({ env: "NEW_RELIC_API_KEY", scheme: "", header: "X-Api-Key" }),
+});
+
+/** Tool names exposed by this connector — for contract/introspection tests. */
+export const NEWRELIC_TOOL_NAMES = [
+  "newrelic_application_list",
+  "newrelic_alert_violations",
+] as const;
+
+export function registerNewrelicTools(server: { tool: (...args: never) => unknown }): void {
+  const reg = createZodToolRegistrar(createRegisterSimpleTool(server));
+
+  reg("newrelic_application_list", "List APM applications.", z.object({}), async () =>
+    jsonResult(await nrGet("/v2/applications.json")),
+  );
+
+  reg(
+    "newrelic_alert_violations",
+    "List recent alert violations.",
+    z.object({ only_open: z.boolean().optional() }),
+    async (p) => {
+      const only = p.only_open === true ? "true" : "false";
+      return jsonResult(await nrGet(`/v2/alerts_violations.json?only_open=${only}`));
+    },
+  );
+}