refactor: simplify dependencies in FileBrowser component and clean up unused console log in rosConnectionManager

Signed-off-by: ola31 <[email protected]>

Author: ola31

physical_ai_manager/src/components/FileBrowser.js

@@ -667,15 +667,8 @@ export default function FileBrowser({
     };
 
     initializeBrowser();
-  }, [
-    initialPath,
-    homePath,
-    browseFile,
-    checkDirectoriesForTargetFile,
-    onPathChange,
-    targetFileName,
-    targetFolderName,
-  ]);
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [initialPath, homePath, targetFileName, targetFolderName]);
 
   const classMainContainer = clsx(
     'h-full',

physical_ai_manager/src/utils/rosConnectionManager.js

@@ -55,7 +55,6 @@ class RosConnectionManager {
 
     // If already connected, return existing connection
     if (this.ros && this.ros.isConnected) {
-      console.log('Using existing ROS connection');
       return this.ros;
     }
 

physical_ai_server/physical_ai_server/communication/communicator.py

@@ -24,13 +24,13 @@
 from physical_ai_interfaces.msg import (
     BrowserItem,
     DatasetInfo,
-    TaskStatus,
+    TaskStatus
 )
 from physical_ai_interfaces.srv import (
     BrowseFile,
     EditDataset,
     GetDatasetInfo,
-    GetImageTopicList,
+    GetImageTopicList
 )
 from physical_ai_server.communication.multi_subscriber import MultiSubscriber
 from physical_ai_server.data_processing.data_editor import DataEditor