@vibescope/mcp-server 0.4.5 → 0.4.7

package/src/handlers/decisions.test.ts

@@ -1,382 +1,382 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { logDecision, getDecisions, getDecisionsStats, deleteDecision } from './decisions.js';
-import { ValidationError } from '../validators.js';
-import { createMockContext } from './__test-utils__.js';
-import { mockApiClient } from './__test-setup__.js';
-
-// ============================================================================
-// logDecision Tests
-// ============================================================================
-
-describe('logDecision', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			logDecision({ title: 'Test', description: 'Desc' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid project_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			logDecision({ project_id: 'invalid', title: 'Test', description: 'Desc' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for missing title', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			logDecision({ project_id: '123e4567-e89b-12d3-a456-426614174000', description: 'Desc' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for missing description', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			logDecision({ project_id: '123e4567-e89b-12d3-a456-426614174000', title: 'Test' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should log decision successfully', async () => {
-		mockApiClient.logDecision.mockResolvedValue({
-			ok: true,
-			data: { decision_id: 'dec-123' },
-		});
-		const ctx = createMockContext();
-
-		const result = await logDecision(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				title: 'Use TypeScript',
-				description: 'We will use TypeScript for type safety',
-			},
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			title: 'Use TypeScript',
-			decision_id: 'dec-123',
-		});
-	});
-
-	it('should log decision with rationale and alternatives', async () => {
-		mockApiClient.logDecision.mockResolvedValue({
-			ok: true,
-			data: { decision_id: 'dec-456' },
-		});
-		const ctx = createMockContext();
-
-		await logDecision(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				title: 'Use PostgreSQL',
-				description: 'We will use PostgreSQL as our database',
-				rationale: 'Better JSON support and performance',
-				alternatives_considered: ['MySQL', 'MongoDB'],
-			},
-			ctx
-		);
-
-		expect(mockApiClient.logDecision).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			{
-				title: 'Use PostgreSQL',
-				description: 'We will use PostgreSQL as our database',
-				rationale: 'Better JSON support and performance',
-				alternatives_considered: ['MySQL', 'MongoDB'],
-			},
-			'session-123'
-		);
-	});
-
-	it('should include session_id in API call', async () => {
-		mockApiClient.logDecision.mockResolvedValue({
-			ok: true,
-			data: { decision_id: 'dec-789' },
-		});
-		const ctx = createMockContext({ sessionId: 'my-session' });
-
-		await logDecision(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				title: 'Test Decision',
-				description: 'Test description',
-			},
-			ctx
-		);
-
-		expect(mockApiClient.logDecision).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			expect.any(Object),
-			'my-session'
-		);
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.logDecision.mockResolvedValue({
-			ok: false,
-			error: 'Insert failed',
-		});
-		const ctx = createMockContext();
-
-		const result = await logDecision(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				title: 'Test',
-				description: 'Test',
-			},
-			ctx
-		);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({
-			error: 'Insert failed',
-		});
-	});
-});
-
-// ============================================================================
-// getDecisions Tests
-// ============================================================================
-
-describe('getDecisions', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(getDecisions({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid project_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			getDecisions({ project_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should return empty list when no decisions', async () => {
-		mockApiClient.getDecisions.mockResolvedValue({
-			ok: true,
-			data: { decisions: [] },
-		});
-		const ctx = createMockContext();
-
-		const result = await getDecisions(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			decisions: [],
-		});
-	});
-
-	it('should return decisions list', async () => {
-		const mockDecisions = [
-			{ id: 'd1', title: 'Decision 1', description: 'Desc 1' },
-			{ id: 'd2', title: 'Decision 2', description: 'Desc 2' },
-		];
-		mockApiClient.getDecisions.mockResolvedValue({
-			ok: true,
-			data: { decisions: mockDecisions },
-		});
-		const ctx = createMockContext();
-
-		const result = await getDecisions(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			decisions: mockDecisions,
-		});
-	});
-
-	it('should call API client getDecisions with options', async () => {
-		mockApiClient.getDecisions.mockResolvedValue({
-			ok: true,
-			data: { decisions: [] },
-		});
-		const ctx = createMockContext();
-
-		await getDecisions(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(mockApiClient.getDecisions).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			{ limit: 10, offset: 0, search_query: undefined }
-		);
-	});
-
-	it('should return error when query fails', async () => {
-		mockApiClient.getDecisions.mockResolvedValue({
-			ok: false,
-			error: 'Query failed',
-		});
-		const ctx = createMockContext();
-
-		const result = await getDecisions(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({
-			error: 'Query failed',
-		});
-	});
-});
-
-// ============================================================================
-// deleteDecision Tests
-// ============================================================================
-
-describe('deleteDecision', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing decision_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(deleteDecision({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid decision_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			deleteDecision({ decision_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should delete decision successfully', async () => {
-		mockApiClient.deleteDecision.mockResolvedValue({
-			ok: true,
-			data: { success: true },
-		});
-		const ctx = createMockContext();
-
-		const result = await deleteDecision(
-			{ decision_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			success: true,
-		});
-	});
-
-	it('should call API client deleteDecision', async () => {
-		mockApiClient.deleteDecision.mockResolvedValue({
-			ok: true,
-			data: { success: true },
-		});
-		const ctx = createMockContext();
-
-		await deleteDecision(
-			{ decision_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(mockApiClient.deleteDecision).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000'
-		);
-	});
-
-	it('should return error when delete fails', async () => {
-		mockApiClient.deleteDecision.mockResolvedValue({
-			ok: false,
-			error: 'Delete failed',
-		});
-		const ctx = createMockContext();
-
-		const result = await deleteDecision(
-			{ decision_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({
-			error: 'Delete failed',
-		});
-	});
-});
-
-// ============================================================================
-// getDecisionsStats Tests
-// ============================================================================
-
-describe('getDecisionsStats', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(getDecisionsStats({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid project_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			getDecisionsStats({ project_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should return decisions stats', async () => {
-		mockApiClient.getDecisionsStats.mockResolvedValue({
-			ok: true,
-			data: { total: 10 },
-		});
-		const ctx = createMockContext();
-
-		const result = await getDecisionsStats(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({ total: 10 });
-	});
-
-	it('should call API client getDecisionsStats', async () => {
-		mockApiClient.getDecisionsStats.mockResolvedValue({
-			ok: true,
-			data: { total: 0 },
-		});
-		const ctx = createMockContext();
-
-		await getDecisionsStats(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(mockApiClient.getDecisionsStats).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000'
-		);
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.getDecisionsStats.mockResolvedValue({
-			ok: false,
-			error: 'Query failed',
-		});
-		const ctx = createMockContext();
-
-		const result = await getDecisionsStats({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({
-			error: 'Query failed',
-		});
-	});
-});
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { logDecision, getDecisions, getDecisionsStats, deleteDecision } from './decisions.js';
+import { ValidationError } from '../validators.js';
+import { createMockContext } from './__test-utils__.js';
+import { mockApiClient } from './__test-setup__.js';
+
+// ============================================================================
+// logDecision Tests
+// ============================================================================
+
+describe('logDecision', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			logDecision({ title: 'Test', description: 'Desc' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			logDecision({ project_id: 'invalid', title: 'Test', description: 'Desc' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing title', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			logDecision({ project_id: '123e4567-e89b-12d3-a456-426614174000', description: 'Desc' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing description', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			logDecision({ project_id: '123e4567-e89b-12d3-a456-426614174000', title: 'Test' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should log decision successfully', async () => {
+		mockApiClient.logDecision.mockResolvedValue({
+			ok: true,
+			data: { decision_id: 'dec-123' },
+		});
+		const ctx = createMockContext();
+
+		const result = await logDecision(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				title: 'Use TypeScript',
+				description: 'We will use TypeScript for type safety',
+			},
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			title: 'Use TypeScript',
+			decision_id: 'dec-123',
+		});
+	});
+
+	it('should log decision with rationale and alternatives', async () => {
+		mockApiClient.logDecision.mockResolvedValue({
+			ok: true,
+			data: { decision_id: 'dec-456' },
+		});
+		const ctx = createMockContext();
+
+		await logDecision(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				title: 'Use PostgreSQL',
+				description: 'We will use PostgreSQL as our database',
+				rationale: 'Better JSON support and performance',
+				alternatives_considered: ['MySQL', 'MongoDB'],
+			},
+			ctx
+		);
+
+		expect(mockApiClient.logDecision).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			{
+				title: 'Use PostgreSQL',
+				description: 'We will use PostgreSQL as our database',
+				rationale: 'Better JSON support and performance',
+				alternatives_considered: ['MySQL', 'MongoDB'],
+			},
+			'session-123'
+		);
+	});
+
+	it('should include session_id in API call', async () => {
+		mockApiClient.logDecision.mockResolvedValue({
+			ok: true,
+			data: { decision_id: 'dec-789' },
+		});
+		const ctx = createMockContext({ sessionId: 'my-session' });
+
+		await logDecision(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				title: 'Test Decision',
+				description: 'Test description',
+			},
+			ctx
+		);
+
+		expect(mockApiClient.logDecision).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			expect.any(Object),
+			'my-session'
+		);
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.logDecision.mockResolvedValue({
+			ok: false,
+			error: 'Insert failed',
+		});
+		const ctx = createMockContext();
+
+		const result = await logDecision(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				title: 'Test',
+				description: 'Test',
+			},
+			ctx
+		);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({
+			error: 'Insert failed',
+		});
+	});
+});
+
+// ============================================================================
+// getDecisions Tests
+// ============================================================================
+
+describe('getDecisions', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(getDecisions({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			getDecisions({ project_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should return empty list when no decisions', async () => {
+		mockApiClient.getDecisions.mockResolvedValue({
+			ok: true,
+			data: { decisions: [] },
+		});
+		const ctx = createMockContext();
+
+		const result = await getDecisions(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			decisions: [],
+		});
+	});
+
+	it('should return decisions list', async () => {
+		const mockDecisions = [
+			{ id: 'd1', title: 'Decision 1', description: 'Desc 1' },
+			{ id: 'd2', title: 'Decision 2', description: 'Desc 2' },
+		];
+		mockApiClient.getDecisions.mockResolvedValue({
+			ok: true,
+			data: { decisions: mockDecisions },
+		});
+		const ctx = createMockContext();
+
+		const result = await getDecisions(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			decisions: mockDecisions,
+		});
+	});
+
+	it('should call API client getDecisions with options', async () => {
+		mockApiClient.getDecisions.mockResolvedValue({
+			ok: true,
+			data: { decisions: [] },
+		});
+		const ctx = createMockContext();
+
+		await getDecisions(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(mockApiClient.getDecisions).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			{ limit: 10, offset: 0, search_query: undefined }
+		);
+	});
+
+	it('should return error when query fails', async () => {
+		mockApiClient.getDecisions.mockResolvedValue({
+			ok: false,
+			error: 'Query failed',
+		});
+		const ctx = createMockContext();
+
+		const result = await getDecisions(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({
+			error: 'Query failed',
+		});
+	});
+});
+
+// ============================================================================
+// deleteDecision Tests
+// ============================================================================
+
+describe('deleteDecision', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing decision_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(deleteDecision({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid decision_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			deleteDecision({ decision_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should delete decision successfully', async () => {
+		mockApiClient.deleteDecision.mockResolvedValue({
+			ok: true,
+			data: { success: true },
+		});
+		const ctx = createMockContext();
+
+		const result = await deleteDecision(
+			{ decision_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+		});
+	});
+
+	it('should call API client deleteDecision', async () => {
+		mockApiClient.deleteDecision.mockResolvedValue({
+			ok: true,
+			data: { success: true },
+		});
+		const ctx = createMockContext();
+
+		await deleteDecision(
+			{ decision_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(mockApiClient.deleteDecision).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000'
+		);
+	});
+
+	it('should return error when delete fails', async () => {
+		mockApiClient.deleteDecision.mockResolvedValue({
+			ok: false,
+			error: 'Delete failed',
+		});
+		const ctx = createMockContext();
+
+		const result = await deleteDecision(
+			{ decision_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({
+			error: 'Delete failed',
+		});
+	});
+});
+
+// ============================================================================
+// getDecisionsStats Tests
+// ============================================================================
+
+describe('getDecisionsStats', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(getDecisionsStats({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			getDecisionsStats({ project_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should return decisions stats', async () => {
+		mockApiClient.getDecisionsStats.mockResolvedValue({
+			ok: true,
+			data: { total: 10 },
+		});
+		const ctx = createMockContext();
+
+		const result = await getDecisionsStats(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({ total: 10 });
+	});
+
+	it('should call API client getDecisionsStats', async () => {
+		mockApiClient.getDecisionsStats.mockResolvedValue({
+			ok: true,
+			data: { total: 0 },
+		});
+		const ctx = createMockContext();
+
+		await getDecisionsStats(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(mockApiClient.getDecisionsStats).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000'
+		);
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.getDecisionsStats.mockResolvedValue({
+			ok: false,
+			error: 'Query failed',
+		});
+		const ctx = createMockContext();
+
+		const result = await getDecisionsStats({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({
+			error: 'Query failed',
+		});
+	});
+});