@memberjunction/react-runtime 2.112.0 → 2.113.1

package/src/runtime/component-hierarchy.ts

@@ -4,12 +4,17 @@
  * @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/global';
+import { UserInfo, Metadata, LogStatus, GetProductionStatus } from '@memberjunction/core';
 import { ComponentLibraryEntity } from '@memberjunction/core-entities';
 
 /**
@@ -65,27 +70,30 @@ 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) {
@@ -109,16 +117,25 @@ 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[] = [];
@@ -128,7 +145,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
       });
     }
 
@@ -136,34 +153,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) => {
@@ -175,7 +192,7 @@ export class ComponentHierarchyRegistrar {
               }
             });
           }
-
+          
           return { success: true };
         } else {
           return {
@@ -183,8 +200,8 @@ export class ComponentHierarchyRegistrar {
             error: {
               componentName: spec.name,
               error: result.error?.message || 'Unknown compilation error',
-              phase: 'compilation',
-            },
+              phase: 'compilation'
+            }
           };
         }
       } catch (error) {
@@ -193,21 +210,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);
@@ -218,7 +235,7 @@ export class ComponentHierarchyRegistrar {
           continue;
         }
       }
-
+      
       const result = await compileOnly(spec);
       if (!result.success) {
         errors.push(result.error!);
@@ -226,12 +243,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) {
@@ -244,11 +261,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) {
@@ -256,13 +273,18 @@ 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);
     }
 
@@ -271,7 +293,7 @@ export class ComponentHierarchyRegistrar {
       registeredComponents,
       errors,
       warnings,
-      resolvedSpec: resolvedRootSpec,
+      resolvedSpec: resolvedRootSpec
     };
   }
 
@@ -298,7 +320,7 @@ export class ComponentHierarchyRegistrar {
       if (!spec.code) {
         return {
           success: true,
-          error: undefined,
+          error: undefined
         };
       }
 
@@ -310,13 +332,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;
@@ -327,7 +349,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) || []
         });
       }
 
@@ -338,7 +360,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);
@@ -349,8 +371,8 @@ export class ComponentHierarchyRegistrar {
           error: {
             componentName: spec.name,
             error: compilationResult.error?.message || 'Unknown compilation error',
-            phase: 'compilation',
-          },
+            phase: 'compilation'
+          }
         };
       }
 
@@ -374,23 +396,29 @@ 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'
+        }
       };
     }
   }
@@ -417,7 +445,7 @@ export class ComponentHierarchyRegistrar {
         namespace: options.namespace,
         version: options.version,
         allowOverride: options.allowOverride,
-        allLibraries: options.allLibraries,
+        allLibraries: options.allLibraries
       });
 
       if (childResult.success) {
@@ -434,7 +462,13 @@ 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
+        );
       }
     }
   }
@@ -486,7 +520,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}`);
     });
   });
@@ -501,9 +535,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));
   });
 
@@ -516,17 +550,20 @@ 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/global';
+import { LogStatus, GetProductionStatus } from '@memberjunction/core';
 
 /**
  * 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,20 +229,22 @@ 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]]) {
@@ -268,21 +270,19 @@ export function createPropsTransformer(transformations: Record<string, (value: a
  * @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(callbacks: ComponentCallbacks, componen
   }
 
   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,9 +329,14 @@ export function wrapCallbacksWithLogging(callbacks: ComponentCallbacks, componen
  */
 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;
@@ -341,4 +346,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/global';
+import { UserInfo } from '@memberjunction/core';
 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,6 +53,7 @@ export interface CompileOptions {
   allLibraries: ComponentLibraryEntity[];
 }
 
+
 /**
  * Registry entry for a compiled component
  */
@@ -229,4 +230,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';