@patch-adams/core 1.5.6 → 1.5.8

package/dist/cli.js

@@ -948,6 +948,14 @@ function generateLrsBridgeCode(options) {
       });
     }
 
+    // Decode HTML entities (e.g., " " & ') to clean text
+    function decodeEntities(str) {
+      if (!str || typeof str !== 'string') return str;
+      var txt = document.createElement('textarea');
+      txt.innerHTML = str;
+      return txt.value;
+    }
+
     LRS.sessionId = generateUUID();
     LRS.courseAttemptId = generateUUID();  // Unique per session for Xyleme correlation
     LRS.launchTime = new Date().toISOString();
@@ -2539,6 +2547,10 @@ function generateLrsBridgeCode(options) {
         info.id = window.location.href.split('#')[0].split('?')[0];
       }
 
+      // Decode any HTML entities in title/description
+      if (info.title) info.title = decodeEntities(info.title);
+      if (info.description) info.description = decodeEntities(info.description);
+
       // Build shared link name if not provided
       if (!info.sharedLinkName && info.title && info.sharedLinkToken) {
         info.sharedLinkName = info.title + ' - LMS Thin Pack';
@@ -2718,7 +2730,7 @@ function generateLrsBridgeCode(options) {
           id: window.location.href.split('#')[0].split('?')[0],
           definition: {
             type: 'http://xyleme.com/bravais/activities/document',
-            name: { 'en-US': document.title || 'Rise Course' }
+            name: { 'en-US': decodeEntities(document.title) || 'Rise Course' }
           }
         };
       }
@@ -2749,10 +2761,11 @@ function generateLrsBridgeCode(options) {
       courseObj.definition.extensions = courseObj.definition.extensions || {};
 
       if (activityDetails) {
-        // Merge activity details into extensions
+        // Merge activity details into extensions, decoding any HTML entities in string values
         for (var key in activityDetails) {
           if (activityDetails.hasOwnProperty(key)) {
-            courseObj.definition.extensions[key] = activityDetails[key];
+            var val = activityDetails[key];
+            courseObj.definition.extensions[key] = typeof val === 'string' ? decodeEntities(val) : val;
           }
         }
       }
@@ -2831,7 +2844,7 @@ function generateLrsBridgeCode(options) {
         id: LRS.courseInfo.id,
         definition: {
           type: 'http://xyleme.com/bravais/activities/document',
-          name: { 'en-US': LRS.courseInfo.title }
+          name: { 'en-US': decodeEntities(LRS.courseInfo.title) }
         }
       };
 
@@ -2881,7 +2894,7 @@ function generateLrsBridgeCode(options) {
         objectType: 'Activity',
         definition: {
           type: 'http://xyleme.com/bravais/activities/document',
-          name: { 'en-US': LRS.courseInfo.title || 'Rise Course' }
+          name: { 'en-US': decodeEntities(LRS.courseInfo.title) || 'Rise Course' }
         }
       };
 
@@ -3003,15 +3016,37 @@ function generateLrsBridgeCode(options) {
      * Build activity object for questions in Xyleme format
      * Name format: "Assessment Name - Q#: Question text..."
      */
+    /**
+     * Simple string hash for generating short, stable IDs from long strings.
+     * Returns a hex string (8 chars). Not cryptographic \u2014 just for uniqueness.
+     */
+    function hashString(str) {
+      var hash = 0;
+      for (var i = 0; i < str.length; i++) {
+        var ch = str.charCodeAt(i);
+        hash = ((hash << 5) - hash) + ch;
+        hash = hash & hash; // Convert to 32-bit integer
+      }
+      return Math.abs(hash).toString(16).padStart(8, '0');
+    }
+
     function buildQuestionActivityObject(questionInfo) {
-      var questionGuid = questionInfo.questionGuid || questionInfo.id || generateUUID();
+      var rawGuid = questionInfo.questionGuid || questionInfo.id || generateUUID();
+
+      // Ensure the object ID stays under 255 chars total to avoid Bravais aggregation failures.
+      // Rise SCORM interaction IDs can be very long (full slugified question text).
+      // Use a truncated prefix + hash to keep it short but unique and stable.
+      var questionGuid = rawGuid;
+      if (rawGuid.length > 80) {
+        questionGuid = rawGuid.substring(0, 60) + '_' + hashString(rawGuid);
+      }
 
       // Build human-readable display name
       var displayName = (questionInfo.assessmentName || 'Knowledge Check');
       if (questionInfo.questionNumber) {
         displayName += ' - Q' + questionInfo.questionNumber;
       }
-      var questionText = questionInfo.text || questionInfo.questionText || '';
+      var questionText = decodeEntities(questionInfo.text || questionInfo.questionText || '');
       if (questionText.length > 50) {
         displayName += ': ' + questionText.substring(0, 47) + '...';
       } else if (questionText.length > 0) {
@@ -4388,10 +4423,12 @@ function generateLrsBridgeCode(options) {
         var interactionId = interaction.id || ('interaction-' + idx);
 
         // Try to make a readable question text from the interaction ID
-        var questionText = interactionId
-          .replace(/_\\d+$/, '')  // remove trailing _0
-          .replace(/_/g, ' ')     // underscores to spaces
-          .substring(0, 200);
+        var questionText = decodeEntities(
+          interactionId
+            .replace(/_\\d+$/, '')  // remove trailing _0
+            .replace(/_/g, ' ')     // underscores to spaces
+            .substring(0, 200)
+        );
 
         // Get lesson context
         var lessonInfo = getCachedLessonInfo();