coding-friend-cli 1.17.4 → 1.18.0

package/dist/{init-73ECEDU7.js → init-2HLPKYXB.js}

@@ -1,9 +1,12 @@
 import {
   applyPermissions,
   buildLearnDirRules,
+  cleanupStalePluginRules,
+  getAllRules,
   getExistingRules,
-  getMissingRules
-} from "./chunk-56U7US6J.js";
+  getMissingRules,
+  logPluginScriptWarning
+} from "./chunk-7CAIGH2Y.js";
 import {
   getLibPath
 } from "./chunk-RZRT7NGT.js";
@@ -20,7 +23,7 @@ import {
 import {
   ensureShellCompletion,
   hasShellCompletion
-} from "./chunk-DVMWMXDZ.js";
+} from "./chunk-VUAUAO2R.js";
 import {
   BACK,
   applyDocsDirChange,
@@ -37,6 +40,7 @@ import {
   run
 } from "./chunk-CYQU33FY.js";
 import {
+  claudeLocalSettingsPath,
   claudeSettingsPath,
   globalConfigPath,
   localConfigPath,
@@ -71,7 +75,9 @@ function printBanner() {
   console.log();
 }
 var _stepIndex = 0;
+var _suppressStepHeaders = false;
 function printStepHeader(label, description) {
+  if (_suppressStepHeaders) return;
   _stepIndex++;
   const line = chalk.hex("#10b981")("\u2500".repeat(44));
   console.log();
@@ -136,6 +142,14 @@ function printSetupStatus(globalCfg, localCfg, gitAvailable) {
     if (scope === "-") notConfiguredCount++;
     if (scope === "-" || scope === "global") notLocalCount++;
   }
+  const projectRules = getExistingRules(claudeLocalSettingsPath());
+  const userRules = getExistingRules(claudeSettingsPath());
+  const allRules = getAllRules();
+  const allRuleStrings = allRules.map((r) => r.rule);
+  const configuredRuleCount = (/* @__PURE__ */ new Set([
+    ...projectRules.filter((r) => allRuleStrings.includes(r)),
+    ...userRules.filter((r) => allRuleStrings.includes(r))
+  ])).size;
   const setupItems = [
     {
       label: ".gitignore",
@@ -144,7 +158,12 @@ function printSetupStatus(globalCfg, localCfg, gitAvailable) {
     },
     { label: "Shell completion", done: hasShellCompletion(), skipped: false },
     { label: "Statusline", done: isStatuslineConfigured(), skipped: false },
-    { label: "CF Memory MCP", done: isMemoryMcpConfigured(), skipped: false }
+    { label: "CF Memory MCP", done: isMemoryMcpConfigured(), skipped: false },
+    {
+      label: `Permissions (${configuredRuleCount}/${allRules.length} rules)`,
+      done: configuredRuleCount > 0,
+      skipped: false
+    }
   ];
   for (const item of setupItems) {
     if (item.skipped) {
@@ -639,33 +658,57 @@ async function stepMemory(docsDir) {
     `Tip: Run ${chalk.cyan("/cf-scan")} in Claude Code to populate memory with project knowledge.`
   );
 }
-async function stepClaudePermissions(outputDir, autoCommit) {
-  const resolved = resolvePath(outputDir);
-  const homePath = resolved.startsWith(homedir()) ? resolved.replace(homedir(), "~") : resolved;
-  const settingsPath = claudeSettingsPath();
-  const existing = getExistingRules(settingsPath);
-  if (existing.length === 0 && !readJson(settingsPath)) {
-    log.warn(
-      "~/.claude/settings.json not found. Create it via Claude Code settings first."
-    );
+async function stepClaudePermissions(externalLearnDir, autoCommit) {
+  const scope = await select({
+    message: "Where should permissions be saved?",
+    choices: [
+      {
+        name: "Project \u2014 .claude/settings.local.json (this project only, gitignored)",
+        value: "project"
+      },
+      {
+        name: "User \u2014 ~/.claude/settings.json (all projects)",
+        value: "user"
+      },
+      { name: "Skip", value: "skip" }
+    ]
+  });
+  if (scope === "skip") {
+    log.dim("Skipped. Run `cf permission` later to configure.");
     return;
   }
-  const learnRules = buildLearnDirRules(homePath, autoCommit);
-  const missing = getMissingRules(existing, learnRules);
+  const settingsPath = scope === "user" ? claudeSettingsPath() : claudeLocalSettingsPath();
+  const existing = getExistingRules(settingsPath);
+  const allRules = getAllRules();
+  const recommended = allRules.filter((r) => r.recommended);
+  let allToAdd = recommended;
+  if (externalLearnDir) {
+    const resolved = resolvePath(externalLearnDir);
+    const homePath = resolved.startsWith(homedir()) ? resolved.replace(homedir(), "~") : resolved;
+    const learnRules = buildLearnDirRules(homePath, autoCommit);
+    allToAdd = [...recommended, ...learnRules];
+  }
+  const missing = getMissingRules(existing, allToAdd);
   if (missing.length === 0) {
-    log.dim("All permission rules already configured.");
+    log.dim("All recommended permission rules already configured.");
     return;
   }
-  console.log("\nTo avoid repeated permission prompts, add these rules:");
+  log.step(`${missing.length} permission rules to add:`);
   for (const r of missing) {
-    console.log(`  ${r.rule}`);
+    console.log(`  ${chalk.green("+")} ${r.rule}`);
   }
+  const hasPluginRule = missing.some((r) => r.category === "Plugin Scripts");
+  if (hasPluginRule) {
+    console.log();
+    logPluginScriptWarning(log, chalk);
+  }
+  console.log();
   const ok = await confirm({
-    message: "Add these to ~/.claude/settings.json?",
+    message: `Add all ${missing.length} recommended rules?`,
     default: true
   });
   if (!ok) {
-    log.dim("Skipped. You'll get prompted each time.");
+    log.dim("Skipped. Run `cf permission` later to configure interactively.");
     return;
   }
   applyPermissions(
@@ -673,8 +716,125 @@ async function stepClaudePermissions(outputDir, autoCommit) {
     missing.map((r) => r.rule),
     []
   );
+  const cleaned = cleanupStalePluginRules(settingsPath);
+  if (cleaned > 0) {
+    log.dim(`Removed ${cleaned} stale old-format plugin rules.`);
+  }
   log.success(`Added ${missing.length} permission rules.`);
-  log.dim("For all plugin permissions, run: `cf permission`");
+  log.dim("Fine-tune later with: `cf permission` or `cf config` \u2192 Permissions");
+}
+async function initMenu(gitAvailable) {
+  while (true) {
+    const globalCfg = readJson(globalConfigPath());
+    const localCfg = readJson(localConfigPath());
+    const docsDir = getDocsDir(globalCfg, localCfg);
+    const docsDirScope = getScopeLabel("docsDir", globalCfg, localCfg);
+    const docsDirVal = getMergedValue("docsDir", globalCfg, localCfg);
+    const langScope = getScopeLabel("language", globalCfg, localCfg);
+    const langVal = getMergedValue("language", globalCfg, localCfg);
+    const learnScope = getScopeLabel("learn", globalCfg, localCfg);
+    const gitignoreStatus = !gitAvailable ? chalk.dim("skipped") : hasGitignoreBlock() ? chalk.green("configured") : chalk.yellow("not configured");
+    const completionStatus = hasShellCompletion() ? chalk.green("installed") : chalk.yellow("not installed");
+    const statuslineStatus = isStatuslineConfigured() ? chalk.green("configured") : chalk.yellow("not configured");
+    const memoryStatus = isMemoryMcpConfigured() ? chalk.green("configured") : chalk.yellow("not configured");
+    const projectRules = getExistingRules(claudeLocalSettingsPath());
+    const userRules = getExistingRules(claudeSettingsPath());
+    const allRules = getAllRules();
+    const allRuleStrings = allRules.map((r) => r.rule);
+    const configuredRuleCount = (/* @__PURE__ */ new Set([
+      ...projectRules.filter((r) => allRuleStrings.includes(r)),
+      ...userRules.filter((r) => allRuleStrings.includes(r))
+    ])).size;
+    const permissionStatus = configuredRuleCount > 0 ? chalk.green(`${configuredRuleCount}/${allRules.length}`) : chalk.yellow(`0/${allRules.length}`);
+    const choices = [
+      {
+        name: `Docs folder ${formatScopeLabel(docsDirScope)}${docsDirVal ? ` (${docsDirVal})` : ""}`,
+        value: "docsDir",
+        description: "  Where plans, memory, research, and session docs are stored"
+      },
+      {
+        name: `.gitignore (${gitignoreStatus})`,
+        value: "gitignore",
+        description: "  Add or update coding-friend artifacts in .gitignore",
+        ...gitAvailable ? {} : { disabled: "not in a git repo" }
+      },
+      {
+        name: `Docs language ${formatScopeLabel(langScope)}${langVal ? ` (${langVal})` : ""}`,
+        value: "language",
+        description: "  Language for /cf-plan, /cf-ask, /cf-remember generated docs"
+      },
+      {
+        name: `/cf-learn config ${formatScopeLabel(learnScope)}`,
+        value: "learn",
+        description: "  Output dir, language, categories, auto-commit, README index"
+      },
+      {
+        name: `Shell completion (${completionStatus})`,
+        value: "completion",
+        description: "  Tab-complete for cf commands in your shell"
+      },
+      {
+        name: `Statusline (${statuslineStatus})`,
+        value: "statusline",
+        description: "  Token count, model, and session info in your terminal prompt"
+      },
+      {
+        name: `CF Memory MCP (${memoryStatus})`,
+        value: "memory",
+        description: "  Connect the memory system to Claude Code via MCP"
+      },
+      {
+        name: `Permissions (${permissionStatus} rules)`,
+        value: "permissions",
+        description: "  Grant Coding Friend skills/hooks the permissions they need"
+      }
+    ];
+    const choice = await select({
+      message: "Which setting to configure?",
+      choices: injectBackChoice(choices, "Exit")
+    });
+    if (choice === BACK) {
+      log.dim("Done. Run `cf init` anytime to reconfigure.");
+      return;
+    }
+    _suppressStepHeaders = true;
+    switch (choice) {
+      case "docsDir":
+        await stepDocsDir(globalCfg, localCfg);
+        break;
+      case "gitignore":
+        await stepGitignore(docsDir);
+        break;
+      case "language":
+        await stepDocsLanguage(globalCfg, localCfg);
+        break;
+      case "learn":
+        await stepLearnConfig(globalCfg, localCfg, gitAvailable);
+        break;
+      case "completion":
+        await stepShellCompletion();
+        break;
+      case "statusline":
+        await stepStatusline();
+        break;
+      case "memory":
+        await stepMemory(docsDir);
+        break;
+      case "permissions": {
+        const learnCfg = localCfg?.learn ?? globalCfg?.learn;
+        const learnOutputDir = learnCfg?.outputDir || `${docsDir}/learn`;
+        const learnIsExternal = !learnOutputDir.startsWith(`${docsDir}/`);
+        const learnAutoCommit = learnCfg?.autoCommit || false;
+        await stepClaudePermissions(
+          learnIsExternal ? learnOutputDir : null,
+          learnAutoCommit
+        );
+        break;
+      }
+    }
+    _suppressStepHeaders = false;
+    console.log();
+  }
 }
 async function initCommand() {
   _stepIndex = 0;
@@ -685,11 +845,43 @@ async function initCommand() {
     log.warn("Not inside a git repo -- git-related steps will be skipped.");
     console.log();
   }
+  const alreadyInitialized = existsSync(join(process.cwd(), ".coding-friend"));
   const globalCfg = readJson(globalConfigPath());
   const localCfg = readJson(localConfigPath());
   console.log("Project status:");
   console.log();
   const { allDone, notLocalCount, notConfiguredCount, missingFolders } = printSetupStatus(globalCfg, localCfg, gitAvailable);
+  if (alreadyInitialized) {
+    if (allDone) {
+      if (notLocalCount > 0) {
+        console.log(
+          chalk.dim(
+            `  ${notLocalCount} setting(s) inherited from global config only.`
+          )
+        );
+        console.log();
+      }
+      log.success("All settings configured!");
+      console.log();
+    } else {
+      const parts = [];
+      if (missingFolders > 0) {
+        parts.push(`${missingFolders} folder(s) missing`);
+      }
+      if (notConfiguredCount > 0) {
+        parts.push(`${notConfiguredCount} setting(s) not configured`);
+      }
+      if (notLocalCount > 0) {
+        parts.push(`${notLocalCount} not set locally`);
+      }
+      if (parts.length > 0) {
+        console.log(`  ${chalk.yellow("\u26A0")} ${parts.join(" \xB7 ")}`);
+        console.log();
+      }
+    }
+    await initMenu(gitAvailable);
+    return;
+  }
   if (allDone) {
     if (notLocalCount > 0) {
       console.log(
@@ -755,20 +947,13 @@ async function initCommand() {
   await stepShellCompletion();
   await stepStatusline();
   await stepMemory(docsDir);
-  if (isExternal) {
-    printStepHeader(
-      "Configure Claude permissions",
-      "Grants Claude read/write access to your external learn folder without repeated prompts."
-    );
-    await stepClaudePermissions(outputDir, autoCommit);
-  } else {
-    printStepHeader(
-      `Configure Claude permissions ${chalk.dim("[skipped]")}`,
-      "Grants Claude read/write access to your external learn folder without repeated prompts."
-    );
-    log.dim(
-      "Skipped \u2014 learn directory is inside the project. Run `cf permission` for other permissions."
-    );
+  printStepHeader(
+    "Configure Claude permissions",
+    "Grants Coding Friend skills/hooks the permissions they need, so you get fewer prompts."
+  );
+  await stepClaudePermissions(isExternal ? outputDir : null, autoCommit);
+  if (!existsSync(localConfigPath())) {
+    writeJson(localConfigPath(), {});
   }
   console.log();
   log.congrats("Setup complete!");

package/dist/{install-USFLRCS5.js → install-3QCRGPTY.js}

@@ -1,7 +1,7 @@
 import {
   getLatestVersion,
   semverCompare
-} from "./chunk-UWQPMVJY.js";
+} from "./chunk-A427XMWE.js";
 import {
   enableMarketplaceAutoUpdate,
   isMarketplaceRegistered,
@@ -13,7 +13,7 @@ import {
 import "./chunk-POC2WHU2.js";
 import {
   ensureShellCompletion
-} from "./chunk-DVMWMXDZ.js";
+} from "./chunk-VUAUAO2R.js";
 import {
   resolveScope
 } from "./chunk-C5LYVVEI.js";

package/dist/{permission-L2QQR5PO.js → permission-Z3LOBJ4X.js}

@@ -1,11 +1,15 @@
 import {
-  PERMISSION_RULES,
+  STATIC_RULES,
   applyPermissions,
+  cleanupStalePluginRules,
+  getAllRules,
   getExistingRules,
-  groupByCategory
-} from "./chunk-56U7US6J.js";
+  groupByCategory,
+  logPluginScriptWarning
+} from "./chunk-7CAIGH2Y.js";
 import {
-  claudeLocalSettingsPath
+  claudeLocalSettingsPath,
+  claudeSettingsPath
 } from "./chunk-RWUTFVRB.js";
 import {
   log
@@ -34,6 +38,39 @@ function colorDescription(desc) {
   }
   return `${main} ${usedBy}`;
 }
+async function resolveSettingsPath(opts) {
+  if (opts.user && opts.project) {
+    log.error("Cannot use both --user and --project. Pick one.");
+    process.exit(1);
+  }
+  if (opts.user) {
+    const p2 = claudeSettingsPath();
+    return { path: p2, label: p2.replace(homedir(), "~"), scope: "user" };
+  }
+  if (opts.project) {
+    const p2 = claudeLocalSettingsPath();
+    return { path: p2, label: p2.replace(homedir(), "~"), scope: "project" };
+  }
+  if (!process.stdin.isTTY) {
+    const p2 = claudeLocalSettingsPath();
+    return { path: p2, label: p2.replace(homedir(), "~"), scope: "project" };
+  }
+  const scope = await select({
+    message: "Where should permissions be saved?",
+    choices: [
+      {
+        name: "Project \u2014 .claude/settings.local.json (this project only, gitignored)",
+        value: "project"
+      },
+      {
+        name: "User \u2014 ~/.claude/settings.json (all projects)",
+        value: "user"
+      }
+    ]
+  });
+  const p = scope === "user" ? claudeSettingsPath() : claudeLocalSettingsPath();
+  return { path: p, label: p.replace(homedir(), "~"), scope };
+}
 async function interactiveFlow(allRules, existing) {
   const groups = groupByCategory(allRules);
   const allRuleStrings = allRules.map((r) => r.rule);
@@ -98,14 +135,17 @@ async function interactiveFlow(allRules, existing) {
 async function permissionCommand(opts) {
   console.log("=== \u{1F33F} Coding Friend Permissions \u{1F33F} ===");
   console.log();
-  const settingsPath = claudeLocalSettingsPath();
-  const settingsLabel = settingsPath.replace(homedir(), "~");
+  const {
+    path: settingsPath,
+    label: settingsLabel,
+    scope
+  } = await resolveSettingsPath(opts);
   const existing = getExistingRules(settingsPath);
-  const allRules = PERMISSION_RULES;
+  const allRules = getAllRules();
   const allRuleStrings = allRules.map((r) => r.rule);
   const managedExisting = existing.filter((r) => allRuleStrings.includes(r));
   const unmanagedExisting = existing.filter((r) => !allRuleStrings.includes(r));
-  log.dim(`Settings: ${settingsLabel} (project-local)`);
+  log.dim(`Settings: ${settingsLabel} (${scope})`);
   log.dim(
     `Current: ${managedExisting.length}/${allRules.length} Coding Friend rules configured`
   );
@@ -122,13 +162,34 @@ async function permissionCommand(opts) {
       log.success("All recommended permissions already configured.");
       return;
     }
-    console.log("Adding recommended permissions:");
+    const staticSet = new Set(STATIC_RULES.map((r) => r.rule));
+    const staticCount = toAdd2.filter((r) => staticSet.has(r)).length;
+    const pluginCount = toAdd2.length - staticCount;
+    console.log(
+      `Adding ${chalk.bold(toAdd2.length)} recommended permissions${pluginCount > 0 ? ` (${staticCount} static + ${pluginCount} plugin scripts)` : ""}:`
+    );
     for (const r of toAdd2) {
       console.log(`  ${chalk.green("+")} ${r}`);
     }
+    if (pluginCount > 0) {
+      console.log();
+      logPluginScriptWarning(log, chalk);
+    }
     console.log();
+    const ok2 = !process.stdin.isTTY || await confirm({
+      message: `Apply ${toAdd2.length} permission rules to ${settingsLabel}?`,
+      default: true
+    });
+    if (!ok2) {
+      log.dim("Skipped.");
+      return;
+    }
     applyPermissions(settingsPath, toAdd2, []);
-    log.success(`Added ${toAdd2.length} permission rules.`);
+    const cleaned = cleanupStalePluginRules(settingsPath);
+    if (cleaned > 0) {
+      log.dim(`Removed ${cleaned} stale old-format plugin rules.`);
+    }
+    log.success(`Added ${toAdd2.length} permission rules to ${settingsLabel}.`);
     return;
   }
   const { toAdd, toRemove } = await interactiveFlow(allRules, existing);
@@ -160,7 +221,7 @@ async function permissionCommand(opts) {
   }
   applyPermissions(settingsPath, toAdd, toRemove);
   log.success(
-    `Done \u2014 added ${toAdd.length}, removed ${toRemove.length} rules.`
+    `Done \u2014 added ${toAdd.length}, removed ${toRemove.length} rules in ${settingsLabel}.`
   );
 }
 export {

package/dist/postinstall.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 import {
   ensureShellCompletion
-} from "./chunk-DVMWMXDZ.js";
+} from "./chunk-VUAUAO2R.js";
 import "./chunk-W5CD7WTX.js";
 
 // src/postinstall.ts

package/dist/{uninstall-QSNKGNHR.js → uninstall-5LRHXFSF.js}

@@ -5,7 +5,7 @@ import {
 import {
   hasShellCompletion,
   removeShellCompletion
-} from "./chunk-DVMWMXDZ.js";
+} from "./chunk-VUAUAO2R.js";
 import {
   resolveScope
 } from "./chunk-C5LYVVEI.js";

package/dist/{update-PNHTIB6M.js → update-4YUSCBCB.js}

@@ -2,10 +2,10 @@ import {
   getLatestVersion,
   semverCompare,
   updateCommand
-} from "./chunk-UWQPMVJY.js";
+} from "./chunk-A427XMWE.js";
 import "./chunk-ORACWEDN.js";
 import "./chunk-POC2WHU2.js";
-import "./chunk-DVMWMXDZ.js";
+import "./chunk-VUAUAO2R.js";
 import "./chunk-C5LYVVEI.js";
 import "./chunk-CYQU33FY.js";
 import "./chunk-RWUTFVRB.js";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "coding-friend-cli",
-  "version": "1.17.4",
+  "version": "1.18.0",
   "description": "CLI for coding-friend — host learning docs, setup MCP server, initialize projects",
   "type": "module",
   "bin": {