@vibescope/mcp-server 0.2.8 → 0.3.0

package/src/handlers/requests.test.ts

@@ -1,303 +1,303 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { getPendingRequests, acknowledgeRequest, answerQuestion } from './requests.js';
-import { ValidationError } from '../validators.js';
-import { createMockContext } from './__test-utils__.js';
-import { mockApiClient } from './__test-setup__.js';
-
-// ============================================================================
-// getPendingRequests Tests
-// ============================================================================
-
-describe('getPendingRequests', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(getPendingRequests({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid project_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			getPendingRequests({ project_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should return empty list when no requests', async () => {
-		mockApiClient.getPendingRequests.mockResolvedValue({
-			ok: true,
-			data: { requests: [], total_count: 0, has_more: false },
-		});
-		const ctx = createMockContext();
-
-		const result = await getPendingRequests(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			requests: [],
-			total_count: 0,
-			has_more: false,
-		});
-	});
-
-	it('should return pending requests with pagination info', async () => {
-		const mockRequests = [
-			{
-				id: 'r1',
-				request_type: 'task',
-				content: 'Please do this',
-				session_id: null,
-				acknowledged_at: null,
-				created_at: '2025-01-14T10:00:00Z',
-			},
-		];
-
-		mockApiClient.getPendingRequests.mockResolvedValue({
-			ok: true,
-			data: { requests: mockRequests, total_count: 5, has_more: true },
-		});
-		const ctx = createMockContext();
-
-		const result = await getPendingRequests(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			requests: mockRequests,
-			total_count: 5,
-			has_more: true,
-		});
-	});
-
-	it('should call API client with project_id and session_id', async () => {
-		mockApiClient.getPendingRequests.mockResolvedValue({
-			ok: true,
-			data: { requests: [], total_count: 0, has_more: false },
-		});
-		const ctx = createMockContext({ sessionId: 'my-session' });
-
-		await getPendingRequests(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(mockApiClient.getPendingRequests).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			'my-session',
-			50,
-			0
-		);
-	});
-
-	it('should pass limit and offset to API client', async () => {
-		mockApiClient.getPendingRequests.mockResolvedValue({
-			ok: true,
-			data: { requests: [], total_count: 100, has_more: true },
-		});
-		const ctx = createMockContext({ sessionId: 'my-session' });
-
-		await getPendingRequests(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				limit: 10,
-				offset: 20,
-			},
-			ctx
-		);
-
-		expect(mockApiClient.getPendingRequests).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			'my-session',
-			10,
-			20
-		);
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.getPendingRequests.mockResolvedValue({
-			ok: false,
-			error: 'Query failed',
-		});
-		const ctx = createMockContext();
-
-		const result = await getPendingRequests(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({
-			error: 'Query failed',
-		});
-	});
-});
-
-// ============================================================================
-// acknowledgeRequest Tests
-// ============================================================================
-
-describe('acknowledgeRequest', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing request_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(acknowledgeRequest({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid request_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			acknowledgeRequest({ request_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should acknowledge request successfully', async () => {
-		mockApiClient.acknowledgeRequest.mockResolvedValue({
-			ok: true,
-			data: { success: true },
-		});
-		const ctx = createMockContext();
-
-		const result = await acknowledgeRequest(
-			{ request_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			success: true,
-		});
-	});
-
-	it('should call API client with request_id and session_id', async () => {
-		mockApiClient.acknowledgeRequest.mockResolvedValue({
-			ok: true,
-			data: { success: true },
-		});
-		const ctx = createMockContext({ sessionId: 'my-session' });
-
-		await acknowledgeRequest(
-			{ request_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(mockApiClient.acknowledgeRequest).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			'my-session'
-		);
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.acknowledgeRequest.mockResolvedValue({
-			ok: false,
-			error: 'Update failed',
-		});
-		const ctx = createMockContext();
-
-		const result = await acknowledgeRequest({ request_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({
-			error: 'Update failed',
-		});
-	});
-});
-
-// ============================================================================
-// answerQuestion Tests
-// ============================================================================
-
-describe('answerQuestion', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing request_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			answerQuestion({ answer: 'The answer' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for missing answer', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			answerQuestion({ request_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid request_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			answerQuestion({ request_id: 'invalid', answer: 'The answer' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should answer question successfully', async () => {
-		mockApiClient.answerQuestion.mockResolvedValue({
-			ok: true,
-			data: { success: true },
-		});
-		const ctx = createMockContext();
-
-		const result = await answerQuestion(
-			{
-				request_id: '123e4567-e89b-12d3-a456-426614174000',
-				answer: 'This is the answer',
-			},
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			message: 'Question answered successfully',
-		});
-	});
-
-	it('should call API client with request_id, answer, and session_id', async () => {
-		mockApiClient.answerQuestion.mockResolvedValue({
-			ok: true,
-			data: { success: true },
-		});
-		const ctx = createMockContext({ sessionId: 'my-session' });
-
-		await answerQuestion(
-			{
-				request_id: '123e4567-e89b-12d3-a456-426614174000',
-				answer: 'The answer to the question',
-			},
-			ctx
-		);
-
-		expect(mockApiClient.answerQuestion).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			'The answer to the question',
-			'my-session'
-		);
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.answerQuestion.mockResolvedValue({
-			ok: false,
-			error: 'Update failed',
-		});
-		const ctx = createMockContext();
-
-		const result = await answerQuestion({
-			request_id: '123e4567-e89b-12d3-a456-426614174000',
-			answer: 'Answer',
-		}, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({
-			error: 'Update failed',
-		});
-	});
-});
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { getPendingRequests, acknowledgeRequest, answerQuestion } from './requests.js';
+import { ValidationError } from '../validators.js';
+import { createMockContext } from './__test-utils__.js';
+import { mockApiClient } from './__test-setup__.js';
+
+// ============================================================================
+// getPendingRequests Tests
+// ============================================================================
+
+describe('getPendingRequests', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(getPendingRequests({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			getPendingRequests({ project_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should return empty list when no requests', async () => {
+		mockApiClient.getPendingRequests.mockResolvedValue({
+			ok: true,
+			data: { requests: [], total_count: 0, has_more: false },
+		});
+		const ctx = createMockContext();
+
+		const result = await getPendingRequests(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			requests: [],
+			total_count: 0,
+			has_more: false,
+		});
+	});
+
+	it('should return pending requests with pagination info', async () => {
+		const mockRequests = [
+			{
+				id: 'r1',
+				request_type: 'task',
+				content: 'Please do this',
+				session_id: null,
+				acknowledged_at: null,
+				created_at: '2025-01-14T10:00:00Z',
+			},
+		];
+
+		mockApiClient.getPendingRequests.mockResolvedValue({
+			ok: true,
+			data: { requests: mockRequests, total_count: 5, has_more: true },
+		});
+		const ctx = createMockContext();
+
+		const result = await getPendingRequests(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			requests: mockRequests,
+			total_count: 5,
+			has_more: true,
+		});
+	});
+
+	it('should call API client with project_id and session_id', async () => {
+		mockApiClient.getPendingRequests.mockResolvedValue({
+			ok: true,
+			data: { requests: [], total_count: 0, has_more: false },
+		});
+		const ctx = createMockContext({ sessionId: 'my-session' });
+
+		await getPendingRequests(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(mockApiClient.getPendingRequests).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			'my-session',
+			50,
+			0
+		);
+	});
+
+	it('should pass limit and offset to API client', async () => {
+		mockApiClient.getPendingRequests.mockResolvedValue({
+			ok: true,
+			data: { requests: [], total_count: 100, has_more: true },
+		});
+		const ctx = createMockContext({ sessionId: 'my-session' });
+
+		await getPendingRequests(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				limit: 10,
+				offset: 20,
+			},
+			ctx
+		);
+
+		expect(mockApiClient.getPendingRequests).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			'my-session',
+			10,
+			20
+		);
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.getPendingRequests.mockResolvedValue({
+			ok: false,
+			error: 'Query failed',
+		});
+		const ctx = createMockContext();
+
+		const result = await getPendingRequests(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({
+			error: 'Query failed',
+		});
+	});
+});
+
+// ============================================================================
+// acknowledgeRequest Tests
+// ============================================================================
+
+describe('acknowledgeRequest', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing request_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(acknowledgeRequest({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid request_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			acknowledgeRequest({ request_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should acknowledge request successfully', async () => {
+		mockApiClient.acknowledgeRequest.mockResolvedValue({
+			ok: true,
+			data: { success: true },
+		});
+		const ctx = createMockContext();
+
+		const result = await acknowledgeRequest(
+			{ request_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+		});
+	});
+
+	it('should call API client with request_id and session_id', async () => {
+		mockApiClient.acknowledgeRequest.mockResolvedValue({
+			ok: true,
+			data: { success: true },
+		});
+		const ctx = createMockContext({ sessionId: 'my-session' });
+
+		await acknowledgeRequest(
+			{ request_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(mockApiClient.acknowledgeRequest).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			'my-session'
+		);
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.acknowledgeRequest.mockResolvedValue({
+			ok: false,
+			error: 'Update failed',
+		});
+		const ctx = createMockContext();
+
+		const result = await acknowledgeRequest({ request_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({
+			error: 'Update failed',
+		});
+	});
+});
+
+// ============================================================================
+// answerQuestion Tests
+// ============================================================================
+
+describe('answerQuestion', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing request_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			answerQuestion({ answer: 'The answer' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing answer', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			answerQuestion({ request_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid request_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			answerQuestion({ request_id: 'invalid', answer: 'The answer' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should answer question successfully', async () => {
+		mockApiClient.answerQuestion.mockResolvedValue({
+			ok: true,
+			data: { success: true },
+		});
+		const ctx = createMockContext();
+
+		const result = await answerQuestion(
+			{
+				request_id: '123e4567-e89b-12d3-a456-426614174000',
+				answer: 'This is the answer',
+			},
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			message: 'Question answered successfully',
+		});
+	});
+
+	it('should call API client with request_id, answer, and session_id', async () => {
+		mockApiClient.answerQuestion.mockResolvedValue({
+			ok: true,
+			data: { success: true },
+		});
+		const ctx = createMockContext({ sessionId: 'my-session' });
+
+		await answerQuestion(
+			{
+				request_id: '123e4567-e89b-12d3-a456-426614174000',
+				answer: 'The answer to the question',
+			},
+			ctx
+		);
+
+		expect(mockApiClient.answerQuestion).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			'The answer to the question',
+			'my-session'
+		);
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.answerQuestion.mockResolvedValue({
+			ok: false,
+			error: 'Update failed',
+		});
+		const ctx = createMockContext();
+
+		const result = await answerQuestion({
+			request_id: '123e4567-e89b-12d3-a456-426614174000',
+			answer: 'Answer',
+		}, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({
+			error: 'Update failed',
+		});
+	});
+});