address comment: update flag

Author: blva

README.md

@@ -150,7 +150,7 @@ The MongoDB MCP Server can be configured using multiple methods, with the follow
 | `connectionString` | MongoDB connection string for direct database connections (optional users may choose to inform it on every tool call) |
 | `logPath`          | Folder to store logs                                                                                                  |
 | `disabledTools`    | An array of tool names, operation types, and/or categories of tools that will be disabled.                            |
-| `readOnlyMode`     | When set to true, only allows read and metadata operation types, disabling create/update/delete operations            |
+| `readOnly`         | When set to true, only allows read and metadata operation types, disabling create/update/delete operations            |
 
 #### `logPath`
 
@@ -184,14 +184,14 @@ Operation types:
 
 #### Read-Only Mode
 
-The `readOnlyMode` configuration option allows you to restrict the MCP server to only use tools with "read" and "metadata" operation types. When enabled, all tools that have "create", "update", "delete", or "cluster" operation types will not be registered with the server.
+The `readOnly` configuration option allows you to restrict the MCP server to only use tools with "read" and "metadata" operation types. When enabled, all tools that have "create", "update", "delete", or "cluster" operation types will not be registered with the server.
 
 This is useful for scenarios where you want to provide access to MongoDB data for analysis without allowing any modifications to the data or infrastructure.
 
 You can enable read-only mode using:
 
 - **Environment variable**: `export MDB_MCP_READ_ONLY_MODE=true`
-- **Command-line argument**: `--readOnlyMode=true`
+- **Command-line argument**: `--readOnly=true`
 
 When read-only mode is active, you'll see a message in the server logs indicating which tools were prevented from registering due to this restriction.
 

src/config.ts

@@ -20,7 +20,7 @@ export interface UserConfig {
         timeoutMS: number;
     };
     disabledTools: Array<string>;
-    readOnlyMode?: boolean;
+    readOnly?: boolean;
 }
 
 const defaults: UserConfig = {
@@ -33,7 +33,7 @@ const defaults: UserConfig = {
     },
     disabledTools: [],
     telemetry: "disabled",
-    readOnlyMode: false,
+    readOnly: false,
 };
 
 export const config = {

src/server.ts

@@ -38,7 +38,7 @@ export class Server {
         this.mcpServer.server.registerCapabilities({ logging: {} });
 
         // Log read-only mode status if enabled
-        if (this.userConfig.readOnlyMode) {
+        if (this.userConfig.readOnly) {
             logger.info(
                 mongoLogId(1_000_005),
                 "server",
@@ -126,7 +126,7 @@ export class Server {
 
         if (command === "start") {
             event.properties.startup_time_ms = commandDuration;
-            event.properties.read_only_mode = this.userConfig.readOnlyMode || false;
+            event.properties.read_only_mode = this.userConfig.readOnly || false;
         }
         if (command === "stop") {
             event.properties.runtime_duration_ms = Date.now() - this.startTime;

src/tools/tool.ts

@@ -88,7 +88,7 @@ export abstract class ToolBase {
         let errorClarification: string | undefined;
 
         // Check read-only mode first
-        if (this.config.readOnlyMode && !["read", "metadata"].includes(this.operationType)) {
+        if (this.config.readOnly && !["read", "metadata"].includes(this.operationType)) {
             logger.debug(
                 mongoLogId(1_000_010),
                 "tool",

tests/integration/server.test.ts

@@ -62,7 +62,9 @@ describe("Server integration test", () => {
     describe("with read-only mode", () => {
         const integration = setupIntegrationTest({
             ...config,
-            readOnlyMode: true,
+            readOnly: true,
+            apiClientId: "test",
+            apiClientSecret: "test",
         });
 
         it("should only register read and metadata operation tools when read-only mode is enabled", async () => {
@@ -74,6 +76,8 @@ describe("Server integration test", () => {
             expect(tools.tools.some((tool) => tool.name === "find")).toBe(true);
             expect(tools.tools.some((tool) => tool.name === "collection-schema")).toBe(true);
             expect(tools.tools.some((tool) => tool.name === "list-databases")).toBe(true);
+            expect(tools.tools.some((tool) => tool.name === "atlas-list-orgs")).toBe(true);
+            expect(tools.tools.some((tool) => tool.name === "atlas-list-projects")).toBe(true);
 
             // Check that non-read tools are NOT available
             expect(tools.tools.some((tool) => tool.name === "insert-one")).toBe(false);