Add 'original_suggestion' parameter to publish_inline_comment methods…

[mrT23]

User description

… across git providers for enhanced inline comment handling

---

PR Type

Enhancement

---

Description

• Added original_suggestion parameter to publish_inline_comment methods across all git providers (Azure DevOps, Bitbucket, Bitbucket Server, CodeCommit, Gerrit, GitHub, GitLab, and Local Git)
• Enhanced GitLab provider to handle original_suggestion in publish_code_suggestions method
• Updated PR reviewer tool to pass suggestion as original_suggestion when publishing inline comments
• Updated model_turbo in configuration to use "gpt-4o-2024-08-06"

---

Changes walkthrough 📝

	Relevant files
Enhancement	10 files azuredevops_provider.py Update Azure DevOps inline comment method                                pr_agent/git_providers/azuredevops_provider.py Added original_suggestion parameter to publish_inline_comment method +1/-1      bitbucket_provider.py Update Bitbucket inline comment method                                      pr_agent/git_providers/bitbucket_provider.py Added original_suggestion parameter to publish_inline_comment method +1/-1      bitbucket_server_provider.py Update Bitbucket Server inline comment method                        pr_agent/git_providers/bitbucket_server_provider.py Added original_suggestion parameter to publish_inline_comment method +1/-1      codecommit_provider.py Update CodeCommit inline comment method                                    pr_agent/git_providers/codecommit_provider.py Added original_suggestion parameter to publish_inline_comment method +1/-1      gerrit_provider.py Update Gerrit inline comment method                                            pr_agent/git_providers/gerrit_provider.py Added original_suggestion parameter to publish_inline_comment method +1/-1      git_provider.py Update abstract Git provider inline comment method              pr_agent/git_providers/git_provider.py Added original_suggestion parameter to abstract publish_inline_comment method +1/-1      github_provider.py Update GitHub inline comment method                                            pr_agent/git_providers/github_provider.py Added original_suggestion parameter to publish_inline_comment method +1/-1      gitlab_provider.py Enhance GitLab provider with original suggestion support  pr_agent/git_providers/gitlab_provider.py Added original_suggestion parameter to publish_inline_comment and send_inline_comment methods Updated publish_code_suggestions to handle cases with and without original_suggestion +8/-4      local_git_provider.py Update Local Git provider inline comment method                    pr_agent/git_providers/local_git_provider.py Added original_suggestion parameter to publish_inline_comment method +1/-1      pr_reviewer.py Update PR reviewer to use original suggestion                        pr_agent/tools/pr_reviewer.py Updated _publish_inline_code_comments to pass suggestion as original_suggestion parameter +1/-1
Configuration changes	1 files configuration.toml Update model configuration                                                              pr_agent/settings/configuration.toml Updated model_turbo to "gpt-4o-2024-08-06" +1/-1

---

💡 PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

[codiumai-pr-agent-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 'original_suggestion' parameter to publish_inline_comment methods Relevant files: pr_agent/git_providers/azuredevops_provider.py pr_agent/git_providers/bitbucket_provider.py pr_agent/git_providers/bitbucket_server_provider.py pr_agent/git_providers/codecommit_provider.py pr_agent/git_providers/gerrit_provider.py pr_agent/git_providers/git_provider.py pr_agent/git_providers/github_provider.py pr_agent/git_providers/local_git_provider.py Sub-PR theme: Enhance GitLab provider to handle 'original_suggestion' in publish_code_suggestions Relevant files: pr_agent/git_providers/gitlab_provider.py Sub-PR theme: Update PR reviewer tool and configuration Relevant files: pr_agent/tools/pr_reviewer.py pr_agent/settings/configuration.toml

⚡ Key issues to review Code Consistency The publish_code_suggestions method now checks for the presence of 'original_suggestion' in the suggestion dictionary. This change might need to be reflected in other parts of the codebase that interact with this method to ensure consistency.

[codiumai-pr-agent-pro]

PR Code Suggestions ✨

Category	Suggestion	Score
Possible issue	Verify the correctness and availability of the specified model name The model_turbo value has been changed from "gpt-4o" to "gpt-4o-2024-08-06". Ensure that this new model name is correct and available in your OpenAI API. If it's a typo or an unavailable model, it could cause issues when the code tries to use this model. pr_agent/settings/configuration.toml [4] -model_turbo="gpt-4o-2024-08-06" +model_turbo="gpt-4-0125-preview" Apply this suggestion Suggestion importance[1-10]: 8 Why: This suggestion highlights a potential critical issue with the model name, which could cause runtime errors if incorrect.	8
Error handling	Add error handling for the API call to improve robustness In the _publish_inline_code_comments method, consider adding error handling when calling publish_inline_comment. This will make the code more robust in case of network issues or API errors. pr_agent/tools/pr_reviewer.py [290] -self.git_provider.publish_inline_comment(content, relevant_file, relevant_line_in_file, suggestion) +try: + self.git_provider.publish_inline_comment(content, relevant_file, relevant_line_in_file, suggestion) +except Exception as e: + get_logger().error(f"Failed to publish inline comment: {e}") Apply this suggestion Suggestion importance[1-10]: 8 Why: This suggestion significantly improves the code's robustness by adding error handling for an external API call.	8
Enhancement	Simplify dictionary value retrieval with a default fallback In the publish_code_suggestions method, consider simplifying the logic for handling the original_suggestion. Instead of using a conditional statement, you could use the get method with a default value. pr_agent/git_providers/gitlab_provider.py [330-333] -if 'original_suggestion' in suggestion: - original_suggestion = suggestion['original_suggestion'] -else: - original_suggestion = suggestion +original_suggestion = suggestion.get('original_suggestion', suggestion) Apply this suggestion Suggestion importance[1-10]: 7 Why: This suggestion offers a more concise and Pythonic way to handle dictionary access, improving code readability.	7
	✅ Handle the case where the optional parameter is not provided Consider handling the case where original_suggestion is not provided in the publish_inline_comment method. You could use a default value or handle it explicitly to ensure consistent behavior. pr_agent/git_providers/gitlab_provider.py [214-219] def publish_inline_comment(self, body: str, relevant_file: str, relevant_line_in_file: str, original_suggestion=None): body = self.limit_output_characters(body, self.max_comment_chars) edit_type, found, source_line_no, target_file, target_line_no = self.search_line(relevant_file, relevant_line_in_file) self.send_inline_comment(body, edit_type, found, relevant_file, relevant_line_in_file, source_line_no, - target_file, target_line_no, original_suggestion) + target_file, target_line_no, original_suggestion or {}) [Suggestion has been applied] Suggestion importance[1-10]: 6 Why: The suggestion is valid and improves error handling, but it's a minor enhancement that doesn't address a critical issue.	6

[mrT23]

Generating PR code suggestions

Work in progress ...

[mrT23]

Generating PR code suggestions

Work in progress ...

[mrT23]

Generating PR code suggestions

Work in progress ...

[mrT23]

Generating PR code suggestions

Work in progress ...