@vibescope/mcp-server 0.0.1

package/src/handlers/tasks.test.ts

@@ -0,0 +1,602 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import type { SupabaseClient } from '@supabase/supabase-js';
+import type { HandlerContext } from './types.js';
+import {
+	getTasks,
+	addTask,
+	updateTask,
+	completeTask,
+	deleteTask,
+	addTaskReference,
+	removeTaskReference,
+	getProjectGitConfig,
+} from './tasks.js';
+import { ValidationError } from '../validators.js';
+
+// ============================================================================
+// Test Utilities
+// ============================================================================
+
+/**
+ * Creates a mock Supabase client with chainable methods
+ */
+function createMockSupabase(overrides: {
+	selectResult?: { data: unknown; error: unknown };
+	insertResult?: { data: unknown; error: unknown };
+	updateResult?: { data: unknown; error: unknown };
+	deleteResult?: { data: unknown; error: unknown };
+} = {}) {
+	const defaultResult = { data: null, error: null };
+
+	// Track both the operation type AND if insert has been followed by select
+	let currentOperation = 'select';
+	let insertThenSelect = false;
+
+	const mock = {
+		from: vi.fn().mockReturnThis(),
+		select: vi.fn(() => {
+			// If we just did an insert and now calling select, it's insert().select() chain
+			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(),
+		lt: vi.fn().mockReturnThis(),
+		order: vi.fn().mockReturnThis(),
+		limit: vi.fn().mockReturnThis(),
+		single: vi.fn(() => {
+			// Handle insert().select().single() pattern
+			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);
+			}
+			if (currentOperation === 'delete') {
+				return Promise.resolve(overrides.deleteResult ?? defaultResult).then(resolve);
+			}
+			return Promise.resolve(defaultResult).then(resolve);
+		}),
+	};
+
+	return mock as unknown as SupabaseClient;
+}
+
+/**
+ * Creates a mock handler context
+ */
+function createMockContext(supabase: SupabaseClient, sessionId: string | null = 'session-123'): HandlerContext {
+	return {
+		supabase,
+		auth: {
+			userId: 'user-123',
+			apiKeyId: 'api-key-123',
+		},
+		session: {
+			currentSessionId: sessionId,
+		},
+	};
+}
+
+// ============================================================================
+// getTasks Tests
+// ============================================================================
+
+describe('getTasks', () => {
+	beforeEach(() => {
+		vi.clearAllMocks();
+	});
+
+	it('should return tasks for a valid project', async () => {
+		const mockTasks = [
+			{ id: 'task-1', title: 'Task 1', status: 'pending', priority: 1 },
+			{ id: 'task-2', title: 'Task 2', status: 'in_progress', priority: 2 },
+		];
+
+		const supabase = createMockSupabase({
+			selectResult: { data: mockTasks, error: null },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await getTasks(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result.tasks).toEqual(mockTasks);
+		expect(supabase.from).toHaveBeenCalledWith('tasks');
+	});
+
+	it('should throw error for missing project_id', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(getTasks({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(getTasks({ project_id: 'not-a-uuid' }, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should filter by status when provided', async () => {
+		const mockTasks = [{ id: 'task-1', title: 'Task 1', status: 'pending', priority: 1 }];
+
+		const supabase = createMockSupabase({
+			selectResult: { data: mockTasks, error: null },
+		});
+		const ctx = createMockContext(supabase);
+
+		await getTasks(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000', status: 'pending' },
+			ctx
+		);
+
+		expect(supabase.eq).toHaveBeenCalledWith('status', 'pending');
+	});
+
+	it('should throw error for invalid status', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			getTasks({ project_id: '123e4567-e89b-12d3-a456-426614174000', status: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should handle database errors', async () => {
+		const supabase = createMockSupabase({
+			selectResult: { data: null, error: { message: 'Database error' } },
+		});
+		const ctx = createMockContext(supabase);
+
+		// The handler uses .then() pattern, so we need to adjust the mock
+		vi.mocked(supabase.from('tasks').select).mockReturnValue({
+			...supabase,
+			then: (resolve: (val: unknown) => void) =>
+				Promise.resolve({ data: null, error: { message: 'Database error' } }).then(resolve),
+		} as unknown as ReturnType<SupabaseClient['from']>);
+
+		await expect(
+			getTasks({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
+		).rejects.toThrow('Failed to fetch tasks');
+	});
+});
+
+// ============================================================================
+// addTask Tests
+// ============================================================================
+
+describe('addTask', () => {
+	beforeEach(() => {
+		vi.clearAllMocks();
+	});
+
+	it('should add a task successfully', async () => {
+		const supabase = createMockSupabase({
+			insertResult: { data: { id: 'new-task-123', blocking: false }, error: null },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await addTask(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				title: 'New Task',
+				description: 'Task description',
+				priority: 2,
+			},
+			ctx
+		);
+
+		expect(result.result.success).toBe(true);
+		expect(result.result.task_id).toBe('new-task-123');
+		expect(result.result.title).toBe('New Task');
+		expect(supabase.from).toHaveBeenCalledWith('tasks');
+		expect(supabase.insert).toHaveBeenCalled();
+	});
+
+	it('should throw error for missing project_id', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(addTask({ title: 'Test' }, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing title', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			addTask({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid priority', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			addTask({
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				title: 'Test',
+				priority: 10, // Invalid: should be 1-5
+			}, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid estimated_minutes', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			addTask({
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				title: 'Test',
+				estimated_minutes: 0, // Invalid: must be positive
+			}, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should include blocking message for blocking tasks', async () => {
+		const supabase = createMockSupabase({
+			insertResult: { data: { id: 'blocking-task', blocking: true }, error: null },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await addTask(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				title: 'Blocking Task',
+				blocking: true,
+			},
+			ctx
+		);
+
+		expect(result.result.blocking).toBe(true);
+		expect(result.result.message).toContain('BLOCKING TASK');
+	});
+
+	it('should use default priority when not provided', async () => {
+		const supabase = createMockSupabase({
+			insertResult: { data: { id: 'task-1', blocking: false }, error: null },
+		});
+		const ctx = createMockContext(supabase);
+
+		await addTask(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				title: 'Test',
+			},
+			ctx
+		);
+
+		// Check that insert was called with priority: 3 (default)
+		expect(supabase.insert).toHaveBeenCalledWith(
+			expect.objectContaining({ priority: 3 })
+		);
+	});
+});
+
+// ============================================================================
+// deleteTask Tests
+// ============================================================================
+
+describe('deleteTask', () => {
+	beforeEach(() => {
+		vi.clearAllMocks();
+	});
+
+	it('should delete a task successfully', async () => {
+		const supabase = createMockSupabase({
+			deleteResult: { data: null, error: null },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await deleteTask(
+			{ task_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result.success).toBe(true);
+		expect(result.result.deleted_id).toBe('123e4567-e89b-12d3-a456-426614174000');
+		expect(supabase.from).toHaveBeenCalledWith('tasks');
+		expect(supabase.delete).toHaveBeenCalled();
+	});
+
+	it('should throw error for missing task_id', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(deleteTask({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid task_id UUID', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(deleteTask({ task_id: 'invalid' }, ctx)).rejects.toThrow(ValidationError);
+	});
+});
+
+// ============================================================================
+// addTaskReference Tests
+// ============================================================================
+
+describe('addTaskReference', () => {
+	beforeEach(() => {
+		vi.clearAllMocks();
+	});
+
+	it('should add a reference successfully', async () => {
+		const supabase = createMockSupabase({
+			selectResult: { data: { references: [] }, error: null },
+			updateResult: { data: null, error: null },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await addTaskReference(
+			{
+				task_id: '123e4567-e89b-12d3-a456-426614174000',
+				url: 'https://github.com/user/repo/pull/123',
+				label: 'PR #123',
+			},
+			ctx
+		);
+
+		expect(result.result.success).toBe(true);
+		expect(result.result.reference).toEqual({
+			url: 'https://github.com/user/repo/pull/123',
+			label: 'PR #123',
+		});
+		expect(result.result.total_references).toBe(1);
+	});
+
+	it('should throw error for missing task_id', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			addTaskReference({ url: 'https://example.com' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing url', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			addTaskReference({ task_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should reject duplicate URL', async () => {
+		const existingRef = { url: 'https://example.com', label: 'Existing' };
+		const supabase = createMockSupabase({
+			selectResult: { data: { references: [existingRef] }, error: null },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await addTaskReference(
+			{
+				task_id: '123e4567-e89b-12d3-a456-426614174000',
+				url: 'https://example.com',
+			},
+			ctx
+		);
+
+		expect(result.result.success).toBe(false);
+		expect(result.result.error).toContain('already exists');
+	});
+});
+
+// ============================================================================
+// removeTaskReference Tests
+// ============================================================================
+
+describe('removeTaskReference', () => {
+	beforeEach(() => {
+		vi.clearAllMocks();
+	});
+
+	it('should remove a reference successfully', async () => {
+		const existingRef = { url: 'https://example.com', label: 'Test' };
+		const supabase = createMockSupabase({
+			selectResult: { data: { references: [existingRef] }, error: null },
+			updateResult: { data: null, error: null },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await removeTaskReference(
+			{
+				task_id: '123e4567-e89b-12d3-a456-426614174000',
+				url: 'https://example.com',
+			},
+			ctx
+		);
+
+		expect(result.result.success).toBe(true);
+		expect(result.result.remaining_references).toBe(0);
+	});
+
+	it('should return error when URL not found', async () => {
+		const supabase = createMockSupabase({
+			selectResult: { data: { references: [] }, error: null },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await removeTaskReference(
+			{
+				task_id: '123e4567-e89b-12d3-a456-426614174000',
+				url: 'https://nonexistent.com',
+			},
+			ctx
+		);
+
+		expect(result.result.success).toBe(false);
+		expect(result.result.error).toContain('not found');
+	});
+});
+
+// ============================================================================
+// getProjectGitConfig Tests
+// ============================================================================
+
+describe('getProjectGitConfig', () => {
+	it('should return git config for a project', async () => {
+		const mockConfig = {
+			git_workflow: 'github-flow',
+			git_main_branch: 'main',
+			git_develop_branch: null,
+			git_auto_branch: true,
+		};
+
+		const supabase = createMockSupabase({
+			selectResult: { data: mockConfig, error: null },
+		});
+
+		const result = await getProjectGitConfig(supabase, '123e4567-e89b-12d3-a456-426614174000');
+
+		expect(result).toEqual(mockConfig);
+		expect(supabase.from).toHaveBeenCalledWith('projects');
+	});
+
+	it('should return null when project not found', async () => {
+		const supabase = createMockSupabase({
+			selectResult: { data: null, error: { message: 'Not found' } },
+		});
+
+		const result = await getProjectGitConfig(supabase, '123e4567-e89b-12d3-a456-426614174000');
+
+		expect(result).toBeNull();
+	});
+});
+
+// ============================================================================
+// completeTask Tests
+// ============================================================================
+
+describe('completeTask', () => {
+	beforeEach(() => {
+		vi.clearAllMocks();
+	});
+
+	it('should throw error for missing task_id', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(completeTask({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid task_id UUID', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(completeTask({ task_id: 'invalid' }, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error when task not found', async () => {
+		const supabase = createMockSupabase({
+			selectResult: { data: null, error: { message: 'Not found' } },
+		});
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			completeTask({ task_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
+		).rejects.toThrow('Task not found');
+	});
+});
+
+// ============================================================================
+// updateTask Tests
+// ============================================================================
+
+describe('updateTask', () => {
+	beforeEach(() => {
+		vi.clearAllMocks();
+	});
+
+	it('should throw error for missing task_id', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(updateTask({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid task_id UUID', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(updateTask({ task_id: 'invalid' }, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid status', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			updateTask({ task_id: '123e4567-e89b-12d3-a456-426614174000', status: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid priority', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			updateTask({ task_id: '123e4567-e89b-12d3-a456-426614174000', priority: 0 }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid progress_percentage', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			updateTask({ task_id: '123e4567-e89b-12d3-a456-426614174000', progress_percentage: 150 }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+});