Addresses copilot review

Author: cveticm

src/common/atlas/cluster.ts

@@ -114,16 +114,14 @@ export async function inspectCluster(apiClient: ApiClient, projectId: string, cl
     }
 }
 
-// getConnectionString returns a connection string given a connectionType.
-// For "privateEndpoint", it returns the first private endpoint connection string available.
+/**
+ * Returns a connection string for the specified connectionType.
+ * For "privateEndpoint", it returns the first private endpoint connection string available.
+ */
 export function getConnectionString(
     connectionStrings: ClusterConnectionStrings,
     connectionType: "standard" | "private" | "privateEndpoint"
 ): string | undefined {
-    if (connectionStrings === undefined) {
-        return undefined;
-    }
-
     switch (connectionType) {
         case "standard":
             return connectionStrings.standardSrv || connectionStrings.standard;

src/tools/atlas/connect/connectCluster.ts

@@ -22,9 +22,9 @@ function sleep(ms: number): Promise<void> {
 export const ConnectClusterArgs = {
     projectId: AtlasArgs.projectId().describe("Atlas project ID"),
     clusterName: AtlasArgs.clusterName().describe("Atlas cluster name"),
-    connectionType: AtlasArgs.connectionType()
-        .optional()
-        .describe("Type of connection (standard, private, or privateEndpoint) to an Atlas cluster"),
+    connectionType: AtlasArgs.connectionType().describe(
+        "Type of connection (standard, private, or privateEndpoint) to an Atlas cluster"
+    ),
 };
 
 export class ConnectClusterTool extends AtlasToolBase {