fix: useCollection useless re-renders (+ missing method in type) (#214)

packages/formiz-core/src/store.ts

@@ -703,7 +703,7 @@ export const createStore = <Values extends object = DefaultFormValues>(
           get().collections.set(
             fieldName.toString(),
             typeof keys === "function"
-              ? keys(get().actions.getCollectionKeys(fieldName) ?? [])
+              ? keys(get().collections.get(fieldName) ?? [])
               : keys
           );
           return {
@@ -712,10 +712,6 @@ export const createStore = <Values extends object = DefaultFormValues>(
         });
       },
 
-      getCollectionKeys: (fieldName) => {
-        return get().collections.get(fieldName.toString());
-      },
-
       setCollectionValues: (fieldName) => (values, options) => {
         set((state) => {
           get().actions.setValues(

packages/formiz-core/src/types.ts

@@ -392,7 +392,6 @@ export interface Store<Values extends object = DefaultFormValues> {
     ): (
       keys: CollectionKey[] | ((oldKeys: CollectionKey[]) => CollectionKey[])
     ) => void;
-    getCollectionKeys(fieldName: string): CollectionKey[] | undefined;
     setCollectionValues(
       fieldName: string
     ): (

packages/formiz-core/src/useCollection.ts

@@ -1,7 +1,7 @@
 import { useCallback, useEffect, useRef, useMemo } from "react";
 import lodashGet from "lodash/get";
 import { StoreApi, UseBoundStore } from "zustand";
-import { Store } from "./types";
+import { CollectionKey, Store } from "./types";
 import { useFormStore } from "./Formiz";
 import { deepEqual } from "fast-equals";
 
@@ -13,7 +13,7 @@ export interface UseCollectionOptions {
 }
 
 export type UseCollectionValues<Data = unknown> = {
-  keys: string[];
+  keys: CollectionKey[];
   insertMultiple(
     index: number,
     data?: Partial<Data>[],
@@ -38,6 +38,9 @@ export type UseCollectionValues<Data = unknown> = {
     values: unknown[],
     options?: Parameters<Store["actions"]["setValues"]>[1]
   ): void;
+  setKeys(
+    keys: CollectionKey[] | ((oldKeys: CollectionKey[]) => CollectionKey[])
+  ): void;
   length: number;
 };
 
@@ -87,14 +90,14 @@ export const useCollection = <Data = unknown>(
     return {
       isReady: state.ready,
       keys: state.ready
-        ? state.actions.getCollectionKeys(name) ??
+        ? state.collections.get(name) ??
           initialValuesArray.map((_, index) => index.toString())
         : [],
       hasInitialValues: Array.isArray(initialValues)
         ? !!initialValues?.length
         : false,
     };
-  });
+  }, deepEqual);
 
   const collectionActions = useMemo(
     () => ({