mongodb 4.3.1 → 4.5.0

package/src/utils.ts

@@ -8,11 +8,12 @@ import type { Connection } from './cmap/connection';
 import { MAX_SUPPORTED_WIRE_VERSION } from './cmap/wire_protocol/constants';
 import type { Collection } from './collection';
 import { LEGACY_HELLO_COMMAND } from './constants';
+import type { AbstractCursor } from './cursor/abstract_cursor';
+import type { FindCursor } from './cursor/find_cursor';
 import type { Db } from './db';
 import {
   AnyError,
   MongoCompatibilityError,
-  MongoExpiredSessionError,
   MongoInvalidArgumentError,
   MongoNotConnectedError,
   MongoParseError,
@@ -37,8 +38,6 @@ import { W, WriteConcern, WriteConcernOptions } from './write_concern';
  * @public
  */
 export type Callback<T = any> = (error?: AnyError, result?: T) => void;
-/** @public */
-export type CallbackWithType<E = AnyError, T0 = any> = (error?: E, result?: T0) => void;
 
 export const MAX_JS_INT = Number.MAX_SAFE_INTEGER + 1;
 
@@ -190,110 +189,6 @@ export function filterOptions(options: AnyOptions, names: string[]): AnyOptions
   return filterOptions;
 }
 
-/**
- * Executes the given operation with provided arguments.
- *
- * @remarks
- * This method reduces large amounts of duplication in the entire codebase by providing
- * a single point for determining whether callbacks or promises should be used. Additionally
- * it allows for a single point of entry to provide features such as implicit sessions, which
- * are required by the Driver Sessions specification in the event that a ClientSession is
- * not provided
- *
- * @internal
- *
- * @param topology - The topology to execute this operation on
- * @param operation - The operation to execute
- * @param args - Arguments to apply the provided operation
- * @param options - Options that modify the behavior of the method
- */
-export function executeLegacyOperation(
-  topology: Topology,
-  operation: (...args: any[]) => void | Promise<Document>,
-  args: any[],
-  options?: AnyOptions
-): void | Promise<any> {
-  const Promise = PromiseProvider.get();
-
-  if (!Array.isArray(args)) {
-    // TODO(NODE-3483)
-    throw new MongoRuntimeError('This method requires an array of arguments to apply');
-  }
-
-  options = options ?? {};
-
-  let callback = args[args.length - 1];
-
-  // The driver sessions spec mandates that we implicitly create sessions for operations
-  // that are not explicitly provided with a session.
-  let session: ClientSession;
-  let opOptions: any;
-  let owner: any;
-  if (!options.skipSessions && topology.hasSessionSupport()) {
-    opOptions = args[args.length - 2];
-    if (opOptions == null || opOptions.session == null) {
-      owner = Symbol();
-      session = topology.startSession({ owner });
-      const optionsIndex = args.length - 2;
-      args[optionsIndex] = Object.assign({}, args[optionsIndex], { session: session });
-    } else if (opOptions.session && opOptions.session.hasEnded) {
-      throw new MongoExpiredSessionError();
-    }
-  }
-
-  function makeExecuteCallback(
-    resolve: (value?: Document) => void,
-    reject: (reason?: AnyError) => void
-  ) {
-    return function (err?: AnyError, result?: any) {
-      if (session && session.owner === owner && !options?.returnsCursor) {
-        session.endSession(() => {
-          delete opOptions.session;
-          if (err) return reject(err);
-          resolve(result);
-        });
-      } else {
-        if (err) return reject(err);
-        resolve(result);
-      }
-    };
-  }
-
-  // Execute using callback
-  if (typeof callback === 'function') {
-    callback = args.pop();
-    const handler = makeExecuteCallback(
-      result => callback(undefined, result),
-      err => callback(err, null)
-    );
-    args.push(handler);
-
-    try {
-      return operation(...args);
-    } catch (e) {
-      handler(e);
-      throw e;
-    }
-  }
-
-  // Return a Promise
-  if (args[args.length - 1] != null) {
-    // TODO(NODE-3483)
-    throw new MongoRuntimeError('Final argument to `executeLegacyOperation` must be a callback');
-  }
-
-  return new Promise<any>((resolve, reject) => {
-    const handler = makeExecuteCallback(resolve, reject);
-    args[args.length - 1] = handler;
-
-    try {
-      return operation(...args);
-    } catch (e) {
-      handler(e);
-    }
-  });
-}
-
 interface HasRetryableWrites {
   retryWrites?: boolean;
 }
@@ -434,17 +329,30 @@ export function decorateWithExplain(command: Document, explain: Explain): Docume
   return { explain: command, verbosity: explain.verbosity };
 }
 
+/**
+ * @internal
+ */
+export type TopologyProvider =
+  | MongoClient
+  | ClientSession
+  | FindCursor
+  | AbstractCursor
+  | Collection<any>
+  | Db;
+
 /**
  * A helper function to get the topology from a given provider. Throws
  * if the topology cannot be found.
+ * @throws MongoNotConnectedError
  * @internal
  */
-export function getTopology<T>(provider: MongoClient | Db | Collection<T>): Topology {
+export function getTopology(provider: TopologyProvider): Topology {
+  // MongoClient or ClientSession or AbstractCursor
   if (`topology` in provider && provider.topology) {
     return provider.topology;
-  } else if ('client' in provider.s && provider.s.client.topology) {
+  } else if ('s' in provider && 'client' in provider.s && provider.s.client.topology) {
     return provider.s.client.topology;
-  } else if ('db' in provider.s && provider.s.db.s.client.topology) {
+  } else if ('s' in provider && 'db' in provider.s && provider.s.db.s.client.topology) {
     return provider.s.db.s.client.topology;
   }
 
@@ -1181,7 +1089,7 @@ export function deepCopy<T>(value: T): T {
       case 'set':
         return new Set(value as any) as unknown as T;
       case 'buffer':
-        return Buffer.from(value as Buffer) as unknown as T;
+        return Buffer.from(value as unknown as Buffer) as unknown as T;
     }
   }
 
@@ -1419,13 +1327,25 @@ export function enumToString(en: Record<string, unknown>): string {
  *
  * @internal
  */
-export function supportsRetryableWrites(server: Server): boolean {
-  return (
-    !!server.loadBalanced ||
-    (server.description.maxWireVersion >= 6 &&
-      !!server.description.logicalSessionTimeoutMinutes &&
-      server.description.type !== ServerType.Standalone)
-  );
+export function supportsRetryableWrites(server?: Server): boolean {
+  if (!server) {
+    return false;
+  }
+
+  if (server.loadBalanced) {
+    // Loadbalanced topologies will always support retry writes
+    return true;
+  }
+
+  if (server.description.logicalSessionTimeoutMinutes != null) {
+    // that supports sessions
+    if (server.description.type !== ServerType.Standalone) {
+      // and that is not a standalone
+      return true;
+    }
+  }
+
+  return false;
 }
 
 export function parsePackageVersion({ version }: { version: string }): {

package/tsconfig.json

@@ -0,0 +1,40 @@
+{
+  "compilerOptions": {
+    "allowJs": false,
+    "checkJs": false,
+    "strict": true,
+    "alwaysStrict": true,
+    "target": "ES2019",
+    "module": "commonJS",
+    "moduleResolution": "node",
+    "skipLibCheck": true,
+    "lib": ["es2020"],
+    // We don't make use of tslib helpers, all syntax used is supported by target engine
+    "importHelpers": false,
+    "noEmitHelpers": true,
+    // Never emit error filled code
+    "noEmitOnError": true,
+    "outDir": "lib",
+    "importsNotUsedAsValues": "error",
+    // We want the sourcemaps in a separate file
+    "inlineSourceMap": false,
+    "sourceMap": true,
+    // API-Extractor uses declaration maps to report problems in source, no need to distribute
+    "declaration": true,
+    "declarationMap": true,
+    // we include sources in the release
+    "inlineSources": false,
+    // Prevents web types from being suggested by vscode.
+    "types": ["node"],
+    "forceConsistentCasingInFileNames": true,
+    "noImplicitOverride": true,
+    "noImplicitReturns": true,
+    // TODO(NODE-3659): Enable useUnknownInCatchVariables and add type assertions or remove unnecessary catch blocks
+    "useUnknownInCatchVariables": false
+  },
+  "ts-node": {
+    "transpileOnly": true,
+    "compiler": "typescript-cached-transpile"
+  },
+  "include": ["src/**/*"]
+}