@azure-devops/mcp 2.10.0 → 2.11.0

package/dist/tools/repositories.js

@@ -1,6 +1,6 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
-import { PullRequestStatus, GitVersionType, GitPullRequestQueryType, CommentThreadStatus, GitPullRequestMergeStrategy, VersionControlRecursionType, } from "azure-devops-node-api/interfaces/GitInterfaces.js";
+import { PullRequestStatus, GitVersionType, GitPullRequestQueryType, CommentThreadStatus, GitPullRequestMergeStrategy, ItemContentType, VersionControlChangeType, VersionControlRecursionType, } from "azure-devops-node-api/interfaces/GitInterfaces.js";
 import { z } from "zod";
 import { getCurrentUserDetails, getUserIdFromEmail } from "./auth.js";
 import { extractAdoStreamError, getEnumKeys, streamToString, apiVersion } from "../utils.js";
@@ -9,9 +9,11 @@ import { createExternalContentResponse } from "../shared/content-safety.js";
 const REPO_TOOLS = {
     repo_repository: "repo_repository",
     repo_pull_request: "repo_pull_request",
+    repo_pull_request_org: "repo_pull_request_org",
     repo_pull_request_thread: "repo_pull_request_thread",
     repo_branch: "repo_branch",
     repo_file: "repo_file",
+    repo_file_write: "repo_file_write",
     repo_search_commits: "repo_search_commits",
     repo_pull_request_write: "repo_pull_request_write",
     repo_pull_request_thread_write: "repo_pull_request_thread_write",
@@ -36,10 +38,8 @@ function trimPullRequestThread(thread) {
         pullRequestThreadContext: thread.pullRequestThreadContext,
     };
 }
-function trimComments(comments) {
-    return comments
-        ?.filter((comment) => !comment.isDeleted)
-        ?.map((comment) => ({
+function trimComment(comment) {
+    return {
         id: comment.id,
         author: {
             displayName: comment.author?.displayName,
@@ -49,7 +49,10 @@ function trimComments(comments) {
         publishedDate: comment.publishedDate,
         lastUpdatedDate: comment.lastUpdatedDate,
         lastContentUpdatedDate: comment.lastContentUpdatedDate,
-    }));
+    };
+}
+function trimComments(comments) {
+    return comments?.filter((comment) => !comment.isDeleted)?.map(trimComment);
 }
 function pullRequestStatusStringToInt(status) {
     switch (status) {
@@ -96,6 +99,36 @@ function trimPullRequest(pr, includeDescription = false) {
         project: pr.repository?.project?.name,
     };
 }
+function trimOrganizationPullRequest(pr) {
+    return {
+        ...trimPullRequest(pr),
+        projectId: pr.repository?.project?.id,
+        repositoryId: pr.repository?.id,
+        createdBy: {
+            id: pr.createdBy?.id,
+            displayName: pr.createdBy?.displayName,
+            uniqueName: pr.createdBy?.uniqueName,
+        },
+        reviewers: pr.reviewers?.map((reviewer) => ({
+            id: reviewer.id,
+            displayName: reviewer.displayName,
+            uniqueName: reviewer.uniqueName,
+            vote: reviewer.vote,
+            isRequired: reviewer.isRequired,
+        })),
+        webUrl: pr.remoteUrl,
+    };
+}
+async function getOrganizationPullRequests(connection, identityFilter, identityId, status, skip, top) {
+    const url = new URL(`${connection.serverUrl.replace(/\/$/, "")}/_apis/git/pullrequests`);
+    url.searchParams.set("api-version", "7.1");
+    url.searchParams.set("$skip", String(skip));
+    url.searchParams.set("$top", String(top));
+    url.searchParams.set(`searchCriteria.${identityFilter}`, identityId);
+    url.searchParams.set("searchCriteria.status", String(status));
+    const response = await connection.rest.get(url.toString(), { deserializeDates: true });
+    return response.result?.value ?? [];
+}
 function buildVersionDescriptor(version, versionType) {
     if (!version)
         return undefined;
@@ -159,18 +192,22 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
         }
     });
     // --- repo_pull_request -----------------------------------------------------
-    server.tool(REPO_TOOLS.repo_pull_request, "Retrieve pull request data. Use the action parameter to specify the operation.", {
+    server.tool(REPO_TOOLS.repo_pull_request, "Retrieve pull request data for a specific repository or project, or retrieve one pull request by ID. For the authenticated user's active pull requests across the entire organization, use repo_pull_request_org.", {
         action: z
             .enum(["get", "list", "list_by_commits"])
             .describe("The action to perform. Options: get (get a pull request by ID), list (list pull requests in a repository or project), list_by_commits (find pull requests that contain specific commit IDs)."),
         repositoryId: z.string().optional().describe("The ID or name of the repository. Required for get. Optional for list. When using a name instead of a GUID, project must also be provided."),
         pullRequestId: z.coerce.number().min(1).optional().describe("The ID of the pull request. Required for get."),
-        project: z.string().optional().describe("Project ID or project name. Required for list_by_commits. Optional for get and list."),
+        project: z.string().optional().describe("Project ID or name. Required for list_by_commits and when repositoryId is a name. Optional when repositoryId is a GUID."),
         includeWorkItemRefs: z.boolean().optional().default(false).describe("Whether to include work item references. Used for get."),
         includeLabels: z.boolean().optional().default(false).describe("Whether to include labels. Used for get."),
-        includeChangedFiles: z.boolean().optional().default(false).describe("Whether to include the list of changed files. Used for get."),
-        top: z.coerce.number().default(100).describe("The maximum number of pull requests to return. Used for list. Defaults to 100."),
-        skip: z.coerce.number().default(0).describe("The number of pull requests to skip. Used for list. Defaults to 0."),
+        includeChangedFiles: z
+            .boolean()
+            .optional()
+            .default(false)
+            .describe("Whether to include the list of changed files. When changedFilesSummary returns a nextSkip greater than 0, pass nextSkip and nextTop back as skip and top for the next page. Used for get."),
+        top: z.coerce.number().default(100).describe("The maximum number of pull requests to return (list), or of changed files to return (get with includeChangedFiles). Defaults to 100."),
+        skip: z.coerce.number().default(0).describe("The number of pull requests to skip (list), or of changed files to skip (get with includeChangedFiles). Defaults to 0."),
         created_by_me: z.boolean().default(false).describe("Filter pull requests created by the current user. Used for list."),
         created_by_user: z.string().optional().describe("Filter pull requests created by a specific user email. Used for list."),
         i_am_reviewer: z.boolean().default(false).describe("Filter pull requests where the current user is a reviewer. Used for list."),
@@ -190,13 +227,19 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
             .describe("Type of commit query. Used for list_by_commits."),
     }, async ({ action, repositoryId, pullRequestId, project, includeWorkItemRefs, includeLabels, includeChangedFiles, top, skip, created_by_me, created_by_user, i_am_reviewer, user_is_reviewer, status, sourceRefName, targetRefName, repository, commits, queryType, }) => {
         try {
-            const connection = await connectionProvider();
-            const gitApi = await connection.getGitApi();
             if (action === "get") {
                 if (!repositoryId)
                     return { content: [{ type: "text", text: "repositoryId is required for get" }], isError: true };
                 if (!pullRequestId)
                     return { content: [{ type: "text", text: "pullRequestId is required for get" }], isError: true };
+                if (!project && !/^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i.test(repositoryId)) {
+                    return {
+                        content: [{ type: "text", text: "When repositoryId is a repository name, project is required. Provide the project name or ID, or use the repository GUID." }],
+                        isError: true,
+                    };
+                }
+                const connection = await connectionProvider();
+                const gitApi = await connection.getGitApi();
                 const pullRequest = await gitApi.getPullRequest(repositoryId, pullRequestId, project, undefined, undefined, undefined, undefined, includeWorkItemRefs);
                 let enhancedResponse = { ...pullRequest };
                 if (includeLabels) {
@@ -218,7 +261,7 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
                         if (iterations?.length) {
                             const latestIteration = iterations[iterations.length - 1];
                             if (latestIteration.id != null) {
-                                const changes = await gitApi.getPullRequestIterationChanges(repositoryId, pullRequestId, latestIteration.id, project);
+                                const changes = await gitApi.getPullRequestIterationChanges(repositoryId, pullRequestId, latestIteration.id, project, top, skip);
                                 enhancedResponse = {
                                     ...enhancedResponse,
                                     changedFilesSummary: {
@@ -246,6 +289,8 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
                 }
                 return createExternalContentResponse(enhancedResponse, "pull request");
             }
+            const connection = await connectionProvider();
+            const gitApi = await connection.getGitApi();
             if (action === "list") {
                 if (!repositoryId && !project) {
                     return { content: [{ type: "text", text: "Either repositoryId or project must be provided." }], isError: true };
@@ -319,6 +364,41 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
             return { content: [{ type: "text", text: `Error with pull request operation: ${errorMessage}` }], isError: true };
         }
     });
+    // --- repo_pull_request_org -------------------------------------------------
+    server.tool(REPO_TOOLS.repo_pull_request_org, "List active pull requests across all projects and repositories in the organization for the authenticated user. Use this tool for organization-wide requests such as 'my pull requests' or 'pull requests awaiting my review'. For a specific project, repository, or pull request ID, use repo_pull_request.", {
+        is_reviewer: z
+            .boolean()
+            .default(false)
+            .describe("Set to true for active pull requests where the authenticated user is a reviewer. Set to false for active pull requests created by the authenticated user."),
+        reviewStatus: z
+            .enum(["all", "approved", "pending"])
+            .default("all")
+            .describe("Filter by the authenticated user's reviewer vote when is_reviewer is true: all includes every vote, approved includes Approved and Approved with suggestions, and pending includes only No vote."),
+        top: z.coerce.number().default(100).describe("The maximum number of active pull requests to retrieve before applying the reviewStatus filter. Defaults to 100."),
+        skip: z.coerce.number().default(0).describe("The number of active pull requests to skip before applying the reviewStatus filter. Defaults to 0."),
+    }, async ({ is_reviewer, reviewStatus, top, skip }) => {
+        try {
+            const currentUser = await getCurrentUserDetails(tokenProvider, connectionProvider, userAgentProvider);
+            const userId = currentUser.authenticatedUser.id;
+            const identityFilter = is_reviewer ? "reviewerId" : "creatorId";
+            const connection = await connectionProvider();
+            const pullRequests = await getOrganizationPullRequests(connection, identityFilter, userId, PullRequestStatus.Active, skip, top);
+            const filteredPullRequests = pullRequests.filter((pullRequest) => {
+                if (!is_reviewer || reviewStatus === "all")
+                    return true;
+                const reviewer = pullRequest.reviewers?.find((item) => item.id === userId);
+                if (reviewStatus === "approved")
+                    return reviewer?.vote === 10 || reviewer?.vote === 5;
+                return reviewer?.vote === 0;
+            });
+            const trimmedPullRequests = filteredPullRequests.map((pullRequest) => trimOrganizationPullRequest(pullRequest));
+            return { content: [{ type: "text", text: JSON.stringify(trimmedPullRequests, null, 2) }] };
+        }
+        catch (error) {
+            const errorMessage = error instanceof Error ? error.message : "Unknown error occurred";
+            return { content: [{ type: "text", text: `Error with organization pull request operation: ${errorMessage}` }], isError: true };
+        }
+    });
     // --- repo_pull_request_thread ----------------------------------------------
     server.tool(REPO_TOOLS.repo_pull_request_thread, "Retrieve pull request thread and comment data. Use the action parameter to specify the operation.", {
         action: z.enum(["list", "list_comments"]).describe("The action to perform. Options: list (list comment threads on a pull request), list_comments (list comments in a specific thread)."),
@@ -440,8 +520,7 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
         versionType: z
             .enum(fileVersionTypeStrings)
             .optional()
-            .default("Commit")
-            .describe("How to interpret the version parameter. Used for get_content. Defaults to 'Commit'."),
+            .describe("How to interpret the version parameter. Defaults to 'Commit' for get_content and 'Branch' for list_directory."),
         recursive: z.boolean().optional().default(false).describe("Whether to list items recursively. Used for list_directory. Defaults to false."),
         recursionDepth: z.coerce.number().min(1).optional().default(1).describe("Maximum depth for recursive listing. Used for list_directory when recursive is true. Defaults to 1."),
     }, async ({ action, repositoryId, path, project, version, versionType, recursive, recursionDepth }) => {
@@ -451,7 +530,7 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
             if (action === "get_content") {
                 if (!path)
                     return { content: [{ type: "text", text: "path is required for get_content" }], isError: true };
-                const versionDescriptor = version ? { version, versionType: GitVersionType[versionType] } : undefined;
+                const versionDescriptor = buildVersionDescriptor(version, versionType ?? "Commit");
                 const stream = await gitApi.getItemText(repositoryId, path, project, undefined, undefined, undefined, undefined, false, versionDescriptor, true);
                 const content = await streamToString(stream);
                 const streamError = extractAdoStreamError(content);
@@ -461,7 +540,7 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
                 return createExternalContentResponse(content, "repository file");
             }
             if (action === "list_directory") {
-                const versionDescriptor = buildVersionDescriptor(version, versionType === "Commit" ? "Branch" : versionType);
+                const versionDescriptor = buildVersionDescriptor(version, versionType);
                 const clampedDepth = Math.min(Math.max(recursionDepth || 1, 1), 10);
                 const recursionType = recursive ? VersionControlRecursionType.Full : VersionControlRecursionType.OneLevel;
                 const items = await gitApi.getItems(repositoryId, project, path, recursionType, true, false, false, false, versionDescriptor);
@@ -502,6 +581,63 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
             return { content: [{ type: "text", text: `Error with file operation: ${errorMessage}` }], isError: true };
         }
     });
+    // --- repo_file_write -------------------------------------------------------
+    server.tool(REPO_TOOLS.repo_file_write, "Create or update a text file in an existing repository branch and commit the change.", {
+        action: z.enum(["create", "update"]).describe("Whether to create a new file or update an existing file."),
+        repositoryId: z.string().describe("The ID or name of the repository."),
+        project: z.string().optional().describe("Project ID or project name. Required when repositoryId is a name."),
+        branchName: z.string().min(1).describe("The name of the existing branch to update, without the 'refs/heads/' prefix."),
+        path: z.string().min(1).describe("The absolute repository path of the file, for example '/src/example.ts'."),
+        content: z.string().describe("The complete UTF-8 text content to write to the file."),
+        commitMessage: z.string().min(1).describe("The commit message for the file change."),
+        expectedOldObjectId: z
+            .string()
+            .optional()
+            .describe("The expected current commit ID of the branch. When provided, the push fails if the branch has moved. If omitted, the current branch head is retrieved before pushing."),
+    }, async ({ action, repositoryId, project, branchName, path, content, commitMessage, expectedOldObjectId }) => {
+        try {
+            const connection = await connectionProvider();
+            const gitApi = await connection.getGitApi();
+            const refName = `refs/heads/${branchName}`;
+            const refs = await gitApi.getRefs(repositoryId, project, "heads/", false, false, undefined, false, undefined, branchName);
+            const branch = refs.find((ref) => ref.name === refName);
+            if (!branch?.objectId) {
+                return { content: [{ type: "text", text: `Error: Branch '${branchName}' not found in repository ${repositoryId}` }], isError: true };
+            }
+            if (expectedOldObjectId && expectedOldObjectId !== branch.objectId) {
+                return { content: [{ type: "text", text: `Error: Branch '${branchName}' has moved from expected commit ${expectedOldObjectId}` }], isError: true };
+            }
+            const oldObjectId = branch.objectId;
+            const filePath = path.startsWith("/") ? path : `/${path}`;
+            const push = await gitApi.createPush({
+                refUpdates: [{ name: refName, oldObjectId }],
+                commits: [
+                    {
+                        comment: commitMessage,
+                        changes: [
+                            {
+                                changeType: action === "create" ? VersionControlChangeType.Add : VersionControlChangeType.Edit,
+                                item: { path: filePath },
+                                newContent: { content, contentType: ItemContentType.RawText },
+                            },
+                        ],
+                    },
+                ],
+            }, repositoryId, project);
+            return {
+                content: [
+                    {
+                        type: "text",
+                        text: JSON.stringify({ pushId: push.pushId, commitId: push.commits?.[0]?.commitId, branchName, path: filePath }, null, 2),
+                    },
+                ],
+            };
+        }
+        catch (error) {
+            const errorMessage = error instanceof Error ? error.message : "Unknown error occurred";
+            return { content: [{ type: "text", text: `Error writing repository file: ${errorMessage}` }], isError: true };
+        }
+    });
     // --- repo_search_commits ---------------------------------------------------
     server.tool(REPO_TOOLS.repo_search_commits, "Search commits with filtering by text, author, date range, and more.", {
         searchText: z.string().describe("Keywords to search for in commit messages"),
@@ -566,7 +702,7 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
         targetRefName: z.string().optional().describe("The target branch name (e.g., 'refs/heads/main'). Required for create. Optional for update."),
         title: z.string().optional().describe("The title of the pull request. Required for create. Optional for update."),
         description: z.string().max(4000).optional().describe("The description of the pull request. Max 4000 characters. Used for create and update."),
-        isDraft: z.boolean().optional().default(false).describe("Whether the pull request is a draft. Used for create and update."),
+        isDraft: z.boolean().optional().describe("Whether the pull request is a draft. Defaults to false for create. When omitted during update, the existing draft state is preserved."),
         workItems: z.string().optional().describe("Work item IDs to associate, space-separated. Used for create."),
         forkSourceRepositoryId: z.string().optional().describe("The ID of the fork repository. Used for create."),
         labels: z.array(z.string()).optional().describe("Array of label names. Used for create and update."),
@@ -602,7 +738,7 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
                     "Try the repository GUID from repo_repository (list action) instead of the Project/RepoName slash format.";
                 const forkSource = forkSourceRepositoryId ? { repository: { id: forkSourceRepositoryId } } : undefined;
                 const labelDefinitions = labels ? labels.map((label) => ({ name: label })) : undefined;
-                let pullRequest = await gitApi.createPullRequest({ sourceRefName, targetRefName, title, description, isDraft, workItemRefs, forkSource, labels: labelDefinitions, supportsIterations: true }, repositoryId, project);
+                let pullRequest = await gitApi.createPullRequest({ sourceRefName, targetRefName, title, description, isDraft: isDraft ?? false, workItemRefs, forkSource, labels: labelDefinitions, 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) {
@@ -766,14 +902,13 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
         status: z
             .enum(getEnumKeys(CommentThreadStatus))
             .optional()
-            .default(CommentThreadStatus[CommentThreadStatus.Active])
-            .describe("The thread status. Used for create (defaults to 'Active') and required for update_status."),
+            .describe("The thread status. Used for create (defaults to 'Active'), optionally for reply, and required for update_status."),
         filePath: z.string().optional().describe("The file path for the comment thread. Used for create."),
         fullResponse: z.boolean().optional().default(false).describe("Return full JSON response. Used for reply and update."),
         rightFileStartLine: z.coerce.number().min(1).optional().describe("Start line in the right file. Used for create."),
         rightFileStartOffset: z.number().optional().describe("Start character offset in the right file. Used for create."),
-        rightFileEndLine: z.number().optional().describe("End line in the right file. Used for create."),
-        rightFileEndOffset: z.number().optional().describe("End character offset in the right file. Used for create."),
+        rightFileEndLine: z.number().optional().describe("End line in the right file. Used for create. Must be supplied together with rightFileEndOffset."),
+        rightFileEndOffset: z.number().optional().describe("End character offset in the right file. Used for create. Must be supplied together with rightFileEndLine."),
         changeTrackingId: z.coerce.number().int().min(1).optional().describe("The file change tracking ID from the pull request iteration changes. Used for create."),
         firstComparingIteration: z.coerce.number().int().min(0).optional().describe("The iteration on the left side of the diff. Used for create."),
         secondComparingIteration: z.coerce.number().int().min(1).optional().describe("The iteration on the right side of the diff. Used for create."),
@@ -837,7 +972,8 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
                 const pullRequestThreadContext = changeTrackingId !== undefined && firstComparingIteration !== undefined && secondComparingIteration !== undefined
                     ? { changeTrackingId, iterationContext: { firstComparingIteration, secondComparingIteration } }
                     : undefined;
-                const thread = await gitApi.createThread({ comments: [{ content, commentType: 1 }], threadContext, pullRequestThreadContext, status: CommentThreadStatus[status] }, repositoryId, pullRequestId, project);
+                const threadStatus = status === undefined ? CommentThreadStatus.Active : CommentThreadStatus[status];
+                const thread = await gitApi.createThread({ comments: [{ content, commentType: 1 }], threadContext, pullRequestThreadContext, status: threadStatus }, repositoryId, pullRequestId, project);
                 return { content: [{ type: "text", text: JSON.stringify(trimPullRequestThread(thread), null, 2) }] };
             }
             if (action === "reply") {
@@ -849,9 +985,55 @@ function configureRepoTools(server, tokenProvider, connectionProvider, userAgent
                 if (!comment) {
                     return { content: [{ type: "text", text: `Error: Failed to add comment to thread ${threadId}. The comment was not created successfully.` }], isError: true };
                 }
+                if (status !== undefined) {
+                    let statusUpdateError;
+                    let resultingThreadStatus;
+                    try {
+                        const thread = await gitApi.updateThread({ status: CommentThreadStatus[status] }, repositoryId, pullRequestId, threadId, project);
+                        if (!thread) {
+                            statusUpdateError = `Thread ${threadId} was not updated successfully.`;
+                        }
+                        else {
+                            resultingThreadStatus = thread.status;
+                        }
+                    }
+                    catch (error) {
+                        statusUpdateError = error instanceof Error ? error.message : "Unknown error occurred";
+                    }
+                    if (statusUpdateError !== undefined) {
+                        return {
+                            content: [
+                                {
+                                    type: "text",
+                                    text: JSON.stringify({
+                                        message: `Comment ${comment.id} was posted successfully, but the thread status update failed. Do not retry the reply.`,
+                                        comment: trimComment(comment),
+                                        statusUpdate: {
+                                            succeeded: false,
+                                            requestedStatus: status,
+                                            error: statusUpdateError,
+                                        },
+                                    }, null, 2),
+                                },
+                            ],
+                            isError: true,
+                        };
+                    }
+                    return {
+                        content: [
+                            {
+                                type: "text",
+                                text: JSON.stringify({
+                                    comment: trimComment(comment),
+                                    status: resultingThreadStatus,
+                                }, null, 2),
+                            },
+                        ],
+                    };
+                }
                 if (fullResponse)
                     return { content: [{ type: "text", text: JSON.stringify(comment, null, 2) }] };
-                return { content: [{ type: "text", text: `Comment successfully added to thread ${threadId}.` }] };
+                return { content: [{ type: "text", text: JSON.stringify(trimComment(comment), null, 2) }] };
             }
             if (action === "update") {
                 if (!threadId)

package/dist/tools/search.js

@@ -63,7 +63,7 @@ function configureSearchTools(server, tokenProvider, connectionProvider, userAge
         const combinedResults = await fetchCombinedResults(resultJson.results ?? [], gitApi);
         return createExternalContentResponse(resultText + JSON.stringify(combinedResults), "code search results");
     });
-    server.tool(SEARCH_TOOLS.search_wiki, "Search Azure DevOps Wiki for a given search text", {
+    server.tool(SEARCH_TOOLS.search_wiki, "Search Azure DevOps Wiki for a given search text. Results include the repository path and a wiki-relative pagePath that can be used with the wiki tool.", {
         searchText: z.string().describe("Keywords to search for wiki pages"),
         project: z.array(z.string()).optional().describe("Filter by projects"),
         wiki: z.array(z.string()).optional().describe("Filter by wiki names"),
@@ -99,8 +99,10 @@ function configureSearchTools(server, tokenProvider, connectionProvider, userAge
         if (!response.ok) {
             throw new Error(`Azure DevOps Wiki Search API error: ${response.status} ${response.statusText}`);
         }
-        const result = await response.text();
-        return createExternalContentResponse(result, "wiki search results");
+        const result = (await response.json());
+        const results = result.results?.map(addWikiPagePath);
+        const enrichedResult = results ? { ...result, results } : result;
+        return createExternalContentResponse(JSON.stringify(enrichedResult), "wiki search results");
     });
     server.tool(SEARCH_TOOLS.search_workitem, "Get Azure DevOps Work Item search results for a given search text", {
         searchText: z.string().describe("Search text to find in work items"),
@@ -151,6 +153,27 @@ function configureSearchTools(server, tokenProvider, connectionProvider, userAge
         return createExternalContentResponse(result, "work item search results");
     });
 }
+function addWikiPagePath(result) {
+    if (!result || result.pagePath || !result.path || !result.wiki?.mappedPath) {
+        return result;
+    }
+    const mappedPath = result.wiki.mappedPath === "/" ? "" : result.wiki.mappedPath.replace(/\/+$/, "");
+    if (mappedPath && result.path !== mappedPath && !result.path.startsWith(`${mappedPath}/`)) {
+        return result;
+    }
+    let relativePath;
+    try {
+        relativePath = decodeURIComponent(result.path.slice(mappedPath.length).replace(/\.md$/i, ""));
+    }
+    catch (error) {
+        if (error instanceof URIError) {
+            return result;
+        }
+        throw error;
+    }
+    const pagePath = relativePath.startsWith("/") ? relativePath : `/${relativePath}`;
+    return { ...result, pagePath };
+}
 async function fetchCombinedResults(topSearchResults, gitApi) {
     const combinedResults = [];
     for (const searchResult of topSearchResults) {

package/dist/tools/test-plans.js

@@ -1,7 +1,27 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
+import { TestOutcome } from "azure-devops-node-api/interfaces/TestInterfaces.js";
 import { z } from "zod";
 import { apiVersion } from "../utils.js";
+const TEST_RESULT_PAGE_SIZE = 1_000;
+const TEST_RESULT_CONCURRENCY = 5;
+const TEST_OUTCOMES = [
+    "Unspecified",
+    "None",
+    "Passed",
+    "Failed",
+    "Inconclusive",
+    "Timeout",
+    "Aborted",
+    "Blocked",
+    "NotExecuted",
+    "Warning",
+    "Error",
+    "NotApplicable",
+    "Paused",
+    "InProgress",
+    "NotImpacted",
+];
 const TEST_PLAN_TOOLS = {
     testplan: "testplan",
     test_results_from_build_id: "testplan_show_test_results_from_build_id",
@@ -137,33 +157,50 @@ function configureTestPlanTools(server, tokenProvider, connectionProvider, userA
     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.coerce.number().min(1).describe("The ID of the build."),
-        outcomes: z.array(z.string()).optional().describe("Filter results by test outcome, e.g. ['Failed', 'Passed', 'Aborted']."),
+        outcomes: z.array(z.enum(TEST_OUTCOMES)).optional().describe("Filter results by test outcome, e.g. ['Failed', 'Passed', 'Aborted']."),
     }, async ({ project, buildid, outcomes }) => {
         try {
             const connection = await connectionProvider();
             const testResultsApi = await connection.getTestResultsApi();
-            const outcomeFilter = outcomes?.length ? `Outcome eq ${outcomes.join(",")}` : undefined;
-            const testResultDetails = await testResultsApi.getTestResultDetailsForBuild(project, buildid, undefined, undefined, outcomeFilter, undefined, true);
+            const buildUri = `vstfs:///Build/Build/${buildid}`;
+            const testRuns = [];
+            for (let skip = 0;; skip += TEST_RESULT_PAGE_SIZE) {
+                const page = await testResultsApi.getTestRuns(project, buildUri, undefined, undefined, undefined, undefined, undefined, skip, TEST_RESULT_PAGE_SIZE);
+                testRuns.push(...page);
+                if (page.length < TEST_RESULT_PAGE_SIZE) {
+                    break;
+                }
+            }
+            const outcomeFilter = outcomes?.map((outcome) => TestOutcome[outcome]);
+            const runsWithIds = testRuns.filter((run) => run.id !== undefined);
             const allResults = [];
-            if (testResultDetails.resultsForGroup) {
-                for (const group of testResultDetails.resultsForGroup) {
-                    if (group.results) {
-                        for (const result of group.results) {
-                            allResults.push(result);
+            for (let index = 0; index < runsWithIds.length; index += TEST_RESULT_CONCURRENCY) {
+                const batch = runsWithIds.slice(index, index + TEST_RESULT_CONCURRENCY);
+                const batchResults = await Promise.all(batch.map(async (run) => {
+                    const results = [];
+                    for (let skip = 0;; skip += TEST_RESULT_PAGE_SIZE) {
+                        const page = await testResultsApi.getTestResults(project, run.id, undefined, skip, TEST_RESULT_PAGE_SIZE, outcomeFilter);
+                        results.push(...page.map((result) => ({ result, runId: run.id })));
+                        if (page.length < TEST_RESULT_PAGE_SIZE) {
+                            break;
                         }
                     }
+                    return results;
+                }));
+                for (const results of batchResults) {
+                    allResults.push(...results);
                 }
             }
-            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,
+            const formattedResults = allResults.map(({ result, runId }) => ({
+                id: result.id,
+                testCaseTitle: result.testCaseTitle,
+                outcome: result.outcome,
+                errorMessage: result.errorMessage,
+                stackTrace: result.stackTrace,
+                automatedTestName: result.automatedTestName,
+                automatedTestStorage: result.automatedTestStorage,
+                durationInMs: result.durationInMs,
+                runId,
             }));
             return {
                 content: [{ type: "text", text: JSON.stringify(formattedResults, null, 2) }],

package/dist/tools/wiki.js

@@ -19,7 +19,7 @@ function configureWikiTools(server, tokenProvider, connectionProvider, userAgent
             .string()
             .optional()
             .describe("The full URL of the wiki page. Used for get_page_content. If provided, wikiIdentifier, project, and path are ignored. Supported patterns: https://dev.azure.com/{org}/{project}/_wiki/wikis/{wikiIdentifier}?pagePath=%2FMy%20Page and https://dev.azure.com/{org}/{project}/_wiki/wikis/{wikiIdentifier}/{pageId}/Page-Title"),
-        top: z.coerce.number().default(20).describe("The maximum number of pages to return. Used for list_pages. Defaults to 20."),
+        top: z.coerce.number().default(20).describe("Maximum number of pages to return for list_pages (up to 100; defaults to 20)."),
         continuationToken: z.string().optional().describe("Token for pagination to retrieve the next set of pages. Used for list_pages."),
         pageViewsForDays: z.coerce.number().optional().describe("Number of days to retrieve page views for. Used for list_pages. If not specified, page views are not included."),
         recursionLevel: z
@@ -178,7 +178,7 @@ function configureWikiTools(server, tokenProvider, connectionProvider, userAgent
                         catch { }
                     }
                 }
-                if (!pageContent) {
+                if (pageContent === undefined) {
                     if (!resolvedPath) {
                         resolvedPath = "/";
                     }
@@ -346,18 +346,18 @@ function parseWikiUrl(url) {
         if (idx < 1 || segments[idx + 1] !== "wikis") {
             return { error: "URL does not match expected wiki pattern (missing /_wiki/wikis/ segment)." };
         }
-        const project = segments[idx - 1];
-        const wikiIdentifier = segments[idx + 2];
-        if (!project || !wikiIdentifier) {
+        const encodedProject = segments[idx - 1];
+        const encodedWikiIdentifier = segments[idx + 2];
+        if (!encodedProject || !encodedWikiIdentifier) {
             return { error: "Could not extract project or wikiIdentifier from URL." };
         }
+        const project = decodeURIComponent(encodedProject);
+        const wikiIdentifier = decodeURIComponent(encodedWikiIdentifier);
         // Query form with pagePath
         const pagePathParam = u.searchParams.get("pagePath");
         if (pagePathParam) {
-            let decoded = decodeURIComponent(pagePathParam);
-            if (!decoded.startsWith("/"))
-                decoded = "/" + decoded;
-            return { project, wikiIdentifier, pagePath: decoded };
+            const pagePath = pagePathParam.startsWith("/") ? pagePathParam : `/${pagePathParam}`;
+            return { project, wikiIdentifier, pagePath };
         }
         // Path ID form: .../wikis/{wikiIdentifier}/{pageId}/...
         const afterWiki = segments.slice(idx + 3); // elements after wikiIdentifier