Summary of Changes

Hello @waruqi, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request enhances the reliability and structure of clang-tidy detection within xmake. By encapsulating the tool discovery process into a dedicated helper function and refining environment management, it ensures clang-tidy is found more consistently and improves the overall robustness of the static analysis integration.

Highlights

• Refactored clang-tidy detection: The logic for finding the clang-tidy executable has been moved into a new dedicated private function, _get_clang_tidy, centralizing the tool discovery process.
• Improved environment handling: Environment restoration (os.setenvs(oldenvs)) is now performed immediately after clang-tidy detection within the new _get_clang_tidy function, ensuring proper cleanup and scope management.
• Streamlined find_tool usage: The version = true option has been removed from find_tool calls when searching for clang-tidy, simplifying the detection process.
• Minor logging adjustment: A cprintf call in has_flags.lua was refactored into a single cprint for cleaner verbose output during flag checking.

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.