@vibescope/mcp-server 0.2.0 → 0.2.1

package/src/handlers/sprints.ts

@@ -16,18 +16,92 @@
  */
 
 import type { Handler, HandlerRegistry } from './types.js';
-import { validateRequired, validateUUID, validateEnum } from '../validators.js';
+import { parseArgs, uuidValidator, createEnumValidator } from '../validators.js';
 import { getApiClient } from '../api-client.js';
 
-type SprintStatus = 'planning' | 'active' | 'in_review' | 'retrospective' | 'completed' | 'cancelled';
-type TaskPhase = 'pre' | 'core' | 'post';
-type DeployEnvironment = 'development' | 'staging' | 'production';
-type VersionBump = 'patch' | 'minor' | 'major';
+const SPRINT_STATUSES = ['planning', 'active', 'in_review', 'retrospective', 'completed', 'cancelled'] as const;
+const TASK_PHASES = ['pre', 'core', 'post'] as const;
+const DEPLOY_ENVIRONMENTS = ['development', 'staging', 'production'] as const;
+const VERSION_BUMPS = ['patch', 'minor', 'major'] as const;
+
+type SprintStatus = typeof SPRINT_STATUSES[number];
+type TaskPhase = typeof TASK_PHASES[number];
+type DeployEnvironment = typeof DEPLOY_ENVIRONMENTS[number];
+type VersionBump = typeof VERSION_BUMPS[number];
+
+// ============================================================================
+// Argument Schemas
+// ============================================================================
+
+const createSprintSchema = {
+	project_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	title: { type: 'string' as const, required: true as const },
+	goal: { type: 'string' as const },
+	start_date: { type: 'string' as const, required: true as const },
+	end_date: { type: 'string' as const, required: true as const },
+	auto_deploy_on_completion: { type: 'boolean' as const },
+	deploy_environment: { type: 'string' as const, validate: createEnumValidator(DEPLOY_ENVIRONMENTS) },
+	deploy_version_bump: { type: 'string' as const, validate: createEnumValidator(VERSION_BUMPS) },
+};
+
+const updateSprintSchema = {
+	sprint_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	title: { type: 'string' as const },
+	goal: { type: 'string' as const },
+	start_date: { type: 'string' as const },
+	end_date: { type: 'string' as const },
+	auto_deploy_on_completion: { type: 'boolean' as const },
+	deploy_environment: { type: 'string' as const, validate: createEnumValidator(DEPLOY_ENVIRONMENTS) },
+	deploy_version_bump: { type: 'string' as const, validate: createEnumValidator(VERSION_BUMPS) },
+};
+
+const getSprintSchema = {
+	sprint_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	summary_only: { type: 'boolean' as const, default: false },
+};
+
+const getSprintsSchema = {
+	project_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	status: { type: 'string' as const, validate: createEnumValidator(SPRINT_STATUSES) },
+	limit: { type: 'number' as const, default: 20 },
+	offset: { type: 'number' as const, default: 0 },
+};
+
+const deleteSprintSchema = {
+	sprint_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+};
+
+const startSprintSchema = {
+	sprint_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+};
+
+const completeSprintSchema = {
+	sprint_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	retrospective_notes: { type: 'string' as const },
+	skip_retrospective: { type: 'boolean' as const },
+};
+
+const addTaskToSprintSchema = {
+	sprint_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	task_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	story_points: { type: 'number' as const },
+	phase: { type: 'string' as const, validate: createEnumValidator(TASK_PHASES) },
+};
+
+const removeTaskFromSprintSchema = {
+	sprint_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	task_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+};
+
+const getSprintBacklogSchema = {
+	project_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	sprint_id: { type: 'string' as const, validate: uuidValidator },
+};
 
-const SPRINT_STATUSES: SprintStatus[] = ['planning', 'active', 'in_review', 'retrospective', 'completed', 'cancelled'];
-const TASK_PHASES: TaskPhase[] = ['pre', 'core', 'post'];
-const DEPLOY_ENVIRONMENTS: DeployEnvironment[] = ['development', 'staging', 'production'];
-const VERSION_BUMPS: VersionBump[] = ['patch', 'minor', 'major'];
+const getSprintVelocitySchema = {
+	project_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	limit: { type: 'number' as const, default: 10 },
+};
 
 export const createSprint: Handler = async (args, ctx) => {
 	const {
@@ -39,29 +113,7 @@ export const createSprint: Handler = async (args, ctx) => {
 		auto_deploy_on_completion,
 		deploy_environment,
 		deploy_version_bump,
-	} = args as {
-		project_id: string;
-		title: string;
-		goal?: string;
-		start_date: string;
-		end_date: string;
-		auto_deploy_on_completion?: boolean;
-		deploy_environment?: DeployEnvironment;
-		deploy_version_bump?: VersionBump;
-	};
-
-	validateRequired(project_id, 'project_id');
-	validateUUID(project_id, 'project_id');
-	validateRequired(title, 'title');
-	validateRequired(start_date, 'start_date');
-	validateRequired(end_date, 'end_date');
-
-	if (deploy_environment) {
-		validateEnum(deploy_environment, DEPLOY_ENVIRONMENTS, 'deploy_environment');
-	}
-	if (deploy_version_bump) {
-		validateEnum(deploy_version_bump, VERSION_BUMPS, 'deploy_version_bump');
-	}
+	} = parseArgs(args, createSprintSchema);
 
 	// Validate date format and order
 	const startDateObj = new Date(start_date);
@@ -126,26 +178,7 @@ export const updateSprint: Handler = async (args, ctx) => {
 		auto_deploy_on_completion,
 		deploy_environment,
 		deploy_version_bump,
-	} = args as {
-		sprint_id: string;
-		title?: string;
-		goal?: string;
-		start_date?: string;
-		end_date?: string;
-		auto_deploy_on_completion?: boolean;
-		deploy_environment?: DeployEnvironment;
-		deploy_version_bump?: VersionBump;
-	};
-
-	validateRequired(sprint_id, 'sprint_id');
-	validateUUID(sprint_id, 'sprint_id');
-
-	if (deploy_environment) {
-		validateEnum(deploy_environment, DEPLOY_ENVIRONMENTS, 'deploy_environment');
-	}
-	if (deploy_version_bump) {
-		validateEnum(deploy_version_bump, VERSION_BUMPS, 'deploy_version_bump');
-	}
+	} = parseArgs(args, updateSprintSchema);
 
 	// Validate dates if provided
 	if (start_date) {
@@ -182,10 +215,7 @@ export const updateSprint: Handler = async (args, ctx) => {
 };
 
 export const getSprint: Handler = async (args, ctx) => {
-	const { sprint_id, summary_only = false } = args as { sprint_id: string; summary_only?: boolean };
-
-	validateRequired(sprint_id, 'sprint_id');
-	validateUUID(sprint_id, 'sprint_id');
+	const { sprint_id, summary_only } = parseArgs(args, getSprintSchema);
 
 	const apiClient = getApiClient();
 
@@ -227,19 +257,7 @@ export const getSprint: Handler = async (args, ctx) => {
 };
 
 export const getSprints: Handler = async (args, ctx) => {
-	const { project_id, status, limit = 20, offset = 0 } = args as {
-		project_id: string;
-		status?: SprintStatus;
-		limit?: number;
-		offset?: number;
-	};
-
-	validateRequired(project_id, 'project_id');
-	validateUUID(project_id, 'project_id');
-
-	if (status) {
-		validateEnum(status, SPRINT_STATUSES, 'status');
-	}
+	const { project_id, status, limit, offset } = parseArgs(args, getSprintsSchema);
 
 	const apiClient = getApiClient();
 
@@ -261,7 +279,7 @@ export const getSprints: Handler = async (args, ctx) => {
 	}>('get_sprints', {
 		project_id,
 		status,
-		limit: Math.min(limit, 100),
+		limit: Math.min(limit ?? 20, 100),
 		offset,
 	});
 
@@ -273,10 +291,7 @@ export const getSprints: Handler = async (args, ctx) => {
 };
 
 export const deleteSprint: Handler = async (args, ctx) => {
-	const { sprint_id } = args as { sprint_id: string };
-
-	validateRequired(sprint_id, 'sprint_id');
-	validateUUID(sprint_id, 'sprint_id');
+	const { sprint_id } = parseArgs(args, deleteSprintSchema);
 
 	const apiClient = getApiClient();
 
@@ -292,10 +307,7 @@ export const deleteSprint: Handler = async (args, ctx) => {
 };
 
 export const startSprint: Handler = async (args, ctx) => {
-	const { sprint_id } = args as { sprint_id: string };
-
-	validateRequired(sprint_id, 'sprint_id');
-	validateUUID(sprint_id, 'sprint_id');
+	const { sprint_id } = parseArgs(args, startSprintSchema);
 
 	const apiClient = getApiClient();
 
@@ -315,14 +327,7 @@ export const startSprint: Handler = async (args, ctx) => {
 };
 
 export const completeSprint: Handler = async (args, ctx) => {
-	const { sprint_id, retrospective_notes, skip_retrospective } = args as {
-		sprint_id: string;
-		retrospective_notes?: string;
-		skip_retrospective?: boolean;
-	};
-
-	validateRequired(sprint_id, 'sprint_id');
-	validateUUID(sprint_id, 'sprint_id');
+	const { sprint_id, retrospective_notes, skip_retrospective } = parseArgs(args, completeSprintSchema);
 
 	const apiClient = getApiClient();
 
@@ -348,21 +353,7 @@ export const completeSprint: Handler = async (args, ctx) => {
 };
 
 export const addTaskToSprint: Handler = async (args, ctx) => {
-	const { sprint_id, task_id, story_points, phase } = args as {
-		sprint_id: string;
-		task_id: string;
-		story_points?: number;
-		phase?: TaskPhase;
-	};
-
-	validateRequired(sprint_id, 'sprint_id');
-	validateUUID(sprint_id, 'sprint_id');
-	validateRequired(task_id, 'task_id');
-	validateUUID(task_id, 'task_id');
-
-	if (phase) {
-		validateEnum(phase, TASK_PHASES, 'phase');
-	}
+	const { sprint_id, task_id, story_points, phase } = parseArgs(args, addTaskToSprintSchema);
 
 	if (story_points !== undefined && (story_points < 0 || !Number.isInteger(story_points))) {
 		throw new Error('story_points must be a non-negative integer');
@@ -392,15 +383,7 @@ export const addTaskToSprint: Handler = async (args, ctx) => {
 };
 
 export const removeTaskFromSprint: Handler = async (args, ctx) => {
-	const { sprint_id, task_id } = args as {
-		sprint_id: string;
-		task_id: string;
-	};
-
-	validateRequired(sprint_id, 'sprint_id');
-	validateUUID(sprint_id, 'sprint_id');
-	validateRequired(task_id, 'task_id');
-	validateUUID(task_id, 'task_id');
+	const { sprint_id, task_id } = parseArgs(args, removeTaskFromSprintSchema);
 
 	const apiClient = getApiClient();
 
@@ -423,17 +406,7 @@ export const removeTaskFromSprint: Handler = async (args, ctx) => {
 };
 
 export const getSprintBacklog: Handler = async (args, ctx) => {
-	const { project_id, sprint_id } = args as {
-		project_id: string;
-		sprint_id?: string;
-	};
-
-	validateRequired(project_id, 'project_id');
-	validateUUID(project_id, 'project_id');
-
-	if (sprint_id) {
-		validateUUID(sprint_id, 'sprint_id');
-	}
+	const { project_id, sprint_id } = parseArgs(args, getSprintBacklogSchema);
 
 	const apiClient = getApiClient();
 
@@ -460,13 +433,7 @@ export const getSprintBacklog: Handler = async (args, ctx) => {
 };
 
 export const getSprintVelocity: Handler = async (args, ctx) => {
-	const { project_id, limit = 10 } = args as {
-		project_id: string;
-		limit?: number;
-	};
-
-	validateRequired(project_id, 'project_id');
-	validateUUID(project_id, 'project_id');
+	const { project_id, limit } = parseArgs(args, getSprintVelocitySchema);
 
 	const apiClient = getApiClient();
 
@@ -482,7 +449,7 @@ export const getSprintVelocity: Handler = async (args, ctx) => {
 		total_sprints: number;
 	}>('get_sprint_velocity', {
 		project_id,
-		limit: Math.min(limit, 50),
+		limit: Math.min(limit ?? 10, 50),
 	});
 
 	if (!response.ok) {