@vibescope/mcp-server 0.0.1

package/src/handlers/project.test.ts

@@ -0,0 +1,562 @@
+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,
+	createProject,
+	updateProject,
+	updateProjectReadme,
+} from './project.js';
+import { ValidationError } from '../validators.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(),
+	};
+}
+
+// ============================================================================
+// getProjectContext Tests
+// ============================================================================
+
+describe('getProjectContext', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should list all projects when no project_id or git_url', async () => {
+		const mockProjects = [
+			{ id: 'proj-1', name: 'Project 1', description: 'Desc 1', status: 'active', git_url: null },
+			{ 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 },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await getProjectContext({}, ctx);
+
+		expect(result.result).toHaveProperty('projects');
+		expect((result.result as { projects: unknown[] }).projects).toEqual(mockProjects);
+		expect(supabase.from).toHaveBeenCalledWith('projects');
+	});
+
+	it('should return found: false when project not found', async () => {
+		const supabase = createMockSupabase({
+			selectResult: { data: null, error: { message: 'Not found' } },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await getProjectContext(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			found: false,
+			message: 'Project not found. Use create_project to create one.',
+		});
+	});
+
+	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,
+			},
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await getProjectContext(
+			{ git_url: 'https://github.com/test' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			found: true,
+		});
+		expect(supabase.eq).toHaveBeenCalledWith('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,
+			},
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await getProjectContext(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toHaveProperty('active_tasks');
+	});
+});
+
+// ============================================================================
+// getGitWorkflow Tests
+// ============================================================================
+
+describe('getGitWorkflow', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(getGitWorkflow({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			getGitWorkflow({ project_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	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,
+			},
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await getGitWorkflow(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			workflow: 'github-flow',
+			main_branch: 'main',
+			auto_branch: true,
+		});
+		expect((result.result as { instructions: string[] }).instructions.length).toBeGreaterThan(0);
+	});
+
+	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,
+			},
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await getGitWorkflow(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			workflow: 'git-flow',
+			main_branch: 'main',
+			develop_branch: 'develop',
+		});
+	});
+
+	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,
+			},
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await getGitWorkflow(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			workflow: 'none',
+		});
+		expect((result.result as { instructions: string[] }).instructions[0]).toContain('No git workflow configured');
+	});
+
+	it('should throw error when project not found', async () => {
+		const supabase = createMockSupabase({
+			selectResult: { data: null, error: { message: 'Not found' } },
+		});
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			getGitWorkflow({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
+		).rejects.toThrow('Project not found');
+	});
+});
+
+// ============================================================================
+// createProject Tests
+// ============================================================================
+
+describe('createProject', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should create project successfully with minimal args', async () => {
+		const mockProject = {
+			id: 'new-proj-1',
+			name: 'New Project',
+			description: null,
+			goal: null,
+			git_url: null,
+			tech_stack: null,
+		};
+
+		const supabase = createMockSupabase({
+			insertResult: { data: mockProject, error: null },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await createProject({ name: 'New Project' }, ctx);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			project: mockProject,
+		});
+		expect(supabase.insert).toHaveBeenCalled();
+	});
+
+	it('should create project with all optional fields', async () => {
+		const mockProject = {
+			id: 'new-proj-2',
+			name: 'Full Project',
+			description: 'A test project',
+			goal: 'Complete the test',
+			git_url: 'https://github.com/test/repo',
+			tech_stack: ['TypeScript', 'Svelte'],
+		};
+
+		const supabase = createMockSupabase({
+			insertResult: { data: mockProject, error: null },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await createProject(
+			{
+				name: 'Full Project',
+				description: 'A test project',
+				goal: 'Complete the test',
+				git_url: 'https://github.com/test/repo',
+				tech_stack: ['TypeScript', 'Svelte'],
+			},
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			project: mockProject,
+		});
+	});
+
+	it('should throw error when insert fails', async () => {
+		const supabase = createMockSupabase({
+			insertResult: { data: null, error: { message: 'Insert failed' } },
+		});
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			createProject({ name: 'Test Project' }, ctx)
+		).rejects.toThrow('Failed to create project: Insert failed');
+	});
+});
+
+// ============================================================================
+// updateProject Tests
+// ============================================================================
+
+describe('updateProject', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(updateProject({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			updateProject({ project_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid status', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			updateProject({
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				status: 'invalid_status',
+			}, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should update project successfully', async () => {
+		const supabase = createMockSupabase({
+			updateResult: { data: null, error: null },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await updateProject(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				name: 'Updated Name',
+				status: 'active',
+			},
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			project_id: '123e4567-e89b-12d3-a456-426614174000',
+		});
+		expect(supabase.update).toHaveBeenCalled();
+	});
+
+	it('should update git workflow settings', async () => {
+		const supabase = createMockSupabase({
+			updateResult: { data: null, error: null },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await updateProject(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				git_workflow: 'github-flow',
+				git_main_branch: 'main',
+				git_auto_branch: true,
+			},
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+		});
+	});
+
+	it('should throw error when update fails', async () => {
+		const supabase = createMockSupabase({
+			updateResult: { data: null, error: { message: 'Update failed' } },
+		});
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			updateProject({
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				name: 'Test',
+			}, ctx)
+		).rejects.toThrow('Failed to update project: Update failed');
+	});
+});
+
+// ============================================================================
+// updateProjectReadme Tests
+// ============================================================================
+
+describe('updateProjectReadme', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			updateProjectReadme({ readme_content: '# README' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing readme_content', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			updateProjectReadme({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			updateProjectReadme({
+				project_id: 'invalid',
+				readme_content: '# README',
+			}, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should update readme successfully', async () => {
+		const supabase = createMockSupabase({
+			updateResult: { data: null, error: null },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await updateProjectReadme(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				readme_content: '# My Project\n\nDescription here.',
+			},
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+		});
+		expect(supabase.update).toHaveBeenCalledWith(
+			expect.objectContaining({
+				readme_content: '# My Project\n\nDescription here.',
+			})
+		);
+	});
+
+	it('should throw error when update fails', async () => {
+		const supabase = createMockSupabase({
+			updateResult: { data: null, error: { message: 'Update failed' } },
+		});
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			updateProjectReadme({
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				readme_content: '# README',
+			}, ctx)
+		).rejects.toThrow('Failed to update README: Update failed');
+	});
+});