Set azure devops default status to ByDesign

[eddielu]

User description

Using a default status of Active = 1 causes issues because a common setup is to enable comment check for Azure PRs. Setting Active=1 means that reviewers need to manually mark each comment as resolved.

Use a better ByDesign status (https://learn.microsoft.com/en-us/javascript/api/azure-devops-extension-api/commentthreadstatus), which better fits what we see in github and gitlab

---

PR Type

Enhancement

---

Description

• Changed the default status of Azure DevOps comment threads from Active (1) to ByDesign (5)
• This change addresses issues with common PR review setups that use comment checks
• The new ByDesign status better aligns with GitHub and GitLab behavior
• Eliminates the need for reviewers to manually mark each comment as resolved

---

Changes walkthrough 📝

	Relevant files
Enhancement	azuredevops_provider.py Update Azure DevOps comment thread default status                pr_agent/git_providers/azuredevops_provider.py Changed the default status of CommentThread from 1 (Active) to 5 (ByDesign) +1/-1

---

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

[qodo-merge-pro]

PR Reviewer Guide 🔍

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

🏅 Score: 95

🧪 No relevant tests

🔒 No security concerns identified

🔀 No multiple PR themes

⚡ No key issues to review

[qodo-merge-pro]

PR Code Suggestions ✨

Category	Suggestion	Score
Maintainability	Use a named constant for the comment thread status value Consider using an enumeration or constant for the status value (5) to improve code readability and maintainability. This will make the intent clearer and reduce the risk of using incorrect status values. pr_agent/git_providers/azuredevops_provider.py [383] -thread = CommentThread(comments=[comment], thread_context=thread_context, status=5) +COMMENT_STATUS_BY_DESIGN = 5 +thread = CommentThread(comments=[comment], thread_context=thread_context, status=COMMENT_STATUS_BY_DESIGN) Apply this suggestion Suggestion importance[1-10]: 7 Why: The suggestion improves code readability and maintainability by using a named constant for the status value, making the intent clearer and reducing the risk of errors.	7

[mrT23]

seems reasonable