Fix/infinite body generation

[blar-app-stg[bot]]

https://app-stg.blar.io/chat/397 Another PR is open; only one can be handled at a time.

[blar-app[bot]]

https://app.blar.io/chat/722 Another PR is open; only one can be handled at a time.

[blar-app[bot]]

Use the URL: https://app.blar.io/chat/722 Summary Report:

• PR Summary

This pull request involves a substantial refactor across key components of the codebase to enhance efficiency and flexibility. It specifically updates the TreeSitterHelper and DefinitionNode classes to switch from using body_text to body_node, improving direct node references. Additionally, it refines the main.py file to utilize environment variables for configuration purposes, promoting a more flexible setup process.

Key Findings

• The TreeSitterHelper and DefinitionNode classes now use body_node instead of body_text, enhancing interaction with TreeSitterNode by maintaining direct node references.
• Adjustments in tree_sitter_helper.py and node_factory.py ensure compatibility with the new body_node setup.
• The DefinitionNode class and its constructor are refactored to handle the updated data structure.
• A minor alteration in main.py removes the LspCaller import and updates root configurations to utilize environment variables instead of hardcoded paths, as shown in the provided code snippet.

[blar-app-stg[bot]]

Use the URL: https://app-stg.blar.io/chat/397 Summary Report:

• PR Summary

This pull request introduces several key refactors and updates to enhance flexibility and improve code handling. It primarily focuses on using structured nodes instead of text representations and utilizing environment variables for better configurability.

Key Findings

• The TreeSitterHelper, DefinitionNode, and NodeFactory classes, along with main.py, have been refactored to use body_node instead of body_text. This ensures consistent handling of TreeSitterNode objects across the codebase, improving flexibility and accuracy in code analysis.
• Within TreeSitterHelper, specific methods such as _create_file_node_from_module_node, _handle_definition_node, and _try_process_body_node_snippet have been updated. Similarly, the skeletonize method and its helpers in DefinitionNode are adjusted to work with body_node.
• The static methods in NodeFactory are now receiving body_node as a parameter, replacing body_text.
• Changes in main.py include updating the root_path and blarignore_path to be retrieved from environment variables, enhancing the application's configurability. The import statement for LspCaller was removed, and the repoId changes from "test" to "test-ruby".
• These changes are aimed at improving codebase consistency and allowing easier deployment across different environments without hardcoding paths.

[blar-app[bot]]

Use the URL: https://app.blar.io/chat/722

Cyber Security Report

❌ - Transition from using hardcoded paths to environment variables in `main.py`, enhancing security by reducing the risk of exposing sensitive information.

✅ Debugger Agent didn't find any issues

Optimizer Report

❌ - Refactoring to use `body_node` instead of `body_text` across `TreeSitterHelper` and `DefinitionNode` for improved accuracy and efficiency in node manipulation.

[blar-app-stg[bot]]

Use the URL: https://app-stg.blar.io/chat/397 ✅ Cyber Security Agent didn't find any issues

Debugger Report

❌ - `_handle_definition_node` does not validate `tree_sitter_node` structure, causing possible unexpected behavior. - Lack of error handling for unset environment variables in `main.py` could lead to runtime errors.

Optimizer Report

❌ - Refactor in `TreeSitterHelper` could increase algorithmic complexity and cause performance bottlenecks. - Possible redundant calculations in `NodeFactory` when handling `body_node` might lead to overhead. - Inefficient looping in `_traverse` method could degrade performance with deeply nested structures. - Potential scalability issues in `main.py` when dealing with large volumes or improperly set environment variables.