mend

src/index.ts

@@ -279,24 +279,21 @@ function setByPath<
   }
 
   // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
-  const parentObject = pathArray.reduce(
-    (current: any, pathPart) => {
-      if (typeof current !== 'object' || !hasOwnProperty.call(current, pathPart)) {
-        throw new Error(`Property ${pathPart} is undefined`)
-      }
-
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-member-access
-      const next = current?.[pathPart]
-
-      if (next === undefined || next === null) {
-        throw new Error(`Property ${pathPart} is undefined`)
-      }
-
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-return
-      return next
-    },
-    object,
-  )
+  const parentObject = pathArray.reduce((current: any, pathPart) => {
+    if (typeof current !== 'object' || !hasOwnProperty.call(current, pathPart)) {
+      throw new Error(`Property ${pathPart} is undefined`)
+    }
+
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-member-access
+    const next = current?.[pathPart]
+
+    if (next === undefined || next === null) {
+      throw new Error(`Property ${pathPart} is undefined`)
+    }
+
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-return
+    return next
+  }, object)
 
   // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
   parentObject[lastKey] = value

test/index.test.ts

@@ -1,4 +1,4 @@
-import { expect, it,  } from 'vitest'
+import { expect, it } from 'vitest'
 
 import { getByPath, setByPath } from '../src'
 
@@ -165,16 +165,20 @@ it('Test for prototype pollution', () => {
   const object = {}
 
   // @ts-expect-error - this is not a valid path for the object
-  expect(() => { setByPath(object, '__proto__.polluted', true) }).toThrowError('__proto__');
+  expect(() => {
+    setByPath(object, '__proto__.polluted', true)
+  }).toThrowError('__proto__')
 
   // @ts-expect-error - this is not a valid path for the object
   expect(getByPath(object, '__proto__')).toBe(undefined)
 
   // @ts-expect-error - this is not a valid path for the object
-  expect(() => { setByPath(object, 'constructor.polluted', true) }).toThrowError('constructor');
+  expect(() => {
+    setByPath(object, 'constructor.polluted', true)
+  }).toThrowError('constructor')
 
   // @ts-expect-error - this is not a valid path for the object
-  expect(getByPath(object, 'constructor')).toBe(undefined);
+  expect(getByPath(object, 'constructor')).toBe(undefined)
 
   // @ts-expect-error - this is should not be defined on the object
   expect(object.polluted).toBe(undefined)
@@ -187,7 +191,6 @@ it('Test for prototype pollution', () => {
 
   expect(object2.constructor.prototype.polluted).toBe(false)
 
-
   // eslint-disable-next-line @typescript-eslint/no-extraneous-class
   const testClass = class TestClass {
     // eslint-disable-next-line @typescript-eslint/no-useless-constructor, @typescript-eslint/no-empty-function
@@ -203,9 +206,12 @@ it('Test for prototype pollution', () => {
   expect(getByPath(object3, 'constructor')).toBe(undefined)
 
   // @ts-expect-error - this is not a valid path for the object
-  expect(() => { setByPath(object3, 'constructor.polluted', true) }).toThrowError('constructor');
+  expect(() => {
+    setByPath(object3, 'constructor.polluted', true)
+  }).toThrowError('constructor')
 
   // @ts-expect-error - this is not a valid path for the object
-  expect(() => { setByPath(object3, '__proto__.polluted', true) }).toThrowError('__proto__');
-
+  expect(() => {
+    setByPath(object3, '__proto__.polluted', true)
+  }).toThrowError('__proto__')
 })