@aaronsb/jira-cloud-mcp 0.2.7 → 0.3.0

package/build/handlers/board-handlers.js

@@ -1,31 +1,7 @@
 import { ErrorCode, McpError } from '@modelcontextprotocol/sdk/types.js';
 import { MarkdownRenderer } from '../mcp/markdown-renderer.js';
-// Helper function to normalize parameter names (support both snake_case and camelCase)
-function normalizeArgs(args) {
-    const normalized = {};
-    for (const [key, value] of Object.entries(args)) {
-        // Convert snake_case to camelCase
-        if (key === 'board_id') {
-            normalized['boardId'] = value;
-        }
-        else if (key === 'include_sprints') {
-            normalized['includeSprints'] = value;
-        }
-        else if (key === 'project_key') {
-            normalized['projectKey'] = value;
-        }
-        else if (key === 'start_at') {
-            normalized['startAt'] = value;
-        }
-        else if (key === 'max_results') {
-            normalized['maxResults'] = value;
-        }
-        else {
-            normalized[key] = value;
-        }
-    }
-    return normalized;
-}
+import { boardNextSteps } from '../utils/next-steps.js';
+import { normalizeArgs } from '../utils/normalize-args.js';
 // Validate the consolidated board management arguments
 function validateManageJiraBoardArgs(args) {
     if (typeof args !== 'object' || args === null) {
@@ -34,28 +10,14 @@ function validateManageJiraBoardArgs(args) {
     const normalizedArgs = normalizeArgs(args);
     // Validate operation parameter
     if (typeof normalizedArgs.operation !== 'string' ||
-        !['get', 'list', 'create', 'update', 'delete', 'get_configuration'].includes(normalizedArgs.operation)) {
-        throw new McpError(ErrorCode.InvalidParams, 'Invalid operation parameter. Valid values are: get, list, create, update, delete, get_configuration');
+        !['get', 'list'].includes(normalizedArgs.operation)) {
+        throw new McpError(ErrorCode.InvalidParams, 'Invalid operation parameter. Valid values are: get, list');
     }
     // Validate parameters based on operation
     switch (normalizedArgs.operation) {
         case 'get':
-        case 'update':
-        case 'delete':
-        case 'get_configuration':
             if (typeof normalizedArgs.boardId !== 'number') {
-                throw new McpError(ErrorCode.InvalidParams, `Missing or invalid boardId parameter. Please provide a valid board ID as a number for the ${normalizedArgs.operation} operation.`);
-            }
-            break;
-        case 'create':
-            if (typeof normalizedArgs.name !== 'string' || normalizedArgs.name.trim() === '') {
-                throw new McpError(ErrorCode.InvalidParams, 'Missing or invalid name parameter. Please provide a valid board name for the create operation.');
-            }
-            if (typeof normalizedArgs.type !== 'string' || !['scrum', 'kanban'].includes(normalizedArgs.type)) {
-                throw new McpError(ErrorCode.InvalidParams, 'Missing or invalid type parameter. Please provide a valid board type (scrum or kanban) for the create operation.');
-            }
-            if (typeof normalizedArgs.projectKey !== 'string' || normalizedArgs.projectKey.trim() === '') {
-                throw new McpError(ErrorCode.InvalidParams, 'Missing or invalid projectKey parameter. Please provide a valid project key for the create operation.');
+                throw new McpError(ErrorCode.InvalidParams, 'Missing or invalid boardId parameter. Please provide a valid board ID as a number for the get operation.');
             }
             break;
     }
@@ -97,7 +59,7 @@ async function handleGetBoard(jiraClient, args) {
         }
     }
     // If include_sprints is true, add sprints to expansions
-    if (args.include_sprints === true) {
+    if (args.includeSprints === true) {
         expansionOptions.sprints = true;
     }
     // Get all boards and find the requested one
@@ -143,7 +105,7 @@ async function handleGetBoard(jiraClient, args) {
         content: [
             {
                 type: 'text',
-                text: markdown,
+                text: markdown + boardNextSteps('get', boardId),
             },
         ],
     };
@@ -152,7 +114,7 @@ async function handleListBoards(jiraClient, args) {
     // Set default pagination values
     const startAt = args.startAt !== undefined ? args.startAt : 0;
     const maxResults = args.maxResults !== undefined ? args.maxResults : 50;
-    const includeSprints = args.include_sprints === true;
+    const includeSprints = args.includeSprints === true;
     // Get all boards
     const boards = await jiraClient.listBoards();
     // Apply pagination
@@ -204,6 +166,7 @@ async function handleListBoards(jiraClient, args) {
     else {
         markdown += `Showing all ${boardDataList.length} board${boardDataList.length !== 1 ? 's' : ''}`;
     }
+    markdown += boardNextSteps('list');
     return {
         content: [
             {
@@ -213,98 +176,8 @@ async function handleListBoards(jiraClient, args) {
         ],
     };
 }
-async function handleCreateBoard(_jiraClient, _args) {
-    // Note: This is a placeholder. The current JiraClient doesn't have a createBoard method.
-    // You would need to implement this in the JiraClient class.
-    throw new McpError(ErrorCode.InternalError, 'Create board operation is not yet implemented');
-    // When implemented, it would look something like this:
-    /*
-    const result = await _jiraClient.createBoard({
-      name: _args.name!,
-      type: _args.type!,
-      projectKey: _args.projectKey!
-    });
-    
-    // Get the created board to return
-    const createdBoard = await _jiraClient.getBoard(result.id);
-    const formattedResponse = BoardFormatter.formatBoard(createdBoard);
-  
-    return {
-      content: [
-        {
-          type: 'text',
-          text: JSON.stringify(formattedResponse, null, 2),
-        },
-      ],
-    };
-    */
-}
-async function handleUpdateBoard(_jiraClient, _args) {
-    // Note: This is a placeholder. The current JiraClient doesn't have an updateBoard method.
-    // You would need to implement this in the JiraClient class.
-    throw new McpError(ErrorCode.InternalError, 'Update board operation is not yet implemented');
-    // When implemented, it would look something like this:
-    /*
-    await _jiraClient.updateBoard(
-      _args.boardId!,
-      _args.name
-    );
-  
-    // Get the updated board to return
-    const updatedBoard = await _jiraClient.getBoard(_args.boardId!);
-    const formattedResponse = BoardFormatter.formatBoard(updatedBoard);
-  
-    return {
-      content: [
-        {
-          type: 'text',
-          text: JSON.stringify(formattedResponse, null, 2),
-        },
-      ],
-    };
-    */
-}
-async function handleDeleteBoard(_jiraClient, _args) {
-    // Note: This is a placeholder. The current JiraClient doesn't have a deleteBoard method.
-    // You would need to implement this in the JiraClient class.
-    throw new McpError(ErrorCode.InternalError, 'Delete board operation is not yet implemented');
-    // When implemented, it would look something like this:
-    /*
-    await _jiraClient.deleteBoard(_args.boardId!);
-  
-    return {
-      content: [
-        {
-          type: 'text',
-          text: JSON.stringify({
-            success: true,
-            message: `Board ${_args.boardId} has been deleted successfully.`,
-          }, null, 2),
-        },
-      ],
-    };
-    */
-}
-async function handleGetBoardConfiguration(_jiraClient, _args) {
-    // Note: This is a placeholder. The current JiraClient doesn't have a getBoardConfiguration method.
-    // You would need to implement this in the JiraClient class.
-    throw new McpError(ErrorCode.InternalError, 'Get board configuration operation is not yet implemented');
-    // When implemented, it would look something like this:
-    /*
-    const configuration = await _jiraClient.getBoardConfiguration(_args.boardId!);
-    
-    return {
-      content: [
-        {
-          type: 'text',
-          text: JSON.stringify(configuration, null, 2),
-        },
-      ],
-    };
-    */
-}
 // Main handler function
-export async function setupBoardHandlers(server, jiraClient, request) {
+export async function handleBoardRequest(jiraClient, request) {
     console.error('Handling board request...');
     const { name } = request.params;
     const args = request.params.arguments || {};
@@ -326,22 +199,6 @@ export async function setupBoardHandlers(server, jiraClient, request) {
                 console.error('Processing list boards operation');
                 return await handleListBoards(jiraClient, normalizedArgs);
             }
-            case 'create': {
-                console.error('Processing create board operation');
-                return await handleCreateBoard(jiraClient, normalizedArgs);
-            }
-            case 'update': {
-                console.error('Processing update board operation');
-                return await handleUpdateBoard(jiraClient, normalizedArgs);
-            }
-            case 'delete': {
-                console.error('Processing delete board operation');
-                return await handleDeleteBoard(jiraClient, normalizedArgs);
-            }
-            case 'get_configuration': {
-                console.error('Processing get board configuration operation');
-                return await handleGetBoardConfiguration(jiraClient, normalizedArgs);
-            }
             default: {
                 console.error(`Unknown operation: ${normalizedArgs.operation}`);
                 throw new McpError(ErrorCode.MethodNotFound, `Unknown operation: ${normalizedArgs.operation}`);

package/build/handlers/filter-handlers.js

@@ -1,28 +1,7 @@
 import { ErrorCode, McpError } from '@modelcontextprotocol/sdk/types.js';
 import { MarkdownRenderer } from '../mcp/markdown-renderer.js';
-// Helper function to normalize parameter names (support both snake_case and camelCase)
-function normalizeArgs(args) {
-    const normalized = {};
-    for (const [key, value] of Object.entries(args)) {
-        // Convert snake_case to camelCase
-        if (key === 'filter_id') {
-            normalized['filterId'] = value;
-        }
-        else if (key === 'share_permissions') {
-            normalized['sharePermissions'] = value;
-        }
-        else if (key === 'start_at') {
-            normalized['startAt'] = value;
-        }
-        else if (key === 'max_results') {
-            normalized['maxResults'] = value;
-        }
-        else {
-            normalized[key] = value;
-        }
-    }
-    return normalized;
-}
+import { filterNextSteps } from '../utils/next-steps.js';
+import { normalizeArgs } from '../utils/normalize-args.js';
 // Validate the consolidated filter management arguments
 function validateManageJiraFilterArgs(args) {
     if (typeof args !== 'object' || args === null) {
@@ -157,7 +136,7 @@ async function handleGetFilter(jiraClient, args) {
             content: [
                 {
                     type: 'text',
-                    text: markdown,
+                    text: markdown + filterNextSteps('get', filterId),
                 },
             ],
         };
@@ -218,6 +197,7 @@ async function handleListFilters(jiraClient, args) {
     else {
         markdown += `Showing all ${filterDataList.length} filter${filterDataList.length !== 1 ? 's' : ''}`;
     }
+    markdown += filterNextSteps('list');
     return {
         content: [
             {
@@ -228,84 +208,13 @@ async function handleListFilters(jiraClient, args) {
     };
 }
 async function handleCreateFilter(_jiraClient, _args) {
-    // Note: This is a placeholder. The current JiraClient doesn't have a createFilter method.
-    // You would need to implement this in the JiraClient class.
     throw new McpError(ErrorCode.InternalError, 'Create filter operation is not yet implemented');
-    // When implemented, it would look something like this:
-    /*
-    const result = await _jiraClient.createFilter({
-      name: _args.name!,
-      jql: _args.jql!,
-      description: _args.description,
-      favourite: _args.favourite,
-      sharePermissions: _args.sharePermissions
-    });
-    
-    // Get the created filter to return
-    const createdFilter = await _jiraClient.getFilter(result.id);
-    const formattedResponse = FilterFormatter.formatFilter(createdFilter);
-  
-    return {
-      content: [
-        {
-          type: 'text',
-          text: JSON.stringify(formattedResponse, null, 2),
-        },
-      ],
-    };
-    */
 }
 async function handleUpdateFilter(_jiraClient, _args) {
-    // Note: This is a placeholder. The current JiraClient doesn't have an updateFilter method.
-    // You would need to implement this in the JiraClient class.
     throw new McpError(ErrorCode.InternalError, 'Update filter operation is not yet implemented');
-    // When implemented, it would look something like this:
-    /*
-    await _jiraClient.updateFilter(
-      _args.filterId!,
-      {
-        name: _args.name,
-        jql: _args.jql,
-        description: _args.description,
-        favourite: _args.favourite,
-        sharePermissions: _args.sharePermissions
-      }
-    );
-  
-    // Get the updated filter to return
-    const updatedFilter = await _jiraClient.getFilter(_args.filterId!);
-    const formattedResponse = FilterFormatter.formatFilter(updatedFilter);
-  
-    return {
-      content: [
-        {
-          type: 'text',
-          text: JSON.stringify(formattedResponse, null, 2),
-        },
-      ],
-    };
-    */
 }
 async function handleDeleteFilter(_jiraClient, _args) {
-    // Note: This is a placeholder. The current JiraClient doesn't have a deleteFilter method.
-    // You would need to implement this in the JiraClient class.
     throw new McpError(ErrorCode.InternalError, 'Delete filter operation is not yet implemented');
-    // When implemented, it would look something like this:
-    /*
-    await _jiraClient.deleteFilter(_args.filterId!);
-  
-    return {
-      content: [
-        {
-          type: 'text',
-          text: JSON.stringify({
-            success: true,
-            message: `Filter ${_args.filterId} has been deleted successfully.`,
-          }, null, 2),
-        },
-      ],
-    };
-    */
 }
 async function handleExecuteFilter(jiraClient, _args) {
     const filterId = _args.filterId;
@@ -322,7 +231,7 @@ async function handleExecuteFilter(jiraClient, _args) {
         content: [
             {
                 type: 'text',
-                text: markdown,
+                text: markdown + filterNextSteps('execute_filter', filterId),
             },
         ],
     };
@@ -353,7 +262,7 @@ async function handleExecuteJql(jiraClient, args) {
             content: [
                 {
                     type: 'text',
-                    text: markdown,
+                    text: markdown + filterNextSteps('execute_jql', undefined, args.jql),
                 },
             ],
         };
@@ -367,7 +276,7 @@ async function handleExecuteJql(jiraClient, args) {
     }
 }
 // Main handler function
-export async function setupFilterHandlers(server, jiraClient, request) {
+export async function handleFilterRequest(jiraClient, request) {
     console.error('Handling filter request...');
     const { name } = request.params;
     const args = request.params.arguments || {};