forgedev 1.2.0 → 1.4.0

package/src/claude-configurator.js

@@ -1,6 +1,6 @@
 import fs from 'node:fs';
 import path from 'node:path';
-import { ROOT_DIR, ensureDir, writeFile, readTemplate, replaceVars, log } from './utils.js';
+import { ROOT_DIR, ensureDir, writeFile, readTemplate, replaceVars, getStackCommands, toPascalCase } from './utils.js';
 
 const CLAUDE_TEMPLATES_DIR = path.join(ROOT_DIR, 'templates', 'claude-code');
 const DOCS_DIR = path.join(ROOT_DIR, 'docs');
@@ -13,79 +13,108 @@ export async function generateClaudeConfig(outputDir, stackConfig, options = {})
   }
   generateHooks(outputDir, stackConfig, { merge: options.mergeSettings });
   generateSkills(outputDir, stackConfig);
-  generateAgents(outputDir, stackConfig, vars);
-  generateCommands(outputDir, stackConfig, vars);
+  const agents = generateAgents(outputDir, stackConfig, vars);
+  const commands = generateCommands(outputDir, stackConfig, vars);
   copyPromptLibrary(outputDir);
+  stampVersion(outputDir);
+
+  return { agents, commands };
+}
+
+function stampVersion(outputDir) {
+  const pkg = JSON.parse(fs.readFileSync(path.join(ROOT_DIR, 'package.json'), 'utf-8'));
+  writeFile(path.join(outputDir, '.claude', '.devforge-version'), JSON.stringify({
+    version: pkg.version,
+    generatedAt: new Date().toISOString(),
+  }, null, 2) + '\n');
 }
 
 function buildClaudeVars(config) {
   const vars = {
     PROJECT_NAME: config.projectName,
-    PROJECT_NAME_PASCAL: config.projectName.split('-').map(w => w.charAt(0).toUpperCase() + w.slice(1)).join(''),
+    PROJECT_NAME_PASCAL: toPascalCase(config.projectName),
   };
 
-  if (config.stackId === 'nextjs-fullstack') {
-    vars.STACK_SUMMARY = 'Next.js 15 (App Router) + TypeScript + Tailwind CSS + Prisma + PostgreSQL';
-    vars.LINT_COMMAND = 'npx eslint .';
-    vars.TYPE_CHECK_COMMAND = 'npx tsc --noEmit';
-    vars.TEST_COMMAND = 'npx vitest run';
-    vars.BUILD_COMMAND = 'npm run build';
-    vars.DEV_COMMAND = 'npm run dev';
-    vars.DIR_MAP = `- src/app/ — Next.js App Router pages and API routes
-- src/lib/ — Shared utilities, database client, error handling
-- src/components/ — React components
-- prisma/ — Database schema and migrations
-- e2e/ — Playwright E2E tests`;
-  } else if (config.stackId === 'fastapi-backend') {
-    vars.STACK_SUMMARY = 'FastAPI + Python + SQLAlchemy 2.0 + PostgreSQL + Alembic';
-    vars.LINT_COMMAND = 'ruff check .';
-    vars.TYPE_CHECK_COMMAND = 'pyright';
-    vars.TEST_COMMAND = 'pytest';
-    vars.BUILD_COMMAND = 'docker build -t app .';
-    vars.DEV_COMMAND = 'uvicorn app.main:app --reload';
-    vars.DIR_MAP = `- backend/app/ — FastAPI application
-- backend/app/api/ — API route handlers
-- backend/app/core/ — Config, security, error handling
-- backend/app/db/ — Database session and models
-- backend/app/models/ — SQLAlchemy models
-- backend/app/schemas/ — Pydantic schemas
-- backend/tests/ — Pytest tests
-- backend/alembic/ — Database migrations`;
-  } else if (config.stackId === 'polyglot-fullstack') {
-    vars.STACK_SUMMARY = 'Next.js 15 (frontend) + FastAPI (backend) + PostgreSQL';
-    vars.LINT_COMMAND = 'cd frontend && npx eslint . && cd ../backend && ruff check .';
-    vars.TYPE_CHECK_COMMAND = 'cd frontend && npx tsc --noEmit';
-    vars.TEST_COMMAND = 'cd frontend && npx vitest run && cd ../backend && pytest';
-    vars.BUILD_COMMAND = 'docker compose build';
-    vars.DEV_COMMAND = 'docker compose up';
-    vars.DIR_MAP = `- frontend/ — Next.js 15 App Router application
-- frontend/src/app/ — Pages and API routes
-- frontend/src/lib/ — Shared utilities
-- backend/ — FastAPI application
-- backend/app/api/ — API route handlers
-- backend/app/core/ — Config, security, error handling
-- backend/app/db/ — Database session and models
-- e2e/ — Playwright E2E tests`;
+  // Commands vary by stack (shared with composer)
+  Object.assign(vars, getStackCommands(config.stackId));
+
+  const claudeVars = {
+    'nextjs-fullstack': {
+      STACK_SUMMARY: 'Next.js 15 (App Router) + TypeScript + Tailwind CSS + Prisma + PostgreSQL',
+      DIR_MAP: `- src/app/ - Next.js App Router pages and API routes
+- src/lib/ - Shared utilities, database client, error handling
+- src/components/ - React components
+- prisma/ - Database schema and migrations
+- e2e/ - Playwright E2E tests`,
+    },
+    'fastapi-backend': {
+      STACK_SUMMARY: 'FastAPI + Python + SQLAlchemy 2.0 + PostgreSQL + Alembic',
+      DIR_MAP: `- backend/app/ - FastAPI application
+- backend/app/api/ - API route handlers
+- backend/app/core/ - Config, security, error handling
+- backend/app/db/ - Database session and models
+- backend/app/models/ - SQLAlchemy models
+- backend/app/schemas/ - Pydantic schemas
+- backend/tests/ - Pytest tests
+- backend/alembic/ - Database migrations`,
+    },
+    'polyglot-fullstack': {
+      STACK_SUMMARY: 'Next.js 15 (frontend) + FastAPI (backend) + PostgreSQL',
+      DIR_MAP: `- frontend/ - Next.js 15 App Router application
+- frontend/src/app/ - Pages and API routes
+- frontend/src/lib/ - Shared utilities
+- backend/ - FastAPI application
+- backend/app/api/ - API route handlers
+- backend/app/core/ - Config, security, error handling
+- backend/app/db/ - Database session and models
+- e2e/ - Playwright E2E tests`,
+    },
+    'react-express': {
+      STACK_SUMMARY: 'React (Vite) + Express + TypeScript + Prisma + PostgreSQL',
+      DIR_MAP: `- frontend/ - React (Vite) SPA
+- frontend/src/ - React components and pages
+- backend/ - Express API server
+- backend/src/ - Express routes and middleware
+- backend/src/routes/ - API route handlers
+- backend/prisma/ - Database schema and migrations`,
+    },
+    'remix-fullstack': {
+      STACK_SUMMARY: 'Remix + Vite + TypeScript + Tailwind CSS + Prisma + PostgreSQL',
+      DIR_MAP: `- app/ - Remix application
+- app/routes/ - File-based routes and API resource routes
+- app/routes/api.health.ts - Health check endpoint
+- prisma/ - Database schema and migrations`,
+    },
+    'hono-api': {
+      STACK_SUMMARY: 'Hono + TypeScript + Prisma + PostgreSQL',
+      DIR_MAP: `- src/ - Hono application
+- src/routes/ - API route handlers
+- src/routes/health.ts - Health check endpoints
+- prisma/ - Database schema and migrations`,
+    },
+  };
+  if (claudeVars[config.stackId]) {
+    Object.assign(vars, claudeVars[config.stackId]);
   }
 
   // Build skills list for CLAUDE.md reference
   const skillsList = [
-    '- `@.claude/skills/git-workflow/` — Git branching, commits, and PR workflow',
-    '- `@.claude/skills/testing-patterns/` — Test pyramid, AAA pattern, mocking strategies',
+    '- `@.claude/skills/git-workflow/`: Git branching, commits, and PR workflow',
+    '- `@.claude/skills/testing-patterns/`: Test pyramid, AAA pattern, mocking strategies',
   ];
   if (config.frontend?.framework === 'nextjs') {
-    skillsList.push('- `@.claude/skills/nextjs/` — Next.js patterns and conventions');
-    skillsList.push('- `@.claude/skills/security-web/` — Frontend security practices');
+    skillsList.push('- `@.claude/skills/nextjs/`: Next.js patterns and conventions');
+    skillsList.push('- `@.claude/skills/security-web/`: Frontend security practices');
   }
   if (config.backend?.framework === 'fastapi') {
-    skillsList.push('- `@.claude/skills/fastapi/` — FastAPI patterns and conventions');
-    skillsList.push('- `@.claude/skills/security-api/` — API security practices');
+    skillsList.push('- `@.claude/skills/fastapi/`: FastAPI patterns and conventions');
+    skillsList.push('- `@.claude/skills/security-api/`: API security practices');
   }
   if (config.testing?.e2e === 'playwright') {
-    skillsList.push('- `@.claude/skills/playwright/` — E2E testing patterns');
+    skillsList.push('- `@.claude/skills/playwright/`: E2E testing patterns');
   }
   if (config.ai) {
-    skillsList.push('- `@.claude/skills/ai-prompts/` — AI/LLM prompt patterns');
+    skillsList.push('- `@.claude/skills/ai-prompts/`: AI/LLM prompt patterns');
   }
   vars.SKILLS_LIST = skillsList.length > 0 ? skillsList.join('\n') : '- (none generated)';
 
@@ -98,14 +127,18 @@ function generateClaudeMd(outputDir, config, vars) {
   let content = readTemplate(basePath);
 
   // Read stack-specific section
-  let stackSection = '';
-  if (config.stackId === 'nextjs-fullstack') {
-    stackSection = readTemplate(path.join(CLAUDE_TEMPLATES_DIR, 'claude-md', 'nextjs.md'));
-  } else if (config.stackId === 'fastapi-backend') {
-    stackSection = readTemplate(path.join(CLAUDE_TEMPLATES_DIR, 'claude-md', 'fastapi.md'));
-  } else if (config.stackId === 'polyglot-fullstack') {
-    stackSection = readTemplate(path.join(CLAUDE_TEMPLATES_DIR, 'claude-md', 'fullstack.md'));
-  }
+  const claudeMdTemplates = {
+    'nextjs-fullstack': 'nextjs.md',
+    'fastapi-backend': 'fastapi.md',
+    'polyglot-fullstack': 'fullstack.md',
+    'react-express': 'fullstack.md',
+    'remix-fullstack': 'remix.md',
+    'hono-api': 'hono.md',
+  };
+  const mdFile = claudeMdTemplates[config.stackId];
+  const stackSection = mdFile
+    ? readTemplate(path.join(CLAUDE_TEMPLATES_DIR, 'claude-md', mdFile))
+    : '';
 
   content = content.replace('{{STACK_SPECIFIC_RULES}}', stackSection);
   content = replaceVars(content, vars);
@@ -114,18 +147,19 @@ function generateClaudeMd(outputDir, config, vars) {
 }
 
 function generateHooks(outputDir, config, options = {}) {
-  let hookFile;
-  let scriptFiles = ['guard-protected-files.mjs'];
-
-  if (config.stackId === 'nextjs-fullstack') {
-    hookFile = 'typescript.json';
-    scriptFiles.push('autofix-typescript.mjs');
-  } else if (config.stackId === 'fastapi-backend') {
-    hookFile = 'python.json';
-    scriptFiles.push('autofix-python.mjs');
-  } else if (config.stackId === 'polyglot-fullstack') {
-    hookFile = 'polyglot.json';
-    scriptFiles.push('autofix-polyglot.mjs');
+  const hookConfig = {
+    'nextjs-fullstack': { file: 'typescript.json', autofix: 'autofix-typescript.mjs' },
+    'fastapi-backend': { file: 'python.json', autofix: 'autofix-python.mjs' },
+    'polyglot-fullstack': { file: 'polyglot.json', autofix: 'autofix-polyglot.mjs' },
+    'react-express': { file: 'typescript.json', autofix: 'autofix-typescript.mjs' },
+    'remix-fullstack': { file: 'typescript.json', autofix: 'autofix-typescript.mjs' },
+    'hono-api': { file: 'typescript.json', autofix: 'autofix-typescript.mjs' },
+  };
+  const hook = hookConfig[config.stackId];
+  const hookFile = hook?.file;
+  const scriptFiles = ['guard-protected-files.mjs', 'code-hygiene.mjs', 'pre-commit-gate.mjs'];
+  if (hook?.autofix) {
+    scriptFiles.push(hook.autofix);
   }
 
   const settingsPath = path.join(outputDir, '.claude', 'settings.json');
@@ -134,7 +168,7 @@ function generateHooks(outputDir, config, options = {}) {
     // Merge hooks into existing settings.json
     const existing = JSON.parse(fs.readFileSync(settingsPath, 'utf-8'));
     const incoming = JSON.parse(readTemplate(path.join(CLAUDE_TEMPLATES_DIR, 'hooks', hookFile)));
-    const merged = deepMergeSettings(existing, incoming);
+    const merged = mergeHooksIntoSettings(existing, incoming);
     writeFile(settingsPath, JSON.stringify(merged, null, 2));
   } else {
     const hookPath = path.join(CLAUDE_TEMPLATES_DIR, 'hooks', hookFile);
@@ -153,7 +187,7 @@ function generateHooks(outputDir, config, options = {}) {
   }
 }
 
-function deepMergeSettings(existing, incoming) {
+function mergeHooksIntoSettings(existing, incoming) {
   const merged = { ...existing };
   if (incoming.hooks) {
     merged.hooks = merged.hooks || {};
@@ -178,7 +212,7 @@ function deepMergeSettings(existing, incoming) {
 }
 
 function generateSkills(outputDir, config) {
-  // Universal skills — always included
+  // Universal skills, always included
   const skillsToInclude = ['git-workflow', 'testing-patterns'];
 
   if (config.frontend?.framework === 'nextjs') {
@@ -227,6 +261,10 @@ function generateAgents(outputDir, config, vars) {
     'loop-operator.md',
     'harness-optimizer.md',
     'product-strategist.md',
+    'prompt-auditor.md',
+    'frontend-builder.md',
+    'enforcement-gate.md',
+    'deep-reviewer.md',
   ];
 
   for (const agent of agents) {
@@ -234,9 +272,14 @@ function generateAgents(outputDir, config, vars) {
     if (fs.existsSync(srcPath)) {
       let content = readTemplate(srcPath);
       content = replaceVars(content, vars);
+      content = stampManaged(content);
       writeFile(path.join(outputDir, '.claude', 'agents', agent), content);
     }
   }
+
+  // Clean up orphaned agents from older versions
+  const removed = cleanOrphans(path.join(outputDir, '.claude', 'agents'), agents);
+  return { count: agents.length, removed };
 }
 
 function generateCommands(outputDir, config, vars) {
@@ -245,7 +288,6 @@ function generateCommands(outputDir, config, vars) {
     'workflows.md',
     'status.md',
     'next.md',
-    'done.md',
     'verify-all.md',
     'audit-spec.md',
     'audit-wiring.md',
@@ -257,11 +299,19 @@ function generateCommands(outputDir, config, vars) {
     'optimize-claude-md.md',
     'plan.md',
     'build-fix.md',
+    'fix-loop.md',
     'code-review.md',
     'tdd.md',
     'save-session.md',
     'resume-session.md',
     'full-audit.md',
+    'verify-intent.md',
+    'build-ui.md',
+    'help.md',
+    'live-uat.md',
+    'simplify.md',
+    'generate-sdd.md',
+    'product-strategist.md',
   ];
 
   for (const cmd of commands) {
@@ -269,9 +319,52 @@ function generateCommands(outputDir, config, vars) {
     if (fs.existsSync(srcPath)) {
       let content = readTemplate(srcPath);
       content = replaceVars(content, vars);
+      content = stampManaged(content);
       writeFile(path.join(outputDir, '.claude', 'commands', cmd), content);
     }
   }
+
+  // Clean up orphaned commands from older versions
+  const removed = cleanOrphans(path.join(outputDir, '.claude', 'commands'), commands);
+  return { count: commands.length, removed };
+}
+
+const MANAGED_MARKER = '<!-- Generated by DevForge -->';
+
+/**
+ * Append the DevForge managed marker to generated file content.
+ */
+function stampManaged(content) {
+  return content.trimEnd() + '\n\n' + MANAGED_MARKER + '\n';
+}
+
+/**
+ * Remove DevForge-managed .md files that are no longer in the current manifest.
+ * Only deletes files stamped with the DevForge marker. User-created custom
+ * files are left untouched. Returns array of removed filenames.
+ */
+function cleanOrphans(dir, expectedFiles) {
+  const removed = [];
+  if (!fs.existsSync(dir)) return removed;
+
+  const entries = fs.readdirSync(dir, { withFileTypes: true });
+  const existing = entries
+    .filter(e => e.isFile() && !e.isSymbolicLink() && e.name.endsWith('.md'))
+    .map(e => e.name);
+  const expectedSet = new Set(expectedFiles);
+  for (const file of existing) {
+    if (!expectedSet.has(file)) {
+      const filePath = path.join(dir, file);
+      const stat = fs.statSync(filePath);
+      if (stat.size > 1_000_000) continue; // skip unexpectedly large files
+      const content = fs.readFileSync(filePath, 'utf-8');
+      if (content.includes(MANAGED_MARKER)) {
+        fs.unlinkSync(filePath);
+        removed.push(file);
+      }
+    }
+  }
+  return removed;
 }
 
 function copyPromptLibrary(outputDir) {

package/src/cli.js

@@ -5,7 +5,13 @@ import { log } from './utils.js';
 
 export async function parseCommand(args) {
   if (args.includes('--version') || args.includes('-v')) {
-    const pkg = JSON.parse(fs.readFileSync(new URL('../package.json', import.meta.url), 'utf-8'));
+    let pkg;
+    try {
+      pkg = JSON.parse(fs.readFileSync(new URL('../package.json', import.meta.url), 'utf-8'));
+    } catch (err) {
+      console.error(`Could not read package.json: ${err.message}`);
+      process.exit(1);
+    }
     console.log(pkg.version);
     process.exit(0);
   }
@@ -18,8 +24,10 @@ export async function parseCommand(args) {
   const command = args[0];
 
   if (!command) {
-    showUsage();
-    process.exit(0);
+    const { showInteractiveMenu, handleMenuSelection } = await import('./menu.js');
+    const selected = await showInteractiveMenu();
+    await handleMenuSelection(selected);
+    return;
   }
 
   if (command === 'new') {
@@ -46,6 +54,12 @@ export async function parseCommand(args) {
     return;
   }
 
+  if (command === 'ci') {
+    const { runCI } = await import('./ci-mode.js');
+    await runCI(process.cwd());
+    return;
+  }
+
   if (command === 'update') {
     const { runUpdate } = await import('./update.js');
     await runUpdate();
@@ -73,12 +87,13 @@ export async function parseCommand(args) {
 
 function showUsage() {
   console.log(`
-  ${chalk.bold.cyan('DevForge')} — AI-first project scaffolding
+  ${chalk.bold.cyan('DevForge')}  AI-first project scaffolding
 
   ${chalk.bold('Usage:')}
     devforge new <name>     Create a new project
     devforge init           Add dev guardrails to current project
     devforge doctor         Diagnose and optimize current project
+    devforge ci             Run health checks for CI/CD (non-interactive)
     devforge update         Check for newer version
     devforge <name>         Shorthand for ${chalk.dim('devforge new <name>')}
 
@@ -92,14 +107,14 @@ function showUsage() {
 
 function showHelp() {
   console.log(`
-  ${chalk.bold.cyan('DevForge')} — Universal, AI-first project scaffolding
+  ${chalk.bold.cyan('DevForge')}  Universal, AI-first project scaffolding
 
   ${chalk.bold('Commands:')}
 
     ${chalk.bold('devforge new <name>')}
       Create a new project. Choose between:
-      ${chalk.dim('• Guided mode — describe what you want in plain English')}
-      ${chalk.dim('• Developer mode — pick your stack directly')}
+      ${chalk.dim('• Guided mode: describe what you want in plain English')}
+      ${chalk.dim('• Developer mode: pick your stack directly')}
 
     ${chalk.bold('devforge init')}
       Add Claude Code infrastructure to an existing project.
@@ -112,6 +127,11 @@ function showHelp() {
       flaky tests, dead code, duplicate code, and more.
       Generates Claude Code prompts to fix each issue.
 
+    ${chalk.bold('devforge ci')}
+      Run project health checks non-interactively for CI/CD.
+      Exits with code 1 if critical issues found.
+      Saves report to docs/ci-report.md if docs/ exists.
+
     ${chalk.bold('devforge update')}
       Check if a newer version of DevForge is available.
       Shows upgrade instructions if an update exists.
@@ -120,6 +140,9 @@ function showHelp() {
     - Next.js full-stack (TypeScript + Prisma + PostgreSQL)
     - FastAPI backend (Python + SQLAlchemy + PostgreSQL)
     - Polyglot full-stack (Next.js + FastAPI monorepo)
+    - React + Express (Vite + Express + Prisma + PostgreSQL)
+    - Remix full-stack (Vite + Tailwind + Prisma + PostgreSQL)
+    - Hono API (TypeScript + Prisma + PostgreSQL)
 
   ${chalk.bold('Examples:')}
     devforge new my-app