@jmruthers/pace-core 0.5.193 → 0.6.2

package/src/components/PasswordChange/PasswordChangeForm.test.tsx

@@ -55,13 +55,13 @@ vi.mock('../Label', () => ({
 
 describe('PasswordChangeForm', () => {
   const mockOnSubmit = vi.fn();
-  const defaultProps: PasswordChangeFormProps = {
+  const baseProps: PasswordChangeFormProps = {
     onSubmit: mockOnSubmit
   };
 
     describe('Rendering', () => {
     it('renders password change form with all elements', () => {
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       expect(screen.getByLabelText('New Password')).toBeInTheDocument();
       expect(screen.getByLabelText('Confirm Password')).toBeInTheDocument();
@@ -70,14 +70,14 @@ describe('PasswordChangeForm', () => {
 
     it('renders with custom className', () => {
       const customClass = 'custom-password-change-form';
-      const { container } = render(<PasswordChangeForm {...defaultProps} className={customClass} />);
+      const { container } = render(<PasswordChangeForm {...baseProps} className={customClass} />);
 
       const form = container.querySelector('form');
       expect(form).toHaveClass(customClass);
     });
 
     it('has proper form structure and accessibility', () => {
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       expect(newPasswordInput).toHaveAttribute('type', 'password');
@@ -97,7 +97,7 @@ describe('PasswordChangeForm', () => {
   describe('Form Interaction', () => {
     it('updates new password input value when user types', async () => {
       const user = userEvent.setup();
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       await user.type(newPasswordInput, 'newpassword123');
@@ -107,7 +107,7 @@ describe('PasswordChangeForm', () => {
 
     it('updates confirm password input value when user types', async () => {
       const user = userEvent.setup();
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
       await user.type(confirmPasswordInput, 'newpassword123');
@@ -117,7 +117,7 @@ describe('PasswordChangeForm', () => {
 
     it('enables submit button when both passwords are provided', async () => {
       const user = userEvent.setup();
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -133,7 +133,7 @@ describe('PasswordChangeForm', () => {
 
     it('disables submit button when new password is empty', async () => {
       const user = userEvent.setup();
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
       const submitButton = screen.getByRole('button', { name: 'Change Password' });
@@ -145,7 +145,7 @@ describe('PasswordChangeForm', () => {
 
     it('disables submit button when confirm password is empty', async () => {
       const user = userEvent.setup();
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const submitButton = screen.getByRole('button', { name: 'Change Password' });
@@ -159,7 +159,7 @@ describe('PasswordChangeForm', () => {
   describe('Form Validation', () => {
     it('shows error when password is too short', async () => {
       const user = userEvent.setup();
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -175,7 +175,7 @@ describe('PasswordChangeForm', () => {
 
     it('shows error when passwords do not match', async () => {
       const user = userEvent.setup();
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -191,7 +191,7 @@ describe('PasswordChangeForm', () => {
 
     it('validates password length before checking match', async () => {
       const user = userEvent.setup();
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -207,7 +207,7 @@ describe('PasswordChangeForm', () => {
 
     it('clears error when form is resubmitted with valid data', async () => {
       const user = userEvent.setup();
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -236,7 +236,7 @@ describe('PasswordChangeForm', () => {
       const user = userEvent.setup();
       mockOnSubmit.mockResolvedValue({});
 
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -254,7 +254,7 @@ describe('PasswordChangeForm', () => {
 
     it('prevents form submission when validation fails', async () => {
       const user = userEvent.setup();
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -273,7 +273,7 @@ describe('PasswordChangeForm', () => {
       const user = userEvent.setup();
       mockOnSubmit.mockImplementation(() => new Promise(resolve => setTimeout(resolve, 100)));
 
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -291,7 +291,7 @@ describe('PasswordChangeForm', () => {
       const user = userEvent.setup();
       mockOnSubmit.mockImplementation(() => new Promise(resolve => setTimeout(resolve, 100)));
 
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -309,7 +309,7 @@ describe('PasswordChangeForm', () => {
       const user = userEvent.setup();
       mockOnSubmit.mockResolvedValue({});
 
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -329,7 +329,7 @@ describe('PasswordChangeForm', () => {
       const user = userEvent.setup();
       mockOnSubmit.mockResolvedValue({ error: { message: 'Test error' } });
 
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -352,7 +352,7 @@ describe('PasswordChangeForm', () => {
       const errorMessage = 'Password change failed';
       mockOnSubmit.mockResolvedValue({ error: { message: errorMessage } });
 
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -373,7 +373,7 @@ describe('PasswordChangeForm', () => {
       const errorMessage = 'Network error';
       mockOnSubmit.mockRejectedValue(new Error(errorMessage));
 
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -393,7 +393,7 @@ describe('PasswordChangeForm', () => {
       const user = userEvent.setup();
       mockOnSubmit.mockRejectedValue('String error');
 
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -412,7 +412,7 @@ describe('PasswordChangeForm', () => {
       const user = userEvent.setup();
       mockOnSubmit.mockResolvedValue({ error: { code: 'INVALID_PASSWORD' } });
 
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -433,7 +433,7 @@ describe('PasswordChangeForm', () => {
         .mockResolvedValueOnce({ error: { message: 'First error' } })
         .mockResolvedValueOnce({});
 
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -457,7 +457,7 @@ describe('PasswordChangeForm', () => {
 
   describe('Accessibility', () => {
     it('has proper form labels and associations', () => {
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       expect(newPasswordInput).toHaveAttribute('id', 'new-password');
@@ -472,7 +472,7 @@ describe('PasswordChangeForm', () => {
       const user = userEvent.setup();
       mockOnSubmit.mockResolvedValue({ error: { message: 'Test error' } });
 
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -492,7 +492,7 @@ describe('PasswordChangeForm', () => {
       const user = userEvent.setup();
       mockOnSubmit.mockResolvedValue({});
 
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -510,7 +510,7 @@ describe('PasswordChangeForm', () => {
 
   describe('Edge Cases', () => {
     it('handles empty password values', () => {
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const submitButton = screen.getByRole('button', { name: 'Change Password' });
 
@@ -519,7 +519,7 @@ describe('PasswordChangeForm', () => {
 
     it('handles whitespace-only passwords', async () => {
       const user = userEvent.setup();
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -538,7 +538,7 @@ describe('PasswordChangeForm', () => {
       const longPassword = 'a'.repeat(1000);
       mockOnSubmit.mockResolvedValue({});
 
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -559,7 +559,7 @@ describe('PasswordChangeForm', () => {
       const specialPassword = 'P@ssw0rd!@#$%^&*()_+-=';
       mockOnSubmit.mockResolvedValue({});
 
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -580,7 +580,7 @@ describe('PasswordChangeForm', () => {
       const unicodePassword = 'pássw0rd_测试_пароль';
       mockOnSubmit.mockResolvedValue({});
 
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       const confirmPasswordInput = screen.getByLabelText('Confirm Password');
@@ -600,7 +600,7 @@ describe('PasswordChangeForm', () => {
   describe('Performance', () => {
     it('handles rapid input changes efficiently', async () => {
       const user = userEvent.setup();
-      render(<PasswordChangeForm {...defaultProps} />);
+      render(<PasswordChangeForm {...baseProps} />);
 
       const newPasswordInput = screen.getByLabelText('New Password');
       

package/src/components/PasswordChange/PasswordChangeForm.tsx

@@ -90,7 +90,7 @@
  * - Proper error handling
  *
  * @dependencies
- * - React 18+ - Hooks and state
+ * - React 19+ - Hooks and state
  * - Button component
  * - Input component
  * - Label component
@@ -103,16 +103,25 @@ import { Input } from '../Input/Input';
 import { Label } from '../Label';
 import { cn } from '../../utils/core/cn';
 
+/**
+ * Form values for password change.
+ */
 export interface PasswordChangeFormValues {
   newPassword: string;
   confirmPassword: string;
 }
 
+/**
+ * Error structure for password change form.
+ */
 export interface PasswordChangeFormError {
   message?: string;
   code?: string;
 }
 
+/**
+ * Props for the PasswordChangeForm component.
+ */
 export interface PasswordChangeFormProps {
   onSubmit: (values: PasswordChangeFormValues) => Promise<{ error?: PasswordChangeFormError }>;
   className?: string;

package/src/components/Progress/Progress.tsx

@@ -57,7 +57,7 @@
  * - Minimal DOM structure (native HTML element)
  *
  * @dependencies
- * - React 18+ - Hooks and refs
+ * - React 19+ - Hooks and refs
  * - Tailwind CSS - Styling
  */
 

package/src/components/ProtectedRoute/ProtectedRoute.tsx

@@ -79,6 +79,9 @@ import { Alert, AlertDescription, AlertTitle } from '../Alert/Alert';
 import { logger } from '../../utils/core/logger';
 import { usePreventTabReload } from '../../hooks/usePreventTabReload';
 
+/**
+ * Props for the ProtectedRoute component.
+ */
 export interface ProtectedRouteProps {
   /**
    * Whether an event is required for routes inside this component.
@@ -88,14 +91,6 @@ export interface ProtectedRouteProps {
    */
   requireEvent?: boolean;
 
-  /**
-   * Whether super admins can bypass event requirement.
-   * Note: This feature requires additional RBAC setup. For simple bypass, set requireEvent={false} instead.
-   * @default false
-   * @deprecated Use requireEvent={false} for routes that don't need events
-   */
-  allowSuperAdminBypass?: boolean;
-
   /**
    * Custom component to render when no events are available.
    * If not provided, a default message is shown.
@@ -133,7 +128,6 @@ export interface ProtectedRouteProps {
  */
 export function ProtectedRoute({
   requireEvent = false,
-  allowSuperAdminBypass = false,
   noEventsFallback,
   loadingFallback,
   loginPath = '/login'

package/src/components/PublicLayout/PublicPageLayout.tsx

@@ -35,9 +35,9 @@
  *       refetch={refetch}
  *     >
  *       <h1>Event Details</h1>
- *       <div className="content">
+ *       <main className="content">
  *         Your public page content
- *       </div>
+ *       </main>
  *     </PublicPageLayout>
  *   );
  * }
@@ -58,7 +58,7 @@
  * - Error handling for invalid routes
  *
  * @dependencies
- * - React 18+ - Component framework
+ * - React 19+ - Component framework
  * - React Router - Routing integration
  * - Public hooks - Data access
  * - Tailwind CSS - Styling
@@ -90,8 +90,6 @@ export interface PublicPageLayoutProps {
   refetch?: () => Promise<void> | void;
   /** Whether to show the footer (default: true) */
   showFooter?: boolean;
-  /** @deprecated Custom CSS classes for the layout - no longer used as wrapper div was removed */
-  className?: string;
   /** Custom error fallback component */
   errorFallback?: React.ComponentType<{ error: Error; retry: () => void }>;
   /** Custom loading fallback component */
@@ -294,7 +292,6 @@ export function PublicPageLayout({
   error = null,
   refetch,
   showFooter = true,
-  className = '',
   errorFallback: ErrorFallback,
   loadingFallback: LoadingFallback,
   customHeader,

package/src/components/PublicLayout/PublicPageProvider.tsx

@@ -50,6 +50,10 @@ interface PublicPageContextType {
   };
 }
 
+/**
+ * Context for public pages.
+ * Provides isolated context for public pages without authentication requirements.
+ */
 export const PublicPageContext = createContext<PublicPageContextType | undefined>(undefined);
 
 export interface PublicPageProviderProps {