@vibescope/mcp-server 0.0.1 → 0.1.0

package/src/handlers/project.test.ts

@@ -1,6 +1,4 @@
 import { describe, it, expect, vi, beforeEach } from 'vitest';
-import type { SupabaseClient } from '@supabase/supabase-js';
-import type { HandlerContext, TokenUsage } from './types.js';
 import {
 	getProjectContext,
 	getGitWorkflow,
@@ -9,114 +7,10 @@ import {
 	updateProjectReadme,
 } from './project.js';
 import { ValidationError } from '../validators.js';
+import { createMockContext } from './__test-utils__.js';
+import { mockApiClient } from './__test-setup__.js';
 
-// ============================================================================
-// Test Utilities
-// ============================================================================
-
-function createMockSupabase(overrides: {
-	selectResult?: { data: unknown; error: unknown };
-	insertResult?: { data: unknown; error: unknown };
-	updateResult?: { data: unknown; error: unknown };
-} = {}) {
-	const defaultResult = { data: null, error: null };
-	let currentOperation = 'select';
-	let insertThenSelect = false;
-
-	const mock = {
-		from: vi.fn().mockReturnThis(),
-		select: vi.fn(() => {
-			if (currentOperation === 'insert') {
-				insertThenSelect = true;
-			} else {
-				currentOperation = 'select';
-				insertThenSelect = false;
-			}
-			return mock;
-		}),
-		insert: vi.fn(() => {
-			currentOperation = 'insert';
-			insertThenSelect = false;
-			return mock;
-		}),
-		update: vi.fn(() => {
-			currentOperation = 'update';
-			insertThenSelect = false;
-			return mock;
-		}),
-		delete: vi.fn(() => {
-			currentOperation = 'delete';
-			insertThenSelect = false;
-			return mock;
-		}),
-		eq: vi.fn().mockReturnThis(),
-		neq: vi.fn().mockReturnThis(),
-		in: vi.fn().mockReturnThis(),
-		is: vi.fn().mockReturnThis(),
-		not: vi.fn().mockReturnThis(),
-		or: vi.fn().mockReturnThis(),
-		gt: vi.fn().mockReturnThis(),
-		gte: vi.fn().mockReturnThis(),
-		lte: vi.fn().mockReturnThis(),
-		lt: vi.fn().mockReturnThis(),
-		order: vi.fn().mockReturnThis(),
-		limit: vi.fn().mockReturnThis(),
-		single: vi.fn(() => {
-			if (currentOperation === 'insert' || insertThenSelect) {
-				return Promise.resolve(overrides.insertResult ?? defaultResult);
-			}
-			if (currentOperation === 'select') {
-				return Promise.resolve(overrides.selectResult ?? defaultResult);
-			}
-			if (currentOperation === 'update') {
-				return Promise.resolve(overrides.updateResult ?? defaultResult);
-			}
-			return Promise.resolve(defaultResult);
-		}),
-		maybeSingle: vi.fn(() => {
-			return Promise.resolve(overrides.selectResult ?? defaultResult);
-		}),
-		then: vi.fn((resolve: (value: unknown) => void) => {
-			if (currentOperation === 'insert' || insertThenSelect) {
-				return Promise.resolve(overrides.insertResult ?? defaultResult).then(resolve);
-			}
-			if (currentOperation === 'select') {
-				return Promise.resolve(overrides.selectResult ?? defaultResult).then(resolve);
-			}
-			if (currentOperation === 'update') {
-				return Promise.resolve(overrides.updateResult ?? defaultResult).then(resolve);
-			}
-			return Promise.resolve(defaultResult).then(resolve);
-		}),
-	};
-
-	return mock as unknown as SupabaseClient;
-}
-
-function createMockContext(
-	supabase: SupabaseClient,
-	options: { sessionId?: string | null } = {}
-): HandlerContext {
-	const defaultTokenUsage: TokenUsage = {
-		callCount: 5,
-		totalTokens: 2500,
-		byTool: {},
-	};
-
-	const sessionId = 'sessionId' in options ? options.sessionId : 'session-123';
-
-	return {
-		supabase,
-		auth: { userId: 'user-123', apiKeyId: 'api-key-123' },
-		session: {
-			instanceId: 'instance-abc',
-			currentSessionId: sessionId,
-			currentPersona: 'Wave',
-			tokenUsage: defaultTokenUsage,
-		},
-		updateSession: vi.fn(),
-	};
-}
+const VALID_UUID = '123e4567-e89b-12d3-a456-426614174000';
 
 // ============================================================================
 // getProjectContext Tests
@@ -131,26 +25,31 @@ describe('getProjectContext', () => {
 			{ id: 'proj-2', name: 'Project 2', description: 'Desc 2', status: 'active', git_url: 'https://github.com/test' },
 		];
 
-		const supabase = createMockSupabase({
-			selectResult: { data: mockProjects, error: null },
+		mockApiClient.listProjects.mockResolvedValue({
+			ok: true,
+			data: { projects: mockProjects },
 		});
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		const result = await getProjectContext({}, ctx);
 
 		expect(result.result).toHaveProperty('projects');
 		expect((result.result as { projects: unknown[] }).projects).toEqual(mockProjects);
-		expect(supabase.from).toHaveBeenCalledWith('projects');
+		expect(mockApiClient.listProjects).toHaveBeenCalled();
 	});
 
 	it('should return found: false when project not found', async () => {
-		const supabase = createMockSupabase({
-			selectResult: { data: null, error: { message: 'Not found' } },
+		mockApiClient.getProject.mockResolvedValue({
+			ok: true,
+			data: {
+				found: false,
+				message: 'Project not found. Use create_project to create one.',
+			},
 		});
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		const result = await getProjectContext(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			{ project_id: VALID_UUID },
 			ctx
 		);
 
@@ -161,13 +60,23 @@ describe('getProjectContext', () => {
 	});
 
 	it('should find project by git_url', async () => {
-		const supabase = createMockSupabase({
-			selectResult: {
-				data: { id: 'proj-1', name: 'Test Project', description: null, goal: null, status: 'active', git_url: 'https://github.com/test', agent_instructions: null, tech_stack: null },
-				error: null,
+		mockApiClient.getProject.mockResolvedValue({
+			ok: true,
+			data: {
+				found: true,
+				project: {
+					id: 'proj-1',
+					name: 'Test Project',
+					description: null,
+					goal: null,
+					status: 'active',
+					git_url: 'https://github.com/test',
+					agent_instructions: null,
+					tech_stack: null,
+				},
 			},
 		});
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		const result = await getProjectContext(
 			{ git_url: 'https://github.com/test' },
@@ -177,25 +86,49 @@ describe('getProjectContext', () => {
 		expect(result.result).toMatchObject({
 			found: true,
 		});
-		expect(supabase.eq).toHaveBeenCalledWith('git_url', 'https://github.com/test');
+		expect(mockApiClient.getProject).toHaveBeenCalledWith(
+			'by-git-url',
+			'https://github.com/test'
+		);
 	});
 
 	it('should include active_tasks in result', async () => {
-		const supabase = createMockSupabase({
-			selectResult: {
-				data: { id: 'proj-1', name: 'Test Project', description: null, goal: null, status: 'active', git_url: null, agent_instructions: null, tech_stack: null },
-				error: null,
+		mockApiClient.getProject.mockResolvedValue({
+			ok: true,
+			data: {
+				found: true,
+				project: {
+					id: 'proj-1',
+					name: 'Test Project',
+					description: null,
+					goal: null,
+					status: 'active',
+					git_url: null,
+					agent_instructions: null,
+					tech_stack: null,
+				},
+				active_tasks: [],
 			},
 		});
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		const result = await getProjectContext(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			{ project_id: VALID_UUID },
 			ctx
 		);
 
 		expect(result.result).toHaveProperty('active_tasks');
 	});
+
+	it('should throw error when API call fails', async () => {
+		mockApiClient.listProjects.mockResolvedValue({
+			ok: false,
+			error: 'Database error',
+		});
+		const ctx = createMockContext();
+
+		await expect(getProjectContext({}, ctx)).rejects.toThrow('Database error');
+	});
 });
 
 // ============================================================================
@@ -206,15 +139,13 @@ describe('getGitWorkflow', () => {
 	beforeEach(() => vi.clearAllMocks());
 
 	it('should throw error for missing project_id', async () => {
-		const supabase = createMockSupabase();
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		await expect(getGitWorkflow({}, ctx)).rejects.toThrow(ValidationError);
 	});
 
 	it('should throw error for invalid project_id UUID', async () => {
-		const supabase = createMockSupabase();
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		await expect(
 			getGitWorkflow({ project_id: 'invalid' }, ctx)
@@ -222,23 +153,26 @@ describe('getGitWorkflow', () => {
 	});
 
 	it('should return workflow instructions for github-flow', async () => {
-		const supabase = createMockSupabase({
-			selectResult: {
-				data: {
-					git_workflow: 'github-flow',
-					git_main_branch: 'main',
-					git_develop_branch: null,
-					git_auto_branch: true,
-					git_auto_tag: false,
-					git_url: 'https://github.com/test',
-				},
-				error: null,
+		mockApiClient.getGitWorkflow.mockResolvedValue({
+			ok: true,
+			data: {
+				workflow: 'github-flow',
+				main_branch: 'main',
+				develop_branch: null,
+				auto_branch: true,
+				auto_tag: false,
+				git_url: 'https://github.com/test',
+				instructions: [
+					'Create feature branch from main',
+					'Make changes and commit',
+					'Create PR to main',
+				],
 			},
 		});
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		const result = await getGitWorkflow(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			{ project_id: VALID_UUID },
 			ctx
 		);
 
@@ -251,23 +185,22 @@ describe('getGitWorkflow', () => {
 	});
 
 	it('should return workflow instructions for git-flow', async () => {
-		const supabase = createMockSupabase({
-			selectResult: {
-				data: {
-					git_workflow: 'git-flow',
-					git_main_branch: 'main',
-					git_develop_branch: 'develop',
-					git_auto_branch: false,
-					git_auto_tag: true,
-					git_url: 'https://github.com/test',
-				},
-				error: null,
+		mockApiClient.getGitWorkflow.mockResolvedValue({
+			ok: true,
+			data: {
+				workflow: 'git-flow',
+				main_branch: 'main',
+				develop_branch: 'develop',
+				auto_branch: false,
+				auto_tag: true,
+				git_url: 'https://github.com/test',
+				instructions: ['Create feature branch from develop'],
 			},
 		});
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		const result = await getGitWorkflow(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			{ project_id: VALID_UUID },
 			ctx
 		);
 
@@ -279,23 +212,22 @@ describe('getGitWorkflow', () => {
 	});
 
 	it('should return no workflow instructions when workflow is none', async () => {
-		const supabase = createMockSupabase({
-			selectResult: {
-				data: {
-					git_workflow: 'none',
-					git_main_branch: 'main',
-					git_develop_branch: null,
-					git_auto_branch: false,
-					git_auto_tag: false,
-					git_url: null,
-				},
-				error: null,
+		mockApiClient.getGitWorkflow.mockResolvedValue({
+			ok: true,
+			data: {
+				workflow: 'none',
+				main_branch: 'main',
+				develop_branch: null,
+				auto_branch: false,
+				auto_tag: false,
+				git_url: null,
+				instructions: ['No git workflow configured for this project.'],
 			},
 		});
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		const result = await getGitWorkflow(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			{ project_id: VALID_UUID },
 			ctx
 		);
 
@@ -306,13 +238,14 @@ describe('getGitWorkflow', () => {
 	});
 
 	it('should throw error when project not found', async () => {
-		const supabase = createMockSupabase({
-			selectResult: { data: null, error: { message: 'Not found' } },
+		mockApiClient.getGitWorkflow.mockResolvedValue({
+			ok: false,
+			error: 'Project not found',
 		});
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		await expect(
-			getGitWorkflow({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
+			getGitWorkflow({ project_id: VALID_UUID }, ctx)
 		).rejects.toThrow('Project not found');
 	});
 });
@@ -334,10 +267,14 @@ describe('createProject', () => {
 			tech_stack: null,
 		};
 
-		const supabase = createMockSupabase({
-			insertResult: { data: mockProject, error: null },
+		mockApiClient.createProject.mockResolvedValue({
+			ok: true,
+			data: {
+				success: true,
+				project: mockProject,
+			},
 		});
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		const result = await createProject({ name: 'New Project' }, ctx);
 
@@ -345,7 +282,13 @@ describe('createProject', () => {
 			success: true,
 			project: mockProject,
 		});
-		expect(supabase.insert).toHaveBeenCalled();
+		expect(mockApiClient.createProject).toHaveBeenCalledWith({
+			name: 'New Project',
+			description: undefined,
+			goal: undefined,
+			git_url: undefined,
+			tech_stack: undefined,
+		});
 	});
 
 	it('should create project with all optional fields', async () => {
@@ -358,10 +301,14 @@ describe('createProject', () => {
 			tech_stack: ['TypeScript', 'Svelte'],
 		};
 
-		const supabase = createMockSupabase({
-			insertResult: { data: mockProject, error: null },
+		mockApiClient.createProject.mockResolvedValue({
+			ok: true,
+			data: {
+				success: true,
+				project: mockProject,
+			},
 		});
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		const result = await createProject(
 			{
@@ -381,15 +328,24 @@ describe('createProject', () => {
 	});
 
 	it('should throw error when insert fails', async () => {
-		const supabase = createMockSupabase({
-			insertResult: { data: null, error: { message: 'Insert failed' } },
+		mockApiClient.createProject.mockResolvedValue({
+			ok: false,
+			error: 'Failed to create project: Insert failed',
 		});
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		await expect(
 			createProject({ name: 'Test Project' }, ctx)
 		).rejects.toThrow('Failed to create project: Insert failed');
 	});
+
+	it('should throw error when name is missing', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			createProject({}, ctx)
+		).rejects.toThrow(ValidationError);
+	});
 });
 
 // ============================================================================
@@ -400,15 +356,13 @@ describe('updateProject', () => {
 	beforeEach(() => vi.clearAllMocks());
 
 	it('should throw error for missing project_id', async () => {
-		const supabase = createMockSupabase();
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		await expect(updateProject({}, ctx)).rejects.toThrow(ValidationError);
 	});
 
 	it('should throw error for invalid project_id UUID', async () => {
-		const supabase = createMockSupabase();
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		await expect(
 			updateProject({ project_id: 'invalid' }, ctx)
@@ -416,26 +370,29 @@ describe('updateProject', () => {
 	});
 
 	it('should throw error for invalid status', async () => {
-		const supabase = createMockSupabase();
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		await expect(
 			updateProject({
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				project_id: VALID_UUID,
 				status: 'invalid_status',
 			}, ctx)
 		).rejects.toThrow(ValidationError);
 	});
 
 	it('should update project successfully', async () => {
-		const supabase = createMockSupabase({
-			updateResult: { data: null, error: null },
+		mockApiClient.updateProject.mockResolvedValue({
+			ok: true,
+			data: {
+				success: true,
+				project_id: VALID_UUID,
+			},
 		});
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		const result = await updateProject(
 			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				project_id: VALID_UUID,
 				name: 'Updated Name',
 				status: 'active',
 			},
@@ -444,20 +401,30 @@ describe('updateProject', () => {
 
 		expect(result.result).toMatchObject({
 			success: true,
-			project_id: '123e4567-e89b-12d3-a456-426614174000',
+			project_id: VALID_UUID,
 		});
-		expect(supabase.update).toHaveBeenCalled();
+		expect(mockApiClient.updateProject).toHaveBeenCalledWith(
+			VALID_UUID,
+			{
+				name: 'Updated Name',
+				status: 'active',
+			}
+		);
 	});
 
 	it('should update git workflow settings', async () => {
-		const supabase = createMockSupabase({
-			updateResult: { data: null, error: null },
+		mockApiClient.updateProject.mockResolvedValue({
+			ok: true,
+			data: {
+				success: true,
+				project_id: VALID_UUID,
+			},
 		});
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		const result = await updateProject(
 			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				project_id: VALID_UUID,
 				git_workflow: 'github-flow',
 				git_main_branch: 'main',
 				git_auto_branch: true,
@@ -468,17 +435,26 @@ describe('updateProject', () => {
 		expect(result.result).toMatchObject({
 			success: true,
 		});
+		expect(mockApiClient.updateProject).toHaveBeenCalledWith(
+			VALID_UUID,
+			{
+				git_workflow: 'github-flow',
+				git_main_branch: 'main',
+				git_auto_branch: true,
+			}
+		);
 	});
 
 	it('should throw error when update fails', async () => {
-		const supabase = createMockSupabase({
-			updateResult: { data: null, error: { message: 'Update failed' } },
+		mockApiClient.updateProject.mockResolvedValue({
+			ok: false,
+			error: 'Failed to update project: Update failed',
 		});
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		await expect(
 			updateProject({
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				project_id: VALID_UUID,
 				name: 'Test',
 			}, ctx)
 		).rejects.toThrow('Failed to update project: Update failed');
@@ -493,8 +469,7 @@ describe('updateProjectReadme', () => {
 	beforeEach(() => vi.clearAllMocks());
 
 	it('should throw error for missing project_id', async () => {
-		const supabase = createMockSupabase();
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		await expect(
 			updateProjectReadme({ readme_content: '# README' }, ctx)
@@ -502,17 +477,15 @@ describe('updateProjectReadme', () => {
 	});
 
 	it('should throw error for missing readme_content', async () => {
-		const supabase = createMockSupabase();
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		await expect(
-			updateProjectReadme({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
+			updateProjectReadme({ project_id: VALID_UUID }, ctx)
 		).rejects.toThrow(ValidationError);
 	});
 
 	it('should throw error for invalid project_id UUID', async () => {
-		const supabase = createMockSupabase();
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		await expect(
 			updateProjectReadme({
@@ -523,14 +496,18 @@ describe('updateProjectReadme', () => {
 	});
 
 	it('should update readme successfully', async () => {
-		const supabase = createMockSupabase({
-			updateResult: { data: null, error: null },
+		mockApiClient.updateProjectReadme.mockResolvedValue({
+			ok: true,
+			data: {
+				success: true,
+				project_id: VALID_UUID,
+			},
 		});
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		const result = await updateProjectReadme(
 			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				project_id: VALID_UUID,
 				readme_content: '# My Project\n\nDescription here.',
 			},
 			ctx
@@ -539,22 +516,22 @@ describe('updateProjectReadme', () => {
 		expect(result.result).toMatchObject({
 			success: true,
 		});
-		expect(supabase.update).toHaveBeenCalledWith(
-			expect.objectContaining({
-				readme_content: '# My Project\n\nDescription here.',
-			})
+		expect(mockApiClient.updateProjectReadme).toHaveBeenCalledWith(
+			VALID_UUID,
+			'# My Project\n\nDescription here.'
 		);
 	});
 
 	it('should throw error when update fails', async () => {
-		const supabase = createMockSupabase({
-			updateResult: { data: null, error: { message: 'Update failed' } },
+		mockApiClient.updateProjectReadme.mockResolvedValue({
+			ok: false,
+			error: 'Failed to update README: Update failed',
 		});
-		const ctx = createMockContext(supabase);
+		const ctx = createMockContext();
 
 		await expect(
 			updateProjectReadme({
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				project_id: VALID_UUID,
 				readme_content: '# README',
 			}, ctx)
 		).rejects.toThrow('Failed to update README: Update failed');