Skip to content

chore: add test for boolean behavior and rename embeddingsValidation #759

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Nov 25, 2025
Merged

chore: add test for boolean behavior and rename embeddingsValidation #759

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
feead60
chore: add tests and pre-processor for other cases
gagik Nov 25, 2025
01c3f47
chore: disableEmbeddingsValidation -> embeddingsValidation
gagik Nov 25, 2025
21d841f
chore: add env tests
gagik Nov 25, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -351,8 +351,8 @@ The MongoDB MCP Server can be configured using multiple methods, with the follow
| `atlasTemporaryDatabaseUserLifetimeMs` | `MDB_MCP_ATLAS_TEMPORARY_DATABASE_USER_LIFETIME_MS` | `14400000` | Time in milliseconds that temporary database users created when connecting to MongoDB Atlas clusters will remain active before being automatically deleted. |
| `confirmationRequiredTools` | `MDB_MCP_CONFIRMATION_REQUIRED_TOOLS` | `"atlas-create-access-list,atlas-create-db-user,drop-database,drop-collection,delete-many,drop-index"` | Comma separated values of tool names that require user confirmation before execution. Requires the client to support elicitation. |
| `connectionString` | `MDB_MCP_CONNECTION_STRING` | `<not set>` | MongoDB connection string for direct database connections. Optional, if not set, you'll need to call the connect tool before interacting with MongoDB data. |
| `disableEmbeddingsValidation` | `MDB_MCP_DISABLE_EMBEDDINGS_VALIDATION` | `false` | When set to true, disables validation of embeddings dimensions. |
| `disabledTools` | `MDB_MCP_DISABLED_TOOLS` | `""` | Comma separated values of tool names, operation types, and/or categories of tools that will be disabled. |
| `embeddingsValidation` | `MDB_MCP_EMBEDDINGS_VALIDATION` | `true` | When set to false, disables validation of embeddings dimensions. |
| `exportCleanupIntervalMs` | `MDB_MCP_EXPORT_CLEANUP_INTERVAL_MS` | `120000` | Time in milliseconds between export cleanup cycles that remove expired export files. |
| `exportTimeoutMs` | `MDB_MCP_EXPORT_TIMEOUT_MS` | `300000` | Time in milliseconds after which an export is considered expired and eligible for cleanup. |
| `exportsPath` | `MDB_MCP_EXPORTS_PATH` | see below\* | Folder to store exported data files. |
Expand Down
7 changes: 1 addition & 6 deletions scripts/generateArguments.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ import { readFileSync, writeFileSync } from "fs";
import { join, dirname } from "path";
import { fileURLToPath } from "url";
import { UserConfigSchema, configRegistry } from "../src/common/config/userConfig.js";
import assert from "assert";
import { execSync } from "child_process";
import { OPTIONS } from "../src/common/config/argsParserOptions.js";

Expand Down Expand Up @@ -69,12 +68,8 @@ function extractZodDescriptions(): Record<string, ConfigMetadata> {
let description = schema.description || `Configuration option: ${key}`;

if ("innerType" in schema.def) {
// "pipe" is used for our comma-separated arrays
// "pipe" is also used for our comma-separated arrays
if (schema.def.innerType.def.type === "pipe") {
assert(
description.startsWith("An array of"),
`Field description for field "${key}" with array type does not start with 'An array of'`
);
description = description.replace("An array of", "Comma separated values of");
}
}
Expand Down
52 changes: 26 additions & 26 deletions server.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,17 +52,17 @@
"isSecret": true
},
{
"name": "MDB_MCP_DISABLE_EMBEDDINGS_VALIDATION",
"description": "When set to true, disables validation of embeddings dimensions.",
"name": "MDB_MCP_DISABLED_TOOLS",
"description": "Comma separated values of tool names, operation types, and/or categories of tools that will be disabled.",
"isRequired": false,
"format": "boolean",
"format": "string",
"isSecret": false
},
{
"name": "MDB_MCP_DISABLED_TOOLS",
"description": "Comma separated values of tool names, operation types, and/or categories of tools that will be disabled.",
"name": "MDB_MCP_EMBEDDINGS_VALIDATION",
"description": "When set to false, disables validation of embeddings dimensions.",
"isRequired": false,
"format": "string",
"format": "boolean",
"isSecret": false
},
{
Expand Down Expand Up @@ -216,19 +216,19 @@
"description": "MongoDB connection string for direct database connections. Optional, if not set, you'll need to call the connect tool before interacting with MongoDB data.",
"isRequired": false
},
{
"type": "named",
"name": "--disableEmbeddingsValidation",
"description": "When set to true, disables validation of embeddings dimensions.",
"isRequired": false,
"format": "boolean"
},
{
"type": "named",
"name": "--disabledTools",
"description": "Comma separated values of tool names, operation types, and/or categories of tools that will be disabled.",
"isRequired": false
},
{
"type": "named",
"name": "--embeddingsValidation",
"description": "When set to false, disables validation of embeddings dimensions.",
"isRequired": false,
"format": "boolean"
},
{
"type": "named",
"name": "--exportCleanupIntervalMs",
Expand Down Expand Up @@ -380,17 +380,17 @@
"isSecret": true
},
{
"name": "MDB_MCP_DISABLE_EMBEDDINGS_VALIDATION",
"description": "When set to true, disables validation of embeddings dimensions.",
"name": "MDB_MCP_DISABLED_TOOLS",
"description": "Comma separated values of tool names, operation types, and/or categories of tools that will be disabled.",
"isRequired": false,
"format": "boolean",
"format": "string",
"isSecret": false
},
{
"name": "MDB_MCP_DISABLED_TOOLS",
"description": "Comma separated values of tool names, operation types, and/or categories of tools that will be disabled.",
"name": "MDB_MCP_EMBEDDINGS_VALIDATION",
"description": "When set to false, disables validation of embeddings dimensions.",
"isRequired": false,
"format": "string",
"format": "boolean",
"isSecret": false
},
{
Expand Down Expand Up @@ -544,19 +544,19 @@
"description": "MongoDB connection string for direct database connections. Optional, if not set, you'll need to call the connect tool before interacting with MongoDB data.",
"isRequired": false
},
{
"type": "named",
"name": "--disableEmbeddingsValidation",
"description": "When set to true, disables validation of embeddings dimensions.",
"isRequired": false,
"format": "boolean"
},
{
"type": "named",
"name": "--disabledTools",
"description": "Comma separated values of tool names, operation types, and/or categories of tools that will be disabled.",
"isRequired": false
},
{
"type": "named",
"name": "--embeddingsValidation",
"description": "When set to false, disables validation of embeddings dimensions.",
"isRequired": false,
"format": "boolean"
},
{
"type": "named",
"name": "--exportCleanupIntervalMs",
Expand Down
Loading