@axiomatic-labs/claudeflow 2.43.30 → 2.43.32

package/lib/install.js

@@ -102,66 +102,90 @@ function syncClaudeMd(cwd, snippetPath) {
 // Place + merge the framework from an extracted payload dir into a project. Pure filesystem, no network —
 // so it is testable on a throwaway dir.
 function installFromPayload(src, cwd) {
-  const out = { playbooks: 0, scripts: 0, skills: 0, agents: 0, claudeMd: false };
+  const out = { playbooks: 0, scripts: 0, skills: 0, agents: 0, claudeMd: false, failed: [] };
+  // Each placement step is ISOLATED: a failure in one (a copy error, a permission hiccup, a weird file in the
+  // user's tree) must NOT abort the rest — above all it must not skip the settings/env + CLAUDE.md config, the
+  // two steps a user most needs (the env block wires agent-teams + the autocompact window, etc.). Before this,
+  // any throw in steps 1–4 silently left settings/CLAUDE.md unconfigured. Each step records its name on failure
+  // so `run()` can warn instead of reporting a clean success over a partial install.
+  const step = (name, fn) => { try { fn(); } catch { out.failed.push(name); } };
 
   // 1) Playbooks — always refreshed (shared system).
-  const srcPlaybooks = path.join(src, '.claudeflow', 'playbooks');
-  if (fs.existsSync(srcPlaybooks)) {
-    const dst = path.join(cwd, '.claudeflow', 'playbooks');
-    rmrf(dst); copyDir(srcPlaybooks, dst);
-    out.playbooks = countFiles(dst);
-  }
+  step('playbooks', () => {
+    const srcPlaybooks = path.join(src, '.claudeflow', 'playbooks');
+    if (fs.existsSync(srcPlaybooks)) {
+      const dst = path.join(cwd, '.claudeflow', 'playbooks');
+      rmrf(dst); copyDir(srcPlaybooks, dst);
+      out.playbooks = countFiles(dst);
+    }
+  });
 
   // 2) Configs — per-project ones create-if-absent (never clobber a customized brand); shared refresh.
-  fs.mkdirSync(path.join(cwd, '.claudeflow'), { recursive: true });
-  for (const c of PER_PROJECT_CONFIGS) {
-    const s = path.join(src, '.claudeflow', c), d = path.join(cwd, '.claudeflow', c);
-    if (fs.existsSync(s) && !fs.existsSync(d)) fs.copyFileSync(s, d);
-  }
-  for (const c of SHARED_CONFIGS) {
-    const s = path.join(src, '.claudeflow', c);
-    if (fs.existsSync(s)) fs.copyFileSync(s, path.join(cwd, '.claudeflow', c));
-  }
-
-  // 3) Scripts (hooks + scans + servers) — always refreshed.
-  const srcScripts = path.join(src, 'scripts', 'claudeflow');
-  if (fs.existsSync(srcScripts)) {
-    const dst = path.join(cwd, 'scripts', 'claudeflow');
-    fs.mkdirSync(dst, { recursive: true });
-    for (const e of fs.readdirSync(srcScripts)) { fs.copyFileSync(path.join(srcScripts, e), path.join(dst, e)); out.scripts++; }
-  }
+  step('configs', () => {
+    fs.mkdirSync(path.join(cwd, '.claudeflow'), { recursive: true });
+    for (const c of PER_PROJECT_CONFIGS) {
+      const s = path.join(src, '.claudeflow', c), d = path.join(cwd, '.claudeflow', c);
+      if (fs.existsSync(s) && !fs.existsSync(d)) fs.copyFileSync(s, d);
+    }
+    for (const c of SHARED_CONFIGS) {
+      const s = path.join(src, '.claudeflow', c);
+      if (fs.existsSync(s)) fs.copyFileSync(s, path.join(cwd, '.claudeflow', c));
+    }
+  });
+
+  // 3) Scripts (hooks + scans + servers) — always refreshed. Per-file isolation so ONE bad file (a socket, a
+  //    permission denial) can't abort the whole copy — let alone the later settings/CLAUDE.md steps.
+  step('scripts', () => {
+    const srcScripts = path.join(src, 'scripts', 'claudeflow');
+    if (fs.existsSync(srcScripts)) {
+      const dst = path.join(cwd, 'scripts', 'claudeflow');
+      fs.mkdirSync(dst, { recursive: true });
+      for (const e of fs.readdirSync(srcScripts)) {
+        try { fs.copyFileSync(path.join(srcScripts, e), path.join(dst, e)); out.scripts++; } catch {}
+      }
+    }
+  });
 
   // 4) Skills — replace each shipped claudeflow-* skill; preserve the user's own skills.
-  const srcSkills = path.join(src, '.claude', 'skills');
-  if (fs.existsSync(srcSkills)) {
-    const dst = path.join(cwd, '.claude', 'skills');
-    fs.mkdirSync(dst, { recursive: true });
-    for (const e of fs.readdirSync(srcSkills, { withFileTypes: true })) {
-      if (!e.isDirectory() || !e.name.startsWith('claudeflow-')) continue;
-      const d = path.join(dst, e.name);
-      rmrf(d); copyDir(path.join(srcSkills, e.name), d); out.skills++;
+  step('skills', () => {
+    const srcSkills = path.join(src, '.claude', 'skills');
+    if (fs.existsSync(srcSkills)) {
+      const dst = path.join(cwd, '.claude', 'skills');
+      fs.mkdirSync(dst, { recursive: true });
+      for (const e of fs.readdirSync(srcSkills, { withFileTypes: true })) {
+        if (!e.isDirectory() || !e.name.startsWith('claudeflow-')) continue;
+        const d = path.join(dst, e.name);
+        try { rmrf(d); copyDir(path.join(srcSkills, e.name), d); out.skills++; } catch {}
+      }
     }
-  }
+  });
 
   // 4b) Agents — replace each shipped claudeflow-* agent; preserve the user's own agents.
-  const srcAgents = path.join(src, '.claude', 'agents');
-  if (fs.existsSync(srcAgents)) {
-    const dst = path.join(cwd, '.claude', 'agents');
-    fs.mkdirSync(dst, { recursive: true });
-    for (const e of fs.readdirSync(srcAgents, { withFileTypes: true })) {
-      if (!e.isFile() || !e.name.startsWith('claudeflow-') || !e.name.endsWith('.md')) continue;
-      fs.copyFileSync(path.join(srcAgents, e.name), path.join(dst, e.name)); out.agents++;
+  step('agents', () => {
+    const srcAgents = path.join(src, '.claude', 'agents');
+    if (fs.existsSync(srcAgents)) {
+      const dst = path.join(cwd, '.claude', 'agents');
+      fs.mkdirSync(dst, { recursive: true });
+      for (const e of fs.readdirSync(srcAgents, { withFileTypes: true })) {
+        if (!e.isFile() || !e.name.startsWith('claudeflow-') || !e.name.endsWith('.md')) continue;
+        try { fs.copyFileSync(path.join(srcAgents, e.name), path.join(dst, e.name)); out.agents++; } catch {}
+      }
     }
-  }
-
-  // 5) settings.json hooks — MERGE (never clobber the user's settings).
-  const srcSettings = path.join(src, '.claude', 'settings.json');
-  if (fs.existsSync(srcSettings)) {
-    try { mergeSettings(cwd, JSON.parse(fs.readFileSync(srcSettings, 'utf8'))); } catch {}
-  }
+  });
+
+  // 5) settings.json — MERGE the shipped hooks + env (add-if-absent) into the user's settings (never clobber).
+  //    ALWAYS runs (isolated above) — this is what wires CLAUDE_CODE_EXPERIMENTAL_AGENT_TEAMS et al.
+  step('settings', () => {
+    const srcSettings = path.join(src, '.claude', 'settings.json');
+    if (fs.existsSync(srcSettings)) {
+      mergeSettings(cwd, JSON.parse(fs.readFileSync(srcSettings, 'utf8')));
+    }
+  });
 
   // 6) CLAUDE.md — upsert the managed guidance block between markers (idempotent; updates propagate on re-install).
-  out.claudeMd = syncClaudeMd(cwd, path.join(src, 'assets', 'CLAUDE.snippet.md'));
+  step('claude-md', () => {
+    out.claudeMd = syncClaudeMd(cwd, path.join(src, 'assets', 'CLAUDE.snippet.md'));
+  });
 
   return out;
 }
@@ -200,6 +224,9 @@ async function run() {
   ui.success(`claudeflow v${version} installed`);
   ui.info(`  ${summary.playbooks} playbooks · ${summary.scripts} scripts · ${summary.skills} skills`
     + (summary.claudeMd ? ' · CLAUDE.md guidance synced' : ''));
+  if (summary.failed && summary.failed.length) {
+    ui.warn(`  some steps did not complete: ${summary.failed.join(', ')}. Re-run \`npx @axiomatic-labs/claudeflow@latest install\` (the @latest avoids a stale npx cache).`);
+  }
   ui.done(`Describe a change and the ${ui.BOLD}claudeflow-implementer${ui.RESET} agent will build it.`);
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@axiomatic-labs/claudeflow",
-  "version": "2.43.30",
+  "version": "2.43.32",
   "description": "Claudeflow — AI-powered development toolkit for Claude Code. Skills, agents, hooks, and quality gates that ship production apps.",
   "bin": {
     "claudeflow": "./bin/cli.js"