agentic-labs / lsproxy

Multi-language code navigation API in a container
https://lsproxy.dev
GNU Affero General Public License v3.0
43 stars 2 forks source link

Fix ranges, workspace folders #51

Closed maks-ivanov closed 1 month ago

maks-ivanov commented 1 month ago

Description


Changes walkthrough

Relevant files
Enhancement
api_types.rs
Remove unused logs and enhance error handling                                   

lsproxy/src/api_types.rs
  • Removed unused debug log imports and statements.
  • Improved error handling with warn logs.
  • +1/-3     
    client.rs
    Optimize workspace folder handling                                                         

    lsproxy/src/lsp/client.rs
  • Removed inefficient workspace folder filtering.
  • Added warning for using root path as workspace.
  • +2/-14   
    python.rs
    Update Python root files list                                                                   

    lsproxy/src/lsp/languages/python.rs - Removed `.git` from Python root files list.
    +0/-1     
    typescript.rs
    Update TypeScript root files list                                                           

    lsproxy/src/lsp/languages/typescript.rs - Removed `.git` from TypeScript root files list.
    +1/-2     
    manager.rs
    Sort files in LspManager                                                                             

    lsproxy/src/lsp/manager.rs - Sorted files in `LspManager`.
    +1/-0     
    workspace_documents.rs
    Add debounce and enhance range extraction                                           

    lsproxy/src/lsp/workspace_documents.rs
  • Replaced notify with notify-debouncer-mini.
  • Added debounce to file watcher.
  • Improved range extraction with warnings for line adjustments.
  • +37/-23 
    Miscellaneous
    main.rs
    Update main imports                                                                                       

    lsproxy/src/main.rs - Added `set_mount_dir` import.
    +1/-1     
    Dependencies
    Cargo.toml
    Add notify-debouncer-mini dependency                                                     

    lsproxy/Cargo.toml - Added `notify-debouncer-mini` dependency.
    +1/-0     
    šŸ’” 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] commented 1 month ago

    Things to consider šŸ›