Refactor: Add Windows support for Debug Console (using threads) and PyInstaller frozen paths

Author: ruflas

README.md

@@ -6,7 +6,7 @@
 
 A lightweight, modern tray indicator that shows your **wireless headset’s battery level**, **charging status**, and lets you **control LEDs and sidetone** — all powered by [HeadsetControl](https://github.com/Sapd/HeadsetControl).
 
-It uses **PySide6 (Qt)** for the graphical interface and works seamlessly with **KDE Plasma**, **GNOME**, **XFCE**, **Cinnamon**, and other Linux desktop environments.
+It uses **PySide6 (Qt)** for the graphical interface and works seamlessly with **KDE Plasma**, **GNOME**, **XFCE**, **Cinnamon**, and other Linux desktop environments ans also macos and Windows.
 
 ![Screenshot of the tray icon](screenshot.png)
 

headset_battery_indicator.py

@@ -8,8 +8,9 @@
 import os
 import logging
 import shutil
+import threading
 from logging.handlers import RotatingFileHandler 
-from PySide6.QtCore import QTimer, QSettings, QSocketNotifier
+from PySide6.QtCore import QTimer, QSettings, QSocketNotifier , Signal, Slot
 from PySide6.QtGui import QIcon, QAction, QActionGroup
 from PySide6.QtWidgets import QApplication, QSystemTrayIcon, QMenu
 
@@ -61,12 +62,18 @@ def setup_logging(debug_to_console=False):
 # --- END LOGGING SETUP ---
 
 class HeadsetBatteryTray(QSystemTrayIcon):
+    command_received = Signal(str)
     def __init__(self, debug_mode=False, parent=None):
         super().__init__(parent)
         self.debug_mode = debug_mode
-        base_path = os.path.dirname(os.path.abspath(__file__))
+
+        if getattr(sys, 'frozen', False):
+            base_path = sys._MEIPASS
+        else:
+            base_path = os.path.dirname(os.path.abspath(__file__))
+            
         self.icons_dir = os.path.join(base_path, 'icons')
-        # Reconfigure logger for console output if in debug mode
+        
         if debug_mode:
             global logger
             logger = setup_logging(debug_to_console=True) 
@@ -77,10 +84,8 @@ def __init__(self, debug_mode=False, parent=None):
         self.settings = QSettings()
         self.load_settings()
 
-        # Initialize and set headsetcontrol_path using robust search
         self.headsetcontrol_path = self._find_headsetcontrol()
 
-        # B. DEPENDENCY CHECK AND ERROR MESSAGE
         if not self.headsetcontrol_path:
             logger.critical("HeadsetControl binary not found. Functionality disabled.")
             self.send_notification(
@@ -99,7 +104,6 @@ def __init__(self, debug_mode=False, parent=None):
         self.timer = QTimer(self)
         self.timer.timeout.connect(self.update_status)
 
-        # Only start timer if binary is found
         if self.headsetcontrol_path:
             self.timer.start(UPDATE_INTERVAL_MS)
 
@@ -118,8 +122,18 @@ def __init__(self, debug_mode=False, parent=None):
             print("  exit                 (quits the application)")
             print("------------------------------------")
 
-            self.stdin_notifier = QSocketNotifier(sys.stdin.fileno(), QSocketNotifier.Type.Read, self)
-            self.stdin_notifier.activated.connect(self.handle_debug_command)
+            self.command_received.connect(self.handle_debug_command)
+
+            def console_listener():
+                while True:
+                    try:
+                        line = sys.stdin.readline()
+                        if line:
+                            self.command_received.emit(line.strip())
+                    except ValueError:
+                        break
+            t = threading.Thread(target=console_listener, daemon=True)
+            t.start()
 
     def _find_headsetcontrol(self):
         """Searches for the headsetcontrol binary using AppImage logic and system PATH."""
@@ -374,10 +388,11 @@ def apply_saved_settings(self):
 
 
     # --- Debug Command Handler ---
-    def handle_debug_command(self):
-        """Processes commands from stdin in debug mode."""
+    @Slot(str)
+    def handle_debug_command(self, line):
+        """Processes commands received from the console thread."""
         try:
-            line = sys.stdin.readline().strip()
+            # We use now 'line' that is an argument
             if not line: return
             parts = line.split()
             if not parts: return

headset_battery_indicator/__main__.py

@@ -8,8 +8,9 @@
 import os
 import logging
 import shutil
+import threading
 from logging.handlers import RotatingFileHandler 
-from PySide6.QtCore import QTimer, QSettings, QSocketNotifier
+from PySide6.QtCore import QTimer, QSettings, QSocketNotifier , Signal, Slot
 from PySide6.QtGui import QIcon, QAction, QActionGroup
 from PySide6.QtWidgets import QApplication, QSystemTrayIcon, QMenu
 
@@ -61,12 +62,18 @@ def setup_logging(debug_to_console=False):
 # --- END LOGGING SETUP ---
 
 class HeadsetBatteryTray(QSystemTrayIcon):
+    command_received = Signal(str)
     def __init__(self, debug_mode=False, parent=None):
         super().__init__(parent)
         self.debug_mode = debug_mode
-        base_path = os.path.dirname(os.path.abspath(__file__))
+
+        if getattr(sys, 'frozen', False):
+            base_path = sys._MEIPASS
+        else:
+            base_path = os.path.dirname(os.path.abspath(__file__))
+            
         self.icons_dir = os.path.join(base_path, 'icons')
-        # Reconfigure logger for console output if in debug mode
+        
         if debug_mode:
             global logger
             logger = setup_logging(debug_to_console=True) 
@@ -77,10 +84,8 @@ def __init__(self, debug_mode=False, parent=None):
         self.settings = QSettings()
         self.load_settings()
 
-        # Initialize and set headsetcontrol_path using robust search
         self.headsetcontrol_path = self._find_headsetcontrol()
 
-        # B. DEPENDENCY CHECK AND ERROR MESSAGE
         if not self.headsetcontrol_path:
             logger.critical("HeadsetControl binary not found. Functionality disabled.")
             self.send_notification(
@@ -99,7 +104,6 @@ def __init__(self, debug_mode=False, parent=None):
         self.timer = QTimer(self)
         self.timer.timeout.connect(self.update_status)
 
-        # Only start timer if binary is found
         if self.headsetcontrol_path:
             self.timer.start(UPDATE_INTERVAL_MS)
 
@@ -118,8 +122,18 @@ def __init__(self, debug_mode=False, parent=None):
             print("  exit                 (quits the application)")
             print("------------------------------------")
 
-            self.stdin_notifier = QSocketNotifier(sys.stdin.fileno(), QSocketNotifier.Type.Read, self)
-            self.stdin_notifier.activated.connect(self.handle_debug_command)
+            self.command_received.connect(self.handle_debug_command)
+
+            def console_listener():
+                while True:
+                    try:
+                        line = sys.stdin.readline()
+                        if line:
+                            self.command_received.emit(line.strip())
+                    except ValueError:
+                        break
+            t = threading.Thread(target=console_listener, daemon=True)
+            t.start()
 
     def _find_headsetcontrol(self):
         """Searches for the headsetcontrol binary using AppImage logic and system PATH."""
@@ -374,10 +388,11 @@ def apply_saved_settings(self):
 
 
     # --- Debug Command Handler ---
-    def handle_debug_command(self):
-        """Processes commands from stdin in debug mode."""
+    @Slot(str)
+    def handle_debug_command(self, line):
+        """Processes commands received from the console thread."""
         try:
-            line = sys.stdin.readline().strip()
+            # We use now 'line' that is an argument
             if not line: return
             parts = line.split()
             if not parts: return

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "headset-battery-indicator"
-version = "1.4.1"
+version = "1.4.2"
 authors = [
     { name = "Ruflas", email = "ruflas@ruflas.dev" },
 ]
@@ -31,7 +31,7 @@ headset-battery-indicator = "headset_battery_indicator.__main__:main"
 [tool.briefcase]
 project_name = "Headset Battery Indicator"
 bundle = "dev.ruflas"
-version = "1.4.1"
+version = "1.4.2"
 app_name = "headset_battery_indicator"
 
 # Define la aplicación