ripperdoc 0.2.7__py3-none-any.whl → 0.2.9__py3-none-any.whl

ripperdoc/core/skills.py

@@ -84,10 +84,15 @@ def _split_frontmatter(raw_text: str) -> Tuple[Dict[str, Any], str]:
                 body = "\n".join(lines[idx + 1 :])
                 try:
                     frontmatter = yaml.safe_load(frontmatter_text) or {}
-                except (yaml.YAMLError, ValueError, TypeError) as exc:  # pragma: no cover - defensive
+                except (
+                    yaml.YAMLError,
+                    ValueError,
+                    TypeError,
+                ) as exc:  # pragma: no cover - defensive
                     logger.warning(
                         "[skills] Invalid frontmatter in SKILL.md: %s: %s",
-                        type(exc).__name__, exc,
+                        type(exc).__name__,
+                        exc,
                     )
                     return {"__error__": f"Invalid frontmatter: {exc}"}, body
                 return frontmatter, body
@@ -118,7 +123,8 @@ def _load_skill_file(
     except (OSError, IOError, UnicodeDecodeError) as exc:
         logger.warning(
             "[skills] Failed to read skill file: %s: %s",
-            type(exc).__name__, exc,
+            type(exc).__name__,
+            exc,
             extra={"path": str(path)},
         )
         return None, SkillLoadError(path=path, reason=f"Failed to read file: {exc}")

ripperdoc/core/system_prompt.py

@@ -49,7 +49,8 @@ def _detect_git_repo(cwd: Path) -> bool:
     except (OSError, subprocess.SubprocessError) as exc:
         logger.warning(
             "[system_prompt] Failed to detect git repository: %s: %s",
-            type(exc).__name__, exc,
+            type(exc).__name__,
+            exc,
             extra={"cwd": str(cwd)},
         )
         return False
@@ -393,7 +394,8 @@ def build_system_prompt(
         except (OSError, ValueError, RuntimeError) as exc:
             logger.warning(
                 "Failed to load agent definitions: %s: %s",
-                type(exc).__name__, exc,
+                type(exc).__name__,
+                exc,
             )
             agent_section = (
                 "# Subagents\nTask tool available, but agent definitions could not be loaded."

ripperdoc/core/tool.py

@@ -37,13 +37,15 @@ class ToolUseContext(BaseModel):
 
     message_id: Optional[str] = None
     agent_id: Optional[str] = None
-    safe_mode: bool = False
+    yolo_mode: bool = False
     verbose: bool = False
     permission_checker: Optional[Any] = None
     read_file_timestamps: Dict[str, float] = Field(default_factory=dict)
     # SkipValidation prevents Pydantic from copying the dict during validation,
     # ensuring Read and Edit tools share the same cache instance
-    file_state_cache: Annotated[Dict[str, FileSnapshot], SkipValidation] = Field(default_factory=dict)
+    file_state_cache: Annotated[Dict[str, FileSnapshot], SkipValidation] = Field(
+        default_factory=dict
+    )
     tool_registry: Optional[Any] = None
     abort_signal: Optional[Any] = None
     # UI control callbacks for tools that need user interaction
@@ -110,7 +112,7 @@ class Tool(ABC, Generic[TInput, TOutput]):
         pass
 
     @abstractmethod
-    async def prompt(self, safe_mode: bool = False) -> str:
+    async def prompt(self, yolo_mode: bool = False) -> str:
         """Get the system prompt for this tool."""
         pass
 
@@ -213,7 +215,8 @@ async def build_tool_description(
     except (TypeError, ValueError, AttributeError, KeyError) as exc:
         logger.warning(
             "[tool] Failed to build input example section: %s: %s",
-            type(exc).__name__, exc,
+            type(exc).__name__,
+            exc,
             extra={"tool": getattr(tool, "name", None)},
         )
         return description_text
@@ -233,7 +236,8 @@ def tool_input_examples(tool: Tool[Any, Any], limit: int = 5) -> List[Dict[str,
         except (TypeError, ValueError, AttributeError) as exc:
             logger.warning(
                 "[tool] Failed to format tool input example: %s: %s",
-                type(exc).__name__, exc,
+                type(exc).__name__,
+                exc,
                 extra={"tool": getattr(tool, "name", None)},
             )
             continue

ripperdoc/sdk/client.py

@@ -83,7 +83,7 @@ class RipperdocOptions:
     tools: Optional[Sequence[Tool[Any, Any]]] = None
     allowed_tools: Optional[Sequence[str]] = None
     disallowed_tools: Optional[Sequence[str]] = None
-    safe_mode: bool = False
+    yolo_mode: bool = False
     verbose: bool = False
     model: str = "main"
     max_thinking_tokens: int = 0
@@ -227,7 +227,7 @@ class RipperdocClient:
         query_context = QueryContext(
             tools=self._tools,
             max_thinking_tokens=self.options.max_thinking_tokens,
-            safe_mode=self.options.safe_mode,
+            yolo_mode=self.options.yolo_mode,
             model=self.options.model,
             verbose=self.options.verbose,
         )

ripperdoc/tools/ask_user_question_tool.py

@@ -228,7 +228,8 @@ async def prompt_user_for_answer(
         except (OSError, RuntimeError, ValueError) as e:
             logger.warning(
                 "[ask_user_question_tool] Error during prompt: %s: %s",
-                type(e).__name__, e,
+                type(e).__name__,
+                e,
             )
             return None
 
@@ -275,7 +276,7 @@ class AskUserQuestionTool(Tool[AskUserQuestionToolInput, AskUserQuestionToolOutp
     def input_schema(self) -> type[AskUserQuestionToolInput]:
         return AskUserQuestionToolInput
 
-    async def prompt(self, safe_mode: bool = False) -> str:  # noqa: ARG002
+    async def prompt(self, yolo_mode: bool = False) -> str:  # noqa: ARG002
         return ASK_USER_QUESTION_PROMPT
 
     def user_facing_name(self) -> str:
@@ -410,7 +411,8 @@ class AskUserQuestionTool(Tool[AskUserQuestionToolInput, AskUserQuestionToolOutp
         except (OSError, RuntimeError, ValueError, KeyError) as exc:
             logger.warning(
                 "[ask_user_question_tool] Error collecting answers: %s: %s",
-                type(exc).__name__, exc,
+                type(exc).__name__,
+                exc,
             )
             output = AskUserQuestionToolOutput(
                 questions=questions,

ripperdoc/tools/background_shell.py

@@ -162,7 +162,8 @@ async def _monitor_task(task: BackgroundTask) -> None:
     except (OSError, RuntimeError, ProcessLookupError) as exc:
         logger.warning(
             "Error monitoring background task: %s: %s",
-            type(exc).__name__, exc,
+            type(exc).__name__,
+            exc,
             extra={"task_id": task.id, "command": task.command},
         )
         with _tasks_lock:

ripperdoc/tools/bash_output_tool.py

@@ -40,7 +40,7 @@ class BashOutputTool(Tool[BashOutputInput, BashOutputData]):
     async def description(self) -> str:
         return "Read output and status from a background bash command started with BashTool(run_in_background=True)."
 
-    async def prompt(self, safe_mode: bool = False) -> str:
+    async def prompt(self, yolo_mode: bool = False) -> str:
         return "Fetch buffered output and status for a background bash task by id."
 
     @property

ripperdoc/tools/bash_tool.py

@@ -150,7 +150,7 @@ build projects, run tests, and interact with the file system."""
             ),
         ]
 
-    async def prompt(self, safe_mode: bool = False) -> str:
+    async def prompt(self, yolo_mode: bool = False) -> str:
         sandbox_available = is_sandbox_available()
         try:
             current_shell = find_suitable_shell()
@@ -398,6 +398,11 @@ build projects, run tests, and interact with the file system."""
 
         validation = validate_shell_command(input_data.command)
         if validation.behavior == "ask":
+            # In yolo mode, allow shell metacharacters
+            if context and hasattr(context, 'yolo_mode') and context.yolo_mode \
+               and "shell metacharacters" in validation.message:
+                # Allow commands with shell metacharacters in yolo mode
+                return ValidationResult(result=True)
             return ValidationResult(result=False, message=validation.message)
 
         return ValidationResult(result=True)
@@ -505,9 +510,7 @@ build projects, run tests, and interact with the file system."""
 
         return command, False
 
-    def _create_error_output(
-        self, command: str, stderr: str, sandbox: bool
-    ) -> BashToolOutput:
+    def _create_error_output(self, command: str, stderr: str, sandbox: bool) -> BashToolOutput:
         """Create a standardized error output."""
         return BashToolOutput(
             stdout="",
@@ -531,9 +534,13 @@ build projects, run tests, and interact with the file system."""
             return command, None, None
 
         if not is_sandbox_available():
-            return None, self._create_error_output(
-                command, "Sandbox mode requested but not available on this system", True
-            ), None
+            return (
+                None,
+                self._create_error_output(
+                    command, "Sandbox mode requested but not available on this system", True
+                ),
+                None,
+            )
 
         try:
             wrapper = create_sandbox_wrapper(command)
@@ -541,12 +548,15 @@ build projects, run tests, and interact with the file system."""
         except (OSError, RuntimeError, ValueError) as exc:
             logger.warning(
                 "[bash_tool] Failed to enable sandbox: %s: %s",
-                type(exc).__name__, exc,
+                type(exc).__name__,
+                exc,
                 extra={"command": command},
             )
-            return None, self._create_error_output(
-                command, f"Failed to enable sandbox: {exc}", True
-            ), None
+            return (
+                None,
+                self._create_error_output(command, f"Failed to enable sandbox: {exc}", True),
+                None,
+            )
 
     async def _run_background_command(
         self,
@@ -570,7 +580,8 @@ build projects, run tests, and interact with the file system."""
             # pragma: no cover - defensive import
             logger.warning(
                 "[bash_tool] Failed to import background shell runner: %s: %s",
-                type(e).__name__, e,
+                type(e).__name__,
+                e,
                 extra={"command": effective_command},
             )
             return self._create_error_output(
@@ -652,7 +663,7 @@ build projects, run tests, and interact with the file system."""
             # Emit progress updates for newly received output chunks
             while not queue.empty():
                 label, text = queue.get_nowait()
-                yield ToolProgress(content=f"{label}: {text}")
+                yield ToolProgress(content=f"{label}: {text}")  # type: ignore[misc]
 
             # Report progress at intervals
             if now - last_progress_time >= PROGRESS_INTERVAL_SECONDS:
@@ -660,7 +671,7 @@ build projects, run tests, and interact with the file system."""
                 if combined_output:
                     preview = get_last_n_lines(combined_output, 5)
                     elapsed = format_duration((now - start_time) * 1000)
-                    yield ToolProgress(content=f"Running... ({elapsed})\n{preview}")
+                    yield ToolProgress(content=f"Running... ({elapsed})\n{preview}")  # type: ignore[misc]
                 last_progress_time = now
 
             # Check timeout
@@ -696,9 +707,7 @@ build projects, run tests, and interact with the file system."""
         # Store results in a way that the caller can access
         self._last_execution_result = (stdout_lines, stderr_lines, timed_out)
 
-    async def _drain_stream(
-        self, stream: Optional[asyncio.StreamReader], sink: list[str]
-    ) -> None:
+    async def _drain_stream(self, stream: Optional[asyncio.StreamReader], sink: list[str]) -> None:
         """Drain any remaining data from a stream."""
         if not stream:
             return
@@ -907,14 +916,14 @@ build projects, run tests, and interact with the file system."""
 
                 while not queue.empty():
                     label, text = queue.get_nowait()
-                    yield ToolProgress(content=f"{label}: {text}")
+                    yield ToolProgress(content=f"{label}: {text}")  # type: ignore[misc]
 
                 if now - last_progress_time >= PROGRESS_INTERVAL_SECONDS:
                     combined_output = "".join(stdout_lines + stderr_lines)
                     if combined_output:
                         preview = get_last_n_lines(combined_output, 5)
                         elapsed = format_duration((now - start) * 1000)
-                        yield ToolProgress(content=f"Running... ({elapsed})\n{preview}")
+                        yield ToolProgress(content=f"Running... ({elapsed})\n{preview}")  # type: ignore[misc]
                     last_progress_time = now
 
                 if deadline is not None and now >= deadline:
@@ -969,7 +978,8 @@ build projects, run tests, and interact with the file system."""
                 raise  # Re-raise cancellation
             logger.warning(
                 "[bash_tool] Error executing command: %s: %s",
-                type(e).__name__, e,
+                type(e).__name__,
+                e,
                 extra={"command": effective_command},
             )
             error_output = self._create_error_output(

ripperdoc/tools/dynamic_mcp_tool.py

@@ -74,7 +74,8 @@ def _annotation_flag(tool_info: Any, key: str) -> bool:
         except (AttributeError, TypeError, KeyError) as exc:
             logger.debug(
                 "[mcp_tools] Failed to read annotation flag: %s: %s",
-                type(exc).__name__, exc,
+                type(exc).__name__,
+                exc,
             )
             return False
     return False
@@ -204,7 +205,7 @@ class DynamicMcpTool(Tool[BaseModel, McpToolCallOutput]):
     def input_schema(self) -> type[BaseModel]:
         return self._input_model
 
-    async def prompt(self, _safe_mode: bool = False) -> str:
+    async def prompt(self, _yolo_mode: bool = False) -> str:
         return await self.description()
 
     def is_read_only(self) -> bool:
@@ -321,7 +322,14 @@ class DynamicMcpTool(Tool[BaseModel, McpToolCallOutput]):
                 data=annotated_output,
                 result_for_assistant=final_text,
             )
-        except (OSError, RuntimeError, ConnectionError, ValueError, KeyError, TypeError) as exc:  # pragma: no cover - runtime errors
+        except (
+            OSError,
+            RuntimeError,
+            ConnectionError,
+            ValueError,
+            KeyError,
+            TypeError,
+        ) as exc:  # pragma: no cover - runtime errors
             output = McpToolCallOutput(
                 server=self.server_name,
                 tool=self.tool_info.name,
@@ -333,7 +341,8 @@ class DynamicMcpTool(Tool[BaseModel, McpToolCallOutput]):
             )
             logger.warning(
                 "Error calling MCP tool: %s: %s",
-                type(exc).__name__, exc,
+                type(exc).__name__,
+                exc,
                 extra={
                     "server": self.server_name,
                     "tool": self.tool_info.name,
@@ -381,10 +390,16 @@ def load_dynamic_mcp_tools_sync(project_path: Optional[Path] = None) -> List[Dyn
 
     try:
         return asyncio.run(_load_and_keep())
-    except (OSError, RuntimeError, ConnectionError, ValueError) as exc:  # pragma: no cover - SDK/runtime failures
+    except (
+        OSError,
+        RuntimeError,
+        ConnectionError,
+        ValueError,
+    ) as exc:  # pragma: no cover - SDK/runtime failures
         logger.warning(
             "Failed to initialize MCP runtime for dynamic tools (sync): %s: %s",
-            type(exc).__name__, exc,
+            type(exc).__name__,
+            exc,
         )
         return []
 
@@ -393,10 +408,16 @@ async def load_dynamic_mcp_tools_async(project_path: Optional[Path] = None) -> L
     """Async loader for MCP tools when already in an event loop."""
     try:
         runtime = await ensure_mcp_runtime(project_path)
-    except (OSError, RuntimeError, ConnectionError, ValueError) as exc:  # pragma: no cover - SDK/runtime failures
+    except (
+        OSError,
+        RuntimeError,
+        ConnectionError,
+        ValueError,
+    ) as exc:  # pragma: no cover - SDK/runtime failures
         logger.warning(
             "Failed to initialize MCP runtime for dynamic tools (async): %s: %s",
-            type(exc).__name__, exc,
+            type(exc).__name__,
+            exc,
         )
         return []
     return _build_dynamic_mcp_tools(runtime)

ripperdoc/tools/enter_plan_mode_tool.py

@@ -151,7 +151,7 @@ class EnterPlanModeTool(Tool[EnterPlanModeToolInput, EnterPlanModeToolOutput]):
     def input_schema(self) -> type[EnterPlanModeToolInput]:
         return EnterPlanModeToolInput
 
-    async def prompt(self, safe_mode: bool = False) -> str:  # noqa: ARG002
+    async def prompt(self, yolo_mode: bool = False) -> str:  # noqa: ARG002
         return ENTER_PLAN_MODE_PROMPT
 
     def user_facing_name(self) -> str:

ripperdoc/tools/exit_plan_mode_tool.py

@@ -84,7 +84,7 @@ class ExitPlanModeTool(Tool[ExitPlanModeToolInput, ExitPlanModeToolOutput]):
     def input_schema(self) -> type[ExitPlanModeToolInput]:
         return ExitPlanModeToolInput
 
-    async def prompt(self, safe_mode: bool = False) -> str:  # noqa: ARG002
+    async def prompt(self, yolo_mode: bool = False) -> str:  # noqa: ARG002
         return EXIT_PLAN_MODE_PROMPT
 
     def user_facing_name(self) -> str:

ripperdoc/tools/file_edit_tool.py

@@ -84,7 +84,7 @@ match exactly (including whitespace and indentation)."""
             ),
         ]
 
-    async def prompt(self, safe_mode: bool = False) -> str:
+    async def prompt(self, yolo_mode: bool = False) -> str:
         return (
             "Performs exact string replacements in files.\n\n"
             "Usage:\n"
@@ -159,7 +159,9 @@ match exactly (including whitespace and indentation)."""
 
         # Check if path is ignored (warning for edit operations)
         file_path_obj = Path(file_path)
-        should_proceed, warning_msg = check_path_for_tool(file_path_obj, tool_name="Edit", warn_only=True)
+        should_proceed, warning_msg = check_path_for_tool(
+            file_path_obj, tool_name="Edit", warn_only=True
+        )
         if warning_msg:
             logger.warning("[file_edit_tool] %s", warning_msg)
 
@@ -238,7 +240,8 @@ match exactly (including whitespace and indentation)."""
             except (OSError, IOError, RuntimeError) as exc:
                 logger.warning(
                     "[file_edit_tool] Failed to record file snapshot: %s: %s",
-                    type(exc).__name__, exc,
+                    type(exc).__name__,
+                    exc,
                     extra={"file_path": abs_file_path},
                 )
 
@@ -330,7 +333,8 @@ match exactly (including whitespace and indentation)."""
         except (OSError, IOError, PermissionError, UnicodeDecodeError, ValueError) as e:
             logger.warning(
                 "[file_edit_tool] Error editing file: %s: %s",
-                type(e).__name__, e,
+                type(e).__name__,
+                e,
                 extra={"file_path": input_data.file_path},
             )
             error_output = FileEditToolOutput(

ripperdoc/tools/file_read_tool.py

@@ -18,7 +18,7 @@ from ripperdoc.core.tool import (
 )
 from ripperdoc.utils.log import get_logger
 from ripperdoc.utils.file_watch import record_snapshot
-from ripperdoc.utils.path_ignore import check_path_for_tool, is_path_ignored
+from ripperdoc.utils.path_ignore import check_path_for_tool
 
 logger = get_logger()
 
@@ -70,7 +70,7 @@ and limit to read only a portion of the file."""
             ),
         ]
 
-    async def prompt(self, safe_mode: bool = False) -> str:
+    async def prompt(self, yolo_mode: bool = False) -> str:
         return (
             "Read a file from the local filesystem.\n\n"
             "Usage:\n"
@@ -106,7 +106,9 @@ and limit to read only a portion of the file."""
 
         # Check if path is ignored (warning only for read operations)
         file_path = Path(input_data.file_path)
-        should_proceed, warning_msg = check_path_for_tool(file_path, tool_name="Read", warn_only=True)
+        should_proceed, warning_msg = check_path_for_tool(
+            file_path, tool_name="Read", warn_only=True
+        )
         if warning_msg:
             logger.info("[file_read_tool] %s", warning_msg)
 
@@ -166,7 +168,8 @@ and limit to read only a portion of the file."""
             except (OSError, IOError, RuntimeError) as exc:
                 logger.warning(
                     "[file_read_tool] Failed to record file snapshot: %s: %s",
-                    type(exc).__name__, exc,
+                    type(exc).__name__,
+                    exc,
                     extra={"file_path": input_data.file_path},
                 )
 
@@ -185,7 +188,8 @@ and limit to read only a portion of the file."""
         except (OSError, IOError, UnicodeDecodeError, ValueError) as e:
             logger.warning(
                 "[file_read_tool] Error reading file: %s: %s",
-                type(e).__name__, e,
+                type(e).__name__,
+                e,
                 extra={"file_path": input_data.file_path},
             )
             # Create an error output

ripperdoc/tools/file_write_tool.py

@@ -72,10 +72,10 @@ the file if it already exists."""
             ),
         ]
 
-    async def prompt(self, safe_mode: bool = False) -> str:
+    async def prompt(self, yolo_mode: bool = False) -> str:
         prompt = """Use the Write tool to create new files. """
 
-        if safe_mode:
+        if not yolo_mode:
             prompt += """IMPORTANT: You must ALWAYS prefer editing existing files.
 NEVER write new files unless explicitly required by the user."""
 
@@ -134,7 +134,9 @@ NEVER write new files unless explicitly required by the user."""
 
         # Check if path is ignored (warning for write operations)
         file_path_obj = Path(file_path)
-        should_proceed, warning_msg = check_path_for_tool(file_path_obj, tool_name="Write", warn_only=True)
+        should_proceed, warning_msg = check_path_for_tool(
+            file_path_obj, tool_name="Write", warn_only=True
+        )
         if warning_msg:
             logger.warning("[file_write_tool] %s", warning_msg)
 
@@ -171,7 +173,8 @@ NEVER write new files unless explicitly required by the user."""
             except (OSError, IOError, RuntimeError) as exc:
                 logger.warning(
                     "[file_write_tool] Failed to record file snapshot: %s: %s",
-                    type(exc).__name__, exc,
+                    type(exc).__name__,
+                    exc,
                     extra={"file_path": abs_file_path},
                 )
 
@@ -189,7 +192,8 @@ NEVER write new files unless explicitly required by the user."""
         except (OSError, IOError, PermissionError, UnicodeEncodeError) as e:
             logger.warning(
                 "[file_write_tool] Error writing file: %s: %s",
-                type(e).__name__, e,
+                type(e).__name__,
+                e,
                 extra={"file_path": input_data.file_path},
             )
             error_output = FileWriteToolOutput(

ripperdoc/tools/glob_tool.py

@@ -76,7 +76,7 @@ class GlobTool(Tool[GlobToolInput, GlobToolOutput]):
             ),
         ]
 
-    async def prompt(self, _safe_mode: bool = False) -> str:
+    async def prompt(self, _yolo_mode: bool = False) -> str:
         return GLOB_USAGE
 
     def is_read_only(self) -> bool:
@@ -169,7 +169,8 @@ class GlobTool(Tool[GlobToolInput, GlobToolOutput]):
         except (OSError, RuntimeError, ValueError) as e:
             logger.warning(
                 "[glob_tool] Error executing glob: %s: %s",
-                type(e).__name__, e,
+                type(e).__name__,
+                e,
                 extra={"pattern": input_data.pattern, "path": input_data.path},
             )
             error_output = GlobToolOutput(matches=[], pattern=input_data.pattern, count=0)

ripperdoc/tools/grep_tool.py

@@ -148,7 +148,7 @@ class GrepTool(Tool[GrepToolInput, GrepToolOutput]):
             ),
         ]
 
-    async def prompt(self, _safe_mode: bool = False) -> str:
+    async def prompt(self, _yolo_mode: bool = False) -> str:
         return GREP_USAGE
 
     def is_read_only(self) -> bool:
@@ -358,7 +358,8 @@ class GrepTool(Tool[GrepToolInput, GrepToolOutput]):
         except (OSError, RuntimeError, ValueError, subprocess.SubprocessError) as e:
             logger.warning(
                 "[grep_tool] Error executing grep: %s: %s",
-                type(e).__name__, e,
+                type(e).__name__,
+                e,
                 extra={"pattern": input_data.pattern, "path": input_data.path},
             )
             error_output = GrepToolOutput(

ripperdoc/tools/kill_bash_tool.py

@@ -53,7 +53,7 @@ class KillBashTool(Tool[KillBashInput, KillBashOutput]):
     async def description(self) -> str:
         return "Kill a background bash shell by ID"
 
-    async def prompt(self, safe_mode: bool = False) -> str:
+    async def prompt(self, yolo_mode: bool = False) -> str:
         return KILL_BASH_PROMPT
 
     @property

ripperdoc/tools/ls_tool.py

@@ -320,7 +320,7 @@ class LSTool(Tool[LSToolInput, LSToolOutput]):
             ),
         ]
 
-    async def prompt(self, safe_mode: bool = False) -> str:
+    async def prompt(self, yolo_mode: bool = False) -> str:
         return (
             "Lists files and directories in a given path. The path parameter must be an absolute path, "
             "not a relative path. You can optionally provide an array of glob patterns to ignore with "