@memberjunction/react-test-harness 2.95.0 → 2.97.0

package/dist/lib/component-runner.d.ts

@@ -1,7 +1,7 @@
 /// <reference types="node" />
 import { BrowserManager } from './browser-context';
 import type { UserInfo } from '@memberjunction/core';
-import { FixSuggestion, Violation } from './component-linter';
+import { Violation } from './component-linter';
 import { ComponentSpec, ComponentUtilities, SimpleAITools } from '@memberjunction/interactive-component-types';
 export interface ComponentExecutionOptions {
     componentSpec: ComponentSpec;
@@ -31,7 +31,6 @@ export interface ComponentExecutionResult {
     executionTime: number;
     renderCount?: number;
     lintViolations?: Violation[];
-    fixSuggestions?: FixSuggestion[];
 }
 /**
  * ComponentRunner that uses the actual React runtime via Playwright UMD bundle
@@ -46,7 +45,6 @@ export declare class ComponentRunner {
      */
     lintComponent(componentCode: string, componentName: string, componentSpec?: any, isRootComponent?: boolean, contextUser?: UserInfo, options?: any): Promise<{
         violations: Violation[];
-        suggestions: FixSuggestion[];
         hasErrors: boolean;
     }>;
     executeComponent(options: ComponentExecutionOptions): Promise<ComponentExecutionResult>;

package/dist/lib/component-runner.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"component-runner.d.ts","sourceRoot":"","sources":["../../src/lib/component-runner.ts"],"names":[],"mappings":";AAAA,OAAO,EAAE,cAAc,EAAE,MAAM,mBAAmB,CAAC;AAEnD,OAAO,KAAK,EAAiC,QAAQ,EAAyD,MAAM,sBAAsB,CAAC;AAC3I,OAAO,EAAmB,aAAa,EAAE,SAAS,EAAE,MAAM,oBAAoB,CAAC;AAC/E,OAAO,EACL,aAAa,EACb,kBAAkB,EAClB,aAAa,EAMd,MAAM,6CAA6C,CAAC;AAMrD,MAAM,WAAW,yBAAyB;IACxC,aAAa,EAAE,aAAa,CAAC;IAC7B,KAAK,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;IAC5B,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,gBAAgB,CAAC,EAAE,MAAM,GAAG,kBAAkB,GAAG,aAAa,CAAC;IAC/D,WAAW,EAAE,QAAQ,CAAC;IACtB,eAAe,CAAC,EAAE,OAAO,CAAC;IAC1B,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,SAAS,CAAC,EAAE,kBAAkB,CAAC;CAChC;AAED,MAAM,WAAW,wBAAwB;IACvC,OAAO,EAAE,OAAO,CAAC;IACjB,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,SAAS,EAAE,CAAC;IACpB,QAAQ,EAAE,SAAS,EAAE,CAAC;IACtB,gBAAgB,EAAE,MAAM,EAAE,CAAC;IAC3B,OAAO,EAAE;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;IAC1C,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,aAAa,EAAE,MAAM,CAAC;IACtB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,cAAc,CAAC,EAAE,SAAS,EAAE,CAAC;IAC7B,cAAc,CAAC,EAAE,aAAa,EAAE,CAAC;CAClC;AAED;;GAEG;AACH,qBAAa,eAAe;IAkBd,OAAO,CAAC,cAAc;IAhBlC,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,yBAAyB,CAS/C;IAKF,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,gBAAgB,CAAS;gBAE7B,cAAc,EAAE,cAAc;IAElD;;OAEG;IACG,aAAa,CACjB,aAAa,EAAE,MAAM,EACrB,aAAa,EAAE,MAAM,EACrB,aAAa,CAAC,EAAE,GAAG,EACnB,eAAe,CAAC,EAAE,OAAO,EACzB,WAAW,CAAC,EAAE,QAAQ,EACtB,OAAO,CAAC,EAAE,GAAG,GACZ,OAAO,CAAC;QAAE,UAAU,EAAE,SAAS,EAAE,CAAC;QAAC,WAAW,EAAE,aAAa,EAAE,CAAC;QAAC,SAAS,EAAE,OAAO,CAAA;KAAE,CAAC;IAoBnF,gBAAgB,CAAC,OAAO,EAAE,yBAAyB,GAAG,OAAO,CAAC,wBAAwB,CAAC;IA+lB7F;;;OAGG;YACW,oBAAoB;IAiElC;;OAEG;YACW,sBAAsB;IAqHpC;;OAEG;YACW,kBAAkB;IAyEhC;;OAEG;YACW,oBAAoB;IAmClC;;OAEG;YACW,eAAe;IAkB7B;;OAEG;IACH,OAAO,CAAC,mBAAmB;YA+Bb,qBAAqB;cAqBnB,uBAAuB,CAAC,WAAW,EAAE,QAAQ,GAAG,OAAO,CAAC,aAAa,CAAC;IA2JtF;;OAEG;YACW,iBAAiB;IAsT/B;;OAEG;IACH,OAAO,CAAC,aAAa;CA6BtB"}
+{"version":3,"file":"component-runner.d.ts","sourceRoot":"","sources":["../../src/lib/component-runner.ts"],"names":[],"mappings":";AAAA,OAAO,EAAE,cAAc,EAAE,MAAM,mBAAmB,CAAC;AAEnD,OAAO,KAAK,EAAiC,QAAQ,EAAyD,MAAM,sBAAsB,CAAC;AAC3I,OAAO,EAAmB,SAAS,EAAE,MAAM,oBAAoB,CAAC;AAChE,OAAO,EACL,aAAa,EACb,kBAAkB,EAClB,aAAa,EAMd,MAAM,6CAA6C,CAAC;AAMrD,MAAM,WAAW,yBAAyB;IACxC,aAAa,EAAE,aAAa,CAAC;IAC7B,KAAK,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;IAC5B,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,gBAAgB,CAAC,EAAE,MAAM,GAAG,kBAAkB,GAAG,aAAa,CAAC;IAC/D,WAAW,EAAE,QAAQ,CAAC;IACtB,eAAe,CAAC,EAAE,OAAO,CAAC;IAC1B,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,SAAS,CAAC,EAAE,kBAAkB,CAAC;CAChC;AAED,MAAM,WAAW,wBAAwB;IACvC,OAAO,EAAE,OAAO,CAAC;IACjB,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,SAAS,EAAE,CAAC;IACpB,QAAQ,EAAE,SAAS,EAAE,CAAC;IACtB,gBAAgB,EAAE,MAAM,EAAE,CAAC;IAC3B,OAAO,EAAE;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;IAC1C,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,aAAa,EAAE,MAAM,CAAC;IACtB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,cAAc,CAAC,EAAE,SAAS,EAAE,CAAC;CAC9B;AAED;;GAEG;AACH,qBAAa,eAAe;IAkBd,OAAO,CAAC,cAAc;IAhBlC,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,yBAAyB,CAS/C;IAKF,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,gBAAgB,CAAS;gBAE7B,cAAc,EAAE,cAAc;IAElD;;OAEG;IACG,aAAa,CACjB,aAAa,EAAE,MAAM,EACrB,aAAa,EAAE,MAAM,EACrB,aAAa,CAAC,EAAE,GAAG,EACnB,eAAe,CAAC,EAAE,OAAO,EACzB,WAAW,CAAC,EAAE,QAAQ,EACtB,OAAO,CAAC,EAAE,GAAG,GACZ,OAAO,CAAC;QAAE,UAAU,EAAE,SAAS,EAAE,CAAC;QAAC,SAAS,EAAE,OAAO,CAAA;KAAE,CAAC;IAmBrD,gBAAgB,CAAC,OAAO,EAAE,yBAAyB,GAAG,OAAO,CAAC,wBAAwB,CAAC;IA6sB7F;;;OAGG;YACW,oBAAoB;IA0ElC;;OAEG;YACW,sBAAsB;IAqHpC;;OAEG;YACW,kBAAkB;IAiFhC;;OAEG;YACW,oBAAoB;IA8ClC;;OAEG;YACW,eAAe;IAkB7B;;OAEG;IACH,OAAO,CAAC,mBAAmB;YA+Bb,qBAAqB;cAqBnB,uBAAuB,CAAC,WAAW,EAAE,QAAQ,GAAG,OAAO,CAAC,aAAa,CAAC;IA2JtF;;OAEG;YACW,iBAAiB;IAsT/B;;OAEG;IACH,OAAO,CAAC,aAAa;CA6BtB"}

package/dist/lib/component-runner.js

@@ -45,7 +45,6 @@ class ComponentRunner {
         const hasErrors = lintResult.violations.some(v => v.severity === 'critical' || v.severity === 'high');
         return {
             violations: lintResult.violations,
-            suggestions: lintResult.suggestions,
             hasErrors
         };
     }
@@ -131,22 +130,25 @@ class ComponentRunner {
                 console.log('  - spec.name:', options.componentSpec.name);
                 console.log('  - spec.code length:', options.componentSpec.code?.length || 0);
                 console.log('  - props:', JSON.stringify(options.props || {}, null, 2));
-                console.log('  - componentLibraries count:', allLibraries?.length || 0);
-                if (allLibraries && allLibraries.length > 0) {
-                    console.log('  - First few libraries:', allLibraries.slice(0, 3).map(lib => ({
-                        Name: lib.Name,
-                        GlobalVariable: lib.GlobalVariable
+                // Show spec-specific libraries, not all available libraries
+                if (options.componentSpec.libraries && options.componentSpec.libraries.length > 0) {
+                    console.log('  - spec requires libraries:', options.componentSpec.libraries.map(lib => ({
+                        name: lib.name,
+                        globalVariable: lib.globalVariable,
+                        version: lib.version
                     })));
                 }
+                else {
+                    console.log('  - spec requires libraries: none');
+                }
+                // Total available libraries in metadata (for context only)
+                console.log('  - total available libraries in metadata:', allLibraries?.length || 0);
             }
             // Execute the component using the real React runtime with timeout (Recommendation #1)
             const executionPromise = page.evaluate(async ({ spec, props, debug, componentLibraries }) => {
                 if (debug) {
                     console.log('🎯 Starting component execution');
-                    console.log('📚 BROWSER: Received componentLibraries:', componentLibraries?.length || 0);
-                    if (componentLibraries?.length > 0) {
-                        console.log('  First library:', componentLibraries[0]);
-                    }
+                    console.log('📚 BROWSER: Component libraries available for loading:', componentLibraries?.length || 0);
                 }
                 // Declare renderCheckInterval at the top scope for cleanup
                 let renderCheckInterval;
@@ -215,8 +217,15 @@ class ComponentRunner {
                     // The spec might not have globalVariable, but we need it for the runtime to work
                     if (spec.libraries && componentLibraries) {
                         for (const specLib of spec.libraries) {
+                            // Skip if library entry is invalid
+                            if (!specLib || !specLib.name) {
+                                if (debug) {
+                                    console.warn('  ⚠️ Skipping invalid library entry (missing name):', specLib);
+                                }
+                                continue;
+                            }
                             if (!specLib.globalVariable) {
-                                const libDef = componentLibraries.find(l => l.Name.toLowerCase() === specLib.name.toLowerCase());
+                                const libDef = componentLibraries.find(l => l && l.Name && l.Name.toLowerCase() === specLib.name.toLowerCase());
                                 if (libDef && libDef.GlobalVariable) {
                                     specLib.globalVariable = libDef.GlobalVariable;
                                     if (debug) {
@@ -257,10 +266,17 @@ class ComponentRunner {
                             message: `Component registration failed: ${registrationError.message || registrationError}`,
                             stack: registrationError.stack,
                             type: 'registration-error',
-                            phase: 'component-compilation'
+                            phase: 'component-compilation',
+                            source: 'runtime-wrapper'
                         });
                         window.__testHarnessTestFailed = true;
-                        throw registrationError; // Re-throw for outer handler
+                        // Don't re-throw - let execution continue to collect this error properly
+                        // Return a failure result so the Promise resolves properly
+                        return {
+                            success: false,
+                            error: `Component registration failed: ${registrationError.message || registrationError}`,
+                            componentCount: 0
+                        };
                     }
                     if (debug && !registrationResult.success) {
                         console.log('❌ Registration failed:', registrationResult.errors);
@@ -319,7 +335,8 @@ class ComponentRunner {
                                 window.__testHarnessRuntimeErrors = window.__testHarnessRuntimeErrors || [];
                                 window.__testHarnessRuntimeErrors.push({
                                     message: `Likely infinite render loop: ${currentRenderCount} createElement calls (max: ${MAX_RENDERS_ALLOWED})`,
-                                    type: 'render-loop'
+                                    type: 'render-loop',
+                                    source: 'test-harness'
                                 });
                                 // Try to unmount to stop the madness
                                 try {
@@ -355,18 +372,29 @@ class ComponentRunner {
                         static getDerivedStateFromError(error) {
                             // Capture the actual error message IMMEDIATELY
                             window.__testHarnessRuntimeErrors = window.__testHarnessRuntimeErrors || [];
+                            // Check if this is a minified React error
+                            const errorMessage = error.message || error.toString();
+                            let enhancedMessage = errorMessage;
+                            if (errorMessage.includes('Minified React error')) {
+                                // Extract error number if present
+                                const match = errorMessage.match(/#(\d+)/);
+                                if (match) {
+                                    enhancedMessage = `React Error #${match[1]} - Visit https://react.dev/errors/${match[1]} for details. ${errorMessage}`;
+                                }
+                            }
                             window.__testHarnessRuntimeErrors.push({
-                                message: error.message || error.toString(),
+                                message: enhancedMessage,
                                 stack: error.stack,
                                 type: 'react-render-error',
-                                phase: 'component-render'
+                                phase: 'component-render',
+                                source: 'user-component' // This is the actual error from user's component
                             });
                             window.__testHarnessTestFailed = true;
                             return { hasError: true, error };
                         }
                         componentDidCatch(error, errorInfo) {
-                            console.error('React Error Boundary caught:', error, errorInfo);
-                            // Update the last error with component stack info
+                            // Don't log here - it creates duplicate messages
+                            // Just update the last error with component stack info
                             const errors = window.__testHarnessRuntimeErrors || [];
                             if (errors.length > 0) {
                                 const lastError = errors[errors.length - 1];
@@ -378,8 +406,8 @@ class ComponentRunner {
                         render() {
                             if (this.state.hasError) {
                                 // DON'T re-throw - this causes "Script error"
-                                // Instead, render an error message
-                                return window.React.createElement('div', { style: { color: 'red', padding: '20px' } }, 'Component failed to render: ' + (this.state.error?.message || 'Unknown error'));
+                                // Instead, render a simple error indicator
+                                return null; // Don't render anything - the error is already captured
                             }
                             return this.props.children;
                         }
@@ -411,7 +439,8 @@ class ComponentRunner {
                     window.__testHarnessRuntimeErrors.push({
                         message: error.message || String(error),
                         stack: error.stack,
-                        type: 'execution-error'
+                        type: 'execution-error',
+                        source: 'runtime-wrapper'
                     });
                     window.__testHarnessTestFailed = true;
                     return {
@@ -429,17 +458,34 @@ class ComponentRunner {
             const timeoutPromise = new Promise((_, reject) => setTimeout(() => reject(new Error(`Component execution timeout after ${globalTimeout}ms`)), globalTimeout));
             // Race between execution and timeout
             let executionResult;
+            let hasTimeout = false;
             try {
                 executionResult = await Promise.race([executionPromise, timeoutPromise]);
             }
             catch (timeoutError) {
                 // Handle timeout gracefully
+                hasTimeout = true;
                 errors.push(`Component execution timed out after ${globalTimeout}ms`);
                 executionResult = {
                     success: false,
                     error: timeoutError instanceof Error ? timeoutError.message : 'Execution timeout'
                 };
             }
+            // Ensure executionResult has proper shape
+            if (!executionResult) {
+                executionResult = {
+                    success: false,
+                    error: 'Component execution returned no result'
+                };
+                errors.push('Component execution failed to return a result');
+            }
+            else if (!executionResult.success && executionResult.error) {
+                // Add the execution error if it hasn't been captured elsewhere
+                const errorMsg = `Component execution failed: ${executionResult.error}`;
+                if (!errors.some(e => e.includes(executionResult.error))) {
+                    errors.push(errorMsg);
+                }
+            }
             if (debug) {
                 console.log('Execution result:', executionResult);
             }
@@ -448,9 +494,9 @@ class ComponentRunner {
             await page.waitForTimeout(renderWaitTime);
             // Get render count
             renderCount = await page.evaluate(() => window.__testHarnessRenderCount || 0);
-            // Collect all errors
-            const runtimeErrors = await this.collectRuntimeErrors(page);
-            errors.push(...runtimeErrors);
+            // Collect all errors with source information
+            const runtimeErrorsWithSource = await this.collectRuntimeErrors(page);
+            errors.push(...runtimeErrorsWithSource.map(e => e.message)); // Extract messages for backward compat
             // Collect warnings (separate from errors)
             const collectedWarnings = await this.collectWarnings(page);
             warnings.push(...collectedWarnings);
@@ -460,8 +506,9 @@ class ComponentRunner {
             const asyncErrors = await this.collectRuntimeErrors(page);
             // Only add new errors
             asyncErrors.forEach(err => {
-                if (!errors.includes(err)) {
-                    errors.push(err);
+                if (!errors.includes(err.message)) {
+                    errors.push(err.message);
+                    runtimeErrorsWithSource.push(err); // Keep the structured version too
                 }
             });
             // Also check for new warnings
@@ -475,26 +522,65 @@ class ComponentRunner {
             const html = await page.content();
             // Take screenshot
             const screenshot = await page.screenshot();
+            // Check for excessive render count first
+            const hasRenderLoop = renderCount > ComponentRunner.MAX_RENDER_COUNT;
+            if (hasRenderLoop) {
+                errors.push(`Possible render loop: ${renderCount} createElement calls detected (likely infinite loop)`);
+            }
             // Determine success
             const success = errors.length === 0 &&
                 criticalWarnings.length === 0 &&
-                renderCount <= ComponentRunner.MAX_RENDER_COUNT &&
+                !hasRenderLoop &&
+                !hasTimeout &&
                 executionResult.success;
-            if (renderCount > ComponentRunner.MAX_RENDER_COUNT) {
-                errors.push(`Possible render loop: ${renderCount} createElement calls detected (likely infinite loop)`);
-            }
             // Combine runtime errors with data errors
             const allErrors = [...errors, ...dataErrors];
-            const result = {
-                success: success && dataErrors.length === 0, // Fail if we have data errors
-                html,
-                errors: allErrors.map(e => ({
+            // Map runtime errors with source info, data errors don't have source
+            const errorViolations = runtimeErrorsWithSource.map(e => ({
+                message: e.message,
+                severity: 'critical',
+                rule: 'runtime-error',
+                line: 0,
+                column: 0,
+                source: e.source
+            }));
+            // Add timeout error if detected
+            if (hasTimeout) {
+                errorViolations.push({
+                    message: `Component execution timed out after ${globalTimeout}ms`,
+                    severity: 'critical',
+                    rule: 'timeout',
+                    line: 0,
+                    column: 0,
+                    source: 'test-harness' // This is a test harness timeout
+                });
+            }
+            // Add render loop error if detected
+            if (hasRenderLoop) {
+                errorViolations.push({
+                    message: `Possible render loop: ${renderCount} createElement calls detected (likely infinite loop)`,
+                    severity: 'critical',
+                    rule: 'render-loop',
+                    line: 0,
+                    column: 0,
+                    source: 'test-harness' // This is a test harness detection
+                });
+            }
+            // Add data errors without source
+            dataErrors.forEach(e => {
+                errorViolations.push({
                     message: e,
                     severity: 'critical',
                     rule: 'runtime-error',
                     line: 0,
-                    column: 0
-                })),
+                    column: 0,
+                    source: 'user-component' // Data errors are from user's data access code
+                });
+            });
+            const result = {
+                success: success && dataErrors.length === 0, // Fail if we have data errors
+                html,
+                errors: errorViolations,
                 warnings: warnings.map(w => ({
                     message: w,
                     severity: 'low',
@@ -514,19 +600,35 @@ class ComponentRunner {
             return result;
         }
         catch (error) {
-            errors.push(error instanceof Error ? error.message : String(error));
-            // Combine runtime errors with data errors
-            const allErrors = [...errors, ...dataErrors];
-            const result = {
-                success: false,
-                html: '',
-                errors: allErrors.map(e => ({
+            // For catch block errors, we need to handle them specially
+            const catchError = {
+                message: error instanceof Error ? error.message : String(error),
+                source: 'test-harness' // Errors caught here are usually test harness issues
+            };
+            // Create error violations including the catch error
+            const errorViolations = [{
+                    message: catchError.message,
+                    severity: 'critical',
+                    rule: 'runtime-error',
+                    line: 0,
+                    column: 0,
+                    source: catchError.source
+                }];
+            // Add any data errors
+            dataErrors.forEach(e => {
+                errorViolations.push({
                     message: e,
                     severity: 'critical',
                     rule: 'runtime-error',
                     line: 0,
-                    column: 0
-                })),
+                    column: 0,
+                    source: 'user-component'
+                });
+            });
+            const result = {
+                success: false,
+                html: '',
+                errors: errorViolations,
                 warnings: warnings.map(w => ({
                     message: w,
                     severity: 'low',
@@ -565,7 +667,12 @@ class ComponentRunner {
      * Component-specific libraries are loaded by the runtime's ComponentCompiler
      */
     async loadRuntimeLibraries(page, debug = false) {
-        // Helper function to load scripts with timeout (Recommendation #3)
+        // Import getCoreRuntimeLibraries to get the correct URLs for React libraries
+        // We can't use LibraryLoader.loadAllLibraries() here because it expects to run in a browser
+        // environment with window/document, but we're in Node.js with Playwright
+        const { getCoreRuntimeLibraries } = await Promise.resolve().then(() => __importStar(require('@memberjunction/react-runtime')));
+        const coreLibraries = getCoreRuntimeLibraries(debug);
+        // Helper function to load scripts with timeout
         const loadScriptWithTimeout = async (url, timeout = 10000) => {
             try {
                 await Promise.race([
@@ -577,11 +684,15 @@ class ComponentRunner {
                 throw new Error(`Failed to load script ${url}: ${error instanceof Error ? error.message : String(error)}`);
             }
         };
-        // Load React and ReactDOM with timeout protection
-        await loadScriptWithTimeout('https://unpkg.com/react@18/umd/react.development.js');
-        await loadScriptWithTimeout('https://unpkg.com/react-dom@18/umd/react-dom.development.js');
-        // Load Babel for JSX transformation with timeout
-        await loadScriptWithTimeout('https://unpkg.com/@babel/standalone/babel.min.js');
+        // Load the core libraries (React, ReactDOM, Babel) into the Playwright page context
+        for (const lib of coreLibraries) {
+            if (lib.cdnUrl) {
+                await loadScriptWithTimeout(lib.cdnUrl);
+                if (debug) {
+                    console.log(`  ✓ Loaded ${lib.displayName} (${lib.globalVariable})`);
+                }
+            }
+        }
         // Load the real MemberJunction React Runtime UMD bundle
         const fs = await Promise.resolve().then(() => __importStar(require('fs')));
         // Resolve the path to the UMD bundle
@@ -768,10 +879,16 @@ class ComponentRunner {
             };
             // Global error handler
             window.addEventListener('error', (event) => {
+                // Determine source based on error content
+                let source = 'user-component'; // Default to user component
+                if (event.message && event.message.includes('Script error')) {
+                    source = 'runtime-wrapper';
+                }
                 window.__testHarnessRuntimeErrors.push({
                     message: event.error?.message || event.message,
                     stack: event.error?.stack,
-                    type: 'runtime'
+                    type: 'runtime',
+                    source: source
                 });
                 window.__testHarnessTestFailed = true;
             });
@@ -780,7 +897,8 @@ class ComponentRunner {
                 window.__testHarnessRuntimeErrors.push({
                     message: 'Unhandled Promise Rejection: ' + (event.reason?.message || event.reason),
                     stack: event.reason?.stack,
-                    type: 'promise-rejection'
+                    type: 'promise-rejection',
+                    source: 'user-component' // Async errors are likely from user code
                 });
                 window.__testHarnessTestFailed = true;
                 event.preventDefault();
@@ -799,23 +917,31 @@ class ComponentRunner {
             };
         });
         const errors = [];
-        // Only add "test failed" message if there are actual errors
-        if (errorData.testFailed && (errorData.runtimeErrors.length > 0 || errorData.consoleErrors.length > 0)) {
-            errors.push('Test marked as failed by error handlers');
-        }
+        // Check if we have any specific React render errors
+        const hasSpecificReactError = errorData.runtimeErrors.some((error) => error.type === 'react-render-error' &&
+            !error.message.includes('Script error'));
+        // Process runtime errors with their source information
         errorData.runtimeErrors.forEach((error) => {
-            // Include phase information if available to help identify where the error occurred
+            // Skip generic "Script error" messages if we have more specific React errors
+            if (hasSpecificReactError &&
+                error.type === 'runtime' &&
+                error.message === 'Script error.') {
+                return; // Skip this generic error
+            }
             const phase = error.phase ? ` (during ${error.phase})` : '';
             const errorMsg = `${error.type} error: ${error.message}${phase}`;
-            if (!errors.includes(errorMsg)) {
-                errors.push(errorMsg);
-            }
+            errors.push({
+                message: errorMsg,
+                source: error.source
+            });
         });
+        // Console errors don't have source info
         errorData.consoleErrors.forEach((error) => {
             const errorMsg = `Console error: ${error}`;
-            if (!errors.includes(errorMsg)) {
-                errors.push(errorMsg);
-            }
+            errors.push({
+                message: errorMsg,
+                source: 'react-framework' // Console errors from React are framework level
+            });
         });
         return errors;
     }