Remove deprecated close_and_link_workitem_duplicates tool and its tests (#316)

Removing `wit_close_and_link_workitem_duplicates`. This is no longer
needed and the scenario can be accomplished using work item link and
work item updates.

## GitHub issue number
N/A

## **Associated Risks**

Low: Existing customers could be used to using the tool

## ✅ **PR Checklist**

- [x] **I have read the [contribution
guidelines](https://github.com/microsoft/azure-devops-mcp/blob/main/CONTRIBUTING.md)**
- [x] **I have read the [code of conduct
guidelines](https://github.com/microsoft/azure-devops-mcp/blob/main/CODE_OF_CONDUCT.md)**
- [ ] Title of the pull request is clear and informative.
- [x] 👌 Code hygiene
- [x] 🔭 Telemetry added, updated, or N/A
- [x] 📄 Documentation added, updated, or N/A
- [x] 🛡️ Automated tests added, or N/A

## 🧪 **How did you test it?**

Tested process manually. Updated tests

Author: danhellem

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
 

src/tools/workitems.ts

@@ -36,7 +36,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",
 };
 
@@ -762,65 +761,6 @@ function configureWorkItemTools(server: McpServer, tokenProvider: () => Promise<
       };
     }
   );
-
-  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 };

test/src/tools/workitems.test.ts

@@ -1055,83 +1055,6 @@ describe("configureWorkItemTools", () => {
     });
   });
 
-  describe("close_and_link_workitem_duplicates tool", () => {
-    it("should close and link duplicate work items successfully", async () => {
-      configureWorkItemTools(server, tokenProvider, connectionProvider, userAgentProvider);
-
-      const call = (server.tool as jest.Mock).mock.calls.find(([toolName]) => toolName === "wit_close_and_link_workitem_duplicates");
-      if (!call) throw new Error("wit_close_and_link_workitem_duplicates tool not registered");
-      const [, , , handler] = call;
-
-      mockConnection.serverUrl = "https://dev.azure.com/contoso";
-      (tokenProvider as jest.Mock).mockResolvedValue({ token: "fake-token" });
-
-      global.fetch = jest.fn().mockResolvedValue({
-        ok: true,
-        json: jest.fn().mockResolvedValue([
-          { id: 2, success: true },
-          { id: 3, success: true },
-        ]),
-      });
-
-      const params = {
-        id: 1,
-        duplicateIds: [2, 3],
-        project: "TestProject",
-        state: "Closed",
-      };
-
-      const result = await handler(params);
-
-      expect(fetch).toHaveBeenCalledWith(
-        "https://dev.azure.com/contoso/_apis/wit/$batch?api-version=5.0",
-        expect.objectContaining({
-          method: "PATCH",
-          headers: expect.objectContaining({
-            "Authorization": "Bearer fake-token",
-            "Content-Type": "application/json",
-          }),
-        })
-      );
-
-      expect(result.content[0].text).toBe(
-        JSON.stringify(
-          [
-            { id: 2, success: true },
-            { id: 3, success: true },
-          ],
-          null,
-          2
-        )
-      );
-    });
-
-    it("should handle duplicate closure failure", async () => {
-      configureWorkItemTools(server, tokenProvider, connectionProvider, userAgentProvider);
-
-      const call = (server.tool as jest.Mock).mock.calls.find(([toolName]) => toolName === "wit_close_and_link_workitem_duplicates");
-      if (!call) throw new Error("wit_close_and_link_workitem_duplicates tool not registered");
-      const [, , , handler] = call;
-
-      mockConnection.serverUrl = "https://dev.azure.com/contoso";
-      (tokenProvider as jest.Mock).mockResolvedValue({ token: "fake-token" });
-
-      global.fetch = jest.fn().mockResolvedValue({
-        ok: false,
-        statusText: "Forbidden",
-      });
-
-      const params = {
-        id: 1,
-        duplicateIds: [2, 3],
-        project: "TestProject",
-        state: "Removed",
-      };
-
-      await expect(handler(params)).rejects.toThrow("Failed to update work items in batch: Forbidden");
-    });
-  });
-
   // Add error handling tests for existing tools
   describe("error handling coverage", () => {
     it("should handle create_work_item errors", async () => {