@vibescope/mcp-server 0.0.1

package/src/handlers/decisions.ts

@@ -0,0 +1,86 @@
+/**
+ * Decisions Handlers
+ *
+ * Handles architectural/technical decisions:
+ * - log_decision
+ * - get_decisions
+ * - delete_decision
+ */
+
+import type { Handler, HandlerRegistry } from './types.js';
+import { validateRequired, validateUUID } from '../validators.js';
+
+export const logDecision: Handler = async (args, ctx) => {
+	const { project_id, title, description, rationale, alternatives_considered } = args as {
+		project_id: string;
+		title: string;
+		description: string;
+		rationale?: string;
+		alternatives_considered?: string[];
+	};
+
+	validateRequired(project_id, 'project_id');
+	validateUUID(project_id, 'project_id');
+	validateRequired(title, 'title');
+	validateRequired(description, 'description');
+
+	const { supabase, session } = ctx;
+
+	const { error } = await supabase
+		.from('decisions')
+		.insert({
+			project_id,
+			title,
+			description,
+			rationale: rationale || null,
+			alternatives_considered: alternatives_considered || null,
+			created_by: 'agent',
+			created_by_session_id: session.currentSessionId,
+		});
+
+	if (error) throw new Error(`Failed to log decision: ${error.message}`);
+
+	return { result: { success: true, title } };
+};
+
+export const getDecisions: Handler = async (args, ctx) => {
+	const { project_id } = args as { project_id: string };
+
+	validateRequired(project_id, 'project_id');
+	validateUUID(project_id, 'project_id');
+
+	const { data, error } = await ctx.supabase
+		.from('decisions')
+		.select('id, title, description, rationale, created_at')
+		.eq('project_id', project_id)
+		.order('created_at', { ascending: false });
+
+	if (error) throw new Error(`Failed to fetch decisions: ${error.message}`);
+
+	return { result: { decisions: data || [] } };
+};
+
+export const deleteDecision: Handler = async (args, ctx) => {
+	const { decision_id } = args as { decision_id: string };
+
+	validateRequired(decision_id, 'decision_id');
+	validateUUID(decision_id, 'decision_id');
+
+	const { error } = await ctx.supabase
+		.from('decisions')
+		.delete()
+		.eq('id', decision_id);
+
+	if (error) throw new Error(`Failed to delete decision: ${error.message}`);
+
+	return { result: { success: true } };
+};
+
+/**
+ * Decisions handlers registry
+ */
+export const decisionHandlers: HandlerRegistry = {
+	log_decision: logDecision,
+	get_decisions: getDecisions,
+	delete_decision: deleteDecision,
+};

package/src/handlers/deployment.test.ts

@@ -0,0 +1,516 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import type { SupabaseClient } from '@supabase/supabase-js';
+import type { HandlerContext, TokenUsage } from './types.js';
+import {
+	claimDeploymentValidation,
+	reportValidation,
+	checkDeploymentStatus,
+	startDeployment,
+	completeDeployment,
+	cancelDeployment,
+	addDeploymentRequirement,
+	getDeploymentRequirements,
+} from './deployment.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(),
+		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(),
+	};
+}
+
+// ============================================================================
+// claimDeploymentValidation Tests
+// ============================================================================
+
+describe('claimDeploymentValidation', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(claimDeploymentValidation({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			claimDeploymentValidation({ project_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should return error when no pending deployment', async () => {
+		const supabase = createMockSupabase({
+			selectResult: { data: null, error: { message: 'Not found' } },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await claimDeploymentValidation(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: false,
+			error: 'No pending deployment found',
+		});
+	});
+});
+
+// ============================================================================
+// reportValidation Tests
+// ============================================================================
+
+describe('reportValidation', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			reportValidation({ build_passed: true }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing build_passed', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			reportValidation({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should return error when no deployment being validated', async () => {
+		const supabase = createMockSupabase({
+			selectResult: { data: null, error: { message: 'Not found' } },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await reportValidation(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000', build_passed: true },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: false,
+			error: 'No deployment being validated. Use claim_deployment_validation first.',
+		});
+	});
+});
+
+// ============================================================================
+// checkDeploymentStatus Tests
+// ============================================================================
+
+describe('checkDeploymentStatus', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(checkDeploymentStatus({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should return no deployment when none found', async () => {
+		const supabase = createMockSupabase({
+			selectResult: { data: null, error: { message: 'Not found' } },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await checkDeploymentStatus(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			has_deployment: false,
+			message: 'No deployments found for this project',
+		});
+	});
+
+	it('should return deployment details when found', async () => {
+		const mockDeployment = {
+			id: 'deploy-1',
+			status: 'deployed',
+			environment: 'production',
+			requested_by: 'agent',
+			build_passed: true,
+			tests_passed: true,
+			created_at: '2025-01-14T10:00:00Z',
+		};
+
+		const supabase = createMockSupabase({
+			selectResult: { data: mockDeployment, error: null },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await checkDeploymentStatus(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			has_deployment: true,
+		});
+		expect((result.result as { deployment: { id: string } }).deployment.id).toBe('deploy-1');
+	});
+});
+
+// ============================================================================
+// startDeployment Tests
+// ============================================================================
+
+describe('startDeployment', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(startDeployment({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should return error when no deployment ready', async () => {
+		const supabase = createMockSupabase({
+			selectResult: { data: null, error: { message: 'Not found' } },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await startDeployment(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: false,
+			error: 'No deployment ready. Must pass validation first.',
+		});
+	});
+});
+
+// ============================================================================
+// completeDeployment Tests
+// ============================================================================
+
+describe('completeDeployment', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			completeDeployment({ success: true }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing success', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			completeDeployment({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should return error when no deployment in progress', async () => {
+		const supabase = createMockSupabase({
+			selectResult: { data: null, error: { message: 'Not found' } },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await completeDeployment(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000', success: true },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: false,
+			error: 'No deployment in progress. Use start_deployment first.',
+		});
+	});
+});
+
+// ============================================================================
+// cancelDeployment Tests
+// ============================================================================
+
+describe('cancelDeployment', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(cancelDeployment({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should return error when no active deployment', async () => {
+		const supabase = createMockSupabase({
+			selectResult: { data: null, error: { message: 'Not found' } },
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await cancelDeployment(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: false,
+			error: 'No active deployment',
+		});
+	});
+});
+
+// ============================================================================
+// addDeploymentRequirement Tests
+// ============================================================================
+
+describe('addDeploymentRequirement', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			addDeploymentRequirement({ type: 'migration', title: 'Test' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing type', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			addDeploymentRequirement({
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				title: 'Test',
+			}, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing title', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			addDeploymentRequirement({
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				type: 'migration',
+			}, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid type', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			addDeploymentRequirement({
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				type: 'invalid_type',
+				title: 'Test',
+			}, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid stage', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(
+			addDeploymentRequirement({
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				type: 'migration',
+				title: 'Test',
+				stage: 'invalid_stage',
+			}, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should add requirement successfully', async () => {
+		const supabase = createMockSupabase({
+			insertResult: {
+				data: { id: 'req-1', type: 'migration', title: 'Test Migration', stage: 'preparation', blocking: false },
+				error: null,
+			},
+		});
+		const ctx = createMockContext(supabase);
+
+		const result = await addDeploymentRequirement(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				type: 'migration',
+				title: 'Test Migration',
+			},
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			requirement_id: 'req-1',
+			stage: 'preparation',
+		});
+	});
+});
+
+// ============================================================================
+// getDeploymentRequirements Tests
+// ============================================================================
+
+describe('getDeploymentRequirements', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const supabase = createMockSupabase();
+		const ctx = createMockContext(supabase);
+
+		await expect(getDeploymentRequirements({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should return empty list when no requirements', async () => {
+		const supabase = createMockSupabase({
+			selectResult: { data: [], error: null },
+		});
+		const ctx = createMockContext(supabase);
+
+		// Need to mock the .then() to return the array result
+		vi.mocked(supabase.from('').select).mockReturnValue({
+			...supabase,
+			then: (resolve: (val: unknown) => void) =>
+				Promise.resolve({ data: [], error: null }).then(resolve),
+		} as unknown as ReturnType<SupabaseClient['from']>);
+
+		const result = await getDeploymentRequirements(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toHaveProperty('requirements');
+		expect(result.result).toHaveProperty('deployment_blocked');
+	});
+
+	it('should query with correct project_id', async () => {
+		const supabase = createMockSupabase({
+			selectResult: { data: [], error: null },
+		});
+		const ctx = createMockContext(supabase);
+
+		await getDeploymentRequirements(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(supabase.from).toHaveBeenCalledWith('deployment_requirements');
+		expect(supabase.eq).toHaveBeenCalled();
+	});
+});