ripperdoc 0.2.10__py3-none-any.whl → 0.3.1__py3-none-any.whl

ripperdoc/__init__.py

@@ -1,3 +1,3 @@
 """Ripperdoc - AI-powered coding agent."""
 
-__version__ = "0.2.10"
+__version__ = "0.3.1"

ripperdoc/cli/cli.py

@@ -44,7 +44,6 @@ from ripperdoc.utils.log import enable_session_file_logging, get_logger
 
 from rich.console import Console
 from rich.markdown import Markdown
-from rich.panel import Panel
 from rich.markup import escape
 
 console = Console()
@@ -98,7 +97,6 @@ async def run_query(
     model: Optional[str] = None,
 ) -> None:
     """Run a single query and print the response."""
-
     logger.info(
         "[cli] Running single prompt session",
         extra={
@@ -212,55 +210,37 @@ async def run_query(
                 mcp_instructions=mcp_instructions,
             )
 
-        # Run the query
+        # Run the query - collect final response text
+        final_response_parts: List[str] = []
         try:
             async for message in query(
                 messages, system_prompt, context, query_context, can_use_tool
             ):
                 if message.type == "assistant" and hasattr(message, "message"):
-                    # Print assistant message
+                    # Collect assistant message text for final output
                     if isinstance(message.message.content, str):
-                        console.print(
-                            Panel(
-                                Markdown(message.message.content),
-                                title="Ripperdoc",
-                                border_style="cyan",
-                                padding=(0, 1),
-                            )
-                        )
+                        final_response_parts.append(message.message.content)
                     else:
                         # Handle structured content
                         for block in message.message.content:
                             if isinstance(block, dict):
                                 if block.get("type") == "text":
-                                    console.print(
-                                        Panel(
-                                            Markdown(block["text"]),
-                                            title="Ripperdoc",
-                                            border_style="cyan",
-                                            padding=(0, 1),
-                                        )
-                                    )
+                                    final_response_parts.append(block["text"])
                             else:
                                 if hasattr(block, "type") and block.type == "text":
-                                    console.print(
-                                        Panel(
-                                            Markdown(block.text or ""),
-                                            title="Ripperdoc",
-                                            border_style="cyan",
-                                            padding=(0, 1),
-                                        )
-                                    )
-
-                elif message.type == "progress" and hasattr(message, "content"):
-                    # Print progress
-                    if verbose:
-                        console.print(f"[dim]Progress: {escape(str(message.content))}[/dim]")
+                                    final_response_parts.append(block.text or "")
+
+                # Skip progress messages entirely for -p mode
 
                 # Add message to history
                 messages.append(message)  # type: ignore[arg-type]
                 session_history.append(message)  # type: ignore[arg-type]
 
+            # Print final response as clean markdown (no panel, no decoration)
+            if final_response_parts:
+                final_text = "\n".join(final_response_parts)
+                console.print(Markdown(final_text))
+
         except KeyboardInterrupt:
             console.print("\n[yellow]Interrupted by user[/yellow]")
         except asyncio.CancelledError:
@@ -308,7 +288,6 @@ async def run_query(
         logger.debug("[cli] Shutdown MCP runtime", extra={"session_id": session_id})
 
 
-
 @click.group(invoke_without_command=True)
 @click.version_option(version=__version__)
 @click.option("--cwd", type=click.Path(exists=True), help="Working directory")
@@ -374,19 +353,51 @@ def cli(
 ) -> None:
     """Ripperdoc - AI-powered coding agent"""
     session_id = str(uuid.uuid4())
+    cwd_changed: Optional[str] = None
 
     # Set working directory
     if cwd:
         import os
 
         os.chdir(cwd)
+        cwd_changed = cwd
+
+    project_path = Path.cwd()
+
+    # Handle --continue option: load the most recent session
+    resume_messages = None
+    most_recent = None
+    if continue_session:
+        summaries = list_session_summaries(project_path)
+        if summaries:
+            most_recent = summaries[0]
+            session_id = most_recent.session_id
+            resume_messages = load_session_messages(project_path, session_id)
+            console.print(f"[dim]Continuing session: {most_recent.last_prompt}[/dim]")
+        else:
+            console.print("[yellow]No previous sessions found in this directory.[/yellow]")
+
+    log_file = enable_session_file_logging(project_path, session_id)
+
+    if cwd_changed:
         logger.debug(
             "[cli] Changed working directory via --cwd",
-            extra={"cwd": cwd, "session_id": session_id},
+            extra={"cwd": cwd_changed, "session_id": session_id},
         )
 
-    project_path = Path.cwd()
-    log_file = enable_session_file_logging(project_path, session_id)
+    if most_recent:
+        logger.info(
+            "[cli] Continuing session",
+            extra={
+                "session_id": session_id,
+                "message_count": len(resume_messages) if resume_messages else 0,
+                "last_prompt": most_recent.last_prompt,
+                "log_file": str(log_file),
+            },
+        )
+    elif continue_session:
+        logger.warning("[cli] No previous sessions found to continue")
+
     logger.info(
         "[cli] Starting CLI invocation",
         extra={
@@ -412,26 +423,39 @@ def cli(
     # Parse --tools option
     allowed_tools = parse_tools_option(tools)
 
-    # Handle --continue option: load the most recent session
-    resume_messages = None
-    if continue_session:
-        summaries = list_session_summaries(project_path)
-        if summaries:
-            most_recent = summaries[0]
-            session_id = most_recent.session_id
-            resume_messages = load_session_messages(project_path, session_id)
-            logger.info(
-                "[cli] Continuing session",
-                extra={
-                    "session_id": session_id,
-                    "message_count": len(resume_messages),
-                    "last_prompt": most_recent.last_prompt,
-                },
-            )
-            console.print(f"[dim]Continuing session: {most_recent.last_prompt}[/dim]")
-        else:
-            logger.warning("[cli] No previous sessions found to continue")
-            console.print("[yellow]No previous sessions found in this directory.[/yellow]")
+    # Handle piped stdin input
+    # - With -p flag: Not applicable (prompt comes from -p argument)
+    # - Without -p: stdin becomes the initial query in interactive mode
+    initial_query: Optional[str] = None
+    if prompt is None and ctx.invoked_subcommand is None:
+        stdin_stream = click.get_text_stream("stdin")
+        try:
+            stdin_is_tty = stdin_stream.isatty()
+        except Exception:
+            stdin_is_tty = True
+
+        if not stdin_is_tty:
+            try:
+                stdin_data = stdin_stream.read()
+            except (OSError, ValueError) as exc:
+                logger.warning(
+                    "[cli] Failed to read stdin for initial query: %s: %s",
+                    type(exc).__name__,
+                    exc,
+                    extra={"session_id": session_id},
+                )
+            else:
+                trimmed = stdin_data.rstrip("\n")
+                if trimmed.strip():
+                    initial_query = trimmed
+                    logger.info(
+                        "[cli] Received initial query from stdin",
+                        extra={
+                            "session_id": session_id,
+                            "query_length": len(initial_query),
+                            "query_preview": initial_query[:200],
+                        },
+                    )
 
     logger.debug(
         "[cli] Configuration initialized",
@@ -480,6 +504,7 @@ def cli(
             append_system_prompt=append_system_prompt,
             model=model,
             resume_messages=resume_messages,
+            initial_query=initial_query,
         )
         return
 
@@ -507,6 +532,88 @@ def config_cmd() -> None:
         console.print()
 
 
+@cli.command(name="stdio")
+@click.option(
+    "--input-format",
+    type=click.Choice(["stream-json", "auto"]),
+    default="stream-json",
+    help="Input format for messages.",
+)
+@click.option(
+    "--output-format",
+    type=click.Choice(["stream-json"]),
+    default="stream-json",
+    help="Output format for messages.",
+)
+@click.option(
+    "--model",
+    type=str,
+    default=None,
+    help="Model profile for the current session.",
+)
+@click.option(
+    "--permission-mode",
+    type=click.Choice(["default", "acceptEdits", "plan", "bypassPermissions"]),
+    default="default",
+    help="Permission mode for tool usage.",
+)
+@click.option(
+    "--max-turns",
+    type=int,
+    default=None,
+    help="Maximum number of conversation turns.",
+)
+@click.option(
+    "--system-prompt",
+    type=str,
+    default=None,
+    help="System prompt to use for the session.",
+)
+@click.option(
+    "--print",
+    "-p",
+    is_flag=True,
+    help="Print mode (for single prompt queries).",
+)
+@click.option(
+    "--",
+    "prompt",
+    type=str,
+    default=None,
+    help="Direct prompt (for print mode).",
+)
+def stdio_cmd(
+    input_format: str,
+    output_format: str,
+    model: str | None,
+    permission_mode: str,
+    max_turns: int | None,
+    system_prompt: str | None,
+    print: bool,
+    prompt: str | None,
+) -> None:
+    """Stdio mode for SDK subprocess communication.
+
+    This command enables Ripperdoc to communicate with SDKs via JSON Control
+    Protocol over stdin/stdout.
+    """
+    from ripperdoc.protocol.stdio import _run_stdio
+    import asyncio
+
+    asyncio.run(
+        _run_stdio(
+            input_format=input_format,
+            output_format=output_format,
+            model=model,
+            permission_mode=permission_mode,
+            max_turns=max_turns,
+            system_prompt=system_prompt,
+            print_mode=print,
+            prompt=prompt,
+        )
+    )
+
+
 @cli.command(name="version")
 def version_cmd() -> None:
     """Show version information"""

ripperdoc/cli/commands/__init__.py

@@ -21,9 +21,11 @@ from .mcp_cmd import command as mcp_command
 from .models_cmd import command as models_command
 from .permissions_cmd import command as permissions_command
 from .resume_cmd import command as resume_command
+from .skills_cmd import command as skills_command
 from .stats_cmd import command as stats_command
 from .tasks_cmd import command as tasks_command
 from .status_cmd import command as status_command
+from .themes_cmd import command as themes_command
 from .todos_cmd import command as todos_command
 from .tools_cmd import command as tools_command
 
@@ -64,7 +66,9 @@ ALL_COMMANDS: List[SlashCommand] = [
     context_command,
     compact_command,
     resume_command,
+    skills_command,
     agents_command,
+    themes_command,
 ]
 
 COMMAND_REGISTRY: Dict[str, SlashCommand] = _build_registry(ALL_COMMANDS)

ripperdoc/cli/commands/agents_cmd.py

@@ -18,7 +18,7 @@ from ripperdoc.tools.task_tool import (
 )
 from ripperdoc.utils.log import get_logger
 
-from typing import Any, Dict, Optional
+from typing import Any, Dict
 from .base import SlashCommand
 
 logger = get_logger()
@@ -101,9 +101,7 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
         for run_id in sorted(run_ids):
             snapshot: Dict[Any, Any] = get_agent_run_snapshot(run_id) or {}
             result_text = snapshot.get("result_text") or snapshot.get("error") or ""
-            result_preview = (
-                result_text if len(result_text) <= 80 else result_text[:77] + "..."
-            )
+            result_preview = result_text if len(result_text) <= 80 else result_text[:77] + "..."
             table.add_row(
                 escape(run_id),
                 escape(snapshot.get("status") or "unknown"),
@@ -136,9 +134,7 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
         details.add_row("Status", escape(snapshot.get("status") or "unknown"))
         details.add_row("Agent", escape(snapshot.get("agent_type") or "unknown"))
         details.add_row("Duration", _format_duration(snapshot.get("duration_ms")))
-        details.add_row(
-            "Background", "yes" if snapshot.get("is_background") else "no"
-        )
+        details.add_row("Background", "yes" if snapshot.get("is_background") else "no")
         if snapshot.get("model_used"):
             details.add_row("Model", escape(str(snapshot.get("model_used"))))
         if snapshot.get("tool_use_count"):

ripperdoc/cli/commands/doctor_cmd.py

@@ -15,6 +15,8 @@ from ripperdoc.core.config import (
     api_key_env_candidates,
     get_global_config,
     get_project_config,
+    get_ripperdoc_env_status,
+    has_ripperdoc_env_overrides,
 )
 from ripperdoc.cli.ui.helpers import get_profile_for_pointer
 from ripperdoc.utils.log import get_logger
@@ -41,6 +43,12 @@ def _api_key_status(provider: ProviderType, profile_key: Optional[str]) -> Tuple
     """Check API key presence and source."""
     import os
 
+    # 首先检查全局 RIPPERDOC_API_KEY
+    if ripperdoc_api_key := os.getenv("RIPPERDOC_API_KEY"):
+        masked = ripperdoc_api_key[:4] + "…" if len(ripperdoc_api_key) > 4 else "set"
+        return ("ok", f"Found in $RIPPERDOC_API_KEY ({masked})")
+
+    # 然后检查 provider 特定的环境变量
     for env_var in api_key_env_candidates(provider):
         if os.environ.get(env_var):
             masked = os.environ[env_var]
@@ -186,6 +194,23 @@ def _project_status(project_path: Path) -> Tuple[str, str, str]:
         )
 
 
+def _ripperdoc_env_status() -> List[Tuple[str, str, str]]:
+    """Check RIPPERDOC_* environment variable overrides."""
+    rows: List[Tuple[str, str, str]] = []
+
+    if not has_ripperdoc_env_overrides():
+        rows.append(_status_row("Env overrides", "ok", "No RIPPERDOC_* overrides active"))
+        return rows
+
+    rows.append(_status_row("Env overrides", "ok", "RIPPERDOC_* variables detected"))
+
+    status = get_ripperdoc_env_status()
+    for key, value in status.items():
+        rows.append(_status_row("", "ok", f"  {key}: {value}"))
+
+    return rows
+
+
 def _render_table(console: Any, rows: List[Tuple[str, str, str]]) -> None:
     table = Table(show_header=True, header_style="bold cyan")
     table.add_column("Check")
@@ -202,6 +227,10 @@ def _handle(ui: Any, _: str) -> bool:
 
     results.append(_onboarding_status())
     results.extend(_model_status(project_path))
+
+    # 检查 RIPPERDOC_* 环境变量
+    results.extend(_ripperdoc_env_status())
+
     project_row = _project_status(project_path)
     results.append(project_row)
 

ripperdoc/cli/commands/memory_cmd.py

@@ -18,6 +18,7 @@ from ripperdoc.utils.memory import (
     MEMORY_FILE_NAME,
     collect_all_memory_files,
 )
+from ripperdoc.utils.platform import is_windows
 
 from .base import SlashCommand
 
@@ -89,7 +90,7 @@ def _determine_editor_command() -> Optional[List[str]]:
             return shlex.split(value)
 
     candidates = ["code", "nano", "vim", "vi"]
-    if os.name == "nt":
+    if is_windows():
         candidates.insert(0, "notepad")
 
     for candidate in candidates:

ripperdoc/cli/commands/models_cmd.py

@@ -155,12 +155,34 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
         context_prompt += "): "
         context_window = parse_int(context_prompt, context_window_default)
 
+        # Vision support prompt
+        supports_vision_default = existing_profile.supports_vision if existing_profile else None
+        supports_vision = None
+        vision_default_display = (
+            "auto"
+            if supports_vision_default is None
+            else ("yes" if supports_vision_default else "no")
+        )
+        supports_vision_input = (
+            console.input(f"Supports vision (images)? [{vision_default_display}] (Y/n/auto): ")
+            .strip()
+            .lower()
+        )
+        if supports_vision_input in ("y", "yes"):
+            supports_vision = True
+        elif supports_vision_input in ("n", "no"):
+            supports_vision = False
+        elif supports_vision_input in ("auto", ""):
+            supports_vision = None
+        else:
+            supports_vision = supports_vision_default
+
         default_set_main = (
             not config.model_profiles
             or getattr(config.model_pointers, "main", "") not in config.model_profiles
         )
         set_main_input = (
-            console.input(f"Set as main model? [{'Y' if default_set_main else 'y'}/N]: ")
+            console.input(f"Set as main model? ({'Y' if default_set_main else 'y'}/N): ")
             .strip()
             .lower()
         )
@@ -175,6 +197,7 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
             temperature=temperature,
             context_window=context_window,
             auth_token=auth_token,
+            supports_vision=supports_vision,
         )
 
         try:
@@ -276,6 +299,31 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
             existing_profile.context_window,
         )
 
+        # Vision support prompt
+        vision_default_display = (
+            "auto"
+            if existing_profile.supports_vision is None
+            else ("yes" if existing_profile.supports_vision else "no")
+        )
+        supports_vision_input = (
+            console.input(
+                f"Supports vision (images)? [{vision_default_display}] (Y/n/auto/C=clear): "
+            )
+            .strip()
+            .lower()
+        )
+        supports_vision = None
+        if supports_vision_input in ("y", "yes"):
+            supports_vision = True
+        elif supports_vision_input in ("n", "no"):
+            supports_vision = False
+        elif supports_vision_input in ("c", "clear", "-"):
+            supports_vision = None
+        elif supports_vision_input in ("auto", ""):
+            supports_vision = existing_profile.supports_vision
+        else:
+            supports_vision = existing_profile.supports_vision
+
         updated_profile = ModelProfile(
             provider=provider,
             model=model_name,
@@ -285,6 +333,7 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
             temperature=temperature,
             context_window=context_window,
             auth_token=auth_token,
+            supports_vision=supports_vision,
         )
 
         try:
@@ -353,10 +402,7 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
                 pointer = "main"
                 target = tokens[1]
         else:
-            pointer = (
-                console.input("Pointer (main/quick) [main]: ").strip().lower()
-                or "main"
-            )
+            pointer = console.input("Pointer (main/quick) [main]: ").strip().lower() or "main"
             if pointer not in valid_pointers:
                 console.print(
                     f"[red]Invalid pointer '{escape(pointer)}'. Valid pointers: {', '.join(valid_pointers)}[/red]"
@@ -415,6 +461,16 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
             )
         if profile.openai_tool_mode:
             console.print(f"      openai_tool_mode: {profile.openai_tool_mode}", markup=False)
+        if profile.thinking_mode:
+            console.print(f"      thinking_mode: {profile.thinking_mode}", markup=False)
+        # Display vision support
+        if profile.supports_vision is None:
+            vision_display = "auto-detect"
+        elif profile.supports_vision:
+            vision_display = "yes"
+        else:
+            vision_display = "no"
+        console.print(f"      supports_vision: {vision_display}", markup=False)
     pointer_labels = ", ".join(f"{p}->{v or '-'}" for p, v in pointer_map.items())
     console.print(f"[dim]Pointers: {escape(pointer_labels)}[/dim]")
     return True

ripperdoc/cli/commands/resume_cmd.py

@@ -132,6 +132,7 @@ command = SlashCommand(
     name="resume",
     description="Resume a previous session conversation",
     handler=_handle,
+    aliases=("sessions",),
 )
 
 

ripperdoc/cli/commands/skills_cmd.py

@@ -0,0 +1,103 @@
+from rich.markup import escape
+
+from ripperdoc.core.skills import (
+    SkillDefinition,
+    SkillLoadResult,
+    SkillLocation,
+    load_all_skills,
+    skill_directories,
+)
+
+from typing import Any
+from .base import SlashCommand
+
+
+def _handle(ui: Any, _: str) -> bool:
+    console = ui.console
+    project_path = getattr(ui, "project_path", None)
+
+    result: SkillLoadResult = load_all_skills(project_path=project_path)
+
+    if not result.skills:
+        dirs = skill_directories(project_path=project_path)
+        dir_paths = [f"'{d}'" for d, _ in dirs]
+        console.print("[yellow]No skills found.[/yellow]")
+        console.print(
+            f"\n[bold]Create skills in:[/bold]\n"
+            f"  • User: {dir_paths[0]}\n"
+            f"  • Project: {dir_paths[1]}\n"
+            f"\n[dim]Each skill needs a SKILL.md file with YAML frontmatter:\n"
+            f"---\n"
+            f"name: my-skill\n"
+            f"description: A helpful skill\n"
+            f"---\n\n"
+            f"Skill content goes here...[/dim]"
+        )
+        if result.errors:
+            console.print("\n[bold red]Errors encountered while loading skills:[/bold red]")
+            for error in result.errors:
+                console.print(f"  • {error.path}: {escape(error.reason)}", markup=False)
+        return True
+
+    console.print("\n[bold]Skills[/bold]")
+
+    # Group skills by location for better organization
+    user_skills = [s for s in result.skills if s.location == SkillLocation.USER]
+    project_skills = [s for s in result.skills if s.location == SkillLocation.PROJECT]
+    other_skills = [s for s in result.skills if s.location == SkillLocation.OTHER]
+
+    def print_skill(skill: SkillDefinition) -> None:
+        location_tag = f"[dim]({skill.location.value})[/dim]" if skill.location else ""
+        console.print(f"\n[bold cyan]{escape(skill.name)}[/bold cyan] {location_tag}")
+        console.print(f"  {escape(skill.description)}")
+
+        if skill.allowed_tools:
+            tools_str = ", ".join(escape(t) for t in skill.allowed_tools)
+            console.print(f"  Tools: {tools_str}")
+
+        if skill.model:
+            console.print(f"  Model: {escape(skill.model)}")
+
+        if skill.max_thinking_tokens:
+            console.print(f"  Max thinking tokens: {skill.max_thinking_tokens}")
+
+        if skill.skill_type != "prompt":
+            console.print(f"  Type: {escape(skill.skill_type)}")
+
+        if skill.disable_model_invocation:
+            console.print("  [yellow]Model invocation disabled[/yellow]")
+
+        console.print(f"  Path: {escape(str(skill.path))}", markup=False)
+
+    # Print project skills first (they have priority), then user skills
+    if project_skills:
+        console.print("\n[bold]Project skills:[/bold]")
+        for skill in project_skills:
+            print_skill(skill)
+
+    if user_skills:
+        console.print("\n[bold]User skills:[/bold]")
+        for skill in user_skills:
+            print_skill(skill)
+
+    if other_skills:
+        console.print("\n[bold]Other skills:[/bold]")
+        for skill in other_skills:
+            print_skill(skill)
+
+    if result.errors:
+        console.print("\n[bold red]Errors encountered while loading skills:[/bold red]")
+        for error in result.errors:
+            console.print(f"  • {error.path}: {escape(error.reason)}", markup=False)
+
+    return True
+
+
+command = SlashCommand(
+    name="skills",
+    description="List available skills",
+    handler=_handle,
+)
+
+
+__all__ = ["command"]

ripperdoc/cli/commands/stats_cmd.py

@@ -165,7 +165,9 @@ def _handle(ui: Any, _args: str) -> bool:
     )
 
     # Row 2: Total tokens | Longest session
-    total_tokens_display = format_large_number(stats.total_tokens) if stats.total_tokens > 0 else "0"
+    total_tokens_display = (
+        format_large_number(stats.total_tokens) if stats.total_tokens > 0 else "0"
+    )
     longest_session_display = ""
     if stats.longest_session_duration.total_seconds() > 0:
         longest_session_display = format_duration(stats.longest_session_duration)
@@ -212,9 +214,7 @@ def _handle(ui: Any, _args: str) -> bool:
     # Create the main panel with statistics
     # Match heatmap width: WEEKDAY_LABEL_WIDTH (8) + weeks_count (52) = 60
     heatmap_width = 8 + weeks_count + 16
-    console.print(
-        Panel(table, title="Statistics", border_style="blue", width=heatmap_width)
-    )
+    console.print(Panel(table, title="Statistics", border_style="blue", width=heatmap_width))
     console.print()
 
     # Fun comparisons