Add correct handling of non string, number or symbol values to unique

cdn/radash.esm.js

@@ -197,12 +197,12 @@ const cluster = (list2, size = 2) => {
 const unique = (array, toKey) => {
   const valueMap = array.reduce((acc, item) => {
     const key = toKey ? toKey(item) : item;
-    if (acc[key])
+    if (acc.get(key))
       return acc;
-    acc[key] = item;
+    acc.set(key, item);
     return acc;
-  }, {});
-  return Object.values(valueMap);
+  }, /* @__PURE__ */ new Map());
+  return Array.from(valueMap.values());
 };
 function* range(startOrLength, end, valueOrMapper = (i) => i, step = 1) {
   const mapper = isFunction(valueOrMapper) ? valueOrMapper : () => valueOrMapper;

cdn/radash.js

@@ -200,12 +200,12 @@ var radash = (function (exports) {
   const unique = (array, toKey) => {
     const valueMap = array.reduce((acc, item) => {
       const key = toKey ? toKey(item) : item;
-      if (acc[key])
+      if (acc.get(key))
         return acc;
-      acc[key] = item;
+      acc.set(key, item);
       return acc;
-    }, {});
-    return Object.values(valueMap);
+    }, /* @__PURE__ */ new Map());
+    return Array.from(valueMap.values());
   };
   function* range(startOrLength, end, valueOrMapper = (i) => i, step = 1) {
     const mapper = isFunction(valueOrMapper) ? valueOrMapper : () => valueOrMapper;

package.json

@@ -1,6 +1,6 @@
 {
   "name": "radash",
-  "version": "11.0.0",
+  "version": "11.0.1",
   "description": "Functional utility library - modern, simple, typed, powerful",
   "main": "dist/cjs/index.cjs",
   "module": "dist/esm/index.mjs",

src/array.ts

@@ -291,17 +291,17 @@ export const cluster = <T>(list: readonly T[], size: number = 2): T[][] => {
  * to convert each item in the list to a comparable identity
  * value
  */
-export const unique = <T, K extends string | number | symbol>(
+export const unique = <T, K>(
   array: readonly T[],
   toKey?: (item: T) => K
 ): T[] => {
   const valueMap = array.reduce((acc, item) => {
-    const key = toKey ? toKey(item) : (item as any as string | number | symbol)
-    if (acc[key]) return acc
-    acc[key] = item
+    const key = toKey ? toKey(item) : item
+    if (acc.get(key)) return acc
+    acc.set(key, item)
     return acc
-  }, {} as Record<string | number | symbol, T>)
-  return Object.values(valueMap)
+  }, new Map())
+  return Array.from(valueMap.values())
 }
 
 /**

src/tests/array.test.ts

@@ -390,6 +390,27 @@ describe('array module', () => {
       assert.equal(c.id, 'c')
       assert.equal(c.word, 'yolo')
     })
+    test('correctly handles non string, number or symbol values', () => {
+      const list = [
+        null,
+        null,
+        true,
+        true,
+        'true',
+        false,
+        { id: 'a', word: 'hello' },
+        { id: 'a', word: 'hello' }
+      ]
+      const result = _.unique(list)
+      assert.deepEqual(result, [
+        null,
+        true,
+        'true',
+        false,
+        { id: 'a', word: 'hello' },
+        { id: 'a', word: 'hello' }
+      ])
+    })
   })
 
   describe('range function', () => {