Tr/update changelog fix

[mrT23]

PR Type

Enhancement

---

Description

• Enhanced changelog update functionality with PR link support
• Improved changelog update prompt with clearer instructions and better formatting
• Added new Bedrock Claude model configuration
• Improved response handling for changelog updates by cleaning up markdown formatting

---

Changes walkthrough 📝

	Relevant files
Configuration changes	__init__.py Add new Bedrock Claude model configuration                              pr_agent/algo/init.py Added new Bedrock Claude model configuration with token limit +1/-0      configuration.toml Add PR link configuration option                                                  pr_agent/settings/configuration.toml Added new configuration option 'add_pr_link' for changelog updates +1/-0
Enhancement	pr_update_changelog.py Enhance changelog updates with PR links and response handling pr_agent/tools/pr_update_changelog.py Added PR link support in changelog updates Improved response handling by cleaning up markdown formatting +10/-0    pr_update_changelog_prompts.toml Improve changelog update prompt instructions                          pr_agent/settings/pr_update_changelog_prompts.toml Updated system prompt for changelog updates with clearer instructions Added support for PR link formatting in changelog entries +13/-5

---

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

[qodo-merge-pro-for-open-source]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

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

🏅 Score: 95

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Code Duplication The strip() operation on response is performed twice - once in _get_prediction() and again in _prepare_changelog_update(). Consider consolidating the cleanup logic in one place. response = response.strip() if response.startswith("```"): response_lines = response.splitlines() response_lines = response_lines[1:] response = "\n".join(response_lines) response = response.strip("`") return response def _prepare_changelog_update(self) -> Tuple[str, str]: answer = self.prediction.strip().strip("```").strip() # noqa B005

[mrT23]

[qodo-merge-pro-for-open-source]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Score
Possible issue	✅ Add null check before string operations to prevent potential runtime errors Add error handling for the case when response is empty or None before attempting string operations. The current implementation could raise AttributeError if response is None. pr_agent/tools/pr_update_changelog.py [115-117] +if not response: + return "" response = response.strip() if response.startswith("```"): response_lines = response.splitlines() [Suggestion has been applied] Suggestion importance[1-10]: 8 Why: This is a critical defensive programming suggestion that prevents potential AttributeError crashes when the response is None or empty, which could happen in production environments.	8
	Add defensive check for configuration setting existence before access Handle potential KeyError when accessing settings by checking if 'add_pr_link' exists in settings before accessing it. pr_agent/tools/pr_update_changelog.py [106-107] -if get_settings().pr_update_changelog.add_pr_link: +if hasattr(get_settings().pr_update_changelog, 'add_pr_link') and get_settings().pr_update_changelog.add_pr_link: variables["pr_link"] = self.git_provider.get_pr_url() Apply this suggestion Suggestion importance[1-10]: 6 Why: Good defensive programming practice to handle potential KeyError when accessing settings, especially important since the 'add_pr_link' setting was just added and might not exist in older configurations.	6

• Author self-review: I have reviewed the PR code suggestions, and addressed the relevant ones.