glaip-sdk 0.1.4__py3-none-any.whl → 0.2.1__py3-none-any.whl

glaip_sdk/__init__.py

@@ -9,4 +9,4 @@ from glaip_sdk.client import Client
 from glaip_sdk.exceptions import AIPError
 from glaip_sdk.models import MCP, Agent, Tool
 
-__all__ = ["MCP", "AIPError", "Agent", "Client", "Tool", "__version__"]
+__all__ = ["Client", "Agent", "Tool", "MCP", "AIPError", "__version__"]

glaip_sdk/_version.py

@@ -56,6 +56,7 @@ def _try_get_dev_version() -> str | None:
 
 
 def _get_version() -> str:
+    """Return the SDK version from install metadata or fallbacks."""
     # Try installed version first
     installed_version = _try_get_installed_version()
     if installed_version:

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, update
+from glaip_sdk.cli.commands import agents, configure, mcps, models, tools, transcripts, update
 
-__all__ = ["agents", "configure", "mcps", "models", "tools", "update"]
+__all__ = ["agents", "configure", "mcps", "models", "tools", "transcripts", "update"]

glaip_sdk/cli/commands/agents.py

@@ -32,7 +32,7 @@ from glaip_sdk.cli.agent_config import (
 from glaip_sdk.cli.agent_config import (
     sanitize_agent_config_for_cli as sanitize_agent_config,
 )
-from glaip_sdk.cli.context import detect_export_format, get_ctx_value, output_flags
+from glaip_sdk.cli.context import get_ctx_value, output_flags
 from glaip_sdk.cli.display import (
     build_resource_result_data,
     display_agent_run_suggestions,
@@ -44,9 +44,6 @@ from glaip_sdk.cli.display import (
     handle_rich_output,
     print_api_error,
 )
-from glaip_sdk.cli.io import (
-    export_resource_to_file_with_validation as export_resource_to_file,
-)
 from glaip_sdk.cli.io import (
     fetch_raw_resource_details,
 )
@@ -360,6 +357,7 @@ def agents_group() -> None:
     pass
 
 
+# pylint: disable=duplicate-code
 def _resolve_agent(
     ctx: Any,
     client: Any,
@@ -493,8 +491,9 @@ def list_agents(
 @output_flags()
 @click.pass_context
 def get(ctx: Any, agent_ref: str, select: int | None, export: str | None) -> None:
-    """Get agent details.
+    r"""Get agent details.
 
+    \b
     Examples:
         aip agents get my-agent
         aip agents get my-agent --export agent.json    # Exports complete configuration as JSON
@@ -508,35 +507,15 @@ def get(ctx: Any, agent_ref: str, select: int | None, export: str | None) -> Non
 
         # Handle export option
         if export:
-            export_path = Path(export)
-            # Auto-detect format from file extension
-            detected_format = detect_export_format(export_path)
-
-            # Always export comprehensive data - re-fetch agent with full details
-            try:
-                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,
-                    markup_text(f"[{WARNING_STYLE}]⚠️  Could not fetch full agent details: {e}[/]"),
-                )
-                handle_rich_output(
-                    ctx,
-                    markup_text(f"[{WARNING_STYLE}]⚠️  Proceeding with available data[/]"),
-                )
-
-            export_resource_to_file(agent, export_path, detected_format)
-            handle_rich_output(
+            from glaip_sdk.cli.utils import handle_resource_export
+
+            handle_resource_export(
                 ctx,
-                markup_text(
-                    f"[{SUCCESS_STYLE}]✅ Complete agent configuration exported to: {export_path} "
-                    f"(format: {detected_format})[/]"
-                ),
+                agent,
+                Path(export),
+                resource_type="agent",
+                get_by_id_func=client.agents.get_agent_by_id,
+                console_override=console,
             )
 
         # Display full agent details using the standardized helper
@@ -706,10 +685,11 @@ def run(
     files: tuple[str, ...] | None,
     verbose: bool,
 ) -> None:
-    """Run an agent with input text.
+    r"""Run an agent with input text.
 
     Usage: aip agents run <agent_ref> <input_text> [OPTIONS]
 
+    \b
     Examples:
         aip agents run my-agent "Hello world"
         aip agents run agent-123 "Process this data" --timeout 600
@@ -778,11 +758,13 @@ def run(
 
 
 def _running_in_slash_mode(ctx: Any) -> bool:
+    """Return True if the command is executing inside the slash session."""
     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:
+    """Explain the modern alternative to the deprecated --verbose flag."""
     if _running_in_slash_mode(ctx):
         message = (
             "[dim]Tip:[/] Verbose streaming has been retired in the command palette. Run the agent normally and open "
@@ -1037,8 +1019,9 @@ def create(
     timeout: float | None,
     import_file: str | None,
 ) -> None:
-    """Create a new agent.
+    r"""Create a new agent.
 
+    \b
     Examples:
         aip agents create --name "My Agent" --instruction "You are a helpful assistant"
         aip agents create --import agent.json
@@ -1232,8 +1215,9 @@ def update(
     timeout: float | None,
     import_file: str | None,
 ) -> None:
-    """Update an existing agent.
+    r"""Update an existing agent.
 
+    \b
     Examples:
         aip agents update my-agent --instruction "New instruction"
         aip agents update my-agent --import agent.json
@@ -1327,7 +1311,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:
-    """Sync agents with LangFlow server flows.
+    r"""Sync agents with LangFlow server flows.
 
     This command fetches all flows from the configured LangFlow server and
     creates/updates corresponding agents in the platform.
@@ -1336,6 +1320,7 @@ def sync_langflow(ctx: Any, base_url: str | None, api_key: str | None) -> None:
     - Command options (--base-url, --api-key)
     - Environment variables (LANGFLOW_BASE_URL, LANGFLOW_API_KEY)
 
+    \b
     Examples:
         aip agents sync-langflow
         aip agents sync-langflow --base-url https://my-langflow.com --api-key my-key

glaip_sdk/cli/commands/configure.py

@@ -48,25 +48,76 @@ def list_config() -> None:
     _render_config_table(config)
 
 
+CONFIG_VALUE_TYPES: dict[str, str] = {
+    "api_url": "string",
+    "api_key": "string",
+    "timeout": "float",
+    "history_default_limit": "int",
+}
+
+
+def _parse_bool_config(value: str) -> bool:
+    """Parse boolean-like CLI input."""
+    normalized = value.strip().lower()
+    if normalized in {"1", "true", "yes", "on"}:
+        return True
+    if normalized in {"0", "false", "no", "off"}:
+        return False
+    raise click.ClickException("Invalid boolean value. Use one of: true, false, yes, no, 1, 0.")
+
+
+def _parse_int_config(value: str) -> int:
+    """Parse integer CLI input with non-negative enforcement."""
+    try:
+        parsed = int(value, 10)
+    except ValueError as exc:
+        raise click.ClickException("Invalid integer value.") from exc
+    if parsed < 0:
+        raise click.ClickException("Value must be greater than or equal to 0.")
+    return parsed
+
+
+def _parse_float_config(value: str) -> float:
+    """Parse float CLI input with non-negative enforcement."""
+    try:
+        parsed = float(value)
+    except ValueError as exc:
+        raise click.ClickException("Invalid float value.") from exc
+    if parsed < 0:
+        raise click.ClickException("Value must be greater than or equal to 0.")
+    return parsed
+
+
+def _coerce_config_value(key: str, raw_value: str) -> str | bool | int | float:
+    """Convert CLI string values to their target config types."""
+    kind = CONFIG_VALUE_TYPES.get(key, "string")
+    if kind == "bool":
+        return _parse_bool_config(raw_value)
+    if kind == "int":
+        return _parse_int_config(raw_value)
+    if kind == "float":
+        return _parse_float_config(raw_value)
+    return raw_value
+
+
 @config_group.command("set")
 @click.argument("key")
 @click.argument("value")
 def set_config(key: str, value: str) -> None:
     """Set a configuration value."""
-    valid_keys = ["api_url", "api_key"]
+    valid_keys = tuple(CONFIG_VALUE_TYPES.keys())
 
     if key not in valid_keys:
         console.print(f"[{ERROR_STYLE}]Error: Invalid key '{key}'. Valid keys are: {', '.join(valid_keys)}[/]")
         raise click.ClickException(f"Invalid configuration key: {key}")
 
+    coerced_value = _coerce_config_value(key, value)
     config = load_config()
-    config[key] = value
+    config[key] = coerced_value
     save_config(config)
 
-    if key == "api_key":
-        console.print(Text(f"✅ Set {key} = {_mask_api_key(value)}", style=SUCCESS_STYLE))
-    else:
-        console.print(Text(f"✅ Set {key} = {value}", style=SUCCESS_STYLE))
+    display_value = _mask_api_key(coerced_value) if key == "api_key" else str(coerced_value)
+    console.print(Text(f"✅ Set {key} = {display_value}", style=SUCCESS_STYLE))
 
 
 @config_group.command("get")
@@ -169,12 +220,14 @@ def configure_command() -> None:
 
 
 def _mask_api_key(value: str | None) -> str:
+    """Return a redacted API key string suitable for display."""
     if not value:
         return ""
     return "***" + value[-4:] if len(value) > 4 else "***"
 
 
 def _print_missing_config_hint() -> None:
+    """Show guidance when no configuration file exists."""
     hint = command_hint("config configure", slash_command="login")
     if hint:
         console.print(f"[{WARNING_STYLE}]No configuration found.[/] Run {format_command_hint(hint) or hint} to set up.")
@@ -183,6 +236,7 @@ def _print_missing_config_hint() -> None:
 
 
 def _render_config_table(config: dict[str, str]) -> None:
+    """Render the current configuration in a friendly table."""
     table = AIPTable(title=f"{ICON_TOOL} AIP Configuration")
     table.add_column("Setting", style=INFO, width=20)
     table.add_column("Value", style=SUCCESS)
@@ -195,6 +249,7 @@ def _render_config_table(config: dict[str, str]) -> None:
 
 
 def _render_configuration_header() -> None:
+    """Display the interactive configuration heading/banner."""
     branding = AIPBranding.create_from_sdk(sdk_version=sdk_version(), package_name="glaip-sdk")
     heading = "[bold]>_ GDP Labs AI Agents Package (AIP CLI)[/bold]"
     console.print(heading)
@@ -204,6 +259,7 @@ def _render_configuration_header() -> None:
 
 
 def _prompt_configuration_inputs(config: dict[str, str]) -> None:
+    """Interactively prompt for configuration values."""
     console.print("\n[bold]Enter your AIP configuration:[/bold]")
     console.print("(Leave blank to keep current values)")
     console.print("─" * 50)
@@ -213,6 +269,7 @@ def _prompt_configuration_inputs(config: dict[str, str]) -> None:
 
 
 def _prompt_api_url(config: dict[str, str]) -> None:
+    """Ask the user for the API URL, preserving existing values by default."""
     current_url = config.get("api_url", "")
     suffix = f"(current: {current_url})" if current_url else ""
     console.print(f"\n[{ACCENT_STYLE}]AIP API URL[/] {suffix}:")
@@ -224,6 +281,7 @@ def _prompt_api_url(config: dict[str, str]) -> None:
 
 
 def _prompt_api_key(config: dict[str, str]) -> None:
+    """Prompt the user for the API key while masking previous input."""
     current_key_masked = _mask_api_key(config.get("api_key"))
     suffix = f"(current: {current_key_masked})" if current_key_masked else ""
     console.print(f"\n[{ACCENT_STYLE}]AIP API Key[/] {suffix}:")
@@ -233,11 +291,13 @@ def _prompt_api_key(config: dict[str, str]) -> None:
 
 
 def _save_configuration(config: dict[str, str]) -> None:
+    """Persist the collected configuration to disk."""
     save_config(config)
     console.print(Text(f"\n✅ Configuration saved to: {CONFIG_FILE}", style=SUCCESS_STYLE))
 
 
 def _test_and_report_connection(config: dict[str, str]) -> None:
+    """Sanity-check the provided credentials against the backend."""
     console.print("\n🔌 Testing connection...")
     client: Client | None = None
     try:
@@ -270,6 +330,7 @@ def _test_and_report_connection(config: dict[str, str]) -> None:
 
 
 def _print_post_configuration_hints() -> None:
+    """Offer next-step guidance after configuration completes."""
     console.print("\n💡 You can now use AIP CLI commands!")
     hint_status = command_hint("status", slash_command="status")
     if hint_status:

glaip_sdk/cli/commands/mcps.py

@@ -53,7 +53,6 @@ from glaip_sdk.config.constants import (
 )
 from glaip_sdk.icons import ICON_TOOL
 from glaip_sdk.rich_components import AIPPanel
-from glaip_sdk.utils import format_datetime
 from glaip_sdk.utils.import_export import convert_export_to_import_format
 from glaip_sdk.utils.serialization import (
     build_mcp_export_payload,
@@ -299,7 +298,7 @@ def mcps_group() -> None:
     pass
 
 
-def _resolve_mcp(ctx: Any, client: Any, ref: str, select: int | None = None) -> Any | None:
+def _resolve_mcp(ctx: Any, client: Any, ref: str, select: int | None = None) -> Any | None:  # pylint: disable=duplicate-code
     """Resolve MCP reference (ID or name) with ambiguity handling.
 
     Args:
@@ -575,7 +574,7 @@ def create(
     auth: str | None,
     import_file: str | None,
 ) -> None:
-    """Create a new MCP with specified configuration.
+    r"""Create a new MCP with specified configuration.
 
     You can create an MCP by providing all parameters via CLI options, or by
     importing from a file and optionally overriding specific fields.
@@ -593,6 +592,7 @@ def create(
     Raises:
         ClickException: If JSON parsing fails or API request fails
 
+    \b
     Examples:
         Create from CLI options:
             aip mcps create --name my-mcp --transport http --config '{"url": "https://api.example.com"}'
@@ -696,19 +696,15 @@ def _handle_mcp_export(
     detected_format = detect_export_format(export_path)
 
     # Always export comprehensive data - re-fetch with full details
-    try:
-        with spinner_context(
-            ctx,
-            "[bold blue]Fetching complete MCP details…[/bold blue]",
-            console_override=console,
-        ):
-            mcp = client.mcps.get_mcp_by_id(mcp.id)
-    except Exception as e:
-        print_markup(
-            f"[{WARNING_STYLE}]⚠️  Could not fetch full MCP details: {e}[/]",
-            console=console,
-        )
-        print_markup(f"[{WARNING_STYLE}]⚠️  Proceeding with available data[/]", console=console)
+    from glaip_sdk.cli.utils import fetch_resource_for_export
+
+    mcp = fetch_resource_for_export(
+        ctx,
+        mcp,
+        resource_type="MCP",
+        get_by_id_func=client.mcps.get_mcp_by_id,
+        console_override=console,
+    )
 
     # Determine if we should prompt for secrets
     prompt_for_secrets = not no_auth_prompt and sys.stdin.isatty()
@@ -779,11 +775,9 @@ def _display_mcp_details(ctx: Any, client: Any, mcp: Any) -> None:
 
     if raw_mcp_data:
         # Use raw API data - this preserves ALL fields
-        formatted_data = raw_mcp_data.copy()
-        if "created_at" in formatted_data:
-            formatted_data["created_at"] = format_datetime(formatted_data["created_at"])
-        if "updated_at" in formatted_data:
-            formatted_data["updated_at"] = format_datetime(formatted_data["updated_at"])
+        from glaip_sdk.cli.utils import format_datetime_fields
+
+        formatted_data = format_datetime_fields(raw_mcp_data)
 
         output_result(
             ctx,
@@ -832,7 +826,7 @@ def get(
     no_auth_prompt: bool,
     auth_placeholder: str,
 ) -> None:
-    """Get MCP details and optionally export configuration to file.
+    r"""Get MCP details and optionally export configuration to file.
 
     Args:
         ctx: Click context containing output format preferences
@@ -844,6 +838,7 @@ def get(
     Raises:
         ClickException: If MCP not found or export fails
 
+    \b
     Examples:
         aip mcps get my-mcp
         aip mcps get my-mcp --export mcp.json    # Export as JSON
@@ -1050,7 +1045,7 @@ def update(
     import_file: str | None,
     y: bool,
 ) -> None:
-    """Update an existing MCP with new configuration values.
+    r"""Update an existing MCP with new configuration values.
 
     You can update an MCP by providing individual fields via CLI options, or by
     importing from a file and optionally overriding specific fields.
@@ -1075,6 +1070,7 @@ def update(
         CLI options override imported values when both are specified.
         Uses PATCH for import-based updates, PUT/PATCH for CLI-only updates.
 
+    \b
     Examples:
         Update with CLI options:
             aip mcps update my-mcp --name new-name --transport sse

glaip_sdk/cli/commands/tools.py

@@ -4,6 +4,7 @@ Authors:
     Raymond Christopher (raymond.christopher@gdplabs.id)
 """
 
+# pylint: disable=duplicate-code
 import json
 import re
 from pathlib import Path
@@ -19,7 +20,7 @@ from glaip_sdk.branding import (
     SUCCESS_STYLE,
     WARNING_STYLE,
 )
-from glaip_sdk.cli.context import detect_export_format, get_ctx_value, output_flags
+from glaip_sdk.cli.context import get_ctx_value, output_flags
 from glaip_sdk.cli.display import (
     display_api_error,
     display_confirmation_prompt,
@@ -29,9 +30,6 @@ from glaip_sdk.cli.display import (
     handle_json_output,
     handle_rich_output,
 )
-from glaip_sdk.cli.io import (
-    export_resource_to_file_with_validation as export_resource_to_file,
-)
 from glaip_sdk.cli.io import (
     fetch_raw_resource_details,
 )
@@ -48,7 +46,6 @@ from glaip_sdk.cli.utils import (
     spinner_context,
 )
 from glaip_sdk.icons import ICON_TOOL
-from glaip_sdk.utils import format_datetime
 from glaip_sdk.utils.import_export import merge_import_with_cli_args
 
 console = Console()
@@ -60,6 +57,7 @@ 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."""
     return resolve_resource_reference(
@@ -118,6 +116,7 @@ def _check_duplicate_name(client: Any, tool_name: str) -> None:
 
 
 def _parse_tags(tags: str | None) -> list[str]:
+    """Return a cleaned list of tag strings from a comma-separated input."""
     return [t.strip() for t in (tags.split(",") if tags else []) if t.strip()]
 
 
@@ -259,8 +258,9 @@ def create(
     tags: tuple[str, ...] | None,
     import_file: str | None,
 ) -> None:
-    """Create a new tool.
+    r"""Create a new tool.
 
+    \b
     Examples:
         aip tools create tool.py  # Create from file
         aip tools create --import tool.json  # Create from exported configuration
@@ -325,8 +325,9 @@ def create(
 @output_flags()
 @click.pass_context
 def get(ctx: Any, tool_ref: str, select: int | None, export: str | None) -> None:
-    """Get tool details.
+    r"""Get tool details.
 
+    \b
     Examples:
         aip tools get my-tool
         aip tools get my-tool --export tool.json    # Exports complete configuration as JSON
@@ -340,38 +341,15 @@ def get(ctx: Any, tool_ref: str, select: int | None, export: str | None) -> None
 
         # Handle export option
         if export:
-            export_path = Path(export)
-            # Auto-detect format from file extension
-            detected_format = detect_export_format(export_path)
-
-            # Always export comprehensive data - re-fetch tool with full details if needed
-            try:
-                with spinner_context(
-                    ctx,
-                    "[bold blue]Fetching complete tool details…[/bold blue]",
-                    console_override=console,
-                ):
-                    tool = client.get_tool_by_id(tool.id)
-            except Exception as e:
-                print_markup(
-                    f"[{WARNING_STYLE}]⚠️  Could not fetch full tool details: {e}[/]",
-                    console=console,
-                )
-                print_markup(
-                    f"[{WARNING_STYLE}]⚠️  Proceeding with available data[/]",
-                    console=console,
-                )
+            from glaip_sdk.cli.utils import handle_resource_export
 
-            with spinner_context(
+            handle_resource_export(
                 ctx,
-                "[bold blue]Exporting tool configuration…[/bold blue]",
+                tool,
+                Path(export),
+                resource_type="tool",
+                get_by_id_func=client.get_tool_by_id,
                 console_override=console,
-            ):
-                export_resource_to_file(tool, export_path, detected_format)
-            print_markup(
-                f"[{SUCCESS_STYLE}]✅ Complete tool configuration exported to: {export_path} "
-                f"(format: {detected_format})[/]",
-                console=console,
             )
 
         # Try to fetch raw API data first to preserve ALL fields
@@ -385,11 +363,9 @@ 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)
-            formatted_data = raw_tool_data.copy()
-            if "created_at" in formatted_data:
-                formatted_data["created_at"] = format_datetime(formatted_data["created_at"])
-            if "updated_at" in formatted_data:
-                formatted_data["updated_at"] = format_datetime(formatted_data["updated_at"])
+            from glaip_sdk.cli.utils import format_datetime_fields
+
+            formatted_data = format_datetime_fields(raw_tool_data)
 
             # Display using output_result with raw data
             output_result(