@programinglive/dev-workflow-mcp-server 1.2.7 → 1.3.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@programinglive/dev-workflow-mcp-server",
-  "version": "1.2.7",
+  "version": "1.3.1",
   "description": "MCP server to enforce development workflow discipline",
   "homepage": "https://devworkflow.programinglive.com/",
   "type": "module",
@@ -18,7 +18,7 @@
     "check": "node --check index.js",
     "prepare": "husky",
     "postinstall": "node install.js",
-    "release": "standard-version",
+    "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",
@@ -41,8 +41,6 @@
     "better-sqlite3": "^11.8.0",
     "esbuild": "^0.25.0",
     "get-port": "^7.1.0",
-    "husky": "^9.1.7",
-    "standard-version": "^9.5.0",
     "vite": "^7.1.12"
   },
   "publishConfig": {
@@ -51,8 +49,10 @@
   "devDependencies": {
     "@commitlint/cli": "^20.1.0",
     "@commitlint/config-conventional": "^20.0.0",
-    "@programinglive/commiter": "^1.1.0",
-    "marked": "^12.0.2"
+    "@programinglive/commiter": "^1.1.8",
+    "husky": "^9.1.7",
+    "marked": "^12.0.2",
+    "standard-version": "^9.5.0"
   },
   "standard-version": {
     "releaseCommitMessageFormat": "chore(release): {{currentTag}} 🚀",

package/release-wrapper.js

@@ -98,7 +98,7 @@ async function main() {
     }
   }
 
-  const commiterReleaseScript = path.join(repoRoot, 'node_modules', '@programinglive', 'commiter', 'scripts', 'release.js');
+  const commiterReleaseScript = path.join(repoRoot, 'scripts', 'release.cjs');
 
   try {
     await fs.access(commiterReleaseScript);

package/scripts/hello-world.js

@@ -0,0 +1 @@
+console.log('Hello from Antigravity!');

package/scripts/preload/fs-f-ok.cjs

@@ -0,0 +1,23 @@
+const fs = require('fs');
+const Module = require('module');
+
+const targetPath = require.resolve('standard-version/lib/lifecycles/changelog');
+
+if (!Module._commiterPatchedFsFok) {
+  const originalLoader = Module._extensions['.js'];
+
+  Module._extensions['.js'] = function patchedLoader(module, filename) {
+    if (filename === targetPath) {
+      let source = fs.readFileSync(filename, 'utf8');
+      if (source.includes('fs.F_OK')) {
+        source = source.replace(/fs\.F_OK/g, 'fs.constants.F_OK');
+      }
+      module._compile(source, filename);
+      return;
+    }
+
+    return originalLoader(module, filename);
+  };
+
+  Module._commiterPatchedFsFok = true;
+}

package/scripts/release.cjs

@@ -0,0 +1,252 @@
+#!/usr/bin/env node
+
+const { spawnSync } = require('child_process');
+const fs = require('fs');
+const path = require('path');
+
+const { updateReleaseNotes } = require('./update-release-notes.cjs');
+
+const fsFokPreloadPath = path.join(__dirname, 'preload', 'fs-f-ok.cjs');
+const FS_FOK_PRELOAD_FLAG = buildPreloadFlag(fsFokPreloadPath);
+
+require('./preload/fs-f-ok.cjs');
+
+const VALID_RELEASE_TYPES = new Set([
+  'major',
+  'minor',
+  'patch',
+  'premajor',
+  'preminor',
+  'prepatch',
+  'prerelease'
+]);
+
+const SEMVER_REGEX = /^\d+\.\d+\.\d+(?:-[0-9A-Za-z.-]+)?$/;
+
+const LOCKFILE_PREFERENCES = [
+  { manager: 'pnpm', file: 'pnpm-lock.yaml' },
+  { manager: 'yarn', file: 'yarn.lock' },
+  { manager: 'bun', file: 'bun.lockb' },
+  { manager: 'npm', file: 'package-lock.json' }
+];
+
+function getCliArguments(argv = process.argv) {
+  const rawArgs = argv.slice(2);
+  if (rawArgs.length === 0) {
+    return { releaseType: undefined, extraArgs: [] };
+  }
+
+  if (rawArgs[0].startsWith('-')) {
+    return { releaseType: undefined, extraArgs: rawArgs };
+  }
+
+  const [firstArg, ...rest] = rawArgs;
+  return { releaseType: firstArg, extraArgs: rest };
+}
+
+function getNpmRunArgument(env = process.env) {
+  try {
+    const npmArgs = JSON.parse(env.npm_config_argv || '{}');
+    const original = npmArgs.original || [];
+    const releaseIndex = original.lastIndexOf('release');
+    if (releaseIndex !== -1) {
+      return original[releaseIndex + 1];
+    }
+  } catch (error) {
+    // Ignore JSON parsing issues and fall back to defaults
+  }
+  return undefined;
+}
+
+function buildStandardVersionArgs({ releaseType, extraArgs }) {
+  const args = [];
+  if (releaseType) {
+    const normalized = releaseType.trim();
+    const isValid = VALID_RELEASE_TYPES.has(normalized) || SEMVER_REGEX.test(normalized);
+    if (!isValid) {
+      const allowed = Array.from(VALID_RELEASE_TYPES).join(', ');
+      throw new Error(`Unknown release type "${normalized}". Use one of ${allowed} or a valid semver version.`);
+    }
+    args.push('--release-as', normalized);
+  }
+
+  if (Array.isArray(extraArgs) && extraArgs.length > 0) {
+    args.push(...extraArgs);
+  }
+
+  return args;
+}
+
+function loadPackageJson(cwd = process.cwd()) {
+  try {
+    const packageJsonPath = path.join(cwd, 'package.json');
+    const contents = fs.readFileSync(packageJsonPath, 'utf8');
+    return JSON.parse(contents);
+  } catch (error) {
+    return null;
+  }
+}
+
+function detectPackageManager({ env = process.env, cwd = process.cwd() } = {}) {
+  const userAgent = env.npm_config_user_agent || '';
+  if (userAgent.startsWith('pnpm/')) return 'pnpm';
+  if (userAgent.startsWith('yarn/')) return 'yarn';
+  if (userAgent.startsWith('bun/')) return 'bun';
+
+  for (const { manager, file } of LOCKFILE_PREFERENCES) {
+    if (fs.existsSync(path.join(cwd, file))) {
+      return manager;
+    }
+  }
+
+  return 'npm';
+}
+
+function buildTestCommand(packageManager) {
+  switch (packageManager) {
+    case 'pnpm':
+      return { command: 'pnpm', args: ['test'] };
+    case 'yarn':
+      return { command: 'yarn', args: ['test'] };
+    case 'bun':
+      return { command: 'bun', args: ['test'] };
+    default:
+      return { command: 'npm', args: ['test'] };
+  }
+}
+
+function runProjectTests({ spawn = spawnSync, env = process.env, cwd = process.cwd() } = {}) {
+  const packageJson = loadPackageJson(cwd);
+  const scripts = packageJson && packageJson.scripts ? packageJson.scripts : {};
+
+  if (!scripts.test) {
+    console.log('ℹ️  Skipping tests: no "test" script detected in package.json');
+    return { status: 0 };
+  }
+
+  const packageManager = detectPackageManager({ env, cwd });
+  const { command, args } = buildTestCommand(packageManager);
+
+  console.log(`🧪 Running tests with ${packageManager} ${args.join(' ')}`.trim());
+  return spawn(command, args, {
+    stdio: 'inherit',
+    env
+  });
+}
+
+function runRelease({
+  argv = process.argv,
+  env = process.env,
+  spawn = spawnSync,
+  cwd = process.cwd(),
+  dependencies = {}
+} = {}) {
+  const { releaseType: cliReleaseType, extraArgs } = getCliArguments(argv);
+  const inferredReleaseType = cliReleaseType || getNpmRunArgument(env);
+  const standardVersionArgs = buildStandardVersionArgs({
+    releaseType: inferredReleaseType,
+    extraArgs
+  });
+
+  const checkWorkingTreeClean = dependencies.isWorkingTreeClean || (() => isWorkingTreeClean({ cwd }));
+  const workingTreeClean = checkWorkingTreeClean();
+  if (!workingTreeClean) {
+    throw new Error('Working tree has uncommitted changes. Commit or stash before running release.');
+  }
+
+  const testResult = runProjectTests({ spawn, env });
+  if (testResult && typeof testResult.status === 'number' && testResult.status !== 0) {
+    return testResult;
+  }
+
+  const standardVersionBin = require.resolve('standard-version/bin/cli.js');
+  const childEnv = appendPreloadToNodeOptions(env, FS_FOK_PRELOAD_FLAG);
+  const releaseResult = spawn(process.execPath, [standardVersionBin, ...standardVersionArgs], {
+    stdio: 'inherit',
+    env: childEnv,
+    cwd
+  });
+
+  if (releaseResult && typeof releaseResult.status === 'number' && releaseResult.status !== 0) {
+    return releaseResult;
+  }
+
+  const releaseNotesPath = dependencies.releaseNotesPath || path.join('docs', 'release-notes', 'RELEASE_NOTES.md');
+  const updateNotes = dependencies.updateReleaseNotes || ((options = {}) => updateReleaseNotes({ rootDir: cwd, ...options }));
+
+  let notesUpdated = false;
+  try {
+    notesUpdated = updateNotes();
+  } catch (error) {
+    console.warn(`⚠️  Skipping release notes update: ${error.message}`);
+  }
+
+  if (notesUpdated) {
+    const gitAddResult = spawnSync('git', ['add', releaseNotesPath], { stdio: 'inherit', cwd });
+    if (!gitAddResult || typeof gitAddResult.status !== 'number' || gitAddResult.status !== 0) {
+      console.warn('⚠️  Release notes updated but failed to stage. Please add manually: git add ' + releaseNotesPath);
+    }
+  }
+
+  return releaseResult;
+}
+
+if (require.main === module) {
+  try {
+    const result = runRelease();
+    if (result.status !== 0) {
+      process.exit(result.status ?? 1);
+    }
+  } catch (error) {
+    console.error(`❌ ${error.message}`);
+    process.exit(1);
+  }
+}
+
+module.exports = {
+  VALID_RELEASE_TYPES,
+  SEMVER_REGEX,
+  getCliArguments,
+  getNpmRunArgument,
+  buildStandardVersionArgs,
+  loadPackageJson,
+  detectPackageManager,
+  runProjectTests,
+  runRelease,
+  isWorkingTreeClean
+};
+
+function buildPreloadFlag(filePath) {
+  const resolved = path.resolve(filePath);
+  const escaped = resolved.includes(' ')
+    ? `"${resolved.replace(/"/g, '\"')}"`
+    : resolved;
+  return `--require ${escaped}`;
+}
+
+function appendPreloadToNodeOptions(env, preloadFlag) {
+  const nextEnv = { ...env };
+  const existing = typeof nextEnv.NODE_OPTIONS === 'string' ? nextEnv.NODE_OPTIONS.trim() : '';
+  nextEnv.NODE_OPTIONS = existing ? `${existing} ${preloadFlag}` : preloadFlag;
+  return nextEnv;
+}
+
+function isWorkingTreeClean({ cwd = process.cwd() } = {}) {
+  const result = spawnSync('git', ['status', '--porcelain'], {
+    cwd,
+    encoding: 'utf8'
+  });
+
+  if (result.error) {
+    throw result.error;
+  }
+
+  if (typeof result.status === 'number' && result.status !== 0) {
+    const stderr = typeof result.stderr === 'string' ? result.stderr.trim() : '';
+    const message = stderr || 'Failed to determine working tree status.';
+    throw new Error(message);
+  }
+
+  const output = typeof result.stdout === 'string' ? result.stdout : '';
+  return output.trim().length === 0;
+}

package/scripts/update-release-notes.cjs

@@ -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/tests/handlers.test.js

@@ -323,7 +323,7 @@ test("run_full_workflow uses provided branch for release push when commit is cle
     const tagPushCommand = commands.find((command) => command.startsWith("git push --follow-tags"));
     assert.ok(tagPushCommand, "release should push tags");
     assert.ok(
-      /origin\s+'?master'?/.test(tagPushCommand),
+      /origin\s+['"]?master['"]?/.test(tagPushCommand),
       `release push should target the provided branch (${branchName})`
     );
 
@@ -837,7 +837,7 @@ test("perform_release runs release command before pushing with tags", async () =
     assert.equal(workflowState.state.currentPhase, "ready_to_complete");
     assert.ok(
       response.content[0].text.includes("No workflow history yet") ||
-        response.content[0].text.includes("Workflow History"),
+      response.content[0].text.includes("Workflow History"),
       "response should describe workflow history or absence thereof"
     );
   });

package/tests/hello-world.test.js

@@ -0,0 +1,13 @@
+import { test } from 'node:test';
+import assert from 'node:assert';
+import { execSync } from 'node:child_process';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+
+test('hello world script runs successfully', (t) => {
+    const scriptPath = path.join(__dirname, '../scripts/hello-world.js');
+    const output = execSync(`node ${scriptPath}`).toString();
+    assert.match(output, /Hello from Antigravity!/);
+});

package/utils.js

@@ -3,8 +3,9 @@ const isWindows = process.platform === "win32";
 export function shellEscape(value) {
   const stringValue = String(value);
   if (isWindows) {
-    // Use PowerShell-friendly single quotes, escape embedded single quotes by doubling
-    return `'${stringValue.replace(/'/g, "''")}'`;
+    // Windows cmd.exe requires double quotes for arguments with spaces.
+    // Escape existing double quotes with backslash.
+    return `"${stringValue.replace(/"/g, '\\"')}"`;
   }
 
   return `'${stringValue.replace(/'/g, "'\\''")}'`;