fix: replace os.startfile with subprocess.run to resolve pylint warning

Author: mashu3

tests/test_pathbrowser.py

@@ -259,20 +259,22 @@ def test_open_file_with_default_app_windows(self):
         with patch('tkface.widget.pathbrowser.utils.IS_WINDOWS', True):
             with patch('tkface.widget.pathbrowser.utils.IS_MACOS', False):
                 with patch('tkface.widget.pathbrowser.utils.IS_LINUX', False):
-                    # Mock os.startfile by patching the module directly with create=True
-                    with patch('os.startfile', create=True) as mock_startfile:
+                    with patch('tkface.widget.pathbrowser.utils.subprocess.run') as mock_run:
                         result = utils.open_file_with_default_app("C:\\test\\file.txt")
                         assert result is True
-                        mock_startfile.assert_called_once_with("C:\\test\\file.txt")
+                        mock_run.assert_called_once_with(
+                            ["cmd", "/c", "start", "", "C:\\test\\file.txt"], 
+                            check=False, 
+                            shell=False
+                        )
 
     def test_open_file_with_default_app_windows_exception(self):
         """Test open_file_with_default_app function on Windows with exception."""
         with patch('tkface.widget.pathbrowser.utils.IS_WINDOWS', True):
             with patch('tkface.widget.pathbrowser.utils.IS_MACOS', False):
                 with patch('tkface.widget.pathbrowser.utils.IS_LINUX', False):
-                    # Mock os.startfile by patching the module directly with create=True
-                    with patch('os.startfile', create=True) as mock_startfile:
-                        mock_startfile.side_effect = Exception("Access denied")
+                    with patch('tkface.widget.pathbrowser.utils.subprocess.run') as mock_run:
+                        mock_run.side_effect = Exception("Access denied")
                         result = utils.open_file_with_default_app("C:\\test\\file.txt")
                         assert result is False
 

tkface/widget/pathbrowser/utils.py

@@ -55,7 +55,8 @@ def open_file_with_default_app(file_path: str) -> bool:
     """
     try:
         if IS_WINDOWS:
-            os.startfile(file_path)  # pylint: disable=no-member
+            # Use subprocess instead of os.startfile to avoid shell process warning
+            subprocess.run(["cmd", "/c", "start", "", file_path], check=False, shell=False)
         elif IS_MACOS:
             open_cmd = shutil.which("open")
             if open_cmd: