5119cd2d8e
* fix(research, tasks): Make research command and task updates tag-aware * refactor(tasks): Prevent automatic task file generation across other locations This commit refactors several core task management functions to prevent them from automatically regenerating individual task files after modifying the main `tasks.json`. Previously, operations like `add-task`, `clear-subtasks`, `expand-task`, and `update-task-by-id` would immediately trigger `generateTaskFiles`. This could be slow and was often unnecessary. The calls to `generateTaskFiles` have been removed or commented out from the core task functions. Users should now run `task-master generate` explicitly to update their individual task files. Additionally, this commit includes fixes to the `move` command to make it fully tag-aware. * fix: move and clear subtasks mcp commands * chore: fix format * chore: fix unit tests --------- Co-authored-by: Ralph Khreish <35776126+Crunchyman-ralph@users.noreply.github.com>
106 lines
2.6 KiB
JavaScript
106 lines
2.6 KiB
JavaScript
/**
|
|
* Direct function wrapper for moveTask
|
|
*/
|
|
|
|
import { moveTask } from '../../../../scripts/modules/task-manager.js';
|
|
import { findTasksPath } from '../utils/path-utils.js';
|
|
import {
|
|
enableSilentMode,
|
|
disableSilentMode
|
|
} from '../../../../scripts/modules/utils.js';
|
|
|
|
/**
|
|
* Move a task or subtask to a new position
|
|
* @param {Object} args - Function arguments
|
|
* @param {string} args.tasksJsonPath - Explicit path to the tasks.json file
|
|
* @param {string} args.sourceId - ID of the task/subtask to move (e.g., '5' or '5.2' or '5,6,7')
|
|
* @param {string} args.destinationId - ID of the destination (e.g., '7' or '7.3' or '7,8,9')
|
|
* @param {string} args.file - Alternative path to the tasks.json file
|
|
* @param {string} args.projectRoot - Project root directory
|
|
* @param {boolean} args.generateFiles - Whether to regenerate task files after moving (default: true)
|
|
* @param {Object} log - Logger object
|
|
* @returns {Promise<{success: boolean, data?: Object, error?: Object}>}
|
|
*/
|
|
export async function moveTaskDirect(args, log, context = {}) {
|
|
const { session } = context;
|
|
|
|
// Validate required parameters
|
|
if (!args.sourceId) {
|
|
return {
|
|
success: false,
|
|
error: {
|
|
message: 'Source ID is required',
|
|
code: 'MISSING_SOURCE_ID'
|
|
}
|
|
};
|
|
}
|
|
|
|
if (!args.destinationId) {
|
|
return {
|
|
success: false,
|
|
error: {
|
|
message: 'Destination ID is required',
|
|
code: 'MISSING_DESTINATION_ID'
|
|
}
|
|
};
|
|
}
|
|
|
|
try {
|
|
// Find tasks.json path if not provided
|
|
let tasksPath = args.tasksJsonPath || args.file;
|
|
if (!tasksPath) {
|
|
if (!args.projectRoot) {
|
|
return {
|
|
success: false,
|
|
error: {
|
|
message:
|
|
'Project root is required if tasksJsonPath is not provided',
|
|
code: 'MISSING_PROJECT_ROOT'
|
|
}
|
|
};
|
|
}
|
|
tasksPath = findTasksPath(args, log);
|
|
}
|
|
|
|
// Enable silent mode to prevent console output during MCP operation
|
|
enableSilentMode();
|
|
|
|
// Call the core moveTask function with file generation control
|
|
const generateFiles = args.generateFiles !== false; // Default to true
|
|
const result = await moveTask(
|
|
tasksPath,
|
|
args.sourceId,
|
|
args.destinationId,
|
|
generateFiles,
|
|
{
|
|
projectRoot: args.projectRoot,
|
|
tag: args.tag
|
|
}
|
|
);
|
|
|
|
// Restore console output
|
|
disableSilentMode();
|
|
|
|
return {
|
|
success: true,
|
|
data: {
|
|
...result,
|
|
message: `Successfully moved task/subtask ${args.sourceId} to ${args.destinationId}`
|
|
}
|
|
};
|
|
} catch (error) {
|
|
// Restore console output in case of error
|
|
disableSilentMode();
|
|
|
|
log.error(`Failed to move task: ${error.message}`);
|
|
|
|
return {
|
|
success: false,
|
|
error: {
|
|
message: error.message,
|
|
code: 'MOVE_TASK_ERROR'
|
|
}
|
|
};
|
|
}
|
|
}
|