fix: some comments

Author: nbbeeken

src/operations/execute_operation.ts

@@ -78,18 +78,21 @@ export function executeOperation<
   return maybeCallback(() => executeOperationAsync(client, operation), callback);
 }
 
-async function assertTopology(client: MongoClient) {
+async function maybeConnect(client: MongoClient): Promise<Topology> {
   const { topology } = client;
   if (topology == null) {
     if (client.s.hasBeenClosed) {
       throw new MongoNotConnectedError('Client must be connected before running operations');
     }
     client.s.options[Symbol.for('@@mdb.skipPingOnConnect')] = true;
-    await client.connect();
-    delete client.s.options[Symbol.for('@@mdb.skipPingOnConnect')];
+    try {
+      await client.connect();
+    } finally {
+      delete client.s.options[Symbol.for('@@mdb.skipPingOnConnect')];
+    }
   }
   if (client.topology == null) {
-    throw new MongoNotConnectedError('Client must be connected before running operations');
+    throw new MongoRuntimeError('client.connect did not create a topology but also did not throw');
   }
   return client.topology;
 }
@@ -103,7 +106,7 @@ async function executeOperationAsync<
     throw new MongoRuntimeError('This method requires a valid operation instance');
   }
 
-  const topology = await assertTopology(client);
+  const topology = await maybeConnect(client);
 
   if (topology.shouldCheckForSessionSupport()) {
     await topology.selectServerAsync(ReadPreference.primaryPreferred, {});

src/operations/rename.ts

@@ -38,7 +38,11 @@ export class RenameOperation extends RunAdminCommandOperation {
     this.newName = newName;
   }
 
-  override execute(server: Server, session: ClientSession | undefined, callback: Callback): void {
+  override execute(
+    server: Server,
+    session: ClientSession | undefined,
+    callback: Callback<Collection>
+  ): void {
     const coll = this.collection;
 
     super.execute(server, session, (err, doc) => {

src/operations/stats.ts

@@ -33,7 +33,11 @@ export class CollStatsOperation extends CommandOperation<Document> {
     this.collectionName = collection.collectionName;
   }
 
-  override execute(server: Server, session: ClientSession | undefined, callback: Callback): void {
+  override execute(
+    server: Server,
+    session: ClientSession | undefined,
+    callback: Callback<CollStats>
+  ): void {
     const command: Document = { collStats: this.collectionName };
     if (this.options.scale != null) {
       command.scale = this.options.scale;