mytunes-pro 1.8.3__py3-none-any.whl → 1.8.5__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.3"
+APP_VERSION = "1.8.5"
 
 # === [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"])
@@ -619,14 +620,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,21 +754,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 +765,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,71 +790,39 @@ 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.2 - Maximum precision: --app flag MUST be exact and first for reliable popup mode
-                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 directly if possible, fallback to cmd.exe)
+            # 3. WSL / Linux (Direct path with session isolation)
             elif self.is_wsl():
-                # v1.8.3 - Direct binary execution for maximum precision (Avoids cmd.exe shell splitting)
-                # We try standard Windows installation paths via /mnt/c/
-                wsl_win_paths = [
+                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",
-                    "/mnt/c/Program Files (x86)/Microsoft/Edge/Application/msedge.exe",
-                    "/mnt/c/Program Files/Microsoft/Edge/Application/msedge.exe",
                 ]
-                # Same precise flags for App Mode
-                wsl_win_flags = [
-                    f'--app={live_url}',
-                    '--window-size=712,800',
-                    '--window-position=100,100',
-                    '--new-window',
-                    '--no-first-run',
-                    '--disable-extensions'
-                ]
-                
-                launched_direct = False
-                for p in wsl_win_paths:
+                for p in wsl_paths:
                     if os.path.exists(p):
                         try:
-                            # Direct execution from WSL to Windows binary is extremely stable
-                            subprocess.Popen([p] + wsl_win_flags, stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)
-                            launched = True; launched_direct = True; break
+                            # 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_direct:
+                if not launched:
                     try:
-                        # Final fallback: cmd.exe start (Literal strings, no title)
-                        cmd_fallback = f'start chrome --app={live_url} --window-size=712,800 --new-window'
-                        subprocess.Popen(["cmd.exe", "/c", cmd_fallback], 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:
-                        try:
-                            subprocess.Popen(["cmd.exe", "/c", "start", live_url], stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)
-                            launched = True
-                        except: pass
+                    except: pass
 
             # 4. Native Linux
             else:
@@ -1470,11 +1426,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.3.dist-info → mytunes_pro-1.8.5.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mytunes-pro
-Version: 1.8.3
+Version: 1.8.5
 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.3)**  
+**현대적인 CLI 유튜브 뮤직 플레이어 (v1.8.5)**  
 터미널 환경에서 **YouTube 음악을 검색하여 듣는** 가볍고 빠른 키보드 중심의 플레이어입니다.  
 한국어 입력 환경에서도 **숫자 키(1~5)**를 통해 지연 없는 쾌적한 조작이 가능합니다.
 
@@ -208,7 +208,7 @@ Windows 환경에서 한글 검색이 안 되거나 설치가 어려운 분들
 
 # 🎵 MyTunes Pro (English)
 
-**Modern CLI YouTube Music Player (v1.8.3)**  
+**Modern CLI YouTube Music Player (v1.8.4)**  
 A lightweight, keyboard-centric terminal player for streaming YouTube music.  
 
 ---
@@ -296,7 +296,19 @@ sudo apt install mpv python3 python3-pip pipx python3-venv -y
 
 ## 🔄 Changelog
 
-### v1.8.3 (Latest)
+### v1.8.5 (Latest)
+
+- **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.

mytunes_pro-1.8.5.dist-info/RECORD

@@ -0,0 +1,8 @@
+mytunes/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
+mytunes/app.py,sha256=HyaRwiTtall9kcCj9JOs-93QPk6Nr15HQC_3ENfvqgI,59636
+mytunes_pro-1.8.5.dist-info/licenses/LICENSE,sha256=lOrP0EIjxcgJia__W3f3PVDZkRd2oRzFkyH2g3LRRCg,1063
+mytunes_pro-1.8.5.dist-info/METADATA,sha256=uFEB7IuCMadyb02juYp_6sCkI6GAuWhJ4_Y67GGo7eI,17561
+mytunes_pro-1.8.5.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
+mytunes_pro-1.8.5.dist-info/entry_points.txt,sha256=6-MsC13nIgzLvrREaGotc32FgxHx_Iuu1z2qCzJs1_4,65
+mytunes_pro-1.8.5.dist-info/top_level.txt,sha256=KWzdFyNNG_sO7GT83-sN5fYArP4_DL5I8HYIwgazXyY,8
+mytunes_pro-1.8.5.dist-info/RECORD,,

mytunes_pro-1.8.3.dist-info/RECORD

@@ -1,8 +0,0 @@
-mytunes/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-mytunes/app.py,sha256=-bEfxmLNP_nSrvdoZNw3WQTgtlkn5dwUON7XgG_7ufw,61946
-mytunes_pro-1.8.3.dist-info/licenses/LICENSE,sha256=lOrP0EIjxcgJia__W3f3PVDZkRd2oRzFkyH2g3LRRCg,1063
-mytunes_pro-1.8.3.dist-info/METADATA,sha256=hY0oW58qdE_X9QE3vH-6LaX5R05XUljQ6IZCAfJbdbE,16657
-mytunes_pro-1.8.3.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
-mytunes_pro-1.8.3.dist-info/entry_points.txt,sha256=6-MsC13nIgzLvrREaGotc32FgxHx_Iuu1z2qCzJs1_4,65
-mytunes_pro-1.8.3.dist-info/top_level.txt,sha256=KWzdFyNNG_sO7GT83-sN5fYArP4_DL5I8HYIwgazXyY,8
-mytunes_pro-1.8.3.dist-info/RECORD,,