synodic-client 0.0.1.dev84__tar.gz → 0.0.1.dev86__tar.gz

{synodic_client-0.0.1.dev84 → synodic_client-0.0.1.dev86}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: synodic_client
-Version: 0.0.1.dev84
+Version: 0.0.1.dev86
 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.dev84 → synodic_client-0.0.1.dev86}/pyproject.toml

@@ -15,7 +15,7 @@ dependencies = [
     "velopack>=0.0.1535.dev45597",
     "typer>=0.24.1",
 ]
-version = "0.0.1.dev84"
+version = "0.0.1.dev86"
 
 [project.license]
 text = "LGPL-3.0-or-later"

{synodic_client-0.0.1.dev84 → synodic_client-0.0.1.dev86}/synodic_client/application/screen/tool_update_controller.py

@@ -29,9 +29,7 @@ from synodic_client.operations.tool import (
     update_all_tools,
     update_tool,
 )
-from synodic_client.resolution import (
-    resolve_update_config,
-)
+from synodic_client.resolution import resolve_update_config
 
 if TYPE_CHECKING:
     from synodic_client.application.config_store import ConfigStore

{synodic_client-0.0.1.dev84 → synodic_client-0.0.1.dev86}/synodic_client/application/screen/tray.py

@@ -55,6 +55,9 @@ class TrayScreen:
         self.tray.activated.connect(self._on_tray_activated)
 
         self._build_menu()
+        # setContextMenu delegates positioning to the OS tray API, which
+        # correctly avoids the taskbar.  Do not manually handle the Context
+        # activation reason — that causes duplicate popups or z-order issues.
         self.tray.setContextMenu(self._menu)
 
         # At early Windows login the notification area may not be ready.

{synodic_client-0.0.1.dev84 → synodic_client-0.0.1.dev86}/synodic_client/application/update_controller.py

@@ -31,10 +31,8 @@ from synodic_client.application.theme import (
 from synodic_client.application.update_model import UpdateModel
 from synodic_client.operations.schema import UpdateCheckResult
 from synodic_client.operations.update import apply_self_update, check_self_update, download_self_update
-from synodic_client.resolution import (
-    ResolvedConfig,
-    resolve_update_config,
-)
+from synodic_client.resolution import resolve_update_config
+from synodic_client.schema import ResolvedConfig, UpdateState
 from synodic_client.startup import sync_startup
 
 if TYPE_CHECKING:
@@ -93,7 +91,7 @@ class UpdateController:
 
         # Track update-relevant config fields to avoid reinitialising
         # on every config save (e.g. timestamp-only changes).
-        self._update_config_key = self._extract_update_key(store.config)
+        self._update_config_key = resolve_update_config(store.config)
 
         # Periodic auto-update timer
         self._auto_update_timer: QTimer | None = None
@@ -225,7 +223,7 @@ class UpdateController:
             return
         self._auto_apply = config.auto_apply
 
-        new_key = self._extract_update_key(config)
+        new_key = resolve_update_config(config)
         if new_key == self._update_config_key:
             return
         self._update_config_key = new_key
@@ -237,16 +235,6 @@ class UpdateController:
     # Updater re-initialisation
     # ------------------------------------------------------------------
 
-    @staticmethod
-    def _extract_update_key(config: ResolvedConfig) -> tuple[object, ...]:
-        """Return a hashable tuple of the fields that affect the updater."""
-        return (
-            config.update_source,
-            config.update_channel,
-            config.auto_update_interval_minutes,
-            config.auto_apply,
-        )
-
     def _reinitialize_updater(self, config: ResolvedConfig) -> None:
         """Re-derive update settings and restart the updater and timer.
 
@@ -289,6 +277,15 @@ class UpdateController:
                 self._model.set_error('Updater is not initialized.')
             return
 
+        # A download may still be running in a thread-pool executor even
+        # after the asyncio Task wrapper has been cancelled.  Starting a
+        # new check would trigger a second concurrent download for the
+        # same package, leading to a file-rename race on Windows.
+        if self._client.updater.state == UpdateState.DOWNLOADING:
+            logger.debug('Skipping update check — download already in progress')
+            self._model.set_check_button_enabled(True)
+            return
+
         # Always disable the button; only show "Checking…" when no
         # download is already pending (to preserve the ready state).
         self._model.set_check_button_enabled(False)

{synodic_client-0.0.1.dev84 → synodic_client-0.0.1.dev86}/synodic_client/config.py

@@ -122,11 +122,16 @@ def load_user_config() -> UserConfig:
 
     try:
         data = json.loads(path.read_text(encoding='utf-8'))
+    except json.JSONDecodeError, OSError:
+        logger.exception('Failed to read config from %s, using defaults', path)
+        return UserConfig()
+
+    try:
         config = UserConfig.model_validate(data)
         logger.debug('Loaded user config from %s', path)
         return config
     except Exception:
-        logger.exception('Failed to load user config from %s, using defaults', path)
+        logger.exception('Failed to validate user config from %s, using defaults', path)
         return UserConfig()
 
 

{synodic_client-0.0.1.dev84 → synodic_client-0.0.1.dev86}/synodic_client/resolution.py

@@ -25,9 +25,7 @@ from synodic_client.config import (
 from synodic_client.schema import (
     DEFAULT_AUTO_UPDATE_INTERVAL_MINUTES,
     DEFAULT_TOOL_UPDATE_INTERVAL_MINUTES,
-    GITHUB_REPO_URL,
     ResolvedConfig,
-    UpdateChannel,
     UpdateConfig,
     UserConfig,
 )
@@ -160,7 +158,11 @@ def update_user_config(**changes: object) -> ResolvedConfig:
 
 
 def resolve_update_config(config: ResolvedConfig) -> UpdateConfig:
-    """Derive an ``UpdateConfig`` from resolved configuration values.
+    """Derive an :class:`UpdateConfig` from resolved configuration values.
+
+    Delegates to :meth:`UpdateConfig.from_resolved` so the type owns
+    its own construction while this module stays the canonical entry
+    point for all resolution logic.
 
     Args:
         config: A resolved configuration snapshot.
@@ -168,11 +170,4 @@ def resolve_update_config(config: ResolvedConfig) -> UpdateConfig:
     Returns:
         An ``UpdateConfig`` ready to initialise the updater.
     """
-    channel = UpdateChannel.DEVELOPMENT if config.update_channel == 'dev' else UpdateChannel.STABLE
-
-    return UpdateConfig(
-        channel=channel,
-        repo_url=config.update_source or GITHUB_REPO_URL,
-        auto_update_interval_minutes=config.auto_update_interval_minutes,
-        tool_update_interval_minutes=config.tool_update_interval_minutes,
-    )
+    return UpdateConfig.from_resolved(config)

{synodic_client-0.0.1.dev84 → synodic_client-0.0.1.dev86}/synodic_client/schema.py

@@ -8,13 +8,16 @@ that every layer can import them without circular dependencies.
 
 from __future__ import annotations
 
+import logging
 import sys
 from dataclasses import dataclass, field
 from enum import Enum, StrEnum, auto
 from typing import Any
 
 from packaging.version import Version
-from pydantic import BaseModel
+from pydantic import BaseModel, ValidationError, model_validator
+
+logger = logging.getLogger(__name__)
 
 # ---------------------------------------------------------------------------
 # BuildConfig — read-only, lives next to the executable
@@ -109,6 +112,31 @@ class UserConfig(BaseModel):
     # tool updates have been recorded.
     last_tool_updates: dict[str, str] | None = None
 
+    @model_validator(mode='wrap')
+    @classmethod
+    def _recover_invalid_fields(cls, data: Any, handler: Any) -> UserConfig:
+        """Silently drop fields that fail validation instead of rejecting the entire config.
+
+        When an on-disk ``config.json`` contains a value whose type no longer
+        matches the schema (e.g. after a version upgrade renames or re-types a
+        field), the normal behaviour is to raise ``ValidationError`` and lose
+        *every* setting.  This wrap validator intercepts the error, removes
+        only the offending fields (so their defaults kick in), and retries.
+        """
+        try:
+            return handler(data)
+        except ValidationError as exc:
+            if not isinstance(data, dict):
+                raise
+
+            bad_fields = {str(e['loc'][0]) for e in exc.errors() if e.get('loc')}
+            cleaned = {k: v for k, v in data.items() if k not in bad_fields}
+
+            for name in bad_fields:
+                logger.warning('Discarding invalid config field %r (using default)', name)
+
+            return handler(cleaned)
+
 
 # ---------------------------------------------------------------------------
 # Update channel & state enums
@@ -201,6 +229,24 @@ class UpdateConfig:
     # Interval in minutes between tool update checks (0 = disabled)
     tool_update_interval_minutes: int = DEFAULT_TOOL_UPDATE_INTERVAL_MINUTES
 
+    @classmethod
+    def from_resolved(cls, config: ResolvedConfig) -> UpdateConfig:
+        """Derive an ``UpdateConfig`` from resolved configuration values.
+
+        Args:
+            config: A resolved configuration snapshot.
+
+        Returns:
+            An ``UpdateConfig`` ready to initialise the updater.
+        """
+        channel = UpdateChannel.DEVELOPMENT if config.update_channel == 'dev' else UpdateChannel.STABLE
+        return cls(
+            channel=channel,
+            repo_url=config.update_source or GITHUB_REPO_URL,
+            auto_update_interval_minutes=config.auto_update_interval_minutes,
+            tool_update_interval_minutes=config.tool_update_interval_minutes,
+        )
+
     @property
     def channel_name(self) -> str:
         """Get the channel name for Velopack.

{synodic_client-0.0.1.dev84 → synodic_client-0.0.1.dev86}/synodic_client/updater.py

@@ -208,7 +208,12 @@ class Updater:
                 # moved past it.  A periodic re-check that discovers the
                 # same release must not regress DOWNLOADED → UPDATE_AVAILABLE,
                 # which would cause apply_update_on_exit() to reject the update.
-                if self._state not in {UpdateState.DOWNLOADED, UpdateState.APPLYING, UpdateState.APPLIED}:
+                if self._state not in {
+                    UpdateState.DOWNLOADING,
+                    UpdateState.DOWNLOADED,
+                    UpdateState.APPLYING,
+                    UpdateState.APPLIED,
+                }:
                     self._state = UpdateState.UPDATE_AVAILABLE
                 logger.info('Update available: %s -> %s', self._current_version, latest)
             else:
@@ -379,20 +384,16 @@ class Updater:
         # Verify checksum — prefer SHA256, fall back to SHA1.
         if asset.SHA256:
             actual = sha256_hash.hexdigest()
-            if not actual.lower() == asset.SHA256.lower():
+            if actual.lower() != asset.SHA256.lower():
                 partial_file.unlink(missing_ok=True)
-                raise RuntimeError(
-                    f'SHA256 mismatch for {asset.FileName}: expected {asset.SHA256}, got {actual}'
-                )
+                raise RuntimeError(f'SHA256 mismatch for {asset.FileName}: expected {asset.SHA256}, got {actual}')
         elif asset.SHA1:
             actual_sha1 = hashlib.sha1(partial_file.read_bytes()).hexdigest()  # noqa: S324 — verifying known digest
-            if not actual_sha1.lower() == asset.SHA1.lower():
+            if actual_sha1.lower() != asset.SHA1.lower():
                 partial_file.unlink(missing_ok=True)
-                raise RuntimeError(
-                    f'SHA1 mismatch for {asset.FileName}: expected {asset.SHA1}, got {actual_sha1}'
-                )
+                raise RuntimeError(f'SHA1 mismatch for {asset.FileName}: expected {asset.SHA1}, got {actual_sha1}')
 
-        partial_file.rename(target_file)
+        partial_file.replace(target_file)
         logger.info('Direct download complete: %s', target_file)
 
     def download_update(self, progress_callback: Callable[[int], None] | None = None) -> bool:
@@ -545,7 +546,7 @@ class Updater:
             raise RuntimeError(f'Failed to create Velopack UpdateManager: {e}') from e
 
 
-def _on_before_uninstall(version: str) -> None:
+def on_before_uninstall(version: str) -> None:
     """Velopack hook: called before the app is uninstalled.
 
     Removes the ``synodic://`` URI protocol handler and auto-startup

{synodic_client-0.0.1.dev84 → synodic_client-0.0.1.dev86}/tests/unit/operations/test_install.py

@@ -61,6 +61,7 @@ def _make_action(
         action.package.constraint = constraint
     else:
         action.package = None
+    action.distro = None
     return action
 
 

{synodic_client-0.0.1.dev84 → synodic_client-0.0.1.dev86}/tests/unit/qt/conftest.py

@@ -97,6 +97,7 @@ def make_action(
     action.command = overrides.get('command')
     action.include_prereleases = overrides.get('include_prereleases', False)
     action.plugin_target = overrides.get('plugin_target')
+    action.distro = overrides.get('distro')
     return action
 
 

{synodic_client-0.0.1.dev84 → synodic_client-0.0.1.dev86}/tests/unit/qt/test_update_controller.py

@@ -4,6 +4,8 @@ from __future__ import annotations
 
 from unittest.mock import MagicMock, patch
 
+import pytest
+
 from synodic_client.application.screen.update_banner import UpdateBanner
 from synodic_client.application.theme import (
     UPDATE_STATUS_AVAILABLE_STYLE,
@@ -13,6 +15,7 @@ from synodic_client.application.theme import (
 from synodic_client.application.update_controller import UpdateController
 from synodic_client.application.update_model import UpdateModel
 from synodic_client.operations.schema import UpdateCheckResult
+from synodic_client.schema import UpdateState
 
 from .conftest import make_config_store, make_resolved_config
 
@@ -576,3 +579,73 @@ class TestApplyGuard:
         assert banner.state.name == 'HIDDEN'
         client.apply_update_on_exit.assert_not_called()
         app.quit.assert_not_called()
+
+
+# ---------------------------------------------------------------------------
+# DOWNLOADING guard in _do_check
+# ---------------------------------------------------------------------------
+
+
+class TestDoCheckDownloadingGuard:
+    """Verify _do_check skips checks when a download is in progress."""
+
+    @staticmethod
+    @pytest.mark.parametrize('silent', [False, True], ids=['manual', 'silent'])
+    def test_check_skipped_when_downloading(silent: bool) -> None:
+        """_do_check should return early without creating a task when DOWNLOADING."""
+        ctrl, _app, client, banner, model = _make_controller()
+        spy = ModelSpy(model)
+        client.updater.state = UpdateState.DOWNLOADING
+
+        with patch.object(ctrl, '_set_task') as mock_set_task:
+            ctrl._do_check(silent=silent)
+
+        mock_set_task.assert_not_called()
+        # Button should be re-enabled
+        assert True in spy.check_button_enabled
+
+    @staticmethod
+    def test_check_proceeds_when_not_downloading() -> None:
+        """_do_check should proceed normally when updater is not DOWNLOADING."""
+        ctrl, _app, client, banner, model = _make_controller()
+        client.updater.state = UpdateState.NO_UPDATE
+
+        with patch.object(ctrl, '_set_task') as mock_set_task:
+            ctrl._do_check(silent=False)
+
+        mock_set_task.assert_called_once()
+
+
+# ---------------------------------------------------------------------------
+# request_retry / request_apply
+# ---------------------------------------------------------------------------
+
+
+class TestRequestRetry:
+    """Verify request_retry clears the failed version and re-checks."""
+
+    @staticmethod
+    def test_clears_failed_version_and_checks() -> None:
+        """request_retry should clear _failed_version then trigger a check."""
+        ctrl, _app, _client, banner, model = _make_controller()
+        ctrl._failed_version = '2.0.0'
+
+        with patch.object(ctrl, 'check_now') as mock_check:
+            ctrl.request_retry()
+
+        assert ctrl._failed_version is None
+        mock_check.assert_called_once_with(silent=True)
+
+
+class TestRequestApply:
+    """Verify request_apply delegates to _apply_update."""
+
+    @staticmethod
+    def test_delegates_to_apply_update() -> None:
+        """request_apply should call _apply_update(silent=False)."""
+        ctrl, _app, _client, banner, model = _make_controller()
+
+        with patch.object(ctrl, '_apply_update') as mock_apply:
+            ctrl.request_apply()
+
+        mock_apply.assert_called_once_with(silent=False)

{synodic_client-0.0.1.dev84 → synodic_client-0.0.1.dev86}/tests/unit/qt/test_update_feedback.py

@@ -512,9 +512,9 @@ class TestCompositeSignalKeys:
         header.set_runtime('3.11')
         spy = MagicMock()
         header.auto_update_toggled.connect(spy)
-        # Find the Auto button and click it
+        # Find the auto-update toggle button (checkable ↺ button)
         for child in header.findChildren(QPushButton):
-            if child.text() == 'Auto':
+            if child.isCheckable():
                 child.click()
                 break
         spy.assert_called_once()

{synodic_client-0.0.1.dev84 → synodic_client-0.0.1.dev86}/tests/unit/test_config.py

@@ -10,6 +10,7 @@ from synodic_client.config import (
     BuildConfig,
     UserConfig,
     config_dir,
+    load_user_config,
     save_user_config,
     set_dev_mode,
 )
@@ -208,3 +209,101 @@ class TestSaveUserConfig:
         assert loaded.update_channel == 'dev'
         assert loaded.auto_start is False
         assert loaded.update_source is None
+
+
+class TestUserConfigRecovery:
+    """Tests for the wrap validator that recovers invalid fields."""
+
+    @staticmethod
+    def test_single_corrupt_field_preserves_others() -> None:
+        """One invalid field is discarded; all other fields survive."""
+        data = {
+            'update_channel': 'dev',
+            'auto_start': False,
+            'auto_update_interval_minutes': 'not_an_int',  # corrupt
+        }
+        config = UserConfig.model_validate(data)
+        assert config.update_channel == 'dev'
+        assert config.auto_start is False
+        assert config.auto_update_interval_minutes is None  # reset to default
+
+    @staticmethod
+    def test_multiple_corrupt_fields() -> None:
+        """Multiple invalid fields are discarded; valid fields survive."""
+        data = {
+            'update_channel': 'stable',
+            'auto_update_interval_minutes': 'bad',
+            'auto_apply': 'not_a_bool',
+            'debug_logging': 42,  # int coerces to bool in Pydantic — this is valid
+            'auto_start': False,
+        }
+        config = UserConfig.model_validate(data)
+        assert config.update_channel == 'stable'
+        assert config.auto_start is False
+        # The corrupt fields revert to defaults
+        assert config.auto_update_interval_minutes is None
+        assert config.auto_apply is None
+
+    @staticmethod
+    def test_all_fields_corrupt() -> None:
+        """When every known field is invalid, result is equivalent to UserConfig()."""
+        data = {
+            'update_source': 123,  # str field, int won't coerce
+            'update_channel': [],
+            'auto_update_interval_minutes': 'bad',
+            'tool_update_interval_minutes': 'bad',
+            'auto_apply': 'nope',
+            'auto_start': 'nope',
+            'debug_logging': 'nope',
+            'plugin_auto_update': 'bad',
+            'prerelease_packages': 42,
+            'last_client_update': [],
+            'last_tool_updates': 'bad',
+        }
+        config = UserConfig.model_validate(data)
+        assert config == UserConfig()
+
+    @staticmethod
+    def test_valid_data_unchanged() -> None:
+        """Fully valid data passes through the wrap validator without modification."""
+        data = {
+            'update_channel': 'dev',
+            'auto_start': True,
+            'auto_update_interval_minutes': 10,
+        }
+        expected_interval = 10
+        config = UserConfig.model_validate(data)
+        assert config.update_channel == 'dev'
+        assert config.auto_start is True
+        assert config.auto_update_interval_minutes == expected_interval
+
+
+class TestLoadUserConfigRecovery:
+    """Integration tests: load_user_config with corrupt files on disk."""
+
+    @staticmethod
+    def test_corrupt_field_recovered_from_disk(tmp_path: Path) -> None:
+        """A config file with one bad field loads with that field reset to default."""
+        config_data = {
+            'update_channel': 'dev',
+            'auto_start': False,
+            'auto_update_interval_minutes': 'not_a_number',
+        }
+        (tmp_path / 'config.json').write_text(json.dumps(config_data), encoding='utf-8')
+
+        with patch('synodic_client.config.config_dir', return_value=tmp_path):
+            config = load_user_config()
+
+        assert config.update_channel == 'dev'
+        assert config.auto_start is False
+        assert config.auto_update_interval_minutes is None
+
+    @staticmethod
+    def test_invalid_json_returns_defaults(tmp_path: Path) -> None:
+        """Completely invalid JSON returns a default UserConfig."""
+        (tmp_path / 'config.json').write_text('{{not valid json', encoding='utf-8')
+
+        with patch('synodic_client.config.config_dir', return_value=tmp_path):
+            config = load_user_config()
+
+        assert config == UserConfig()