Filtering repos by name substring

Also improving code formatting

Author: nikolapeja6

src/tools/core.ts

@@ -6,11 +6,24 @@ import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { WebApi } from "azure-devops-node-api";
 import { z } from "zod";
 
+import type { ProjectInfo } from "azure-devops-node-api/interfaces/CoreInterfaces.js";
+
 const CORE_TOOLS = {
   list_project_teams: "core_list_project_teams",
   list_projects: "core_list_projects",  
 };
 
+function filterProjectsByName(
+  projects: ProjectInfo[],
+  projectNameFilter: string
+): ProjectInfo[]
+{
+  const lowerCaseFilter = projectNameFilter.toLowerCase();
+  return projects.filter((project) =>
+    project.name?.toLowerCase().includes(lowerCaseFilter)
+  );
+}
+
 function configureCoreTools(
   server: McpServer,
   tokenProvider: () => Promise<AccessToken>,
@@ -82,21 +95,12 @@ function configureCoreTools(
           return { content: [{ type: "text", text: "No projects found" }], isError: true };
         }
 
-        // If projectNameFilter is provided, filter the projects
-        if (projectNameFilter) {
-          const filteredProjects = projects.filter(project =>
-            project.name?.toLowerCase().includes(projectNameFilter.toLowerCase())
-          );
-          if (filteredProjects.length === 0) {
-            return { content: [{ type: "text", text: "No projects found matching the filter" }], isError: true };
-          }
-          return {
-            content: [{ type: "text", text: JSON.stringify(filteredProjects, null, 2) }],
-          };
-        }
+        const filteredProject = projectNameFilter
+          ? filterProjectsByName(projects, projectNameFilter)
+          : projects;
 
         return {
-          content: [{ type: "text", text: JSON.stringify(projects, null, 2) }],
+          content: [{ type: "text", text: JSON.stringify(filteredProject, null, 2) }],
         };
       } catch (error) {
         const errorMessage = error instanceof Error ? error.message : 'Unknown error occurred';

src/tools/repos.ts

@@ -7,6 +7,7 @@ import { WebApi } from "azure-devops-node-api";
 import { GitRef, PullRequestStatus } from "azure-devops-node-api/interfaces/GitInterfaces.js";
 import { z } from "zod";
 import { getCurrentUserDetails } from "./auth.js";
+import { GitRepository } from "azure-devops-node-api/interfaces/TfvcInterfaces.js";
 
 const REPO_TOOLS = {
   list_repos_by_project: "repo_list_repos_by_project",
@@ -55,6 +56,18 @@ function pullRequestStatusStringToInt(
   }
 }
 
+function filterReposByName(
+  repositories: GitRepository[],
+  repoNameFilter: string
+): GitRepository[] {
+  const lowerCaseFilter = repoNameFilter.toLowerCase();
+  const filteredByName = repositories?.filter((repo) =>
+    repo.name?.toLowerCase().includes(lowerCaseFilter)
+  );
+
+  return filteredByName;
+}
+
 function configureRepoTools(
   server: McpServer,
   tokenProvider: () => Promise<AccessToken>,
@@ -137,9 +150,10 @@ function configureRepoTools(
     REPO_TOOLS.list_repos_by_project,
     "Retrieve a list of repositories for a given project",
     { 
-      project: z.string().describe("The name or ID of the Azure DevOps project."), 
+      project: z.string().describe("The name or ID of the Azure DevOps project."),
+      repoNameFilter: z.string().optional().describe("Optional filter to search for repositories by name. If provided, only repositories with names containing this string will be returned."), 
     },
-    async ({ project }) => {
+    async ({ project, repoNameFilter }) => {
       const connection = await connectionProvider();
       const gitApi = await connection.getGitApi();
       const repositories = await gitApi.getRepositories(
@@ -149,8 +163,12 @@ function configureRepoTools(
         false
       );
 
+      const filteredRepositories = repoNameFilter
+        ? filterReposByName(repositories, repoNameFilter)
+        : repositories;
+
       // Filter out the irrelevant properties
-      const filteredRepositories = repositories?.map((repo) => ({
+      const trimmedRepositories = filteredRepositories?.map((repo) => ({
         id: repo.id,
         name: repo.name,
         isDisabled: repo.isDisabled,
@@ -162,7 +180,7 @@ function configureRepoTools(
 
       return {
         content: [
-          { type: "text", text: JSON.stringify(filteredRepositories, null, 2) },
+          { type: "text", text: JSON.stringify(trimmedRepositories, null, 2) },
         ],
       };
     }