Better filtering in branch tools (#440)

Adding option to filter branches by name when listing branches.

When getting branches, not all branches will be fetched and then
filtered on the client side, but we'll only fetch the relevant branch.

## GitHub issue number
/

## **Associated Risks**

## ✅ **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)**
- [x] 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?**
Manually tested, and updated tests.

Author: nikolapeja6

src/tools/repos.ts

@@ -486,11 +486,12 @@ function configureRepoTools(server: McpServer, tokenProvider: () => Promise<Acce
     {
       repositoryId: z.string().describe("The ID of the repository where the branches are located."),
       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 }) => {
+    async ({ repositoryId, top, filterContains }) => {
       const connection = await connectionProvider();
       const gitApi = await connection.getGitApi();
-      const branches = await gitApi.getRefs(repositoryId, undefined);
+      const branches = await gitApi.getRefs(repositoryId, undefined, "heads/", undefined, undefined, undefined, undefined, undefined, filterContains);
 
       const filteredBranches = branchesFilterOutIrrelevantProperties(branches, top);
 
@@ -506,11 +507,12 @@ function configureRepoTools(server: McpServer, tokenProvider: () => Promise<Acce
     {
       repositoryId: z.string().describe("The ID of the repository where the branches are located."),
       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 }) => {
+    async ({ repositoryId, top, filterContains }) => {
       const connection = await connectionProvider();
       const gitApi = await connection.getGitApi();
-      const branches = await gitApi.getRefs(repositoryId, undefined, undefined, undefined, undefined, true);
+      const branches = await gitApi.getRefs(repositoryId, undefined, "heads/", undefined, undefined, true, undefined, undefined, filterContains);
 
       const filteredBranches = branchesFilterOutIrrelevantProperties(branches, top);
 
@@ -554,8 +556,8 @@ function configureRepoTools(server: McpServer, tokenProvider: () => Promise<Acce
     async ({ repositoryId, branchName }) => {
       const connection = await connectionProvider();
       const gitApi = await connection.getGitApi();
-      const branches = await gitApi.getRefs(repositoryId);
-      const branch = branches?.find((branch) => branch.name === `refs/heads/${branchName}`);
+      const branches = await gitApi.getRefs(repositoryId, undefined, "heads/", false, false, undefined, false, undefined, branchName);
+      const branch = branches.find((branch) => branch.name === `refs/heads/${branchName}` || branch.name === branchName);
       if (!branch) {
         return {
           content: [
@@ -564,6 +566,7 @@ function configureRepoTools(server: McpServer, tokenProvider: () => Promise<Acce
               text: `Branch ${branchName} not found in repository ${repositoryId}`,
             },
           ],
+          isError: true,
         };
       }
       return {

test/src/tools/repos.test.ts

@@ -763,7 +763,7 @@ describe("repos tools", () => {
 
       const result = await handler(params);
 
-      expect(mockGitApi.getRefs).toHaveBeenCalledWith("repo123", undefined);
+      expect(mockGitApi.getRefs).toHaveBeenCalledWith("repo123", undefined, "heads/", undefined, undefined, undefined, undefined, undefined, undefined);
 
       const expectedResult = ["main", "feature-2", "feature-1"]; // Sorted reverse alphabetically
       expect(result.content[0].text).toBe(JSON.stringify(expectedResult, null, 2));
@@ -788,7 +788,7 @@ describe("repos tools", () => {
 
       const result = await handler(params);
 
-      expect(mockGitApi.getRefs).toHaveBeenCalledWith("repo123", undefined, undefined, undefined, undefined, true);
+      expect(mockGitApi.getRefs).toHaveBeenCalledWith("repo123", undefined, "heads/", undefined, undefined, true, undefined, undefined, undefined);
 
       const expectedResult = ["my-feature", "main"];
       expect(result.content[0].text).toBe(JSON.stringify(expectedResult, null, 2));
@@ -882,7 +882,7 @@ describe("repos tools", () => {
 
       const result = await handler(params);
 
-      expect(mockGitApi.getRefs).toHaveBeenCalledWith("repo123");
+      expect(mockGitApi.getRefs).toHaveBeenCalledWith("repo123", undefined, "heads/", false, false, undefined, false, undefined, "main");
       expect(result.content[0].text).toBe(JSON.stringify(mockBranches[0], null, 2));
     });