chore: made eslint happy

Author: jeroenvervaeke

src/tools/atlasLocal/atlasLocalTool.ts

@@ -1,7 +1,7 @@
 import type { CallToolResult } from "@modelcontextprotocol/sdk/types.js";
 import type { TelemetryToolMetadata, ToolArgs, ToolCategory } from "../tool.js";
 import { ToolBase } from "../tool.js";
-import { ToolCallback } from "@modelcontextprotocol/sdk/server/mcp.js";
+import type { ToolCallback } from "@modelcontextprotocol/sdk/server/mcp.js";
 import type AtlasLocal from "@mongodb-js-preview/atlas-local";
 
 export abstract class AtlasLocalToolBase extends ToolBase {
@@ -27,6 +27,7 @@ export abstract class AtlasLocalToolBase extends ToolBase {
         ...args: Parameters<ToolCallback<typeof this.argsShape>>
     ): TelemetryToolMetadata {
         // TODO: include deployment id in the metadata where possible
+        void args; // this shuts up the eslint rule until we implement the TODO above
         return {};
     }
 }

src/tools/atlasLocal/read/listDeployments.ts

@@ -1,6 +1,6 @@
 import type { CallToolResult } from "@modelcontextprotocol/sdk/types.js";
 import { AtlasLocalToolBase } from "../atlasLocalTool.js";
-import type { ToolArgs, OperationType, TelemetryToolMetadata } from "../../tool.js";
+import type { OperationType } from "../../tool.js";
 import { formatUntrustedData } from "../../tool.js";
 import type { Deployment } from "@mongodb-js-preview/atlas-local";
 
@@ -10,7 +10,7 @@ export class ListDeploymentsTool extends AtlasLocalToolBase {
     public operationType: OperationType = "read";
     protected argsShape = {};
 
-    protected async execute({}: ToolArgs<typeof this.argsShape>): Promise<CallToolResult> {
+    protected async execute(): Promise<CallToolResult> {
         // Get the client
         const client = this.client;
 
@@ -20,18 +20,15 @@ export class ListDeploymentsTool extends AtlasLocalToolBase {
         if (!client) {
             throw new Error("Atlas Local client not found, tool should have been disabled.");
         }
-        
+
         // List the deployments
         const deployments = await client.listDeployments();
 
         // Format the deployments
         return this.formatDeploymentsTable(deployments);
     }
 
-
-    private formatDeploymentsTable(
-        deployments: Deployment[]
-    ): CallToolResult {
+    private formatDeploymentsTable(deployments: Deployment[]): CallToolResult {
         // Check if deployments are absent
         if (!deployments?.length) {
             return {
@@ -42,10 +39,10 @@ export class ListDeploymentsTool extends AtlasLocalToolBase {
         // Turn the deployments into a markdown table
         const rows = deployments
             .map((deployment) => {
-                return `${deployment.name || "Unknown"} | ${deployment.state} | ${deployment.mongodbVersion}`
+                return `${deployment.name || "Unknown"} | ${deployment.state} | ${deployment.mongodbVersion}`;
             })
             .join("\n");
-            
+
         return {
             content: formatUntrustedData(
                 `Found ${deployments.length} deployments:`,

src/tools/atlasLocal/tools.ts

@@ -5,15 +5,15 @@ import type AtlasLocal from "@mongodb-js-preview/atlas-local";
 const atlasLocalTools = [ListDeploymentsTool];
 
 // Build the Atlas Local tools
-export const BuildAtlasLocalTools = async () => {
+export const BuildAtlasLocalTools = async (): Promise<typeof atlasLocalTools> => {
     // Initialize the Atlas Local client
     const client = await GetAtlasLocalClient();
 
     // If the client is found, set it on the tools
     // On unsupported platforms, the client will be undefined
     if (client) {
         // Set the client on the tools
-        atlasLocalTools.forEach(tool => {
+        atlasLocalTools.forEach((tool) => {
             tool.prototype.client = client;
         });
     }
@@ -30,4 +30,4 @@ export const GetAtlasLocalClient = async (): Promise<AtlasLocal.Client | undefin
         console.warn("Atlas Local native binding not available:", error);
         return undefined;
     }
-}
+};

tests/integration/tools/atlas-local/listDeployments.test.ts

@@ -1,14 +1,18 @@
-import { defaultDriverOptions, defaultTestConfig, expectDefined, getResponseElements, setupIntegrationTest } from "../../helpers.js";
+import {
+    defaultDriverOptions,
+    defaultTestConfig,
+    expectDefined,
+    getResponseElements,
+    setupIntegrationTest,
+} from "../../helpers.js";
 import { describe, expect, it } from "vitest";
 
-
-
 describe("atlas-local-list-deployments", () => {
     const integration = setupIntegrationTest(
         () => defaultTestConfig,
         () => defaultDriverOptions
     );
-    
+
     it("should have correct metadata", async () => {
         const { tools } = await integration.mcpClient().listTools();
         const listDeployments = tools.find((tool) => tool.name === "atlas-local-list-deployments");
@@ -26,6 +30,8 @@ describe("atlas-local-list-deployments", () => {
         const elements = getResponseElements(response.content);
         expect(elements).toHaveLength(2);
         expect(elements[0]?.text).toMatch(/Found \d+ deployments/);
-        expect(elements[1]?.text).toContain("Deployment Name | State | MongoDB Version\n----------------|----------------|----------------\n");
+        expect(elements[1]?.text).toContain(
+            "Deployment Name | State | MongoDB Version\n----------------|----------------|----------------\n"
+        );
     });
-});
+});