Refactoring

src/config/normalize/lib/wild_wild_path/remove.js

@@ -3,34 +3,11 @@ import omit from 'omit.js'
 import { setArray } from '../../../../utils/set.js'
 
 import { handleMissingValue } from './iterate/expand.js'
-import { iterate } from './iterate/main.js'
-import { parent } from './parsing/compare.js'
-import { setValue } from './set.js'
+import { reduceParents, setValue } from './set.js'
 
 // Same as `set()` but removing a value
 export const remove = function (target, queryOrPath) {
-  // eslint-disable-next-line fp/no-let
-  let newTarget = target
-
-  const paths = []
-
-  // eslint-disable-next-line fp/no-loops
-  for (const { path } of iterate(target, queryOrPath)) {
-    // eslint-disable-next-line max-depth
-    if (!parentIsRemoved(paths, path)) {
-      // eslint-disable-next-line fp/no-mutating-methods
-      paths.push(path)
-      // eslint-disable-next-line fp/no-mutation
-      newTarget = removeAnyEntry(newTarget, path, 0)
-    }
-  }
-
-  return newTarget
-}
-
-// If both a parent and a child property are removed, the parent prevails
-const parentIsRemoved = function (paths, path) {
-  return paths.some((previousPath) => parent(previousPath, path))
+  return reduceParents(target, queryOrPath, removeAnyEntry)
 }
 
 const removeAnyEntry = function (target, path, index) {