chore: use a dictionary of flags with urls instead

Author: kmruiz

src/common/config.ts

@@ -87,7 +87,7 @@ const OPTIONS = {
         "greedy-arrays": true,
         "short-option-groups": false,
     },
-};
+} as const;
 
 function isConnectionSpecifier(arg: string | undefined): boolean {
     return (
@@ -185,14 +185,15 @@ function getExportsPath(): string {
 // are prefixed with `MDB_MCP_` and the keys match the UserConfig keys, but are converted
 // to SNAKE_UPPER_CASE.
 function parseEnvConfig(env: Record<string, unknown>): Partial<UserConfig> {
+    const CONFIG_WITH_URLS: Set<string> = new Set<(typeof OPTIONS)["string"][number]>(["connectionString"]);
+
     function setValue(obj: Record<string, unknown>, path: string[], value: string): void {
         const currentField = path.shift();
         if (!currentField) {
             return;
         }
         if (path.length === 0) {
-            // MongoDB URLs must not be preprocessed
-            if (typeof value === "string" && (value.startsWith("mongodb://") || value.startsWith("mongodb+srv://"))) {
+            if (CONFIG_WITH_URLS.has(currentField)) {
                 obj[currentField] = value;
                 return;
             }