@vibescope/mcp-server 0.4.4 → 0.4.6

package/src/cli.test.ts

@@ -1,284 +1,284 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-
-// Mock child_process before importing cli module
-vi.mock('child_process', () => ({
-	execSync: vi.fn(),
-}));
-
-// Mock fetch
-const mockFetch = vi.fn();
-global.fetch = mockFetch;
-
-import { execSync } from 'child_process';
-import {
-	detectGitUrl,
-	normalizeGitUrl,
-	verify,
-	type VerificationResult,
-} from './cli.js';
-
-describe('CLI verification logic', () => {
-	beforeEach(() => {
-		vi.clearAllMocks();
-	});
-
-	describe('normalizeGitUrl', () => {
-		it('should remove .git suffix and lowercase', () => {
-			expect(normalizeGitUrl('https://github.com/User/Repo.git')).toBe('https://github.com/user/repo');
-		});
-
-		it('should convert SSH to HTTPS format and lowercase', () => {
-			expect(normalizeGitUrl('git@github.com:User/Repo.git')).toBe('https://github.com/user/repo');
-		});
-
-		it('should handle HTTPS URLs without .git suffix', () => {
-			expect(normalizeGitUrl('https://github.com/user/repo')).toBe('https://github.com/user/repo');
-		});
-
-		it('should handle GitLab SSH URLs and lowercase', () => {
-			expect(normalizeGitUrl('git@gitlab.com:User/Repo.git')).toBe('https://gitlab.com/user/repo');
-		});
-
-		it('should handle Bitbucket SSH URLs and lowercase', () => {
-			expect(normalizeGitUrl('git@bitbucket.org:User/Repo.git')).toBe('https://bitbucket.org/user/repo');
-		});
-
-		it('should return null for empty string', () => {
-			expect(normalizeGitUrl('')).toBe(null);
-		});
-
-		it('should handle http and upgrade to https', () => {
-			expect(normalizeGitUrl('http://github.com/user/repo')).toBe('https://github.com/user/repo');
-		});
-
-		it('should handle trailing slashes', () => {
-			expect(normalizeGitUrl('https://github.com/user/repo/')).toBe('https://github.com/user/repo');
-		});
-
-		it('should add https protocol to bare URLs', () => {
-			expect(normalizeGitUrl('github.com/user/repo')).toBe('https://github.com/user/repo');
-		});
-	});
-
-	describe('detectGitUrl', () => {
-		it('should return normalized git URL when git command succeeds', () => {
-			vi.mocked(execSync).mockReturnValue('git@github.com:user/repo.git\n');
-			const result = detectGitUrl();
-			expect(result).toBe('https://github.com/user/repo');
-		});
-
-		it('should return null when git command fails', () => {
-			vi.mocked(execSync).mockImplementation(() => {
-				throw new Error('Not a git repository');
-			});
-			const result = detectGitUrl();
-			expect(result).toBeNull();
-		});
-
-		it('should handle HTTPS URLs', () => {
-			vi.mocked(execSync).mockReturnValue('https://github.com/user/repo.git\n');
-			const result = detectGitUrl();
-			expect(result).toBe('https://github.com/user/repo');
-		});
-
-		it('should call execSync with correct options', () => {
-			vi.mocked(execSync).mockReturnValue('https://github.com/user/repo.git');
-			detectGitUrl();
-			expect(execSync).toHaveBeenCalledWith('git config --get remote.origin.url', {
-				encoding: 'utf8',
-				timeout: 5000,
-				stdio: ['pipe', 'pipe', 'pipe'],
-			});
-		});
-
-		it('should trim whitespace from git URL', () => {
-			vi.mocked(execSync).mockReturnValue('  https://github.com/user/repo.git  \n');
-			const result = detectGitUrl();
-			expect(result).toBe('https://github.com/user/repo');
-		});
-
-		it('should return null on timeout', () => {
-			vi.mocked(execSync).mockImplementation(() => {
-				const error = new Error('Command timed out');
-				(error as NodeJS.ErrnoException).code = 'ETIMEDOUT';
-				throw error;
-			});
-			const result = detectGitUrl();
-			expect(result).toBeNull();
-		});
-	});
-});
-
-describe('verify function', () => {
-	const originalEnv = process.env;
-
-	beforeEach(() => {
-		vi.clearAllMocks();
-		mockFetch.mockReset();
-		process.env = { ...originalEnv };
-		process.env.VIBESCOPE_API_KEY = 'test-api-key';
-	});
-
-	afterEach(() => {
-		process.env = originalEnv;
-	});
-
-	it('should return error when API key is not set', async () => {
-		delete process.env.VIBESCOPE_API_KEY;
-		// Need to re-import to get the new env value
-		// Since the module caches the env var, we test the behavior directly
-		const result = await verify('https://github.com/test/repo');
-		// The module reads env at import time, so this test may not work as expected
-		// But the function should handle missing API key
-		expect(result.status).toBeDefined();
-	});
-
-	it('should call API with correct parameters', async () => {
-		mockFetch.mockResolvedValueOnce({
-			json: () => Promise.resolve({
-				status: 'compliant',
-				reason: 'All good',
-			}),
-		});
-
-		await verify('https://github.com/test/repo', 'proj-123');
-
-		expect(mockFetch).toHaveBeenCalledWith(
-			'https://vibescope.dev/api/mcp/verify',
-			expect.objectContaining({
-				method: 'POST',
-				headers: { 'Content-Type': 'application/json' },
-				body: expect.stringContaining('test-api-key'),
-			})
-		);
-
-		const body = JSON.parse(mockFetch.mock.calls[0][1].body);
-		expect(body.git_url).toBe('https://github.com/test/repo');
-		expect(body.project_id).toBe('proj-123');
-	});
-
-	it('should return compliant status from API', async () => {
-		mockFetch.mockResolvedValueOnce({
-			json: () => Promise.resolve({
-				status: 'compliant',
-				reason: 'All tracked work completed properly',
-				details: {
-					session_started: true,
-					project_id: 'proj-123',
-					project_name: 'Test Project',
-					git_url: 'https://github.com/test/repo',
-					in_progress_tasks: 0,
-					tasks_completed_this_session: 5,
-					progress_logs_this_session: 3,
-					blockers_logged_this_session: 0,
-					session_duration_minutes: 30,
-				},
-			}),
-		});
-
-		const result = await verify('https://github.com/test/repo');
-
-		expect(result.status).toBe('compliant');
-		expect(result.details?.tasks_completed_this_session).toBe(5);
-	});
-
-	it('should return non_compliant status from API', async () => {
-		mockFetch.mockResolvedValueOnce({
-			json: () => Promise.resolve({
-				status: 'non_compliant',
-				reason: 'You have 2 task(s) still in_progress',
-				continuation_prompt: 'Complete your tasks before exiting',
-			}),
-		});
-
-		const result = await verify('https://github.com/test/repo');
-
-		expect(result.status).toBe('non_compliant');
-		expect(result.continuation_prompt).toBeDefined();
-	});
-
-	it('should handle network errors', async () => {
-		mockFetch.mockRejectedValueOnce(new Error('Network error'));
-
-		const result = await verify('https://github.com/test/repo');
-
-		expect(result.status).toBe('error');
-		expect(result.reason).toBe('Network error');
-	});
-
-	it('should auto-detect git URL if not provided', async () => {
-		vi.mocked(execSync).mockReturnValue('https://github.com/auto/detected.git');
-		mockFetch.mockResolvedValueOnce({
-			json: () => Promise.resolve({ status: 'compliant', reason: 'OK' }),
-		});
-
-		await verify();
-
-		const body = JSON.parse(mockFetch.mock.calls[0][1].body);
-		expect(body.git_url).toBe('https://github.com/auto/detected');
-	});
-
-	it('should use custom API URL from env', async () => {
-		process.env.VIBESCOPE_API_URL = 'https://custom.api.com';
-		mockFetch.mockResolvedValueOnce({
-			json: () => Promise.resolve({ status: 'compliant', reason: 'OK' }),
-		});
-
-		// Note: The module caches VIBESCOPE_API_URL at import time
-		// This test verifies the URL construction logic
-		await verify('https://github.com/test/repo');
-
-		// The actual URL used depends on when the module was loaded
-		expect(mockFetch).toHaveBeenCalled();
-	});
-});
-
-describe('VerificationResult types', () => {
-	it('should have correct status types', () => {
-		const compliant: VerificationResult = {
-			status: 'compliant',
-			reason: 'All good',
-		};
-		expect(compliant.status).toBe('compliant');
-
-		const nonCompliant: VerificationResult = {
-			status: 'non_compliant',
-			reason: 'Tasks in progress',
-			continuation_prompt: 'Complete your tasks',
-		};
-		expect(nonCompliant.status).toBe('non_compliant');
-
-		const error: VerificationResult = {
-			status: 'error',
-			reason: 'Missing env vars',
-		};
-		expect(error.status).toBe('error');
-
-		const noSession: VerificationResult = {
-			status: 'no_session',
-			reason: 'No git URL detected',
-		};
-		expect(noSession.status).toBe('no_session');
-	});
-
-	it('should support optional details', () => {
-		const result: VerificationResult = {
-			status: 'compliant',
-			reason: 'All good',
-			details: {
-				session_started: true,
-				project_id: 'proj-123',
-				project_name: 'Test Project',
-				git_url: 'https://github.com/user/repo',
-				in_progress_tasks: 0,
-				tasks_completed_this_session: 5,
-				progress_logs_this_session: 3,
-				blockers_logged_this_session: 1,
-				session_duration_minutes: 45,
-			},
-		};
-
-		expect(result.details?.session_started).toBe(true);
-		expect(result.details?.tasks_completed_this_session).toBe(5);
-	});
-});
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+
+// Mock child_process before importing cli module
+vi.mock('child_process', () => ({
+	execSync: vi.fn(),
+}));
+
+// Mock fetch
+const mockFetch = vi.fn();
+global.fetch = mockFetch;
+
+import { execSync } from 'child_process';
+import {
+	detectGitUrl,
+	normalizeGitUrl,
+	verify,
+	type VerificationResult,
+} from './cli.js';
+
+describe('CLI verification logic', () => {
+	beforeEach(() => {
+		vi.clearAllMocks();
+	});
+
+	describe('normalizeGitUrl', () => {
+		it('should remove .git suffix and lowercase', () => {
+			expect(normalizeGitUrl('https://github.com/User/Repo.git')).toBe('https://github.com/user/repo');
+		});
+
+		it('should convert SSH to HTTPS format and lowercase', () => {
+			expect(normalizeGitUrl('git@github.com:User/Repo.git')).toBe('https://github.com/user/repo');
+		});
+
+		it('should handle HTTPS URLs without .git suffix', () => {
+			expect(normalizeGitUrl('https://github.com/user/repo')).toBe('https://github.com/user/repo');
+		});
+
+		it('should handle GitLab SSH URLs and lowercase', () => {
+			expect(normalizeGitUrl('git@gitlab.com:User/Repo.git')).toBe('https://gitlab.com/user/repo');
+		});
+
+		it('should handle Bitbucket SSH URLs and lowercase', () => {
+			expect(normalizeGitUrl('git@bitbucket.org:User/Repo.git')).toBe('https://bitbucket.org/user/repo');
+		});
+
+		it('should return null for empty string', () => {
+			expect(normalizeGitUrl('')).toBe(null);
+		});
+
+		it('should handle http and upgrade to https', () => {
+			expect(normalizeGitUrl('http://github.com/user/repo')).toBe('https://github.com/user/repo');
+		});
+
+		it('should handle trailing slashes', () => {
+			expect(normalizeGitUrl('https://github.com/user/repo/')).toBe('https://github.com/user/repo');
+		});
+
+		it('should add https protocol to bare URLs', () => {
+			expect(normalizeGitUrl('github.com/user/repo')).toBe('https://github.com/user/repo');
+		});
+	});
+
+	describe('detectGitUrl', () => {
+		it('should return normalized git URL when git command succeeds', () => {
+			vi.mocked(execSync).mockReturnValue('git@github.com:user/repo.git\n');
+			const result = detectGitUrl();
+			expect(result).toBe('https://github.com/user/repo');
+		});
+
+		it('should return null when git command fails', () => {
+			vi.mocked(execSync).mockImplementation(() => {
+				throw new Error('Not a git repository');
+			});
+			const result = detectGitUrl();
+			expect(result).toBeNull();
+		});
+
+		it('should handle HTTPS URLs', () => {
+			vi.mocked(execSync).mockReturnValue('https://github.com/user/repo.git\n');
+			const result = detectGitUrl();
+			expect(result).toBe('https://github.com/user/repo');
+		});
+
+		it('should call execSync with correct options', () => {
+			vi.mocked(execSync).mockReturnValue('https://github.com/user/repo.git');
+			detectGitUrl();
+			expect(execSync).toHaveBeenCalledWith('git config --get remote.origin.url', {
+				encoding: 'utf8',
+				timeout: 5000,
+				stdio: ['pipe', 'pipe', 'pipe'],
+			});
+		});
+
+		it('should trim whitespace from git URL', () => {
+			vi.mocked(execSync).mockReturnValue('  https://github.com/user/repo.git  \n');
+			const result = detectGitUrl();
+			expect(result).toBe('https://github.com/user/repo');
+		});
+
+		it('should return null on timeout', () => {
+			vi.mocked(execSync).mockImplementation(() => {
+				const error = new Error('Command timed out');
+				(error as NodeJS.ErrnoException).code = 'ETIMEDOUT';
+				throw error;
+			});
+			const result = detectGitUrl();
+			expect(result).toBeNull();
+		});
+	});
+});
+
+describe('verify function', () => {
+	const originalEnv = process.env;
+
+	beforeEach(() => {
+		vi.clearAllMocks();
+		mockFetch.mockReset();
+		process.env = { ...originalEnv };
+		process.env.VIBESCOPE_API_KEY = 'test-api-key';
+	});
+
+	afterEach(() => {
+		process.env = originalEnv;
+	});
+
+	it('should return error when API key is not set', async () => {
+		delete process.env.VIBESCOPE_API_KEY;
+		// Need to re-import to get the new env value
+		// Since the module caches the env var, we test the behavior directly
+		const result = await verify('https://github.com/test/repo');
+		// The module reads env at import time, so this test may not work as expected
+		// But the function should handle missing API key
+		expect(result.status).toBeDefined();
+	});
+
+	it('should call API with correct parameters', async () => {
+		mockFetch.mockResolvedValueOnce({
+			json: () => Promise.resolve({
+				status: 'compliant',
+				reason: 'All good',
+			}),
+		});
+
+		await verify('https://github.com/test/repo', 'proj-123');
+
+		expect(mockFetch).toHaveBeenCalledWith(
+			'https://vibescope.dev/api/mcp/verify',
+			expect.objectContaining({
+				method: 'POST',
+				headers: { 'Content-Type': 'application/json' },
+				body: expect.stringContaining('test-api-key'),
+			})
+		);
+
+		const body = JSON.parse(mockFetch.mock.calls[0][1].body);
+		expect(body.git_url).toBe('https://github.com/test/repo');
+		expect(body.project_id).toBe('proj-123');
+	});
+
+	it('should return compliant status from API', async () => {
+		mockFetch.mockResolvedValueOnce({
+			json: () => Promise.resolve({
+				status: 'compliant',
+				reason: 'All tracked work completed properly',
+				details: {
+					session_started: true,
+					project_id: 'proj-123',
+					project_name: 'Test Project',
+					git_url: 'https://github.com/test/repo',
+					in_progress_tasks: 0,
+					tasks_completed_this_session: 5,
+					progress_logs_this_session: 3,
+					blockers_logged_this_session: 0,
+					session_duration_minutes: 30,
+				},
+			}),
+		});
+
+		const result = await verify('https://github.com/test/repo');
+
+		expect(result.status).toBe('compliant');
+		expect(result.details?.tasks_completed_this_session).toBe(5);
+	});
+
+	it('should return non_compliant status from API', async () => {
+		mockFetch.mockResolvedValueOnce({
+			json: () => Promise.resolve({
+				status: 'non_compliant',
+				reason: 'You have 2 task(s) still in_progress',
+				continuation_prompt: 'Complete your tasks before exiting',
+			}),
+		});
+
+		const result = await verify('https://github.com/test/repo');
+
+		expect(result.status).toBe('non_compliant');
+		expect(result.continuation_prompt).toBeDefined();
+	});
+
+	it('should handle network errors', async () => {
+		mockFetch.mockRejectedValueOnce(new Error('Network error'));
+
+		const result = await verify('https://github.com/test/repo');
+
+		expect(result.status).toBe('error');
+		expect(result.reason).toBe('Network error');
+	});
+
+	it('should auto-detect git URL if not provided', async () => {
+		vi.mocked(execSync).mockReturnValue('https://github.com/auto/detected.git');
+		mockFetch.mockResolvedValueOnce({
+			json: () => Promise.resolve({ status: 'compliant', reason: 'OK' }),
+		});
+
+		await verify();
+
+		const body = JSON.parse(mockFetch.mock.calls[0][1].body);
+		expect(body.git_url).toBe('https://github.com/auto/detected');
+	});
+
+	it('should use custom API URL from env', async () => {
+		process.env.VIBESCOPE_API_URL = 'https://custom.api.com';
+		mockFetch.mockResolvedValueOnce({
+			json: () => Promise.resolve({ status: 'compliant', reason: 'OK' }),
+		});
+
+		// Note: The module caches VIBESCOPE_API_URL at import time
+		// This test verifies the URL construction logic
+		await verify('https://github.com/test/repo');
+
+		// The actual URL used depends on when the module was loaded
+		expect(mockFetch).toHaveBeenCalled();
+	});
+});
+
+describe('VerificationResult types', () => {
+	it('should have correct status types', () => {
+		const compliant: VerificationResult = {
+			status: 'compliant',
+			reason: 'All good',
+		};
+		expect(compliant.status).toBe('compliant');
+
+		const nonCompliant: VerificationResult = {
+			status: 'non_compliant',
+			reason: 'Tasks in progress',
+			continuation_prompt: 'Complete your tasks',
+		};
+		expect(nonCompliant.status).toBe('non_compliant');
+
+		const error: VerificationResult = {
+			status: 'error',
+			reason: 'Missing env vars',
+		};
+		expect(error.status).toBe('error');
+
+		const noSession: VerificationResult = {
+			status: 'no_session',
+			reason: 'No git URL detected',
+		};
+		expect(noSession.status).toBe('no_session');
+	});
+
+	it('should support optional details', () => {
+		const result: VerificationResult = {
+			status: 'compliant',
+			reason: 'All good',
+			details: {
+				session_started: true,
+				project_id: 'proj-123',
+				project_name: 'Test Project',
+				git_url: 'https://github.com/user/repo',
+				in_progress_tasks: 0,
+				tasks_completed_this_session: 5,
+				progress_logs_this_session: 3,
+				blockers_logged_this_session: 1,
+				session_duration_minutes: 45,
+			},
+		};
+
+		expect(result.details?.session_started).toBe(true);
+		expect(result.details?.tasks_completed_this_session).toBe(5);
+	});
+});