refactor: stabilise filtering function

apps/client/src/common/hooks-query/useRundown.ts

@@ -65,6 +65,10 @@ export function useFlatRundownWithMetadata() {
 
 /**
  * Provides access to a partial rundown based on a filter callback
+ *
+ * Callers MUST memoize the callback with useCallback to prevent
+ * re-filtering on every render.
+ *
  */
 export function usePartialRundown(cb: (event: ExtendedEntry<OntimeEntry>) => boolean) {
   const { data, status } = useFlatRundownWithMetadata();

apps/client/src/views/common/schedule/ScheduleContext.tsx

@@ -1,4 +1,14 @@
-import { createContext, PropsWithChildren, RefObject, use, useEffect, useLayoutEffect, useRef, useState } from 'react';
+import {
+  createContext,
+  PropsWithChildren,
+  RefObject,
+  use,
+  useCallback,
+  useEffect,
+  useLayoutEffect,
+  useRef,
+  useState,
+} from 'react';
 import { EntryId, isOntimeEvent, OntimeEntry, OntimeEvent } from 'ontime-types';
 
 import { usePartialRundown } from '../../../common/hooks-query/useRundown';
@@ -22,14 +32,20 @@ interface ScheduleProviderProps {
 
 export const ScheduleProvider = ({ children, selectedEventId }: PropsWithChildren<ScheduleProviderProps>) => {
   const { cycleInterval, stopCycle, filter } = useScheduleOptions();
-  const { data: events } = usePartialRundown((entry: ExtendedEntry<OntimeEntry>) => {
+
-    if (filter) {
+  const filterCallback = useCallback(
-      // custom keys are prepended with custom-
+    (entry: ExtendedEntry<OntimeEntry>) => {
-      const customKey = filter.startsWith('custom-') ? filter.slice('custom-'.length) : filter;
+      if (filter) {
-      return isOntimeEvent(entry) && Boolean(entry.custom[customKey]);
+        // custom keys are prepended with custom-
-    }
+        const customKey = filter.startsWith('custom-') ? filter.slice('custom-'.length) : filter;
-    return isOntimeEvent(entry);
+        return isOntimeEvent(entry) && Boolean(entry.custom[customKey]);
-  });
+      }
+      return isOntimeEvent(entry);
+    },
+    [filter],
+  );
+
+  const { data: events } = usePartialRundown(filterCallback);
 
   const [firstIndex, setFirstIndex] = useState(-1);
   const [numPages, setNumPages] = useState(0);