Revert "perf(NODE-6356): Improve serialization performance (#709)"

This reverts commit 61537f5.
mongodb · Nov 21, 2024 · 43f6403 · 43f6403
1 parent dbeda2f
commit 43f6403
Show file tree

Hide file tree

Showing 7 changed files with 202 additions and 249 deletions.
diff --git a/src/bson_value.ts b/src/bson_value.ts
@@ -1,14 +1,13 @@
 import { BSON_MAJOR_VERSION } from './constants';
 import { type InspectFn } from './parser/utils';
-import { BSON_VERSION_SYMBOL } from './constants';
 
 /** @public */
 export abstract class BSONValue {
   /** @public */
   public abstract get _bsontype(): string;
 
   /** @internal */
-  get [BSON_VERSION_SYMBOL](): typeof BSON_MAJOR_VERSION {
+  get [Symbol.for('@@mdb.bson.version')](): typeof BSON_MAJOR_VERSION {
     return BSON_MAJOR_VERSION;
   }
 

diff --git a/src/constants.ts b/src/constants.ts
@@ -1,9 +1,6 @@
 /** @internal */
 export const BSON_MAJOR_VERSION = 6;
 
-/** @internal */
-export const BSON_VERSION_SYMBOL = Symbol.for('@@mdb.bson.version');
-
 /** @internal */
 export const BSON_INT32_MAX = 0x7fffffff;
 /** @internal */

diff --git a/src/decimal128.ts b/src/decimal128.ts
@@ -142,7 +142,7 @@ export class Decimal128 extends BSONValue {
     super();
     if (typeof bytes === 'string') {
       this.bytes = Decimal128.fromString(bytes).bytes;
-    } else if (bytes instanceof Uint8Array || isUint8Array(bytes)) {
+    } else if (isUint8Array(bytes)) {
       if (bytes.byteLength !== 16) {
         throw new BSONError('Decimal128 must take a Buffer of 16 bytes');
       }

diff --git a/src/extended_json.ts b/src/extended_json.ts
@@ -6,8 +6,7 @@ import {
   BSON_INT32_MIN,
   BSON_INT64_MAX,
   BSON_INT64_MIN,
-  BSON_MAJOR_VERSION,
-  BSON_VERSION_SYMBOL
+  BSON_MAJOR_VERSION
 } from './constants';
 import { DBRef, isDBRefLike } from './db_ref';
 import { Decimal128 } from './decimal128';
@@ -359,7 +358,7 @@ function serializeDocument(doc: any, options: EJSONSerializeOptions) {
     doc != null &&
     typeof doc === 'object' &&
     typeof doc._bsontype === 'string' &&
-    doc[BSON_VERSION_SYMBOL] !== BSON_MAJOR_VERSION
+    doc[Symbol.for('@@mdb.bson.version')] !== BSON_MAJOR_VERSION
   ) {
     throw new BSONVersionError();
   } else if (isBSONType(doc)) {

diff --git a/src/parser/calculate_size.ts b/src/parser/calculate_size.ts
@@ -81,7 +81,7 @@ function calculateElement(
       if (
         value != null &&
         typeof value._bsontype === 'string' &&
-        value[constants.BSON_VERSION_SYMBOL] !== constants.BSON_MAJOR_VERSION
+        value[Symbol.for('@@mdb.bson.version')] !== constants.BSON_MAJOR_VERSION
       ) {
         throw new BSONVersionError();
       } else if (value == null || value._bsontype === 'MinKey' || value._bsontype === 'MaxKey') {