@memberjunction/react-runtime 2.102.0 → 2.104.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@memberjunction/react-runtime",
-  "version": "2.102.0",
+  "version": "2.104.0",
   "description": "Platform-agnostic React component runtime for MemberJunction. Provides core compilation, registry, and execution capabilities for React components in any JavaScript environment.",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -27,11 +27,11 @@
   },
   "homepage": "https://github.com/MemberJunction/MJ#readme",
   "dependencies": {
-    "@memberjunction/core": "2.102.0",
-    "@memberjunction/global": "2.102.0",
-    "@memberjunction/interactive-component-types": "2.102.0",
-    "@memberjunction/core-entities": "2.102.0",
-    "@memberjunction/graphql-dataprovider": "2.102.0",
+    "@memberjunction/core": "2.100.3",
+    "@memberjunction/global": "2.100.3",
+    "@memberjunction/interactive-component-types": "2.104.0",
+    "@memberjunction/core-entities": "2.104.0",
+    "@memberjunction/graphql-dataprovider": "2.104.0",
     "@babel/standalone": "^7.23.5",
     "rxjs": "^7.8.1"
   },
@@ -47,4 +47,4 @@
     "@babel/preset-env": "^7.23.5",
     "path-browserify": "^1.0.1"
   }
-}
+}

package/src/runtime/component-hierarchy.ts

@@ -4,7 +4,7 @@
  * @module @memberjunction/react-runtime/hierarchy
  */
 
-import { 
+import {
   CompilationResult,
   CompileOptions,
   RuntimeContext,
@@ -14,7 +14,7 @@ import { ComponentCompiler } from '../compiler';
 import { ComponentRegistry } from '../registry';
 
 import { ComponentSpec, ComponentStyles } from '@memberjunction/interactive-component-types';
-import { UserInfo, Metadata } from '@memberjunction/core';
+import { UserInfo, Metadata, LogStatus, GetProductionStatus } from '@memberjunction/core';
 import { ComponentLibraryEntity } from '@memberjunction/core-entities';
 
 /**
@@ -95,9 +95,11 @@ export class ComponentHierarchyRegistrar {
         name: spec.name,
         version: spec.version || 'latest'
       });
-      
+
       if (fullSpec && fullSpec.code) {
-        console.log(`✅ [ComponentHierarchyRegistrar] Fetched external component ${spec.name} with code (${fullSpec.code.length} chars)`);
+        if (!GetProductionStatus()) {
+          LogStatus(`✅ [ComponentHierarchyRegistrar] Fetched external component ${spec.name} with code (${fullSpec.code.length} chars)`);
+        }
         return fullSpec;
       } else {
         console.warn(`⚠️ [ComponentHierarchyRegistrar] Failed to fetch external component ${spec.name} or no code`);
@@ -122,7 +124,9 @@ export class ComponentHierarchyRegistrar {
     // If this is an external registry component without code, fetch it first
     let resolvedRootSpec = rootSpec;
     if (rootSpec.location === 'registry' && rootSpec.registry && !rootSpec.code) {
-      console.log(`🌐 [ComponentHierarchyRegistrar] Fetching external registry component: ${rootSpec.registry}/${rootSpec.name}`);
+      if (!GetProductionStatus()) {
+        LogStatus(`🌐 [ComponentHierarchyRegistrar] Fetching external registry component: ${rootSpec.registry}/${rootSpec.name}`);
+      }
       resolvedRootSpec = await this.fetchExternalComponent(rootSpec, options.contextUser) || rootSpec;
     }
     const {
@@ -133,17 +137,18 @@ export class ComponentHierarchyRegistrar {
       allowOverride = true
     } = options;
 
-    console.log('🌳 ComponentHierarchyRegistrar.registerHierarchy:', {
-      rootComponent: resolvedRootSpec.name,
-      hasLibraries: !!(resolvedRootSpec.libraries && resolvedRootSpec.libraries.length > 0),
-      libraryCount: resolvedRootSpec.libraries?.length || 0,
-      libraries: resolvedRootSpec.libraries?.map(l => l.name)
-    });
-
     const registeredComponents: string[] = [];
     const errors: ComponentRegistrationError[] = [];
     const warnings: string[] = [];
 
+    if (!GetProductionStatus()) {
+      LogStatus('🌳 ComponentHierarchyRegistrar.registerHierarchy:', undefined, {
+        rootComponent: resolvedRootSpec.name,
+        hasLibraries: !!(resolvedRootSpec.libraries && resolvedRootSpec.libraries.length > 0),
+        libraryCount: resolvedRootSpec.libraries?.length || 0
+      });
+    }
+
     // PHASE 1: Compile all components first (but defer factory execution)
     const compiledMap = new Map<string, CompiledComponent>();
     const specMap = new Map<string, ComponentSpec>();
@@ -181,7 +186,9 @@ export class ComponentHierarchyRegistrar {
             result.loadedLibraries.forEach((value, key) => {
               if (!allLoadedLibraries.has(key)) {
                 allLoadedLibraries.set(key, value);
-                console.log(`📚 [registerHierarchy] Added library ${key} to accumulated libraries`);
+                if (!GetProductionStatus()) {
+                  LogStatus(`📚 [registerHierarchy] Added library ${key} to accumulated libraries`);
+                }
               }
             });
           }
@@ -249,7 +256,9 @@ export class ComponentHierarchyRegistrar {
       }
       allLoadedLibraries.forEach((value, key) => {
         this.runtimeContext.libraries![key] = value;
-        console.log(`✅ [registerHierarchy] Added ${key} to runtime context libraries`);
+        if (!GetProductionStatus()) {
+          LogStatus(`✅ [registerHierarchy] Added ${key} to runtime context libraries`);
+        }
       });
     }
     
@@ -335,7 +344,15 @@ export class ComponentHierarchyRegistrar {
         if (!lib.globalVariable || lib.globalVariable === 'undefined' || lib.globalVariable === 'null') return false;
         return true;
       });
-      
+
+      if (!GetProductionStatus()) {
+        LogStatus(`🔧 Compiling component ${spec.name} with libraries:`, undefined, {
+          originalCount: spec.libraries?.length || 0,
+          filteredCount: validLibraries?.length || 0,
+          libraries: validLibraries?.map(l => l.name) || []
+        });
+      }
+
       // Compile the component
       const compileOptions: CompileOptions = {
         componentName: spec.name,
@@ -346,12 +363,6 @@ export class ComponentHierarchyRegistrar {
         allLibraries: options.allLibraries
       };
 
-      console.log(`🔧 Compiling component ${spec.name} with libraries:`, {
-        originalCount: spec.libraries?.length || 0,
-        filteredCount: validLibraries?.length || 0,
-        validLibraries: validLibraries?.map(l => ({ name: l.name, globalVariable: l.globalVariable }))
-      });
-
       const compilationResult = await this.compiler.compile(compileOptions);
 
       if (!compilationResult.success) {
@@ -372,18 +383,22 @@ export class ComponentHierarchyRegistrar {
         }
         compilationResult.loadedLibraries.forEach((value, key) => {
           this.runtimeContext.libraries![key] = value;
-          console.log(`✅ [registerSingleComponent] Added ${key} to runtime context libraries`);
+          if (!GetProductionStatus()) {
+            LogStatus(`✅ [registerSingleComponent] Added ${key} to runtime context libraries`);
+          }
         });
       }
 
       // Call the factory to create the ComponentObject
       // IMPORTANT: We don't pass components here because child components may not be registered yet
       // Components are resolved later when the component is actually rendered
-      console.log(`🏭 Calling factory for ${spec.name} with runtime context:`, {
-        hasReact: !!this.runtimeContext.React,
-        hasReactDOM: !!this.runtimeContext.ReactDOM,
-        libraryKeys: Object.keys(this.runtimeContext.libraries || {})
-      });
+      if (!GetProductionStatus()) {
+        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
+        });
+      }
       const componentObject = compilationResult.component!.factory(this.runtimeContext, styles);
 
       // Register the full ComponentObject (not just the React component)

package/src/runtime/prop-builder.ts

@@ -7,6 +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';
 
 /**
  * Options for building component props
@@ -283,7 +284,9 @@ export function wrapCallbacksWithLogging(
   Object.keys(callbacks).forEach(key => {
     if (key !== 'OpenEntityRecord' && key !== 'RegisterMethod' && key !== 'CreateSimpleNotification' && typeof (callbacks as any)[key] === 'function') {
       (wrapped as any)[key] = (...args: any[]) => {
-        console.log(`[${componentName}] ${key} called with args:`, args);
+        if (!GetProductionStatus()) {
+          LogStatus(`[${componentName}] ${key} called with args:`, undefined, args);
+        }
         return (callbacks as any)[key](...args);
       };
     }
@@ -291,21 +294,27 @@ export function wrapCallbacksWithLogging(
 
   if (callbacks.OpenEntityRecord) {
     wrapped.OpenEntityRecord = (entityName: string, key: any) => {
-      console.log(`[${componentName}] OpenEntityRecord called:`, { entityName, key });
+      if (!GetProductionStatus()) {
+        LogStatus(`[${componentName}] OpenEntityRecord called:`, undefined, { entityName, key });
+      }
       callbacks.OpenEntityRecord!(entityName, key);
     };
   }
 
   if (callbacks.RegisterMethod) {
     wrapped.RegisterMethod = (methodName: any, handler: any) => {
-      console.log(`[${componentName}] RegisterMethod called for:`, methodName);
+      if (!GetProductionStatus()) {
+        LogStatus(`[${componentName}] RegisterMethod called for:`, undefined, methodName);
+      }
       callbacks.RegisterMethod!(methodName, handler);
     };
   }
 
   if (callbacks.CreateSimpleNotification) {
     wrapped.CreateSimpleNotification = (message: string, style: "none" | "success" | "error" | "warning" | "info", hideAfter?: number) => {
-      console.log(`[${componentName}] CreateSimpleNotification called:`, { message, style, hideAfter });
+      if (!GetProductionStatus()) {
+        LogStatus(`[${componentName}] CreateSimpleNotification called:`, undefined, { message, style, hideAfter });
+      }
       callbacks.CreateSimpleNotification!(message, style, hideAfter);
     };
   }