chore: style changes, use a getter for isConnectedToMongoDB

Author: kmruiz

src/common/session.ts

@@ -105,22 +105,22 @@ export class Session extends EventEmitter<SessionEvents> {
         await this.apiClient.close();
     }
 
-    async connectToMongoDB(settings: ConnectionSettings): Promise<AnyConnectionState> {
+    async connectToMongoDB(settings: ConnectionSettings): Promise<void> {
         try {
-            return await this.connectionManager.connect({ ...settings });
+            await this.connectionManager.connect({ ...settings });
         } catch (error: unknown) {
             const message = error instanceof Error ? error.message : (error as string);
             this.emit("connection-error", message);
             throw error;
         }
     }
 
-    isConnectedToMongoDB(): boolean {
+    get isConnectedToMongoDB(): boolean {
         return this.connectionManager.currentConnectionState.tag === "connected";
     }
 
     get serviceProvider(): NodeDriverServiceProvider {
-        if (this.isConnectedToMongoDB()) {
+        if (this.isConnectedToMongoDB) {
             const state = this.connectionManager.currentConnectionState as ConnectionStateConnected;
             return state.serviceProvider;
         }

src/tools/mongodb/connect/connect.ts

@@ -87,7 +87,7 @@ export class ConnectTool extends MongoDBToolBase {
     }
 
     private updateMetadata(): void {
-        if (this.session.isConnectedToMongoDB()) {
+        if (this.session.isConnectedToMongoDB) {
             this.update?.({
                 name: connectedName,
                 description: connectedDescription,

src/tools/mongodb/mongodbTool.ts

@@ -17,7 +17,7 @@ export abstract class MongoDBToolBase extends ToolBase {
     public category: ToolCategory = "mongodb";
 
     protected async ensureConnected(): Promise<NodeDriverServiceProvider> {
-        if (!this.session.isConnectedToMongoDB()) {
+        if (!this.session.isConnectedToMongoDB) {
             if (this.session.connectedAtlasCluster) {
                 throw new MongoDBError(
                     ErrorCodes.NotConnectedToMongoDB,
@@ -39,7 +39,7 @@ export abstract class MongoDBToolBase extends ToolBase {
             }
         }
 
-        if (!this.session.isConnectedToMongoDB()) {
+        if (!this.session.isConnectedToMongoDB) {
             throw new MongoDBError(ErrorCodes.NotConnectedToMongoDB, "Not connected to MongoDB");
         }
 
@@ -117,7 +117,7 @@ export abstract class MongoDBToolBase extends ToolBase {
         return super.handleError(error, args);
     }
 
-    protected connectToMongoDB(connectionString: string): Promise<AnyConnectionState> {
+    protected connectToMongoDB(connectionString: string): Promise<void> {
         return this.session.connectToMongoDB({ connectionString, ...this.config.connectOptions });
     }