lean-explore 0.3.0__py3-none-any.whl → 1.0.1__py3-none-any.whl

lean_explore/cli/main.py

@@ -1,599 +1,87 @@
-# src/lean_explore/cli/main.py
-
 """Command-Line Interface for Lean Explore.
 
-Provides commands to configure the CLI, search for Lean statement groups
-via the remote API, interact with AI agents, manage local data, and other utilities.
+Provides commands to search for Lean declarations via the remote API,
+interact with AI agents, and manage local data.
 """
 
-import subprocess  # For running the MCP server
-import sys  # For sys.executable
-import textwrap
-from typing import List, Optional
+import asyncio
+import logging
+import os
+import subprocess
+import sys
 
-import httpx
 import typer
 from rich.console import Console
-from rich.panel import Panel
-from rich.table import Table
 
-from lean_explore.api.client import Client as APIClient
-from lean_explore.cli import (
-    config_utils,
-    data_commands,  # For data management subcommands
-)
-from lean_explore.shared.models.api import APISearchResponse
+from lean_explore.api import ApiClient
+from lean_explore.cli import data_commands
+from lean_explore.cli.display import display_search_results
 
-# Import the specific command function and its async wrapper from agent.py
-from .agent import agent_chat_command
-from .agent import typer_async as agent_typer_async
+logger = logging.getLogger(__name__)
 
-# Initialize Typer app and Rich console
 app = typer.Typer(
-    name="leanexplore",
+    name="lean-explore",
     help="A CLI tool to explore and search Lean mathematical libraries.",
     add_completion=False,
-    rich_markup_mode="markdown",  # Enables rich markup in help text
+    rich_markup_mode="markdown",
 )
-configure_app = typer.Typer(
-    name="configure", help="Configure leanexplore CLI settings."
-)
-app.add_typer(configure_app)
 
 mcp_app = typer.Typer(
     name="mcp", help="Manage and run the Model Context Protocol (MCP) server."
 )
 app.add_typer(mcp_app)
 
-# Add the data_commands.app as a subcommand group named "data"
 app.add_typer(
     data_commands.app,
     name="data",
     help="Manage local data toolchains.",
 )
 
-# Register the agent_chat_command directly on the main app as "chat"
-# The agent_chat_command is already decorated with @typer_async in agent.py
-app.command("chat", help="Interact with an AI agent using Lean Explore tools.")(
-    agent_chat_command
-)
-
-
-console = Console()
-error_console = Console(stderr=True)
-
-# Content width for panels.
-PANEL_CONTENT_WIDTH = 80
-
-
-@configure_app.command("api-key")
-def configure_lean_explore_api_key(
-    api_key: Optional[str] = typer.Option(
-        None,
-        prompt="Please enter your Lean Explore API key",
-        help="Your personal API key for accessing Lean Explore services.",
-        hide_input=True,
-        confirmation_prompt=True,
-    ),
-):
-    """Configure and save your Lean Explore API key.
-
-    Args:
-        api_key: The API key string to save. Prompts if not provided.
-    """
-    if not api_key:
-        error_console.print(
-            "[bold red]Lean Explore API key cannot be empty.[/bold red]"
-        )
-        raise typer.Abort()
-
-    if config_utils.save_api_key(api_key):
-        config_path = config_utils.get_config_file_path()
-        console.print(
-            f"[bold green]Lean Explore API key saved successfully to: "
-            f"{config_path}[/bold green]"
-        )
-    else:
-        error_console.print(
-            "[bold red]Failed to save Lean Explore API key. "
-            "Check logs or permissions.[/bold red]"
-        )
-        raise typer.Abort()
-
-
-@configure_app.command("openai-key")
-def configure_openai_api_key(
-    api_key: Optional[str] = typer.Option(
-        None,
-        prompt="Please enter your OpenAI API key",
-        help="Your personal API key for OpenAI services (e.g., GPT-4).",
-        hide_input=True,
-        confirmation_prompt=True,
-    ),
-):
-    """Configure and save your OpenAI API key.
-
-    This key is used by agent functionalities that leverage OpenAI models.
-
-    Args:
-        api_key: The OpenAI API key string to save. Prompts if not provided.
-    """
-    if not api_key:
-        error_console.print("[bold red]OpenAI API key cannot be empty.[/bold red]")
-        raise typer.Abort()
-
-    if config_utils.save_openai_api_key(api_key):
-        config_path = config_utils.get_config_file_path()
-        console.print(
-            f"[bold green]OpenAI API key saved successfully to: "
-            f"{config_path}[/bold green]"
-        )
-    else:
-        error_console.print(
-            "[bold red]Failed to save OpenAI API key. "
-            "Check logs or permissions.[/bold red]"
-        )
-        raise typer.Abort()
-
-
-def _get_api_client() -> Optional[APIClient]:
-    """Loads Lean Explore API key and initializes the APIClient.
-
-    Returns:
-        Optional[APIClient]: APIClient instance if key is found, None otherwise.
-    """
-    api_key = config_utils.load_api_key()
-    if not api_key:
-        config_path = config_utils.get_config_file_path()
-        error_console.print(
-            "[bold yellow]Lean Explore API key not configured. Please run:"
-            "[/bold yellow]\n"
-            f"  `leanexplore configure api-key`\n"
-            f"Your API key will be stored in: {config_path}"
-        )
-        return None
-    return APIClient(api_key=api_key)
-
 
-def _format_text_for_fixed_panel(text_content: Optional[str], width: int) -> str:
-    """Wraps text and pads lines to ensure fixed content width for a Panel.
+def _get_console(use_stderr: bool = False) -> Console:
+    """Create a Rich console instance for output.
 
     Args:
-        text_content: The text content to wrap and pad.
-        width: The target width for text wrapping and padding.
+        use_stderr: If True, output to stderr instead of stdout.
 
     Returns:
-        A string with wrapped and padded text suitable for fixed-width display.
+        A configured Console instance.
     """
-    if not text_content:
-        return " " * width
-
-    final_output_lines = []
-    paragraphs = text_content.split("\n\n")
-
-    for i, paragraph in enumerate(paragraphs):
-        if not paragraph.strip() and i < len(paragraphs) - 1:
-            final_output_lines.append(" " * width)
-            continue
-
-        lines_in_paragraph = paragraph.splitlines()
-        if not lines_in_paragraph and paragraph.strip() == "":
-            final_output_lines.append(" " * width)
-            continue
-        if not lines_in_paragraph and not paragraph:
-            final_output_lines.append(" " * width)
-            continue
-
-        for line in lines_in_paragraph:
-            if not line.strip():
-                final_output_lines.append(" " * width)
-                continue
-
-            wrapped_segments = textwrap.wrap(
-                line,
-                width=width,
-                replace_whitespace=True,
-                drop_whitespace=True,
-                break_long_words=True,
-                break_on_hyphens=True,
-            )
-            if not wrapped_segments:
-                final_output_lines.append(" " * width)
-            else:
-                for segment in wrapped_segments:
-                    final_output_lines.append(segment.ljust(width))
-
-        if i < len(paragraphs) - 1 and (
-            paragraph.strip() or (not paragraph.strip() and not lines_in_paragraph)
-        ):
-            # Add a blank padded line between paragraphs
-            final_output_lines.append(" " * width)
-
-    if not final_output_lines and text_content.strip():
-        # Fallback for content that becomes empty after processing but was not initially
-        return " " * width
-
-    return "\n".join(final_output_lines)
-
-
-def _display_search_results(response: APISearchResponse, display_limit: int = 5):
-    """Displays search results using fixed-width Panels for each item.
-
-    Args:
-        response: The APISearchResponse object from the backend.
-        display_limit: The maximum number of individual results to display in detail.
-    """
-    console.print(
-        Panel(
-            f"[bold cyan]Search Query:[/bold cyan] {response.query}",
-            expand=False,
-            border_style="dim",
-        )
-    )
-    if response.packages_applied:
-        console.print(
-            f"[bold cyan]Package Filters:[/bold cyan] "
-            f"{', '.join(response.packages_applied)}"
-        )
-
-    num_results_to_show = min(len(response.results), display_limit)
-    console.print(
-        f"Showing {num_results_to_show} of {response.count} "
-        f"(out of {response.total_candidates_considered} candidates considered by "
-        f"server). Time: {response.processing_time_ms}ms"
-    )
-
-    if not response.results:
-        console.print("[yellow]No results found.[/yellow]")
-        return
-
-    console.print("")  # Adds a blank line for spacing
-
-    for i, item in enumerate(response.results):
-        if i >= display_limit:
-            break
-
-        lean_name = (
-            item.primary_declaration.lean_name if item.primary_declaration else "N/A"
-        )
-
-        console.rule(f"[bold]Result {i + 1}[/bold]", style="dim")
-        console.print(f"[bold cyan]ID:[/bold cyan] [dim]{item.id}[/dim]")
-        console.print(f"[bold cyan]Name:[/bold cyan] {lean_name}")
-        console.print(
-            f"[bold cyan]File:[/bold cyan] [green]{item.source_file}[/green]:"
-            f"[dim]{item.range_start_line}[/dim]"
-        )
-
-        code_to_display = item.display_statement_text or item.statement_text
-        if code_to_display:
-            formatted_code = _format_text_for_fixed_panel(
-                code_to_display, PANEL_CONTENT_WIDTH
-            )
-            console.print(
-                Panel(
-                    formatted_code,
-                    title="[bold green]Code[/bold green]",
-                    border_style="green",
-                    expand=False,
-                    padding=(0, 1),
-                )
-            )
-
-        if item.docstring:
-            formatted_doc = _format_text_for_fixed_panel(
-                item.docstring, PANEL_CONTENT_WIDTH
-            )
-            console.print(
-                Panel(
-                    formatted_doc,
-                    title="[bold blue]Docstring[/bold blue]",
-                    border_style="blue",
-                    expand=False,
-                    padding=(0, 1),
-                )
-            )
-
-        if item.informal_description:
-            formatted_informal = _format_text_for_fixed_panel(
-                item.informal_description, PANEL_CONTENT_WIDTH
-            )
-            console.print(
-                Panel(
-                    formatted_informal,
-                    title="[bold magenta]Informal Description[/bold magenta]",
-                    border_style="magenta",
-                    expand=False,
-                    padding=(0, 1),
-                )
-            )
-        elif not item.docstring and not code_to_display:
-            console.print(
-                "[dim]No further textual details (docstring, informal description, "
-                "code) available for this item.[/dim]"
-            )
-
-        if i < num_results_to_show - 1:  # Add spacing between items
-            console.print("")
-
-    console.rule(style="dim")
-    if len(response.results) > num_results_to_show:
-        console.print(
-            f"...and {len(response.results) - num_results_to_show} more results "
-            "received from server but not shown due to limit."
-        )
-    elif response.count > len(
-        response.results
-    ):  # Should be total_candidates_considered
-        console.print(
-            f"...and {response.total_candidates_considered - len(response.results)} "
-            "more results available "
-            "on server."
-        )
+    return Console(stderr=use_stderr)
 
 
 @app.command("search")
-@agent_typer_async
-async def search_command(
+def search_command(
     query_string: str = typer.Argument(..., help="The search query string."),
-    package: Optional[List[str]] = typer.Option(
-        None,
-        "--package",
-        "-p",
-        help="Filter by package name(s). Can be used multiple times.",
-    ),
     limit: int = typer.Option(
         5, "--limit", "-n", help="Number of search results to display."
     ),
-):
-    """Search for Lean statement groups using the Lean Explore API.
-
-    Args:
-        query_string: The natural language query to search for.
-        package: An optional list of package names to filter results by.
-        limit: The maximum number of search results to display to the user.
-    """
-    client = _get_api_client()
-    if not client:
-        raise typer.Exit(code=1)
-
-    console.print(f"Searching for: '{query_string}'...")
-    try:
-        response = await client.search(query=query_string, package_filters=package)
-        _display_search_results(response, display_limit=limit)
-    except httpx.HTTPStatusError as e:
-        if e.response.status_code == 401:
-            error_console.print(
-                f"[bold red]API Error {e.response.status_code}: Unauthorized. "
-                "Your API key might be invalid or expired.[/bold red]"
-            )
-            error_console.print(
-                "Please reconfigure your API key using: `leanexplore configure api-key`"
-            )
-        else:
-            try:
-                error_detail = e.response.json().get("detail", e.response.text)
-                error_console.print(
-                    f"[bold red]API Error {e.response.status_code}: "
-                    f"{error_detail}[/bold red]"
-                )
-            except Exception:
-                error_console.print(
-                    f"[bold red]API Error {e.response.status_code}: "
-                    f"{e.response.text}[/bold red]"
-                )
-        raise typer.Exit(code=1)
-    except httpx.RequestError as e:
-        error_console.print(
-            f"[bold red]Network Error: Could not connect to the API. {e}[/bold red]"
-        )
-        raise typer.Exit(code=1)
-    except Exception as e:
-        error_console.print(f"[bold red]An unexpected error occurred: {e}[/bold red]")
-        raise typer.Exit(code=1)
-
-
-@app.command("get")
-@agent_typer_async
-async def get_by_id_command(
-    group_id: int = typer.Argument(
-        ..., help="The ID of the statement group to retrieve."
+    packages: list[str] | None = typer.Option(
+        None, "--package", "-p", help="Filter by package (e.g., -p Mathlib -p Std)."
     ),
 ):
-    """Get detailed information about a specific statement group by its ID.
-
-    Args:
-        group_id: The unique integer identifier of the statement group.
-    """
-    client = _get_api_client()
-    if not client:
-        raise typer.Exit(code=1)
+    """Search for Lean declarations using the Lean Explore API."""
+    asyncio.run(_search_async(query_string, limit, packages))
 
-    console.print(f"Fetching statement group ID: {group_id}...")
-    try:
-        item = await client.get_by_id(group_id)
-        if item:
-            console.print(
-                Panel(
-                    f"[bold green]Statement Group ID: {item.id}[/bold green]",
-                    expand=False,
-                    border_style="dim",
-                )
-            )
-            lean_name = (
-                item.primary_declaration.lean_name
-                if item.primary_declaration
-                else "N/A"
-            )
-            console.print(f"  [bold cyan]Lean Name:[/bold cyan] {lean_name}")
-            console.print(
-                f"  [bold cyan]Source File:[/bold cyan] "
-                f"[green]{item.source_file}[/green]:"
-                f"[dim]{item.range_start_line}[/dim]"
-            )
 
-            if item.statement_text:
-                formatted_stmt_text = _format_text_for_fixed_panel(
-                    item.statement_text, PANEL_CONTENT_WIDTH
-                )
-                console.print(
-                    Panel(
-                        formatted_stmt_text,
-                        title="[bold green]Code[/bold green]",
-                        border_style="green",
-                        expand=False,
-                        padding=(0, 1),
-                    )
-                )
-
-            if item.docstring:
-                formatted_docstring = _format_text_for_fixed_panel(
-                    item.docstring, PANEL_CONTENT_WIDTH
-                )
-                console.print(
-                    Panel(
-                        formatted_docstring,
-                        title="[bold blue]Docstring[/bold blue]",
-                        border_style="blue",
-                        expand=False,
-                        padding=(0, 1),
-                    )
-                )
-
-            if item.informal_description:
-                formatted_informal = _format_text_for_fixed_panel(
-                    item.informal_description, PANEL_CONTENT_WIDTH
-                )
-                console.print(
-                    Panel(
-                        formatted_informal,
-                        title="[bold magenta]Informal Description[/bold magenta]",
-                        border_style="magenta",
-                        expand=False,
-                        padding=(0, 1),
-                    )
-                )
-
-        else:
-            error_console.print(  # Changed to error_console for error/warning message
-                f"[yellow]Statement group with ID {group_id} not found.[/yellow]"
-            )
+async def _search_async(
+    query_string: str, limit: int, packages: list[str] | None
+) -> None:
+    """Async implementation of search command."""
+    console = _get_console()
+    error_console = _get_console(use_stderr=True)
 
-    except httpx.HTTPStatusError as e:
-        if e.response.status_code == 401:
-            error_console.print(
-                f"[bold red]API Error {e.response.status_code}: Unauthorized."
-                " Your API key might be invalid or expired.[/bold red]"
-            )
-        else:
-            try:
-                error_detail = e.response.json().get("detail", e.response.text)
-                error_console.print(
-                    f"[bold red]API Error {e.response.status_code}: "
-                    f"{error_detail}[/bold red]"
-                )
-            except Exception:
-                error_console.print(
-                    f"[bold red]API Error {e.response.status_code}: "
-                    f"{e.response.text}[/bold red]"
-                )
-        raise typer.Exit(code=1)
-    except httpx.RequestError as e:
-        error_console.print(
-            f"[bold red]Network Error: Could not connect to the API. {e}[/bold red]"
-        )
-        raise typer.Exit(code=1)
-    except Exception as e:
-        error_console.print(f"[bold red]An unexpected error occurred: {e}[/bold red]")
-        raise typer.Exit(code=1)
-
-
-@app.command("dependencies")
-@agent_typer_async
-async def get_dependencies_command(
-    group_id: int = typer.Argument(
-        ..., help="The ID of the statement group to get dependencies for."
-    ),
-):
-    """Get dependencies (citations) for a specific statement group by its ID.
-
-    Args:
-        group_id: The unique integer identifier of the statement group.
-    """
-    client = _get_api_client()
-    if not client:
-        raise typer.Exit(code=1)
-
-    console.print(f"Fetching dependencies for statement group ID: {group_id}...")
     try:
-        response = await client.get_dependencies(group_id)
-        if response:
-            console.print(
-                Panel(
-                    f"[bold green]Citations for Statement Group ID: "
-                    f"{response.source_group_id}[/bold green]",
-                    expand=False,
-                    border_style="dim",
-                )
-            )
-            console.print(f"Found {response.count} direct citations.")
-
-            if response.citations:
-                citation_table = Table(show_header=True, header_style="bold magenta")
-                citation_table.add_column("ID", style="dim", width=6)
-                citation_table.add_column("Cited Lean Name", width=40)
-                citation_table.add_column("File", style="green")
-                citation_table.add_column("Line", style="dim")
-
-                for item in response.citations:
-                    lean_name = (
-                        item.primary_declaration.lean_name
-                        if item.primary_declaration
-                        else "N/A"
-                    )
-                    citation_table.add_row(
-                        str(item.id),
-                        lean_name,
-                        item.source_file,
-                        str(item.range_start_line),
-                    )
-                console.print(citation_table)
-            else:
-                console.print("[yellow]No citations found for this group.[/yellow]")
-        else:
-            error_console.print(  # Changed to error_console for error/warning message
-                f"[yellow]Statement group with ID {group_id} not found or no "
-                "citations data available.[/yellow]"
-            )
-
-    except httpx.HTTPStatusError as e:
-        if e.response.status_code == 401:
-            error_console.print(
-                f"[bold red]API Error {e.response.status_code}: Unauthorized."
-                " Your API key might be invalid or expired.[/bold red]"
-            )
-        else:
-            try:
-                error_detail = e.response.json().get("detail", e.response.text)
-                error_console.print(
-                    f"[bold red]API Error {e.response.status_code}: "
-                    f"{error_detail}[/bold red]"
-                )
-            except Exception:
-                error_console.print(
-                    f"[bold red]API Error {e.response.status_code}: "
-                    f"{e.response.text}[/bold red]"
-                )
-        raise typer.Exit(code=1)
-    except httpx.RequestError as e:
-        error_console.print(
-            f"[bold red]Network Error: Could not connect to the API. {e}[/bold red]"
-        )
-        raise typer.Exit(code=1)
-    except Exception as e:
-        error_console.print(f"[bold red]An unexpected error occurred: {e}[/bold red]")
+        client = ApiClient()
+    except ValueError as error:
+        logger.error("Failed to initialize API client: %s", error)
+        error_console.print(f"[bold red]Error: {error}[/bold red]")
         raise typer.Exit(code=1)
 
+    console.print(f"Searching for: '{query_string}'...")
+    response = await client.search(query=query_string, limit=limit, packages=packages)
+    display_search_results(response, display_limit=limit, console=console)
+
 
 @mcp_app.command("serve")
 def mcp_serve_command(
@@ -605,23 +93,15 @@ def mcp_serve_command(
         case_sensitive=False,
         show_choices=True,
     ),
-    api_key_override: Optional[str] = typer.Option(
+    api_key_override: str | None = typer.Option(
         None,
         "--api-key",
-        help="API key to use if backend is 'api'. Overrides stored key. "
-        "Not used for 'local' backend.",
+        help="API key to use if backend is 'api'. Overrides env var.",
     ),
 ):
-    """Launch the Lean Explore MCP (Model Context Protocol) server.
-
-    The server communicates via stdio and provides Lean search functionalities
-    as MCP tools. The actual checks for local data presence or API key validity
-    are handled by the 'lean_explore.mcp.server' module when it starts.
+    """Launch the Lean Explore MCP (Model Context Protocol) server."""
+    error_console = _get_console(use_stderr=True)
 
-    Args:
-        backend: The backend choice ('api' or 'local').
-        api_key_override: Optional API key to override any stored key.
-    """
     command_parts = [
         sys.executable,
         "-m",
@@ -631,60 +111,23 @@ def mcp_serve_command(
     ]
 
     if backend.lower() == "api":
-        effective_lean_explore_api_key = api_key_override or config_utils.load_api_key()
-        if not effective_lean_explore_api_key:
+        effective_api_key = api_key_override or os.getenv("LEANEXPLORE_API_KEY")
+        if not effective_api_key:
+            logger.error("API key required for 'api' backend but not provided")
             error_console.print(
-                "[bold red]Lean Explore API key is required for 'api' backend."
-                "[/bold red]\n"
-                "Please configure it using `leanexplore configure api-key` "
-                "or provide it with the `--api-key` option for this command."
+                "[bold red]API key required for 'api' backend.[/bold red]\n"
+                "Set LEANEXPLORE_API_KEY or use --api-key option."
             )
             raise typer.Abort()
         if api_key_override:
             command_parts.extend(["--api-key", api_key_override])
-    elif backend.lower() == "local":
-        error_console.print(  # Changed to error_console for consistency
-            "[dim]Attempting to start MCP server with 'local' backend. "
-            "The server will verify local data availability.[/dim]"
-        )
-    else:
-        error_console.print(
-            f"[bold red]Invalid backend: '{backend}'. Must be 'api' or 'local'."
-            "[/bold red]"
-        )
-        raise typer.Abort()
 
-    error_console.print(
-        f"[green]Launching MCP server subprocess with '{backend}' backend...[/green]"
-    )
-    error_console.print(
-        "[dim]The server will now take over stdio. To stop it, the connected MCP "
-        "client should disconnect, or you may need to manually terminate this process "
-        "(e.g., Ctrl+C if no client is managing it).[/dim]"
-    )
+    logger.info("Starting MCP server with backend: %s", backend.lower())
+    result = subprocess.run(command_parts, check=False)
 
-    try:
-        process_result = subprocess.run(command_parts, check=False)
-        if process_result.returncode != 0:
-            error_console.print(
-                f"[bold red]MCP server subprocess exited with code: "
-                f"{process_result.returncode}. Check server logs above for "
-                f"details.[/bold red]"
-            )
-    except FileNotFoundError:
-        error_console.print(
-            f"[bold red]Error: Could not find Python interpreter '{sys.executable}' "
-            f"or the MCP server module 'lean_explore.mcp.server'.[/bold red]"
-        )
-        error_console.print(
-            "Please ensure the package is installed correctly and "
-            "`python -m lean_explore.mcp.server` is runnable."
-        )
-    except Exception as e:
-        error_console.print(
-            f"[bold red]An error occurred while trying to launch or run the MCP "
-            f"server: {e}[/bold red]"
-        )
+    if result.returncode != 0:
+        logger.error("MCP server exited with code %d", result.returncode)
+        raise typer.Exit(code=result.returncode)
 
 
 if __name__ == "__main__":