claudex-setup 1.0.0 → 1.0.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claudex-setup",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "Audit and optimize any project for Claude Code. Powered by 1107 verified techniques.",
   "main": "src/index.js",
   "bin": {

package/src/setup.js

@@ -29,12 +29,20 @@ function detectScripts(ctx) {
 // Helper: detect main directories
 // ============================================================
 function detectMainDirs(ctx) {
-  const candidates = ['src', 'lib', 'app', 'pages', 'components', 'api', 'routes', 'utils', 'helpers', 'services', 'models', 'controllers', 'views', 'public', 'assets', 'config', 'tests', 'test', '__tests__', 'spec', 'scripts', 'prisma', 'db', 'middleware'];
+  const candidates = ['src', 'lib', 'app', 'pages', 'components', 'api', 'routes', 'utils', 'helpers', 'services', 'models', 'controllers', 'views', 'public', 'assets', 'config', 'tests', 'test', '__tests__', 'spec', 'scripts', 'prisma', 'db', 'middleware', 'hooks'];
+  // Also check inside src/ for nested structure (common in Next.js, React)
+  const srcNested = ['src/components', 'src/app', 'src/pages', 'src/api', 'src/lib', 'src/hooks', 'src/utils', 'src/services', 'src/models', 'src/middleware', 'src/app/api', 'app/api'];
   const found = [];
-  for (const dir of candidates) {
+  const seenNames = new Set();
+
+  for (const dir of [...candidates, ...srcNested]) {
     if (ctx.hasDir(dir)) {
       const files = ctx.dirFiles(dir);
-      found.push({ name: dir, fileCount: files.length, files: files.slice(0, 10) });
+      const displayName = dir.includes('/') ? dir : dir;
+      if (!seenNames.has(displayName)) {
+        found.push({ name: displayName, fileCount: files.length, files: files.slice(0, 10) });
+        seenNames.add(displayName);
+      }
     }
   }
   return found;
@@ -61,31 +69,63 @@ function generateMermaid(dirs, stacks) {
     return `    ${id}[${label}]`;
   }
 
-  // Entry point
-  nodes.push(addNode('Entry Point', 'round'));
+  // Detect Next.js App Router specifically
+  const hasAppRouter = dirNames.includes('app') || dirNames.includes('src/app');
+  const hasPages = dirNames.includes('pages') || dirNames.includes('src/pages');
+  const hasAppApi = dirNames.includes('app/api') || dirNames.includes('src/app/api');
+  const hasSrcComponents = dirNames.includes('src/components') || dirNames.includes('components');
+  const hasSrcHooks = dirNames.includes('src/hooks') || dirNames.includes('hooks');
+  const hasSrcLib = dirNames.includes('src/lib') || dirNames.includes('lib');
+
+  // Smart entry point based on framework
+  const isNextJs = stackKeys.includes('nextjs');
+  const isDjango = stackKeys.includes('django');
+  const isFastApi = stackKeys.includes('fastapi');
+
+  if (isNextJs) {
+    nodes.push(addNode('Next.js', 'round'));
+  } else if (isDjango) {
+    nodes.push(addNode('Django', 'round'));
+  } else if (isFastApi) {
+    nodes.push(addNode('FastAPI', 'round'));
+  } else {
+    nodes.push(addNode('Entry Point', 'round'));
+  }
+
+  const root = ids['Next.js'] || ids['Django'] || ids['FastAPI'] || ids['Entry Point'];
 
   // Detect layers
-  if (dirNames.includes('app') || dirNames.includes('pages')) {
-    nodes.push(addNode('Pages / Routes', 'default'));
-    edges.push(`    ${ids['Entry Point']} --> ${ids['Pages / Routes']}`);
+  if (hasAppRouter || hasPages) {
+    const label = hasAppRouter ? 'App Router' : 'Pages';
+    nodes.push(addNode(label, 'default'));
+    edges.push(`    ${root} --> ${ids[label]}`);
+  }
+
+  if (hasAppApi) {
+    nodes.push(addNode('API Routes', 'default'));
+    const parent = ids['App Router'] || ids['Pages'] || root;
+    edges.push(`    ${parent} --> ${ids['API Routes']}`);
   }
 
-  if (dirNames.includes('components')) {
+  if (hasSrcComponents) {
     nodes.push(addNode('Components', 'default'));
-    const parent = ids['Pages / Routes'] || ids['Entry Point'];
+    const parent = ids['App Router'] || ids['Pages'] || root;
     edges.push(`    ${parent} --> ${ids['Components']}`);
   }
 
-  if (dirNames.includes('src')) {
-    nodes.push(addNode('src/', 'default'));
-    const parent = ids['Pages / Routes'] || ids['Entry Point'];
-    edges.push(`    ${parent} --> ${ids['src/']}`);
+  if (hasSrcHooks) {
+    nodes.push(addNode('Hooks', 'default'));
+    const parent = ids['Components'] || root;
+    edges.push(`    ${parent} --> ${ids['Hooks']}`);
   }
 
-  if (dirNames.includes('lib')) {
+  if (hasSrcLib) {
     nodes.push(addNode('lib/', 'default'));
-    const parent = ids['src/'] || ids['Entry Point'];
+    const parent = ids['API Routes'] || ids['Hooks'] || ids['Components'] || root;
     edges.push(`    ${parent} --> ${ids['lib/']}`);
+  } else if (dirNames.includes('src') && !hasAppRouter && !hasPages) {
+    nodes.push(addNode('src/', 'default'));
+    edges.push(`    ${root} --> ${ids['src/']}`);
   }
 
   if (dirNames.includes('api') || dirNames.includes('routes') || dirNames.includes('controllers')) {
@@ -379,6 +419,9 @@ ${verificationSteps.join('\n')}
 - Use descriptive commit messages (why, not what)
 - Create focused PRs — one concern per PR
 - Document non-obvious decisions in code comments
+
+---
+*Generated by [claudex-setup](https://github.com/DnaFin/claudex-setup) v${require('../package.json').version} on ${new Date().toISOString().split('T')[0]}. Customize this file for your project — a hand-crafted CLAUDE.md will always be better than a generated one.*
 `;
   },
 
@@ -413,8 +456,9 @@ echo '{"decision": "allow"}'
 # Appends to .claude/logs/file-changes.log
 
 INPUT=$(cat -)
-TOOL_NAME=$(echo "$INPUT" | grep -oP '"tool_name"\\s*:\\s*"\\K[^"]+' 2>/dev/null || echo "unknown")
-FILE_PATH=$(echo "$INPUT" | grep -oP '"file_path"\\s*:\\s*"\\K[^"]+' 2>/dev/null || echo "")
+TOOL_NAME=$(echo "$INPUT" | sed -n 's/.*"tool_name"[[:space:]]*:[[:space:]]*"\\([^"]*\\)".*/\\1/p')
+TOOL_NAME=\${TOOL_NAME:-unknown}
+FILE_PATH=$(echo "$INPUT" | sed -n 's/.*"file_path"[[:space:]]*:[[:space:]]*"\\([^"]*\\)".*/\\1/p')
 
 if [ -z "$FILE_PATH" ]; then
   exit 0
@@ -590,7 +634,13 @@ async function setup(options) {
 
     if (typeof result === 'string') {
       // Single file template (like CLAUDE.md)
-      const filePath = key === 'claudeMd' ? 'CLAUDE.md' : key;
+      // Map technique keys to actual file paths
+      const filePathMap = {
+        'claudeMd': 'CLAUDE.md',
+        'mermaidArchitecture': 'CLAUDE.md', // mermaid is part of CLAUDE.md, skip separate file
+      };
+      if (key === 'mermaidArchitecture') continue; // Mermaid is generated inside CLAUDE.md template
+      const filePath = filePathMap[key] || key;
       const fullPath = path.join(options.dir, filePath);
 
       if (!fs.existsSync(fullPath)) {