@programinglive/dev-workflow-mcp-server 1.8.1 → 1.9.1

package/tools/definitions.js

@@ -35,6 +35,25 @@ export function getToolList() {
         required: ["summary"],
       },
     },
+    {
+      name: "create_feature_flow",
+      description:
+        "Record the feature flow using Mermaid syntax. This helps visualize the implementation plan.",
+      inputSchema: {
+        type: "object",
+        properties: {
+          mermaidCode: {
+            type: "string",
+            description: "The Mermaid code representing the feature flow",
+          },
+          description: {
+            type: "string",
+            description: "A brief description of this flow",
+          },
+        },
+        required: ["mermaidCode", "description"],
+      },
+    },
     {
       name: "create_tests",
       description: "Mark that tests covering the change have been created.",

package/tools/handler-modules/commit-handlers.js

@@ -1,4 +1,5 @@
-import { textResponse, isReleaseSatisfied, formatChecklist, handleReadyCheck } from "./shared.js";
+import { textResponse, isReleaseSatisfied, formatChecklist } from "./shared.js";
+import { handleReadyCheck } from "./status-handlers.js";
 import { containsTestFilesInStatus } from "../../git-helpers.js";
 
 export async function handleCommitAndPush(args, context) {

package/tools/handler-modules/index.js

@@ -1,15 +1,10 @@
-import { textResponse, isReleaseSatisfied } from "./shared.js";
-import { handleStartTask, handleMarkBugFixed, handleCreateTests, handleSkipTests, handleRunTests, handleCreateDocumentation } from "./workflow-handlers.js";
+import { textResponse, isReleaseSatisfied, isErrorResponse } from "./shared.js";
+import { handleStartTask, handleMarkBugFixed, handleCreateTests, handleSkipTests, handleRunTests, handleCreateDocumentation, handleCreateFeatureFlow } from "./workflow-handlers.js";
 import { handleSkipRelease, handlePerformRelease } from "./release-handlers.js";
 import { handleCommitAndPush, handleCompleteTask, handleForceCompleteTask, handleDropTask } from "./commit-handlers.js";
 import { handleProjectSummary, handleProjectSummaryData, handleProjectSummaryDb, handleViewHistory } from "./summary-handlers.js";
 import { handleGetWorkflowStatus, handleReadyCheck, handleContinueWorkflow, handleRerunWorkflow } from "./status-handlers.js";
 
-function isErrorResponse(response) {
-  const text = response?.content?.[0]?.text || "";
-  return /^\s*[⚠️❌]/u.test(text);
-}
-
 function summarizeResponse(label, response) {
   const text = response?.content?.[0]?.text || "";
   const firstLine = text.split("\n")[0] || "";
@@ -84,6 +79,15 @@ async function handleRunFullWorkflow(args, { workflowState, exec, git, utils })
       continue;
     }
 
+    if (!state.featureFlowCreated) {
+      if (!args.mermaidCode || !args.featureFlowDescription) {
+        return textResponse("⚠️ 'run_full_workflow' now requires 'mermaidCode' and 'featureFlowDescription' to support the Mermaid step.");
+      }
+      const resp = await executeStep("create_feature_flow", handleCreateFeatureFlow, [{ mermaidCode: args.mermaidCode, description: args.featureFlowDescription }, workflowState]);
+      if (isErrorResponse(resp)) return resp;
+      continue;
+    }
+
     if (!state.testsCreated) {
       const resp = await executeStep("create_tests", handleCreateTests, [workflowState]);
       if (isErrorResponse(resp)) return resp;
@@ -172,6 +176,8 @@ export async function handleToolCall({
         return handleStartTask(args, workflowState);
       case "mark_bug_fixed":
         return handleMarkBugFixed(args, workflowState);
+      case "create_feature_flow":
+        return handleCreateFeatureFlow(args, workflowState);
       case "create_tests":
         return handleCreateTests(workflowState);
       case "skip_tests":

package/tools/handler-modules/shared.js

@@ -21,6 +21,11 @@ export function isReleaseSatisfied(state) {
   return Boolean(state?.released || state?.releaseSkipped);
 }
 
+export function isErrorResponse(response) {
+  const text = response?.content?.[0]?.text || "";
+  return /^\s*[⚠️❌]/u.test(text);
+}
+
 export async function resetToCommitIfWorkingChanges(workflowState, git) {
   if (!workflowState.state.commitAndPushCompleted) {
     return false;
@@ -92,7 +97,7 @@ export function getContinueGuidance(status) {
   switch (status.currentPhase) {
     case "coding": {
       const task = status.taskDescription ? ` "${status.taskDescription}"` : "";
-      return `Finish implementing${task} and then run 'mark_bug_fixed' with a brief summary of the change.`;
+      return `Describe your feature flow with Mermaid using 'create_feature_flow', then finish implementing${task} and run 'mark_bug_fixed' with a brief summary of the change.`;
     }
     case "testing":
       if (!status.testsCreated) {
@@ -127,45 +132,6 @@ export function getContinueGuidance(status) {
   }
 }
 
-export async function handleReadyCheck(workflowState) {
-  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 },
-    {
-      name: status.testsSkipped ? "Tests skipped (manual QA documented)" : "Tests created",
-      done: testsCreatedDone,
-    },
-    {
-      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 = formatChecklist(checks);
-
-  workflowState.state.readyCheckCompleted = allDone;
-  if (allDone) {
-    if (workflowState.state.commitAndPushCompleted) {
-      workflowState.state.currentPhase = isReleaseSatisfied(workflowState.state) ? "ready_to_complete" : "release";
-    } else {
-      workflowState.state.currentPhase = "commit";
-    }
-  }
-  await workflowState.save();
-
-  if (allDone) {
-    return textResponse(
-      `🎉 ALL CHECKS PASSED!\n\n${checkList}\n\n✅ Next actions:\n1. Run 'commit_and_push' (commits and pushes your changes)\n2. Run 'perform_release' (handles versioning, tags, and final push)\n3. Finish with 'complete_task'\n\nTip: Provide the optional 'branch' argument to 'commit_and_push' to push to a non-default branch.\n\nTask: ${status.taskDescription}`
-    );
-  }
-
-  return textResponse(`⚠️ NOT READY TO COMMIT!\n\n${checkList}\n\nPlease complete all steps before committing.`);
-}
 
 export async function loadProjectSummary(workflowState) {
   const summaryPath = path.join(path.dirname(workflowState.stateFile), PROJECT_SUMMARY_FILENAME);
@@ -180,7 +146,7 @@ export async function loadProjectSummary(workflowState) {
   }
 }
 
-export const prdPath = path.join(__dirname, "..", "docs", "product", "PRD.md");
+export const prdPath = path.join(__dirname, "..", "..", "docs", "PRD.md");
 
 export function ensurePrdExists() {
   return existsSync(prdPath);

package/tools/handler-modules/status-handlers.js

@@ -10,6 +10,7 @@ export function handleGetWorkflowStatus(workflowState) {
 
 Phase: ${status.currentPhase}
 Task: ${status.taskDescription || "(none)"}
+Feature flow: ${status.featureFlowCreated ? "✅" : "❌"}
 Bug fixed: ${status.bugFixed ? "✅" : "❌"}
 Tests created: ${status.testsCreated ? "✅" : "❌"}
 Tests passed: ${status.testsPassed ? "✅" : "❌"}
@@ -28,6 +29,7 @@ export async function handleReadyCheck(workflowState) {
   const testsPassedDone = status.testsSkipped || status.testsPassed;
   const checks = [
     { name: "Task started", done: status.currentPhase !== "idle" },
+    { name: "Feature flow described", done: status.featureFlowCreated },
     { name: "Feature/bug fixed", done: status.bugFixed },
     {
       name: status.testsSkipped ? "Tests skipped (manual QA documented)" : "Tests created",
@@ -129,6 +131,6 @@ export async function handleRerunWorkflow(workflowState) {
   await workflowState.save();
 
   return textResponse(
-    `🔄 Rerunning workflow from the start!\n\nTask: ${currentDescription}\nType: ${currentType}\n\nWorkflow Steps:\n1. ✓ Start task (current)\n2. ⏳ Fix/implement the feature\n3. ⏳ Create tests\n4. ⏳ Run tests (must pass!)\n5. ⏳ Create documentation\n6. ⏳ Run 'check_ready_to_commit'\n7. ⏳ Run 'commit_and_push' (commits and pushes)\n8. ⏳ Run 'perform_release' (handles versioning and tags)\n9. ⏳ Complete task\n\n🎯 Be conscious about what you're coding!`
+    `🔄 Rerunning workflow from the start!\n\nTask: ${currentDescription}\nType: ${currentType}\n\nWorkflow Steps:\n1. ✓ Start task (current)\n2. ⏳ Describe feature flow with Mermaid\n3. ⏳ Fix/implement the feature\n4. ⏳ Create tests\n5. ⏳ Run tests (must pass!)\n6. ⏳ Create documentation\n7. ⏳ Run 'check_ready_to_commit'\n8. ⏳ Run 'commit_and_push' (commits and pushes)\n9. ⏳ Run 'perform_release' (handles versioning and tags)\n10. ⏳ Complete task\n\n🎯 Be conscious about what you're coding!`
   );
 }

package/tools/handler-modules/workflow-handlers.js

@@ -1,6 +1,4 @@
-import path from "path";
-import { existsSync } from "fs";
-import { textResponse, isReleaseSatisfied, resetToCommitIfWorkingChanges } from "./shared.js";
+import { textResponse, isReleaseSatisfied, isErrorResponse, ensurePrdExists } from "./shared.js";
 
 export async function handleStartTask(args, workflowState) {
   workflowState.reset();
@@ -15,7 +13,7 @@ export async function handleStartTask(args, workflowState) {
   await workflowState.save();
 
   return textResponse(
-    `✅ 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. ⏳ Run 'check_ready_to_commit'\n7. ⏳ Run 'commit_and_push' (commits and pushes)\n8. ⏳ Run 'perform_release' (handles versioning and tags) or 'skip_release' (when no release is needed)\n9. ⏳ Complete task\n\nReminder: Focus on writing clean, maintainable code!`
+    `✅ Task Started: ${args.description}\n\n🎯 Be conscious about what you're coding!\n\nWorkflow Steps:\n1. ✓ Start task (current)\n2. ⏳ Describe feature flow with Mermaid\n3. ⏳ Fix/implement the feature\n4. ⏳ Create tests\n5. ⏳ Run tests (must pass!)\n6. ⏳ Create documentation\n7. ⏳ Run 'check_ready_to_commit'\n8. ⏳ Run 'commit_and_push' (commits and pushes)\n9. ⏳ Run 'perform_release' (handles versioning and tags) or 'skip_release' (when no release is needed)\n10. ⏳ Complete task\n\nReminder: Focus on writing clean, maintainable code!`
   );
 }
 
@@ -67,6 +65,29 @@ export async function handleMarkBugFixed(args, workflowState) {
   );
 }
 
+export async function handleCreateFeatureFlow(args, workflowState) {
+  if (workflowState.state.currentPhase === "idle") {
+    return textResponse("⚠️ Please start a task first using 'start_task'!");
+  }
+
+  const mermaidCode = typeof args.mermaidCode === "string" ? args.mermaidCode.trim() : "";
+  const description = typeof args.description === "string" ? args.description.trim() : "";
+
+  if (!mermaidCode || !description) {
+    return textResponse("⚠️ Please provide both 'mermaidCode' and 'description' when creating a feature flow.");
+  }
+
+  workflowState.state.featureFlowCreated = true;
+  workflowState.state.mermaidCode = mermaidCode;
+  workflowState.state.featureFlowDescription = description;
+  workflowState.state.readyCheckCompleted = false;
+  await workflowState.save();
+
+  return textResponse(
+    `✅ Feature flow recorded!\n\nDescription: ${description}\n\nNext Steps:\n1. ✓ Start task\n2. ✓ Describe feature flow\n3. ⏳ Fix/implement the feature\n4. ⏳ Create tests\n5. ⏳ Run tests\n6. ⏳ Create documentation\n7. ⏳ Run 'check_ready_to_commit'\n8. ⏳ Run 'commit_and_push'\n9. ⏳ Run 'perform_release'\n10. ⏳ Complete task`
+  );
+}
+
 export async function handleCreateTests(workflowState) {
   if (!workflowState.state.bugFixed) {
     return textResponse("⚠️ Please mark your feature/bug as fixed first using 'mark_bug_fixed'!");
@@ -190,10 +211,7 @@ export async function handleCreateDocumentation(args, workflowState) {
     return textResponse("⚠️ Please ensure tests are passing first! Run 'run_tests' with passed=true.");
   }
 
-  const prdPath = path.join(process.cwd(), "docs", "PRD.md");
-  const prdExists = existsSync(prdPath);
-
-  if (!prdExists) {
+  if (!ensurePrdExists()) {
     return textResponse(
       `⚠️ PRD file not found!\n\nExpected location: docs/PRD.md\n\nPlease create or update the PRD before marking documentation as complete.`
     );

package/workflow-state.js

@@ -447,6 +447,8 @@ export class WorkflowState {
       commitAndPushCompleted: false,
       lastCommitMessage: "",
       lastPushBranch: "",
+      featureFlowCreated: false,
+      featureFlowDescription: "",
       history: [],
       taskType: "",
     };
@@ -500,6 +502,12 @@ export class WorkflowState {
       if (typeof this.state.lastPushBranch !== "string") {
         this.state.lastPushBranch = "";
       }
+      if (typeof this.state.featureFlowCreated !== "boolean") {
+        this.state.featureFlowCreated = false;
+      }
+      if (typeof this.state.featureFlowDescription !== "string") {
+        this.state.featureFlowDescription = "";
+      }
     } catch (error) {
       if (error?.code !== "ENOENT") {
         throw error;
@@ -637,6 +645,8 @@ export class WorkflowState {
       commitAndPushCompleted: false,
       lastCommitMessage: "",
       lastPushBranch: "",
+      featureFlowCreated: false,
+      featureFlowDescription: "",
       history: this.state.history,
     };
   }

package/.agent/workflows/dev-workflow-bugfix.md

@@ -1,250 +0,0 @@
----
-description: Bug fix workflow using dev-workflow MCP server
----
-
-# Bug Fix Workflow
-
-This workflow guides you through fixing bugs with the dev-workflow MCP server, ensuring proper testing and documentation.
-
-## When to Use
-
-- Fixing defects
--修复 unexpected behavior
-- Addressing user-reported issues
-- Fixing regressions
-
-## Workflow Steps
-
-### 1. Start the Task
-
-Begin by defining the bug:
-
-```
-Use start_task tool:
-- description: "Fix [bug description]"
-- type: "bugfix"
-```
-
-### 2. Reproduce and Fix
-
-1. **Reproduce** the bug locally
-2. **Identify** the root cause
-3. **Fix** the issue
-4. **Verify** the fix works
-
-Tips:
-- Document expected vs actual behavior
-- Take notes on the root cause
-- Test edge cases
-
-### 3. Mark as Fixed
-
-Once the fix is implemented:
-
-```
-Use mark_bug_fixed tool:
-- summary: "[Brief description of the fix]"
-```
-
-### 4. Create Regression Tests
-
-**Critical:** Write tests that would have caught this bug!
-
-Create regression tests:
-- Test the specific bug scenario
-- Test edge cases around the bug
-- Update existing tests if needed
-
-Then mark tests as created:
-
-```
-Use create_tests tool
-```
-
-### 5. Run Tests
-
-// turbo
-Execute your test suite:
-
-```bash
-npm test
-```
-
-Ensure the new regression tests pass:
-
-```
-Use run_tests tool:
-- passed: true
-- testCommand: "npm test"
-- details: "[Include regression test results]"
-```
-
-### 6. Document the Fix
-
-Update documentation:
-- **CHANGELOG** - Add bug fix entry
-- **Code comments** - Explain why the fix was necessary
-- **Issue tracker** - Reference issue number
-
-```
-Use create_documentation tool:
-- documentationType: "changelog"
-- summary: "[What you documented]"
-```
-
-### 7. Commit and Push
-
-Use conventional commit with `fix` type:
-
-```
-Use commit_and_push tool:
-- commitMessage: "fix([scope]): [description]"
-```
-
-Examples:
-- `fix(auth): prevent token expiration edge case`
-- `fix(ui): correct button alignment on mobile`
-
-### 8. Perform Release
-
-Bug fixes are patch releases:
-
-```
-Use perform_release tool:
-- command: "npm run release:patch"
-- notes: "v[X.Y.Z] - Bugfix for [issue]"
-```
-
-### 9. Complete Task
-
-```
-Use complete_task tool:
-- commitMessage: "fix([scope]): [description]"
-```
-
-## Example: Date Formatting Bug
-
-```
-1. start_task:
-     description: "Fix incorrect date format in transaction history"
-     type: "bugfix"
-
-2. [Reproduce bug, find issue in formatDate.js, implement fix]
-
-3. mark_bug_fixed:
-     summary: "Corrected date formatting to use ISO 8601 format"
-
-4. [Write regression test in formatDate.test.js]
-
-5. create_tests
-
-6. Run: npm test
-   
-7. run_tests:
-     passed: true
-     testCommand: "npm test -- utils/formatDate.test.js"
-     details: "2 new regression tests pass, all 45 tests pass"
-
-8. [Update CHANGELOG.md with fix entry]
-
-9. create_documentation:
-     documentationType: "changelog"
-     summary: "Added bug fix entry for date formatting issue"
-
-10. commit_and_push:
-      commitMessage: "fix(utils): correct date formatting to use ISO 8601"
-
-11. perform_release:
-      command: "npm run release:patch"
-      notes: "v1.2.1 - Date formatting hotfix"
-
-12. complete_task:
-      commitMessage: "fix(utils): correct date formatting to use ISO 8601"
-```
-
-## Handling Test Failures
-
-If tests fail during development:
-
-```
-run_tests:
-  passed: false
-  testCommand: "npm test"
-  details: "3 tests failed in auth module"
-```
-
-Response:
-```
-❌ Tests FAILED!
-🚫 Cannot proceed with failing tests
-```
-
-Fix the issues, then re-run:
-
-```
-run_tests:
-  passed: true
-  testCommand: "npm test"
-  details: "All tests now pass"
-```
-
-## Quick Reference
-
-```
-1. start_task → 2. Reproduce & fix → 3. mark_bug_fixed 
-→ 4. Write regression tests → 5. create_tests → 6. run_tests
-→ 7. Document → 8. create_documentation → 9. commit_and_push
-→ 10. perform_release → 11. complete_task
-```
-
-## Tips
-
-- **Regression tests** - Always write tests that would catch this bug
-- **Root cause** - Fix the underlying issue, not just symptoms
-- **Documentation** - Update CHANGELOG for user visibility
-- **Patch versions** - Bug fixes bump patch version (X.Y.Z)
-- **Conventional commits** - Use `fix` prefix
-
-## Hotfix Workflow
-
-For critical production bugs:
-
-```
-start_task:
-  description: "HOTFIX: [critical issue]"
-  type: "bugfix"
-
-[Implement minimal fix]
-
-mark_bug_fixed:
-  summary: "[Fix description]"
-
-[Focused regression test]
-
-create_tests
-
-run_tests:
-  passed: true
-  testCommand: "npm test -- [critical-path]"
-  details: "[Critical tests pass]"
-
-create_documentation:
-  documentationType: "changelog"
-  summary: "HOTFIX: [issue] - [fix]"
-
-commit_and_push:
-  commitMessage: "fix([scope])!: HOTFIX - [description]"
-
-perform_release:
-  command: "npm run release:patch"
-  notes: "v[X.Y.Z] - HOTFIX: [issue]"
-
-complete_task:
-  commitMessage: "fix([scope])!: HOTFIX - [description]"
-```
-
-## Need Help?
-
-- `get_workflow_status` - Check current phase
-- [EXAMPLES.md](../docs/antigravity/EXAMPLES.md) - Detailed examples
-- [TROUBLESHOOTING.md](../docs/antigravity/TROUBLESHOOTING.md) - Common issues