@memberjunction/react-runtime 2.111.1 → 2.112.0

package/src/runtime/component-hierarchy.ts

@@ -4,17 +4,12 @@
  * @module @memberjunction/react-runtime/hierarchy
  */
 
-import {
-  CompilationResult,
-  CompileOptions,
-  RuntimeContext,
-  CompiledComponent
-} from '../types';
+import { CompilationResult, CompileOptions, RuntimeContext, CompiledComponent } from '../types';
 import { ComponentCompiler } from '../compiler';
 import { ComponentRegistry } from '../registry';
 
 import { ComponentSpec, ComponentStyles } from '@memberjunction/interactive-component-types';
-import { UserInfo, Metadata, LogStatus, GetProductionStatus } from '@memberjunction/core';
+import { UserInfo, Metadata, LogStatus, GetProductionStatus } from '@memberjunction/global';
 import { ComponentLibraryEntity } from '@memberjunction/core-entities';
 
 /**
@@ -70,30 +65,27 @@ export class ComponentHierarchyRegistrar {
     private registry: ComponentRegistry,
     private runtimeContext: RuntimeContext
   ) {}
-  
+
   /**
    * Fetches a component specification from an external registry
    */
-  private async fetchExternalComponent(
-    spec: ComponentSpec,
-    contextUser?: UserInfo
-  ): Promise<ComponentSpec | null> {
+  private async fetchExternalComponent(spec: ComponentSpec, contextUser?: UserInfo): Promise<ComponentSpec | null> {
     try {
       const provider = Metadata?.Provider;
       if (!provider || !(provider as any).ExecuteGQL) {
         console.warn('⚠️ [ComponentHierarchyRegistrar] No GraphQL provider available for external registry fetch');
         return null;
       }
-      
+
       // Dynamically import the GraphQL client to avoid circular dependencies
       const { GraphQLComponentRegistryClient } = await import('@memberjunction/graphql-dataprovider');
       const graphQLClient = new GraphQLComponentRegistryClient(provider as any);
-      
+
       const fullSpec = await graphQLClient.GetRegistryComponent({
         registryName: spec.registry!,
         namespace: spec.namespace || 'Global',
         name: spec.name,
-        version: spec.version || 'latest'
+        version: spec.version || 'latest',
       });
 
       if (fullSpec && fullSpec.code) {
@@ -117,25 +109,16 @@ export class ComponentHierarchyRegistrar {
    * @param options - Registration options
    * @returns Registration result with details about success/failures
    */
-  async registerHierarchy(
-    rootSpec: ComponentSpec,
-    options: HierarchyRegistrationOptions
-  ): Promise<HierarchyRegistrationResult> {
+  async registerHierarchy(rootSpec: ComponentSpec, options: HierarchyRegistrationOptions): Promise<HierarchyRegistrationResult> {
     // If this is an external registry component without code, fetch it first
     let resolvedRootSpec = rootSpec;
     if (rootSpec.location === 'registry' && rootSpec.registry && !rootSpec.code) {
       if (!GetProductionStatus()) {
         LogStatus(`🌐 [ComponentHierarchyRegistrar] Fetching external registry component: ${rootSpec.registry}/${rootSpec.name}`);
       }
-      resolvedRootSpec = await this.fetchExternalComponent(rootSpec, options.contextUser) || rootSpec;
+      resolvedRootSpec = (await this.fetchExternalComponent(rootSpec, options.contextUser)) || rootSpec;
     }
-    const {
-      styles,
-      namespace = 'Global',
-      version = 'v1',
-      continueOnError = true,
-      allowOverride = true
-    } = options;
+    const { styles, namespace = 'Global', version = 'v1', continueOnError = true, allowOverride = true } = options;
 
     const registeredComponents: string[] = [];
     const errors: ComponentRegistrationError[] = [];
@@ -145,7 +128,7 @@ export class ComponentHierarchyRegistrar {
       LogStatus('🌳 ComponentHierarchyRegistrar.registerHierarchy:', undefined, {
         rootComponent: resolvedRootSpec.name,
         hasLibraries: !!(resolvedRootSpec.libraries && resolvedRootSpec.libraries.length > 0),
-        libraryCount: resolvedRootSpec.libraries?.length || 0
+        libraryCount: resolvedRootSpec.libraries?.length || 0,
       });
     }
 
@@ -153,34 +136,34 @@ export class ComponentHierarchyRegistrar {
     const compiledMap = new Map<string, CompiledComponent>();
     const specMap = new Map<string, ComponentSpec>();
     const allLoadedLibraries = new Map<string, any>(); // Track all loaded libraries
-    
+
     // Helper to compile a component without calling its factory
     const compileOnly = async (spec: ComponentSpec): Promise<{ success: boolean; error?: ComponentRegistrationError }> => {
       if (!spec.code) return { success: true };
-      
+
       try {
         // Filter out invalid library entries before compilation
-        const validLibraries = spec.libraries?.filter(lib => {
+        const validLibraries = spec.libraries?.filter((lib) => {
           if (!lib || typeof lib !== 'object') return false;
           if (!lib.name || lib.name === 'unknown' || lib.name === 'null' || lib.name === 'undefined') return false;
           if (!lib.globalVariable || lib.globalVariable === 'undefined' || lib.globalVariable === 'null') return false;
           return true;
         });
-        
+
         const compileOptions: CompileOptions = {
           componentName: spec.name,
           componentCode: spec.code,
           styles,
           libraries: validLibraries,
           dependencies: spec.dependencies,
-          allLibraries: options.allLibraries
+          allLibraries: options.allLibraries,
         };
-        
+
         const result = await this.compiler.compile(compileOptions);
         if (result.success && result.component) {
           compiledMap.set(spec.name, result.component);
           specMap.set(spec.name, spec);
-          
+
           // Extract and accumulate loaded libraries from the compilation
           if (result.loadedLibraries) {
             result.loadedLibraries.forEach((value, key) => {
@@ -192,7 +175,7 @@ export class ComponentHierarchyRegistrar {
               }
             });
           }
-          
+
           return { success: true };
         } else {
           return {
@@ -200,8 +183,8 @@ export class ComponentHierarchyRegistrar {
             error: {
               componentName: spec.name,
               error: result.error?.message || 'Unknown compilation error',
-              phase: 'compilation'
-            }
+              phase: 'compilation',
+            },
           };
         }
       } catch (error) {
@@ -210,21 +193,21 @@ export class ComponentHierarchyRegistrar {
           error: {
             componentName: spec.name,
             error: error instanceof Error ? error.message : String(error),
-            phase: 'compilation'
-          }
+            phase: 'compilation',
+          },
         };
       }
     };
-    
+
     // Compile all components in hierarchy
     const compileQueue = [resolvedRootSpec];
     const visited = new Set<string>();
-    
+
     while (compileQueue.length > 0) {
       let spec = compileQueue.shift()!;
       if (visited.has(spec.name)) continue;
       visited.add(spec.name);
-      
+
       // If this is an external registry component without code, fetch it first
       if (spec.location === 'registry' && spec.registry && !spec.code) {
         const fetched = await this.fetchExternalComponent(spec, options.contextUser);
@@ -235,7 +218,7 @@ export class ComponentHierarchyRegistrar {
           continue;
         }
       }
-      
+
       const result = await compileOnly(spec);
       if (!result.success) {
         errors.push(result.error!);
@@ -243,12 +226,12 @@ export class ComponentHierarchyRegistrar {
           return { success: false, registeredComponents, errors, warnings, resolvedSpec: resolvedRootSpec };
         }
       }
-      
+
       if (spec.dependencies) {
         compileQueue.push(...spec.dependencies);
       }
     }
-    
+
     // Add all accumulated libraries to runtime context
     if (allLoadedLibraries.size > 0) {
       if (!this.runtimeContext.libraries) {
@@ -261,11 +244,11 @@ export class ComponentHierarchyRegistrar {
         }
       });
     }
-    
+
     // PHASE 2: Execute all factories with components available
     for (const [name, compiled] of compiledMap) {
       const spec = specMap.get(name)!;
-      
+
       // Build components object from all registered components
       const components: Record<string, any> = {};
       for (const [depName, depCompiled] of compiledMap) {
@@ -273,18 +256,13 @@ export class ComponentHierarchyRegistrar {
         const depObject = depCompiled.factory(this.runtimeContext, styles);
         components[depName] = depObject.component;
       }
-      
+
       // Now call factory with components available
       const componentObject = compiled.factory(this.runtimeContext, styles, components);
-      
+
       // Register in registry
-      this.registry.register(
-        spec.name,
-        componentObject,
-        spec.namespace || namespace,
-        version
-      );
-      
+      this.registry.register(spec.name, componentObject, spec.namespace || namespace, version);
+
       registeredComponents.push(spec.name);
     }
 
@@ -293,7 +271,7 @@ export class ComponentHierarchyRegistrar {
       registeredComponents,
       errors,
       warnings,
-      resolvedSpec: resolvedRootSpec
+      resolvedSpec: resolvedRootSpec,
     };
   }
 
@@ -320,7 +298,7 @@ export class ComponentHierarchyRegistrar {
       if (!spec.code) {
         return {
           success: true,
-          error: undefined
+          error: undefined,
         };
       }
 
@@ -332,13 +310,13 @@ export class ComponentHierarchyRegistrar {
           error: {
             componentName: spec.name,
             error: `Component already registered in ${namespace}/${version}`,
-            phase: 'registration'
-          }
+            phase: 'registration',
+          },
         };
       }
 
       // Filter out invalid library entries before compilation
-      const validLibraries = spec.libraries?.filter(lib => {
+      const validLibraries = spec.libraries?.filter((lib) => {
         if (!lib || typeof lib !== 'object') return false;
         if (!lib.name || lib.name === 'unknown' || lib.name === 'null' || lib.name === 'undefined') return false;
         if (!lib.globalVariable || lib.globalVariable === 'undefined' || lib.globalVariable === 'null') return false;
@@ -349,7 +327,7 @@ export class ComponentHierarchyRegistrar {
         LogStatus(`🔧 Compiling component ${spec.name} with libraries:`, undefined, {
           originalCount: spec.libraries?.length || 0,
           filteredCount: validLibraries?.length || 0,
-          libraries: validLibraries?.map(l => l.name) || []
+          libraries: validLibraries?.map((l) => l.name) || [],
         });
       }
 
@@ -360,7 +338,7 @@ export class ComponentHierarchyRegistrar {
         styles,
         libraries: validLibraries, // Pass along filtered library dependencies
         dependencies: spec.dependencies, // Pass along child component dependencies
-        allLibraries: options.allLibraries
+        allLibraries: options.allLibraries,
       };
 
       const compilationResult = await this.compiler.compile(compileOptions);
@@ -371,8 +349,8 @@ export class ComponentHierarchyRegistrar {
           error: {
             componentName: spec.name,
             error: compilationResult.error?.message || 'Unknown compilation error',
-            phase: 'compilation'
-          }
+            phase: 'compilation',
+          },
         };
       }
 
@@ -396,29 +374,23 @@ export class ComponentHierarchyRegistrar {
         LogStatus(`🏭 Calling factory for ${spec.name} with runtime context:`, undefined, {
           hasReact: !!this.runtimeContext.React,
           hasReactDOM: !!this.runtimeContext.ReactDOM,
-          libraryCount: Object.keys(this.runtimeContext.libraries || {}).length
+          libraryCount: Object.keys(this.runtimeContext.libraries || {}).length,
         });
       }
       const componentObject = compilationResult.component!.factory(this.runtimeContext, styles);
 
       // Register the full ComponentObject (not just the React component)
-      this.registry.register(
-        spec.name,
-        componentObject,
-        spec.namespace || namespace,
-        version
-      );
+      this.registry.register(spec.name, componentObject, spec.namespace || namespace, version);
 
       return { success: true };
-
     } catch (error) {
       return {
         success: false,
         error: {
           componentName: spec.name,
           error: error instanceof Error ? error.message : String(error),
-          phase: 'registration'
-        }
+          phase: 'registration',
+        },
       };
     }
   }
@@ -445,7 +417,7 @@ export class ComponentHierarchyRegistrar {
         namespace: options.namespace,
         version: options.version,
         allowOverride: options.allowOverride,
-        allLibraries: options.allLibraries
+        allLibraries: options.allLibraries,
       });
 
       if (childResult.success) {
@@ -462,13 +434,7 @@ export class ComponentHierarchyRegistrar {
       // Register nested children recursively
       const nestedChildren = child.dependencies || [];
       if (nestedChildren.length > 0) {
-        await this.registerChildComponents(
-          nestedChildren,
-          options,
-          registeredComponents,
-          errors,
-          warnings
-        );
+        await this.registerChildComponents(nestedChildren, options, registeredComponents, errors, warnings);
       }
     }
   }
@@ -520,7 +486,7 @@ export function validateComponentSpec(spec: ComponentSpec): string[] {
   const children = spec.dependencies || [];
   children.forEach((child, index) => {
     const childErrors = validateComponentSpec(child);
-    childErrors.forEach(error => {
+    childErrors.forEach((error) => {
       errors.push(`Child ${index} (${child.name || 'unnamed'}): ${error}`);
     });
   });
@@ -535,9 +501,9 @@ export function validateComponentSpec(spec: ComponentSpec): string[] {
  */
 export function flattenComponentHierarchy(rootSpec: ComponentSpec): ComponentSpec[] {
   const components: ComponentSpec[] = [rootSpec];
-  
+
   const children = rootSpec.dependencies || [];
-  children.forEach(child => {
+  children.forEach((child) => {
     components.push(...flattenComponentHierarchy(child));
   });
 
@@ -550,20 +516,17 @@ export function flattenComponentHierarchy(rootSpec: ComponentSpec): ComponentSpe
  * @param includeEmpty - Whether to include components without code
  * @returns Total component count
  */
-export function countComponentsInHierarchy(
-  rootSpec: ComponentSpec,
-  includeEmpty: boolean = false
-): number {
+export function countComponentsInHierarchy(rootSpec: ComponentSpec, includeEmpty: boolean = false): number {
   let count = 0;
-  
+
   if (includeEmpty || rootSpec.code) {
     count = 1;
   }
 
   const children = rootSpec.dependencies || [];
-  children.forEach(child => {
+  children.forEach((child) => {
     count += countComponentsInHierarchy(child, includeEmpty);
   });
 
   return count;
-}
+}

package/src/runtime/prop-builder.ts

@@ -7,7 +7,7 @@
 import { ComponentStyles, ComponentCallbacks } from '@memberjunction/interactive-component-types';
 import { ComponentProps } from '../types';
 import { Subject, debounceTime, Subscription } from 'rxjs';
-import { LogStatus, GetProductionStatus } from '@memberjunction/core';
+import { LogStatus, GetProductionStatus } from '@memberjunction/global';
 
 /**
  * Options for building component props
@@ -43,7 +43,7 @@ export function buildComponentProps(
   callbacks: ComponentCallbacks = {
     OpenEntityRecord: () => {},
     RegisterMethod: () => {},
-    CreateSimpleNotification: () => {}
+    CreateSimpleNotification: () => {},
   },
   components: Record<string, any> = {},
   styles?: ComponentStyles,
@@ -54,7 +54,7 @@ export function buildComponentProps(
     validate = true,
     transformData,
     transformState,
-    debounceUpdateUserState = 3000 // Default 3 seconds
+    debounceUpdateUserState = 3000, // Default 3 seconds
   } = options;
 
   // Transform data if transformer provided
@@ -69,7 +69,7 @@ export function buildComponentProps(
     callbacks: normalizeCallbacks(callbacks, debounceUpdateUserState),
     components,
     styles: normalizeStyles(styles),
-    onStateChanged
+    onStateChanged,
   };
 
   // Validate if enabled
@@ -97,20 +97,20 @@ const loopDetectionStates = new WeakMap<Function, LoopDetectionState>();
 // Deep equality check for objects
 function deepEqual(obj1: any, obj2: any): boolean {
   if (obj1 === obj2) return true;
-  
+
   if (!obj1 || !obj2) return false;
   if (typeof obj1 !== 'object' || typeof obj2 !== 'object') return false;
-  
+
   const keys1 = Object.keys(obj1);
   const keys2 = Object.keys(obj2);
-  
+
   if (keys1.length !== keys2.length) return false;
-  
+
   for (const key of keys1) {
     if (!keys2.includes(key)) return false;
     if (!deepEqual(obj1[key], obj2[key])) return false;
   }
-  
+
   return true;
 }
 
@@ -125,12 +125,12 @@ export function normalizeCallbacks(callbacks: any, debounceMs: number = 3000): C
   const normalized: ComponentCallbacks = {
     OpenEntityRecord: callbacks?.OpenEntityRecord || (() => {}),
     RegisterMethod: callbacks?.RegisterMethod || (() => {}),
-    CreateSimpleNotification: callbacks?.CreateSimpleNotification || (() => {})
+    CreateSimpleNotification: callbacks?.CreateSimpleNotification || (() => {}),
   };
 
   // Copy any additional callbacks that might exist
   if (callbacks) {
-    Object.keys(callbacks).forEach(key => {
+    Object.keys(callbacks).forEach((key) => {
       if (typeof callbacks[key] === 'function' && !normalized.hasOwnProperty(key)) {
         (normalized as any)[key] = callbacks[key];
       }
@@ -198,7 +198,7 @@ export function mergeProps(...propsList: Partial<ComponentProps>[]): ComponentPr
     utilities: {},
     callbacks: {},
     components: {},
-    styles: {} as ComponentStyles
+    styles: {} as ComponentStyles,
   };
 
   for (const props of propsList) {
@@ -229,22 +229,20 @@ export function mergeProps(...propsList: Partial<ComponentProps>[]): ComponentPr
 
   return merged;
 }
- 
+
 /**
  * Creates a props transformer function
  * @param transformations - Map of prop paths to transformer functions
  * @returns Props transformer
  */
-export function createPropsTransformer(
-  transformations: Record<string, (value: any) => any>
-): (props: ComponentProps) => ComponentProps {
+export function createPropsTransformer(transformations: Record<string, (value: any) => any>): (props: ComponentProps) => ComponentProps {
   return (props: ComponentProps) => {
     const transformed = { ...props };
 
     for (const [path, transformer] of Object.entries(transformations)) {
       const pathParts = path.split('.');
       let current: any = transformed;
-      
+
       // Navigate to the parent of the target property
       for (let i = 0; i < pathParts.length - 1; i++) {
         if (!current[pathParts[i]]) {
@@ -270,19 +268,21 @@ export function createPropsTransformer(
  * @param componentName - Component name for logging
  * @returns Wrapped callbacks
  */
-export function wrapCallbacksWithLogging(
-  callbacks: ComponentCallbacks,
-  componentName: string
-): ComponentCallbacks {
+export function wrapCallbacksWithLogging(callbacks: ComponentCallbacks, componentName: string): ComponentCallbacks {
   const wrapped: ComponentCallbacks = {
     OpenEntityRecord: callbacks?.OpenEntityRecord || (() => {}),
     RegisterMethod: callbacks?.RegisterMethod || (() => {}),
-    CreateSimpleNotification: callbacks?.CreateSimpleNotification || (() => {})
+    CreateSimpleNotification: callbacks?.CreateSimpleNotification || (() => {}),
   };
 
   // Wrap any additional callbacks that might exist
-  Object.keys(callbacks).forEach(key => {
-    if (key !== 'OpenEntityRecord' && key !== 'RegisterMethod' && key !== 'CreateSimpleNotification' && typeof (callbacks as any)[key] === 'function') {
+  Object.keys(callbacks).forEach((key) => {
+    if (
+      key !== 'OpenEntityRecord' &&
+      key !== 'RegisterMethod' &&
+      key !== 'CreateSimpleNotification' &&
+      typeof (callbacks as any)[key] === 'function'
+    ) {
       (wrapped as any)[key] = (...args: any[]) => {
         if (!GetProductionStatus()) {
           LogStatus(`[${componentName}] ${key} called with args:`, undefined, args);
@@ -311,7 +311,7 @@ export function wrapCallbacksWithLogging(
   }
 
   if (callbacks.CreateSimpleNotification) {
-    wrapped.CreateSimpleNotification = (message: string, style: "none" | "success" | "error" | "warning" | "info", hideAfter?: number) => {
+    wrapped.CreateSimpleNotification = (message: string, style: 'none' | 'success' | 'error' | 'warning' | 'info', hideAfter?: number) => {
       if (!GetProductionStatus()) {
         LogStatus(`[${componentName}] CreateSimpleNotification called:`, undefined, { message, style, hideAfter });
       }
@@ -329,14 +329,9 @@ export function wrapCallbacksWithLogging(
  */
 export function extractPropPaths(componentCode: string): string[] {
   const paths: string[] = [];
-  
+
   // Simple regex patterns to find prop access
-  const patterns = [
-    /props\.data\.(\w+)/g,
-    /props\.userState\.(\w+)/g,
-    /props\.utilities\.(\w+)/g,
-    /props\.callbacks\.(\w+)/g
-  ];
+  const patterns = [/props\.data\.(\w+)/g, /props\.userState\.(\w+)/g, /props\.utilities\.(\w+)/g, /props\.callbacks\.(\w+)/g];
 
   for (const pattern of patterns) {
     let match;
@@ -346,4 +341,4 @@ export function extractPropPaths(componentCode: string): string[] {
   }
 
   return [...new Set(paths)]; // Remove duplicates
-}
+}

package/src/types/index.ts

@@ -4,7 +4,7 @@
  * @module @memberjunction/react-runtime/types
  */
 
-import { UserInfo } from '@memberjunction/core';
+import { UserInfo } from '@memberjunction/global';
 import { ComponentLibraryEntity } from '@memberjunction/core-entities';
 import { ComponentLibraryDependency, ComponentStyles, ComponentObject } from '@memberjunction/interactive-component-types';
 
@@ -43,7 +43,7 @@ export interface CompileOptions {
 
   /** Library dependencies that the component requires */
   libraries?: ComponentLibraryDependency[];
-  
+
   /** Child component dependencies that the component requires */
   dependencies?: Array<{ name: string; code?: string }>;
 
@@ -53,7 +53,6 @@ export interface CompileOptions {
   allLibraries: ComponentLibraryEntity[];
 }
 
-
 /**
  * Registry entry for a compiled component
  */
@@ -230,4 +229,4 @@ export * from './library-config';
 export * from './dependency-types';
 
 // Re-export ComponentObject for convenience
-export { ComponentObject } from '@memberjunction/interactive-component-types';
+export { ComponentObject } from '@memberjunction/interactive-component-types';