lisichatbot 1.3.1 → 1.3.3

package/package.json

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

package/src/index.js

@@ -922,6 +922,9 @@ function handleCustomSelectClick(element, field, customConfig) {
 function validateMinMax(field, customConfig) {
     console.log(`\n🔍 === VALIDATING MIN/MAX for field: ${field} ===`);
     
+    // Always use global config errors
+    const errors = config.customRangeErrors;
+    
     // Fixed selectors - input has all attributes on it directly
     const minInput = document.querySelector(`[data-field="${field}"][data-input-type="min"][data-chat-input-element="input"]`);
     const maxInput = document.querySelector(`[data-field="${field}"][data-input-type="max"][data-chat-input-element="input"]`);
@@ -976,7 +979,7 @@ function validateMinMax(field, customConfig) {
     
     // 1. Check if only one is filled
     if (minFilled && !maxFilled) {
-        const error = config.customRangeErrors.maxRequired;
+        const error = errors.maxRequired;
         showErrorDiv(error);
         console.log(`❌ FAIL: Only min filled`);
         disableNextButton();
@@ -984,7 +987,7 @@ function validateMinMax(field, customConfig) {
     }
     
     if (!minFilled && maxFilled) {
-        const error = config.customRangeErrors.minRequired;
+        const error = errors.minRequired;
         showErrorDiv(error);
         console.log(`❌ FAIL: Only max filled`);
         disableNextButton();
@@ -1003,7 +1006,7 @@ function validateMinMax(field, customConfig) {
 
     // 3. Check if values are valid numbers
     if (isNaN(minValue) || isNaN(maxValue)) {
-        const error = config.customRangeErrors.bothRequired;
+        const error = errors.bothRequired;
         showErrorDiv(error);
         console.log(`❌ FAIL: Invalid numbers (min: ${minValue}, max: ${maxValue})`);
         disableNextButton();
@@ -1013,7 +1016,7 @@ function validateMinMax(field, customConfig) {
     // 4. Check min constraint
     console.log(`🔍 Checking: ${minValue} < ${minConstraint}?`);
     if (minValue < minConstraint) {
-        const error = config.customRangeErrors.minBelowConstraint.replace('{min}', minConstraint);
+        const error = errors.minBelowConstraint.replace('{min}', minConstraint);
         showErrorDiv(error);
         console.log(`❌ FAIL: Min (${minValue}) < constraint (${minConstraint})`);
         disableNextButton();
@@ -1024,7 +1027,7 @@ function validateMinMax(field, customConfig) {
     // 5. Check max constraint
     console.log(`🔍 Checking: ${maxValue} > ${maxConstraint}?`);
     if (maxValue > maxConstraint) {
-        const error = config.customRangeErrors.maxAboveConstraint.replace('{max}', maxConstraint);
+        const error = errors.maxAboveConstraint.replace('{max}', maxConstraint);
         showErrorDiv(error);
         console.log(`❌ FAIL: Max (${maxValue}) > constraint (${maxConstraint})`);
         disableNextButton();
@@ -1035,7 +1038,7 @@ function validateMinMax(field, customConfig) {
     // 6. Check min < max
     console.log(`🔍 Checking: ${minValue} >= ${maxValue}?`);
     if (minValue >= maxValue) {
-        const error = config.customRangeErrors.minGreaterThanMax;
+        const error = errors.minGreaterThanMax;
         showErrorDiv(error);
         console.log(`❌ FAIL: Min (${minValue}) >= Max (${maxValue})`);
         disableNextButton();
@@ -1380,6 +1383,7 @@ async function handleNext() {
         if (isCustomRange && customConfig) {
             // Validate the custom range one more time before proceeding
             console.log(`🔄 Running final validation before proceeding...`);
+            
             const validation = validateMinMax(field, customConfig);
             
             console.log(`Validation result:`, validation);
@@ -1620,27 +1624,19 @@ async function showNextStep() {
             }
         }
     } else {
-        // No input
-        const shouldAutoAdvance = nextStep.nextButtonDisplay !== false;
-        
-        if (shouldAutoAdvance) {
-            // Auto-advance for steps without input (default behavior)
-            setTimeout(() => {
-                chatState.step++;
-                
-                // Update edit icons after auto-advance
-                updateEditIcons();
-                
-                if (chatState.step < flowData.flow.length) {
-                    showNextStep();
-                } else {
-                    handleCompletion();
-                }
-            }, config.autoAdvanceDelay);
-        } else {
-            // Don't auto-advance if nextButtonDisplay is explicitly false
-            console.log('  ⏸️ Not auto-advancing (nextButtonDisplay: false)');
-        }
+        // No input - always auto-advance (nextButtonDisplay only controls button visibility)
+        setTimeout(() => {
+            chatState.step++;
+            
+            // Update edit icons after auto-advance
+            updateEditIcons();
+            
+            if (chatState.step < flowData.flow.length) {
+                showNextStep();
+            } else {
+                handleCompletion();
+            }
+        }, config.autoAdvanceDelay);
     }
     
     // Handle nextButtonDisplay - default is true
@@ -1698,7 +1694,7 @@ function handleCompletion() {
 // INITIALIZATION
 // =============================================================================
 
-function init(flowName, flowConfig) {
+function init(flowName, flowConfig, options = {}) {
     // Find container by data-chat-element="chat-wrapper"
     elements.container = document.querySelector('[data-chat-element="chat-wrapper"]');
     
@@ -1713,10 +1709,24 @@ function init(flowName, flowConfig) {
         Object.assign(config, flowConfig.config);
     }
     
+    // Merge customRangeErrors - priority: options param > flow object > defaults
+    // 1. First, try flow object customRangeErrors
+    if (flowConfig.customRangeErrors) {
+        console.log('✅ Using custom range errors from flow object');
+        Object.assign(config.customRangeErrors, flowConfig.customRangeErrors);
+    }
+    
+    // 2. Then, options parameter (overrides flow object if both provided)
+    if (options.customRangeErrors) {
+        console.log('✅ Using custom range errors from options parameter');
+        Object.assign(config.customRangeErrors, options.customRangeErrors);
+    }
+    
     console.log('✅ Flow initialized:', {
         flowName,
         selectedBackground: config.selectedBackground,
-        autoAdvanceDelay: config.autoAdvanceDelay
+        autoAdvanceDelay: config.autoAdvanceDelay,
+        customRangeErrors: config.customRangeErrors
     });
 
     // Store flow data
@@ -1843,15 +1853,10 @@ function editStep(stepNumber) {
     // Clear history from this step forward
     chatState.history = chatState.history.filter(h => h.step < stepNumber);
     
-    console.log(`   Resetting data for step ${stepNumber} and beyond...`);
-    // Reset data for this field and beyond
-    const stepsToReset = flowData.flow.slice(stepNumber);
-    stepsToReset.forEach(step => {
-        if (step.input && step.input.field) {
-            console.log(`     Deleting field: ${step.input.field}`);
-            delete chatState.data[step.input.field];
-        }
-    });
+    console.log(`   ✅ Keeping ALL data intact for editing (no data deleted)`);
+    console.log(`   Current data:`, chatState.data);
+    // Don't delete any data - keep everything for pre-filling
+    // Data will be replaced when user submits the edited step
 
     console.log(`   Calling goToStep(${stepNumber})...`);
     // Go to that step