glaip-sdk 0.7.15__py3-none-any.whl → 0.7.16__py3-none-any.whl

glaip_sdk/cli/slash/session.py

@@ -333,7 +333,8 @@ class SlashSession:
         current_status[0] = "Checking for updates..."
         if status_callback:
             status_callback(current_status[0])
-        self._maybe_show_update_prompt()
+        # Defer update prompt if we are in animated initialization to avoid blocking/cluttering
+        self._maybe_show_update_prompt(defer=bool(status_callback is None))
         return True
 
     def _update_pulse_step(
@@ -1619,9 +1620,12 @@ class SlashSession:
         self.console.print()
         self.console.print(banner)
 
-    def _maybe_show_update_prompt(self) -> None:
+    def _maybe_show_update_prompt(self, *, defer: bool = False) -> None:
         """Display update prompt once per session when applicable."""
-        if self._update_prompt_shown:
+        if self._update_prompt_shown or (defer and not self._update_prompt_shown):
+            if defer:
+                # Just mark as ready to show, but don't show yet
+                return
             return
 
         self._update_notifier(

glaip_sdk/cli/slash/tui/__init__.py

@@ -2,19 +2,20 @@
 
 from glaip_sdk.cli.slash.tui.clipboard import ClipboardAdapter, ClipboardResult
 from glaip_sdk.cli.slash.tui.context import TUIContext
+from glaip_sdk.cli.slash.tui.indicators import PulseIndicator
 from glaip_sdk.cli.slash.tui.keybind_registry import (
     Keybind,
     KeybindRegistry,
     format_key_sequence,
     parse_key_sequence,
 )
-from glaip_sdk.cli.slash.tui.toast import ToastBus, ToastVariant
 from glaip_sdk.cli.slash.tui.remote_runs_app import (
     RemoteRunsTextualApp,
     RemoteRunsTUICallbacks,
     run_remote_runs_textual,
 )
 from glaip_sdk.cli.slash.tui.terminal import TerminalCapabilities, detect_terminal_background
+from glaip_sdk.cli.slash.tui.toast import ToastBus, ToastVariant
 
 __all__ = [
     "TUIContext",
@@ -31,4 +32,5 @@ __all__ = [
     "format_key_sequence",
     "ClipboardAdapter",
     "ClipboardResult",
+    "PulseIndicator",
 ]

glaip_sdk/cli/slash/tui/accounts.tcss

@@ -79,7 +79,6 @@ Button:hover {
 
 #accounts-loading {
     width: 8;
-    display: none;
 }
 
 #status {
@@ -165,6 +164,12 @@ Button:hover {
     height: auto;
 }
 
+#harlequin-loading {
+    width: auto;
+    height: 3;
+    padding: 0 1;
+}
+
 #harlequin-status {
     padding: 1;
     margin-top: 1;

glaip_sdk/cli/slash/tui/accounts_app.py

@@ -30,77 +30,40 @@ from glaip_sdk.cli.slash.accounts_shared import (
 from glaip_sdk.cli.slash.tui.background_tasks import BackgroundTaskMixin
 from glaip_sdk.cli.slash.tui.clipboard import ClipboardAdapter, ClipboardResult
 from glaip_sdk.cli.slash.tui.context import TUIContext
+from glaip_sdk.cli.slash.tui.indicators import PulseIndicator
 from glaip_sdk.cli.slash.tui.keybind_registry import KeybindRegistry
 from glaip_sdk.cli.slash.tui.layouts.harlequin import HarlequinScreen
 from glaip_sdk.cli.slash.tui.loading import hide_loading_indicator, show_loading_indicator
 from glaip_sdk.cli.slash.tui.terminal import TerminalCapabilities
 from glaip_sdk.cli.slash.tui.theme.catalog import _BUILTIN_THEMES
 
-try:  # pragma: no cover - optional dependency
-    from glaip_sdk.cli.slash.tui.toast import (
-        ClipboardToastMixin,
-        Toast,
-        ToastBus,
-        ToastContainer,
-        ToastHandlerMixin,
-        ToastVariant,
-    )
-except Exception:  # pragma: no cover - optional dependency
-    ClipboardToastMixin = object  # type: ignore[assignment, misc]
-    Toast = None  # type: ignore[assignment]
-    ToastBus = None  # type: ignore[assignment]
-    ToastContainer = None  # type: ignore[assignment]
-    ToastHandlerMixin = object  # type: ignore[assignment, misc]
-    ToastVariant = None  # type: ignore[assignment]
+from glaip_sdk.cli.slash.tui.toast import (
+    ClipboardToastMixin,
+    Toast,
+    ToastBus,
+    ToastContainer,
+    ToastHandlerMixin,
+    ToastVariant,
+)
 from glaip_sdk.cli.validators import validate_api_key
 from glaip_sdk.utils.validation import validate_url
 
-try:  # pragma: no cover - optional dependency
-    from textual import events
-    from textual.app import App, ComposeResult
-    from textual.binding import Binding
-    from textual.containers import Horizontal, Vertical
-    from textual.coordinate import Coordinate
-    from textual.screen import ModalScreen
-    from textual.suggester import SuggestFromList
-    from textual.widgets import Button, Checkbox, DataTable, Footer, Header, Input, LoadingIndicator, Static
-except Exception:  # pragma: no cover - optional dependency
-    events = None  # type: ignore[assignment]
-    App = None  # type: ignore[assignment]
-    ComposeResult = None  # type: ignore[assignment]
-    Binding = None  # type: ignore[assignment]
-    Horizontal = None  # type: ignore[assignment]
-    Vertical = None  # type: ignore[assignment]
-    Coordinate = None  # type: ignore[assignment]
-    Button = None  # type: ignore[assignment]
-    Checkbox = None  # type: ignore[assignment]
-    DataTable = None  # type: ignore[assignment]
-    Footer = None  # type: ignore[assignment]
-    Header = None  # type: ignore[assignment]
-    Input = None  # type: ignore[assignment]
-    LoadingIndicator = None  # type: ignore[assignment]
-    ModalScreen = None  # type: ignore[assignment]
-    Static = None  # type: ignore[assignment]
-    SuggestFromList = None  # type: ignore[assignment]
-    Theme = None  # type: ignore[assignment]
-
-if App is not None:
-    try:  # pragma: no cover - optional dependency
-        from textual.theme import Theme
-    except Exception:  # pragma: no cover - optional dependency
-        Theme = None  # type: ignore[assignment]
-
-TEXTUAL_SUPPORTED = App is not None and DataTable is not None
-
-# Use safe bases so the module remains importable without Textual installed.
-if TEXTUAL_SUPPORTED:
-    _AccountFormBase = ModalScreen[dict[str, Any] | None]
-    _ConfirmDeleteBase = ModalScreen[str | None]
-    _AppBase = App[None]
-else:
-    _AccountFormBase = object
-    _ConfirmDeleteBase = object
-    _AppBase = object
+from textual.app import App, ComposeResult
+from textual.binding import Binding
+from textual.containers import Horizontal, Vertical
+from textual.coordinate import Coordinate
+from textual.screen import ModalScreen
+from textual.suggester import SuggestFromList
+from textual.theme import Theme
+from textual.widgets import Button, Checkbox, DataTable, Footer, Input, Static
+
+# Harlequin layout requires specific widget support
+TEXTUAL_SUPPORTED = True
+
+# Use standard Textual base classes
+_AccountFormBase = ModalScreen[dict[str, Any] | None]
+_ConfirmDeleteBase = ModalScreen[str | None]
+_AppBase = App[None]
 
 # Widget IDs for Textual UI
 ACCOUNTS_TABLE_ID = "#accounts-table"
@@ -390,6 +353,10 @@ class AccountFormModal(_AccountFormBase):  # pragma: no cover - interactive
         if btn_id == "form-save":
             self._handle_submit()
 
+    def on_input_submitted(self, _event: Input.Submitted) -> None:
+        """Handle Enter key to save."""
+        self._handle_submit()
+
     def _handle_submit(self) -> None:
         """Validate inputs and dismiss with payload on success."""
         status = self.query_one("#form-status", Static)
@@ -457,6 +424,10 @@ class ConfirmDeleteModal(_ConfirmDeleteBase):  # pragma: no cover - interactive
         if btn_id == "confirm-delete":
             self._handle_confirm()
 
+    def on_input_submitted(self, _event: Input.Submitted) -> None:
+        """Handle Enter key in confirmation input."""
+        self._handle_confirm()
+
     def _handle_confirm(self) -> None:
         """Dismiss with name when confirmation matches."""
         status = self.query_one("#confirm-status", Static)
@@ -520,6 +491,7 @@ class AccountsHarlequinScreen(  # pragma: no cover - interactive
             ctx: Shared TUI context.
         """
         super().__init__(ctx=ctx)
+        self._ctx = ctx
         self._store = get_account_store()
         self._all_rows = rows
         self._active_account = active_account
@@ -562,6 +534,7 @@ class AccountsHarlequinScreen(  # pragma: no cover - interactive
                     yield Button("(e) Edit", id="action-edit")
                     yield Button("(d) Delete", id="action-delete")
                     yield Button("(c) Copy", id="action-copy")
+                yield PulseIndicator(id="harlequin-loading")
                 yield Static("", id="harlequin-status")
                 # Help text showing keyboard shortcuts at the bottom
                 yield Static(
@@ -588,6 +561,7 @@ class AccountsHarlequinScreen(  # pragma: no cover - interactive
         self._prepare_toasts()
         self._register_keybinds()
         self._update_detail_pane()
+        self._hide_loading()
 
     def _initialize_context_services(self) -> None:
         """Initialize TUI context services."""
@@ -753,6 +727,12 @@ class AccountsHarlequinScreen(  # pragma: no cover - interactive
             return None
         return str(self._selected_account.get("name", ""))
 
+    def _show_loading(self, message: str | None = None) -> None:
+        show_loading_indicator(self, "#harlequin-loading", message=message, set_status=self._set_status)
+
+    def _hide_loading(self) -> None:
+        hide_loading_indicator(self, "#harlequin-loading")
+
     def action_switch_account(self) -> None:
         """Switch to the currently selected account."""
         if self._env_lock:
@@ -806,6 +786,7 @@ class AccountsHarlequinScreen(  # pragma: no cover - interactive
                 self._set_status(f"Switch failed: {exc}", "red")
                 return
             finally:
+                self._hide_loading()
                 self._is_switching = False
 
             if switched:
@@ -822,8 +803,10 @@ class AccountsHarlequinScreen(  # pragma: no cover - interactive
                 self._set_status(message or "Switch failed; kept previous account.", "yellow")
 
         try:
+            self._show_loading(f"Connecting to '{name}'...")
             self.track_task(perform(), logger=logging.getLogger(__name__))
         except Exception as exc:
+            self._hide_loading()
             self._is_switching = False
             self._set_status(f"Switch failed to start: {exc}", "red")
 
@@ -881,6 +864,15 @@ class AccountsHarlequinScreen(  # pragma: no cover - interactive
             self._filter_text = (event.value or "").strip()
             self._reload_accounts_list()
 
+    def on_input_submitted(self, event: Input.Submitted) -> None:
+        """Handle Enter key in Harlequin filter input."""
+        if event.input.id == "harlequin-filter":
+            try:
+                table = self.query_one(HARLEQUIN_ACCOUNTS_LIST_ID, DataTable)
+                table.focus()
+            except Exception:
+                pass
+
     def action_add_account(self) -> None:
         """Open add account modal."""
         if self._check_env_lock():
@@ -1144,6 +1136,8 @@ class AccountsHarlequinScreen(  # pragma: no cover - interactive
                     warning=tokens.warning,
                     error=tokens.error,
                     success=tokens.success,
+                    background=tokens.background,
+                    surface=tokens.background_panel,
                 )
             )
 
@@ -1371,16 +1365,23 @@ class AccountsTextualApp(  # pragma: no cover - interactive
 
     def on_input_submitted(self, event: Input.Submitted) -> None:
         """Apply filter when user presses Enter inside filter input."""
+        # Skip if a screen other than the default app screen is active (e.g., Harlequin or Modal)
+        if self.screen.id != "_default":
+            return
+
         self._filter_text = (event.value or "").strip()
         self._reload_rows()
-        table = self.query_one(ACCOUNTS_TABLE_ID, DataTable)
-        table.focus()
+        try:
+            table = self.query_one(ACCOUNTS_TABLE_ID, DataTable)
+            table.focus()
+        except Exception:
+            pass
         self._update_filter_button_visibility()
 
     def on_input_changed(self, event: Input.Changed) -> None:
         """Apply filter live as the user types."""
-        # Skip if Harlequin screen is active (it handles its own filtering)
-        if isinstance(self.screen, AccountsHarlequinScreen):
+        # Skip if a screen other than the default app screen is active (e.g., Harlequin or Modal)
+        if self.screen.id != "_default":
             return
         self._filter_text = (event.value or "").strip()
         self._reload_rows()

glaip_sdk/cli/slash/tui/indicators.py

@@ -0,0 +1,341 @@
+"""TUI animated indicators for waiting states."""
+
+from __future__ import annotations
+
+from typing import Any
+
+from rich.text import Text
+from textual._context import NoActiveAppError
+from textual.timer import Timer
+from textual.widgets import Static
+
+from glaip_sdk.cli.slash.tui.theme.catalog import _BUILTIN_THEMES
+
+DEFAULT_MESSAGE = "Processing…"
+DEFAULT_WIDTH = 20
+DEFAULT_SPEED_MS = 40
+
+BAR_GLYPH = " "
+PULSE_GLYPH = "█"
+
+VARIANT_STYLES: dict[str, str] = {
+    # Default hex colors matching gl-dark theme (see theme/catalog.py)
+    # These are used as fallbacks when the app theme is not active
+    "accent": "#C77DFF",
+    "primary": "#6EA8FE",
+    "success": "#34D399",
+    "warning": "#FBBF24",
+    "error": "#F87171",
+    "info": "#60A5FA",
+    "subtle": "#9CA3AF",
+}
+
+
+class PulseIndicator(Static):
+    """A Codex-style moving light/pulse indicator for waiting states.
+
+    Mirrors the 'Knight Rider' / Cylon scanner animation pattern.
+    Specified in specs/architecture/cli-textual-animated-indicators/spec.md
+    """
+
+    DEFAULT_CSS = """
+    PulseIndicator {
+        width: auto;
+        height: 3;
+        content-align: center middle;
+        padding: 0 2;
+        border: round #666666;
+        color: $text;
+        background: $surface;
+    }
+    """
+
+    def __init__(
+        self,
+        message: str | None = None,
+        *,
+        width: int = DEFAULT_WIDTH,
+        speed_ms: int = DEFAULT_SPEED_MS,
+        variant: str = "accent",
+        low_motion: bool = False,
+        **kwargs: Any,
+    ) -> None:
+        """Initialize the PulseIndicator."""
+        super().__init__(**kwargs)
+        self._width = self._coerce_width(width)
+        self._speed_ms = self._coerce_speed(speed_ms)
+        self._variant = self._coerce_variant(variant)
+        self._message = self._normalize_message(message)
+        self._low_motion = bool(low_motion)
+        self._position = 0
+        self._direction = 1
+        self._timer: Timer | None = None
+        self._pending_render: Text | None = None
+        self.can_focus = False
+        self.accessible_label = self._message
+
+    def on_mount(self) -> None:
+        """Handle component mounting."""
+        # Initial render happens here to ensure component is ready for updates
+        self._safe_update(self._render_static() if self._low_motion else self._render_frame())
+        if self._pending_render is not None:
+            return
+        if self._timer is None and not self._low_motion:
+            self._timer = self.set_interval(self._speed_ms / 1000, self._tick)
+
+    def start(self, message: str | None = None) -> None:
+        """Start the pulse animation."""
+        if message is not None:
+            self.update_message(message)
+        self._apply_pending_render()
+        self._cancel_timer()
+        if self._low_motion:
+            self._position = 0
+            self._safe_update(self._render_static())
+            return
+        self._timer = self.set_interval(self._speed_ms / 1000, self._tick)
+        self._safe_update(self._render_frame())
+
+    def stop(self, message: str | None = None) -> None:
+        """Stop the pulse animation."""
+        if message is not None:
+            self.update_message(message)
+        self._cancel_timer()
+        self._position = 0
+        self._direction = 1
+        self._safe_update(self._render_static())
+
+    def update_message(self, message: str) -> None:
+        """Update the display message."""
+        self._message = self._normalize_message(message)
+        self.accessible_label = self._message
+        self._safe_update(self._render_static() if self._low_motion else self._render_frame())
+
+    def _tick(self) -> None:
+        self._position += self._direction
+        if self._position >= self._width - 1:
+            self._position = self._width - 1
+            self._direction = -1
+        elif self._position <= 0:
+            self._position = 0
+            self._direction = 1
+        self._safe_update(self._render_frame())
+
+    def _render_frame(self) -> Text:
+        bar = self._render_bar(self._position, active=True)
+        bar.append(" ")
+        bar.append(self._message, style=self._message_style)
+        return bar
+
+    def _render_static(self) -> Text:
+        bar = self._render_bar(0, active=False)
+        bar.append(" ")
+        bar.append(self._message, style=self._message_style)
+        return bar
+
+    def _render_bar(self, position: int, *, active: bool) -> Text:
+        bg = self._resolve_style("on #111111", "$surface", is_bg=True)
+        bar = Text("[", style=f"grey37 {bg}")
+
+        p = position
+        v = self._active_style
+
+        for index in range(self._width):
+            if not active:
+                glyph = "█"
+                style = f"dim {v} {bg}"
+            else:
+                glyph, style = self._get_pulse_glyph_and_style(index, p, v, bg)
+
+            bar.append(glyph, style=style)
+
+        bar.append("]", style=f"grey37 {bg}")
+        return bar
+
+    def _get_pulse_glyph_and_style(self, index: int, p: int, v: str, bg: str) -> tuple[str, str]:
+        """Determine glyph and style for a bar position during animation."""
+        dist = abs(index - p)
+        if dist == 0:
+            return "█", f"bold white {bg}"
+        if dist == 1:
+            return "█", f"{v} {bg}"
+        if dist == 2:
+            return "▓", f"dim {v} {bg}"
+        if dist == 3:
+            return "▒", f"dim {v} {bg}"
+        return " ", bg
+
+    @property
+    def _active_style(self) -> str:
+        token = f"${self._variant}"
+        fallback = VARIANT_STYLES.get(self._variant, VARIANT_STYLES["accent"])
+        return self._resolve_style(fallback, token)
+
+    @property
+    def _message_style(self) -> str:
+        token = "$text-muted" if self._variant == "subtle" else "$text"
+        fallback = VARIANT_STYLES["subtle"] if self._variant == "subtle" else "white"
+        return self._resolve_style(fallback, token)
+
+    def _resolve_style(self, fallback: str, token: str | None = None, *, is_bg: bool = False) -> str:
+        """Resolve a theme token to a Rich style string with fallback."""
+        try:
+            # Standard resolution sequence
+            res = self._do_resolve(token, is_bg)
+            if res:
+                return res
+
+            # Specific background resolution fallback
+            if is_bg:
+                res = self._do_resolve("$surface", True) or self._do_resolve("$background", True)
+                if res:
+                    return res
+        except (NoActiveAppError, AttributeError):
+            pass
+        return fallback
+
+    def _do_resolve(self, token: str | None, is_bg: bool) -> str | None:
+        """Internal resolver that tries multiple sources."""
+        if not token:
+            return None
+
+        # 1. Try resolving via component styles
+        if token.startswith("$"):
+            res = self._resolve_from_component(token, is_bg)
+            if res:
+                return res
+
+        # 2. Try direct variable lookup (App.theme_variables or Theme.variables)
+        res = self._resolve_from_theme_vars(token.lstrip("$"), is_bg)
+        if res:
+            return res
+
+        # 3. Try our built-in theme catalog
+        return self._resolve_from_catalog(token.lstrip("$"), is_bg)
+
+    def _resolve_from_component(self, token: str, is_bg: bool) -> str | None:
+        """Resolve style from Textual component registry."""
+        try:
+            style = self.app.get_component_rich_style(token)
+            color = style.bgcolor if is_bg else style.color
+            if color:
+                return self._color_to_rich_style(color, is_bg)
+        except Exception:
+            pass
+        return None
+
+    def _resolve_from_theme_vars(self, var_name: str, is_bg: bool) -> str | None:
+        """Resolve color from theme variables dictionary."""
+        try:
+            app = self.app
+            # Check theme_variables first
+            val = getattr(app, "theme_variables", {}).get(var_name)
+            if val is None:
+                # Fallback to current theme object's variables (Textual 0.52+)
+                theme_obj = app.get_theme(app.theme)
+                if theme_obj and hasattr(theme_obj, "variables"):
+                    val = theme_obj.variables.get(var_name)
+
+            if val:
+                return self._color_to_rich_style(val, is_bg)
+        except Exception:
+            pass
+        return None
+
+    def _resolve_from_catalog(self, var_name: str, is_bg: bool) -> str | None:
+        """Resolve color from our built-in theme catalog."""
+        try:
+            theme_name = getattr(self.app, "theme", "gl-dark")
+            theme_tokens = _BUILTIN_THEMES.get(theme_name, _BUILTIN_THEMES["gl-dark"])
+            val = getattr(theme_tokens, var_name.replace("-", "_"), None)
+            if val:
+                return self._color_to_rich_style(val, is_bg)
+        except Exception:
+            pass
+        return None
+
+    def _color_to_rich_style(self, color: Any, is_bg: bool) -> str | None:
+        """Convert any color-like object to a Rich-compatible style string."""
+        if not color:
+            return None
+
+        # 1. Textual Color objects
+        if hasattr(color, "hex") and color.hex.startswith("#"):
+            return f"on {color.hex}" if is_bg else color.hex
+
+        # 2. Rich Color objects (with triplets)
+        if hasattr(color, "triplet") and color.triplet:
+            hex_val = color.triplet.hex
+            return f"on {hex_val}" if is_bg else hex_val
+
+        # 3. Strings or named colors
+        return self._str_color_to_style(color, is_bg)
+
+    def _str_color_to_style(self, color: Any, is_bg: bool) -> str | None:
+        """Helper to convert string-based colors to style."""
+        if color is None:
+            return None
+        c_str = str(color).strip()
+        if not c_str:
+            return None
+
+        if c_str.startswith("#"):
+            return f"on {c_str}" if is_bg else c_str
+
+        # If it's a named color like 'white', Rich understands it directly
+        # but we skip Textual's 'color(N)' internal format.
+        if not c_str.startswith("color(") and not c_str.startswith("auto"):
+            return f"on {c_str}" if is_bg else c_str
+
+        return None
+
+    def _safe_update(self, renderable: Text) -> None:
+        try:
+            self.update(renderable)
+            self._pending_render = None
+        except NoActiveAppError:
+            self._pending_render = renderable
+
+    def _apply_pending_render(self) -> None:
+        if self._pending_render is None:
+            return
+        try:
+            self.update(self._pending_render)
+            self._pending_render = None
+        except NoActiveAppError:
+            return
+
+    def _cancel_timer(self) -> None:
+        if self._timer is None:
+            return
+        try:
+            self._timer.stop()
+        except Exception:
+            pass
+        self._timer = None
+
+    @staticmethod
+    def _normalize_message(message: str | None) -> str:
+        if message is None:
+            return DEFAULT_MESSAGE
+        cleaned = str(message).strip()
+        return cleaned if cleaned else DEFAULT_MESSAGE
+
+    @staticmethod
+    def _coerce_width(width: int) -> int:
+        if not isinstance(width, int):
+            return DEFAULT_WIDTH
+        return width if width > 0 else DEFAULT_WIDTH
+
+    @staticmethod
+    def _coerce_speed(speed_ms: int) -> int:
+        if not isinstance(speed_ms, int):
+            return DEFAULT_SPEED_MS
+        return speed_ms if speed_ms > 0 else DEFAULT_SPEED_MS
+
+    @staticmethod
+    def _coerce_variant(variant: str) -> str:
+        if not isinstance(variant, str):
+            return "accent"
+        normalized = variant.strip().lower()
+        return normalized if normalized in VARIANT_STYLES else "accent"

glaip_sdk/cli/slash/tui/loading.py

@@ -1,58 +1,80 @@
 """Shared helpers for toggling Textual loading indicators.
 
-Note: uses Textual's built-in LoadingIndicator as the MVP; upgrade to the
-PulseIndicator from cli-textual-animated-indicators.md when shipped.
+This module provides unified helpers for showing/hiding both the built-in
+Textual LoadingIndicator and the custom PulseIndicator.
 """
 
 from __future__ import annotations
 
 from collections.abc import Callable
-from typing import TYPE_CHECKING
+from typing import Any
 
-try:  # pragma: no cover - optional dependency
-    from textual.widgets import LoadingIndicator
-except Exception:  # pragma: no cover - optional dependency
-    LoadingIndicator = None  # type: ignore[assignment]
+from textual.widgets import LoadingIndicator
 
-if TYPE_CHECKING:  # pragma: no cover - type checking aid
-    from textual.widgets import LoadingIndicator as _LoadingIndicatorType
+from glaip_sdk.cli.slash.tui.indicators import PulseIndicator
 
-    LoadingIndicator: type[_LoadingIndicatorType] | None
 
-
-def _set_indicator_display(app: object, selector: str, visible: bool) -> None:
-    """Safely toggle a LoadingIndicator's display property."""
-    if LoadingIndicator is None:
+def _set_indicator_display(app: Any, selector: str, visible: bool) -> None:
+    try:
+        indicator = app.query_one(selector, PulseIndicator)
+        if visible:
+            indicator.display = True
+            indicator.start()
+        else:
+            indicator.stop()
+            indicator.display = False
         return
+    except Exception:
+        pass
+
     try:
-        indicator = app.query_one(selector, LoadingIndicator)  # type: ignore[arg-type]
+        indicator = app.query_one(selector, LoadingIndicator)
         indicator.display = visible
     except Exception:
-        # Ignore lookup/rendering errors to keep UI resilient
         return
 
 
 def show_loading_indicator(
-    app: object,
+    app: Any,
     selector: str,
     *,
     message: str | None = None,
     set_status: Callable[..., None] | None = None,
     status_style: str = "cyan",
 ) -> None:
-    """Show a loading indicator and optionally set a status message."""
+    """Show a loading indicator (PulseIndicator or LoadingIndicator) and optionally set a status message.
+
+    Args:
+        app: Textual app instance containing the indicator widget
+        selector: CSS selector for the indicator widget
+        message: Optional message to display in the indicator
+        set_status: Optional callback to set status message (for fallback display)
+        status_style: Style for status message if set_status is provided
+    """
     _set_indicator_display(app, selector, True)
+
+    try:
+        indicator = app.query_one(selector, PulseIndicator)
+        if message:
+            indicator.update_message(message)
+    except Exception:
+        pass
+
     if message and set_status:
         try:
             set_status(message, status_style)
         except TypeError:
-            # Fallback for setters that accept only a single arg or kwargs
             try:
                 set_status(message)
             except Exception:
                 return
 
 
-def hide_loading_indicator(app: object, selector: str) -> None:
-    """Hide a loading indicator."""
+def hide_loading_indicator(app: Any, selector: str) -> None:
+    """Hide a loading indicator (PulseIndicator or LoadingIndicator).
+
+    Args:
+        app: Textual app instance containing the indicator widget
+        selector: CSS selector for the indicator widget
+    """
     _set_indicator_display(app, selector, False)

glaip_sdk/cli/slash/tui/remote_runs_app.py

@@ -18,32 +18,17 @@ from typing import Any
 
 from rich.text import Text
 
-try:  # pragma: no cover - optional dependency
-    from textual.app import App, ComposeResult
-    from textual.binding import Binding
-    from textual.containers import Horizontal, Vertical
-    from textual.coordinate import Coordinate
-    from textual.reactive import ReactiveError
-    from textual.screen import ModalScreen
-    from textual.widgets import DataTable, Footer, Header, LoadingIndicator, RichLog, Static
-except Exception:  # pragma: no cover - optional dependency
-    App = None  # type: ignore[assignment]
-    ComposeResult = None  # type: ignore[assignment]
-    Binding = None  # type: ignore[assignment]
-    Horizontal = None  # type: ignore[assignment]
-    Vertical = None  # type: ignore[assignment]
-    Coordinate = None  # type: ignore[assignment]
-    ReactiveError = Exception  # type: ignore[assignment, misc]
-    ModalScreen = object  # type: ignore[assignment, misc]
-    DataTable = None  # type: ignore[assignment]
-    Footer = None  # type: ignore[assignment]
-    Header = None  # type: ignore[assignment]
-    LoadingIndicator = None  # type: ignore[assignment]
-    RichLog = None  # type: ignore[assignment]
-    Static = None  # type: ignore[assignment]
+from textual.app import App, ComposeResult
+from textual.binding import Binding
+from textual.containers import Horizontal, Vertical
+from textual.coordinate import Coordinate
+from textual.reactive import ReactiveError
+from textual.screen import ModalScreen
+from textual.widgets import DataTable, Footer, Header, RichLog, Static
 
 from glaip_sdk.cli.slash.tui.clipboard import ClipboardAdapter
 from glaip_sdk.cli.slash.tui.context import TUIContext
+from glaip_sdk.cli.slash.tui.indicators import PulseIndicator
 from glaip_sdk.cli.slash.tui.loading import hide_loading_indicator, show_loading_indicator
 from glaip_sdk.cli.slash.tui.toast import ClipboardToastMixin, Toast, ToastBus, ToastContainer, ToastHandlerMixin
 
@@ -327,7 +312,6 @@ class RemoteRunsTextualApp(ToastHandlerMixin, App[None]):
     """Textual application for browsing remote runs."""
 
     CSS = f"""
-    Screen {{ layout: vertical; layers: base toasts; }}
     #toast-container {{
         width: 100%;
         height: auto;
@@ -335,10 +319,14 @@ class RemoteRunsTextualApp(ToastHandlerMixin, App[None]):
         align: right top;
         layer: toasts;
     }}
-    #status-bar {{ height: 3; padding: 0 1; }}
-    #agent-context {{ min-width: 25; padding-right: 1; }}
-    #{RUNS_LOADING_ID} {{ width: 8; }}
-    #status {{ padding-left: 1; }}
+    #{RUNS_LOADING_ID} {{
+        width: auto;
+        display: none;
+    }}
+    #status-bar {{
+        height: 3;
+        padding: 0 1;
+    }}
     """
 
     BINDINGS = [
@@ -400,7 +388,7 @@ class RemoteRunsTextualApp(ToastHandlerMixin, App[None]):
         )
         yield table  # pragma: no cover - interactive UI, tested via integration
         yield Horizontal(  # pragma: no cover - interactive UI, tested via integration
-            LoadingIndicator(id=RUNS_LOADING_ID),
+            PulseIndicator(id=RUNS_LOADING_ID),
             Static(id="status"),
             id="status-bar",
         )
@@ -557,7 +545,7 @@ class RemoteRunsTextualApp(ToastHandlerMixin, App[None]):
             self._update_status("Already loading run detail. Please wait…", append=True)
             return
         run_id = str(run.id)
-        self._show_loading("Loading run detail…", table_spinner=False)
+        self._show_loading("Loading run detail…", table_spinner=False, footer_message=False)
         self._queue_detail_load(run_id)
 
     async def action_export_run(self) -> None:
@@ -750,7 +738,7 @@ class RemoteRunsTextualApp(ToastHandlerMixin, App[None]):
         show_loading_indicator(
             self,
             RUNS_LOADING_SELECTOR,
-            message=message if footer_message else None,
+            message=message,
             set_status=self._update_status if footer_message else None,
         )
         self._set_table_loading(table_spinner)

glaip_sdk/runner/deps.py

@@ -31,7 +31,10 @@ def _probe_aip_agents_import() -> bool:
     Returns:
         True if aip_agents appears importable, False otherwise.
     """
-    return importlib.util.find_spec("aip_agents") is not None
+    try:
+        return importlib.util.find_spec("aip_agents") is not None
+    except (ImportError, ValueError):
+        return False
 
 
 def check_local_runtime_available() -> bool:

{glaip_sdk-0.7.15.dist-info → glaip_sdk-0.7.16.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: glaip-sdk
-Version: 0.7.15
+Version: 0.7.16
 Summary: Python SDK and CLI for GL AIP (GDP Labs AI Agent Package) - Build, run, and manage AI agents
 Author-email: Raymond Christopher <raymond.christopher@gdplabs.id>
 License: MIT

{glaip_sdk-0.7.15.dist-info → glaip_sdk-0.7.16.dist-info}/RECORD

@@ -78,16 +78,17 @@ glaip_sdk/cli/slash/accounts_shared.py,sha256=Mq5HxlI0YsVEQ0KKISWvyBZhzOFFWCzwRb
 glaip_sdk/cli/slash/agent_session.py,sha256=tuVOme-NbEyr6rwJvsBEKZYWQmsaRf4piJeRvIGu0ns,11384
 glaip_sdk/cli/slash/prompt.py,sha256=q4f1c2zr7ZMUeO6AgOBF2Nz4qgMOXrVPt6WzPRQMbAM,8501
 glaip_sdk/cli/slash/remote_runs_controller.py,sha256=iLl4a-mu9QU7dcedgEILewPtDIVtFUJkbKGtcx1F66U,21445
-glaip_sdk/cli/slash/session.py,sha256=YJ7UIeWyged1znmBVnGweOzH2l4NKeF5lT9VGdDvQWo,75998
-glaip_sdk/cli/slash/tui/__init__.py,sha256=oBUzeoslYwPKVlhqhgg4I7480b77vQNc9ec0NgdTC1s,977
-glaip_sdk/cli/slash/tui/accounts.tcss,sha256=GA5NFu_VSPRQdpfTaO0iUHjGDh8U0QDYqiI-WY59ZTc,2356
-glaip_sdk/cli/slash/tui/accounts_app.py,sha256=mv4SzBZrAvS-q-_kMEQQbQStiJPaJ71AmvcaoXiQm0A,73518
+glaip_sdk/cli/slash/session.py,sha256=jWTPrt374tDTt3tN-nBQ5wb2ssc60yMSAcSp4FNej2Y,76308
+glaip_sdk/cli/slash/tui/__init__.py,sha256=hAjH4ULBhRpQzA6fBLWRV6LiVm8UM3lgPurjoX9muYU,1061
+glaip_sdk/cli/slash/tui/accounts.tcss,sha256=5iVZZfS10CTJhnoZ9AFJejtj8nyQXH9xV7u9k8jSkGE,2411
+glaip_sdk/cli/slash/tui/accounts_app.py,sha256=5FtQy57xdUtBfOVRsqjXSbWLsLna7wIoMOz9t3h_ptQ,73187
 glaip_sdk/cli/slash/tui/background_tasks.py,sha256=SAe1mV2vXB3mJcSGhelU950vf8Lifjhws9iomyIVFKw,2422
 glaip_sdk/cli/slash/tui/clipboard.py,sha256=7fEshhTwHYaj-n7n0W0AsWTs8W0RLZw_9luXxrFTrtw,6227
 glaip_sdk/cli/slash/tui/context.py,sha256=mzI4TDXnfZd42osACp5uo10d10y1_A0z6IxRK1KVoVk,3320
+glaip_sdk/cli/slash/tui/indicators.py,sha256=jV3fFvEVWQ0inWJJ-B1fMsdkF0Uq2zwX3xcl0YWPHSE,11768
 glaip_sdk/cli/slash/tui/keybind_registry.py,sha256=_rK05BxTxNudYc4iJ9gDxpgeUkjDAq8rarIT-9A-jyM,6739
-glaip_sdk/cli/slash/tui/loading.py,sha256=nW5pv_Tnl9FUOPR3Qf2O5gt1AGHSo3b5-Uofg34F6AE,1909
-glaip_sdk/cli/slash/tui/remote_runs_app.py,sha256=F50pn2VmCN6RQImhQjNZvpE4gU3eUj6dV32_w1Gg5JU,30282
+glaip_sdk/cli/slash/tui/loading.py,sha256=Ku7HyQ_h-r2dJQ5aIEaCOi5PUu5gSsYle8oiKHIxfKI,2336
+glaip_sdk/cli/slash/tui/remote_runs_app.py,sha256=MVbzx-VJGi-wGSzr_CjQbdZbcWboVEbrCZUCuH2AeJM,29388
 glaip_sdk/cli/slash/tui/terminal.py,sha256=ZAC3sB17TGpl-GFeRVm_nI8DQTN3pyti3ynlZ41wT_A,12323
 glaip_sdk/cli/slash/tui/toast.py,sha256=XGITLHhO40xIGmtg9hanPmDsPCQY2hQXzoM_9mJXQyg,12442
 glaip_sdk/cli/slash/tui/layouts/__init__.py,sha256=KT77pZHa7Wz84QlHYT2mfhQ_AXUA-T0eHv_HtAvc1ac,473
@@ -150,7 +151,7 @@ glaip_sdk/registry/mcp.py,sha256=kNJmiijIbZL9Btx5o2tFtbaT-WG6O4Xf_nl3wz356Ow,797
 glaip_sdk/registry/tool.py,sha256=c0Ja4rFYMOKs_1yjDLDZxCId4IjQzprwXzX0iIL8Fio,14979
 glaip_sdk/runner/__init__.py,sha256=orJ3nLR9P-n1qMaAMWZ_xRS4368YnDpdltg-bX5BlUk,2210
 glaip_sdk/runner/base.py,sha256=KIjcSAyDCP9_mn2H4rXR5gu1FZlwD9pe0gkTBmr6Yi4,2663
-glaip_sdk/runner/deps.py,sha256=Du3hr2R5RHOYCRAv7RVmx661x-ayVXIeZ8JD7ODirTA,3884
+glaip_sdk/runner/deps.py,sha256=Lv8LdIF6H4JGzzvLmi-MgG72RJYgB-MsQNRx8yY7cl4,3956
 glaip_sdk/runner/langgraph.py,sha256=HB1n6w44LdyE7OSB2iSEtwhm_IeD7fGi_20erCZurX4,40869
 glaip_sdk/runner/logging_config.py,sha256=OrQgW23t42qQRqEXKH8U4bFg4JG5EEkUJTlbvtU65iE,2528
 glaip_sdk/runner/mcp_adapter/__init__.py,sha256=Rdttfg3N6kg3-DaTCKqaGXKByZyBt0Mwf6FV8s_5kI8,462
@@ -216,8 +217,8 @@ glaip_sdk/utils/rendering/steps/format.py,sha256=Chnq7OBaj8XMeBntSBxrX5zSmrYeGcO
 glaip_sdk/utils/rendering/steps/manager.py,sha256=BiBmTeQMQhjRMykgICXsXNYh1hGsss-fH9BIGVMWFi0,13194
 glaip_sdk/utils/rendering/viewer/__init__.py,sha256=XrxmE2cMAozqrzo1jtDFm8HqNtvDcYi2mAhXLXn5CjI,457
 glaip_sdk/utils/rendering/viewer/presenter.py,sha256=mlLMTjnyeyPVtsyrAbz1BJu9lFGQSlS-voZ-_Cuugv0,5725
-glaip_sdk-0.7.15.dist-info/METADATA,sha256=L7wF6kfY6yiNHvUsl6gGSVhFjMGllD9K3ZHA3UmIbkA,8528
-glaip_sdk-0.7.15.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-glaip_sdk-0.7.15.dist-info/entry_points.txt,sha256=NkhO6FfgX9Zrjn63GuKphf-dLw7KNJvucAcXc7P3aMk,54
-glaip_sdk-0.7.15.dist-info/top_level.txt,sha256=td7yXttiYX2s94-4wFhv-5KdT0rSZ-pnJRSire341hw,10
-glaip_sdk-0.7.15.dist-info/RECORD,,
+glaip_sdk-0.7.16.dist-info/METADATA,sha256=oqaUXPHF7LzD6CCvuJoHe10WD8OMhzMlKOKSgqwpx-E,8528
+glaip_sdk-0.7.16.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+glaip_sdk-0.7.16.dist-info/entry_points.txt,sha256=NkhO6FfgX9Zrjn63GuKphf-dLw7KNJvucAcXc7P3aMk,54
+glaip_sdk-0.7.16.dist-info/top_level.txt,sha256=td7yXttiYX2s94-4wFhv-5KdT0rSZ-pnJRSire341hw,10
+glaip_sdk-0.7.16.dist-info/RECORD,,