glaip-sdk 0.0.5b1__py3-none-any.whl → 0.0.7__py3-none-any.whl

glaip_sdk/cli/display.py

@@ -21,7 +21,7 @@ console = Console()
 
 
 def display_creation_success(
-    resource_type: str, resource_name: str, resource_id: str, **additional_fields
+    resource_type: str, resource_name: str, resource_id: str, **additional_fields: Any
 ) -> Panel:
     """Create standardized success message for resource creation.
 
@@ -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:
@@ -157,9 +158,38 @@ def _normalise_field_value(field: str, value: Any) -> Any:
     return value
 
 
-def handle_json_output(
-    ctx, data: Any = None, error: Exception = None
-) -> None:  # pragma: no cover - formatting covered via integration tests
+def _get_context_object(ctx: Any) -> dict[str, Any]:
+    """Get context object safely."""
+    ctx_obj = getattr(ctx, "obj", {}) if ctx is not None else {}
+    return ctx_obj if isinstance(ctx_obj, dict) else {}
+
+
+def _should_output_json(ctx_obj: dict[str, Any]) -> bool:
+    """Check if output should be in JSON format."""
+    return ctx_obj.get("view") == "json"
+
+
+def _build_error_output_data(error: Exception) -> dict[str, Any]:
+    """Build error output data with additional error details."""
+    output_data = {"error": str(error)}
+
+    # Add additional error details if available
+    if hasattr(error, "status_code"):
+        output_data["status_code"] = error.status_code
+    if hasattr(error, "error_type"):
+        output_data["error_type"] = error.error_type
+    if hasattr(error, "payload"):
+        output_data["details"] = error.payload
+
+    return output_data
+
+
+def _build_success_output_data(data: Any) -> dict[str, Any]:
+    """Build success output data."""
+    return data if data is not None else {"success": True}
+
+
+def handle_json_output(ctx: Any, data: Any = None, error: Exception = None) -> None:
     """Handle JSON output format for CLI commands.
 
     Args:
@@ -167,27 +197,18 @@ def handle_json_output(
         data: Data to output (for successful operations)
         error: Error to output (for failed operations)
     """
-    ctx_obj = getattr(ctx, "obj", {}) if ctx is not None else {}
-    if not isinstance(ctx_obj, dict):
-        ctx_obj = {}
+    ctx_obj = _get_context_object(ctx)
 
-    if ctx_obj.get("view") == "json":
+    if _should_output_json(ctx_obj):
         if error:
-            output_data = {"error": str(error)}
-            # Add additional error details if available
-            if hasattr(error, "status_code"):
-                output_data["status_code"] = error.status_code
-            if hasattr(error, "error_type"):
-                output_data["error_type"] = error.error_type
-            if hasattr(error, "payload"):
-                output_data["details"] = error.payload
+            output_data = _build_error_output_data(error)
         else:
-            output_data = data if data is not None else {"success": True}
+            output_data = _build_success_output_data(data)
 
         click.echo(json.dumps(output_data, indent=2, default=str))
 
 
-def handle_rich_output(ctx, rich_content: Any = None) -> None:
+def handle_rich_output(ctx: Any, rich_content: Any = None) -> None:
     """Handle Rich output format for CLI commands.
 
     Args:

glaip_sdk/cli/io.py

@@ -65,7 +65,7 @@ def export_resource_to_file_with_validation(
         raise click.ClickException(f"Failed to export resource: {e}")
 
 
-def fetch_raw_resource_details(client, resource, resource_type: str):
+def fetch_raw_resource_details(client: Any, resource: Any, resource_type: str) -> Any:
     """Fetch raw resource details directly from API to preserve ALL fields.
 
     Args:

glaip_sdk/cli/main.py

@@ -7,6 +7,7 @@ Authors:
 import os
 import subprocess
 import sys
+from typing import Any
 
 import click
 from rich.console import Console
@@ -41,7 +42,14 @@ from glaip_sdk.rich_components import AIPPanel, AIPTable
 )
 @click.option("--no-tty", is_flag=True, help="Disable TTY renderer")
 @click.pass_context
-def main(ctx, api_url, api_key, timeout, view, no_tty):
+def main(
+    ctx: Any,
+    api_url: str | None,
+    api_key: str | None,
+    timeout: float | None,
+    view: str | None,
+    no_tty: bool,
+) -> None:
     """GL AIP SDK Command Line Interface.
 
     A comprehensive CLI for managing GL AIP resources including
@@ -81,7 +89,7 @@ main.add_command(configure_command)
 
 @main.command()
 @click.pass_context
-def status(ctx):
+def status(ctx: Any) -> None:
     """Show connection status and basic info."""
     config = {}
     try:
@@ -91,7 +99,12 @@ def status(ctx):
         branding = AIPBranding.create_from_sdk(
             sdk_version=_SDK_VERSION, package_name="glaip-sdk"
         )
-        branding.display_status_banner("ready")
+        branding.display_welcome_panel(title="🚀 AIP Status")
+
+        # Show AIP Ready status
+        console.print(
+            f"\n[bold green]✅ AIP - Ready[/bold green] (SDK v{_SDK_VERSION})"
+        )
 
         # Load config from file and merge with context
         file_config = load_config()
@@ -125,6 +138,9 @@ def status(ctx):
                     border_style="red",
                 )
             )
+            console.print(
+                f"\n[bold green]✅ AIP - Ready[/bold green] (SDK v{_SDK_VERSION}) - Configure to connect"
+            )
             sys.exit(1)
 
         # Try to create client
@@ -163,6 +179,11 @@ def status(ctx):
             console.print(table)
 
         except Exception as e:
+            # Show AIP Ready status even if connection fails
+            console.print(
+                f"\n[bold green]✅ AIP - Ready[/bold green] (SDK v{_SDK_VERSION})"
+            )
+
             console.print(
                 AIPPanel(
                     f"[bold yellow]⚠️  Connection established but API call failed[/bold yellow]\n"
@@ -197,7 +218,7 @@ def status(ctx):
 
 
 @main.command()
-def version():
+def version() -> None:
     """Show version information."""
     branding = AIPBranding.create_from_sdk(
         sdk_version=_SDK_VERSION, package_name="glaip-sdk"
@@ -214,7 +235,7 @@ def version():
     is_flag=True,
     help="Force reinstall even if already up-to-date (adds --force-reinstall)",
 )
-def update(check_only: bool, force: bool):
+def update(check_only: bool, force: bool) -> None:
     """Update AIP SDK to the latest version from PyPI."""
     try:
         console = Console()

glaip_sdk/cli/resolution.py

@@ -8,6 +8,7 @@ Authors:
 """
 
 from collections.abc import Callable
+from typing import Any
 
 import click
 
@@ -15,16 +16,16 @@ from glaip_sdk.cli.utils import resolve_resource
 
 
 def resolve_resource_reference(
-    ctx,
-    _client,
+    ctx: Any,
+    _client: Any,
     reference: str,
     resource_type: str,
     get_by_id_func: Callable,
     find_by_name_func: Callable,
     label: str,
-    select: int = None,
+    select: int | None = None,
     interface_preference: str | None = None,
-):
+) -> Any | None:
     """Resolve resource reference (ID or name) with ambiguity handling.
 
     This is a common pattern used across all resource types.