Solves issue #365 with crush() not handling dots in object keys

.vscode/settings.json

@@ -0,0 +1,3 @@
+{
+  "autoHide.autoHidePanel": false
+}

cdn/radash.esm.js

@@ -770,10 +770,16 @@ const keys = (value) => {
 const crush = (value) => {
   if (!value)
     return {};
-  return objectify(
-    keys(value),
-    (k) => k,
-    (k) => get(value, k)
+  const getKeys = (nested, paths) => {
+    const inner = (k, v) => {
+      if (!(isObject(v) || isArray(v)))
+        return [{ [[...paths, `${k}`].join(".")]: v }];
+      return getKeys(v, [...paths, `${k}`]);
+    };
+    return isArray(nested) ? nested.flatMap((v, i) => inner(`${i}`, v)) : Object.entries(nested).flatMap(([k, v]) => inner(k, v));
+  };
+  return Object.fromEntries(
+    getKeys(value, []).flatMap((obj) => Object.entries(obj))
   );
 };
 const construct = (obj) => {

cdn/radash.js

@@ -773,10 +773,16 @@ var radash = (function (exports) {
   const crush = (value) => {
     if (!value)
       return {};
-    return objectify(
-      keys(value),
-      (k) => k,
-      (k) => get(value, k)
+    const getKeys = (nested, paths) => {
+      const inner = (k, v) => {
+        if (!(isObject(v) || isArray(v)))
+          return [{ [[...paths, `${k}`].join(".")]: v }];
+        return getKeys(v, [...paths, `${k}`]);
+      };
+      return isArray(nested) ? nested.flatMap((v, i) => inner(`${i}`, v)) : Object.entries(nested).flatMap(([k, v]) => inner(k, v));
+    };
+    return Object.fromEntries(
+      getKeys(value, []).flatMap((obj) => Object.entries(obj))
     );
   };
   const construct = (obj) => {

src/object.ts

@@ -321,7 +321,7 @@ export const keys = <TValue extends object>(value: TValue): string[] => {
  * crush({ name: 'ra', children: [{ name: 'hathor' }] })
  * // { name: 'ra', 'children.0.name': 'hathor' }
  */
-export const crush = <TValue extends object>(value: TValue): object => {
+export const old_crush = <TValue extends object>(value: TValue): object => {
   if (!value) return {}
   return objectify(
     keys(value),
@@ -330,6 +330,24 @@ export const crush = <TValue extends object>(value: TValue): object => {
   )
 }
 
+export const crush = <TValue extends object>(value: TValue): object => {
+  if (!value) return {}
+  const getKeys = (nested: any, paths: string[]): object[] => {
+    const inner = (k: string, v: any) => {
+      if (!(isObject(v) || isArray(v)))
+        return [{ [[...paths, `${k}`].join('.')]: v }]
+      return getKeys(v, [...paths, `${k}`])
+    }
+
+    return isArray(nested)
+      ? nested.flatMap((v, i) => inner(`${i}`, v))
+      : Object.entries(nested).flatMap(([k, v]) => inner(k, v))
+  }
+  return Object.fromEntries(
+    getKeys(value, []).flatMap(obj => Object.entries(obj))
+  )
+}
+
 /**
  * The opposite of crush, given an object that was
  * crushed into key paths and values will return

src/tests/object.test.ts

@@ -517,16 +517,8 @@ describe('object module', () => {
       const ra = {
         name: 'ra',
         power: 100,
-        friend: {
-          name: 'loki',
-          power: 80
-        },
-        enemies: [
-          {
-            name: 'hathor',
-            power: 12
-          }
-        ],
+        friend: { name: 'loki', power: 80 },
+        enemies: [{ name: 'hathor', power: 12 }],
         timestamp: now
       }
       assert.deepEqual(_.crush(ra), {
@@ -539,6 +531,23 @@ describe('object module', () => {
         timestamp: now
       })
     })
+    test('handles key in dots correctly', () => {
+      const now = new Date()
+      const ra = {
+        key1: 'value1',
+        obj1: { x: 'x', y: 10 },
+        arr1: ['xx', { key2: 'value2' }, { key3: [false, now] }]
+      }
+      assert.deepEqual(_.crush(ra), {
+        key1: 'value1',
+        'obj1.x': 'x',
+        'obj1.y': 10,
+        'arr1.0': 'xx',
+        'arr1.1.key2': 'value2',
+        'arr1.2.key3.0': false,
+        'arr1.2.key3.1': now
+      })
+    })
   })
 
   describe('construct function', () => {