glaip-sdk 0.0.19__py3-none-any.whl → 0.0.20__py3-none-any.whl

glaip_sdk/_version.py

@@ -6,6 +6,8 @@ Falls back to a dev marker when running from source without installation.
 
 from __future__ import annotations
 
+from pathlib import Path
+
 try:
     from importlib.metadata import PackageNotFoundError, version  # Python 3.8+
 except Exception:  # pragma: no cover - extremely unlikely
@@ -38,8 +40,6 @@ def _try_get_dev_version() -> str | None:
         return None
 
     try:
-        from pathlib import Path
-
         here = Path(__file__).resolve()
         root = here.parent.parent  # project root (contains pyproject.toml)
         pyproject = root / "pyproject.toml"

glaip_sdk/branding.py

@@ -36,9 +36,27 @@ SECONDARY_DARK = "#003A5C"  # Darkest variant for emphasis
 SECONDARY_MEDIUM = "#005CB8"  # Medium variant for UI elements
 SECONDARY_LIGHT = "#40B4E5"  # Light variant for highlights
 
+# Neutral companion palette (optimized for dark terminals)
+SUCCESS = "#7FA089"  # Muted teal-green for success messaging
+WARNING = "#C3A46F"  # Soft amber for warnings
+ERROR = "#C97B6C"  # Tempered coral-red for errors
+INFO = "#9CA3AF"  # Cool grey for informational accents
+NEUTRAL = "#D1D5DB"  # Light grey for muted text and dividers
+
 BORDER = PRIMARY  # Keep borders aligned with primary brand tone
 TITLE_STYLE = f"bold {PRIMARY}"
 LABEL = "bold"
+SUCCESS_STYLE = f"bold {SUCCESS}"
+WARNING_STYLE = f"bold {WARNING}"
+ERROR_STYLE = f"bold {ERROR}"
+INFO_STYLE = f"bold {INFO}"
+ACCENT_STYLE = INFO  # For subdued inline highlights
+
+# Hint styling (slash command helpers, tips, quick actions)
+HINT_TITLE_STYLE = f"bold {SECONDARY_LIGHT}"
+HINT_COMMAND_STYLE = f"bold {SECONDARY_LIGHT}"
+HINT_DESCRIPTION_COLOR = NEUTRAL
+HINT_PREFIX_STYLE = INFO_STYLE
 
 
 class AIPBranding:
@@ -118,11 +136,17 @@ GDP Labs AI Agents Package
             "architecture": platform.architecture()[0],
         }
 
-    def display_welcome_panel(self, title: str = "Welcome to AIP") -> None:
+    def display_welcome_panel(
+        self,
+        title: str = "Welcome to AIP",
+        *,
+        console: Console | None = None,
+    ) -> None:
         """Display a welcome panel with branding.
 
         Args:
             title: Custom title for the welcome panel
+            console: Optional console instance to print to. If None, uses self.console
         """
         banner = self.get_welcome_banner()
         panel = AIPPanel(
@@ -131,7 +155,8 @@ GDP Labs AI Agents Package
             border_style=BORDER,
             padding=(1, 2),
         )
-        self.console.print(panel)
+        target_console = console or self.console
+        target_console.print(panel)
 
     def display_version_panel(self) -> None:
         """Display a panel with comprehensive version information."""

glaip_sdk/cli/auth.py

@@ -3,6 +3,10 @@
 This module provides utilities for preparing authentication data for export,
 including interactive secret capture and placeholder generation.
 
+These helpers are distinct from the AIP CLI's own authentication, which always
+relies on the API URL and API key managed via ``aip configure`` / `AIP_API_*`
+environment variables.
+
 Authors:
     Raymond Christopher (raymond.christopher@gdplabs.id)
 """
@@ -13,6 +17,9 @@ from typing import Any
 import click
 from rich.console import Console
 
+from glaip_sdk.branding import HINT_PREFIX_STYLE, WARNING_STYLE
+from glaip_sdk.cli.utils import command_hint, format_command_hint
+
 
 def prepare_authentication_export(
     auth: dict[str, Any] | None,
@@ -92,21 +99,19 @@ def _get_token_value(
         The token string, either provided by the user or the placeholder.
     """
     if prompt_for_secrets:
-        console.print(
-            "[yellow]Bearer token is missing or redacted. "
-            "Please provide the token.[/yellow]"
-        )
-        token_value = click.prompt(
-            "Bearer token (leave blank for placeholder)",
-            default="",
-            show_default=False,
+        return _prompt_secret_with_placeholder(
+            console,
+            warning_message="Bearer token is missing or redacted. Please provide the token.",
+            prompt_message="Bearer token (leave blank for placeholder)",
+            placeholder=placeholder,
+            tip_cli_command="configure",
+            tip_slash_command="configure",
         )
-        return token_value.strip() or placeholder
 
     if not click.get_text_stream("stdin").isatty():
         console.print(
-            "[yellow]⚠️  Non-interactive mode: "
-            "using placeholder for bearer token[/yellow]"
+            f"[{WARNING_STYLE}]⚠️  Non-interactive mode: "
+            "using placeholder for bearer token[/]"
         )
     return placeholder
 
@@ -205,20 +210,19 @@ def _get_api_key_value(
         The API key value, either provided by the user or the placeholder.
     """
     if prompt_for_secrets:
-        console.print(
-            f"[yellow]API key value for '{key_name}' is missing or redacted.[/yellow]"
-        )
-        key_value = click.prompt(
-            f"API key value for '{key_name}' (leave blank for placeholder)",
-            default="",
-            show_default=False,
+        return _prompt_secret_with_placeholder(
+            console,
+            warning_message=f"API key value for '{key_name}' is missing or redacted.",
+            prompt_message=f"API key value for '{key_name}' (leave blank for placeholder)",
+            placeholder=placeholder,
+            tip_cli_command="configure api-key",
+            tip_slash_command="configure",
         )
-        return key_value.strip() or placeholder
 
     if not click.get_text_stream("stdin").isatty():
         console.print(
-            f"[yellow]⚠️  Non-interactive mode: "
-            f"using placeholder for API key '{key_name}'[/yellow]"
+            f"[{WARNING_STYLE}]⚠️  Non-interactive mode: "
+            f"using placeholder for API key '{key_name}'[/]"
         )
     return placeholder
 
@@ -362,17 +366,18 @@ def _prompt_or_placeholder(
         The provided value or the placeholder.
     """
     if prompt_for_secrets:
-        console.print(f"[yellow]Header '{name}' is missing or redacted.[/yellow]")
-        value = click.prompt(
-            f"Value for header '{name}' (leave blank for placeholder)",
-            default="",
-            show_default=False,
+        return _prompt_secret_with_placeholder(
+            console,
+            warning_message=f"Header '{name}' is missing or redacted.",
+            prompt_message=f"Value for header '{name}' (leave blank for placeholder)",
+            placeholder=placeholder,
+            tip_cli_command="configure",
+            tip_slash_command="configure",
         )
-        return value.strip() or placeholder
 
     if not click.get_text_stream("stdin").isatty():
         console.print(
-            f"[yellow]⚠️  Non-interactive mode: using placeholder for header '{name}'[/yellow]"
+            f"[{WARNING_STYLE}]⚠️  Non-interactive mode: using placeholder for header '{name}'[/]"
         )
     return placeholder
 
@@ -412,3 +417,63 @@ def _build_custom_headers(
         )
 
     return headers
+
+
+def _prompt_secret_with_placeholder(
+    console: Console,
+    *,
+    warning_message: str,
+    prompt_message: str,
+    placeholder: str,
+    tip_cli_command: str | None = "configure",
+    tip_slash_command: str | None = "configure",
+    mask_input: bool = True,
+    retry_limit: int = 1,
+) -> str:
+    """Prompt for a secret value with masking, retries, and placeholder fallback.
+
+    Args:
+        console: Rich console used to render messaging.
+        warning_message: Message shown before prompting (rendered with warning style).
+        prompt_message: The message passed to :func:`click.prompt`.
+        placeholder: Placeholder value inserted when the user skips input.
+        tip_cli_command: CLI command (without ``aip`` prefix) used to build hints.
+        tip_slash_command: Slash command counterpart used in hints.
+        mask_input: Whether to hide user input while typing.
+        retry_limit: Number of additional attempts when the user submits empty input.
+
+    Returns:
+        The value entered by the user or the provided placeholder.
+    """
+    console.print(f"[{WARNING_STYLE}]{warning_message}[/]")
+
+    tip = command_hint(tip_cli_command, tip_slash_command)
+    if tip:
+        console.print(
+            f"[{HINT_PREFIX_STYLE}]Tip:[/] use {format_command_hint(tip) or tip} later if you want to update these credentials."
+        )
+
+    attempts = 0
+    while attempts <= retry_limit:
+        response = click.prompt(
+            prompt_message,
+            default="",
+            show_default=False,
+            hide_input=mask_input,
+        )
+        value = response.strip()
+        if value:
+            return value
+
+        if attempts < retry_limit:
+            console.print(
+                f"[{WARNING_STYLE}]No value entered. Enter a value or press Enter again to use the placeholder.[/]"
+            )
+            attempts += 1
+            continue
+
+        console.print("[dim]Using placeholder value.[/dim]")
+        return placeholder
+
+    # This line is unreachable as the loop always returns
+    # return placeholder

glaip_sdk/cli/commands/__init__.py

@@ -1,5 +1,5 @@
 """CLI commands package exports."""
 
-from glaip_sdk.cli.commands import agents, configure, mcps, models, tools
+from glaip_sdk.cli.commands import agents, configure, mcps, models, tools, update
 
-__all__ = ["agents", "configure", "mcps", "models", "tools"]
+__all__ = ["agents", "configure", "mcps", "models", "tools", "update"]

glaip_sdk/cli/commands/agents.py

@@ -4,6 +4,8 @@ Authors:
     Raymond Christopher (raymond.christopher@gdplabs.id)
 """
 
+from __future__ import annotations
+
 import json
 import os
 from collections.abc import Mapping
@@ -13,6 +15,14 @@ from typing import Any
 import click
 from rich.console import Console
 
+from glaip_sdk.branding import (
+    ACCENT_STYLE,
+    ERROR_STYLE,
+    INFO,
+    SUCCESS,
+    SUCCESS_STYLE,
+    WARNING_STYLE,
+)
 from glaip_sdk.cli.agent_config import (
     merge_agent_config_with_cli_args as merge_import_with_cli_args,
 )
@@ -45,6 +55,10 @@ from glaip_sdk.cli.io import (
 )
 from glaip_sdk.cli.resolution import resolve_resource_reference
 from glaip_sdk.cli.rich_helpers import markup_text, print_markup
+from glaip_sdk.cli.transcript import (
+    maybe_launch_post_run_viewer,
+    store_transcript_for_session,
+)
 from glaip_sdk.cli.utils import (
     _fuzzy_pick_for_resources,
     build_renderer,
@@ -65,6 +79,7 @@ from glaip_sdk.cli.validators import (
 )
 from glaip_sdk.config.constants import DEFAULT_AGENT_RUN_TIMEOUT, DEFAULT_MODEL
 from glaip_sdk.exceptions import AgentTimeoutError
+from glaip_sdk.icons import ICON_AGENT
 from glaip_sdk.utils import format_datetime, is_uuid
 from glaip_sdk.utils.agent_config import normalize_agent_config_for_import
 from glaip_sdk.utils.import_export import convert_export_to_import_format
@@ -168,17 +183,41 @@ def _fetch_full_agent_details(client: Any, agent: Any) -> Any | None:
     return agent
 
 
+def _normalise_model_name(value: Any) -> str | None:
+    """Return a cleaned model name or None when not usable."""
+    if value is None:
+        return None
+    if isinstance(value, str):
+        cleaned = value.strip()
+        return cleaned or None
+    if isinstance(value, bool):
+        return None
+    return str(value)
+
+
+def _model_from_config(agent: Any) -> str | None:
+    """Extract a usable model name from an agent's configuration mapping."""
+    config = getattr(agent, "agent_config", None)
+    if not config or not isinstance(config, dict):
+        return None
+
+    for key in ("lm_name", "model"):
+        normalised = _normalise_model_name(config.get(key))
+        if normalised:
+            return normalised
+    return None
+
+
 def _get_agent_model_name(agent: Any) -> str | None:
     """Extract model name from agent configuration."""
-    # Try different possible locations for model name
-    if hasattr(agent, "agent_config") and agent.agent_config:
-        if isinstance(agent.agent_config, dict):
-            return agent.agent_config.get("lm_name") or agent.agent_config.get("model")
+    config_model = _model_from_config(agent)
+    if config_model:
+        return config_model
 
-    if hasattr(agent, "model") and agent.model:
-        return agent.model
+    normalised_attr = _normalise_model_name(getattr(agent, "model", None))
+    if normalised_attr:
+        return normalised_attr
 
-    # Default fallback
     return DEFAULT_MODEL
 
 
@@ -198,7 +237,7 @@ def _resolve_resources_by_name(
         List of resolved resource IDs
     """
     out = []
-    for ref in list(items or ()):
+    for ref in items or ():
         if is_uuid(ref):
             out.append(ref)
             continue
@@ -275,7 +314,7 @@ def _format_fallback_agent_data(client: Any, agent: Any) -> dict:
 def _display_agent_details(ctx: Any, client: Any, agent: Any) -> None:
     """Display full agent details using raw API data to preserve ALL fields."""
     if agent is None:
-        handle_rich_output(ctx, markup_text("[red]❌ No agent provided[/red]"))
+        handle_rich_output(ctx, markup_text(f"[{ERROR_STYLE}]❌ No agent provided[/]"))
         return
 
     # Try to fetch and format raw agent data first
@@ -288,7 +327,7 @@ def _display_agent_details(ctx: Any, client: Any, agent: Any) -> None:
 
     if formatted_data:
         # Use raw API data - this preserves ALL fields including account_id
-        panel_title = f"🤖 {formatted_data.get('name', 'Unknown')}"
+        panel_title = f"{ICON_AGENT} {formatted_data.get('name', 'Unknown')}"
         output_result(
             ctx,
             formatted_data,
@@ -298,7 +337,7 @@ def _display_agent_details(ctx: Any, client: Any, agent: Any) -> None:
         # Fall back to Pydantic model data if raw fetch fails
         handle_rich_output(
             ctx,
-            markup_text("[yellow]Falling back to Pydantic model data[/yellow]"),
+            markup_text(f"[{WARNING_STYLE}]Falling back to Pydantic model data[/]"),
         )
 
         with spinner_context(
@@ -401,10 +440,10 @@ def list_agents(
         # Define table columns: (data_key, header, style, width)
         columns = [
             ("id", "ID", "dim", 36),
-            ("name", "Name", "cyan", None),
-            ("type", "Type", "yellow", None),
-            ("framework", "Framework", "blue", None),
-            ("version", "Version", "green", None),
+            ("name", "Name", ACCENT_STYLE, None),
+            ("type", "Type", WARNING_STYLE, None),
+            ("framework", "Framework", INFO, None),
+            ("version", "Version", SUCCESS, None),
         ]
 
         # Transform function for safe attribute access
@@ -439,7 +478,7 @@ def list_agents(
         output_list(
             ctx,
             agents,
-            "🤖 Available Agents",
+            f"{ICON_AGENT} Available Agents",
             columns,
             transform_agent,
             skip_picker=simple
@@ -497,19 +536,21 @@ def get(ctx: Any, agent_ref: str, select: int | None, export: str | None) -> Non
                 handle_rich_output(
                     ctx,
                     markup_text(
-                        f"[yellow]⚠️  Could not fetch full agent details: {e}[/yellow]"
+                        f"[{WARNING_STYLE}]⚠️  Could not fetch full agent details: {e}[/]"
                     ),
                 )
                 handle_rich_output(
                     ctx,
-                    markup_text("[yellow]⚠️  Proceeding with available data[/yellow]"),
+                    markup_text(
+                        f"[{WARNING_STYLE}]⚠️  Proceeding with available data[/]"
+                    ),
                 )
 
             export_resource_to_file(agent, export_path, detected_format)
             handle_rich_output(
                 ctx,
                 markup_text(
-                    f"[green]✅ Complete agent configuration exported to: {export_path} (format: {detected_format})[/green]"
+                    f"[{SUCCESS_STYLE}]✅ Complete agent configuration exported to: {export_path} (format: {detected_format})[/]"
                 ),
             )
 
@@ -621,7 +662,9 @@ def _save_run_transcript(save: str | None, result: Any, working_console: Any) ->
 
     with open(save, "w", encoding="utf-8") as f:
         f.write(content)
-    print_markup(f"[green]Full debug output saved to: {save}[/green]", console=console)
+    print_markup(
+        f"[{SUCCESS_STYLE}]Full debug output saved to: {save}[/]", console=console
+    )
 
 
 @agents_group.command()
@@ -678,6 +721,10 @@ def run(
     """
     final_input_text = _validate_run_input(input_option, input_text)
 
+    if verbose:
+        _emit_verbose_guidance(ctx)
+        return
+
     try:
         client = get_client(ctx)
         agent = _resolve_agent(
@@ -703,8 +750,29 @@ def run(
 
         result = client.agents.run_agent(**run_kwargs, timeout=timeout)
 
+        slash_mode = _running_in_slash_mode(ctx)
+        agent_id = str(_safe_agent_attribute(agent, "id") or "") or None
+        agent_name = _safe_agent_attribute(agent, "name")
+        model_hint = _get_agent_model_name(agent)
+
+        transcript_context = store_transcript_for_session(
+            ctx,
+            renderer,
+            final_result=result,
+            agent_id=agent_id,
+            agent_name=agent_name,
+            model=model_hint,
+            source="slash" if slash_mode else "cli",
+        )
+
         _handle_run_output(ctx, result, renderer)
         _save_run_transcript(save, result, working_console)
+        maybe_launch_post_run_viewer(
+            ctx,
+            transcript_context,
+            console=console,
+            slash_mode=slash_mode,
+        )
 
     except AgentTimeoutError as e:
         error_msg = str(e)
@@ -714,6 +782,25 @@ def run(
         _handle_command_exception(ctx, e)
 
 
+def _running_in_slash_mode(ctx: Any) -> bool:
+    ctx_obj = getattr(ctx, "obj", None)
+    return isinstance(ctx_obj, dict) and bool(ctx_obj.get("_slash_session"))
+
+
+def _emit_verbose_guidance(ctx: Any) -> None:
+    if _running_in_slash_mode(ctx):
+        message = (
+            "[dim]Tip:[/] Verbose streaming has been retired in the command palette. Run the agent normally and open "
+            "the post-run viewer (Ctrl+T) to inspect the transcript."
+        )
+    else:
+        message = (
+            "[dim]Tip:[/] `--verbose` is no longer supported. Re-run without the flag and toggle the post-run viewer "
+            "(Ctrl+T) for detailed output."
+        )
+    handle_rich_output(ctx, markup_text(message))
+
+
 def _handle_import_file_logic(
     import_file: str,
     model: str | None,
@@ -1317,7 +1404,7 @@ def sync_langflow(ctx: Any, base_url: str | None, api_key: str | None) -> None:
             handle_rich_output(
                 ctx,
                 markup_text(
-                    f"[green]✅ Successfully synced {success_count} LangFlow agents ({total_count} total processed)[/green]"
+                    f"[{SUCCESS_STYLE}]✅ Successfully synced {success_count} LangFlow agents ({total_count} total processed)[/]"
                 ),
             )