@elixium.ai/mcp-server 0.2.1 → 0.2.2

package/dist/index.js

@@ -10,6 +10,14 @@ const API_URL = process.env.ELIXIUM_API_URL || "https://elixium.ai/api";
 const BOARD_SLUG = process.env.ELIXIUM_BOARD_SLUG;
 const LANE_STYLE_ENV = process.env.ELIXIUM_LANE_STYLE;
 const USER_EMAIL = process.env.ELIXIUM_USER_EMAIL; // Optional: Override requester email for stories
+// UUID v4 format validation — prevents 500s from partial/truncated IDs
+const UUID_RE = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;
+function assertUUID(value, label) {
+    if (!UUID_RE.test(value)) {
+        throw new Error(`Invalid ${label}: "${value}" is not a valid UUID. ` +
+            `Expected format: xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx (full UUID from list_stories/list_epics).`);
+    }
+}
 const CLI_ARGS = process.argv.slice(2);
 const hasArg = (flag) => CLI_ARGS.includes(flag);
 const getArgValue = (flag) => {
@@ -604,6 +612,20 @@ const createServer = () => {
                     required: ["storyId", "testPlan"],
                 },
             },
+            {
+                name: "approve_tests",
+                description: "Approve a proposed test plan so implementation can proceed. Transitions workflow from tests_proposed to tests_approved. This is the human approval gate in the TDD workflow.",
+                inputSchema: {
+                    type: "object",
+                    properties: {
+                        storyId: {
+                            type: "string",
+                            description: "ID of the story whose test plan to approve",
+                        },
+                    },
+                    required: ["storyId"],
+                },
+            },
             {
                 name: "submit_for_review",
                 description: "Submit implementation for human review. Only works if tests are approved. Sets state to finished.",
@@ -836,6 +858,7 @@ ${config.infrastructureProfile?.provider ? `- Provider: ${config.infrastructureP
                 case "record_learning": {
                     const args = request.params.arguments;
                     const { storyId, outcome_summary } = args;
+                    assertUUID(storyId, "storyId");
                     const response = await client.patch(`/stories/${storyId}`, {
                         outcome_summary,
                     });
@@ -851,6 +874,7 @@ ${config.infrastructureProfile?.provider ? `- Provider: ${config.infrastructureP
                     if (!storyId) {
                         throw new Error("storyId is required");
                     }
+                    assertUUID(storyId, "storyId");
                     // Guardrail: Block AI from setting accepted/rejected states (human-in-the-loop)
                     const blockedStates = ["accepted", "rejected"];
                     if (state && blockedStates.includes(state.toLowerCase())) {
@@ -904,6 +928,7 @@ ${config.infrastructureProfile?.provider ? `- Provider: ${config.infrastructureP
                     if (!epicId) {
                         throw new Error("epicId is required");
                     }
+                    assertUUID(epicId, "epicId");
                     const payload = Object.fromEntries(Object.entries(rest).filter(([, value]) => value !== undefined));
                     const response = await client.patch(`/epics/${epicId}`, payload);
                     return {
@@ -915,6 +940,7 @@ ${config.infrastructureProfile?.provider ? `- Provider: ${config.infrastructureP
                 case "prepare_implementation": {
                     const args = request.params.arguments;
                     const { storyId } = args;
+                    assertUUID(storyId, "storyId");
                     const storyResponse = await client.get(`/stories/${storyId}`);
                     const story = storyResponse.data;
                     // Validation & Guardrails
@@ -959,6 +985,7 @@ Here’s the smallest change that will validate it:
                     if (!storyId) {
                         throw new Error("storyId is required");
                     }
+                    assertUUID(storyId, "storyId");
                     const response = await client.post(`/stories/${storyId}/start`, {
                         branchPrefix: branchPrefix || "feat",
                         trunkBased,
@@ -1041,6 +1068,7 @@ ${result.workflow_reminder}
                     if (!storyId) {
                         throw new Error("storyId is required");
                     }
+                    assertUUID(storyId, "storyId");
                     if (!testPlan) {
                         throw new Error("testPlan is required");
                     }
@@ -1062,6 +1090,30 @@ ${(result.test_file_paths || []).map((p) => `- \`${p}\``).join("\n") || "No test
 ${result.message}
 
 > 🛑 **BLOCKED:** Implementation cannot proceed until a human approves this test plan.
+`;
+                    return {
+                        content: [{ type: "text", text: formattedResult.trim() }],
+                    };
+                }
+                case "approve_tests": {
+                    const args = request.params.arguments;
+                    const { storyId } = args;
+                    if (!storyId) {
+                        throw new Error("storyId is required");
+                    }
+                    assertUUID(storyId, "storyId");
+                    const response = await client.post(`/stories/${storyId}/approve-tests`);
+                    const result = response.data;
+                    const formattedResult = `
+# Test Plan Approved
+
+**Story ID:** ${storyId}
+**Workflow Stage:** ${result.workflow_stage}
+
+## Status
+${result.message}
+
+> ✅ **Approved:** Agent may now proceed with implementation.
 `;
                     return {
                         content: [{ type: "text", text: formattedResult.trim() }],
@@ -1073,6 +1125,7 @@ ${result.message}
                     if (!storyId) {
                         throw new Error("storyId is required");
                     }
+                    assertUUID(storyId, "storyId");
                     const response = await client.post(`/stories/${storyId}/submit-review`, {
                         commitHash,
                         testResults,
@@ -1131,6 +1184,7 @@ ${result.next_step}
                     let { provider } = args;
                     if (!storyId)
                         throw new Error("storyId is required");
+                    assertUUID(storyId, "storyId");
                     // Default provider from infrastructure profile if not explicitly provided
                     if (!provider) {
                         const costConfig = await fetchFeatureConfig();
@@ -1200,6 +1254,7 @@ ${(estimate.assumptions || []).map((a) => `- ${a}`).join("\n")}
                     const { epicId } = args;
                     if (!epicId)
                         throw new Error("epicId is required");
+                    assertUUID(epicId, "epicId");
                     // Fetch all stories and filter by epicId
                     const allStories = await fetchStories();
                     const epicStories = allStories.filter((s) => s.epicId === epicId);
@@ -1302,11 +1357,20 @@ ${unestimated.length === 0
             if (error.response) {
                 console.error("Response data:", error.response.data);
             }
+            let errorText = `Error: ${error.message}`;
+            if (error.response?.status) {
+                errorText += ` (HTTP ${error.response.status})`;
+            }
+            if (error.response?.data) {
+                const data = error.response.data;
+                const detail = typeof data === "string" ? data : JSON.stringify(data);
+                errorText += `\nDetails: ${detail}`;
+            }
             return {
                 content: [
                     {
                         type: "text",
-                        text: `Error: ${error.message}`,
+                        text: errorText,
                     },
                 ],
                 isError: true,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@elixium.ai/mcp-server",
-  "version": "0.2.1",
+  "version": "0.2.2",
   "type": "module",
   "description": "MCP Server for Elixium.ai",
   "publishConfig": {