@vibescope/mcp-server 0.2.9 → 0.3.0

package/src/handlers/ideas.test.ts

@@ -1,644 +1,644 @@
-import { describe, it, expect, beforeEach } from 'vitest';
-import {
-	addIdea,
-	updateIdea,
-	getIdeas,
-	deleteIdea,
-	convertIdeaToTask,
-} from './ideas.js';
-import { ValidationError } from '../validators.js';
-import { createMockContext } from './__test-utils__.js';
-import { mockApiClient } from './__test-setup__.js';
-
-// ============================================================================
-// addIdea Tests
-// ============================================================================
-
-describe('addIdea', () => {
-	beforeEach(() => {});
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			addIdea({ title: 'New Feature' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid project_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			addIdea({ project_id: 'invalid', title: 'New Feature' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for missing title', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			addIdea({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should add idea successfully', async () => {
-		mockApiClient.addIdea.mockResolvedValue({
-			ok: true,
-			data: { idea_id: 'idea-1' },
-		});
-		const ctx = createMockContext();
-
-		const result = await addIdea(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				title: 'Dark Mode Support',
-			},
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			idea_id: 'idea-1',
-			title: 'Dark Mode Support',
-		});
-	});
-
-	it('should call API client with default status "raw"', async () => {
-		mockApiClient.addIdea.mockResolvedValue({
-			ok: true,
-			data: { idea_id: 'idea-1' },
-		});
-		const ctx = createMockContext();
-
-		await addIdea(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				title: 'New Feature',
-			},
-			ctx
-		);
-
-		expect(mockApiClient.addIdea).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			{
-				title: 'New Feature',
-				description: undefined,
-				status: undefined, // Default status applied server-side
-			},
-			'session-123'
-		);
-	});
-
-	it('should call API client with custom status', async () => {
-		mockApiClient.addIdea.mockResolvedValue({
-			ok: true,
-			data: { idea_id: 'idea-1' },
-		});
-		const ctx = createMockContext();
-
-		await addIdea(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				title: 'New Feature',
-				status: 'exploring',
-			},
-			ctx
-		);
-
-		expect(mockApiClient.addIdea).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			{
-				title: 'New Feature',
-				description: undefined,
-				status: 'exploring',
-			},
-			'session-123'
-		);
-	});
-
-	it('should include session_id in API call', async () => {
-		mockApiClient.addIdea.mockResolvedValue({
-			ok: true,
-			data: { idea_id: 'idea-1' },
-		});
-		const ctx = createMockContext({ sessionId: 'my-session' });
-
-		await addIdea(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				title: 'New Feature',
-			},
-			ctx
-		);
-
-		expect(mockApiClient.addIdea).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			expect.any(Object),
-			'my-session'
-		);
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.addIdea.mockResolvedValue({
-			ok: false,
-			error: 'Insert failed',
-		});
-		const ctx = createMockContext();
-
-		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' });
-	});
-});
-
-// ============================================================================
-// updateIdea Tests
-// ============================================================================
-
-describe('updateIdea', () => {
-	beforeEach(() => {});
-
-	it('should throw error for missing idea_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(updateIdea({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid idea_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			updateIdea({ idea_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should update idea title', async () => {
-		mockApiClient.updateIdea.mockResolvedValue({
-			ok: true,
-			data: { idea: { id: 'idea-1', title: 'Updated Title' } },
-		});
-		const ctx = createMockContext();
-
-		const result = await updateIdea(
-			{
-				idea_id: '123e4567-e89b-12d3-a456-426614174000',
-				title: 'Updated Title',
-			},
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			idea_id: '123e4567-e89b-12d3-a456-426614174000',
-		});
-	});
-
-	it('should call API client with status update', async () => {
-		mockApiClient.updateIdea.mockResolvedValue({
-			ok: true,
-			data: { idea: { id: 'idea-1', status: 'planned' } },
-		});
-		const ctx = createMockContext();
-
-		await updateIdea(
-			{
-				idea_id: '123e4567-e89b-12d3-a456-426614174000',
-				status: 'planned',
-			},
-			ctx
-		);
-
-		expect(mockApiClient.updateIdea).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			{
-				title: undefined,
-				description: undefined,
-				status: 'planned',
-				doc_url: undefined,
-			}
-		);
-	});
-
-	it('should update doc_url', async () => {
-		mockApiClient.updateIdea.mockResolvedValue({
-			ok: true,
-			data: { idea: { id: 'idea-1' } },
-		});
-		const ctx = createMockContext();
-
-		await updateIdea(
-			{
-				idea_id: '123e4567-e89b-12d3-a456-426614174000',
-				doc_url: 'https://docs.example.com/feature',
-			},
-			ctx
-		);
-
-		expect(mockApiClient.updateIdea).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			{
-				title: undefined,
-				description: undefined,
-				status: undefined,
-				doc_url: 'https://docs.example.com/feature',
-			}
-		);
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.updateIdea.mockResolvedValue({
-			ok: false,
-			error: 'Idea not found',
-		});
-		const ctx = createMockContext();
-
-		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' });
-	});
-});
-
-// ============================================================================
-// getIdeas Tests
-// ============================================================================
-
-describe('getIdeas', () => {
-	beforeEach(() => {});
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(getIdeas({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid project_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			getIdeas({ project_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should return empty list when no ideas', async () => {
-		mockApiClient.getIdeas.mockResolvedValue({
-			ok: true,
-			data: { ideas: [] },
-		});
-		const ctx = createMockContext();
-
-		const result = await getIdeas(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({ ideas: [] });
-	});
-
-	it('should return ideas list', async () => {
-		const mockIdeas = [
-			{ id: 'i1', title: 'Idea 1', description: null, status: 'raw', doc_url: null },
-			{ id: 'i2', title: 'Idea 2', description: 'Some desc', status: 'exploring', doc_url: null },
-		];
-
-		mockApiClient.getIdeas.mockResolvedValue({
-			ok: true,
-			data: { ideas: mockIdeas },
-		});
-		const ctx = createMockContext();
-
-		const result = await getIdeas(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect((result.result as { ideas: unknown[] }).ideas).toHaveLength(2);
-	});
-
-	it('should pass status filter to API', async () => {
-		mockApiClient.getIdeas.mockResolvedValue({
-			ok: true,
-			data: { ideas: [] },
-		});
-		const ctx = createMockContext();
-
-		await getIdeas(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000', status: 'planned' },
-			ctx
-		);
-
-		expect(mockApiClient.getIdeas).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			{
-				status: 'planned',
-				limit: 10,
-				offset: 0,
-				search_query: undefined,
-			}
-		);
-	});
-
-	it('should pass limit and offset to API', async () => {
-		mockApiClient.getIdeas.mockResolvedValue({
-			ok: true,
-			data: { ideas: [] },
-		});
-		const ctx = createMockContext();
-
-		await getIdeas(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000', limit: 10, offset: 5 },
-			ctx
-		);
-
-		expect(mockApiClient.getIdeas).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			{
-				status: undefined,
-				limit: 10,
-				offset: 5,
-				search_query: undefined,
-			}
-		);
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.getIdeas.mockResolvedValue({
-			ok: false,
-			error: 'Query failed',
-		});
-		const ctx = createMockContext();
-
-		const result = await getIdeas({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Query failed' });
-	});
-});
-
-// ============================================================================
-// deleteIdea Tests
-// ============================================================================
-
-describe('deleteIdea', () => {
-	beforeEach(() => {});
-
-	it('should throw error for missing idea_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(deleteIdea({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid idea_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			deleteIdea({ idea_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should delete idea successfully', async () => {
-		mockApiClient.deleteIdea.mockResolvedValue({
-			ok: true,
-			data: { success: true },
-		});
-		const ctx = createMockContext();
-
-		const result = await deleteIdea(
-			{ idea_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({ success: true });
-	});
-
-	it('should call API client deleteIdea', async () => {
-		mockApiClient.deleteIdea.mockResolvedValue({
-			ok: true,
-			data: { success: true },
-		});
-		const ctx = createMockContext();
-
-		await deleteIdea(
-			{ idea_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(mockApiClient.deleteIdea).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000'
-		);
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.deleteIdea.mockResolvedValue({
-			ok: false,
-			error: 'Delete failed',
-		});
-		const ctx = createMockContext();
-
-		const result = await deleteIdea({ idea_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Delete failed' });
-	});
-});
-
-// ============================================================================
-// convertIdeaToTask Tests
-// ============================================================================
-
-describe('convertIdeaToTask', () => {
-	beforeEach(() => {});
-
-	it('should throw error for missing idea_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(convertIdeaToTask({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid idea_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			convertIdeaToTask({ idea_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid priority', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			convertIdeaToTask({
-				idea_id: '123e4567-e89b-12d3-a456-426614174000',
-				priority: 10, // Invalid
-			}, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should return error when idea already converted', async () => {
-		mockApiClient.proxy.mockResolvedValue({
-			ok: true,
-			data: {
-				success: false,
-				error: 'Idea has already been converted to a task',
-				existing_task_id: 'existing-task-id',
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await convertIdeaToTask(
-			{ idea_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			success: false,
-			error: 'Idea has already been converted to a task',
-			existing_task_id: 'existing-task-id',
-		});
-	});
-
-	it('should convert idea to task successfully', async () => {
-		mockApiClient.proxy.mockResolvedValue({
-			ok: true,
-			data: {
-				success: true,
-				task_id: 'task-1',
-				task_title: 'Feature Request',
-				idea_id: 'idea-1',
-				idea_status: 'in_development',
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await convertIdeaToTask(
-			{ idea_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			task_id: 'task-1',
-			task_title: 'Feature Request',
-			idea_id: 'idea-1',
-		});
-	});
-
-	it('should use default priority of 3', async () => {
-		mockApiClient.proxy.mockResolvedValue({
-			ok: true,
-			data: { success: true, task_id: 'task-1' },
-		});
-		const ctx = createMockContext();
-
-		await convertIdeaToTask(
-			{ idea_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(mockApiClient.proxy).toHaveBeenCalledWith(
-			'convert_idea_to_task',
-			{
-				idea_id: '123e4567-e89b-12d3-a456-426614174000',
-				priority: 3,
-				estimated_minutes: undefined,
-				update_status: true,
-			}
-		);
-	});
-
-	it('should use custom priority when provided', async () => {
-		mockApiClient.proxy.mockResolvedValue({
-			ok: true,
-			data: { success: true, task_id: 'task-1' },
-		});
-		const ctx = createMockContext();
-
-		await convertIdeaToTask(
-			{
-				idea_id: '123e4567-e89b-12d3-a456-426614174000',
-				priority: 1,
-			},
-			ctx
-		);
-
-		expect(mockApiClient.proxy).toHaveBeenCalledWith(
-			'convert_idea_to_task',
-			{
-				idea_id: '123e4567-e89b-12d3-a456-426614174000',
-				priority: 1,
-				estimated_minutes: undefined,
-				update_status: true,
-			}
-		);
-	});
-
-	it('should update idea status to in_development by default', async () => {
-		mockApiClient.proxy.mockResolvedValue({
-			ok: true,
-			data: {
-				success: true,
-				task_id: 'task-1',
-				task_title: 'Feature Request',
-				idea_id: 'idea-1',
-				idea_status: 'in_development',
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await convertIdeaToTask(
-			{ idea_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect((result.result as { idea_status: string }).idea_status).toBe('in_development');
-	});
-
-	it('should not update status when update_status is false', async () => {
-		mockApiClient.proxy.mockResolvedValue({
-			ok: true,
-			data: {
-				success: true,
-				task_id: 'task-1',
-				task_title: 'Feature Request',
-				idea_id: 'idea-1',
-				idea_status: 'planned',
-			},
-		});
-		const ctx = createMockContext();
-
-		await convertIdeaToTask(
-			{
-				idea_id: '123e4567-e89b-12d3-a456-426614174000',
-				update_status: false,
-			},
-			ctx
-		);
-
-		expect(mockApiClient.proxy).toHaveBeenCalledWith(
-			'convert_idea_to_task',
-			{
-				idea_id: '123e4567-e89b-12d3-a456-426614174000',
-				priority: 3,
-				estimated_minutes: undefined,
-				update_status: false,
-			}
-		);
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.proxy.mockResolvedValue({
-			ok: false,
-			error: 'Idea not found',
-		});
-		const ctx = createMockContext();
-
-		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' });
-	});
-});
+import { describe, it, expect, beforeEach } from 'vitest';
+import {
+	addIdea,
+	updateIdea,
+	getIdeas,
+	deleteIdea,
+	convertIdeaToTask,
+} from './ideas.js';
+import { ValidationError } from '../validators.js';
+import { createMockContext } from './__test-utils__.js';
+import { mockApiClient } from './__test-setup__.js';
+
+// ============================================================================
+// addIdea Tests
+// ============================================================================
+
+describe('addIdea', () => {
+	beforeEach(() => {});
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			addIdea({ title: 'New Feature' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			addIdea({ project_id: 'invalid', title: 'New Feature' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing title', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			addIdea({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should add idea successfully', async () => {
+		mockApiClient.addIdea.mockResolvedValue({
+			ok: true,
+			data: { idea_id: 'idea-1' },
+		});
+		const ctx = createMockContext();
+
+		const result = await addIdea(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				title: 'Dark Mode Support',
+			},
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			idea_id: 'idea-1',
+			title: 'Dark Mode Support',
+		});
+	});
+
+	it('should call API client with default status "raw"', async () => {
+		mockApiClient.addIdea.mockResolvedValue({
+			ok: true,
+			data: { idea_id: 'idea-1' },
+		});
+		const ctx = createMockContext();
+
+		await addIdea(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				title: 'New Feature',
+			},
+			ctx
+		);
+
+		expect(mockApiClient.addIdea).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			{
+				title: 'New Feature',
+				description: undefined,
+				status: undefined, // Default status applied server-side
+			},
+			'session-123'
+		);
+	});
+
+	it('should call API client with custom status', async () => {
+		mockApiClient.addIdea.mockResolvedValue({
+			ok: true,
+			data: { idea_id: 'idea-1' },
+		});
+		const ctx = createMockContext();
+
+		await addIdea(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				title: 'New Feature',
+				status: 'exploring',
+			},
+			ctx
+		);
+
+		expect(mockApiClient.addIdea).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			{
+				title: 'New Feature',
+				description: undefined,
+				status: 'exploring',
+			},
+			'session-123'
+		);
+	});
+
+	it('should include session_id in API call', async () => {
+		mockApiClient.addIdea.mockResolvedValue({
+			ok: true,
+			data: { idea_id: 'idea-1' },
+		});
+		const ctx = createMockContext({ sessionId: 'my-session' });
+
+		await addIdea(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				title: 'New Feature',
+			},
+			ctx
+		);
+
+		expect(mockApiClient.addIdea).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			expect.any(Object),
+			'my-session'
+		);
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.addIdea.mockResolvedValue({
+			ok: false,
+			error: 'Insert failed',
+		});
+		const ctx = createMockContext();
+
+		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' });
+	});
+});
+
+// ============================================================================
+// updateIdea Tests
+// ============================================================================
+
+describe('updateIdea', () => {
+	beforeEach(() => {});
+
+	it('should throw error for missing idea_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(updateIdea({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid idea_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			updateIdea({ idea_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should update idea title', async () => {
+		mockApiClient.updateIdea.mockResolvedValue({
+			ok: true,
+			data: { idea: { id: 'idea-1', title: 'Updated Title' } },
+		});
+		const ctx = createMockContext();
+
+		const result = await updateIdea(
+			{
+				idea_id: '123e4567-e89b-12d3-a456-426614174000',
+				title: 'Updated Title',
+			},
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			idea_id: '123e4567-e89b-12d3-a456-426614174000',
+		});
+	});
+
+	it('should call API client with status update', async () => {
+		mockApiClient.updateIdea.mockResolvedValue({
+			ok: true,
+			data: { idea: { id: 'idea-1', status: 'planned' } },
+		});
+		const ctx = createMockContext();
+
+		await updateIdea(
+			{
+				idea_id: '123e4567-e89b-12d3-a456-426614174000',
+				status: 'planned',
+			},
+			ctx
+		);
+
+		expect(mockApiClient.updateIdea).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			{
+				title: undefined,
+				description: undefined,
+				status: 'planned',
+				doc_url: undefined,
+			}
+		);
+	});
+
+	it('should update doc_url', async () => {
+		mockApiClient.updateIdea.mockResolvedValue({
+			ok: true,
+			data: { idea: { id: 'idea-1' } },
+		});
+		const ctx = createMockContext();
+
+		await updateIdea(
+			{
+				idea_id: '123e4567-e89b-12d3-a456-426614174000',
+				doc_url: 'https://docs.example.com/feature',
+			},
+			ctx
+		);
+
+		expect(mockApiClient.updateIdea).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			{
+				title: undefined,
+				description: undefined,
+				status: undefined,
+				doc_url: 'https://docs.example.com/feature',
+			}
+		);
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.updateIdea.mockResolvedValue({
+			ok: false,
+			error: 'Idea not found',
+		});
+		const ctx = createMockContext();
+
+		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' });
+	});
+});
+
+// ============================================================================
+// getIdeas Tests
+// ============================================================================
+
+describe('getIdeas', () => {
+	beforeEach(() => {});
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(getIdeas({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			getIdeas({ project_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should return empty list when no ideas', async () => {
+		mockApiClient.getIdeas.mockResolvedValue({
+			ok: true,
+			data: { ideas: [] },
+		});
+		const ctx = createMockContext();
+
+		const result = await getIdeas(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({ ideas: [] });
+	});
+
+	it('should return ideas list', async () => {
+		const mockIdeas = [
+			{ id: 'i1', title: 'Idea 1', description: null, status: 'raw', doc_url: null },
+			{ id: 'i2', title: 'Idea 2', description: 'Some desc', status: 'exploring', doc_url: null },
+		];
+
+		mockApiClient.getIdeas.mockResolvedValue({
+			ok: true,
+			data: { ideas: mockIdeas },
+		});
+		const ctx = createMockContext();
+
+		const result = await getIdeas(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect((result.result as { ideas: unknown[] }).ideas).toHaveLength(2);
+	});
+
+	it('should pass status filter to API', async () => {
+		mockApiClient.getIdeas.mockResolvedValue({
+			ok: true,
+			data: { ideas: [] },
+		});
+		const ctx = createMockContext();
+
+		await getIdeas(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000', status: 'planned' },
+			ctx
+		);
+
+		expect(mockApiClient.getIdeas).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			{
+				status: 'planned',
+				limit: 10,
+				offset: 0,
+				search_query: undefined,
+			}
+		);
+	});
+
+	it('should pass limit and offset to API', async () => {
+		mockApiClient.getIdeas.mockResolvedValue({
+			ok: true,
+			data: { ideas: [] },
+		});
+		const ctx = createMockContext();
+
+		await getIdeas(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000', limit: 10, offset: 5 },
+			ctx
+		);
+
+		expect(mockApiClient.getIdeas).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			{
+				status: undefined,
+				limit: 10,
+				offset: 5,
+				search_query: undefined,
+			}
+		);
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.getIdeas.mockResolvedValue({
+			ok: false,
+			error: 'Query failed',
+		});
+		const ctx = createMockContext();
+
+		const result = await getIdeas({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Query failed' });
+	});
+});
+
+// ============================================================================
+// deleteIdea Tests
+// ============================================================================
+
+describe('deleteIdea', () => {
+	beforeEach(() => {});
+
+	it('should throw error for missing idea_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(deleteIdea({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid idea_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			deleteIdea({ idea_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should delete idea successfully', async () => {
+		mockApiClient.deleteIdea.mockResolvedValue({
+			ok: true,
+			data: { success: true },
+		});
+		const ctx = createMockContext();
+
+		const result = await deleteIdea(
+			{ idea_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({ success: true });
+	});
+
+	it('should call API client deleteIdea', async () => {
+		mockApiClient.deleteIdea.mockResolvedValue({
+			ok: true,
+			data: { success: true },
+		});
+		const ctx = createMockContext();
+
+		await deleteIdea(
+			{ idea_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(mockApiClient.deleteIdea).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000'
+		);
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.deleteIdea.mockResolvedValue({
+			ok: false,
+			error: 'Delete failed',
+		});
+		const ctx = createMockContext();
+
+		const result = await deleteIdea({ idea_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Delete failed' });
+	});
+});
+
+// ============================================================================
+// convertIdeaToTask Tests
+// ============================================================================
+
+describe('convertIdeaToTask', () => {
+	beforeEach(() => {});
+
+	it('should throw error for missing idea_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(convertIdeaToTask({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid idea_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			convertIdeaToTask({ idea_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid priority', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			convertIdeaToTask({
+				idea_id: '123e4567-e89b-12d3-a456-426614174000',
+				priority: 10, // Invalid
+			}, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should return error when idea already converted', async () => {
+		mockApiClient.proxy.mockResolvedValue({
+			ok: true,
+			data: {
+				success: false,
+				error: 'Idea has already been converted to a task',
+				existing_task_id: 'existing-task-id',
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await convertIdeaToTask(
+			{ idea_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: false,
+			error: 'Idea has already been converted to a task',
+			existing_task_id: 'existing-task-id',
+		});
+	});
+
+	it('should convert idea to task successfully', async () => {
+		mockApiClient.proxy.mockResolvedValue({
+			ok: true,
+			data: {
+				success: true,
+				task_id: 'task-1',
+				task_title: 'Feature Request',
+				idea_id: 'idea-1',
+				idea_status: 'in_development',
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await convertIdeaToTask(
+			{ idea_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			task_id: 'task-1',
+			task_title: 'Feature Request',
+			idea_id: 'idea-1',
+		});
+	});
+
+	it('should use default priority of 3', async () => {
+		mockApiClient.proxy.mockResolvedValue({
+			ok: true,
+			data: { success: true, task_id: 'task-1' },
+		});
+		const ctx = createMockContext();
+
+		await convertIdeaToTask(
+			{ idea_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(mockApiClient.proxy).toHaveBeenCalledWith(
+			'convert_idea_to_task',
+			{
+				idea_id: '123e4567-e89b-12d3-a456-426614174000',
+				priority: 3,
+				estimated_minutes: undefined,
+				update_status: true,
+			}
+		);
+	});
+
+	it('should use custom priority when provided', async () => {
+		mockApiClient.proxy.mockResolvedValue({
+			ok: true,
+			data: { success: true, task_id: 'task-1' },
+		});
+		const ctx = createMockContext();
+
+		await convertIdeaToTask(
+			{
+				idea_id: '123e4567-e89b-12d3-a456-426614174000',
+				priority: 1,
+			},
+			ctx
+		);
+
+		expect(mockApiClient.proxy).toHaveBeenCalledWith(
+			'convert_idea_to_task',
+			{
+				idea_id: '123e4567-e89b-12d3-a456-426614174000',
+				priority: 1,
+				estimated_minutes: undefined,
+				update_status: true,
+			}
+		);
+	});
+
+	it('should update idea status to in_development by default', async () => {
+		mockApiClient.proxy.mockResolvedValue({
+			ok: true,
+			data: {
+				success: true,
+				task_id: 'task-1',
+				task_title: 'Feature Request',
+				idea_id: 'idea-1',
+				idea_status: 'in_development',
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await convertIdeaToTask(
+			{ idea_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect((result.result as { idea_status: string }).idea_status).toBe('in_development');
+	});
+
+	it('should not update status when update_status is false', async () => {
+		mockApiClient.proxy.mockResolvedValue({
+			ok: true,
+			data: {
+				success: true,
+				task_id: 'task-1',
+				task_title: 'Feature Request',
+				idea_id: 'idea-1',
+				idea_status: 'planned',
+			},
+		});
+		const ctx = createMockContext();
+
+		await convertIdeaToTask(
+			{
+				idea_id: '123e4567-e89b-12d3-a456-426614174000',
+				update_status: false,
+			},
+			ctx
+		);
+
+		expect(mockApiClient.proxy).toHaveBeenCalledWith(
+			'convert_idea_to_task',
+			{
+				idea_id: '123e4567-e89b-12d3-a456-426614174000',
+				priority: 3,
+				estimated_minutes: undefined,
+				update_status: false,
+			}
+		);
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.proxy.mockResolvedValue({
+			ok: false,
+			error: 'Idea not found',
+		});
+		const ctx = createMockContext();
+
+		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' });
+	});
+});