@vibescope/mcp-server 0.2.8 → 0.3.0

package/src/handlers/__test-setup__.ts

@@ -1,236 +1,236 @@
-/**
- * Global Test Setup for Handler Tests
- *
- * This file is loaded before all tests via vitest.config.ts setupFiles.
- * It provides a global mock for the API client used by migrated handlers.
- */
-
-import { vi, beforeEach } from 'vitest';
-
-// Default success response for API calls
-const defaultSuccess = { ok: true, data: {} };
-const defaultError = { ok: false, error: 'Mock error' };
-
-// ============================================================================
-// Global Mock API Client
-// ============================================================================
-
-/**
- * Mock API client with all methods stubbed.
- * Tests can configure return values using mockResolvedValue/mockImplementation.
- * Default return is a success response { ok: true, data: {} }
- */
-export const mockApiClient = {
-	// Session
-	startSession: vi.fn(),
-	heartbeat: vi.fn(),
-	syncSession: vi.fn(),
-	endSession: vi.fn(),
-	reportTokenUsage: vi.fn(),
-
-	// Project
-	listProjects: vi.fn(),
-	getProject: vi.fn(),
-	getProjectContext: vi.fn(),
-	createProject: vi.fn(),
-	updateProject: vi.fn(),
-	updateProjectReadme: vi.fn(),
-	getGitWorkflow: vi.fn(),
-
-	// Tasks
-	getTasks: vi.fn(),
-	getNextTask: vi.fn(),
-	createTask: vi.fn(),
-	updateTask: vi.fn(),
-	completeTask: vi.fn(),
-	deleteTask: vi.fn(),
-	addTaskReference: vi.fn(),
-	removeTaskReference: vi.fn(),
-	batchUpdateTasks: vi.fn(),
-	batchCompleteTasks: vi.fn(),
-
-	// Subtasks
-	addSubtask: vi.fn(),
-	getSubtasks: vi.fn(),
-
-	// Progress
-	logProgress: vi.fn(),
-	getActivityFeed: vi.fn(),
-
-	// Blockers
-	addBlocker: vi.fn(),
-	resolveBlocker: vi.fn(),
-	getBlockers: vi.fn(),
-	getBlockersStats: vi.fn(),
-	deleteBlocker: vi.fn(),
-
-	// Decisions
-	logDecision: vi.fn(),
-	getDecisions: vi.fn(),
-	getDecisionsStats: vi.fn(),
-	deleteDecision: vi.fn(),
-
-	// Ideas
-	addIdea: vi.fn(),
-	updateIdea: vi.fn(),
-	getIdeas: vi.fn(),
-	deleteIdea: vi.fn(),
-	convertIdeaToTask: vi.fn(),
-
-	// Findings
-	addFinding: vi.fn(),
-	getFindings: vi.fn(),
-	getFindingsStats: vi.fn(),
-	updateFinding: vi.fn(),
-	deleteFinding: vi.fn(),
-	queryKnowledgeBase: vi.fn(),
-
-	// Requests
-	getPendingRequests: vi.fn(),
-	acknowledgeRequest: vi.fn(),
-	answerQuestion: vi.fn(),
-
-	// Milestones
-	addMilestone: vi.fn(),
-	updateMilestone: vi.fn(),
-	completeMilestone: vi.fn(),
-	deleteMilestone: vi.fn(),
-	getMilestones: vi.fn(),
-
-	// Fallback
-	startFallbackActivity: vi.fn(),
-	stopFallbackActivity: vi.fn(),
-	getActivityHistory: vi.fn(),
-	getActivitySchedules: vi.fn(),
-
-	// Cost
-	getCostSummary: vi.fn(),
-	getCostAlerts: vi.fn(),
-	addCostAlert: vi.fn(),
-	updateCostAlert: vi.fn(),
-	deleteCostAlert: vi.fn(),
-	getTaskCosts: vi.fn(),
-
-	// Git Issues
-	addGitIssue: vi.fn(),
-	resolveGitIssue: vi.fn(),
-	getGitIssues: vi.fn(),
-	deleteGitIssue: vi.fn(),
-
-	// Organizations
-	listOrganizations: vi.fn(),
-	createOrganization: vi.fn(),
-	updateOrganization: vi.fn(),
-	deleteOrganization: vi.fn(),
-	listOrgMembers: vi.fn(),
-	inviteMember: vi.fn(),
-	updateMemberRole: vi.fn(),
-	removeMember: vi.fn(),
-	leaveOrganization: vi.fn(),
-	shareProjectWithOrg: vi.fn(),
-	updateProjectShare: vi.fn(),
-	unshareProject: vi.fn(),
-	listProjectShares: vi.fn(),
-
-	// Bodies of Work
-	createBodyOfWork: vi.fn(),
-	updateBodyOfWork: vi.fn(),
-	getBodyOfWork: vi.fn(),
-	getBodiesOfWork: vi.fn(),
-	deleteBodyOfWork: vi.fn(),
-	addTaskToBodyOfWork: vi.fn(),
-	removeTaskFromBodyOfWork: vi.fn(),
-	activateBodyOfWork: vi.fn(),
-	addTaskDependency: vi.fn(),
-	removeTaskDependency: vi.fn(),
-	getTaskDependencies: vi.fn(),
-	getNextBodyOfWorkTask: vi.fn(),
-
-	// Deployment
-	requestDeployment: vi.fn(),
-	claimDeploymentValidation: vi.fn(),
-	reportValidation: vi.fn(),
-	checkDeploymentStatus: vi.fn(),
-	startDeployment: vi.fn(),
-	completeDeployment: vi.fn(),
-	cancelDeployment: vi.fn(),
-	addDeploymentRequirement: vi.fn(),
-	completeDeploymentRequirement: vi.fn(),
-	getDeploymentRequirements: vi.fn(),
-	getDeploymentRequirementsStats: vi.fn(),
-	scheduleDeployment: vi.fn(),
-	getScheduledDeployments: vi.fn(),
-	updateScheduledDeployment: vi.fn(),
-	deleteScheduledDeployment: vi.fn(),
-	triggerScheduledDeployment: vi.fn(),
-	checkDueDeployments: vi.fn(),
-
-	// Validation
-	getTasksAwaitingValidation: vi.fn(),
-	claimValidation: vi.fn(),
-	validateTask: vi.fn(),
-
-	// Help Topics
-	getHelpTopic: vi.fn(),
-	getHelpTopics: vi.fn(),
-
-	// File Checkouts
-	checkoutFile: vi.fn(),
-	checkinFile: vi.fn(),
-	getFileCheckouts: vi.fn(),
-	getFileCheckoutsStats: vi.fn(),
-	abandonCheckout: vi.fn(),
-
-	// Connectors
-	getConnectors: vi.fn(),
-	getConnector: vi.fn(),
-	addConnector: vi.fn(),
-	updateConnector: vi.fn(),
-	deleteConnector: vi.fn(),
-	testConnector: vi.fn(),
-	getConnectorEvents: vi.fn(),
-
-	// Proxy (generic)
-	proxy: vi.fn(),
-};
-
-// ============================================================================
-// Mock the API Client Module
-// ============================================================================
-
-// Note: The path is relative to the test file location, not this setup file
-// This setup file is in src/handlers/, tests import from ../api-client.js
-vi.mock('../api-client.js', () => ({
-	getApiClient: () => mockApiClient,
-	initApiClient: vi.fn(),
-	VibescopeApiClient: vi.fn(),
-}));
-
-// Also mock with absolute-style path for tests outside handlers/
-vi.mock('../../api-client.js', () => ({
-	getApiClient: () => mockApiClient,
-	initApiClient: vi.fn(),
-	VibescopeApiClient: vi.fn(),
-}));
-
-// Export for test files that need direct access
-export { mockApiClient as __mockApiClient__ };
-
-// ============================================================================
-// Reset Mocks Before Each Test
-// ============================================================================
-
-/**
- * Reset all mock functions before each test and set default return values.
- * This ensures tests start with a clean slate and have sensible defaults.
- */
-beforeEach(() => {
-	// Reset all mocks
-	Object.values(mockApiClient).forEach((mockFn) => {
-		if (typeof mockFn === 'function' && mockFn.mockReset) {
-			mockFn.mockReset();
-			// Set default return value to success
-			mockFn.mockResolvedValue({ ok: true, data: {} });
-		}
-	});
-});
+/**
+ * Global Test Setup for Handler Tests
+ *
+ * This file is loaded before all tests via vitest.config.ts setupFiles.
+ * It provides a global mock for the API client used by migrated handlers.
+ */
+
+import { vi, beforeEach } from 'vitest';
+
+// Default success response for API calls
+const defaultSuccess = { ok: true, data: {} };
+const defaultError = { ok: false, error: 'Mock error' };
+
+// ============================================================================
+// Global Mock API Client
+// ============================================================================
+
+/**
+ * Mock API client with all methods stubbed.
+ * Tests can configure return values using mockResolvedValue/mockImplementation.
+ * Default return is a success response { ok: true, data: {} }
+ */
+export const mockApiClient = {
+	// Session
+	startSession: vi.fn(),
+	heartbeat: vi.fn(),
+	syncSession: vi.fn(),
+	endSession: vi.fn(),
+	reportTokenUsage: vi.fn(),
+
+	// Project
+	listProjects: vi.fn(),
+	getProject: vi.fn(),
+	getProjectContext: vi.fn(),
+	createProject: vi.fn(),
+	updateProject: vi.fn(),
+	updateProjectReadme: vi.fn(),
+	getGitWorkflow: vi.fn(),
+
+	// Tasks
+	getTasks: vi.fn(),
+	getNextTask: vi.fn(),
+	createTask: vi.fn(),
+	updateTask: vi.fn(),
+	completeTask: vi.fn(),
+	deleteTask: vi.fn(),
+	addTaskReference: vi.fn(),
+	removeTaskReference: vi.fn(),
+	batchUpdateTasks: vi.fn(),
+	batchCompleteTasks: vi.fn(),
+
+	// Subtasks
+	addSubtask: vi.fn(),
+	getSubtasks: vi.fn(),
+
+	// Progress
+	logProgress: vi.fn(),
+	getActivityFeed: vi.fn(),
+
+	// Blockers
+	addBlocker: vi.fn(),
+	resolveBlocker: vi.fn(),
+	getBlockers: vi.fn(),
+	getBlockersStats: vi.fn(),
+	deleteBlocker: vi.fn(),
+
+	// Decisions
+	logDecision: vi.fn(),
+	getDecisions: vi.fn(),
+	getDecisionsStats: vi.fn(),
+	deleteDecision: vi.fn(),
+
+	// Ideas
+	addIdea: vi.fn(),
+	updateIdea: vi.fn(),
+	getIdeas: vi.fn(),
+	deleteIdea: vi.fn(),
+	convertIdeaToTask: vi.fn(),
+
+	// Findings
+	addFinding: vi.fn(),
+	getFindings: vi.fn(),
+	getFindingsStats: vi.fn(),
+	updateFinding: vi.fn(),
+	deleteFinding: vi.fn(),
+	queryKnowledgeBase: vi.fn(),
+
+	// Requests
+	getPendingRequests: vi.fn(),
+	acknowledgeRequest: vi.fn(),
+	answerQuestion: vi.fn(),
+
+	// Milestones
+	addMilestone: vi.fn(),
+	updateMilestone: vi.fn(),
+	completeMilestone: vi.fn(),
+	deleteMilestone: vi.fn(),
+	getMilestones: vi.fn(),
+
+	// Fallback
+	startFallbackActivity: vi.fn(),
+	stopFallbackActivity: vi.fn(),
+	getActivityHistory: vi.fn(),
+	getActivitySchedules: vi.fn(),
+
+	// Cost
+	getCostSummary: vi.fn(),
+	getCostAlerts: vi.fn(),
+	addCostAlert: vi.fn(),
+	updateCostAlert: vi.fn(),
+	deleteCostAlert: vi.fn(),
+	getTaskCosts: vi.fn(),
+
+	// Git Issues
+	addGitIssue: vi.fn(),
+	resolveGitIssue: vi.fn(),
+	getGitIssues: vi.fn(),
+	deleteGitIssue: vi.fn(),
+
+	// Organizations
+	listOrganizations: vi.fn(),
+	createOrganization: vi.fn(),
+	updateOrganization: vi.fn(),
+	deleteOrganization: vi.fn(),
+	listOrgMembers: vi.fn(),
+	inviteMember: vi.fn(),
+	updateMemberRole: vi.fn(),
+	removeMember: vi.fn(),
+	leaveOrganization: vi.fn(),
+	shareProjectWithOrg: vi.fn(),
+	updateProjectShare: vi.fn(),
+	unshareProject: vi.fn(),
+	listProjectShares: vi.fn(),
+
+	// Bodies of Work
+	createBodyOfWork: vi.fn(),
+	updateBodyOfWork: vi.fn(),
+	getBodyOfWork: vi.fn(),
+	getBodiesOfWork: vi.fn(),
+	deleteBodyOfWork: vi.fn(),
+	addTaskToBodyOfWork: vi.fn(),
+	removeTaskFromBodyOfWork: vi.fn(),
+	activateBodyOfWork: vi.fn(),
+	addTaskDependency: vi.fn(),
+	removeTaskDependency: vi.fn(),
+	getTaskDependencies: vi.fn(),
+	getNextBodyOfWorkTask: vi.fn(),
+
+	// Deployment
+	requestDeployment: vi.fn(),
+	claimDeploymentValidation: vi.fn(),
+	reportValidation: vi.fn(),
+	checkDeploymentStatus: vi.fn(),
+	startDeployment: vi.fn(),
+	completeDeployment: vi.fn(),
+	cancelDeployment: vi.fn(),
+	addDeploymentRequirement: vi.fn(),
+	completeDeploymentRequirement: vi.fn(),
+	getDeploymentRequirements: vi.fn(),
+	getDeploymentRequirementsStats: vi.fn(),
+	scheduleDeployment: vi.fn(),
+	getScheduledDeployments: vi.fn(),
+	updateScheduledDeployment: vi.fn(),
+	deleteScheduledDeployment: vi.fn(),
+	triggerScheduledDeployment: vi.fn(),
+	checkDueDeployments: vi.fn(),
+
+	// Validation
+	getTasksAwaitingValidation: vi.fn(),
+	claimValidation: vi.fn(),
+	validateTask: vi.fn(),
+
+	// Help Topics
+	getHelpTopic: vi.fn(),
+	getHelpTopics: vi.fn(),
+
+	// File Checkouts
+	checkoutFile: vi.fn(),
+	checkinFile: vi.fn(),
+	getFileCheckouts: vi.fn(),
+	getFileCheckoutsStats: vi.fn(),
+	abandonCheckout: vi.fn(),
+
+	// Connectors
+	getConnectors: vi.fn(),
+	getConnector: vi.fn(),
+	addConnector: vi.fn(),
+	updateConnector: vi.fn(),
+	deleteConnector: vi.fn(),
+	testConnector: vi.fn(),
+	getConnectorEvents: vi.fn(),
+
+	// Proxy (generic)
+	proxy: vi.fn(),
+};
+
+// ============================================================================
+// Mock the API Client Module
+// ============================================================================
+
+// Note: The path is relative to the test file location, not this setup file
+// This setup file is in src/handlers/, tests import from ../api-client.js
+vi.mock('../api-client.js', () => ({
+	getApiClient: () => mockApiClient,
+	initApiClient: vi.fn(),
+	VibescopeApiClient: vi.fn(),
+}));
+
+// Also mock with absolute-style path for tests outside handlers/
+vi.mock('../../api-client.js', () => ({
+	getApiClient: () => mockApiClient,
+	initApiClient: vi.fn(),
+	VibescopeApiClient: vi.fn(),
+}));
+
+// Export for test files that need direct access
+export { mockApiClient as __mockApiClient__ };
+
+// ============================================================================
+// Reset Mocks Before Each Test
+// ============================================================================
+
+/**
+ * Reset all mock functions before each test and set default return values.
+ * This ensures tests start with a clean slate and have sensible defaults.
+ */
+beforeEach(() => {
+	// Reset all mocks
+	Object.values(mockApiClient).forEach((mockFn) => {
+		if (typeof mockFn === 'function' && mockFn.mockReset) {
+			mockFn.mockReset();
+			// Set default return value to success
+			mockFn.mockResolvedValue({ ok: true, data: {} });
+		}
+	});
+});

package/src/handlers/__test-utils__.ts

@@ -1,87 +1,87 @@
-/**
- * Shared Test Utilities
- *
- * Common mock factories and test helpers used across handler tests.
- * Handlers now use API client instead of direct Supabase access.
- */
-
-import { vi } from 'vitest';
-import type { HandlerContext, TokenUsage } from './types.js';
-
-// ============================================================================
-// Mock Handler Context Factory
-// ============================================================================
-
-export interface MockContextOptions {
-	sessionId?: string | null;
-	userId?: string;
-	apiKeyId?: string;
-	instanceId?: string;
-	persona?: string;
-	tokenUsage?: TokenUsage;
-}
-
-/**
- * Create a mock HandlerContext for testing.
- *
- * @example
- * ```ts
- * const ctx = createMockContext({ sessionId: 'test-session' });
- * ```
- */
-export function createMockContext(
-	options: MockContextOptions = {}
-): HandlerContext {
-	const defaultTokenUsage: TokenUsage = {
-		callCount: 5,
-		totalTokens: 2500,
-		byTool: {},
-		byModel: {},
-		currentModel: null,
-	};
-
-	const sessionId = 'sessionId' in options ? (options.sessionId ?? null) : 'session-123';
-
-	return {
-		auth: {
-			userId: options.userId ?? 'user-123',
-			apiKeyId: options.apiKeyId ?? 'api-key-123',
-			scope: 'personal' as const,
-		},
-		session: {
-			instanceId: options.instanceId ?? 'instance-abc',
-			currentSessionId: sessionId,
-			currentPersona: options.persona ?? 'Wave',
-			tokenUsage: options.tokenUsage ?? defaultTokenUsage,
-		},
-		updateSession: vi.fn(),
-	};
-}
-
-// ============================================================================
-// Test Data Generators
-// ============================================================================
-
-/**
- * Generate a valid UUID for testing.
- */
-export function testUUID(): string {
-	return '123e4567-e89b-12d3-a456-426614174000';
-}
-
-/**
- * Generate a random-ish UUID for testing (deterministic based on seed).
- */
-export function testUUIDSeeded(seed: number): string {
-	const hex = seed.toString(16).padStart(8, '0');
-	return `${hex}-e89b-12d3-a456-426614174000`;
-}
-
-/**
- * Create a mock timestamp for consistent testing.
- */
-export function testTimestamp(offsetMinutes = 0): string {
-	const date = new Date('2025-01-14T12:00:00Z');
-	date.setMinutes(date.getMinutes() + offsetMinutes);
-	return date.toISOString();
-}
+/**
+ * Shared Test Utilities
+ *
+ * Common mock factories and test helpers used across handler tests.
+ * Handlers now use API client instead of direct Supabase access.
+ */
+
+import { vi } from 'vitest';
+import type { HandlerContext, TokenUsage } from './types.js';
+
+// ============================================================================
+// Mock Handler Context Factory
+// ============================================================================
+
+export interface MockContextOptions {
+	sessionId?: string | null;
+	userId?: string;
+	apiKeyId?: string;
+	instanceId?: string;
+	persona?: string;
+	tokenUsage?: TokenUsage;
+}
+
+/**
+ * Create a mock HandlerContext for testing.
+ *
+ * @example
+ * ```ts
+ * const ctx = createMockContext({ sessionId: 'test-session' });
+ * ```
+ */
+export function createMockContext(
+	options: MockContextOptions = {}
+): HandlerContext {
+	const defaultTokenUsage: TokenUsage = {
+		callCount: 5,
+		totalTokens: 2500,
+		byTool: {},
+		byModel: {},
+		currentModel: null,
+	};
+
+	const sessionId = 'sessionId' in options ? (options.sessionId ?? null) : 'session-123';
+
+	return {
+		auth: {
+			userId: options.userId ?? 'user-123',
+			apiKeyId: options.apiKeyId ?? 'api-key-123',
+			scope: 'personal' as const,
+		},
+		session: {
+			instanceId: options.instanceId ?? 'instance-abc',
+			currentSessionId: sessionId,
+			currentPersona: options.persona ?? 'Wave',
+			tokenUsage: options.tokenUsage ?? defaultTokenUsage,
+		},
+		updateSession: vi.fn(),
+	};
+}
+
+// ============================================================================
+// Test Data Generators
+// ============================================================================
+
+/**
+ * Generate a valid UUID for testing.
+ */
+export function testUUID(): string {
+	return '123e4567-e89b-12d3-a456-426614174000';
+}
+
+/**
+ * Generate a random-ish UUID for testing (deterministic based on seed).
+ */
+export function testUUIDSeeded(seed: number): string {
+	const hex = seed.toString(16).padStart(8, '0');
+	return `${hex}-e89b-12d3-a456-426614174000`;
+}
+
+/**
+ * Create a mock timestamp for consistent testing.
+ */
+export function testTimestamp(offsetMinutes = 0): string {
+	const date = new Date('2025-01-14T12:00:00Z');
+	date.setMinutes(date.getMinutes() + offsetMinutes);
+	return date.toISOString();
+}