glaip-sdk 0.6.26__py3-none-any.whl → 0.7.1__py3-none-any.whl

glaip_sdk/cli/commands/mcps/__init__.py

@@ -14,8 +14,6 @@ from glaip_sdk.cli.commands.mcps._common import (  # noqa: E402
     console,
     _load_import_ready_payload,
     _merge_import_payload,
-    _assemble_update_data_from_import_payload,
-    _assemble_update_data_from_cli_options,
     _strip_server_only_fields,
     _coerce_cli_string,
     _collect_cli_overrides,
@@ -73,8 +71,6 @@ __all__ = [
     "console",
     "_load_import_ready_payload",
     "_merge_import_payload",
-    "_assemble_update_data_from_import_payload",
-    "_assemble_update_data_from_cli_options",
     "_strip_server_only_fields",
     "_coerce_cli_string",
     "_collect_cli_overrides",

glaip_sdk/cli/commands/mcps/_common.py

@@ -328,66 +328,35 @@ def _handle_cli_error(ctx: Any, error: Exception, operation: str) -> None:
     ctx.exit(1)
 
 
-def _assemble_update_data_from_import_payload(import_payload: dict[str, Any] | None) -> dict[str, Any]:
-    """Assemble update_data dictionary from import payload.
-
-    Args:
-        import_payload: Import payload dictionary or None
-
-    Returns:
-        Dictionary with update fields from import payload
-    """
-    update_data: dict[str, Any] = {}
-    if import_payload:
-        for field in ("name", "transport", "description", "config", "authentication"):
-            if field in import_payload:
-                update_data[field] = import_payload[field]
-    return update_data
-
-
-def _assemble_update_data_from_cli_options(
-    update_data: dict[str, Any],
-    name: str | None,
-    transport: str | None,
-    description: str | None,
+def _parse_and_validate_config_auth(
+    update_dict: dict[str, Any],
     config: str | None,
     auth: str | None,
+    transport: str | None,
     import_payload: dict[str, Any] | None,
     mcp: Any,
-) -> dict[str, Any]:
-    """Build update_data dictionary from CLI options, overriding import values.
+) -> None:
+    """Parse and validate config and auth CLI options, updating dict in-place.
 
     Args:
-        update_data: Existing update_data dictionary (from import payload)
-        name: MCP name option
-        transport: Transport option
-        description: Description option
-        config: Config option
-        auth: Auth option
+        update_dict: Dictionary to update with parsed config/auth
+        config: Config option string
+        auth: Auth option string
+        transport: Transport option for config validation
         import_payload: Import payload dictionary or None
         mcp: Current MCP object
-
-    Returns:
-        Updated dictionary with CLI option values
     """
-    if name is not None:
-        update_data["name"] = name
-    if transport is not None:
-        update_data["transport"] = transport
-    if description is not None:
-        update_data["description"] = description
     if config is not None:
         parsed_config = parse_json_input(config)
         config_transport = _get_config_transport(transport, import_payload, mcp)
-        update_data["config"] = validate_mcp_config_structure(
+        update_dict["config"] = validate_mcp_config_structure(
             parsed_config,
             transport=config_transport,
             source="--config",
         )
     if auth is not None:
         parsed_auth = parse_json_input(auth)
-        update_data["authentication"] = validate_mcp_auth_structure(parsed_auth, source="--auth")
-    return update_data
+        update_dict["authentication"] = validate_mcp_auth_structure(parsed_auth, source="--auth")
 
 
 def _generate_update_preview(mcp: Any, update_data: dict[str, Any], cli_overrides: dict[str, Any]) -> str:

glaip_sdk/cli/commands/mcps/create.py

@@ -111,29 +111,28 @@ def create(
         effective_description = merged_payload.get("description")
         effective_config = merged_payload.get("config") or {}
         effective_auth = merged_payload.get("authentication")
+        mcp_metadata = merged_payload.get("mcp_metadata")
 
         with spinner_context(
             ctx,
             "[bold blue]Creating MCP…[/bold blue]",
             console_override=console,
         ):
+            # Use SDK client method to create MCP
             create_kwargs: dict[str, Any] = {
-                "name": effective_name,
-                "config": effective_config,
                 "transport": effective_transport,
             }
-
-            if effective_description is not None:
-                create_kwargs["description"] = effective_description
-
             if effective_auth:
                 create_kwargs["authentication"] = effective_auth
-
-            mcp_metadata = merged_payload.get("mcp_metadata")
             if mcp_metadata is not None:
                 create_kwargs["mcp_metadata"] = mcp_metadata
 
-            mcp = api_client.mcps.create_mcp(**create_kwargs)
+            mcp = api_client.mcps.create_mcp(
+                name=effective_name,
+                description=effective_description,
+                config=effective_config,
+                **create_kwargs,
+            )
 
         # Handle JSON output
         handle_json_output(ctx, mcp.model_dump())

glaip_sdk/cli/commands/mcps/update.py

@@ -16,11 +16,10 @@ from glaip_sdk.cli.core.context import get_client
 from glaip_sdk.cli.core.rendering import spinner_context
 
 from ._common import (
-    _assemble_update_data_from_cli_options,
-    _assemble_update_data_from_import_payload,
     _handle_cli_error,
     _handle_update_preview_and_confirmation,
     _load_import_ready_payload,
+    _parse_and_validate_config_auth,
     _resolve_mcp,
     _validate_import_payload_fields,
     _validate_update_inputs,
@@ -29,6 +28,49 @@ from ._common import (
 )
 
 
+def _merge_update_kwargs(
+    import_payload: dict[str, Any] | None,
+    name: str | None,
+    transport: str | None,
+    description: str | None,
+    config: str | None,
+    auth: str | None,
+    mcp: Any,
+) -> dict[str, Any]:
+    """Merge import payload and CLI options into kwargs for SDK builder.
+
+    Args:
+        import_payload: Import payload dictionary or None
+        name: MCP name option
+        transport: Transport option
+        description: Description option
+        config: Config option
+        auth: Auth option
+        mcp: Current MCP object
+
+    Returns:
+        Dictionary with merged update kwargs
+    """
+    update_kwargs: dict[str, Any] = {}
+
+    # Start with import payload fields
+    if import_payload:
+        for field in ("name", "transport", "description", "config", "authentication"):
+            if field in import_payload:
+                update_kwargs[field] = import_payload[field]
+
+    # Override with CLI options (CLI takes precedence)
+    if name is not None:
+        update_kwargs["name"] = name
+    if transport is not None:
+        update_kwargs["transport"] = transport
+    if description is not None:
+        update_kwargs["description"] = description
+    _parse_and_validate_config_auth(update_kwargs, config, auth, transport, import_payload, mcp)
+
+    return update_kwargs
+
+
 @mcps_group.command()
 @click.argument("mcp_ref")
 @click.option("--name", help="New MCP name")
@@ -87,7 +129,8 @@ def update(
     Note:
         Must specify either --import OR at least one CLI field.
         CLI options override imported values when both are specified.
-        Uses PATCH for import-based updates, PUT/PATCH for CLI-only updates.
+        Method selection (PATCH vs PUT) is handled automatically by the SDK client
+        based on the fields provided.
 
     \b
     Examples:
@@ -116,28 +159,29 @@ def update(
             if not _validate_import_payload_fields(import_payload):
                 return
 
-        # Build update_data from import payload and CLI options
-        update_data = _assemble_update_data_from_import_payload(import_payload)
-        update_data = _assemble_update_data_from_cli_options(
-            update_data, name, transport, description, config, auth, import_payload, mcp
-        )
+        # Merge import payload and CLI options into kwargs for SDK builder
+        update_kwargs = _merge_update_kwargs(import_payload, name, transport, description, config, auth, mcp)
 
-        if not update_data:
+        if not update_kwargs:
             raise click.ClickException("No update fields specified")
 
+        # Build preview data for confirmation (using the same structure as before)
+        preview_data = update_kwargs.copy()
+
         # Show confirmation preview for import-based updates (unless -y flag)
         if not _handle_update_preview_and_confirmation(
-            import_payload, y, mcp, update_data, name, transport, description, config, auth
+            import_payload, y, mcp, preview_data, name, transport, description, config, auth
         ):
             return
 
-        # Update MCP
+        # Use SDK client method to update MCP
+        # Pass mcp object (not mcp.id) to avoid extra fetch; SDK accepts str | MCP
         with spinner_context(
             ctx,
             "[bold blue]Updating MCP…[/bold blue]",
             console_override=console,
         ):
-            updated_mcp = client.mcps.update_mcp(mcp, **update_data)
+            updated_mcp = client.mcps.update_mcp(mcp, **update_kwargs)
 
         handle_json_output(ctx, updated_mcp.model_dump())
         handle_rich_output(ctx, display_update_success("MCP", updated_mcp.name))

glaip_sdk/cli/commands/tools/create.py

@@ -13,14 +13,14 @@ from typing import Any
 import click
 
 from glaip_sdk.cli.context import get_ctx_value, output_flags
+from glaip_sdk.cli.core.context import get_client, handle_best_effort_check
+from glaip_sdk.cli.core.rendering import spinner_context
 from glaip_sdk.cli.display import (
     display_api_error,
     display_creation_success,
     handle_json_output,
     handle_rich_output,
 )
-from glaip_sdk.cli.core.context import get_client, handle_best_effort_check
-from glaip_sdk.cli.core.rendering import spinner_context
 from glaip_sdk.cli.io import load_resource_from_file_with_validation as load_resource_from_file
 from glaip_sdk.utils.import_export import merge_import_with_cli_args
 

glaip_sdk/cli/slash/accounts_controller.py

@@ -170,7 +170,9 @@ class AccountsController:
         callbacks = AccountsTUICallbacks(switch_account=_switch_in_textual)
         active = next((row["name"] for row in rows if row.get("active")), None)
         try:
-            run_accounts_textual(rows, active_account=active, env_lock=env_lock, callbacks=callbacks)
+            # Inject TUI context for theme support
+            tui_ctx = getattr(self.session, "tui_ctx", None)
+            run_accounts_textual(rows, active_account=active, env_lock=env_lock, callbacks=callbacks, ctx=tui_ctx)
         except Exception as exc:  # pragma: no cover - defensive around Textual failures
             self.console.print(f"[{WARNING_STYLE}]Accounts browser exited unexpectedly: {exc}[/]")
 

glaip_sdk/cli/slash/session.py

@@ -6,6 +6,7 @@ Authors:
 
 from __future__ import annotations
 
+import asyncio
 import importlib
 import os
 import shlex
@@ -51,6 +52,7 @@ from glaip_sdk.cli.slash.prompt import (
     to_formatted_text,
 )
 from glaip_sdk.cli.slash.remote_runs_controller import RemoteRunsController
+from glaip_sdk.cli.slash.tui.context import TUIContext
 from glaip_sdk.cli.transcript import (
     export_cached_transcript,
     load_history_snapshot,
@@ -186,6 +188,7 @@ class SlashSession:
         self._update_notifier = maybe_notify_update
         self._home_hint_shown = False
         self._agent_transcript_ready: dict[str, str] = {}
+        self.tui_ctx: TUIContext | None = None
 
     # ------------------------------------------------------------------
     # Session orchestration
@@ -215,6 +218,22 @@ class SlashSession:
 
     def run(self, initial_commands: Iterable[str] | None = None) -> None:
         """Start the command palette session loop."""
+        # Initialize TUI context asynchronously
+        try:
+            self.tui_ctx = asyncio.run(TUIContext.create())
+        except RuntimeError:
+            try:
+                loop = asyncio.get_event_loop()
+            except RuntimeError:
+                self.tui_ctx = None
+            else:
+                if loop.is_running():
+                    self.tui_ctx = None
+                else:
+                    self.tui_ctx = loop.run_until_complete(TUIContext.create())
+        except Exception:
+            self.tui_ctx = None
+
         ctx_obj = self.ctx.obj if isinstance(self.ctx.obj, dict) else None
         previous_session = None
         if ctx_obj is not None:

glaip_sdk/cli/slash/tui/__init__.py

@@ -1,9 +1,34 @@
 """Textual UI helpers for slash commands."""
 
+from glaip_sdk.cli.slash.tui.clipboard import ClipboardAdapter, ClipboardResult
+from glaip_sdk.cli.slash.tui.context import TUIContext
+from glaip_sdk.cli.slash.tui.keybind_registry import (
+    Keybind,
+    KeybindRegistry,
+    format_key_sequence,
+    parse_key_sequence,
+)
+from glaip_sdk.cli.slash.tui.toast import ToastBus, ToastVariant
 from glaip_sdk.cli.slash.tui.remote_runs_app import (
     RemoteRunsTextualApp,
     RemoteRunsTUICallbacks,
     run_remote_runs_textual,
 )
+from glaip_sdk.cli.slash.tui.terminal import TerminalCapabilities, detect_terminal_background
 
-__all__ = ["RemoteRunsTextualApp", "RemoteRunsTUICallbacks", "run_remote_runs_textual"]
+__all__ = [
+    "TUIContext",
+    "ToastBus",
+    "ToastVariant",
+    "TerminalCapabilities",
+    "detect_terminal_background",
+    "RemoteRunsTextualApp",
+    "RemoteRunsTUICallbacks",
+    "run_remote_runs_textual",
+    "KeybindRegistry",
+    "Keybind",
+    "parse_key_sequence",
+    "format_key_sequence",
+    "ClipboardAdapter",
+    "ClipboardResult",
+]

glaip_sdk/cli/slash/tui/accounts.tcss

@@ -11,7 +11,7 @@
 
 #env-lock {
     padding: 0 1 0 1;
-    color: yellow;
+    color: $warning;
     height: 1;
 }
 
@@ -45,6 +45,7 @@
     padding: 0 1 0 1;
     margin: 0 0 0 0;
     height: 1fr;
+    border: tall $primary;
 }
 
 #status-bar {
@@ -58,11 +59,12 @@
 }
 
 #status {
-    padding: 0 1 0 1;
-    margin: 0;
-    color: cyan;
+    height: 3;
+    padding: 0 1;
+    color: $secondary;
 }
 
+
 .form-label {
     padding: 0 1 0 1;
 }
@@ -73,7 +75,7 @@
 
 #form-status, #confirm-status {
     padding: 0 1;
-    color: yellow;
+    color: $warning;
 }
 
 #form-test {

glaip_sdk/cli/slash/tui/accounts_app.py

@@ -23,7 +23,10 @@ from glaip_sdk.cli.slash.accounts_shared import (
     env_credentials_present,
 )
 from glaip_sdk.cli.slash.tui.background_tasks import BackgroundTaskMixin
+from glaip_sdk.cli.slash.tui.clipboard import ClipboardAdapter
+from glaip_sdk.cli.slash.tui.context import TUIContext
 from glaip_sdk.cli.slash.tui.loading import hide_loading_indicator, show_loading_indicator
+from glaip_sdk.cli.slash.tui.theme.catalog import _BUILTIN_THEMES
 from glaip_sdk.cli.validators import validate_api_key
 from glaip_sdk.utils.validation import validate_url
 
@@ -51,6 +54,13 @@ except Exception:  # pragma: no cover - optional dependency
     LoadingIndicator = None  # type: ignore[assignment]
     ModalScreen = None  # type: ignore[assignment]
     Static = None  # type: ignore[assignment]
+    Theme = None  # type: ignore[assignment]
+
+if App is not None:
+    try:  # pragma: no cover - optional dependency
+        from textual.theme import Theme
+    except Exception:  # pragma: no cover - optional dependency
+        Theme = None  # type: ignore[assignment]
 
 TEXTUAL_SUPPORTED = App is not None and DataTable is not None
 
@@ -201,11 +211,12 @@ def run_accounts_textual(
     active_account: str | None,
     env_lock: bool,
     callbacks: AccountsTUICallbacks,
+    ctx: TUIContext | None = None,
 ) -> None:
     """Launch the Textual accounts browser if dependencies are available."""
     if not TEXTUAL_SUPPORTED:
         return
-    app = AccountsTextualApp(rows, active_account, env_lock, callbacks)
+    app = AccountsTextualApp(rows, active_account, env_lock, callbacks, ctx=ctx)
     app.run()
 
 
@@ -379,16 +390,18 @@ class AccountsTextualApp(BackgroundTaskMixin, _AppBase):  # pragma: no cover - i
 
     CSS_PATH = CSS_FILE_NAME
     BINDINGS = [
-        Binding("enter", "switch_row", "Switch", show=True),
-        Binding("return", "switch_row", "Switch", show=False),
-        Binding("/", "focus_filter", "Filter", show=True),
-        Binding("a", "add_account", "Add", show=True),
-        Binding("e", "edit_account", "Edit", show=True),
-        Binding("d", "delete_account", "Delete", show=True),
+        Binding("enter", "switch_row", "Switch", show=True) if Binding else None,
+        Binding("return", "switch_row", "Switch", show=False) if Binding else None,
+        Binding("/", "focus_filter", "Filter", show=True) if Binding else None,
+        Binding("a", "add_account", "Add", show=True) if Binding else None,
+        Binding("e", "edit_account", "Edit", show=True) if Binding else None,
+        Binding("d", "delete_account", "Delete", show=True) if Binding else None,
+        Binding("c", "copy_account", "Copy", show=True) if Binding else None,
         # Esc clears filter when focused/non-empty; otherwise exits
-        Binding("escape", "clear_or_exit", "Close", priority=True),
-        Binding("q", "app_exit", "Close", priority=True),
+        Binding("escape", "clear_or_exit", "Close", priority=True) if Binding else None,
+        Binding("q", "app_exit", "Close", priority=True) if Binding else None,
     ]
+    BINDINGS = [b for b in BINDINGS if b is not None]
 
     def __init__(
         self,
@@ -396,6 +409,7 @@ class AccountsTextualApp(BackgroundTaskMixin, _AppBase):  # pragma: no cover - i
         active_account: str | None,
         env_lock: bool,
         callbacks: AccountsTUICallbacks,
+        ctx: TUIContext | None = None,
     ) -> None:
         """Initialize the Textual accounts app.
 
@@ -404,6 +418,7 @@ class AccountsTextualApp(BackgroundTaskMixin, _AppBase):  # pragma: no cover - i
             active_account: Name of the currently active account.
             env_lock: Whether environment credentials are locking account switching.
             callbacks: Callbacks for account switching operations.
+            ctx: Shared TUI context.
         """
         super().__init__()
         self._store = get_account_store()
@@ -411,6 +426,7 @@ class AccountsTextualApp(BackgroundTaskMixin, _AppBase):  # pragma: no cover - i
         self._active_account = active_account
         self._env_lock = env_lock
         self._callbacks = callbacks
+        self._ctx = ctx
         self._filter_text: str = ""
         self._is_switching = False
 
@@ -449,6 +465,7 @@ class AccountsTextualApp(BackgroundTaskMixin, _AppBase):  # pragma: no cover - i
 
     def on_mount(self) -> None:
         """Configure table columns and load rows."""
+        self._apply_theme()
         table = self.query_one(ACCOUNTS_TABLE_ID, DataTable)
         table.add_column("Name", width=20)
         table.add_column("API URL", width=40)
@@ -752,6 +769,26 @@ class AccountsTextualApp(BackgroundTaskMixin, _AppBase):  # pragma: no cover - i
             return
         self.push_screen(ConfirmDeleteModal(name), self._on_delete_result)
 
+    def action_copy_account(self) -> None:
+        """Copy selected account name and URL to clipboard."""
+        name = self._get_selected_name()
+        if not name:
+            self._set_status("Select an account to copy.", "yellow")
+            return
+
+        account = self._store.get_account(name)
+        if not account:
+            return
+
+        text = f"Account: {name}\nURL: {account.get('api_url', '')}"
+        adapter = ClipboardAdapter()
+        result = adapter.copy(text)
+
+        if result.success:
+            self._set_status(f"Copied '{name}' to clipboard.", "green")
+        else:
+            self._set_status(f"Copy failed: {result.message}", "red")
+
     def _check_env_lock_hotkey(self) -> bool:
         """Prevent mutations when env credentials are present."""
         if not self._is_env_locked():
@@ -874,3 +911,23 @@ class AccountsTextualApp(BackgroundTaskMixin, _AppBase):  # pragma: no cover - i
         filter_input = self.query_one(FILTER_INPUT_ID, Input)
         clear_btn = self.query_one("#filter-clear", Button)
         clear_btn.display = bool(filter_input.value or self._filter_text)
+
+    def _apply_theme(self) -> None:
+        """Register built-in themes and set the active one from context."""
+        if not self._ctx or not self._ctx.theme or Theme is None:
+            return
+
+        for name, tokens in _BUILTIN_THEMES.items():
+            self.register_theme(
+                Theme(
+                    name=name,
+                    primary=tokens.primary,
+                    secondary=tokens.secondary,
+                    accent=tokens.accent,
+                    warning=tokens.warning,
+                    error=tokens.error,
+                    success=tokens.success,
+                )
+            )
+
+        self.theme = self._ctx.theme.theme_name