mytunes-pro 1.8.1__py3-none-any.whl → 1.8.4__py3-none-any.whl

mytunes/app.py

@@ -35,7 +35,7 @@ MPV_SOCKET = "/tmp/mpv_socket"
 LOG_FILE = "/tmp/mytunes_mpv.log"
 PID_FILE = "/tmp/mytunes_mpv.pid"
 APP_NAME = "MyTunes Pro"
-APP_VERSION = "1.8.1"
+APP_VERSION = "1.8.4"
 
 # === [Strings & Localization] ===
 STRINGS = {
@@ -245,10 +245,12 @@ class Player:
         # self.cleanup_orphaned_mpv() # Moved to play() per user request
         
     def cleanup_orphaned_mpv(self):
-        # User requested revert to aggressive pkill for reliability
-        # This ensures any previous background instances are killed
+        # Precise pkill to avoid matching the main TUI process
+        # Matches 'mpv ' (with space) or 'mpv' as exact process name
         try:
-            subprocess.run(["pkill", "-f", "mpv"], stderr=subprocess.DEVNULL)
+            subprocess.run(["pkill", "-x", "mpv"], stderr=subprocess.DEVNULL)
+            # Second pass for variants or sub-arguments if needed
+            subprocess.run(["pkill", "-f", "mpv --video=no"], stderr=subprocess.DEVNULL)
         except: pass
         
     def play(self, url, start_pos=0):
@@ -331,7 +333,6 @@ class Player:
             try:
                 self.current_proc.terminate()
                 self.current_proc.wait(timeout=1)
-                self.current_proc.wait(timeout=1)
             except:
                 # If terminate fails, try socket quit
                 try: self.send_cmd(["quit"])
@@ -743,21 +744,9 @@ class MyTunesApp:
                     if self.is_remote():
                         self.show_copy_dialog("YouTube", url)
                     else:
-                        try:
-                            # Robust multi-platform open
-                            if sys.platform == 'darwin':
-                                subprocess.Popen(["open", url], stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)
-                            elif sys.platform == 'win32':
-                                os.startfile(url)
-                            elif self.is_wsl():
-                                # In WSL, call the Windows shell to open the URL in Windows browser
-                                subprocess.Popen(["cmd.exe", "/c", "start", url.replace("&", "^&")], stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)
-                            else:
-                                webbrowser.open(url)
-                            self.status_msg = "🌐 Opening YouTube in Browser..."
-                        except:
-                            webbrowser.open(url)
-                            self.status_msg = "🌐 Opening YouTube..."
+                        # v1.8.4 - Use standard webbrowser library for maximum stability on F7
+                        self.status_msg = "🌐 Opening YouTube in Browser..."
+                        threading.Thread(target=webbrowser.open, args=(url,), daemon=True).start()
 
         # Open Live Station: F8
         elif key == curses.KEY_F8:
@@ -766,35 +755,24 @@ class MyTunesApp:
                 self.show_copy_dialog("Live Station", live_url)
                 return
 
-            # Add timestamp to user-data-dir to force size/position flags to be respected (prevents "remembering")
-            # Using int(time.time() / 3600) to keep it stable within the same hour but fresh enough for new versions
-            temp_user_data = os.path.join(tempfile.gettempdir(), f"mytunes_v174_{int(time.time() / 10)}")
-            
-            # Universal flags
+            # App Mode Flags
             flags = [
                 f"--app={live_url}", 
                 "--window-size=712,800", 
                 "--window-position=100,100",
-                f"--user-data-dir={temp_user_data}", 
+                "--new-window",
                 "--no-first-run",
-                "--disable-extensions",
-                "--disable-default-apps",
-                "--disable-features=Translation",
-                "--disable-save-password-bubble",
-                "--disable-translate"
+                "--disable-extensions"
             ]
             
             launched = False
-            # 1. macOS (Avoid AppleScript to prevent permission prompts)
+            # v1.8.4 - Subprocess Isolation (start_new_session) to prevent crashes on WSL/Linux
+            # 1. macOS
             if sys.platform == 'darwin':
-                browsers = [
-                    "/Applications/Google Chrome.app/Contents/MacOS/Google Chrome",
-                    "/Applications/Brave Browser.app/Contents/MacOS/Brave Browser"
-                ]
+                browsers = ["/Applications/Google Chrome.app/Contents/MacOS/Google Chrome", "/Applications/Brave Browser.app/Contents/MacOS/Brave Browser"]
                 for b_path in browsers:
                     if os.path.exists(b_path):
                         try:
-                            # Use 'open -na' but without AppleScript to stay 'standard' and avoid prompts
                             subprocess.Popen(["open", "-na", b_path, "--args"] + flags, stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)
                             launched = True; break
                         except: pass
@@ -802,53 +780,37 @@ class MyTunesApp:
             # 2. Windows Native
             elif sys.platform == 'win32':
                 win_paths = [
-                    os.path.join(os.environ.get('PROGRAMFILES', 'C:\\Program Files'), 'Google\\Chrome\\Application\\chrome.exe'),
-                    os.path.join(os.environ.get('PROGRAMFILES(X86)', 'C:\\Program Files (x86)'), 'Google\\Chrome\\Application\\chrome.exe'),
+                    os.path.join(os.environ.get('PROGRAMFILES', ''), 'Google\\Chrome\\Application\\chrome.exe'),
+                    os.path.join(os.environ.get('PROGRAMFILES(X86)', ''), 'Google\\Chrome\\Application\\chrome.exe'),
                     os.path.join(os.environ.get('LOCALAPPDATA', ''), 'Google\\Chrome\\Application\\chrome.exe'),
-                    os.path.join(os.environ.get('PROGRAMFILES', 'C:\\Program Files'), 'BraveSoftware\\Brave-Browser\\Application\\brave.exe'),
-                    os.path.join(os.environ.get('PROGRAMFILES(X86)', 'C:\\Program Files (x86)'), 'Microsoft\\Edge\\Application\\msedge.exe'),
-                    os.path.join(os.environ.get('PROGRAMFILES', 'C:\\Program Files'), 'Microsoft\\Edge\\Application\\msedge.exe'),
-                ]
-                # v1.8.1 - Precise: No internal quotes around URL to avoid misparsing as literal parts of URI
-                win_flags = [
-                    f'--app={live_url}',
-                    '--window-size=712,800',
-                    '--window-position=100,100',
-                    '--new-window',
-                    '--no-first-run',
-                    '--disable-extensions'
                 ]
                 for p in win_paths:
-                    if os.path.exists(p):
+                    if p and os.path.exists(p):
                         try:
-                            # Use list-based Popen for native Windows
-                            subprocess.Popen([p] + win_flags, stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)
+                            subprocess.Popen([p] + flags, stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)
                             launched = True; break
                         except: pass
             
-            # 3. WSL (Run Windows Chrome via cmd.exe)
+            # 3. WSL / Linux (Direct path with session isolation)
             elif self.is_wsl():
-                try:
-                    # v1.8.1 - Pure and Precise for WSL->CMD
-                    # 1. No internal quotes for the URL (prevents "Empty URL" / navigation failure)
-                    # 2. Use start "" chrome (Prevents 'start' from hijacking the first flag as a title)
-                    # 3. Combined flags for maximum compatibility
-                    c_args = [
-                        f'--app={live_url}',
-                        '--window-size=712,800',
-                        '--window-position=100,100',
-                        '--new-window',
-                        '--no-first-run',
-                        '--disable-extensions'
-                    ]
-                    # Direct call to chrome via cmd start
-                    full_cmd = f'start "" chrome {" ".join(c_args)}'
-                    subprocess.Popen(["cmd.exe", "/c", full_cmd], stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)
-                    launched = True
-                except:
-                    # Fallback to general start
+                wsl_paths = [
+                    "/mnt/c/Program Files/Google/Chrome/Application/chrome.exe",
+                    "/mnt/c/Program Files (x86)/Google/Chrome/Application/chrome.exe",
+                    "/mnt/c/Program Files/BraveSoftware/Brave-Browser/Application/brave.exe",
+                ]
+                for p in wsl_paths:
+                    if os.path.exists(p):
+                        try:
+                            # CRITICAL: start_new_session=True isolates the browser from the TUI process group
+                            # This prevents the TUI from dying when navigating or if the browser has shell issues.
+                            subprocess.Popen([p] + flags, stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL, start_new_session=True)
+                            launched = True; break
+                        except: pass
+                
+                if not launched:
                     try:
-                        subprocess.Popen(["cmd.exe", "/c", "start", live_url], stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)
+                        # Fallback for WSL to CMD
+                        subprocess.Popen(["cmd.exe", "/c", f"start chrome --app={live_url}"], stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL, start_new_session=True)
                         launched = True
                     except: pass
 
@@ -1454,11 +1416,20 @@ class MyTunesApp:
 
     def run(self):
         while self.running:
-            self.loop_count = (self.loop_count + 1) % 1000
-            self.update_playback_state()
-            self.check_autoplay()
-            self.draw()
-            self.handle_input()
+            try:
+                self.loop_count = (self.loop_count + 1) % 1000
+                self.update_playback_state()
+                self.check_autoplay()
+                self.draw()
+                self.handle_input()
+            except Exception as e:
+                # v1.8.4 - Global resilience: Catch and log loop errors instead of crashing
+                try: 
+                    with open("/tmp/mytunes_error.log", "a") as f:
+                        f.write(f"[{time.ctime()}] Loop Error: {str(e)}\n")
+                except: pass
+                # Small sleep to prevent infinite tight loop on persistent error
+                time.sleep(0.1)
         
         if self.stop_on_exit:
             self.player.stop()

{mytunes_pro-1.8.1.dist-info → mytunes_pro-1.8.4.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mytunes-pro
-Version: 1.8.1
+Version: 1.8.4
 Summary: A lightweight, keyboard-centric terminal player for streaming YouTube music.
 Author-email: loxo <loxo5432@gmail.com>
 Project-URL: Homepage, https://github.com/postgresql-co-kr/mytunes
@@ -19,7 +19,7 @@ Dynamic: license-file
 
 # 🎵 MyTunes Pro (Korean)
 
-**현대적인 CLI 유튜브 뮤직 플레이어 (v1.8.1)**  
+**현대적인 CLI 유튜브 뮤직 플레이어 (v1.8.4)**  
 터미널 환경에서 **YouTube 음악을 검색하여 듣는** 가볍고 빠른 키보드 중심의 플레이어입니다.  
 한국어 입력 환경에서도 **숫자 키(1~5)**를 통해 지연 없는 쾌적한 조작이 가능합니다.
 
@@ -208,7 +208,7 @@ Windows 환경에서 한글 검색이 안 되거나 설치가 어려운 분들
 
 # 🎵 MyTunes Pro (English)
 
-**Modern CLI YouTube Music Player (v1.8.1)**  
+**Modern CLI YouTube Music Player (v1.8.4)**  
 A lightweight, keyboard-centric terminal player for streaming YouTube music.  
 
 ---
@@ -296,7 +296,20 @@ sudo apt install mpv python3 python3-pip pipx python3-venv -y
 
 ## 🔄 Changelog
 
-### v1.8.1 (Latest)
+### v1.8.4 (Latest)
+
+- **Python Crash Fix (WSL)**: Eliminated premature termination by implementing `start_new_session=True` for browser launches, isolating them from the TUI process group.
+- **Hybrid Browser Strategy**: Switched to the standard `webbrowser` library for F7 (YouTube links) for maximum internal stability.
+- **Global Error Protection**: Wrapped the main application loop in an exception guard to catch and log transient OS errors without crashing the entire app.
+- **Refined Process Cleanup**: Specialized the `pkill` logic to prevent accidental self-termination while maintaining reliable MPV management.
+
+### v1.8.3
+
+- **Direct Binary Execution (WSL)**: Resolved shell parsing issues by bypassing `cmd.exe` and directly executing Windows browser binaries via `/mnt/c/` paths.
+- **App Mode Reliability**: Guaranteed 712x800 popup mode by ensuring flags are delivered directly to the browser process without intermediate shell mangling.
+- **Fixed URL Resolution**: Eliminated the "Empty URL" bug by standardizing argument passing between WSL and Windows.
+
+### v1.8.1
 
 - **Fixed App Mode (WSL/Win)**: Guaranteed the browser opens in a clean "App Mode" popup by fixing shell quoting issues in the launch command.
 - **URL Resolution Fix**: Resolved the "Empty URL" bug on WSL/Windows by ensuring the `--app` flag is correctly parsed by the native Windows shell.

mytunes_pro-1.8.4.dist-info/RECORD

@@ -0,0 +1,8 @@
+mytunes/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
+mytunes/app.py,sha256=l9KMZxjy_BvLGI9Pphk5E1QYBMFWQ2ICWijCjWYM3ks,59186
+mytunes_pro-1.8.4.dist-info/licenses/LICENSE,sha256=lOrP0EIjxcgJia__W3f3PVDZkRd2oRzFkyH2g3LRRCg,1063
+mytunes_pro-1.8.4.dist-info/METADATA,sha256=tHjXf9El5TQnO2Ty-uq-ARJB2Yya4sR4puj5QAWIL_8,17271
+mytunes_pro-1.8.4.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
+mytunes_pro-1.8.4.dist-info/entry_points.txt,sha256=6-MsC13nIgzLvrREaGotc32FgxHx_Iuu1z2qCzJs1_4,65
+mytunes_pro-1.8.4.dist-info/top_level.txt,sha256=KWzdFyNNG_sO7GT83-sN5fYArP4_DL5I8HYIwgazXyY,8
+mytunes_pro-1.8.4.dist-info/RECORD,,

mytunes_pro-1.8.1.dist-info/RECORD

@@ -1,8 +0,0 @@
-mytunes/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-mytunes/app.py,sha256=cMlDP9j0M7AtoJtgCQiYDysG9TmoDknq-yZG3Th76Oc,60962
-mytunes_pro-1.8.1.dist-info/licenses/LICENSE,sha256=lOrP0EIjxcgJia__W3f3PVDZkRd2oRzFkyH2g3LRRCg,1063
-mytunes_pro-1.8.1.dist-info/METADATA,sha256=WMwanWpyV1OEIPMJJHgiHjSqZ0RhY9Yrf1NcOwjvqpw,16208
-mytunes_pro-1.8.1.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
-mytunes_pro-1.8.1.dist-info/entry_points.txt,sha256=6-MsC13nIgzLvrREaGotc32FgxHx_Iuu1z2qCzJs1_4,65
-mytunes_pro-1.8.1.dist-info/top_level.txt,sha256=KWzdFyNNG_sO7GT83-sN5fYArP4_DL5I8HYIwgazXyY,8
-mytunes_pro-1.8.1.dist-info/RECORD,,