@jmruthers/pace-core 0.6.11 → 0.6.12

package/docs/api/variables/Tooltip.md

@@ -0,0 +1,34 @@
+[**@jmruthers/pace-core v0.6.12**](../README.md)
+
+***
+
+[@jmruthers/pace-core](../globals.md) / Tooltip
+
+# Variable: Tooltip
+
+> `const` **Tooltip**: `ForwardRefExoticComponent`\<`TooltipProps` & `RefAttributes`\<`HTMLButtonElement`\>\>
+
+Defined in: [components/Tooltip/Tooltip.tsx:142](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Tooltip/Tooltip.tsx#L142)
+
+Tooltip component
+A simplified tooltip component for easy usage
+
+## Param
+
+Tooltip configuration
+
+## Param
+
+Forwarded ref to the trigger element
+
+## Returns
+
+JSX.Element - The tooltip with trigger and content
+
+## Example
+
+```tsx
+<Tooltip content="This is a helpful tooltip">
+  <Button>Hover me</Button>
+</Tooltip>
+```

package/docs/api/variables/TooltipContent.md

@@ -0,0 +1,34 @@
+[**@jmruthers/pace-core v0.6.12**](../README.md)
+
+***
+
+[@jmruthers/pace-core](../globals.md) / TooltipContent
+
+# Variable: TooltipContent
+
+> `const` **TooltipContent**: `ForwardRefExoticComponent`\<`Omit`\<`TooltipContentProps` & `RefAttributes`\<`HTMLDivElement`\>, `"ref"`\> & `RefAttributes`\<`HTMLDivElement`\>\>
+
+Defined in: [components/Tooltip/Tooltip.tsx:99](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Tooltip/Tooltip.tsx#L99)
+
+TooltipContent component
+The content container for tooltip information
+
+## Param
+
+Content configuration and styling
+
+## Param
+
+Forwarded ref to the content element
+
+## Returns
+
+JSX.Element - The tooltip content container
+
+## Example
+
+```tsx
+<TooltipContent side="top" className="bg-main-500">
+  Custom tooltip content
+</TooltipContent>
+```

package/docs/api/variables/TooltipProvider.md

@@ -0,0 +1,11 @@
+[**@jmruthers/pace-core v0.6.12**](../README.md)
+
+***
+
+[@jmruthers/pace-core](../globals.md) / TooltipProvider
+
+# Variable: TooltipProvider
+
+> `const` **TooltipProvider**: `FC`\<`TooltipProviderProps`\> = `TooltipPrimitive.Provider`
+
+Defined in: [components/Tooltip/Tooltip.tsx:78](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Tooltip/Tooltip.tsx#L78)

package/docs/api/variables/TooltipRoot.md

@@ -0,0 +1,11 @@
+[**@jmruthers/pace-core v0.6.12**](../README.md)
+
+***
+
+[@jmruthers/pace-core](../globals.md) / TooltipRoot
+
+# Variable: TooltipRoot
+
+> `const` **TooltipRoot**: `FC`\<`TooltipProps`\> = `TooltipPrimitive.Root`
+
+Defined in: [components/Tooltip/Tooltip.tsx:80](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Tooltip/Tooltip.tsx#L80)

package/docs/api/variables/TooltipTrigger.md

@@ -0,0 +1,11 @@
+[**@jmruthers/pace-core v0.6.12**](../README.md)
+
+***
+
+[@jmruthers/pace-core](../globals.md) / TooltipTrigger
+
+# Variable: TooltipTrigger
+
+> `const` **TooltipTrigger**: `ForwardRefExoticComponent`\<`TooltipTriggerProps` & `RefAttributes`\<`HTMLButtonElement`\>\> = `TooltipPrimitive.Trigger`
+
+Defined in: [components/Tooltip/Tooltip.tsx:82](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/Tooltip/Tooltip.tsx#L82)

package/docs/api/variables/UserMenu.md

@@ -0,0 +1,11 @@
+[**@jmruthers/pace-core v0.6.12**](../README.md)
+
+***
+
+[@jmruthers/pace-core](../globals.md) / UserMenu
+
+# Variable: UserMenu
+
+> `const` **UserMenu**: `NamedExoticComponent`\<[`UserMenuProps`](../interfaces/UserMenuProps.md)\>
+
+Defined in: [components/UserMenu/UserMenu.tsx:137](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/components/UserMenu/UserMenu.tsx#L137)

package/docs/api/variables/emailSchema.md

@@ -0,0 +1,13 @@
+[**@jmruthers/pace-core v0.6.12**](../README.md)
+
+***
+
+[@jmruthers/pace-core](../globals.md) / emailSchema
+
+# Variable: emailSchema
+
+> `const` **emailSchema**: `ZodString`
+
+Defined in: [utils/validation/common.ts:12](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/utils/validation/common.ts#L12)
+
+Email validation schema

package/docs/api/variables/logger.md

@@ -0,0 +1,203 @@
+[**@jmruthers/pace-core v0.6.12**](../README.md)
+
+***
+
+[@jmruthers/pace-core](../globals.md) / logger
+
+# Variable: logger
+
+> `const` **logger**: `object`
+
+Defined in: [utils/core/logger.ts:167](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/utils/core/logger.ts#L167)
+
+Default logger instance for convenience
+
+## Type Declaration
+
+### debug()
+
+> **debug**: (`component`, `message`, ...`args`) => `void`
+
+#### Parameters
+
+##### component
+
+`string`
+
+##### message
+
+`string`
+
+##### args
+
+...`unknown`[]
+
+#### Returns
+
+`void`
+
+### info()
+
+> **info**: (`component`, `message`, ...`args`) => `void`
+
+#### Parameters
+
+##### component
+
+`string`
+
+##### message
+
+`string`
+
+##### args
+
+...`unknown`[]
+
+#### Returns
+
+`void`
+
+### warn()
+
+> **warn**: (`component`, `message`, ...`args`) => `void`
+
+#### Parameters
+
+##### component
+
+`string`
+
+##### message
+
+`string`
+
+##### args
+
+...`unknown`[]
+
+#### Returns
+
+`void`
+
+### error()
+
+> **error**: (`component`, `message`, ...`args`) => `void`
+
+#### Parameters
+
+##### component
+
+`string`
+
+##### message
+
+`string`
+
+##### args
+
+...`unknown`[]
+
+#### Returns
+
+`void`
+
+### createScopedLogger()
+
+> **createScopedLogger**: (`component`) => `object`
+
+#### Parameters
+
+##### component
+
+`string`
+
+#### Returns
+
+`object`
+
+##### debug()
+
+> **debug**: (`message`, ...`args`) => `void`
+
+###### Parameters
+
+###### message
+
+`string`
+
+###### args
+
+...`unknown`[]
+
+###### Returns
+
+`void`
+
+##### info()
+
+> **info**: (`message`, ...`args`) => `void`
+
+###### Parameters
+
+###### message
+
+`string`
+
+###### args
+
+...`unknown`[]
+
+###### Returns
+
+`void`
+
+##### warn()
+
+> **warn**: (`message`, ...`args`) => `void`
+
+###### Parameters
+
+###### message
+
+`string`
+
+###### args
+
+...`unknown`[]
+
+###### Returns
+
+`void`
+
+##### error()
+
+> **error**: (`message`, ...`args`) => `void`
+
+###### Parameters
+
+###### message
+
+`string`
+
+###### args
+
+...`unknown`[]
+
+###### Returns
+
+`void`
+
+### configure()
+
+> **configure**: (`config`) => `void`
+
+#### Parameters
+
+##### config
+
+`Partial`\<[`LoggerConfig`](../interfaces/LoggerConfig.md)\>
+
+#### Returns
+
+`void`

package/docs/api/variables/nameSchema.md

@@ -0,0 +1,13 @@
+[**@jmruthers/pace-core v0.6.12**](../README.md)
+
+***
+
+[@jmruthers/pace-core](../globals.md) / nameSchema
+
+# Variable: nameSchema
+
+> `const` **nameSchema**: `ZodString`
+
+Defined in: [utils/validation/common.ts:21](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/utils/validation/common.ts#L21)
+
+Name validation schema

package/docs/api/variables/passwordSchema.md

@@ -0,0 +1,13 @@
+[**@jmruthers/pace-core v0.6.12**](../README.md)
+
+***
+
+[@jmruthers/pace-core](../globals.md) / passwordSchema
+
+# Variable: passwordSchema
+
+> `const` **passwordSchema**: `ZodString`
+
+Defined in: [utils/validation/passwordSchema.ts:67](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/utils/validation/passwordSchema.ts#L67)
+
+Basic password schema for less strict requirements

package/docs/api/variables/phoneSchema.md

@@ -0,0 +1,13 @@
+[**@jmruthers/pace-core v0.6.12**](../README.md)
+
+***
+
+[@jmruthers/pace-core](../globals.md) / phoneSchema
+
+# Variable: phoneSchema
+
+> `const` **phoneSchema**: `ZodString`
+
+Defined in: [utils/validation/common.ts:30](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/utils/validation/common.ts#L30)
+
+Phone number validation schema

package/docs/api/variables/rbacCache.md

@@ -0,0 +1,16 @@
+[**@jmruthers/pace-core v0.6.12**](../README.md)
+
+***
+
+[@jmruthers/pace-core](../globals.md) / rbacCache
+
+# Variable: rbacCache
+
+> `const` **rbacCache**: [`RBACCache`](../classes/RBACCache.md)
+
+Defined in: [rbac/cache.ts:302](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/rbac/cache.ts#L302)
+
+Global cache instance
+
+This is the default cache instance used by the RBAC system.
+You can create additional instances if needed for different contexts.

package/docs/api/variables/styleConfig.md

@@ -0,0 +1,25 @@
+[**@jmruthers/pace-core v0.6.12**](../README.md)
+
+***
+
+[@jmruthers/pace-core](../globals.md) / styleConfig
+
+# Variable: styleConfig
+
+> `const` **styleConfig**: `object`
+
+Defined in: [styles/index.ts:32](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/styles/index.ts#L32)
+
+## Type Declaration
+
+### core
+
+> `readonly` **core**: `object`
+
+#### core.path
+
+> `readonly` **path**: `"./core.css"` = `'./core.css'`
+
+#### core.description
+
+> `readonly` **description**: `"Complete CSS foundation including base styles, theme mappings, fonts, resets, and neutral tokens"` = `'Complete CSS foundation including base styles, theme mappings, fonts, resets, and neutral tokens'`

package/docs/api/variables/urlSchema.md

@@ -0,0 +1,13 @@
+[**@jmruthers/pace-core v0.6.12**](../README.md)
+
+***
+
+[@jmruthers/pace-core](../globals.md) / urlSchema
+
+# Variable: urlSchema
+
+> `const` **urlSchema**: `ZodString`
+
+Defined in: [utils/validation/common.ts:39](https://github.com/jmruthers/pace-core/blob/main/packages/core/src/utils/validation/common.ts#L39)
+
+URL validation schema

package/docs/api-reference/hooks.md

@@ -288,6 +288,8 @@ function UserActions({ userId, scope }) {
 }
 ```
 
+**Page-name permission checks:** For checks that use a page name (e.g. `useCan(userId, scope, 'read:page.items', 'items')`), scope should include `appId` so the backend can resolve event roles. For event-scoped pages, include both `eventId` and `appId`; when `eventId` is present but `appId` is missing, pace-core may resolve `appId` from the current app name. You can also use `useResolvedScope` to obtain a scope with `organisationId`, `eventId`, and `appId`.
+
 ### usePermissions
 
 Get all permissions for a user in a specific scope.

package/docs/implementation-guides/data-tables.md

@@ -50,6 +50,14 @@ When `deleteSelected` is enabled, a **confirmation dialog** is shown by pace-cor
 >
 > The DataTable simply calls your provided handlers (`onDeleteRow`, `onEditRow`, `onCreateRow`) - it does not intercept or modify them to add audit logging.
 
+### Organisation and event context (edit/delete buttons)
+
+DataTable permission checks (and thus visibility of edit/delete/import buttons) depend on **RBAC scope**: organisation and, for event-scoped pages, **event** must be available when the table mounts.
+
+- **If organisation or event is set asynchronously** (e.g. after navigation or from a restored session), the first render may have no scope or incomplete scope, so permission checks return "no access" and buttons stay hidden. After a **refresh**, context is often already in state, so scope is ready and buttons appear.
+- **Fix in the app:** Ensure organisation and, for event-scoped pages, event are set **before** or **as** the user reaches the page (e.g. from URL, storage, or a layout that sets context on load). That way scope is ready when the DataTable runs its permission checks and edit/delete/import buttons show without needing a refresh.
+- **Event-scoped pages (e.g. CAKE units/menus):** The selected **event** must be in scope. If the user navigates to the page without an event selected, scope is incomplete and buttons will not show until context is set (or the page is refreshed with event in state).
+
 ## Basic Usage
 
 ### Simple Data Table

package/docs/rbac/getting-started.md

@@ -837,6 +837,13 @@ function AdaptiveComponent({ appId }) {
 }
 ```
 
+### Page permissions and scope
+
+For page permission checks that use a **page name** (e.g. `useCan(userId, scope, 'read:page.items', 'items')`), the backend can only return **event-role** permissions when the scope includes both `eventId` and `appId`. For event-scoped pages, include both in scope so the RPC considers event roles (e.g. `planner`, `event_admin`).
+
+- **Scope:** When calling `useCan` for event-scoped page permissions, pass a scope that includes `eventId` and `appId` (and `organisationId` when required). The library may resolve `appId` from the current app name when `eventId` is present but `appId` is missing, so you can pass `{ organisationId, eventId }` and pace-core will resolve `appId` before calling the backend.
+- **useResolvedScope:** For event-scoped pages, you can use the `useResolvedScope` hook to obtain a scope with `organisationId`, `eventId`, and `appId` resolved from the current app and selection context.
+
 ## Next Steps
 
 - [API Reference](./api-reference.md) - Complete API documentation

package/docs/rbac/troubleshooting.md

@@ -143,9 +143,13 @@ setupRBAC(supabase, {
 - Console shows `scope: { organisationId: "...", eventId: "..." }` (missing `appId`)
 - All permission checks return `false`
 
-**Root Cause:** App name not being set globally for RBAC resolution
+**Root Cause:** For page permission checks with a page name, the backend returns event-role permissions only when both `eventId` and `appId` are provided. If scope is missing `appId`, event-scoped page checks may deny. App name may also not be set globally for RBAC resolution.
 
 **Fix:**
+
+For **event-scoped page checks**, scope should include both `eventId` and `appId`. pace-core can resolve `appId` from the current app name when `eventId` is present (so passing `{ organisationId, eventId }` is enough in that case). Alternatively, use the `useResolvedScope` hook to obtain a scope with `organisationId`, `eventId`, and `appId` before calling `useCan`.
+
+**Fix (app name not set):**
 ```typescript
 // ❌ WRONG - Missing setRBACAppName call
 import { UnifiedAuthProvider } from '@jmruthers/pace-core/providers'

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jmruthers/pace-core",
-  "version": "0.6.11",
+  "version": "0.6.12",
   "description": "React component library with Tailwind v4",
   "private": false,
   "publishConfig": {
@@ -268,8 +268,8 @@
     "jsdom": "^25.0.1",
     "react-router-dom": "^6.26.2",
     "tsup": "^8.5.1",
-    "typedoc": "^0.27.0",
-    "typedoc-plugin-markdown": "^3.17.1",
+    "typedoc": "^0.28.0",
+    "typedoc-plugin-markdown": "^4.10.0",
     "typedoc-plugin-merge-modules": "^5.1.0",
     "typescript": "^5.9.3",
     "typescript-eslint": "^8.39.0",

package/src/components/DataTable/hooks/useDataTableEffectiveActions.ts

@@ -172,26 +172,36 @@ export function useDataTableEffectiveActions<TData extends DataRecord>(
   ]);
 
   const effectiveActions = useMemo(() => {
+    const defaultEditOnClick = (row: TData) => {
+      if (!permissions.canUpdate.can) {
+        throw new Error('Insufficient permissions to edit this resource');
+      }
+      const rowIndex = data.findIndex((r) => r === row);
+      const rowId = resolvedGetRowId(row, rowIndex >= 0 ? rowIndex : 0);
+      stateActions.setEditingRow(rowId, toCellValueRecord(row));
+    };
+
     const result = [...actions];
-    if (
-      secureFeatures.editing &&
-      secureHandlers.onEditRow &&
-      !result.some((a) => a.label === 'Edit')
-    ) {
-      result.push({
-        label: 'Edit',
-        onClick: (row: TData) => {
-          if (!permissions.canUpdate.can) {
-            throw new Error('Insufficient permissions to edit this resource');
-          }
-          const rowIndex = data.findIndex((r) => r === row);
-          const rowId = resolvedGetRowId(row, rowIndex >= 0 ? rowIndex : 0);
-          stateActions.setEditingRow(rowId, toCellValueRecord(row));
-        },
-        icon: Edit,
-        testId: 'edit',
-        hidden: !permissions.canUpdate.can,
-      });
+    if (secureFeatures.editing && secureHandlers.onEditRow) {
+      const existingEditIndex = result.findIndex((a) => a.label === 'Edit');
+      if (existingEditIndex >= 0) {
+        const customEdit = result[existingEditIndex];
+        result[existingEditIndex] = {
+          ...customEdit,
+          onClick: defaultEditOnClick,
+          icon: customEdit.icon ?? Edit,
+          testId: customEdit.testId ?? 'edit',
+          hidden: customEdit.hidden ?? !permissions.canUpdate.can,
+        };
+      } else {
+        result.push({
+          label: 'Edit',
+          onClick: defaultEditOnClick,
+          icon: Edit,
+          testId: 'edit',
+          hidden: !permissions.canUpdate.can,
+        });
+      }
     }
     if (
       secureFeatures.deletion &&

package/src/components/DataTable/hooks/useDataTableScope.test.ts

@@ -49,7 +49,7 @@ describe('[unit] useDataTableScope', () => {
     });
   }, TEST_TIMEOUT);
 
-  it('returns empty scope when resolvedScope is null and not loading', () => {
+  it('returns null scope when resolvedScope is null and not loading', () => {
     const { result } = renderHook(() =>
       useDataTableScope({
         selectedOrganisation: null,
@@ -59,17 +59,13 @@ describe('[unit] useDataTableScope', () => {
         pageName: undefined,
       })
     );
-    expect(result.current.consistentScope).toEqual({
-      organisationId: undefined,
-      eventId: undefined,
-      appId: undefined,
-    });
+    expect(result.current.consistentScope).toBeNull();
   }, TEST_TIMEOUT);
 
-  it('with pageId and no pageName, can proceed with empty scope', () => {
+  it('with pageId and no pageName, returns null when no scope values (never pass empty scope to RPC)', () => {
     mockUseResolvedScope.mockReturnValue({
       resolvedScope: null,
-      isLoading: true,
+      isLoading: false,
     });
     const { result } = renderHook(() =>
       useDataTableScope({
@@ -80,11 +76,7 @@ describe('[unit] useDataTableScope', () => {
         pageName: undefined,
       })
     );
-    expect(result.current.consistentScope).toEqual({
-      organisationId: undefined,
-      eventId: undefined,
-      appId: undefined,
-    });
+    expect(result.current.consistentScope).toBeNull();
   }, TEST_TIMEOUT);
 
   it('calls useResolvedScope with supabase and selected ids', () => {

package/src/components/DataTable/hooks/useDataTableScope.ts

@@ -34,7 +34,8 @@ export interface UseDataTableScopeOptions {
 }
 
 export interface UseDataTableScopeReturn {
-  consistentScope: DataTableScopeShape;
+  /** Scope for permission checks; null when not ready (e.g. appId still resolving). */
+  consistentScope: DataTableScopeShape | null;
 }
 
 function scopeValuesEqual(
@@ -82,10 +83,14 @@ export function useDataTableScope(
   const stableScopeRef = useRef<DataTableScopeShape>({ ...EMPTY_SCOPE });
 
   if (rawResolvedScope) {
+    // Normalise empty strings to undefined so scope never fails validateScope (engine invalid_input)
+    const org = rawResolvedScope.organisationId;
+    const ev = rawResolvedScope.eventId;
+    const app = rawResolvedScope.appId;
     const newScope: DataTableScopeShape = {
-      organisationId: rawResolvedScope.organisationId,
-      appId: rawResolvedScope.appId,
-      eventId: rawResolvedScope.eventId,
+      organisationId: typeof org === 'string' && org.trim() !== '' ? org : undefined,
+      eventId: typeof ev === 'string' && ev.trim() !== '' ? ev : undefined,
+      appId: typeof app === 'string' && app.trim() !== '' ? app : undefined,
     };
     if (!scopeValuesEqual(stableScopeRef.current, newScope)) {
       stableScopeRef.current = { ...newScope };
@@ -105,19 +110,16 @@ export function useDataTableScope(
     (scopeLoading && !canProceedWithoutFullScope);
 
   const hasScopeValues =
-    stableScope.organisationId ||
-    stableScope.appId ||
-    stableScope.eventId ||
-    canProceedWithoutFullScope;
+    !!stableScope.organisationId ||
+    !!stableScope.appId ||
+    !!stableScope.eventId;
 
-  const effectiveScope: DataTableScopeShape | null = !shouldWaitForScope &&
-    hasScopeValues
+  // Never pass EMPTY_SCOPE to useCan — validateScope requires at least one of org/event/app.
+  const effectiveScope: DataTableScopeShape | null = !shouldWaitForScope && hasScopeValues
     ? stableScope
-    : canProceedWithoutFullScope
-      ? { ...EMPTY_SCOPE }
-      : null;
+    : null;
 
-  const consistentScope = effectiveScope ?? { ...EMPTY_SCOPE };
+  const consistentScope: DataTableScopeShape | null = effectiveScope;
 
   return { consistentScope };
 }