update_pull_request_reviewers should accept email addresses, not just identity GUIDs
#1043
kimsey0
started this conversation in
Backlog of Suggestions
Replies: 1 comment
-
|
we made making a pull request a skill and added a mapping table with name, mail, guid as resources so the skill easy finds the correct guid. Problem is that it is maintainable. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Summary
Adding a reviewer to a pull request currently requires a 3-step chain:
repo_get_repo_by_name_or_id-- resolve repository name to GUID (unless already known)core_get_identity_ids-- resolve email address (e.g.bfs@templafy.com) to identity GUID (e.g.3e13f7bd-d0b0-6ed9-a45d-b4e22eb7dc22)repo_update_pull_request_reviewers-- actually add the reviewer using the GUIDStep 2 exists solely because
reviewerIdsonly accepts identity GUIDs. The server already has thecore_get_identity_idstool and thus the capability to resolve identities -- it just doesn't do it inline.Example of the current workflow
Suggested fix
Accept email addresses or display names in
reviewerIdsin addition to GUIDs, and resolve them server-side. The server can detect whether a value is a GUID (matches UUID format) or a search string, and call the identity resolution API internally for non-GUID values.Alternatively, add a parallel parameter like
reviewerEmailsthat resolves emails to IDs before calling the Azure DevOps API.Why this matters
This is one of the most common PR workflows - adding a reviewer. The extra lookup step:
In my Claude Code sessions, this pattern appeared repeatedly across PR creation workflows.
Beta Was this translation helpful? Give feedback.
All reactions