coding-friend-cli 1.27.1 → 1.28.0

package/README.md

@@ -147,19 +147,25 @@ npm ls -g coding-friend-cli
 
 ### Running tests
 
-Tests are written with [Vitest](https://vitest.dev/) and live in `src/lib/__tests__/`.
+Tests are written with [Vitest](https://vitest.dev/).
 
 ```bash
 cd cli
 
-# Run all tests once
+# Run all unit tests once
 npm test
 
 # Watch mode (re-runs on file changes)
 npm run test:watch
+
+# Run E2E tests (real CLI commands against temp directories)
+npm run test:e2e
+
+# Run E2E tests in Docker (isolated, no host pollution)
+cd e2e && docker compose up --build --abort-on-container-exit
 ```
 
-Current coverage: `lib/json.ts`, `lib/paths.ts`, `lib/exec.ts`.
+Unit tests live in `src/**/__tests__/` and use mocks. E2E tests live in `e2e/` and run actual CLI commands — see [`e2e/README.md`](e2e/README.md) for details.
 
 ## Publish CLI to npm
 

package/dist/{chunk-PRIH34UB.js → chunk-DHH6SRXV.js}

@@ -6,12 +6,16 @@ var STATUSLINE_COMPONENTS = [
   { id: "branch", label: "Git branch (\u2387 main)" },
   {
     id: "account",
-    label: "Account info \u2014 name, email & org (from ~/.claude.json)"
+    label: "Account info \u2014 name, email (from ~/.claude.json)"
   },
   { id: "context", label: "Context window usage (ctx 42%)" },
   {
     id: "rate_limit",
     label: "Rate limit \u2014 current & weekly usage (requires curl & jq)"
+  },
+  {
+    id: "task_agent",
+    label: "Task progress & active agent (Tasks: 2/5 | Agent: cf-reviewer)"
   }
 ];
 var ALL_COMPONENT_IDS = STATUSLINE_COMPONENTS.map((c) => c.id);

package/dist/{chunk-75IEE2OE.js → chunk-E7OY4UWW.js}

@@ -7,14 +7,14 @@ import {
   getMemoryMcpStatus,
   memoryConfigMenu,
   writeMemoryMcpEntry
-} from "./chunk-BX3DRKTU.js";
+} from "./chunk-XROT7L2F.js";
 import {
   getLibPath
 } from "./chunk-RZRT7NGT.js";
 import {
   loadConfig,
   resolveMemoryDir
-} from "./chunk-2HQVNQB7.js";
+} from "./chunk-J4N2ODQ5.js";
 import {
   showConfigHint
 } from "./chunk-HPNRQYLM.js";

package/dist/chunk-J4N2ODQ5.js

@@ -0,0 +1,191 @@
+import {
+  DEFAULT_CONFIG
+} from "./chunk-DHH6SRXV.js";
+import {
+  globalConfigPath,
+  localConfigPath,
+  resolvePath
+} from "./chunk-TWKNGPBO.js";
+import {
+  log
+} from "./chunk-NREZK463.js";
+import {
+  readJson
+} from "./chunk-5UVDWG5L.js";
+
+// src/lib/config.ts
+import { z } from "zod";
+function deepMerge(base, override) {
+  const result = { ...base };
+  for (const key of Object.keys(override)) {
+    const baseVal = result[key];
+    const overVal = override[key];
+    if (baseVal && overVal && typeof baseVal === "object" && typeof overVal === "object" && !Array.isArray(baseVal) && !Array.isArray(overVal)) {
+      result[key] = deepMerge(
+        baseVal,
+        overVal
+      );
+    } else {
+      result[key] = overVal;
+    }
+  }
+  return result;
+}
+var LearnCategorySchema = z.object({
+  name: z.string(),
+  description: z.string()
+});
+var LearnConfigSchema = z.object({
+  language: z.string().optional(),
+  outputDir: z.string().optional(),
+  categories: z.array(LearnCategorySchema).optional(),
+  autoCommit: z.boolean().optional(),
+  readmeIndex: z.union([z.boolean(), z.literal("per-category")]).optional()
+});
+var StatuslineComponentSchema = z.enum([
+  "version",
+  "folder",
+  "model",
+  "branch",
+  "account",
+  "context",
+  "rate_limit",
+  "task_agent"
+]);
+var StatuslineConfigSchema = z.object({
+  components: z.array(StatuslineComponentSchema).optional()
+});
+var MemoryConfigSchema = z.object({
+  tier: z.enum(["auto", "full", "lite", "markdown"]).optional(),
+  daemon: z.object({
+    idleTimeout: z.number().optional()
+  }).optional(),
+  embedding: z.object({
+    provider: z.enum(["transformers", "ollama"]).optional(),
+    model: z.string().optional(),
+    ollamaUrl: z.string().optional()
+  }).optional(),
+  autoCapture: z.boolean().optional(),
+  autoStart: z.boolean().optional()
+});
+var ConfigSchema = z.strictObject({
+  language: z.string().optional(),
+  docsDir: z.string().optional(),
+  learn: LearnConfigSchema.optional(),
+  statusline: StatuslineConfigSchema.optional(),
+  memory: MemoryConfigSchema.optional(),
+  autoApprove: z.boolean().optional()
+});
+var KNOWN_KEYS = [
+  "language",
+  "docsDir",
+  "learn",
+  "statusline",
+  "memory",
+  "autoApprove"
+];
+function suggestKey(unknown) {
+  let best = null;
+  let bestDist = Infinity;
+  for (const known of KNOWN_KEYS) {
+    const dist = levenshtein(unknown.toLowerCase(), known.toLowerCase());
+    if (dist < bestDist && dist <= 3) {
+      bestDist = dist;
+      best = known;
+    }
+  }
+  return best;
+}
+function levenshtein(a, b) {
+  const m = a.length;
+  const n = b.length;
+  const dp = Array.from(
+    { length: m + 1 },
+    () => Array(n + 1).fill(0)
+  );
+  for (let i = 0; i <= m; i++) dp[i][0] = i;
+  for (let j = 0; j <= n; j++) dp[0][j] = j;
+  for (let i = 1; i <= m; i++) {
+    for (let j = 1; j <= n; j++) {
+      dp[i][j] = a[i - 1] === b[j - 1] ? dp[i - 1][j - 1] : 1 + Math.min(dp[i - 1][j], dp[i][j - 1], dp[i - 1][j - 1]);
+    }
+  }
+  return dp[m][n];
+}
+function validateConfig(merged) {
+  const result = ConfigSchema.safeParse(merged);
+  if (result.success) return result.data;
+  const sanitized = { ...merged };
+  for (const issue of result.error.issues) {
+    const pathStr = issue.path.join(".");
+    if (issue.code === "unrecognized_keys") {
+      const keys = issue.keys ?? [];
+      for (const key of keys) {
+        const suggestion = suggestKey(key);
+        const hint = suggestion ? ` Did you mean '${suggestion}'?` : "";
+        log.warn(`Unknown config key '${key}'.${hint}`);
+        delete sanitized[key];
+      }
+    } else {
+      log.warn(`Config: ${pathStr} \u2014 ${issue.message}. Using default.`);
+      stripPath(sanitized, issue.path);
+    }
+  }
+  return deepMerge(
+    DEFAULT_CONFIG,
+    sanitized
+  );
+}
+function stripPath(obj, path) {
+  if (path.length === 0) return;
+  let current = obj;
+  for (let i = 0; i < path.length - 1; i++) {
+    const key = String(path[i]);
+    if (current[key] && typeof current[key] === "object") {
+      current = current[key];
+    } else {
+      return;
+    }
+  }
+  delete current[String(path[path.length - 1])];
+}
+function loadConfig() {
+  const global = readJson(globalConfigPath());
+  const local = readJson(localConfigPath());
+  const base = deepMerge(
+    DEFAULT_CONFIG,
+    global ?? {}
+  );
+  const merged = deepMerge(base, local ?? {});
+  return validateConfig(merged);
+}
+function resolveDocsDir(explicitPath) {
+  if (explicitPath) {
+    return resolvePath(explicitPath);
+  }
+  const local = readJson(localConfigPath());
+  if (local?.learn?.outputDir) {
+    return resolvePath(local.learn.outputDir);
+  }
+  const global = readJson(globalConfigPath());
+  if (global?.learn?.outputDir) {
+    return resolvePath(global.learn.outputDir);
+  }
+  return resolvePath("docs/learn");
+}
+function resolveMemoryDir(explicitPath) {
+  if (explicitPath) {
+    return resolvePath(explicitPath);
+  }
+  const config = loadConfig();
+  if (config.docsDir) {
+    return resolvePath(`${config.docsDir}/memory`);
+  }
+  return resolvePath("docs/memory");
+}
+
+export {
+  loadConfig,
+  resolveDocsDir,
+  resolveMemoryDir
+};

package/dist/{chunk-YCWRBOB3.js → chunk-RGQGKOBH.js}

@@ -1,7 +1,7 @@
 import {
   ensureStatusline,
   getInstalledVersion
-} from "./chunk-GDZTU2Z4.js";
+} from "./chunk-XQ5KAXKL.js";
 import {
   ensureShellCompletion
 } from "./chunk-DMJCO6LJ.js";

package/dist/{chunk-GDZTU2Z4.js → chunk-XQ5KAXKL.js}

@@ -1,7 +1,7 @@
 import {
   ALL_COMPONENT_IDS,
   STATUSLINE_COMPONENTS
-} from "./chunk-PRIH34UB.js";
+} from "./chunk-DHH6SRXV.js";
 import {
   claudeSettingsPath,
   globalConfigPath,

package/dist/{chunk-BX3DRKTU.js → chunk-XROT7L2F.js}

@@ -3,7 +3,7 @@ import {
 } from "./chunk-RZRT7NGT.js";
 import {
   resolveMemoryDir
-} from "./chunk-2HQVNQB7.js";
+} from "./chunk-J4N2ODQ5.js";
 import {
   BACK,
   askScope,
@@ -222,16 +222,17 @@ async function editMemoryEmbedding(globalCfg, localCfg) {
 async function editMemoryDaemonTimeout(globalCfg, localCfg) {
   const currentDaemon = getMergedMemoryValue("daemon", globalCfg, localCfg);
   const currentMs = currentDaemon?.idleTimeout;
-  const currentMin = currentMs ? currentMs / 6e4 : void 0;
+  const currentMin = currentMs !== void 0 ? currentMs / 6e4 : void 0;
   if (currentMin !== void 0) {
     log.dim(`Current: ${currentMin} minutes`);
   }
   const value = await input({
-    message: "Daemon idle timeout (minutes):",
-    default: String(currentMin ?? 30),
+    message: "Daemon idle timeout (minutes, 0 = no timeout):",
+    default: String(currentMin ?? 0),
     validate: (val) => {
       const n = Number(val);
-      if (isNaN(n) || n < 1) return "Must be a positive number";
+      if (isNaN(n) || n < 0)
+        return "Must be 0 (no timeout) or a positive number";
       return true;
     }
   });

package/dist/{config-N55LTJ76.js → config-U2WEZXF6.js}

@@ -1,19 +1,19 @@
 import {
   memoryConfigMenu
-} from "./chunk-BX3DRKTU.js";
+} from "./chunk-XROT7L2F.js";
 import "./chunk-RZRT7NGT.js";
-import "./chunk-2HQVNQB7.js";
+import "./chunk-J4N2ODQ5.js";
 import {
   findStatuslineHookPath,
   isStatuslineConfigured,
   saveStatuslineConfig,
   selectStatuslineComponents,
   writeStatuslineSettings
-} from "./chunk-GDZTU2Z4.js";
+} from "./chunk-XQ5KAXKL.js";
 import {
   ALL_COMPONENT_IDS,
   DEFAULT_CONFIG
-} from "./chunk-PRIH34UB.js";
+} from "./chunk-DHH6SRXV.js";
 import {
   ensureShellCompletion,
   hasShellCompletion,

package/dist/{dev-BSXR55PZ.js → dev-4OAJCQRQ.js}

@@ -1,7 +1,7 @@
 import {
   ensureStatusline
-} from "./chunk-GDZTU2Z4.js";
-import "./chunk-PRIH34UB.js";
+} from "./chunk-XQ5KAXKL.js";
+import "./chunk-DHH6SRXV.js";
 import {
   ensureShellCompletion
 } from "./chunk-DMJCO6LJ.js";
@@ -32,6 +32,7 @@ import {
   existsSync,
   unlinkSync,
   readdirSync,
+  readFileSync,
   statSync,
   mkdirSync,
   copyFileSync
@@ -255,11 +256,29 @@ async function devSyncCommand() {
   }
   const shortDest = cacheVersionDir.replace(homedir(), "~");
   log.step(`Syncing ${chalk.cyan(pluginSrcDir)} \u2192 ${chalk.dim(shortDest)}`);
+  const cachedHooksPath = join(cacheVersionDir, "hooks", "hooks.json");
+  const sourceHooksPath = join(pluginSrcDir, "hooks", "hooks.json");
+  let hooksEventsChanged = false;
+  try {
+    const cachedHooks = JSON.parse(readFileSync(cachedHooksPath, "utf-8"));
+    const sourceHooks = JSON.parse(readFileSync(sourceHooksPath, "utf-8"));
+    const cachedEvents = Object.keys(cachedHooks.hooks || {}).sort();
+    const sourceEvents = Object.keys(sourceHooks.hooks || {}).sort();
+    hooksEventsChanged = JSON.stringify(cachedEvents) !== JSON.stringify(sourceEvents);
+  } catch {
+  }
   const fileCount = { n: 0 };
   copyDirRecursive(pluginSrcDir, cacheVersionDir, fileCount);
-  log.success(
-    `Synced ${chalk.green(fileCount.n)} files. Restart Claude Code to apply changes.`
-  );
+  if (hooksEventsChanged) {
+    log.success(`Synced ${chalk.green(fileCount.n)} files.`);
+    log.warn(
+      `hooks.json event types changed. Run ${chalk.bold("cf dev restart")} for Claude Code to register the new hook events.`
+    );
+  } else {
+    log.success(
+      `Synced ${chalk.green(fileCount.n)} files. Restart Claude Code to apply changes.`
+    );
+  }
 }
 async function devReinstall(path, label) {
   const state = getDevState();
@@ -288,6 +307,11 @@ async function devStatusCommand() {
   if (state) {
     log.info(`Dev mode: ${chalk.green("ON")}`);
     log.info(`Local path: ${chalk.cyan(state.localPath)}`);
+    if (!existsSync(state.localPath)) {
+      log.warn(
+        `Dev mode path no longer exists: ${state.localPath}. Run 'cf host <new-path>' to update.`
+      );
+    }
     log.dim(`Since: ${state.savedAt}`);
   } else {
     log.info(`Dev mode: ${chalk.yellow("OFF")}`);

package/dist/guide-F2FLCDYO.js

@@ -0,0 +1,113 @@
+import {
+  devStatePath,
+  pluginCachePath
+} from "./chunk-TWKNGPBO.js";
+import {
+  log
+} from "./chunk-NREZK463.js";
+import {
+  readJson
+} from "./chunk-5UVDWG5L.js";
+
+// src/commands/guide.ts
+import { existsSync, mkdirSync, readdirSync, writeFileSync } from "fs";
+import { join, resolve } from "path";
+var GUIDE_TEMPLATE = `# Custom Guide for {{SKILL_NAME}}
+
+<!-- Sections are optional \u2014 include only the ones you need. -->
+<!-- This file is loaded automatically when the skill runs. -->
+
+## Before
+
+<!-- Steps to execute BEFORE the skill's first step. -->
+<!-- Example: run a linter, check prerequisites, set up context. -->
+
+## Rules
+
+<!-- Additional rules to apply THROUGHOUT all steps of the skill. -->
+<!-- Example: "always use TypeScript strict mode", "never modify tests in src/__tests__". -->
+
+## After
+
+<!-- Steps to execute AFTER the skill's final step. -->
+<!-- Example: run a formatter, update a changelog, notify a channel. -->
+`;
+function findPluginSkillsDir() {
+  const devState = readJson(devStatePath());
+  if (devState?.localPath) {
+    const dir = join(devState.localPath, "plugin", "skills");
+    if (existsSync(dir)) return dir;
+  }
+  const cachePath = pluginCachePath();
+  if (existsSync(cachePath)) {
+    try {
+      const versions = readdirSync(cachePath, { withFileTypes: true }).filter((e) => e.isDirectory() && /^\d/.test(e.name)).map((e) => e.name).sort((a, b) => b.localeCompare(a, void 0, { numeric: true }));
+      for (const ver of versions) {
+        const dir = join(cachePath, ver, "plugin", "skills");
+        if (existsSync(dir)) return dir;
+      }
+    } catch {
+    }
+  }
+  return null;
+}
+function skillExists(skillsDir, skillName) {
+  return existsSync(join(skillsDir, skillName, "SKILL.md"));
+}
+function guideCreateCommand(skillName) {
+  if (!/^[a-z0-9-]+$/.test(skillName)) {
+    log.error(
+      `Invalid skill name "${skillName}". Use lowercase letters, numbers, and hyphens only.`
+    );
+    return;
+  }
+  const skillsDir = findPluginSkillsDir();
+  if (!skillsDir || !skillExists(skillsDir, skillName)) {
+    log.error(
+      `Skill "${skillName}" not found. Run "cf guide list" to see available skills.`
+    );
+    return;
+  }
+  const guideDir = resolve(".coding-friend", "skills", `${skillName}-custom`);
+  const guidePath = join(guideDir, "SKILL.md");
+  if (existsSync(guidePath)) {
+    log.warn(`Custom guide already exists: ${guidePath}`);
+    return;
+  }
+  mkdirSync(guideDir, { recursive: true });
+  writeFileSync(
+    guidePath,
+    GUIDE_TEMPLATE.replace(/\{\{SKILL_NAME\}\}/g, skillName)
+  );
+  log.success(`Created custom guide: ${guidePath}`);
+  log.dim("Edit the file to add your Before/Rules/After sections.");
+}
+function guideListCommand() {
+  const customDir = resolve(".coding-friend", "skills");
+  if (!existsSync(customDir)) {
+    log.dim(
+      "No custom guides found. Run `cf guide create <skill-name>` to create one."
+    );
+    return;
+  }
+  const entries = readdirSync(customDir, { withFileTypes: true }).filter(
+    (e) => e.isDirectory() && e.name.endsWith("-custom") && existsSync(join(customDir, e.name, "SKILL.md"))
+  ).map((e) => e.name.replace(/-custom$/, ""));
+  if (entries.length === 0) {
+    log.dim(
+      "No custom guides found. Run `cf guide create <skill-name>` to create one."
+    );
+    return;
+  }
+  const skillsDir = findPluginSkillsDir();
+  log.info(`Custom guides (${entries.length}):`);
+  for (const name of entries) {
+    const path = join(customDir, `${name}-custom`, "SKILL.md");
+    const exists = skillsDir != null && skillExists(skillsDir, name) ? "\u2714" : "\u26A0 skill not found";
+    log.info(`  ${name}  \u2192  ${path}  (${exists})`);
+  }
+}
+export {
+  guideCreateCommand,
+  guideListCommand
+};

package/dist/{host-JVLWYY5A.js → host-2KBOPER6.js}

@@ -3,8 +3,8 @@ import {
 } from "./chunk-RZRT7NGT.js";
 import {
   resolveDocsDir
-} from "./chunk-2HQVNQB7.js";
-import "./chunk-PRIH34UB.js";
+} from "./chunk-J4N2ODQ5.js";
+import "./chunk-DHH6SRXV.js";
 import {
   run,
   streamExec