@bunshin-cc/bunshin 0.3.1 → 0.3.3

package/dist/bin.js

@@ -3352,7 +3352,7 @@ function repoCommit(root) {
 }
 function listDocs(root) {
   const listed = git(root, ["ls-files", "-co", "--exclude-standard", "-z"], 3e3);
-  if (listed !== null) return [...new Set(listed.split("\0").filter(Boolean).filter(isDoc))].filter((rel) => fs7.existsSync(path6.join(root, rel))).sort();
+  if (listed !== null) return [...new Set(listed.split("\0").filter(Boolean).filter(isDoc))].filter((rel) => fs7.existsSync(path6.join(root, rel)) && !deployedSkill(root, rel)).sort();
   const out = [];
   let visited = 0;
   const walk = (dir, rel, depth) => {
@@ -3365,10 +3365,10 @@ function listDocs(root) {
     }
     for (const e of entries) {
       if (e.isDirectory()) {
-        if (!SKIP_DIRS.has(e.name)) walk(path6.join(dir, e.name), rel ? `${rel}/${e.name}` : e.name, depth + 1);
+        if (!SKIP_DIRS.has(e.name) && !fs7.existsSync(path6.join(dir, e.name, ".git"))) walk(path6.join(dir, e.name), rel ? `${rel}/${e.name}` : e.name, depth + 1);
       } else if (e.isFile()) {
         const r = rel ? `${rel}/${e.name}` : e.name;
-        if (isDoc(r)) out.push(r);
+        if (isDoc(r) && !deployedSkill(root, r)) out.push(r);
       }
     }
   };
@@ -3448,7 +3448,7 @@ function relativeToRoot(root, file, cwd = root) {
   }
   return null;
 }
-var DOC_NAMES, SKIP_DIRS, LIST_TTL_MS, rootCache, isDoc;
+var DOC_NAMES, SKIP_DIRS, LIST_TTL_MS, rootCache, isDoc, deployedSkill;
 var init_repo = __esm({
   "src/v2/repo.ts"() {
     "use strict";
@@ -3462,6 +3462,7 @@ var init_repo = __esm({
       if (parts.some((p) => SKIP_DIRS.has(p))) return false;
       return DOC_NAMES.has(parts[parts.length - 1].toLowerCase());
     };
+    deployedSkill = (root, rel) => /(^|\/)SKILL\.md$/i.test(rel) && fs7.existsSync(path6.join(root, path6.posix.dirname(rel), ".bunshin.json"));
   }
 });
 
@@ -4175,11 +4176,13 @@ var init_exports = {};
 __export(init_exports, {
   CODEX_HOOK_EVENTS: () => CODEX_HOOK_EVENTS,
   HOOK_EVENTS: () => HOOK_EVENTS,
+  LEARN_COMMAND: () => LEARN_COMMAND,
   PRE_TOOL_MATCHER: () => PRE_TOOL_MATCHER,
   bunshinHookEntry: () => bunshinHookEntry,
   claudeSettingsPath: () => claudeSettingsPath,
   countBunshinHooks: () => countBunshinHooks,
   installClaudeHooks: () => installClaudeHooks,
+  installLearnCommand: () => installLearnCommand,
   installMcpConfig: () => installMcpConfig,
   installPiExtension: () => installPiExtension,
   isBunshinEntry: () => isBunshinEntry,
@@ -4290,6 +4293,24 @@ function mcpLauncher(bin) {
   }
   return { command: process.execPath, args: [script, "mcp"] };
 }
+function installLearnCommand({ file, remove = false }) {
+  try {
+    const current = fileExists(file) ? fs13.readFileSync(file, "utf8") : null;
+    const ours = current === null || current === LEARN_COMMAND;
+    if (remove) {
+      if (current !== null && ours) fs13.unlinkSync(file);
+      return { file, installed: 0, changed: current !== null && ours };
+    }
+    if (!ours) return { file, installed: 0, changed: false, error: "a different learn.md is already there; left as it is" };
+    if (current === null) {
+      fs13.mkdirSync(path10.dirname(file), { recursive: true });
+      fs13.writeFileSync(file, LEARN_COMMAND);
+    }
+    return { file, installed: 1, changed: current === null };
+  } catch (err) {
+    return { file, installed: 0, changed: false, error: err instanceof Error ? err.message : String(err) };
+  }
+}
 function installMcpConfig({ file, launcher, remove = false }) {
   try {
     const before = fileExists(file) ? JSON.parse(fs13.readFileSync(file, "utf8")) : {};
@@ -4362,15 +4383,18 @@ function runInit(opts = {}) {
     if (hooks.error) warnings.push(`could not update ${hooks.file}: ${hooks.error}`);
   }
   let mcp;
+  let learnCommand;
   if ((opts.removeHooks || !opts.noHooks) && config.harness === "claude-code" && scope === "repo") {
     mcp = installMcpConfig({ file: path10.join(cwd, ".mcp.json"), launcher: opts.mcp ?? mcpLauncher(opts.bin), remove: opts.removeHooks });
     if (mcp.error) warnings.push(`could not update ${mcp.file}: ${mcp.error}`);
+    learnCommand = installLearnCommand({ file: path10.join(cwd, ".claude", "commands", "learn.md"), remove: opts.removeHooks });
+    if (learnCommand.error) warnings.push(`/learn not installed at ${learnCommand.file}: ${learnCommand.error}`);
   }
   if (!resolveBackend(config)) {
     warnings.push("No model backend found. Install and sign in to Codex or Claude Code before using automatic learning.");
   }
   store.log("init", { root, scope, role: config.role, orgUnit: config.orgUnit, hooks: hooks?.installed ?? 0 });
-  return { root, config, created: !existed, hooks, ...mcp ? { mcp } : {}, warnings };
+  return { root, config, created: !existed, hooks, ...mcp ? { mcp } : {}, ...learnCommand ? { learnCommand } : {}, warnings };
 }
 function otherHarnessInstructions() {
   return [
@@ -4386,7 +4410,7 @@ function otherHarnessInstructions() {
     "  Claude Code plugin users need no settings.json wiring \u2014 the plugin ships hooks/hooks.json."
   ].join("\n");
 }
-var PRE_TOOL_MATCHER, HOOK_EVENTS, CODEX_HOOK_EVENTS;
+var PRE_TOOL_MATCHER, HOOK_EVENTS, CODEX_HOOK_EVENTS, LEARN_COMMAND;
 var init_init = __esm({
   "src/init.ts"() {
     "use strict";
@@ -4410,6 +4434,7 @@ var init_init = __esm({
       { event: "PreCompact", arg: "pre-compact", timeout: 10 },
       { event: "SessionEnd", arg: "session-end", timeout: 3 }
     ];
+    LEARN_COMMAND = '---\ndescription: Capture what this session learned into the Bunshin library now (applied locally as a trial, submitted for the nightly review)\nargument-hint: "[what to capture, e.g. \\"the ledger must be reconciled before closing\\"]"\nallowed-tools: Bash(bunshin learn:*)\n---\n\nCapture knowledge from this session into Bunshin.\n\nRun exactly this with the Bash tool (it takes up to two minutes; use a timeout of 180000 ms):\n\n```bash\nbunshin learn "$ARGUMENTS"\n```\n\n`bunshin learn` forks this conversation (`claude -p --resume <this session> --fork-session`),\nso the reflection sees everything you saw without re-sending the transcript. It applies what it\nfinds locally as a trial and queues it for the nightly review:\n\n- edits to this repo\'s SPEC.md / TESTING.md / SKILL.md land in the working tree (they show up\n  in `git diff`);\n- knowledge with no repo home becomes a local overlay on the Bunshin library;\n- per-machine values stay private.\n\nRelay its output to the user as it is. Each line reads\n`captured: <summary> \u2192 <section or file#anchor>`. If it prints `nothing new to capture.`, say so\nin one line; that is the normal outcome. If it reports an error, show it and stop; do not retry\nin a loop, and do not edit the library files by hand.\n\nIf the user named nothing specific, run it without arguments. Do not continue with other work in\nthe same reply unless the user asked you to.\n';
   }
 });
 
@@ -4491,6 +4516,7 @@ __export(commands_exports, {
   cmdRecall: () => cmdRecall,
   cmdReflectFork: () => cmdReflectFork,
   cmdReview: () => cmdReview,
+  cmdReviewDecide: () => cmdReviewDecide,
   cmdSuggest: () => cmdSuggest,
   cmdSync: () => cmdSync,
   robotAttrsFromFlags: () => robotAttrsFromFlags,
@@ -4532,7 +4558,7 @@ function cmdReview(store, args, cwd, out, err) {
 `);
     }
     out(`
-bunshin review apply <id> | submit <id> | skip <id> | apply-all
+bunshin review apply <id> | submit <id> | skip <id> | apply-all   (owners: approve <id> | reject <id> [--note])
 `);
     return 0;
   }
@@ -4574,6 +4600,29 @@ bunshin review apply <id> | submit <id> | skip <id> | apply-all
 `);
   return 1;
 }
+async function cmdReviewDecide(store, args, out, err) {
+  const sub = args._[1];
+  const id = args._[2];
+  if (!id) {
+    err(`bunshin review ${sub}: a suggestion id is required (see the console's Review, or bunshin review list --all)
+`);
+    return 1;
+  }
+  const config = store.config();
+  if (!canCall(config)) {
+    err(`bunshin review ${sub}: this project is not connected to Bunshin
+`);
+    return 1;
+  }
+  const note = str(args.flags.note);
+  const res = await v2api(config, "POST", `/suggestions/${encodeURIComponent(id)}/review`, {
+    approve: sub === "approve",
+    ...note ? { note } : {}
+  });
+  out(`${res.suggestion.id}: ${res.suggestion.status}${res.suggestion.decision?.reason ? ` (${res.suggestion.decision.reason})` : ""}
+`);
+  return 0;
+}
 async function cmdRecall(store, args, cwd, out, err) {
   const abstract = args._.slice(1).join(" ").trim() || str(args.flags.abstract) || "";
   if (!abstract) {
@@ -8320,8 +8369,9 @@ Usage: bunshin <command> [options]
                                 print one document (or one part of it) and mark it loaded
   learn [focus...]              capture what this session learned now (fork reflection),
                                 apply it locally as a trial and submit it
-  review [list [--all]|apply <id>|submit <id>|skip <id>|apply-all]
-                                suggestions awaiting review (apply = local trial + submit)
+  review [list [--all]|apply <id>|submit <id>|skip <id>|apply-all|approve <id>|reject <id> [--note <text>]]
+                                suggestions awaiting review (apply = local trial + submit;
+                                approve / reject = owner decision: merge now, or keep it out)
   index [--upload] [--json]     parse SPEC.md / TESTING.md / SKILL.md into sections
   sync [--index] [--json]       upload episodes + suggestions (incl. in-place skill edits), pull
                                 the library, install skills / instructions / files, settle trials
@@ -8440,6 +8490,10 @@ Key saved privately outside the project.
 `);
       if (result.hooks?.error) throw new Error(`Connected, but hooks could not be configured: ${result.hooks.error}. Fix the file and run setup again.`);
       if (result.hooks) out(`${result.hooks.installed} ${harness === "pi" ? "extension" : "hooks"} configured in ${result.hooks.file}
+`);
+      if (result.learnCommand?.installed) out(`/learn installed in ${result.learnCommand.file}
+`);
+      else if (result.learnCommand?.error) out(`/learn not installed: ${result.learnCommand.error} (${result.learnCommand.file})
 `);
       if (result.hooks?.sharedWithWorktrees) out("Codex shares these hooks with the repository\u2019s linked worktrees. Each connected worktree keeps its own Bunshin store.\n");
       if (harness === "codex" && result.hooks) out("Run codex interactively in this project. Review and accept project trust, then run /hooks and review and trust the five Bunshin hooks.\n");
@@ -8487,6 +8541,7 @@ Key saved privately outside the project.
       case "suggest":
         return (await Promise.resolve().then(() => (init_commands(), commands_exports))).cmdSuggest(store, args, cwd, out, err);
       case "review":
+        if (args._[1] === "approve" || args._[1] === "reject") return await (await Promise.resolve().then(() => (init_commands(), commands_exports))).cmdReviewDecide(store, args, out, err);
         return (await Promise.resolve().then(() => (init_commands(), commands_exports))).cmdReview(store, args, cwd, out, err);
       case "questions":
       case "question":

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bunshin-cc/bunshin",
-  "version": "0.3.1",
+  "version": "0.3.3",
   "description": "Shared memory for company AI agents. Connect Codex, Claude Code, or Pi to Bunshin.",
   "type": "module",
   "bin": {