glaip-sdk 0.2.2__py3-none-any.whl → 0.4.0__py3-none-any.whl

glaip_sdk/cli/commands/tools.py

@@ -4,7 +4,6 @@ Authors:
     Raymond Christopher (raymond.christopher@gdplabs.id)
 """
 
-# pylint: disable=duplicate-code
 import json
 import re
 from pathlib import Path
@@ -40,7 +39,10 @@ from glaip_sdk.cli.resolution import resolve_resource_reference
 from glaip_sdk.cli.rich_helpers import markup_text, print_markup
 from glaip_sdk.cli.utils import (
     coerce_to_row,
+    format_datetime_fields,
     get_client,
+    handle_best_effort_check,
+    handle_resource_export,
     output_list,
     output_result,
     spinner_context,
@@ -57,16 +59,32 @@ def tools_group() -> None:
     pass
 
 
-# pylint: disable=duplicate-code
 def _resolve_tool(ctx: Any, client: Any, ref: str, select: int | None = None) -> Any | None:
-    """Resolve tool reference (ID or name) with ambiguity handling."""
+    """Resolve a tool by ID or name, handling ambiguous matches interactively.
+
+    This function provides tool-specific resolution logic. It uses
+    resolve_resource_reference to find tools by UUID or name, with interactive
+    selection when multiple matches are found.
+
+    Args:
+        ctx: Click context for CLI operations.
+        client: API client instance.
+        ref: Tool reference (UUID string or name).
+        select: Pre-selected index for non-interactive mode (1-based).
+
+    Returns:
+        Tool object if found, None otherwise.
+    """
+    # Configure tool-specific resolution with standard fuzzy matching
+    get_by_id = client.get_tool
+    find_by_name = client.find_tools
     return resolve_resource_reference(
         ctx,
         client,
         ref,
         "tool",
-        client.get_tool,
-        client.find_tools,
+        get_by_id,
+        find_by_name,
         "Tool",
         select=select,
     )
@@ -100,19 +118,16 @@ def _validate_name_match(provided: str | None, internal: str) -> str:
 
 def _check_duplicate_name(client: Any, tool_name: str) -> None:
     """Raise if a tool with the same name already exists."""
-    try:
+
+    def _check_duplicate() -> None:
         existing = client.find_tools(name=tool_name)
         if existing:
             raise click.ClickException(
                 f"A tool named '{tool_name}' already exists. "
                 "Please change your plugin's 'name' to a unique value, then re-run."
             )
-    except click.ClickException:
-        # Re-raise ClickException (intended error)
-        raise
-    except Exception:
-        # Non-fatal: best-effort duplicate check for other errors
-        pass
+
+    handle_best_effort_check(_check_duplicate)
 
 
 def _parse_tags(tags: str | None) -> list[str]:
@@ -349,8 +364,6 @@ def get(ctx: Any, tool_ref: str, select: int | None, export: str | None) -> None
 
         # Handle export option
         if export:
-            from glaip_sdk.cli.utils import handle_resource_export
-
             handle_resource_export(
                 ctx,
                 tool,
@@ -371,8 +384,6 @@ def get(ctx: Any, tool_ref: str, select: int | None, export: str | None) -> None
         if raw_tool_data:
             # Use raw API data - this preserves ALL fields
             # Format dates for better display (minimal postprocessing)
-            from glaip_sdk.cli.utils import format_datetime_fields
-
             formatted_data = format_datetime_fields(raw_tool_data)
 
             # Display using output_result with raw data

glaip_sdk/cli/commands/transcripts.py

@@ -37,7 +37,7 @@ from glaip_sdk.cli.transcript.viewer import ViewerContext, run_viewer_session
 from glaip_sdk.cli.utils import format_size, get_ctx_value, parse_json_line
 from glaip_sdk.rich_components import AIPTable
 from glaip_sdk.utils.rendering.renderer.debug import render_debug_event
-from glaip_sdk.utils.rendering.renderer.panels import create_final_panel
+from glaip_sdk.utils.rendering.layout.panels import create_final_panel
 
 console = Console()
 

glaip_sdk/cli/constants.py

@@ -33,3 +33,6 @@ UPDATE_CHECK_ENABLED = True
 
 # Agent instruction preview defaults
 DEFAULT_AGENT_INSTRUCTION_PREVIEW_LIMIT = 800
+
+# Remote runs defaults
+DEFAULT_REMOTE_RUNS_PAGE_LIMIT = 20

glaip_sdk/cli/display.py

@@ -17,7 +17,7 @@ from rich.text import Text
 
 from glaip_sdk.branding import ERROR_STYLE, SUCCESS, SUCCESS_STYLE, WARNING_STYLE
 from glaip_sdk.cli.rich_helpers import markup_text
-from glaip_sdk.cli.utils import command_hint, format_command_hint, in_slash_mode
+from glaip_sdk.cli.hints import command_hint, format_command_hint, in_slash_mode
 from glaip_sdk.icons import ICON_AGENT, ICON_TOOL
 from glaip_sdk.rich_components import AIPPanel
 

glaip_sdk/cli/hints.py

@@ -0,0 +1,58 @@
+"""Helpers for formatting CLI/slash command hints.
+
+Authors:
+    Raymond Christopher (raymond.christopher@gdplabs.id)
+"""
+
+from __future__ import annotations
+
+
+import click
+
+from glaip_sdk.branding import HINT_COMMAND_STYLE, HINT_DESCRIPTION_COLOR
+
+
+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."""
+    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 format_command_hint(command: str | None, description: str | None = None) -> str | None:
+    """Return a Rich markup string that highlights a command hint."""
+    if not command:
+        return None
+
+    highlighted = f"[{HINT_COMMAND_STYLE}]{command}[/]"
+    if description:
+        highlighted += f"  [{HINT_DESCRIPTION_COLOR}]{description}[/{HINT_DESCRIPTION_COLOR}]"
+    return highlighted

glaip_sdk/cli/io.py

@@ -7,6 +7,7 @@ Authors:
     Raymond Christopher (raymond.christopher@gdplabs.id)
 """
 
+from importlib import import_module
 from pathlib import Path
 from typing import TYPE_CHECKING, Any
 
@@ -25,9 +26,11 @@ if TYPE_CHECKING:  # pragma: no cover - typing-only imports
 
 def _create_console() -> "Console":
     """Return a Console instance (lazy import for easier testing)."""
-    from rich.console import Console  # Local import for test patching
-
-    return Console()
+    try:
+        console_module = import_module("rich.console")
+    except ImportError as exc:  # pragma: no cover - optional dependency missing
+        raise RuntimeError("Rich Console is not available") from exc
+    return console_module.Console()
 
 
 def load_resource_from_file_with_validation(file_path: Path, resource_type: str) -> dict[str, Any]:

glaip_sdk/cli/main.py

@@ -34,11 +34,12 @@ 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.cli.commands.transcripts import transcripts_group
-from glaip_sdk.cli.commands.update import update_command
+from glaip_sdk.cli.commands.update import _build_upgrade_command, update_command
 from glaip_sdk.cli.config import load_config
 from glaip_sdk.cli.transcript import get_transcript_cache_stats
 from glaip_sdk.cli.update_notifier import maybe_notify_update
-from glaip_sdk.cli.utils import format_size, in_slash_mode, sdk_version, spinner_context, update_spinner
+from glaip_sdk.cli.hints import in_slash_mode
+from glaip_sdk.cli.utils import format_size, sdk_version, spinner_context, update_spinner
 from glaip_sdk.config.constants import (
     DEFAULT_AGENT_RUN_TIMEOUT,
 )
@@ -418,8 +419,6 @@ def update(check_only: bool, force: bool) -> None:
 
         # Update using pip
         try:
-            from glaip_sdk.cli.commands.update import _build_upgrade_command
-
             cmd = list(_build_upgrade_command(include_prerelease=False))
             # Replace package name with "glaip-sdk" (main.py uses different name)
             cmd[-1] = "glaip-sdk"

glaip_sdk/cli/slash/agent_session.py

@@ -16,7 +16,8 @@ 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.constants import DEFAULT_AGENT_INSTRUCTION_PREVIEW_LIMIT
 from glaip_sdk.cli.slash.prompt import _HAS_PROMPT_TOOLKIT, FormattedText
-from glaip_sdk.cli.utils import format_command_hint
+from glaip_sdk.cli.hints import format_command_hint
+from glaip_sdk.cli.utils import bind_slash_session_context
 
 if TYPE_CHECKING:  # pragma: no cover - type checking only
     from glaip_sdk.cli.slash.session import SlashSession
@@ -41,6 +42,7 @@ class AgentRunSession:
         self._contextual_completion_help: dict[str, str] = {
             "details": "Show this agent's configuration (+ expands prompt).",
             "help": "Display this context-aware menu.",
+            "runs": "✨ NEW · Browse remote run history for this agent.",
             "exit": "Return to the command palette.",
             "q": "Return to the command palette.",
         }
@@ -252,19 +254,8 @@ class AgentRunSession:
     @contextmanager
     def _bind_session_context(self) -> Any:
         """Temporarily attach this slash session to the Click context."""
-        ctx_obj = getattr(self.session.ctx, "obj", None)
-        has_context = isinstance(ctx_obj, dict)
-        previous_session = ctx_obj.get("_slash_session") if has_context else None
-        if has_context:
-            ctx_obj["_slash_session"] = self.session
-        try:
+        with bind_slash_session_context(self.session.ctx, self.session):
             yield
-        finally:
-            if has_context:
-                if previous_session is None:
-                    ctx_obj.pop("_slash_session", None)
-                else:
-                    ctx_obj["_slash_session"] = previous_session
 
     def _run_agent(self, agent_id: str, message: str) -> None:
         """Execute the agents run command for the active agent."""

glaip_sdk/cli/slash/prompt.py

@@ -177,8 +177,11 @@ def _iter_command_completions(
         return []
 
     commands = sorted(session._unique_commands.values(), key=lambda c: c.name)
+    agent_context = bool(getattr(session, "_current_agent", None))
 
     for cmd in commands:
+        if getattr(cmd, "agent_only", False) and not agent_context:
+            continue
         yield from _generate_command_completions(cmd, prefix, text, seen)