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

ripperdoc/core/permissions.py

@@ -26,8 +26,29 @@ class PermissionResult:
     decision: Optional[PermissionDecision] = None
 
 
-def _format_input_preview(parsed_input: Any) -> str:
-    """Create a short, human-friendly preview for prompts."""
+def _format_input_preview(parsed_input: Any, tool_name: Optional[str] = None) -> str:
+    """Create a human-friendly preview for prompts.
+
+    For Bash commands, shows full details for security review.
+    For other tools, shows a concise preview.
+    """
+    # For Bash tool, show full command details for security review
+    if tool_name == "Bash" and hasattr(parsed_input, "command"):
+        lines = [f"Command: {getattr(parsed_input, 'command')}"]
+
+        # Add other relevant parameters
+        if hasattr(parsed_input, "timeout") and parsed_input.timeout:
+            lines.append(f"Timeout: {parsed_input.timeout}ms")
+        if hasattr(parsed_input, "sandbox"):
+            lines.append(f"Sandbox: {parsed_input.sandbox}")
+        if hasattr(parsed_input, "run_in_background"):
+            lines.append(f"Background: {parsed_input.run_in_background}")
+        if hasattr(parsed_input, "shell_executable") and parsed_input.shell_executable:
+            lines.append(f"Shell: {parsed_input.shell_executable}")
+
+        return "\n  ".join(lines)
+
+    # For other tools with commands, show concise preview
     if hasattr(parsed_input, "command"):
         return f"command='{getattr(parsed_input, 'command')}'"
     if hasattr(parsed_input, "file_path"):
@@ -94,10 +115,13 @@ def _rule_strings(rule_suggestions: Optional[Any]) -> list[str]:
 
 def make_permission_checker(
     project_path: Path,
-    safe_mode: bool,
+    yolo_mode: bool,
     prompt_fn: Optional[Callable[[str], str]] = None,
 ) -> Callable[[Tool[Any, Any], Any], Awaitable[PermissionResult]]:
-    """Create a permission checking function for the current project."""
+    """Create a permission checking function for the current project.
+
+    In yolo mode, all tool calls are allowed without prompting.
+    """
 
     project_path = project_path.resolve()
     config_manager.get_project_config(project_path)
@@ -120,7 +144,7 @@ def make_permission_checker(
         """Check and optionally persist permission for a tool invocation."""
         config = config_manager.get_project_config(project_path)
 
-        if not safe_mode:
+        if yolo_mode:
             return PermissionResult(result=True)
 
         try:
@@ -130,7 +154,11 @@ def make_permission_checker(
             # Tool implementation error - log and deny for safety
             logger.warning(
                 "[permissions] Tool needs_permissions check failed",
-                extra={"tool": getattr(tool, "name", None), "error": str(exc), "error_type": type(exc).__name__},
+                extra={
+                    "tool": getattr(tool, "name", None),
+                    "error": str(exc),
+                    "error_type": type(exc).__name__,
+                },
             )
             return PermissionResult(
                 result=False,
@@ -172,7 +200,11 @@ def make_permission_checker(
                 # Tool implementation error - fall back to asking user
                 logger.warning(
                     "[permissions] Tool check_permissions failed",
-                    extra={"tool": getattr(tool, "name", None), "error": str(exc), "error_type": type(exc).__name__},
+                    extra={
+                        "tool": getattr(tool, "name", None),
+                        "error": str(exc),
+                        "error_type": type(exc).__name__,
+                    },
                 )
                 decision = PermissionDecision(
                     behavior="ask",
@@ -203,7 +235,7 @@ def make_permission_checker(
             )
 
         # Ask/passthrough flows prompt the user.
-        input_preview = _format_input_preview(parsed_input)
+        input_preview = _format_input_preview(parsed_input, tool_name=tool.name)
         prompt_lines = [
             f"{tool.name}",
             "",