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

package/scripts/update-release-notes.cjs

@@ -1,178 +1,178 @@
-#!/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');
-
-  // Find the "## Unreleased" section
-  const unreleasedIndex = lines.findIndex((line) => line.trim().startsWith('## Unreleased'));
-  if (unreleasedIndex === -1) {
-    throw new Error('Unable to locate "## Unreleased" section in release notes.');
-  }
-
-  const sectionLines = [];
-  sectionLines.push('');
-  const trimmedHeading = typeof sectionHeading === 'string' ? sectionHeading.trim() : '';
-  const headingSuffix = trimmedHeading ? ` — ${trimmedHeading}` : '';
-
-  // Format: ## [Version] — [Date]
-  sectionLines.push(`## ${version} — ${releaseDate}${headingSuffix}`);
-
-  for (const bullet of detailBullets) {
-    sectionLines.push(`- ${bullet}`);
-  }
-
-  // Insert after the Unreleased header (and potentially some blank lines)
-  // We want to keep "## Unreleased" empty and add the new version below it.
-  // So we insert after the Unreleased line.
-  lines.splice(unreleasedIndex + 1, 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);
-  }
-}
+#!/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');
+
+  // Find the "## Unreleased" section
+  const unreleasedIndex = lines.findIndex((line) => line.trim().startsWith('## Unreleased'));
+  if (unreleasedIndex === -1) {
+    throw new Error('Unable to locate "## Unreleased" section in release notes.');
+  }
+
+  const sectionLines = [];
+  sectionLines.push('');
+  const trimmedHeading = typeof sectionHeading === 'string' ? sectionHeading.trim() : '';
+  const headingSuffix = trimmedHeading ? ` — ${trimmedHeading}` : '';
+
+  // Format: ## [Version] — [Date]
+  sectionLines.push(`## ${version} — ${releaseDate}${headingSuffix}`);
+
+  for (const bullet of detailBullets) {
+    sectionLines.push(`- ${bullet}`);
+  }
+
+  // Insert after the Unreleased header (and potentially some blank lines)
+  // We want to keep "## Unreleased" empty and add the new version below it.
+  // So we insert after the Unreleased line.
+  lines.splice(unreleasedIndex + 1, 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,91 +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 version dynamically', async () => {
-  const heroPath = path.join(process.cwd(), 'web', 'components', 'Hero.tsx');
-  const heroContent = await readFile(heroPath, 'utf-8');
-
-  // Verify no hardcoded version strings
-  assert(!heroContent.includes('v1.3.12'), 'Hero should not have hardcoded v1.3.12');
-  assert(!heroContent.includes('"1.3.12"'), 'Hero should not have hardcoded "1.3.12"');
-
-  // Verify it fetches from /api/version
-  assert(heroContent.includes('fetch("/api/version")'), 'Hero should fetch version from API');
-  assert(heroContent.includes('useState<string | null>(null)'), 'Hero should have version state');
-});
-
-test('Netlify deployment configuration exists', 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('@netlify/plugin-nextjs'), 'netlify.toml should use Next.js plugin');
-});
-
-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('API routes return static data for Netlify deployment', async () => {
-  const summaryRoutePath = path.join(process.cwd(), 'web', 'app', 'api', 'summary', 'route.ts');
-  const historyRoutePath = path.join(process.cwd(), 'web', 'app', 'api', 'history', 'route.ts');
-  
-  const summaryContent = await readFile(summaryRoutePath, 'utf-8');
-  const historyContent = await readFile(historyRoutePath, 'utf-8');
-  
-  // Verify no database imports
-  assert(!summaryContent.includes('from "@/lib/db"'), 'Summary route should not import from db');
-  assert(!historyContent.includes('from "@/lib/db"'), 'History route should not import from db');
-});
+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');
+});