@patch-adams/core 1.4.24 → 1.4.26

package/dist/cli.js

@@ -613,13 +613,13 @@ html.${htmlClass}.${loadingClass} body {
   var link = loadCSSSync(REMOTE_URL);
 
   link.onerror = function() {
-    console.warn('[Patch-Adams] CSS before failed to load from remote, using local fallback');
+    console.warn('[PA-Patcher] CSS before failed to load from remote, using local fallback');
     document.head.removeChild(link);
     loadCSSSync(LOCAL_PATH);
   };
 
   link.onload = function() {
-    console.log('[Patch-Adams] CSS before loaded from remote:', REMOTE_URL);
+    console.log('[PA-Patcher] CSS before loaded from remote:', REMOTE_URL);
   };
 })();
 </script>`;
@@ -673,7 +673,7 @@ function generateCssAfterLoader(options) {
         if (loaded) return;
         loaded = true;
         clearTimeout(timeoutId);
-        console.log('[Patch-Adams] CSS after loaded from remote:', REMOTE_URL);
+        console.log('[PA-Patcher] CSS after loaded from remote:', REMOTE_URL);
       },
       function() {
         if (loaded) return;
@@ -687,7 +687,7 @@ function generateCssAfterLoader(options) {
     timeoutId = setTimeout(function() {
       if (loaded) return;
       loaded = true;
-      console.warn('[Patch-Adams] CSS after timed out, using local fallback');
+      console.warn('[PA-Patcher] CSS after timed out, using local fallback');
       if (remoteLink.parentNode) {
         document.head.removeChild(remoteLink);
       }
@@ -699,10 +699,10 @@ function generateCssAfterLoader(options) {
     loadCSS(
       LOCAL_PATH,
       function() {
-        console.log('[Patch-Adams] CSS after loaded from local fallback:', LOCAL_PATH);
+        console.log('[PA-Patcher] CSS after loaded from local fallback:', LOCAL_PATH);
       },
       function() {
-        console.error('[Patch-Adams] CSS after failed to load from both remote and local');
+        console.error('[PA-Patcher] CSS after failed to load from both remote and local');
       }
     );
   }
@@ -1667,7 +1667,9 @@ function generateLrsBridgeCode(options) {
                         ((employee.fname || employee.firstName || '') + ' ' + (employee.lname || employee.lastName || '')).trim(),
                   firstName: employee.fname || employee.firstName || employee.first_name,
                   lastName: employee.lname || employee.lastName || employee.last_name,
-                  employeeId: employee.emp_id || employee.employeeId || employee.employee_id || employeeId
+                  employeeId: employee.emp_id || employee.employeeId || employee.employee_id || employeeId,
+                  bravaisUserId: employee.bravaisUserId || null,
+                  tenantUrl: employee.tenantUrl || null
                 };
                 log('Parsed employee data:', employeeLookupData);
                 // Persist to localStorage for cross-session caching
@@ -1741,6 +1743,16 @@ function generateLrsBridgeCode(options) {
           if (employeeData.name && (!actor.name || actor.name === 'Unknown Learner')) {
             actor.name = employeeData.name;
           }
+
+          // Update account with Bravais user ID and tenant URL if available
+          if (employeeData.bravaisUserId) {
+            var originalAccountName = actor.account ? actor.account.name : null;
+            actor.account = {
+              name: employeeData.bravaisUserId,
+              homePage: employeeData.tenantUrl || (actor.account && actor.account.homePage) || window.location.origin
+            };
+            log('Updated actor account: bravaisUserId=' + employeeData.bravaisUserId + ', homePage=' + actor.account.homePage + ' (was: ' + originalAccountName + ')');
+          }
         }
         callback(actor);
       });
@@ -1823,16 +1835,9 @@ function generateLrsBridgeCode(options) {
                 name: scormLearnerId
               }
             };
-            if (scormLearnerName) {
-              // Handle "LastName, FirstName" format
-              if (scormLearnerName.indexOf(',') > -1) {
-                var parts = scormLearnerName.split(',');
-                actor.name = (parts[1] || '').trim() + ' ' + (parts[0] || '').trim();
-              } else {
-                actor.name = scormLearnerName;
-              }
-            }
-            log('Actor source: SCORM API');
+            // Don't use SCORM student_name - it's unreliable ("Last, First" format varies by LMS).
+            // The employee API lookup will provide the correct name and email.
+            log('Actor source: SCORM API (id only, name from employee lookup)');
             LRS.actor = actor;
             return actor;
           }
@@ -2412,7 +2417,7 @@ function generateLrsBridgeCode(options) {
         tenantHomepage: null         // https://{tenant}.bravais.com format
       };
 
-      // 0. Use baked-in GUIDs from Patch-Adams config (highest priority - set at wrap time)
+      // 0. Use baked-in GUIDs from PA-Patcher config (highest priority - set at wrap time)
       if (DOCUMENT_GUID) {
         info.guid = DOCUMENT_GUID;
         info.id = 'http://xyleme.com/bravais/document/' + DOCUMENT_GUID;
@@ -2921,6 +2926,11 @@ function generateLrsBridgeCode(options) {
       // Publish index (default to 1)
       ctx.extensions['publishIndex'] = (LRS.courseInfo && LRS.courseInfo.publishIndex) ? LRS.courseInfo.publishIndex : 1;
 
+      // Employee ID from employee lookup (if available)
+      if (employeeLookupData && employeeLookupData.employeeId) {
+        ctx.extensions['employeeId'] = employeeLookupData.employeeId;
+      }
+
       // Xyleme schema version and PII flag
       ctx.extensions['http://xyleme.com/bravais/extensions/statement-schema'] = 'xyleme_10';
       ctx.extensions['http://xyleme.com/bravais/extensions/protect_pii'] = false;
@@ -3640,6 +3650,27 @@ function generateLrsBridgeCode(options) {
      * because the bridge initializes in <head> before LMSInitialize runs.
      */
     LRS.refreshActor = function(callback) {
+      // If SCORM API wasn't found during <head> init, re-scan now
+      // (some LMS inject window.API after page starts loading)
+      if (!LRS.scormApi) {
+        log('refreshActor: SCORM API missing, re-scanning...');
+        var api2004 = findAPIInFrameHierarchy('API_1484_11', 10);
+        if (api2004) {
+          LRS.scormApi = api2004.api;
+          LRS.scormApiFound = true;
+          LRS.scormApiType = '2004';
+          log('refreshActor: found SCORM 2004 API');
+        } else {
+          var api12 = findAPIInFrameHierarchy('API', 10);
+          if (api12) {
+            LRS.scormApi = api12.api;
+            LRS.scormApiFound = true;
+            LRS.scormApiType = '1.2';
+            log('refreshActor: found SCORM 1.2 API');
+          }
+        }
+      }
+
       // Always-visible diagnostic: confirm refreshActor is being called and show SCORM data (TEMPORARY)
       if (window.console && window.console.info) {
         var scormId = 'n/a', scormName = 'n/a';
@@ -4568,7 +4599,7 @@ ${courseLines.join("\n")}
   },` : "";
   return `<!-- === PATCH-ADAMS: JS BEFORE (blocking) === -->
 <script data-pa="js-before-loader">
-// Initialize Patch-Adams global namespace IMMEDIATELY (before IIFE)
+// Initialize PA-Patcher global namespace IMMEDIATELY (before IIFE)
 window.pa_patcher = window.pa_patcher || {
   version: '1.0.25',
   htmlClass: '${htmlClass}',
@@ -4613,7 +4644,7 @@ window.pa_patcher = window.pa_patcher || {
     if (isExtensionError(event.reason)) {
       event.preventDefault();
       event.stopImmediatePropagation();
-      console.debug('[Patch-Adams] Suppressed browser extension error:', event.reason);
+      console.debug('[PA-Patcher] Suppressed browser extension error:', event.reason);
       return false;
     }
   }, true);
@@ -4623,7 +4654,7 @@ window.pa_patcher = window.pa_patcher || {
     if (isExtensionError(event.error || event.message)) {
       event.preventDefault();
       event.stopImmediatePropagation();
-      console.debug('[Patch-Adams] Suppressed browser extension error:', event.message);
+      console.debug('[PA-Patcher] Suppressed browser extension error:', event.message);
       return false;
     }
   }, true);
@@ -4648,17 +4679,17 @@ window.pa_patcher = window.pa_patcher || {
   var script = loadJSSync(REMOTE_URL);
 
   script.onerror = function() {
-    console.warn('[Patch-Adams] JS before failed to load from remote, using local fallback');
+    console.warn('[PA-Patcher] JS before failed to load from remote, using local fallback');
     document.head.removeChild(script);
     var fallback = loadJSSync(LOCAL_PATH);
     fallback.onload = function() {
       if (window.pa_patcher && window.pa_patcher.loaded) {
         window.pa_patcher.loaded.jsBefore = true;
       }
-      console.log('[Patch-Adams] JS before loaded from local fallback:', LOCAL_PATH);
+      console.log('[PA-Patcher] JS before loaded from local fallback:', LOCAL_PATH);
     };
     fallback.onerror = function() {
-      console.error('[Patch-Adams] JS before failed to load from both remote and local');
+      console.error('[PA-Patcher] JS before failed to load from both remote and local');
     };
   };
 
@@ -4666,7 +4697,7 @@ window.pa_patcher = window.pa_patcher || {
     if (window.pa_patcher && window.pa_patcher.loaded) {
       window.pa_patcher.loaded.jsBefore = true;
     }
-    console.log('[Patch-Adams] JS before loaded from remote:', REMOTE_URL);
+    console.log('[PA-Patcher] JS before loaded from remote:', REMOTE_URL);
   };
 })();
 ${lrsBridgeCode}
@@ -4737,14 +4768,14 @@ function generateJsAfterLoader(options) {
     if (window.pa_patcher && window.pa_patcher.loaded) {
       window.pa_patcher.loaded.jsAfter = true;
     }
-    console.log('[Patch-Adams] Loading complete, content revealed');
+    console.log('[PA-Patcher] Loading complete, content revealed');
 
     // Add visual badge to indicate patching is active
     var badge = document.createElement('div');
     badge.setAttribute('data-pa', 'badge');
     badge.style.cssText = 'position:fixed;bottom:10px;right:10px;background:#4CAF50;color:white;padding:4px 8px;border-radius:4px;font-size:11px;font-family:sans-serif;z-index:999999;opacity:0.8;cursor:pointer;';
     badge.textContent = 'PA';
-    badge.title = 'Patch-Adams Active';
+    badge.title = 'PA-Patcher Active';
     badge.onclick = function() { badge.style.display = 'none'; };
     document.body.appendChild(badge);
   }
@@ -4760,7 +4791,7 @@ function generateJsAfterLoader(options) {
         if (loaded) return;
         loaded = true;
         clearTimeout(timeoutId);
-        console.log('[Patch-Adams] JS after loaded from remote:', REMOTE_URL);
+        console.log('[PA-Patcher] JS after loaded from remote:', REMOTE_URL);
         // Give the script a moment to execute, then remove loading class
         setTimeout(removeLoadingClass, 50);
       },
@@ -4776,7 +4807,7 @@ function generateJsAfterLoader(options) {
     timeoutId = setTimeout(function() {
       if (loaded) return;
       loaded = true;
-      console.warn('[Patch-Adams] JS after timed out, using local fallback');
+      console.warn('[PA-Patcher] JS after timed out, using local fallback');
       if (remoteScript.parentNode) {
         document.body.removeChild(remoteScript);
       }
@@ -4788,11 +4819,11 @@ function generateJsAfterLoader(options) {
     loadJS(
       LOCAL_PATH,
       function() {
-        console.log('[Patch-Adams] JS after loaded from local fallback:', LOCAL_PATH);
+        console.log('[PA-Patcher] JS after loaded from local fallback:', LOCAL_PATH);
         setTimeout(removeLoadingClass, 50);
       },
       function() {
-        console.error('[Patch-Adams] JS after failed to load from both remote and local');
+        console.error('[PA-Patcher] JS after failed to load from both remote and local');
         // Still remove loading class so content is visible even if JS fails
         removeLoadingClass();
       }
@@ -5252,7 +5283,7 @@ var ManifestUpdater = class {
       const resourceClosePattern = /(\s*)<\/resource>/i;
       const match = xmlContent.match(resourceClosePattern);
       if (!match) {
-        console.warn("[Patch-Adams] Could not find </resource> tag in imsmanifest.xml");
+        console.warn("[PA-Patcher] Could not find </resource> tag in imsmanifest.xml");
         return [];
       }
       const updatedXml = xmlContent.replace(
@@ -5264,7 +5295,7 @@ $1</resource>`
       zip.updateFile("imsmanifest.xml", Buffer.from(updatedXml, "utf-8"));
       return ["imsmanifest.xml"];
     } catch (error) {
-      console.error("[Patch-Adams] Failed to update imsmanifest.xml:", error);
+      console.error("[PA-Patcher] Failed to update imsmanifest.xml:", error);
       return [];
     }
   }
@@ -5586,7 +5617,7 @@ ${js.after}
 var pluginRegistry = new PluginRegistry();
 
 // src/patcher/index.ts
-var DEFAULT_CSS_BEFORE = `/* Patch-Adams: CSS Before (blocking)
+var DEFAULT_CSS_BEFORE = `/* PA-Patcher: CSS Before (blocking)
  * This file loads at the start of <head> and blocks rendering.
  * Use it to hide content and prevent flash of unstyled content.
  *
@@ -5596,7 +5627,7 @@ var DEFAULT_CSS_BEFORE = `/* Patch-Adams: CSS Before (blocking)
  * }
  */
 `;
-var DEFAULT_CSS_AFTER = `/* Patch-Adams: CSS After (async)
+var DEFAULT_CSS_AFTER = `/* PA-Patcher: CSS After (async)
  * This file loads at the end of <head> with remote fallback.
  * Use it for style overrides that take precedence over Rise styles.
  *
@@ -5606,7 +5637,7 @@ var DEFAULT_CSS_AFTER = `/* Patch-Adams: CSS After (async)
  * }
  */
 `;
-var DEFAULT_JS_BEFORE = `// Patch-Adams: JS Before (blocking)
+var DEFAULT_JS_BEFORE = `// PA-Patcher: JS Before (blocking)
 // This file loads at the start of <head> and blocks rendering.
 // Use it for setup, API interception, and preparing globals.
 //
@@ -5616,9 +5647,9 @@ var DEFAULT_JS_BEFORE = `// Patch-Adams: JS Before (blocking)
 //   analytics: true
 // };
 
-console.log('[Patch-Adams] JS Before loaded');
+console.log('[PA-Patcher] JS Before loaded');
 `;
-var DEFAULT_JS_AFTER = `// Patch-Adams: JS After (async)
+var DEFAULT_JS_AFTER = `// PA-Patcher: JS After (async)
 // This file loads at the end of <body> with remote fallback.
 // Use it for DOM manipulation after Rise has initialized.
 //
@@ -5630,7 +5661,7 @@ var DEFAULT_JS_AFTER = `// Patch-Adams: JS After (async)
 //   // Your modifications here
 // });
 
-console.log('[Patch-Adams] JS After loaded');
+console.log('[PA-Patcher] JS After loaded');
 `;
 var Patcher = class {
   config;