chore: generate the connection string properly on first connection

This is necessary when connecting for the first time.

Author: kmruiz

src/common/config.ts

@@ -1,8 +1,8 @@
 import path from "path";
 import os from "os";
 import argv from "yargs-parser";
-import type { CliOptions } from "@mongosh/arg-parser";
-import type { ConnectionInfo } from "@mongosh/arg-parser";
+import type { CliOptions, ConnectionInfo } from "@mongosh/arg-parser";
+import { generateConnectionInfoFromCliArgs } from "@mongosh/arg-parser";
 
 // From: https://github.com/mongodb-js/mongosh/blob/main/packages/cli-repl/src/arg-parser.ts
 const OPTIONS = {
@@ -286,21 +286,9 @@ export function setupUserConfig({
     userConfig.loggers = commaSeparatedToArray(userConfig.loggers);
 
     if (userConfig.connectionString && userConfig.connectionSpecifier) {
-        delete userConfig.connectionString;
+        const connectionInfo = generateConnectionInfoFromCliArgs(userConfig);
+        userConfig.connectionString = connectionInfo.connectionString;
     }
 
     return userConfig;
 }
-
-/**
-                   readConcern: {
-                    level: settings.readConcern,
-                },
-                readPreference: settings.readPreference,
-                writeConcern: {
-                    w: settings.writeConcern,
-                },
-                timeoutMS: settings.timeoutMS,
-                proxy: { useEnvironmentVariableProxies: true },
-                applyProxyToOIDC: true,
-**/

src/lib.ts

@@ -1,4 +1,4 @@
 export { Server, type ServerOptions } from "./server.js";
 export { Telemetry } from "./telemetry/telemetry.js";
 export { Session, type SessionOptions } from "./common/session.js";
-export type { UserConfig, ConnectOptions } from "./common/config.js";
+export type { UserConfig } from "./common/config.js";

tests/unit/common/config.test.ts

@@ -404,7 +404,10 @@ describe("config", () => {
                 defaults: {},
             });
 
-            expect(actual.connectionString).toBe(undefined);
+            // the shell specifies directConnection=true and serverSelectionTimeoutMS=2000 by default
+            expect(actual.connectionString).toBe(
+                "mongodb://localhost/?directConnection=true&serverSelectionTimeoutMS=2000"
+            );
             expect(actual.connectionSpecifier).toBe("mongodb://localhost");
         });
     });