@programinglive/dev-workflow-mcp-server 1.5.0 → 1.5.2

package/install.js

@@ -1,73 +1,73 @@
-#!/usr/bin/env node
-
-/**
- * Installation script for dev-workflow MCP server
- * Sets up per-project workflow state file in the target project directory
- */
-
-import fs from 'fs/promises';
-import path from 'path';
-import { fileURLToPath } from 'url';
-
-const __filename = fileURLToPath(import.meta.url);
-const __dirname = path.dirname(__filename);
-
-async function install() {
-  try {
-    // npm sets INIT_CWD to the directory where `npm install` was originally invoked.
-    // Fallback to process.cwd() if it's not present (e.g., older npm versions).
-    const projectRoot = process.env.INIT_CWD || process.cwd();
-
-    // If INIT_CWD points inside node_modules, this script is being executed
-    // while installing the package itself. In that case, skip creating a state file.
-    if (projectRoot.includes(`node_modules${path.sep}`)) {
-      console.log('ℹ️ Skipping workflow state creation inside node_modules.');
-      return;
-    }
-
-    const stateDir = path.join(projectRoot, '.state');
-    const stateFile = path.join(stateDir, 'workflow-state.json');
-
-    // Create .state directory if it doesn't exist
-    await fs.mkdir(stateDir, { recursive: true });
-
-    // Check if state file already exists
-    try {
-      await fs.access(stateFile);
-      console.log(`✓ Workflow state file already exists at ${stateFile}`);
-      return;
-    } catch {
-      // File doesn't exist, create it
-    }
-
-    // Create initial workflow state
-    const initialState = {
-      currentPhase: 'idle',
-      taskDescription: '',
-      bugFixed: false,
-      testsCreated: false,
-      testsPassed: false,
-      testsSkipped: false,
-      testsSkippedReason: '',
-      documentationCreated: false,
-      readyToCommit: false,
-      readyCheckCompleted: false,
-      released: false,
-      releaseCommand: '',
-      releaseNotes: '',
-      commitAndPushCompleted: false,
-      lastCommitMessage: '',
-      lastPushBranch: '',
-      history: [],
-    };
-
-    await fs.writeFile(stateFile, JSON.stringify(initialState, null, 2));
-    console.log(`✓ Created workflow state file at ${stateFile}`);
-    console.log(`✓ Each project now has its own isolated workflow history`);
-  } catch (error) {
-    console.error('✗ Installation failed:', error.message);
-    process.exit(1);
-  }
-}
-
-install();
+#!/usr/bin/env node
+
+/**
+ * Installation script for dev-workflow MCP server
+ * Sets up per-project workflow state file in the target project directory
+ */
+
+import fs from 'fs/promises';
+import path from 'path';
+import { fileURLToPath } from 'url';
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
+
+async function install() {
+  try {
+    // npm sets INIT_CWD to the directory where `npm install` was originally invoked.
+    // Fallback to process.cwd() if it's not present (e.g., older npm versions).
+    const projectRoot = process.env.INIT_CWD || process.cwd();
+
+    // If INIT_CWD points inside node_modules, this script is being executed
+    // while installing the package itself. In that case, skip creating a state file.
+    if (projectRoot.includes(`node_modules${path.sep}`)) {
+      console.log('ℹ️ Skipping workflow state creation inside node_modules.');
+      return;
+    }
+
+    const stateDir = path.join(projectRoot, '.state');
+    const stateFile = path.join(stateDir, 'workflow-state.json');
+
+    // Create .state directory if it doesn't exist
+    await fs.mkdir(stateDir, { recursive: true });
+
+    // Check if state file already exists
+    try {
+      await fs.access(stateFile);
+      console.log(`✓ Workflow state file already exists at ${stateFile}`);
+      return;
+    } catch {
+      // File doesn't exist, create it
+    }
+
+    // Create initial workflow state
+    const initialState = {
+      currentPhase: 'idle',
+      taskDescription: '',
+      bugFixed: false,
+      testsCreated: false,
+      testsPassed: false,
+      testsSkipped: false,
+      testsSkippedReason: '',
+      documentationCreated: false,
+      readyToCommit: false,
+      readyCheckCompleted: false,
+      released: false,
+      releaseCommand: '',
+      releaseNotes: '',
+      commitAndPushCompleted: false,
+      lastCommitMessage: '',
+      lastPushBranch: '',
+      history: [],
+    };
+
+    await fs.writeFile(stateFile, JSON.stringify(initialState, null, 2));
+    console.log(`✓ Created workflow state file at ${stateFile}`);
+    console.log(`✓ Each project now has its own isolated workflow history`);
+  } catch (error) {
+    console.error('✗ Installation failed:', error.message);
+    process.exit(1);
+  }
+}
+
+install();

package/package.json

@@ -1,18 +1,20 @@
 {
   "name": "@programinglive/dev-workflow-mcp-server",
-  "version": "1.5.0",
+  "version": "1.5.2",
   "description": "MCP server to enforce development workflow discipline",
   "homepage": "https://devworkflow.programinglive.com/",
+  "mcpName": "io.github.programinglive/dev-workflow-mcp-server",
   "type": "module",
   "main": "index.js",
   "bin": {
     "dev-workflow-mcp": "index.js"
   },
   "scripts": {
-    "build": "vite build",
-    "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",
+    "build": "npm run build:server && npm run build:web",
+    "build:server": "vite build",
+    "build:web": "npm run build --prefix web",
+    "watch:web": "npm run dev --prefix web",
+    "postbuild": "node scripts/build-docs.js",
     "dev": "node --watch index.js",
     "start": "node index.js",
     "local": "node index.js",
@@ -21,10 +23,10 @@
     "prepare": "husky",
     "postinstall": "node install.js",
     "release": "node scripts/release.cjs",
-    "release:major": "node release-wrapper.js major",
-    "release:minor": "node release-wrapper.js minor",
-    "release:patch": "node release-wrapper.js patch",
-    "web": "node web/server.js"
+    "release:major": "node scripts/release.cjs major",
+    "release:minor": "node scripts/release.cjs minor",
+    "release:patch": "node scripts/release.cjs patch",
+    "web": "npm start --prefix web"
   },
   "keywords": [
     "mcp",

package/release-wrapper.js

@@ -1,157 +1,192 @@
-#!/usr/bin/env node
-
-import { spawn } from 'node:child_process';
-import fs from 'node:fs/promises';
-import path from 'node:path';
-
-const RELEASE_TYPES = new Set(['major', 'minor', 'patch']);
-
-async function main() {
-  const releaseTypeArg = (process.argv[2] || '').toLowerCase();
-  const forceFlag = process.argv.includes('--force') || process.env.DEV_WORKFLOW_FORCE_RELEASE === '1';
-  const additionalArgs = process.argv.slice(3).filter((arg) => arg !== '--force');
-
-  if (!RELEASE_TYPES.has(releaseTypeArg)) {
-    console.error(
-      '❌ Release guard: expected release type argument (major | minor | patch). Example: npm run release:patch'
-    );
-    process.exit(1);
-  }
-
-  const repoRoot = process.cwd();
-  const projectRootCandidate = process.env.INIT_CWD || repoRoot;
-  if (projectRootCandidate.includes(`node_modules${path.sep}`)) {
-    console.error('❌ Release guard: refusing to run inside node_modules.');
-    process.exit(1);
-  }
-  const projectRoot = projectRootCandidate;
-
-  // Prefer state under INIT_CWD to support per-project runs and tests.
-  // If DEV_WORKFLOW_USER_ID is set and a user-scoped file exists, use it; otherwise fall back to legacy path.
-  const baseStateDir = path.join(projectRoot, '.state');
-  const legacyStatePath = path.join(baseStateDir, 'workflow-state.json');
-  let resolvedUserId = (process.env.DEV_WORKFLOW_USER_ID || '').trim();
-  if (!resolvedUserId) {
-    try {
-      const fileUserId = (await fs.readFile(path.join(baseStateDir, 'user-id'), 'utf8')).trim();
-      if (fileUserId) {
-        resolvedUserId = fileUserId;
-      }
-    } catch {
-      // no user-id file; legacy layout likely
-    }
-  }
-  const userStatePath = resolvedUserId ? path.join(baseStateDir, 'users', resolvedUserId, 'workflow-state.json') : null;
-
-  let statePath = legacyStatePath;
-  if (userStatePath) {
-    try {
-      await fs.access(userStatePath);
-      statePath = userStatePath;
-    } catch {
-      // fall back to legacy path
-    }
-  }
-  let state;
-
-  try {
-    const raw = await fs.readFile(statePath, 'utf8');
-    state = JSON.parse(raw);
-  } catch (error) {
-    console.error(
-      `❌ Release guard: cannot locate workflow state at ${statePath}. Make sure the dev-workflow MCP is installed and you have started a task.`
-    );
-    process.exit(1);
-  }
-
-  const errors = [];
-  if (state.currentPhase !== 'release') {
-    errors.push(
-      "Current workflow phase isn't 'release'. Complete documentation, commit, and push steps first."
-    );
-  }
-  if (!state.readyCheckCompleted) {
-    errors.push("'check_ready_to_commit' has not been completed.");
-  }
-  if (!state.commitAndPushCompleted) {
-    errors.push("'commit_and_push' has not been completed.");
-  }
-  if (state.released) {
-    errors.push('A release has already been recorded for this task.');
-  }
-
-  if (errors.length && !forceFlag) {
-    console.error('❌ Release guard blocked the release:');
-    for (const message of errors) {
-      console.error(`  • ${message}`);
-    }
-    console.error("\nUse 'perform_release' through the dev-workflow MCP after completing the missing steps.");
-    console.error("Or use: npm run release:patch -- --force (to bypass guard)");
-    process.exit(1);
-  }
-
-  if (forceFlag && errors.length) {
-    console.warn('⚠️ --force flag detected. Bypassing workflow guard.');
-    console.warn('Errors that were skipped:');
-    for (const message of errors) {
-      console.warn(`  • ${message}`);
-    }
-  }
-
-  const commiterReleaseScript = path.join(repoRoot, 'scripts', 'release.cjs');
-
-  try {
-    await fs.access(commiterReleaseScript);
-  } catch {
-    console.error('❌ Release guard: @programinglive/commiter release script not found.');
-    console.error('   Ensure @programinglive/commiter is installed as a dev dependency.');
-    console.error('   Try: npm install --save-dev @programinglive/commiter');
-    process.exit(1);
-  }
-
-  const runRelease = async () => {
-    if (process.env.DEV_WORKFLOW_SKIP_RELEASE === '1') {
-      console.log('ℹ️ DEV_WORKFLOW_SKIP_RELEASE=1 detected. Skipping actual release command.');
-      return 0;
-    }
-
-    return await new Promise((resolve, reject) => {
-      const child = spawn(
-        process.execPath,
-        [commiterReleaseScript, releaseTypeArg, ...additionalArgs],
-        { stdio: 'inherit', cwd: repoRoot }
-      );
-
-      child.on('error', (error) => reject(error));
-      child.on('close', (code) => resolve(code));
-    });
-  };
-
-  try {
-    const exitCode = await runRelease();
-    if (exitCode !== 0) {
-      console.error(`❌ Release command exited with code ${exitCode}.`);
-      process.exit(exitCode);
-    }
-  } catch (error) {
-    console.error(`❌ Failed to run release command: ${error.message}`);
-    process.exit(1);
-  }
-
-  const recordedReleaseCommand = `node node_modules/@programinglive/commiter/scripts/release.js ${releaseTypeArg}`;
-
-  state.released = true;
-  state.releaseCommand = recordedReleaseCommand;
-  state.currentPhase = 'ready_to_complete';
-
-  try {
-    await fs.writeFile(statePath, JSON.stringify(state, null, 2));
-  } catch (error) {
-    console.error(`❌ Release guard: failed to update workflow state. ${error.message}`);
-    process.exit(1);
-  }
-
-  console.log('✅ Release recorded in workflow state. You can now run complete_task.');
-}
-
-main();
+#!/usr/bin/env node
+
+import { spawn } from 'node:child_process';
+import fs from 'node:fs/promises';
+import path from 'node:path';
+import { resolveStateFile } from './workflow-state.js';
+
+const RELEASE_TYPES = new Set(['major', 'minor', 'patch']);
+
+async function readStateCandidate(filePath) {
+  if (!filePath) {
+    return null;
+  }
+
+  try {
+    const [raw, stats] = await Promise.all([fs.readFile(filePath, 'utf8'), fs.stat(filePath)]);
+    return {
+      state: JSON.parse(raw),
+      path: filePath,
+      mtimeMs: stats.mtimeMs,
+    };
+  } catch {
+    return null;
+  }
+}
+
+async function loadWorkflowStateCandidates(projectRoot) {
+  const candidates = [];
+
+  try {
+    candidates.push(resolveStateFile());
+  } catch {
+    // resolver may throw before state exists; fall through to other candidates
+  }
+
+  const legacyStatePath = path.join(projectRoot, '.state', 'workflow-state.json');
+  candidates.push(legacyStatePath);
+
+  // Add explicit DEV_WORKFLOW_STATE_FILE if provided (resolver would have used it, but keep explicit just in case)
+  if (process.env.DEV_WORKFLOW_STATE_FILE) {
+    candidates.push(path.resolve(process.env.DEV_WORKFLOW_STATE_FILE));
+  }
+
+  const seen = new Set();
+  let freshest = null;
+
+  for (const candidate of candidates) {
+    if (!candidate) {
+      continue;
+    }
+
+    const resolved = path.resolve(candidate);
+    if (seen.has(resolved)) {
+      continue;
+    }
+    seen.add(resolved);
+
+    const result = await readStateCandidate(resolved);
+    if (!result) {
+      continue;
+    }
+
+    if (!freshest || result.mtimeMs > freshest.mtimeMs) {
+      freshest = result;
+    }
+  }
+
+  return freshest;
+}
+
+async function main() {
+  const releaseTypeArg = (process.argv[2] || '').toLowerCase();
+  const forceFlag = process.argv.includes('--force') || process.env.DEV_WORKFLOW_FORCE_RELEASE === '1';
+  const additionalArgs = process.argv.slice(3).filter((arg) => arg !== '--force');
+
+  if (!RELEASE_TYPES.has(releaseTypeArg)) {
+    console.error(
+      '❌ Release guard: expected release type argument (major | minor | patch). Example: npm run release:patch'
+    );
+    process.exit(1);
+  }
+
+  const repoRoot = process.cwd();
+  const projectRootCandidate = process.env.INIT_CWD || repoRoot;
+  if (projectRootCandidate.includes(`node_modules${path.sep}`)) {
+    console.error('❌ Release guard: refusing to run inside node_modules.');
+    process.exit(1);
+  }
+  const projectRoot = projectRootCandidate;
+
+  const result = await loadWorkflowStateCandidates(projectRoot);
+  if (!result) {
+    console.error(
+      `❌ Release guard: cannot locate workflow state under ${path.join(projectRoot, '.state')}.
+Make sure the dev-workflow MCP is installed and you have started a task.`
+    );
+    process.exit(1);
+  }
+
+  const { state, path: statePath } = result;
+
+  const errors = [];
+  if (state.currentPhase !== 'release') {
+    errors.push(
+      "Current workflow phase isn't 'release'. Complete documentation, commit, and push steps first."
+    );
+  }
+  if (!state.readyCheckCompleted) {
+    errors.push("'check_ready_to_commit' has not been completed.");
+  }
+  if (!state.commitAndPushCompleted) {
+    errors.push("'commit_and_push' has not been completed.");
+  }
+  if (state.released) {
+    errors.push('A release has already been recorded for this task.');
+  }
+
+  if (errors.length && !forceFlag) {
+    console.error('❌ Release guard blocked the release:');
+    for (const message of errors) {
+      console.error(`  • ${message}`);
+    }
+    console.error("\nUse 'perform_release' through the dev-workflow MCP after completing the missing steps.");
+    console.error("Or use: npm run release:patch -- --force (to bypass guard)");
+    process.exit(1);
+  }
+
+  if (forceFlag && errors.length) {
+    console.warn('⚠️ --force flag detected. Bypassing workflow guard.');
+    console.warn('Errors that were skipped:');
+    for (const message of errors) {
+      console.warn(`  • ${message}`);
+    }
+  }
+
+  const commiterReleaseScript = path.join(repoRoot, 'scripts', 'release.cjs');
+
+  try {
+    await fs.access(commiterReleaseScript);
+  } catch {
+    console.error('❌ Release guard: @programinglive/commiter release script not found.');
+    console.error('   Ensure @programinglive/commiter is installed as a dev dependency.');
+    console.error('   Try: npm install --save-dev @programinglive/commiter');
+    process.exit(1);
+  }
+
+  const runRelease = async () => {
+    if (process.env.DEV_WORKFLOW_SKIP_RELEASE === '1') {
+      console.log('ℹ️ DEV_WORKFLOW_SKIP_RELEASE=1 detected. Skipping actual release command.');
+      return 0;
+    }
+
+    return await new Promise((resolve, reject) => {
+      const child = spawn(
+        process.execPath,
+        [commiterReleaseScript, releaseTypeArg, ...additionalArgs],
+        { stdio: 'inherit', cwd: repoRoot }
+      );
+
+      child.on('error', (error) => reject(error));
+      child.on('close', (code) => resolve(code));
+    });
+  };
+
+  try {
+    const exitCode = await runRelease();
+    if (exitCode !== 0) {
+      console.error(`❌ Release command exited with code ${exitCode}.`);
+      process.exit(exitCode);
+    }
+  } catch (error) {
+    console.error(`❌ Failed to run release command: ${error.message}`);
+    process.exit(1);
+  }
+
+  const recordedReleaseCommand = `node node_modules/@programinglive/commiter/scripts/release.js ${releaseTypeArg}`;
+
+  state.released = true;
+  state.releaseCommand = recordedReleaseCommand;
+  state.currentPhase = 'ready_to_complete';
+
+  try {
+    await fs.writeFile(statePath, JSON.stringify(state, null, 2));
+  } catch (error) {
+    console.error(`❌ Release guard: failed to update workflow state. ${error.message}`);
+    process.exit(1);
+  }
+
+  console.log('✅ Release recorded in workflow state. You can now run complete_task.');
+}
+
+main();