npm - @patch-adams/core - Versions diffs - 1.4.28 → 1.4.30 - Mend

@patch-adams/core 1.4.28 → 1.4.30

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (9) hide show

package/dist/cli.js CHANGED Viewed

@@ -1735,6 +1735,10 @@ function generateLrsBridgeCode(options) {
       log('Looking up email for employee ID:', employeeId);
       fetchEmployeeEmail(employeeId, function(employeeData) {
+        console.info('[PA-LRS] Employee lookup result for ' + employeeId + ': ' +
+          (employeeData ? 'name=' + employeeData.name + ', email=' + employeeData.email +
+            ', bravaisUserId=' + employeeData.bravaisUserId + ', tenantUrl=' + employeeData.tenantUrl
+            : 'NO DATA'));
         if (employeeData && employeeData.email) {
           actor.mbox = 'mailto:' + employeeData.email;
           log('Enhanced actor with email:', employeeData.email);
@@ -4440,14 +4444,83 @@ function generateLrsBridgeCode(options) {
         // Launch events will NOT be sent until this completes
         extractActorAsync(function(actor) {
           log('Actor updated after async fetch:', actor);
-          actorReady = true;
-          // Log final actor state for diagnostics
-          if (window.console && window.console.info) {
-            console.info('[PA-LRS] Actor resolved: name=' + (actor ? actor.name : 'none') +
-              ', mbox=' + (actor ? actor.mbox : 'none') +
-              ', account=' + (actor && actor.account ? actor.account.name : 'none'));
+          // Check if actor is still "Unknown Learner" - SCORM API might not be available yet
+          // The SCORM API is often injected by scorm-calls.js AFTER <head> init completes
+          var isUnknown = !actor || actor.name === 'Unknown Learner' ||
+                          !actor.account || actor.account.name === 'unknown';
+          if (isUnknown) {
+            log('Actor is Unknown Learner - SCORM API likely not available yet, polling for it...');
+            var scormPollCount = 0;
+            var scormMaxPolls = 20; // wait up to 20 seconds
+            var scormPollTimer = setInterval(function() {
+              scormPollCount++;
+              // Search parent frames for SCORM API (not just current window)
+              var api2004 = findAPIInFrameHierarchy('API_1484_11', 10);
+              var api12 = !api2004 ? findAPIInFrameHierarchy('API', 10) : null;
+              var foundApi = api2004 || api12;
+              if (foundApi) {
+                clearInterval(scormPollTimer);
+                var foundType = api2004 ? '2004' : '1.2';
+                log('SCORM API found on poll ' + scormPollCount + ' at level ' + foundApi.level);
+                // Update LRS SCORM API reference
+                if (!LRS.scormApi) {
+                  LRS.scormApi = foundApi.api;
+                  LRS.scormApiFound = true;
+                  LRS.scormApiType = foundType;
+                }
+                // Read learner_id right now for diagnostics
+                var scormLearnerId = 'n/a', scormLearnerName = 'n/a';
+                try {
+                  if (foundType === '2004') {
+                    scormLearnerId = foundApi.api.GetValue('cmi.learner_id');
+                    scormLearnerName = foundApi.api.GetValue('cmi.learner_name');
+                  } else {
+                    scormLearnerId = foundApi.api.LMSGetValue('cmi.core.student_id');
+                    scormLearnerName = foundApi.api.LMSGetValue('cmi.core.student_name');
+                  }
+                } catch(e) { scormLearnerId = 'error'; scormLearnerName = 'error'; }
+                console.info('[PA-LRS] SCORM API found (poll ' + scormPollCount + ', type=' + foundType +
+                  ', level=' + foundApi.level + '): learner_id=' + scormLearnerId + ', learner_name=' + scormLearnerName);
+                // refreshActor re-reads learner_id from SCORM and runs employee lookup
+                LRS.refreshActor(function(refreshedActor) {
+                  if (window.console && window.console.info) {
+                    console.info('[PA-LRS] Actor resolved (after SCORM wait, poll ' + scormPollCount + '): name=' +
+                      (refreshedActor ? refreshedActor.name : 'none') +
+                      ', mbox=' + (refreshedActor ? refreshedActor.mbox : 'none') +
+                      ', account=' + (refreshedActor && refreshedActor.account ? refreshedActor.account.name : 'none'));
+                  }
+                  actorReady = true;
+                  tryLaunchEvents();
+                });
+              } else if (scormPollCount >= scormMaxPolls) {
+                clearInterval(scormPollTimer);
+                log('SCORM API not found after ' + scormMaxPolls + 's, proceeding with Unknown Learner');
+                if (window.console && window.console.info) {
+                  console.info('[PA-LRS] Actor resolved (timeout): name=' + (actor ? actor.name : 'none') +
+                    ', account=' + (actor && actor.account ? actor.account.name : 'none'));
+                }
+                actorReady = true;
+                tryLaunchEvents();
+              }
+            }, 1000);
+          } else {
+            // Actor is valid - proceed immediately
+            if (window.console && window.console.info) {
+              console.info('[PA-LRS] Actor resolved: name=' + (actor ? actor.name : 'none') +
+                ', mbox=' + (actor ? actor.mbox : 'none') +
+                ', account=' + (actor && actor.account ? actor.account.name : 'none'));
+            }
+            actorReady = true;
+            tryLaunchEvents();
           }
-          tryLaunchEvents();
         });
       } else {
         warn('Bridge setup failed - operating in offline mode');
@@ -4570,32 +4643,6 @@ function generateLrsBridgeCode(options) {
         }, 1000);
       }
-      // If actor is still Unknown Learner, poll for SCORM API availability
-      // The SCORM API may not be available during <head> init but appears later
-      // (e.g., scorm-calls.js finds and initializes it after DOM ready)
-      if (!LRS.actor || LRS.actor.name === 'Unknown Learner' || !LRS.actor.account || LRS.actor.account.name === 'unknown') {
-        var actorRetryCount = 0;
-        var actorMaxRetries = 15;
-        var actorRetryInterval = setInterval(function() {
-          actorRetryCount++;
-          // Check if SCORM API has become available (injected by scorm-calls.js or LMS)
-          var hasScormApi = LRS.scormApi ||
-            (typeof window.API !== 'undefined' && window.API) ||
-            (typeof window.API_1484_11 !== 'undefined' && window.API_1484_11);
-          if (hasScormApi) {
-            log('SCORM API now available (attempt ' + actorRetryCount + '), refreshing actor...');
-            clearInterval(actorRetryInterval);
-            LRS.refreshActor(function(actor) {
-              log('Actor refreshed after SCORM API detected:', actor ? actor.name : 'none');
-            });
-          } else if (actorRetryCount >= actorMaxRetries) {
-            log('SCORM API not found after ' + actorMaxRetries + ' attempts, actor stays as-is');
-            clearInterval(actorRetryInterval);
-          }
-        }, 1000);
-      }
       log('LRS bridge setup complete');
     }