chore: some minor refactors and linter checks

Author: kmruiz

src/common/config.ts

@@ -164,6 +164,11 @@ export const defaultDriverOptions: ConnectionInfo["driverOptions"] = {
     applyProxyToOIDC: true,
 };
 
+export const driverOptions = setupDriverConfig({
+    config,
+    defaults: defaultDriverOptions,
+});
+
 function getLogPath(): string {
     const logPath = path.join(getLocalDataPath(), "mongodb-mcp", ".app-logs");
     return logPath;
@@ -302,3 +307,17 @@ export function setupUserConfig({
 
     return userConfig;
 }
+
+export function setupDriverConfig({
+    config,
+    defaults,
+}: {
+    config: UserConfig;
+    defaults: ConnectionInfo["driverOptions"];
+}): ConnectionInfo["driverOptions"] {
+    const { driverOptions } = generateConnectionInfoFromCliArgs(config);
+    return {
+        ...defaults,
+        ...driverOptions,
+    };
+}

src/common/connectionManager.ts

@@ -1,4 +1,4 @@
-import { defaultDriverOptions } from "./config.js";
+import { driverOptions } from "./config.js";
 import { NodeDriverServiceProvider } from "@mongosh/service-provider-node-driver";
 import EventEmitter from "events";
 import { setAppNameParamIfMissing } from "../helpers/connectionOptions.js";
@@ -92,7 +92,7 @@ export class ConnectionManager extends EventEmitter<ConnectionManagerEvents> {
             serviceProvider = await NodeDriverServiceProvider.connect(settings.connectionString, {
                 productDocsLink: "https://github.com/mongodb-js/mongodb-mcp-server/",
                 productName: "MongoDB MCP",
-                ...defaultDriverOptions,
+                ...driverOptions,
             });
         } catch (error: unknown) {
             const errorReason = error instanceof Error ? error.message : `${error as string}`;

src/index.ts

@@ -88,7 +88,7 @@ main().catch((error: unknown) => {
     process.exit(1);
 });
 
-function assertFIPSMode() {
+function assertFIPSMode(): void | never {
     let fipsError: Error | undefined = undefined;
     if (config.tlsFIPSMode) {
         if (!fipsError && !crypto.getFips()) {
@@ -110,15 +110,15 @@ function assertFIPSMode() {
     }
 }
 
-function assertHelpMode() {
+function assertHelpMode(): void | never {
     if (config.help) {
         console.log("For usage information refer to the README.md:");
         console.log("https://github.com/mongodb-js/mongodb-mcp-server?tab=readme-ov-file#quick-start");
         process.exit(0);
     }
 }
 
-function assertVersionMode() {
+function assertVersionMode(): void | never {
     if (config.version) {
         console.log(packageInfo.version);
         process.exit(0);