@azure-devops/mcp 1.2.0 → 1.2.1

package/README.md

@@ -73,12 +73,12 @@ Interact with these Azure DevOps services:
 - **wit_get_query**: Get a query by its ID or path.
 - **wit_get_query_results_by_id**: Retrieve the results of a work item query given the query ID.
 - **wit_update_work_items_batch**: Update work items in batch.
-- **wit_close_and_link_workitem_duplicates**: Close duplicate work items by ID.
 - **wit_work_items_link**: Link work items together in batch.
 
 #### Deprecated Tools
 
 - **wit_add_child_work_item**: Replaced by `wit_add_child_work_items` to allow creating one or more child items per call.
+- **wit_close_and_link_workitem_duplicates**: This tool is no longer needed. Finding and marking duplicates can be done with other tools.
 
 ### 📁 Repositories
 
@@ -96,9 +96,10 @@ Interact with these Azure DevOps services:
 - **repo_create_pull_request**: Create a new pull request.
 - **repo_update_pull_request_status**: Update the status of an existing pull request to active or abandoned.
 - **repo_update_pull_request_reviewers**: Add or remove reviewers for an existing pull request.
-- **repo_reply_to_comment**: Reply to a specific comment on a pull request.
-- **repo_resolve_comment**: Resolve a specific comment thread on a pull request.
-- **repo_search_commits**: Search for commits.
+- **repo_reply_to_comment**: Replies to a specific comment on a pull request.
+- **repo_resolve_comment**: Resolves a specific comment thread on a pull request.
+- **repo_search_commits**: Searches for commits.
+- **repo_create_pull_request_thread**: Creates a new comment thread on a pull request.
 
 ### 🛰️ Builds
 

package/dist/tools/repos.js

@@ -19,6 +19,7 @@ const REPO_TOOLS = {
     update_pull_request_status: "repo_update_pull_request_status",
     update_pull_request_reviewers: "repo_update_pull_request_reviewers",
     reply_to_comment: "repo_reply_to_comment",
+    create_pull_request_thread: "repo_create_pull_request_thread",
     resolve_comment: "repo_resolve_comment",
     search_commits: "repo_search_commits",
     list_pull_requests_by_commits: "repo_list_pull_requests_by_commits",
@@ -31,6 +32,26 @@ function branchesFilterOutIrrelevantProperties(branches, top) {
         .sort((a, b) => b.localeCompare(a))
         .slice(0, top);
 }
+/**
+ * Trims comment data to essential properties, filtering out deleted comments
+ * @param comments Array of comments to trim (can be undefined/null)
+ * @returns Array of trimmed comment objects with essential properties only
+ */
+function trimComments(comments) {
+    return comments
+        ?.filter((comment) => !comment.isDeleted) // Exclude deleted comments
+        ?.map((comment) => ({
+        id: comment.id,
+        author: {
+            displayName: comment.author?.displayName,
+            uniqueName: comment.author?.uniqueName,
+        },
+        content: comment.content,
+        publishedDate: comment.publishedDate,
+        lastUpdatedDate: comment.lastUpdatedDate,
+        lastContentUpdatedDate: comment.lastContentUpdatedDate,
+    }));
+}
 function pullRequestStatusStringToInt(status) {
     switch (status) {
         case "Abandoned":
@@ -242,13 +263,27 @@ function configureRepoTools(server, tokenProvider, connectionProvider) {
         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."),
         skip: z.number().default(0).describe("The number of threads to skip."),
-    }, async ({ repositoryId, pullRequestId, project, iteration, baseIteration, top, skip }) => {
+        fullResponse: z.boolean().optional().default(false).describe("Return full thread JSON response instead of trimmed data."),
+    }, async ({ repositoryId, pullRequestId, project, iteration, baseIteration, top, skip, fullResponse }) => {
         const connection = await connectionProvider();
         const gitApi = await connection.getGitApi();
         const threads = await gitApi.getThreads(repositoryId, pullRequestId, project, iteration, baseIteration);
         const paginatedThreads = threads?.sort((a, b) => (a.id ?? 0) - (b.id ?? 0)).slice(skip, skip + top);
+        if (fullResponse) {
+            return {
+                content: [{ type: "text", text: JSON.stringify(paginatedThreads, null, 2) }],
+            };
+        }
+        // Return trimmed thread data focusing on essential information
+        const trimmedThreads = paginatedThreads?.map((thread) => ({
+            id: thread.id,
+            publishedDate: thread.publishedDate,
+            lastUpdatedDate: thread.lastUpdatedDate,
+            status: thread.status,
+            comments: trimComments(thread.comments),
+        }));
         return {
-            content: [{ type: "text", text: JSON.stringify(paginatedThreads, null, 2) }],
+            content: [{ type: "text", text: JSON.stringify(trimmedThreads, null, 2) }],
         };
     });
     server.tool(REPO_TOOLS.list_pull_request_thread_comments, "Retrieve a list of comments in a pull request thread.", {
@@ -258,14 +293,22 @@ function configureRepoTools(server, tokenProvider, connectionProvider) {
         project: z.string().optional().describe("Project ID or project name (optional)"),
         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."),
-    }, async ({ repositoryId, pullRequestId, threadId, project, top, skip }) => {
+        fullResponse: z.boolean().optional().default(false).describe("Return full comment JSON response instead of trimmed data."),
+    }, async ({ repositoryId, pullRequestId, threadId, project, top, skip, fullResponse }) => {
         const connection = await connectionProvider();
         const gitApi = await connection.getGitApi();
         // Get thread comments - GitApi uses getComments for retrieving comments from a specific thread
         const comments = await gitApi.getComments(repositoryId, pullRequestId, threadId, project);
         const paginatedComments = comments?.sort((a, b) => (a.id ?? 0) - (b.id ?? 0)).slice(skip, skip + top);
+        if (fullResponse) {
+            return {
+                content: [{ type: "text", text: JSON.stringify(paginatedComments, null, 2) }],
+            };
+        }
+        // Return trimmed comment data focusing on essential information
+        const trimmedComments = trimComments(paginatedComments);
         return {
-            content: [{ type: "text", text: JSON.stringify(paginatedComments, null, 2) }],
+            content: [{ type: "text", text: JSON.stringify(trimmedComments, null, 2) }],
         };
     });
     server.tool(REPO_TOOLS.list_branches_by_repo, "Retrieve a list of branches for a given repository.", {
@@ -346,25 +389,106 @@ function configureRepoTools(server, tokenProvider, connectionProvider) {
         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)"),
-    }, async ({ repositoryId, pullRequestId, threadId, content, project }) => {
+        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 }) => {
         const connection = await connectionProvider();
         const gitApi = await connection.getGitApi();
         const comment = await gitApi.createComment({ content }, repositoryId, pullRequestId, threadId, project);
+        // Check if the comment was successfully created
+        if (!comment) {
+            return {
+                content: [{ type: "text", text: `Error: Failed to add comment to thread ${threadId}. The comment was not created successfully.` }],
+                isError: true,
+            };
+        }
+        if (fullResponse) {
+            return {
+                content: [{ type: "text", text: JSON.stringify(comment, null, 2) }],
+            };
+        }
         return {
-            content: [{ type: "text", text: JSON.stringify(comment, null, 2) }],
+            content: [{ type: "text", text: `Comment successfully added to thread ${threadId}.` }],
+        };
+    });
+    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."),
+        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)"),
+        filePath: z.string().optional().describe("The path of the file where the comment thread will be created. (optional)"),
+        rightFileStartLine: z.number().optional().describe("Position of first character of the thread's span in right file. The line number of a thread's position. Starts at 1. (optional)"),
+        rightFileStartOffset: z
+            .number()
+            .optional()
+            .describe("Position of first character of the thread's span in right file. The line number of a thread's position. The character offset of a thread's position inside of a line. Starts at 1. Must only be set if rightFileStartLine is also specified. (optional)"),
+        rightFileEndLine: z
+            .number()
+            .optional()
+            .describe("Position of last character of the thread's span in right file. The line number of a thread's position. Starts at 1. Must only be set if rightFileStartLine is also specified. (optional)"),
+        rightFileEndOffset: z
+            .number()
+            .optional()
+            .describe("Position of last character of the thread's span in right file. The character offset of a thread's position inside of a line. Must only be set if rightFileEndLine is also specified. (optional)"),
+    }, async ({ repositoryId, pullRequestId, content, project, filePath, rightFileStartLine, rightFileStartOffset, rightFileEndLine, rightFileEndOffset }) => {
+        const connection = await connectionProvider();
+        const gitApi = await connection.getGitApi();
+        const threadContext = { filePath: filePath };
+        if (rightFileStartLine !== undefined) {
+            if (rightFileStartLine < 1) {
+                throw new Error("rightFileStartLine must be greater than or equal to 1.");
+            }
+            threadContext.rightFileStart = { line: rightFileStartLine };
+            if (rightFileStartOffset !== undefined) {
+                if (rightFileStartOffset < 1) {
+                    throw new Error("rightFileStartOffset must be greater than or equal to 1.");
+                }
+                threadContext.rightFileStart.offset = rightFileStartOffset;
+            }
+        }
+        if (rightFileEndLine !== undefined) {
+            if (rightFileStartLine === undefined) {
+                throw new Error("rightFileEndLine must only be specified if rightFileStartLine is also specified.");
+            }
+            if (rightFileEndLine < 1) {
+                throw new Error("rightFileEndLine must be greater than or equal to 1.");
+            }
+            threadContext.rightFileEnd = { line: rightFileEndLine };
+            if (rightFileEndOffset !== undefined) {
+                if (rightFileEndOffset < 1) {
+                    throw new Error("rightFileEndOffset must be greater than or equal to 1.");
+                }
+                threadContext.rightFileEnd.offset = rightFileEndOffset;
+            }
+        }
+        const thread = await gitApi.createThread({ comments: [{ content: content }], threadContext: threadContext }, repositoryId, pullRequestId, project);
+        return {
+            content: [{ type: "text", text: JSON.stringify(thread, null, 2) }],
         };
     });
     server.tool(REPO_TOOLS.resolve_comment, "Resolves a specific comment thread on a pull request.", {
         repositoryId: z.string().describe("The ID of the repository where the pull request is located."),
         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 be resolved."),
-    }, async ({ repositoryId, pullRequestId, threadId }) => {
+        fullResponse: z.boolean().optional().default(false).describe("Return full thread JSON response instead of a simple confirmation message."),
+    }, async ({ repositoryId, pullRequestId, threadId, fullResponse }) => {
         const connection = await connectionProvider();
         const gitApi = await connection.getGitApi();
         const thread = await gitApi.updateThread({ status: 2 }, // 2 corresponds to "Resolved" status
         repositoryId, pullRequestId, threadId);
+        // Check if the thread was successfully resolved
+        if (!thread) {
+            return {
+                content: [{ type: "text", text: `Error: Failed to resolve thread ${threadId}. The thread status was not updated successfully.` }],
+                isError: true,
+            };
+        }
+        if (fullResponse) {
+            return {
+                content: [{ type: "text", text: JSON.stringify(thread, null, 2) }],
+            };
+        }
         return {
-            content: [{ type: "text", text: JSON.stringify(thread, null, 2) }],
+            content: [{ type: "text", text: `Thread ${threadId} was successfully resolved.` }],
         };
     });
     const gitVersionTypeStrings = Object.values(GitVersionType).filter((value) => typeof value === "string");

package/dist/tools/search.js

@@ -17,17 +17,17 @@ function configureSearchTools(server, tokenProvider, connectionProvider, userAge
         path: z.array(z.string()).optional().describe("Filter by paths"),
         branch: z.array(z.string()).optional().describe("Filter by branches"),
         includeFacets: z.boolean().default(false).describe("Include facets in the search results"),
-        $skip: z.number().default(0).describe("Number of results to skip"),
-        $top: z.number().default(5).describe("Maximum number of results to return"),
-    }, async ({ searchText, project, repository, path, branch, includeFacets, $skip, $top }) => {
+        skip: z.number().default(0).describe("Number of results to skip"),
+        top: z.number().default(5).describe("Maximum number of results to return"),
+    }, async ({ searchText, project, repository, path, branch, includeFacets, skip, top }) => {
         const accessToken = await tokenProvider();
         const connection = await connectionProvider();
         const url = `https://almsearch.dev.azure.com/${orgName}/_apis/search/codesearchresults?api-version=${apiVersion}`;
         const requestBody = {
             searchText,
             includeFacets,
-            $skip,
-            $top,
+            $skip: skip,
+            $top: top,
         };
         const filters = {};
         if (project && project.length > 0)
@@ -66,16 +66,16 @@ function configureSearchTools(server, tokenProvider, connectionProvider, userAge
         project: z.array(z.string()).optional().describe("Filter by projects"),
         wiki: z.array(z.string()).optional().describe("Filter by wiki names"),
         includeFacets: z.boolean().default(false).describe("Include facets in the search results"),
-        $skip: z.number().default(0).describe("Number of results to skip"),
-        $top: z.number().default(10).describe("Maximum number of results to return"),
-    }, async ({ searchText, project, wiki, includeFacets, $skip, $top }) => {
+        skip: z.number().default(0).describe("Number of results to skip"),
+        top: z.number().default(10).describe("Maximum number of results to return"),
+    }, async ({ searchText, project, wiki, includeFacets, skip, top }) => {
         const accessToken = await tokenProvider();
         const url = `https://almsearch.dev.azure.com/${orgName}/_apis/search/wikisearchresults?api-version=${apiVersion}`;
         const requestBody = {
             searchText,
             includeFacets,
-            $skip,
-            $top,
+            $skip: skip,
+            $top: top,
         };
         const filters = {};
         if (project && project.length > 0)
@@ -110,16 +110,16 @@ function configureSearchTools(server, tokenProvider, connectionProvider, userAge
         state: z.array(z.string()).optional().describe("Filter by work item states"),
         assignedTo: z.array(z.string()).optional().describe("Filter by assigned to users"),
         includeFacets: z.boolean().default(false).describe("Include facets in the search results"),
-        $skip: z.number().default(0).describe("Number of results to skip for pagination"),
-        $top: z.number().default(10).describe("Number of results to return"),
-    }, async ({ searchText, project, areaPath, workItemType, state, assignedTo, includeFacets, $skip, $top }) => {
+        skip: z.number().default(0).describe("Number of results to skip for pagination"),
+        top: z.number().default(10).describe("Number of results to return"),
+    }, async ({ searchText, project, areaPath, workItemType, state, assignedTo, includeFacets, skip, top }) => {
         const accessToken = await tokenProvider();
         const url = `https://almsearch.dev.azure.com/${orgName}/_apis/search/workitemsearchresults?api-version=${apiVersion}`;
         const requestBody = {
             searchText,
             includeFacets,
-            $skip,
-            $top,
+            $skip: skip,
+            $top: top,
         };
         const filters = {};
         if (project && project.length > 0)

package/dist/tools/workitems.js

@@ -3,14 +3,6 @@
 import { QueryExpand } from "azure-devops-node-api/interfaces/WorkItemTrackingInterfaces.js";
 import { z } from "zod";
 import { batchApiVersion, markdownCommentsApiVersion, getEnumKeys, safeEnumConvert } from "../utils.js";
-/**
- * Converts Operation enum key to lowercase string for API usage
- * @param operation The Operation enum key (e.g., "Add", "Replace", "Remove")
- * @returns Lowercase string for API usage (e.g., "add", "replace", "remove")
- */
-function operationToApiString(operation) {
-    return operation.toLowerCase();
-}
 const WORKITEM_TOOLS = {
     my_work_items: "wit_my_work_items",
     list_backlogs: "wit_list_backlogs",
@@ -28,7 +20,6 @@ const WORKITEM_TOOLS = {
     get_query: "wit_get_query",
     get_query_results_by_id: "wit_get_query_results_by_id",
     update_work_items_batch: "wit_update_work_items_batch",
-    close_and_link_workitem_duplicates: "wit_close_and_link_workitem_duplicates",
     work_items_link: "wit_work_items_link",
 };
 function getLinkTypeFromName(name) {
@@ -353,7 +344,12 @@ function configureWorkItemTools(server, tokenProvider, connectionProvider, userA
         id: z.number().describe("The ID of the work item to update."),
         updates: z
             .array(z.object({
-            op: z.enum(["Add", "Replace", "Remove"]).default("Add").describe("The operation to perform on the field."),
+            op: z
+                .string()
+                .transform((val) => val.toLowerCase())
+                .pipe(z.enum(["add", "replace", "remove"]))
+                .default("add")
+                .describe("The operation to perform on the field."),
             path: z.string().describe("The path of the field to update, e.g., '/fields/System.Title'."),
             value: z.string().describe("The new value for the field. This is required for 'Add' and 'Replace' operations, and should be omitted for 'Remove' operations."),
         }))
@@ -364,7 +360,7 @@ function configureWorkItemTools(server, tokenProvider, connectionProvider, userA
         // Convert operation names to lowercase for API
         const apiUpdates = updates.map((update) => ({
             ...update,
-            op: operationToApiString(update.op),
+            op: update.op,
         }));
         const updatedWorkItem = await workItemApi.updateWorkItem(null, apiUpdates, id);
         return {
@@ -577,52 +573,5 @@ function configureWorkItemTools(server, tokenProvider, connectionProvider, userA
             content: [{ type: "text", text: JSON.stringify(result, null, 2) }],
         };
     });
-    server.tool(WORKITEM_TOOLS.close_and_link_workitem_duplicates, "Close duplicate work items by id.", {
-        id: z.number().describe("The ID of the work item to close and link duplicates to."),
-        duplicateIds: z.array(z.number()).describe("An array of IDs of the duplicate work items to close and link to the specified work item."),
-        project: z.string().describe("The name or ID of the Azure DevOps project."),
-        state: z.string().default("Removed").describe("The state to set for the duplicate work items. Defaults to 'Removed'."),
-    }, async ({ id, duplicateIds, project, state }) => {
-        const connection = await connectionProvider();
-        const body = duplicateIds.map((duplicateId) => ({
-            method: "PATCH",
-            uri: `/_apis/wit/workitems/${duplicateId}?api-version=${batchApiVersion}`,
-            headers: {
-                "Content-Type": "application/json-patch+json",
-            },
-            body: [
-                {
-                    op: "add",
-                    path: "/fields/System.State",
-                    value: `${state}`,
-                },
-                {
-                    op: "add",
-                    path: "/relations/-",
-                    value: {
-                        rel: "System.LinkTypes.Duplicate-Reverse",
-                        url: `${connection.serverUrl}/${project}/_apis/wit/workItems/${id}`,
-                    },
-                },
-            ],
-        }));
-        const accessToken = await tokenProvider();
-        const response = await fetch(`${connection.serverUrl}/_apis/wit/$batch?api-version=${batchApiVersion}`, {
-            method: "PATCH",
-            headers: {
-                "Authorization": `Bearer ${accessToken.token}`,
-                "Content-Type": "application/json",
-                "User-Agent": userAgentProvider(),
-            },
-            body: JSON.stringify(body),
-        });
-        if (!response.ok) {
-            throw new Error(`Failed to update work items in batch: ${response.statusText}`);
-        }
-        const result = await response.json();
-        return {
-            content: [{ type: "text", text: JSON.stringify(result, null, 2) }],
-        };
-    });
 }
 export { WORKITEM_TOOLS, configureWorkItemTools };

package/dist/version.js

@@ -1 +1 @@
-export const packageVersion = "1.2.0";
+export const packageVersion = "1.2.1";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@azure-devops/mcp",
-  "version": "1.2.0",
+  "version": "1.2.1",
   "description": "MCP server for interacting with Azure DevOps",
   "license": "MIT",
   "author": "Microsoft Corporation",