Wrap every tool with exception handling

Author: CTY-git

patchwork/common/tools/grep_tool.py

@@ -164,7 +164,7 @@ def execute(
         is_case_sensitive: bool = False,
     ) -> str:
         if pattern is None:
-            raise ValueError("pattern argument is required!")
+            return "`pattern` argument is required!"
 
         if path is None:
             path = Path(self.__working_dir)
@@ -173,9 +173,12 @@ def execute(
         if is_case_sensitive:
             matcher = fnmatch.fnmatchcase
 
-        path = Path(path).resolve()
+        try:
+            path = Path(path).resolve()
+        except FileNotFoundError:
+            return f"`path` does not exist"
         if not path.is_relative_to(self.__working_dir):
-            raise ValueError("Path must be relative to working dir")
+            return f"Path must be relative to working dir {self.__working_dir}"
 
         if path.is_file():
             paths = [path]

patchwork/common/tools/tool.py

@@ -74,6 +74,9 @@ def execute_logging_wrapper(self, *args, **kwargs):
                 arg_text += f"kwargs: {kwargs}"
 
             logger.info(f"Executing Tool: {self.name} with {arg_text}")
-            return func(self, *args, **kwargs)
+            try:
+                return func(self, *args, **kwargs)
+            except Exception as e:
+                return f"Error: {e}"
 
         return execute_logging_wrapper

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "patchwork-cli"
-version = "0.0.112"
+version = "0.0.113.dev0"
 description = ""
 authors = ["patched.codes"]
 license = "AGPL"