feat: add listDevlogNotes method and schema for retrieving devlog notes

Author: tikazyq

packages/mcp/src/adapters/mcp-adapter.ts

@@ -17,6 +17,7 @@ import type {
   GetCurrentProjectArgs,
   GetDevlogArgs,
   ListDevlogArgs,
+  ListDevlogNotesArgs,
   ListProjectsArgs,
   SwitchProjectArgs,
   UpdateDevlogArgs,
@@ -197,19 +198,10 @@ export class MCPAdapter {
 
       // Handle update with optional note
       if (args.note) {
-        // Then add the note
         await this.apiClient.addDevlogNote(args.id, args.note.content, args.note.category);
-
-        return this.toStandardResponse(true, { id: args.id }, `Updated entry ${args.id} with note`);
-      } else {
-        // Regular update without note
-        const entry = await this.apiClient.updateDevlog(args.id, {
-          status: args.status,
-          priority: args.priority,
-        });
-
-        return this.toStandardResponse(true, { id: args.id }, `Updated entry ${args.id}`);
       }
+
+      return this.toStandardResponse(true, { id: args.id }, `Updated entry ${args.id}`);
     } catch (error) {
       return this.handleError('Failed to update entry', error);
     }
@@ -242,22 +234,6 @@ export class MCPAdapter {
     }
   }
 
-  async addDevlogNote(args: AddDevlogNoteArgs): Promise<CallToolResult> {
-    await this.ensureInitialized();
-
-    try {
-      await this.apiClient.addDevlogNote(args.id, args.content, args.category);
-
-      return this.toStandardResponse(
-        true,
-        { id: args.id },
-        `Added ${args.category} note to entry ${args.id}`,
-      );
-    } catch (error) {
-      return this.handleError('Failed to add note', error);
-    }
-  }
-
   async findRelatedDevlogs(args: FindRelatedDevlogsArgs): Promise<CallToolResult> {
     await this.ensureInitialized();
 
@@ -294,6 +270,42 @@ export class MCPAdapter {
     }
   }
 
+  async addDevlogNote(args: AddDevlogNoteArgs): Promise<CallToolResult> {
+    await this.ensureInitialized();
+
+    try {
+      await this.apiClient.addDevlogNote(args.id, args.content, args.category);
+
+      return this.toStandardResponse(
+        true,
+        { id: args.id },
+        `Added ${args.category} note to entry ${args.id}`,
+      );
+    } catch (error) {
+      return this.handleError('Failed to add note', error);
+    }
+  }
+
+  async listDevlogNotes(args: ListDevlogNotesArgs): Promise<CallToolResult> {
+    await this.ensureInitialized();
+
+    try {
+      const result = await this.apiClient.listDevlogNotes(args.id, args.category, args.limit);
+
+      return this.toStandardResponse(
+        true,
+        {
+          devlogId: args.id,
+          total: result.total,
+          notes: result.notes,
+        },
+        `Found ${result.notes.length} notes for devlog ${args.id}`,
+      );
+    } catch (error) {
+      return this.handleError('Failed to list notes', error);
+    }
+  }
+
   // === PROJECT OPERATIONS ===
 
   async listProjects(args: ListProjectsArgs): Promise<CallToolResult> {

packages/mcp/src/api/devlog-api-client.ts

@@ -7,6 +7,8 @@ import type {
   CreateDevlogRequest,
   DevlogEntry,
   DevlogFilter,
+  DevlogNote,
+  DevlogNoteCategory,
   PaginatedResult,
   PaginationMeta,
   SortOptions,
@@ -286,6 +288,23 @@ export class DevlogApiClient {
     return this.unwrapApiResponse<DevlogEntry>(response);
   }
 
+  async listDevlogNotes(
+    devlogId: number,
+    category?: DevlogNoteCategory,
+    limit?: number,
+  ): Promise<{ devlogId: number; total: number; notes: DevlogNote[] }> {
+    const params = new URLSearchParams();
+
+    if (category) params.append('category', category);
+    if (limit) params.append('limit', limit.toString());
+
+    const query = params.toString() ? `?${params.toString()}` : '';
+    const response = await this.get(
+      `${this.getProjectEndpoint()}/devlogs/${devlogId}/notes${query}`,
+    );
+    return this.unwrapApiResponse(response);
+  }
+
   // Health check
   async healthCheck(): Promise<{ status: string; timestamp: string }> {
     try {

packages/mcp/src/handlers/tool-handlers.ts

@@ -16,6 +16,8 @@ import {
   type GetDevlogArgs,
   GetDevlogSchema,
   type ListDevlogArgs,
+  ListDevlogNotesArgs,
+  ListDevlogNotesSchema,
   ListDevlogSchema,
   type ListProjectsArgs,
   ListProjectsSchema,
@@ -75,6 +77,14 @@ export const toolHandlers = {
       adapter.listDevlogs(validArgs),
     ),
 
+  find_related_devlogs: (adapter: MCPAdapter, args: unknown) =>
+    validateAndHandle<FindRelatedDevlogsArgs>(
+      FindRelatedDevlogsSchema,
+      args,
+      'find_related_devlogs',
+      (validArgs) => adapter.findRelatedDevlogs(validArgs),
+    ),
+
   add_devlog_note: (adapter: MCPAdapter, args: unknown) =>
     validateAndHandle<AddDevlogNoteArgs>(
       AddDevlogNoteSchema,
@@ -83,12 +93,12 @@ export const toolHandlers = {
       (validArgs) => adapter.addDevlogNote(validArgs),
     ),
 
-  find_related_devlogs: (adapter: MCPAdapter, args: unknown) =>
-    validateAndHandle<FindRelatedDevlogsArgs>(
-      FindRelatedDevlogsSchema,
+  list_devlog_notes: (adapter: MCPAdapter, args: unknown) =>
+    validateAndHandle<ListDevlogNotesArgs>(
+      ListDevlogNotesSchema,
       args,
-      'find_related_devlogs',
-      (validArgs) => adapter.findRelatedDevlogs(validArgs),
+      'list_devlog_notes',
+      (validArgs) => adapter.listDevlogNotes(validArgs),
     ),
 
   // Project operations

packages/mcp/src/schemas/devlog-schemas.ts

@@ -54,7 +54,7 @@ export const UpdateDevlogSchema = z.object({
     .optional(),
 });
 
-// === LIST/SEARCH DEVLOGS ===
+// === LIST DEVLOGS ===
 export const ListDevlogSchema = z.object({
   status: DevlogStatusSchema.optional(),
   type: DevlogTypeSchema.optional(),
@@ -65,18 +65,25 @@ export const ListDevlogSchema = z.object({
   sortOrder: SortOrderSchema,
 });
 
+// === FIND RELATED DEVLOGS ===
+export const FindRelatedDevlogsSchema = z.object({
+  description: DescriptionSchema,
+  type: DevlogTypeSchema.optional(),
+  keywords: KeywordsSchema,
+  limit: LimitSchema,
+});
+
 // === ADD DEVLOG NOTE ===
 export const AddDevlogNoteSchema = z.object({
   id: DevlogIdSchema,
   content: DevlogNoteContentSchema,
   category: DevlogNoteCategorySchema,
 });
 
-// === FIND RELATED DEVLOGS ===
-export const FindRelatedDevlogsSchema = z.object({
-  description: DescriptionSchema,
-  type: DevlogTypeSchema.optional(),
-  keywords: KeywordsSchema,
+// === LIST DEVLOG NOTES ===
+export const ListDevlogNotesSchema = z.object({
+  id: DevlogIdSchema,
+  category: DevlogNoteCategorySchema,
   limit: LimitSchema,
 });
 
@@ -85,5 +92,6 @@ export type CreateDevlogArgs = z.infer<typeof CreateDevlogSchema>;
 export type GetDevlogArgs = z.infer<typeof GetDevlogSchema>;
 export type UpdateDevlogArgs = z.infer<typeof UpdateDevlogSchema>;
 export type ListDevlogArgs = z.infer<typeof ListDevlogSchema>;
-export type AddDevlogNoteArgs = z.infer<typeof AddDevlogNoteSchema>;
 export type FindRelatedDevlogsArgs = z.infer<typeof FindRelatedDevlogsSchema>;
+export type AddDevlogNoteArgs = z.infer<typeof AddDevlogNoteSchema>;
+export type ListDevlogNotesArgs = z.infer<typeof ListDevlogNotesSchema>;