@onebrain-ai/cli 2.3.0 → 2.3.1

package/README.md

@@ -528,6 +528,22 @@ Or use the interactive wizards from inside your vault:
 
 Output goes to `[logs_folder]/scheduler/YYYY/MM/YYYY-MM-DD-{skill}.md` as readable markdown.
 
+### Command mode (CLI binaries, hook-style)
+
+For CLI maintenance tasks that aren't OneBrain skills, use the `command + args[]` shape:
+
+```yaml
+schedule:
+  - cron: "0 3 * * 0"
+    command: onebrain
+    args: [qmd-reindex]
+  - cron: "0 5 * * *"
+    command: rsync
+    args: [-av, /vault, /backup]
+```
+
+This matches the same shape Claude Code uses for `hooks` in `settings.json` — direct binary invocation with positional argv. No wrapper skill needed.
+
 CLI flags:
 
 | Flag | Purpose |

package/dist/onebrain

@@ -9560,7 +9560,7 @@ var init_lib = __esm(() => {
 var require_package = __commonJS((exports, module) => {
   module.exports = {
     name: "@onebrain-ai/cli",
-    version: "2.3.0",
+    version: "2.3.1",
     description: "CLI for OneBrain \u2014 personal AI OS for Obsidian with persistent memory, 24+ skills, and Claude Code integration",
     keywords: [
       "onebrain",
@@ -11048,7 +11048,7 @@ var import_picocolors5 = __toESM(require_picocolors(), 1);
 var import_picocolors = __toESM(require_picocolors(), 1);
 function resolveBinaryVersion() {
   if (true)
-    return "2.3.0";
+    return "2.3.1";
   try {
     const pkg = require_package();
     return pkg.version ?? "dev";
@@ -13197,40 +13197,94 @@ function atToLaunchd(at) {
 function isOneShot(entry) {
   return entry.at !== undefined;
 }
+function isSkillMode(entry) {
+  return entry.skill !== undefined;
+}
+function isCommandMode(entry) {
+  return entry.command !== undefined;
+}
 function validateEntry(entry) {
   const hasCron = entry.cron !== undefined;
   const hasAt = entry.at !== undefined;
   if (hasCron === hasAt) {
     return { valid: false, reason: "entry must have exactly one of `cron` or `at`" };
   }
-  if (!entry.skill)
-    return { valid: false, reason: "entry.skill is required" };
+  const hasSkill = entry.skill !== undefined;
+  const hasCommand = entry.command !== undefined;
+  if (hasSkill === hasCommand) {
+    return { valid: false, reason: "entry must have exactly one of `skill` or `command`" };
+  }
+  if (hasSkill && !entry.skill) {
+    return { valid: false, reason: "entry.skill must not be empty" };
+  }
+  if (hasCommand && !entry.command) {
+    return { valid: false, reason: "entry.command must not be empty" };
+  }
+  if (entry.args !== undefined) {
+    const isArray = Array.isArray(entry.args);
+    if (hasSkill && isArray) {
+      return {
+        valid: false,
+        reason: "skill-mode entries require `args` as a map (Record<string, string>), not an array"
+      };
+    }
+    if (hasCommand && !isArray) {
+      return {
+        valid: false,
+        reason: "command-mode entries require `args` as a string array, not a map"
+      };
+    }
+    if (isArray) {
+      for (const v2 of entry.args) {
+        if (typeof v2 !== "string") {
+          return { valid: false, reason: "command-mode `args` must contain only strings" };
+        }
+      }
+    }
+  }
   return { valid: true };
 }
 
 // src/lib/scheduler/launchd.ts
 var xmlEscape = (s) => s.replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/"/g, "&quot;");
+function labelForEntry(entry) {
+  const raw = isCommandMode(entry) ? entry.command : (entry.skill ?? "").replace(/^\//, "");
+  return raw.replace(/[^a-zA-Z0-9-]/g, "-");
+}
 function generatePlist(entry, ctx) {
-  const labelSafe = entry.skill.replace(/^\//, "").replace(/[^a-zA-Z0-9-]/g, "-");
+  const labelSafe = labelForEntry(entry);
   const label = `com.onebrain.${labelSafe}`;
-  let programArgumentsBlock;
-  let calendarXml;
-  if (entry.at !== undefined) {
-    const calendar = atToLaunchd(entry.at);
-    calendarXml = Object.entries(calendar).map(([k2, v2]) => `        <key>${k2}</key>
+  const calendar = isOneShot(entry) ? atToLaunchd(entry.at) : cronFieldsToLaunchd(entry.cron);
+  const calendarXml = Object.entries(calendar).map(([k2, v2]) => `        <key>${k2}</key>
         <integer>${v2}</integer>`).join(`
 `);
-    const plistFilePath = plistPath(entry.skill, ctx.homedir);
-    const argsFlags = entry.args ? ` ${Object.entries(entry.args).map(([k2, v2]) => `--${k2}="${v2}"`).join(" ")}` : "";
-    const shellLine = xmlEscape(`"${ctx.skillCliPath}" --vault="${ctx.vaultPath}" --skill="${entry.skill}" --headless${argsFlags}; launchctl bootout gui/${ctx.uid}/${label}; rm -f "${plistFilePath}"`);
-    programArgumentsBlock = `        <string>/bin/sh</string>
+  let programArgumentsBlock;
+  if (isOneShot(entry)) {
+    if (isCommandMode(entry)) {
+      const argv = entry.args ?? [];
+      const quotedArgs = argv.map((a2) => `"${a2}"`).join(" ");
+      const innerCommand = `"${entry.command}"${quotedArgs ? ` ${quotedArgs}` : ""}`;
+      const plistFilePath = `${ctx.homedir}/Library/LaunchAgents/${label}.plist`;
+      const shellLine = xmlEscape(`${innerCommand}; launchctl bootout gui/${ctx.uid}/${label}; rm -f "${plistFilePath}"`);
+      programArgumentsBlock = `        <string>/bin/sh</string>
         <string>-c</string>
         <string>${shellLine}</string>`;
-  } else {
-    const calendar = cronFieldsToLaunchd(entry.cron);
-    calendarXml = Object.entries(calendar).map(([k2, v2]) => `        <key>${k2}</key>
-        <integer>${v2}</integer>`).join(`
+    } else {
+      const plistFilePath = plistPath(entry.skill ?? "", ctx.homedir);
+      const argsFlags = entry.args ? ` ${Object.entries(entry.args).map(([k2, v2]) => `--${k2}="${v2}"`).join(" ")}` : "";
+      const shellLine = xmlEscape(`"${ctx.skillCliPath}" --vault="${ctx.vaultPath}" --skill="${entry.skill}" --headless${argsFlags}; launchctl bootout gui/${ctx.uid}/${label}; rm -f "${plistFilePath}"`);
+      programArgumentsBlock = `        <string>/bin/sh</string>
+        <string>-c</string>
+        <string>${shellLine}</string>`;
+    }
+  } else if (isCommandMode(entry)) {
+    const argv = entry.args ?? [];
+    programArgumentsBlock = [
+      `        <string>${xmlEscape(entry.command)}</string>`,
+      ...argv.map((a2) => `        <string>${xmlEscape(a2)}</string>`)
+    ].join(`
 `);
+  } else {
     const argsBlock = entry.args ? `
 ${Object.entries(entry.args).map(([k2, v2]) => `        <string>--${xmlEscape(k2)}=${xmlEscape(v2)}</string>`).join(`
 `)}` : "";
@@ -13238,7 +13292,7 @@ ${Object.entries(entry.args).map(([k2, v2]) => `        <string>--${xmlEscape(k2
         <string>--vault</string>
         <string>${xmlEscape(ctx.vaultPath)}</string>
         <string>--skill</string>
-        <string>${xmlEscape(entry.skill)}</string>
+        <string>${xmlEscape(entry.skill ?? "")}</string>
         <string>--headless</string>${argsBlock}`;
   }
   return `<?xml version="1.0" encoding="UTF-8"?>
@@ -13264,8 +13318,8 @@ ${calendarXml}
 </dict>
 </plist>`;
 }
-function plistPath(skill, homedir4) {
-  const labelSafe = skill.replace(/^\//, "").replace(/[^a-zA-Z0-9-]/g, "-");
+function plistPath(skillOrLabel, homedir4) {
+  const labelSafe = skillOrLabel.startsWith("/") ? skillOrLabel.replace(/^\//, "").replace(/[^a-zA-Z0-9-]/g, "-") : skillOrLabel.replace(/[^a-zA-Z0-9-]/g, "-");
   return `${homedir4}/Library/LaunchAgents/com.onebrain.${labelSafe}.plist`;
 }
 
@@ -13296,12 +13350,15 @@ async function registerSchedule(opts) {
       const va = validateAt(entry.at);
       if (!va.valid)
         throw new Error(`Invalid at "${entry.at}": ${va.reason}`);
+      sanitizeArgsForOneShot(entry);
     } else if (entry.cron !== undefined) {
       const vc = validateCron(entry.cron);
       if (!vc.valid)
         throw new Error(`Invalid cron "${entry.cron}": ${vc.reason}`);
     }
-    await validateSchedulable(opts.vault, entry);
+    if (isSkillMode(entry)) {
+      await validateSchedulable(opts.vault, entry);
+    }
   }
   const skillCliPath = process.argv[1] ?? "onebrain";
   const ctx = {
@@ -13313,15 +13370,20 @@ async function registerSchedule(opts) {
   };
   const seen = new Map;
   for (const entry of entries) {
-    const target = plistPath(entry.skill, ctx.homedir);
+    const target = plistPath(labelForEntry(entry), ctx.homedir);
     if (seen.has(target)) {
-      throw new Error(`Conflict: ${entry.skill} and ${seen.get(target)?.skill} normalize to the same plist path ${target}`);
+      const existing = seen.get(target);
+      if (existing) {
+        const existingLabel = isCommandMode(existing) ? `command:${existing.command}` : `skill:${existing.skill}`;
+        const newLabel = isCommandMode(entry) ? `command:${entry.command}` : `skill:${entry.skill}`;
+        throw new Error(`Conflict: ${newLabel} and ${existingLabel} normalize to the same plist path ${target}`);
+      }
     }
     seen.set(target, entry);
   }
   for (const entry of entries) {
     const plistContent = generatePlist(entry, ctx);
-    const targetPath = plistPath(entry.skill, ctx.homedir);
+    const targetPath = plistPath(labelForEntry(entry), ctx.homedir);
     if (opts.dryRun) {
       console.log(import_picocolors8.default.cyan(`---  ${targetPath}  ---`));
       console.log(plistContent);
@@ -13334,7 +13396,7 @@ async function registerSchedule(opts) {
 Registered ${entries.length} schedule entries.`));
   console.log(import_picocolors8.default.dim("Use launchctl to load (or restart launchd):"));
   for (const entry of entries) {
-    const target = plistPath(entry.skill, ctx.homedir);
+    const target = plistPath(labelForEntry(entry), ctx.homedir);
     console.log(import_picocolors8.default.dim(`  launchctl load ${target}`));
   }
 }
@@ -13345,7 +13407,18 @@ async function readVaultConfig(vault) {
   const raw = await readFile6(yamlPath, "utf8");
   return import_yaml7.parse(raw) ?? {};
 }
+function sanitizeArgsForOneShot(entry) {
+  const values = isCommandMode(entry) ? entry.args ?? [] : Object.values(entry.args ?? {});
+  for (const v2 of values) {
+    if (/["$`\\]/.test(v2)) {
+      throw new Error(`Arg value must not contain shell-special chars (", $, \`, \\): ${v2}`);
+    }
+  }
+}
 async function validateSchedulable(vault, entry) {
+  if (!entry.skill) {
+    throw new Error("validateSchedulable invoked on non-skill entry \u2014 caller bug");
+  }
   const skillName = entry.skill.replace(/^\//, "");
   const skillPath = join11(vault, ".claude/plugins/onebrain/skills", skillName, "SKILL.md");
   if (!existsSync(skillPath)) {
@@ -13373,7 +13446,7 @@ async function validateSchedulable(vault, entry) {
   if (entry.args) {
     for (const [k2, v2] of Object.entries(entry.args)) {
       if (/["$`\\]/.test(v2)) {
-        throw new Error(`Arg "${k2}" value must not contain shell-special chars (", $, backtick, \\): ${v2}`);
+        throw new Error(`Arg "${k2}" value must not contain shell-special chars (", $, \`, \\): ${v2}`);
       }
     }
   }
@@ -13382,7 +13455,7 @@ async function removeAll(vault) {
   const config = await readVaultConfig(vault);
   const entries = config.schedule ?? [];
   for (const entry of entries) {
-    const target = plistPath(entry.skill, homedir4());
+    const target = plistPath(labelForEntry(entry), homedir4());
     if (existsSync(target)) {
       await unlink4(target);
       console.log(import_picocolors8.default.green(`\u2713 Removed ${target}`));
@@ -13394,11 +13467,21 @@ async function printStatus(vault) {
   const entries = config.schedule ?? [];
   console.log(import_picocolors8.default.cyan(`Registered schedules: ${entries.length}`));
   for (const entry of entries) {
-    const target = plistPath(entry.skill, homedir4());
+    const target = plistPath(labelForEntry(entry), homedir4());
     const installed = existsSync(target) ? "\u2713" : "\u2717";
     const when = entry.at ?? entry.cron ?? "?";
     const tag = entry.at ? import_picocolors8.default.magenta("[once]") : import_picocolors8.default.dim("[cron]");
-    console.log(`  ${installed} ${tag} ${when}  ${entry.skill}`);
+    let targetLabel;
+    if (isCommandMode(entry)) {
+      const argv = entry.args ?? [];
+      const argStr = argv.length ? ` ${argv.join(" ")}` : "";
+      targetLabel = `${import_picocolors8.default.yellow("cmd:")} ${entry.command}${argStr}`;
+    } else {
+      const argsMap = entry.args ?? {};
+      const argStr = Object.keys(argsMap).length ? ` (${Object.entries(argsMap).map(([k2, v2]) => `${k2}=${v2}`).join(", ")})` : "";
+      targetLabel = `${import_picocolors8.default.green("skill:")} ${entry.skill}${argStr}`;
+    }
+    console.log(`  ${installed} ${tag} ${when}  ${targetLabel}`);
   }
 }
 async function testRun(vault, skill) {
@@ -13645,7 +13728,7 @@ function patchUtf8(stream) {
 }
 
 // src/index.ts
-var VERSION = "2.3.0";
+var VERSION = "2.3.1";
 var RELEASE_DATE = "2026-05-12";
 patchUtf8(process.stdout);
 patchUtf8(process.stderr);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onebrain-ai/cli",
-  "version": "2.3.0",
+  "version": "2.3.1",
   "description": "CLI for OneBrain — personal AI OS for Obsidian with persistent memory, 24+ skills, and Claude Code integration",
   "keywords": [
     "onebrain",