mytunes-pro 1.8.0__py3-none-any.whl → 1.9.4__py3-none-any.whl

mytunes/app.py

@@ -17,6 +17,7 @@ import socket
 import locale
 import signal
 import warnings
+import webbrowser
 # Suppress urllib3 warning about LibreSSL compatibility
 warnings.filterwarnings("ignore", message=".*urllib3 v2 only supports OpenSSL 1.1.1+.*")
 import webbrowser
@@ -35,7 +36,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.0"
+APP_VERSION = "1.9.4"
 
 # === [Strings & Localization] ===
 STRINGS = {
@@ -245,10 +246,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 +334,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"])
@@ -619,14 +621,24 @@ class MyTunesApp:
             if self.selection_idx > 0:
                 self.selection_idx -= 1
                 if self.selection_idx < self.scroll_offset: self.scroll_offset = self.selection_idx
+            elif current_list:
+                # v1.8.5 - Wrapping: Top to Bottom
+                self.selection_idx = len(current_list) - 1
+                h, _ = self.stdscr.getmaxyx()
+                # Maintain scroll consistency (h - 10 matches draw() layout)
+                list_area_height = h - 10
+                self.scroll_offset = max(0, self.selection_idx - list_area_height + 1)
         elif key == curses.KEY_DOWN or k_char in ['j', 'ㅓ']:
             if self.selection_idx < len(current_list) - 1:
                 self.selection_idx += 1
                 h, _ = self.stdscr.getmaxyx()
-                # Use h - 10 to match inner_h in draw() (h - footer_h(5) - header_top(3) - borders(2))
                 list_area_height = h - 10
                 if self.selection_idx >= self.scroll_offset + list_area_height:
                     self.scroll_offset = self.selection_idx - list_area_height + 1
+            elif current_list:
+                # v1.8.5 - Wrapping: Bottom to Top
+                self.selection_idx = 0
+                self.scroll_offset = 0
 
         # Enter / Select: Enter Only (L moved to Forward)
         elif key == '\n' or key == 10 or key == 13:
@@ -743,58 +755,55 @@ 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
+        # Open Live Station (F8): App Mode with Optimized Flags (v1.8.6)
         elif key == curses.KEY_F8:
             live_url = "https://mytunes.postgresql.co.kr/live/"
             if self.is_remote():
                 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
+            # v1.9.4 - Ultimate WSL Fix: Use Standard Webbrowser Module
+            # Subprocess/cmd.exe based launching in WSL is unstable.
+            # We switch to the standard `webbrowser` module which handles system default browser reliably.
+            # This sacrifices window sizing but guarantees the URL opens.
+            if self.is_wsl():
+                threading.Thread(target=webbrowser.open, args=(live_url,), daemon=True).start()
+                return
+
+            # Native (Mac/Windows/Linux) Logic Continues Below...
+            temp_user_data = os.path.join(tempfile.gettempdir(), f"mytunes_v190_{int(time.time() / 10)}")
+
+            # Optimized Flag Set (Context7 Research)
             flags = [
                 f"--app={live_url}", 
                 "--window-size=712,800", 
                 "--window-position=100,100",
-                f"--user-data-dir={temp_user_data}", 
+                f"--user-data-dir={temp_user_data}",
                 "--no-first-run",
-                "--disable-extensions",
+                "--no-default-browser-check",
                 "--disable-default-apps",
+                "--disable-infobars",
+                "--disable-translate",
                 "--disable-features=Translation",
                 "--disable-save-password-bubble",
-                "--disable-translate"
+                "--autoplay-policy=no-user-gesture-required",
+                "--new-window",
+                "--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
+                            # Use -na to open a fresh instance
                             subprocess.Popen(["open", "-na", b_path, "--args"] + flags, stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)
                             launched = True; break
                         except: pass
@@ -802,52 +811,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'),
-                ]
-                # In native Windows, we remove user-data-dir to avoid permission/expansion errors
-                # app mode + new-window + window-size is sufficient.
-                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:
-                    # Pure CMD start without user-data-dir to avoid expansion/path issues.
-                    # Window sizing works reliably with just these flags.
-                    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 (path is already converted upstream)
+                        subprocess.Popen(["cmd.exe", "/c", f"start chrome --app={live_url} --user-data-dir={temp_user_data}"], stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL, start_new_session=True)
                         launched = True
                     except: pass
 
@@ -1453,11 +1447,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.0.dist-info → mytunes_pro-1.9.4.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mytunes-pro
-Version: 1.8.0
+Version: 1.9.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.7.9)**  
+**현대적인 CLI 유튜브 뮤직 플레이어 (v1.9.4)**  
 터미널 환경에서 **YouTube 음악을 검색하여 듣는** 가볍고 빠른 키보드 중심의 플레이어입니다.  
 한국어 입력 환경에서도 **숫자 키(1~5)**를 통해 지연 없는 쾌적한 조작이 가능합니다.
 
@@ -208,7 +208,7 @@ Windows 환경에서 한글 검색이 안 되거나 설치가 어려운 분들
 
 # 🎵 MyTunes Pro (English)
 
-**Modern CLI YouTube Music Player (v1.7.9)**  
+**Modern CLI YouTube Music Player (v1.9.4)**  
 A lightweight, keyboard-centric terminal player for streaming YouTube music.  
 
 ---
@@ -296,7 +296,74 @@ sudo apt install mpv python3 python3-pip pipx python3-venv -y
 
 ## 🔄 Changelog
 
-### v1.7.9 (Latest)
+### v1.9.4 (Latest)
+
+- **Ultimate WSL Fix**: Switched to using Python's standard `webbrowser` module for opening links in WSL. This fully delegates browser launching to the system (Windows host), ensuring maximum stability and eliminating all `subprocess` or `cmd.exe` related conflicts.
+
+### v1.9.3
+
+- **Hotfix for Startup**: Fixed a syntax error introduced in v1.9.2 that prevented the application from starting.
+
+### v1.9.2
+
+- **Disable WSL Profile Isolation**: To ensure maximum stability and prevent `cmd.exe` conflicts, MyTunes now temporarily disables profile isolation (forced window size/position) on WSL. It runs using the default Chrome profile, guaranteeing reliable launching.
+
+### v1.9.1
+
+- **Fix CMD Output Pollution (WSL)**: Resolved an issue where `cmd.exe` printed "UNC paths are not supported" warnings when executed from a WSL directory, corrupting the temporary path retrieval. Now parses output safely and executes from `/mnt/c` to prevent warnings.
+
+### v1.9.0
+
+- **Fix WSL Profile Error**: Switched to using the **native Windows TEMP directory** (e.g., `C:\Users\...\AppData\Local\Temp`) for the browser profile in WSL. This prevents file locking issues caused by Chrome treating `\\wsl$\` paths as network drives.
+
+### v1.8.9
+
+- **Robust WSL Path Fix**: Resolved an issue where direct browser launching (non-fallback) in WSL was still using Linux paths for the profile, causing "User Data Directory" creation errors. Path conversion is now applied globally before launch.
+
+### v1.8.8
+
+- **WSL Path Conversion**: Implemented `wslpath -w` logic to correctly convert Linux-style temp paths to Windows format when launching Chrome via `cmd.exe` on WSL.
+
+### v1.8.7
+
+- **Syntax Fix (WSL)**: Corrected a typo in the browser launch command that caused a crash on Linux/WSL systems.
+
+### v1.8.6
+
+- **Browser Popup Optimization (Context7)**: Improved Live Station (F8) experience with optimized CLI flags for a perfectly minimalist UI.
+- **Forced Window Dimensions**: Implemented profile isolation using a timestamped `user-data-dir` to ensure window size and position are always respected, overriding session memory.
+- **UI Cleanup**: Automatically hides distraction-bars (translation, password, automation infobars) and enables instant autoplay for live streams.
+
+### v1.8.5
+
+- **Looping Navigation (Menu Wrapping)**: Pressing UP at the first item now wraps to the last item, and pressing DOWN at the last item wraps to the first.
+- **Improved UI Flow**: Enhanced keyboard navigation experience across all list views (Main, Search, Favorites, History).
+
+### v1.8.4
+
+- **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.
+- **Reliable Popup UI**: Standardized on `start "" chrome` for WSL to ensure flags are never misidentified as window titles.
+
+### v1.8.0
+
+- **Stabilized Browser Launch (Windows/WSL)**: Completely removed the `--user-data-dir` flag for all Windows-based environments. This permanently resolves the "cannot read or write" directory errors while maintaining reliable 712x800 window sizing through pure app-mode flags.
+- **Clean CMD Execution**: Simplified the WSL-to-Windows transition by using standard `cmd.exe` calls without complex path or variable expansion, ensuring consistent behavior across all systems.
+
+### v1.7.9
 
 - **Pure CMD-based Launch (WSL/Win)**: Final fix for WSL-to-Windows browser launch using `cmd.exe /c` with native `%LOCALAPPDATA%` expansion.
 - **Directory Reliability**: Ensured Chrome data directory creation and access by using native Windows shell commands, eliminating the "cannot read or write" errors seen in v1.7.8.

mytunes_pro-1.9.4.dist-info/RECORD

@@ -0,0 +1,8 @@
+mytunes/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
+mytunes/app.py,sha256=9qM1blYaDozAHFHk83xfZ2qg_Ijc4BgFY5KNM4VN9Us,60877
+mytunes_pro-1.9.4.dist-info/licenses/LICENSE,sha256=lOrP0EIjxcgJia__W3f3PVDZkRd2oRzFkyH2g3LRRCg,1063
+mytunes_pro-1.9.4.dist-info/METADATA,sha256=hxsZZRaOEmZy7i8duKAC3IjWxtHMZytMsekV4jhx-b8,19834
+mytunes_pro-1.9.4.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
+mytunes_pro-1.9.4.dist-info/entry_points.txt,sha256=6-MsC13nIgzLvrREaGotc32FgxHx_Iuu1z2qCzJs1_4,65
+mytunes_pro-1.9.4.dist-info/top_level.txt,sha256=KWzdFyNNG_sO7GT83-sN5fYArP4_DL5I8HYIwgazXyY,8
+mytunes_pro-1.9.4.dist-info/RECORD,,

mytunes_pro-1.8.0.dist-info/RECORD

@@ -1,8 +0,0 @@
-mytunes/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-mytunes/app.py,sha256=EBMpGbTzRmynKsNHk1P-5qKH43LZn6KHgRM1SVKKedQ,60855
-mytunes_pro-1.8.0.dist-info/licenses/LICENSE,sha256=lOrP0EIjxcgJia__W3f3PVDZkRd2oRzFkyH2g3LRRCg,1063
-mytunes_pro-1.8.0.dist-info/METADATA,sha256=ihPxSaFC7mLvJfJrIXqY6v55M3h8Qm2yts71GJuBjCg,15294
-mytunes_pro-1.8.0.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
-mytunes_pro-1.8.0.dist-info/entry_points.txt,sha256=6-MsC13nIgzLvrREaGotc32FgxHx_Iuu1z2qCzJs1_4,65
-mytunes_pro-1.8.0.dist-info/top_level.txt,sha256=KWzdFyNNG_sO7GT83-sN5fYArP4_DL5I8HYIwgazXyY,8
-mytunes_pro-1.8.0.dist-info/RECORD,,