Update: Handle additional edge cases in set function

- Add checks to prevent invalid paths like empty strings or '[]'
- Ensure correct behavior when path is just a number index

Author: nightohl

src/object.ts

@@ -245,6 +245,13 @@ export const set = <T extends object, K>(
 ): T => {
   if (!initial) return {} as T
   if (!path || value === undefined) return initial
+  if (
+    typeof path !== 'string' ||
+    !/(\w+|\[\d+\])/.test(path) ||
+    path === '[]'
+  ) {
+    return initial
+  }
 
   const segments: (string | number)[] =
     path
@@ -253,6 +260,8 @@ export const set = <T extends object, K>(
         segment.startsWith('[') ? Number(segment.slice(1, -1)) : segment
       ) ?? []
 
+  if (segments.length === 1 && typeof segments[0] === 'number') return initial
+
   const _set = (node: any) => {
     if (segments.length > 1) {
       const key = segments.shift()!

src/tests/object.test.ts

@@ -488,6 +488,11 @@ describe('object module', () => {
       assert.deepEqual(_.set(null as any, null as any, null as any), {})
       assert.deepEqual(_.set({ foo: true }, 'foo', false), { foo: false })
       assert.deepEqual(_.set({}, 'foo', 0), { foo: 0 })
+      assert.deepEqual(_.set({}, '', 2), {})
+      assert.deepEqual(_.set({}, '[]', 2), {})
+      assert.deepEqual(_.set({}, '[0]', 2), {})
+      assert.deepEqual(_.set({}, undefined as any, 2), {})
+      assert.deepEqual(_.set({}, null as any, 2), {})
     })
     test('sets deep values correctly', () => {
       assert.deepEqual(_.set({}, 'cards.value', 2), {