lisichatbot 1.6.0 → 1.6.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "lisichatbot",
-  "version": "1.6.0",
+  "version": "1.6.2",
   "type": "module",
   "main": "./src/index.js",
   "exports": {

package/src/index.js

@@ -49,9 +49,14 @@ let flowData = null;
 
 function scrollToBottom() {
     if (elements.messages) {
-        setTimeout(() => {
-            elements.messages.scrollTop = elements.messages.scrollHeight;
-        }, 100);
+        // ✅ Use requestAnimationFrame to ensure DOM has updated
+        requestAnimationFrame(() => {
+            requestAnimationFrame(() => {
+                // Double RAF ensures layout is complete
+                elements.messages.scrollTop = elements.messages.scrollHeight;
+                console.log(`  📜 Scrolled to bottom: ${elements.messages.scrollHeight}px`);
+            });
+        });
     }
 }
 
@@ -293,7 +298,8 @@ function renderMultiSelectDropdown(options, field) {
     // ✅ Get existing data for pre-filling (edit mode)
     // ✅ NEW: Check if prefill is disabled for this step
     const currentStep = flowData.flow[chatState.step];
-    const disablePrefill = currentStep?.disableInputValuePrefill === true && !chatState.prefillOverride;
+    const hasOverride = chatState.prefillOverrideFields && chatState.prefillOverrideFields.includes(field);
+    const disablePrefill = currentStep?.disableInputValuePrefill === true && !hasOverride;
     
     const existingData = disablePrefill ? [] : (chatState.data[field] || []);
     const selectedValues = new Set(existingData);
@@ -301,8 +307,8 @@ function renderMultiSelectDropdown(options, field) {
     if (disablePrefill) {
         console.log(`⏭️ Pre-fill disabled for ${field} - starting fresh`);
     } else {
-        if (chatState.prefillOverride) {
-            console.log(`   🔓 Pre-fill override active - filling ${field}:`, Array.from(selectedValues));
+        if (hasOverride) {
+            console.log(`   🔓 Pre-fill override active for ${field}:`, Array.from(selectedValues));
         } else {
             console.log(`📝 Pre-filling selections for ${field}:`, Array.from(selectedValues));
         }
@@ -556,15 +562,16 @@ function renderOptions(options, field, isSingleSelect = true) {
 
     // ✅ NEW: Check if prefill is disabled for this step
     const currentStep = flowData.flow[chatState.step];
-    const disablePrefill = currentStep?.disableInputValuePrefill === true && !chatState.prefillOverride;
+    const hasOverride = chatState.prefillOverrideFields && chatState.prefillOverrideFields.includes(field);
+    const disablePrefill = currentStep?.disableInputValuePrefill === true && !hasOverride;
     
     const existingData = disablePrefill ? (isSingleSelect ? null : []) : chatState.data[field];
     
     if (disablePrefill) {
         console.log(`⏭️ Pre-fill disabled for ${field} - starting fresh`);
     } else {
-        if (chatState.prefillOverride) {
-            console.log(`   🔓 Pre-fill override active - filling ${field}:`, existingData);
+        if (hasOverride) {
+            console.log(`   🔓 Pre-fill override active for ${field}:`, existingData);
         } else {
             console.log(`📝 Pre-filling ${field}:`, existingData);
         }
@@ -694,15 +701,16 @@ function renderColorOptions(options, field) {
 
     // ✅ NEW: Check if prefill is disabled for this step
     const currentStep = flowData.flow[chatState.step];
-    const disablePrefill = currentStep?.disableInputValuePrefill === true && !chatState.prefillOverride;
+    const hasOverride = chatState.prefillOverrideFields && chatState.prefillOverrideFields.includes(field);
+    const disablePrefill = currentStep?.disableInputValuePrefill === true && !hasOverride;
     
     const existingData = disablePrefill ? [] : chatState.data[field];
     
     if (disablePrefill) {
         console.log(`⏭️ Pre-fill disabled for ${field} (color) - starting fresh`);
     } else {
-        if (chatState.prefillOverride) {
-            console.log(`   🔓 Pre-fill override active - filling ${field} (color):`, existingData);
+        if (hasOverride) {
+            console.log(`   🔓 Pre-fill override active for ${field} (color):`, existingData);
         } else {
             console.log(`📝 Pre-filling ${field} (color):`, existingData);
         }
@@ -829,15 +837,16 @@ function renderCustomSelectOptions(options, field, customConfig) {
 
     // ✅ NEW: Check if prefill is disabled for this step
     const currentStep = flowData.flow[chatState.step];
-    const disablePrefill = currentStep?.disableInputValuePrefill === true && !chatState.prefillOverride;
+    const hasOverride = chatState.prefillOverrideFields && chatState.prefillOverrideFields.includes(field);
+    const disablePrefill = currentStep?.disableInputValuePrefill === true && !hasOverride;
     
     const existingData = disablePrefill ? null : chatState.data[field];
     
     if (disablePrefill) {
         console.log(`⏭️ Pre-fill disabled for ${field} (custom) - starting fresh`);
     } else {
-        if (chatState.prefillOverride) {
-            console.log(`   🔓 Pre-fill override active - filling ${field} (custom):`, existingData);
+        if (hasOverride) {
+            console.log(`   🔓 Pre-fill override active for ${field} (custom):`, existingData);
         } else {
             console.log(`📝 Pre-filling ${field} (custom):`, existingData);
         }
@@ -1455,15 +1464,16 @@ function renderTextInput(field, inputType = 'text', inputConfig = {}) {
 
     // ✅ NEW: Check if prefill is disabled for this step
     const currentStep = flowData.flow[chatState.step];
-    const disablePrefill = currentStep?.disableInputValuePrefill === true && !chatState.prefillOverride;
+    const hasOverride = chatState.prefillOverrideFields && chatState.prefillOverrideFields.includes(field);
+    const disablePrefill = currentStep?.disableInputValuePrefill === true && !hasOverride;
     
     const existingValue = disablePrefill ? null : chatState.data[field];
     
     if (disablePrefill) {
         console.log(`⏭️ Pre-fill disabled for ${field} - starting fresh`);
     } else {
-        if (chatState.prefillOverride) {
-            console.log(`   🔓 Pre-fill override active - filling ${field}:`, existingValue);
+        if (hasOverride) {
+            console.log(`   🔓 Pre-fill override active for ${field}:`, existingValue);
         } else {
             console.log(`📝 Pre-filling ${field}:`, existingValue);
         }
@@ -2001,6 +2011,10 @@ async function handleNext() {
                 console.log('💬 showMessage called:', message);
                 addMessage(message, 'bot', false, null);
                 scrollToBottom();
+                // ✅ Delayed scroll to ensure message is fully rendered
+                setTimeout(() => {
+                    scrollToBottom();
+                }, 100);
             };
             
             // ✅ Pass showMessage as third parameter
@@ -2015,6 +2029,10 @@ async function handleNext() {
                 console.log('💬 onNext displaying message from return:', result.showMessage);
                 addMessage(result.showMessage, 'bot', false, null);
                 scrollToBottom();
+                // ✅ Delayed scroll to ensure message is fully rendered
+                setTimeout(() => {
+                    scrollToBottom();
+                }, 100);
             }
             
             // ✅ NEW: Check if onNext wants to update steps by field name
@@ -2091,8 +2109,9 @@ async function handleNext() {
                             console.log(`      → Set ${key}:`, result.prefillData[key]);
                         });
                         
-                        // ✅ Store prefill override flag
-                        chatState.prefillOverride = true;
+                        // ✅ Store which fields should override disableInputValuePrefill
+                        chatState.prefillOverrideFields = Object.keys(result.prefillData);
+                        console.log(`   🔓 Override active for fields:`, chatState.prefillOverrideFields);
                     }
                     
                     chatState.step = targetStepIndex;
@@ -2122,8 +2141,9 @@ async function handleNext() {
                             console.log(`      → Set ${key}:`, result.prefillData[key]);
                         });
                         
-                        // ✅ Store prefill override flag
-                        chatState.prefillOverride = true;
+                        // ✅ Store which fields should override disableInputValuePrefill
+                        chatState.prefillOverrideFields = Object.keys(result.prefillData);
+                        console.log(`   🔓 Override active for fields:`, chatState.prefillOverrideFields);
                     }
                     
                     chatState.step = targetStep;
@@ -2336,7 +2356,8 @@ async function handleNext() {
         }
     }
 
-    chatState.step++;
+    // ✅ NEW: Find next accessible step (skip accessibleViaJumpOnly steps)
+    chatState.step = findNextAccessibleStep(chatState.step);
     
     const allRangeWrappers = document.querySelectorAll('[data-chat-element="range-wrapper"]');
     allRangeWrappers.forEach(wrapper => {
@@ -2354,6 +2375,24 @@ async function handleNext() {
     await showNextStep();
 }
 
+// ✅ NEW: Helper function to find next step that is accessible in normal flow
+function findNextAccessibleStep(currentStep) {
+    let nextStep = currentStep + 1;
+    
+    while (nextStep < flowData.flow.length) {
+        const stepConfig = flowData.flow[nextStep];
+        
+        if (stepConfig.accessibleViaJumpOnly === true) {
+            console.log(`  ⏭️ Skipping step ${nextStep} (accessibleViaJumpOnly)`);
+            nextStep++;
+        } else {
+            return nextStep;
+        }
+    }
+    
+    return nextStep;  // Return even if >= flow.length (will be handled by caller)
+}
+
 async function showNextStep() {
     const nextStep = flowData.flow[chatState.step];
 
@@ -2400,6 +2439,10 @@ async function showNextStep() {
                 console.log('💬 showMessage called:', message);
                 addMessage(message, 'bot', false, null);
                 scrollToBottom();
+                // ✅ Delayed scroll to ensure message is fully rendered
+                setTimeout(() => {
+                    scrollToBottom();
+                }, 100);
             };
             
             // ✅ Pass showMessage as second parameter
@@ -2410,6 +2453,10 @@ async function showNextStep() {
                 console.log('💬 onStart displaying message from return:', result.showMessage);
                 addMessage(result.showMessage, 'bot', false, null);
                 scrollToBottom();
+                // ✅ Delayed scroll to ensure message is fully rendered
+                setTimeout(() => {
+                    scrollToBottom();
+                }, 100);
             }
             
             // ✅ NEW: Check if onStart wants to update steps by field name
@@ -2580,8 +2627,9 @@ async function showNextStep() {
                             console.log(`      → Set ${key}:`, result.prefillData[key]);
                         });
                         
-                        // ✅ Store prefill override flag
-                        chatState.prefillOverride = true;
+                        // ✅ Store which fields should override disableInputValuePrefill
+                        chatState.prefillOverrideFields = Object.keys(result.prefillData);
+                        console.log(`   🔓 Override active for fields:`, chatState.prefillOverrideFields);
                     }
                     
                     chatState.step = targetStepIndex;
@@ -2611,8 +2659,9 @@ async function showNextStep() {
                             console.log(`      → Set ${key}:`, result.prefillData[key]);
                         });
                         
-                        // ✅ Store prefill override flag
-                        chatState.prefillOverride = true;
+                        // ✅ Store which fields should override disableInputValuePrefill
+                        chatState.prefillOverrideFields = Object.keys(result.prefillData);
+                        console.log(`   🔓 Override active for fields:`, chatState.prefillOverrideFields);
                     }
                     
                     chatState.step = targetStep;
@@ -2628,7 +2677,8 @@ async function showNextStep() {
             // ✅ NEW: Check if onStart wants to block this step and skip to next
             if (result && typeof result === 'object' && result.blockStep === true) {
                 console.log('⏭️ Step blocked by onStart - skipping to next step');
-                chatState.step++;
+                // ✅ NEW: Find next accessible step
+                chatState.step = findNextAccessibleStep(chatState.step);
                 
                 if (chatState.step >= flowData.flow.length) {
                     handleCompletion();
@@ -2776,7 +2826,8 @@ async function showNextStep() {
         console.log(`  ⏱️ Auto-advance delay: ${delay}ms ${nextStep.autoAdvanceDelay !== undefined ? '(step-level)' : '(global)'}`);
         
         setTimeout(() => {
-            chatState.step++;
+            // ✅ NEW: Find next accessible step
+            chatState.step = findNextAccessibleStep(chatState.step);
             updateEditIcons();
             
             if (chatState.step < flowData.flow.length) {
@@ -2815,16 +2866,30 @@ async function showNextStep() {
         }
     }
     
-    // ✅ Clear prefillOverride flag after rendering this step
-    // This ensures it only applies to the first step after goToStep with prefillData
-    if (chatState.prefillOverride) {
-        console.log(`   🔓 Clearing prefill override flag`);
-        chatState.prefillOverride = false;
+    // ✅ Remove current field from prefillOverride list after rendering
+    // This ensures override only applies to fields specified in prefillData
+    if (chatState.prefillOverrideFields && chatState.prefillOverrideFields.length > 0) {
+        const currentField = nextStep.input?.field;
+        if (currentField && chatState.prefillOverrideFields.includes(currentField)) {
+            chatState.prefillOverrideFields = chatState.prefillOverrideFields.filter(f => f !== currentField);
+            console.log(`   🔓 Removed ${currentField} from override list. Remaining:`, chatState.prefillOverrideFields);
+            
+            // Clear the list entirely if empty
+            if (chatState.prefillOverrideFields.length === 0) {
+                console.log(`   🔓 All override fields processed - clearing list`);
+                delete chatState.prefillOverrideFields;
+            }
+        }
     }
     
     setTimeout(() => {
         updateEditIcons();
     }, 10);
+    
+    // ✅ NEW: Delayed scroll to ensure inputs are fully rendered and visible
+    setTimeout(() => {
+        scrollToBottom();
+    }, 150);
 }
 
 function handleCompletion() {