@vibescope/mcp-server 0.2.3 → 0.2.5

package/src/handlers/blockers.ts

@@ -1,109 +1,163 @@
-/**
- * Blockers Handlers
- *
- * Handles blocker management:
- * - add_blocker
- * - resolve_blocker
- * - get_blockers
- * - delete_blocker
- */
-
-import type { Handler, HandlerRegistry } from './types.js';
-import { success, error } from './types.js';
-import {
-	parseArgs,
-	uuidValidator,
-	createEnumValidator,
-	VALID_BLOCKER_STATUSES,
-} from '../validators.js';
-import { getApiClient } from '../api-client.js';
-
-// Argument schemas for type-safe parsing
-const addBlockerSchema = {
-	project_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
-	description: { type: 'string' as const, required: true as const },
-};
-
-const resolveBlockerSchema = {
-	blocker_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
-	resolution_note: { type: 'string' as const },
-};
-
-const getBlockersSchema = {
-	project_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
-	status: { type: 'string' as const, default: 'open', validate: createEnumValidator(VALID_BLOCKER_STATUSES) },
-	limit: { type: 'number' as const, default: 50 },
-	offset: { type: 'number' as const, default: 0 },
-	search_query: { type: 'string' as const },
-};
-
-const deleteBlockerSchema = {
-	blocker_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
-};
-
-export const addBlocker: Handler = async (args, ctx) => {
-	const { project_id, description } = parseArgs(args, addBlockerSchema);
-
-	const apiClient = getApiClient();
-	const response = await apiClient.addBlocker(project_id, description, ctx.session.currentSessionId || undefined);
-
-	if (!response.ok) {
-		return error(response.error || 'Failed to add blocker');
-	}
-
-	return success(response.data);
-};
-
-export const resolveBlocker: Handler = async (args, _ctx) => {
-	const { blocker_id, resolution_note } = parseArgs(args, resolveBlockerSchema);
-
-	const apiClient = getApiClient();
-	const response = await apiClient.resolveBlocker(blocker_id, resolution_note);
-
-	if (!response.ok) {
-		return error(response.error || 'Failed to resolve blocker');
-	}
-
-	return success(response.data);
-};
-
-export const getBlockers: Handler = async (args, _ctx) => {
-	const { project_id, status, limit, offset, search_query } = parseArgs(args, getBlockersSchema);
-
-	const apiClient = getApiClient();
-	const response = await apiClient.getBlockers(project_id, {
-		status,
-		limit,
-		offset,
-		search_query
-	});
-
-	if (!response.ok) {
-		return error(response.error || 'Failed to fetch blockers');
-	}
-
-	return success(response.data);
-};
-
-export const deleteBlocker: Handler = async (args, _ctx) => {
-	const { blocker_id } = parseArgs(args, deleteBlockerSchema);
-
-	const apiClient = getApiClient();
-	const response = await apiClient.deleteBlocker(blocker_id);
-
-	if (!response.ok) {
-		return error(response.error || 'Failed to delete blocker');
-	}
-
-	return success(response.data);
-};
-
-/**
- * Blockers handlers registry
- */
-export const blockerHandlers: HandlerRegistry = {
-	add_blocker: addBlocker,
-	resolve_blocker: resolveBlocker,
-	get_blockers: getBlockers,
-	delete_blocker: deleteBlocker,
-};
+/**
+ * Blockers Handlers
+ *
+ * Handles blocker management:
+ * - add_blocker
+ * - resolve_blocker
+ * - get_blockers
+ * - delete_blocker
+ */
+
+import type { Handler, HandlerRegistry } from './types.js';
+import { success, error } from './types.js';
+import {
+	parseArgs,
+	uuidValidator,
+	createEnumValidator,
+	VALID_BLOCKER_STATUSES,
+} from '../validators.js';
+import { getApiClient } from '../api-client.js';
+
+// Argument schemas for type-safe parsing
+const addBlockerSchema = {
+	project_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	description: { type: 'string' as const, required: true as const },
+};
+
+const resolveBlockerSchema = {
+	blocker_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	resolution_note: { type: 'string' as const },
+};
+
+const getBlockerSchema = {
+	blocker_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+};
+
+const getBlockersSchema = {
+	project_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	status: { type: 'string' as const, default: 'open', validate: createEnumValidator(VALID_BLOCKER_STATUSES) },
+	limit: { type: 'number' as const, default: 10 },
+	offset: { type: 'number' as const, default: 0 },
+	search_query: { type: 'string' as const },
+};
+
+const deleteBlockerSchema = {
+	blocker_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+};
+
+const getBlockersStatsSchema = {
+	project_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+};
+
+export const addBlocker: Handler = async (args, ctx) => {
+	const { project_id, description } = parseArgs(args, addBlockerSchema);
+
+	const apiClient = getApiClient();
+	const response = await apiClient.addBlocker(project_id, description, ctx.session.currentSessionId || undefined);
+
+	if (!response.ok) {
+		return error(response.error || 'Failed to add blocker');
+	}
+
+	return success(response.data);
+};
+
+export const resolveBlocker: Handler = async (args, _ctx) => {
+	const { blocker_id, resolution_note } = parseArgs(args, resolveBlockerSchema);
+
+	const apiClient = getApiClient();
+	const response = await apiClient.resolveBlocker(blocker_id, resolution_note);
+
+	if (!response.ok) {
+		return error(response.error || 'Failed to resolve blocker');
+	}
+
+	return success(response.data);
+};
+
+/**
+ * Get a single blocker by ID.
+ * More token-efficient than get_blockers when you need details for a specific blocker.
+ */
+export const getBlocker: Handler = async (args, _ctx) => {
+	const { blocker_id } = parseArgs(args, getBlockerSchema);
+
+	const apiClient = getApiClient();
+	const response = await apiClient.proxy<{
+		blocker: {
+			id: string;
+			description: string;
+			status: string;
+			resolution_note?: string;
+			created_at: string;
+			resolved_at?: string;
+		};
+	}>('get_blocker', { blocker_id });
+
+	if (!response.ok) {
+		return error(response.error || 'Failed to get blocker');
+	}
+
+	return success(response.data);
+};
+
+export const getBlockers: Handler = async (args, _ctx) => {
+	const { project_id, status, limit, offset, search_query } = parseArgs(args, getBlockersSchema);
+
+	const apiClient = getApiClient();
+	const response = await apiClient.getBlockers(project_id, {
+		status,
+		limit: Math.min(limit ?? 10, 200),
+		offset,
+		search_query
+	});
+
+	if (!response.ok) {
+		return error(response.error || 'Failed to fetch blockers');
+	}
+
+	return success(response.data);
+};
+
+export const deleteBlocker: Handler = async (args, _ctx) => {
+	const { blocker_id } = parseArgs(args, deleteBlockerSchema);
+
+	const apiClient = getApiClient();
+	const response = await apiClient.deleteBlocker(blocker_id);
+
+	if (!response.ok) {
+		return error(response.error || 'Failed to delete blocker');
+	}
+
+	return success(response.data);
+};
+
+/**
+ * Get aggregate statistics about blockers for a project.
+ * Returns total count and breakdown by status without the actual blocker data.
+ * This is much more token-efficient than get_blockers for understanding the overall state.
+ */
+export const getBlockersStats: Handler = async (args, _ctx) => {
+	const { project_id } = parseArgs(args, getBlockersStatsSchema);
+
+	const apiClient = getApiClient();
+	const response = await apiClient.getBlockersStats(project_id);
+
+	if (!response.ok) {
+		return error(response.error || 'Failed to get blockers stats');
+	}
+
+	return success(response.data);
+};
+
+/**
+ * Blockers handlers registry
+ */
+export const blockerHandlers: HandlerRegistry = {
+	add_blocker: addBlocker,
+	resolve_blocker: resolveBlocker,
+	get_blocker: getBlocker,
+	get_blockers: getBlockers,
+	get_blockers_stats: getBlockersStats,
+	delete_blocker: deleteBlocker,
+};