@2112-lab/central-plant 0.3.9 → 0.3.10

package/dist/bundle/index.js

@@ -37716,39 +37716,152 @@ var CentralPlantInternals = /*#__PURE__*/function () {
         return false;
       }
       try {
+        var _this$centralPlant$sc0, _this$centralPlant$sc1, _sceneData$scene2, _sceneData$scene3;
         console.log("\uD83D\uDDD1\uFE0F deleteComponent(): Deleting component ".concat(componentId));
+        var threeScene = (_this$centralPlant$sc0 = this.centralPlant.sceneViewer) === null || _this$centralPlant$sc0 === void 0 ? void 0 : _this$centralPlant$sc0.scene;
+        var sceneData = (_this$centralPlant$sc1 = this.centralPlant.sceneViewer) === null || _this$centralPlant$sc1 === void 0 ? void 0 : _this$centralPlant$sc1.currentSceneData;
+
+        // Step 1: Resolve the actual Three.js UUID from componentId.
+        // The UI emits object.name (e.g. "Pump (PUMP-1)") as the selection ID, but
+        // sceneData.scene.children stores the raw uuid (e.g. "PUMP-1").  We must
+        // find the live Three.js object first so we can use its true .uuid for all
+        // sceneData lookups.
+        var resolvedUuid = componentId;
+        if (threeScene) {
+          var threeObj = findObjectByHardcodedUuid(threeScene, componentId);
+          if (!threeObj) threeObj = threeScene.getObjectByProperty('uuid', componentId);
+          if (!threeObj) threeObj = threeScene.getObjectByProperty('name', componentId);
+          if (threeObj) {
+            var _threeObj$userData;
+            resolvedUuid = ((_threeObj$userData = threeObj.userData) === null || _threeObj$userData === void 0 ? void 0 : _threeObj$userData.originalUuid) || threeObj.uuid || componentId;
+            console.log("\uD83D\uDD0D deleteComponent(): Resolved UUID \"".concat(componentId, "\" \u2192 \"").concat(resolvedUuid, "\""));
+          }
+        }
+
+        // Step 2: Collect connector UUIDs from sceneData.scene.children FIRST.
+        var connectorIds = new Set();
+        var sceneDataComponentIndex = -1;
+        if (sceneData !== null && sceneData !== void 0 && (_sceneData$scene2 = sceneData.scene) !== null && _sceneData$scene2 !== void 0 && _sceneData$scene2.children) {
+          sceneDataComponentIndex = sceneData.scene.children.findIndex(function (c) {
+            var _c$userData2;
+            return c.uuid === resolvedUuid || ((_c$userData2 = c.userData) === null || _c$userData2 === void 0 ? void 0 : _c$userData2.originalUuid) === resolvedUuid;
+          });
+          if (sceneDataComponentIndex !== -1) {
+            var componentNode = sceneData.scene.children[sceneDataComponentIndex];
+            if (Array.isArray(componentNode.children)) {
+              componentNode.children.forEach(function (child) {
+                var _child$userData2;
+                if (((_child$userData2 = child.userData) === null || _child$userData2 === void 0 ? void 0 : _child$userData2.objectType) === 'connector' && child.uuid) {
+                  connectorIds.add(child.uuid);
+                }
+              });
+            }
+          }
+        }
 
-        // Use componentManager to remove the component
+        // Fallback: also traverse the live Three.js scene in case sceneData is missing
+        // the component (e.g., dynamically added but not yet synced to sceneData)
+        if (connectorIds.size === 0 && threeScene) {
+          threeScene.traverse(function (obj) {
+            var _obj$userData2, _obj$userData3;
+            if ((obj.uuid === resolvedUuid || ((_obj$userData2 = obj.userData) === null || _obj$userData2 === void 0 ? void 0 : _obj$userData2.originalUuid) === resolvedUuid) && ((_obj$userData3 = obj.userData) === null || _obj$userData3 === void 0 ? void 0 : _obj$userData3.objectType) === 'component') {
+              obj.children.forEach(function (child) {
+                var _child$userData3;
+                if (((_child$userData3 = child.userData) === null || _child$userData3 === void 0 ? void 0 : _child$userData3.objectType) === 'connector') {
+                  connectorIds.add(child.uuid);
+                }
+              });
+            }
+          });
+        }
+        console.log("\uD83D\uDD0C deleteComponent(): Found ".concat(connectorIds.size, " connector(s) for component ").concat(componentId));
+
+        // Step 3: Remove connections from sceneData BEFORE removing from Three.js
+        // so the pathfinder fingerprint reflects the updated state on next updatePaths() call.
+        if (connectorIds.size > 0 && sceneData && Array.isArray(sceneData.connections)) {
+          var beforeCount = sceneData.connections.length;
+          sceneData.connections = sceneData.connections.filter(function (conn) {
+            return !connectorIds.has(conn.from) && !connectorIds.has(conn.to);
+          });
+          var removedCount = beforeCount - sceneData.connections.length;
+          if (removedCount > 0) {
+            console.log("\uD83D\uDD17 deleteComponent(): Removed ".concat(removedCount, " connection(s) for deleted component"));
+          }
+        }
+
+        // Step 4: Remove component from sceneData.scene.children BEFORE removing from Three.js
+        if (sceneDataComponentIndex !== -1) {
+          sceneData.scene.children.splice(sceneDataComponentIndex, 1);
+          console.log('✅ Removed component from sceneData.scene.children');
+        } else if (sceneData !== null && sceneData !== void 0 && (_sceneData$scene3 = sceneData.scene) !== null && _sceneData$scene3 !== void 0 && _sceneData$scene3.children) {
+          // Retry in case findIndex above didn't run (e.g. no sceneData at that point)
+          var retryIndex = sceneData.scene.children.findIndex(function (c) {
+            var _c$userData3;
+            return c.uuid === resolvedUuid || ((_c$userData3 = c.userData) === null || _c$userData3 === void 0 ? void 0 : _c$userData3.originalUuid) === resolvedUuid;
+          });
+          if (retryIndex !== -1) {
+            sceneData.scene.children.splice(retryIndex, 1);
+            console.log('✅ Removed component from sceneData.scene.children (retry)');
+          }
+        }
+
+        // Step 5: Remove the component from the Three.js scene
         var success = componentManager.removeComponentFromScene(componentId);
         if (success) {
-          // Also remove from scene data if available
-          if (this.centralPlant.sceneViewer.sceneOperationsManager && this.centralPlant.sceneViewer.currentSceneData) {
-            // Note: componentManager.removeComponentFromScene might not update sceneData
-            // We should verify if sceneOperationsManager has a method for this or if we need to manually update
-            // For now, assuming componentManager handles the scene object removal, 
-            // we might need to update the data structure manually if componentManager doesn't.
-
-            // Let's implement a safe removal from sceneData here just in case
-            var sceneData = this.centralPlant.sceneViewer.currentSceneData;
-            if (sceneData.scene && sceneData.scene.children) {
-              var index = sceneData.scene.children.findIndex(function (c) {
-                var _c$userData2;
-                return c.uuid === componentId || ((_c$userData2 = c.userData) === null || _c$userData2 === void 0 ? void 0 : _c$userData2.originalUuid) === componentId;
+          // Step 6: Directly remove SEGMENT-* and Gateway objects from Three.js whose
+          // pathFrom/pathTo references one of this component's connectors.
+          // This is surgical cleanup that works regardless of shouldUpdatePaths.
+          if (connectorIds.size > 0 && threeScene) {
+            var objectsToRemove = [];
+            threeScene.traverse(function (obj) {
+              var _obj$uuid, _obj$userData4, _obj$uuid2, _obj$userData5;
+              var isComputedSegment = ((_obj$uuid = obj.uuid) === null || _obj$uuid === void 0 ? void 0 : _obj$uuid.startsWith('SEGMENT-')) && ((_obj$userData4 = obj.userData) === null || _obj$userData4 === void 0 ? void 0 : _obj$userData4.isDeclared) !== true;
+              var isComputedGateway = ((_obj$uuid2 = obj.uuid) === null || _obj$uuid2 === void 0 ? void 0 : _obj$uuid2.includes('Gateway')) && ((_obj$userData5 = obj.userData) === null || _obj$userData5 === void 0 ? void 0 : _obj$userData5.isDeclared) !== true;
+              if (isComputedSegment || isComputedGateway) {
+                var _obj$userData6, _obj$userData7;
+                if (connectorIds.has((_obj$userData6 = obj.userData) === null || _obj$userData6 === void 0 ? void 0 : _obj$userData6.pathFrom) || connectorIds.has((_obj$userData7 = obj.userData) === null || _obj$userData7 === void 0 ? void 0 : _obj$userData7.pathTo)) {
+                  objectsToRemove.push(obj);
+                }
+              }
+            });
+            objectsToRemove.forEach(function (obj) {
+              var _obj$geometry;
+              var childrenCopy = _toConsumableArray(obj.children);
+              childrenCopy.forEach(function (child) {
+                var _child$geometry;
+                obj.remove(child);
+                (_child$geometry = child.geometry) === null || _child$geometry === void 0 || _child$geometry.dispose();
+                if (Array.isArray(child.material)) {
+                  child.material.forEach(function (m) {
+                    return m === null || m === void 0 ? void 0 : m.dispose();
+                  });
+                } else {
+                  var _child$material;
+                  (_child$material = child.material) === null || _child$material === void 0 || _child$material.dispose();
+                }
               });
-              if (index !== -1) {
-                sceneData.scene.children.splice(index, 1);
-                console.log('✅ Removed component from sceneData');
+              threeScene.remove(obj);
+              (_obj$geometry = obj.geometry) === null || _obj$geometry === void 0 || _obj$geometry.dispose();
+              if (Array.isArray(obj.material)) {
+                obj.material.forEach(function (m) {
+                  return m === null || m === void 0 ? void 0 : m.dispose();
+                });
+              } else {
+                var _obj$material;
+                (_obj$material = obj.material) === null || _obj$material === void 0 || _obj$material.dispose();
               }
+            });
+            if (objectsToRemove.length > 0) {
+              console.log("\uD83D\uDEBF deleteComponent(): Removed ".concat(objectsToRemove.length, " pipe segment(s)/gateway(s) for deleted component"));
             }
           }
 
           // Deselect if the deleted component was selected
-          // We check if the selected object matches the deleted ID, OR if the selected object is now detached from the scene (parent is null)
           var transformManager = this.centralPlant.sceneViewer.transformManager;
           if (transformManager && transformManager.selectedObjectForTransform) {
             var _selectedObj$userData;
             var selectedObj = transformManager.selectedObjectForTransform;
-            if (selectedObj.uuid === componentId || selectedObj.name === componentId || ((_selectedObj$userData = selectedObj.userData) === null || _selectedObj$userData === void 0 ? void 0 : _selectedObj$userData.originalUuid) === componentId || selectedObj.parent === null) {
+            if (selectedObj.uuid === resolvedUuid || selectedObj.uuid === componentId || selectedObj.name === componentId || ((_selectedObj$userData = selectedObj.userData) === null || _selectedObj$userData === void 0 ? void 0 : _selectedObj$userData.originalUuid) === resolvedUuid || selectedObj.parent === null) {
               console.log('🎯 deleteComponent(): Deselecting deleted object');
               transformManager.deselectObject();
             }
@@ -37761,14 +37874,6 @@ var CentralPlantInternals = /*#__PURE__*/function () {
             });
             console.log("\uD83D\uDCE1 Emitted 'component-removed' event for ".concat(componentId));
           }
-
-          // Auto-update paths if enabled (mirrors behaviour of transformOperationsManager)
-          if (this.centralPlant.sceneViewer.shouldUpdatePaths) {
-            if (typeof this.centralPlant.sceneViewer.updatePaths === 'function') {
-              console.log('🔄 deleteComponent(): Auto-updating paths after delete...');
-              this.centralPlant.sceneViewer.updatePaths();
-            }
-          }
           console.log("\u2705 deleteComponent(): Component ".concat(componentId, " deleted successfully"));
           return true;
         } else {
@@ -37793,8 +37898,8 @@ var CentralPlantInternals = /*#__PURE__*/function () {
       }
       var componentIds = [];
       this.centralPlant.sceneViewer.scene.traverse(function (child) {
-        var _child$userData2;
-        if (((_child$userData2 = child.userData) === null || _child$userData2 === void 0 ? void 0 : _child$userData2.objectType) === 'component') {
+        var _child$userData4;
+        if (((_child$userData4 = child.userData) === null || _child$userData4 === void 0 ? void 0 : _child$userData4.objectType) === 'component') {
           componentIds.push(child.uuid || child.userData.originalUuid);
         }
       });
@@ -37829,7 +37934,7 @@ var CentralPlant = /*#__PURE__*/function (_BaseDisposable) {
    * Initialize the CentralPlant manager
    * 
    * @constructor
-   * @version 0.3.9
+   * @version 0.3.10
    * @updated 2025-10-22
    * 
    * @description Creates a new CentralPlant instance and initializes internal managers and utilities.

package/dist/cjs/src/core/centralPlant.js

@@ -35,7 +35,7 @@ var CentralPlant = /*#__PURE__*/function (_BaseDisposable) {
    * Initialize the CentralPlant manager
    * 
    * @constructor
-   * @version 0.3.9
+   * @version 0.3.10
    * @updated 2025-10-22
    * 
    * @description Creates a new CentralPlant instance and initializes internal managers and utilities.

package/dist/cjs/src/core/centralPlantInternals.js

@@ -1223,39 +1223,152 @@ var CentralPlantInternals = /*#__PURE__*/function () {
         return false;
       }
       try {
+        var _this$centralPlant$sc0, _this$centralPlant$sc1, _sceneData$scene2, _sceneData$scene3;
         console.log("\uD83D\uDDD1\uFE0F deleteComponent(): Deleting component ".concat(componentId));
+        var threeScene = (_this$centralPlant$sc0 = this.centralPlant.sceneViewer) === null || _this$centralPlant$sc0 === void 0 ? void 0 : _this$centralPlant$sc0.scene;
+        var sceneData = (_this$centralPlant$sc1 = this.centralPlant.sceneViewer) === null || _this$centralPlant$sc1 === void 0 ? void 0 : _this$centralPlant$sc1.currentSceneData;
+
+        // Step 1: Resolve the actual Three.js UUID from componentId.
+        // The UI emits object.name (e.g. "Pump (PUMP-1)") as the selection ID, but
+        // sceneData.scene.children stores the raw uuid (e.g. "PUMP-1").  We must
+        // find the live Three.js object first so we can use its true .uuid for all
+        // sceneData lookups.
+        var resolvedUuid = componentId;
+        if (threeScene) {
+          var threeObj = nameUtils.findObjectByHardcodedUuid(threeScene, componentId);
+          if (!threeObj) threeObj = threeScene.getObjectByProperty('uuid', componentId);
+          if (!threeObj) threeObj = threeScene.getObjectByProperty('name', componentId);
+          if (threeObj) {
+            var _threeObj$userData;
+            resolvedUuid = ((_threeObj$userData = threeObj.userData) === null || _threeObj$userData === void 0 ? void 0 : _threeObj$userData.originalUuid) || threeObj.uuid || componentId;
+            console.log("\uD83D\uDD0D deleteComponent(): Resolved UUID \"".concat(componentId, "\" \u2192 \"").concat(resolvedUuid, "\""));
+          }
+        }
+
+        // Step 2: Collect connector UUIDs from sceneData.scene.children FIRST.
+        var connectorIds = new Set();
+        var sceneDataComponentIndex = -1;
+        if (sceneData !== null && sceneData !== void 0 && (_sceneData$scene2 = sceneData.scene) !== null && _sceneData$scene2 !== void 0 && _sceneData$scene2.children) {
+          sceneDataComponentIndex = sceneData.scene.children.findIndex(function (c) {
+            var _c$userData2;
+            return c.uuid === resolvedUuid || ((_c$userData2 = c.userData) === null || _c$userData2 === void 0 ? void 0 : _c$userData2.originalUuid) === resolvedUuid;
+          });
+          if (sceneDataComponentIndex !== -1) {
+            var componentNode = sceneData.scene.children[sceneDataComponentIndex];
+            if (Array.isArray(componentNode.children)) {
+              componentNode.children.forEach(function (child) {
+                var _child$userData2;
+                if (((_child$userData2 = child.userData) === null || _child$userData2 === void 0 ? void 0 : _child$userData2.objectType) === 'connector' && child.uuid) {
+                  connectorIds.add(child.uuid);
+                }
+              });
+            }
+          }
+        }
+
+        // Fallback: also traverse the live Three.js scene in case sceneData is missing
+        // the component (e.g., dynamically added but not yet synced to sceneData)
+        if (connectorIds.size === 0 && threeScene) {
+          threeScene.traverse(function (obj) {
+            var _obj$userData2, _obj$userData3;
+            if ((obj.uuid === resolvedUuid || ((_obj$userData2 = obj.userData) === null || _obj$userData2 === void 0 ? void 0 : _obj$userData2.originalUuid) === resolvedUuid) && ((_obj$userData3 = obj.userData) === null || _obj$userData3 === void 0 ? void 0 : _obj$userData3.objectType) === 'component') {
+              obj.children.forEach(function (child) {
+                var _child$userData3;
+                if (((_child$userData3 = child.userData) === null || _child$userData3 === void 0 ? void 0 : _child$userData3.objectType) === 'connector') {
+                  connectorIds.add(child.uuid);
+                }
+              });
+            }
+          });
+        }
+        console.log("\uD83D\uDD0C deleteComponent(): Found ".concat(connectorIds.size, " connector(s) for component ").concat(componentId));
+
+        // Step 3: Remove connections from sceneData BEFORE removing from Three.js
+        // so the pathfinder fingerprint reflects the updated state on next updatePaths() call.
+        if (connectorIds.size > 0 && sceneData && Array.isArray(sceneData.connections)) {
+          var beforeCount = sceneData.connections.length;
+          sceneData.connections = sceneData.connections.filter(function (conn) {
+            return !connectorIds.has(conn.from) && !connectorIds.has(conn.to);
+          });
+          var removedCount = beforeCount - sceneData.connections.length;
+          if (removedCount > 0) {
+            console.log("\uD83D\uDD17 deleteComponent(): Removed ".concat(removedCount, " connection(s) for deleted component"));
+          }
+        }
+
+        // Step 4: Remove component from sceneData.scene.children BEFORE removing from Three.js
+        if (sceneDataComponentIndex !== -1) {
+          sceneData.scene.children.splice(sceneDataComponentIndex, 1);
+          console.log('✅ Removed component from sceneData.scene.children');
+        } else if (sceneData !== null && sceneData !== void 0 && (_sceneData$scene3 = sceneData.scene) !== null && _sceneData$scene3 !== void 0 && _sceneData$scene3.children) {
+          // Retry in case findIndex above didn't run (e.g. no sceneData at that point)
+          var retryIndex = sceneData.scene.children.findIndex(function (c) {
+            var _c$userData3;
+            return c.uuid === resolvedUuid || ((_c$userData3 = c.userData) === null || _c$userData3 === void 0 ? void 0 : _c$userData3.originalUuid) === resolvedUuid;
+          });
+          if (retryIndex !== -1) {
+            sceneData.scene.children.splice(retryIndex, 1);
+            console.log('✅ Removed component from sceneData.scene.children (retry)');
+          }
+        }
 
-        // Use componentManager to remove the component
+        // Step 5: Remove the component from the Three.js scene
         var success = componentManager.removeComponentFromScene(componentId);
         if (success) {
-          // Also remove from scene data if available
-          if (this.centralPlant.sceneViewer.sceneOperationsManager && this.centralPlant.sceneViewer.currentSceneData) {
-            // Note: componentManager.removeComponentFromScene might not update sceneData
-            // We should verify if sceneOperationsManager has a method for this or if we need to manually update
-            // For now, assuming componentManager handles the scene object removal, 
-            // we might need to update the data structure manually if componentManager doesn't.
-
-            // Let's implement a safe removal from sceneData here just in case
-            var sceneData = this.centralPlant.sceneViewer.currentSceneData;
-            if (sceneData.scene && sceneData.scene.children) {
-              var index = sceneData.scene.children.findIndex(function (c) {
-                var _c$userData2;
-                return c.uuid === componentId || ((_c$userData2 = c.userData) === null || _c$userData2 === void 0 ? void 0 : _c$userData2.originalUuid) === componentId;
+          // Step 6: Directly remove SEGMENT-* and Gateway objects from Three.js whose
+          // pathFrom/pathTo references one of this component's connectors.
+          // This is surgical cleanup that works regardless of shouldUpdatePaths.
+          if (connectorIds.size > 0 && threeScene) {
+            var objectsToRemove = [];
+            threeScene.traverse(function (obj) {
+              var _obj$uuid, _obj$userData4, _obj$uuid2, _obj$userData5;
+              var isComputedSegment = ((_obj$uuid = obj.uuid) === null || _obj$uuid === void 0 ? void 0 : _obj$uuid.startsWith('SEGMENT-')) && ((_obj$userData4 = obj.userData) === null || _obj$userData4 === void 0 ? void 0 : _obj$userData4.isDeclared) !== true;
+              var isComputedGateway = ((_obj$uuid2 = obj.uuid) === null || _obj$uuid2 === void 0 ? void 0 : _obj$uuid2.includes('Gateway')) && ((_obj$userData5 = obj.userData) === null || _obj$userData5 === void 0 ? void 0 : _obj$userData5.isDeclared) !== true;
+              if (isComputedSegment || isComputedGateway) {
+                var _obj$userData6, _obj$userData7;
+                if (connectorIds.has((_obj$userData6 = obj.userData) === null || _obj$userData6 === void 0 ? void 0 : _obj$userData6.pathFrom) || connectorIds.has((_obj$userData7 = obj.userData) === null || _obj$userData7 === void 0 ? void 0 : _obj$userData7.pathTo)) {
+                  objectsToRemove.push(obj);
+                }
+              }
+            });
+            objectsToRemove.forEach(function (obj) {
+              var _obj$geometry;
+              var childrenCopy = _rollupPluginBabelHelpers.toConsumableArray(obj.children);
+              childrenCopy.forEach(function (child) {
+                var _child$geometry;
+                obj.remove(child);
+                (_child$geometry = child.geometry) === null || _child$geometry === void 0 || _child$geometry.dispose();
+                if (Array.isArray(child.material)) {
+                  child.material.forEach(function (m) {
+                    return m === null || m === void 0 ? void 0 : m.dispose();
+                  });
+                } else {
+                  var _child$material;
+                  (_child$material = child.material) === null || _child$material === void 0 || _child$material.dispose();
+                }
               });
-              if (index !== -1) {
-                sceneData.scene.children.splice(index, 1);
-                console.log('✅ Removed component from sceneData');
+              threeScene.remove(obj);
+              (_obj$geometry = obj.geometry) === null || _obj$geometry === void 0 || _obj$geometry.dispose();
+              if (Array.isArray(obj.material)) {
+                obj.material.forEach(function (m) {
+                  return m === null || m === void 0 ? void 0 : m.dispose();
+                });
+              } else {
+                var _obj$material;
+                (_obj$material = obj.material) === null || _obj$material === void 0 || _obj$material.dispose();
               }
+            });
+            if (objectsToRemove.length > 0) {
+              console.log("\uD83D\uDEBF deleteComponent(): Removed ".concat(objectsToRemove.length, " pipe segment(s)/gateway(s) for deleted component"));
             }
           }
 
           // Deselect if the deleted component was selected
-          // We check if the selected object matches the deleted ID, OR if the selected object is now detached from the scene (parent is null)
           var transformManager = this.centralPlant.sceneViewer.transformManager;
           if (transformManager && transformManager.selectedObjectForTransform) {
             var _selectedObj$userData;
             var selectedObj = transformManager.selectedObjectForTransform;
-            if (selectedObj.uuid === componentId || selectedObj.name === componentId || ((_selectedObj$userData = selectedObj.userData) === null || _selectedObj$userData === void 0 ? void 0 : _selectedObj$userData.originalUuid) === componentId || selectedObj.parent === null) {
+            if (selectedObj.uuid === resolvedUuid || selectedObj.uuid === componentId || selectedObj.name === componentId || ((_selectedObj$userData = selectedObj.userData) === null || _selectedObj$userData === void 0 ? void 0 : _selectedObj$userData.originalUuid) === resolvedUuid || selectedObj.parent === null) {
               console.log('🎯 deleteComponent(): Deselecting deleted object');
               transformManager.deselectObject();
             }
@@ -1268,14 +1381,6 @@ var CentralPlantInternals = /*#__PURE__*/function () {
             });
             console.log("\uD83D\uDCE1 Emitted 'component-removed' event for ".concat(componentId));
           }
-
-          // Auto-update paths if enabled (mirrors behaviour of transformOperationsManager)
-          if (this.centralPlant.sceneViewer.shouldUpdatePaths) {
-            if (typeof this.centralPlant.sceneViewer.updatePaths === 'function') {
-              console.log('🔄 deleteComponent(): Auto-updating paths after delete...');
-              this.centralPlant.sceneViewer.updatePaths();
-            }
-          }
           console.log("\u2705 deleteComponent(): Component ".concat(componentId, " deleted successfully"));
           return true;
         } else {
@@ -1300,8 +1405,8 @@ var CentralPlantInternals = /*#__PURE__*/function () {
       }
       var componentIds = [];
       this.centralPlant.sceneViewer.scene.traverse(function (child) {
-        var _child$userData2;
-        if (((_child$userData2 = child.userData) === null || _child$userData2 === void 0 ? void 0 : _child$userData2.objectType) === 'component') {
+        var _child$userData4;
+        if (((_child$userData4 = child.userData) === null || _child$userData4 === void 0 ? void 0 : _child$userData4.objectType) === 'component') {
           componentIds.push(child.uuid || child.userData.originalUuid);
         }
       });

package/dist/esm/src/core/centralPlant.js

@@ -31,7 +31,7 @@ var CentralPlant = /*#__PURE__*/function (_BaseDisposable) {
    * Initialize the CentralPlant manager
    * 
    * @constructor
-   * @version 0.3.9
+   * @version 0.3.10
    * @updated 2025-10-22
    * 
    * @description Creates a new CentralPlant instance and initializes internal managers and utilities.

package/dist/esm/src/core/centralPlantInternals.js

@@ -1,4 +1,4 @@
-import { createClass as _createClass, objectSpread2 as _objectSpread2, createForOfIteratorHelper as _createForOfIteratorHelper, typeof as _typeof, classCallCheck as _classCallCheck, asyncToGenerator as _asyncToGenerator, regenerator as _regenerator } from '../../_virtual/_rollupPluginBabelHelpers.js';
+import { createClass as _createClass, objectSpread2 as _objectSpread2, createForOfIteratorHelper as _createForOfIteratorHelper, typeof as _typeof, toConsumableArray as _toConsumableArray, classCallCheck as _classCallCheck, asyncToGenerator as _asyncToGenerator, regenerator as _regenerator } from '../../_virtual/_rollupPluginBabelHelpers.js';
 import * as THREE from 'three';
 import { CentralPlantValidator } from './centralPlantValidator.js';
 import { createTransformControls } from '../managers/controls/transformControlsManager.js';
@@ -1199,39 +1199,152 @@ var CentralPlantInternals = /*#__PURE__*/function () {
         return false;
       }
       try {
+        var _this$centralPlant$sc0, _this$centralPlant$sc1, _sceneData$scene2, _sceneData$scene3;
         console.log("\uD83D\uDDD1\uFE0F deleteComponent(): Deleting component ".concat(componentId));
+        var threeScene = (_this$centralPlant$sc0 = this.centralPlant.sceneViewer) === null || _this$centralPlant$sc0 === void 0 ? void 0 : _this$centralPlant$sc0.scene;
+        var sceneData = (_this$centralPlant$sc1 = this.centralPlant.sceneViewer) === null || _this$centralPlant$sc1 === void 0 ? void 0 : _this$centralPlant$sc1.currentSceneData;
+
+        // Step 1: Resolve the actual Three.js UUID from componentId.
+        // The UI emits object.name (e.g. "Pump (PUMP-1)") as the selection ID, but
+        // sceneData.scene.children stores the raw uuid (e.g. "PUMP-1").  We must
+        // find the live Three.js object first so we can use its true .uuid for all
+        // sceneData lookups.
+        var resolvedUuid = componentId;
+        if (threeScene) {
+          var threeObj = findObjectByHardcodedUuid(threeScene, componentId);
+          if (!threeObj) threeObj = threeScene.getObjectByProperty('uuid', componentId);
+          if (!threeObj) threeObj = threeScene.getObjectByProperty('name', componentId);
+          if (threeObj) {
+            var _threeObj$userData;
+            resolvedUuid = ((_threeObj$userData = threeObj.userData) === null || _threeObj$userData === void 0 ? void 0 : _threeObj$userData.originalUuid) || threeObj.uuid || componentId;
+            console.log("\uD83D\uDD0D deleteComponent(): Resolved UUID \"".concat(componentId, "\" \u2192 \"").concat(resolvedUuid, "\""));
+          }
+        }
+
+        // Step 2: Collect connector UUIDs from sceneData.scene.children FIRST.
+        var connectorIds = new Set();
+        var sceneDataComponentIndex = -1;
+        if (sceneData !== null && sceneData !== void 0 && (_sceneData$scene2 = sceneData.scene) !== null && _sceneData$scene2 !== void 0 && _sceneData$scene2.children) {
+          sceneDataComponentIndex = sceneData.scene.children.findIndex(function (c) {
+            var _c$userData2;
+            return c.uuid === resolvedUuid || ((_c$userData2 = c.userData) === null || _c$userData2 === void 0 ? void 0 : _c$userData2.originalUuid) === resolvedUuid;
+          });
+          if (sceneDataComponentIndex !== -1) {
+            var componentNode = sceneData.scene.children[sceneDataComponentIndex];
+            if (Array.isArray(componentNode.children)) {
+              componentNode.children.forEach(function (child) {
+                var _child$userData2;
+                if (((_child$userData2 = child.userData) === null || _child$userData2 === void 0 ? void 0 : _child$userData2.objectType) === 'connector' && child.uuid) {
+                  connectorIds.add(child.uuid);
+                }
+              });
+            }
+          }
+        }
+
+        // Fallback: also traverse the live Three.js scene in case sceneData is missing
+        // the component (e.g., dynamically added but not yet synced to sceneData)
+        if (connectorIds.size === 0 && threeScene) {
+          threeScene.traverse(function (obj) {
+            var _obj$userData2, _obj$userData3;
+            if ((obj.uuid === resolvedUuid || ((_obj$userData2 = obj.userData) === null || _obj$userData2 === void 0 ? void 0 : _obj$userData2.originalUuid) === resolvedUuid) && ((_obj$userData3 = obj.userData) === null || _obj$userData3 === void 0 ? void 0 : _obj$userData3.objectType) === 'component') {
+              obj.children.forEach(function (child) {
+                var _child$userData3;
+                if (((_child$userData3 = child.userData) === null || _child$userData3 === void 0 ? void 0 : _child$userData3.objectType) === 'connector') {
+                  connectorIds.add(child.uuid);
+                }
+              });
+            }
+          });
+        }
+        console.log("\uD83D\uDD0C deleteComponent(): Found ".concat(connectorIds.size, " connector(s) for component ").concat(componentId));
+
+        // Step 3: Remove connections from sceneData BEFORE removing from Three.js
+        // so the pathfinder fingerprint reflects the updated state on next updatePaths() call.
+        if (connectorIds.size > 0 && sceneData && Array.isArray(sceneData.connections)) {
+          var beforeCount = sceneData.connections.length;
+          sceneData.connections = sceneData.connections.filter(function (conn) {
+            return !connectorIds.has(conn.from) && !connectorIds.has(conn.to);
+          });
+          var removedCount = beforeCount - sceneData.connections.length;
+          if (removedCount > 0) {
+            console.log("\uD83D\uDD17 deleteComponent(): Removed ".concat(removedCount, " connection(s) for deleted component"));
+          }
+        }
+
+        // Step 4: Remove component from sceneData.scene.children BEFORE removing from Three.js
+        if (sceneDataComponentIndex !== -1) {
+          sceneData.scene.children.splice(sceneDataComponentIndex, 1);
+          console.log('✅ Removed component from sceneData.scene.children');
+        } else if (sceneData !== null && sceneData !== void 0 && (_sceneData$scene3 = sceneData.scene) !== null && _sceneData$scene3 !== void 0 && _sceneData$scene3.children) {
+          // Retry in case findIndex above didn't run (e.g. no sceneData at that point)
+          var retryIndex = sceneData.scene.children.findIndex(function (c) {
+            var _c$userData3;
+            return c.uuid === resolvedUuid || ((_c$userData3 = c.userData) === null || _c$userData3 === void 0 ? void 0 : _c$userData3.originalUuid) === resolvedUuid;
+          });
+          if (retryIndex !== -1) {
+            sceneData.scene.children.splice(retryIndex, 1);
+            console.log('✅ Removed component from sceneData.scene.children (retry)');
+          }
+        }
 
-        // Use componentManager to remove the component
+        // Step 5: Remove the component from the Three.js scene
         var success = componentManager.removeComponentFromScene(componentId);
         if (success) {
-          // Also remove from scene data if available
-          if (this.centralPlant.sceneViewer.sceneOperationsManager && this.centralPlant.sceneViewer.currentSceneData) {
-            // Note: componentManager.removeComponentFromScene might not update sceneData
-            // We should verify if sceneOperationsManager has a method for this or if we need to manually update
-            // For now, assuming componentManager handles the scene object removal, 
-            // we might need to update the data structure manually if componentManager doesn't.
-
-            // Let's implement a safe removal from sceneData here just in case
-            var sceneData = this.centralPlant.sceneViewer.currentSceneData;
-            if (sceneData.scene && sceneData.scene.children) {
-              var index = sceneData.scene.children.findIndex(function (c) {
-                var _c$userData2;
-                return c.uuid === componentId || ((_c$userData2 = c.userData) === null || _c$userData2 === void 0 ? void 0 : _c$userData2.originalUuid) === componentId;
+          // Step 6: Directly remove SEGMENT-* and Gateway objects from Three.js whose
+          // pathFrom/pathTo references one of this component's connectors.
+          // This is surgical cleanup that works regardless of shouldUpdatePaths.
+          if (connectorIds.size > 0 && threeScene) {
+            var objectsToRemove = [];
+            threeScene.traverse(function (obj) {
+              var _obj$uuid, _obj$userData4, _obj$uuid2, _obj$userData5;
+              var isComputedSegment = ((_obj$uuid = obj.uuid) === null || _obj$uuid === void 0 ? void 0 : _obj$uuid.startsWith('SEGMENT-')) && ((_obj$userData4 = obj.userData) === null || _obj$userData4 === void 0 ? void 0 : _obj$userData4.isDeclared) !== true;
+              var isComputedGateway = ((_obj$uuid2 = obj.uuid) === null || _obj$uuid2 === void 0 ? void 0 : _obj$uuid2.includes('Gateway')) && ((_obj$userData5 = obj.userData) === null || _obj$userData5 === void 0 ? void 0 : _obj$userData5.isDeclared) !== true;
+              if (isComputedSegment || isComputedGateway) {
+                var _obj$userData6, _obj$userData7;
+                if (connectorIds.has((_obj$userData6 = obj.userData) === null || _obj$userData6 === void 0 ? void 0 : _obj$userData6.pathFrom) || connectorIds.has((_obj$userData7 = obj.userData) === null || _obj$userData7 === void 0 ? void 0 : _obj$userData7.pathTo)) {
+                  objectsToRemove.push(obj);
+                }
+              }
+            });
+            objectsToRemove.forEach(function (obj) {
+              var _obj$geometry;
+              var childrenCopy = _toConsumableArray(obj.children);
+              childrenCopy.forEach(function (child) {
+                var _child$geometry;
+                obj.remove(child);
+                (_child$geometry = child.geometry) === null || _child$geometry === void 0 || _child$geometry.dispose();
+                if (Array.isArray(child.material)) {
+                  child.material.forEach(function (m) {
+                    return m === null || m === void 0 ? void 0 : m.dispose();
+                  });
+                } else {
+                  var _child$material;
+                  (_child$material = child.material) === null || _child$material === void 0 || _child$material.dispose();
+                }
               });
-              if (index !== -1) {
-                sceneData.scene.children.splice(index, 1);
-                console.log('✅ Removed component from sceneData');
+              threeScene.remove(obj);
+              (_obj$geometry = obj.geometry) === null || _obj$geometry === void 0 || _obj$geometry.dispose();
+              if (Array.isArray(obj.material)) {
+                obj.material.forEach(function (m) {
+                  return m === null || m === void 0 ? void 0 : m.dispose();
+                });
+              } else {
+                var _obj$material;
+                (_obj$material = obj.material) === null || _obj$material === void 0 || _obj$material.dispose();
               }
+            });
+            if (objectsToRemove.length > 0) {
+              console.log("\uD83D\uDEBF deleteComponent(): Removed ".concat(objectsToRemove.length, " pipe segment(s)/gateway(s) for deleted component"));
             }
           }
 
           // Deselect if the deleted component was selected
-          // We check if the selected object matches the deleted ID, OR if the selected object is now detached from the scene (parent is null)
           var transformManager = this.centralPlant.sceneViewer.transformManager;
           if (transformManager && transformManager.selectedObjectForTransform) {
             var _selectedObj$userData;
             var selectedObj = transformManager.selectedObjectForTransform;
-            if (selectedObj.uuid === componentId || selectedObj.name === componentId || ((_selectedObj$userData = selectedObj.userData) === null || _selectedObj$userData === void 0 ? void 0 : _selectedObj$userData.originalUuid) === componentId || selectedObj.parent === null) {
+            if (selectedObj.uuid === resolvedUuid || selectedObj.uuid === componentId || selectedObj.name === componentId || ((_selectedObj$userData = selectedObj.userData) === null || _selectedObj$userData === void 0 ? void 0 : _selectedObj$userData.originalUuid) === resolvedUuid || selectedObj.parent === null) {
               console.log('🎯 deleteComponent(): Deselecting deleted object');
               transformManager.deselectObject();
             }
@@ -1244,14 +1357,6 @@ var CentralPlantInternals = /*#__PURE__*/function () {
             });
             console.log("\uD83D\uDCE1 Emitted 'component-removed' event for ".concat(componentId));
           }
-
-          // Auto-update paths if enabled (mirrors behaviour of transformOperationsManager)
-          if (this.centralPlant.sceneViewer.shouldUpdatePaths) {
-            if (typeof this.centralPlant.sceneViewer.updatePaths === 'function') {
-              console.log('🔄 deleteComponent(): Auto-updating paths after delete...');
-              this.centralPlant.sceneViewer.updatePaths();
-            }
-          }
           console.log("\u2705 deleteComponent(): Component ".concat(componentId, " deleted successfully"));
           return true;
         } else {
@@ -1276,8 +1381,8 @@ var CentralPlantInternals = /*#__PURE__*/function () {
       }
       var componentIds = [];
       this.centralPlant.sceneViewer.scene.traverse(function (child) {
-        var _child$userData2;
-        if (((_child$userData2 = child.userData) === null || _child$userData2 === void 0 ? void 0 : _child$userData2.objectType) === 'component') {
+        var _child$userData4;
+        if (((_child$userData4 = child.userData) === null || _child$userData4 === void 0 ? void 0 : _child$userData4.objectType) === 'component') {
           componentIds.push(child.uuid || child.userData.originalUuid);
         }
       });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@2112-lab/central-plant",
-  "version": "0.3.9",
+  "version": "0.3.10",
   "description": "Utility modules for the Central Plant Application",
   "main": "dist/bundle/index.js",
   "module": "dist/esm/src/index.js",