lisichatbot 1.2.1 → 1.2.3

package/package.json

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

package/src/index.js

@@ -16,7 +16,8 @@ let chatState = {
     data: {},
     history: [],
     currentSelection: null,
-    returnToStep: null  // Track where to return after editing
+    returnToStep: null,  // Track where to return after editing
+    completed: false     // Track if flow is completed
 };
 
 let elements = {
@@ -89,6 +90,17 @@ function addMessage(content, type = 'bot', hasInput = false, stepNumber = null)
     // Add step number if provided
     if (stepNumber !== null) {
         clone.setAttribute('data-chat-step', stepNumber);
+        
+        // Mark all previous instances of this step as NOT latest
+        const previousInstances = elements.messages.querySelectorAll(
+            `[data-chat-element="${type}-message-wrapper"][data-chat-step="${stepNumber}"]`
+        );
+        previousInstances.forEach(prev => {
+            prev.removeAttribute('data-chat-latest');
+        });
+        
+        // Mark this as the latest instance of this step
+        clone.setAttribute('data-chat-latest', 'true');
     }
 
     // Find text element in clone and set content
@@ -111,8 +123,11 @@ function addMessage(content, type = 'bot', hasInput = false, stepNumber = null)
             editIcon.style.setProperty('display', 'none', 'important');
             console.log('  ✓ Edit icon hidden (display: none !important)');
             
-            // Only show if this step has input AND it's a PREVIOUS step (not current)
-            if (hasInput && stepNumber !== null && stepNumber < chatState.step) {
+            // Check if this is the latest instance (it should be since we just marked it)
+            const isLatest = clone.hasAttribute('data-chat-latest');
+            
+            // Only show if this step has input AND it's a PREVIOUS step AND it's the latest instance
+            if (hasInput && stepNumber !== null && stepNumber < chatState.step && isLatest) {
                 editIcon.setAttribute('data-chat-step', stepNumber);
                 editIcon.onclick = (e) => {
                     e.stopPropagation();
@@ -125,9 +140,11 @@ function addMessage(content, type = 'bot', hasInput = false, stepNumber = null)
                 editIcon.style.setProperty('display', 'block', 'important');
                 editIcon.style.setProperty('margin-left', '8px', 'important');
                 editIcon.style.setProperty('cursor', 'pointer', 'important');
-                console.log(`  ✏️ Edit icon SHOWN (step ${stepNumber} is previous step with input)`);
+                console.log(`  ✏️ Edit icon SHOWN (step ${stepNumber} is latest previous step with input)`);
             } else if (hasInput && stepNumber === chatState.step) {
                 console.log(`  ⏸️ Edit icon HIDDEN (step ${stepNumber} is CURRENT step)`);
+            } else if (!isLatest && hasInput && stepNumber < chatState.step) {
+                console.log(`  📜 Edit icon HIDDEN (step ${stepNumber} is OLD instance)`);
             } else {
                 console.log(`  ⚪ Edit icon HIDDEN (step ${stepNumber || 'N/A'} has no input)`);
             }
@@ -155,8 +172,10 @@ function addMessage(content, type = 'bot', hasInput = false, stepNumber = null)
             editIconAfterAppend.style.setProperty('display', 'none', 'important');
             editIconAfterAppend.style.setProperty('margin-left', '8px', 'important');
             
-            // Only show if has input AND it's a previous step
-            if (hasInput && stepNumber !== null && stepNumber < chatState.step) {
+            const isLatest = clone.hasAttribute('data-chat-latest');
+            
+            // Only show if has input AND it's a previous step AND it's latest instance
+            if (hasInput && stepNumber !== null && stepNumber < chatState.step && isLatest) {
                 editIconAfterAppend.style.setProperty('display', 'block', 'important');
                 
                 // Debug: Check spacing
@@ -180,19 +199,27 @@ function updateEditIcons() {
     // Find all rendered bot message wrappers
     const allBotMessages = elements.messages.querySelectorAll('[data-chat-element="bot-message-wrapper"]');
     
-    console.log(`\n🔄 Updating edit icons - Current step: ${chatState.step}`);
+    console.log(`\n🔄 Updating edit icons - Current step: ${chatState.step}, Completed: ${chatState.completed}`);
     
     allBotMessages.forEach(wrapper => {
         const stepNumber = parseInt(wrapper.getAttribute('data-chat-step'));
         const editIcon = wrapper.querySelector('[data-chat-element="bot-edit-icon"]');
+        const isLatest = wrapper.hasAttribute('data-chat-latest');
         
         if (editIcon && !isNaN(stepNumber)) {
             // Check if this step has input by looking at the flow
             const stepData = flowData.flow[stepNumber];
             const hasInput = stepData && !!stepData.input;
             
-            // Show icon if: has input AND is previous step
-            if (hasInput && stepNumber < chatState.step) {
+            // Hide ALL icons if flow is completed
+            if (chatState.completed) {
+                editIcon.style.setProperty('display', 'none', 'important');
+                console.log(`  🏁 Step ${stepNumber}: Icon HIDDEN (flow completed)`);
+                return;
+            }
+            
+            // Only show icon if: has input AND is previous step AND is latest instance
+            if (hasInput && stepNumber < chatState.step && isLatest) {
                 // Set onclick handler
                 editIcon.onclick = (e) => {
                     e.stopPropagation();
@@ -206,10 +233,12 @@ function updateEditIcons() {
                 editIcon.style.setProperty('display', 'block', 'important');
                 editIcon.style.setProperty('cursor', 'pointer', 'important');
                 editIcon.style.setProperty('margin-left', '8px', 'important');
-                console.log(`  ✏️ Step ${stepNumber}: Icon SHOWN (previous step)`);
+                console.log(`  ✏️ Step ${stepNumber}: Icon SHOWN (latest previous step)`);
             } else {
                 editIcon.style.setProperty('display', 'none', 'important');
-                if (stepNumber === chatState.step) {
+                if (!isLatest && hasInput && stepNumber < chatState.step) {
+                    console.log(`  📜 Step ${stepNumber}: Icon HIDDEN (old instance)`);
+                } else if (stepNumber === chatState.step) {
                     console.log(`  ⏸️ Step ${stepNumber}: Icon HIDDEN (current step)`);
                 } else if (!hasInput) {
                     console.log(`  ⚪ Step ${stepNumber}: Icon HIDDEN (no input)`);
@@ -675,6 +704,13 @@ async function showNextStep() {
 function handleCompletion() {
     console.log('✅ Flow completed. Data:', chatState.data);
 
+    // Mark flow as completed
+    chatState.completed = true;
+    console.log('   🏁 Flow marked as completed');
+    
+    // Hide all edit icons
+    updateEditIcons();
+
     // Hide Next button
     if (elements.nextBtn) {
         elements.nextBtn.style.display = 'none';
@@ -814,10 +850,9 @@ function goToStep(stepNumber) {
     chatState.step = stepNumber;
     chatState.currentSelection = null;
     
-    console.log(`   Clearing messages container...`);
-    if (elements.messages) {
-        elements.messages.innerHTML = '';
-    }
+    // DON'T clear messages - keep chat history visible!
+    // User can see their previous answers
+    console.log(`   Keeping messages visible (not clearing)...`);
 
     console.log(`   Disabling next button and calling showNextStep()...`);
     disableNextButton();
@@ -836,6 +871,12 @@ function editStep(stepNumber) {
     // Save where we are now so we can return after editing
     chatState.returnToStep = chatState.step;
     console.log(`   💾 Saved return step: ${chatState.returnToStep}`);
+    
+    // Reset completed flag since we're editing
+    if (chatState.completed) {
+        chatState.completed = false;
+        console.log(`   🔓 Flow unmarked as completed (editing)`);
+    }
 
     console.log(`   Clearing history from step ${stepNumber} forward...`);
     // Clear history from this step forward