synodic-client 0.0.1.dev23__tar.gz → 0.0.1.dev25__tar.gz

{synodic_client-0.0.1.dev23 → synodic_client-0.0.1.dev25}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: synodic_client
-Version: 0.0.1.dev23
+Version: 0.0.1.dev25
 Author-Email: Synodic Software <contact@synodic.software>
 License: LGPL-3.0-or-later
 Project-URL: homepage, https://github.com/synodic/synodic-client

{synodic_client-0.0.1.dev23 → synodic_client-0.0.1.dev25}/pyproject.toml

@@ -14,7 +14,7 @@ dependencies = [
     "velopack>=0.0.1369.dev7516",
     "typer>=0.23.1",
 ]
-version = "0.0.1.dev23"
+version = "0.0.1.dev25"
 
 [project.license]
 text = "LGPL-3.0-or-later"

synodic_client-0.0.1.dev25/synodic_client/_version.py

@@ -0,0 +1 @@
+__version__ = '0.0.1.dev25'

synodic_client-0.0.1.dev25/synodic_client/application/bootstrap.py

@@ -0,0 +1,39 @@
+"""Bootstrap entry point for PyInstaller builds.
+
+Runs the lightweight startup preamble — logging, Velopack hooks, and
+protocol registration — **before** importing heavy modules (PySide6,
+porringer).  Velopack's install/uninstall/update hooks have strict
+timeouts (15–30 s) and must complete before the process is killed.
+
+Import order matters:
+    1. stdlib + config (pure-Python, fast)
+    2. configure_logging() — now Qt-free
+    3. initialize_velopack() — hooks run with logging active
+    4. register_protocol() — stdlib only
+    5. import qt.application — PySide6 / porringer loaded here
+"""
+
+import sys
+
+from synodic_client.config import set_dev_mode
+from synodic_client.logging import configure_logging
+from synodic_client.protocol import register_protocol
+from synodic_client.updater import initialize_velopack
+
+_PROTOCOL_SCHEME = 'synodic'
+
+# Parse --dev flag early so logging uses the right filename.
+_dev_mode = '--dev' in sys.argv[1:]
+set_dev_mode(_dev_mode)
+
+configure_logging()
+initialize_velopack()
+
+if not _dev_mode:
+    register_protocol(sys.executable)
+
+# Heavy imports happen here — PySide6, porringer, etc.
+from synodic_client.application.qt import application  # noqa: E402
+
+_uri = next((a for a in sys.argv[1:] if a.lower().startswith(f'{_PROTOCOL_SCHEME}://')), None)
+application(uri=_uri, dev_mode=_dev_mode)

{synodic_client-0.0.1.dev23 → synodic_client-0.0.1.dev25}/synodic_client/application/screen/tray.py

@@ -6,8 +6,8 @@ from pathlib import Path
 
 from porringer.api import API
 from porringer.schema import SetupParameters, SyncStrategy
-from PySide6.QtCore import QThread, QTimer, Signal
-from PySide6.QtGui import QAction
+from PySide6.QtCore import QThread, QTimer, QUrl, Signal
+from PySide6.QtGui import QAction, QDesktopServices
 from PySide6.QtWidgets import (
     QApplication,
     QDialog,
@@ -29,7 +29,7 @@ from synodic_client.application.screen.screen import MainWindow
 from synodic_client.application.theme import UPDATE_SOURCE_DIALOG_MIN_WIDTH
 from synodic_client.client import Client
 from synodic_client.config import GlobalConfiguration
-from synodic_client.logging import open_log
+from synodic_client.logging import log_path
 from synodic_client.resolution import resolve_config, resolve_enabled_plugins, resolve_update_config, update_and_resolve
 from synodic_client.updater import GITHUB_REPO_URL, UpdateChannel, UpdateInfo
 
@@ -281,7 +281,7 @@ class TrayScreen:
         self.settings_menu.addSeparator()
 
         self.open_log_action = QAction('Open Log...', self.settings_menu)
-        self.open_log_action.triggered.connect(open_log)
+        self.open_log_action.triggered.connect(self._open_log)
         self.settings_menu.addAction(self.open_log_action)
 
         self.menu.addSeparator()
@@ -294,6 +294,14 @@ class TrayScreen:
 
     # -- Config helpers --
 
+    @staticmethod
+    def _open_log() -> None:
+        """Open the log file in the system's default editor."""
+        path = log_path()
+        if not path.exists():
+            path.touch()
+        QDesktopServices.openUrl(QUrl.fromLocalFile(str(path)))
+
     def _resolve_config(self) -> GlobalConfiguration:
         """Return the injected config or resolve from disk."""
         if self._config is not None:

{synodic_client-0.0.1.dev23 → synodic_client-0.0.1.dev25}/synodic_client/cli.py

@@ -5,7 +5,6 @@ from typing import Annotated
 import typer
 
 from synodic_client import __version__
-from synodic_client.application.qt import application
 
 app = typer.Typer(
     name='synodic-c',
@@ -38,4 +37,6 @@ def main(
     ] = False,
 ) -> None:
     """Launch the Synodic Client GUI application."""
+    from synodic_client.application.qt import application  # noqa: PLC0415
+
     application(uri=uri, dev_mode=dev)

{synodic_client-0.0.1.dev23 → synodic_client-0.0.1.dev25}/synodic_client/logging.py

@@ -1,7 +1,6 @@
 """Centralised logging configuration for the Synodic Client.
 
-Provides a rotating file handler with eager flushing and a helper to open
-the current log file in the system's default editor.
+Provides a rotating file handler with eager flushing.
 """
 
 import logging
@@ -9,9 +8,6 @@ import tempfile
 from logging.handlers import RotatingFileHandler
 from pathlib import Path
 
-from PySide6.QtCore import QUrl
-from PySide6.QtGui import QDesktopServices
-
 from synodic_client.config import is_dev_mode
 
 _LOG_FILENAME = 'synodic.log'
@@ -52,7 +48,15 @@ def configure_logging() -> None:
     Attaches a :class:`EagerRotatingFileHandler` to the ``synodic_client``
     and ``porringer`` loggers and configures :func:`logging.basicConfig`
     for ``INFO`` level output on *stderr*.
+
+    Safe to call more than once — subsequent calls are no-ops.
     """
+    app_logger = logging.getLogger('synodic_client')
+
+    # Guard: skip if already configured (e.g. by bootstrap.py)
+    if any(isinstance(h, EagerRotatingFileHandler) for h in app_logger.handlers):
+        return
+
     logging.basicConfig(level=logging.INFO)
 
     handler = EagerRotatingFileHandler(
@@ -63,22 +67,8 @@ def configure_logging() -> None:
     )
     handler.setFormatter(logging.Formatter(_FORMAT))
 
-    app_logger = logging.getLogger('synodic_client')
     app_logger.addHandler(handler)
 
     porringer_logger = logging.getLogger('porringer')
     porringer_logger.addHandler(handler)
     porringer_logger.setLevel(logging.INFO)
-
-
-def open_log() -> None:
-    """Open the log file in the system's default editor.
-
-    Creates an empty file if one does not yet exist so that the OS always
-    has something to open.
-    """
-    path = log_path()
-    if not path.exists():
-        path.touch()
-
-    QDesktopServices.openUrl(QUrl.fromLocalFile(str(path)))

{synodic_client-0.0.1.dev23 → synodic_client-0.0.1.dev25}/synodic_client/updater.py

@@ -33,7 +33,7 @@ _PLATFORM_SUFFIXES: dict[str, str] = {
 }
 
 
-def _platform_suffix() -> str:
+def platform_suffix() -> str:
     """Return the Velopack channel suffix for the current platform."""
     try:
         return _PLATFORM_SUFFIXES[sys.platform]
@@ -104,8 +104,8 @@ class UpdateConfig:
         so each OS has its own release manifest and nupkg files.
         """
         base = 'dev' if self.channel == UpdateChannel.DEVELOPMENT else 'stable'
-        platform_suffix = _platform_suffix()
-        return f'{base}-{platform_suffix}'
+        suffix = platform_suffix()
+        return f'{base}-{suffix}'
 
 
 class Updater:
@@ -393,13 +393,16 @@ def initialize_velopack() -> None:
     before any UI is shown. Velopack may need to perform cleanup or apply
     pending updates.
 
-    On Windows, the uninstall hook removes the ``synodic://`` URI protocol.
     Protocol registration happens on every app launch (see ``qt.application``).
+
+    .. note::
+
+        The SDK's callback hooks only accept ``PyCFunction`` — add an
+        uninstall hook here when that is fixed upstream.
     """
     logger.info('Initializing Velopack (exe=%s)', sys.executable)
     try:
         app = velopack.App()
-        app.on_before_uninstall_fast_callback(_on_before_uninstall)
         app.run()
         logger.info('Velopack initialized successfully')
     except Exception as e:

{synodic_client-0.0.1.dev23 → synodic_client-0.0.1.dev25}/tests/unit/qt/test_logging.py

@@ -5,12 +5,12 @@ import tempfile
 from pathlib import Path
 from unittest.mock import patch
 
+from synodic_client.application.screen.tray import TrayScreen
 from synodic_client.config import set_dev_mode
 from synodic_client.logging import (
     EagerRotatingFileHandler,
     configure_logging,
     log_path,
-    open_log,
 )
 
 
@@ -106,31 +106,31 @@ class TestConfigureLogging:
 
 
 class TestOpenLog:
-    """Tests for open_log()."""
+    """Tests for TrayScreen._open_log()."""
 
     @staticmethod
     def test_creates_file_if_missing(tmp_path: Path) -> None:
-        """open_log() should create the log file when it does not exist."""
+        """_open_log() should create the log file when it does not exist."""
         log_file = tmp_path / 'synodic.log'
         assert not log_file.exists()
 
         with (
-            patch('synodic_client.logging.log_path', return_value=log_file),
-            patch('synodic_client.logging.QDesktopServices') as mock_ds,
+            patch('synodic_client.application.screen.tray.log_path', return_value=log_file),
+            patch('synodic_client.application.screen.tray.QDesktopServices') as mock_ds,
         ):
-            open_log()
+            TrayScreen._open_log()
             assert log_file.exists()
             mock_ds.openUrl.assert_called_once()
 
     @staticmethod
     def test_opens_existing_file(tmp_path: Path) -> None:
-        """open_log() should open an existing log file without error."""
+        """_open_log() should open an existing log file without error."""
         log_file = tmp_path / 'synodic.log'
         log_file.write_text('existing content', encoding='utf-8')
 
         with (
-            patch('synodic_client.logging.log_path', return_value=log_file),
-            patch('synodic_client.logging.QDesktopServices') as mock_ds,
+            patch('synodic_client.application.screen.tray.log_path', return_value=log_file),
+            patch('synodic_client.application.screen.tray.QDesktopServices') as mock_ds,
         ):
-            open_log()
+            TrayScreen._open_log()
             mock_ds.openUrl.assert_called_once()

{synodic_client-0.0.1.dev23 → synodic_client-0.0.1.dev25}/tests/unit/test_cli.py

@@ -33,7 +33,7 @@ class TestCli:
     @staticmethod
     def test_launches_application_without_uri() -> None:
         """Verify invoking with no args calls application(uri=None, dev_mode=False)."""
-        with patch('synodic_client.cli.application') as mock_app:
+        with patch('synodic_client.application.qt.application') as mock_app:
             result = runner.invoke(app, [])
             assert result.exit_code == 0
             mock_app.assert_called_once_with(uri=None, dev_mode=False)
@@ -42,7 +42,7 @@ class TestCli:
     def test_launches_application_with_uri() -> None:
         """Verify invoking with a URI passes it to application()."""
         test_uri = 'synodic://install?manifest=https://example.com/foo.json'
-        with patch('synodic_client.cli.application') as mock_app:
+        with patch('synodic_client.application.qt.application') as mock_app:
             result = runner.invoke(app, [test_uri])
             assert result.exit_code == 0
             mock_app.assert_called_once_with(uri=test_uri, dev_mode=False)
@@ -50,7 +50,7 @@ class TestCli:
     @staticmethod
     def test_launches_application_with_dev_flag() -> None:
         """Verify --dev flag sets dev_mode=True."""
-        with patch('synodic_client.cli.application') as mock_app:
+        with patch('synodic_client.application.qt.application') as mock_app:
             result = runner.invoke(app, ['--dev'])
             assert result.exit_code == 0
             mock_app.assert_called_once_with(uri=None, dev_mode=True)

{synodic_client-0.0.1.dev23 → synodic_client-0.0.1.dev25}/tests/unit/test_updater.py

@@ -14,8 +14,8 @@ from synodic_client.updater import (
     UpdateInfo,
     Updater,
     UpdateState,
-    _platform_suffix,
     initialize_velopack,
+    platform_suffix,
 )
 
 
@@ -127,13 +127,13 @@ class TestUpdateConfig:
     def test_channel_name_stable() -> None:
         """Verify STABLE channel returns platform-specific 'stable' name."""
         config = UpdateConfig(channel=UpdateChannel.STABLE)
-        assert config.channel_name == f'stable-{_platform_suffix()}'
+        assert config.channel_name == f'stable-{platform_suffix()}'
 
     @staticmethod
     def test_channel_name_development() -> None:
         """Verify DEVELOPMENT channel returns platform-specific 'dev' name."""
         config = UpdateConfig(channel=UpdateChannel.DEVELOPMENT)
-        assert config.channel_name == f'dev-{_platform_suffix()}'
+        assert config.channel_name == f'dev-{platform_suffix()}'
 
 
 @pytest.fixture

synodic_client-0.0.1.dev23/synodic_client/_version.py

@@ -1 +0,0 @@
-__version__ = '0.0.1.dev23'