@programinglive/dev-workflow-mcp-server 1.3.9 → 1.3.11

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.3.11](https://github.com/programinglive/dev-workflow-mcp-server/compare/v1.3.10...v1.3.11) (2025-11-22)
+
+
+### ♻️ Refactors
+
+* **web:** move static files to public/ subdirectory for Plesk security ([3e5c321](https://github.com/programinglive/dev-workflow-mcp-server/commit/3e5c321d569e3594d9f29c8762513b95df8dbd84))
+
+### [1.3.10](https://github.com/programinglive/dev-workflow-mcp-server/compare/v1.3.9...v1.3.10) (2025-11-22)
+
+
+### 📝 Documentation
+
+* add workflow best practices to eliminate forced releases ([6bcac42](https://github.com/programinglive/dev-workflow-mcp-server/commit/6bcac42bc9f30cbfa0a27e36a90af303d244d863))
+
 ### [1.3.9](https://github.com/programinglive/dev-workflow-mcp-server/compare/v1.3.8...v1.3.9) (2025-11-22)
 
 

package/docs/WORKFLOW-BEST-PRACTICES.md

@@ -0,0 +1,44 @@
+# Workflow Best Practices
+
+## Correct Release Workflow Order
+
+To avoid needing `--force` flag for releases, follow this exact order:
+
+1. **Start Task**: `mcp0_start_task`
+2. **Fix/Implement**: Make your code changes
+3. **Mark Fixed**: `mcp0_mark_bug_fixed`
+4. **Tests**: `mcp0_create_tests` + `mcp0_run_tests` (or `mcp0_skip_tests` with reason)
+5. **Documentation**: `mcp0_create_documentation`
+6. **Ready Check**: `mcp0_check_ready_to_commit`
+7. **Commit & Push**: `mcp0_commit_and_push`
+8. **Release**: `mcp0_perform_release` ⚠️ **MUST come BEFORE complete_task**
+9. **Complete**: `mcp0_complete_task`
+
+## Common Mistakes
+
+### ❌ Wrong Order (causes forced releases)
+```
+check_ready → commit → complete → release  // WRONG!
+```
+
+### ✅ Correct Order
+```
+check_ready → commit → release → complete  // CORRECT!
+```
+
+## Why This Matters
+
+- `complete_task` **resets the workflow state**
+- `perform_release` **checks the workflow state** before running
+- If you complete before releasing, the state is cleared and release guards fail
+- Using `--force` bypasses guards but doesn't fix the root cause
+
+## State Transitions
+
+1. After `commit_and_push`: `currentPhase = "release"`
+2. After `perform_release`: `currentPhase = "ready_to_complete"`, `released = true`
+3. After `complete_task`: State is reset to idle
+
+## Solution
+
+Always call `perform_release` BEFORE `complete_task`. Never use `--force` unless absolutely necessary.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@programinglive/dev-workflow-mcp-server",
-  "version": "1.3.9",
+  "version": "1.3.11",
   "description": "MCP server to enforce development workflow discipline",
   "homepage": "https://devworkflow.programinglive.com/",
   "type": "module",
@@ -10,8 +10,8 @@
   },
   "scripts": {
     "build": "vite build",
-    "build:web": "npx tailwindcss -i ./web/input.css -o ./web/output.css --minify",
-    "watch:web": "npx tailwindcss -i ./web/input.css -o ./web/output.css --watch",
+    "build:web": "npx tailwindcss -i ./web/public/input.css -o ./web/public/output.css --minify",
+    "watch:web": "npx tailwindcss -i ./web/public/input.css -o ./web/public/output.css --watch",
     "postbuild": "npm run build:web && node scripts/build-docs.js",
     "dev": "node --watch index.js",
     "start": "node index.js",

package/tailwind.config.js

@@ -1,8 +1,6 @@
 /** @type {import('tailwindcss').Config} */
-export default {
-    content: [
-        "./web/**/*.{html,js}",
-    ],
+module.exports = {
+    content: ["./web/public/**/*.{html,js}"],
     theme: {
         extend: {},
     },

package/web/server.js

@@ -83,11 +83,11 @@ async function start() {
       const userId = url.searchParams.get("user") || "default";
 
       if (req.method === "GET" && url.pathname === "/output.css") {
-        const css = await readFile(join(__dirname, "output.css"), "utf-8");
+        const css = await readFile(join(__dirname, "public", "output.css"), "utf-8");
         res.writeHead(200, { "Content-Type": "text/css" });
         res.end(css);
       } else if (req.method === "GET" && url.pathname === "/") {
-        const html = await readFile(join(__dirname, "index.html"), "utf-8");
+        const html = await readFile(join(__dirname, "public", "index.html"), "utf-8");
         res.writeHead(200, { "Content-Type": "text/html" });
         res.end(html);
       } else if (req.method === "GET" && url.pathname === "/api/summary") {