Skip to content

Conversation

@JegernOUTT
Copy link
Member

  • Introduce ToolRegexSearch to perform regex-based searches in specified file scopes.
  • Integrate new tool into tools_description and tools module.
  • Implement functionality for searching within entire workspace, specific directories, or individual files.
  • Support case-sensitive searches and limit results with 'max_results' parameter.
  • Provide detailed progress updates during search execution.
  • Handle invalid regex patterns gracefully with error messages.

@JegernOUTT JegernOUTT self-assigned this Feb 25, 2025
@JegernOUTT JegernOUTT force-pushed the engine/regex-search-tool branch 3 times, most recently from 96bb12d to b9a9adf Compare February 26, 2025 10:02
- Introduce `ToolRegexSearch` to perform regex-based searches in specified file scopes.
- Integrate new tool into tools_description and tools module.
- Implement functionality for searching within entire workspace, specific directories, or individual files.
- Support case-sensitive searches and limit results with 'max_results' parameter.
- Provide detailed progress updates during search execution.
- Handle invalid regex patterns gracefully with error messages.
Eliminate `case_sensitive` and `max_results` parameters to simplify the regex search tool. Update the regex pattern description to indicate the use of `(?i)` for case-insensitivity directly within the pattern. Adjust related code logic to accommodate the removal, ensuring that the search function no longer checks or enforces maximum result limits and handles pattern validation without conditional alterations for case sensitivity. This streamlines the function and reduces unnecessary complexity.
Introduce a new `scope_utils` module to streamline the process of resolving scope strings into file lists and creating SQL-like filters. Refactor existing search functionalities to utilize this common utility, reducing redundancy and improving maintainability. Implement functions `resolve_scope`, `create_scope_filter`, and `validate_scope_files` to handle scope-related operations efficiently. Update `tool_search.rs` and `tool_regex_search.rs` to integrate with the new utility, ensuring consistent scope handling across different tools.
@JegernOUTT JegernOUTT force-pushed the engine/regex-search-tool branch from b9a9adf to 0f1e741 Compare February 26, 2025 12:29
@olegklimov olegklimov merged commit a6c115f into main Feb 26, 2025
7 checks passed
@olegklimov olegklimov deleted the engine/regex-search-tool branch February 26, 2025 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants