@onebrain-ai/cli 2.0.0 → 2.0.2

package/dist/onebrain

@@ -10742,7 +10742,7 @@ init_dist();
 import { mkdir as mkdir3, readFile as readFile3, rename as rename3, stat as stat3, writeFile as writeFile3 } from "node:fs/promises";
 import { homedir as homedir3 } from "node:os";
 import { dirname as dirname3, join as join5 } from "node:path";
-var binaryVersion = typeof BUILD_VERSION !== "undefined" ? BUILD_VERSION : "dev";
+var binaryVersion = "2.0.2";
 var STANDARD_FOLDERS = [
   "00-inbox",
   "01-projects",
@@ -12598,8 +12598,8 @@ async function vaultSyncCommand2(vaultRoot, opts = {}) {
 }
 
 // src/index.ts
-var VERSION = typeof BUILD_VERSION !== "undefined" ? BUILD_VERSION : "0.0.0-dev";
-var RELEASE_DATE = typeof BUILD_DATE !== "undefined" ? BUILD_DATE : "dev";
+var VERSION = "2.0.2";
+var RELEASE_DATE = "2026-04-25";
 var VERSION_STRING = `OneBrain v${VERSION} \u2014 released ${RELEASE_DATE}`;
 if (process.argv.slice(2).length === 0) {
   console.log(VERSION_STRING);

package/package.json

@@ -1,6 +1,28 @@
 {
   "name": "@onebrain-ai/cli",
-  "version": "2.0.0",
+  "version": "2.0.2",
+  "description": "CLI for OneBrain — personal AI OS for Obsidian with persistent memory, 24+ skills, and Claude Code integration",
+  "keywords": [
+    "onebrain",
+    "obsidian",
+    "ai",
+    "cli",
+    "memory",
+    "knowledge-management",
+    "claude",
+    "agent",
+    "pkm",
+    "productivity",
+    "vault"
+  ],
+  "homepage": "https://github.com/kengio/onebrain",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/kengio/onebrain.git"
+  },
+  "bugs": "https://github.com/kengio/onebrain/issues",
+  "license": "MIT",
+  "files": ["dist/onebrain"],
   "type": "module",
   "bin": {
     "onebrain": "dist/onebrain"
@@ -11,12 +33,12 @@
     "bump": "echo 'bump not yet implemented'"
   },
   "dependencies": {
-    "@onebrain/core": "workspace:*",
     "@clack/prompts": "^0.9",
     "commander": "^12",
     "yaml": "^2"
   },
   "devDependencies": {
+    "@onebrain/core": "workspace:*",
     "@types/bun": "latest",
     "@types/node": "^20"
   }

package/src/commands/doctor.test.ts

@@ -1,416 +0,0 @@
-/**
- * Tests for `onebrain doctor` — runDoctor()
- *
- * All @onebrain/core validators are injected via opts so tests are
- * fast, offline, and deterministic. No mock.module needed.
- */
-
-import { afterEach, beforeEach, describe, expect, it, spyOn } from 'bun:test';
-import { mkdir, rm } from 'node:fs/promises';
-import { tmpdir } from 'node:os';
-import { join } from 'node:path';
-
-import type { VaultConfig } from '@onebrain/core';
-import { type DoctorOptions, runDoctor } from './doctor.js';
-
-// ---------------------------------------------------------------------------
-// Helpers
-// ---------------------------------------------------------------------------
-
-async function makeTempVault(): Promise<string> {
-  const dir = join(
-    tmpdir(),
-    `onebrain-doctor-test-${Date.now()}-${Math.random().toString(36).slice(2)}`,
-  );
-  await mkdir(dir, { recursive: true });
-  return dir;
-}
-
-const DEFAULT_CONFIG: VaultConfig = {
-  folders: {
-    inbox: '00-inbox',
-    projects: '01-projects',
-    areas: '02-areas',
-    knowledge: '03-knowledge',
-    resources: '04-resources',
-    agent: '05-agent',
-    archive: '06-archive',
-    logs: '07-logs',
-  },
-};
-
-function makeAllOkValidators(): Required<
-  Pick<
-    DoctorOptions,
-    | 'checkVaultYmlFn'
-    | 'loadVaultConfigFn'
-    | 'checkFoldersFn'
-    | 'checkHarnessBinaryFn'
-    | 'checkQmdEmbeddingsFn'
-    | 'checkVersionDriftFn'
-    | 'checkOrphanCheckpointsFn'
-    | 'checkSandboxFn'
-  >
-> {
-  return {
-    checkVaultYmlFn: async () => ({ check: 'vault.yml', status: 'ok', message: 'valid' }),
-    loadVaultConfigFn: async () => DEFAULT_CONFIG,
-    checkFoldersFn: async () => ({ check: 'folders', status: 'ok', message: '8/8 present' }),
-    checkHarnessBinaryFn: async () => ({
-      check: 'runtime.harness',
-      status: 'ok',
-      message: 'claude-code (found)',
-    }),
-    checkQmdEmbeddingsFn: async () => ({
-      check: 'qmd-embeddings',
-      status: 'ok',
-      message: 'all embedded',
-    }),
-    checkVersionDriftFn: async () => ({ check: 'version-drift', status: 'ok', message: 'v1.0.0' }),
-    checkOrphanCheckpointsFn: async () => ({
-      check: 'orphan-checkpoints',
-      status: 'ok',
-      message: '0 orphans',
-    }),
-    checkSandboxFn: async () => ({ check: 'sandbox', status: 'ok', message: 'enabled' }),
-  };
-}
-
-let tempDir: string;
-
-beforeEach(async () => {
-  tempDir = await makeTempVault();
-});
-
-afterEach(async () => {
-  await rm(tempDir, { recursive: true, force: true });
-});
-
-// ---------------------------------------------------------------------------
-// Tests
-// ---------------------------------------------------------------------------
-
-describe('runDoctor', () => {
-  // ── Exit codes ─────────────────────────────────────────────────────────────
-
-  describe('exit codes', () => {
-    it('returns exitCode 1 when any check returns status error', async () => {
-      const validators = makeAllOkValidators();
-      validators.checkVaultYmlFn = async () => ({
-        check: 'vault.yml',
-        status: 'error',
-        message: 'not found',
-      });
-
-      const result = await runDoctor({ vaultDir: tempDir, isTTY: false, ...validators });
-
-      expect(result.exitCode).toBe(1);
-      expect(result.ok).toBe(false);
-      expect(result.errorCount).toBeGreaterThanOrEqual(1);
-    });
-
-    it('returns exitCode 0 when checks return only warnings (no errors)', async () => {
-      const validators = makeAllOkValidators();
-      validators.checkFoldersFn = async () => ({
-        check: 'folders',
-        status: 'warn',
-        message: '7/8 present',
-      });
-      validators.checkSandboxFn = async () => ({
-        check: 'sandbox',
-        status: 'warn',
-        message: 'disabled',
-      });
-
-      const result = await runDoctor({ vaultDir: tempDir, isTTY: false, ...validators });
-
-      expect(result.exitCode).toBe(0);
-      expect(result.ok).toBe(true);
-      expect(result.warningCount).toBeGreaterThanOrEqual(2);
-      expect(result.errorCount).toBe(0);
-    });
-
-    it('returns exitCode 0 when all checks pass', async () => {
-      const result = await runDoctor({ vaultDir: tempDir, isTTY: false, ...makeAllOkValidators() });
-
-      expect(result.exitCode).toBe(0);
-      expect(result.ok).toBe(true);
-      expect(result.errorCount).toBe(0);
-      expect(result.warningCount).toBe(0);
-    });
-  });
-
-  // ── binaryVersion forwarding ───────────────────────────────────────────────
-
-  describe('binaryVersion forwarding', () => {
-    it('forwards binaryVersion to checkVersionDriftFn when provided', async () => {
-      let capturedBinaryVersion: string | undefined = 'not-set';
-      const validators = makeAllOkValidators();
-      validators.checkVersionDriftFn = async (_vaultDir, _config, bv) => {
-        capturedBinaryVersion = bv;
-        return { check: 'version-drift', status: 'ok', message: 'ok' };
-      };
-
-      await runDoctor({ vaultDir: tempDir, isTTY: false, binaryVersion: 'v2.0.0', ...validators });
-
-      expect(capturedBinaryVersion).toBe('v2.0.0');
-    });
-
-    it('passes undefined binaryVersion to checkVersionDriftFn when omitted', async () => {
-      let capturedBinaryVersion: string | undefined = 'not-set';
-      const validators = makeAllOkValidators();
-      validators.checkVersionDriftFn = async (_vaultDir, _config, bv) => {
-        capturedBinaryVersion = bv;
-        return { check: 'version-drift', status: 'ok', message: 'ok' };
-      };
-
-      await runDoctor({ vaultDir: tempDir, isTTY: false, ...validators });
-
-      expect(capturedBinaryVersion).toBeUndefined();
-    });
-  });
-
-  // ── Summary line selection ─────────────────────────────────────────────────
-
-  describe('summary line selection', () => {
-    it('shows "N errors, N warnings" when both errors and warnings exist', async () => {
-      const logLines: string[] = [];
-      const spy = spyOn(console, 'log').mockImplementation((msg: string) => {
-        logLines.push(msg);
-      });
-      try {
-        const validators = makeAllOkValidators();
-        validators.checkVaultYmlFn = async () => ({
-          check: 'vault.yml',
-          status: 'error',
-          message: 'not found',
-        });
-        validators.checkFoldersFn = async () => ({
-          check: 'folders',
-          status: 'warn',
-          message: '7/8 present',
-        });
-        await runDoctor({ vaultDir: tempDir, isTTY: false, ...validators });
-      } finally {
-        spy.mockRestore();
-      }
-
-      expect(logLines.join('\n')).toMatch(/Summary: 1 errors, 1 warnings/);
-    });
-
-    it('shows "N errors" (no warnings mention) when only errors', async () => {
-      const logLines: string[] = [];
-      const spy = spyOn(console, 'log').mockImplementation((msg: string) => {
-        logLines.push(msg);
-      });
-      try {
-        const validators = makeAllOkValidators();
-        validators.checkVaultYmlFn = async () => ({
-          check: 'vault.yml',
-          status: 'error',
-          message: 'not found',
-        });
-        await runDoctor({ vaultDir: tempDir, isTTY: false, ...validators });
-      } finally {
-        spy.mockRestore();
-      }
-
-      const output = logLines.join('\n');
-      expect(output).toMatch(/Summary: 1 errors$/m);
-      expect(output).not.toMatch(/warnings/);
-    });
-
-    it('shows "N warnings — ok to run" when only warnings (no errors)', async () => {
-      const logLines: string[] = [];
-      const spy = spyOn(console, 'log').mockImplementation((msg: string) => {
-        logLines.push(msg);
-      });
-      try {
-        const validators = makeAllOkValidators();
-        validators.checkSandboxFn = async () => ({
-          check: 'sandbox',
-          status: 'warn',
-          message: 'disabled',
-        });
-        await runDoctor({ vaultDir: tempDir, isTTY: false, ...validators });
-      } finally {
-        spy.mockRestore();
-      }
-
-      expect(logLines.join('\n')).toMatch(/Summary: 1 warnings — ok to run/);
-    });
-
-    it('shows "All checks passed" when no errors or warnings', async () => {
-      const logLines: string[] = [];
-      const spy = spyOn(console, 'log').mockImplementation((msg: string) => {
-        logLines.push(msg);
-      });
-      try {
-        await runDoctor({ vaultDir: tempDir, isTTY: false, ...makeAllOkValidators() });
-      } finally {
-        spy.mockRestore();
-      }
-
-      expect(logLines.join('\n')).toMatch(/Summary: All checks passed/);
-    });
-  });
-
-  // ── TTY vs non-TTY output formatting ──────────────────────────────────────
-
-  describe('TTY vs non-TTY output', () => {
-    it('non-TTY: plain title without leading blank line', async () => {
-      const logLines: string[] = [];
-      const spy = spyOn(console, 'log').mockImplementation((msg: string) => {
-        logLines.push(msg);
-      });
-      try {
-        await runDoctor({ vaultDir: tempDir, isTTY: false, ...makeAllOkValidators() });
-      } finally {
-        spy.mockRestore();
-      }
-
-      expect(logLines.join('\n')).toMatch(/^OneBrain Doctor 🔍/);
-    });
-
-    it('TTY: title is padded with surrounding blank lines', async () => {
-      const logLines: string[] = [];
-      const spy = spyOn(console, 'log').mockImplementation((msg: string) => {
-        logLines.push(msg);
-      });
-      try {
-        await runDoctor({ vaultDir: tempDir, isTTY: true, ...makeAllOkValidators() });
-      } finally {
-        spy.mockRestore();
-      }
-
-      const output = logLines.join('\n');
-      expect(output).toMatch(/^\n\s+OneBrain Doctor 🔍/);
-      expect(output).toMatch(/Summary: All checks passed\n$/);
-    });
-  });
-
-  // ── loadVaultConfig failure resilience ────────────────────────────────────
-
-  describe('loadVaultConfig failure resilience', () => {
-    it('continues with default config when loadVaultConfigFn throws after valid vault.yml', async () => {
-      let foldersConfigReceived: VaultConfig | undefined;
-      const validators = makeAllOkValidators();
-      validators.checkVaultYmlFn = async () => ({
-        check: 'vault.yml',
-        status: 'ok',
-        message: 'valid',
-      });
-      validators.loadVaultConfigFn = async () => {
-        throw new Error('parse error');
-      };
-      validators.checkFoldersFn = async (_vaultDir, config) => {
-        foldersConfigReceived = config;
-        return { check: 'folders', status: 'ok', message: '8/8 present' };
-      };
-
-      const result = await runDoctor({ vaultDir: tempDir, isTTY: false, ...validators });
-
-      expect(result.ok).toBe(true);
-      expect(result.exitCode).toBe(0);
-      expect(foldersConfigReceived?.folders.inbox).toBe('00-inbox');
-      expect(foldersConfigReceived?.folders.logs).toBe('07-logs');
-    });
-
-    it('skips loadVaultConfigFn when checkVaultYml returns error', async () => {
-      let loadCalled = false;
-      const validators = makeAllOkValidators();
-      validators.checkVaultYmlFn = async () => ({
-        check: 'vault.yml',
-        status: 'error',
-        message: 'not found',
-      });
-      validators.loadVaultConfigFn = async () => {
-        loadCalled = true;
-        return DEFAULT_CONFIG;
-      };
-
-      await runDoctor({ vaultDir: tempDir, isTTY: false, ...validators });
-
-      expect(loadCalled).toBe(false);
-    });
-  });
-
-  // ── Hint lines ────────────────────────────────────────────────────────────
-
-  describe('hint lines', () => {
-    it('includes hint line in output when a check returns a hint', async () => {
-      const logLines: string[] = [];
-      const spy = spyOn(console, 'log').mockImplementation((msg: string) => {
-        logLines.push(msg);
-      });
-      try {
-        const validators = makeAllOkValidators();
-        validators.checkVaultYmlFn = async () => ({
-          check: 'vault.yml',
-          status: 'error',
-          message: 'vault.yml not found',
-          hint: 'Run onebrain init to create vault.yml',
-        });
-        await runDoctor({ vaultDir: tempDir, isTTY: false, ...validators });
-      } finally {
-        spy.mockRestore();
-      }
-
-      expect(logLines.join('\n')).toContain('→ Run onebrain init to create vault.yml');
-    });
-
-    it('does not include a hint line when check has no hint', async () => {
-      const logLines: string[] = [];
-      const spy = spyOn(console, 'log').mockImplementation((msg: string) => {
-        logLines.push(msg);
-      });
-      try {
-        await runDoctor({ vaultDir: tempDir, isTTY: false, ...makeAllOkValidators() });
-      } finally {
-        spy.mockRestore();
-      }
-
-      expect(logLines.join('\n')).not.toContain('→');
-    });
-  });
-
-  // ── errorCount / warningCount accuracy ────────────────────────────────────
-
-  describe('result counts', () => {
-    it('accurately counts multiple errors and warnings across all checks', async () => {
-      const validators = makeAllOkValidators();
-      validators.checkVaultYmlFn = async () => ({
-        check: 'vault.yml',
-        status: 'error',
-        message: 'not found',
-      });
-      validators.checkFoldersFn = async () => ({
-        check: 'folders',
-        status: 'error',
-        message: '0/8 present',
-      });
-      validators.checkSandboxFn = async () => ({
-        check: 'sandbox',
-        status: 'warn',
-        message: 'disabled',
-      });
-      validators.checkHarnessBinaryFn = async () => ({
-        check: 'runtime.harness',
-        status: 'warn',
-        message: 'not found',
-      });
-
-      const result = await runDoctor({ vaultDir: tempDir, isTTY: false, ...validators });
-
-      expect(result.errorCount).toBe(2);
-      expect(result.warningCount).toBe(2);
-      expect(result.exitCode).toBe(1);
-    });
-
-    it('returns errorCount 0 and warningCount 0 when all checks pass', async () => {
-      const result = await runDoctor({ vaultDir: tempDir, isTTY: false, ...makeAllOkValidators() });
-      expect(result.errorCount).toBe(0);
-      expect(result.warningCount).toBe(0);
-    });
-  });
-});

package/src/commands/doctor.ts

@@ -1,203 +0,0 @@
-import {
-  type DoctorResult,
-  type VaultConfig,
-  checkFolders,
-  checkHarnessBinary,
-  checkOrphanCheckpoints,
-  checkQmdEmbeddings,
-  checkSandbox,
-  checkVaultYml,
-  checkVersionDrift,
-  loadVaultConfig,
-} from '@onebrain/core';
-
-// ---------------------------------------------------------------------------
-// Types
-// ---------------------------------------------------------------------------
-
-export interface DoctorOptions {
-  /** Vault root directory (default: process.cwd()). */
-  vaultDir?: string;
-  /** Whether stdout is a TTY (default: process.stdout.isTTY). */
-  isTTY?: boolean;
-  /** Compiled binary version (BUILD_VERSION). When provided, compared against plugin.json instead of vault.yml onebrain_version. */
-  binaryVersion?: string;
-  /** Injectable validators — real implementations are used when absent. */
-  checkVaultYmlFn?: (vaultDir: string) => Promise<DoctorResult>;
-  loadVaultConfigFn?: (vaultDir: string) => Promise<VaultConfig>;
-  checkFoldersFn?: (vaultDir: string, config: VaultConfig) => Promise<DoctorResult>;
-  checkHarnessBinaryFn?: (config: VaultConfig) => Promise<DoctorResult>;
-  checkQmdEmbeddingsFn?: (config: VaultConfig) => Promise<DoctorResult>;
-  checkVersionDriftFn?: (
-    vaultDir: string,
-    config: VaultConfig,
-    binaryVersion?: string,
-  ) => Promise<DoctorResult>;
-  checkOrphanCheckpointsFn?: (vaultDir: string, config: VaultConfig) => Promise<DoctorResult>;
-  checkSandboxFn?: (config: VaultConfig) => DoctorResult | Promise<DoctorResult>;
-}
-
-export interface DoctorCommandResult {
-  ok: boolean;
-  exitCode: number;
-  errorCount: number;
-  warningCount: number;
-}
-
-// ---------------------------------------------------------------------------
-// Main runDoctor (pure, testable)
-// ---------------------------------------------------------------------------
-
-export async function runDoctor(opts: DoctorOptions = {}): Promise<DoctorCommandResult> {
-  const vaultDir = opts.vaultDir ?? process.cwd();
-  const isTTY = opts.isTTY ?? process.stdout.isTTY ?? false;
-  const binaryVersion = opts.binaryVersion;
-
-  const checkVaultYmlFn = opts.checkVaultYmlFn ?? checkVaultYml;
-  const loadVaultConfigFn = opts.loadVaultConfigFn ?? loadVaultConfig;
-  const checkFoldersFn = opts.checkFoldersFn ?? checkFolders;
-  const checkHarnessBinaryFn = opts.checkHarnessBinaryFn ?? checkHarnessBinary;
-  const checkQmdEmbeddingsFn = opts.checkQmdEmbeddingsFn ?? checkQmdEmbeddings;
-  const checkVersionDriftFn = opts.checkVersionDriftFn ?? checkVersionDrift;
-  const checkOrphanCheckpointsFn = opts.checkOrphanCheckpointsFn ?? checkOrphanCheckpoints;
-  const checkSandboxFn = opts.checkSandboxFn ?? checkSandbox;
-
-  const vaultYmlResult = await checkVaultYmlFn(vaultDir);
-
-  let config: VaultConfig = {
-    folders: {
-      inbox: '00-inbox',
-      projects: '01-projects',
-      areas: '02-areas',
-      knowledge: '03-knowledge',
-      resources: '04-resources',
-      agent: '05-agent',
-      archive: '06-archive',
-      logs: '07-logs',
-    },
-  };
-
-  if (vaultYmlResult.status === 'ok') {
-    try {
-      config = await loadVaultConfigFn(vaultDir);
-    } catch {
-      // If loading fails, use default config above
-    }
-  }
-
-  const [
-    foldersResult,
-    harnessResult,
-    qmdResult,
-    versionDriftResult,
-    orphanCheckpointsResult,
-    sandboxResult,
-  ] = await Promise.all([
-    checkFoldersFn(vaultDir, config),
-    checkHarnessBinaryFn(config),
-    checkQmdEmbeddingsFn(config),
-    checkVersionDriftFn(vaultDir, config, binaryVersion),
-    checkOrphanCheckpointsFn(vaultDir, config),
-    checkSandboxFn(config),
-  ]);
-
-  const results = [
-    vaultYmlResult,
-    foldersResult,
-    harnessResult,
-    qmdResult,
-    versionDriftResult,
-    orphanCheckpointsResult,
-    sandboxResult,
-  ];
-
-  const errorCount = results.filter((r) => r.status === 'error').length;
-  const warningCount = results.filter((r) => r.status === 'warn').length;
-
-  printDoctorOutput(results, isTTY, errorCount, warningCount);
-
-  return {
-    ok: errorCount === 0,
-    exitCode: errorCount > 0 ? 1 : 0,
-    errorCount,
-    warningCount,
-  };
-}
-
-// ---------------------------------------------------------------------------
-// CLI entry point — thin wrapper, calls process.exit
-// ---------------------------------------------------------------------------
-
-export async function doctorCommand(opts: DoctorOptions = {}): Promise<void> {
-  const result = await runDoctor(opts);
-  process.exit(result.exitCode);
-}
-
-// ---------------------------------------------------------------------------
-// Formatting
-// ---------------------------------------------------------------------------
-
-function printDoctorOutput(
-  results: DoctorResult[],
-  isTTY: boolean,
-  errorCount: number,
-  warningCount: number,
-): void {
-  const lines: string[] = [];
-
-  if (isTTY) {
-    lines.push('');
-    lines.push('  OneBrain Doctor 🔍');
-    lines.push('');
-  } else {
-    lines.push('OneBrain Doctor 🔍');
-    lines.push('');
-  }
-
-  for (const result of results) {
-    const statusIcon = getStatusIcon(result.status);
-    lines.push(formatCheckLine(result, statusIcon));
-    if (result.hint) {
-      lines.push(formatHintLine(result.hint));
-    }
-  }
-
-  lines.push('');
-
-  if (errorCount > 0 && warningCount > 0) {
-    lines.push(`Summary: ${errorCount} errors, ${warningCount} warnings`);
-  } else if (errorCount > 0) {
-    lines.push(`Summary: ${errorCount} errors`);
-  } else if (warningCount > 0) {
-    lines.push(`Summary: ${warningCount} warnings — ok to run`);
-  } else {
-    lines.push('Summary: All checks passed');
-  }
-
-  if (isTTY) {
-    lines.push('');
-  }
-
-  console.log(lines.join('\n'));
-}
-
-function getStatusIcon(status: 'ok' | 'warn' | 'error'): string {
-  switch (status) {
-    case 'ok':
-      return '[✓]';
-    case 'warn':
-      return '[!]';
-    case 'error':
-      return '[✗]';
-    default:
-      return '[?]';
-  }
-}
-
-function formatCheckLine(result: DoctorResult, icon: string): string {
-  return `  ${icon} ${result.check.padEnd(20)} ${result.message}`;
-}
-
-function formatHintLine(hint: string): string {
-  return `        → ${hint}`;
-}