Merge branch 'next' of github.com:eyaltoledano/claude-task-master into v017-adds

mcp-server/src/tools/utils.js

@@ -304,6 +304,7 @@ function getProjectRootFromSession(session, log) {
  * @param {string} [projectRoot] - Optional project root for tag information
  * @returns {Object} - Standardized MCP response object
  */
+async function handleApiResult(
 async function handleApiResult(
 	result,
 	log,
@@ -329,11 +330,14 @@ async function handleApiResult(
 		: result.data;
 
 	log.info('Successfully completed operation');
+	log.info('Successfully completed operation');
 
 	// Create the response payload including version info and tag info
 	const responsePayload = {
 		data: processedData,
 		version: versionInfo
+		data: processedData,
+		version: versionInfo
 	};
 
 	// Add tag information if available
@@ -436,6 +440,8 @@ function executeTaskMasterCommand(
  * @param {Object} options.log - The logger instance.
  * @returns {Promise<Object>} - An object containing the result.
  *                              Format: { success: boolean, data?: any, error?: { code: string, message: string } }
+ * @returns {Promise<Object>} - An object containing the result.
+ *                              Format: { success: boolean, data?: any, error?: { code: string, message: string } }
  */
 async function getCachedOrExecute({ cacheKey, actionFn, log }) {
 	// Check cache first
@@ -444,6 +450,7 @@ async function getCachedOrExecute({ cacheKey, actionFn, log }) {
 	if (cachedResult !== undefined) {
 		log.info(`Cache hit for key: ${cacheKey}`);
 		return cachedResult;
+		return cachedResult;
 	}
 
 	log.info(`Cache miss for key: ${cacheKey}. Executing action function.`);
@@ -451,10 +458,12 @@ async function getCachedOrExecute({ cacheKey, actionFn, log }) {
 	// Execute the action function if cache missed
 	const result = await actionFn();
 
+	// If the action was successful, cache the result
 	// If the action was successful, cache the result
 	if (result.success && result.data !== undefined) {
 		log.info(`Action successful. Caching result for key: ${cacheKey}`);
 		contextManager.setCachedData(cacheKey, result);
+		contextManager.setCachedData(cacheKey, result);
 	} else if (!result.success) {
 		log.warn(
 			`Action failed for cache key ${cacheKey}. Result not cached. Error: ${result.error?.message}`
@@ -466,6 +475,7 @@ async function getCachedOrExecute({ cacheKey, actionFn, log }) {
 	}
 
 	return result;
+	return result;
 }
 
 /**