Merge branch 'undefined-default-value' of github.com:LeCarbonator/tanstack-form into undefined-default-value

Author: LeCarbonator

packages/form-core/src/utils.ts

@@ -31,21 +31,24 @@ export function functionalUpdate<TInput, TOutput = TInput>(
  * Get a value from an object using a path, including dot notation.
  * @private
  */
-export function getBy(obj: unknown, path: string | (string | number)[]): { found: boolean; value: any } {
+export function getBy(
+  obj: unknown,
+  path: string | (string | number)[],
+): { found: boolean; value: any } {
   const pathObj = makePathArray(path)
   let current: unknown = obj
   for (const pathPart of pathObj) {
     // path is trying to access props of undefined/null, so it doesn't exist
     if (typeof current === 'undefined' || current === null) {
       return { found: false, value: undefined }
     }
-    if (typeof current === 'object' && (pathPart in current)) {
+    if (typeof current === 'object' && pathPart in current) {
       current = current[pathPart as never]
     } else {
       return { found: false, value: undefined }
     }
   }
-  return { found: true, value: current };
+  return { found: true, value: current }
 }
 
 /**

packages/form-core/tests/utils.spec.ts

@@ -21,17 +21,19 @@ describe('getBy', () => {
     mother: {
       name: 'Lisa',
     },
-    siblings: [{
-      name: undefined
-    }]
+    siblings: [
+      {
+        name: undefined,
+      },
+    ],
   }
 
   it('should get subfields by path', () => {
-    const name = getBy(structure, 'name');
+    const name = getBy(structure, 'name')
     expect(name.value).toBe(structure.name)
     expect(name.found).toBe(true)
 
-    const motherName = getBy(structure, 'mother.name');
+    const motherName = getBy(structure, 'mother.name')
     expect(motherName.value).toBe(structure.mother.name)
     expect(motherName.found).toBe(true)
   })
@@ -47,22 +49,18 @@ describe('getBy', () => {
   })
 
   it('should get nested array subfields by path', () => {
-    const hobbies0 = getBy(structure, 'kids[0].hobbies[0]');
-    expect(hobbies0.value).toBe(
-      structure.kids[0]!.hobbies[0],
-    )
+    const hobbies0 = getBy(structure, 'kids[0].hobbies[0]')
+    expect(hobbies0.value).toBe(structure.kids[0]!.hobbies[0])
     expect(hobbies0.found).toBe(true)
 
-    const hobbies1 = getBy(structure, 'kids[0].hobbies[1]');
-    expect(hobbies1.value).toBe(
-      structure.kids[0]!.hobbies[1],
-    )
+    const hobbies1 = getBy(structure, 'kids[0].hobbies[1]')
+    expect(hobbies1.value).toBe(structure.kids[0]!.hobbies[1])
     expect(hobbies1.found).toBe(true)
   })
 
   it('should differentiate between explicit undefined vs. no path', () => {
-    const sibling0 = getBy(structure, 'siblings[0].name');
-    const sibling1 = getBy(structure, 'siblings[1].name');
+    const sibling0 = getBy(structure, 'siblings[0].name')
+    const sibling1 = getBy(structure, 'siblings[1].name')
 
     expect(sibling0.value).toBeUndefined()
     expect(sibling1.value).toBeUndefined()