Updates tool descriptions for clarity

Improves the descriptions of the google_search, list_files, read_file, and write_file tools to provide more context and detail about their functionality, security measures, and usage. This enhances understanding and usability.

Updates base_dir and relative paths descriptions in file tools.

Author: nullchimp

tests/test_tools_google_search_fixed.py

@@ -15,7 +15,7 @@ def test_name_property(self, google_search_tool):
         assert google_search_tool.name == "google_search"
 
     def test_description_property(self, google_search_tool):
-        expected_description = "Search the web for relevant information."
+        expected_description = "Perform web searches using Google Custom Search API to retrieve relevant information from the internet. Returns structured search results including titles, URLs, snippets, and metadata. Supports configurable result limits and provides search performance metrics. Requires valid Google API credentials and custom search engine configuration."
         assert google_search_tool.description == expected_description
 
     def test_parameters_property(self, google_search_tool):

tests/test_tools_list_files.py

@@ -7,7 +7,7 @@ class TestListFiles:
     def test_list_files_properties(self):
         list_files = ListFiles()
         assert list_files.name == "list_files"
-        assert list_files.description == "List files in a specified directory within a secure base directory."
+        assert list_files.description == "List files and directories within a specified directory path, constrained to operate within a secure base directory for security. Returns comprehensive file listing with metadata including file names, types, and directory structure. Supports recursive directory traversal within security boundaries."
 
         parameters = list_files.parameters
         assert parameters["type"] == "object"
@@ -26,7 +26,7 @@ def test_list_files_define_method(self):
 
         assert definition["type"] == "function"
         assert definition["function"]["name"] == "list_files"
-        assert definition["function"]["description"] == "List files in a specified directory within a secure base directory."
+        assert definition["function"]["description"] == "List files and directories within a specified directory path, constrained to operate within a secure base directory for security. Returns comprehensive file listing with metadata including file names, types, and directory structure. Supports recursive directory traversal within security boundaries."
         assert definition["function"]["parameters"]["type"] == "object"
 
     @pytest.mark.asyncio
@@ -122,13 +122,13 @@ def test_list_files_parameters_validation(self):
         # Check base_dir parameter
         base_dir_prop = params["properties"]["base_dir"]
         assert base_dir_prop["type"] == "string"
-        assert "Base directory" in base_dir_prop["description"]
+        assert "Absolute path to the base directory" in base_dir_prop["description"]
 
         # Check directory parameter
         directory_prop = params["properties"]["directory"]
         assert directory_prop["type"] == "string"
-        assert "relative subdirectory" in directory_prop["description"]
-        assert "default: '.'" in directory_prop["description"]
+        assert "Relative path to the subdirectory" in directory_prop["description"]
+        assert "Defaults to '.'" in directory_prop["description"]
 
     @pytest.mark.asyncio
     @patch('libs.fileops.file.FileService')

tests/test_tools_read_file.py

@@ -7,7 +7,7 @@ class TestReadFile:
     def test_read_file_properties(self):
         read_file = ReadFile()
         assert read_file.name == "read_file"
-        assert read_file.description == "Read content from a specified file within a secure base directory."
+        assert read_file.description == "Read and return the complete content of a specified file within security-constrained base directory boundaries. Supports text and binary file reading with proper error handling for missing files, permission issues, and encoding problems. File access is restricted to the specified base directory to prevent path traversal vulnerabilities."
 
         parameters = read_file.parameters
         assert parameters["type"] == "object"
@@ -26,7 +26,7 @@ def test_read_file_define_method(self):
 
         assert definition["type"] == "function"
         assert definition["function"]["name"] == "read_file"
-        assert definition["function"]["description"] == "Read content from a specified file within a secure base directory."
+        assert definition["function"]["description"] == "Read and return the complete content of a specified file within security-constrained base directory boundaries. Supports text and binary file reading with proper error handling for missing files, permission issues, and encoding problems. File access is restricted to the specified base directory to prevent path traversal vulnerabilities."
         assert definition["function"]["parameters"]["type"] == "object"
 
     @pytest.mark.asyncio
@@ -120,12 +120,12 @@ def test_read_file_parameters_validation(self):
         # Check base_dir parameter
         base_dir_prop = params["properties"]["base_dir"]
         assert base_dir_prop["type"] == "string"
-        assert "Base directory" in base_dir_prop["description"]
+        assert "Absolute path to the base directory" in base_dir_prop["description"]
 
         # Check filename parameter
         filename_prop = params["properties"]["filename"]
         assert filename_prop["type"] == "string"
-        assert "relative path" in filename_prop["description"]
+        assert "Relative path to the target file" in filename_prop["description"]
 
     @pytest.mark.asyncio
     @patch('libs.fileops.file.FileService')

tests/test_tools_write_file.py

@@ -6,7 +6,7 @@ class TestWriteFile:
     def test_write_file_properties(self):
         write_file = WriteFile()
         assert write_file.name == "write_file"
-        assert write_file.description == "Write content to a specified file within a secure base directory."
+        assert write_file.description == "Write or overwrite content to a specified file within security-constrained base directory boundaries. Creates directories as needed and handles text encoding automatically. Supports creating new files or updating existing ones with comprehensive error handling for permission issues, disk space, and path validation. File operations are restricted to the specified base directory to prevent path traversal vulnerabilities."
 
         parameters = write_file.parameters
         assert parameters["type"] == "object"
@@ -26,7 +26,7 @@ def test_write_file_define_method(self):
 
         assert definition["type"] == "function"
         assert definition["function"]["name"] == "write_file"
-        assert definition["function"]["description"] == "Write content to a specified file within a secure base directory."
+        assert definition["function"]["description"] == "Write or overwrite content to a specified file within security-constrained base directory boundaries. Creates directories as needed and handles text encoding automatically. Supports creating new files or updating existing ones with comprehensive error handling for permission issues, disk space, and path validation. File operations are restricted to the specified base directory to prevent path traversal vulnerabilities."
         assert definition["function"]["parameters"]["type"] == "object"
 
     @pytest.mark.asyncio
@@ -134,12 +134,12 @@ def test_write_file_parameters_validation(self):
         # Check base_dir parameter
         base_dir_prop = params["properties"]["base_dir"]
         assert base_dir_prop["type"] == "string"
-        assert "Base directory" in base_dir_prop["description"]
+        assert "Absolute path to the base directory" in base_dir_prop["description"]
 
         # Check filename parameter
         filename_prop = params["properties"]["filename"]
         assert filename_prop["type"] == "string"
-        assert "relative path" in filename_prop["description"]
+        assert "Relative path to the target file" in filename_prop["description"]
 
         # Check content parameter
         content_prop = params["properties"]["content"]