glaip-sdk 0.0.7__py3-none-any.whl → 0.0.8__py3-none-any.whl

glaip_sdk/cli/utils.py

@@ -8,13 +8,16 @@ from __future__ import annotations
 
 import io
 import json
+import logging
 import os
 import platform
 import shlex
 import shutil
 import subprocess
+import sys
 import tempfile
 from collections.abc import Callable
+from contextlib import AbstractContextManager, nullcontext
 from pathlib import Path
 from typing import TYPE_CHECKING, Any
 
@@ -50,6 +53,7 @@ from glaip_sdk.utils.rendering.renderer import (
 )
 
 console = Console()
+logger = logging.getLogger("glaip_sdk.cli.utils")
 
 
 # ----------------------------- Context helpers ---------------------------- #
@@ -225,6 +229,76 @@ def _get_view(ctx: Any) -> str:
     return fallback or "rich"
 
 
+def spinner_context(
+    ctx: Any | None,
+    message: str,
+    *,
+    console_override: Console | None = None,
+    spinner: str = "dots",
+    spinner_style: str = "cyan",
+) -> AbstractContextManager[Any]:
+    """Return a context manager that renders a spinner when appropriate."""
+
+    active_console = console_override or console
+    if not _can_use_spinner(ctx, active_console):
+        return nullcontext()
+
+    return active_console.status(
+        message,
+        spinner=spinner,
+        spinner_style=spinner_style,
+    )
+
+
+def _can_use_spinner(ctx: Any | None, active_console: Console) -> bool:
+    """Check if spinner output is allowed in the current environment."""
+
+    if ctx is not None:
+        tty_enabled = bool(get_ctx_value(ctx, "tty", True))
+        view = (_get_view(ctx) or "rich").lower()
+        if not tty_enabled or view not in {"", "rich"}:
+            return False
+
+    if not active_console.is_terminal:
+        return False
+
+    return _stream_supports_tty(getattr(active_console, "file", None))
+
+
+def _stream_supports_tty(stream: Any) -> bool:
+    """Return True if the provided stream can safely render a spinner."""
+
+    target = stream if hasattr(stream, "isatty") else sys.stdout
+    try:
+        return bool(target.isatty())
+    except Exception:
+        return False
+
+
+def _spinner_update(status_indicator: Any | None, message: str) -> None:
+    """Update spinner text when a status indicator is active."""
+
+    if status_indicator is None:
+        return
+
+    try:
+        status_indicator.update(message)
+    except Exception:  # pragma: no cover - defensive update
+        pass
+
+
+def _spinner_stop(status_indicator: Any | None) -> None:
+    """Stop an active spinner safely."""
+
+    if status_indicator is None:
+        return
+
+    try:
+        status_indicator.stop()
+    except Exception:  # pragma: no cover - defensive stop
+        pass
+
+
 # ----------------------------- Client config ----------------------------- #
 
 
@@ -507,6 +581,8 @@ def _fuzzy_pick(
         )
     except (KeyboardInterrupt, EOFError):  # pragma: no cover - user cancelled input
         return None
+    except Exception:  # pragma: no cover - prompt_toolkit not available in headless env
+        return None
 
     return _perform_fuzzy_search(answer, labels, by_label) if answer else None
 
@@ -798,11 +874,14 @@ def _handle_fuzzy_pick_selection(
     rows: list[dict[str, Any]], columns: list[tuple], title: str
 ) -> bool:
     """Handle fuzzy picker selection, returns True if selection was made."""
-    picked = (
-        _fuzzy_pick(rows, columns, title)
-        if console.is_terminal and os.isatty(1)
-        else None
-    )
+    picked = None
+    if console.is_terminal and os.isatty(1):
+        try:
+            picked = _fuzzy_pick(rows, columns, title)
+        except Exception:
+            logger.debug(
+                "Fuzzy picker failed; falling back to table output", exc_info=True
+            )
     if picked:
         table = _create_table(columns, title)
         table.add_row(*[str(picked.get(key, "N/A")) for key, _, _, _ in columns])
@@ -1096,6 +1175,7 @@ def resolve_resource(
     label: str,
     select: int | None = None,
     interface_preference: str = "fuzzy",
+    status_indicator: Any | None = None,
 ) -> Any | None:
     """Resolve resource reference (ID or name) with ambiguity handling.
 
@@ -1107,32 +1187,46 @@ def resolve_resource(
         label: Resource type label for error messages
         select: Optional selection index for ambiguity resolution
         interface_preference: "fuzzy" for fuzzy picker, "questionary" for up/down list
+        status_indicator: Optional Rich status indicator for wait animations
 
     Returns:
         Resolved resource object
     """
+    spinner = status_indicator
+    _spinner_update(spinner, f"[bold blue]Resolving {label}…[/bold blue]")
+
     # Try to resolve by ID first
+    _spinner_update(spinner, f"[bold blue]Fetching {label} by ID…[/bold blue]")
     result = _resolve_by_id(ref, get_by_id)
     if result is not None:
+        _spinner_update(spinner, f"[bold green]{label} found[/bold green]")
         return result
 
     # If get_by_id returned None, the resource doesn't exist
     if is_uuid(ref):
+        _spinner_stop(spinner)
         raise click.ClickException(f"{label} '{ref}' not found")
 
     # Find resources by name
+    _spinner_update(
+        spinner, f"[bold blue]Searching {label}s matching '{ref}'…[/bold blue]"
+    )
     matches = find_by_name(name=ref)
     if not matches:
+        _spinner_stop(spinner)
         raise click.ClickException(f"{label} '{ref}' not found")
 
     if len(matches) == 1:
+        _spinner_update(spinner, f"[bold green]{label} found[/bold green]")
         return matches[0]
 
     # Multiple matches found, handle ambiguity
     if select:
+        _spinner_stop(spinner)
         return _resolve_by_name_multiple_with_select(matches, select)
 
     # Choose interface based on preference
+    _spinner_stop(spinner)
     if interface_preference == "fuzzy":
         return _resolve_by_name_multiple_fuzzy(ctx, ref, matches, label)
     else:

glaip_sdk/utils/rendering/renderer/base.py

@@ -179,7 +179,7 @@ class RichStreamRenderer:
             AIPPanel(
                 Markdown(f"**Query:** {query}"),
                 title="User Request",
-                border_style="yellow",
+                border_style="#d97706",
                 padding=(0, 1),
             )
         )
@@ -311,6 +311,24 @@ class RichStreamRenderer:
     def _ensure_live(self) -> None:
         """Ensure live display is updated."""
         # Lazily create Live if needed
+        # Rich's Live expects the console to maintain a _live_stack list. When tests
+        # or callers provide a lightweight console double (e.g. unittest.mock.Mock),
+        # the attribute might be missing or replaced with another type which breaks
+        # the background refresh thread. Normal Rich consoles always expose
+        # _live_stack as a list, so we defensively initialise it if needed.
+        live_stack = getattr(self.console, "_live_stack", None)
+        if not isinstance(live_stack, list):
+            try:
+                self.console._live_stack = []  # type: ignore[attr-defined]
+            except Exception:
+                # If the console forbids attribute assignment we simply skip the
+                # live update for this cycle and fall back to buffered printing.
+                logger.debug(
+                    "Console missing _live_stack; skipping live UI initialisation",
+                    exc_info=True,
+                )
+                return
+
         if self.live is None and self.cfg.live:
             try:
                 self.live = Live(

glaip_sdk/utils/serialization.py

@@ -8,7 +8,7 @@ Authors:
 """
 
 import json
-from collections.abc import Iterable
+from collections.abc import Callable, Iterable
 from pathlib import Path
 from typing import Any
 
@@ -217,13 +217,13 @@ def _coerce_resource_to_mapping(resource: Any) -> dict[str, Any] | None:
     if isinstance(resource, dict):
         return resource
 
-    if hasattr(resource, "__dict__"):
-        try:
+    try:
+        if hasattr(resource, "__dict__"):
             return dict(resource.__dict__)
-        except (
-            Exception
-        ):  # pragma: no cover - pathological objects can still defeat coercion
-            return None
+    except (
+        Exception
+    ):  # pragma: no cover - pathological objects can still defeat coercion
+        return None
 
     return None
 
@@ -235,27 +235,59 @@ def _iter_public_attribute_names(resource: Any) -> Iterable[str]:
     names: list[str] = []
 
     def _collect(candidates: Iterable[str] | None) -> None:
-        if candidates is None:
-            return
-        for candidate in candidates:
+        for candidate in candidates or ():
             if candidate not in seen:
                 seen.add(candidate)
                 names.append(candidate)
 
-    _collect(
-        getattr(resource, "__dict__", {}).keys()
-        if hasattr(resource, "__dict__")
-        else None
-    )
-    _collect(getattr(resource, "__annotations__", {}).keys())
+    # Collect from __dict__
+    _collect_from_dict(resource, _collect)
+
+    # Collect from __annotations__
+    _collect_from_annotations(resource, _collect)
+
+    # Collect from __slots__
     _collect(getattr(resource, "__slots__", ()))
 
+    # Fallback to dir() if no names found
     if not names:
-        _collect(name for name in dir(resource) if not name.startswith("__"))
+        _collect_from_dir(resource, _collect)
 
     return iter(names)
 
 
+def _collect_from_dict(
+    resource: Any, collect_func: Callable[[Iterable[str]], None]
+) -> None:
+    """Safely collect attribute names from __dict__."""
+    try:
+        if hasattr(resource, "__dict__"):
+            dict_keys = getattr(resource, "__dict__", {})
+            if dict_keys:
+                collect_func(dict_keys.keys())
+    except Exception:  # pragma: no cover - defensive programming
+        pass
+
+
+def _collect_from_annotations(
+    resource: Any, collect_func: Callable[[Iterable[str]], None]
+) -> None:
+    """Safely collect attribute names from __annotations__."""
+    annotations = getattr(resource, "__annotations__", {})
+    if annotations:
+        collect_func(annotations.keys())
+
+
+def _collect_from_dir(
+    resource: Any, collect_func: Callable[[Iterable[str]], None]
+) -> None:
+    """Safely collect attribute names from dir()."""
+    try:
+        collect_func(name for name in dir(resource) if not name.startswith("__"))
+    except Exception:  # pragma: no cover - defensive programming
+        pass
+
+
 def _safe_getattr(resource: Any, name: str) -> Any:
     try:
         return getattr(resource, name)

{glaip_sdk-0.0.7.dist-info → glaip_sdk-0.0.8.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: glaip-sdk
-Version: 0.0.7
+Version: 0.0.8
 Summary: Python SDK for GL AIP (GDP Labs AI Agent Package) - Simplified CLI Design
 License: MIT
 Author: Raymond Christopher

{glaip_sdk-0.0.7.dist-info → glaip_sdk-0.0.8.dist-info}/RECORD

@@ -1,19 +1,23 @@
 glaip_sdk/__init__.py,sha256=FD-oTyFUKsTB9xTuGiqvkhuFXfeZ-TspjkeXERglha8,370
 glaip_sdk/_version.py,sha256=tGkFWAVu2ry4Hy7j-u7ophGbPRX8y-ngBbXDhN1VBIQ,2007
-glaip_sdk/branding.py,sha256=d-yGCYbUcdOlEH87E5PmGIyj6uBywsnf-Yd7zs__mUs,5371
+glaip_sdk/branding.py,sha256=_TiQtiwH3ZSWaQhooWPHioriL0goTiYANeN6DT8lQX8,5398
 glaip_sdk/cli/__init__.py,sha256=xCCfuF1Yc7mpCDcfhHZTX0vizvtrDSLeT8MJ3V7m5A0,156
 glaip_sdk/cli/agent_config.py,sha256=VHjebw68wAdhGUzYdPH8qz10oADZPRgUQcPW6F7iHIU,2421
 glaip_sdk/cli/commands/__init__.py,sha256=x0CZlZbZHoHvuzfoTWIyEch6WmNnbPzxajrox6riYp0,173
-glaip_sdk/cli/commands/agents.py,sha256=xiFKK76AeqkHyyQo7asdqNDy_VqbIzFHBUEvdE6lnwQ,36943
+glaip_sdk/cli/commands/agents.py,sha256=1xGSoH6C9hSaGglXf4pnloqX5gJQIPsgHhkliiWQQcY,39988
 glaip_sdk/cli/commands/configure.py,sha256=eRDzsaKV4fl2lJt8ieS4g2-xRnaa02eAAPW8xBf-tqA,7507
 glaip_sdk/cli/commands/mcps.py,sha256=ZfmFkisXHHqgFH6WU7W5qPvfdt5Dxd5ZFZb_Ml3POoQ,12794
 glaip_sdk/cli/commands/models.py,sha256=lHEVuXT8Sq239C_2LneV8PimVZSm9DXJKajdSVO-N50,1496
 glaip_sdk/cli/commands/tools.py,sha256=Xm_qCe0h2EqhbqNuKTBvow1HMc7MbLYx6j92A6rLwu0,16743
-glaip_sdk/cli/display.py,sha256=Wr86LTdwUaaTceXwi2v5rJAYaWaYYYhnt3PinSSAQP8,8605
+glaip_sdk/cli/display.py,sha256=jE20swoRKzpYUmc0jgbeonaXKeE9x95hfjWAEdnBYRc,8727
 glaip_sdk/cli/io.py,sha256=GPkw3pQMLBGoD5GH-KlbKpNRlVWFZOXHE17F7V3kQsI,3343
-glaip_sdk/cli/main.py,sha256=Z1uMlmU72FFXC2ushNTWpemhQWKbIJNFJauH6FOCPqA,11059
-glaip_sdk/cli/resolution.py,sha256=4PEW_KZfN13k7GGEWpx3dOTywE_d-ujSlUhO4liov5w,1673
-glaip_sdk/cli/utils.py,sha256=GQ_VwY374D8aLlySWYdExOZMap7Bb0nFIzS9gRhR5gs,37660
+glaip_sdk/cli/main.py,sha256=1RSR3YmJECnpErD_y9NXqCHjyho3JkYkVc26J6NAbzE,12694
+glaip_sdk/cli/resolution.py,sha256=BOw2NchReLKewAwBAZLWw_3_bI7u3tfzQEO7kQbIiGE,2067
+glaip_sdk/cli/slash/__init__.py,sha256=Vdv6Y8bu-pA8dxDlyP4XrhudBPivztUozhLAz9vaLig,682
+glaip_sdk/cli/slash/agent_session.py,sha256=JFIFBxB4xzigqHtpLGbl2fgg7RHNwy3e-kUMPMK9MdM,5006
+glaip_sdk/cli/slash/prompt.py,sha256=QFWYgNt5AhBRYGGyUkH30ToZE7B6LS-eZGzoyibYrK8,6064
+glaip_sdk/cli/slash/session.py,sha256=GtnnBqV79KtFJHeLfCKKnCXJ6SU1AmeXquPjy72T20E,24215
+glaip_sdk/cli/utils.py,sha256=Jhjv9ggwZGQ8oJZGIPIpOc29LqSZAks9FLVWtTpklYU,40629
 glaip_sdk/cli/validators.py,sha256=USbBgY86AwuDHO-Q_g8g7hu-ot4NgITBsWjTWIl62ms,5569
 glaip_sdk/client/__init__.py,sha256=nYLXfBVTTWwKjP0e63iumPYO4k5FifwWaELQPaPIKIg,188
 glaip_sdk/client/agents.py,sha256=FSKubF40wptMNIheC3_iawiX2CRbhTcNLFiz4qkPC6k,34659
@@ -36,7 +40,7 @@ glaip_sdk/utils/rendering/__init__.py,sha256=vXjwk5rPhhfPyD8S0DnV4GFFEtPJp4HCCg1
 glaip_sdk/utils/rendering/formatting.py,sha256=_k8tkcobctmHvdygMljZF7-ALGXpD9-hHF1CNtM2KMU,7201
 glaip_sdk/utils/rendering/models.py,sha256=SS34_00FaoGuSYn-viBkAtIbq7cJNwwPjpxnvyeUmxI,1567
 glaip_sdk/utils/rendering/renderer/__init__.py,sha256=EXwVBmGkSYcype4ocAXo69Z1kXu0gpNXmhH5LW0_B7A,2939
-glaip_sdk/utils/rendering/renderer/base.py,sha256=4WdvYwsvCBtiIaB-LVteDORyxZN1HCT2jFfU0X0ReXY,39977
+glaip_sdk/utils/rendering/renderer/base.py,sha256=OqRBfdE1cwYhaB-1wXwc4pyGaagnX2duistDOIaVsD0,40969
 glaip_sdk/utils/rendering/renderer/config.py,sha256=E4ER8TJJbqr1hcWjkwG7XROqLuccQy4EL99CbuLvSXE,783
 glaip_sdk/utils/rendering/renderer/console.py,sha256=4cLOw4Q1fkHkApuj6dWW8eYpeYdcT0t2SO5MbVt5UTc,1844
 glaip_sdk/utils/rendering/renderer/debug.py,sha256=FEYxAu4ZB0CjrJKevqQ2TKDgElA2cf6GqZXCNm12sNQ,3721
@@ -47,9 +51,9 @@ glaip_sdk/utils/rendering/steps.py,sha256=4zdeyKxMbUzCal4-yv8yf18144cs5wwXaxhe6m
 glaip_sdk/utils/resource_refs.py,sha256=0YzblJNfRhz9xhpaKE9aE68XEV-6_ssr0fIkiMVOka0,5489
 glaip_sdk/utils/rich_utils.py,sha256=-Ij-1bIJvnVAi6DrfftchIlMcvOTjVmSE0Qqax0EY_s,763
 glaip_sdk/utils/run_renderer.py,sha256=d_VMI6LbvHPUUeRmGqh5wK_lHqDEIAcym2iqpbtDad0,1365
-glaip_sdk/utils/serialization.py,sha256=DAOxi42xgE3EFCcdMLxPhQblL3uWVhndJXA2Jc-UF-I,8051
+glaip_sdk/utils/serialization.py,sha256=cUE6PxqTsfJuEEmsk_Li3QmaDavTIPotEA-BQ-v5exY,9043
 glaip_sdk/utils/validation.py,sha256=QNORcdyvuliEs4EH2_mkDgmoyT9utgl7YNhaf45SEf8,6992
-glaip_sdk-0.0.7.dist-info/METADATA,sha256=2NLkepPjdP_WEon0dKF2IZGi3Co6a4rzPznhpP_HAUY,4942
-glaip_sdk-0.0.7.dist-info/WHEEL,sha256=sP946D7jFCHeNz5Iq4fL4Lu-PrWrFsgfLXbbkciIZwg,88
-glaip_sdk-0.0.7.dist-info/entry_points.txt,sha256=EGs8NO8J1fdFMWA3CsF7sKBEvtHb_fujdCoNPhfMouE,47
-glaip_sdk-0.0.7.dist-info/RECORD,,
+glaip_sdk-0.0.8.dist-info/METADATA,sha256=yWuZjBH9LvHlgsGZ8NP8GsUpsAM2X2ScE9uNCDnRPls,4942
+glaip_sdk-0.0.8.dist-info/WHEEL,sha256=sP946D7jFCHeNz5Iq4fL4Lu-PrWrFsgfLXbbkciIZwg,88
+glaip_sdk-0.0.8.dist-info/entry_points.txt,sha256=EGs8NO8J1fdFMWA3CsF7sKBEvtHb_fujdCoNPhfMouE,47
+glaip_sdk-0.0.8.dist-info/RECORD,,