d63964a10e
This commit introduces several improvements and refactorings across MCP tools, core logic, and configuration.
**Major Changes:**
1. **Refactor updateSubtaskById:**
- Switched from generateTextService to generateObjectService for structured AI responses, using a Zod schema (subtaskSchema) for validation.
- Revised prompts to have the AI generate relevant content based on user request and context (parent/sibling tasks), while explicitly preventing AI from handling timestamp/tag formatting.
- Implemented **local timestamp generation (new Date().toISOString()) and formatting** (using <info added on ...> tags) within the function *after* receiving the AI response. This ensures reliable and correctly formatted details are appended.
- Corrected logic to append only the locally formatted, AI-generated content block to the existing subtask.details.
2. **Consolidate MCP Utilities:**
- Moved/consolidated the withNormalizedProjectRoot HOF into mcp-server/src/tools/utils.js.
- Updated MCP tools (like update-subtask.js) to import withNormalizedProjectRoot from the new location.
3. **Refactor Project Initialization:**
- Deleted the redundant mcp-server/src/core/direct-functions/initialize-project-direct.js file.
- Updated mcp-server/src/core/task-master-core.js to import initializeProjectDirect from its correct location (./direct-functions/initialize-project.js).
**Other Changes:**
- Updated .taskmasterconfig fallback model to claude-3-7-sonnet-20250219.
- Clarified model cost representation in the models tool description (taskmaster.mdc and mcp-server/src/tools/models.js).
90 lines
2.5 KiB
JavaScript
90 lines
2.5 KiB
JavaScript
/**
|
|
* tools/update-subtask.js
|
|
* Tool to append additional information to a specific subtask
|
|
*/
|
|
|
|
import { z } from 'zod';
|
|
import {
|
|
handleApiResult,
|
|
createErrorResponse,
|
|
withNormalizedProjectRoot
|
|
} from './utils.js';
|
|
import { updateSubtaskByIdDirect } from '../core/task-master-core.js';
|
|
import { findTasksJsonPath } from '../core/utils/path-utils.js';
|
|
|
|
/**
|
|
* Register the update-subtask tool with the MCP server
|
|
* @param {Object} server - FastMCP server instance
|
|
*/
|
|
export function registerUpdateSubtaskTool(server) {
|
|
server.addTool({
|
|
name: 'update_subtask',
|
|
description:
|
|
'Appends timestamped information to a specific subtask without replacing existing content',
|
|
parameters: z.object({
|
|
id: z
|
|
.string()
|
|
.describe(
|
|
'ID of the subtask to update in format "parentId.subtaskId" (e.g., "5.2"). Parent ID is the ID of the task that contains the subtask.'
|
|
),
|
|
prompt: z.string().describe('Information to add to the subtask'),
|
|
research: z
|
|
.boolean()
|
|
.optional()
|
|
.describe('Use Perplexity AI for research-backed updates'),
|
|
file: z.string().optional().describe('Absolute path to the tasks file'),
|
|
projectRoot: z
|
|
.string()
|
|
.describe('The directory of the project. Must be an absolute path.')
|
|
}),
|
|
execute: withNormalizedProjectRoot(async (args, { log, session }) => {
|
|
const toolName = 'update_subtask';
|
|
try {
|
|
log.info(`Updating subtask with args: ${JSON.stringify(args)}`);
|
|
|
|
let tasksJsonPath;
|
|
try {
|
|
tasksJsonPath = findTasksJsonPath(
|
|
{ projectRoot: args.projectRoot, file: args.file },
|
|
log
|
|
);
|
|
} catch (error) {
|
|
log.error(`${toolName}: Error finding tasks.json: ${error.message}`);
|
|
return createErrorResponse(
|
|
`Failed to find tasks.json: ${error.message}`
|
|
);
|
|
}
|
|
|
|
const result = await updateSubtaskByIdDirect(
|
|
{
|
|
tasksJsonPath: tasksJsonPath,
|
|
id: args.id,
|
|
prompt: args.prompt,
|
|
research: args.research,
|
|
projectRoot: args.projectRoot
|
|
},
|
|
log,
|
|
{ session }
|
|
);
|
|
|
|
if (result.success) {
|
|
log.info(`Successfully updated subtask with ID ${args.id}`);
|
|
} else {
|
|
log.error(
|
|
`Failed to update subtask: ${result.error?.message || 'Unknown error'}`
|
|
);
|
|
}
|
|
|
|
return handleApiResult(result, log, 'Error updating subtask');
|
|
} catch (error) {
|
|
log.error(
|
|
`Critical error in ${toolName} tool execute: ${error.message}`
|
|
);
|
|
return createErrorResponse(
|
|
`Internal tool error (${toolName}): ${error.message}`
|
|
);
|
|
}
|
|
})
|
|
});
|
|
}
|