ripperdoc 0.2.0__py3-none-any.whl → 0.2.3__py3-none-any.whl

ripperdoc/cli/commands/memory_cmd.py

@@ -0,0 +1,202 @@
+"""Slash command to view and edit AGENTS memory files."""
+
+from __future__ import annotations
+
+import os
+import shlex
+import shutil
+import subprocess
+from pathlib import Path
+from typing import Any, List, Optional
+
+from rich.markup import escape
+from rich.panel import Panel
+from rich.table import Table
+
+from ripperdoc.utils.memory import (
+    LOCAL_MEMORY_FILE_NAME,
+    MEMORY_FILE_NAME,
+    collect_all_memory_files,
+)
+
+from .base import SlashCommand
+
+
+def _shorten_path(path: Path, project_path: Path) -> str:
+    """Return a short, user-friendly path."""
+    try:
+        return str(path.resolve().relative_to(project_path.resolve()))
+    except Exception:
+        pass
+
+    home = Path.home()
+    try:
+        rel_home = path.resolve().relative_to(home)
+        return f"~/{rel_home}"
+    except Exception:
+        return str(path)
+
+
+def _preferred_user_memory_path() -> Path:
+    """Pick the user-level memory path, preferring ~/.ripperdoc/AGENTS.md."""
+    home = Path.home()
+    preferred_dir = home / ".ripperdoc"
+    preferred_path = preferred_dir / MEMORY_FILE_NAME
+    fallback_path = home / MEMORY_FILE_NAME
+
+    if preferred_path.exists():
+        return preferred_path
+    if fallback_path.exists():
+        return fallback_path
+
+    preferred_dir.mkdir(parents=True, exist_ok=True)
+    return preferred_path
+
+
+def _ensure_file(path: Path) -> bool:
+    """Ensure the target file exists. Returns True if created."""
+    path.parent.mkdir(parents=True, exist_ok=True)
+    if path.exists():
+        return False
+    path.write_text("", encoding="utf-8")
+    return True
+
+
+def _ensure_gitignore_entry(project_path: Path, entry: str) -> bool:
+    """Ensure an entry exists in .gitignore. Returns True if added."""
+    gitignore_path = project_path / ".gitignore"
+    try:
+        text = ""
+        if gitignore_path.exists():
+            text = gitignore_path.read_text(encoding="utf-8", errors="ignore")
+            existing_lines = text.splitlines()
+            if entry in existing_lines:
+                return False
+        with gitignore_path.open("a", encoding="utf-8") as f:
+            if text and not text.endswith("\n"):
+                f.write("\n")
+            f.write(f"{entry}\n")
+        return True
+    except Exception:
+        return False
+
+
+def _determine_editor_command() -> Optional[List[str]]:
+    """Resolve the editor command from environment or common defaults."""
+    for env_var in ("VISUAL", "EDITOR"):
+        value = os.environ.get(env_var)
+        if value:
+            return shlex.split(value)
+
+    candidates = ["code", "nano", "vim", "vi"]
+    if os.name == "nt":
+        candidates.insert(0, "notepad")
+
+    for candidate in candidates:
+        if shutil.which(candidate):
+            return [candidate]
+    return None
+
+
+def _open_in_editor(path: Path, console: Any) -> bool:
+    """Open the file in a text editor; returns True if an editor was launched."""
+    editor_cmd = _determine_editor_command()
+    if not editor_cmd:
+        console.print(
+            f"[yellow]No editor configured. Set $EDITOR or $VISUAL, "
+            f"or manually edit: {escape(str(path))}[/yellow]"
+        )
+        return False
+
+    cmd = [*editor_cmd, str(path)]
+    try:
+        console.print(f"[dim]Opening with: {' '.join(editor_cmd)}[/dim]")
+        subprocess.run(cmd, check=False)
+        return True
+    except FileNotFoundError:
+        console.print(f"[red]Editor command not found: {escape(editor_cmd[0])}[/red]")
+        return False
+    except Exception as exc:  # pragma: no cover - best-effort logging
+        console.print(f"[red]Failed to launch editor: {escape(str(exc))}[/red]")
+        return False
+
+
+def _render_memory_table(console: Any, project_path: Path) -> None:
+    files = collect_all_memory_files()
+    table = Table(title="Memory files", show_header=True, header_style="bold cyan")
+    table.add_column("Type", style="bold")
+    table.add_column("Location")
+    table.add_column("Nested", justify="center")
+
+    for memory_file in files:
+        display_path = _shorten_path(Path(memory_file.path), project_path)
+        nested = "yes" if getattr(memory_file, "is_nested", False) else ""
+        table.add_row(memory_file.type, escape(display_path), nested)
+
+    if files:
+        console.print(table)
+    else:
+        console.print("[yellow]No memory files found yet.[/yellow]")
+
+
+def _handle(ui: Any, trimmed_arg: str) -> bool:
+    project_path = getattr(ui, "project_path", Path.cwd())
+    scope = trimmed_arg.strip().lower()
+
+    if scope:
+        scope_aliases = {
+            "project": "project",
+            "workspace": "project",
+            "local": "local",
+            "private": "local",
+            "user": "user",
+            "global": "user",
+        }
+        if scope not in scope_aliases:
+            ui.console.print("[red]Unknown scope. Use one of: project, local, user.[/red]")
+            return True
+
+        resolved_scope = scope_aliases[scope]
+        if resolved_scope == "project":
+            target_path = project_path / MEMORY_FILE_NAME
+            heading = "Project memory (checked in)"
+        elif resolved_scope == "local":
+            target_path = project_path / LOCAL_MEMORY_FILE_NAME
+            heading = "Local memory (not checked in)"
+        else:
+            target_path = _preferred_user_memory_path()
+            heading = "User memory (home directory)"
+
+        created = _ensure_file(target_path)
+        gitignore_added = False
+        if resolved_scope == "local":
+            gitignore_added = _ensure_gitignore_entry(project_path, LOCAL_MEMORY_FILE_NAME)
+
+        _open_in_editor(target_path, ui.console)
+
+        messages: List[str] = [f"{heading}: {escape(_shorten_path(target_path, project_path))}"]
+        if created:
+            messages.append("Created new memory file.")
+        if gitignore_added:
+            messages.append("Added AGENTS.local.md to .gitignore.")
+        if not created:
+            messages.append("Opened existing memory file.")
+
+        ui.console.print(Panel("\n".join(messages), title="/memory"))
+        return True
+
+    _render_memory_table(ui.console, project_path)
+    ui.console.print("[dim]Usage: /memory project | /memory local | /memory user[/dim]")
+    ui.console.print("[dim]Project and user memories feed directly into the system prompt.[/dim]")
+    return True
+
+
+command = SlashCommand(
+    name="memory",
+    description="List and edit AGENTS memory files",
+    handler=_handle,
+    aliases=("mem",),
+)
+
+
+__all__ = ["command"]

ripperdoc/cli/commands/models_cmd.py

@@ -1,6 +1,4 @@
-from typing import Any
-from getpass import getpass
-from typing import Optional
+from typing import Any, Optional
 
 from rich.markup import escape
 
@@ -13,15 +11,23 @@ from ripperdoc.core.config import (
     get_global_config,
     set_model_pointer,
 )
+from ripperdoc.utils.log import get_logger
+from ripperdoc.utils.prompt import prompt_secret
 
 from .base import SlashCommand
 
+logger = get_logger()
+
 
 def _handle(ui: Any, trimmed_arg: str) -> bool:
     console = ui.console
     tokens = trimmed_arg.split()
     subcmd = tokens[0].lower() if tokens else ""
     config = get_global_config()
+    logger.info(
+        "[models_cmd] Handling /models command",
+        extra={"subcommand": subcmd or "list", "session_id": getattr(ui, "session_id", None)},
+    )
 
     def print_models_usage() -> None:
         console.print("[bold]/models[/bold] — list configured models")
@@ -103,9 +109,18 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
             console.print("[red]Model name is required.[/red]")
             return True
 
-        api_key_input = getpass("API key (leave blank to keep unset): ").strip()
+        api_key_input = prompt_secret("API key (leave blank to keep unset)").strip()
         api_key = api_key_input or (existing_profile.api_key if existing_profile else None)
 
+        auth_token = existing_profile.auth_token if existing_profile else None
+        if provider == ProviderType.ANTHROPIC:
+            auth_token_input = prompt_secret(
+                "Auth token (Anthropic only, leave blank to keep unset)"
+            ).strip()
+            auth_token = auth_token_input or auth_token
+        else:
+            auth_token = None
+
         api_base_default = existing_profile.api_base if existing_profile else ""
         api_base = (
             console.input(
@@ -156,6 +171,7 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
             max_tokens=max_tokens,
             temperature=temperature,
             context_window=context_window,
+            auth_token=auth_token,
         )
 
         try:
@@ -167,6 +183,10 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
             )
         except Exception as exc:
             console.print(f"[red]Failed to save model: {escape(str(exc))}[/red]")
+            logger.exception(
+                "[models_cmd] Failed to save model profile",
+                extra={"profile": profile_name, "session_id": getattr(ui, "session_id", None)},
+            )
             return True
 
         marker = " (main)" if set_as_main else ""
@@ -202,8 +222,8 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
         )
 
         api_key_label = "[set]" if existing_profile.api_key else "[not set]"
-        api_key_prompt = f"API key {api_key_label} (Enter=keep, '-'=clear): "
-        api_key_input = getpass(api_key_prompt).strip()
+        api_key_prompt = f"API key {api_key_label} (Enter=keep, '-'=clear)"
+        api_key_input = prompt_secret(api_key_prompt).strip()
         if api_key_input == "-":
             api_key = None
         elif api_key_input:
@@ -211,6 +231,21 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
         else:
             api_key = existing_profile.api_key
 
+        auth_token = existing_profile.auth_token
+        if (
+            provider == ProviderType.ANTHROPIC
+            or existing_profile.provider == ProviderType.ANTHROPIC
+        ):
+            auth_label = "[set]" if auth_token else "[not set]"
+            auth_prompt = f"Auth token (Anthropic only) {auth_label} (Enter=keep, '-'=clear)"
+            auth_token_input = prompt_secret(auth_prompt).strip()
+            if auth_token_input == "-":
+                auth_token = None
+            elif auth_token_input:
+                auth_token = auth_token_input
+        else:
+            auth_token = None
+
         api_base = (
             console.input(f"API base (optional) [{existing_profile.api_base or ''}]: ").strip()
             or existing_profile.api_base
@@ -244,6 +279,7 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
             max_tokens=max_tokens,
             temperature=temperature,
             context_window=context_window,
+            auth_token=auth_token,
         )
 
         try:
@@ -255,6 +291,10 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
             )
         except Exception as exc:
             console.print(f"[red]Failed to update model: {escape(str(exc))}[/red]")
+            logger.exception(
+                "[models_cmd] Failed to update model profile",
+                extra={"profile": profile_name, "session_id": getattr(ui, "session_id", None)},
+            )
             return True
 
         console.print(f"[green]✓ Model '{escape(profile_name)}' updated[/green]")
@@ -274,6 +314,10 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
         except Exception as exc:
             console.print(f"[red]Failed to delete model: {escape(str(exc))}[/red]")
             print_models_usage()
+            logger.exception(
+                "[models_cmd] Failed to delete model profile",
+                extra={"profile": target, "session_id": getattr(ui, "session_id", None)},
+            )
         return True
 
     if subcmd in ("use", "main", "set-main"):
@@ -288,6 +332,10 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
         except Exception as exc:
             console.print(f"[red]{escape(str(exc))}[/red]")
             print_models_usage()
+            logger.exception(
+                "[models_cmd] Failed to set main model pointer",
+                extra={"profile": target, "session_id": getattr(ui, "session_id", None)},
+            )
         return True
 
     print_models_usage()
@@ -312,6 +360,13 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
             markup=False,
         )
         console.print(f"      api_key: {'***' if profile.api_key else 'Not set'}", markup=False)
+        if profile.provider == ProviderType.ANTHROPIC:
+            console.print(
+                f"      auth_token: {'***' if getattr(profile, 'auth_token', None) else 'Not set'}",
+                markup=False,
+            )
+        if profile.openai_tool_mode:
+            console.print(f"      openai_tool_mode: {profile.openai_tool_mode}", 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

@@ -30,7 +30,8 @@ def _choose_session(ui: Any, arg: str) -> Optional[SessionSummary]:
             if 0 <= idx < len(sessions):
                 return sessions[idx]
             ui.console.print(
-                f"[red]Invalid session index {escape(str(idx))}. Choose 0-{len(sessions) - 1}.[/red]"
+                f"[red]Invalid session index {escape(str(idx))}. "
+                f"Choose 0-{len(sessions) - 1}.[/red]"
             )
         else:
             # Treat arg as session id if it matches.
@@ -60,7 +61,8 @@ def _choose_session(ui: Any, arg: str) -> Optional[SessionSummary]:
     idx = int(choice_text)
     if idx < 0 or idx >= len(sessions):
         ui.console.print(
-            f"[red]Invalid session index {escape(str(idx))}. Choose 0-{len(sessions) - 1}.[/red]"
+            f"[red]Invalid session index {escape(str(idx))}. "
+            f"Choose 0-{len(sessions) - 1}.[/red]"
         )
         return None
     return sessions[idx]

ripperdoc/cli/commands/status_cmd.py

@@ -34,7 +34,7 @@ def _auth_token_display(profile: Optional[ModelProfile]) -> Tuple[str, Optional[
     env_var = next((name for name in env_candidates if os.environ.get(name)), None)
     if env_var:
         return (f"{env_var} (env)", env_var)
-    if profile.api_key:
+    if profile.api_key or getattr(profile, "auth_token", None):
         return ("Configured in profile", None)
     return ("Missing", None)
 

ripperdoc/cli/commands/tasks_cmd.py

@@ -13,11 +13,15 @@ from ripperdoc.tools.background_shell import (
     kill_background_task,
     list_background_tasks,
 )
+from ripperdoc.utils.log import get_logger
 
 from typing import Any, Optional
 from .base import SlashCommand
 
 
+logger = get_logger()
+
+
 def _format_duration(duration_ms: Optional[float]) -> str:
     """Render milliseconds into a short human-readable duration."""
     if duration_ms is None:
@@ -103,6 +107,10 @@ def _list_tasks(ui: Any) -> bool:
             status = get_background_status(task_id, consume=False)
         except Exception as exc:
             table.add_row(escape(task_id), "[red]error[/]", escape(str(exc)), "-")
+            logger.exception(
+                "[tasks_cmd] Failed to read background task status",
+                extra={"task_id": task_id, "session_id": getattr(ui, "session_id", None)},
+            )
             continue
 
         command = status.get("command") or ""
@@ -137,6 +145,10 @@ def _kill_task(ui: Any, task_id: str) -> bool:
         return True
     except Exception as exc:
         console.print(f"[red]Failed to read task '{escape(task_id)}': {escape(str(exc))}[/red]")
+        logger.exception(
+            "[tasks_cmd] Failed to read task before kill",
+            extra={"task_id": task_id, "session_id": getattr(ui, "session_id", None)},
+        )
         return True
 
     if status.get("status") != "running":
@@ -149,6 +161,10 @@ def _kill_task(ui: Any, task_id: str) -> bool:
         killed = asyncio.run(kill_background_task(task_id))
     except Exception as exc:
         console.print(f"[red]Error stopping task {escape(task_id)}: {escape(str(exc))}[/red]")
+        logger.exception(
+            "[tasks_cmd] Error stopping background task",
+            extra={"task_id": task_id, "session_id": getattr(ui, "session_id", None)},
+        )
         return True
 
     if killed:
@@ -169,6 +185,10 @@ def _show_task(ui: Any, task_id: str) -> bool:
         return True
     except Exception as exc:
         console.print(f"[red]Failed to read task '{escape(task_id)}': {escape(str(exc))}[/red]")
+        logger.exception(
+            "[tasks_cmd] Failed to read task for detail view",
+            extra={"task_id": task_id, "session_id": getattr(ui, "session_id", None)},
+        )
         return True
 
     details = Table(box=box.SIMPLE_HEAVY, show_header=False)
@@ -208,6 +228,13 @@ def _show_task(ui: Any, task_id: str) -> bool:
 
 def _handle(ui: Any, args: str) -> bool:
     parts = args.split()
+    logger.info(
+        "[tasks_cmd] Handling /tasks command",
+        extra={
+            "session_id": getattr(ui, "session_id", None),
+            "raw_args": args,
+        },
+    )
     if not parts:
         return _list_tasks(ui)