ripperdoc 0.1.0__py3-none-any.whl → 0.2.2__py3-none-any.whl

ripperdoc/cli/commands/memory_cmd.py

@@ -0,0 +1,209 @@
+"""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, 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

@@ -13,15 +13,22 @@ from ripperdoc.core.config import (
     get_global_config,
     set_model_pointer,
 )
+from ripperdoc.utils.log import get_logger
 
 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")
@@ -167,6 +174,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 ""
@@ -255,6 +266,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 +289,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 +307,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 +335,8 @@ 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.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

@@ -17,7 +17,7 @@ def _format_time(dt: datetime) -> str:
     return dt.strftime("%Y-%m-%d %H:%M")
 
 
-def _choose_session(ui, arg: str) -> Optional[SessionSummary]:
+def _choose_session(ui: Any, arg: str) -> Optional[SessionSummary]:
     sessions = list_session_summaries(ui.project_path)
     if not sessions:
         ui.console.print("[yellow]No saved sessions found for this project.[/yellow]")
@@ -30,7 +30,7 @@ def _choose_session(ui, 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))}. Choose 0-{len(sessions) - 1}.[/red]"
             )
         else:
             # Treat arg as session id if it matches.
@@ -60,7 +60,7 @@ def _choose_session(ui, 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))}. Choose 0-{len(sessions) - 1}.[/red]"
         )
         return None
     return sessions[idx]

ripperdoc/cli/commands/status_cmd.py

@@ -22,8 +22,8 @@ def _auth_token_display(profile: Optional[ModelProfile]) -> Tuple[str, Optional[
     if not profile:
         return ("Not configured", None)
 
-    provider_value = profile.provider.value if hasattr(profile.provider, "value") else str(
-        profile.provider
+    provider_value = (
+        profile.provider.value if hasattr(profile.provider, "value") else str(profile.provider)
     )
 
     env_candidates = api_key_env_candidates(profile.provider)
@@ -50,8 +50,8 @@ def _api_base_display(profile: Optional[ModelProfile]) -> str:
         ProviderType.GEMINI: "Gemini base URL",
     }
     label = label_map.get(profile.provider, "API base URL")
-    provider_value = profile.provider.value if hasattr(profile.provider, "value") else str(
-        profile.provider
+    provider_value = (
+        profile.provider.value if hasattr(profile.provider, "value") else str(profile.provider)
     )
 
     env_candidates = api_base_env_candidates(profile.provider)
@@ -81,7 +81,7 @@ def _memory_status_lines(memory_files: List[MemoryFile]) -> List[str]:
 
 
 def _setting_sources_summary(
-    config,
+    config: Any,
     profile: Optional[ModelProfile],
     memory_files: List[MemoryFile],
     auth_env_var: Optional[str],

ripperdoc/cli/commands/tasks_cmd.py

@@ -13,12 +13,16 @@ from ripperdoc.tools.background_shell import (
     kill_background_task,
     list_background_tasks,
 )
+from ripperdoc.utils.log import get_logger
 
-from typing import Any
+from typing import Any, Optional
 from .base import SlashCommand
 
 
-def _format_duration(duration_ms) -> str:
+logger = get_logger()
+
+
+def _format_duration(duration_ms: Optional[float]) -> str:
     """Render milliseconds into a short human-readable duration."""
     if duration_ms is None:
         return "-"
@@ -82,7 +86,7 @@ def _tail_lines(text: str, max_lines: int = 20, max_chars: int = 4000) -> str:
     return content
 
 
-def _list_tasks(ui) -> bool:
+def _list_tasks(ui: Any) -> bool:
     console = ui.console
     task_ids = list_background_tasks()
 
@@ -103,6 +107,10 @@ def _list_tasks(ui) -> 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 ""
@@ -128,7 +136,7 @@ def _list_tasks(ui) -> bool:
     return True
 
 
-def _kill_task(ui, task_id: str) -> bool:
+def _kill_task(ui: Any, task_id: str) -> bool:
     console = ui.console
     try:
         status = get_background_status(task_id, consume=False)
@@ -137,6 +145,10 @@ def _kill_task(ui, 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, 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:
@@ -160,7 +176,7 @@ def _kill_task(ui, task_id: str) -> bool:
     return True
 
 
-def _show_task(ui, task_id: str) -> bool:
+def _show_task(ui: Any, task_id: str) -> bool:
     console = ui.console
     try:
         status = get_background_status(task_id, consume=False)
@@ -169,6 +185,10 @@ def _show_task(ui, 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, 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)
 

ripperdoc/cli/ui/context_display.py

@@ -3,7 +3,7 @@
 from __future__ import annotations
 
 import re
-from typing import Any, Dict, List, Optional
+from typing import Any, Dict, List, Optional, Tuple
 
 from ripperdoc.utils.message_compaction import ContextBreakdown
 
@@ -205,7 +205,8 @@ def make_segment_grid(
 
     rows: List[str] = []
     for start in range(0, total_slots, per_row):
-        rows.append(" ".join(icons[start : start + per_row]))
+        row_icons = [icon for icon in icons[start : start + per_row] if icon is not None]
+        rows.append(" ".join(row_icons))
     return rows
 
 
@@ -229,7 +230,7 @@ def context_usage_lines(
             grid_lines.append(f"     {row}")
 
     # Textual stats (without additional mini bars).
-    stats = [
+    stats: List[Tuple[str, Optional[int], Optional[float]]] = [
         (
             f"{styled_symbol('⛁', 'grey58')} System prompt",
             breakdown.system_prompt_tokens,