Implement validate-dependencies MCP command for checking dependency validity
This commit is contained in:
@@ -0,0 +1,55 @@
|
||||
/**
|
||||
* Direct function wrapper for validateDependenciesCommand
|
||||
*/
|
||||
|
||||
import { validateDependenciesCommand } from '../../../../scripts/modules/dependency-manager.js';
|
||||
import { findTasksJsonPath } from '../utils/path-utils.js';
|
||||
import fs from 'fs';
|
||||
|
||||
/**
|
||||
* Validate dependencies in tasks.json
|
||||
* @param {Object} args - Function arguments
|
||||
* @param {string} [args.file] - Path to the tasks file
|
||||
* @param {string} [args.projectRoot] - Project root directory
|
||||
* @param {Object} log - Logger object
|
||||
* @returns {Promise<{success: boolean, data?: Object, error?: {code: string, message: string}}>}
|
||||
*/
|
||||
export async function validateDependenciesDirect(args, log) {
|
||||
try {
|
||||
log.info(`Validating dependencies in tasks...`);
|
||||
|
||||
// Determine the tasks file path
|
||||
const tasksPath = args.file || await findTasksJsonPath(args.projectRoot);
|
||||
|
||||
// Verify the file exists
|
||||
if (!fs.existsSync(tasksPath)) {
|
||||
return {
|
||||
success: false,
|
||||
error: {
|
||||
code: 'FILE_NOT_FOUND',
|
||||
message: `Tasks file not found at ${tasksPath}`
|
||||
}
|
||||
};
|
||||
}
|
||||
|
||||
// Call the original command function
|
||||
await validateDependenciesCommand(tasksPath);
|
||||
|
||||
return {
|
||||
success: true,
|
||||
data: {
|
||||
message: 'Dependencies validated successfully',
|
||||
tasksPath
|
||||
}
|
||||
};
|
||||
} catch (error) {
|
||||
log.error(`Error validating dependencies: ${error.message}`);
|
||||
return {
|
||||
success: false,
|
||||
error: {
|
||||
code: 'VALIDATION_ERROR',
|
||||
message: error.message
|
||||
}
|
||||
};
|
||||
}
|
||||
}
|
||||
@@ -23,6 +23,7 @@ import { analyzeTaskComplexityDirect } from './direct-functions/analyze-task-com
|
||||
import { clearSubtasksDirect } from './direct-functions/clear-subtasks.js';
|
||||
import { expandAllTasksDirect } from './direct-functions/expand-all-tasks.js';
|
||||
import { removeDependencyDirect } from './direct-functions/remove-dependency.js';
|
||||
import { validateDependenciesDirect } from './direct-functions/validate-dependencies.js';
|
||||
|
||||
// Re-export utility functions
|
||||
export { findTasksJsonPath } from './utils/path-utils.js';
|
||||
@@ -46,7 +47,8 @@ export const directFunctions = new Map([
|
||||
['analyzeTaskComplexityDirect', analyzeTaskComplexityDirect],
|
||||
['clearSubtasksDirect', clearSubtasksDirect],
|
||||
['expandAllTasksDirect', expandAllTasksDirect],
|
||||
['removeDependencyDirect', removeDependencyDirect]
|
||||
['removeDependencyDirect', removeDependencyDirect],
|
||||
['validateDependenciesDirect', validateDependenciesDirect]
|
||||
]);
|
||||
|
||||
// Re-export all direct function implementations
|
||||
@@ -68,5 +70,6 @@ export {
|
||||
analyzeTaskComplexityDirect,
|
||||
clearSubtasksDirect,
|
||||
expandAllTasksDirect,
|
||||
removeDependencyDirect
|
||||
removeDependencyDirect,
|
||||
validateDependenciesDirect
|
||||
};
|
||||
@@ -21,31 +21,41 @@ import { registerAnalyzeTool } from "./analyze.js";
|
||||
import { registerClearSubtasksTool } from "./clear-subtasks.js";
|
||||
import { registerExpandAllTool } from "./expand-all.js";
|
||||
import { registerRemoveDependencyTool } from "./remove-dependency.js";
|
||||
import { registerValidateDependenciesTool } from "./validate-dependencies.js";
|
||||
|
||||
/**
|
||||
* Register all Task Master tools with the MCP server
|
||||
* @param {Object} server - FastMCP server instance
|
||||
*/
|
||||
export function registerTaskMasterTools(server) {
|
||||
registerListTasksTool(server);
|
||||
registerSetTaskStatusTool(server);
|
||||
registerParsePRDTool(server);
|
||||
registerUpdateTool(server);
|
||||
registerUpdateTaskTool(server);
|
||||
registerUpdateSubtaskTool(server);
|
||||
registerGenerateTool(server);
|
||||
registerShowTaskTool(server);
|
||||
registerNextTaskTool(server);
|
||||
registerExpandTaskTool(server);
|
||||
registerAddTaskTool(server);
|
||||
registerAddSubtaskTool(server);
|
||||
registerRemoveSubtaskTool(server);
|
||||
registerAnalyzeTool(server);
|
||||
registerClearSubtasksTool(server);
|
||||
registerExpandAllTool(server);
|
||||
registerRemoveDependencyTool(server);
|
||||
logger.info("Registering Task Master tools with MCP server");
|
||||
|
||||
logger.info("Registered all Task Master tools with MCP server");
|
||||
try {
|
||||
// Register each tool
|
||||
registerListTasksTool(server);
|
||||
registerSetTaskStatusTool(server);
|
||||
registerParsePRDTool(server);
|
||||
registerUpdateTool(server);
|
||||
registerUpdateTaskTool(server);
|
||||
registerUpdateSubtaskTool(server);
|
||||
registerGenerateTool(server);
|
||||
registerShowTaskTool(server);
|
||||
registerNextTaskTool(server);
|
||||
registerExpandTaskTool(server);
|
||||
registerAddTaskTool(server);
|
||||
registerAddSubtaskTool(server);
|
||||
registerRemoveSubtaskTool(server);
|
||||
registerAnalyzeTool(server);
|
||||
registerClearSubtasksTool(server);
|
||||
registerExpandAllTool(server);
|
||||
registerRemoveDependencyTool(server);
|
||||
registerValidateDependenciesTool(server);
|
||||
|
||||
logger.info("Successfully registered all Task Master tools");
|
||||
} catch (error) {
|
||||
logger.error(`Error registering Task Master tools: ${error.message}`);
|
||||
throw error;
|
||||
}
|
||||
}
|
||||
|
||||
export default {
|
||||
|
||||
34
mcp-server/src/tools/validate-dependencies.js
Normal file
34
mcp-server/src/tools/validate-dependencies.js
Normal file
@@ -0,0 +1,34 @@
|
||||
/**
|
||||
* tools/validate-dependencies.js
|
||||
* Tool for validating task dependencies
|
||||
*/
|
||||
|
||||
import { z } from "zod";
|
||||
import {
|
||||
handleApiResult,
|
||||
createErrorResponse
|
||||
} from "./utils.js";
|
||||
import { validateDependenciesDirect } from "../core/task-master-core.js";
|
||||
|
||||
/**
|
||||
* Register the validateDependencies tool with the MCP server
|
||||
* @param {Object} server - FastMCP server instance
|
||||
*/
|
||||
export function registerValidateDependenciesTool(server) {
|
||||
server.addTool({
|
||||
name: "validate_dependencies",
|
||||
description: "Identify invalid dependencies in tasks without fixing them",
|
||||
parameters: z.object({
|
||||
file: z.string().optional().describe("Path to the tasks file"),
|
||||
projectRoot: z.string().optional().describe("Root directory of the project (default: current working directory)")
|
||||
}),
|
||||
handler: async ({ file, projectRoot }, { logger }) => {
|
||||
try {
|
||||
const result = await validateDependenciesDirect({ file, projectRoot }, logger);
|
||||
return handleApiResult(result);
|
||||
} catch (error) {
|
||||
return createErrorResponse(error);
|
||||
}
|
||||
}
|
||||
});
|
||||
}
|
||||
Reference in New Issue
Block a user