lisichatbot 1.3.8 → 1.4.0

package/package.json

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

package/src/index.js

@@ -58,16 +58,18 @@ function scrollToBottom() {
 function enableNextButton() {
     if (elements.nextBtn) {
         elements.nextBtn.disabled = false;
-        elements.nextBtn.style.opacity = '1';
-        elements.nextBtn.style.cursor = 'pointer';
+        elements.nextBtn.style.setProperty('opacity', '1', 'important');
+        elements.nextBtn.style.setProperty('cursor', 'pointer', 'important');
+        console.log('  ✅ Next button ENABLED (opacity: 1, cursor: pointer)');
     }
 }
 
 function disableNextButton() {
     if (elements.nextBtn) {
         elements.nextBtn.disabled = true;
-        elements.nextBtn.style.opacity = '0.5';
-        elements.nextBtn.style.cursor = 'not-allowed';
+        elements.nextBtn.style.setProperty('opacity', '0.5', 'important');
+        elements.nextBtn.style.setProperty('cursor', 'not-allowed', 'important');
+        console.log('  🔒 Next button DISABLED (opacity: 0.5, cursor: not-allowed)');
     }
 }
 
@@ -1275,9 +1277,19 @@ function renderTextInput(field, inputType = 'text', inputConfig = {}) {
             }
         }
         
-        if (existingValue !== undefined && existingValue !== null) {
+        // ✅ FIX: Handle pre-filled values for both validation and non-validation cases
+        const hasValue = existingValue !== undefined && existingValue !== null && existingValue !== '';
+        
+        if (hasValue) {
             inputElement.value = existingValue;
-            console.log(`  ✅ Pre-filled with: ${existingValue}`);
+            console.log(`  ✅ Pre-filled with: "${existingValue}"`);
+            
+            // Set currentSelection for pre-filled value
+            chatState.currentSelection = { 
+                field, 
+                value: existingValue, 
+                name: existingValue.toString() 
+            };
             
             if (hasValidation) {
                 const value = parseFloat(existingValue);
@@ -1290,23 +1302,50 @@ function renderTextInput(field, inputType = 'text', inputConfig = {}) {
                 
                 if (isValid) {
                     enableNextButton();
-                    console.log(`  ✅ Pre-filled value is valid: ${value}`);
+                    console.log(`  ✅ Pre-filled number is valid - Next button enabled`);
                 } else {
                     disableNextButton();
-                    console.log(`  ❌ Pre-filled value is invalid: ${value}`);
+                    console.log(`  ❌ Pre-filled number is invalid - Next button disabled`);
                 }
+            } else {
+                // ✅ FIX: Enable button for pre-filled text input
+                enableNextButton();
+                console.log(`  ✅ Pre-filled text - Next button enabled`);
             }
         } else {
+            // ✅ No pre-filled value - set initial button state
             inputElement.value = '';
+            
+            const currentStep = flowData.flow[chatState.step];
+            const inputRequired = currentStep.inputRequired === true;
+            
+            if (inputRequired || hasValidation) {
+                disableNextButton();
+                console.log(`  🔒 Empty input (required or has validation) - Next button disabled`);
+            } else {
+                enableNextButton();
+                console.log(`  🔓 Empty input (optional, no validation) - Next button enabled`);
+            }
         }
         
         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;
+            
+            // ✅ FIX: Only set data and currentSelection if value is valid
+            const isEmpty = rawValue === '' || (inputType === 'number' && isNaN(value));
+            const isEmptyText = inputType === 'text' && (!rawValue || rawValue.trim() === '');
             
-            chatState.data[field] = value;
-            chatState.currentSelection = { field, value, name: value };
+            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;
@@ -1315,7 +1354,7 @@ function renderTextInput(field, inputType = 'text', inputConfig = {}) {
                 let isValid = true;
                 let errorMessage = '';
                 
-                if (value === '' || isNaN(value)) {
+                if (rawValue === '' || (inputType === 'number' && isNaN(value))) {
                     isValid = false;
                     errorMessage = 'Please enter a valid number';
                 } else if (min !== undefined && value < min) {
@@ -1328,13 +1367,51 @@ function renderTextInput(field, inputType = 'text', inputConfig = {}) {
                 
                 if (isValid) {
                     enableNextButton();
-                    console.log(`  ✅ Number input valid: ${value}`);
+                    console.log(`  ✅ Number input valid: ${value} - Next button enabled`);
                 } else {
                     disableNextButton();
-                    console.log(`  ❌ Number input invalid: ${errorMessage}`);
+                    console.log(`  ❌ Number input invalid: ${errorMessage} - Next button disabled`);
                 }
             } else {
-                console.log(`  📝 Input updated: ${value} (no validation)`);
+                // ✅ 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(`  ✅ 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`);
+                        }
+                    }
+                }
             }
         };
     }
@@ -1472,6 +1549,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;
@@ -1655,18 +1759,8 @@ async function showNextStep() {
         const inputType = nextStep.inputType || 'single-select';
         
         if (inputType === 'text' || inputType === 'number') {
+            // ✅ renderTextInput handles ALL button state logic (pre-filled, typing, validation)
             renderTextInput(nextStep.input.field, inputType, nextStep.input);
-            
-            const hasValidation = inputType === 'number' && 
-                (nextStep.input.min !== undefined || nextStep.input.max !== undefined);
-            
-            if (inputRequired || hasValidation) {
-                disableNextButton();
-                console.log('  🔒 Next button disabled initially (input required or validation enabled)');
-            } else {
-                enableNextButton();
-                console.log('  🔓 Next button enabled (optional input, no validation)');
-            }
         } else if (inputType === 'multi-select-color') {
             renderColorOptions(nextStep.input.options, nextStep.input.field);