small tweaks

[robmck1995]

Description

• Refactored function names in LspManager for improved clarity and consistency.
• Enhanced file watching with debounce logic using notify_debouncer_mini.
• Improved range extraction logic with better error handling.
• Added PendingRequests handling in RustAnalyzerClient.
• Updated test cases to reflect changes in function names and logic.

---

Changes walkthrough

Relevant files

Enhancement

manager.rs Refactor LSP Manager function names and update tests                      lsproxy/src/lsp/manager.rs Refactored function names for clarity. Updated method calls to use new function names. Sorted workspace files for consistent output. Modified test cases to reflect function name changes. +43/-40  workspace_documents.rs Enhance file watching with debounce and improve range handling  lsproxy/src/lsp/workspace_documents.rs Replaced notify with notify_debouncer_mini for file watching. Added debounce logic to file change events. Improved range extraction logic with better error handling. Updated file change subscription to use debounced events. +56/-62  rust.rs Add pending requests handling in Rust Analyzer Client                    lsproxy/src/lsp/languages/rust.rs Added PendingRequests to RustAnalyzerClient. Updated workspace setup to use send_request. +9/-3

💡 Usage Guide

### Checking Your Pull Request Every time you make a pull request, our system automatically looks through it. We check for security issues, mistakes in how you're setting up your infrastructure, and common code problems. We do this to make sure your changes are solid and won't cause any trouble later. ### Talking to CodeAnt AI Got a question or need a hand with something in your pull request? You can easily get in touch with CodeAnt AI right here. Just type the following in a comment on your pull request, and replace "Your question here" with whatever you want to ask:

@codeant-ai ask: Your question here

This lets you have a chat with CodeAnt AI about your pull request, making it easier to understand and improve your code. ### Retrigger review Ask CodeAnt AI to review the PR again, by typing:

@codeant-ai: review

### Check Your Repository Health To analyze the health of your code repository, visit our dashboard at [app.codeant.ai](https://app.codeant.ai). This tool helps you identify potential issues and areas for improvement in your codebase, ensuring your repository maintains high standards of code health.

[codeant-ai[bot]]

Things to consider 🐛

• In the function definitions_in_file at line 34 in 'lsproxy/src/handlers/definitions_in_file.rs', the lock on lsp_manager is held while awaiting the result of definitions_in_file. This could lead to a deadlock if definitions_in_file tries to acquire the same lock internally. Consider restructuring the code to release the lock before awaiting.

• In the list_files function at line 199 in 'lsproxy/src/lsp/manager.rs', the method list_files is used to replace workspace_files. Ensure that the behavior of list_files is consistent with workspace_files, especially in terms of error handling and returned data, to avoid any regression issues.

• The test at line 91 in 'lsproxy/src/handlers/definitions_in_file.rs' checks for a 200 OK status but does not verify the actual content of the response. This could miss cases where the response structure or data is incorrect. Consider adding assertions to validate the response body.