@rebasepro/plugin-insights 0.2.4 → 0.2.5

package/dist/types/src/types/backend_hooks.d.ts

@@ -1,4 +1,4 @@
-import type { AdminUser, AdminRole } from "../controllers/client";
+import type { AdminUser } from "../controllers/client";
 /**
  * Context passed to every backend hook.
  * Provides information about the request that triggered the hook.
@@ -64,19 +64,6 @@ export interface UserHooks {
      */
     afterDelete?(userId: string, context: BackendHookContext): void | Promise<void>;
 }
-/**
- * Hooks for intercepting Admin Role data at the API boundary.
- * @group Backend Hooks
- */
-export interface RoleHooks {
-    /**
-     * Transform a role record after it's read from the database,
-     * before it's returned to the client.
-     *
-     * Return the modified role, or `null` to filter it out entirely.
-     */
-    afterRead?(role: AdminRole, context: BackendHookContext): AdminRole | null | Promise<AdminRole | null>;
-}
 /**
  * Hooks for intercepting collection entity data at the REST API boundary.
  *
@@ -146,7 +133,7 @@ export interface DataHooks {
  * These hooks run server-side after database operations complete and before
  * API responses are sent.
  *
- * - `users` / `roles` — intercept admin user and role management endpoints
+ * - `users` — intercept admin user management endpoints
  * - `data` — intercept ALL collection entity data flowing through the REST API
  *
  * `data` hooks complement per-collection `EntityCallbacks`. Entity callbacks
@@ -180,8 +167,6 @@ export interface DataHooks {
 export interface BackendHooks {
     /** Hooks for intercepting user management data */
     users?: UserHooks;
-    /** Hooks for intercepting role management data */
-    roles?: RoleHooks;
     /** Hooks for intercepting ALL collection entity data via the REST API */
     data?: DataHooks;
 }

package/dist/types/src/types/properties.d.ts

@@ -1,6 +1,6 @@
 import type { ComponentRef } from "./component_ref";
-import type { EntityReference, EntityRelation, EntityValues, GeoPoint, Entity, Vector } from "./entities";
-import type { Relation, JoinStep, OnAction } from "./relations";
+import type { Entity, EntityReference, EntityRelation, EntityValues, GeoPoint, Vector } from "./entities";
+import type { JoinStep, OnAction, Relation } from "./relations";
 import type { EntityCollection, FilterValues } from "./collections";
 import type { ColorKey, ColorScheme } from "./chips";
 import type { AuthController } from "../controllers/auth";
@@ -185,7 +185,7 @@ export interface StringProperty extends BaseProperty {
      * Optional database column type. If not set, it defaults to `varchar` or `uuid` depending on `isId` configuration.
      * Use `text` for strings with unbound length, `char` for fixed-length strings, or `varchar` for variable-length strings with a limit.
      */
-    columnType?: "varchar" | "text" | "char";
+    columnType?: "varchar" | "text" | "char" | "uuid";
     /**
      * Rules for validating this property
      */
@@ -541,9 +541,11 @@ export interface ArrayProperty extends BaseProperty {
     ui?: ArrayUIConfig;
     type: "array";
     /**
-     * Optional database column type. Defaults to `jsonb`.
+     * Optional database column type. By default, maps to a native Postgres array
+     * (e.g. `text[]`, `integer[]`/`numeric[]`, `boolean[]`) if the element type
+     * is a primitive, otherwise defaults to `jsonb`.
      */
-    columnType?: "json" | "jsonb";
+    columnType?: "json" | "jsonb" | "text[]" | "integer[]" | "boolean[]" | "numeric[]";
     /**
      * The property of this array.
      * You can specify any property (except another Array property)

package/dist/types/src/types/user_management_delegate.d.ts

@@ -1,4 +1,4 @@
-import { Role, User } from "../users";
+import type { User } from "../users";
 /**
  * Result of creating a new user via admin flow.
  * Contains the created user plus information about how credentials were delivered.
@@ -15,56 +15,46 @@ export interface UserCreationResult<USER extends User = User> {
     temporaryPassword?: string;
 }
 /**
- * Delegate to manage users, roles, and their permissions.
- * This interface allows the CMS to be completely agnostic of the underlying
- * authentication provider or backend.
+ * Delegate to manage auth-specific user operations.
+ *
+ * This interface allows the CMS to be agnostic of the underlying
+ * authentication provider or backend. User/role CRUD is now handled
+ * by the collection system; this delegate only exposes auth-specific
+ * operations (password hashing, invitations, bootstrap).
  *
  * @group Models
  */
 export interface UserManagementDelegate<USER extends User = User> {
     /**
-     * Are the users and roles currently being fetched?
+     * Are auth-related operations currently loading?
      */
     loading: boolean;
     /**
-     * List of users managed by the CMS.
+     * In-memory list of users (used for client-side filtering fallback).
      */
-    users: USER[];
+    users?: USER[];
     /**
-     * Optional error if users failed to load.
+     * Error from fetching the users list, if any.
      */
     usersError?: Error;
     /**
-     * Function to get a user by its uid. This is used to show
-     * user information when assigning ownership of an entity.
-     * @param uid
+     * Look up a single user by UID from the in-memory cache.
      */
-    getUser: (uid: string) => USER | null;
+    getUser?: (uid: string) => USER | null;
     /**
-     * Search users with server-side pagination.
-     * When provided, the CMS will use this for the users table
-     * instead of loading all users into memory.
+     * Server-side user search with pagination.
      */
-    searchUsers?: (options: {
+    searchUsers?: (params: {
         search?: string;
         limit?: number;
         offset?: number;
-        orderBy?: string;
-        orderDir?: "asc" | "desc";
-        roleId?: string;
     }) => Promise<{
         users: USER[];
         total: number;
     }>;
-    /**
-     * Save a user (create or update)
-     * @param user
-     */
-    saveUser?: (user: USER) => Promise<USER>;
     /**
      * Create a new user with invitation/password generation support.
      * Returns additional info about how the credentials were delivered.
-     * Falls back to saveUser if not provided.
      */
     createUser?: (user: USER) => Promise<UserCreationResult<USER>>;
     /**
@@ -73,42 +63,15 @@ export interface UserManagementDelegate<USER extends User = User> {
      * or a flag indicating an email invitation was sent.
      */
     resetPassword?: (user: USER) => Promise<UserCreationResult<USER>>;
-    /**
-     * Delete a user
-     * @param user
-     */
-    deleteUser?: (user: USER) => Promise<void>;
-    /**
-     * List of roles defined in the CMS.
-     */
-    roles?: Role[];
-    /**
-     * Optional error if roles failed to load.
-     */
-    rolesError?: Error;
-    /**
-     * Save a role (create or update)
-     * @param role
-     */
-    saveRole?: (role: Role) => Promise<void>;
-    /**
-     * Delete a role
-     * @param role
-     */
-    deleteRole?: (role: Role) => Promise<void>;
     /**
      * Is the currently logged in user an admin?
      */
     isAdmin?: boolean;
-    /**
-     * If true, the UI will allow the user to create the default roles (admin, editor, viewer).
-     */
-    allowDefaultRolesCreation?: boolean;
     /**
      * Optionally define roles for a given user. This is useful when the roles
      * are coming from a separate provider than the one issuing the tokens.
      */
-    defineRolesFor?: (user: USER) => Promise<Role[] | undefined> | Role[] | undefined;
+    defineRolesFor?: (user: USER) => Promise<string[] | undefined> | string[] | undefined;
     /**
      * Whether any admin users exist. Used by the bootstrap banner to decide
      * whether to prompt.  Populated via a lightweight check (e.g. `limit=1`

package/dist/types/src/users/index.d.ts

@@ -1,2 +1 @@
 export * from "./user";
-export * from "./roles";

package/dist/types/src/users/user.d.ts

@@ -35,7 +35,6 @@ export type User = {
     readonly isAnonymous: boolean;
     /**
      * Role IDs assigned to this user (e.g. ["admin", "editor"]).
-     * These are plain string IDs — use the UserManagementDelegate to look up full Role objects.
      */
     roles?: string[];
     /**

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@rebasepro/plugin-insights",
   "type": "module",
-  "version": "0.2.4",
+  "version": "0.2.5",
   "main": "./dist/index.umd.js",
   "module": "./dist/index.es.js",
   "types": "./dist/index.d.ts",
@@ -16,9 +16,9 @@
     "./package.json": "./package.json"
   },
   "dependencies": {
-    "@rebasepro/core": "0.2.4",
-    "@rebasepro/types": "0.2.4",
-    "@rebasepro/ui": "0.2.4"
+    "@rebasepro/types": "0.2.5",
+    "@rebasepro/ui": "0.2.5",
+    "@rebasepro/core": "0.2.5"
   },
   "peerDependencies": {
     "react": ">=19.0.0",

package/src/components/InsightWidget.tsx

@@ -1,14 +1,36 @@
 import React from "react";
-import type { InsightDefinition, DataRow } from "../types";
+import type { InsightDefinition, DataRow, ScorecardConfig } from "../types";
 import { useInsightsData } from "../engine/useInsightsData";
 import { InsightsScorecardView } from "./InsightsScorecardView";
 import { InsightWidgetSkeleton } from "./InsightWidgetSkeleton";
 
+/**
+ * Compute a deterministic fixed height for a standard scorecard based
+ * on which optional elements the config declares.  This eliminates
+ * layout shift between skeleton and loaded states.
+ *
+ * Breakdown (non-compact, non-small):
+ *   py-4 padding:     16 + 16 = 32
+ *   title row:        16.5  (text-xs leading-snug)
+ *   mb-2 margin:      8
+ *   value:            30    (text-2xl leading-tight)
+ *   ---
+ *   base:             86.5
+ *   + dateRange:      +16   (14px text + 2px mt-0.5)
+ *   + comparison:     +20   (16px text + 4px mt-1)
+ */
+function computeFixedHeight(config: ScorecardConfig): number {
+    let h = 86.5; // base: padding + title + mb-2 + value
+    if (config.dateRange) h += 16;
+    if (config.comparison) h += 20;
+    return Math.ceil(h);
+}
+
 /**
  * Single insight widget orchestrator.
  *
- * Fetches data via the engine, renders the scorecard visualization,
- * and manages loading/error states.
+ * Wraps skeleton and loaded states in a fixed-height container
+ * (computed from the scorecard config) to prevent layout shift.
  *
  * All theme-awareness is handled via Tailwind `dark:` classes.
  */
@@ -30,14 +52,19 @@ export function InsightWidget({
 }) {
     const { data, loading, error } = useInsightsData(definition, { path, collectionSlug, parentCollectionSlugs });
 
+    // For non-compact, non-embedded standard scorecards, use a fixed height
+    // derived from the config to prevent layout shift between skeleton → loaded.
+    const fixedHeight = (!compact && !embedded) ? computeFixedHeight(definition.scorecard) : undefined;
+
     if (loading) {
-        return <InsightWidgetSkeleton config={definition.scorecard} compact={compact} embedded={embedded} />;
+        return <InsightWidgetSkeleton config={definition.scorecard} compact={compact} embedded={embedded} fixedHeight={fixedHeight} />;
     }
 
     if (error) {
         return (
             <div
                 className={`text-red-500/70 dark:text-red-400/70 text-[0.8125rem] ${embedded ? "px-5 py-4 h-full" : `rounded-lg bg-red-500/5 dark:bg-red-400/5 border border-red-500/10 dark:border-red-400/10 ${compact ? "px-3.5 py-3" : "px-5 py-4"}`}`}
+                style={fixedHeight ? { height: fixedHeight } : undefined}
             >
                 <div className="font-semibold mb-1">{definition.title}</div>
                 <div>{error.message}</div>
@@ -49,6 +76,7 @@ export function InsightWidget({
         return (
             <div
                 className={`text-surface-400 dark:text-surface-500 text-[0.8125rem] ${embedded ? "px-5 py-4 h-full" : `rounded-lg bg-surface-100 dark:bg-surface-800 border border-surface-200 dark:border-surface-700 ${compact ? "px-3.5 py-3" : "px-5 py-4"}`}`}
+                style={fixedHeight ? { height: fixedHeight } : undefined}
             >
                 {definition.title} — No data
             </div>
@@ -62,6 +90,7 @@ export function InsightWidget({
             title={definition.title}
             compact={compact}
             embedded={embedded}
+            fixedHeight={fixedHeight}
         />
     );
 }

package/src/components/InsightWidgetSkeleton.tsx

@@ -21,12 +21,15 @@ export function InsightWidgetSkeleton({
     config,
     compact = false,
     embedded = false,
+    fixedHeight,
 }: {
     /** Scorecard config — used to match optional elements (comparison, dateRange, icon). */
     config: ScorecardConfig;
     compact?: boolean;
     /** When true, skip own border since the parent card provides it. */
     embedded?: boolean;
+    /** Explicit height to prevent layout shift between skeleton → loaded. */
+    fixedHeight?: number;
 }) {
     const hasComparison = Boolean(config.comparison);
     const hasIcon = Boolean(config.icon);
@@ -74,6 +77,7 @@ export function InsightWidgetSkeleton({
         hasIcon={hasIcon}
         hasDateRange={hasDateRange}
         embedded={embedded}
+        fixedHeight={fixedHeight}
     />;
 }
 
@@ -112,11 +116,13 @@ function StandardSkeleton({
     hasIcon,
     hasDateRange,
     embedded,
+    fixedHeight,
 }: {
     hasComparison: boolean;
     hasIcon: boolean;
     hasDateRange: boolean;
     embedded: boolean;
+    fixedHeight?: number;
 }) {
     const containerRef = useRef<HTMLDivElement>(null);
     const [containerWidth, setContainerWidth] = useState<number | null>(null);
@@ -158,7 +164,7 @@ function StandardSkeleton({
         <div
             ref={containerRef}
             className={cls("animate-pulse", baseClass)}
-            style={embedded ? undefined : { minHeight: isSmall ? 68 : 92 }}
+            style={embedded ? undefined : fixedHeight ? { height: fixedHeight } : { minHeight: isSmall ? 68 : 92 }}
         >
             {/* Title row — identical flex structure to InsightsScorecardView */}
             <div className={`flex items-center justify-between ${isSmall ? "mb-1" : "mb-2"}`}>

package/src/components/InsightsScorecardView.tsx

@@ -39,6 +39,7 @@ export function InsightsScorecardView({
     title,
     compact = false,
     embedded = false,
+    fixedHeight,
 }: {
     config: ScorecardConfig;
     data: DataRow;
@@ -46,6 +47,8 @@ export function InsightsScorecardView({
     compact?: boolean;
     /** When true, skip own border/bg since the parent card provides them. */
     embedded?: boolean;
+    /** Explicit height to prevent layout shift between skeleton → loaded. */
+    fixedHeight?: number;
 }) {
     const containerRef = useRef<HTMLDivElement>(null);
     const [containerWidth, setContainerWidth] = useState<number | null>(null);
@@ -124,7 +127,7 @@ export function InsightsScorecardView({
         : cls("rounded-lg flex flex-col min-w-0 bg-transparent border", defaultBorderMixin, isSmall ? "px-3.5 py-3" : "px-5 py-4");
 
     return (
-        <div ref={containerRef} className={baseClass} style={embedded ? undefined : { minHeight: isSmall ? 68 : 92 }}>
+        <div ref={containerRef} className={baseClass} style={embedded ? undefined : fixedHeight ? { height: fixedHeight } : { minHeight: isSmall ? 68 : 92 }}>
             {/* Title row */}
             <div className={`flex items-center justify-between ${isSmall ? "mb-1" : "mb-2"}`}>
                 <div className="flex flex-col min-w-0">

package/dist/types/src/users/roles.d.ts

@@ -1,14 +0,0 @@
-export type Role = {
-    /**
-     * ID of the role
-     */
-    id: string;
-    /**
-     * Name of the role
-     */
-    name: string;
-    /**
-     * If this flag is true, the user can perform any action
-     */
-    isAdmin?: boolean;
-};