fix: Objects without prototype are now properly stringified

SkeLLLa · Dec 17, 2024 · a6d1c08 · a6d1c08
1 parent 088a352
commit a6d1c08
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/__tests__/objectSorter.spec.ts b/__tests__/objectSorter.spec.ts
@@ -93,5 +93,13 @@ describe('Sorter', () => {
       const hash = hasher();
       expect(hash.sort(Object.create(null))).toBe('<:unknonw>:unknown');
     });
+    test('unknown with data', () => {
+      const hash = hasher();
+
+      const obj = Object.create(null) as Record<string, number>;
+      obj['a'] = 1;
+
+      expect(hash.sort(obj)).toBe('<:unknonw>:{"a":1}');
+    });
   });
 });
diff --git a/src/objectSorter.ts b/src/objectSorter.ts
@@ -226,7 +226,14 @@ export const objectSorter = (options?: SorterOptions): StringifyFn => {
     unknown: function _unknown(obj: Object) {
       // `unknonw` - is a typo, saved for backward compatibility
       const constructorName: string = obj.constructor?.name ?? 'unknonw';
-      const objectName = typeof obj.toString === 'function' ? obj.toString() : 'unknown';
+
+      let objectName = 'unknown';
+
+      if (typeof obj.toString === 'function') {
+        objectName = obj.toString();
+      } else if (Object.keys(obj).length > 0) {
+        objectName = JSON.stringify(obj);
+      }
 
       return `<:${constructorName}>:${objectName}`;
     },