@vibescope/mcp-server 0.2.1 → 0.2.2

package/src/handlers/ideas.test.ts

@@ -138,19 +138,20 @@ describe('addIdea', () => {
 		);
 	});
 
-	it('should throw error when API call fails', async () => {
+	it('should return error when API call fails', async () => {
 		mockApiClient.addIdea.mockResolvedValue({
 			ok: false,
 			error: 'Insert failed',
 		});
 		const ctx = createMockContext();
 
-		await expect(
-			addIdea({
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				title: 'New Feature',
-			}, ctx)
-		).rejects.toThrow('Failed to add idea');
+		const result = await addIdea({
+			project_id: '123e4567-e89b-12d3-a456-426614174000',
+			title: 'New Feature',
+		}, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Insert failed' });
 	});
 });
 
@@ -248,16 +249,17 @@ describe('updateIdea', () => {
 		);
 	});
 
-	it('should throw error when API call fails', async () => {
+	it('should return error when API call fails', async () => {
 		mockApiClient.updateIdea.mockResolvedValue({
 			ok: false,
 			error: 'Idea not found',
 		});
 		const ctx = createMockContext();
 
-		await expect(
-			updateIdea({ idea_id: '123e4567-e89b-12d3-a456-426614174000', title: 'New' }, ctx)
-		).rejects.toThrow('Failed to update idea');
+		const result = await updateIdea({ idea_id: '123e4567-e89b-12d3-a456-426614174000', title: 'New' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Idea not found' });
 	});
 });
 
@@ -363,16 +365,17 @@ describe('getIdeas', () => {
 		);
 	});
 
-	it('should throw error when API call fails', async () => {
+	it('should return error when API call fails', async () => {
 		mockApiClient.getIdeas.mockResolvedValue({
 			ok: false,
 			error: 'Query failed',
 		});
 		const ctx = createMockContext();
 
-		await expect(
-			getIdeas({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
-		).rejects.toThrow('Failed to fetch ideas');
+		const result = await getIdeas({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Query failed' });
 	});
 });
 
@@ -429,16 +432,17 @@ describe('deleteIdea', () => {
 		);
 	});
 
-	it('should throw error when API call fails', async () => {
+	it('should return error when API call fails', async () => {
 		mockApiClient.deleteIdea.mockResolvedValue({
 			ok: false,
 			error: 'Delete failed',
 		});
 		const ctx = createMockContext();
 
-		await expect(
-			deleteIdea({ idea_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
-		).rejects.toThrow('Failed to delete idea');
+		const result = await deleteIdea({ idea_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Delete failed' });
 	});
 });
 
@@ -625,15 +629,16 @@ describe('convertIdeaToTask', () => {
 		);
 	});
 
-	it('should throw error when API call fails', async () => {
+	it('should return error when API call fails', async () => {
 		mockApiClient.proxy.mockResolvedValue({
 			ok: false,
 			error: 'Idea not found',
 		});
 		const ctx = createMockContext();
 
-		await expect(
-			convertIdeaToTask({ idea_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
-		).rejects.toThrow('Failed to convert idea');
+		const result = await convertIdeaToTask({ idea_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Idea not found' });
 	});
 });

package/src/handlers/ideas.ts

@@ -72,7 +72,7 @@ export const addIdea: Handler = async (args, ctx) => {
 	}, session.currentSessionId || undefined);
 
 	if (!response.ok) {
-		throw new Error(`Failed to add idea: ${response.error}`);
+		return { result: { error: response.error || 'Failed to add idea' }, isError: true };
 	}
 
 	return { result: { success: true, idea_id: response.data?.idea_id, title } };
@@ -91,7 +91,7 @@ export const updateIdea: Handler = async (args, _ctx) => {
 	});
 
 	if (!response.ok) {
-		throw new Error(`Failed to update idea: ${response.error}`);
+		return { result: { error: response.error || 'Failed to update idea' }, isError: true };
 	}
 
 	return { result: { success: true, idea_id } };
@@ -110,7 +110,7 @@ export const getIdeas: Handler = async (args, _ctx) => {
 	});
 
 	if (!response.ok) {
-		throw new Error(`Failed to fetch ideas: ${response.error}`);
+		return { result: { error: response.error || 'Failed to fetch ideas' }, isError: true };
 	}
 
 	return {
@@ -128,7 +128,7 @@ export const deleteIdea: Handler = async (args, _ctx) => {
 	const response = await apiClient.deleteIdea(idea_id);
 
 	if (!response.ok) {
-		throw new Error(`Failed to delete idea: ${response.error}`);
+		return { result: { error: response.error || 'Failed to delete idea' }, isError: true };
 	}
 
 	return { result: { success: true } };
@@ -157,7 +157,7 @@ export const convertIdeaToTask: Handler = async (args, _ctx) => {
 	});
 
 	if (!response.ok) {
-		throw new Error(`Failed to convert idea: ${response.error}`);
+		return { result: { error: response.error || 'Failed to convert idea' }, isError: true };
 	}
 
 	return { result: response.data };

package/src/handlers/index.ts

@@ -27,6 +27,7 @@ export * from './git-issues.js';
 export * from './sprints.js';
 export * from './file-checkouts.js';
 export * from './roles.js';
+export * from './connectors.js';
 
 import type { HandlerRegistry } from './types.js';
 import { milestoneHandlers } from './milestones.js';
@@ -50,6 +51,7 @@ import { gitIssueHandlers } from './git-issues.js';
 import { sprintHandlers } from './sprints.js';
 import { fileCheckoutHandlers } from './file-checkouts.js';
 import { roleHandlers } from './roles.js';
+import { connectorHandlers } from './connectors.js';
 
 /**
  * Build the complete handler registry from all modules
@@ -77,5 +79,6 @@ export function buildHandlerRegistry(): HandlerRegistry {
 		...sprintHandlers,
 		...fileCheckoutHandlers,
 		...roleHandlers,
+		...connectorHandlers,
 	};
 }

package/src/handlers/milestones.test.ts

@@ -90,19 +90,20 @@ describe('addMilestone', () => {
 		);
 	});
 
-	it('should throw error when API call fails', async () => {
+	it('should return error when API call fails', async () => {
 		mockApiClient.addMilestone.mockResolvedValue({
 			ok: false,
 			error: 'Task not found',
 		});
 		const ctx = createMockContext();
 
-		await expect(
-			addMilestone({
-				task_id: '123e4567-e89b-12d3-a456-426614174000',
-				title: 'Milestone 1',
-			}, ctx)
-		).rejects.toThrow('Failed to add milestone: Task not found');
+		const result = await addMilestone({
+			task_id: '123e4567-e89b-12d3-a456-426614174000',
+			title: 'Milestone 1',
+		}, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Task not found' });
 	});
 });
 
@@ -132,7 +133,7 @@ describe('updateMilestone', () => {
 
 		await expect(
 			updateMilestone({ milestone_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
-		).rejects.toThrow('At least one field to update is required');
+		).rejects.toThrow(ValidationError);
 	});
 
 	it('should throw error for invalid status', async () => {
@@ -143,7 +144,7 @@ describe('updateMilestone', () => {
 				milestone_id: '123e4567-e89b-12d3-a456-426614174000',
 				status: 'invalid_status',
 			}, ctx)
-		).rejects.toThrow('Invalid status: "invalid_status"');
+		).rejects.toThrow(ValidationError);
 	});
 
 	it('should update title successfully', async () => {
@@ -192,19 +193,20 @@ describe('updateMilestone', () => {
 		);
 	});
 
-	it('should throw error when API call fails', async () => {
+	it('should return error when API call fails', async () => {
 		mockApiClient.updateMilestone.mockResolvedValue({
 			ok: false,
 			error: 'Milestone not found',
 		});
 		const ctx = createMockContext();
 
-		await expect(
-			updateMilestone({
-				milestone_id: '123e4567-e89b-12d3-a456-426614174000',
-				title: 'New Title',
-			}, ctx)
-		).rejects.toThrow('Failed to update milestone: Milestone not found');
+		const result = await updateMilestone({
+			milestone_id: '123e4567-e89b-12d3-a456-426614174000',
+			title: 'New Title',
+		}, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Milestone not found' });
 	});
 });
 
@@ -263,16 +265,17 @@ describe('completeMilestone', () => {
 		);
 	});
 
-	it('should throw error when API call fails', async () => {
+	it('should return error when API call fails', async () => {
 		mockApiClient.completeMilestone.mockResolvedValue({
 			ok: false,
 			error: 'Milestone not found',
 		});
 		const ctx = createMockContext();
 
-		await expect(
-			completeMilestone({ milestone_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
-		).rejects.toThrow('Failed to complete milestone: Milestone not found');
+		const result = await completeMilestone({ milestone_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Milestone not found' });
 	});
 });
 
@@ -332,16 +335,17 @@ describe('deleteMilestone', () => {
 		);
 	});
 
-	it('should throw error when API call fails', async () => {
+	it('should return error when API call fails', async () => {
 		mockApiClient.deleteMilestone.mockResolvedValue({
 			ok: false,
 			error: 'Milestone not found',
 		});
 		const ctx = createMockContext();
 
-		await expect(
-			deleteMilestone({ milestone_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
-		).rejects.toThrow('Failed to delete milestone: Milestone not found');
+		const result = await deleteMilestone({ milestone_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Milestone not found' });
 	});
 });
 
@@ -456,15 +460,16 @@ describe('getMilestones', () => {
 		);
 	});
 
-	it('should throw error when API call fails', async () => {
+	it('should return error when API call fails', async () => {
 		mockApiClient.getMilestones.mockResolvedValue({
 			ok: false,
 			error: 'Task not found',
 		});
 		const ctx = createMockContext();
 
-		await expect(
-			getMilestones({ task_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
-		).rejects.toThrow('Failed to get milestones: Task not found');
+		const result = await getMilestones({ task_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Task not found' });
 	});
 });

package/src/handlers/milestones.ts

@@ -64,7 +64,7 @@ export const addMilestone: Handler = async (args, ctx) => {
 	}, session.currentSessionId || undefined);
 
 	if (!response.ok) {
-		throw new Error(`Failed to add milestone: ${response.error}`);
+		return { result: { error: response.error || 'Failed to add milestone' }, isError: true };
 	}
 
 	return {
@@ -93,7 +93,7 @@ export const updateMilestone: Handler = async (args, _ctx) => {
 	});
 
 	if (!response.ok) {
-		throw new Error(`Failed to update milestone: ${response.error}`);
+		return { result: { error: response.error || 'Failed to update milestone' }, isError: true };
 	}
 
 	return {
@@ -112,7 +112,7 @@ export const completeMilestone: Handler = async (args, _ctx) => {
 	const response = await apiClient.completeMilestone(milestone_id);
 
 	if (!response.ok) {
-		throw new Error(`Failed to complete milestone: ${response.error}`);
+		return { result: { error: response.error || 'Failed to complete milestone' }, isError: true };
 	}
 
 	return {
@@ -131,7 +131,7 @@ export const deleteMilestone: Handler = async (args, _ctx) => {
 	const response = await apiClient.deleteMilestone(milestone_id);
 
 	if (!response.ok) {
-		throw new Error(`Failed to delete milestone: ${response.error}`);
+		return { result: { error: response.error || 'Failed to delete milestone' }, isError: true };
 	}
 
 	return {
@@ -150,7 +150,7 @@ export const getMilestones: Handler = async (args, _ctx) => {
 	const response = await apiClient.getMilestones(task_id);
 
 	if (!response.ok) {
-		throw new Error(`Failed to get milestones: ${response.error}`);
+		return { result: { error: response.error || 'Failed to get milestones' }, isError: true };
 	}
 
 	// Stats are calculated server-side now