@vibescope/mcp-server 0.1.0 → 0.2.1

package/src/handlers/git-issues.ts

@@ -9,7 +9,7 @@
  */
 
 import type { Handler, HandlerRegistry } from './types.js';
-import { validateRequired, validateUUID } from '../validators.js';
+import { parseArgs, uuidValidator, createEnumValidator } from '../validators.js';
 import { getApiClient } from '../api-client.js';
 
 const VALID_GIT_ISSUE_TYPES = [
@@ -25,41 +25,38 @@ const VALID_GIT_ISSUE_STATUSES = ['open', 'resolved'] as const;
 type GitIssueType = (typeof VALID_GIT_ISSUE_TYPES)[number];
 type GitIssueStatus = (typeof VALID_GIT_ISSUE_STATUSES)[number];
 
-export const addGitIssue: Handler = async (args, ctx) => {
-	const {
-		project_id,
-		issue_type,
-		branch,
-		target_branch,
-		pr_url,
-		conflicting_files,
-		error_message,
-		task_id,
-	} = args as {
-		project_id: string;
-		issue_type: string;
-		branch: string;
-		target_branch?: string;
-		pr_url?: string;
-		conflicting_files?: string[];
-		error_message?: string;
-		task_id?: string;
-	};
-
-	validateRequired(project_id, 'project_id');
-	validateUUID(project_id, 'project_id');
-	validateRequired(issue_type, 'issue_type');
-	validateRequired(branch, 'branch');
-
-	if (!VALID_GIT_ISSUE_TYPES.includes(issue_type as GitIssueType)) {
-		throw new Error(
-			`Invalid issue_type. Valid types: ${VALID_GIT_ISSUE_TYPES.join(', ')}`
-		);
-	}
+// Argument schemas for type-safe parsing
+const addGitIssueSchema = {
+	project_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	issue_type: { type: 'string' as const, required: true as const, validate: createEnumValidator(VALID_GIT_ISSUE_TYPES) },
+	branch: { type: 'string' as const, required: true as const },
+	target_branch: { type: 'string' as const },
+	pr_url: { type: 'string' as const },
+	conflicting_files: { type: 'array' as const },
+	error_message: { type: 'string' as const },
+	task_id: { type: 'string' as const, validate: uuidValidator },
+};
 
-	if (task_id) {
-		validateUUID(task_id, 'task_id');
-	}
+const resolveGitIssueSchema = {
+	git_issue_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	resolution_note: { type: 'string' as const },
+	auto_resolved: { type: 'boolean' as const },
+};
+
+const getGitIssuesSchema = {
+	project_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	status: { type: 'string' as const, default: 'open', validate: createEnumValidator(VALID_GIT_ISSUE_STATUSES) },
+	issue_type: { type: 'string' as const, validate: createEnumValidator(VALID_GIT_ISSUE_TYPES) },
+	branch: { type: 'string' as const },
+	limit: { type: 'number' as const, default: 50 },
+};
+
+const deleteGitIssueSchema = {
+	git_issue_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+};
+
+export const addGitIssue: Handler = async (args, ctx) => {
+	const { project_id, issue_type, branch, target_branch, pr_url, conflicting_files, error_message, task_id } = parseArgs(args, addGitIssueSchema);
 
 	const apiClient = getApiClient();
 	const response = await apiClient.addGitIssue(project_id, {
@@ -67,7 +64,7 @@ export const addGitIssue: Handler = async (args, ctx) => {
 		branch,
 		target_branch,
 		pr_url,
-		conflicting_files,
+		conflicting_files: conflicting_files as string[] | undefined,
 		error_message,
 		task_id
 	}, ctx.session.currentSessionId || undefined);
@@ -80,14 +77,7 @@ export const addGitIssue: Handler = async (args, ctx) => {
 };
 
 export const resolveGitIssue: Handler = async (args, ctx) => {
-	const { git_issue_id, resolution_note, auto_resolved } = args as {
-		git_issue_id: string;
-		resolution_note?: string;
-		auto_resolved?: boolean;
-	};
-
-	validateRequired(git_issue_id, 'git_issue_id');
-	validateUUID(git_issue_id, 'git_issue_id');
+	const { git_issue_id, resolution_note, auto_resolved } = parseArgs(args, resolveGitIssueSchema);
 
 	const apiClient = getApiClient();
 	const response = await apiClient.resolveGitIssue(git_issue_id, {
@@ -102,40 +92,13 @@ export const resolveGitIssue: Handler = async (args, ctx) => {
 	return { result: response.data };
 };
 
-export const getGitIssues: Handler = async (args, ctx) => {
-	const {
-		project_id,
-		status = 'open',
-		issue_type,
-		branch,
-		limit = 50,
-	} = args as {
-		project_id: string;
-		status?: string;
-		issue_type?: string;
-		branch?: string;
-		limit?: number;
-	};
-
-	validateRequired(project_id, 'project_id');
-	validateUUID(project_id, 'project_id');
-
-	if (status && !VALID_GIT_ISSUE_STATUSES.includes(status as GitIssueStatus)) {
-		throw new Error(
-			`Invalid status. Valid statuses: ${VALID_GIT_ISSUE_STATUSES.join(', ')}`
-		);
-	}
-
-	if (issue_type && !VALID_GIT_ISSUE_TYPES.includes(issue_type as GitIssueType)) {
-		throw new Error(
-			`Invalid issue_type. Valid types: ${VALID_GIT_ISSUE_TYPES.join(', ')}`
-		);
-	}
+export const getGitIssues: Handler = async (args, _ctx) => {
+	const { project_id, status, issue_type, branch, limit } = parseArgs(args, getGitIssuesSchema);
 
 	const apiClient = getApiClient();
 	const response = await apiClient.getGitIssues(project_id, {
-		status,
-		issue_type,
+		status: status as GitIssueStatus | undefined,
+		issue_type: issue_type as GitIssueType | undefined,
 		branch,
 		limit
 	});
@@ -147,11 +110,8 @@ export const getGitIssues: Handler = async (args, ctx) => {
 	return { result: response.data };
 };
 
-export const deleteGitIssue: Handler = async (args, ctx) => {
-	const { git_issue_id } = args as { git_issue_id: string };
-
-	validateRequired(git_issue_id, 'git_issue_id');
-	validateUUID(git_issue_id, 'git_issue_id');
+export const deleteGitIssue: Handler = async (args, _ctx) => {
+	const { git_issue_id } = parseArgs(args, deleteGitIssueSchema);
 
 	const apiClient = getApiClient();
 	const response = await apiClient.deleteGitIssue(git_issue_id);

package/src/handlers/ideas.ts

@@ -12,22 +12,55 @@
  */
 
 import type { Handler, HandlerRegistry } from './types.js';
-import { validateRequired, validateUUID, validatePriority, validateEstimatedMinutes } from '../validators.js';
+import {
+	parseArgs,
+	uuidValidator,
+	priorityValidator,
+	minutesValidator,
+	createEnumValidator,
+} from '../validators.js';
 import { getApiClient } from '../api-client.js';
 
-type IdeaStatus = 'raw' | 'exploring' | 'planned' | 'in_development' | 'shipped';
+const VALID_IDEA_STATUSES = ['raw', 'exploring', 'planned', 'in_development', 'shipped'] as const;
+type IdeaStatus = typeof VALID_IDEA_STATUSES[number];
 
-export const addIdea: Handler = async (args, ctx) => {
-	const { project_id, title, description, status } = args as {
-		project_id: string;
-		title: string;
-		description?: string;
-		status?: IdeaStatus;
-	};
+// Argument schemas for type-safe parsing
+const addIdeaSchema = {
+	project_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	title: { type: 'string' as const, required: true as const },
+	description: { type: 'string' as const },
+	status: { type: 'string' as const, validate: createEnumValidator(VALID_IDEA_STATUSES) },
+};
+
+const updateIdeaSchema = {
+	idea_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	title: { type: 'string' as const },
+	description: { type: 'string' as const },
+	status: { type: 'string' as const, validate: createEnumValidator(VALID_IDEA_STATUSES) },
+	doc_url: { type: 'string' as const },
+};
 
-	validateRequired(project_id, 'project_id');
-	validateUUID(project_id, 'project_id');
-	validateRequired(title, 'title');
+const getIdeasSchema = {
+	project_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	status: { type: 'string' as const, validate: createEnumValidator(VALID_IDEA_STATUSES) },
+	limit: { type: 'number' as const, default: 50 },
+	offset: { type: 'number' as const, default: 0 },
+	search_query: { type: 'string' as const },
+};
+
+const deleteIdeaSchema = {
+	idea_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+};
+
+const convertIdeaToTaskSchema = {
+	idea_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	priority: { type: 'number' as const, default: 3, validate: priorityValidator },
+	estimated_minutes: { type: 'number' as const, validate: minutesValidator },
+	update_status: { type: 'boolean' as const, default: true },
+};
+
+export const addIdea: Handler = async (args, ctx) => {
+	const { project_id, title, description, status } = parseArgs(args, addIdeaSchema);
 
 	const { session } = ctx;
 	const apiClient = getApiClient();
@@ -35,7 +68,7 @@ export const addIdea: Handler = async (args, ctx) => {
 	const response = await apiClient.addIdea(project_id, {
 		title,
 		description,
-		status
+		status: status as IdeaStatus | undefined
 	}, session.currentSessionId || undefined);
 
 	if (!response.ok) {
@@ -45,24 +78,15 @@ export const addIdea: Handler = async (args, ctx) => {
 	return { result: { success: true, idea_id: response.data?.idea_id, title } };
 };
 
-export const updateIdea: Handler = async (args, ctx) => {
-	const { idea_id, title, description, status, doc_url } = args as {
-		idea_id: string;
-		title?: string;
-		description?: string;
-		status?: IdeaStatus;
-		doc_url?: string;
-	};
-
-	validateRequired(idea_id, 'idea_id');
-	validateUUID(idea_id, 'idea_id');
+export const updateIdea: Handler = async (args, _ctx) => {
+	const { idea_id, title, description, status, doc_url } = parseArgs(args, updateIdeaSchema);
 
 	const apiClient = getApiClient();
 
 	const response = await apiClient.updateIdea(idea_id, {
 		title,
 		description,
-		status,
+		status: status as IdeaStatus | undefined,
 		doc_url
 	});
 
@@ -73,22 +97,13 @@ export const updateIdea: Handler = async (args, ctx) => {
 	return { result: { success: true, idea_id } };
 };
 
-export const getIdeas: Handler = async (args, ctx) => {
-	const { project_id, status, limit = 50, offset = 0, search_query } = args as {
-		project_id: string;
-		status?: IdeaStatus;
-		limit?: number;
-		offset?: number;
-		search_query?: string;
-	};
-
-	validateRequired(project_id, 'project_id');
-	validateUUID(project_id, 'project_id');
+export const getIdeas: Handler = async (args, _ctx) => {
+	const { project_id, status, limit, offset, search_query } = parseArgs(args, getIdeasSchema);
 
 	const apiClient = getApiClient();
 
 	const response = await apiClient.getIdeas(project_id, {
-		status,
+		status: status as IdeaStatus | undefined,
 		limit,
 		offset,
 		search_query
@@ -105,11 +120,8 @@ export const getIdeas: Handler = async (args, ctx) => {
 	};
 };
 
-export const deleteIdea: Handler = async (args, ctx) => {
-	const { idea_id } = args as { idea_id: string };
-
-	validateRequired(idea_id, 'idea_id');
-	validateUUID(idea_id, 'idea_id');
+export const deleteIdea: Handler = async (args, _ctx) => {
+	const { idea_id } = parseArgs(args, deleteIdeaSchema);
 
 	const apiClient = getApiClient();
 
@@ -122,18 +134,8 @@ export const deleteIdea: Handler = async (args, ctx) => {
 	return { result: { success: true } };
 };
 
-export const convertIdeaToTask: Handler = async (args, ctx) => {
-	const { idea_id, priority = 3, estimated_minutes, update_status = true } = args as {
-		idea_id: string;
-		priority?: number;
-		estimated_minutes?: number;
-		update_status?: boolean;
-	};
-
-	validateRequired(idea_id, 'idea_id');
-	validateUUID(idea_id, 'idea_id');
-	validatePriority(priority);
-	validateEstimatedMinutes(estimated_minutes);
+export const convertIdeaToTask: Handler = async (args, _ctx) => {
+	const { idea_id, priority, estimated_minutes, update_status } = parseArgs(args, convertIdeaToTaskSchema);
 
 	const apiClient = getApiClient();
 

package/src/handlers/index.ts

@@ -25,6 +25,8 @@ export * from './organizations.js';
 export * from './cost.js';
 export * from './git-issues.js';
 export * from './sprints.js';
+export * from './file-checkouts.js';
+export * from './roles.js';
 
 import type { HandlerRegistry } from './types.js';
 import { milestoneHandlers } from './milestones.js';
@@ -46,6 +48,8 @@ import { organizationHandlers } from './organizations.js';
 import { costHandlers } from './cost.js';
 import { gitIssueHandlers } from './git-issues.js';
 import { sprintHandlers } from './sprints.js';
+import { fileCheckoutHandlers } from './file-checkouts.js';
+import { roleHandlers } from './roles.js';
 
 /**
  * Build the complete handler registry from all modules
@@ -71,5 +75,7 @@ export function buildHandlerRegistry(): HandlerRegistry {
 		...costHandlers,
 		...gitIssueHandlers,
 		...sprintHandlers,
+		...fileCheckoutHandlers,
+		...roleHandlers,
 	};
 }

package/src/handlers/milestones.test.ts

@@ -143,7 +143,7 @@ describe('updateMilestone', () => {
 				milestone_id: '123e4567-e89b-12d3-a456-426614174000',
 				status: 'invalid_status',
 			}, ctx)
-		).rejects.toThrow('status must be pending, in_progress, or completed');
+		).rejects.toThrow('Invalid status: "invalid_status"');
 	});
 
 	it('should update title successfully', async () => {

package/src/handlers/milestones.ts

@@ -12,20 +12,47 @@
  */
 
 import type { Handler, HandlerRegistry } from './types.js';
-import { ValidationError, validateRequired, validateUUID } from '../validators.js';
+import {
+	parseArgs,
+	uuidValidator,
+	createEnumValidator,
+	ValidationError,
+} from '../validators.js';
 import { getApiClient } from '../api-client.js';
 
-export const addMilestone: Handler = async (args, ctx) => {
-	const { task_id, title, description, order_index } = args as {
-		task_id: string;
-		title: string;
-		description?: string;
-		order_index?: number;
-	};
+const VALID_MILESTONE_STATUSES = ['pending', 'in_progress', 'completed'] as const;
+type MilestoneStatus = typeof VALID_MILESTONE_STATUSES[number];
+
+// Argument schemas for type-safe parsing
+const addMilestoneSchema = {
+	task_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	title: { type: 'string' as const, required: true as const },
+	description: { type: 'string' as const },
+	order_index: { type: 'number' as const },
+};
+
+const updateMilestoneSchema = {
+	milestone_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	title: { type: 'string' as const },
+	description: { type: 'string' as const },
+	status: { type: 'string' as const, validate: createEnumValidator(VALID_MILESTONE_STATUSES) },
+	order_index: { type: 'number' as const },
+};
+
+const completeMilestoneSchema = {
+	milestone_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+};
+
+const deleteMilestoneSchema = {
+	milestone_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+};
 
-	validateRequired(task_id, 'task_id');
-	validateUUID(task_id, 'task_id');
-	validateRequired(title, 'title');
+const getMilestonesSchema = {
+	task_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+};
+
+export const addMilestone: Handler = async (args, ctx) => {
+	const { task_id, title, description, order_index } = parseArgs(args, addMilestoneSchema);
 
 	const { session } = ctx;
 	const apiClient = getApiClient();
@@ -48,24 +75,8 @@ export const addMilestone: Handler = async (args, ctx) => {
 	};
 };
 
-export const updateMilestone: Handler = async (args, ctx) => {
-	const { milestone_id, title, description, status, order_index } = args as {
-		milestone_id: string;
-		title?: string;
-		description?: string;
-		status?: string;
-		order_index?: number;
-	};
-
-	validateRequired(milestone_id, 'milestone_id');
-	validateUUID(milestone_id, 'milestone_id');
-
-	// Validate status if provided
-	if (status !== undefined) {
-		if (!['pending', 'in_progress', 'completed'].includes(status)) {
-			throw new ValidationError('status must be pending, in_progress, or completed');
-		}
-	}
+export const updateMilestone: Handler = async (args, _ctx) => {
+	const { milestone_id, title, description, status, order_index } = parseArgs(args, updateMilestoneSchema);
 
 	// Check that at least one field is provided
 	if (title === undefined && description === undefined && status === undefined && order_index === undefined) {
@@ -77,7 +88,7 @@ export const updateMilestone: Handler = async (args, ctx) => {
 	const response = await apiClient.updateMilestone(milestone_id, {
 		title,
 		description,
-		status: status as 'pending' | 'in_progress' | 'completed' | undefined,
+		status: status as MilestoneStatus | undefined,
 		order_index
 	});
 
@@ -93,11 +104,8 @@ export const updateMilestone: Handler = async (args, ctx) => {
 	};
 };
 
-export const completeMilestone: Handler = async (args, ctx) => {
-	const { milestone_id } = args as { milestone_id: string };
-
-	validateRequired(milestone_id, 'milestone_id');
-	validateUUID(milestone_id, 'milestone_id');
+export const completeMilestone: Handler = async (args, _ctx) => {
+	const { milestone_id } = parseArgs(args, completeMilestoneSchema);
 
 	const apiClient = getApiClient();
 
@@ -115,11 +123,8 @@ export const completeMilestone: Handler = async (args, ctx) => {
 	};
 };
 
-export const deleteMilestone: Handler = async (args, ctx) => {
-	const { milestone_id } = args as { milestone_id: string };
-
-	validateRequired(milestone_id, 'milestone_id');
-	validateUUID(milestone_id, 'milestone_id');
+export const deleteMilestone: Handler = async (args, _ctx) => {
+	const { milestone_id } = parseArgs(args, deleteMilestoneSchema);
 
 	const apiClient = getApiClient();
 
@@ -137,11 +142,8 @@ export const deleteMilestone: Handler = async (args, ctx) => {
 	};
 };
 
-export const getMilestones: Handler = async (args, ctx) => {
-	const { task_id } = args as { task_id: string };
-
-	validateRequired(task_id, 'task_id');
-	validateUUID(task_id, 'task_id');
+export const getMilestones: Handler = async (args, _ctx) => {
+	const { task_id } = parseArgs(args, getMilestonesSchema);
 
 	const apiClient = getApiClient();