@jmruthers/pace-core 0.5.126 → 0.5.127

package/docs/api/interfaces/StyleImport.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / StyleImport
+[@jmruthers/pace-core - v0.5.127](../README.md) / [Exports](../modules.md) / StyleImport
 
 # Interface: StyleImport
 

package/docs/api/interfaces/SwitchProps.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / SwitchProps
+[@jmruthers/pace-core - v0.5.127](../README.md) / [Exports](../modules.md) / SwitchProps
 
 # Interface: SwitchProps
 

package/docs/api/interfaces/ToastActionElement.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / ToastActionElement
+[@jmruthers/pace-core - v0.5.127](../README.md) / [Exports](../modules.md) / ToastActionElement
 
 # Interface: ToastActionElement
 

package/docs/api/interfaces/ToastProps.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / ToastProps
+[@jmruthers/pace-core - v0.5.127](../README.md) / [Exports](../modules.md) / ToastProps
 
 # Interface: ToastProps
 

package/docs/api/interfaces/UnifiedAuthContextType.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UnifiedAuthContextType
+[@jmruthers/pace-core - v0.5.127](../README.md) / [Exports](../modules.md) / UnifiedAuthContextType
 
 # Interface: UnifiedAuthContextType
 

package/docs/api/interfaces/UnifiedAuthProviderProps.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UnifiedAuthProviderProps
+[@jmruthers/pace-core - v0.5.127](../README.md) / [Exports](../modules.md) / UnifiedAuthProviderProps
 
 # Interface: UnifiedAuthProviderProps
 

package/docs/api/interfaces/UseInactivityTrackerOptions.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UseInactivityTrackerOptions
+[@jmruthers/pace-core - v0.5.127](../README.md) / [Exports](../modules.md) / UseInactivityTrackerOptions
 
 # Interface: UseInactivityTrackerOptions
 

package/docs/api/interfaces/UseInactivityTrackerReturn.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UseInactivityTrackerReturn
+[@jmruthers/pace-core - v0.5.127](../README.md) / [Exports](../modules.md) / UseInactivityTrackerReturn
 
 # Interface: UseInactivityTrackerReturn
 

package/docs/api/interfaces/UsePublicEventOptions.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UsePublicEventOptions
+[@jmruthers/pace-core - v0.5.127](../README.md) / [Exports](../modules.md) / UsePublicEventOptions
 
 # Interface: UsePublicEventOptions
 

package/docs/api/interfaces/UsePublicEventReturn.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UsePublicEventReturn
+[@jmruthers/pace-core - v0.5.127](../README.md) / [Exports](../modules.md) / UsePublicEventReturn
 
 # Interface: UsePublicEventReturn
 

package/docs/api/interfaces/UsePublicFileDisplayOptions.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UsePublicFileDisplayOptions
+[@jmruthers/pace-core - v0.5.127](../README.md) / [Exports](../modules.md) / UsePublicFileDisplayOptions
 
 # Interface: UsePublicFileDisplayOptions
 

package/docs/api/interfaces/UsePublicFileDisplayReturn.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UsePublicFileDisplayReturn
+[@jmruthers/pace-core - v0.5.127](../README.md) / [Exports](../modules.md) / UsePublicFileDisplayReturn
 
 # Interface: UsePublicFileDisplayReturn
 

package/docs/api/interfaces/UsePublicRouteParamsReturn.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UsePublicRouteParamsReturn
+[@jmruthers/pace-core - v0.5.127](../README.md) / [Exports](../modules.md) / UsePublicRouteParamsReturn
 
 # Interface: UsePublicRouteParamsReturn
 

package/docs/api/interfaces/UseResolvedScopeOptions.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UseResolvedScopeOptions
+[@jmruthers/pace-core - v0.5.127](../README.md) / [Exports](../modules.md) / UseResolvedScopeOptions
 
 # Interface: UseResolvedScopeOptions
 

package/docs/api/interfaces/UseResolvedScopeReturn.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UseResolvedScopeReturn
+[@jmruthers/pace-core - v0.5.127](../README.md) / [Exports](../modules.md) / UseResolvedScopeReturn
 
 # Interface: UseResolvedScopeReturn
 

package/docs/api/interfaces/UserEventAccess.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UserEventAccess
+[@jmruthers/pace-core - v0.5.127](../README.md) / [Exports](../modules.md) / UserEventAccess
 
 # Interface: UserEventAccess
 

package/docs/api/interfaces/UserMenuProps.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UserMenuProps
+[@jmruthers/pace-core - v0.5.127](../README.md) / [Exports](../modules.md) / UserMenuProps
 
 # Interface: UserMenuProps
 

package/docs/api/interfaces/UserProfile.md

@@ -1,4 +1,4 @@
-[@jmruthers/pace-core - v0.5.126](../README.md) / [Exports](../modules.md) / UserProfile
+[@jmruthers/pace-core - v0.5.127](../README.md) / [Exports](../modules.md) / UserProfile
 
 # Interface: UserProfile
 

package/docs/api/modules.md

@@ -1,6 +1,6 @@
-[@jmruthers/pace-core - v0.5.126](README.md) / Exports
+[@jmruthers/pace-core - v0.5.127](README.md) / Exports
 
-# @jmruthers/pace-core - v0.5.126
+# @jmruthers/pace-core - v0.5.127
 
 **`File`**
 
@@ -1558,7 +1558,7 @@ JSX.Element - The dialog footer container using semantic <footer> element
 
 #### Defined in
 
-[packages/core/src/components/Dialog/Dialog.tsx:690](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Dialog/Dialog.tsx#L690)
+[packages/core/src/components/Dialog/Dialog.tsx:677](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Dialog/Dialog.tsx#L677)
 
 ___
 
@@ -1578,7 +1578,7 @@ ___
 
 #### Defined in
 
-[packages/core/src/components/Dialog/Dialog.tsx:707](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Dialog/Dialog.tsx#L707)
+[packages/core/src/components/Dialog/Dialog.tsx:694](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Dialog/Dialog.tsx#L694)
 
 ___
 
@@ -1598,7 +1598,7 @@ ___
 
 #### Defined in
 
-[packages/core/src/components/Dialog/Dialog.tsx:742](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Dialog/Dialog.tsx#L742)
+[packages/core/src/components/Dialog/Dialog.tsx:729](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Dialog/Dialog.tsx#L729)
 
 ___
 
@@ -1668,7 +1668,7 @@ React element with file display
 
 #### Defined in
 
-[packages/core/src/components/FileDisplay/FileDisplay.tsx:727](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/FileDisplay/FileDisplay.tsx#L727)
+[packages/core/src/components/FileDisplay/FileDisplay.tsx:803](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/FileDisplay/FileDisplay.tsx#L803)
 
 ___
 
@@ -1767,6 +1767,11 @@ and customizable branding support.
 A flexible header component that supports various configurations including custom logos,
 navigation menus, user authentication, event selection, and custom actions.
 
+**Logo Display:** When used via PaceAppLayout, the logo URL is automatically constructed
+from the appName prop as `/${appName.toLowerCase()}_logo_wide.svg`. The appName should
+come from an APP_NAME constant declared in your App.tsx file to ensure consistency across
+authenticated and public pages.
+
 Features:
 - Customizable logo (URL or custom component)
 - Clickable logo that automatically routes to dashboard (configurable via logoHref)
@@ -1872,7 +1877,7 @@ function MinimalHeader() {
 
 #### Defined in
 
-[packages/core/src/components/Header/Header.tsx:229](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Header/Header.tsx#L229)
+[packages/core/src/components/Header/Header.tsx:234](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Header/Header.tsx#L234)
 
 ___
 
@@ -2304,6 +2309,11 @@ This component is designed to work with React Router's nested routing pattern us
 Outlet to render child routes. It provides integrated authentication, navigation,
 and user management functionality.
 
+**Important:** The appName prop should use an APP_NAME constant declared in your App.tsx
+file. This ensures consistency with public pages (via PublicPageProvider) which should
+also receive the same APP_NAME constant. The logo URL is automatically constructed as
+`/${appName.toLowerCase()}_logo_wide.svg` from the public folder.
+
 Features:
 - React Router v6 integration with nested routing
 - Unified authentication integration
@@ -2334,17 +2344,20 @@ Basic React Router setup with permission enforcement (RECOMMENDED):
 ```tsx
 import { BrowserRouter as Router, Routes, Route } from 'react-router-dom';
 import { UnifiedAuthProvider } from '@jmruthers/pace-core/providers';
-import { PaceAppLayout, PaceLoginPage } from '@jmruthers/pace-core';
+import { PaceAppLayout, PaceLoginPage, PublicPageApp } from '@jmruthers/pace-core';
+
+const APP_NAME = 'CORE';
 
 function App() {
   return (
-    <UnifiedAuthProvider supabaseClient={supabase} appName="My App">
+    <UnifiedAuthProvider supabaseClient={supabase} appName={APP_NAME}>
       <Router>
         <Routes>
-          <Route path="/login" element={<PaceLoginPage appName="My App" />} />
+          <Route path="/login" element={<PaceLoginPage appName={APP_NAME} />} />
+          <Route path="/events/*" element={<PublicPageApp appName={APP_NAME} />} />
           <Route path="/" element={
             <PaceAppLayout 
-              appName="My Application"
+              appName={APP_NAME}
               enforcePermissions={true}
               defaultPermission="read"
             />
@@ -2442,7 +2455,7 @@ function AdminApp() {
 
 #### Defined in
 
-[packages/core/src/components/PaceAppLayout/PaceAppLayout.tsx:328](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PaceAppLayout/PaceAppLayout.tsx#L328)
+[packages/core/src/components/PaceAppLayout/PaceAppLayout.tsx:336](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PaceAppLayout/PaceAppLayout.tsx#L336)
 
 ___
 
@@ -2823,14 +2836,18 @@ Header component for public pages with event-specific branding
 This component displays the app logo, event logo, and event information
 in a clean, accessible layout suitable for public pages.
 
-Logo handling follows a priority order:
-1. customAppLogo prop (if provided) - highest priority
-2. logoUrl prop (if provided) - direct URL override
-3. Auto-generated path from appName via useAppConfig() - convention-based
-4. Default SVG fallback - lowest priority
+The app logo is automatically generated from the appName using the pattern:
+`/{appName.toLowerCase()}_logo_wide.svg` from the public folder.
+
+**Important:** The appName is obtained from PublicPageProvider context, which should
+receive the APP_NAME constant from your App.tsx file. This ensures consistency with
+authenticated pages that use the same APP_NAME constant.
 
-The logo can be made clickable by providing the logoHref prop, which will
-wrap the logo in a Link component for navigation.
+**Event Logo Requirements:**
+- Event logo files must be marked as `is_public = true` in the `file_references` table
+- The RPC function `data_file_reference_by_category_list` must be accessible to the anonymous/public role
+- Storage bucket must allow public read access for logo files
+- If no public logo is available, a fallback UI with event initials will be displayed
 
 #### Parameters
 
@@ -2846,7 +2863,7 @@ React element with public page header
 
 #### Defined in
 
-[packages/core/src/components/PublicLayout/PublicPageHeader.tsx:124](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PublicLayout/PublicPageHeader.tsx#L124)
+[packages/core/src/components/PublicLayout/PublicPageHeader.tsx:103](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PublicLayout/PublicPageHeader.tsx#L103)
 
 ___
 
@@ -2922,6 +2939,7 @@ This provider:
 - Provides environment variables for public data access
 - Includes error boundary for graceful error handling
 - Is completely separate from the main app context
+- Provides appName for consistent logo display
 
 #### Parameters
 
@@ -2935,7 +2953,7 @@ This provider:
 
 #### Defined in
 
-[packages/core/src/components/PublicLayout/PublicPageProvider.tsx:64](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PublicLayout/PublicPageProvider.tsx#L64)
+[packages/core/src/components/PublicLayout/PublicPageProvider.tsx:70](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PublicLayout/PublicPageProvider.tsx#L70)
 
 ___
 
@@ -2953,7 +2971,7 @@ Public page context with environment variables
 
 #### Defined in
 
-[packages/core/src/components/PublicLayout/PublicPageProvider.tsx:119](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PublicLayout/PublicPageProvider.tsx#L119)
+[packages/core/src/components/PublicLayout/PublicPageProvider.tsx:128](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PublicLayout/PublicPageProvider.tsx#L128)
 
 ___
 
@@ -2971,7 +2989,7 @@ True if we're in a public page context
 
 #### Defined in
 
-[packages/core/src/components/PublicLayout/PublicPageProvider.tsx:134](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PublicLayout/PublicPageProvider.tsx#L134)
+[packages/core/src/components/PublicLayout/PublicPageProvider.tsx:143](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PublicLayout/PublicPageProvider.tsx#L143)
 
 ___
 
@@ -3858,7 +3876,7 @@ Useful for testing or when you need to force refresh all data
 
 #### Defined in
 
-[packages/core/src/hooks/public/usePublicFileDisplay.ts:373](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/public/usePublicFileDisplay.ts#L373)
+[packages/core/src/hooks/public/usePublicFileDisplay.ts:542](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/public/usePublicFileDisplay.ts#L542)
 
 ___
 
@@ -3879,7 +3897,7 @@ Get cache statistics for debugging
 
 #### Defined in
 
-[packages/core/src/hooks/public/usePublicFileDisplay.ts:384](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/public/usePublicFileDisplay.ts#L384)
+[packages/core/src/hooks/public/usePublicFileDisplay.ts:553](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/public/usePublicFileDisplay.ts#L553)
 
 ___
 
@@ -3935,7 +3953,7 @@ Useful when you only need the event code and will fetch data separately
 
 #### Defined in
 
-[packages/core/src/hooks/public/usePublicRouteParams.ts:224](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/public/usePublicRouteParams.ts#L224)
+[packages/core/src/hooks/public/usePublicRouteParams.ts:212](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/public/usePublicRouteParams.ts#L212)
 
 ___
 
@@ -3958,7 +3976,7 @@ Utility function to generate public route paths
 
 #### Defined in
 
-[packages/core/src/hooks/public/usePublicRouteParams.ts:260](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/public/usePublicRouteParams.ts#L260)
+[packages/core/src/hooks/public/usePublicRouteParams.ts:248](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/public/usePublicRouteParams.ts#L248)
 
 ___
 
@@ -3981,7 +3999,7 @@ Supports 2-50 character event codes
 
 #### Defined in
 
-[packages/core/src/hooks/public/usePublicRouteParams.ts:275](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/public/usePublicRouteParams.ts#L275)
+[packages/core/src/hooks/public/usePublicRouteParams.ts:263](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/public/usePublicRouteParams.ts#L263)
 
 ___
 

package/docs/architecture/rpc-function-standards.md

@@ -75,8 +75,9 @@ get_cake_meals                    -- Wrong: wrong prefix order
 
 ```sql
 CREATE OR REPLACE FUNCTION public.<family>_<domain>_<verb>(
-    p_param1 TYPE,
-    p_param2 TYPE DEFAULT NULL,
+    p_param1 TEXT,  -- ⚠️ Use TEXT for strings, NOT CHARACTER VARYING
+    p_param2 INTEGER DEFAULT NULL,  -- Use INTEGER, DOUBLE PRECISION, SMALLINT for numbers
+    p_param3 BOOLEAN DEFAULT NULL,  -- Use BOOLEAN for boolean values
     p_user_id UUID DEFAULT auth.uid(),
     p_organisation_id UUID DEFAULT NULL
 )
@@ -150,11 +151,23 @@ $$;
 
 2. **Always set `SET search_path TO 'public'`** - Prevents SQL injection attacks
 
-3. **Parameter Naming**:
+3. **Parameter Naming and Types**:
    - Prefix all parameters with `p_` (e.g., `p_event_id`, `p_user_id`)
    - Use `p_user_id UUID DEFAULT auth.uid()` for user context
    - Use `p_organisation_id UUID DEFAULT NULL` for organisation context
    - Use descriptive names
+   - **⚠️ CRITICAL: Parameter Type Requirements**:
+     - **String parameters MUST use `TEXT`**, NOT `CHARACTER VARYING` or `VARCHAR`
+     - **Numeric parameters**: Use `DOUBLE PRECISION` for decimal numbers, `INTEGER` for whole numbers, `SMALLINT` for small integers
+     - **Boolean parameters**: Use `BOOLEAN`
+     - **Date/Time parameters**: Use `DATE`, `TIMESTAMP`, or `TIMESTAMPTZ` as appropriate
+     - **UUID parameters**: Use `UUID`
+     - **Why TEXT instead of CHARACTER VARYING?**
+       - Application code calls functions with TEXT parameters
+       - PostgreSQL cannot automatically choose between TEXT and CHARACTER VARYING versions
+       - Having both creates ambiguity errors: "Could not choose the best candidate function"
+       - TEXT is the standard PostgreSQL type for string parameters in RPC functions
+       - Even though database columns may use CHARACTER VARYING, function parameters should use TEXT
 
 4. **Variable Naming**:
    - Prefix local variables with `v_` (e.g., `v_is_super_admin`, `v_event_exists`)
@@ -710,7 +723,22 @@ rbac_check_permission_simplified(
 
 3. **Missing page_id parameter**: Always provide the page name as the last parameter (e.g., `'dishes'`, `'meals'`, `'distribution'`)
 
-4. **Ambiguous column references**: 
+4. **Using CHARACTER VARYING instead of TEXT for string parameters**:
+   - ❌ **WRONG**: `p_unit_id CHARACTER VARYING` or `p_dish_name VARCHAR`
+   - ✅ **CORRECT**: `p_unit_id TEXT` or `p_dish_name TEXT`
+   - **CRITICAL**: All string parameters MUST use `TEXT`, NOT `CHARACTER VARYING` or `VARCHAR`
+   - Having both TEXT and CHARACTER VARYING versions causes "Could not choose the best candidate function" errors
+   - Application code calls functions with TEXT parameters, so functions must accept TEXT
+   - Even though database columns may use CHARACTER VARYING, function parameters should use TEXT
+
+5. **Creating duplicate function versions**:
+   - ❌ **WRONG**: Creating both `app_cake_unit_update(TEXT, ...)` and `app_cake_unit_update(CHARACTER VARYING, ...)`
+   - ✅ **CORRECT**: Only create `app_cake_unit_update(TEXT, ...)`
+   - Before creating a new function, check if a version already exists
+   - If updating a function, use `CREATE OR REPLACE FUNCTION` to update the existing version
+   - If you need to change parameter types, drop the old version first: `DROP FUNCTION IF EXISTS ... CASCADE;`
+
+6. **Ambiguous column references**: 
    - ❌ **WRONG**: `SELECT mealtype_name FROM cake_mealtype`
    - ✅ **CORRECT**: `SELECT cake_mealtype.mealtype_name FROM cake_mealtype`
    - Always fully qualify table names when there might be ambiguity
@@ -718,7 +746,7 @@ rbac_check_permission_simplified(
    - Common ambiguous columns: `organisation_id`, `created_by`, `updated_by`, `created_at`, `updated_at`
    - See "Avoiding Ambiguous Column References" section below for detailed guidance
 
-5. **Not checking existing migrations**: Before creating a new migration, check if a similar one already exists to avoid duplicates
+7. **Not checking existing migrations**: Before creating a new migration, check if a similar one already exists to avoid duplicates
 
 ### Updating Existing Functions
 
@@ -1046,6 +1074,12 @@ When creating or reviewing an RPC function, ensure:
 - ✅ Includes `GRANT EXECUTE` statement
 - ✅ Tested with various scenarios
 
+### ⚠️ Critical Parameter Type Requirements
+- ✅ **String parameters use TEXT**: All string parameters MUST use `TEXT`, NOT `CHARACTER VARYING` or `VARCHAR`
+- ✅ **No duplicate function versions**: Only one version of each function exists (TEXT version, not CHARACTER VARYING)
+- ✅ **Numeric parameters use correct types**: `DOUBLE PRECISION` for decimals, `INTEGER` for whole numbers, `SMALLINT` for small integers
+- ✅ **Why TEXT?**: Application code calls with TEXT parameters; having both TEXT and CHARACTER VARYING versions causes ambiguity errors
+
 ### ⚠️ Critical Permission Format
 - ✅ **Permission format is correct**: `'operation:page.{pageName}'` (e.g., `'create:page.dishes'`, `'read:page.meals'`)
 - ✅ **NOT using wrong format**: `'operation:resource'` (e.g., `'create:dishes'`, `'read:meals'`)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jmruthers/pace-core",
-  "version": "0.5.126",
+  "version": "0.5.127",
   "description": "Clean, modern React component library with Tailwind v4 styling and native utilities",
   "private": false,
   "publishConfig": {

package/src/components/Button/Button.tsx

@@ -74,7 +74,7 @@ function getButtonClasses(variant: ButtonProps['variant'] = 'default', size: But
     outline: 'border border-main-300 bg-background shadow-sm hover:bg-acc-400',
     secondary: 'bg-sec-100 text-sec-900 shadow-sm hover:bg-acc-400',
     ghost: 'hover:bg-acc-400',
-    link: 'text-primary underline-offset-4 hover:underline text-shadow-md hover:text-shadow-accent',
+    link: 'text-main-700 underline-offset-4 hover:underline hover:drop-shadow-lg hover:drop-shadow-acc-400',
   };
   
   const sizeClasses = {

package/src/components/DataTable/components/ImportModal.tsx

@@ -39,6 +39,7 @@ import React, { useState, useRef, useEffect } from 'react';
 import { Dialog, DialogContent, DialogDescription, DialogHeader, DialogTitle } from '../../Dialog';
 import { Button } from '../../Button/Button';
 import { Input } from '../../Input/Input';
+import { Progress } from '../../Progress/Progress';
 import { Upload, FileText, AlertCircle } from 'lucide-react';
 import { createLogger } from '../../../utils/logger';
 
@@ -115,6 +116,7 @@ export function ImportModal({ isOpen, onClose, onImport, config = {} }: ImportMo
   const [previewData, setPreviewData] = useState<Array<Record<string, unknown>> | null>(null);
   const [totalCount, setTotalCount] = useState<number>(0);
   const [validationErrors, setValidationErrors] = useState<Array<{row: number; field: string; message: string}>>([]);
+  const [importProgress, setImportProgress] = useState<{ current: number; total: number; stage: 'parsing' | 'importing' } | null>(null);
   const fileInputRef = useRef<HTMLInputElement>(null);
   const isMountedRef = useRef(true);
 
@@ -135,6 +137,7 @@ export function ImportModal({ isOpen, onClose, onImport, config = {} }: ImportMo
       setError(null);
       setValidationErrors([]);
       setIsProcessing(false);
+      setImportProgress(null);
       // Reset file input
       if (fileInputRef.current) {
         fileInputRef.current.value = '';
@@ -211,24 +214,126 @@ export function ImportModal({ isOpen, onClose, onImport, config = {} }: ImportMo
 
     setIsProcessing(true);
     setError(null);
+    setImportProgress({ current: 0, total: 0, stage: 'parsing' });
 
     try {
+      // Step 1: Parse CSV with progress indication
       const text = await file.text();
-      const data = processCSV(text);
+      const lines = text.split('\n').filter(line => line.trim());
+      const totalLines = lines.length;
+      
+      if (totalLines < 2) {
+        throw new Error('CSV must have at least a header row and one data row');
+      }
+
+      // For large files, process in chunks to show progress
+      const CHUNK_SIZE = 1000; // Process 1000 rows at a time
+      const data: Array<Record<string, unknown>> = [];
+      
+      if (totalLines > CHUNK_SIZE) {
+        // Large file - process in chunks with progress
+        const parseCSVLine = (line: string): string[] => {
+          const result: string[] = [];
+          let current = '';
+          let inQuotes = false;
+          
+          for (let i = 0; i < line.length; i++) {
+            const char = line[i];
+            
+            if (char === '"') {
+              inQuotes = !inQuotes;
+            } else if (char === ',' && !inQuotes) {
+              result.push(current.trim());
+              current = '';
+            } else {
+              current += char;
+            }
+          }
+          result.push(current.trim());
+          return result;
+        };
+
+        const headers = parseCSVLine(lines[0]).map(h => h.replace(/"/g, '').trim());
+        
+        // Process data rows in chunks
+        for (let i = 1; i < totalLines; i += CHUNK_SIZE) {
+          const chunkEnd = Math.min(i + CHUNK_SIZE, totalLines);
+          const chunk = lines.slice(i, chunkEnd);
+          
+          chunk.forEach((line, index) => {
+            const values = parseCSVLine(line).map(v => v.replace(/"/g, '').trim());
+            const row: Record<string, unknown> = {};
+            headers.forEach((header, colIndex) => {
+              row[header] = values[colIndex] || '';
+            });
+            data.push(row);
+          });
+          
+          // Update progress
+          const processed = Math.min(chunkEnd - 1, totalLines - 1);
+          if (isMountedRef.current) {
+            setImportProgress({ 
+              current: processed, 
+              total: totalLines - 1, 
+              stage: 'parsing' 
+            });
+          }
+          
+          // Yield to browser to update UI
+          await new Promise(resolve => setTimeout(resolve, 0));
+        }
+      } else {
+        // Small file - process normally
+        const parsedData = processCSV(text);
+        data.push(...parsedData);
+        if (isMountedRef.current) {
+          setImportProgress({ 
+            current: totalLines - 1, 
+            total: totalLines - 1, 
+            stage: 'parsing' 
+          });
+        }
+      }
+      
+      // Step 2: Import data with progress indication
+      if (isMountedRef.current) {
+        setImportProgress({ 
+          current: 0, 
+          total: data.length, 
+          stage: 'importing' 
+        });
+      }
       
       // Await the onImport callback in case it returns a promise
       const result = onImport(data);
       if (result && typeof result.then === 'function') {
+        // For async imports, we can't track exact progress, but we show it's processing
+        // The progress will remain at 0 until the import completes
         await result;
       }
+      // Note: For synchronous imports, the progress stays at 0 until we mark it complete
+      
+      // Mark as complete
+      if (isMountedRef.current) {
+        setImportProgress({ 
+          current: data.length, 
+          total: data.length, 
+          stage: 'importing' 
+        });
+      }
+      
+      // Small delay to show completion
+      await new Promise(resolve => setTimeout(resolve, 300));
       
       onClose();
       setFile(null);
     } catch (err) {
       setError(err instanceof Error ? err.message : 'Failed to process file');
+      setImportProgress(null);
     } finally {
       if (isMountedRef.current) {
         setIsProcessing(false);
+        setImportProgress(null);
       }
     }
   };
@@ -239,6 +344,7 @@ export function ImportModal({ isOpen, onClose, onImport, config = {} }: ImportMo
     setPreviewData(null);
     setTotalCount(0);
     setValidationErrors([]);
+    setImportProgress(null);
     onClose();
   };
   
@@ -338,7 +444,33 @@ export function ImportModal({ isOpen, onClose, onImport, config = {} }: ImportMo
           )}
 
 
-          {file && previewData && previewData.length > 0 ? (
+          {importProgress && isProcessing && (
+            <div className="space-y-2 p-4 bg-sec-50 rounded-lg border border-sec-200">
+              <div className="flex items-center justify-between">
+                <span className="text-sm font-medium text-sec-900">
+                  {importProgress.stage === 'parsing' ? 'Parsing CSV file...' : 'Importing data...'}
+                </span>
+                <span className="text-sm text-sec-600">
+                  {importProgress.current.toLocaleString()} / {importProgress.total.toLocaleString()} rows
+                </span>
+              </div>
+              <Progress 
+                value={importProgress.total > 0 ? (importProgress.current / importProgress.total) * 100 : 0} 
+                className="h-2 bg-sec-200"
+              />
+              <p className="text-xs text-sec-500">
+                {importProgress.total > 0 && importProgress.current < importProgress.total
+                  ? `${Math.round((importProgress.current / importProgress.total) * 100)}% complete`
+                  : importProgress.stage === 'importing' && importProgress.current === 0
+                  ? 'Processing your data...'
+                  : importProgress.current === importProgress.total
+                  ? 'Complete!'
+                  : 'Processing...'}
+              </p>
+            </div>
+          )}
+
+          {file && previewData && previewData.length > 0 && !isProcessing ? (
             <div className="space-y-3">
               <h4 className="text-sec-900">{previewHeaderText}</h4>
               <div className="border rounded-lg overflow-hidden">