claude-launchpad 0.7.8 → 0.8.0

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,89 +1306,6 @@ async function analyzeMcp(config) {
   return { name: "MCP Servers", issues, score };
 }
 
-// src/commands/doctor/analyzers/quality.ts
-var ESSENTIAL_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" }
-];
-var VAGUE_PATTERNS = [
-  { pattern: /write (good|clean|quality|nice) code/i, label: "write good code" },
-  { pattern: /be (careful|thorough|diligent)/i, label: "be careful" },
-  { pattern: /follow best practices/i, label: "follow best practices" },
-  { pattern: /make sure (everything|it) works/i, label: "make sure it works" }
-];
-var SECRET_PATTERNS = [
-  { pattern: /sk-[a-zA-Z0-9]{20,}/, label: "OpenAI API key" },
-  { pattern: /ghp_[a-zA-Z0-9]{36}/, label: "GitHub personal token" },
-  { pattern: /AKIA[0-9A-Z]{16}/, label: "AWS access key" },
-  { pattern: /xoxb-[0-9]+-[a-zA-Z0-9]+/, label: "Slack bot token" }
-];
-async function analyzeQuality(config) {
-  const issues = [];
-  const content = config.claudeMdContent;
-  if (content === null) {
-    issues.push({
-      analyzer: "Quality",
-      severity: "high",
-      message: "No CLAUDE.md found",
-      fix: "Run `claude-launchpad init` to generate one"
-    });
-    return { name: "CLAUDE.md Quality", issues, score: 0 };
-  }
-  let sectionsFound = 0;
-  for (const section of ESSENTIAL_SECTIONS) {
-    if (section.pattern.test(content)) {
-      sectionsFound++;
-    } else {
-      issues.push({
-        analyzer: "Quality",
-        severity: "medium",
-        message: `Missing "## ${section.name}" section \u2014 ${section.why}`,
-        fix: `Add a ## ${section.name} section to CLAUDE.md`
-      });
-    }
-  }
-  for (const vague of VAGUE_PATTERNS) {
-    if (vague.pattern.test(content)) {
-      issues.push({
-        analyzer: "Quality",
-        severity: "low",
-        message: `Vague instruction detected: "${vague.label}" \u2014 zero signal, wastes budget`,
-        fix: "Replace with specific, actionable instructions"
-      });
-    }
-  }
-  for (const secret of SECRET_PATTERNS) {
-    if (secret.pattern.test(content)) {
-      issues.push({
-        analyzer: "Quality",
-        severity: "critical",
-        message: `Possible ${secret.label} found in CLAUDE.md \u2014 secrets must never be in config files`,
-        fix: "Remove the secret immediately and rotate it"
-      });
-    }
-  }
-  const todoCount = (content.match(/<!--\s*TODO/gi) ?? []).length;
-  if (todoCount > 3) {
-    issues.push({
-      analyzer: "Quality",
-      severity: "medium",
-      message: `${todoCount} TODO placeholders \u2014 CLAUDE.md is mostly unfinished`,
-      fix: "Fill in the TODO sections or remove them"
-    });
-  }
-  const criticals = issues.filter((i) => i.severity === "critical").length;
-  const highs = issues.filter((i) => i.severity === "high").length;
-  const mediums = issues.filter((i) => i.severity === "medium").length;
-  const lows = issues.filter((i) => i.severity === "low").length;
-  const score = Math.max(0, 100 - criticals * 40 - highs * 30 - mediums * 15 - lows * 5);
-  return { name: "CLAUDE.md Quality", issues, score };
-}
-
 // src/commands/doctor/analyzers/memory.ts
 var MEMORY_MCP_TOOLS = [
   "mcp__agentic-memory__memory_store",
@@ -1484,6 +1393,90 @@ async function analyzeMemory(config) {
   return { name: "Memory", issues, score };
 }
 
+// src/commands/doctor/analyzers/quality.ts
+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" }
+];
+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" },
+  { pattern: /follow best practices/i, label: "follow best practices" },
+  { pattern: /make sure (everything|it) works/i, label: "make sure it works" }
+];
+var SECRET_PATTERNS = [
+  { pattern: /sk-[a-zA-Z0-9]{20,}/, label: "OpenAI API key" },
+  { pattern: /ghp_[a-zA-Z0-9]{36}/, label: "GitHub personal token" },
+  { pattern: /AKIA[0-9A-Z]{16}/, label: "AWS access key" },
+  { pattern: /xoxb-[0-9]+-[a-zA-Z0-9]+/, label: "Slack bot token" }
+];
+async function analyzeQuality(config) {
+  const issues = [];
+  const content = config.claudeMdContent;
+  if (content === null) {
+    issues.push({
+      analyzer: "Quality",
+      severity: "high",
+      message: "No CLAUDE.md found",
+      fix: "Run `claude-launchpad init` to generate one"
+    });
+    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 sections) {
+    if (section.pattern.test(content)) {
+      sectionsFound++;
+    } else {
+      issues.push({
+        analyzer: "Quality",
+        severity: "medium",
+        message: `Missing "## ${section.name}" section \u2014 ${section.why}`,
+        fix: `Add a ## ${section.name} section to CLAUDE.md`
+      });
+    }
+  }
+  for (const vague of VAGUE_PATTERNS) {
+    if (vague.pattern.test(content)) {
+      issues.push({
+        analyzer: "Quality",
+        severity: "low",
+        message: `Vague instruction detected: "${vague.label}" \u2014 zero signal, wastes budget`,
+        fix: "Replace with specific, actionable instructions"
+      });
+    }
+  }
+  for (const secret of SECRET_PATTERNS) {
+    if (secret.pattern.test(content)) {
+      issues.push({
+        analyzer: "Quality",
+        severity: "critical",
+        message: `Possible ${secret.label} found in CLAUDE.md \u2014 secrets must never be in config files`,
+        fix: "Remove the secret immediately and rotate it"
+      });
+    }
+  }
+  const todoCount = (content.match(/<!--\s*TODO/gi) ?? []).length;
+  if (todoCount > 3) {
+    issues.push({
+      analyzer: "Quality",
+      severity: "medium",
+      message: `${todoCount} TODO placeholders \u2014 CLAUDE.md is mostly unfinished`,
+      fix: "Fill in the TODO sections or remove them"
+    });
+  }
+  const criticals = issues.filter((i) => i.severity === "critical").length;
+  const highs = issues.filter((i) => i.severity === "high").length;
+  const mediums = issues.filter((i) => i.severity === "medium").length;
+  const lows = issues.filter((i) => i.severity === "low").length;
+  const score = Math.max(0, 100 - criticals * 40 - highs * 30 - mediums * 15 - lows * 5);
+  return { name: "CLAUDE.md Quality", 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 +1517,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) },
@@ -2626,7 +2618,7 @@ function createMemoryCommand() {
         log.info("Skipped.");
         return;
       }
-      const { runInstall } = await import("./install-56GBDZHM.js");
+      const { runInstall } = await import("./install-3IW2PDOS.js");
       await runInstall({});
     } else {
       const { requireMemoryDeps } = await import("./require-deps-6D6IBICL.js");
@@ -2660,7 +2652,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.8", "-v, --version").action(async () => {
+var program = new Command5().name("claude-launchpad").description("CLI toolkit that makes Claude Code setups measurably good").version("0.8.0", "-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" });