From cca2ac28e069cef5de9ece6c64f1801fa357bd10 Mon Sep 17 00:00:00 2001
From: filtered <176114999+webfiltered@users.noreply.github.com>
Date: Thu, 13 Mar 2025 10:12:04 +1100
Subject: [PATCH] [Refactor] Prefer for loop over reduce (#764)

- Follow-up on #763
---
 src/utils/object.ts | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/src/utils/object.ts b/src/utils/object.ts
index 15ab92d08..93d421fb0 100644
--- a/src/utils/object.ts
+++ b/src/utils/object.ts
@@ -12,11 +12,12 @@ export function omitBy<T extends object>(obj: T, predicate: (value: any) => bool
  * @returns A new object with just the picked properties
  */
 export function pick<T extends object, K extends keyof T>(obj: T, keys: K[]): Pick<T, K> {
-  // eslint-disable-next-line unicorn/no-array-reduce
-  return keys.reduce((result, key) => {
+  // Assertion: Required to avoid verbose runtime guards
+  const picked = {} as Pick<T, K>
+  for (const key of keys) {
     if (key in obj) {
-      result[key] = obj[key]
+      picked[key] = obj[key]
     }
-    return result
-  }, {} as Pick<T, K>)
+  }
+  return picked
 }