include path remove with smart components detection logic

Author: Jason2866

builder/frameworks/component_manager.py

@@ -116,6 +116,110 @@ def _get_lib_ignore_entries(self) -> List[str]:
         except Exception:
             return []
 
+    def _analyze_project_dependencies(self) -> Set[str]:
+        """Analyze project files to detect actually used components/libraries."""
+        used_components = set()
+        
+        try:
+            # Analyze project source files
+            src_dir = self.env.subst("$PROJECT_SRC_DIR")
+            if os.path.exists(src_dir):
+                for root, dirs, files in os.walk(src_dir):
+                    for file in files:
+                        if file.endswith(('.cpp', '.c', '.h', '.hpp', '.ino')):
+                            file_path = os.path.join(root, file)
+                            used_components.update(self._extract_components_from_file(file_path))
+            
+            # Analyze lib_deps for explicit dependencies (if available)
+            lib_deps = self.env.GetProjectOption("lib_deps", [])
+            if isinstance(lib_deps, str):
+                lib_deps = [lib_deps]
+            
+            for dep in lib_deps:
+                used_components.update(self._extract_components_from_lib_dep(str(dep)))
+                
+        except Exception:
+            pass
+        
+        return used_components
+    
+    def _extract_components_from_file(self, file_path: str) -> Set[str]:
+        """Extract component usage from a single file by analyzing includes and function calls."""
+        components = set()
+        
+        # Component detection patterns - maps component names to search patterns
+        component_patterns = {
+            'bt': ['bluetooth', 'ble', 'nimble', 'bt_', 'esp_bt', 'esp_ble'],
+            'esp_wifi': ['wifi', 'esp_wifi', 'tcpip_adapter'],
+            'esp_dsp': ['dsps_', 'esp_dsp', 'fft2r'],
+            'esp_http_client': ['esp_http_client', 'http_client'],
+            'esp_https_ota': ['esp_https_ota', 'esp_ota'],
+            'mdns': ['mdns', 'esp_mdns'],
+            'mqtt': ['mqtt', 'esp_mqtt'],
+            'spiffs': ['spiffs', 'esp_spiffs'],
+            'fatfs': ['fatfs', 'ff.h'],
+            'nvs_flash': ['nvs', 'nvs_flash'],
+            'esp_timer': ['esp_timer', 'timer_'],
+            'driver': ['gpio_', 'uart_', 'spi_', 'i2c_', 'adc_', 'dac_'],
+            'esp_camera': ['esp_camera', 'camera.h'],
+            'esp_now': ['esp_now', 'espnow'],
+            'esp_smartconfig': ['smartconfig', 'esp_smartconfig']
+        }
+        
+        try:
+            with open(file_path, 'r', encoding='utf-8', errors='ignore') as f:
+                content = f.read().lower()
+                
+                # Check each component pattern against file content
+                for component, patterns in component_patterns.items():
+                    if any(pattern in content for pattern in patterns):
+                        components.add(component)
+                        
+        except Exception:
+            pass
+        
+        return components
+    
+    def _extract_components_from_lib_dep(self, lib_dep: str) -> Set[str]:
+        """Extract components from lib_deps entry by mapping library names to components."""
+        components = set()
+        lib_dep_upper = lib_dep.upper()
+        
+        # Map lib_deps entries to ESP-IDF components
+        lib_dep_mapping = {
+            'bt': ['BLE', 'BT', 'BLUETOOTH', 'NIMBLE'],
+            'esp_wifi': ['WIFI', 'ASYNCTCP', 'ESPASYNCWEBSERVER'],
+            'esp_dsp': ['DSP', 'FFT', 'JPEG'],
+            'esp_http_client': ['HTTP', 'HTTPCLIENT'],
+            'mqtt': ['MQTT', 'PUBSUB'],
+            'esp_camera': ['CAMERA', 'ESP32CAM']
+        }
+        
+        for component, keywords in lib_dep_mapping.items():
+            if any(keyword in lib_dep_upper for keyword in keywords):
+                components.add(component)
+        
+        return components
+    
+    def _is_component_used_in_project(self, lib_name: str) -> bool:
+        """Check if a component/library is actually used in the project."""
+        # Cache project analysis results for performance
+        if not hasattr(self, '_project_components_cache'):
+            self._project_components_cache = self._analyze_project_dependencies()
+        
+        lib_name_lower = lib_name.lower()
+        
+        # Direct match
+        if lib_name_lower in self._project_components_cache:
+            return True
+        
+        # Partial match for related components
+        for used_component in self._project_components_cache:
+            if lib_name_lower in used_component or used_component in lib_name_lower:
+                return True
+        
+        return False
+    
     def _get_arduino_core_libraries(self) -> Dict[str, str]:
         """Get all Arduino core libraries and their corresponding include paths."""
         libraries_mapping = {}
@@ -297,64 +401,13 @@ def _convert_lib_name_to_include(self, lib_name: str) -> str:
 
         return cleaned_name
 
-    def _has_bt_ble_dependencies(self) -> bool:
-        """Check if lib_deps contains any BT/BLE related dependencies."""
-        try:
-            # Get lib_deps from current environment
-            lib_deps = self.env.GetProjectOption("lib_deps", [])
-            
-            if isinstance(lib_deps, str):
-                lib_deps = [lib_deps]
-            elif lib_deps is None:
-                lib_deps = []
-            
-            # Convert to string and check for BT/BLE keywords
-            lib_deps_str = ' '.join(str(dep) for dep in lib_deps).upper()
-            
-            bt_ble_keywords = ['BLE', 'BT', 'NIMBLE', 'BLUETOOTH']
-            
-            for keyword in bt_ble_keywords:
-                if keyword in lib_deps_str:
-                    return True
-            
-            return False
-            
-        except Exception:
-            return False
-    
-    def _is_bt_related_library(self, lib_name: str) -> bool:
-        """Check if a library name is related to Bluetooth/BLE functionality."""
-        lib_name_upper = lib_name.upper()
-        
-        bt_related_names = [
-            'BT',
-            'BLE', 
-            'BLUETOOTH',
-            'NIMBLE',
-            'ESP32_BLE',
-            'ESP32BLE',
-            'BLUETOOTHSERIAL',
-            'BLE_ARDUINO',
-            'ESP_BLE',
-            'ESP_BT'
-        ]
-        
-        for bt_name in bt_related_names:
-            if bt_name in lib_name_upper:
-                return True
-        
-        return False
-    
     def _remove_ignored_lib_includes(self) -> None:
         """Remove include entries for ignored libraries from pioarduino-build.py."""
         build_py_path = join(self.arduino_libs_mcu, "pioarduino-build.py")
 
         if not os.path.exists(build_py_path):
             return
 
-        # Check if BT/BLE dependencies exist in lib_deps
-        bt_ble_protected = self._has_bt_ble_dependencies()
-        
         try:
             with open(build_py_path, 'r') as f:
                 content = f.read()
@@ -364,9 +417,9 @@ def _remove_ignored_lib_includes(self) -> None:
 
             # Remove CPPPATH entries for each ignored library
             for lib_name in self.ignored_libs:
-                # Skip BT-related libraries if BT/BLE dependencies are present
-                if bt_ble_protected and self._is_bt_related_library(lib_name):
-                    print(f"Skipping removal of BT-related library '{lib_name}' due to BT/BLE dependency in lib_deps")
+                # Universal protection: Skip if component is actually used in project
+                if self._is_component_used_in_project(lib_name):
+                    print(f"Skipping removal of library '{lib_name}' - detected as used in project")
                     continue
 
                 # Multiple patterns to catch different include formats