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

package/scripts/update-release-notes.js

@@ -0,0 +1,182 @@
+#!/usr/bin/env node
+
+const fs = require('fs');
+const path = require('path');
+
+function updateReleaseNotes({
+  rootDir = process.cwd(),
+  releaseNotesRelativePath = path.join('docs', 'release-notes', 'RELEASE_NOTES.md'),
+  changelogRelativePath = 'CHANGELOG.md',
+  packageJsonRelativePath = 'package.json',
+  now = () => new Date()
+} = {}) {
+  const releaseNotesPath = path.join(rootDir, releaseNotesRelativePath);
+  if (!fs.existsSync(releaseNotesPath)) {
+    console.log(`ℹ️  Release notes file not found at ${releaseNotesPath}; skipping update.`);
+    return false;
+  }
+
+  const packageJsonPath = path.join(rootDir, packageJsonRelativePath);
+  if (!fs.existsSync(packageJsonPath)) {
+    console.log(`ℹ️  package.json not found at ${packageJsonPath}; skipping release notes update.`);
+    return false;
+  }
+
+  const changelogPath = path.join(rootDir, changelogRelativePath);
+  if (!fs.existsSync(changelogPath)) {
+    console.log(`ℹ️  CHANGELOG not found at ${changelogPath}; skipping release notes update.`);
+    return false;
+  }
+
+  const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
+  const version = packageJson.version;
+  if (!version) {
+    console.log('ℹ️  package.json version is missing; skipping release notes update.');
+    return false;
+  }
+
+  const releaseNotesContent = fs.readFileSync(releaseNotesPath, 'utf8');
+  if (releaseNotesContent.includes(`| ${version} `) || releaseNotesContent.includes(`## ${version}`)) {
+    console.log(`ℹ️  Release notes already contain version ${version}; skipping.`);
+    return false;
+  }
+
+  const changelogContent = fs.readFileSync(changelogPath, 'utf8');
+  const changelogInfo = extractChangelogInfo({ changelogContent, version });
+
+  const releaseDate = changelogInfo.date || formatDate(now());
+  const highlight = changelogInfo.highlight || changelogInfo.heading || 'See CHANGELOG for details.';
+  const sectionHeading = changelogInfo.heading || '';
+  const detailBullets = Array.isArray(changelogInfo.bullets) && changelogInfo.bullets.length > 0
+    ? changelogInfo.bullets
+    : [highlight];
+
+  const updatedContent = insertReleaseNotesEntry({
+    content: releaseNotesContent,
+    version,
+    releaseDate,
+    highlight,
+    sectionHeading,
+    detailBullets
+  });
+
+  fs.writeFileSync(releaseNotesPath, updatedContent);
+  console.log(`✅ Added release notes entry for version ${version}.`);
+  return true;
+}
+
+function insertReleaseNotesEntry({ content, version, releaseDate, highlight, sectionHeading, detailBullets }) {
+  const lines = content.split('\n');
+  const headerSeparatorIndex = lines.findIndex((line) => line.trim().startsWith('|---------'));
+  if (headerSeparatorIndex === -1) {
+    throw new Error('Unable to locate release notes table header.');
+  }
+
+  const newRow = `| ${version} | ${releaseDate} | ${escapePipes(highlight)} |`;
+  lines.splice(headerSeparatorIndex + 1, 0, newRow);
+
+  const sectionLines = [];
+  sectionLines.push('');
+  const trimmedHeading = typeof sectionHeading === 'string' ? sectionHeading.trim() : '';
+  const headingSuffix = trimmedHeading ? ` – ${trimmedHeading}` : '';
+  sectionLines.push(`## ${version}${headingSuffix}`);
+  sectionLines.push('');
+  sectionLines.push(`Released on **${releaseDate}**.`);
+  sectionLines.push('');
+  for (const bullet of detailBullets) {
+    sectionLines.push(`- ${bullet}`);
+  }
+  sectionLines.push('');
+
+  const firstSectionIndex = lines.findIndex((line, idx) => idx > headerSeparatorIndex && line.startsWith('## '));
+  if (firstSectionIndex === -1) {
+    lines.push(...sectionLines);
+  } else {
+    lines.splice(firstSectionIndex, 0, ...sectionLines);
+  }
+
+  return lines.join('\n');
+}
+
+function extractChangelogInfo({ changelogContent, version }) {
+  const versionHeadingRegex = new RegExp(`^### \\[${escapeRegExp(version)}\\][^\n]*`, 'm');
+  const match = changelogContent.match(versionHeadingRegex);
+  if (!match) {
+    return {};
+  }
+
+  const headingIndex = match.index;
+  const rest = changelogContent.slice(headingIndex + match[0].length);
+  const nextVersionRegex = /^### \[[^\n]*\]/m;
+  const nextVersionMatch = nextVersionRegex.exec(rest);
+  const section = nextVersionMatch ? rest.slice(0, nextVersionMatch.index) : rest;
+
+  const dateMatch = match[0].match(/\((\d{4}-\d{2}-\d{2})\)/);
+  const date = dateMatch ? dateMatch[1] : undefined;
+
+  const categoryRegex = /^###\s+([^\n]+)$/m;
+  const categoryMatch = section.match(categoryRegex);
+  let heading;
+  if (categoryMatch) {
+    heading = sanitizeText(categoryMatch[1]);
+  }
+
+  const bullets = [];
+  const bulletRegex = /^\*\s+([^\n]+)$/gm;
+  let bulletMatch;
+  while ((bulletMatch = bulletRegex.exec(section)) !== null) {
+    const sanitized = sanitizeText(bulletMatch[1]);
+    if (!bullets.includes(sanitized)) {
+      bullets.push(sanitized);
+    }
+  }
+
+  const highlight = bullets.length > 0 ? bullets[0] : undefined;
+
+  return {
+    date,
+    highlight,
+    heading,
+    bullets
+  };
+}
+
+function sanitizeText(text) {
+  return text
+    .replace(/\[([^\]]+)\]\([^\)]+\)/g, '$1')
+    .replace(/\s+/g, ' ')
+    .trim();
+}
+
+function escapeRegExp(string) {
+  return string.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+}
+
+function escapePipes(text) {
+  return text.replace(/\|/g, '\\|');
+}
+
+function formatDate(date) {
+  const year = date.getFullYear();
+  const month = String(date.getMonth() + 1).padStart(2, '0');
+  const day = String(date.getDate()).padStart(2, '0');
+  return `${year}-${month}-${day}`;
+}
+
+module.exports = {
+  updateReleaseNotes,
+  insertReleaseNotesEntry,
+  extractChangelogInfo,
+  sanitizeText,
+  escapePipes,
+  formatDate
+};
+
+if (require.main === module) {
+  try {
+    updateReleaseNotes();
+  } catch (error) {
+    console.error(`❌ Failed to update release notes: ${error.message}`);
+    process.exit(1);
+  }
+}

package/server.json

@@ -1,23 +1,23 @@
-{
-  "$schema": "https://static.modelcontextprotocol.io/schemas/2025-10-17/server.schema.json",
-  "name": "io.github.programinglive/dev-workflow-mcp-server",
-  "description": "MCP server to enforce development workflow discipline",
-  "title": "Dev Workflow MCP Server",
-  "websiteUrl": "https://devworkflow.programinglive.com/",
-  "repository": {
-    "url": "https://github.com/programinglive/dev-workflow-mcp-server",
-    "source": "github"
-  },
-  "version": "1.4.6",
-  "packages": [
-    {
-      "registryType": "npm",
-      "registryBaseUrl": "https://registry.npmjs.org",
-      "identifier": "@programinglive/dev-workflow-mcp-server",
-      "version": "1.4.6",
-      "transport": {
-        "type": "stdio"
-      }
-    }
-  ]
-}
+{
+  "$schema": "https://static.modelcontextprotocol.io/schemas/2025-10-17/server.schema.json",
+  "name": "io.github.programinglive/dev-workflow-mcp-server",
+  "description": "MCP server to enforce development workflow discipline",
+  "title": "Dev Workflow MCP Server",
+  "websiteUrl": "https://devworkflow.programinglive.com/",
+  "repository": {
+    "url": "https://github.com/programinglive/dev-workflow-mcp-server",
+    "source": "github"
+  },
+  "version": "1.4.6",
+  "packages": [
+    {
+      "registryType": "npm",
+      "registryBaseUrl": "https://registry.npmjs.org",
+      "identifier": "@programinglive/dev-workflow-mcp-server",
+      "version": "1.4.6",
+      "transport": {
+        "type": "stdio"
+      }
+    }
+  ]
+}

package/tailwind.config.js

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

package/tests/build.test.js

@@ -1,83 +1,83 @@
-import { test } from 'node:test';
-import assert from 'node:assert';
-import { readFile, access } from 'fs/promises';
-import { exec as execCallback } from 'child_process';
-import { promisify } from 'util';
-import path from 'path';
-
-const exec = promisify(execCallback);
-
-test('build generates dist/index.mjs file', async () => {
-  const { stderr } = await exec('npm run build:server && npm run postbuild');
-  // Allow the known vite warning about node:process
-  const knownWarning = '[plugin vite:resolve] Module "node:process" has been externalized for browser compatibility';
-  const tailwindOutput = '≈ tailwindcss';
-  const browserslistWarning = 'Browserslist: caniuse-lite is outdated';
-  const actualStderr = stderr.replace(/\x1B\[[0-9;]*m/g, ''); // Remove ANSI codes
-  assert(
-    actualStderr === '' || actualStderr.includes(knownWarning) || actualStderr.includes(tailwindOutput) || actualStderr.includes(browserslistWarning),
-    `Build should not have unexpected errors: ${actualStderr}`
-  );
-
-  // Verify the file exists
-  const distPath = path.join(process.cwd(), 'dist', 'index.mjs');
-  await access(distPath);
-
-  // Verify it contains expected content
-  const content = await readFile(distPath, 'utf-8');
-  assert(content.includes('main'), 'Bundle should contain main function');
-  assert(content.length > 1000, 'Bundle should be substantial');
-
-  const docDir = path.join(process.cwd(), 'dist', 'docs');
-  const docFiles = ['README.html', 'web-dashboard.html'];
-  for (const file of docFiles) {
-    const fullPath = path.join(docDir, file);
-    await access(fullPath);
-    const docContent = await readFile(fullPath, 'utf-8');
-    assert(docContent.includes('<h1'), `${file} should contain HTML output`);
-  }
-});
-
-test('build script exists in package.json', async () => {
-  const packagePath = path.join(process.cwd(), 'package.json');
-  const content = await readFile(packagePath, 'utf-8');
-  const pkg = JSON.parse(content);
-
-  assert.strictEqual(typeof pkg.scripts['build:server'], 'string');
-  assert(pkg.scripts['build:server'].includes('vite build'));
-});
-
-test('Hero component fetches npm downloads directly', async () => {
-  const heroPath = path.join(process.cwd(), 'web', 'components', 'Hero.tsx');
-  const heroContent = await readFile(heroPath, 'utf-8');
-
-  // Verify it fetches npm downloads directly from npm API
-  assert(heroContent.includes('api.npmjs.org'), 'Hero should fetch downloads from npm API');
-  assert(heroContent.includes('useState<string | null>(null)'), 'Hero should have version state');
-});
-
-test('Netlify deployment configuration exists for static export', async () => {
-  const netlifyTomlPath = path.join(process.cwd(), 'web', 'netlify.toml');
-  await access(netlifyTomlPath);
-  
-  const content = await readFile(netlifyTomlPath, 'utf-8');
-  assert(content.includes('[build]'), 'netlify.toml should have build section');
-  assert(content.includes('npm run build'), 'netlify.toml should have build command');
-  assert(content.includes('publish = "out"'), 'netlify.toml should publish out directory for static export');
-});
-
-test('Web package.json does not include better-sqlite3', async () => {
-  const webPackagePath = path.join(process.cwd(), 'web', 'package.json');
-  const content = await readFile(webPackagePath, 'utf-8');
-  const pkg = JSON.parse(content);
-  
-  assert(!pkg.dependencies['better-sqlite3'], 'Web should not depend on better-sqlite3 for Netlify compatibility');
-});
-
-test('Next.js config uses static export for Netlify', async () => {
-  const nextConfigPath = path.join(process.cwd(), 'web', 'next.config.ts');
-  const content = await readFile(nextConfigPath, 'utf-8');
-  
-  assert(content.includes("output: 'export'"), 'next.config.ts should use static export');
-  assert(content.includes('unoptimized: true'), 'next.config.ts should disable image optimization for static export');
-});
+import { test } from 'node:test';
+import assert from 'node:assert';
+import { readFile, access } from 'fs/promises';
+import { exec as execCallback } from 'child_process';
+import { promisify } from 'util';
+import path from 'path';
+
+const exec = promisify(execCallback);
+
+test('build generates dist/index.mjs file', async () => {
+  const { stderr } = await exec('npm run build:server && npm run postbuild');
+  // Allow the known vite warning about node:process
+  const knownWarning = '[plugin vite:resolve] Module "node:process" has been externalized for browser compatibility';
+  const tailwindOutput = '≈ tailwindcss';
+  const browserslistWarning = 'Browserslist: caniuse-lite is outdated';
+  const actualStderr = stderr.replace(/\x1B\[[0-9;]*m/g, ''); // Remove ANSI codes
+  assert(
+    actualStderr === '' || actualStderr.includes(knownWarning) || actualStderr.includes(tailwindOutput) || actualStderr.includes(browserslistWarning),
+    `Build should not have unexpected errors: ${actualStderr}`
+  );
+
+  // Verify the file exists
+  const distPath = path.join(process.cwd(), 'dist', 'index.mjs');
+  await access(distPath);
+
+  // Verify it contains expected content
+  const content = await readFile(distPath, 'utf-8');
+  assert(content.includes('main'), 'Bundle should contain main function');
+  assert(content.length > 1000, 'Bundle should be substantial');
+
+  const docDir = path.join(process.cwd(), 'dist', 'docs');
+  const docFiles = ['README.html', 'web-dashboard.html'];
+  for (const file of docFiles) {
+    const fullPath = path.join(docDir, file);
+    await access(fullPath);
+    const docContent = await readFile(fullPath, 'utf-8');
+    assert(docContent.includes('<h1'), `${file} should contain HTML output`);
+  }
+});
+
+test('build script exists in package.json', async () => {
+  const packagePath = path.join(process.cwd(), 'package.json');
+  const content = await readFile(packagePath, 'utf-8');
+  const pkg = JSON.parse(content);
+
+  assert.strictEqual(typeof pkg.scripts['build:server'], 'string');
+  assert(pkg.scripts['build:server'].includes('vite build'));
+});
+
+test('Hero component fetches npm downloads directly', async () => {
+  const heroPath = path.join(process.cwd(), 'web', 'components', 'Hero.tsx');
+  const heroContent = await readFile(heroPath, 'utf-8');
+
+  // Verify it fetches npm downloads directly from npm API
+  assert(heroContent.includes('api.npmjs.org'), 'Hero should fetch downloads from npm API');
+  assert(heroContent.includes('useState<string | null>(null)'), 'Hero should have version state');
+});
+
+test('Netlify deployment configuration exists for static export', async () => {
+  const netlifyTomlPath = path.join(process.cwd(), 'web', 'netlify.toml');
+  await access(netlifyTomlPath);
+  
+  const content = await readFile(netlifyTomlPath, 'utf-8');
+  assert(content.includes('[build]'), 'netlify.toml should have build section');
+  assert(content.includes('npm run build'), 'netlify.toml should have build command');
+  assert(content.includes('publish = "out"'), 'netlify.toml should publish out directory for static export');
+});
+
+test('Web package.json does not include better-sqlite3', async () => {
+  const webPackagePath = path.join(process.cwd(), 'web', 'package.json');
+  const content = await readFile(webPackagePath, 'utf-8');
+  const pkg = JSON.parse(content);
+  
+  assert(!pkg.dependencies['better-sqlite3'], 'Web should not depend on better-sqlite3 for Netlify compatibility');
+});
+
+test('Next.js config uses static export for Netlify', async () => {
+  const nextConfigPath = path.join(process.cwd(), 'web', 'next.config.ts');
+  const content = await readFile(nextConfigPath, 'utf-8');
+  
+  assert(content.includes("output: 'export'"), 'next.config.ts should use static export');
+  assert(content.includes('unoptimized: true'), 'next.config.ts should disable image optimization for static export');
+});

package/tests/exec.test.js

@@ -1,35 +1,35 @@
-import { test } from 'node:test';
-import assert from 'node:assert';
-import { exec } from '../exec.js';
-import { tmpdir } from 'os';
-import { mkdtemp, rm } from 'fs/promises';
-import path from 'path';
-
-test('exec runs commands successfully', async () => {
-  const echoCommand = process.platform === 'win32' ? 'echo hello world' : 'echo "hello world"';
-  const { stdout } = await exec(echoCommand);
-  const normalized = stdout.trim().replace(/^"(.*)"$/, '$1');
-  assert.strictEqual(normalized, 'hello world');
-});
-
-test('exec respects explicit cwd option', async () => {
-  const tempDir = await mkdtemp(path.join(tmpdir(), 'mcp-test-'));
-  
-  try {
-    const dirCommand = process.platform === 'win32' ? 'cd' : 'pwd';
-    const { stdout } = await exec(dirCommand, { cwd: tempDir });
-    // Just verify it runs in the specified directory (path may vary by platform)
-    assert(stdout.trim().toLowerCase().includes('mcp-test-'));
-  } finally {
-    await rm(tempDir, { recursive: true, force: true });
-  }
-});
-
-test('exec handles command errors', async () => {
-  try {
-    await exec('command-that-does-not-exist');
-    assert.fail('Should have thrown an error');
-  } catch (error) {
-    assert(error.code !== undefined || error.message.includes('command-not-found'));
-  }
-});
+import { test } from 'node:test';
+import assert from 'node:assert';
+import { exec } from '../exec.js';
+import { tmpdir } from 'os';
+import { mkdtemp, rm } from 'fs/promises';
+import path from 'path';
+
+test('exec runs commands successfully', async () => {
+  const echoCommand = process.platform === 'win32' ? 'echo hello world' : 'echo "hello world"';
+  const { stdout } = await exec(echoCommand);
+  const normalized = stdout.trim().replace(/^"(.*)"$/, '$1');
+  assert.strictEqual(normalized, 'hello world');
+});
+
+test('exec respects explicit cwd option', async () => {
+  const tempDir = await mkdtemp(path.join(tmpdir(), 'mcp-test-'));
+  
+  try {
+    const dirCommand = process.platform === 'win32' ? 'cd' : 'pwd';
+    const { stdout } = await exec(dirCommand, { cwd: tempDir });
+    // Just verify it runs in the specified directory (path may vary by platform)
+    assert(stdout.trim().toLowerCase().includes('mcp-test-'));
+  } finally {
+    await rm(tempDir, { recursive: true, force: true });
+  }
+});
+
+test('exec handles command errors', async () => {
+  try {
+    await exec('command-that-does-not-exist');
+    assert.fail('Should have thrown an error');
+  } catch (error) {
+    assert(error.code !== undefined || error.message.includes('command-not-found'));
+  }
+});