feat: add api rename code

[zong-zhe]

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

• N
• Y

add rename code api

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

• Affects user behaviors
• Contains syntax changes
• Contains variable changes
• Contains experimental features
• Performance regression: Consumes more CPU
• Performance regression: Consumes more Memory
• Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

• N
• Y

add api rename code

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

• Unit test
• Integration test
• Benchmark (add benchmark stats below)
• Manual test (add detailed scripts or steps below)
• Other

[coveralls]

Pull Request Test Coverage Report for Build 7337094836

• 540 of 565 (95.58%) changed or added relevant lines in 4 files are covered.
• 241 unchanged lines in 8 files lost coverage.
• Overall coverage decreased (-15.0%) to 72.768%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
kclvm/tools/src/LSP/src/util.rs	45	51	88.24%
kclvm/tools/src/LSP/src/rename.rs	481	500	96.2%

Files with Coverage Reduction	New Missed Lines	%
compiler_base/error/src/diagnostic/diagnostic_message.rs	3	0.0%
compiler_base/error/src/diagnostic/components.rs	6	0.0%
compiler_base/error/src/errors.rs	8	0.0%
compiler_base/error/src/diagnostic/diagnostic_handler.rs	11	0.0%
compiler_base/session/src/lib.rs	26	58.06%
compiler_base/error/src/diagnostic/mod.rs	27	0.0%
compiler_base/error/src/diagnostic/style.rs	41	0.0%
compiler_base/error/src/emitter.rs	119	0.0%

Totals
Change from base Build 7328439211:	-15.0%
Covered Lines:	43192
Relevant Lines:	59356

---

💛 - Coveralls

[Peefy]

@CodiumAI-Agent /review

[CodiumAI-Agent]

PR Analysis

(review updated until commit fc68e48)

• 🎯 Main theme: The PR introduces a new feature to rename symbols in the source code. It also includes some refactoring and bug fixes.
• 📝 PR summary: This PR adds a new feature to rename symbols in the source code. It provides two services: rename_symbol_on_file and rename_symbol_on_code. The former renames symbols in the files, while the latter renames symbols in the source code. The PR also includes some refactoring and bug fixes, such as handling cases where the file path is not valid and improving error handling.
• 📌 Type of PR: Enhancement
• 🧪 Relevant tests added: Yes
• ⏱️ Estimated effort to review [1-5]: 4, because the PR involves a significant amount of new code and changes to existing code. It also introduces a new feature which requires a good understanding of the existing codebase to review effectively.
• 🔒 Security concerns: No security concerns found

PR Feedback

• 💡 General suggestions: The PR is well-structured and the new feature is implemented in a clear and understandable way. However, there are several debug print statements left in the code which should be removed. Additionally, the PR could benefit from more detailed comments explaining the logic and purpose of the new functions and changes.

• 🤖 Code feedback:

  • relevant file: kclvm/tools/src/LSP/src/rename.rs
    suggestion: Remove debug print statements. They are useful during development but should be removed in the final version of the code. [important]
    relevant line: println!("symbol_spec: {:?}", symbol_spec);

  • relevant file: kclvm/tools/src/LSP/src/rename.rs
    suggestion: Consider adding more detailed comments to explain the logic and purpose of the new functions and changes. This will make the code easier to understand and maintain. [medium]
    relevant line: pub fn rename_symbol_on_code(

  • relevant file: kclvm/tools/src/LSP/src/rename.rs
    suggestion: The function apply_rename_changes could be refactored to reduce its complexity and improve readability. Consider breaking it down into smaller, more manageable functions. [medium]
    relevant line: fn apply_rename_changes(

  • relevant file: kclvm/tools/src/LSP/src/rename.rs
    suggestion: The function rename_symbol has a high cyclomatic complexity due to nested conditionals and loops. Consider refactoring it to reduce complexity and improve readability. [medium]
    relevant line: pub fn rename_symbol(

How to use

Instructions

Tag me in a comment '@CodiumAI-Agent' and add one of the following commands:
/review: Request a review of your Pull Request.
/describe: Update the PR title and description based on the contents of the PR.
/improve [--extended]: Suggest code improvements. Extended mode provides a higher quality feedback.
/ask <QUESTION>: Ask a question about the PR.
/update_changelog: Update the changelog based on the PR's contents.
/add_docs: Generate docstring for new components introduced in the PR.
/generate_labels: Generate labels for the PR based on the PR's contents.
see the tools guide for more details.

To edit any configuration parameter from the configuration.toml, add --config_path=new_value.
For example: /review --pr_reviewer.extra_instructions="focus on the file: ..."
To list the possible configuration parameters, add a /config comment.

[Peefy]

I think the path and source_codes in all functions are confusing and need to be readjusted or merged into one parameter.

[Peefy]

@CodiumAI-Agent /review

[CodiumAI-Agent]

Persistent review updated to latest commit fc68e48

[Peefy]

LGTM