[0.5.x] Improve forgetFiles to handle nested structures and edge cases (#119)

* Improve forgetFiles to handle nested structures and edge cases

- Added recursive support to remove files from deeply nested arrays and objects.
- Improved compatibility with non-iterable objects using Object.getOwnPropertyNames.

* fix: remove files from nested arrays and objects without leaving empty values

* test: add coverage for deeply nested files (files in objects within arrays)

* formatting

---------

Co-authored-by: rottifant <[email protected]>

Author: timacdonald

packages/core/src/validator.ts

@@ -453,7 +453,7 @@ const forgetFiles = (data: Record<string, unknown>): Record<string, unknown> =>
         }
 
         if (Array.isArray(value)) {
-            newData[name] = value.filter((value) => !isFile(value))
+            newData[name] = Object.values(forgetFiles({ ...value }))
 
             return
         }

packages/core/tests/validator.test.js

@@ -260,6 +260,12 @@ it('filters out files', async () => {
             'apple',
             'banana',
             new Blob([], { type: 'image/png' }),
+            ['nested', new Blob([], { type: 'image/png' })],
+            {
+                name: 'Tim',
+                email: null,
+                avatar: new Blob([], { type: 'image/png' }),
+            },
         ],
         avatar: new Blob([], { type: 'image/png' }),
         nested: {
@@ -292,6 +298,11 @@ it('filters out files', async () => {
         fruits: [
             'apple',
             'banana',
+            ['nested'],
+            {
+                name: 'Tim',
+                email: null,
+            },
         ],
         nested: {
             name: 'Tim',