Skip to content

fix: enable Azure DevOps thread resolution - #3146

Merged
IsmaelMartinez merged 1 commit into
The-PR-Agent:mainfrom
Ricky-7-Yan:fix/3144-azure-thread-resolution
Sep 7, 2026
Merged

IsmaelMartinez merged 1 commit into
The-PR-Agent:mainfrom
Ricky-7-Yan:fix/3144-azure-thread-resolution

Conversation

@Ricky-7-Yan

Copy link
Copy Markdown
Contributor

Summary

  • advertise Azure DevOps' existing thread-resolution capability to /ask_line
  • map the generic resolve operation to set_thread_status(thread_id, "closed")
  • keep the provider-gated prompt comment accurate and add focused regression tests

Fixes #3144

Validation

  • uv run --python 3.12 --frozen pytest tests/unittest/test_azure_devops_provider.py tests/unittest/test_pr_questions_helpers.py -q — 192 passed
  • uv run --python 3.12 --frozen ruff check pr_agent/git_providers/azuredevops_provider.py pr_agent/tools/pr_line_questions.py tests/unittest/test_azure_devops_provider.py — passed

@github-actions github-actions Bot added the bug label Sep 7, 2026
@qodo-code-review

qodo-code-review Bot commented Sep 7, 2026 •

Copy link
Copy Markdown
Contributor

Code Review by Qodo

Grey Divider

Sorry, something went wrong

We weren't able to complete the code review on our side. Please try again manually by commenting /agentic_review on this PR.

Grey Divider

Qodo Logo

@IsmaelMartinez IsmaelMartinez left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for picking this up within twenty minutes of the issue going up, and for pinning both methods with tests. Verified end to end: the prompt block renders and the thread closes with set_thread_status(id, "closed").

@IsmaelMartinez
IsmaelMartinez merged commit 782a4e3 into The-PR-Agent:main Sep 7, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Azure DevOps ignores pr_questions.resolve_threads; /ask_line never closes the thread

2 participants