@patch-adams/core 1.5.1 → 1.5.3

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
@@ -4175,16 +4176,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 +4211,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 +4242,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),
@@ -5086,8 +5098,9 @@ function generateSkinCssLoader(options) {
 }
 function buildSkinCssOptions(config) {
   if (!config.skin) return null;
+  const cacheBuster = Date.now().toString(36) + Math.random().toString(36).slice(2, 6);
   return {
-    remoteUrl: `${config.remoteDomain}/skin/${config.skin}/style.css`,
+    remoteUrl: `${config.remoteDomain}/skin/${config.skin}/style.css?v=${cacheBuster}`,
     localPath: `skin/${config.skin}/style.css`,
     timeout: config.cssAfter.timeout
     // reuse cssAfter timeout
@@ -5187,8 +5200,9 @@ function generateSkinJsLoader(options) {
 }
 function buildSkinJsOptions(config) {
   if (!config.skin) return null;
+  const cacheBuster = Date.now().toString(36) + Math.random().toString(36).slice(2, 6);
   return {
-    remoteUrl: `${config.remoteDomain}/skin/${config.skin}/script.js`,
+    remoteUrl: `${config.remoteDomain}/skin/${config.skin}/script.js?v=${cacheBuster}`,
     localPath: `skin/${config.skin}/script.js`,
     timeout: config.jsAfter.timeout
     // reuse jsAfter timeout