@patch-adams/core 1.5.20 → 1.5.21

package/dist/cli.js

@@ -1971,22 +1971,28 @@ function generateLrsBridgeCode(options) {
 
     /**
      * Get the best available actor for statement building.
-     * Priority: LRS.actor (if non-anonymous) > localStorage shared actor > LRS.actor > extractActor()
+     * ALWAYS checks localStorage because the skin overlay in another frame
+     * may have updated the actor AFTER this bridge instance initialized.
+     * (e.g., user enters email in skin overlay \u2192 actor persisted to localStorage,
+     *  but the bridge in scormcontent/index.html already loaded a stale actor at init)
      */
     function getActor() {
-      // If current actor is non-anonymous, use it
-      if (LRS.actor && !isAnonymousActor(LRS.actor)) {
-        return LRS.actor;
-      }
-      // Check localStorage for actor set by another frame (e.g., skin overlay)
+      // Always check localStorage for the freshest actor \u2014 it may have been
+      // updated by the skin overlay in another frame since our init
       var shared = loadSharedActor();
       if (shared) {
-        // Update local actor so future calls are fast
+        // Only update if different from current (avoid unnecessary log spam)
+        if (!LRS.actor || LRS.actor.name !== shared.name ||
+            (LRS.actor.account && shared.account && LRS.actor.account.name !== shared.account.name)) {
+          log('Actor updated from cross-frame storage:', shared.name);
+        }
         LRS.actor = shared;
-        log('Actor loaded from cross-frame storage:', shared.name);
         return shared;
       }
       // Fallback to current actor or re-extract
+      if (LRS.actor && !isAnonymousActor(LRS.actor)) {
+        return LRS.actor;
+      }
       return LRS.actor || extractActor();
     }
 
@@ -1997,7 +2003,9 @@ function generateLrsBridgeCode(options) {
     LRS.setActor = function(actor) {
       LRS.actor = actor;
       persistActor(actor);
-      // Also try to propagate to other frames in the hierarchy
+
+      // Propagate to ALL frames: walk UP parent chain AND DOWN into child iframes
+      // UP: parent frames (e.g., if skin is in a child iframe)
       try {
         var w = window;
         for (var i = 0; i < 10; i++) {
@@ -2010,9 +2018,29 @@ function generateLrsBridgeCode(options) {
           w = w.parent;
         }
       } catch (e) {}
+
+      // DOWN: child iframes (e.g., scormcontent/index.html has its own bridge)
+      function setActorInChildren(win) {
+        try {
+          var frames = win.frames;
+          for (var j = 0; j < frames.length; j++) {
+            try {
+              if (frames[j].pa_patcher && frames[j].pa_patcher.lrs) {
+                frames[j].pa_patcher.lrs.actor = actor;
+                log('Actor propagated to child frame', j);
+              }
+              // Recurse into nested iframes
+              setActorInChildren(frames[j]);
+            } catch (e) { /* cross-origin child */ }
+          }
+        } catch (e) {}
+      }
+      setActorInChildren(window);
+
       if (window.console && window.console.info) {
         console.info('[PA-LRS] Actor set:', actor.name,
           actor.mbox ? '(' + actor.mbox + ')' : '',
+          actor.account ? '(account: ' + actor.account.name + ')' : '',
           '\u2014 shared across frames');
       }
     };