refactor(search): remove max_line_length parameter from search strategies

Author: johnhuang316

src/code_index_mcp/search/ag.py

@@ -27,8 +27,7 @@ def search(
         context_lines: int = 0,
         file_pattern: Optional[str] = None,
         fuzzy: bool = False,
-        regex: bool = False,
-        max_line_length: Optional[int] = None
+        regex: bool = False
     ) -> Dict[str, List[Tuple[int, str]]]:
         """
         Execute a search using The Silver Searcher (ag).
@@ -41,7 +40,6 @@ def search(
             file_pattern: File pattern to filter
             fuzzy: Enable word boundary matching (not true fuzzy search)
             regex: Enable regex pattern matching
-            max_line_length: Optional. Limit the length of lines when context_lines is used
         """
         # ag prints line numbers and groups by file by default, which is good.
         # --noheading is used to be consistent with other tools' output format.
@@ -138,7 +136,7 @@ def search(
             if process.returncode > 1:
                  raise RuntimeError(f"ag failed with exit code {process.returncode}: {process.stderr}")
 
-            return parse_search_output(process.stdout, base_path, max_line_length)
+            return parse_search_output(process.stdout, base_path)
 
         except FileNotFoundError:
             raise RuntimeError("'ag' (The Silver Searcher) not found. Please install it and ensure it's in your PATH.")

src/code_index_mcp/search/base.py

@@ -19,16 +19,14 @@
 
 def parse_search_output(
     output: str,
-    base_path: str,
-    max_line_length: Optional[int] = None
+    base_path: str
 ) -> Dict[str, List[Tuple[int, str]]]:
     """
     Parse the output of command-line search tools (grep, ag, rg).
 
     Args:
         output: The raw output from the command-line tool.
         base_path: The base path of the project to make file paths relative.
-        max_line_length: Optional maximum line length to truncate long lines.
 
     Returns:
         A dictionary where keys are file paths and values are lists of (line_number, line_content) tuples.
@@ -84,10 +82,6 @@ def parse_search_output(
             # Normalize path separators for consistency
             relative_path = normalize_file_path(relative_path)
 
-            # Truncate content if it exceeds max_line_length
-            if max_line_length and len(content) > max_line_length:
-                content = content[:max_line_length] + '... (truncated)'
-
             if relative_path not in results:
                 results[relative_path] = []
             results[relative_path].append((line_number, content))
@@ -220,8 +214,7 @@ def search(
         context_lines: int = 0,
         file_pattern: Optional[str] = None,
         fuzzy: bool = False,
-        regex: bool = False,
-        max_line_length: Optional[int] = None
+        regex: bool = False
     ) -> Dict[str, List[Tuple[int, str]]]:
         """
         Execute a search using the specific strategy.

src/code_index_mcp/search/basic.py

@@ -47,8 +47,7 @@ def search(
         context_lines: int = 0,
         file_pattern: Optional[str] = None,
         fuzzy: bool = False,
-        regex: bool = False,
-        max_line_length: Optional[int] = None
+        regex: bool = False
     ) -> Dict[str, List[Tuple[int, str]]]:
         """
         Execute a basic, line-by-line search.
@@ -62,7 +61,6 @@ def search(
             file_pattern: File pattern to filter
             fuzzy: Enable word boundary matching
             regex: Enable regex pattern matching
-            max_line_length: Optional. Limit the length of lines when context_lines is used
         """
         results: Dict[str, List[Tuple[int, str]]] = {}
 
@@ -107,9 +105,6 @@ def search(
                         for line_num, line in enumerate(f, 1):
                             if search_regex.search(line):
                                 content = line.rstrip('\n')
-                                if max_line_length and len(content) > max_line_length:
-                                    content = content[:max_line_length] + '... (truncated)'
-
                                 if rel_path not in results:
                                     results[rel_path] = []
                                 results[rel_path].append((line_num, content))

src/code_index_mcp/search/grep.py

@@ -32,8 +32,7 @@ def search(
         context_lines: int = 0,
         file_pattern: Optional[str] = None,
         fuzzy: bool = False,
-        regex: bool = False,
-        max_line_length: Optional[int] = None
+        regex: bool = False
     ) -> Dict[str, List[Tuple[int, str]]]:
         """
         Execute a search using standard grep.
@@ -46,7 +45,6 @@ def search(
             file_pattern: File pattern to filter
             fuzzy: Enable word boundary matching
             regex: Enable regex pattern matching
-            max_line_length: Optional. Limit the length of lines when context_lines is used
         """
         # -r: recursive, -n: line number
         cmd = ['grep', '-r', '-n']
@@ -125,7 +123,7 @@ def search(
             if process.returncode > 1:
                  raise RuntimeError(f"grep failed with exit code {process.returncode}: {process.stderr}")
 
-            return parse_search_output(process.stdout, base_path, max_line_length)
+            return parse_search_output(process.stdout, base_path)
 
         except FileNotFoundError:
             raise RuntimeError("'grep' not found. Please install it and ensure it's in your PATH.")

src/code_index_mcp/search/ripgrep.py

@@ -27,8 +27,7 @@ def search(
         context_lines: int = 0,
         file_pattern: Optional[str] = None,
         fuzzy: bool = False,
-        regex: bool = False,
-        max_line_length: Optional[int] = None
+        regex: bool = False
     ) -> Dict[str, List[Tuple[int, str]]]:
         """
         Execute a search using ripgrep.
@@ -41,7 +40,6 @@ def search(
             file_pattern: File pattern to filter
             fuzzy: Enable word boundary matching (not true fuzzy search)
             regex: Enable regex pattern matching
-            max_line_length: Optional. Limit the length of lines when context_lines is used
         """
         cmd = ['rg', '--line-number', '--no-heading', '--color=never', '--no-ignore']
 
@@ -114,7 +112,7 @@ def search(
             if process.returncode > 1:
                 raise RuntimeError(f"ripgrep failed with exit code {process.returncode}: {process.stderr}")
 
-            return parse_search_output(process.stdout, base_path, max_line_length)
+            return parse_search_output(process.stdout, base_path)
 
         except FileNotFoundError:
             raise RuntimeError("ripgrep (rg) not found. Please install it and ensure it's in your PATH.")

src/code_index_mcp/search/ugrep.py

@@ -27,8 +27,7 @@ def search(
         context_lines: int = 0,
         file_pattern: Optional[str] = None,
         fuzzy: bool = False,
-        regex: bool = False,
-        max_line_length: Optional[int] = None
+        regex: bool = False
     ) -> Dict[str, List[Tuple[int, str]]]:
         """
         Execute a search using the 'ug' command-line tool.
@@ -41,7 +40,6 @@ def search(
             file_pattern: File pattern to filter
             fuzzy: Enable true fuzzy search (ugrep native support)
             regex: Enable regex pattern matching
-            max_line_length: Optional. Limit the length of lines when context_lines is used
         """
         if not self.is_available():
             return {"error": "ugrep (ug) command not found."}
@@ -115,7 +113,7 @@ def search(
                 error_output = process.stderr.strip()
                 return {"error": f"ugrep execution failed with code {process.returncode}", "details": error_output}
 
-            return parse_search_output(process.stdout, base_path, max_line_length)
+            return parse_search_output(process.stdout, base_path)
 
         except FileNotFoundError:
             return {"error": "ugrep (ug) command not found. Please ensure it's installed and in your PATH."}

src/code_index_mcp/server.py

@@ -164,8 +164,7 @@ def search_code_advanced(
     context_lines: int = 0,
     file_pattern: str = None,
     fuzzy: bool = False,
-    regex: bool = None,
-    max_line_length: int = None
+    regex: bool = None
 ) -> Dict[str, Any]:
     """
     Search for a code pattern in the project using an advanced, fast tool.
@@ -179,7 +178,6 @@ def search_code_advanced(
         context_lines: Number of lines to show before and after the match.
         file_pattern: A glob pattern to filter files to search in
                      (e.g., "*.py", "*.js", "test_*.py").
-        max_line_length: Optional. Default None (no limit). Limits the length of lines when context_lines is used.
                      All search tools now handle glob patterns consistently:
                      - ugrep: Uses glob patterns (*.py, *.{js,ts})
                      - ripgrep: Uses glob patterns (*.py, *.{js,ts})
@@ -208,8 +206,7 @@ def search_code_advanced(
         context_lines=context_lines,
         file_pattern=file_pattern,
         fuzzy=fuzzy,
-        regex=regex,
-        max_line_length=max_line_length
+        regex=regex
     )
 
 @mcp.tool()

src/code_index_mcp/services/search_service.py

@@ -27,8 +27,7 @@ def search_code(  # pylint: disable=too-many-arguments
         context_lines: int = 0,
         file_pattern: Optional[str] = None,
         fuzzy: bool = False,
-        regex: Optional[bool] = None,
-        max_line_length: Optional[int] = None
+        regex: Optional[bool] = None
     ) -> Dict[str, Any]:
         """Search for code patterns in the project."""
         self._require_project_setup()
@@ -62,8 +61,7 @@ def search_code(  # pylint: disable=too-many-arguments
                 context_lines=context_lines,
                 file_pattern=file_pattern,
                 fuzzy=fuzzy,
-                regex=regex,
-                max_line_length=max_line_length
+                regex=regex
             )
             filtered = self._filter_results(results)
             return ResponseFormatter.search_results_response(filtered)