fix(splitProps): support Symbol keys (#2814)

* fix(splitProps): support `Symbol` keys

* chore(splitPropsTests): simplify description

Author: Hugos68

.changeset/sunny-peaches-think.md

@@ -0,0 +1,5 @@
+---
+"@zag-js/utils": patch
+---
+
+fix: support `Symbol` keys in `splitProps`

packages/utilities/core/src/object.ts

@@ -28,15 +28,15 @@ export function pick<T extends Record<string, any>, K extends keyof T>(obj: T, k
   return filtered as any
 }
 
-type Dict = Record<string, any>
+type Dict = Record<string | symbol, any>
 
 export function splitProps<T extends Dict>(props: T, keys: (keyof T)[]) {
   const rest: Dict = {}
   const result: Dict = {}
 
   const keySet = new Set(keys)
 
-  for (const key in props) {
+  for (const key of Reflect.ownKeys(props)) {
     if (keySet.has(key)) {
       result[key] = props[key]
     } else {

packages/utilities/core/tests/object.test.ts

@@ -1,5 +1,5 @@
 import { describe, expect, test } from "vitest"
-import { compact } from "../src"
+import { compact, splitProps } from "../src"
 
 describe("compact()", () => {
   test("should compact object", () => {
@@ -45,3 +45,30 @@ describe("compact()", () => {
     expect(compact(obj)).toEqual(expected)
   })
 })
+
+describe("splitProps()", () => {
+  test("should split props", () => {
+    const props = { a: 1, b: 2, c: 3 }
+    const [result, rest] = splitProps(props, ["a", "c"])
+
+    expect(result).toEqual({ a: 1, c: 3 })
+    expect(rest).toEqual({ b: 2 })
+  })
+
+  test("should handle missing keys gracefully", () => {
+    const props = { a: 1 }
+    const [result, rest] = splitProps(props, ["b" as keyof typeof props])
+
+    expect(result).toEqual({})
+    expect(rest).toEqual({ a: 1 })
+  })
+
+  test("should perserve symbol keys", () => {
+    const symA = Symbol("a")
+    const symB = Symbol("b")
+    const props = { [symA]: 1, [symB]: 2, c: 3 }
+    const [result, rest] = splitProps(props, [symA, "c"])
+    expect(result).toEqual({ [symA]: 1, c: 3 })
+    expect(rest).toEqual({ [symB]: 2 })
+  })
+})