@jmruthers/pace-core 0.5.193 → 0.6.1

package/src/hooks/__tests__/hooks.integration.test.tsx

@@ -1,6 +1,7 @@
 import React, { useState, useEffect } from 'react';
 import { screen, fireEvent, waitFor, act } from '@testing-library/react';
 import { renderHook } from '@testing-library/react';
+import userEvent from '@testing-library/user-event';
 import '@testing-library/jest-dom';
 import { vi, describe, it, beforeEach, afterEach } from 'vitest';
 import { renderWithProviders } from '../../__tests__/helpers/test-utils';
@@ -16,7 +17,7 @@ import { Button } from '../../components/Button/Button';
 import { Input } from '../../components/Input/Input';
 import { Form } from '../../components/Form/Form';
 import { z } from 'zod';
-import { useForm } from 'react-hook-form';
+import { useForm, useFormState } from 'react-hook-form';
 
 // Mock the Logger module
 vi.mock('../../utils/core/logger', () => {
@@ -137,47 +138,57 @@ const ValidatedForm = ({ onSubmit }: ValidatedFormProps = {}) => {
       onSubmit={handleSubmit}
     >
       {(methods) => (
-        <>
-          <div>
-            <Input
-              {...methods.register('name')}
-              data-testid="name-input"
-              placeholder="Name"
-            />
-            {methods.formState.errors.name && (
-              <div data-testid="name-error">{methods.formState.errors.name.message}</div>
-            )}
-          </div>
-          <div>
-            <Input
-              {...methods.register('email')}
-              data-testid="email-input"
-              placeholder="Email"
-            />
-            {methods.formState.errors.email && (
-              <div data-testid="email-error">{methods.formState.errors.email.message}</div>
-            )}
-          </div>
-          <div>
-            <Input
-              {...methods.register('age', { valueAsNumber: true })}
-              data-testid="age-input"
-              type="number"
-              placeholder="Age"
-            />
-            {methods.formState.errors.age && (
-              <div data-testid="age-error">{methods.formState.errors.age.message}</div>
-            )}
-          </div>
-          <Button type="submit" data-testid="submit-button">
-            Submit
-          </Button>
-        </>
+        <ValidatedFormFields methods={methods} />
       )}
     </Form>
   );
 };
 
+// Separate component to use useFormState hook for reactive error updates
+const ValidatedFormFields = ({ methods }: { methods: ReturnType<typeof useForm> }) => {
+  // Use useFormState to subscribe to form state changes, including errors
+  const { errors } = useFormState({ control: methods.control });
+  
+  return (
+    <>
+      <div>
+        <Input
+          {...methods.register('name')}
+          data-testid="name-input"
+          placeholder="Name"
+        />
+        {errors.name && (
+          <div data-testid="name-error">{errors.name.message}</div>
+        )}
+      </div>
+      <div>
+        <Input
+          {...methods.register('email')}
+          data-testid="email-input"
+          placeholder="Email"
+        />
+        {errors.email && (
+          <div data-testid="email-error">{errors.email.message}</div>
+        )}
+      </div>
+      <div>
+        <Input
+          {...methods.register('age', { valueAsNumber: true })}
+          data-testid="age-input"
+          type="number"
+          placeholder="Age"
+        />
+        {errors.age && (
+          <div data-testid="age-error">{errors.age.message}</div>
+        )}
+      </div>
+      <Button type="submit" data-testid="submit-button">
+        Submit
+      </Button>
+    </>
+  );
+};
+
 // Test component for useKeyboardShortcuts integration
 const KeyboardShortcutsComponent = () => {
   const [isModalOpen, setIsModalOpen] = useState(false);
@@ -312,14 +323,13 @@ describe('Hooks Integration', () => {
     });
 
     it('handles rapid input changes correctly', async () => {
+      const user = userEvent.setup();
       renderWithProviders(<DebouncedSearch />);
 
       const searchInput = screen.getByTestId('search-input');
       
-      // Type rapidly
-      fireEvent.change(searchInput, { target: { value: 'a' } });
-      fireEvent.change(searchInput, { target: { value: 'ab' } });
-      fireEvent.change(searchInput, { target: { value: 'abc' } });
+      // Type rapidly using userEvent
+      await user.type(searchInput, 'abc');
 
       // Should not show results immediately
       expect(screen.queryByTestId('result-0')).not.toBeInTheDocument();
@@ -329,7 +339,10 @@ describe('Hooks Integration', () => {
         expect(screen.getByTestId('debounced-term')).toHaveTextContent('Debounced: abc');
       }, { timeout: 400 });
 
-      expect(screen.getByTestId('result-0')).toHaveTextContent('Result for: abc');
+      // Wait for results to appear after debounced term updates
+      await waitFor(() => {
+        expect(screen.getByTestId('result-0')).toHaveTextContent('Result for: abc');
+      }, { timeout: 500 });
     });
   });
 
@@ -362,6 +375,7 @@ describe('Hooks Integration', () => {
 
   describe('useZodForm Integration', () => {
     it('validates form with real-time feedback', async () => {
+      const user = userEvent.setup();
       renderWithProviders(<ValidatedForm />);
 
       const nameInput = screen.getByTestId('name-input');
@@ -369,21 +383,32 @@ describe('Hooks Integration', () => {
       const ageInput = screen.getByTestId('age-input');
       const submitButton = screen.getByTestId('submit-button');
 
-      // Test validation errors
-      await act(async () => {
-        fireEvent.change(nameInput, { target: { value: 'a' } });
-        fireEvent.change(emailInput, { target: { value: 'invalid-email' } });
-        fireEvent.change(ageInput, { target: { value: '15' } });
-      });
-      await act(async () => {
-        fireEvent.click(submitButton);
-      });
+      // Test validation errors - use userEvent for proper async handling
+      await user.clear(nameInput);
+      await user.type(nameInput, 'a');
+      await user.clear(emailInput);
+      await user.type(emailInput, 'invalid-email');
+      await user.clear(ageInput);
+      await user.type(ageInput, '15');
+      
+      // Submit the form to trigger validation
+      await user.click(submitButton);
 
+      // Wait for validation errors to appear after form submission
+      // The render prop function needs to re-render when formState.errors changes
+      // Wait for all errors to appear together
       await waitFor(() => {
-        expect(screen.getByTestId('name-error')).toHaveTextContent('Name must be at least 2 characters');
-        expect(screen.getByTestId('email-error')).toHaveTextContent('Invalid email address');
-        expect(screen.getByTestId('age-error')).toHaveTextContent('Must be at least 18 years old');
-      });
+        const nameError = screen.queryByTestId('name-error');
+        const emailError = screen.queryByTestId('email-error');
+        const ageError = screen.queryByTestId('age-error');
+        
+        expect(nameError).toBeInTheDocument();
+        expect(nameError).toHaveTextContent('Name must be at least 2 characters');
+        expect(emailError).toBeInTheDocument();
+        expect(emailError).toHaveTextContent('Invalid email address');
+        expect(ageError).toBeInTheDocument();
+        expect(ageError).toHaveTextContent('Must be at least 18 years old');
+      }, { timeout: 5000 });
     });
 
     it('submits form with valid data', async () => {

package/src/hooks/__tests__/useStorage.unit.test.ts

@@ -38,7 +38,7 @@ const mockSupabase = {
 const mockFile = new File(['test content'], 'test.txt', { type: 'text/plain' });
 
 describe('useStorage Hook', () => {
-  const defaultProps = {
+  const baseProps = {
     supabase: mockSupabase,
     appName: 'test-app',
     orgId: 'test-org',
@@ -46,7 +46,7 @@ describe('useStorage Hook', () => {
 
     describe('Initial State', () => {
     it('should initialize with correct default values', () => {
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       expect(result.current.isUploading).toBe(false);
       expect(result.current.uploadError).toBe(null);
@@ -58,7 +58,7 @@ describe('useStorage Hook', () => {
     });
 
     it('should provide all required functions', () => {
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       expect(typeof result.current.uploadFile).toBe('function');
       expect(typeof result.current.getPublicUrl).toBe('function');
@@ -106,7 +106,7 @@ describe('useStorage Hook', () => {
       vi.mocked(uploadFile).mockResolvedValue(mockUploadResult);
       vi.mocked(listFiles).mockResolvedValue(mockListResult);
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       let uploadResult: any;
       await act(async () => {
@@ -132,7 +132,7 @@ describe('useStorage Hook', () => {
 
       vi.mocked(uploadFile).mockResolvedValue(mockUploadResult);
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       let uploadResult: any;
       await act(async () => {
@@ -153,7 +153,7 @@ describe('useStorage Hook', () => {
       vi.mocked(uploadFile).mockResolvedValue(mockUploadResult);
       vi.mocked(listFiles).mockResolvedValue({ files: [], totalCount: 0, hasMore: false });
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       const customOptions = {
         isPublic: true,
@@ -179,7 +179,7 @@ describe('useStorage Hook', () => {
     it('should handle upload exception', async () => {
       vi.mocked(uploadFile).mockRejectedValue(new Error('Network error'));
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       let uploadResult: any;
       await act(async () => {
@@ -197,7 +197,7 @@ describe('useStorage Hook', () => {
     it('should handle upload exception with non-Error object', async () => {
       vi.mocked(uploadFile).mockRejectedValue('String error');
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       let uploadResult: any;
       await act(async () => {
@@ -217,7 +217,7 @@ describe('useStorage Hook', () => {
       const mockUrl = 'https://example.com/public/test-file.txt';
       vi.mocked(getPublicUrl).mockReturnValue(mockUrl);
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       const url = result.current.getPublicUrl('test-org/public/test-file.txt');
 
@@ -229,7 +229,7 @@ describe('useStorage Hook', () => {
       const mockUrl = 'https://example.com/public/test-file.txt';
       vi.mocked(getPublicUrl).mockReturnValue(mockUrl);
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       const url = result.current.getFileUrl('test-org/public/test-file.txt');
 
@@ -245,7 +245,7 @@ describe('useStorage Hook', () => {
 
       vi.mocked(getSignedUrl).mockResolvedValue(mockSignedUrl);
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       let signedUrl: string | null = null;
       await act(async () => {
@@ -263,7 +263,7 @@ describe('useStorage Hook', () => {
     it('should handle signed URL failure', async () => {
       vi.mocked(getSignedUrl).mockRejectedValue(new Error('Signing failed'));
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       let signedUrl: string | null = null;
       await act(async () => {
@@ -276,7 +276,7 @@ describe('useStorage Hook', () => {
     it('should handle signed URL with null result', async () => {
       vi.mocked(getSignedUrl).mockResolvedValue(null);
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       let signedUrl: string | null = null;
       await act(async () => {
@@ -295,7 +295,7 @@ describe('useStorage Hook', () => {
       vi.mocked(deleteFile).mockResolvedValue(mockDeleteResult);
       vi.mocked(listFiles).mockResolvedValue(mockListResult);
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       let deleteResult: any;
       await act(async () => {
@@ -311,7 +311,7 @@ describe('useStorage Hook', () => {
 
       vi.mocked(deleteFile).mockResolvedValue(mockDeleteResult);
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       let deleteResult: any;
       await act(async () => {
@@ -324,7 +324,7 @@ describe('useStorage Hook', () => {
     it('should handle delete file exception', async () => {
       vi.mocked(deleteFile).mockRejectedValue(new Error('Network error'));
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       let deleteResult: any;
       await act(async () => {
@@ -344,7 +344,7 @@ describe('useStorage Hook', () => {
       vi.mocked(archiveFile).mockResolvedValue(mockArchiveResult);
       vi.mocked(listFiles).mockResolvedValue(mockListResult);
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       let archiveResult: any;
       await act(async () => {
@@ -363,7 +363,7 @@ describe('useStorage Hook', () => {
 
       vi.mocked(archiveFile).mockResolvedValue(mockArchiveResult);
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       let archiveResult: any;
       await act(async () => {
@@ -376,7 +376,7 @@ describe('useStorage Hook', () => {
     it('should handle archive file exception', async () => {
       vi.mocked(archiveFile).mockRejectedValue(new Error('Network error'));
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       let archiveResult: any;
       await act(async () => {
@@ -417,7 +417,7 @@ describe('useStorage Hook', () => {
 
       vi.mocked(listFiles).mockResolvedValue(mockListResult);
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       let listResult: any;
       await act(async () => {
@@ -439,7 +439,7 @@ describe('useStorage Hook', () => {
 
       vi.mocked(listFiles).mockResolvedValue(mockListResult);
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       const customOptions = {
         pathPrefix: 'documents',
@@ -465,7 +465,7 @@ describe('useStorage Hook', () => {
     it('should handle list files failure', async () => {
       vi.mocked(listFiles).mockRejectedValue(new Error('List failed'));
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       let listResult: any;
       await act(async () => {
@@ -481,7 +481,7 @@ describe('useStorage Hook', () => {
     it('should handle list files exception with non-Error object', async () => {
       vi.mocked(listFiles).mockRejectedValue('String error');
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       let listResult: any;
       await act(async () => {
@@ -497,7 +497,7 @@ describe('useStorage Hook', () => {
 
       vi.mocked(listFiles).mockResolvedValue(mockListResult);
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       await act(async () => {
         await result.current.refreshFiles();
@@ -519,7 +519,7 @@ describe('useStorage Hook', () => {
 
       vi.mocked(uploadFile).mockReturnValue(uploadPromise);
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       expect(result.current.isUploading).toBe(false);
 
@@ -545,7 +545,7 @@ describe('useStorage Hook', () => {
 
       vi.mocked(listFiles).mockReturnValue(listPromise);
 
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       expect(result.current.isListing).toBe(false);
 
@@ -566,13 +566,13 @@ describe('useStorage Hook', () => {
 
   describe('Aliases', () => {
     it('should provide isLoading alias for isListing', () => {
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       expect(result.current.isLoading).toBe(result.current.isListing);
     });
 
     it('should provide error alias for listError', () => {
-      const { result } = renderHook(() => useStorage(defaultProps));
+      const { result } = renderHook(() => useStorage(baseProps));
 
       expect(result.current.error).toBe(result.current.listError);
     });
@@ -580,7 +580,7 @@ describe('useStorage Hook', () => {
 });
 
 describe('useFileUpload Hook', () => {
-  const defaultProps = {
+  const baseProps = {
     supabase: mockSupabase,
     appName: 'test-app',
     orgId: 'test-org',
@@ -588,7 +588,7 @@ describe('useFileUpload Hook', () => {
 
     describe('Initial State', () => {
     it('should initialize with correct default values', () => {
-      const { result } = renderHook(() => useFileUpload(defaultProps));
+      const { result } = renderHook(() => useFileUpload(baseProps));
 
       expect(result.current.uploadProgress).toBe(0);
       expect(result.current.isUploading).toBe(false);
@@ -605,7 +605,7 @@ describe('useFileUpload Hook', () => {
 
       vi.mocked(uploadFile).mockResolvedValue(mockUploadResult);
 
-      const { result } = renderHook(() => useFileUpload(defaultProps));
+      const { result } = renderHook(() => useFileUpload(baseProps));
 
       let uploadResult: any;
       await act(async () => {
@@ -628,7 +628,7 @@ describe('useFileUpload Hook', () => {
 
       vi.mocked(uploadFile).mockReturnValue(uploadPromise);
 
-      const { result } = renderHook(() => useFileUpload(defaultProps));
+      const { result } = renderHook(() => useFileUpload(baseProps));
 
       act(() => {
         result.current.uploadWithProgress(mockFile);
@@ -670,7 +670,7 @@ describe('useFileUpload Hook', () => {
 
       vi.mocked(uploadFile).mockResolvedValue(mockUploadResult);
 
-      const { result } = renderHook(() => useFileUpload(defaultProps));
+      const { result } = renderHook(() => useFileUpload(baseProps));
 
       let uploadResult: any;
       await act(async () => {
@@ -686,7 +686,7 @@ describe('useFileUpload Hook', () => {
     it('should handle upload exception', async () => {
       vi.mocked(uploadFile).mockRejectedValue(new Error('Network error'));
 
-      const { result } = renderHook(() => useFileUpload(defaultProps));
+      const { result } = renderHook(() => useFileUpload(baseProps));
 
       let uploadResult: any;
       await act(async () => {
@@ -707,7 +707,7 @@ describe('useFileUpload Hook', () => {
 
       vi.mocked(uploadFile).mockResolvedValue(mockUploadResult);
 
-      const { result } = renderHook(() => useFileUpload(defaultProps));
+      const { result } = renderHook(() => useFileUpload(baseProps));
 
       await act(async () => {
         await result.current.uploadWithProgress(mockFile);
@@ -729,7 +729,7 @@ describe('useFileUpload Hook', () => {
 
       vi.mocked(uploadFile).mockResolvedValue(mockUploadResult);
 
-      const { result } = renderHook(() => useFileUpload(defaultProps));
+      const { result } = renderHook(() => useFileUpload(baseProps));
 
       const customOptions = {
         isPublic: true,

package/src/hooks/public/usePublicEvent.ts

@@ -54,7 +54,7 @@
  * - Error boundary integration
  *
  * @dependencies
- * - React 18+ - Hooks and effects
+ * - React 19+ - Hooks and effects
  * - @supabase/supabase-js - Database integration
  * - Event types - Type definitions
  */

package/src/hooks/public/usePublicEventLogo.ts

@@ -59,7 +59,7 @@
  * - Lazy loading support
  *
  * @dependencies
- * - React 18+ - Hooks and effects
+ * - React 19+ - Hooks and effects
  * - @supabase/supabase-js - Storage integration
  * - Event types - Type definitions
  */

package/src/hooks/public/usePublicRouteParams.ts

@@ -51,7 +51,7 @@
  * - Caching integration
  *
  * @dependencies
- * - React 18+ - Hooks and effects
+ * - React 19+ - Hooks and effects
  * - React Router - URL parameter extraction
  * - Event types - Type definitions
  */

package/src/hooks/useDataTableState.ts

@@ -3,7 +3,7 @@
  * @description Hook for managing DataTable state
  */
 
-import { useState, useCallback, useMemo } from 'react';
+import { useState, useCallback } from 'react';
 import type { SortingState, ColumnFiltersState, ExpandedState } from '@tanstack/react-table';
 
 export interface DataTableState {
@@ -63,24 +63,14 @@ export function useDataTableState(options: UseDataTableStateOptions) {
     setSelectedRows([]);
   }, [initialPageSize]);
 
-  // Computed values
-  const paginatedData = useMemo(() => {
-    const start = pageIndex * pageSize;
-    const end = start + pageSize;
-    return data.slice(start, end);
-  }, [data, pageIndex, pageSize]);
+  // Computed values - React Compiler handles memoization automatically
+  const start = pageIndex * pageSize;
+  const end = start + pageSize;
+  const paginatedData = data.slice(start, end);
 
-  const totalPages = useMemo(() => {
-    return Math.ceil(data.length / pageSize);
-  }, [data.length, pageSize]);
-
-  const hasNextPage = useMemo(() => {
-    return pageIndex < totalPages - 1;
-  }, [pageIndex, totalPages]);
-
-  const hasPreviousPage = useMemo(() => {
-    return pageIndex > 0;
-  }, [pageIndex]);
+  const totalPages = Math.ceil(data.length / pageSize);
+  const hasNextPage = pageIndex < totalPages - 1;
+  const hasPreviousPage = pageIndex > 0;
 
   return {
     state: {

package/src/hooks/useFocusManagement.ts

@@ -11,8 +11,8 @@ export interface FocusManagementOptions {
 }
 
 export interface FocusManagementReturn {
-  containerRef: React.RefObject<HTMLDivElement>;
-  focusRef: React.RefObject<HTMLElement>;
+  containerRef: React.RefObject<HTMLDivElement | null>;
+  focusRef: React.RefObject<HTMLElement | null>;
   setFocus: (element: HTMLElement | null) => void;
   focusFirst: () => void;
   focusLast: () => void;

package/src/hooks/useFocusTrap.ts

@@ -16,7 +16,7 @@ export interface FocusTrapOptions {
 
 export interface FocusTrapReturn {
   /** Ref to attach to the container element */
-  containerRef: React.RefObject<HTMLElement>;
+  containerRef: React.RefObject<HTMLElement | null>;
   /** Focus the first focusable element */
   focusFirst: () => void;
   /** Focus the last focusable element */
@@ -58,9 +58,9 @@ export function useFocusTrap(options: FocusTrapOptions = {}): FocusTrapReturn {
       containerRef.current.querySelectorAll<HTMLElement>(focusableSelector)
     ).filter((element) => {
       return (
-        !element.hasAttribute('disabled') &&
-        !element.hasAttribute('hidden') &&
-        element.offsetParent !== null // visible check
+        // visible check
+        (!element.hasAttribute('disabled') &&
+        !element.hasAttribute('hidden') && element.offsetParent !== null)
       );
     });
   }, [focusableSelector]);

package/src/hooks/useFormDialog.ts

@@ -23,7 +23,7 @@
  * ```
  */
 
-import { useState, useCallback } from 'react';
+import { useState } from 'react';
 
 /**
  * Options for configuring the useFormDialog hook behavior
@@ -115,27 +115,28 @@ export function useFormDialog<T = unknown>({
   const [isOpen, setIsOpen] = useState(false);
   const [formData, setFormData] = useState<T | null>(null);
 
-  const openDialog = useCallback((data: T | null = null) => {
+  // React Compiler handles memoization automatically
+  const openDialog = (data: T | null = null) => {
     setFormData(data);
     setIsOpen(true);
     onOpenChange?.(true);
-  }, [onOpenChange]);
+  };
 
-  const closeDialog = useCallback(() => {
+  const closeDialog = () => {
     setIsOpen(false);
     if (resetOnClose) {
       setFormData(null);
     }
     onOpenChange?.(false);
-  }, [resetOnClose, onOpenChange]);
+  };
 
-  const handleOpenChange = useCallback((open: boolean) => {
+  const handleOpenChange = (open: boolean) => {
     setIsOpen(open);
     if (!open && resetOnClose) {
       setFormData(null);
     }
     onOpenChange?.(open);
-  }, [resetOnClose, onOpenChange]);
+  };
 
   return {
     isOpen,

package/src/hooks/useInactivityTracker.ts

@@ -48,7 +48,7 @@
  * - Cross-tab optimization
  *
  * @dependencies
- * - React 18+ - Hooks and effects
+ * - React 19+ - Hooks and effects
  * - Browser APIs - BroadcastChannel, localStorage, timers
  */
 

package/src/hooks/usePermissionCache.ts

@@ -46,7 +46,7 @@
  * - Memory-efficient storage
  *
  * @dependencies
- * - React 18+ - Hooks and effects
+ * - React 19+ - Hooks and effects
  * - RBAC API functions - isPermittedCached for permission checking
  * - RBAC types - Type definitions
  *