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

ripperdoc/tools/bash_tool.py

@@ -49,6 +49,9 @@ from ripperdoc.utils.permissions.tool_permission_utils import (
 from ripperdoc.utils.permissions import PermissionDecision
 from ripperdoc.utils.sandbox_utils import create_sandbox_wrapper, is_sandbox_available
 from ripperdoc.utils.safe_get_cwd import get_original_cwd, safe_get_cwd
+from ripperdoc.utils.log import get_logger
+
+logger = get_logger()
 
 
 DEFAULT_TIMEOUT_MS = get_bash_default_timeout_ms()
@@ -134,11 +137,11 @@ build projects, run tests, and interact with the file system."""
         return [
             ToolUseExample(
                 description="Run a read-only listing in sandboxed mode",
-                input={"command": "ls -la", "sandbox": True, "timeout": 10000},
+                example={"command": "ls -la", "sandbox": True, "timeout": 10000},
             ),
             ToolUseExample(
                 description="Start a long task in the background with a timeout",
-                input={
+                example={
                     "command": "npm test",
                     "run_in_background": True,
                     "timeout": 600000,
@@ -395,10 +398,10 @@ build projects, run tests, and interact with the file system."""
         if output.duration_ms:
             timing = f" ({format_duration(output.duration_ms)}"
             if output.timeout_ms:
-                timing += f" / timeout {output.timeout_ms/1000:.0f}s"
+                timing += f" / timeout {output.timeout_ms / 1000:.0f}s"
             timing += ")"
         elif output.timeout_ms:
-            timing = f" (timeout {output.timeout_ms/1000:.0f}s)"
+            timing = f" (timeout {output.timeout_ms / 1000:.0f}s)"
 
         result_parts.append(f"{exit_code_text}{timing}")
 
@@ -516,6 +519,10 @@ build projects, run tests, and interact with the file system."""
                 final_command = wrapper.final_command
                 sandbox_cleanup = wrapper.cleanup
             except Exception as exc:
+                logger.exception(
+                    "[bash_tool] Failed to enable sandbox",
+                    extra={"command": effective_command, "error": str(exc)},
+                )
                 error_output = BashToolOutput(
                     stdout="",
                     stderr=f"Failed to enable sandbox: {exc}",
@@ -561,6 +568,10 @@ build projects, run tests, and interact with the file system."""
                 try:
                     from ripperdoc.tools.background_shell import start_background_command
                 except Exception as e:  # pragma: no cover - defensive import
+                    logger.exception(
+                        "[bash_tool] Failed to import background shell runner",
+                        extra={"command": effective_command},
+                    )
                     error_output = BashToolOutput(
                         stdout="",
                         stderr=f"Failed to start background task: {str(e)}",
@@ -767,6 +778,10 @@ build projects, run tests, and interact with the file system."""
             )
 
         except Exception as e:
+            logger.exception(
+                "[bash_tool] Error executing command",
+                extra={"command": effective_command, "error": str(e)},
+            )
             error_output = BashToolOutput(
                 stdout="",
                 stderr=f"Error executing command: {str(e)}",

ripperdoc/tools/file_edit_tool.py

@@ -15,6 +15,9 @@ from ripperdoc.core.tool import (
     ToolUseExample,
     ValidationResult,
 )
+from ripperdoc.utils.log import get_logger
+
+logger = get_logger()
 
 
 class FileEditToolInput(BaseModel):
@@ -60,7 +63,7 @@ match exactly (including whitespace and indentation)."""
         return [
             ToolUseExample(
                 description="Rename a function definition once",
-                input={
+                example={
                     "file_path": "/repo/src/app.py",
                     "old_string": "def old_name(",
                     "new_string": "def new_name(",
@@ -69,7 +72,7 @@ match exactly (including whitespace and indentation)."""
             ),
             ToolUseExample(
                 description="Replace every occurrence of a constant across a file",
-                input={
+                example={
                     "file_path": "/repo/src/config.ts",
                     "old_string": 'API_BASE = "http://localhost"',
                     "new_string": 'API_BASE = "https://api.example.com"',
@@ -268,6 +271,10 @@ match exactly (including whitespace and indentation)."""
             )
 
         except Exception as e:
+            logger.exception(
+                "[file_edit_tool] Error editing file",
+                extra={"file_path": input_data.file_path, "error": str(e)},
+            )
             error_output = FileEditToolOutput(
                 file_path=input_data.file_path,
                 replacements_made=0,

ripperdoc/tools/file_read_tool.py

@@ -15,6 +15,9 @@ from ripperdoc.core.tool import (
     ToolUseExample,
     ValidationResult,
 )
+from ripperdoc.utils.log import get_logger
+
+logger = get_logger()
 
 
 class FileReadToolInput(BaseModel):
@@ -56,11 +59,11 @@ and limit to read only a portion of the file."""
         return [
             ToolUseExample(
                 description="Read the top of a file to understand structure",
-                input={"file_path": "/repo/src/main.py", "limit": 50},
+                example={"file_path": "/repo/src/main.py", "limit": 50},
             ),
             ToolUseExample(
                 description="Inspect a slice of a large log without loading everything",
-                input={"file_path": "/repo/logs/server.log", "offset": 200, "limit": 40},
+                example={"file_path": "/repo/logs/server.log", "offset": 200, "limit": 40},
             ),
         ]
 
@@ -153,6 +156,10 @@ and limit to read only a portion of the file."""
             )
 
         except Exception as e:
+            logger.exception(
+                "[file_read_tool] Error reading file",
+                extra={"file_path": input_data.file_path, "error": str(e)},
+            )
             # Create an error output
             error_output = FileReadToolOutput(
                 content=f"Error reading file: {str(e)}",

ripperdoc/tools/file_write_tool.py

@@ -16,6 +16,9 @@ from ripperdoc.core.tool import (
     ToolUseExample,
     ValidationResult,
 )
+from ripperdoc.utils.log import get_logger
+
+logger = get_logger()
 
 
 class FileWriteToolInput(BaseModel):
@@ -53,11 +56,17 @@ the file if it already exists."""
         return [
             ToolUseExample(
                 description="Create a JSON fixture file",
-                input={"file_path": "/repo/tests/fixtures/sample.json", "content": '{\n  "items": []\n}\n'},
+                example={
+                    "file_path": "/repo/tests/fixtures/sample.json",
+                    "content": '{\n  "items": []\n}\n',
+                },
             ),
             ToolUseExample(
                 description="Write a short markdown note",
-                input={"file_path": "/repo/docs/USAGE.md", "content": "# Usage\n\nRun `make test`.\n"},
+                example={
+                    "file_path": "/repo/docs/USAGE.md",
+                    "content": "# Usage\n\nRun `make test`.\n",
+                },
             ),
         ]
 
@@ -128,6 +137,10 @@ NEVER write new files unless explicitly required by the user."""
             )
 
         except Exception as e:
+            logger.exception(
+                "[file_write_tool] Error writing file",
+                extra={"file_path": input_data.file_path, "error": str(e)},
+            )
             error_output = FileWriteToolOutput(
                 file_path=input_data.file_path,
                 bytes_written=0,

ripperdoc/tools/glob_tool.py

@@ -15,17 +15,22 @@ from ripperdoc.core.tool import (
     ToolUseExample,
     ValidationResult,
 )
+from ripperdoc.utils.log import get_logger
+
+logger = get_logger()
 
 
 GLOB_USAGE = (
-    "- Fast file pattern matching tool for any codebase size\n"
+    "- Fast file pattern matching tool that works with any codebase size\n"
     '- Supports glob patterns like "**/*.js" or "src/**/*.ts"\n'
-    "- Returns matching file paths sorted by modification time (newest first)\n"
-    "- Use this when you need to find files by name patterns\n"
-    "- For open-ended searches that need multiple rounds of globbing and grepping, run the searches iteratively with these tools\n"
-    "- You can call multiple tools in a single response; speculatively batch useful searches together"
+    "- Returns matching file paths sorted by modification time\n"
+    "- Use this tool when you need to find files by name patterns\n"
+    "- When you are doing an open ended search that may require multiple rounds of globbing and grepping, use the Agent tool instead\n"
+    "- You have the capability to call multiple tools in a single response. It is always better to speculatively perform multiple searches as a batch that are potentially useful.\n"
 )
 
+RESULT_LIMIT = 100
+
 
 class GlobToolInput(BaseModel):
     """Input schema for GlobTool."""
@@ -42,6 +47,7 @@ class GlobToolOutput(BaseModel):
     matches: List[str]
     pattern: str
     count: int
+    truncated: bool = False
 
 
 class GlobTool(Tool[GlobToolInput, GlobToolOutput]):
@@ -62,11 +68,11 @@ class GlobTool(Tool[GlobToolInput, GlobToolOutput]):
         return [
             ToolUseExample(
                 description="Find Python sources inside src",
-                input={"pattern": "src/**/*.py"},
+                example={"pattern": "src/**/*.py"},
             ),
             ToolUseExample(
                 description="Locate snapshot files within tests",
-                input={"pattern": "tests/**/__snapshots__/*.snap", "path": "/repo"},
+                example={"pattern": "tests/**/__snapshots__/*.snap", "path": "/repo"},
             ),
         ]
 
@@ -92,14 +98,34 @@ class GlobTool(Tool[GlobToolInput, GlobToolOutput]):
         if not output.matches:
             return f"No files found matching pattern: {output.pattern}"
 
-        result = f"Found {output.count} file(s) matching '{output.pattern}':\n\n"
-        result += "\n".join(output.matches)
-
-        return result
+        lines = list(output.matches)
+        if output.truncated:
+            lines.append("(Results are truncated. Consider using a more specific path or pattern.)")
+        return "\n".join(lines)
 
     def render_tool_use_message(self, input_data: GlobToolInput, verbose: bool = False) -> str:
         """Format the tool use for display."""
-        return f"Glob: {input_data.pattern}"
+        if not input_data.pattern:
+            return "Glob"
+
+        base_path = Path.cwd()
+        rendered_path = ""
+        if input_data.path:
+            candidate_path = Path(input_data.path)
+            absolute_path = candidate_path if candidate_path.is_absolute() else (base_path / candidate_path).resolve()
+
+            try:
+                relative_path = absolute_path.relative_to(base_path)
+            except ValueError:
+                relative_path = None
+
+            if verbose or not relative_path or str(relative_path) == ".":
+                rendered_path = str(absolute_path)
+            else:
+                rendered_path = str(relative_path)
+
+        path_fragment = f', path: "{rendered_path}"' if rendered_path else ""
+        return f'pattern: "{input_data.pattern}"{path_fragment}'
 
     async def call(
         self, input_data: GlobToolInput, context: ToolUseContext
@@ -108,9 +134,8 @@ class GlobTool(Tool[GlobToolInput, GlobToolOutput]):
 
         try:
             search_path = Path(input_data.path) if input_data.path else Path.cwd()
-
-            # Use glob to find matches, sorted by modification time (newest first)
-            paths = list(search_path.glob(input_data.pattern))
+            if not search_path.is_absolute():
+                search_path = (Path.cwd() / search_path).resolve()
 
             def _mtime(path: Path) -> float:
                 try:
@@ -118,15 +143,30 @@ class GlobTool(Tool[GlobToolInput, GlobToolOutput]):
                 except OSError:
                     return float("-inf")
 
-            matches = [str(p) for p in sorted(paths, key=_mtime, reverse=True)]
+            # Find matching files, sorted by modification time
+            paths = sorted(
+                (p for p in search_path.glob(input_data.pattern) if p.is_file()),
+                key=_mtime,
+            )
+
+            truncated = len(paths) > RESULT_LIMIT
+            paths = paths[:RESULT_LIMIT]
 
-            output = GlobToolOutput(matches=matches, pattern=input_data.pattern, count=len(matches))
+            matches = [str(p) for p in paths]
+
+            output = GlobToolOutput(
+                matches=matches, pattern=input_data.pattern, count=len(matches), truncated=truncated
+            )
 
             yield ToolResult(
                 data=output, result_for_assistant=self.render_result_for_assistant(output)
             )
 
         except Exception as e:
+            logger.exception(
+                "[glob_tool] Error executing glob",
+                extra={"pattern": input_data.pattern, "path": input_data.path},
+            )
             error_output = GlobToolOutput(matches=[], pattern=input_data.pattern, count=0)
 
             yield ToolResult(

ripperdoc/tools/grep_tool.py

@@ -15,6 +15,9 @@ from ripperdoc.core.tool import (
     ToolUseExample,
     ValidationResult,
 )
+from ripperdoc.utils.log import get_logger
+
+logger = get_logger()
 
 
 GREP_USAGE = (
@@ -80,11 +83,11 @@ class GrepTool(Tool[GrepToolInput, GrepToolOutput]):
         return [
             ToolUseExample(
                 description="Find TODO comments in TypeScript files",
-                input={"pattern": "TODO", "glob": "**/*.ts", "output_mode": "content"},
+                example={"pattern": "TODO", "glob": "**/*.ts", "output_mode": "content"},
             ),
             ToolUseExample(
                 description="List files referencing a function name",
-                input={
+                example={
                     "pattern": "fetchUserData",
                     "output_mode": "files_with_matches",
                     "path": "/repo/src",
@@ -223,6 +226,10 @@ class GrepTool(Tool[GrepToolInput, GrepToolOutput]):
             )
 
         except Exception as e:
+            logger.exception(
+                "[grep_tool] Error executing grep",
+                extra={"pattern": input_data.pattern, "path": input_data.path},
+            )
             error_output = GrepToolOutput(
                 matches=[], pattern=input_data.pattern, total_files=0, total_matches=0
             )