@programinglive/dev-workflow-mcp-server 1.0.7 → 1.0.9

package/CHANGELOG.md

@@ -2,6 +2,20 @@
 
 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.9](https://github.com/programinglive/dev-workflow-mcp-server/compare/v1.0.8...v1.0.9) (2025-10-23)
+
+
+### 📝 Documentation
+
+* explain local workflow state ([cb6e333](https://github.com/programinglive/dev-workflow-mcp-server/commit/cb6e33353b4e1c0f693146f6cb07c9437a7f5aa8))
+
+### [1.0.8](https://github.com/programinglive/dev-workflow-mcp-server/compare/v1.0.7...v1.0.8) (2025-10-21)
+
+
+### ✨ Features
+
+* allow skipping tests and dropping workflow tasks ([2b8b27c](https://github.com/programinglive/dev-workflow-mcp-server/commit/2b8b27cca611d2fe266c1b814c21b6fec2472856))
+
 ### [1.0.7](https://github.com/programinglive/dev-workflow-mcp-server/compare/v1.0.6...v1.0.7) (2025-10-20)
 
 ### [1.0.6](https://github.com/programinglive/dev-workflow-mcp-server/compare/v1.0.5...v1.0.6) (2025-10-20)

package/README.md

@@ -158,6 +158,12 @@ Confirm that you've created the necessary tests covering your change. Required b
 
 **Parameters:** _none_
 
+### `skip_tests`
+Record an explicit justification when automated tests aren't feasible. Marks testing as satisfied so you can proceed with documentation and verification, while flagging the task for manual QA.
+
+**Parameters:**
+- `reason` (string, required): Why automated tests were skipped
+
 ### `run_tests`
 Record test results. **NEVER commit if tests fail!** Only proceed if all tests are green.
 
@@ -204,6 +210,12 @@ Mark the task as complete after successful commit & push. Resets workflow for ne
 **Parameters:**
 - `commitMessage` (string, required): The commit message used
 
+### `drop_task`
+Abandon the current task without completing the workflow. Preserves an audit entry with context, then resets the state so you can start fresh.
+
+**Parameters:**
+- `reason` (string, optional): Additional detail about why the task was dropped
+
 ### `get_workflow_status`
 Get current workflow status and what needs to be done next.
 
@@ -249,31 +261,36 @@ Here's how you'd use this MCP server in a typical coding session:
    ```
    - If tests fail, the server will **block** you from proceeding!
 
-6. **Document:**
+7. **Document:**
    ```
    "Create documentation: type=README, summary='Added user profile section to docs'"
    ```
 
-7. **Check readiness:**
+8. **Check readiness:**
    ```
    "Check if I'm ready to commit"
    ```
 
-8. **Commit & Push:**
+9. **Commit & Push:**
    ```
    "Commit and push: commitMessage='feat: add user profile page with tests and docs'"
    ```
 
-9. **Record release:**
+10. **Record release:**
    ```
    "Record release: command='npm run release', notes='v1.2.3'"
    ```
 
-10. **Complete:**
+11. **Complete:**
    ```
    "Complete the task with commit message: 'feat: add user profile page'"
    ```
 
+12. **Drop task (optional):**
+   ```
+   "Drop task: reason='Switching to a different feature'"
+   ```
+
 ## 🎯 Key Features
 
 - **Enforces discipline**: Won't let you skip steps
@@ -294,7 +311,7 @@ Here's how you'd use this MCP server in a typical coding session:
 ## 💡 Tips
 
 1. **Always start with `start_task`** - This sets your intention
-2. **Never skip tests** - The server will remind you!
+2. **Never skip tests without justification** - Use `skip_tests` only when absolutely necessary and document the reason for manual QA
 3. **Use `get_workflow_status`** - Check where you are anytime
 4. **Review history** - Learn from your past tasks
 5. **Follow the prompts** - They contain best practices
@@ -316,6 +333,7 @@ The server maintains state in `.workflow-state.json`:
 - History of completed tasks
 
 This file is automatically created and managed by the server.
+It contains local, machine-specific progress and is ignored by git so each environment can manage its own workflow history without cross-contamination.
 
 ## 🤝 Integration with Your Rules
 

package/index.js

@@ -27,6 +27,8 @@ export class WorkflowState {
       bugFixed: false,
       testsCreated: false,
       testsPassed: false,
+      testsSkipped: false,
+      testsSkippedReason: "",
       documentationCreated: false,
       readyToCommit: false,
       readyCheckCompleted: false,
@@ -51,6 +53,12 @@ export class WorkflowState {
       if (typeof this.state.testsCreated !== "boolean") {
         this.state.testsCreated = false;
       }
+      if (typeof this.state.testsSkipped !== "boolean") {
+        this.state.testsSkipped = false;
+      }
+      if (typeof this.state.testsSkippedReason !== "string") {
+        this.state.testsSkippedReason = "";
+      }
       if (typeof this.state.readyCheckCompleted !== "boolean") {
         this.state.readyCheckCompleted = false;
       }
@@ -88,6 +96,8 @@ export class WorkflowState {
       bugFixed: false,
       testsCreated: false,
       testsPassed: false,
+      testsSkipped: false,
+      testsSkippedReason: "",
       documentationCreated: false,
       readyToCommit: false,
       readyCheckCompleted: false,
@@ -529,6 +539,21 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
           properties: {},
         },
       },
+      {
+        name: "skip_tests",
+        description:
+          "Record that tests were intentionally skipped, with a justification.",
+        inputSchema: {
+          type: "object",
+          properties: {
+            reason: {
+              type: "string",
+              description: "Why tests cannot be created or executed",
+            },
+          },
+          required: ["reason"],
+        },
+      },
       {
         name: "run_tests",
         description:
@@ -635,6 +660,20 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
           required: ["commitMessage"],
         },
       },
+      {
+        name: "drop_task",
+        description:
+          "Abandon the current task and reset the workflow without completing it.",
+        inputSchema: {
+          type: "object",
+          properties: {
+            reason: {
+              type: "string",
+              description: "Optional explanation for dropping the task",
+            },
+          },
+        },
+      },
       {
         name: "get_workflow_status",
         description: "Get current workflow status and what needs to be done next.",
@@ -685,6 +724,8 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         workflowState.state.currentPhase = "coding";
         workflowState.state.taskDescription = args.description;
         workflowState.state.taskType = args.type;
+        workflowState.state.testsSkipped = false;
+        workflowState.state.testsSkippedReason = "";
         workflowState.state.commitAndPushCompleted = false;
         workflowState.state.lastCommitMessage = "";
         workflowState.state.lastPushBranch = "";
@@ -715,12 +756,13 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         workflowState.state.bugFixed = true;
         workflowState.state.testsCreated = false;
         workflowState.state.currentPhase = "testing";
+        workflowState.state.testsSkipped = false;
+        workflowState.state.testsSkippedReason = "";
         workflowState.state.fixSummary = args.summary;
         workflowState.state.readyCheckCompleted = false;
         workflowState.state.released = false;
         workflowState.state.releaseCommand = "";
         workflowState.state.releaseNotes = "";
-        workflowState.state.readyToCommit = false;
         workflowState.state.commitAndPushCompleted = false;
         workflowState.state.lastCommitMessage = "";
         workflowState.state.lastPushBranch = "";
@@ -749,6 +791,8 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         }
 
         workflowState.state.testsCreated = true;
+        workflowState.state.testsSkipped = false;
+        workflowState.state.testsSkippedReason = "";
         workflowState.state.readyCheckCompleted = false;
         workflowState.state.released = false;
         workflowState.state.commitAndPushCompleted = false;
@@ -766,6 +810,78 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         };
       }
 
+      case "skip_tests": {
+        if (workflowState.state.currentPhase === "idle") {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "⚠️ No active task. Use 'start_task' before skipping tests.",
+              },
+            ],
+          };
+        }
+
+        if (!workflowState.state.bugFixed) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "⚠️ Please mark your feature/bug as fixed first using 'mark_bug_fixed'!",
+              },
+            ],
+          };
+        }
+
+        const reason =
+          typeof args.reason === "string" ? args.reason.trim() : "";
+
+        if (!reason) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "⚠️ Provide a non-empty justification when skipping tests.",
+              },
+            ],
+          };
+        }
+
+        workflowState.state.testsCreated = true;
+        workflowState.state.testsPassed = true;
+        workflowState.state.testsSkipped = true;
+        workflowState.state.testsSkippedReason = reason;
+        workflowState.state.currentPhase = "documentation";
+        workflowState.state.readyToCommit = false;
+        workflowState.state.readyCheckCompleted = false;
+        workflowState.state.released = false;
+        workflowState.state.commitAndPushCompleted = false;
+        workflowState.state.lastCommitMessage = "";
+        workflowState.state.lastPushBranch = "";
+        await workflowState.save();
+
+        return {
+          content: [
+            {
+              type: "text",
+              text: `⚠️ Tests skipped for this task.
+Reason: ${reason}
+
+Manual verification is now required.
+
+Next Steps:
+1. ✓ Fix/implement feature
+2. ⚠️ Tests skipped (ensure manual QA)
+3. ⏳ Create/update documentation
+4. ⏳ Run 'check_ready_to_commit'
+5. ⏳ Commit & push
+6. ⏳ Run 'perform_release'
+7. ⏳ Complete task`,
+            },
+          ],
+        };
+      }
+
       case "run_tests": {
         if (!workflowState.state.bugFixed) {
           return {
@@ -808,6 +924,8 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         workflowState.state.testCommand = args.testCommand;
         workflowState.state.testDetails =
           typeof args.details === "string" ? args.details : "";
+        workflowState.state.testsSkipped = false;
+        workflowState.state.testsSkippedReason = "";
         workflowState.state.readyCheckCompleted = false;
         workflowState.state.released = false;
         workflowState.state.commitAndPushCompleted = false;
@@ -841,75 +959,86 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         };
       }
 
-    case "create_documentation": {
-      if (!workflowState.state.testsPassed) {
+      case "create_documentation": {
+        if (!workflowState.state.testsPassed) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "⚠️ Please ensure tests are passing first! Run 'run_tests' with passed=true.",
+              },
+            ],
+          };
+        }
+
+        workflowState.state.documentationCreated = true;
+        workflowState.state.documentationType = args.documentationType;
+        workflowState.state.documentationSummary = args.summary;
+        workflowState.state.currentPhase = "ready";
+        workflowState.state.readyToCommit = true;
+        workflowState.state.readyCheckCompleted = false;
+        workflowState.state.released = false;
+        workflowState.state.commitAndPushCompleted = false;
+        workflowState.state.lastCommitMessage = "";
+        workflowState.state.lastPushBranch = "";
+        await workflowState.save();
+
         return {
           content: [
             {
               type: "text",
-              text: "⚠️ Please ensure tests are passing first! Run 'run_tests' with passed=true.",
+              text: `✅ Documentation created!\n\nType: ${args.documentationType}\nSummary: ${args.summary}\n\n🎉 You're ready to verify your work!\n\nNext Steps:\n1. ✓ Fix/implement feature\n2. ✓ Create tests\n3. ✓ Run tests (GREEN!)\n4. ✓ Create documentation\n5. ⏳ Run 'check_ready_to_commit' to verify\n6. ⏳ Commit & push, then run 'perform_release'\n7. ⏳ Mark as complete with 'complete_task'\n\nRemember: git add . && git commit && git push`,
             },
           ],
         };
       }
 
-      workflowState.state.documentationCreated = true;
-      workflowState.state.documentationType = args.documentationType;
-      workflowState.state.documentationSummary = args.summary;
-      workflowState.state.currentPhase = "ready";
-      workflowState.state.readyToCommit = true;
-      workflowState.state.readyCheckCompleted = false;
-      workflowState.state.released = false;
-      workflowState.state.commitAndPushCompleted = false;
-      workflowState.state.lastCommitMessage = "";
-      workflowState.state.lastPushBranch = "";
-      await workflowState.save();
-
-      return {
-        content: [
+      case "check_ready_to_commit": {
+        const status = workflowState.state;
+        const testsCreatedDone = status.testsSkipped || status.testsCreated;
+        const testsPassedDone = status.testsSkipped || status.testsPassed;
+        const checks = [
+          { name: "Task started", done: status.currentPhase !== "idle" },
+          { name: "Feature/bug fixed", done: status.bugFixed },
           {
-            type: "text",
-            text: `✅ Documentation created!\n\nType: ${args.documentationType}\nSummary: ${args.summary}\n\n🎉 You're ready to verify your work!\n\nNext Steps:\n1. ✓ Fix/implement feature\n2. ✓ Create tests\n3. ✓ Run tests (GREEN!)\n4. ✓ Create documentation\n5. ⏳ Run 'check_ready_to_commit' to verify\n6. ⏳ Commit & push, then run 'perform_release'\n7. ⏳ Mark as complete with 'complete_task'\n\nRemember: git add . && git commit && git push`,
+            name: status.testsSkipped
+              ? "Tests skipped (manual QA documented)"
+              : "Tests created",
+            done: testsCreatedDone,
           },
-        ],
-      };
-    }
-
-    case "check_ready_to_commit": {
-      const status = workflowState.state;
-      const checks = [
-        { name: "Task started", done: status.currentPhase !== "idle" },
-        { name: "Feature/bug fixed", done: status.bugFixed },
-        { name: "Tests created", done: status.testsCreated },
-        { name: "Tests created and passed", done: status.testsPassed },
-        { name: "Documentation created", done: status.documentationCreated },
-      ];
-
-      const allDone = checks.every((c) => c.done);
-      const checkList = checks
-        .map((c) => `${c.done ? "✅" : "❌"} ${c.name}`)
-        .join("\n");
+          {
+            name: status.testsSkipped ? "Tests skipped acknowledged" : "Tests passed",
+            done: testsPassedDone,
+          },
+          { name: "Documentation created", done: status.documentationCreated },
+        ];
+
+        const allDone = checks.every((c) => c.done);
+        const checkList = checks
+          .map((c) => `${c.done ? "✅" : "❌"} ${c.name}`)
+          .join("\n");
+
+        workflowState.state.readyCheckCompleted = allDone;
+        if (allDone) {
+          workflowState.state.currentPhase = workflowState.state.commitAndPushCompleted
+            ? workflowState.state.released
+              ? "ready_to_complete"
+              : "release"
+            : "commit";
+        }
+        await workflowState.save();
 
-      workflowState.state.readyCheckCompleted = allDone;
-      if (allDone) {
-        workflowState.state.currentPhase = workflowState.state.commitAndPushCompleted
-          ? workflowState.state.released
-            ? "ready_to_complete"
-            : "release"
-          : "commit";
-      }
-      await workflowState.save();
+        if (allDone) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: `🎉 ALL CHECKS PASSED!\n\n${checkList}\n\n✅ Next actions:\n1. Run 'commit_and_push' with your commit message\n2. Run 'perform_release' to record the release\n3. Finish with 'complete_task'\n\nTip: Provide the optional 'branch' argument to push to a non-default branch.\n\nTask: ${status.taskDescription}`,
+              },
+            ],
+          };
+        }
 
-      if (allDone) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: `🎉 ALL CHECKS PASSED!\n\n${checkList}\n\n✅ Next actions:\n1. Run 'commit_and_push' with your commit message\n2. Run 'perform_release' to record the release\n3. Finish with 'complete_task'\n\nTip: Provide the optional 'branch' argument to push to a non-default branch.\n\nTask: ${status.taskDescription}`,
-            },
-          ],
-        };
-      } else {
         return {
           content: [
             {
@@ -919,371 +1048,243 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
           ],
         };
       }
-    }
-
-    case "commit_and_push": {
-      if (!workflowState.state.readyToCommit) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: "⚠️ Not ready yet! Run 'check_ready_to_commit' and complete all steps first.",
-            },
-          ],
-        };
-      }
-
-      if (!workflowState.state.readyCheckCompleted) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: "⚠️ Please run 'check_ready_to_commit' and resolve any outstanding items before committing!",
-            },
-          ],
-        };
-      }
-
-      const providedCommitMessage =
-        typeof args.commitMessage === "string" ? args.commitMessage.trim() : "";
-
-      if (!(await hasWorkingChanges())) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: "⚠️ No changes detected. Make sure you have modifications to commit before running 'commit_and_push'.",
-            },
-          ],
-        };
-      }
 
-      if (!(await hasTestChanges())) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: "⚠️ Please include test updates in your changes before running 'commit_and_push'. Ensure at least one test file is modified.",
-            },
-          ],
-        };
-      }
-
-      try {
-        await exec("git add .");
-      } catch (error) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: `❌ Failed to stage files:\n\n${error.stderr || error.stdout || error.message}`,
-            },
-          ],
-        };
-      }
-
-      const stagedChanges = await getStagedChanges();
-      const { summary: generatedSummary, body: commitBody } =
-        createCommitMessageParts(stagedChanges, providedCommitMessage);
-
-      if (!generatedSummary) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: "⚠️ Unable to generate commit message. Please provide a 'commitMessage' manually.",
-            },
-          ],
-        };
-      }
-
-      try {
-        if (commitBody) {
-          await exec(
-            `git commit -m ${shellEscape(generatedSummary)} -m ${shellEscape(commitBody)}`
-          );
-        } else {
-          await exec(`git commit -m ${shellEscape(generatedSummary)}`);
-        }
-      } catch (error) {
-        const output = error.stderr || error.stdout || error.message || "Unknown error";
-        if (output.includes("nothing to commit")) {
+      case "commit_and_push": {
+        if (!workflowState.state.readyToCommit) {
           return {
             content: [
               {
                 type: "text",
-                text: "⚠️ Nothing to commit. Make additional changes before running 'commit_and_push'.",
+                text: "⚠️ Not ready yet! Run 'check_ready_to_commit' and complete all steps first.",
               },
             ],
           };
         }
 
-        return {
-          content: [
-            {
-              type: "text",
-              text: `❌ git commit failed:\n\n${output}`,
-            },
-          ],
-        };
-      }
-
-      const requestedBranch =
-        typeof args.branch === "string" ? args.branch.trim() : "";
-      const currentBranch = await getCurrentBranch();
-      const pushCommand = requestedBranch
-        ? `git push origin ${shellEscape(requestedBranch)}`
-        : "git push";
-
-      try {
-        await exec(pushCommand);
-      } catch (error) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: `❌ git push failed:\n\n${error.stderr || error.stdout || error.message}`,
-            },
-          ],
-        };
-      }
-
-      workflowState.state.commitAndPushCompleted = true;
-      workflowState.state.lastCommitMessage = generatedSummary;
-      workflowState.state.lastPushBranch =
-        requestedBranch || currentBranch || "";
-      workflowState.state.currentPhase = workflowState.state.released
-        ? "ready_to_complete"
-        : "release";
-      await workflowState.save();
-
-      return {
-        content: [
-          {
-            type: "text",
-            text: `✅ Commit & push completed!\n\nCommit message: ${generatedSummary}\nPushed to: ${
-              requestedBranch || currentBranch || "(default upstream)"
-            }\n\nNext: Run 'perform_release' to record the release.`,
-          },
-        ],
-      };
-    }
-
-    case "perform_release": {
-      if (workflowState.state.currentPhase === "idle") {
-        return {
-          content: [
-            {
-              type: "text",
-              text: "⚠️ Please start a task first using 'start_task'!",
-            },
-          ],
-        };
-      }
+        if (!workflowState.state.readyCheckCompleted) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "⚠️ Please run 'check_ready_to_commit' and resolve any outstanding items before committing!",
+              },
+            ],
+          };
+        }
 
-      if (!workflowState.state.commitAndPushCompleted) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: "⚠️ Please run 'commit_and_push' after the ready check before recording a release!",
-            },
-          ],
-        };
-      }
+        const providedCommitMessage =
+          typeof args.commitMessage === "string" ? args.commitMessage.trim() : "";
 
-      if (!workflowState.state.readyCheckCompleted) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: "⚠️ Please run 'check_ready_to_commit' and ensure all checks pass before releasing!",
-            },
-          ],
-        };
-      }
+        if (!(await hasWorkingChanges())) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "⚠️ No changes detected. Make sure you have modifications to commit before running 'commit_and_push'.",
+              },
+            ],
+          };
+        }
 
-      const releaseCommand =
-        typeof args.command === "string" ? args.command.trim() : "";
+        if (!(await hasTestChanges()) && !workflowState.state.testsSkipped) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "⚠️ Please include test updates in your changes before running 'commit_and_push'. Ensure at least one test file is modified.",
+              },
+            ],
+          };
+        }
 
-      if (!releaseCommand) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: "⚠️ Please provide a non-empty 'command' when using 'perform_release'.",
-            },
-          ],
-        };
-      }
+        try {
+          await exec("git add .");
+        } catch (error) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: `❌ Failed to stage files:\n\n${error.stderr || error.stdout || error.message}`,
+              },
+            ],
+          };
+        }
 
-      if (await hasWorkingChanges()) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: "⚠️ Working tree not clean! Please commit or stash all changes before running 'perform_release'.",
-            },
-          ],
-        };
-      }
+        const stagedChanges = await getStagedChanges();
+        const { summary: generatedSummary, body: commitBody } =
+          createCommitMessageParts(stagedChanges, providedCommitMessage);
 
-      const lastCommitMessage = await getLastCommitMessage();
-      const releaseType = determineReleaseTypeFromCommit(
-        lastCommitMessage || workflowState.state.lastCommitMessage
-      );
+        if (!generatedSummary) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "⚠️ Unable to generate commit message. Please provide a 'commitMessage' manually.",
+              },
+            ],
+          };
+        }
 
-      const finalCommand = releaseCommand.includes("--release-as")
-        ? releaseCommand
-        : `${releaseCommand} -- --release-as ${releaseType}`;
+        try {
+          if (commitBody) {
+            await exec(
+              `git commit -m ${shellEscape(generatedSummary)} -m ${shellEscape(commitBody)}`
+            );
+          } else {
+            await exec(`git commit -m ${shellEscape(generatedSummary)}`);
+          }
+        } catch (error) {
+          const output = error.stderr || error.stdout || error.message || "Unknown error";
+          if (output.includes("nothing to commit")) {
+            return {
+              content: [
+                {
+                  type: "text",
+                  text: "⚠️ Nothing to commit. Make additional changes before running 'commit_and_push'.",
+                },
+              ],
+            };
+          }
 
-      try {
-        await exec(finalCommand);
-      } catch (error) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: `❌ Release command failed:\n\n${error.stderr || error.stdout || error.message}`,
-            },
-          ],
-        };
-      }
+          return {
+            content: [
+              {
+                type: "text",
+                text: `❌ git commit failed:\n\n${output}`,
+              },
+            ],
+          };
+        }
 
-      workflowState.state.released = true;
-      workflowState.state.releaseCommand = finalCommand;
-      workflowState.state.releaseNotes =
-        typeof args.notes === "string" ? args.notes : "";
-      workflowState.state.currentPhase = "ready_to_complete";
-      await workflowState.save();
+        const requestedBranch =
+          typeof args.branch === "string" ? args.branch.trim() : "";
+        const currentBranch = await getCurrentBranch();
+        const pushCommand = requestedBranch
+          ? `git push origin ${shellEscape(requestedBranch)}`
+          : "git push";
 
-      return {
-        content: [
-          {
-            type: "text",
-            text: `🚀 Release recorded!\n\nCommand: ${finalCommand}\nRelease type: ${releaseType}\n${workflowState.state.releaseNotes ? `Notes: ${workflowState.state.releaseNotes}\n` : ""}\n✅ Next: Run 'complete_task' to wrap up.`,
-          },
-        ],
-      };
-    }
+        try {
+          await exec(pushCommand);
+        } catch (error) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: `❌ git push failed:\n\n${error.stderr || error.stdout || error.message}`,
+              },
+            ],
+          };
+        }
 
-    case "complete_task": {
-      if (!workflowState.state.readyToCommit) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: "⚠️ Not ready to complete! Run 'check_ready_to_commit' first.",
-            },
-          ],
-        };
-      }
+        workflowState.state.commitAndPushCompleted = true;
+        workflowState.state.lastCommitMessage = generatedSummary;
+        workflowState.state.lastPushBranch =
+          requestedBranch || currentBranch || "";
+        workflowState.state.currentPhase = workflowState.state.released
+          ? "ready_to_complete"
+          : "release";
+        await workflowState.save();
 
-      if (!workflowState.state.readyCheckCompleted) {
         return {
           content: [
             {
               type: "text",
-              text: "⚠️ Please run 'check_ready_to_commit' and ensure all checks pass before completing the task!",
+              text: `✅ Commit & push completed!\n\nCommit message: ${generatedSummary}\nPushed to: ${
+                requestedBranch || currentBranch || "(default upstream)"
+              }\n\nNext: Run 'perform_release' to record the release.`,
             },
           ],
         };
       }
 
-      if (!workflowState.state.commitAndPushCompleted) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: "⚠️ Commit and push not completed! Run 'commit_and_push' first.",
-            },
-          ],
-        };
-      }
+      case "perform_release": {
+        if (workflowState.state.currentPhase === "idle") {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "⚠️ Please start a task first using 'start_task'!",
+              },
+            ],
+          };
+        }
 
-      if (!workflowState.state.released) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: "⚠️ Release not recorded! Run 'perform_release' after committing and pushing.",
-            },
-          ],
-        };
-      }
+        if (!workflowState.state.commitAndPushCompleted) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "⚠️ Please run 'commit_and_push' after the ready check before recording a release!",
+              },
+            ],
+          };
+        }
 
-      if (await hasStagedChanges()) {
-        return {
-          content: [
-            {
-              type: "text",
-              text: "⚠️ Staged files detected! Please commit or unstage all files before pushing. Run 'git status' to confirm a clean working tree, then try again.",
-            },
-          ],
-        };
-      }
+        if (!workflowState.state.readyCheckCompleted) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "⚠️ Please run 'check_ready_to_commit' and ensure all checks pass before releasing!",
+              },
+            ],
+          };
+        }
 
-      workflowState.addToHistory({
-        taskDescription: workflowState.state.taskDescription,
-        taskType: workflowState.state.taskType,
-        fixSummary: workflowState.state.fixSummary,
-        testCommand: workflowState.state.testCommand,
-        documentationType: workflowState.state.documentationType,
-        commitMessage: args.commitMessage,
-        releaseCommand: workflowState.state.releaseCommand,
-        releaseNotes: workflowState.state.releaseNotes,
-      });
+        const releaseCommand =
+          typeof args.command === "string" ? args.command.trim() : "";
 
-      const completedTask = workflowState.state.taskDescription;
-      workflowState.reset();
-      workflowState.state.currentPhase = "idle";
-      await workflowState.save();
+        if (!releaseCommand) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "⚠️ Please provide a non-empty 'command' when using 'perform_release'.",
+              },
+            ],
+          };
+        }
 
-      return {
-        content: [
-          {
-            type: "text",
-            text: `🎉 Task completed successfully!\n\nCompleted: ${completedTask}\nCommit: ${args.commitMessage}\n\n✅ Workflow reset. Ready for next task!\n\nUse 'start_task' when you begin your next coding session.`,
-          },
-        ],
-      };
-    }
+        if (await hasWorkingChanges()) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "⚠️ Working tree not clean! Please commit or stash all changes before running 'perform_release'.",
+              },
+            ],
+          };
+        }
 
-    case "get_workflow_status": {
-      const status = workflowState.state;
+        const lastCommitMessage = await getLastCommitMessage();
+        const releaseType = determineReleaseTypeFromCommit(
+          lastCommitMessage || workflowState.state.lastCommitMessage
+        );
 
-      if (status.currentPhase === "idle") {
-        return {
-          content: [
-            {
-              type: "text",
-              text: "📋 No active task.\n\nUse 'start_task' to begin a new coding session.",
-            },
-          ],
-        };
-      }
+        const finalCommand = releaseCommand.includes("--release-as")
+          ? releaseCommand
+          : `${releaseCommand} -- --release-as ${releaseType}`;
 
-      const phaseMessages = {
-        coding: "🔨 Currently coding",
-        testing: "🧪 Ready for testing",
-        documentation: "📝 Ready for documentation",
-        ready: "✅ Ready for verification",
-        commit: "📦 Ready to commit & push",
-        release: "🚀 Ready to release",
-        ready_to_complete: "✅ Release completed",
-      };
+        try {
+          await exec(finalCommand);
+        } catch (error) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: `❌ Release command failed:\n\n${error.stderr || error.stdout || error.message}`,
+              },
+            ],
+          };
+        }
 
-      return {
-        content: [
-          {
-            type: "text",
-            text: `📋 Current Workflow Status\n\nTask: ${status.taskDescription}\nPhase: ${phaseMessages[status.currentPhase]}\n\nProgress:\n${status.bugFixed ? "✅" : "⏳"} Feature/bug fixed\n${status.testsCreated ? "✅" : "⏳"} Tests created\n${status.testsPassed ? "✅" : "⏳"} Tests passed\n${status.documentationCreated ? "✅" : "⏳"} Documentation created\n${status.readyCheckCompleted ? "✅" : "⏳"} Ready check completed\n${status.commitAndPushCompleted ? "✅" : "⏳"} Commit & push completed\n${status.released ? "✅" : "⏳"} Release recorded\n\nNext: ${getNextStep(status)}`,
-          },
-        ],
-      };
-    }
+        workflowState.state.released = true;
+        workflowState.state.releaseCommand = finalCommand;
+        workflowState.state.releaseNotes =
+          typeof args.notes === "string" ? args.notes : "";
+        workflowState.state.currentPhase = "ready_to_complete";
+        await workflowState.save();
 
-    case "view_history": {
       const limit = args.limit || 10;
       const history = workflowState.state.history.slice(-limit).reverse();
 

package/package.json

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

package/tests/workflowState.test.js

@@ -38,6 +38,8 @@ test("WorkflowState persists data across save/load", async () => {
     state.state.commitAndPushCompleted = true;
     state.state.lastCommitMessage = "test: persistence";
     state.state.lastPushBranch = "main";
+    state.state.testsSkipped = true;
+    state.state.testsSkippedReason = "Legacy app uses manual QA";
     await state.save();
 
     const reloaded = new WorkflowState(state.stateFile);
@@ -54,6 +56,11 @@ test("WorkflowState persists data across save/load", async () => {
     assert.equal(reloaded.state.commitAndPushCompleted, true);
     assert.equal(reloaded.state.lastCommitMessage, "test: persistence");
     assert.equal(reloaded.state.lastPushBranch, "main");
+    assert.equal(reloaded.state.testsSkipped, true);
+    assert.equal(
+      reloaded.state.testsSkippedReason,
+      "Legacy app uses manual QA"
+    );
   });
 });
 
@@ -66,6 +73,8 @@ test("WorkflowState reset keeps history but clears progress", async () => {
     state.state.readyCheckCompleted = true;
     state.state.released = true;
     state.state.commitAndPushCompleted = true;
+    state.state.testsSkipped = true;
+    state.state.testsSkippedReason = "Manual QA";
     state.addToHistory({ taskDescription: "Sample task" });
 
     state.reset();
@@ -77,11 +86,29 @@ test("WorkflowState reset keeps history but clears progress", async () => {
     assert.equal(state.state.readyCheckCompleted, false);
     assert.equal(state.state.released, false);
     assert.equal(state.state.commitAndPushCompleted, false);
+    assert.equal(state.state.testsSkipped, false);
+    assert.equal(state.state.testsSkippedReason, "");
     assert.equal(state.state.history.length, 1);
     assert.equal(state.state.history[0].taskDescription, "Sample task");
   });
 });
 
+test("getNextStep treats skipped tests as satisfied", () => {
+  const skippedState = {
+    bugFixed: true,
+    testsCreated: true,
+    testsPassed: true,
+    testsSkipped: true,
+    testsSkippedReason: "Legacy system",
+    documentationCreated: false,
+    readyCheckCompleted: false,
+    commitAndPushCompleted: false,
+    released: false,
+  };
+
+  assert.equal(getNextStep(skippedState), "Create documentation");
+});
+
 test("getNextStep guides workflow progression", () => {
   const baseState = {
     bugFixed: false,