@programinglive/dev-workflow-mcp-server 1.1.5 → 1.1.7

package/CHANGELOG.md

@@ -2,6 +2,21 @@
 
 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.1.7](https://github.com/programinglive/dev-workflow-mcp-server/compare/v1.1.6...v1.1.7) (2025-10-26)
+
+
+### ✨ Features
+
+* prompt for documentation after tests pass ([a87e44f](https://github.com/programinglive/dev-workflow-mcp-server/commit/a87e44f2a78ea4c9220d62868ee1f9c7ad3b149b))
+
+### [1.1.6](https://github.com/programinglive/dev-workflow-mcp-server/compare/v1.1.5...v1.1.6) (2025-10-26)
+
+
+### 🐛 Bug Fixes
+
+* avoid root workflow state path ([a9bd01b](https://github.com/programinglive/dev-workflow-mcp-server/commit/a9bd01bac6b0a17307385688a51873668498e0a0))
+* require explicit commit before release ([5f58ead](https://github.com/programinglive/dev-workflow-mcp-server/commit/5f58ead9006595b7e57460fef585e3f9ea9de409))
+
 ### [1.1.5](https://github.com/programinglive/dev-workflow-mcp-server/compare/v1.1.4...v1.1.5) (2025-10-25)
 
 

package/package.json

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

package/tests/handlers.test.js

@@ -437,7 +437,7 @@ test("perform_release respects explicit releaseType option", async () => {
   });
 });
 
-test("perform_release auto-detects commit when tree is clean", async () => {
+test("perform_release requires commit_and_push completion when tree is clean", async () => {
   await withWorkflowState(async (workflowState) => {
     workflowState.state.currentPhase = "release";
     workflowState.state.commitAndPushCompleted = false;
@@ -449,10 +449,6 @@ test("perform_release auto-detects commit when tree is clean", async () => {
     const commands = [];
     const execStub = async (command) => {
       commands.push(command);
-      if (command === "git status --porcelain") {
-        return { stdout: "" };
-      }
-
       return { stdout: "" };
     };
 
@@ -475,15 +471,12 @@ test("perform_release auto-detects commit when tree is clean", async () => {
       utils,
     });
 
-    const pushCommand = commands.find((command) => command.startsWith("git push --follow-tags"));
-    assert.ok(pushCommand, "git push with tags should still run");
-    assert.equal(workflowState.state.commitAndPushCompleted, true);
-    assert.equal(workflowState.state.lastCommitMessage, "chore: already pushed");
-    assert.equal(workflowState.state.lastPushBranch, "main");
-    assert.equal(workflowState.state.currentPhase, "ready_to_complete");
+    assert.equal(commands.length, 0, "no release commands should run when commit_and_push not completed");
+    assert.equal(workflowState.state.currentPhase, "commit");
+    assert.equal(workflowState.state.commitAndPushCompleted, false);
     assert.ok(
-      response.content[0].text.includes("Detected clean working tree"),
-      "response should mention auto-detected commit"
+      response.content[0].text.includes("Please run 'commit_and_push' after the ready check"),
+      "response should instruct user to run commit_and_push"
     );
   });
 });

package/tools/handlers.js

@@ -290,7 +290,7 @@ async function handleRunTests(args, workflowState) {
   await workflowState.save();
 
   return textResponse(
-    `✅ 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. ⏳ Run 'check_ready_to_commit'\n6. ⏳ Run 'commit_and_push'\n7. ⏳ Run 'perform_release'\n8. ⏳ Complete task\n\nReminder: Document what you did before committing!`
+    `✅ All tests passed! 🎉\n\nTest command: ${args.testCommand}\n\n📝 Now create documentation using 'create_documentation' with:\n- documentationType: "README", "inline-comments", "API-docs", "changelog", or "other"\n- summary: Brief description of what was documented\n\nNext Steps:\n1. ✓ Fix/implement feature\n2. ✓ Create tests\n3. ✓ Run tests (GREEN!)\n4. ⏳ Create/update documentation\n5. ⏳ Run 'check_ready_to_commit'\n6. ⏳ Run 'commit_and_push'\n7. ⏳ Run 'perform_release'\n8. ⏳ Complete task`
   );
 }
 
@@ -480,8 +480,6 @@ async function handlePerformRelease(args, context) {
     );
   }
 
-  let autoDetectedCommit = false;
-
   if (!workflowState.state.commitAndPushCompleted) {
     const hasWorkingChanges = await git.hasWorkingChanges();
 
@@ -491,16 +489,12 @@ async function handlePerformRelease(args, context) {
       );
     }
 
-    const lastCommitMessage = await git.getLastCommitMessage();
-    const currentBranch = await git.getCurrentBranch();
-
-    workflowState.state.commitAndPushCompleted = true;
-    workflowState.state.lastCommitMessage =
-      lastCommitMessage || workflowState.state.lastCommitMessage || "";
-    workflowState.state.lastPushBranch = currentBranch || workflowState.state.lastPushBranch || "";
-    workflowState.state.currentPhase = "release";
-    autoDetectedCommit = true;
+    workflowState.state.currentPhase = "commit";
     await workflowState.save();
+
+    return textResponse(
+      "⚠️ Please run 'commit_and_push' after the ready check before recording a release!"
+    );
   }
 
   const rawReleaseCommand = typeof args.command === "string" ? args.command.trim() : "";
@@ -610,16 +604,10 @@ async function handlePerformRelease(args, context) {
 
   const message = summarizeHistory(workflowState.state.history, args.limit || 10);
   if (!message) {
-    const autoNote = autoDetectedCommit
-      ? "\n\nℹ️ Detected clean working tree and treated commit/push as already completed."
-      : "";
-    return textResponse(`📜 No workflow history yet.${autoNote}`);
+    return textResponse("📜 No workflow history yet.");
   }
 
-  const autoNote = autoDetectedCommit
-    ? "\n\nℹ️ Detected clean working tree and treated commit/push as already completed."
-    : "";
-  return textResponse(`${message}${autoNote}`);
+  return textResponse(message);
 }
 
 async function handleCompleteTask(args, workflowState) {

package/workflow-state.js

@@ -4,6 +4,16 @@ import { fileURLToPath } from "url";
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
+const STATE_FILENAME = ".workflow-state.json";
+
+function isRootPath(candidate) {
+  if (!candidate) {
+    return true;
+  }
+
+  const resolved = path.resolve(candidate);
+  return path.parse(resolved).root === resolved;
+}
 
 function resolveDefaultStateFile() {
   if (process.env.DEV_WORKFLOW_STATE_FILE) {
@@ -11,20 +21,20 @@ function resolveDefaultStateFile() {
   }
 
   const cwd = process.cwd();
-  if (!cwd.includes(`node_modules${path.sep}`)) {
-    return path.join(cwd, ".workflow-state.json");
+  if (!cwd.includes(`node_modules${path.sep}`) && !isRootPath(cwd)) {
+    return path.join(cwd, STATE_FILENAME);
   }
 
-  if (process.env.INIT_CWD) {
-    return path.join(process.env.INIT_CWD, ".workflow-state.json");
+  if (process.env.INIT_CWD && !isRootPath(process.env.INIT_CWD)) {
+    return path.join(process.env.INIT_CWD, STATE_FILENAME);
   }
 
   const [beforeNodeModules] = cwd.split(`node_modules${path.sep}`);
-  if (beforeNodeModules) {
-    return path.join(beforeNodeModules, ".workflow-state.json");
+  if (beforeNodeModules && !isRootPath(beforeNodeModules)) {
+    return path.join(beforeNodeModules, STATE_FILENAME);
   }
 
-  return path.join(__dirname, ".workflow-state.json");
+  return path.join(__dirname, STATE_FILENAME);
 }
 
 export class WorkflowState {