fix: use prototypeless objects for field-to-type mappings

Fixes: #211
mongodb-js · Dec 15, 2023 · a6b79ca · a6b79ca
1 parent ac1616d
commit a6b79ca
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 7 deletions.
diff --git a/src/schema-analyzer.ts b/src/schema-analyzer.ts
@@ -1,6 +1,6 @@
 
 import Reservoir from 'reservoir';
-import type {
+import {
   Document,
   ObjectId,
   MinKey,
@@ -432,7 +432,7 @@ export class SchemaAnalyzer {
   options: SchemaParseOptions;
   documentsAnalyzed = 0;
   schemaAnalysisRoot: SchemaAnalysisRoot = {
-    fields: {},
+    fields: Object.create(null),
     count: 0
   };
 
@@ -513,7 +513,7 @@ export class SchemaAnalyzer {
         (value as BSONValue[]).forEach((v: BSONValue) => addToType(path, v, type.types));
       } else if (isDocumentType(type)) {
         // Recurse into nested documents by calling `addToField` for all sub-fields.
-        type.fields = type.fields ?? {};
+        type.fields = type.fields ?? Object.create(null);
         Object.entries(value as Document).forEach(
           ([fieldName, v]) => addToField(fieldName, [...path, fieldName], v, type.fields)
         );

diff --git a/test/basic.test.ts b/test/basic.test.ts
@@ -19,7 +19,8 @@ describe('using only basic fields', function() {
       last_address_longitude: null,
       created_at: new Date(),
       length: 29,
-      'name[]': 'Annabeth Frankie'
+      'name[]': 'Annabeth Frankie',
+      toString: 42
     }
   ];
 
@@ -38,7 +39,8 @@ describe('using only basic fields', function() {
       'name',
       'stats_friends',
       'twitter_username',
-      'name[]'
+      'name[]',
+      'toString'
     ];
     assert.deepEqual(users.fields.map(v => v.name).sort(), fieldNames.sort());
   });
@@ -59,5 +61,6 @@ describe('using only basic fields', function() {
     assert.equal(users.fields.find(v => v.name === 'name')?.type, 'String');
     assert.equal(users.fields.find(v => v.name === 'stats_friends')?.type, 'Number');
     assert.equal(users.fields.find(v => v.name === 'twitter_username')?.type, 'String');
+    assert.equal(users.fields.find(v => v.name === 'toString')?.type, 'Number');
   });
 });
diff --git a/test/mixed-type-nested.test.ts b/test/mixed-type-nested.test.ts
@@ -30,10 +30,11 @@ describe('mixed types nested', function() {
     {
       _id: 5,
       address: {
-        valid: true
+        valid: true,
+        toString: { value: false }
       }
     }
-  ];
+  ] as const;
 
   let schema: Schema;
   let valid: SchemaField | undefined;