docs: update task status for get_repository_details

Author: openhands-agent

project-management/task-management/doing.md

@@ -1,17 +1,3 @@
 ## Current Task
 
-- [ ] **Task**: Implement get_repository_details core functionality
-  - **Role**: Full-Stack Developer
-  - **Phase**: Research
-  - **Notes**:
-    - Implement a handler to fetch detailed information about a specific Git repository in Azure DevOps
-    - Include repository metadata, branch statistics, and refs information
-    - Create appropriate tests and documentation
-  - **Sub-tasks**:
-    1. Create schema for get_repository_details
-    2. Implement core functionality to fetch repository details
-    3. Add support for branch statistics
-    4. Add support for repository refs
-    5. Write unit tests
-    6. Write integration tests
-    7. Update documentation
+No task is currently in progress. Please take the next task from todo.md.

project-management/task-management/done.md

@@ -1,5 +1,23 @@
 ## Completed Tasks
 
+- [x] **Task 3.1**: Implement get_repository_details core functionality
+  - **Role**: Full-Stack Developer
+  - **Phase**: Completed
+  - **Notes**:
+    - Implemented a handler to fetch detailed information about a specific Git repository in Azure DevOps
+    - Added support for repository metadata, branch statistics, and refs information
+    - Created unit tests and integration tests
+    - Updated documentation
+  - **Sub-tasks**:
+    - [x] Created schema for get_repository_details
+    - [x] Implemented core functionality to fetch repository details
+    - [x] Added support for branch statistics
+    - [x] Added support for repository refs
+    - [x] Wrote unit tests
+    - [x] Wrote integration tests
+    - [x] Updated documentation
+  - **Completed**: April 2, 2025
+
 - [x] **Task 2.8**: Allow `get_work_item` to default to 'Expand All' when no specific fields are requested. There isn't usually enough information on the default Get_work_item response now.
   - **Role**: Full-Stack Developer
   - **Phase**: Completed

src/features/repositories/get-repository-details/feature.spec.int.ts

@@ -42,13 +42,10 @@ describe('getRepositoryDetails integration', () => {
     const testRepo = repos[0];
 
     // Act - make an actual API call to Azure DevOps
-    const result = await getRepositoryDetails(
-      connection,
-      {
-        projectId: projectName,
-        repositoryId: testRepo.name || testRepo.id || '',
-      },
-    );
+    const result = await getRepositoryDetails(connection, {
+      projectId: projectName,
+      repositoryId: testRepo.name || testRepo.id || '',
+    });
 
     // Assert on the actual response
     expect(result).toBeDefined();
@@ -88,14 +85,11 @@ describe('getRepositoryDetails integration', () => {
     const testRepo = repos[0];
 
     // Act - make an actual API call to Azure DevOps
-    const result = await getRepositoryDetails(
-      connection,
-      {
-        projectId: projectName,
-        repositoryId: testRepo.name || testRepo.id || '',
-        includeStatistics: true,
-      },
-    );
+    const result = await getRepositoryDetails(connection, {
+      projectId: projectName,
+      repositoryId: testRepo.name || testRepo.id || '',
+      includeStatistics: true,
+    });
 
     // Assert on the actual response
     expect(result).toBeDefined();
@@ -132,14 +126,11 @@ describe('getRepositoryDetails integration', () => {
     const testRepo = repos[0];
 
     // Act - make an actual API call to Azure DevOps
-    const result = await getRepositoryDetails(
-      connection,
-      {
-        projectId: projectName,
-        repositoryId: testRepo.name || testRepo.id || '',
-        includeRefs: true,
-      },
-    );
+    const result = await getRepositoryDetails(connection, {
+      projectId: projectName,
+      repositoryId: testRepo.name || testRepo.id || '',
+      includeRefs: true,
+    });
 
     // Assert on the actual response
     expect(result).toBeDefined();
@@ -178,25 +169,22 @@ describe('getRepositoryDetails integration', () => {
     const testRepo = repos[0];
 
     // Act - make an actual API call to Azure DevOps
-    const result = await getRepositoryDetails(
-      connection,
-      {
-        projectId: projectName,
-        repositoryId: testRepo.name || testRepo.id || '',
-        includeRefs: true,
-        refFilter: 'heads/',
-      },
-    );
+    const result = await getRepositoryDetails(connection, {
+      projectId: projectName,
+      repositoryId: testRepo.name || testRepo.id || '',
+      includeRefs: true,
+      refFilter: 'heads/',
+    });
 
     // Assert on the actual response
     expect(result).toBeDefined();
     expect(result.repository).toBeDefined();
     expect(result.refs).toBeDefined();
     expect(result.refs?.value).toBeDefined();
-    
+
     // All refs should start with refs/heads/
     if (result.refs && result.refs.value.length > 0) {
-      result.refs.value.forEach(ref => {
+      result.refs.value.forEach((ref) => {
         expect(ref.name).toMatch(/^refs\/heads\//);
       });
     }
@@ -226,4 +214,4 @@ describe('getRepositoryDetails integration', () => {
       }),
     ).rejects.toThrow(/not found|Failed to get repository/);
   });
-});
+});

src/features/repositories/get-repository-details/feature.spec.unit.ts

@@ -182,7 +182,7 @@ describe('getRepositoryDetails unit', () => {
     expect(getRefs).toHaveBeenCalledWith(
       mockRepository.id,
       'test-project',
-      'heads/'
+      'heads/',
     );
   });
 
@@ -210,8 +210,8 @@ describe('getRepositoryDetails unit', () => {
       'test-project',
       {
         version: 'main',
-        versionType: GitVersionType.Branch
-      }
+        versionType: GitVersionType.Branch,
+      },
     );
   });
 
@@ -305,4 +305,4 @@ describe('getRepositoryDetails unit', () => {
     expect(result.refs?.value).toEqual([]);
     expect(result.refs?.count).toBe(0);
   });
-});
+});

src/features/repositories/get-repository-details/feature.ts

@@ -20,9 +20,12 @@ export async function getRepositoryDetails(
 ): Promise<RepositoryDetails> {
   try {
     const gitApi = await connection.getGitApi();
-    
+
     // Get the basic repository information
-    const repository = await gitApi.getRepository(options.repositoryId, options.projectId);
+    const repository = await gitApi.getRepository(
+      options.repositoryId,
+      options.projectId,
+    );
 
     if (!repository) {
       throw new AzureDevOpsResourceNotFoundError(
@@ -38,19 +41,19 @@ export async function getRepositoryDetails(
     // Get branch statistics if requested
     if (options.includeStatistics) {
       let baseVersionDescriptor = undefined;
-      
+
       // If a specific branch name is provided, create a version descriptor for it
       if (options.branchName) {
         baseVersionDescriptor = {
           version: options.branchName,
-          versionType: GitVersionType.Branch
+          versionType: GitVersionType.Branch,
         };
       }
-      
+
       const branchStats = await gitApi.getBranches(
         repository.id || '',
         options.projectId,
-        baseVersionDescriptor
+        baseVersionDescriptor,
       );
 
       response.statistics = {
@@ -64,7 +67,7 @@ export async function getRepositoryDetails(
       const refs = await gitApi.getRefs(
         repository.id || '',
         options.projectId,
-        filter
+        filter,
       );
 
       if (refs) {
@@ -89,4 +92,4 @@ export async function getRepositoryDetails(
       `Failed to get repository details: ${error instanceof Error ? error.message : String(error)}`,
     );
   }
-}
+}

src/features/repositories/get-repository-details/index.ts

@@ -1,2 +1,2 @@
 export * from './schema';
-export * from './feature';
+export * from './feature';

src/features/repositories/get-repository-details/schema.ts

@@ -1,3 +1,3 @@
 import { GetRepositoryDetailsSchema } from '../schemas';
 
-export { GetRepositoryDetailsSchema };
+export { GetRepositoryDetailsSchema };

src/features/repositories/schemas.ts

@@ -31,7 +31,9 @@ export const GetRepositoryDetailsSchema = z.object({
   branchName: z
     .string()
     .optional()
-    .describe('Name of specific branch to get statistics for (if includeStatistics is true)'),
+    .describe(
+      'Name of specific branch to get statistics for (if includeStatistics is true)',
+    ),
 });
 
 /**

src/features/repositories/types.ts

@@ -1,7 +1,7 @@
-import { 
-  GitRepository, 
-  GitBranchStats, 
-  GitRef 
+import {
+  GitRepository,
+  GitBranchStats,
+  GitRef,
 } from 'azure-devops-node-api/interfaces/GitInterfaces';
 
 /**

src/server.ts

@@ -137,7 +137,8 @@ export function createAzureDevOpsServer(config: AzureDevOpsConfig): Server {
         },
         {
           name: 'get_repository_details',
-          description: 'Get detailed information about a repository including statistics and refs',
+          description:
+            'Get detailed information about a repository including statistics and refs',
           inputSchema: zodToJsonSchema(GetRepositoryDetailsSchema),
         },
         {
@@ -276,18 +277,17 @@ export function createAzureDevOpsServer(config: AzureDevOpsConfig): Server {
           };
         }
         case 'get_repository_details': {
-          const args = GetRepositoryDetailsSchema.parse(request.params.arguments);
-          const result = await getRepositoryDetails(
-            connection,
-            {
-              projectId: args.projectId,
-              repositoryId: args.repositoryId,
-              includeStatistics: args.includeStatistics,
-              includeRefs: args.includeRefs,
-              refFilter: args.refFilter,
-              branchName: args.branchName,
-            },
+          const args = GetRepositoryDetailsSchema.parse(
+            request.params.arguments,
           );
+          const result = await getRepositoryDetails(connection, {
+            projectId: args.projectId,
+            repositoryId: args.repositoryId,
+            includeStatistics: args.includeStatistics,
+            includeRefs: args.includeRefs,
+            refFilter: args.refFilter,
+            branchName: args.branchName,
+          });
           return {
             content: [{ type: 'text', text: JSON.stringify(result, null, 2) }],
           };