Add intro and chat text options for PR code suggestions in configuration

[mrT23]

PR Type

Enhancement

---

Description

• Introduced new configuration options to enhance PR code suggestions:
  • enable_intro_text: Adds an introductory text for code suggestions
  • enable_chat_text: Provides a link to start a PR chat for additional feedback
• Updated pr_agent/tools/pr_code_suggestions.py to implement these new features:
  • Conditional rendering of intro text based on configuration
  • Addition of chat text with a link to PR chat extension (for GitHub only)
• Modified pr_agent/settings/configuration.toml to include new configuration options
• Improved user experience by providing more context and interaction options in PR reviews

---

Changes walkthrough 📝

	Relevant files
Enhancement	pr_code_suggestions.py Enhance PR code suggestions with intro and chat options    pr_agent/tools/pr_code_suggestions.py Added option to display chat text for PR feedback Introduced intro text for code suggestions Implemented conditional rendering of chat and intro text based on configuration +7/-0
Configuration changes	configuration.toml Add new configuration options for PR suggestions                  pr_agent/settings/configuration.toml Added new configuration options: enable_intro_text and enable_chat_text Set default values for new options +2/-0

---

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

[qodo-merge-pro]

PR Reviewer Guide 🔍

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🏅 Score: 95

🧪 No relevant tests

🔒 No security concerns identified

🔀 Multiple PR themes Sub-PR theme: Add intro and chat text options for PR code suggestions Relevant files: pr_agent/tools/pr_code_suggestions.py Sub-PR theme: Update configuration with new PR code suggestion options Relevant files: pr_agent/settings/configuration.toml

⚡ Key issues to review Code Duplication The condition get_settings().config.is_auto_command is repeated in two places. Consider extracting it to a variable for better maintainability.

[qodo-merge-pro]

PR Code Suggestions ✨

Category	Suggestion	Score
Maintainability	Extract hardcoded URLs to constants or configuration values Consider extracting the PR chat link to a constant or configuration value for easier maintenance and potential reuse. pr_agent/tools/pr_code_suggestions.py [148] -pr_body += "\n\n>💡 Need additional feedback ? start a [PR chat](https://chromewebstore.google.com/detail/ephlnjeghhogofkifjloamocljapahnl) \n\n" +PR_CHAT_LINK = "https://chromewebstore.google.com/detail/ephlnjeghhogofkifjloamocljapahnl" +pr_body += f"\n\n>💡 Need additional feedback ? start a [PR chat]({PR_CHAT_LINK}) \n\n" Apply this suggestion Suggestion importance[1-10]: 8 Why: Extracting hardcoded URLs to constants improves maintainability and makes future updates easier, reducing the risk of errors when the URL needs to be changed.	8
Organization best practice	Break down complex conditions into multiple lines or use intermediate variables Consider breaking down the long condition into multiple lines or using intermediate variables for better readability. pr_agent/tools/pr_code_suggestions.py [146-147] -if (get_settings().pr_code_suggestions.enable_chat_text and get_settings().config.is_auto_command - and isinstance(self.git_provider, GithubProvider)): +enable_chat = get_settings().pr_code_suggestions.enable_chat_text +is_auto_command = get_settings().config.is_auto_command +is_github_provider = isinstance(self.git_provider, GithubProvider) +if enable_chat and is_auto_command and is_github_provider: Apply this suggestion Suggestion importance[1-10]: 7 Why: The suggestion improves code readability by breaking down a complex condition, making it easier to understand and maintain.	7
	Group related configuration options together Group related configuration options together for better organization and readability. pr_agent/settings/configuration.toml [72-74] enable_intro_text=true +enable_chat_text=false enable_help_text=false # Determines whether to include help text in the PR review. Enabled by default. -enable_chat_text=false Apply this suggestion Suggestion importance[1-10]: 6 Why: Grouping related configuration options enhances code organization and readability, making it easier for developers to locate and modify settings.	6
	Remove unnecessary blank lines from the code Remove the unnecessary blank line to keep the code concise and clean. pr_agent/tools/pr_code_suggestions.py [154] - # Output the relevant configurations if enabled Apply this suggestion Suggestion importance[1-10]: 3 Why: While removing unnecessary blank lines can improve code cleanliness, it's a minor change with minimal impact on functionality or readability.	3
Best practice	Use f-strings for string formatting instead of concatenation Consider using f-strings for string formatting instead of concatenation for better readability and performance. pr_agent/tools/pr_code_suggestions.py [619] -pr_body += "Explore these optional code suggestions:\n\n" +pr_body += f"Explore these optional code suggestions:\n\n" Apply this suggestion Suggestion importance[1-10]: 4 Why: Using f-strings can improve readability slightly, but in this case, the performance gain is negligible, and the current code is already quite clear.	4

💡 Need additional feedback ? start a PR chat