glaip-sdk 0.0.15__py3-none-any.whl → 0.0.16__py3-none-any.whl

glaip_sdk/cli/utils.py

@@ -21,6 +21,8 @@ from rich.markdown import Markdown
 from rich.pretty import Pretty
 from rich.text import Text
 
+from glaip_sdk.rich_components import AIPPanel
+
 # Optional interactive deps (fuzzy palette)
 try:
     from prompt_toolkit.completion import Completion
@@ -38,9 +40,15 @@ except Exception:  # pragma: no cover - optional dependency
 if TYPE_CHECKING:  # pragma: no cover - import-only during type checking
     from glaip_sdk import Client
 from glaip_sdk.cli import masking, pager
-from glaip_sdk.cli.commands.configure import load_config
-from glaip_sdk.cli.context import _get_view, get_ctx_value
-from glaip_sdk.rich_components import AIPPanel, AIPTable
+from glaip_sdk.cli.config import load_config
+from glaip_sdk.cli.context import (
+    _get_view,
+    get_ctx_value,
+)
+from glaip_sdk.cli.context import (
+    detect_export_format as _detect_export_format,
+)
+from glaip_sdk.rich_components import AIPTable
 from glaip_sdk.utils import is_uuid
 from glaip_sdk.utils.rendering.renderer import (
     CapturingConsole,
@@ -53,6 +61,59 @@ pager.console = console
 logger = logging.getLogger("glaip_sdk.cli.utils")
 
 
+# ----------------------------- Context helpers ---------------------------- #
+
+
+def detect_export_format(file_path: str | os.PathLike[str]) -> str:
+    """Backward-compatible proxy to `glaip_sdk.cli.context.detect_export_format`."""
+    return _detect_export_format(file_path)
+
+
+def in_slash_mode(ctx: click.Context | None = None) -> bool:
+    """Return True when running inside the slash command palette."""
+    if ctx is None:
+        try:
+            ctx = click.get_current_context(silent=True)
+        except RuntimeError:
+            ctx = None
+
+    if ctx is None:
+        return False
+
+    obj = getattr(ctx, "obj", None)
+    if isinstance(obj, dict):
+        return bool(obj.get("_slash_session"))
+
+    return bool(getattr(obj, "_slash_session", False))
+
+
+def command_hint(
+    cli_command: str | None,
+    slash_command: str | None = None,
+    *,
+    ctx: click.Context | None = None,
+) -> str | None:
+    """Return the appropriate command string for the current mode.
+
+    Args:
+        cli_command: Command string without the ``aip`` prefix (e.g., ``"status"``).
+        slash_command: Slash command counterpart (e.g., ``"status"`` or ``"/status"``).
+        ctx: Optional Click context override.
+
+    Returns:
+        The formatted command string for the active mode, or ``None`` when no
+        equivalent command exists in that mode.
+    """
+    if in_slash_mode(ctx):
+        if not slash_command:
+            return None
+        return slash_command if slash_command.startswith("/") else f"/{slash_command}"
+
+    if not cli_command:
+        return None
+    return f"aip {cli_command}"
+
+
 def spinner_context(
     ctx: Any | None,
     message: str,
@@ -62,7 +123,6 @@ def spinner_context(
     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()
@@ -76,7 +136,6 @@ def spinner_context(
 
 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()
@@ -91,7 +150,6 @@ def _can_use_spinner(ctx: Any | None, active_console: Console) -> bool:
 
 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())
@@ -101,7 +159,6 @@ def _stream_supports_tty(stream: Any) -> bool:
 
 def update_spinner(status_indicator: Any | None, message: str) -> None:
     """Update spinner text when a status indicator is active."""
-
     if status_indicator is None:
         return
 
@@ -113,7 +170,6 @@ def update_spinner(status_indicator: Any | None, message: str) -> None:
 
 def stop_spinner(status_indicator: Any | None) -> None:
     """Stop an active spinner safely."""
-
     if status_indicator is None:
         return
 
@@ -160,9 +216,12 @@ def get_client(ctx: Any) -> Client:  # pragma: no cover
     }
 
     if not config.get("api_url") or not config.get("api_key"):
-        raise click.ClickException(
-            "Missing api_url/api_key. Run `aip configure` or set AIP_* env vars."
-        )
+        configure_hint = command_hint("configure", slash_command="login", ctx=ctx)
+        actions = []
+        if configure_hint:
+            actions.append(f"Run `{configure_hint}`")
+        actions.append("set AIP_* env vars")
+        raise click.ClickException(f"Missing api_url/api_key. {' or '.join(actions)}.")
 
     return Client(
         api_url=config.get("api_url"),
@@ -171,6 +230,8 @@ def get_client(ctx: Any) -> Client:  # pragma: no cover
     )
 
 
+# ----------------------------- Secret masking ---------------------------- #
+
 # ----------------------------- Fuzzy palette ----------------------------- #
 
 
@@ -237,8 +298,8 @@ def _build_display_parts(
 
 
 def _row_display(row: dict[str, Any], columns: list[tuple]) -> str:
-    """
-    Build a compact text label for the palette.
+    """Build a compact text label for the palette.
+
     Prefers: name • type • framework • [id] (when available)
     Falls back to first 2 columns + [id].
     """
@@ -331,8 +392,8 @@ def _perform_fuzzy_search(
 def _fuzzy_pick(
     rows: list[dict[str, Any]], columns: list[tuple], title: str
 ) -> dict[str, Any] | None:  # pragma: no cover - requires interactive prompt toolkit
-    """
-    Open a minimal fuzzy palette using prompt_toolkit.
+    """Open a minimal fuzzy palette using prompt_toolkit.
+
     Returns the selected row (dict) or None if cancelled/missing deps.
     """
     if not _check_fuzzy_pick_requirements():
@@ -401,8 +462,8 @@ def _calculate_length_bonus(search: str, target: str) -> int:
 
 
 def _fuzzy_score(search: str, target: str) -> int:
-    """
-    Calculate fuzzy match score.
+    """Calculate fuzzy match score.
+
     Higher score = better match.
     Returns -1 if no match possible.
     """
@@ -465,8 +526,15 @@ def output_result(
     result: Any,
     title: str = "Result",
     panel_title: str | None = None,
-    success_message: str | None = None,
 ) -> None:
+    """Output a result to the console with optional title.
+
+    Args:
+        ctx: Click context
+        result: Result data to output
+        title: Optional title for the output
+        panel_title: Optional Rich panel title for structured output
+    """
     fmt = _get_view(ctx)
 
     data = _coerce_result_payload(result)
@@ -485,20 +553,12 @@ def output_result(
         _render_markdown_output(data)
         return
 
-    if success_message:
-        console.print(Text(f"[green]✅ {success_message}[/green]"))
-
+    renderable = Pretty(data)
     if panel_title:
-        console.print(
-            AIPPanel(
-                Pretty(data),
-                title=panel_title,
-                border_style="blue",
-            )
-        )
+        console.print(AIPPanel(renderable, title=panel_title))
     else:
         console.print(Text(f"[cyan]{title}:[/cyan]"))
-        console.print(Pretty(data))
+        console.print(renderable)
 
 
 # ----------------------------- List rendering ---------------------------- #
@@ -610,7 +670,6 @@ def _handle_empty_items(title: str) -> None:
 
 def _should_use_fuzzy_picker() -> bool:
     """Return True when the interactive fuzzy picker can be shown."""
-
     return console.is_terminal and os.isatty(1)
 
 
@@ -618,7 +677,6 @@ def _try_fuzzy_pick(
     rows: list[dict[str, Any]], columns: list[tuple], title: str
 ) -> dict[str, Any] | None:
     """Best-effort fuzzy selection; returns None if the picker fails."""
-
     if not _should_use_fuzzy_picker():
         return None
 
@@ -629,36 +687,34 @@ def _try_fuzzy_pick(
         return None
 
 
-def _resource_tip_command(title: str) -> str:
+def _resource_tip_command(title: str) -> str | None:
     """Resolve the follow-up command hint for the given table title."""
-
     title_lower = title.lower()
     mapping = {
-        "agent": "aip agents get",
-        "tool": "aip tools get",
-        "mcp": "aip mcps get",
-        "model": "aip models list",  # models only ship a list command
+        "agent": ("agents get", "agents"),
+        "tool": ("tools get", None),
+        "mcp": ("mcps get", None),
+        "model": ("models list", None),  # models only ship a list command
     }
-    for keyword, command in mapping.items():
+    for keyword, (cli_command, slash_command) in mapping.items():
         if keyword in title_lower:
-            return command
-    return "aip agents get"
+            return command_hint(cli_command, slash_command=slash_command)
+    return command_hint("agents get", slash_command="agents")
 
 
 def _print_selection_tip(title: str) -> None:
     """Print the contextual follow-up tip after a fuzzy selection."""
-
     tip_cmd = _resource_tip_command(title)
-    console.print(
-        Text.from_markup(f"\n[dim]Tip: use `{tip_cmd} <ID>` for details[/dim]")
-    )
+    if tip_cmd:
+        console.print(
+            Text.from_markup(f"\n[dim]Tip: use `{tip_cmd} <ID>` for details[/dim]")
+        )
 
 
 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 = _try_fuzzy_pick(rows, columns, title)
     if picked is None:
         return False
@@ -773,14 +829,16 @@ def build_renderer(
     """Build renderer and capturing console for CLI commands.
 
     Args:
-        ctx: Click context
-        save_path: Path to save output to (enables capturing)
-        theme: Color theme ("dark" or "light")
-        verbose: Whether to enable verbose mode
-        tty_enabled: Whether TTY is available
+        _ctx: Click context object for CLI operations.
+        save_path: Path to save output to (enables capturing console).
+        theme: Color theme ("dark" or "light").
+        verbose: Whether to enable verbose mode.
+        _tty_enabled: Whether TTY is available for interactive features.
+        live: Whether to enable live rendering mode (overrides verbose default).
+        snapshots: Whether to capture and store snapshots.
 
     Returns:
-        Tuple of (renderer, capturing_console)
+        Tuple of (renderer, capturing_console) for streaming output.
     """
     # Use capturing console if saving output
     working_console = console
@@ -859,8 +917,7 @@ def _build_resource_labels(resources: list[Any]) -> tuple[list[str], dict[str, A
 def _fuzzy_pick_for_resources(
     resources: list[Any], resource_type: str, _search_term: str
 ) -> Any | None:  # pragma: no cover - interactive selection helper
-    """
-    Fuzzy picker for resource objects, similar to _fuzzy_pick but without column dependencies.
+    """Fuzzy picker for resource objects, similar to _fuzzy_pick but without column dependencies.
 
     Args:
         resources: List of resource objects to choose from