@vibescope/mcp-server 0.2.1 → 0.2.3

package/src/handlers/bodies-of-work.ts

@@ -142,7 +142,7 @@ export const createBodyOfWork: Handler = async (args, ctx) => {
 	});
 
 	if (!response.ok) {
-		throw new Error(`Failed to create body of work: ${response.error}`);
+		return { result: { error: response.error || 'Failed to create body of work' }, isError: true };
 	}
 
 	return {
@@ -186,7 +186,7 @@ export const updateBodyOfWork: Handler = async (args, ctx) => {
 	});
 
 	if (!response.ok) {
-		throw new Error(`Failed to update body of work: ${response.error}`);
+		return { result: { error: response.error || 'Failed to update body of work' }, isError: true };
 	}
 
 	return { result: { success: true, body_of_work_id } };
@@ -226,7 +226,7 @@ export const getBodyOfWork: Handler = async (args, ctx) => {
 	}>('get_body_of_work', { body_of_work_id, summary_only });
 
 	if (!response.ok) {
-		throw new Error(`Failed to get body of work: ${response.error}`);
+		return { result: { error: response.error || 'Failed to get body of work' }, isError: true };
 	}
 
 	return { result: response.data };
@@ -261,7 +261,7 @@ export const getBodiesOfWork: Handler = async (args, ctx) => {
 	});
 
 	if (!response.ok) {
-		throw new Error(`Failed to fetch bodies of work: ${response.error}`);
+		return { result: { error: response.error || 'Failed to fetch bodies of work' }, isError: true };
 	}
 
 	return { result: response.data };
@@ -277,7 +277,7 @@ export const deleteBodyOfWork: Handler = async (args, ctx) => {
 	});
 
 	if (!response.ok) {
-		throw new Error(`Failed to delete body of work: ${response.error}`);
+		return { result: { error: response.error || 'Failed to delete body of work' }, isError: true };
 	}
 
 	return { result: { success: true, message: 'Body of work deleted. Tasks are preserved.' } };
@@ -302,7 +302,7 @@ export const addTaskToBodyOfWork: Handler = async (args, ctx) => {
 	});
 
 	if (!response.ok) {
-		throw new Error(`Failed to add task to body of work: ${response.error}`);
+		return { result: { error: response.error || 'Failed to add task to body of work' }, isError: true };
 	}
 
 	return { result: response.data };
@@ -320,7 +320,7 @@ export const removeTaskFromBodyOfWork: Handler = async (args, ctx) => {
 	}>('remove_task_from_body_of_work', { task_id });
 
 	if (!response.ok) {
-		throw new Error(`Failed to remove task from body of work: ${response.error}`);
+		return { result: { error: response.error || 'Failed to remove task from body of work' }, isError: true };
 	}
 
 	return { result: response.data };
@@ -340,7 +340,7 @@ export const activateBodyOfWork: Handler = async (args, ctx) => {
 	}>('activate_body_of_work', { body_of_work_id });
 
 	if (!response.ok) {
-		throw new Error(`Failed to activate body of work: ${response.error}`);
+		return { result: { error: response.error || 'Failed to activate body of work' }, isError: true };
 	}
 
 	return { result: response.data };
@@ -350,7 +350,7 @@ export const addTaskDependency: Handler = async (args, ctx) => {
 	const { body_of_work_id, task_id, depends_on_task_id } = parseArgs(args, addTaskDependencySchema);
 
 	if (task_id === depends_on_task_id) {
-		throw new Error('A task cannot depend on itself');
+		return { result: { error: 'A task cannot depend on itself' }, isError: true };
 	}
 
 	const apiClient = getApiClient();
@@ -368,7 +368,7 @@ export const addTaskDependency: Handler = async (args, ctx) => {
 	});
 
 	if (!response.ok) {
-		throw new Error(`Failed to add task dependency: ${response.error}`);
+		return { result: { error: response.error || 'Failed to add task dependency' }, isError: true };
 	}
 
 	return { result: response.data };
@@ -389,7 +389,7 @@ export const removeTaskDependency: Handler = async (args, ctx) => {
 	});
 
 	if (!response.ok) {
-		throw new Error(`Failed to remove task dependency: ${response.error}`);
+		return { result: { error: response.error || 'Failed to remove task dependency' }, isError: true };
 	}
 
 	return { result: response.data };
@@ -399,7 +399,7 @@ export const getTaskDependencies: Handler = async (args, ctx) => {
 	const { body_of_work_id, task_id } = parseArgs(args, getTaskDependenciesSchema);
 
 	if (!body_of_work_id && !task_id) {
-		throw new Error('Either body_of_work_id or task_id is required');
+		return { result: { error: 'Either body_of_work_id or task_id is required' }, isError: true };
 	}
 
 	const apiClient = getApiClient();
@@ -417,7 +417,7 @@ export const getTaskDependencies: Handler = async (args, ctx) => {
 	});
 
 	if (!response.ok) {
-		throw new Error(`Failed to fetch task dependencies: ${response.error}`);
+		return { result: { error: response.error || 'Failed to fetch task dependencies' }, isError: true };
 	}
 
 	return { result: response.data };
@@ -443,7 +443,7 @@ export const getNextBodyOfWorkTask: Handler = async (args, ctx) => {
 	}>('get_next_body_of_work_task', { body_of_work_id });
 
 	if (!response.ok) {
-		throw new Error(`Failed to get next body of work task: ${response.error}`);
+		return { result: { error: response.error || 'Failed to get next body of work task' }, isError: true };
 	}
 
 	return { result: response.data };