@azure-devops/mcp 2.4.0 → 2.5.0-nightly.20260319

package/dist/tools/repositories.js

@@ -1,6 +1,6 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
-import { PullRequestStatus, GitVersionType, GitPullRequestQueryType, CommentThreadStatus, GitPullRequestMergeStrategy, } from "azure-devops-node-api/interfaces/GitInterfaces.js";
+import { PullRequestStatus, GitVersionType, GitPullRequestQueryType, CommentThreadStatus, GitPullRequestMergeStrategy, VersionControlRecursionType, } from "azure-devops-node-api/interfaces/GitInterfaces.js";
 import { z } from "zod";
 import { getCurrentUserDetails, getUserIdFromEmail } from "./auth.js";
 import { getEnumKeys } from "../utils.js";
@@ -23,6 +23,8 @@ const REPO_TOOLS = {
     update_pull_request_thread: "repo_update_pull_request_thread",
     search_commits: "repo_search_commits",
     list_pull_requests_by_commits: "repo_list_pull_requests_by_commits",
+    vote_pull_request: "repo_vote_pull_request",
+    list_directory: "repo_list_directory",
 };
 function branchesFilterOutIrrelevantProperties(branches, top) {
     return branches
@@ -84,6 +86,9 @@ function filterReposByName(repositories, repoNameFilter) {
     return filteredByName;
 }
 function trimPullRequest(pr, includeDescription = false) {
+    if (!pr) {
+        return null;
+    }
     return {
         pullRequestId: pr.pullRequestId,
         codeReviewId: pr.codeReviewId,
@@ -103,18 +108,36 @@ function trimPullRequest(pr, includeDescription = false) {
         project: pr.repository?.project?.name,
     };
 }
+// Helper function to build a version descriptor from branch or commit
+function buildVersionDescriptor(version, versionType) {
+    if (!version) {
+        return undefined;
+    }
+    const versionTypeMap = {
+        Branch: GitVersionType.Branch,
+        Commit: GitVersionType.Commit,
+        Tag: GitVersionType.Tag,
+    };
+    return {
+        version: version,
+        versionType: versionTypeMap[versionType || "Branch"] ?? GitVersionType.Branch,
+    };
+}
 function configureRepoTools(server, tokenProvider, connectionProvider, userAgentProvider) {
     server.tool(REPO_TOOLS.create_pull_request, "Create a new pull request.", {
-        repositoryId: z.string().describe("The ID of the repository where the pull request will be created."),
+        repositoryId: z
+            .string()
+            .describe("The ID or name of the repository where the pull request will be created. When using a repository name instead of a GUID, the project parameter must also be provided."),
         sourceRefName: z.string().describe("The source branch name for the pull request, e.g., 'refs/heads/feature-branch'."),
         targetRefName: z.string().describe("The target branch name for the pull request, e.g., 'refs/heads/main'."),
         title: z.string().describe("The title of the pull request."),
         description: z.string().max(4000).optional().describe("The description of the pull request. Must not be longer than 4000 characters. Optional."),
         isDraft: z.boolean().optional().default(false).describe("Indicates whether the pull request is a draft. Defaults to false."),
+        project: z.string().optional().describe("Project ID or project name. Required when repositoryId is a repository name instead of a GUID."),
         workItems: z.string().optional().describe("Work item IDs to associate with the pull request, space-separated."),
         forkSourceRepositoryId: z.string().optional().describe("The ID of the fork repository that the pull request originates from. Optional, used when creating a pull request from a fork."),
         labels: z.array(z.string()).optional().describe("Array of label names to add to the pull request after creation."),
-    }, async ({ repositoryId, sourceRefName, targetRefName, title, description, isDraft, workItems, forkSourceRepositoryId, labels }) => {
+    }, async ({ repositoryId, sourceRefName, targetRefName, title, description, isDraft, project, workItems, forkSourceRepositoryId, labels }) => {
         try {
             const connection = await connectionProvider();
             const gitApi = await connection.getGitApi();
@@ -127,7 +150,7 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
                 }
                 : undefined;
             const labelDefinitions = labels ? labels.map((label) => ({ name: label })) : undefined;
-            const pullRequest = await gitApi.createPullRequest({
+            let pullRequest = await gitApi.createPullRequest({
                 sourceRefName,
                 targetRefName,
                 title,
@@ -136,8 +159,25 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
                 workItemRefs: workItemRefs,
                 forkSource,
                 labels: labelDefinitions,
-            }, repositoryId);
+                supportsIterations: true,
+            }, repositoryId, project);
+            if (!pullRequest) {
+                const prs = await gitApi.getPullRequests(repositoryId, { sourceRefName, targetRefName, status: PullRequestStatus.Active }, project, undefined, 0, 1);
+                if (prs && prs.length > 0) {
+                    pullRequest = prs[0];
+                }
+                else {
+                    return {
+                        content: [{ type: "text", text: "Pull request created but API returned no data." }],
+                    };
+                }
+            }
             const trimmedPullRequest = trimPullRequest(pullRequest, true);
+            if (!trimmedPullRequest) {
+                return {
+                    content: [{ type: "text", text: "Pull request created but API returned no data." }],
+                };
+            }
             return {
                 content: [{ type: "text", text: JSON.stringify(trimmedPullRequest, null, 2) }],
             };
@@ -151,11 +191,14 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
         }
     });
     server.tool(REPO_TOOLS.create_branch, "Create a new branch in the repository.", {
-        repositoryId: z.string().describe("The ID of the repository where the branch will be created."),
+        repositoryId: z
+            .string()
+            .describe("The ID or name of the repository where the branch will be created. When using a repository name instead of a GUID, the project parameter must also be provided."),
         branchName: z.string().describe("The name of the new branch to create, e.g., 'feature-branch'."),
         sourceBranchName: z.string().optional().default("main").describe("The name of the source branch to create the new branch from. Defaults to 'main'."),
         sourceCommitId: z.string().optional().describe("The commit ID to create the branch from. If not provided, uses the latest commit of the source branch."),
-    }, async ({ repositoryId, branchName, sourceBranchName, sourceCommitId }) => {
+        project: z.string().optional().describe("Project ID or project name. Required when repositoryId is a repository name instead of a GUID."),
+    }, async ({ repositoryId, branchName, sourceBranchName, sourceCommitId, project }) => {
         try {
             const connection = await connectionProvider();
             const gitApi = await connection.getGitApi();
@@ -164,7 +207,7 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
             if (!commitId) {
                 const sourceRefName = `refs/heads/${sourceBranchName}`;
                 try {
-                    const sourceBranch = await gitApi.getRefs(repositoryId, undefined, "heads/", false, false, undefined, false, undefined, sourceBranchName);
+                    const sourceBranch = await gitApi.getRefs(repositoryId, project, "heads/", false, false, undefined, false, undefined, sourceBranchName);
                     const branch = sourceBranch.find((b) => b.name === sourceRefName);
                     if (!branch || !branch.objectId) {
                         return {
@@ -199,7 +242,7 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
                 oldObjectId: "0000000000000000000000000000000000000000", // All zeros indicates creating a new ref
             };
             try {
-                const result = await gitApi.updateRefs([refUpdate], repositoryId);
+                const result = await gitApi.updateRefs([refUpdate], repositoryId, project);
                 // Check if the branch creation was successful
                 if (result && result.length > 0 && result[0].success) {
                     return {
@@ -245,8 +288,9 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
         }
     });
     server.tool(REPO_TOOLS.update_pull_request, "Update a Pull Request by ID with specified fields, including setting autocomplete with various completion options.", {
-        repositoryId: z.string().describe("The ID of the repository where the pull request exists."),
+        repositoryId: z.string().describe("The ID or name of the repository where the pull request exists. When using a repository name instead of a GUID, the project parameter must also be provided."),
         pullRequestId: z.number().describe("The ID of the pull request to update."),
+        project: z.string().optional().describe("Project ID or project name. Required when repositoryId is a repository name instead of a GUID."),
         title: z.string().optional().describe("The new title for the pull request."),
         description: z.string().max(4000).optional().describe("The new description for the pull request. Must not be longer than 4000 characters."),
         isDraft: z.boolean().optional().describe("Whether the pull request should be a draft."),
@@ -261,7 +305,7 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
         transitionWorkItems: z.boolean().optional().default(true).describe("Whether to transition associated work items to the next state when the pull request autocompletes. Defaults to true."),
         bypassReason: z.string().optional().describe("Reason for bypassing branch policies. When provided, branch policies will be automatically bypassed during autocompletion."),
         labels: z.array(z.string()).optional().describe("Array of label names to replace existing labels on the pull request. This will remove all current labels and add the specified ones."),
-    }, async ({ repositoryId, pullRequestId, title, description, isDraft, targetRefName, status, autoComplete, mergeStrategy, deleteSourceBranch, transitionWorkItems, bypassReason, labels }) => {
+    }, async ({ repositoryId, pullRequestId, project, title, description, isDraft, targetRefName, status, autoComplete, mergeStrategy, deleteSourceBranch, transitionWorkItems, bypassReason, labels, }) => {
         try {
             const connection = await connectionProvider();
             const gitApi = await connection.getGitApi();
@@ -310,25 +354,30 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
             }
             // Update labels if provided
             if (labels) {
-                const currentLabels = await gitApi.getPullRequestLabels(repositoryId, pullRequestId);
+                const currentLabels = await gitApi.getPullRequestLabels(repositoryId, pullRequestId, project);
                 for (const currentLabel of currentLabels) {
                     if (currentLabel.id) {
-                        await gitApi.deletePullRequestLabels(repositoryId, pullRequestId, currentLabel.id);
+                        await gitApi.deletePullRequestLabels(repositoryId, pullRequestId, currentLabel.id, project);
                     }
                 }
                 for (const label of labels) {
-                    await gitApi.createPullRequestLabel({ name: label }, repositoryId, pullRequestId);
+                    await gitApi.createPullRequestLabel({ name: label }, repositoryId, pullRequestId, project);
                 }
             }
             let updatedPullRequest;
             if (Object.keys(updateRequest).length > 0) {
-                updatedPullRequest = await gitApi.updatePullRequest(updateRequest, repositoryId, pullRequestId);
+                updatedPullRequest = await gitApi.updatePullRequest(updateRequest, repositoryId, pullRequestId, project);
             }
             else {
                 // If only labels were updated, get the current pull request
-                updatedPullRequest = await gitApi.getPullRequest(repositoryId, pullRequestId);
+                updatedPullRequest = await gitApi.getPullRequest(repositoryId, pullRequestId, project);
             }
             const trimmedUpdatedPullRequest = trimPullRequest(updatedPullRequest, true);
+            if (!trimmedUpdatedPullRequest) {
+                return {
+                    content: [{ type: "text", text: "Pull request updated but API returned no data." }],
+                };
+            }
             return {
                 content: [{ type: "text", text: JSON.stringify(trimmedUpdatedPullRequest, null, 2) }],
             };
@@ -342,17 +391,18 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
         }
     });
     server.tool(REPO_TOOLS.update_pull_request_reviewers, "Add or remove reviewers for an existing pull request.", {
-        repositoryId: z.string().describe("The ID of the repository where the pull request exists."),
+        repositoryId: z.string().describe("The ID or name of the repository where the pull request exists. When using a repository name instead of a GUID, the project parameter must also be provided."),
         pullRequestId: z.number().describe("The ID of the pull request to update."),
         reviewerIds: z.array(z.string()).describe("List of reviewer ids to add or remove from the pull request."),
         action: z.enum(["add", "remove"]).describe("Action to perform on the reviewers. Can be 'add' or 'remove'."),
-    }, async ({ repositoryId, pullRequestId, reviewerIds, action }) => {
+        project: z.string().optional().describe("Project ID or project name. Required when repositoryId is a repository name instead of a GUID."),
+    }, async ({ repositoryId, pullRequestId, reviewerIds, action, project }) => {
         try {
             const connection = await connectionProvider();
             const gitApi = await connection.getGitApi();
             let updatedPullRequest;
             if (action === "add") {
-                updatedPullRequest = await gitApi.createPullRequestReviewers(reviewerIds.map((id) => ({ id: id })), repositoryId, pullRequestId);
+                updatedPullRequest = await gitApi.createPullRequestReviewers(reviewerIds.map((id) => ({ id: id })), repositoryId, pullRequestId, project);
                 const trimmedResponse = updatedPullRequest.map((item) => ({
                     displayName: item.displayName,
                     id: item.id,
@@ -367,7 +417,7 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
             }
             else {
                 for (const reviewerId of reviewerIds) {
-                    await gitApi.deletePullRequestReviewer(repositoryId, pullRequestId, reviewerId);
+                    await gitApi.deletePullRequestReviewer(repositoryId, pullRequestId, reviewerId, project);
                 }
                 return {
                     content: [{ type: "text", text: `Reviewers with IDs ${reviewerIds.join(", ")} removed from pull request ${pullRequestId}.` }],
@@ -417,8 +467,11 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
         }
     });
     server.tool(REPO_TOOLS.list_pull_requests_by_repo_or_project, "Retrieve a list of pull requests for a given repository. Either repositoryId or project must be provided.", {
-        repositoryId: z.string().optional().describe("The ID of the repository where the pull requests are located."),
-        project: z.string().optional().describe("The ID of the project where the pull requests are located."),
+        repositoryId: z
+            .string()
+            .optional()
+            .describe("The ID or name of the repository where the pull requests are located. When using a repository name instead of a GUID, the project parameter must also be provided."),
+        project: z.string().optional().describe("Project ID or project name. Required when repositoryId is a repository name instead of a GUID, or to scope the search to a specific project."),
         top: z.number().default(100).describe("The maximum number of pull requests to return."),
         skip: z.number().default(0).describe("The number of pull requests to skip."),
         created_by_me: z.boolean().default(false).describe("Filter pull requests created by the current user."),
@@ -543,9 +596,11 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
         }
     });
     server.tool(REPO_TOOLS.list_pull_request_threads, "Retrieve a list of comment threads for a pull request.", {
-        repositoryId: z.string().describe("The ID of the repository where the pull request is located."),
+        repositoryId: z
+            .string()
+            .describe("The ID or name of the repository where the pull request is located. When using a repository name instead of a GUID, the project parameter must also be provided."),
         pullRequestId: z.number().describe("The ID of the pull request for which to retrieve threads."),
-        project: z.string().optional().describe("Project ID or project name (optional)"),
+        project: z.string().optional().describe("Project ID or project name. Required when repositoryId is a repository name instead of a GUID."),
         iteration: z.number().optional().describe("The iteration ID for which to retrieve threads. Optional, defaults to the latest iteration."),
         baseIteration: z.number().optional().describe("The base iteration ID for which to retrieve threads. Optional, defaults to the latest base iteration."),
         top: z.number().default(100).describe("The maximum number of threads to return after filtering."),
@@ -601,10 +656,12 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
         }
     });
     server.tool(REPO_TOOLS.list_pull_request_thread_comments, "Retrieve a list of comments in a pull request thread.", {
-        repositoryId: z.string().describe("The ID of the repository where the pull request is located."),
+        repositoryId: z
+            .string()
+            .describe("The ID or name of the repository where the pull request is located. When using a repository name instead of a GUID, the project parameter must also be provided."),
         pullRequestId: z.number().describe("The ID of the pull request for which to retrieve thread comments."),
         threadId: z.number().describe("The ID of the thread for which to retrieve comments."),
-        project: z.string().optional().describe("Project ID or project name (optional)"),
+        project: z.string().optional().describe("Project ID or project name. Required when repositoryId is a repository name instead of a GUID."),
         top: z.number().default(100).describe("The maximum number of comments to return."),
         skip: z.number().default(0).describe("The number of comments to skip."),
         fullResponse: z.boolean().optional().default(false).describe("Return full comment JSON response instead of trimmed data."),
@@ -635,14 +692,17 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
         }
     });
     server.tool(REPO_TOOLS.list_branches_by_repo, "Retrieve a list of branches for a given repository.", {
-        repositoryId: z.string().describe("The ID of the repository where the branches are located."),
+        repositoryId: z
+            .string()
+            .describe("The ID or name of the repository where the branches are located. When using a repository name instead of a GUID, the project parameter must also be provided."),
         top: z.number().default(100).describe("The maximum number of branches to return. Defaults to 100."),
         filterContains: z.string().optional().describe("Filter to find branches that contain this string in their name."),
-    }, async ({ repositoryId, top, filterContains }) => {
+        project: z.string().optional().describe("Project ID or project name. Required when repositoryId is a repository name instead of a GUID."),
+    }, async ({ repositoryId, top, filterContains, project }) => {
         try {
             const connection = await connectionProvider();
             const gitApi = await connection.getGitApi();
-            const branches = await gitApi.getRefs(repositoryId, undefined, "heads/", undefined, undefined, undefined, undefined, undefined, filterContains);
+            const branches = await gitApi.getRefs(repositoryId, project, "heads/", undefined, undefined, undefined, undefined, undefined, filterContains);
             const filteredBranches = branchesFilterOutIrrelevantProperties(branches, top);
             return {
                 content: [{ type: "text", text: JSON.stringify(filteredBranches, null, 2) }],
@@ -657,14 +717,17 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
         }
     });
     server.tool(REPO_TOOLS.list_my_branches_by_repo, "Retrieve a list of my branches for a given repository Id.", {
-        repositoryId: z.string().describe("The ID of the repository where the branches are located."),
+        repositoryId: z
+            .string()
+            .describe("The ID or name of the repository where the branches are located. When using a repository name instead of a GUID, the project parameter must also be provided."),
         top: z.number().default(100).describe("The maximum number of branches to return."),
         filterContains: z.string().optional().describe("Filter to find branches that contain this string in their name."),
-    }, async ({ repositoryId, top, filterContains }) => {
+        project: z.string().optional().describe("Project ID or project name. Required when repositoryId is a repository name instead of a GUID."),
+    }, async ({ repositoryId, top, filterContains, project }) => {
         try {
             const connection = await connectionProvider();
             const gitApi = await connection.getGitApi();
-            const branches = await gitApi.getRefs(repositoryId, undefined, "heads/", undefined, undefined, true, undefined, undefined, filterContains);
+            const branches = await gitApi.getRefs(repositoryId, project, "heads/", undefined, undefined, true, undefined, undefined, filterContains);
             const filteredBranches = branchesFilterOutIrrelevantProperties(branches, top);
             return {
                 content: [{ type: "text", text: JSON.stringify(filteredBranches, null, 2) }],
@@ -706,13 +769,14 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
         }
     });
     server.tool(REPO_TOOLS.get_branch_by_name, "Get a branch by its name.", {
-        repositoryId: z.string().describe("The ID of the repository where the branch is located."),
+        repositoryId: z.string().describe("The ID or name of the repository where the branch is located. When using a repository name instead of a GUID, the project parameter must also be provided."),
         branchName: z.string().describe("The name of the branch to retrieve, e.g., 'main' or 'feature-branch'."),
-    }, async ({ repositoryId, branchName }) => {
+        project: z.string().optional().describe("Project ID or project name. Required when repositoryId is a repository name instead of a GUID."),
+    }, async ({ repositoryId, branchName, project }) => {
         try {
             const connection = await connectionProvider();
             const gitApi = await connection.getGitApi();
-            const branches = await gitApi.getRefs(repositoryId, undefined, "heads/", false, false, undefined, false, undefined, branchName);
+            const branches = await gitApi.getRefs(repositoryId, project, "heads/", false, false, undefined, false, undefined, branchName);
             const branch = branches.find((branch) => branch.name === `refs/heads/${branchName}` || branch.name === branchName);
             if (!branch) {
                 return {
@@ -738,15 +802,18 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
         }
     });
     server.tool(REPO_TOOLS.get_pull_request_by_id, "Get a pull request by its ID.", {
-        repositoryId: z.string().describe("The ID of the repository where the pull request is located."),
+        repositoryId: z
+            .string()
+            .describe("The ID or name of the repository where the pull request is located. When using a repository name instead of a GUID, the project parameter must also be provided."),
         pullRequestId: z.number().describe("The ID of the pull request to retrieve."),
+        project: z.string().optional().describe("Project ID or project name. Required when repositoryId is a repository name instead of a GUID."),
         includeWorkItemRefs: z.boolean().optional().default(false).describe("Whether to reference work items associated with the pull request."),
         includeLabels: z.boolean().optional().default(false).describe("Whether to include a summary of labels in the response."),
-    }, async ({ repositoryId, pullRequestId, includeWorkItemRefs, includeLabels }) => {
+    }, async ({ repositoryId, pullRequestId, project, includeWorkItemRefs, includeLabels }) => {
         try {
             const connection = await connectionProvider();
             const gitApi = await connection.getGitApi();
-            const pullRequest = await gitApi.getPullRequest(repositoryId, pullRequestId, undefined, undefined, undefined, undefined, undefined, includeWorkItemRefs);
+            const pullRequest = await gitApi.getPullRequest(repositoryId, pullRequestId, project, undefined, undefined, undefined, undefined, includeWorkItemRefs);
             if (includeLabels) {
                 try {
                     const projectId = pullRequest.repository?.project?.id;
@@ -789,11 +856,13 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
         }
     });
     server.tool(REPO_TOOLS.reply_to_comment, "Replies to a specific comment on a pull request.", {
-        repositoryId: z.string().describe("The ID of the repository where the pull request is located."),
+        repositoryId: z
+            .string()
+            .describe("The ID or name of the repository where the pull request is located. When using a repository name instead of a GUID, the project parameter must also be provided."),
         pullRequestId: z.number().describe("The ID of the pull request where the comment thread exists."),
         threadId: z.number().describe("The ID of the thread to which the comment will be added."),
         content: z.string().describe("The content of the comment to be added."),
-        project: z.string().optional().describe("Project ID or project name (optional)"),
+        project: z.string().optional().describe("Project ID or project name. Required when repositoryId is a repository name instead of a GUID."),
         fullResponse: z.boolean().optional().default(false).describe("Return full comment JSON response instead of a simple confirmation message."),
     }, async ({ repositoryId, pullRequestId, threadId, content, project, fullResponse }) => {
         try {
@@ -825,10 +894,12 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
         }
     });
     server.tool(REPO_TOOLS.create_pull_request_thread, "Creates a new comment thread on a pull request.", {
-        repositoryId: z.string().describe("The ID of the repository where the pull request is located."),
+        repositoryId: z
+            .string()
+            .describe("The ID or name of the repository where the pull request is located. When using a repository name instead of a GUID, the project parameter must also be provided."),
         pullRequestId: z.number().describe("The ID of the pull request where the comment thread exists."),
         content: z.string().describe("The content of the comment to be added."),
-        project: z.string().optional().describe("Project ID or project name (optional)"),
+        project: z.string().optional().describe("Project ID or project name. Required when repositoryId is a repository name instead of a GUID."),
         filePath: z.string().optional().describe("The path of the file where the comment thread will be created. (optional)"),
         status: z
             .enum(getEnumKeys(CommentThreadStatus))
@@ -939,10 +1010,12 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
         }
     });
     server.tool(REPO_TOOLS.update_pull_request_thread, "Updates an existing comment thread on a pull request.", {
-        repositoryId: z.string().describe("The ID of the repository where the pull request is located."),
+        repositoryId: z
+            .string()
+            .describe("The ID or name of the repository where the pull request is located. When using a repository name instead of a GUID, the project parameter must also be provided."),
         pullRequestId: z.number().describe("The ID of the pull request where the comment thread exists."),
         threadId: z.number().describe("The ID of the thread to update."),
-        project: z.string().optional().describe("Project ID or project name (optional)"),
+        project: z.string().optional().describe("Project ID or project name. Required when repositoryId is a repository name instead of a GUID."),
         status: z
             .enum(getEnumKeys(CommentThreadStatus))
             .optional()
@@ -1149,5 +1222,101 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
             };
         }
     });
+    server.tool(REPO_TOOLS.vote_pull_request, "Cast a vote on a pull request. Automatically adds the current user as a reviewer if they are not already one.", {
+        repositoryId: z.string().describe("The ID or name of the repository. When using a repository name instead of a GUID, the project parameter must also be provided."),
+        pullRequestId: z.number().describe("The ID of the pull request."),
+        vote: z.enum(["Approved", "ApprovedWithSuggestions", "NoVote", "WaitingForAuthor", "Rejected"]).describe("The vote to cast: Approved(10), Suggestions(5), None(0), Waiting(-5), Rejected(-10)."),
+        project: z.string().optional().describe("Project ID or project name. Required when repositoryId is a repository name instead of a GUID."),
+    }, async ({ repositoryId, pullRequestId, vote, project }) => {
+        const connection = await connectionProvider();
+        const gitApi = await connection.getGitApi();
+        const userDetails = await getCurrentUserDetails(tokenProvider, connectionProvider, userAgentProvider);
+        const userId = userDetails.authenticatedUser.id;
+        if (!userId) {
+            throw new Error("Could not determine authenticated user ID.");
+        }
+        const voteMap = {
+            Approved: 10,
+            ApprovedWithSuggestions: 5,
+            NoVote: 0,
+            WaitingForAuthor: -5,
+            Rejected: -10,
+        };
+        await gitApi.createPullRequestReviewer({ vote: voteMap[vote], id: userId }, repositoryId, pullRequestId, userId, project);
+        return {
+            content: [
+                {
+                    type: "text",
+                    text: `Successfully cast vote '${vote}' on PR #${pullRequestId}.`,
+                },
+            ],
+        };
+    });
+    server.tool(REPO_TOOLS.list_directory, "List files and folders in a directory within a repository. Useful for exploring the structure of a codebase or finding related files.", {
+        repositoryId: z.string().describe("The ID or name of the repository."),
+        path: z.string().optional().default("/").describe("The directory path to list (e.g., '/src' or '/src/components'). Defaults to repository root."),
+        project: z.string().optional().describe("Project ID or name. Required if repositoryId is a name rather than a GUID."),
+        version: z.string().optional().describe("The version identifier - branch name (e.g., 'main'), tag name, or commit SHA. Defaults to the repository's default branch."),
+        versionType: z.enum(["Branch", "Commit", "Tag"]).optional().default("Branch").describe("The type of version identifier: 'Branch', 'Commit', or 'Tag'. Defaults to 'Branch'."),
+        recursive: z.boolean().optional().default(false).describe("Whether to list items recursively. Defaults to false."),
+        recursionDepth: z.number().optional().default(1).describe("Maximum depth for recursive listing (1-10). Only applies when recursive is true. Defaults to 1."),
+    }, async ({ repositoryId, path, project, version, versionType, recursive, recursionDepth }) => {
+        try {
+            const connection = await connectionProvider();
+            const gitApi = await connection.getGitApi();
+            const versionDescriptor = buildVersionDescriptor(version, versionType);
+            const clampedDepth = Math.min(Math.max(recursionDepth || 1, 1), 10);
+            let recursionType = VersionControlRecursionType.OneLevel;
+            if (recursive) {
+                recursionType = VersionControlRecursionType.Full;
+            }
+            const items = await gitApi.getItems(repositoryId, project, path, recursionType, true, false, false, false, versionDescriptor);
+            if (!items || items.length === 0) {
+                return {
+                    content: [{ type: "text", text: `No items found at path: ${path}` }],
+                };
+            }
+            let filteredItems = items;
+            if (recursive && clampedDepth < 10) {
+                const basePath = path === "/" ? "" : path;
+                const baseDepth = basePath.split("/").filter((p) => p).length;
+                filteredItems = items.filter((item) => {
+                    if (!item.path)
+                        return false;
+                    const itemDepth = item.path.split("/").filter((p) => p).length;
+                    return itemDepth <= baseDepth + clampedDepth;
+                });
+            }
+            const formattedItems = filteredItems.map((item) => ({
+                path: item.path,
+                isFolder: item.isFolder,
+                gitObjectType: item.gitObjectType,
+                commitId: item.commitId,
+                contentMetadata: item.contentMetadata
+                    ? {
+                        contentType: item.contentMetadata.contentType,
+                        fileName: item.contentMetadata.fileName,
+                    }
+                    : undefined,
+            }));
+            const response = {
+                count: formattedItems.length,
+                path: path,
+                recursive: recursive,
+                recursionDepth: recursive ? clampedDepth : undefined,
+                items: formattedItems,
+            };
+            return {
+                content: [{ type: "text", text: JSON.stringify(response, null, 2) }],
+            };
+        }
+        catch (error) {
+            const errorMessage = error instanceof Error ? error.message : "Unknown error occurred";
+            return {
+                content: [{ type: "text", text: `Error listing directory: ${errorMessage}` }],
+                isError: true,
+            };
+        }
+    });
 }
 export { REPO_TOOLS, configureRepoTools };

package/dist/tools/test-plans.js

@@ -76,29 +76,47 @@ function configureTestPlanTools(server, _, connectionProvider) {
         parentSuiteId: z.number().describe("ID of the parent suite under which the new suite will be created, if not given by user this can be id of a root suite of the test plan"),
         name: z.string().describe("Name of the child test suite"),
     }, async ({ project, planId, parentSuiteId, name }) => {
-        try {
-            const connection = await connectionProvider();
-            const testPlanApi = await connection.getTestPlanApi();
-            const testSuiteToCreate = {
-                name,
-                parentSuite: {
-                    id: parentSuiteId,
-                    name: "",
-                },
-                suiteType: 2,
-            };
-            const createdTestSuite = await testPlanApi.createTestSuite(testSuiteToCreate, project, planId);
-            return {
-                content: [{ type: "text", text: JSON.stringify(createdTestSuite, null, 2) }],
-            };
-        }
-        catch (error) {
-            const errorMessage = error instanceof Error ? error.message : "Unknown error occurred";
-            return {
-                content: [{ type: "text", text: `Error creating test suite: ${errorMessage}` }],
-                isError: true,
-            };
+        const maxRetries = 5;
+        const baseDelay = 500; // milliseconds
+        for (let attempt = 0; attempt <= maxRetries; attempt++) {
+            try {
+                const connection = await connectionProvider();
+                const testPlanApi = await connection.getTestPlanApi();
+                const testSuiteToCreate = {
+                    name,
+                    parentSuite: {
+                        id: parentSuiteId,
+                        name: "",
+                    },
+                    suiteType: 2,
+                };
+                const createdTestSuite = await testPlanApi.createTestSuite(testSuiteToCreate, project, planId);
+                return {
+                    content: [{ type: "text", text: JSON.stringify(createdTestSuite, null, 2) }],
+                };
+            }
+            catch (error) {
+                const errorMessage = error instanceof Error ? error.message : "Unknown error occurred";
+                // Check if it's a concurrency conflict error
+                const isConcurrencyError = errorMessage.includes("TF26071") || errorMessage.includes("got update") || errorMessage.includes("changed by someone else");
+                // If it's a concurrency error and we have retries left, wait and retry
+                if (isConcurrencyError && attempt < maxRetries) {
+                    const delay = baseDelay * Math.pow(2, attempt) + Math.random() * 200; // Exponential backoff with jitter
+                    await new Promise((resolve) => setTimeout(resolve, delay));
+                    continue; // Retry
+                }
+                // If not a concurrency error or out of retries, return error
+                return {
+                    content: [{ type: "text", text: `Error creating test suite: ${errorMessage}` }],
+                    isError: true,
+                };
+            }
         }
+        // This should never be reached, but TypeScript requires a return value
+        return {
+            content: [{ type: "text", text: "Error creating test suite: Maximum retries exceeded" }],
+            isError: true,
+        };
     });
     server.tool(Test_Plan_Tools.add_test_cases_to_suite, "Adds existing test cases to a test suite.", {
         project: z.string().describe("The unique identifier (ID or name) of the Azure DevOps project."),
@@ -257,16 +275,48 @@ function configureTestPlanTools(server, _, connectionProvider) {
             };
         }
     });
-    server.tool(Test_Plan_Tools.test_results_from_build_id, "Gets a list of test results for a given project and build ID.", {
+    server.tool(Test_Plan_Tools.test_results_from_build_id, "Gets a list of test results for a given project and build ID. Can filter by test outcome (e.g. Failed, Passed, Aborted). Returns test case titles, error messages, stack traces, and outcomes. Efficiently handles builds with large numbers of test runs.", {
         project: z.string().describe("The unique identifier (ID or name) of the Azure DevOps project."),
         buildid: z.number().describe("The ID of the build."),
-    }, async ({ project, buildid }) => {
+        outcomes: z.array(z.string()).optional().describe("Filter results by test outcome, e.g. ['Failed', 'Passed', 'Aborted']."),
+    }, async ({ project, buildid, outcomes }) => {
         try {
             const connection = await connectionProvider();
-            const coreApi = await connection.getTestResultsApi();
-            const testResults = await coreApi.getTestResultDetailsForBuild(project, buildid);
+            const testResultsApi = await connection.getTestResultsApi();
+            // Build filter expression for outcomes if specified
+            const outcomeFilter = outcomes?.map((o) => `Outcome eq '${o}'`).join(" or ");
+            // Fetch test result details for the build in a single API call
+            // This is more efficient than getTestRuns + getTestResults per run,
+            // especially for builds with many test runs (e.g., cloud testing with one run per test case)
+            const testResultDetails = await testResultsApi.getTestResultDetailsForBuild(project, buildid, undefined, // publishContext
+            undefined, // groupBy
+            outcomeFilter, // filter by outcome
+            undefined, // orderby
+            true // shouldIncludeResults - get individual test results, not just aggregates
+            );
+            // Extract individual test results from the grouped response
+            const allResults = [];
+            if (testResultDetails.resultsForGroup) {
+                for (const group of testResultDetails.resultsForGroup) {
+                    if (group.results) {
+                        allResults.push(...group.results);
+                    }
+                }
+            }
+            // Format results to extract useful fields
+            const formattedResults = allResults.map((r) => ({
+                id: r.id,
+                testCaseTitle: r.testCaseTitle,
+                outcome: r.outcome,
+                errorMessage: r.errorMessage,
+                stackTrace: r.stackTrace,
+                automatedTestName: r.automatedTestName,
+                automatedTestStorage: r.automatedTestStorage,
+                durationInMs: r.durationInMs,
+                runId: r.testRun?.id,
+            }));
             return {
-                content: [{ type: "text", text: JSON.stringify(testResults, null, 2) }],
+                content: [{ type: "text", text: JSON.stringify(formattedResults, null, 2) }],
             };
         }
         catch (error) {