coding-friend-cli 1.27.1 → 1.28.0

package/lib/cf-memory/src/lib/claude-md.ts

@@ -14,7 +14,10 @@
 import fs from "node:fs";
 import path from "node:path";
 
-export const SECTION_HEADER = "## CF Memory: Conventions";
+export const SECTION_HEADER = "## CF Memory: Project Rules";
+
+/** @deprecated Old header — kept for migration. Removed after reading. */
+const LEGACY_HEADER = "## CF Memory: Conventions";
 
 // Tested per-line via .split("\n"), not against the full document,
 // so the $ anchor correctly matches end-of-line without the `m` flag.
@@ -52,13 +55,19 @@ function parseClaudeMd(content: string): {
   after: string;
   untrackedLines: string[];
 } {
-  const headerIdx = content.indexOf(SECTION_HEADER);
+  // Try current header first, then legacy header for migration
+  let headerIdx = content.indexOf(SECTION_HEADER);
+  let headerLen = SECTION_HEADER.length;
+  if (headerIdx === -1) {
+    headerIdx = content.indexOf(LEGACY_HEADER);
+    headerLen = LEGACY_HEADER.length;
+  }
   if (headerIdx === -1) {
     return { before: content, entries: [], after: "", untrackedLines: [] };
   }
 
   const before = content.slice(0, headerIdx);
-  const rest = content.slice(headerIdx + SECTION_HEADER.length);
+  const rest = content.slice(headerIdx + headerLen);
 
   // Find the next section header (## ) to determine where our section ends
   const nextSectionMatch = rest.match(/\n(## )/);

package/lib/cf-memory/src/lib/tier.ts

@@ -50,17 +50,29 @@ export async function detectTier(configTier?: TierConfig): Promise<TierInfo> {
   return TIERS.markdown;
 }
 
-/** Build a respawn callback that DaemonClient can call when the daemon is gone. */
-function makeRespawn(
+const RESPAWN_MAX_RETRIES = 3;
+const RESPAWN_RETRY_DELAY_MS = 1000;
+
+/** Build a respawn callback with retry logic (up to 3 attempts). */
+export function createRespawnWithRetry(
   docsDir: string,
   embeddingConfig?: Partial<EmbeddingConfig>,
   idleTimeoutMs?: number,
 ): () => Promise<boolean> {
   return async () => {
-    const result = await spawnDaemon(docsDir, embeddingConfig, {
-      idleTimeoutMs,
-    });
-    return result !== null;
+    for (let attempt = 1; attempt <= RESPAWN_MAX_RETRIES; attempt++) {
+      // spawnDaemon returns null both when "already running" and "truly failed"
+      // Check isDaemonRunning after null to distinguish the two cases
+      const result = await spawnDaemon(docsDir, embeddingConfig, {
+        idleTimeoutMs,
+      });
+      if (result !== null) return true;
+      if (await isDaemonRunning()) return true;
+      if (attempt < RESPAWN_MAX_RETRIES) {
+        await new Promise((r) => setTimeout(r, RESPAWN_RETRY_DELAY_MS));
+      }
+    }
+    return false;
   };
 }
 
@@ -75,7 +87,7 @@ export async function createBackendForTier(
   daemonOptions?: { idleTimeoutMs?: number },
 ): Promise<{ backend: MemoryBackend; tier: TierInfo }> {
   const tier = await detectTier(configTier);
-  const respawn = makeRespawn(
+  const respawn = createRespawnWithRetry(
     docsDir,
     embeddingConfig,
     daemonOptions?.idleTimeoutMs,

package/lib/cf-memory/src/tools/delete.ts

@@ -28,12 +28,12 @@ export function registerDelete(
           isError: true,
         };
       }
-      // Remove from CLAUDE.md if it was a conventions entry
-      if (id.startsWith("conventions/") && ctx?.docsDir) {
+      // Remove from CLAUDE.md (any category may have been synced via sync_to_claude_md)
+      if (ctx?.docsDir) {
         try {
           removeFromClaudeMd(ctx.docsDir, id);
         } catch {
-          // Best-effort
+          // Best-effort — removeFromClaudeMd is idempotent
         }
       }
 

package/lib/cf-memory/src/tools/store.ts

@@ -40,6 +40,14 @@ export function registerStore(
         .describe(
           "When true, skip file creation and return a Memory object for indexing. File must already exist on disk.",
         ),
+      sync_to_claude_md: z
+        .boolean()
+        .optional()
+        .describe(
+          "When true, sync this memory to the project's CLAUDE.md regardless of category. " +
+            "Use for memories containing project-wide rules, conventions, or decisions that should be visible in CLAUDE.md. " +
+            "Convention memories (type: preference) are always synced automatically.",
+        ),
     },
     async ({
       title,
@@ -50,6 +58,7 @@ export function registerStore(
       importance,
       source,
       index_only,
+      sync_to_claude_md,
     }) => {
       const input = {
         title,
@@ -73,9 +82,12 @@ export function registerStore(
 
       const memory = await backend.store(input);
 
-      // Sync convention memories to CLAUDE.md
+      // Sync to CLAUDE.md: auto for conventions, opt-in for other categories
       let claudeMdUpdated = false;
-      if (memory.category === "conventions" && ctx?.docsDir && !index_only) {
+      const shouldSync =
+        (memory.category === "conventions" || sync_to_claude_md) &&
+        ctx?.docsDir;
+      if (shouldSync) {
         try {
           syncToClaudeMd(ctx.docsDir, memory.id, description);
           claudeMdUpdated = true;

package/lib/cf-memory/src/tools/update.ts

@@ -29,8 +29,23 @@ export function registerUpdate(
         .max(5)
         .optional()
         .describe("New importance"),
+      sync_to_claude_md: z
+        .boolean()
+        .optional()
+        .describe(
+          "When true, sync this memory to the project's CLAUDE.md regardless of category. " +
+            "Convention memories (type: preference) are always synced automatically.",
+        ),
     },
-    async ({ id, title, description, tags, content, importance }) => {
+    async ({
+      id,
+      title,
+      description,
+      tags,
+      content,
+      importance,
+      sync_to_claude_md,
+    }) => {
       const memory = await backend.update({
         id,
         title,
@@ -51,9 +66,12 @@ export function registerUpdate(
         };
       }
 
-      // Sync convention memories to CLAUDE.md
+      // Sync to CLAUDE.md: auto for conventions, opt-in for other categories
       let claudeMdUpdated = false;
-      if (memory.category === "conventions" && ctx?.docsDir) {
+      const shouldSync =
+        (memory.category === "conventions" || sync_to_claude_md) &&
+        ctx?.docsDir;
+      if (shouldSync) {
         try {
           if (description) {
             // Description changed — update the CLAUDE.md entry

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "coding-friend-cli",
-  "version": "1.27.1",
+  "version": "1.28.0",
   "description": "CLI for coding-friend — host learning docs, setup MCP server, initialize projects",
   "type": "module",
   "bin": {
@@ -12,8 +12,10 @@
     "prepublishOnly": "npm run build",
     "dev": "tsx src/index.ts",
     "watch": "tsup src/index.ts src/postinstall.ts --format esm --dts --watch",
-    "test": "vitest run",
-    "test:watch": "vitest"
+    "test": "vitest run && vitest run --config vitest.hooks.config.ts",
+    "test:watch": "vitest",
+    "test:hooks": "vitest run --config vitest.hooks.config.ts",
+    "test:e2e": "vitest run --config vitest.e2e.config.ts"
   },
   "files": [
     "dist",
@@ -43,7 +45,8 @@
   "dependencies": {
     "@inquirer/prompts": "^7.0.0",
     "chalk": "^5.0.0",
-    "commander": "^13.0.0"
+    "commander": "^13.0.0",
+    "zod": "^4.3.6"
   },
   "devDependencies": {
     "@types/node": "^22.0.0",

package/dist/chunk-2HQVNQB7.js

@@ -1,71 +0,0 @@
-import {
-  DEFAULT_CONFIG
-} from "./chunk-PRIH34UB.js";
-import {
-  globalConfigPath,
-  localConfigPath,
-  resolvePath
-} from "./chunk-TWKNGPBO.js";
-import {
-  readJson
-} from "./chunk-5UVDWG5L.js";
-
-// src/lib/config.ts
-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;
-}
-function loadConfig() {
-  const global = readJson(globalConfigPath());
-  const local = readJson(localConfigPath());
-  const base = deepMerge(
-    DEFAULT_CONFIG,
-    global ?? {}
-  );
-  return deepMerge(
-    base,
-    local ?? {}
-  );
-}
-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
-};