ci: add eslint

Author: fmenezes

.github/workflows/code_health.yaml

@@ -37,22 +37,47 @@ jobs:
                     exit 1
                   fi
     prettier:
-        name: Prettier Check
         runs-on: ubuntu-latest
         steps:
             - uses: GitHubSecurityLab/actions-permissions/monitor@v1
               with:
                   config: ${{ vars.PERMISSIONS_CONFIG }}
             - name: Checkout Repository
               uses: actions/checkout@v2
-            - name: Run Prettier
-              id: prettier-run
-              uses: rutajdash/prettier-cli-action@d42c4325a3b344f3bd4be482bc34de521998d557
+            - uses: actions/setup-node@v4
               with:
-                  config_path: ./.prettierrc.yml
-            - name: Prettier Output
-              if: ${{ failure() }}
-              shell: bash
+                  node-version-file: package.json
+            - name: install dependencies
+              run: |
+                  npm ci
+            - run: |
+                  npm run reformat
+            - name: Check for uncommitted files
               run: |
-                  echo "The following files are not formatted:"
-                  echo "${{steps.prettier-run.outputs.prettier_output}}"
+                  export FILES=
+                  FILES=$(git ls-files -o -m --directory --exclude-standard --no-empty-directory)
+                  export LINES=
+                  LINES=$(echo "$FILES" | awk 'NF' | wc -l)
+                  if [ "$LINES" -ne 0 ]; then
+                    echo "Detected files that need to be committed:"
+                    echo "${FILES}"
+                    echo ""
+                    echo "Try running: npm run reformat"
+                    exit 1
+                  fi
+    eslint:
+        runs-on: ubuntu-latest
+        steps:
+            - uses: GitHubSecurityLab/actions-permissions/monitor@v1
+              with:
+                  config: ${{ vars.PERMISSIONS_CONFIG }}
+            - name: Checkout Repository
+              uses: actions/checkout@v4
+            - uses: actions/setup-node@v4
+              with:
+                  node-version-file: package.json
+            - name: install dependencies
+              run: |
+                  npm ci
+            - run: |
+                  npm run lint

CONTRIBUTING.md

@@ -0,0 +1,100 @@
+# Contributing to MongoDB MCP Server
+
+Thank you for your interest in contributing to the MongoDB MCP Server project! This document provides guidelines and instructions for contributing.
+
+## Project Overview
+
+This project implements a Model Context Protocol (MCP) server for MongoDB and MongoDB Atlas, enabling AI assistants to interact with MongoDB Atlas resources through natural language.
+
+## Development Setup
+
+### Prerequisites
+
+- Node.js (v23 or later)
+- npm
+
+### Getting Started
+
+1. Clone the repository:
+
+   ```
+   git clone https://github.com/mongodb-labs/mongodb-mcp-server.git
+   cd mongodb-mcp-server
+   ```
+
+2. Install dependencies:
+
+   ```
+   npm install
+   ```
+
+3. Add the mcp server to your IDE of choice
+   ```json
+   {
+     "mcpServers": {
+       "MongoDB": {
+         "command": "/path/to/mongodb-mcp-server/dist/index.js"
+       }
+     }
+   }
+   ```
+
+## Code Contribution Workflow
+
+1. Create a new branch for your feature or bugfix:
+
+   ```
+   git checkout -b feature/your-feature-name
+   ```
+
+2. Make your changes, following the code style of the project
+
+3. Run the inspector and double check your changes:
+
+   ```
+   npm run inspect
+   ```
+
+4. Commit your changes with a descriptive commit message
+
+## Pull Request Guidelines
+
+1. Update documentation if necessary
+2. Ensure your PR includes only relevant changes
+3. Link any related issues in your PR description
+4. Keep PRs focused on a single topic
+
+## Code Standards
+
+- Use TypeScript for all new code
+- Follow the existing code style (indentation, naming conventions, etc.)
+- Comment your code when necessary, especially for complex logic
+- Use meaningful variable and function names
+
+## Reporting Issues
+
+When reporting issues, please include:
+
+- A clear description of the problem
+- Steps to reproduce
+- Expected vs. actual behavior
+- Version information
+- Environment details
+
+## Adding New Tools
+
+When adding new tools to the MCP server:
+
+1. Follow the existing pattern in `server.ts`
+2. Define clear parameter schemas using Zod
+3. Implement thorough error handling
+4. Add proper documentation for the tool
+5. Include examples of how to use the tool
+
+## License
+
+By contributing to this project, you agree that your contributions will be licensed under the project's license.
+
+## Questions?
+
+If you have any questions or need help, please open an issue or reach out to the maintainers.

dist/client.js

@@ -112,7 +112,7 @@ export class ApiClient {
                 throw new ApiClientError("Device code expired. Please restart the authentication process.", response);
             }
         }
-        catch (error) {
+        catch {
             throw new ApiClientError("Failed to retrieve token. Please check your device code.", response);
         }
     }
@@ -182,7 +182,7 @@ export class ApiClient {
             const expiryWithDelta = new Date(token.expiry.getTime() - expiryDelta);
             return expiryWithDelta.getTime() > Date.now();
         }
-        catch (error) {
+        catch {
             return false;
         }
     }
@@ -194,7 +194,7 @@ export class ApiClient {
             await this.refreshToken(token);
             return true;
         }
-        catch (error) {
+        catch {
             return false;
         }
     }

dist/server.js

@@ -43,7 +43,7 @@ export class Server {
                     await this.apiClient.retrieveToken(this.state.auth.code.device_code);
                     return !!this.state.auth.token;
                 }
-                catch (error) {
+                catch {
                     return false;
                 }
             case "issued":
@@ -178,7 +178,7 @@ export class Server {
             const header = `Project Name | Project ID | Created At
 ----------------|----------------|----------------`;
             const rows = projects
-                .map((project) => {
+                .map(project => {
                 const createdAt = project.created ? new Date(project.created.$date).toLocaleString() : "N/A";
                 return `${project.name} | ${project.id} | ${createdAt}`;
             })
@@ -217,7 +217,7 @@ export class Server {
             name: "MongoDB Atlas",
             version: config.version,
         });
-        server.tool("auth", "Authenticate to Atlas", async ({}) => this.authTool());
+        server.tool("auth", "Authenticate to Atlas", async () => this.authTool());
         let projectIdFilter = z.string().describe("Optional Atlas project ID to filter clusters");
         if (config.projectID) {
             projectIdFilter = projectIdFilter.optional();

eslint.config.js

@@ -0,0 +1,12 @@
+import { defineConfig } from "eslint/config";
+import js from "@eslint/js";
+import globals from "globals";
+import tseslint from "typescript-eslint";
+import eslintConfigPrettier from "eslint-config-prettier/flat";
+
+export default defineConfig([
+    { files: ["src/**/*.ts"], plugins: { js }, extends: ["js/recommended"] },
+    { files: ["src/**/*.ts"], languageOptions: { globals: globals.node } },
+    tseslint.configs.recommended,
+    eslintConfigPrettier
+]);