starbash 0.3.2__tar.gz → 0.3.4__tar.gz

{starbash-0.3.2 → starbash-0.3.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: starbash
-Version: 0.3.2
+Version: 0.3.4
 Summary: A tool for automating/standardizing/sharing astrophotography workflows.
 License-File: LICENSE
 Author: Kevin Hester
@@ -10,23 +10,22 @@ Classifier: Programming Language :: Python :: 3
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.13
 Classifier: Programming Language :: Python :: 3.14
-Requires-Dist: astropy (>=7.1.1,<8.0.0)
-Requires-Dist: click (>=8.0,<8.5)
-Requires-Dist: doit (>=0.36.0,<0.37.0)
+Requires-Dist: astropy (>=7.1.1)
+Requires-Dist: doit (>=0.36.0)
 Requires-Dist: graxpert[cpuonly] (>=3.2.0a2,<4.0.0)
 Requires-Dist: jinja2 (>=3.1,<4.0)
-Requires-Dist: keyring (>=25.6,<26.0)
+Requires-Dist: keyring (>=25.6)
 Requires-Dist: numpy (==2.2.6)
 Requires-Dist: platformdirs (>=4.5.0,<5.0.0)
 Requires-Dist: pygal (>=3.0,<4.0)
 Requires-Dist: pyqt6 (>=6.10.1,<7.0.0)
-Requires-Dist: restrictedpython (>=8.1,<9.0)
-Requires-Dist: rich (>=14.2.0,<15.0.0)
-Requires-Dist: sentry-sdk (>=2.42.1,<3.0.0)
-Requires-Dist: textual (>=6.8.0,<7.0.0)
-Requires-Dist: toml-repo (>=0.1.6,<0.2.0)
-Requires-Dist: typer (>=0.20.0,<0.21.0)
-Requires-Dist: update-checker (>=0.18.0,<0.19.0)
+Requires-Dist: pyside6 (>=6.11.2,<7.0.0)
+Requires-Dist: restrictedpython (>=8.1)
+Requires-Dist: rich (>=14.2.0)
+Requires-Dist: sentry-sdk (>=2.42.1)
+Requires-Dist: toml-repo (>=0.1.7,<0.2.0)
+Requires-Dist: typer (>=0.24.2)
+Requires-Dist: update-checker (>=1.0.0,<2.0.0)
 Project-URL: Documentation, https://github.com/geeksville/starbash/blob/main/README.md
 Project-URL: Homepage, https://github.com/geeksville/starbash
 Project-URL: Repository, https://github.com/geeksville/starbash
@@ -80,7 +79,12 @@ This project is still very young - but making good progress 😊!
 
 If you are interested in alpha-testing we ❤️ you.  This README should have enough instructions to get you going, but if you encounter **any** problems please file a github issue and we'll work together to fix them.
 
-![Sample session movie](https://raw.githubusercontent.com/geeksville/starbash/refs/heads/main/doc/vhs/sample-session.gif)
+(new!) We now have a full GUI (click the video to play)...
+
+<video src="https://github.com/user-attachments/assets/0791c47f-f8fa-4279-be06-f869d7212d8c" autoplay controls loop muted playsinline width="100%"></video>
+
+We have a CLI interface also...
+![CLI sample session](https://raw.githubusercontent.com/geeksville/starbash/refs/heads/main/doc/vhs/sample-session.gif)
 
 ## Current features
 
@@ -225,6 +229,22 @@ sb publish github               # Publish to github
 See the [GitHub publishing guide](doc/publish/github.md) for prerequisites,
 preview instructions, authentication, and troubleshooting.
 
+### Desktop GUI
+
+Prefer clicking to typing? Starbash ships a desktop app (built on PySide6) that
+covers the same ground as the CLI — browse sessions and preview frames, edit the
+selection, run the pipeline and watch every stage live, tweak per-target stage
+exclusions, manage repositories, and build the report site.
+
+```bash
+sb gui
+```
+
+PySide6 is installed with Starbash, so there is nothing extra to add. The GUI is
+optional in the sense that you can ignore it completely and keep driving Starbash
+from the command line: both front ends share the same selection, config and
+database, so you can mix them freely.
+
 ## Supported commands
 
 ### Repository Management
@@ -306,3 +326,4 @@ However, in some of my other open-source projects, I've seen the current hell PR
 
 Copyright 2025 Kevin Hester, kevinh@geeksville.com.
 Licensed under the [GPL v3](https://raw.githubusercontent.com/geeksville/starbash/refs/heads/main/LICENSE)
+

{starbash-0.3.2 → starbash-0.3.4}/README.md

@@ -46,7 +46,12 @@ This project is still very young - but making good progress 😊!
 
 If you are interested in alpha-testing we ❤️ you.  This README should have enough instructions to get you going, but if you encounter **any** problems please file a github issue and we'll work together to fix them.
 
-![Sample session movie](https://raw.githubusercontent.com/geeksville/starbash/refs/heads/main/doc/vhs/sample-session.gif)
+(new!) We now have a full GUI (click the video to play)...
+
+<video src="https://github.com/user-attachments/assets/0791c47f-f8fa-4279-be06-f869d7212d8c" autoplay controls loop muted playsinline width="100%"></video>
+
+We have a CLI interface also...
+![CLI sample session](https://raw.githubusercontent.com/geeksville/starbash/refs/heads/main/doc/vhs/sample-session.gif)
 
 ## Current features
 
@@ -191,6 +196,22 @@ sb publish github               # Publish to github
 See the [GitHub publishing guide](doc/publish/github.md) for prerequisites,
 preview instructions, authentication, and troubleshooting.
 
+### Desktop GUI
+
+Prefer clicking to typing? Starbash ships a desktop app (built on PySide6) that
+covers the same ground as the CLI — browse sessions and preview frames, edit the
+selection, run the pipeline and watch every stage live, tweak per-target stage
+exclusions, manage repositories, and build the report site.
+
+```bash
+sb gui
+```
+
+PySide6 is installed with Starbash, so there is nothing extra to add. The GUI is
+optional in the sense that you can ignore it completely and keep driving Starbash
+from the command line: both front ends share the same selection, config and
+database, so you can mix them freely.
+
 ## Supported commands
 
 ### Repository Management
@@ -271,4 +292,4 @@ However, in some of my other open-source projects, I've seen the current hell PR
 ## License
 
 Copyright 2025 Kevin Hester, kevinh@geeksville.com.
-Licensed under the [GPL v3](https://raw.githubusercontent.com/geeksville/starbash/refs/heads/main/LICENSE)
+Licensed under the [GPL v3](https://raw.githubusercontent.com/geeksville/starbash/refs/heads/main/LICENSE)

{starbash-0.3.2 → starbash-0.3.4}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "starbash"
-version = "0.3.2"
+version = "0.3.4"
 description = "A tool for automating/standardizing/sharing astrophotography workflows."
 authors = ["Kevin Hester <kevinh@geeksville.com>"]
 readme = "README.md"
@@ -16,22 +16,19 @@ python = ">=3.12,<3.15"         # RestrictedPython doesn't yet work on 3.15
 
 # Note: to use toml-repo from the local submodule rather than pypi use:
 # toml-repo = {path = "toml-repo", develop = true}
-rich = ">=14.2.0,<15.0.0"
-restrictedpython = ">=8.1,<9.0"
-astropy = ">=7.1.1,<8.0.0"
+rich = ">=14.2.0"
+restrictedpython = ">=8.1"
+astropy = ">=7.1.1"
 platformdirs = ">=4.5.0,<5.0.0"
-typer = "^0.20.0"
-# Typer 0.20.1 still imports Click's deprecated stream helpers. Click 8.5
-# warns about those imports, so keep the compatible pre-8.5 release line until
-# Typer removes that dependency on deprecated APIs.
-click = ">=8.0,<8.5"
-sentry-sdk = "^2.42.1"
-update-checker = "^0.18.0"
-doit = "^0.36.0"
-textual = "^6.8.0"
+typer = ">=0.24.2"
+sentry-sdk = ">=2.42.1"
+update-checker = "^1.0.0"
+doit = ">=0.36.0"
 jinja2 = ">=3.1,<4.0"
 pygal = ">=3.0,<4.0"
-keyring = ">=25.6,<26.0"
+keyring = ">=25.6"
+# 0.1.7 added make_file_url()/path_from_file_url(); starbash.url re-exports them.
+toml-repo = "^0.1.7"
 
 # Note: to use graxpert from the local tree rather than pypi use:
 # graxpert = {path = "GraXpert", extras = ["cpuonly"], develop = true}
@@ -39,24 +36,32 @@ graxpert = {version = "^3.2.0a2", extras = ["cpuonly"]}
 
 # The following two libs are only needed for the (temporary) experiment of running siril scripts directly...
 # experimenting with siril-scripts/processing/VeraLux_HyperMetric_Stretch.py
+# pyqt6 is used *only* by those out-of-process siril scripts; Starbash itself never imports it,
+# so it can safely coexist with PySide6 (which the GUI *does* import).
 numpy = "2.2.6"
 pyqt6 = "^6.10.1"
-toml-repo = "^0.1.6"
+# The desktop GUI (`sb gui`) binding. A normal dependency: the GUI is a first-class
+# way to drive Starbash. "Optional" only means users may keep using the CLI instead.
+pyside6 = "^6.11.2"
 
 [tool.poetry.group.dev.dependencies]
 pytest = ">=8.4.2,<10.0.0"
-pytest-cov = "^6.0.0"
-pytest-xdist = "^3.8.0"
-pre-commit = "^4.3.0"
-ruff = "^0.14.4"
-basedpyright = "^1.33.0"
-textual-dev = "^1.8.0"
+pytest-cov = ">=6.0.0"
+pytest-xdist = ">=3.8.0"
+pre-commit = ">=4.3.0"
+ruff = ">=0.14.4"
+basedpyright = ">=1.33.0"
+# Qt widget testing for the `sb gui` desktop app (drives real widgets offscreen).
+pytest-qt = ">=4.4.0"
 
 # Adds the `graph` command to doit (GraphViz output) for `sb process doit graph`.
-doit-graph = "^0.3.0"
+doit-graph = ">=0.3.0"
 
 # We don't currently use this text based sql client
 # harlequin = "^2.4.1"
+library-skills = "^0.0.19"
+semble = {version = "^0.6.0", extras = ["mcp"]}
+imageio-ffmpeg = "^0.6.0"
 
 [tool.poetry.scripts]
 starbash = "starbash.main:app"
@@ -66,14 +71,23 @@ sb = "starbash.main:app"       # std alias for convenience
 markers = [
     "slow: marks tests as slow (deselect with '-m \"not slow\"')",
     "integration: marks tests as integration tests requiring /test-data (deselect with '-m \"not integration\"')",
+    "gui: marks tests that build real Qt widgets (offscreen; deselect with '-m \"not gui\"')",
+    "gui_integration: marks the scripted gui_integration movie runs (needs ffmpeg + /test-data; deselect with '-m \"not gui_integration\"')",
 ]
 # -n auto enables parallel test execution plugin
-# By default, exclude slow and integration tests
+# By default we exclude only slow and integration tests. GUI tests DO run: PySide6
+# ships as a normal dependency, and tests/conftest.py forces QT_QPA_PLATFORM=offscreen
+# so they work headless. If Qt cannot start at all on a machine, they skip cleanly.
 # --tb=line shows minimal tracebacks (just one line per failure)
 # --tb=short shows shorter tracebacks
 # -rfE shows summary of failed and error test outcomes at end (not passed)
 # --ignore excludes third-party test directories
-addopts = "-m 'not slow and not integration' -n auto --tb=short -rfE --ignore=GraXpert --ignore=textual --ignore=starbash-recipes"
+addopts = "-m 'not slow and not integration and not gui_integration' -n auto --tb=short -rfE --ignore=GraXpert --ignore=starbash-recipes"
+
+# Note: the `gui_integration` movies are deselected by default on purpose: they drive a
+# whole user journey (setup wizard -> reindex -> processing run) and take minutes, so they
+# run on demand - `just test-integration-gui`, or `pytest -m gui_integration -n 0`.
+# The marker name uses an underscore so it is usable as `pytest.mark.gui_integration`.
 
 # Note: To limit captured output further, you can:
 # 1. Use --tb=line for minimal tracebacks
@@ -119,9 +133,7 @@ exclude = [
     "starbash-recipes/**",
     "test-data/**",
     "GraXpert/**",
-    "textual/**",
-    "bin/**",
-    "*.py"  # Exclude root-level Python files
+    "bin/**"
 ]
 
 [tool.ruff.lint]
@@ -160,7 +172,8 @@ ignore = [
 
 [tool.basedpyright]
 # Enable import validation to catch runtime import errors at analysis time
-include = ["src"]
+# Type-check both the package and the test suite.
+include = ["src", "tests"]
 exclude = ["**/__pycache__", ".venv", "build", "dist"]
 extraPaths = ["GraXpert", "toml-repo"]  # Include local sources for type checking
 typeCheckingMode = "standard"                          # Standard type checking: catches type errors, missing annotations, etc.

{starbash-0.3.2 → starbash-0.3.4}/src/starbash/analytics.py

@@ -14,6 +14,15 @@ from starbash.check_version import is_connected
 # Default to no analytics/auto crash reports
 analytics_allowed = False
 
+# Canonical defaults for the user's analytics preferences.  These mirror the
+# comments in ``templates/userconfig.toml`` and are the source of truth for every
+# front end: the core (``app.py``), the CLI setup (``commands/user.py``) and the
+# GUI settings page / first-run wizard all read the preference through
+# ``analytics_enabled`` / ``analytics_include_user`` below, so an unset
+# preference is treated identically everywhere.
+DEFAULT_ANALYTICS_ENABLED = True
+DEFAULT_ANALYTICS_INCLUDE_USER = False
+
 __all__ = [
     "analytics_setup",
     "analytics_shutdown",
@@ -22,9 +31,31 @@ __all__ = [
     "analytics_exception",
     "is_development_environment",
     "NopAnalytics",
+    "DEFAULT_ANALYTICS_ENABLED",
+    "DEFAULT_ANALYTICS_INCLUDE_USER",
+    "analytics_enabled",
+    "analytics_include_user",
 ]
 
 
+def analytics_enabled(repo: Any) -> bool:
+    """Return whether the user has opted into analytics/crash reports.
+
+    Args:
+        repo: the user preferences repo (or any object with a ``get(key, default)``).
+    """
+    return bool(repo.get("analytics.enabled", DEFAULT_ANALYTICS_ENABLED))
+
+
+def analytics_include_user(repo: Any) -> bool:
+    """Return whether the user's email may be attached to reports.
+
+    Args:
+        repo: the user preferences repo (or any object with a ``get(key, default)``).
+    """
+    return bool(repo.get("analytics.include_user", DEFAULT_ANALYTICS_INCLUDE_USER))
+
+
 def analytics_setup(allowed: bool = False, user_email: str | None = None) -> None:
     global analytics_allowed
     analytics_allowed = allowed

{starbash-0.3.2 → starbash-0.3.4}/src/starbash/app.py

@@ -11,11 +11,10 @@ from typing import Any
 import rich.console
 import typer
 from rich.logging import RichHandler
-from rich.progress import track
 from toml_repo import Repo, RepoManager, get_config_suffix
 
 import starbash
-from starbash import doit_types
+from starbash import doit_types, events
 from starbash.aliases import (
     Aliases,
     get_aliases,
@@ -24,7 +23,9 @@ from starbash.aliases import (
 )
 from starbash.analytics import (
     NopAnalytics,
+    analytics_enabled,
     analytics_exception,
+    analytics_include_user,
     analytics_setup,
     analytics_shutdown,
     analytics_start_transaction,
@@ -47,6 +48,7 @@ from starbash.score import ScoredCandidate, score_candidates
 from starbash.selection import Selection, build_search_conditions
 from starbash.toml import toml_from_template
 from starbash.tool import init_tools
+from starbash.url import make_file_url
 from starbash.windows import windows_init
 
 critical_keys = [Database.DATE_OBS_KEY, Database.IMAGETYP_KEY]
@@ -131,13 +133,13 @@ def remap_expected_errors(exc: BaseException | None) -> BaseException | None:
         return None
 
     # Some OSErrors definitely don't indicate a code level bug...
-    expected_errors = [ "Read-only file system", "No space left on device" ]
+    expected_errors = ["Read-only file system", "No space left on device"]
     # Update: I give up, keep seeing misc reports from field.  I think all OSErrors should be considered
     # 'not a bug' until proven otherwise.
 
     if isinstance(exc, OperationalError):
         return NonSoftwareError(f"[red]Database IO error:[/red] {exc}")
-    elif isinstance(exc, OSError): # Was FileNotFoundError
+    elif isinstance(exc, OSError):  # Was FileNotFoundError
         return NonSoftwareError(f"[red]OS error:[/red] {exc}")
     elif isinstance(exc, OSError):
         exc_str = str(exc)
@@ -215,7 +217,7 @@ class Starbash:
         submodule_path = Path(__file__).parent.parent.parent / "starbash-recipes"
         if submodule_path.exists():
             logging.info(f"Using local recipes from {submodule_path}")
-            self.repo_manager.add_repo(f"file://{submodule_path}")
+            self.repo_manager.add_repo(make_file_url(submodule_path))
             return True
         return False
 
@@ -225,7 +227,7 @@ class Starbash:
         self.repo_manager.add_repo("pkg://defaults")
 
         # Add user prefs as a repo
-        self.user_repo = self.repo_manager.add_repo("file://" + str(create_user()))
+        self.user_repo = self.repo_manager.add_repo(make_file_url(create_user()))
 
         # We always need at least one set of recipes.  If the user hasn't specified one use the default.
         if self.repo_manager.get_repo_by_kind("std-recipe") is None:
@@ -251,8 +253,8 @@ class Starbash:
 
     def _init_analytics(self, cmd: str) -> None:
         self.analytics = NopAnalytics()
-        if self.user_repo.get("analytics.enabled", True):
-            include_user = self.user_repo.get("analytics.include_user", False)
+        if analytics_enabled(self.user_repo):
+            include_user = analytics_include_user(self.user_repo)
             user_email = self.user_repo.get("user.email", None) if include_user else None
             if user_email is not None:
                 user_email = str(user_email)
@@ -302,7 +304,12 @@ class Starbash:
     def __enter__(self) -> "Starbash":
         return self
 
-    def __exit__(self, exc_type: type[BaseException] | None, exc: BaseException | None, tb: types.TracebackType | None) -> bool:
+    def __exit__(
+        self,
+        exc_type: type[BaseException] | None,
+        exc: BaseException | None,
+        tb: types.TracebackType | None,
+    ) -> bool:
         from starbash import _is_test_env
         from starbash.exception import UserHandledError
 
@@ -326,12 +333,18 @@ class Starbash:
                 self.close()
                 if not handled:
                     # Show the full exception for developers
-                    starbash.console.print_exception(show_locals=False) # Locals are too verbose
+                    starbash.console.print_exception(show_locals=False)  # Locals are too verbose
 
-                # In test environments, let exceptions propagate naturally for better test diagnostics
-                if not _is_test_env:
-                    # But in any case, make our app exit with an error code
-                    raise typer.Exit(code=1)
+                # In test environments, let exceptions propagate naturally for better
+                # test diagnostics.  Return False *explicitly*: ``analytics_exception``
+                # is usually mocked in tests, and a mock returns a truthy value which
+                # would otherwise be returned from ``__exit__`` and silently suppress
+                # the exception (hiding real failures).
+                if _is_test_env:
+                    return False
+
+                # But in any case, make our app exit with an error code
+                raise typer.Exit(code=1)
         else:
             self.close()
 
@@ -366,9 +379,14 @@ class Starbash:
             if filter:
                 new[get_column_name(Database.FILTER_KEY)] = filter
 
-            telescop = header.get(Database.TELESCOP_KEY)
-            if telescop:
-                new[get_column_name(Database.TELESCOP_KEY)] = telescop
+            # The sessions table declares `telescop` NOT NULL, but a frame without a
+            # TELESCOP header is legitimate - `_extend_image_header` even falls back
+            # to CREATOR for such files.  Store "" ("unknown") rather than omitting
+            # the key: omitting it made upsert_session insert NULL and abort the whole
+            # repo scan with an IntegrityError.  `get_session` treats an empty
+            # telescope as "match any", so these frames still join the same rig's
+            # existing session when there is one.
+            new[get_column_name(Database.TELESCOP_KEY)] = header.get(Database.TELESCOP_KEY) or ""
 
             obj = header.get(Database.OBJECT_KEY)
             if obj:
@@ -551,7 +569,10 @@ class Starbash:
         #                 filtered_images.append(img)
         #         images = filtered_images
 
-        return images
+        # Reconstruct absolute paths, exactly like get_session_images() does.  Master
+        # rows only carry a repo-relative ``path``, so without this callers (e.g. the
+        # GUI's master preview) have no way to open the file.
+        return [self._add_image_abspath(image) for image in images]
 
     def add_filter_not_masters(self, conditions: list[SearchCondition]) -> None:
         """Add conditions to filter out master and processed repos from image searches."""
@@ -627,36 +648,69 @@ class Starbash:
         # Reconstruct absolute paths for all images
         return [self._add_image_abspath(img) for img in filtered_images]
 
-    def remove_repo_ref(self, url: str) -> None:
+    def _find_user_repo_ref(self, url: str) -> dict | None:
         """
-        Remove a repository reference from the user configuration.
+        Find the user configuration ``[[repo-ref]]`` entry matching a repository.
 
         Args:
-            url: The repository URL to remove (e.g., 'file:///path/to/repo')
+            url: The repository URL or directory to look for.  Both are accepted,
+                because the CLI passes whatever the user typed on the command line
+                (a number, a path or a URL) and the GUI passes the URL from its
+                repository table.  (e.g., 'file:///path/to/repo')
 
-        Raises:
-            ValueError: If the repository URL is not found in user configuration
+        Returns:
+            The matching repo-ref entry, or None if the URL is not user configured.
         """
-        self.db.remove_repo(url)
+        for ref in self.user_repo.config.get("repo-ref", []):
+            ref_dir = ref.get("dir", "")
+            if not ref_dir:
+                continue
+            # ``dir`` is always recorded as a plain filesystem path
+            # (Repo.add_repo_ref writes str(path)), so compare paths directly and
+            # fall back to the canonical URL -- that is the spelling Starbash
+            # stores elsewhere, and on Windows it differs far more than the
+            # "file://" prefix (file:///C:/data vs file://C:\data).
+            if ref_dir == url or make_file_url(ref_dir) == url:
+                return ref
+        return None
 
-        # Get the repo-ref list from user config
-        repo_refs: list = self.user_repo.config.get("repo-ref", [])
+    def is_repo_removable(self, url: str) -> bool:
+        """
+        Whether remove_repo_ref() would be able to remove a repository.
 
-        # Find and remove the matching repo-ref
-        found = False
-        refs_copy = [r for r in repo_refs]  # Make a copy to iterate
-        for ref in refs_copy:
-            ref_dir = ref.get("dir", "")
-            # Match by converting to file:// URL format if needed
-            if ref_dir == url or f"file://{ref_dir}" == url:
-                repo_refs.remove(ref)
+        Only repositories the user added are recorded in the user configuration;
+        the ones Starbash manages itself (``pkg://defaults``, the recipes checkout
+        and the preferences repo) have no entry there and therefore cannot be
+        removed individually.
+
+        Args:
+            url: The repository URL to check (e.g., 'file:///path/to/repo')
+
+        Returns:
+            True if the repository is recorded in the user configuration.
+        """
+        return self._find_user_repo_ref(url) is not None
+
+    def remove_repo_ref(self, url: str) -> None:
+        """
+        Remove a repository reference from the user configuration.
 
-                found = True
-                break
+        Args:
+            url: The repository URL to remove (e.g., 'file:///path/to/repo')
 
-        if not found:
+        Raises:
+            UserHandledError: If the repository URL is not found in user configuration
+        """
+        ref = self._find_user_repo_ref(url)
+        if ref is None:
+            # Check before touching the database: for a repository we refuse to
+            # remove (the recipes checkout, pkg://defaults, ...) dropping its
+            # indexed rows would only force a pointless re-index later.
             raise UserHandledError(f"Repository '{url}' not found in user configuration.")
 
+        self.db.remove_repo(url)
+        self.user_repo.config.get("repo-ref", []).remove(ref)
+
         # Write the updated config
         self.user_repo.write_config()
 
@@ -757,7 +811,21 @@ class Starbash:
             self._add_session(headers)
 
     def reindex_repo(self, repo: Repo, subdir: str | None = None) -> None:
-        """Reindex all repositories managed by the RepoManager."""
+        """Scan one repo's files into the database.
+
+        Progress is *reported*, never drawn: the scan publishes
+        ``reindex.progress`` / ``reindex.finished`` per repo (see
+        :mod:`starbash.events`) and the front ends render those -- the CLI's
+        :class:`~starbash.ui.cli.ReindexView`, a processing run's own live view,
+        or the GUI's Repositories page.  A ``rich`` bar drawn from here would be a
+        *second* display racing the run's live view on the terminal, which is the
+        problem ``doc/plans/cli-live-display.md`` already removed from
+        ``starbash.tool``.
+
+        Args:
+            repo: the repo to scan.
+            subdir: scan only this subdirectory of the repo (for debugging).
+        """
 
         # make sure this new repo is listed in the repos table
         self.repo_db_update()  # not really ideal, a more optimal version would just add the new repo
@@ -774,10 +842,19 @@ class Starbash:
 
             # Find all FITS files under this repo path
             all_files = list(path.rglob("*.fit")) + list(path.rglob("*.fits"))
-            for f in track(
-                all_files,
-                description=f"Indexing {repo.url}...",
-            ):
+            total_files = len(all_files)
+            events.publish(
+                events.EVENT_REINDEX_PROGRESS,
+                {"repo": repo.url, "done": 0, "total": total_files},
+            )
+            for index, f in enumerate(all_files, start=1):
+                # Throttle progress events: a repo can hold tens of thousands of
+                # frames and we don't want to flood the event bus / GUI.
+                if index % 25 == 0 or index == total_files:
+                    events.publish(
+                        events.EVENT_REINDEX_PROGRESS,
+                        {"repo": repo.url, "done": index, "total": total_files, "file": str(f)},
+                    )
                 if ".sbignore" in str(f):
                     logging.warning('Skipping "%s": path contains ".sbignore".', f)
                     continue
@@ -794,11 +871,19 @@ class Starbash:
                 except OSError as e:
                     logging.error(f'Skipping "{f}" due to: [red]{e}[/red]')
 
+            events.publish(
+                events.EVENT_REINDEX_FINISHED,
+                {"repo": repo.url, "indexed": total_files},
+            )
+
     def reindex_repos(self) -> None:
-        """Reindex all repositories managed by the RepoManager."""
+        """Reindex all repositories managed by the RepoManager.
+
+        Reports progress through the event bus only -- see :meth:`reindex_repo`.
+        """
         logging.debug("Reindexing all repositories...")
 
-        for repo in track(self.repo_manager.repos, description="Reindexing repos..."):
+        for repo in self.repo_manager.repos:
             self.reindex_repo(repo)
 
     def get_recipes(self) -> list[Repo]:

starbash-0.3.4/src/starbash/assets/icon.svg

@@ -0,0 +1,49 @@
+<?xml version="1.0" encoding="iso-8859-1"?>
+<!-- Uploaded to: SVG Repo, www.svgrepo.com, Generator: SVG Repo Mixer Tools -->
+<svg version="1.1" id="Layer_1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" 
+	 viewBox="0 0 241.168 241.168" xml:space="preserve">
+<g>
+	<g>
+		<g id="XMLID_28_">
+			<g>
+				<rect x="114.406" y="167.755" style="fill:#EAEEEF;" width="27.663" height="2.643"/>
+				<path style="fill:#EAEEEF;" d="M128.733,144.086c1.352,0,2.463,1.081,2.463,2.403c0,1.352-1.111,2.433-2.463,2.433
+					c-1.382,0-2.493-1.081-2.493-2.433C126.24,145.167,127.351,144.086,128.733,144.086z"/>
+				<polygon style="fill:#EAEEEF;" points="45.142,119.997 47.725,124.502 43.73,126.695 41.117,122.219 				"/>
+				<polygon style="fill:#EAEEEF;" points="144.262,38.929 169.222,82.061 88.364,121.469 68.059,86.386 				"/>
+				<polygon style="fill:#505967;" points="22.945,124.773 32.106,140.632 27.51,143.155 18.349,127.326 				"/>
+				<path style="fill:#2D213F;" d="M236.744,17.002c9.281,16.069,3.484,36.554-12.946,45.625l-4.596,2.553l0.33,0.541
+					c1.051,1.862,1.292,4.055,0.661,6.067c-0.571,1.892-1.892,3.514-3.664,4.475c-0.12,0.06-0.24,0.12-0.36,0.18l-79.056,38.537
+					v15.739c5.857,3.004,9.882,8.891,9.882,15.769c0,2.042-0.421,3.995-1.081,5.797h4.055c4.355,0,7.9,3.484,7.9,7.749v14.898
+					l53.855,52.684c3.094,3.004,3.094,7.9,0,10.933c-1.532,1.502-3.574,2.253-5.587,2.253c-2.012,0-4.055-0.751-5.587-2.253
+					l-53.855-52.684h-10.062v47.578c0,4.265-3.544,7.719-7.9,7.719c-4.385,0-7.9-3.454-7.9-7.719v-47.578H109.78l-52.654,51.452
+					c-1.532,1.532-3.544,2.283-5.587,2.283c-2.012,0-4.025-0.751-5.587-2.283c-3.064-3.004-3.064-7.9,0-10.903l52.654-51.482
+					v-14.898c0-4.265,3.514-7.749,7.9-7.749h5.016c-0.661-1.802-1.081-3.755-1.081-5.797c0-7.269,4.475-13.516,10.873-16.28v-7.539
+					l-51.512,25.11c-3.755,1.832-8.35,0.481-10.393-3.124l-3.905-6.728l-5.346,2.974c0.901,3.214-0.601,6.788-3.845,8.59
+					l-18.352,10.152c-3.785,2.103-8.53,0.961-10.543-2.523l-16.49-28.535c-2.042-3.484-0.601-8.05,3.214-10.152l18.322-10.122
+					c3.274-1.802,7.209-1.231,9.552,1.201l5.346-2.974l-4.535-7.837c-2.073-3.574-0.931-8.11,2.613-10.333L173.397,2.465
+					c1.832-1.111,4.055-1.472,6.157-0.961c2.103,0.511,3.905,1.832,4.956,3.694l0.991,1.682l4.566-2.553
+					C206.527-4.744,227.432,0.933,236.744,17.002z M216.018,49.171c8.831-4.896,11.954-15.919,6.968-24.57
+					c-5.016-8.65-16.28-11.714-25.14-6.818l-4.566,2.553l18.142,31.388L216.018,49.171z M201.661,66.262l-26.792-46.376
+					l-17.241,10.753l25.741,44.544L201.661,66.262z M169.222,82.061l-24.96-43.132L68.059,86.386l20.305,35.082L169.222,82.061z
+					 M142.069,170.398v-2.643h-27.663v2.643H142.069z M131.196,146.489c0-1.322-1.111-2.403-2.463-2.403
+					c-1.382,0-2.493,1.081-2.493,2.403c0,1.352,1.111,2.433,2.493,2.433C130.084,148.922,131.196,147.84,131.196,146.489z
+					 M74.187,128.377L54.723,94.706l-4.445,2.763l19.193,33.22L74.187,128.377z M47.725,124.502l-2.583-4.505l-4.025,2.223
+					l2.613,4.475L47.725,124.502z M32.106,140.632l-9.161-15.859l-4.596,2.553l9.161,15.829L32.106,140.632z"/>
+				<path style="fill:#FEDE94;" d="M222.987,24.601c4.986,8.65,1.862,19.674-6.968,24.57l-4.596,2.553l-18.142-31.388l4.566-2.553
+					C206.707,12.887,217.971,15.951,222.987,24.601z"/>
+				<polygon style="fill:#505967;" points="174.869,19.886 201.661,66.262 183.369,75.183 157.628,30.639 				"/>
+				<polygon style="fill:#505967;" points="54.723,94.706 74.187,128.377 69.471,130.69 50.278,97.47 				"/>
+			</g>
+		</g>
+		<polygon style="fill:#454D5B;" points="183.369,75.183 201.661,66.262 192.857,51.024 175.304,61.228 		"/>
+		<polygon style="fill:#C5CBCF;" points="88.364,121.469 169.222,82.061 161.731,69.118 84.159,114.203 		"/>
+		<polygon style="fill:#454D5B;" points="69.471,130.69 74.187,128.377 70.561,122.105 66.032,124.74 		"/>
+		<polygon style="fill:#454D5B;" points="174.869,19.886 157.628,30.639 165.305,43.927 182.981,33.928 		"/>
+		<polygon style="fill:#C5CBCF;" points="144.262,38.929 68.059,86.386 73.535,95.848 151.63,51.664 		"/>
+		<polygon style="fill:#454D5B;" points="54.723,94.706 50.278,97.47 55.3,106.165 59.856,103.588 		"/>
+		<path style="fill:#F2CE75;" d="M216.018,49.171c8.831-4.896,11.954-15.919,6.968-24.57c-1.114-1.922-2.55-3.55-4.187-4.893
+			c3.664,8.641,2.454,21.996-6.845,31.721L216.018,49.171z"/>
+	</g>
+</g>
+</svg>