@jmruthers/pace-core 0.5.114 → 0.5.116

package/docs/api/interfaces/UnifiedAuthContextType.md

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

package/docs/api/interfaces/UnifiedAuthProviderProps.md

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

package/docs/api/interfaces/UseInactivityTrackerOptions.md

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

package/docs/api/interfaces/UseInactivityTrackerReturn.md

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

package/docs/api/interfaces/UsePublicEventOptions.md

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

package/docs/api/interfaces/UsePublicEventReturn.md

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

package/docs/api/interfaces/UsePublicFileDisplayOptions.md

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

package/docs/api/interfaces/UsePublicFileDisplayReturn.md

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

package/docs/api/interfaces/UsePublicRouteParamsReturn.md

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

package/docs/api/interfaces/UseResolvedScopeOptions.md

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

package/docs/api/interfaces/UseResolvedScopeReturn.md

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

package/docs/api/interfaces/UserEventAccess.md

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

package/docs/api/interfaces/UserMenuProps.md

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

package/docs/api/interfaces/UserProfile.md

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

package/docs/api/modules.md

@@ -1,6 +1,6 @@
-[@jmruthers/pace-core - v0.5.114](README.md) / Exports
+[@jmruthers/pace-core - v0.5.116](README.md) / Exports
 
-# @jmruthers/pace-core - v0.5.114
+# @jmruthers/pace-core - v0.5.116
 
 **`File`**
 
@@ -114,6 +114,10 @@ import { Dialog, NavigationMenu } from '@jmruthers/pace-core/components';
 - [RBACLogger](interfaces/RBACLogger.md)
 - [UseResolvedScopeOptions](interfaces/UseResolvedScopeOptions.md)
 - [UseResolvedScopeReturn](interfaces/UseResolvedScopeReturn.md)
+- [EventAppRoleData](interfaces/EventAppRoleData.md)
+- [RevokeEventAppRoleParams](interfaces/RevokeEventAppRoleParams.md)
+- [GrantEventAppRoleParams](interfaces/GrantEventAppRoleParams.md)
+- [RoleManagementResult](interfaces/RoleManagementResult.md)
 - [StyleImport](interfaces/StyleImport.md)
 - [ColorShade](interfaces/ColorShade.md)
 - [ColorPalette](interfaces/ColorPalette.md)
@@ -348,6 +352,7 @@ import { Dialog, NavigationMenu } from '@jmruthers/pace-core/components';
 - [useCachedPermissions](modules.md#usecachedpermissions)
 - [useRBAC](modules.md#userbac)
 - [useResolvedScope](modules.md#useresolvedscope)
+- [useRoleManagement](modules.md#userolemanagement)
 - [isValidPermission](modules.md#isvalidpermission)
 - [getPermissionsForRole](modules.md#getpermissionsforrole)
 - [createSecureClient](modules.md#createsecureclient)
@@ -2436,7 +2441,7 @@ function AdminApp() {
 
 #### Defined in
 
-[packages/core/src/components/PaceAppLayout/PaceAppLayout.tsx:327](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PaceAppLayout/PaceAppLayout.tsx#L327)
+[packages/core/src/components/PaceAppLayout/PaceAppLayout.tsx:328](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/PaceAppLayout/PaceAppLayout.tsx#L328)
 
 ___
 
@@ -3013,7 +3018,7 @@ ___
 
 #### Defined in
 
-[packages/core/src/components/Select/Select.tsx:465](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Select/Select.tsx#L465)
+[packages/core/src/components/Select/Select.tsx:473](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Select/Select.tsx#L473)
 
 ___
 
@@ -3033,7 +3038,7 @@ ___
 
 #### Defined in
 
-[packages/core/src/components/Select/Select.tsx:482](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Select/Select.tsx#L482)
+[packages/core/src/components/Select/Select.tsx:490](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Select/Select.tsx#L490)
 
 ___
 
@@ -3053,7 +3058,7 @@ ___
 
 #### Defined in
 
-[packages/core/src/components/Select/Select.tsx:585](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Select/Select.tsx#L585)
+[packages/core/src/components/Select/Select.tsx:593](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Select/Select.tsx#L593)
 
 ___
 
@@ -3073,7 +3078,7 @@ ___
 
 #### Defined in
 
-[packages/core/src/components/Select/Select.tsx:686](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Select/Select.tsx#L686)
+[packages/core/src/components/Select/Select.tsx:694](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Select/Select.tsx#L694)
 
 ___
 
@@ -3093,7 +3098,7 @@ ___
 
 #### Defined in
 
-[packages/core/src/components/Select/Select.tsx:697](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Select/Select.tsx#L697)
+[packages/core/src/components/Select/Select.tsx:705](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Select/Select.tsx#L705)
 
 ___
 
@@ -3113,7 +3118,7 @@ ___
 
 #### Defined in
 
-[packages/core/src/components/Select/Select.tsx:708](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Select/Select.tsx#L708)
+[packages/core/src/components/Select/Select.tsx:716](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Select/Select.tsx#L716)
 
 ___
 
@@ -4062,7 +4067,7 @@ void - This is an effect hook with no return value
 
 #### Defined in
 
-[packages/core/src/hooks/useEventTheme.ts:41](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/useEventTheme.ts#L41)
+[packages/core/src/hooks/useEventTheme.ts:85](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/useEventTheme.ts#L85)
 
 ___
 
@@ -4386,12 +4391,12 @@ Object with toast state and control methods
 | Name | Type | Description |
 | :------ | :------ | :------ |
 | `toasts` | `ToasterToast`[] | Array of active toasts |
-| `toast` | (`props`: `Toast`) => \{ `id`: `string` = id; `dismiss`: () => `void` ; `update`: (`props`: `Partial`\<`Omit`\<`ToasterToast`, ``"id"``\>\>) => `void`  } | Creates a new toast notification |
+| `toast` | (`props`: `Toast`) => \{ `id`: `string` = id; `dismiss`: () => `void` ; `update`: (`props`: `Partial`\<`Omit`\<`ToasterToast`, ``"id"`` \| ``"duration"``\>\>) => `void`  } | Creates a new toast notification |
 | `dismiss` | (`toastId?`: `string`) => `void` | - |
 
 #### Defined in
 
-[packages/core/src/hooks/useToast.ts:230](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/useToast.ts#L230)
+[packages/core/src/hooks/useToast.ts:229](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/hooks/useToast.ts#L229)
 
 ___
 
@@ -5273,7 +5278,7 @@ RBACAuditManager instance
 
 #### Defined in
 
-[packages/core/src/rbac/audit.ts:340](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/rbac/audit.ts#L340)
+[packages/core/src/rbac/audit.ts:375](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/rbac/audit.ts#L375)
 
 ___
 
@@ -5295,7 +5300,7 @@ Set the global audit manager
 
 #### Defined in
 
-[packages/core/src/rbac/audit.ts:356](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/rbac/audit.ts#L356)
+[packages/core/src/rbac/audit.ts:391](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/rbac/audit.ts#L391)
 
 ___
 
@@ -5313,7 +5318,7 @@ Global audit manager or null if not set
 
 #### Defined in
 
-[packages/core/src/rbac/audit.ts:365](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/rbac/audit.ts#L365)
+[packages/core/src/rbac/audit.ts:400](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/rbac/audit.ts#L400)
 
 ___
 
@@ -5335,7 +5340,7 @@ Emit an audit event using the global audit manager
 
 #### Defined in
 
-[packages/core/src/rbac/audit.ts:374](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/rbac/audit.ts#L374)
+[packages/core/src/rbac/audit.ts:409](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/rbac/audit.ts#L409)
 
 ___
 
@@ -6171,6 +6176,28 @@ const permission = useCan(userId, resolvedScope, permission);
 
 ___
 
+### useRoleManagement
+
+▸ **useRoleManagement**(): `Object`
+
+#### Returns
+
+`Object`
+
+| Name | Type |
+| :------ | :------ |
+| `revokeEventAppRole` | (`params`: [`RevokeEventAppRoleParams`](interfaces/RevokeEventAppRoleParams.md)) => `Promise`\<[`RoleManagementResult`](interfaces/RoleManagementResult.md)\> |
+| `grantEventAppRole` | (`params`: [`GrantEventAppRoleParams`](interfaces/GrantEventAppRoleParams.md)) => `Promise`\<[`RoleManagementResult`](interfaces/RoleManagementResult.md)\> |
+| `revokeRoleById` | (`roleId`: `string`) => `Promise`\<[`RoleManagementResult`](interfaces/RoleManagementResult.md)\> |
+| `isLoading` | `boolean` |
+| `error` | ``null`` \| `Error` |
+
+#### Defined in
+
+[packages/core/src/rbac/hooks/useRoleManagement.ts:68](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/rbac/hooks/useRoleManagement.ts#L68)
+
+___
+
 ### isValidPermission
 
 ▸ **isValidPermission**(`permission`): permission is \`create:$\{string}\` \| \`update:$\{string}\` \| \`read:$\{string}\` \| \`delete:$\{string}\`

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

@@ -714,6 +714,9 @@ rbac_check_permission_simplified(
    - ❌ **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
+   - **MANDATORY**: When using JOINs, always use table aliases and prefix ALL column references
+   - 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
 
@@ -736,6 +739,190 @@ rbac_check_permission_simplified(
 
 3. **Update references** - Update all callers before removing
 
+## Avoiding Ambiguous Column References
+
+### Overview
+
+Ambiguous column reference errors occur when PostgreSQL cannot determine which table a column belongs to in a query with JOINs. This is a common source of silent failures in RPC functions.
+
+### Mandatory Rules
+
+1. **Always use table aliases in JOINs**: Never write JOINs without table aliases
+2. **Fully qualify ALL column references**: Every column in SELECT, WHERE, ORDER BY, GROUP BY, and HAVING clauses must be prefixed with its table alias
+3. **Common ambiguous columns**: The following columns exist in multiple tables and MUST always be prefixed:
+   - `organisation_id`
+   - `created_by`, `updated_by`
+   - `created_at`, `updated_at`
+   - `event_id` and its variations (e.g., `dish_event_id`, `meal_event_id`)
+
+### Correct Patterns
+
+```sql
+-- ✅ CORRECT: All columns properly prefixed with table aliases
+SELECT 
+  d.dish_id,
+  d.dish_name,
+  d.organisation_id,
+  d.created_at,
+  mt.mealtype_name,
+  e.event_name,
+  e.organisation_id AS event_organisation_id
+FROM cake_dish d
+LEFT JOIN cake_mealtype mt ON d.dish_mealtype_id = mt.mealtype_id
+LEFT JOIN event e ON d.dish_event_id = e.event_id
+WHERE d.dish_event_id = p_event_id
+  AND d.organisation_id = p_organisation_id
+ORDER BY d.dish_code, mt.mealtype_name;
+
+-- ✅ CORRECT: CTEs with proper aliases
+WITH distribution_items AS (
+  SELECT
+    d.item_id,
+    d.collection_date,
+    SUM(d.required_quantity) as total_quantity
+  FROM data_cake_distribution_list(p_event_id, p_user_id, p_organisation_id) d
+  GROUP BY d.item_id, d.collection_date
+)
+SELECT 
+  di.item_id,
+  di.collection_date,
+  di.total_quantity
+FROM distribution_items di
+WHERE di.item_id = p_item_id;
+```
+
+### Incorrect Patterns
+
+```sql
+-- ❌ WRONG: Missing table prefixes in SELECT
+SELECT 
+  dish_id,
+  dish_name,
+  organisation_id,  -- AMBIGUOUS: Could be from cake_dish or event
+  mealtype_name
+FROM cake_dish d
+LEFT JOIN cake_mealtype mt ON d.dish_mealtype_id = mt.mealtype_id
+LEFT JOIN event e ON d.dish_event_id = e.event_id;
+
+-- ❌ WRONG: Missing table prefix in WHERE clause
+SELECT d.dish_id, d.dish_name
+FROM cake_dish d
+LEFT JOIN event e ON d.dish_event_id = e.event_id
+WHERE organisation_id = p_organisation_id;  -- AMBIGUOUS: Which table?
+
+-- ❌ WRONG: Missing table prefix in ORDER BY
+SELECT d.dish_id, d.dish_name, e.event_name
+FROM cake_dish d
+LEFT JOIN event e ON d.dish_event_id = e.event_id
+ORDER BY created_at;  -- AMBIGUOUS: Which table's created_at?
+
+-- ❌ WRONG: SELECT INTO with multiple columns but wrong variable count
+SELECT d.dish_code, d.dish_name, mt.mealtype_name
+INTO v_mealtype_name  -- WRONG: Only one variable for three columns
+FROM cake_dish d
+LEFT JOIN cake_mealtype mt ON d.dish_mealtype_id = mt.mealtype_id;
+```
+
+### SELECT INTO Statements
+
+When using `SELECT ... INTO`, ensure:
+1. The number of columns matches the number of variables
+2. All columns are properly prefixed with table aliases
+3. Use separate variables for each column
+
+```sql
+-- ✅ CORRECT: Proper SELECT INTO with multiple variables
+DECLARE
+  v_dish_code TEXT;
+  v_dish_name TEXT;
+  v_mealtype_name TEXT;
+  v_updated_at TIMESTAMPTZ;
+BEGIN
+  SELECT 
+    d.dish_code,
+    d.dish_name,
+    mt.mealtype_name,
+    d.updated_at
+  INTO 
+    v_dish_code,
+    v_dish_name,
+    v_mealtype_name,
+    v_updated_at
+  FROM cake_dish d
+  LEFT JOIN cake_mealtype mt ON d.dish_mealtype_id = mt.mealtype_id
+  WHERE d.dish_id = p_dish_id;
+END;
+```
+
+### CTEs and Subqueries
+
+When using CTEs (Common Table Expressions) or subqueries:
+1. Use descriptive aliases for CTEs
+2. Fully qualify all column references within CTEs
+3. When referencing CTEs in the main query, use the CTE alias
+
+```sql
+-- ✅ CORRECT: CTE with proper aliases
+WITH unit_diners AS (
+  SELECT 
+    d.diner_unit_id,
+    d.diner_diettype_id,
+    SUM(COALESCE(d.diner_adult, 0)) as adults,
+    SUM(COALESCE(d.diner_youth, 0)) as youth
+  FROM cake_diner d
+  INNER JOIN event_units eu ON d.diner_unit_id = eu.unit_id
+  WHERE (d.diner_adult > 0 OR d.diner_youth > 0)
+  GROUP BY d.diner_unit_id, d.diner_diettype_id
+)
+SELECT 
+  ud.diner_unit_id,
+  ud.adults,
+  ud.youth
+FROM unit_diners ud
+WHERE ud.diner_unit_id = p_unit_id;
+```
+
+### Table Alias Naming Conventions
+
+Use short, descriptive aliases:
+- `d` for `cake_dish`
+- `r` for `cake_recipe`
+- `m` for `cake_meal`
+- `mt` for `cake_mealtype`
+- `i` for `cake_item`
+- `dt` for `cake_diettype`
+- `e` for `event`
+- `u` for `cake_unit`
+- `mp` for `cake_mealplan`
+
+### UPDATE Statements
+
+For UPDATE statements, the WHERE clause should reference the table being updated:
+
+```sql
+-- ✅ CORRECT: WHERE clause uses table name or alias
+UPDATE cake_dish
+SET dish_name = p_dish_name,
+    updated_by = p_user_id,
+    updated_at = now()
+WHERE dish_id = p_dish_id;  -- dish_id is unambiguous (primary key)
+
+-- ✅ CORRECT: When updating with JOIN, qualify all columns
+UPDATE cake_dish d
+SET d.dish_name = p_dish_name
+FROM cake_mealtype mt
+WHERE d.dish_mealtype_id = mt.mealtype_id
+  AND d.dish_id = p_dish_id;
+```
+
+### Testing for Ambiguous References
+
+Before deploying, test your function:
+1. Call the function with valid parameters
+2. Check PostgreSQL logs for "ambiguous column reference" errors
+3. Verify all column references are prefixed in JOINs
+4. Test with multiple tables that share column names
+
 ## Common Patterns
 
 ### Pattern 1: Simple Data Retrieval
@@ -871,6 +1058,12 @@ When creating or reviewing an RPC function, ensure:
 - ✅ **File naming**: `YYYYMMDDHHMMSS_descriptive_name.sql`
 - ✅ **No ambiguous column references**: Fully qualify table names (e.g., `cake_mealtype.mealtype_name`)
 
+### ⚠️ Column Reference Requirements
+- ✅ **All column references in JOINs are fully qualified with table aliases**: Every column in SELECT, WHERE, ORDER BY, GROUP BY, HAVING must have table prefix
+- ✅ **No ambiguous column references in WHERE, ORDER BY, GROUP BY, HAVING clauses**: All columns must be prefixed with table alias
+- ✅ **Table aliases used consistently and meaningfully**: Use short, descriptive aliases (e.g., `d` for dish, `r` for recipe)
+- ✅ **SELECT INTO statements match variable count**: Number of columns must match number of variables
+
 ## References
 
 - [RBAC System Documentation](../core-concepts/rbac-system.md)

package/package.json

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