@programinglive/dev-workflow-mcp-server 1.0.1 → 1.0.2

package/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+### [1.0.2](https://github.com/programinglive/dev-workflow-mcp-server/compare/v1.0.1...v1.0.2) (2025-10-19)
+
+
+### 🐛 Bug Fixes
+
+* guard run_tests arguments ([2004c59](https://github.com/programinglive/dev-workflow-mcp-server/commit/2004c590fabe359d068122cf240a731382f81ca0))
+* normalize tool call arguments ([44d5a3d](https://github.com/programinglive/dev-workflow-mcp-server/commit/44d5a3d35f191012445fa02cee6254afbc993c98))
+
+
+### 📝 Documentation
+
+* clarify tool argument requirements ([4528614](https://github.com/programinglive/dev-workflow-mcp-server/commit/45286146661ad850eccb04849c11819e91ddd580))
+
 ### [1.0.1](https://github.com/programinglive/dev-workflow-mcp-server/compare/v1.0.0...v1.0.1) (2025-10-18)
 
 

package/README.md

@@ -57,6 +57,21 @@ After adding the configuration, restart the application to load the MCP server.
 
 ## 🛠️ Available Tools
 
+### Tool Argument Requirements
+
+All tool invocations validate their `arguments` payload before running:
+- Strings are parsed as JSON and must resolve to an object (key/value pairs).
+- Passing non-object data (numbers, arrays, plain text) triggers a guidance error.
+- Missing or malformed arguments safely default to empty input so the tool can respond with actionable reminders.
+
+Example (stringified JSON object):
+```
+{
+  "name": "start_task",
+  "arguments": "{\"description\":\"Add reporting endpoint\",\"type\":\"feature\"}"
+}
+```
+
 ### `start_task`
 Start a new coding task. This is your first step - be conscious about what you're coding.
 

package/index.js

@@ -70,6 +70,42 @@ await workflowState.load();
 
 const exec = promisify(execCallback);
 
+function normalizeRequestArgs(rawArgs) {
+  if (rawArgs === undefined || rawArgs === null) {
+    return { args: {}, error: null };
+  }
+
+  if (typeof rawArgs === "string") {
+    try {
+      const parsed = JSON.parse(rawArgs);
+      if (parsed && typeof parsed === "object") {
+        return { args: parsed, error: null };
+      }
+
+      return {
+        args: {},
+        error:
+          "⚠️ Tool arguments must be a JSON object. Please provide key/value pairs.",
+      };
+    } catch (parseError) {
+      return {
+        args: {},
+        error:
+          "⚠️ Unable to parse tool arguments. Please provide valid JSON-formatted data.",
+      };
+    }
+  }
+
+  if (typeof rawArgs === "object") {
+    return { args: rawArgs, error: null };
+  }
+
+  return {
+    args: {},
+    error: "⚠️ Unsupported tool arguments format. Expected a JSON object.",
+  };
+}
+
 async function hasStagedChanges() {
   try {
     const { stdout } = await exec("git status --porcelain");
@@ -232,96 +268,127 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
 
 // Handle tool calls
 server.setRequestHandler(CallToolRequestSchema, async (request) => {
-  const { name, arguments: args } = request.params;
-
-  switch (name) {
-    case "start_task": {
-      workflowState.reset();
-      workflowState.state.currentPhase = "coding";
-      workflowState.state.taskDescription = args.description;
-      workflowState.state.taskType = args.type;
-      await workflowState.save();
+  try {
+    const { name } = request.params;
+    const { args, error: argumentsError } = normalizeRequestArgs(
+      request.params.arguments
+    );
 
+    if (argumentsError) {
       return {
         content: [
           {
             type: "text",
-            text: `✅ Task Started: ${args.description}\n\n🎯 Be conscious about what you're coding!\n\nWorkflow Steps:\n1. ✓ Start task (current)\n2. ⏳ Fix/implement the feature\n3. ⏳ Create tests\n4. ⏳ Run tests (must pass!)\n5. ⏳ Create documentation\n6. ⏳ Commit & push\n\nReminder: Focus on writing clean, maintainable code!`,
+            text: argumentsError,
           },
         ],
       };
     }
 
-    case "mark_bug_fixed": {
-      if (workflowState.state.currentPhase === "idle") {
+    switch (name) {
+      case "start_task": {
+        workflowState.reset();
+        workflowState.state.currentPhase = "coding";
+        workflowState.state.taskDescription = args.description;
+        workflowState.state.taskType = args.type;
+        await workflowState.save();
+
         return {
           content: [
             {
               type: "text",
-              text: "⚠️ Please start a task first using 'start_task'!",
+              text: `✅ Task Started: ${args.description}\n\n🎯 Be conscious about what you're coding!\n\nWorkflow Steps:\n1. ✓ Start task (current)\n2. ⏳ Fix/implement the feature\n3. ⏳ Create tests\n4. ⏳ Run tests (must pass!)\n5. ⏳ Create documentation\n6. ⏳ Commit & push\n\nReminder: Focus on writing clean, maintainable code!`,
             },
           ],
         };
       }
 
-      workflowState.state.bugFixed = true;
-      workflowState.state.currentPhase = "testing";
-      workflowState.state.fixSummary = args.summary;
-      await workflowState.save();
-
-      return {
-        content: [
-          {
-            type: "text",
-            text: `✅ Feature/Bug marked as fixed!\n\n⚠️ CRITICAL REMINDER: You MUST create tests now!\n\nNext Steps:\n1. ✓ Fix/implement feature\n2. ⏳ Create tests for: ${args.summary}\n3. ⏳ Run tests (must be green!)\n4. ⏳ Create documentation\n5. ⏳ Commit & push\n\n🚫 DO NOT SKIP TESTING!`,
-          },
-        ],
-      };
-    }
+      case "mark_bug_fixed": {
+        if (workflowState.state.currentPhase === "idle") {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "⚠️ Please start a task first using 'start_task'!",
+              },
+            ],
+          };
+        }
+
+        workflowState.state.bugFixed = true;
+        workflowState.state.currentPhase = "testing";
+        workflowState.state.fixSummary = args.summary;
+        await workflowState.save();
 
-    case "run_tests": {
-      if (!workflowState.state.bugFixed) {
         return {
           content: [
             {
               type: "text",
-              text: "⚠️ Please mark your feature/bug as fixed first using 'mark_bug_fixed'!",
+              text: `✅ Feature/Bug marked as fixed!\n\n⚠️ CRITICAL REMINDER: You MUST create tests now!\n\nNext Steps:\n1. ✓ Fix/implement feature\n2. ⏳ Create tests for: ${args.summary}\n3. ⏳ Run tests (must be green!)\n4. ⏳ Create documentation\n5. ⏳ Commit & push\n\n🚫 DO NOT SKIP TESTING!`,
             },
           ],
         };
       }
 
-      workflowState.state.testsPassed = args.passed;
-      workflowState.state.testCommand = args.testCommand;
-      workflowState.state.testDetails = args.details || "";
-
-      if (!args.passed) {
-        workflowState.state.currentPhase = "testing";
+      case "run_tests": {
+        if (!workflowState.state.bugFixed) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "⚠️ Please mark your feature/bug as fixed first using 'mark_bug_fixed'!",
+              },
+            ],
+          };
+        }
+
+        if (
+          typeof args.passed !== "boolean" ||
+          typeof args.testCommand !== "string" ||
+          args.testCommand.trim() === ""
+        ) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "⚠️ Please provide both 'passed' (boolean) and 'testCommand' (non-empty string) when recording test results.",
+              },
+            ],
+          };
+        }
+
+        workflowState.state.testsPassed = args.passed;
+        workflowState.state.testCommand = args.testCommand;
+        workflowState.state.testDetails =
+          typeof args.details === "string" ? args.details : "";
+
+        if (!args.passed) {
+          workflowState.state.currentPhase = "testing";
+          await workflowState.save();
+
+          return {
+            content: [
+              {
+                type: "text",
+                text: `❌ TESTS FAILED!\n\n🚫 STOP! DO NOT COMMIT OR PUSH!\n\nYou must:\n1. Fix the failing tests\n2. Run tests again until they pass\n\nTest command: ${args.testCommand}\n\nNever skip or ignore failing tests!`,
+              },
+            ],
+          };
+        }
+
+        workflowState.state.currentPhase = "documentation";
         await workflowState.save();
 
         return {
           content: [
             {
               type: "text",
-              text: `❌ TESTS FAILED!\n\n🚫 STOP! DO NOT COMMIT OR PUSH!\n\nYou must:\n1. Fix the failing tests\n2. Run tests again until they pass\n\nTest command: ${args.testCommand}\n\nNever skip or ignore failing tests!`,
+              text: `✅ All tests passed! 🎉\n\nTest command: ${args.testCommand}\n\nNext Steps:\n1. ✓ Fix/implement feature\n2. ✓ Create tests\n3. ✓ Run tests (GREEN!)\n4. ⏳ Create/update documentation\n5. ⏳ Commit & push\n\nReminder: Document what you did before committing!`,
             },
           ],
         };
       }
 
-      workflowState.state.currentPhase = "documentation";
-      await workflowState.save();
-
-      return {
-        content: [
-          {
-            type: "text",
-            text: `✅ All tests passed! 🎉\n\nTest command: ${args.testCommand}\n\nNext Steps:\n1. ✓ Fix/implement feature\n2. ✓ Create tests\n3. ✓ Run tests (GREEN!)\n4. ⏳ Create/update documentation\n5. ⏳ Commit & push\n\nReminder: Document what you did before committing!`,
-          },
-        ],
-      };
-    }
-
     case "create_documentation": {
       if (!workflowState.state.testsPassed) {
         return {
@@ -496,8 +563,19 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
       };
     }
 
-    default:
-      throw new Error(`Unknown tool: ${name}`);
+      default:
+        throw new Error(`Unknown tool: ${name}`);
+    }
+  } catch (error) {
+    console.error("Tool handler error:", error);
+    return {
+      content: [
+        {
+          type: "text",
+          text: "⚠️ Internal server error while processing the tool request. Please try again.",
+        },
+      ],
+    };
   }
 });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@programinglive/dev-workflow-mcp-server",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "MCP server to enforce development workflow discipline",
   "type": "module",
   "main": "index.js",