fix TS issue

Author: baileympearson

src/operations/execute_operation.ts

@@ -295,7 +295,7 @@ async function tryOperation<
         const result = await server.modernCommand(operation, timeoutContext);
         return operation.handleOk(result);
       } catch (error) {
-        operation.handleError(error);
+        return operation.handleError(error);
       }
     } catch (operationError) {
       if (!(operationError instanceof MongoError)) throw operationError;
@@ -313,8 +313,8 @@ async function tryOperation<
     }
   }
 
-  if (previousOperationError) throw previousOperationError;
-
-  // @ts-expect-error asdf
-  return;
+  throw (
+    previousOperationError ??
+    new MongoRuntimeError('Tried to propagate retryability error, but no error was found.')
+  );
 }

src/operations/operation.ts

@@ -144,8 +144,14 @@ export abstract class ModernizedOperation<TResult> extends AbstractOperation<TRe
     throw new Error('cannot execute!!');
   }
 
+  /**
+   * Build a raw command document.
+   */
   abstract buildCommand(connection: Connection, session?: ClientSession): Document;
 
+  /**
+   * Builds an instance of `ServerCommandOptions` to be used for operation execution.
+   */
   abstract buildOptions(timeoutContext: TimeoutContext): ServerCommandOptions;
 
   /**
@@ -168,10 +174,12 @@ export abstract class ModernizedOperation<TResult> extends AbstractOperation<TRe
   }
 
   /**
-   * Optional - if the operation performs error handling, such as wrapping or renaming the error,
+   * Optional.
+   *
+   * If the operation performs error handling, such as wrapping, renaming the error, or squashing errors
    * this method can be overridden.
    */
-  handleError(error: MongoError): void {
+  handleError(error: MongoError): TResult | never {
     throw error;
   }
 }

src/operations/search_indexes/drop.ts

@@ -36,6 +36,7 @@ export class DropSearchIndexOperation extends ModernizedOperation<void> {
     if (typeof this.name === 'string') {
       command.name = this.name;
     }
+
     return command;
   }