@patch-adams/core 1.5.2 → 1.5.4

package/dist/cli.js

@@ -4142,8 +4142,9 @@ function generateLrsBridgeCode(options) {
       });
     }
 
-    // Track submitted Knowledge Check blocks to avoid duplicates
+    // Track submitted Knowledge Check questions to avoid duplicates
     var submittedKnowledgeChecks = {};
+    var kcQuestionCounter = 0;
 
     /**
      * Set up interceptors specifically for Rise Knowledge Check blocks
@@ -4153,7 +4154,11 @@ function generateLrsBridgeCode(options) {
       if (!TRACK_QUIZZES) return;
 
       // Intercept submit button clicks on Knowledge Check blocks
+      // NOTE: If the SCORM tracker is active, it handles quiz tracking via
+      // cmi.interactions \u2014 skip DOM scraping to avoid duplicate statements
       document.addEventListener('click', function(e) {
+        if (scormTrackerActive) return;  // SCORM tracker handles this
+
         var submitBtn = e.target.closest('.quiz-card__button');
         if (!submitBtn) return;
 
@@ -4175,16 +4180,26 @@ function generateLrsBridgeCode(options) {
      * Extract and send xAPI statement for a Knowledge Check submission
      */
     function extractKnowledgeCheckResult(kcBlock) {
-      // Get block ID for deduplication
+      // Get block ID for context
       var blockContainer = kcBlock.closest('[data-block-id]');
       var blockId = blockContainer ? blockContainer.getAttribute('data-block-id') : null;
 
+      // Get question text first \u2014 needed for per-question dedup key
+      var questionText = '';
+      var questionTextEl = kcBlock.querySelector('.quiz-card__title .fr-view, .quiz-card__title');
+      if (questionTextEl) {
+        questionText = questionTextEl.textContent.trim();
+      }
+
       // Get question ID from the title element
       var questionTitleEl = kcBlock.querySelector('.quiz-card__title');
-      var questionId = questionTitleEl ? questionTitleEl.id : (blockId ? 'q-' + blockId : 'q-' + generateUUID());
+      var questionId = questionTitleEl ? questionTitleEl.id : null;
+
+      // Build a question-specific dedup key using question text hash
+      // This ensures each question in a multi-question quiz block gets its own key
+      var questionHash = questionText ? questionText.substring(0, 100) : (questionId || blockId || generateUUID());
+      var submissionKey = 'kc-' + questionHash;
 
-      // Check if we already processed this submission (avoid duplicates)
-      var submissionKey = blockId || questionId;
       var feedbackLabel = kcBlock.querySelector('.quiz-card__feedback-label');
       if (!feedbackLabel) {
         log('Knowledge Check: No feedback visible yet');
@@ -4200,11 +4215,9 @@ function generateLrsBridgeCode(options) {
       }
       submittedKnowledgeChecks[submissionId] = true;
 
-      // Get question text
-      var questionText = '';
-      var questionTextEl = kcBlock.querySelector('.quiz-card__title .fr-view, .quiz-card__title');
-      if (questionTextEl) {
-        questionText = questionTextEl.textContent.trim();
+      // Use question-specific ID for the statement (not the shared block ID)
+      if (!questionId) {
+        questionId = blockId ? 'q-' + blockId + '-' + questionHash.substring(0, 20) : 'q-' + generateUUID();
       }
 
       // Determine question type from aria-label
@@ -4233,11 +4246,14 @@ function generateLrsBridgeCode(options) {
         correct: isCorrect
       });
 
+      // Increment question counter for this session
+      kcQuestionCounter++;
+
       // Send question answered statement using existing LRS method
       LRS.questionAnswered({
         questionId: questionId,
         questionGuid: blockId || generateUUID(),
-        questionNumber: 1,
+        questionNumber: kcQuestionCounter,
         questionText: questionText.substring(0, 500),
         questionType: questionType,
         answer: answerText.substring(0, 500),
@@ -4298,6 +4314,142 @@ function generateLrsBridgeCode(options) {
       return answerText;
     }
 
+    // ========================================================================
+    // SCORM INTERACTION TRACKER
+    // Intercepts cmi.interactions.N.* SetValue calls to capture quiz answers
+    // directly from the SCORM data model \u2014 works regardless of Rise UI format
+    // (Knowledge Check blocks, quiz lessons, etc.)
+    // ========================================================================
+    var scormInteractions = {};  // Pending interactions keyed by index N
+    var scormInteractionsSent = {};  // Track which interactions were already sent
+    var scormTrackerActive = false;  // Set true when SCORM tracker wraps SetValue \u2014 KC handler defers
+
+    function setupScormInteractionTracker() {
+      if (!TRACK_QUIZZES) return;
+      if (!LRS.scormApi) {
+        log('SCORM Interaction Tracker: No SCORM API found, skipping');
+        return;
+      }
+
+      var api = LRS.scormApi;
+      var apiType = LRS.scormApiType;
+
+      // Determine which SetValue function to wrap
+      var setValueFn = apiType === '2004' ? 'SetValue' : 'LMSSetValue';
+      var originalSetValue = api[setValueFn];
+
+      if (typeof originalSetValue !== 'function') {
+        log('SCORM Interaction Tracker: No ' + setValueFn + ' function found');
+        return;
+      }
+
+      // SCORM 1.2 uses cmi.interactions.N.*, SCORM 2004 uses cmi.interactions.N.*
+      var interactionPattern = /^cmi\\.interactions\\.(\\d+)\\.(.+)$/;
+
+      api[setValueFn] = function(key, value) {
+        // Always call the original first
+        var result = originalSetValue.apply(api, arguments);
+
+        // Check if this is an interaction data element
+        if (typeof key === 'string') {
+          var match = key.match(interactionPattern);
+          if (match) {
+            var idx = match[1];
+            var field = match[2];
+
+            // Initialize interaction tracking for this index
+            if (!scormInteractions[idx]) {
+              scormInteractions[idx] = {};
+            }
+
+            // Store the field value
+            scormInteractions[idx][field] = value;
+
+            log('SCORM Interaction [' + idx + '].' + field + ' = ' + (value ? value.substring(0, 80) : value));
+
+            // When 'result' is set, the interaction is complete \u2014 fire xAPI statement
+            if (field === 'result' && !scormInteractionsSent[idx]) {
+              scormInteractionsSent[idx] = true;
+              var interaction = scormInteractions[idx];
+
+              // Map SCORM interaction type to readable type
+              var typeMap = {
+                'choice': 'multiple-choice',
+                'true-false': 'true-false',
+                'fill-in': 'fill-in-blank',
+                'matching': 'matching',
+                'performance': 'performance',
+                'sequencing': 'sequencing',
+                'likert': 'likert',
+                'numeric': 'numeric'
+              };
+
+              var scormType = interaction.type || 'unknown';
+              var questionType = typeMap[scormType] || scormType;
+
+              // Determine correctness from SCORM result value
+              // SCORM 1.2: 'correct', 'wrong', 'unanticipated', 'neutral'
+              // SCORM 2004: 'correct', 'incorrect', 'unanticipated', 'neutral'
+              var isCorrect = value === 'correct';
+
+              // Get student response \u2014 for 'choice' type, this might be indices like '1,3'
+              var studentResponse = interaction.student_response || interaction['student_response'] || '';
+
+              // Get correct response pattern
+              var correctResponse = '';
+              // SCORM 1.2: correct_responses.0.pattern
+              // Check stored fields for correct_responses
+              Object.keys(interaction).forEach(function(k) {
+                if (k.indexOf('correct_responses') > -1 && k.indexOf('pattern') > -1) {
+                  correctResponse = interaction[k];
+                }
+              });
+
+              // Get interaction ID (Rise sets this to a unique question identifier)
+              var interactionId = interaction.id || ('interaction-' + idx);
+
+              // Get lesson context
+              var lessonInfo = getCachedLessonInfo();
+
+              // Increment question counter
+              kcQuestionCounter++;
+
+              log('SCORM Interaction complete [' + idx + ']:', {
+                id: interactionId,
+                type: questionType,
+                response: studentResponse,
+                correctResponse: correctResponse,
+                result: value,
+                correct: isCorrect,
+                questionNumber: kcQuestionCounter
+              });
+
+              // Send xAPI answered statement
+              LRS.questionAnswered({
+                questionId: interactionId,
+                questionGuid: interactionId,
+                questionNumber: kcQuestionCounter,
+                questionText: 'Question ' + kcQuestionCounter + ' (' + interactionId + ')',
+                questionType: questionType,
+                answer: studentResponse,
+                correctAnswer: correctResponse,
+                correct: isCorrect,
+                result: isCorrect ? 'correct' : 'incorrect',
+                assessmentName: lessonInfo.name || 'Quiz',
+                lessonName: lessonInfo.name,
+                sectionName: lessonInfo.sectionName
+              });
+            }
+          }
+        }
+
+        return result;
+      };
+
+      scormTrackerActive = true;
+      log('SCORM Interaction Tracker: Wrapped ' + setValueFn + ' (apiType=' + apiType + ') \u2014 KC DOM handler will defer to SCORM tracker');
+    }
+
     function setupInteractionInterceptors() {
       if (!TRACK_INTERACTIONS) return;
 
@@ -4516,6 +4668,9 @@ function generateLrsBridgeCode(options) {
               LRS.scormApiFound = true;
               LRS.scormApiType = result.type;
 
+              // Wrap SCORM SetValue to intercept cmi.interactions for quiz tracking
+              setupScormInteractionTracker();
+
               // Read learner_id for diagnostics
               var scormLearnerId = 'n/a', scormLearnerName = 'n/a';
               try {