glaip-sdk 0.0.6a0__py3-none-any.whl → 0.0.8__py3-none-any.whl

glaip_sdk/_version.py

@@ -24,28 +24,51 @@ except Exception:  # pragma: no cover
         _toml = None  # type: ignore
 
 
-def _get_version() -> str:
+def _try_get_installed_version() -> str | None:
+    """Try to get version from installed package."""
     try:
         return version("glaip-sdk")
     except PackageNotFoundError:
-        # Not installed; try reading from local pyproject for dev
-        if _toml is not None:
-            try:
-                from pathlib import Path
-
-                here = Path(__file__).resolve()
-                root = here.parent.parent  # project root (contains pyproject.toml)
-                pyproject = root / "pyproject.toml"
-                if pyproject.is_file():
-                    data = _toml.loads(pyproject.read_text(encoding="utf-8"))
-                    ver = data.get("project", {}).get("version") or data.get(
-                        "tool", {}
-                    ).get("poetry", {}).get("version")
-                    if isinstance(ver, str) and ver:
-                        return ver
-            except Exception:
-                pass
-        return "0.0.0.dev0"
+        return None
+
+
+def _try_get_dev_version() -> str | None:
+    """Try to get version from local pyproject.toml for development."""
+    if _toml is 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"
+        if not pyproject.is_file():
+            return None
+
+        data = _toml.loads(pyproject.read_text(encoding="utf-8"))
+        ver = data.get("project", {}).get("version") or data.get("tool", {}).get(
+            "poetry", {}
+        ).get("version")
+        if isinstance(ver, str) and ver:
+            return ver
+    except Exception:
+        pass
+    return None
+
+
+def _get_version() -> str:
+    # Try installed version first
+    installed_version = _try_get_installed_version()
+    if installed_version:
+        return installed_version
+
+    # Try development version
+    dev_version = _try_get_dev_version()
+    if dev_version:
+        return dev_version
+
+    return "0.0.0.dev0"
 
 
 __version__ = _get_version()

glaip_sdk/branding.py

@@ -31,8 +31,8 @@ except Exception:  # pragma: no cover
 
 
 # ---- minimal, readable styles (light blue + white theme) -----------------
-PRIMARY = "bright_blue"  # Light blue for main elements
-BORDER = "bright_blue"  # Light blue borders
+PRIMARY = "#15a2d8"  # GDP Labs brand blue
+BORDER = PRIMARY  # Keep borders aligned with brand tone
 TITLE_STYLE = f"bold {PRIMARY}"
 LABEL = "bold"
 
@@ -95,7 +95,7 @@ GDP Labs AI Agents Package
     # ---- public API -----------------------------------------------------------
     def get_welcome_banner(self) -> str:
         """Get AIP banner with version info."""
-        banner = self.AIP_LOGO
+        banner = f"[{PRIMARY}]{self.AIP_LOGO}[/{PRIMARY}]"
         line = f"Version: {self.version}"
         banner = f"{banner}\n{line}"
         return banner

glaip_sdk/cli/commands/agents.py

@@ -6,6 +6,7 @@ Authors:
 
 import json
 import os
+from collections.abc import Mapping
 from pathlib import Path
 from typing import Any
 
@@ -47,11 +48,13 @@ from glaip_sdk.cli.utils import (
     _fuzzy_pick_for_resources,
     build_renderer,
     coerce_to_row,
+    detect_export_format,
     get_client,
     get_ctx_value,
     output_flags,
     output_list,
     output_result,
+    spinner_context,
 )
 from glaip_sdk.cli.validators import (
     validate_agent_instruction_cli as validate_agent_instruction,
@@ -75,6 +78,91 @@ console = Console()
 AGENT_NOT_FOUND_ERROR = "Agent not found"
 
 
+def _safe_agent_attribute(agent: Any, name: str) -> Any:
+    """Return attribute value for ``name`` while filtering Mock sentinels."""
+
+    try:
+        value = getattr(agent, name)
+    except Exception:
+        return None
+
+    if hasattr(value, "_mock_name"):
+        return None
+    return value
+
+
+def _coerce_mapping_candidate(candidate: Any) -> dict[str, Any] | None:
+    """Convert a mapping-like candidate to a plain dict when possible."""
+
+    if candidate is None:
+        return None
+    if isinstance(candidate, Mapping):
+        return dict(candidate)
+    return None
+
+
+def _call_agent_method(agent: Any, method_name: str) -> dict[str, Any] | None:
+    """Attempt to call the named method and coerce its output to a dict."""
+
+    method = getattr(agent, method_name, None)
+    if not callable(method):
+        return None
+    try:
+        candidate = method()
+    except Exception:
+        return None
+    return _coerce_mapping_candidate(candidate)
+
+
+def _coerce_agent_via_methods(agent: Any) -> dict[str, Any] | None:
+    """Try standard serialisation helpers to produce a mapping."""
+
+    for attr in ("model_dump", "dict", "to_dict"):
+        mapping = _call_agent_method(agent, attr)
+        if mapping is not None:
+            return mapping
+    return None
+
+
+def _build_fallback_agent_mapping(agent: Any) -> dict[str, Any]:
+    """Construct a minimal mapping from well-known agent attributes."""
+
+    fallback_fields = (
+        "id",
+        "name",
+        "instruction",
+        "description",
+        "model",
+        "agent_config",
+        "tools",
+        "agents",
+        "mcps",
+        "timeout",
+    )
+
+    fallback: dict[str, Any] = {}
+    for field in fallback_fields:
+        value = _safe_agent_attribute(agent, field)
+        if value is not None:
+            fallback[field] = value
+
+    return fallback or {"name": str(agent)}
+
+
+def _prepare_agent_output(agent: Any) -> dict[str, Any]:
+    """Build a JSON-serialisable mapping for CLI output."""
+
+    method_mapping = _coerce_agent_via_methods(agent)
+    if method_mapping is not None:
+        return method_mapping
+
+    intrinsic = _coerce_mapping_candidate(agent)
+    if intrinsic is not None:
+        return intrinsic
+
+    return _build_fallback_agent_mapping(agent)
+
+
 def _fetch_full_agent_details(client: Any, agent: Any) -> Any | None:
     """Fetch full agent details by ID to ensure all fields are populated."""
     try:
@@ -180,7 +268,7 @@ def _format_fallback_agent_data(client: Any, agent: Any) -> dict:
     result_data = build_resource_result_data(full_agent, fields)
 
     # Handle missing instruction
-    if not result_data.get("instruction"):
+    if result_data.get("instruction") in ["N/A", None, ""]:
         result_data["instruction"] = "-"
 
     # Format dates for better display
@@ -198,7 +286,12 @@ def _display_agent_details(ctx: Any, client: Any, agent: Any) -> None:
         return
 
     # Try to fetch and format raw agent data first
-    formatted_data = _fetch_and_format_raw_agent_data(client, agent)
+    with spinner_context(
+        ctx,
+        "[bold blue]Loading agent details…[/bold blue]",
+        console_override=console,
+    ):
+        formatted_data = _fetch_and_format_raw_agent_data(client, agent)
 
     if formatted_data:
         # Use raw API data - this preserves ALL fields including account_id
@@ -215,7 +308,12 @@ def _display_agent_details(ctx: Any, client: Any, agent: Any) -> None:
             ctx, Text("[yellow]Falling back to Pydantic model data[/yellow]")
         )
 
-        result_data = _format_fallback_agent_data(client, agent)
+        with spinner_context(
+            ctx,
+            "[bold blue]Preparing fallback agent details…[/bold blue]",
+            console_override=console,
+        ):
+            result_data = _format_fallback_agent_data(client, agent)
 
         # Display using output_result
         output_result(
@@ -288,13 +386,18 @@ def list_agents(
     """List agents with optional filtering."""
     try:
         client = get_client(ctx)
-        agents = client.agents.list_agents(
-            agent_type=agent_type,
-            framework=framework,
-            name=name,
-            version=version,
-            sync_langflow_agents=sync_langflow,
-        )
+        with spinner_context(
+            ctx,
+            "[bold blue]Fetching agents…[/bold blue]",
+            console_override=console,
+        ):
+            agents = client.agents.list_agents(
+                agent_type=agent_type,
+                framework=framework,
+                name=name,
+                version=version,
+                sync_langflow_agents=sync_langflow,
+            )
 
         # Define table columns: (data_key, header, style, width)
         columns = [
@@ -356,18 +459,20 @@ def get(ctx: Any, agent_ref: str, select: int | None, export: str | None) -> Non
         )
 
         # Handle export option
-        if export:  # pragma: no cover - requires filesystem verification
+        if export:
             export_path = Path(export)
             # Auto-detect format from file extension
-            if export_path.suffix.lower() in [".yaml", ".yml"]:
-                detected_format = "yaml"
-            else:
-                detected_format = "json"
+            detected_format = detect_export_format(export_path)
 
             # Always export comprehensive data - re-fetch agent with full details
             try:
-                agent = client.agents.get_agent_by_id(agent.id)
-            except Exception as e:  # pragma: no cover - best-effort fallback messaging
+                with spinner_context(
+                    ctx,
+                    "[bold blue]Fetching complete agent data…[/bold blue]",
+                    console_override=console,
+                ):
+                    agent = client.agents.get_agent_by_id(agent.id)
+            except Exception as e:
                 handle_rich_output(
                     ctx,
                     Text(
@@ -584,8 +689,7 @@ def run(
         handle_json_output(ctx, error=Exception(error_msg))
         raise click.ClickException(error_msg)
     except Exception as e:
-        handle_json_output(ctx, error=e)
-        raise click.ClickException(str(e))
+        _handle_command_exception(ctx, e)
 
 
 def _handle_import_file_logic(
@@ -779,7 +883,7 @@ def _get_language_model_display_name(agent: Any, model: str | None) -> str:
 
 def _handle_successful_creation(ctx: Any, agent: Any, model: str | None) -> None:
     """Handle successful agent creation output."""
-    handle_json_output(ctx, agent.model_dump())
+    handle_json_output(ctx, _prepare_agent_output(agent))
 
     lm_display = _get_language_model_display_name(agent, model)
 
@@ -798,8 +902,8 @@ def _handle_successful_creation(ctx: Any, agent: Any, model: str | None) -> None
     handle_rich_output(ctx, display_agent_run_suggestions(agent))
 
 
-def _handle_creation_exception(ctx: Any, e: Exception) -> None:
-    """Handle exceptions during agent creation."""
+def _handle_command_exception(ctx: Any, e: Exception) -> None:
+    """Handle exceptions during command execution with consistent error handling."""
     if isinstance(e, click.ClickException):
         if get_ctx_value(ctx, "view") == "json":
             handle_json_output(ctx, error=Exception(AGENT_NOT_FOUND_ERROR))
@@ -811,6 +915,11 @@ def _handle_creation_exception(ctx: Any, e: Exception) -> None:
     raise click.ClickException(str(e))
 
 
+def _handle_creation_exception(ctx: Any, e: Exception) -> None:
+    """Handle exceptions during agent creation."""
+    _handle_command_exception(ctx, e)
+
+
 @agents_group.command()
 @click.option("--name", help="Agent name")
 @click.option("--instruction", help="Agent instruction (prompt)")
@@ -1078,7 +1187,7 @@ def update(
 
         updated_agent = client.agents.update_agent(agent.id, **update_data)
 
-        handle_json_output(ctx, updated_agent.model_dump())
+        handle_json_output(ctx, _prepare_agent_output(updated_agent))
         handle_rich_output(ctx, display_update_success("Agent", updated_agent.name))
         handle_rich_output(ctx, display_agent_run_suggestions(updated_agent))
 
@@ -1089,10 +1198,7 @@ def update(
         # Re-raise ClickExceptions without additional processing
         raise
     except Exception as e:
-        handle_json_output(ctx, error=e)
-        if get_ctx_value(ctx, "view") != "json":
-            print_api_error(e)
-        raise click.ClickException(str(e))
+        _handle_command_exception(ctx, e)
 
 
 @agents_group.command()
@@ -1133,10 +1239,7 @@ def delete(ctx: Any, agent_id: str, yes: bool) -> None:
         # Re-raise ClickExceptions without additional processing
         raise
     except Exception as e:
-        handle_json_output(ctx, error=e)
-        if get_ctx_value(ctx, "view") != "json":
-            print_api_error(e)
-        raise click.ClickException(str(e))
+        _handle_command_exception(ctx, e)
 
 
 @agents_group.command()
@@ -1149,9 +1252,7 @@ def delete(ctx: Any, agent_id: str, yes: bool) -> None:
 )
 @output_flags()
 @click.pass_context
-def sync_langflow(
-    ctx: Any, base_url: str | None, api_key: str | None
-) -> None:  # pragma: no cover - integration-only path
+def sync_langflow(ctx: Any, base_url: str | None, api_key: str | None) -> None:
     """Sync agents with LangFlow server flows.
 
     This command fetches all flows from the configured LangFlow server and
@@ -1192,7 +1293,4 @@ def sync_langflow(
             )
 
     except Exception as e:
-        handle_json_output(ctx, error=e)
-        if get_ctx_value(ctx, "view") != "json":
-            print_api_error(e)
-        raise click.ClickException(str(e))
+        _handle_command_exception(ctx, e)

glaip_sdk/cli/commands/configure.py

@@ -47,8 +47,8 @@ def save_config(config: dict[str, Any]) -> None:
     # Set secure file permissions
     try:
         os.chmod(CONFIG_FILE, 0o600)
-    except Exception:
-        pass  # Best effort
+    except Exception:  # pragma: no cover - platform dependent best effort
+        pass
 
 
 @click.group()

glaip_sdk/cli/commands/mcps.py

@@ -30,6 +30,7 @@ from glaip_sdk.cli.io import (
 from glaip_sdk.cli.resolution import resolve_resource_reference
 from glaip_sdk.cli.utils import (
     coerce_to_row,
+    detect_export_format,
     get_client,
     get_ctx_value,
     output_flags,
@@ -178,10 +179,7 @@ def get(ctx: Any, mcp_ref: str, export: str | None) -> None:
         if export:
             export_path = Path(export)
             # Auto-detect format from file extension
-            if export_path.suffix.lower() in [".yaml", ".yml"]:
-                detected_format = "yaml"
-            else:
-                detected_format = "json"
+            detected_format = detect_export_format(export_path)
 
             # Always export comprehensive data - re-fetch MCP with full details if needed
             try:

glaip_sdk/cli/commands/tools.py

@@ -34,6 +34,7 @@ from glaip_sdk.cli.io import (
 from glaip_sdk.cli.resolution import resolve_resource_reference
 from glaip_sdk.cli.utils import (
     coerce_to_row,
+    detect_export_format,
     get_client,
     get_ctx_value,
     output_flags,
@@ -328,10 +329,7 @@ def get(ctx: Any, tool_ref: str, select: int | None, export: str | None) -> None
         if export:
             export_path = Path(export)
             # Auto-detect format from file extension
-            if export_path.suffix.lower() in [".yaml", ".yml"]:
-                detected_format = "yaml"
-            else:
-                detected_format = "json"
+            detected_format = detect_export_format(export_path)
 
             # Always export comprehensive data - re-fetch tool with full details if needed
             try:

glaip_sdk/cli/display.py

@@ -134,17 +134,18 @@ _MISSING = object()
 def build_resource_result_data(resource: Any, fields: list[str]) -> dict[str, Any]:
     """Return a normalized mapping of ``fields`` extracted from ``resource``."""
 
-    return {
-        field: _normalise_field_value(field, _safe_get_attr(resource, field))
-        for field in fields
-    }
+    result: dict[str, Any] = {}
+    for field in fields:
+        try:
+            value = getattr(resource, field)
+        except AttributeError:
+            value = _MISSING
+        except Exception:
+            value = _MISSING
 
+        result[field] = _normalise_field_value(field, value)
 
-def _safe_get_attr(resource: Any, field: str) -> Any:
-    try:
-        return getattr(resource, field, _MISSING)
-    except Exception:
-        return _MISSING
+    return result
 
 
 def _normalise_field_value(field: str, value: Any) -> Any:
@@ -188,9 +189,7 @@ def _build_success_output_data(data: Any) -> dict[str, Any]:
     return data if data is not None else {"success": True}
 
 
-def handle_json_output(
-    ctx: Any, data: Any = None, error: Exception = None
-) -> None:  # pragma: no cover - formatting covered via integration tests
+def handle_json_output(ctx: Any, data: Any = None, error: Exception = None) -> None:
     """Handle JSON output format for CLI commands.
 
     Args:
@@ -248,9 +247,10 @@ def display_agent_run_suggestions(agent: Any) -> Panel:
 
     return AIPPanel(
         f"[bold blue]💡 Next Steps:[/bold blue]\n\n"
-        f"🚀 Run this agent:\n"
-        f'   [green]aip agents run {agent.id} "Your message here"[/green]\n\n'
-        f"📋 Or use the agent name:\n"
+        f"🚀 Start chatting with [bold]{agent.name}[/bold] right here:\n"
+        f"   Type your message below and press Enter to run it immediately.\n\n"
+        f"📋 Prefer the CLI instead?\n"
+        f'   [green]aip agents run {agent.id} "Your message here"[/green]\n'
         f'   [green]aip agents run "{agent.name}" "Your message here"[/green]\n\n'
         f"🔧 Available options:\n"
         f"   [dim]--chat-history[/dim]  Include previous conversation\n"

glaip_sdk/cli/main.py

@@ -24,11 +24,21 @@ from glaip_sdk.cli.commands.configure import (
 from glaip_sdk.cli.commands.mcps import mcps_group
 from glaip_sdk.cli.commands.models import models_group
 from glaip_sdk.cli.commands.tools import tools_group
-from glaip_sdk.config.constants import DEFAULT_AGENT_RUN_TIMEOUT
+from glaip_sdk.cli.utils import spinner_context
+from glaip_sdk.config.constants import (
+    DEFAULT_AGENT_RUN_TIMEOUT,
+)
 from glaip_sdk.rich_components import AIPPanel, AIPTable
 
+# Import SlashSession for potential mocking in tests
+try:
+    from glaip_sdk.cli.slash import SlashSession
+except ImportError:  # pragma: no cover - optional slash dependencies
+    # Slash dependencies might not be available in all environments
+    SlashSession = None
+
 
-@click.group()
+@click.group(invoke_without_command=True)
 @click.version_option(version=_SDK_VERSION, prog_name="aip")
 @click.option("--api-url", envvar="AIP_API_URL", help="AIP API URL")
 @click.option("--api-key", envvar="AIP_API_KEY", help="AIP API Key")
@@ -72,6 +82,15 @@ def main(
 
     ctx.obj["tty"] = not no_tty
 
+    if ctx.invoked_subcommand is None and not ctx.resilient_parsing:
+        if _should_launch_slash(ctx) and SlashSession is not None:
+            session = SlashSession(ctx)
+            session.run()
+            ctx.exit()
+        else:
+            click.echo(ctx.get_help())
+            ctx.exit()
+
 
 # Add command groups
 main.add_command(agents_group)
@@ -87,6 +106,19 @@ main.add_command(configure_command)
 # Tip: `--version` is provided by click.version_option above.
 
 
+def _should_launch_slash(ctx: click.Context) -> bool:
+    """Determine whether to open the command palette automatically."""
+
+    ctx_obj = ctx.obj or {}
+    if not bool(ctx_obj.get("tty", True)):
+        return False
+
+    if not (sys.stdin.isatty() and sys.stdout.isatty()):
+        return False
+
+    return True
+
+
 @main.command()
 @click.pass_context
 def status(ctx: Any) -> None:
@@ -152,9 +184,23 @@ def status(ctx: Any) -> None:
 
         # Test connection by listing resources
         try:
-            agents = client.list_agents()
-            tools = client.list_tools()
-            mcps = client.list_mcps()
+            with spinner_context(
+                ctx,
+                "[bold blue]Checking GL AIP status…[/bold blue]",
+                console_override=console,
+                spinner_style="cyan",
+            ) as status_indicator:
+                if status_indicator is not None:
+                    status_indicator.update("[bold blue]Fetching agents…[/bold blue]")
+                agents = client.list_agents()
+
+                if status_indicator is not None:
+                    status_indicator.update("[bold blue]Fetching tools…[/bold blue]")
+                tools = client.list_tools()
+
+                if status_indicator is not None:
+                    status_indicator.update("[bold blue]Fetching MCPs…[/bold blue]")
+                mcps = client.list_mcps()
 
             # Create status table
             table = AIPTable(title="🔗 GL AIP Status")

glaip_sdk/cli/resolution.py

@@ -12,7 +12,7 @@ from typing import Any
 
 import click
 
-from glaip_sdk.cli.utils import resolve_resource
+from glaip_sdk.cli.utils import resolve_resource, spinner_context
 
 
 def resolve_resource_reference(
@@ -25,6 +25,7 @@ def resolve_resource_reference(
     label: str,
     select: int | None = None,
     interface_preference: str | None = None,
+    spinner_message: str | None = None,
 ) -> Any | None:
     """Resolve resource reference (ID or name) with ambiguity handling.
 
@@ -47,14 +48,21 @@ def resolve_resource_reference(
         click.ClickException: If resolution fails
     """
     try:
-        return resolve_resource(
-            ctx,
-            reference,
-            get_by_id=get_by_id_func,
-            find_by_name=find_by_name_func,
-            label=label,
-            select=select,
-            interface_preference=interface_preference,
+        message = (
+            spinner_message
+            if spinner_message is not None
+            else f"[bold blue]Fetching {label}…[/bold blue]"
         )
+        with spinner_context(ctx, message, spinner_style="cyan") as status_indicator:
+            return resolve_resource(
+                ctx,
+                reference,
+                get_by_id=get_by_id_func,
+                find_by_name=find_by_name_func,
+                label=label,
+                select=select,
+                interface_preference=interface_preference,
+                status_indicator=status_indicator,
+            )
     except Exception as e:
         raise click.ClickException(f"Failed to resolve {resource_type.lower()}: {e}")

glaip_sdk/cli/slash/__init__.py

@@ -0,0 +1,25 @@
+"""Slash command palette entrypoints.
+
+Authors:
+    Raymond Christopher (raymond.christopher@gdplabs.id)
+"""
+
+from glaip_sdk.cli.commands.agents import get as agents_get_command
+from glaip_sdk.cli.commands.agents import run as agents_run_command
+from glaip_sdk.cli.commands.configure import configure_command, load_config
+from glaip_sdk.cli.utils import get_client
+
+from .agent_session import AgentRunSession
+from .prompt import _HAS_PROMPT_TOOLKIT
+from .session import SlashSession
+
+__all__ = [
+    "AgentRunSession",
+    "SlashSession",
+    "_HAS_PROMPT_TOOLKIT",
+    "agents_get_command",
+    "agents_run_command",
+    "configure_command",
+    "get_client",
+    "load_config",
+]