lisichatbot 1.3.9 → 1.4.1

package/package.json

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

package/src/index.js

@@ -540,6 +540,8 @@ function renderOptions(options, field, isSingleSelect = true) {
     optionsWrapper.style.alignItems = 'flex-start';
     optionsWrapper.style.gap = '8px';
 
+    let hasPreselectedOptions = false; // ✅ Track if any options are pre-selected
+
     options.forEach((option, index) => {
         const optionName = option.name || option;
         const optionValue = option.value !== undefined ? option.value : option;
@@ -562,6 +564,7 @@ function renderOptions(options, field, isSingleSelect = true) {
             clone.classList.add('cf-checked');
             clone.style.backgroundColor = config.selectedBackground;
             console.log(`  ✅ Pre-selected: ${optionName}`);
+            hasPreselectedOptions = true; // ✅ Mark that we have pre-selections
         } else {
             clone.classList.remove('cf-checked');
             clone.style.backgroundColor = 'transparent';
@@ -610,6 +613,31 @@ function renderOptions(options, field, isSingleSelect = true) {
             handleOptionClick(el, field, isSingleSelect);
         };
     });
+
+    // ✅ FIX: Enable Next button if options are pre-selected
+    if (hasPreselectedOptions) {
+        // Set currentSelection for pre-selected options
+        if (isSingleSelect) {
+            chatState.currentSelection = {
+                field,
+                value: existingData,
+                name: options.find(o => JSON.stringify(o.value !== undefined ? o.value : o) === JSON.stringify(existingData))?.name || existingData
+            };
+        } else {
+            const selectedNames = existingData.map(val => {
+                const opt = options.find(o => JSON.stringify(o.value !== undefined ? o.value : o) === JSON.stringify(val));
+                return opt ? (opt.name || opt) : val;
+            }).join(', ');
+            chatState.currentSelection = {
+                field,
+                value: existingData,
+                name: selectedNames
+            };
+        }
+        
+        enableNextButton();
+        console.log(`  ✅ Pre-selected options found - Next button enabled`);
+    }
 }
 
 // =============================================================================
@@ -637,6 +665,8 @@ function renderColorOptions(options, field) {
     optionsWrapper.style.alignItems = 'flex-start';
     optionsWrapper.style.gap = '8px';
 
+    let hasPreselectedOptions = false; // ✅ Track if any options are pre-selected
+
     options.forEach((option, index) => {
         const optionName = option.name || option;
         const optionValue = option.value !== undefined ? option.value : option;
@@ -654,6 +684,7 @@ function renderColorOptions(options, field) {
             clone.classList.add('cf-checked');
             clone.style.backgroundColor = config.selectedBackground;
             console.log(`  ✅ Pre-selected color: ${optionName}`);
+            hasPreselectedOptions = true; // ✅ Mark that we have pre-selections
         } else {
             clone.classList.remove('cf-checked');
             clone.style.backgroundColor = 'transparent';
@@ -711,6 +742,24 @@ function renderColorOptions(options, field) {
             handleOptionClick(el, field, false);
         };
     });
+
+    // ✅ FIX: Enable Next button if options are pre-selected
+    if (hasPreselectedOptions) {
+        // Set currentSelection for pre-selected options
+        const selectedNames = existingData.map(val => {
+            const opt = options.find(o => JSON.stringify(o.value !== undefined ? o.value : o) === JSON.stringify(val));
+            return opt ? (opt.name || opt) : val;
+        }).join(', ');
+        
+        chatState.currentSelection = {
+            field,
+            value: existingData,
+            name: selectedNames
+        };
+        
+        enableNextButton();
+        console.log(`  ✅ Pre-selected color options found - Next button enabled`);
+    }
 }
 
 // =============================================================================
@@ -738,6 +787,8 @@ function renderCustomSelectOptions(options, field, customConfig) {
     optionsWrapper.style.alignItems = 'flex-start';
     optionsWrapper.style.gap = '8px';
 
+    let hasPreselectedOption = false; // ✅ Track if any option is pre-selected
+
     options.forEach((option, index) => {
         const optionName = option.name || option;
         const optionValue = option.value !== undefined ? option.value : option;
@@ -755,6 +806,7 @@ function renderCustomSelectOptions(options, field, customConfig) {
             clone.classList.add('cf-checked');
             clone.style.backgroundColor = config.selectedBackground;
             console.log(`  ✅ Pre-selected: ${optionName}`);
+            hasPreselectedOption = true; // ✅ Mark that we have a pre-selection
         } else {
             clone.classList.remove('cf-checked');
             clone.style.backgroundColor = 'transparent';
@@ -799,6 +851,7 @@ function renderCustomSelectOptions(options, field, customConfig) {
             customClone.classList.add('cf-checked');
             customClone.style.backgroundColor = config.selectedBackground;
             console.log(`  ✅ Pre-selected: Custom Range [${existingData[0]}, ${existingData[1]}]`);
+            hasPreselectedOption = true; // ✅ Mark that custom is pre-selected
         } else {
             customClone.classList.remove('cf-checked');
             customClone.style.backgroundColor = 'transparent';
@@ -842,6 +895,35 @@ function renderCustomSelectOptions(options, field, customConfig) {
             handleCustomSelectClick(el, field, customConfig);
         };
     });
+
+    // ✅ FIX: Enable Next button if option is pre-selected
+    if (hasPreselectedOption) {
+        // Set currentSelection
+        const isCustomSelected = Array.isArray(existingData) && existingData.length === 2;
+        
+        if (isCustomSelected) {
+            // Custom range selected - currentSelection will be set by validateMinMax
+            const validation = validateMinMax(field, customConfig);
+            if (validation.valid) {
+                enableNextButton();
+                console.log(`  ✅ Pre-selected custom range is valid - Next button enabled`);
+            }
+        } else {
+            // Regular option selected
+            const selectedOption = options.find(o => 
+                JSON.stringify(o.value !== undefined ? o.value : o) === JSON.stringify(existingData)
+            );
+            
+            chatState.currentSelection = {
+                field,
+                value: existingData,
+                name: selectedOption ? (selectedOption.name || selectedOption) : existingData
+            };
+            
+            enableNextButton();
+            console.log(`  ✅ Pre-selected option found - Next button enabled`);
+        }
+    }
 }
 
 function renderMinMaxInputs(field, customConfig, existingData) {
@@ -1331,10 +1413,21 @@ function renderTextInput(field, inputType = 'text', inputConfig = {}) {
         inputElement.type = inputType === 'number' ? 'number' : 'text';
         
         inputElement.oninput = (e) => {
-            const value = inputType === 'number' ? parseFloat(e.target.value) : e.target.value;
+            const rawValue = e.target.value;
+            const value = inputType === 'number' ? parseFloat(rawValue) : rawValue;
             
-            chatState.data[field] = value;
-            chatState.currentSelection = { field, value, name: value.toString() };
+            // ✅ FIX: Only set data and currentSelection if value is valid
+            const isEmpty = rawValue === '' || (inputType === 'number' && isNaN(value));
+            const isEmptyText = inputType === 'text' && (!rawValue || rawValue.trim() === '');
+            
+            if (!isEmpty && !isEmptyText) {
+                chatState.data[field] = value;
+                chatState.currentSelection = { field, value, name: value.toString() };
+            } else {
+                // Clear data and selection for empty values
+                delete chatState.data[field];
+                chatState.currentSelection = null;
+            }
             
             if (hasValidation) {
                 const min = inputConfig.min;
@@ -1343,7 +1436,7 @@ function renderTextInput(field, inputType = 'text', inputConfig = {}) {
                 let isValid = true;
                 let errorMessage = '';
                 
-                if (e.target.value === '' || (inputType === 'number' && isNaN(value))) {
+                if (rawValue === '' || (inputType === 'number' && isNaN(value))) {
                     isValid = false;
                     errorMessage = 'Please enter a valid number';
                 } else if (min !== undefined && value < min) {
@@ -1362,21 +1455,43 @@ function renderTextInput(field, inputType = 'text', inputConfig = {}) {
                     console.log(`  ❌ Number input invalid: ${errorMessage} - Next button disabled`);
                 }
             } else {
-                // ✅ FIX: For text inputs without validation, enable button as user types
-                if (e.target.value && e.target.value.trim() !== '') {
-                    enableNextButton();
-                    console.log(`  ✅ Text input has value - Next button enabled`);
-                } else {
-                    // ✅ Check if input is required
-                    const currentStep = flowData.flow[chatState.step];
-                    const inputRequired = currentStep.inputRequired === true;
-                    
-                    if (inputRequired) {
-                        disableNextButton();
-                        console.log(`  🔒 Text input empty and required - Next button disabled`);
+                // ✅ FIX: Handle number inputs without validation
+                if (inputType === 'number') {
+                    // For number inputs, check if value is valid
+                    if (rawValue === '' || isNaN(value)) {
+                        // Empty or NaN
+                        const currentStep = flowData.flow[chatState.step];
+                        const inputRequired = currentStep.inputRequired === true;
+                        
+                        if (inputRequired) {
+                            disableNextButton();
+                            console.log(`  🔒 Number input empty/NaN and required - Next button disabled`);
+                        } else {
+                            enableNextButton();
+                            console.log(`  🔓 Number input empty/NaN but not required - Next button enabled`);
+                        }
                     } else {
+                        // Has valid number value
                         enableNextButton();
-                        console.log(`  🔓 Text input empty but not required - Next button enabled`);
+                        console.log(`  ✅ Number input has value: ${value} - Next button enabled`);
+                    }
+                } else {
+                    // Text input logic
+                    if (rawValue && rawValue.trim() !== '') {
+                        enableNextButton();
+                        console.log(`  ✅ Text input has value - Next button enabled`);
+                    } else {
+                        // ✅ Check if input is required
+                        const currentStep = flowData.flow[chatState.step];
+                        const inputRequired = currentStep.inputRequired === true;
+                        
+                        if (inputRequired) {
+                            disableNextButton();
+                            console.log(`  🔒 Text input empty and required - Next button disabled`);
+                        } else {
+                            enableNextButton();
+                            console.log(`  🔓 Text input empty but not required - Next button enabled`);
+                        }
                     }
                 }
             }
@@ -1516,6 +1631,33 @@ async function handleNext() {
         console.log('Selection required but none made');
         return;
     }
+    
+    // ✅ FIX: Validate text/number inputs when required
+    if (currentStep.input && inputRequired) {
+        const inputType = currentStep.inputType || 'single-select';
+        
+        if (inputType === 'text' || inputType === 'number') {
+            const field = currentStep.input.field;
+            const value = chatState.data[field];
+            
+            // Check for empty or invalid values
+            if (inputType === 'text') {
+                if (value === undefined || value === null || value === '' || (typeof value === 'string' && value.trim() === '')) {
+                    console.log('❌ Text input required but empty - cannot proceed');
+                    disableNextButton();
+                    return;
+                }
+            } else if (inputType === 'number') {
+                if (value === undefined || value === null || value === '' || isNaN(value)) {
+                    console.log('❌ Number input required but empty/NaN - cannot proceed');
+                    disableNextButton();
+                    return;
+                }
+            }
+            
+            console.log(`✅ ${inputType} input validation passed:`, value);
+        }
+    }
 
     if (currentStep.inputType === 'single-select-custom' && currentStep.input) {
         const field = currentStep.input.field;
@@ -1704,9 +1846,19 @@ async function showNextStep() {
         } else if (inputType === 'multi-select-color') {
             renderColorOptions(nextStep.input.options, nextStep.input.field);
             
-            if (!inputRequired) {
+            // ✅ Check if pre-filled - renderColorOptions handles enabling if pre-filled
+            // Only disable if NOT pre-filled and required
+            const existingData = chatState.data[nextStep.input.field];
+            const hasPreFill = Array.isArray(existingData) && existingData.length > 0;
+            
+            if (inputRequired && !hasPreFill) {
+                disableNextButton();
+                console.log('  🔒 Multi-select-color required with no pre-fill - Next button disabled');
+            } else if (!inputRequired && !hasPreFill) {
                 enableNextButton();
+                console.log('  🔓 Multi-select-color not required - Next button enabled');
             }
+            // If hasPreFill, renderColorOptions already handled it
         } else if (inputType === 'single-select-custom') {
             renderCustomSelectOptions(
                 nextStep.input.options, 
@@ -1714,11 +1866,18 @@ async function showNextStep() {
                 nextStep.input.custom
             );
             
-            if (inputRequired) {
+            // ✅ Check if pre-filled - renderCustomSelectOptions handles enabling if pre-filled
+            const existingData = chatState.data[nextStep.input.field];
+            const hasPreFill = existingData !== undefined && existingData !== null;
+            
+            if (inputRequired && !hasPreFill) {
                 disableNextButton();
-            } else {
+                console.log('  🔒 Single-select-custom required with no pre-fill - Next button disabled');
+            } else if (!inputRequired && !hasPreFill) {
                 enableNextButton();
+                console.log('  🔓 Single-select-custom not required - Next button enabled');
             }
+            // If hasPreFill, renderCustomSelectOptions already handled it
         } else if (inputType === 'multi-select-dropdown') {
             // ✅ Render multi-select dropdown
             renderMultiSelectDropdown(nextStep.input.options, nextStep.input.field);
@@ -1735,9 +1894,24 @@ async function showNextStep() {
             const isSingleSelect = inputType === 'single-select';
             renderOptions(nextStep.input.options, nextStep.input.field, isSingleSelect);
             
-            if (!inputRequired) {
+            // ✅ Check if pre-filled - renderOptions handles enabling if pre-filled
+            const existingData = chatState.data[nextStep.input.field];
+            let hasPreFill = false;
+            
+            if (isSingleSelect) {
+                hasPreFill = existingData !== undefined && existingData !== null;
+            } else {
+                hasPreFill = Array.isArray(existingData) && existingData.length > 0;
+            }
+            
+            if (inputRequired && !hasPreFill) {
+                disableNextButton();
+                console.log(`  🔒 ${inputType} required with no pre-fill - Next button disabled`);
+            } else if (!inputRequired && !hasPreFill) {
                 enableNextButton();
+                console.log(`  🔓 ${inputType} not required - Next button enabled`);
             }
+            // If hasPreFill, renderOptions already handled it
         }
     } else {
         const delay = nextStep.autoAdvanceDelay !== undefined ? nextStep.autoAdvanceDelay : config.autoAdvanceDelay;