synodic-client 0.0.1.dev64__tar.gz → 0.0.1.dev66__tar.gz

{synodic_client-0.0.1.dev64 → synodic_client-0.0.1.dev66}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: synodic_client
-Version: 0.0.1.dev64
+Version: 0.0.1.dev66
 Author-Email: Synodic Software <contact@synodic.software>
 License: LGPL-3.0-or-later
 Project-URL: homepage, https://github.com/synodic/synodic-client
@@ -8,7 +8,7 @@ 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.dev77
+Requires-Dist: porringer>=0.2.1.dev78
 Requires-Dist: qasync>=0.28.0
 Requires-Dist: velopack>=0.0.1444.dev49733
 Requires-Dist: typer>=0.24.1

{synodic_client-0.0.1.dev64 → synodic_client-0.0.1.dev66}/pyproject.toml

@@ -10,12 +10,12 @@ requires-python = ">=3.14, <3.15"
 dependencies = [
     "pyside6>=6.10.2",
     "packaging>=26.0",
-    "porringer>=0.2.1.dev77",
+    "porringer>=0.2.1.dev78",
     "qasync>=0.28.0",
     "velopack>=0.0.1444.dev49733",
     "typer>=0.24.1",
 ]
-version = "0.0.1.dev64"
+version = "0.0.1.dev66"
 
 [project.license]
 text = "LGPL-3.0-or-later"

{synodic_client-0.0.1.dev64 → synodic_client-0.0.1.dev66}/synodic_client/application/config_store.py

@@ -24,7 +24,7 @@ class ConfigStore(QObject):
 
         store = ConfigStore(initial_config)
         store.changed.connect(some_consumer.on_config_changed)
-        store.update(auto_apply=False)   # persists + emits
+        store.update(auto_apply=False)  # persists + emits
     """
 
     changed = Signal(object)

synodic_client-0.0.1.dev66/synodic_client/application/package_state.py

@@ -0,0 +1,125 @@
+"""Shared package update state registry.
+
+Provides :class:`PackageStateStore`, a centralised record of
+per-package update status used by both ToolsView and ProjectsView
+so that version/update information discovered in one view is
+immediately available to the other.
+"""
+
+from __future__ import annotations
+
+from dataclasses import dataclass
+
+from PySide6.QtCore import QObject, Signal
+
+
+@dataclass(slots=True)
+class PackageState:
+    """Canonical update state for a single package.
+
+    Keyed by ``(signal_key, name)`` inside :class:`PackageStateStore`.
+    """
+
+    name: str
+    """Package name (e.g. ``"ruff"``)."""
+
+    installed_version: str = ''
+    """Currently installed version, or empty if unknown."""
+
+    available_version: str = ''
+    """Latest available version, or empty if unknown."""
+
+    has_update: bool = False
+    """Whether the package has a newer version available."""
+
+
+class PackageStateStore(QObject):
+    """Shared registry of package update states across views.
+
+    Both ToolsView and ProjectsView write discovered update information
+    into the store; each view can then read the canonical state
+    regardless of which view made the discovery.
+
+    :attr:`state_changed` is emitted whenever new data arrives so
+    listeners can refresh their badges/labels without polling.
+    """
+
+    state_changed = Signal()
+    """Emitted whenever any package state is created or modified."""
+
+    def __init__(self, parent: QObject | None = None) -> None:
+        """Initialise with an empty registry."""
+        super().__init__(parent)
+        self._data: dict[str, dict[str, PackageState]] = {}
+
+    # -- Bulk write (ToolsView check_updates) ----------------------------
+
+    def set_check_results(self, available: dict[str, dict[str, str]]) -> None:
+        """Populate from ``check_updates`` results.
+
+        *available* maps ``{signal_key: {package_name: latest_version}}``
+        — the same shape previously stored in
+        ``ToolsView._updates_available``.
+        """
+        self._data.clear()
+        for key, packages in available.items():
+            for pkg_name, latest in packages.items():
+                self._data.setdefault(key, {})[pkg_name] = PackageState(
+                    name=pkg_name,
+                    available_version=latest,
+                    has_update=True,
+                )
+        self.state_changed.emit()
+
+    # -- Single-action write (ProjectsView dry-run) ----------------------
+
+    def record_action_result(
+        self,
+        signal_key: str,
+        pkg_name: str,
+        *,
+        installed_version: str = '',
+        available_version: str = '',
+        has_update: bool = False,
+    ) -> None:
+        """Record a single dry-run result.
+
+        Merges with any existing state so that data discovered by
+        different views accumulates rather than overwrites.
+        """
+        existing = self._data.get(signal_key, {}).get(pkg_name)
+        state = PackageState(
+            name=pkg_name,
+            installed_version=installed_version or (existing.installed_version if existing else ''),
+            available_version=available_version or (existing.available_version if existing else ''),
+            has_update=has_update or (existing.has_update if existing else False),
+        )
+        self._data.setdefault(signal_key, {})[pkg_name] = state
+        self.state_changed.emit()
+
+    # -- Read API --------------------------------------------------------
+
+    def get_updates(self, signal_key: str) -> dict[str, str]:
+        """Return ``{package_name: latest_version}`` for packages with updates.
+
+        Drop-in replacement for ``_updates_available.get(key, {})``.
+        """
+        bucket = self._data.get(signal_key, {})
+        return {name: s.available_version for name, s in bucket.items() if s.has_update}
+
+    def has_updates_for(self, signal_key: str) -> bool:
+        """Return whether any package under *signal_key* has an update."""
+        return any(s.has_update for s in self._data.get(signal_key, {}).values())
+
+    def get(self, signal_key: str, pkg_name: str) -> PackageState | None:
+        """Return the state for a specific package, or ``None``."""
+        return self._data.get(signal_key, {}).get(pkg_name)
+
+    @property
+    def has_data(self) -> bool:
+        """Return whether any update data has been recorded."""
+        return bool(self._data)
+
+    def clear(self) -> None:
+        """Remove all recorded state."""
+        self._data.clear()

{synodic_client-0.0.1.dev64 → synodic_client-0.0.1.dev66}/synodic_client/application/screen/install.py

@@ -43,6 +43,7 @@ from PySide6.QtWidgets import (
     QWidget,
 )
 
+from synodic_client.application.package_state import PackageStateStore
 from synodic_client.application.screen import skip_reason_label
 from synodic_client.application.screen.action_card import ActionCardList, action_key
 from synodic_client.application.screen.card import CardFrame
@@ -113,6 +114,7 @@ class SetupPreviewWidget(QWidget):
         *,
         show_close: bool = True,
         config: ResolvedConfig | None = None,
+        package_store: PackageStateStore | None = None,
     ) -> None:
         """Initialize the preview widget.
 
@@ -122,11 +124,13 @@ class SetupPreviewWidget(QWidget):
             show_close: Whether to show the Close button.  Set ``False``
                 when embedding inside a persistent view (e.g. a tab).
             config: Global configuration for per-manifest pre-release state.
+            package_store: Shared package update state registry.
         """
         super().__init__(parent)
         self._porringer = porringer
         self._show_close = show_close
         self._config = config
+        self._package_store = package_store
         self._discovered_plugins: DiscoveredPlugins | None = None
 
         self._model = PreviewModel()
@@ -293,7 +297,6 @@ class SetupPreviewWidget(QWidget):
         path_or_url: str,
         *,
         project_directory: Path | None = None,
-        detect_updates: bool = True,
     ) -> None:
         """Load a manifest preview, or skip if the same manifest is already showing results.
 
@@ -305,7 +308,6 @@ class SetupPreviewWidget(QWidget):
         Args:
             path_or_url: Manifest path or URL.
             project_directory: Working directory for project sync actions.
-            detect_updates: Query package indices for newer versions.
         """
         key = normalize_manifest_key(path_or_url)
 
@@ -345,7 +347,6 @@ class SetupPreviewWidget(QWidget):
             self._run_preview_task(
                 path_or_url,
                 project_directory=self._model.project_directory,
-                detect_updates=detect_updates,
                 prerelease_packages=overrides,
             ),
         )
@@ -474,7 +475,6 @@ class SetupPreviewWidget(QWidget):
         path_or_url: str,
         *,
         project_directory: Path | None = None,
-        detect_updates: bool = True,
         prerelease_packages: set[str] | None = None,
     ) -> None:
         """Run the preview coroutine and route completion/errors."""
@@ -484,7 +484,6 @@ class SetupPreviewWidget(QWidget):
                 path_or_url,
                 config=PreviewConfig(
                     project_directory=project_directory,
-                    detect_updates=detect_updates,
                     prerelease_packages=prerelease_packages,
                 ),
                 callbacks=PreviewCallbacks(
@@ -640,6 +639,16 @@ class SetupPreviewWidget(QWidget):
             if card is not None:
                 card.set_check_result(result)
 
+            # Record in shared store so ToolsView can reflect the update
+            if self._package_store is not None and action.installer and action.package:
+                self._package_store.record_action_result(
+                    action.installer,
+                    str(action.package.name),
+                    installed_version=result.installed_version or '',
+                    available_version=result.available_version or '',
+                    has_update=result.skip_reason == SkipReason.UPDATE_AVAILABLE,
+                )
+
         # Update phase text
         m.checked_count += 1
         total = len(m.action_states)
@@ -987,11 +996,9 @@ class InstallPreviewWindow(QMainWindow):
         logger.info('Starting install preview for: %s', self._manifest_url)
         self._url_label.setText(f'<b>Manifest:</b> {self._manifest_url}')
 
-        detect = self._config.detect_updates if self._config else True
         self._preview_widget.load(
             self._manifest_url,
             project_directory=self._project_directory,
-            detect_updates=detect,
         )
 
     # --- Callbacks ---

{synodic_client-0.0.1.dev64 → synodic_client-0.0.1.dev66}/synodic_client/application/screen/install_workers.py

@@ -232,7 +232,6 @@ async def run_preview(
             paths=[manifest_path],
             dry_run=True,
             project_directory=cfg.project_directory,
-            detect_updates=cfg.detect_updates,
             prerelease_packages=cfg.prerelease_packages,
         )
         state = _DispatchState()

{synodic_client-0.0.1.dev64 → synodic_client-0.0.1.dev66}/synodic_client/application/screen/projects.py

@@ -20,6 +20,7 @@ from PySide6.QtWidgets import (
 )
 
 from synodic_client.application.data import DataCoordinator
+from synodic_client.application.package_state import PackageStateStore
 from synodic_client.application.screen.install import SetupPreviewWidget
 from synodic_client.application.screen.schema import PreviewPhase
 from synodic_client.application.screen.sidebar import ManifestSidebar
@@ -48,6 +49,7 @@ class ProjectsView(QWidget):
         parent: QWidget | None = None,
         *,
         coordinator: DataCoordinator | None = None,
+        package_store: PackageStateStore | None = None,
     ) -> None:
         """Initialize the projects view.
 
@@ -57,11 +59,13 @@ class ProjectsView(QWidget):
             parent: Optional parent widget.
             coordinator: Shared data coordinator for validated directory
                 data.
+            package_store: Shared package update state registry.
         """
         super().__init__(parent)
         self._porringer = porringer
         self._store = store
         self._coordinator = coordinator
+        self._package_store = package_store
         self._refresh_in_progress = False
         self._pending_select: Path | None = None
         self._widgets: dict[Path, SetupPreviewWidget] = {}
@@ -166,7 +170,6 @@ class ProjectsView(QWidget):
                     widget.load(
                         str(path),
                         project_directory=path if path.is_dir() else path.parent,
-                        detect_updates=self._store.config.detect_updates,
                     )
 
         except Exception:
@@ -200,6 +203,7 @@ class ProjectsView(QWidget):
                     self,
                     show_close=False,
                     config=self._store.config,
+                    package_store=self._package_store,
                 )
                 widget._discovered_plugins = discovered
                 widget.install_finished.connect(self._on_install_finished)

{synodic_client-0.0.1.dev64 → synodic_client-0.0.1.dev66}/synodic_client/application/screen/schema.py

@@ -11,9 +11,7 @@ from __future__ import annotations
 import enum
 from collections.abc import Callable
 from dataclasses import dataclass, field
-from enum import Enum, auto
 from pathlib import Path
-from typing import Protocol, runtime_checkable
 
 from porringer.schema import (
     PluginInfo,
@@ -334,7 +332,6 @@ class PreviewConfig:
     """Optional execution parameters for :func:`run_preview`."""
 
     project_directory: Path | None = None
-    detect_updates: bool = True
     prerelease_packages: set[str] | None = None
 
 
@@ -344,62 +341,3 @@ class _DispatchState:
 
     action_index: dict[int, int] = field(default_factory=dict)
     got_parsed: bool = False
-
-
-# ---------------------------------------------------------------------------
-# Update view protocol & banner data models
-# ---------------------------------------------------------------------------
-
-
-@runtime_checkable
-class UpdateView(Protocol):
-    """Minimal display contract for the self-update lifecycle.
-
-    :class:`UpdateBanner` satisfies this protocol implicitly via
-    structural typing.  The controller broadcasts state transitions
-    through a ``list[UpdateView]`` so that every window showing update
-    status stays in sync.
-    """
-
-    def show_downloading(self, version: str) -> None:
-        """Indicate that *version* is being downloaded."""
-        ...
-
-    def show_downloading_progress(self, percentage: int) -> None:
-        """Update the download progress indicator."""
-        ...
-
-    def show_ready(self, version: str) -> None:
-        """Indicate that *version* is downloaded and ready to install."""
-        ...
-
-    def show_error(self, message: str) -> None:
-        """Display an error *message* in the update area."""
-        ...
-
-    def hide_banner(self) -> None:
-        """Hide the update banner."""
-        ...
-
-
-class UpdateBannerState(Enum):
-    """Visual states for the update banner."""
-
-    HIDDEN = auto()
-    DOWNLOADING = auto()
-    READY = auto()
-    ERROR = auto()
-
-
-@dataclass(frozen=True, slots=True)
-class _BannerConfig:
-    """Bundled visual configuration for a banner state transition."""
-
-    state: UpdateBannerState
-    style: str
-    icon: str
-    text: str
-    text_style: str
-    version: str = ''
-    action_label: str = ''
-    show_progress: bool = False

{synodic_client-0.0.1.dev64 → synodic_client-0.0.1.dev66}/synodic_client/application/screen/screen.py

@@ -37,6 +37,7 @@ from PySide6.QtWidgets import (
 from synodic_client.application.config_store import ConfigStore
 from synodic_client.application.data import DataCoordinator
 from synodic_client.application.icon import app_icon
+from synodic_client.application.package_state import PackageStateStore
 from synodic_client.application.screen.plugin_row import (
     FilterChip,
     PluginKindHeader,
@@ -115,6 +116,7 @@ class ToolsView(QWidget):
         parent: QWidget | None = None,
         *,
         coordinator: DataCoordinator | None = None,
+        package_store: PackageStateStore | None = None,
     ) -> None:
         """Initialize the tools view.
 
@@ -125,22 +127,25 @@ class ToolsView(QWidget):
             coordinator: Shared data coordinator.  When provided, the
                 view delegates plugin/directory fetching to the
                 coordinator instead of calling porringer directly.
+            package_store: Shared package update state registry.
         """
         super().__init__(parent)
         self._porringer = porringer
         self._store = store
         self._coordinator = coordinator
+        self._package_store = package_store
         self._section_widgets: list[QWidget] = []
         self._filter_chips: dict[str, FilterChip] = {}
         self._deselected_plugins: set[str] = set()
         self._refresh_in_progress = False
         self._check_in_progress = False
-        self._updates_checked = False
-        self._updates_available: dict[str, dict[str, str]] = {}
         self._directories: list[ManifestDirectory] = []
         self._timestamp_timer: QTimer | None = None
         self._init_ui()
 
+        if self._package_store is not None:
+            self._package_store.state_changed.connect(self._on_package_state_changed)
+
     def _init_ui(self) -> None:
         """Initialize the UI components."""
         outer = QVBoxLayout(self)
@@ -234,6 +239,11 @@ class ToolsView(QWidget):
 
     # --- Public API ---
 
+    def invalidate_update_data(self) -> None:
+        """Clear cached update state so the next refresh re-checks."""
+        if self._package_store is not None:
+            self._package_store.clear()
+
     def refresh(self) -> None:
         """Schedule an asynchronous rebuild of the tool list."""
         if self._refresh_in_progress:
@@ -257,7 +267,7 @@ class ToolsView(QWidget):
 
         try:
             data = await self._gather_refresh_data()
-            need_deferred_check = not self._updates_checked
+            need_deferred_check = not self._has_update_data
             self._build_widget_tree(data)
         except Exception:
             logger.exception('Failed to refresh tools')
@@ -272,7 +282,7 @@ class ToolsView(QWidget):
         # Fire-and-forget: detect updates in the background, then patch
         # the just-rendered widget tree with update badges.
         if need_deferred_check:
-            asyncio.create_task(self._deferred_update_check(self._directories))
+            asyncio.create_task(self._deferred_update_check())
 
     # ------------------------------------------------------------------
     # _async_refresh helper methods
@@ -445,7 +455,6 @@ class ToolsView(QWidget):
             return
 
         auto_val = auto_update_map.get(plugin.name, True)
-        plugin_updates = self._updates_available.get(plugin.name, {})
         tool_timestamps = self._store.config.last_tool_updates or {}
         default_exe = data.default_runtime_executable
 
@@ -462,11 +471,14 @@ class ToolsView(QWidget):
             if is_default:
                 tag_text += ' (default)'
 
+            # Runtime updates use composite keys "plugin:tag"
+            rt_updates = self._get_plugin_updates(f'{plugin.name}:{rt.tag}')
+
             provider = PluginProviderHeader(
                 plugin,
                 auto_val is not False,
                 show_controls=True,
-                has_updates=bool(plugin_updates),
+                has_updates=bool(rt_updates),
                 parent=self._container,
             )
             provider.set_runtime(rt.tag, label=tag_text)
@@ -496,7 +508,7 @@ class ToolsView(QWidget):
                         plugin_name=plugin.name,
                         auto_update=pkg_auto,
                         show_toggle=True,
-                        has_update=pkg.name in plugin_updates,
+                        has_update=pkg.name in rt_updates,
                         is_global=True,
                         host_tool=pkg.host_tool,
                         runtime_tag=rt.tag,
@@ -518,7 +530,7 @@ class ToolsView(QWidget):
         ``ProjectChildRow`` widgets are no longer used.
         """
         auto_val = auto_update_map.get(plugin.name, True)
-        plugin_updates = self._updates_available.get(plugin.name, {})
+        plugin_updates = self._get_plugin_updates(plugin.name)
 
         provider = PluginProviderHeader(
             plugin,
@@ -1121,11 +1133,15 @@ class ToolsView(QWidget):
         asyncio.create_task(self._run_inline_update_check())
 
     async def _run_inline_update_check(self) -> None:
-        """Check for updates with inline spinners (no overlay / rebuild)."""
+        """Check for updates with inline spinners (no overlay / rebuild).
+
+        Used by both the manual *Check for Updates* button and the
+        automatic deferred check after initial refresh.
+        """
         self._check_in_progress = True
         try:
-            self._updates_available = await self._check_for_updates(self._directories)
-            self._updates_checked = True
+            available = await self._check_for_updates(self._directories)
+            self._store_check_results(available)
             self._apply_update_badges()
         except Exception:
             logger.debug('Inline update check failed', exc_info=True)
@@ -1225,7 +1241,6 @@ class ToolsView(QWidget):
             params = SetupParameters(
                 paths=[str(manifest_path)],
                 dry_run=True,
-                detect_updates=True,
                 project_directory=path,
             )
             async for event in self._porringer.sync.execute_stream(params):
@@ -1247,31 +1262,42 @@ class ToolsView(QWidget):
             )
         return available
 
-    async def _deferred_update_check(
-        self,
-        directories: list[ManifestDirectory],
-    ) -> None:
+    async def _deferred_update_check(self) -> None:
         """Run update detection in the background, then patch the widget tree.
 
         Called after the initial render so the user sees the tool list
         immediately while update badges are populated asynchronously.
-        Inline per-row spinners provide visual feedback.
+        Delegates to :meth:`_run_inline_update_check`.
         """
-        self._check_in_progress = True
         self._check_btn.setEnabled(False)
         self._check_btn.setText('Checking\u2026')
         self._set_all_checking(True)
-        try:
-            self._updates_available = await self._check_for_updates(directories)
-            self._updates_checked = True
+        await self._run_inline_update_check()
+
+    def _get_plugin_updates(self, signal_key: str) -> dict[str, str]:
+        """Return ``{package_name: latest_version}`` for *signal_key*.
+
+        Reads from the shared :class:`PackageStateStore` when available,
+        otherwise returns an empty dict.
+        """
+        if self._package_store is not None:
+            return self._package_store.get_updates(signal_key)
+        return {}
+
+    def _store_check_results(self, available: dict[str, dict[str, str]]) -> None:
+        """Push check results into the store."""
+        if self._package_store is not None:
+            self._package_store.set_check_results(available)
+
+    @property
+    def _has_update_data(self) -> bool:
+        """Return whether update data has been fetched at least once."""
+        return self._package_store is not None and self._package_store.has_data
+
+    def _on_package_state_changed(self) -> None:
+        """Re-apply badges when another view writes to the shared store."""
+        if not self._check_in_progress and not self._refresh_in_progress:
             self._apply_update_badges()
-        except Exception:
-            logger.debug('Deferred update check failed', exc_info=True)
-        finally:
-            self._set_all_checking(False)
-            self._check_btn.setEnabled(True)
-            self._check_btn.setText('Check for Updates')
-            self._check_in_progress = False
 
     def _apply_update_badges(self) -> None:
         """Walk existing widgets and show/hide Update buttons + set inline status."""
@@ -1279,12 +1305,12 @@ class ToolsView(QWidget):
         for widget in self._section_widgets:
             if isinstance(widget, PluginProviderHeader):
                 current_plugin = widget._signal_key
-                plugin_updates = self._updates_available.get(current_plugin, {})
+                plugin_updates = self._get_plugin_updates(current_plugin)
                 has = bool(plugin_updates)
                 if widget._update_btn is not None:
                     widget._update_btn.setVisible(has)
             elif isinstance(widget, PluginRow) and widget._plugin_name:
-                plugin_updates = self._updates_available.get(widget._signal_key, {})
+                plugin_updates = self._get_plugin_updates(widget._signal_key)
                 latest_version = plugin_updates.get(widget._package_name)
                 has_update = latest_version is not None
 
@@ -1295,7 +1321,7 @@ class ToolsView(QWidget):
                 if has_update:
                     version_text = f'v{latest_version} available' if latest_version else 'Update available'
                     widget.set_update_status(version_text, PLUGIN_ROW_STATUS_AVAILABLE_STYLE)
-                elif self._updates_checked:
+                elif self._has_update_data:
                     widget.set_update_status('Up to date', PLUGIN_ROW_STATUS_UP_TO_DATE_STYLE)
 
     def _set_all_checking(self, checking: bool) -> None:
@@ -1401,6 +1427,7 @@ class MainWindow(QMainWindow):
         self._porringer = porringer
         self._store = store
         self._coordinator: DataCoordinator | None = DataCoordinator(porringer) if porringer is not None else None
+        self._package_store: PackageStateStore | None = PackageStateStore(self) if porringer is not None else None
         self.setWindowTitle('Synodic Client')
         self.setMinimumSize(*MAIN_WINDOW_MIN_SIZE)
         self.setWindowIcon(app_icon())
@@ -1453,6 +1480,7 @@ class MainWindow(QMainWindow):
                 self._store,
                 self,
                 coordinator=self._coordinator,
+                package_store=self._package_store,
             )
             self._tabs.addTab(self._projects_view, 'Projects')
 
@@ -1461,6 +1489,7 @@ class MainWindow(QMainWindow):
                 self._store,
                 self,
                 coordinator=self._coordinator,
+                package_store=self._package_store,
             )
             self._tabs.addTab(self._tools_view, 'Tools')
             self.tools_view_created.emit(self._tools_view)