claude-launchpad 0.7.9 → 0.8.1

package/dist/cli.js

@@ -270,14 +270,6 @@ When all tasks in the current sprint are complete, do a quick quality check befo
 - Never hardcode secrets \u2014 use environment variables
 - Never write to \`.env\` files
 - Never expose internal error details in API responses`);
-  sections.push("", `## Memory & Learnings
-Use the built-in memory system to persist knowledge across sessions:
-- **Save immediately** when you discover: a non-obvious fix, a gotcha, an external resource, a decision with context that would be lost, or a known issue to fix later
-- **Categories**: \`decision\` (why X over Y), \`gotcha\` (non-obvious pitfall), \`deferred\` (known issue, not urgent), \`reference\` (where to find things)
-- **Where**: project memory for this repo, global memory for cross-project learnings
-- **Format**: one fact per memory, include date and why \u2014 not just what
-- **Prune**: check if a memory on this topic exists before saving \u2014 update, don't duplicate
-- Before starting work, check memory for relevant context from previous sessions`);
   sections.push("", `## Key Decisions
 <!-- Record architectural decisions as you make them -->`);
   return sections.join("\n") + "\n";
@@ -561,7 +553,7 @@ CLAUDE.md must stay UNDER 120 lines of actionable content (not counting headings
 2. **## Architecture** - 3-5 bullet points describing the codebase shape (not a full directory tree)
 3. **## Conventions** - max 8 key patterns. Move detailed rules to .claude/rules/conventions.md
 4. **## Off-Limits** - max 8 guardrails specific to this project
-5. **## Memory & Learnings** - max 6 bullets. If missing, add instructions for using the built-in memory system: what to save (gotchas, decisions, deferred issues, references), where (project vs global memory), and the rule to check existing memories before creating duplicates
+5. **## Memory & Learnings** - ONLY if the project already has a ## Memory section or agentic-memory is configured in .claude/settings.json. If present, keep to max 6 bullets. If the project does NOT use memory, do NOT add this section
 6. **## Key Decisions** - only decisions that affect how Claude should work in this codebase
 7. **MCP server suggestions** - look at what external services the project uses (databases, APIs, storage). If you spot Postgres, Redis, Stripe, GitHub API, or similar, suggest relevant MCP servers. Print as suggestions at the end, not in CLAUDE.md.
 
@@ -1314,15 +1306,93 @@ async function analyzeMcp(config) {
   return { name: "MCP Servers", issues, score };
 }
 
+// src/commands/doctor/analyzers/memory.ts
+var MEMORY_MCP_TOOLS = [
+  "mcp__agentic-memory__memory_store",
+  "mcp__agentic-memory__memory_search",
+  "mcp__agentic-memory__memory_recent",
+  "mcp__agentic-memory__memory_forget",
+  "mcp__agentic-memory__memory_relate",
+  "mcp__agentic-memory__memory_stats",
+  "mcp__agentic-memory__memory_update"
+];
+function hasMemoryIndicators(config) {
+  const hasMcpServer = config.mcpServers.some((s) => s.name === "agentic-memory");
+  const hasHookRef = config.hooks.some(
+    (h) => h.command?.includes("memory context") || h.command?.includes("memory extract")
+  );
+  return hasMcpServer || hasHookRef;
+}
+async function analyzeMemory(config) {
+  if (!hasMemoryIndicators(config)) return null;
+  const issues = [];
+  const hasSessionStart = config.hooks.some(
+    (h) => h.event === "SessionStart" && h.command?.includes("memory context")
+  );
+  if (!hasSessionStart) {
+    issues.push({
+      analyzer: "Memory",
+      severity: "high",
+      message: "No SessionStart hook with memory context injection",
+      fix: "Add a SessionStart hook that runs `memory context` to inject relevant memories"
+    });
+  }
+  const hasStopHook = config.hooks.some(
+    (h) => h.event === "Stop" && h.command?.includes("memory extract")
+  );
+  if (!hasStopHook) {
+    issues.push({
+      analyzer: "Memory",
+      severity: "medium",
+      message: "No Stop hook with memory extract for session learnings",
+      fix: "Add a Stop hook that runs `memory extract` to capture session insights"
+    });
+  }
+  const autoMemoryDisabled = config.settings?.autoMemoryEnabled === false;
+  if (!autoMemoryDisabled) {
+    issues.push({
+      analyzer: "Memory",
+      severity: "medium",
+      message: "autoMemoryEnabled not disabled \u2014 built-in memory may conflict with agentic-memory",
+      fix: "Set autoMemoryEnabled: false in .claude/settings.json"
+    });
+  }
+  const hasMemoryGuidance = config.claudeMdContent?.includes("agentic-memory") || config.claudeMdContent?.includes("## Memory");
+  if (!hasMemoryGuidance) {
+    issues.push({
+      analyzer: "Memory",
+      severity: "low",
+      message: "CLAUDE.md missing memory guidance section",
+      fix: "Add a ## Memory section to CLAUDE.md describing when and how to use agentic-memory"
+    });
+  }
+  const allowList = config.settings?.allowedTools ?? [];
+  const missingTools = MEMORY_MCP_TOOLS.filter((t) => !allowList.includes(t));
+  if (missingTools.length > 0) {
+    issues.push({
+      analyzer: "Memory",
+      severity: "low",
+      message: `${missingTools.length} agentic-memory MCP tool permission(s) missing from allowedTools`,
+      fix: "Add all agentic-memory tool names to allowedTools in .claude/settings.json"
+    });
+  }
+  const critical = issues.filter((i) => i.severity === "critical").length;
+  const high = issues.filter((i) => i.severity === "high").length;
+  const medium = issues.filter((i) => i.severity === "medium").length;
+  const low = issues.filter((i) => i.severity === "low").length;
+  const score = Math.max(0, 100 - (critical * 40 + high * 20 + medium * 10 + low * 5));
+  return { name: "Memory", issues, score };
+}
+
 // src/commands/doctor/analyzers/quality.ts
-var ESSENTIAL_SECTIONS = [
+var BASE_SECTIONS = [
   { pattern: /^##\s+(Tech )?Stack/m, name: "Stack", why: "Claude performs worse without knowing the tech stack" },
   { pattern: /^##\s+Commands/m, name: "Commands", why: "Claude guesses wrong without explicit dev/build/test commands" },
   { pattern: /^##\s+Session Start/m, name: "Session Start", why: "Without this, Claude won't read TASKS.md or maintain continuity" },
   { pattern: /^##\s+Off.?Limits/m, name: "Off-Limits", why: "Without guardrails, Claude has no boundaries beyond defaults" },
-  { pattern: /^##\s+(Architecture|Project Structure)/m, name: "Architecture/Structure", why: "Claude makes better decisions when it understands the codebase shape" },
-  { pattern: /^##\s+Memory/m, name: "Memory & Learnings", why: "Without memory instructions, Claude forgets learnings and repeats mistakes across sessions" }
+  { pattern: /^##\s+(Architecture|Project Structure)/m, name: "Architecture/Structure", why: "Claude makes better decisions when it understands the codebase shape" }
 ];
+var MEMORY_SECTION = { pattern: /^##\s+Memory/m, name: "Memory & Learnings", why: "Without memory instructions, Claude forgets learnings and repeats mistakes across sessions" };
 var VAGUE_PATTERNS = [
   { pattern: /write (good|clean|quality|nice) code/i, label: "write good code" },
   { pattern: /be (careful|thorough|diligent)/i, label: "be careful" },
@@ -1347,8 +1417,9 @@ async function analyzeQuality(config) {
     });
     return { name: "CLAUDE.md Quality", issues, score: 0 };
   }
+  const sections = hasMemoryIndicators(config) ? [...BASE_SECTIONS, MEMORY_SECTION] : [...BASE_SECTIONS];
   let sectionsFound = 0;
-  for (const section of ESSENTIAL_SECTIONS) {
+  for (const section of sections) {
     if (section.pattern.test(content)) {
       sectionsFound++;
     } else {
@@ -1397,93 +1468,6 @@ async function analyzeQuality(config) {
   return { name: "CLAUDE.md Quality", issues, score };
 }
 
-// src/commands/doctor/analyzers/memory.ts
-var MEMORY_MCP_TOOLS = [
-  "mcp__agentic-memory__memory_store",
-  "mcp__agentic-memory__memory_search",
-  "mcp__agentic-memory__memory_recent",
-  "mcp__agentic-memory__memory_forget",
-  "mcp__agentic-memory__memory_relate",
-  "mcp__agentic-memory__memory_stats",
-  "mcp__agentic-memory__memory_update"
-];
-function hasMemoryIndicators(config) {
-  const hasMcpServer = config.mcpServers.some((s) => s.name === "agentic-memory");
-  const hasHookRef = config.hooks.some(
-    (h) => h.command?.includes("memory context") || h.command?.includes("memory extract")
-  );
-  return hasMcpServer || hasHookRef;
-}
-async function analyzeMemory(config) {
-  if (!hasMemoryIndicators(config)) return null;
-  const issues = [];
-  const hasMcpServer = config.mcpServers.some((s) => s.name === "agentic-memory");
-  if (!hasMcpServer) {
-    issues.push({
-      analyzer: "Memory",
-      severity: "high",
-      message: "agentic-memory MCP server not found in mcpServers",
-      fix: "Add agentic-memory to mcpServers in .claude/settings.json"
-    });
-  }
-  const hasSessionStart = config.hooks.some(
-    (h) => h.event === "SessionStart" && h.command?.includes("memory context")
-  );
-  if (!hasSessionStart) {
-    issues.push({
-      analyzer: "Memory",
-      severity: "high",
-      message: "No SessionStart hook with memory context injection",
-      fix: "Add a SessionStart hook that runs `memory context` to inject relevant memories"
-    });
-  }
-  const hasStopHook = config.hooks.some(
-    (h) => h.event === "Stop" && h.command?.includes("memory extract")
-  );
-  if (!hasStopHook) {
-    issues.push({
-      analyzer: "Memory",
-      severity: "medium",
-      message: "No Stop hook with memory extract for session learnings",
-      fix: "Add a Stop hook that runs `memory extract` to capture session insights"
-    });
-  }
-  const autoMemoryDisabled = config.settings?.autoMemoryEnabled === false;
-  if (!autoMemoryDisabled) {
-    issues.push({
-      analyzer: "Memory",
-      severity: "medium",
-      message: "autoMemoryEnabled not disabled \u2014 built-in memory may conflict with agentic-memory",
-      fix: "Set autoMemoryEnabled: false in .claude/settings.json"
-    });
-  }
-  const hasMemoryGuidance = config.claudeMdContent?.includes("agentic-memory") || config.claudeMdContent?.includes("## Memory");
-  if (!hasMemoryGuidance) {
-    issues.push({
-      analyzer: "Memory",
-      severity: "low",
-      message: "CLAUDE.md missing memory guidance section",
-      fix: "Add a ## Memory section to CLAUDE.md describing when and how to use agentic-memory"
-    });
-  }
-  const allowList = config.settings?.allowedTools ?? [];
-  const missingTools = MEMORY_MCP_TOOLS.filter((t) => !allowList.includes(t));
-  if (missingTools.length > 0) {
-    issues.push({
-      analyzer: "Memory",
-      severity: "low",
-      message: `${missingTools.length} agentic-memory MCP tool permission(s) missing from allowedTools`,
-      fix: "Add all agentic-memory tool names to allowedTools in .claude/settings.json"
-    });
-  }
-  const critical = issues.filter((i) => i.severity === "critical").length;
-  const high = issues.filter((i) => i.severity === "high").length;
-  const medium = issues.filter((i) => i.severity === "medium").length;
-  const low = issues.filter((i) => i.severity === "low").length;
-  const score = Math.max(0, 100 - (critical * 40 + high * 20 + medium * 10 + low * 5));
-  return { name: "Memory", issues, score };
-}
-
 // src/commands/doctor/fixer.ts
 import { readFile as readFile4, writeFile as writeFile2, mkdir as mkdir2, access as access4 } from "fs/promises";
 import { join as join5 } from "path";
@@ -1524,7 +1508,6 @@ var FIX_TABLE = [
   { analyzer: "Quality", match: "Session Start", fix: (root) => addClaudeMdSection(root, "## Session Start", "- ALWAYS read @TASKS.md first - it tracks progress across sessions\n- Update TASKS.md as you complete work") },
   { analyzer: "Rules", match: "No .claudeignore", fix: (root, detected) => createClaudeignore(root, detected) },
   { analyzer: "Rules", match: "No .claude/rules/", fix: (root) => createStarterRules(root) },
-  { analyzer: "Quality", match: "Memory", fix: (root) => addClaudeMdSection(root, "## Memory & Learnings", "Use the built-in memory system to persist knowledge across sessions:\n- **Save immediately** when you discover: a non-obvious fix, a gotcha, an external resource, a decision with context that would be lost, or a known issue to fix later\n- **Categories**: `decision` (why X over Y), `gotcha` (non-obvious pitfall), `deferred` (known issue, not urgent), `reference` (where to find things)\n- **Where**: project memory for this repo, global memory for cross-project learnings\n- **Format**: one fact per memory, include date and why \u2014 not just what\n- **Prune**: check if a memory on this topic exists before saving \u2014 update, don't duplicate\n- Before starting work, check memory for relevant context from previous sessions") },
   { analyzer: "Hooks", match: "PostCompact", fix: (root) => addPostCompactHook(root) },
   { analyzer: "Permissions", match: "force-push", fix: (root) => addForcePushProtection(root) },
   { analyzer: "Permissions", match: "Credential files not blocked", fix: (root) => addCredentialDenyRules(root) },
@@ -1947,23 +1930,23 @@ function createDoctorCommand() {
         if (fixed > 0) {
           log.success(`Applied ${fixed} fix(es). Re-scanning...`);
           log.blank();
-          const updatedConfig = await parseClaudeConfig(opts.path);
-          const updatedResults = await Promise.all([
-            analyzeBudget(updatedConfig),
-            analyzeQuality(updatedConfig),
-            analyzeSettings(updatedConfig),
-            analyzeHooks(updatedConfig),
-            analyzeRules(updatedConfig),
-            analyzePermissions(updatedConfig),
-            analyzeMcp(updatedConfig)
-          ]);
-          const updatedMemoryResult = await analyzeMemory(updatedConfig);
-          if (updatedMemoryResult) {
-            updatedResults.push(updatedMemoryResult);
-          }
-          renderDoctorReport(updatedResults, { afterFix: true });
-          log.info(`Then use ${chalk.bold("/lp-enhance")} inside Claude Code to have Claude restructure and complete your CLAUDE.md.`);
         }
+        const updatedConfig = await parseClaudeConfig(opts.path);
+        const updatedResults = await Promise.all([
+          analyzeBudget(updatedConfig),
+          analyzeQuality(updatedConfig),
+          analyzeSettings(updatedConfig),
+          analyzeHooks(updatedConfig),
+          analyzeRules(updatedConfig),
+          analyzePermissions(updatedConfig),
+          analyzeMcp(updatedConfig)
+        ]);
+        const updatedMemoryResult = await analyzeMemory(updatedConfig);
+        if (updatedMemoryResult) {
+          updatedResults.push(updatedMemoryResult);
+        }
+        renderDoctorReport(updatedResults, { afterFix: true });
+        log.info(`Then use ${chalk.bold("/lp-enhance")} inside Claude Code to have Claude restructure and complete your CLAUDE.md.`);
       }
     }
     if (opts.minScore) {
@@ -2660,7 +2643,7 @@ function createMemoryCommand() {
 }
 
 // src/cli.ts
-var program = new Command5().name("claude-launchpad").description("CLI toolkit that makes Claude Code setups measurably good").version("0.7.9", "-v, --version").action(async () => {
+var program = new Command5().name("claude-launchpad").description("CLI toolkit that makes Claude Code setups measurably good").version("0.8.1", "-v, --version").action(async () => {
   const hasConfig = await fileExists(join11(process.cwd(), "CLAUDE.md")) || await fileExists(join11(process.cwd(), ".claude", "settings.json"));
   if (hasConfig) {
     await program.commands.find((c) => c.name() === "doctor")?.parseAsync([], { from: "user" });