@programinglive/dev-workflow-mcp-server 1.0.8 → 1.0.11

package/CONTRIBUTING.md

@@ -1,60 +1,60 @@
-# Contributing to dev-workflow-mcp-server
-
-Thanks for your interest in contributing! Follow the guidelines below to keep the development process smooth and aligned with the workflow enforced by this project.
-
-## Development Workflow
-
-1. **Start a task** using the MCP server's `start_task` tool with a clear description and task type.
-2. **Implement the change** following the single flow rules documented in the README.
-3. **Write or update tests**. All code changes must include tests when applicable.
-4. **Run tests** with `npm test` and ensure they pass.
-5. **Document changes**. Update `README.md` or other docs as needed.
-6. **Commit** following conventional commits (e.g., `feat: add new tool`).
-7. **Push** your branch and open a pull request using the provided template.
-
-## Getting Started
-
-- Fork the repository and clone it locally.
-- Install dependencies:
-
-```bash
-npm install
-```
-
-- Run the test suite to verify your setup:
-
-```bash
-npm test
-```
-
-## Pull Requests
-
-- Use a descriptive branch name (e.g., `feat/new-tool` or `fix/workflow-reset`).
-- Fill out the pull request template completely, including tests and documentation details.
-- Link related issues in the PR description.
-- Ensure CI checks pass before requesting review.
-
-## Coding Standards
-
-- Use TypeScript-friendly patterns even in JavaScript (avoid dynamic typing pitfalls).
-- Remove unused imports and variables before committing.
-- Follow existing code style and formatting.
-
-## Commit Messages
-
-- Follow [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/).
-- Example: `feat: enforce documentation reminder before commit`.
-
-## Reporting Issues
-
-- Search open issues to avoid duplicates.
-- When filing a bug, include reproduction steps, expected behavior, and environment details.
-- Feature requests should explain the problem being solved and potential benefits.
-
-## Code of Conduct
-
-By participating, you agree to abide by the [Code of Conduct](./CODE_OF_CONDUCT.md).
-
-## Contact
-
-Questions? Reach out at `opensource@programinglive.com`.
+# Contributing to dev-workflow-mcp-server
+
+Thanks for your interest in contributing! Follow the guidelines below to keep the development process smooth and aligned with the workflow enforced by this project.
+
+## Development Workflow
+
+1. **Start a task** using the MCP server's `start_task` tool with a clear description and task type.
+2. **Implement the change** following the single flow rules documented in the README.
+3. **Write or update tests**. All code changes must include tests when applicable.
+4. **Run tests** with `npm test` and ensure they pass.
+5. **Document changes**. Update `README.md` or other docs as needed.
+6. **Commit** following conventional commits (e.g., `feat: add new tool`).
+7. **Push** your branch and open a pull request using the provided template.
+
+## Getting Started
+
+- Fork the repository and clone it locally.
+- Install dependencies:
+
+```bash
+npm install
+```
+
+- Run the test suite to verify your setup:
+
+```bash
+npm test
+```
+
+## Pull Requests
+
+- Use a descriptive branch name (e.g., `feat/new-tool` or `fix/workflow-reset`).
+- Fill out the pull request template completely, including tests and documentation details.
+- Link related issues in the PR description.
+- Ensure CI checks pass before requesting review.
+
+## Coding Standards
+
+- Use TypeScript-friendly patterns even in JavaScript (avoid dynamic typing pitfalls).
+- Remove unused imports and variables before committing.
+- Follow existing code style and formatting.
+
+## Commit Messages
+
+- Follow [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/).
+- Example: `feat: enforce documentation reminder before commit`.
+
+## Reporting Issues
+
+- Search open issues to avoid duplicates.
+- When filing a bug, include reproduction steps, expected behavior, and environment details.
+- Feature requests should explain the problem being solved and potential benefits.
+
+## Code of Conduct
+
+By participating, you agree to abide by the [Code of Conduct](./CODE_OF_CONDUCT.md).
+
+## Contact
+
+Questions? Reach out at `opensource@programinglive.com`.

package/LICENSE

@@ -1,21 +1,21 @@
-MIT License
-
-Copyright (c) 2025 programinglive
-
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in all
-copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
-SOFTWARE.
+MIT License
+
+Copyright (c) 2025 programinglive
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

package/README.md

@@ -1,66 +1,66 @@
-# Development Workflow MCP Server
-
-An MCP (Model Context Protocol) server that helps enforce development discipline and workflow best practices. This server acts as your coding conscience, reminding you to follow proper development workflows.
-
-## 🎯 Purpose
-
-This MCP server guides you through a disciplined development workflow:
-
-1. **Start Conscious** - Be clear about what you're coding
-2. **Fix/Implement** - Write your code
-3. **Create Tests** - Always test your changes
-4. **Run Tests** - Tests must pass (GREEN)
-5. **Document** - Update documentation
-6. **Commit & Push** - Let the server drive the git commands once verification passes
-7. **Release** - Record the release details before closing out the task
-
-## 🚀 Installation
-
-### 1. Install Dependencies
-
-```bash
-npm install
-```
-
-### 2. Configure in Windsurf/Claude Desktop
-
-Point your MCP client to the server entry point. Replace `<PROJECT_ROOT>` with the absolute path to this repository on your machine.
-
-#### macOS
-
-- **Windsurf** (`~/Library/Application Support/Windsurf/config.json`):
-
-```json
-{
-  "mcpServers": {
-    "dev-workflow": {
-      "command": "node",
-      "args": ["<PROJECT_ROOT>/index.js"]
-    }
-  }
-}
-```
-
-- **Claude Desktop** (`~/Library/Application Support/Claude/claude_desktop_config.json`):
-
-```json
-{
-  "mcpServers": {
-    "dev-workflow": {
-      "command": "node",
-      "args": ["<PROJECT_ROOT>/index.js"]
-    }
-  }
-}
-```
-
-#### Windows
-
-- **Windsurf** (`%APPDATA%\Windsurf\config.json`):
-
-```json
-{
-  "mcpServers": {
+# Development Workflow MCP Server
+
+An MCP (Model Context Protocol) server that helps enforce development discipline and workflow best practices. This server acts as your coding conscience, reminding you to follow proper development workflows.
+
+## 🎯 Purpose
+
+This MCP server guides you through a disciplined development workflow:
+
+1. **Start Conscious** - Be clear about what you're coding
+2. **Fix/Implement** - Write your code
+3. **Create Tests** - Always test your changes
+4. **Run Tests** - Tests must pass (GREEN)
+5. **Document** - Update documentation
+6. **Commit & Push** - Let the server drive the git commands once verification passes
+7. **Release** - Record the release details before closing out the task
+
+## 🚀 Installation
+
+### 1. Install Dependencies
+
+```bash
+npm install
+```
+
+### 2. Configure in Windsurf/Claude Desktop
+
+Point your MCP client to the server entry point. Replace `<PROJECT_ROOT>` with the absolute path to this repository on your machine.
+
+#### macOS
+
+- **Windsurf** (`~/Library/Application Support/Windsurf/config.json`):
+
+```json
+{
+  "mcpServers": {
+    "dev-workflow": {
+      "command": "node",
+      "args": ["<PROJECT_ROOT>/index.js"]
+    }
+  }
+}
+```
+
+- **Claude Desktop** (`~/Library/Application Support/Claude/claude_desktop_config.json`):
+
+```json
+{
+  "mcpServers": {
+    "dev-workflow": {
+      "command": "node",
+      "args": ["<PROJECT_ROOT>/index.js"]
+    }
+  }
+}
+```
+
+#### Windows
+
+- **Windsurf** (`%APPDATA%\Windsurf\config.json`):
+
+```json
+{
+  "mcpServers": {
     "dev-workflow": {
       "command": "node",
       "args": ["<PROJECT_ROOT>\\index.js"]
@@ -333,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/SECURITY.md

@@ -1,20 +1,20 @@
-# Security Policy
-
-## Supported Versions
-
-We release patches for security vulnerabilities under new minor or patch versions. Always use the latest release.
-
-## Reporting a Vulnerability
-
-If you discover a vulnerability, please email `security@programinglive.com` with:
-
-- Description of the issue
-- Steps to reproduce
-- Potential impact
-- Suggested remediation (if known)
-
-We aim to acknowledge reports within 72 hours and will keep you informed as we work on a fix. Please do not disclose the vulnerability publicly until we have released an update.
-
-## Disclosure Policy
-
-Once a fix is available, we will publish a security advisory in the repository releases and notify contributors via the README.
+# Security Policy
+
+## Supported Versions
+
+We release patches for security vulnerabilities under new minor or patch versions. Always use the latest release.
+
+## Reporting a Vulnerability
+
+If you discover a vulnerability, please email `security@programinglive.com` with:
+
+- Description of the issue
+- Steps to reproduce
+- Potential impact
+- Suggested remediation (if known)
+
+We aim to acknowledge reports within 72 hours and will keep you informed as we work on a fix. Please do not disclose the vulnerability publicly until we have released an update.
+
+## Disclosure Policy
+
+Once a fix is available, we will publish a security advisory in the repository releases and notify contributors via the README.

package/commit-helpers.js

@@ -0,0 +1,166 @@
+function formatFileList(paths) {
+  const uniquePaths = [...new Set(paths.filter(Boolean))];
+
+  if (uniquePaths.length === 0) {
+    return "updates";
+  }
+
+  if (uniquePaths.length === 1) {
+    return uniquePaths[0];
+  }
+
+  if (uniquePaths.length === 2) {
+    return `${uniquePaths[0]} and ${uniquePaths[1]}`;
+  }
+
+  if (uniquePaths.length === 3) {
+    return `${uniquePaths[0]}, ${uniquePaths[1]}, and ${uniquePaths[2]}`;
+  }
+
+  const remaining = uniquePaths.length - 3;
+  return `${uniquePaths[0]}, ${uniquePaths[1]}, ${uniquePaths[2]} +${remaining} more`;
+}
+
+function determineCommitType(changes) {
+  const hasTestChange = changes.some((change) =>
+    [change.path, change.originalPath].some((p) => isTestFilePath(p))
+  );
+  const hasDocsChange = changes.some((change) =>
+    [change.path, change.originalPath].some((p) => isDocumentationFilePath(p))
+  );
+
+  if (hasTestChange) {
+    return "test";
+  }
+
+  if (hasDocsChange) {
+    return "docs";
+  }
+
+  return "chore";
+}
+
+function isTestFilePath(filePath) {
+  if (!filePath) {
+    return false;
+  }
+
+  if (/\.(test|spec)\.[^.]+$/i.test(filePath)) {
+    return true;
+  }
+
+  return /(\/|^)(__tests__|tests?|spec)\//i.test(filePath);
+}
+
+function isDocumentationFilePath(filePath) {
+  if (!filePath) {
+    return false;
+  }
+
+  const normalized = filePath.toLowerCase();
+  return (
+    normalized.startsWith("docs/") ||
+    normalized.startsWith("documentation/") ||
+    normalized.endsWith(".md") ||
+    normalized.endsWith(".rst") ||
+    normalized.endsWith(".adoc") ||
+    normalized.includes("readme")
+  );
+}
+
+function describeChange(change) {
+  const statusCode = change.status || "";
+  const path = change.path || change.originalPath || "";
+
+  if (statusCode.startsWith("R")) {
+    return `renamed ${change.originalPath} -> ${change.path}`;
+  }
+
+  if (statusCode.startsWith("C")) {
+    return `copied ${change.originalPath} -> ${change.path}`;
+  }
+
+  switch (statusCode[0]) {
+    case "A":
+      return `added ${path}`;
+    case "M":
+      return `modified ${path}`;
+    case "D":
+      return `removed ${path}`;
+    case "U":
+      return `updated ${path}`;
+    default:
+      return `${statusCode} ${path}`.trim();
+  }
+}
+
+export function createCommitMessageParts(changes, providedSummary = "") {
+  const summaryInput = typeof providedSummary === "string" ? providedSummary.trim() : "";
+
+  if (!Array.isArray(changes) || changes.length === 0) {
+    return {
+      summary: summaryInput,
+      body: "",
+    };
+  }
+
+  const commitType = determineCommitType(changes);
+  const summaryPaths = changes
+    .map((change) => change.path || change.originalPath)
+    .filter(Boolean);
+  const generatedSummary = `${commitType}: update ${formatFileList(summaryPaths)}`;
+  const summary = summaryInput || generatedSummary;
+
+  const detailLines = changes
+    .map((change) => describeChange(change))
+    .filter((line) => line && line.trim().length > 0)
+    .map((line) => `- ${line}`);
+
+  return {
+    summary,
+    body: detailLines.join("\n"),
+  };
+}
+
+function normalizeCommitHeader(message) {
+  if (typeof message !== "string") {
+    return "";
+  }
+
+  const header = message.split("\n")[0] || "";
+  return header.trim();
+}
+
+function containsBreakingChange(message) {
+  if (typeof message !== "string") {
+    return false;
+  }
+
+  return /breaking change/i.test(message);
+}
+
+export function determineReleaseTypeFromCommit(message) {
+  const header = normalizeCommitHeader(message).toLowerCase();
+  const hasBreaking = containsBreakingChange(message) || /!/.test(header.split(":")[0] || "");
+
+  if (hasBreaking) {
+    return "major";
+  }
+
+  const typeMatch = header.match(/^([a-z]+)(\(|:)/i);
+  const commitType = typeMatch ? typeMatch[1] : "";
+
+  if (commitType === "feat") {
+    return "minor";
+  }
+
+  if (commitType === "fix" || commitType === "perf") {
+    return "patch";
+  }
+
+  if (!commitType && containsBreakingChange(message)) {
+    return "major";
+  }
+
+  return "patch";
+}

package/commitlint.config.js

@@ -1,3 +1,3 @@
-export default {
-  extends: ["@commitlint/config-conventional"],
-};
+export default {
+  extends: ["@commitlint/config-conventional"],
+};

package/exec.js

@@ -0,0 +1,4 @@
+import { exec as execCallback } from "child_process";
+import { promisify } from "util";
+
+export const exec = promisify(execCallback);

package/git-helpers.js

@@ -0,0 +1,152 @@
+import { exec } from "./exec.js";
+
+export async function hasStagedChanges() {
+  try {
+    const { stdout } = await exec("git status --porcelain");
+    return stdout
+      .split("\n")
+      .filter(Boolean)
+      .some((line) => {
+        const indexStatus = line[0];
+        return indexStatus && indexStatus !== " " && indexStatus !== "?";
+      });
+  } catch (error) {
+    return false;
+  }
+}
+
+export async function hasWorkingChanges() {
+  try {
+    const { stdout } = await exec("git status --porcelain");
+    return stdout.split("\n").some((line) => line && line.trim().length > 0);
+  } catch (error) {
+    return false;
+  }
+}
+
+export function workingTreeSummary(statusOutput) {
+  if (typeof statusOutput !== "string" || statusOutput.trim() === "") {
+    return {
+      hasChanges: false,
+      lines: [],
+    };
+  }
+
+  const lines = statusOutput
+    .split("\n")
+    .map((line) => line.replace(/\s+/g, " ").trim())
+    .filter(Boolean);
+
+  return {
+    hasChanges: lines.length > 0,
+    lines,
+  };
+}
+
+function isTestFilePath(filePath) {
+  if (!filePath) {
+    return false;
+  }
+
+  if (/\.(test|spec)\.[^.]+$/i.test(filePath)) {
+    return true;
+  }
+
+  return /(\/|^)(__tests__|tests?|spec)\//i.test(filePath);
+}
+
+function extractPathFromStatusLine(line) {
+  if (!line || line.length < 4) {
+    return "";
+  }
+
+  const rawPath = line.slice(3).trim();
+  if (!rawPath) {
+    return "";
+  }
+
+  if (rawPath.includes(" -> ")) {
+    const parts = rawPath.split(" -> ");
+    return parts[parts.length - 1].trim();
+  }
+
+  return rawPath;
+}
+
+export function containsTestFilesInStatus(statusOutput) {
+  if (typeof statusOutput !== "string" || statusOutput.trim() === "") {
+    return false;
+  }
+
+  return statusOutput
+    .split("\n")
+    .map((line) => extractPathFromStatusLine(line))
+    .some((path) => isTestFilePath(path));
+}
+
+function parseNameStatusLine(line) {
+  if (!line) {
+    return null;
+  }
+
+  const parts = line.split("\t").map((part) => part.trim());
+  const status = parts[0];
+  if (!status) {
+    return null;
+  }
+
+  if (status.startsWith("R") || status.startsWith("C")) {
+    const originalPath = parts[1] || "";
+    const newPath = parts[2] || originalPath;
+    return {
+      status,
+      path: newPath,
+      originalPath,
+    };
+  }
+
+  return {
+    status,
+    path: parts[1] || "",
+    originalPath: "",
+  };
+}
+
+export async function getStagedChanges() {
+  try {
+    const { stdout } = await exec("git diff --cached --name-status");
+    return stdout
+      .split("\n")
+      .map((line) => parseNameStatusLine(line))
+      .filter(Boolean);
+  } catch (error) {
+    return [];
+  }
+}
+
+export async function hasTestChanges() {
+  try {
+    const { stdout } = await exec("git status --porcelain");
+    return containsTestFilesInStatus(stdout);
+  } catch (error) {
+    return false;
+  }
+}
+
+export async function getLastCommitMessage() {
+  try {
+    const { stdout } = await exec("git log -1 --pretty=%B");
+    return stdout.trim();
+  } catch (error) {
+    return "";
+  }
+}
+
+export async function getCurrentBranch() {
+  try {
+    const { stdout } = await exec("git rev-parse --abbrev-ref HEAD");
+    return stdout.trim();
+  } catch (error) {
+    return "";
+  }
+}