synodic-client 0.0.1.dev29__tar.gz → 0.0.1.dev30__tar.gz

{synodic_client-0.0.1.dev29 → synodic_client-0.0.1.dev30}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: synodic_client
-Version: 0.0.1.dev29
+Version: 0.0.1.dev30
 Author-Email: Synodic Software <contact@synodic.software>
 License: LGPL-3.0-or-later
 Project-URL: homepage, https://github.com/synodic/synodic-client
@@ -8,9 +8,9 @@ Project-URL: repository, https://github.com/synodic/synodic-client
 Requires-Python: <3.15,>=3.14
 Requires-Dist: pyside6>=6.10.2
 Requires-Dist: packaging>=26.0
-Requires-Dist: porringer>=0.2.1.dev48
+Requires-Dist: porringer>=0.2.1.dev49
 Requires-Dist: qasync>=0.28.0
-Requires-Dist: velopack>=0.0.1369.dev7516
+Requires-Dist: velopack>=0.0.1442.dev64255
 Requires-Dist: typer>=0.24.1
 Description-Content-Type: text/markdown
 

{synodic_client-0.0.1.dev29 → synodic_client-0.0.1.dev30}/pyproject.toml

@@ -10,12 +10,12 @@ requires-python = ">=3.14, <3.15"
 dependencies = [
     "pyside6>=6.10.2",
     "packaging>=26.0",
-    "porringer>=0.2.1.dev48",
+    "porringer>=0.2.1.dev49",
     "qasync>=0.28.0",
-    "velopack>=0.0.1369.dev7516",
+    "velopack>=0.0.1442.dev64255",
     "typer>=0.24.1",
 ]
-version = "0.0.1.dev29"
+version = "0.0.1.dev30"
 
 [project.license]
 text = "LGPL-3.0-or-later"

synodic_client-0.0.1.dev30/synodic_client/_version.py

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

{synodic_client-0.0.1.dev29 → synodic_client-0.0.1.dev30}/synodic_client/application/qt.py

@@ -44,11 +44,16 @@ def _init_services(logger: logging.Logger) -> tuple[Client, API, GlobalConfigura
     update_config = resolve_update_config(config)
     client.initialize_updater(update_config)
 
+    cached_dirs = porringer.cache.list_directories()
+
     logger.info(
-        'Synodic Client v%s started (channel: %s, source: %s)',
+        'Synodic Client v%s started (channel: %s, source: %s, '
+        'config_fields_set: %s, cached_projects: %d)',
         client.version,
         update_config.channel.name,
         update_config.repo_url,
+        sorted(config.model_fields_set),
+        len(cached_dirs),
     )
 
     return client, porringer, config

{synodic_client-0.0.1.dev29 → synodic_client-0.0.1.dev30}/synodic_client/config.py

@@ -194,6 +194,12 @@ def _load_global_config() -> GlobalConfiguration:
 def save_config(config: GlobalConfiguration) -> None:
     """Save configuration to the global (system) config directory.
 
+    Only fields that have been explicitly set (either loaded from the
+    existing config file or changed at runtime) are written.  This
+    sparse serialisation ensures that build-time local-config values
+    do not leak into the user's global config and that future defaults
+    can take effect for fields the user has not customised.
+
     Args:
         config: The configuration to persist.
     """
@@ -202,7 +208,10 @@ def save_config(config: GlobalConfiguration) -> None:
     path = directory / _CONFIG_FILENAME
 
     try:
-        path.write_text(config.model_dump_json(indent=2), encoding='utf-8')
+        path.write_text(
+            config.model_dump_json(indent=2, exclude_unset=True),
+            encoding='utf-8',
+        )
         logger.info('Saved config to %s', path)
     except Exception:
         logger.exception('Failed to save config to %s', path)

{synodic_client-0.0.1.dev29 → synodic_client-0.0.1.dev30}/synodic_client/resolution.py

@@ -32,8 +32,13 @@ def merge_config(
 ) -> GlobalConfiguration:
     """Merge local overrides into a global configuration.
 
-    Fields explicitly set (not None) in the local config override the
-    corresponding global values.
+    Fields that the user has explicitly saved (present in the global
+    config file) take priority over local overrides.  Local config
+    fields only fill in values the user has **not** set.
+
+    The returned object preserves the global config's
+    ``model_fields_set`` so that :func:`save_config` can use
+    ``exclude_unset=True`` to write only user-changed fields.
 
     Args:
         global_config: The user-scoped global configuration.
@@ -45,12 +50,13 @@ def merge_config(
     if local_config is None:
         return global_config
 
+    user_set = global_config.model_fields_set
     merged = global_config.model_dump()
     for field_name, value in local_config.model_dump().items():
-        if value is not None:
+        if value is not None and field_name not in user_set:
             merged[field_name] = value
 
-    return GlobalConfiguration.model_validate(merged)
+    return GlobalConfiguration.model_construct(_fields_set=set(user_set), **merged)
 
 
 def resolve_config() -> GlobalConfiguration:

{synodic_client-0.0.1.dev29 → synodic_client-0.0.1.dev30}/tests/unit/test_config.py

@@ -165,6 +165,20 @@ class TestSaveConfig:
         assert data['update_source'] == '/my/releases'
         assert data['update_channel'] == 'stable'
 
+    @staticmethod
+    def test_sparse_serialization(tmp_path: Path) -> None:
+        """Verify save_config only writes user-set fields (exclude_unset)."""
+        config = GlobalConfiguration(update_channel='dev')
+
+        with patch('synodic_client.config.config_dir', return_value=tmp_path):
+            save_config(config)
+
+        data = json.loads((tmp_path / 'config.json').read_text(encoding='utf-8'))
+        # Only 'update_channel' should be in the file
+        assert data == {'update_channel': 'dev'}
+        assert 'update_source' not in data
+        assert 'auto_update_interval_minutes' not in data
+
     @staticmethod
     def test_creates_directory(tmp_path: Path) -> None:
         """Verify save_config creates the directory if missing."""
@@ -189,3 +203,19 @@ class TestSaveConfig:
 
         data = json.loads(config_path.read_text(encoding='utf-8'))
         assert data['update_source'] == 'http://new-source'
+
+    @staticmethod
+    def test_save_load_round_trip(tmp_path: Path) -> None:
+        """Verify saved config can be loaded back with correct fields_set."""
+        original = GlobalConfiguration(update_channel='dev', auto_start=False)
+
+        with patch('synodic_client.config.config_dir', return_value=tmp_path):
+            save_config(original)
+
+        data = json.loads((tmp_path / 'config.json').read_text(encoding='utf-8'))
+        loaded = GlobalConfiguration.model_validate(data)
+        assert loaded.update_channel == 'dev'
+        assert loaded.auto_start is False
+        # Only saved fields should be in model_fields_set
+        assert loaded.model_fields_set == {'update_channel', 'auto_start'}
+        assert loaded.update_source is None

{synodic_client-0.0.1.dev29 → synodic_client-0.0.1.dev30}/tests/unit/test_resolution.py

@@ -34,8 +34,8 @@ class TestMergeConfig:
 
     @staticmethod
     def test_local_overrides_global() -> None:
-        """Verify local fields override global values."""
-        global_cfg = GlobalConfiguration(update_source='/system', update_channel='stable')
+        """Verify local fields fill in values the user hasn't set."""
+        global_cfg = GlobalConfiguration(update_channel='stable')
         local_cfg = LocalConfiguration(update_source='/local')
         result = merge_config(global_cfg, local_cfg)
         assert result.update_source == '/local'
@@ -52,21 +52,57 @@ class TestMergeConfig:
 
     @staticmethod
     def test_full_override() -> None:
-        """Verify all local fields override when set."""
+        """Verify local fields are ignored when the user has saved both."""
         global_cfg = GlobalConfiguration(update_source='/system', update_channel='stable')
         local_cfg = LocalConfiguration(update_source='/local', update_channel='dev')
         result = merge_config(global_cfg, local_cfg)
-        assert result.update_source == '/local'
+        assert result.update_source == '/system'
+        assert result.update_channel == 'stable'
+
+    @staticmethod
+    def test_user_saved_wins_over_local() -> None:
+        """Verify a user-saved field takes priority over local config."""
+        global_cfg = GlobalConfiguration(update_channel='dev')
+        local_cfg = LocalConfiguration(update_channel='stable')
+        result = merge_config(global_cfg, local_cfg)
+        assert result.update_channel == 'dev'
+
+    @staticmethod
+    def test_local_fills_unsaved_fields() -> None:
+        """Verify local config fills in fields the user hasn't saved."""
+        global_cfg = GlobalConfiguration(update_channel='dev')
+        local_cfg = LocalConfiguration(update_source='/local/releases')
+        result = merge_config(global_cfg, local_cfg)
         assert result.update_channel == 'dev'
+        assert result.update_source == '/local/releases'
+
+    @staticmethod
+    def test_preserves_model_fields_set() -> None:
+        """Verify merge preserves the global config's model_fields_set."""
+        global_cfg = GlobalConfiguration(update_channel='dev')
+        local_cfg = LocalConfiguration(update_source='/local')
+        result = merge_config(global_cfg, local_cfg)
+        # Only 'update_channel' was in the user's config file
+        assert result.model_fields_set == {'update_channel'}
+        # But the runtime value from local config is available
+        assert result.update_source == '/local'
 
     @staticmethod
     def test_local_overrides_plugin_auto_update() -> None:
-        """Verify local plugin_auto_update overrides global."""
-        global_cfg = GlobalConfiguration(plugin_auto_update={'pip': False})
+        """Verify local plugin_auto_update fills in when user hasn't set it."""
+        global_cfg = GlobalConfiguration()
         local_cfg = LocalConfiguration(plugin_auto_update={'pip': True, 'pipx': False})
         result = merge_config(global_cfg, local_cfg)
         assert result.plugin_auto_update == {'pip': True, 'pipx': False}
 
+    @staticmethod
+    def test_user_plugin_auto_update_wins() -> None:
+        """Verify user-saved plugin_auto_update wins over local."""
+        global_cfg = GlobalConfiguration(plugin_auto_update={'pip': False})
+        local_cfg = LocalConfiguration(plugin_auto_update={'pip': True, 'pipx': False})
+        result = merge_config(global_cfg, local_cfg)
+        assert result.plugin_auto_update == {'pip': False}
+
 
 class TestResolveAutoStart:
     """Tests for resolve_auto_start."""
@@ -169,7 +205,7 @@ class TestResolveConfig:
 
     @staticmethod
     def test_local_overrides_global_per_field(tmp_path: Path) -> None:
-        """Verify local config overrides global on a per-field basis."""
+        """Verify local config fills in fields the user hasn't saved."""
         local_data = {'update_source': '/local/releases'}
         local_path = tmp_path / 'local' / 'config.json'
         local_path.parent.mkdir()
@@ -177,7 +213,8 @@ class TestResolveConfig:
 
         system_dir = tmp_path / 'system'
         system_dir.mkdir()
-        system_data = {'update_source': '/system/releases', 'update_channel': 'stable'}
+        # User has only saved update_channel, not update_source
+        system_data = {'update_channel': 'stable'}
         (system_dir / 'config.json').write_text(json.dumps(system_data), encoding='utf-8')
 
         with (
@@ -186,7 +223,9 @@ class TestResolveConfig:
         ):
             config = resolve_config()
 
+        # Local fills in update_source since user didn't set it
         assert config.update_source == '/local/releases'
+        # User's saved update_channel is preserved
         assert config.update_channel == 'stable'
 
     @staticmethod
@@ -226,15 +265,15 @@ class TestResolveConfig:
 
     @staticmethod
     def test_portable_takes_precedence(tmp_path: Path) -> None:
-        """Verify portable config values override system config."""
+        """Verify portable config fills in fields user hasn't saved."""
         portable_data = {'update_source': '/portable/releases', 'update_channel': 'dev'}
         portable_path = tmp_path / 'config.json'
         portable_path.write_text(json.dumps(portable_data), encoding='utf-8')
 
         system_dir = tmp_path / 'system'
         system_dir.mkdir()
-        system_data = {'update_source': '/system/releases', 'update_channel': 'stable'}
-        (system_dir / 'config.json').write_text(json.dumps(system_data), encoding='utf-8')
+        # User has NOT saved any config (empty file or missing)
+        (system_dir / 'config.json').write_text('{}', encoding='utf-8')
 
         with (
             patch('synodic_client.config._portable_config_path', return_value=portable_path),
@@ -245,6 +284,27 @@ class TestResolveConfig:
         assert config.update_source == '/portable/releases'
         assert config.update_channel == 'dev'
 
+    @staticmethod
+    def test_user_saved_wins_over_portable(tmp_path: Path) -> None:
+        """Verify user-saved values in global config win over portable."""
+        portable_data = {'update_source': '/portable/releases', 'update_channel': 'dev'}
+        portable_path = tmp_path / 'config.json'
+        portable_path.write_text(json.dumps(portable_data), encoding='utf-8')
+
+        system_dir = tmp_path / 'system'
+        system_dir.mkdir()
+        system_data = {'update_source': '/system/releases', 'update_channel': 'stable'}
+        (system_dir / 'config.json').write_text(json.dumps(system_data), encoding='utf-8')
+
+        with (
+            patch('synodic_client.config._portable_config_path', return_value=portable_path),
+            patch('synodic_client.config.config_dir', return_value=system_dir),
+        ):
+            config = resolve_config()
+
+        assert config.update_source == '/system/releases'
+        assert config.update_channel == 'stable'
+
 
 class TestResolveUpdateConfig:
     """Tests for resolve_update_config."""
@@ -339,7 +399,9 @@ class TestUpdateAndResolve:
         assert result.channel == UpdateChannel.DEVELOPMENT
         assert result.repo_url == '/my/source'
 
-        # Verify file was saved
+        # Verify file was saved (sparse — only user-set fields)
         saved = json.loads((tmp_path / 'config.json').read_text(encoding='utf-8'))
         assert saved['update_source'] == '/my/source'
         assert saved['update_channel'] == 'dev'
+        # Unset fields should not appear in the sparse output
+        assert 'auto_update_interval_minutes' not in saved

{synodic_client-0.0.1.dev29 → synodic_client-0.0.1.dev30}/tests/unit/windows/test_startup.py

@@ -3,8 +3,6 @@
 import winreg
 from unittest.mock import MagicMock, patch
 
-import pytest
-
 from synodic_client.startup import (
     RUN_KEY_PATH,
     STARTUP_VALUE_NAME,
@@ -13,9 +11,6 @@ from synodic_client.startup import (
     remove_startup,
 )
 
-_TEST_VALUE_NAME = f'{STARTUP_VALUE_NAME}_test'
-"""Temporary value name used by integration tests to avoid clobbering the real registration."""
-
 
 class TestRegisterStartup:
     """Tests for register_startup."""
@@ -123,70 +118,4 @@ class TestIsStartupRegistered:
             patch.object(winreg, 'QueryValueEx', side_effect=FileNotFoundError),
         ):
             assert is_startup_registered() is False
-
-
-class TestStartupIntegration:
-    """Integration tests that read/write real registry values under a test name."""
-
-    @staticmethod
-    def test_register_creates_valid_entry() -> None:
-        """Register under a test value name, verify, then clean up."""
-        test_exe = r'C:\test\synodic_test.exe'
-
-        try:
-            with patch('synodic_client.startup.STARTUP_VALUE_NAME', _TEST_VALUE_NAME):
-                register_startup(test_exe)
-
-            with winreg.OpenKey(winreg.HKEY_CURRENT_USER, RUN_KEY_PATH, 0, winreg.KEY_QUERY_VALUE) as key:
-                value, reg_type = winreg.QueryValueEx(key, _TEST_VALUE_NAME)
-                assert reg_type == winreg.REG_SZ
-                assert test_exe in value
-
-        finally:
-            with patch('synodic_client.startup.STARTUP_VALUE_NAME', _TEST_VALUE_NAME):
-                remove_startup()
-
-    @staticmethod
-    def test_remove_deletes_entry() -> None:
-        """Register then remove under a test value name, verify it is gone."""
-        with patch('synodic_client.startup.STARTUP_VALUE_NAME', _TEST_VALUE_NAME):
-            register_startup(r'C:\test\synodic_test.exe')
-            remove_startup()
-
-        with (
-            winreg.OpenKey(winreg.HKEY_CURRENT_USER, RUN_KEY_PATH, 0, winreg.KEY_QUERY_VALUE) as key,
-            pytest.raises(FileNotFoundError),
-        ):
-            winreg.QueryValueEx(key, _TEST_VALUE_NAME)
-
-    @staticmethod
-    def test_register_is_idempotent() -> None:
-        """Calling register twice with a different exe updates the value."""
-        exe_v1 = r'C:\test\v1\synodic.exe'
-        exe_v2 = r'C:\test\v2\synodic.exe'
-
-        try:
-            with patch('synodic_client.startup.STARTUP_VALUE_NAME', _TEST_VALUE_NAME):
-                register_startup(exe_v1)
-                register_startup(exe_v2)
-
-            with winreg.OpenKey(winreg.HKEY_CURRENT_USER, RUN_KEY_PATH, 0, winreg.KEY_QUERY_VALUE) as key:
-                value, _ = winreg.QueryValueEx(key, _TEST_VALUE_NAME)
-                assert exe_v2 in value
-                assert exe_v1 not in value
-
-        finally:
-            with patch('synodic_client.startup.STARTUP_VALUE_NAME', _TEST_VALUE_NAME):
-                remove_startup()
-
-    @staticmethod
-    def test_is_startup_registered_reflects_state() -> None:
-        """Verify is_startup_registered returns the correct state."""
-        with patch('synodic_client.startup.STARTUP_VALUE_NAME', _TEST_VALUE_NAME):
-            assert is_startup_registered() is False
-
-            register_startup(r'C:\test\synodic_test.exe')
-            assert is_startup_registered() is True
-
-            remove_startup()
             assert is_startup_registered() is False

synodic_client-0.0.1.dev29/synodic_client/_version.py

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