ultra-builder-pro-cli 0.6.0 → 0.8.1

package/ultra-tools/system-command.test.cjs

@@ -0,0 +1,53 @@
+'use strict';
+
+const { test } = require('node:test');
+const assert = require('node:assert/strict');
+const fs = require('node:fs');
+const os = require('node:os');
+const path = require('node:path');
+const { spawnSync } = require('node:child_process');
+
+const { initStateDb, closeStateDb } = require('../mcp-server/lib/state-db.cjs');
+
+const CLI = path.join(__dirname, 'cli.cjs');
+
+function fixture() {
+  const rootDir = fs.mkdtempSync(path.join(os.tmpdir(), 'ubp-system-cli-'));
+  const { db } = initStateDb(path.join(rootDir, '.ultra', 'state.db'));
+  closeStateDb(db);
+  return rootDir;
+}
+
+function invoke(rootDir, args) {
+  return spawnSync(process.execPath, [CLI, 'system', ...args], {
+    cwd: rootDir, encoding: 'utf8', stdio: ['ignore', 'pipe', 'pipe'],
+  });
+}
+
+test('system doctor provides a read-only JSON health envelope', () => {
+  const rootDir = fixture();
+  try {
+    const result = invoke(rootDir, ['doctor']);
+    assert.equal(result.status, 0, result.stderr);
+    const envelope = JSON.parse(result.stdout.trim().split('\n').at(-1));
+    assert.equal(envelope.ok, true);
+    assert.equal(envelope.data.status, 'healthy');
+    assert.equal(envelope.data.repair_performed, false);
+  } finally {
+    fs.rmSync(rootDir, { recursive: true, force: true });
+  }
+});
+
+test('system doctor --repair creates a backup before recovery', () => {
+  const rootDir = fixture();
+  try {
+    const result = invoke(rootDir, ['doctor', '--repair']);
+    assert.equal(result.status, 0, result.stderr);
+    const envelope = JSON.parse(result.stdout.trim().split('\n').at(-1));
+    assert.equal(envelope.ok, true);
+    assert.equal(envelope.data.repair_performed, true);
+    assert.ok(fs.existsSync(envelope.data.backup_path));
+  } finally {
+    fs.rmSync(rootDir, { recursive: true, force: true });
+  }
+});

package/adapters/_shared/rtk-detect.cjs

@@ -1,86 +0,0 @@
-'use strict';
-
-// Phase 6.1 — RTK soft dependency.
-//
-// RTK (Rust Token Killer) is an external CLI proxy the user installs
-// separately. We never hard-require it: installation continues even when
-// rtk is missing, we just print an install hint. This mirrors PLAN Phase
-// 6.1: "无 rtk → 提示安装 + 跳过（不硬依赖）".
-//
-// Tests inject a PATH override so we can exercise the happy/missing paths
-// without touching the real rtk binary on the machine.
-
-const { execFileSync } = require('node:child_process');
-
-const INSTALL_HINT =
-  'rtk not found. RTK is a token-saving proxy that rewrites Bash/git/npm\n' +
-  '  commands transparently. Install: https://github.com/rcrocky/rtk\n' +
-  '  (or run `cargo install rtk`). Continuing without RTK.';
-
-function envWithPath(pathOverride) {
-  if (!pathOverride) return process.env;
-  // Isolate rtk lookup to pathOverride + minimal system dirs (so shebangs
-  // like `/usr/bin/env bash` still resolve). Tests use this to cut off the
-  // real rtk binary on the developer's machine.
-  return { ...process.env, PATH: `${pathOverride}:/usr/bin:/bin` };
-}
-
-function detectRtk({ pathOverride = null, timeoutMs = 3000 } = {}) {
-  try {
-    const stdout = execFileSync('rtk', ['--version'], {
-      env: envWithPath(pathOverride),
-      stdio: ['ignore', 'pipe', 'pipe'],
-      timeout: timeoutMs,
-    });
-    return { available: true, version: stdout.toString().trim() };
-  } catch (err) {
-    return {
-      available: false,
-      reason: err && err.code === 'ENOENT' ? 'binary not on PATH' : (err && err.message) || 'unknown',
-    };
-  }
-}
-
-function initRtk({ pathOverride = null, scope = 'global', timeoutMs = 10000 } = {}) {
-  const args = ['init'];
-  if (scope === 'global') args.push('-g');
-  try {
-    const stdout = execFileSync('rtk', args, {
-      env: envWithPath(pathOverride),
-      stdio: ['ignore', 'pipe', 'pipe'],
-      timeout: timeoutMs,
-    });
-    return { ok: true, stdout: stdout.toString() };
-  } catch (err) {
-    return { ok: false, error: (err && err.message) || 'unknown' };
-  }
-}
-
-// High-level entry: detect + optionally init, never throws.
-// Caller passes {skip:true} to honor --skip-rtk flag.
-function installHook({ skip = false, pathOverride = null, scope = 'global' } = {}) {
-  if (skip) return { skipped: true };
-  const detection = detectRtk({ pathOverride });
-  if (!detection.available) {
-    return {
-      available: false,
-      initialized: false,
-      reason: detection.reason,
-      hint: INSTALL_HINT,
-    };
-  }
-  const initResult = initRtk({ pathOverride, scope });
-  return {
-    available: true,
-    initialized: initResult.ok,
-    version: detection.version,
-    init_error: initResult.ok ? undefined : initResult.error,
-  };
-}
-
-module.exports = {
-  detectRtk,
-  initRtk,
-  installHook,
-  INSTALL_HINT,
-};

package/adapters/_shared/tests/rtk-detect.test.cjs

@@ -1,116 +0,0 @@
-'use strict';
-
-// Phase 6.1 — RTK soft dependency:
-//   • detectRtk(opts): locates the `rtk` binary via PATH; returns false
-//     (never throws) when missing or on any exec error.
-//   • initRtk(opts): runs `rtk init -g`; only call after detectRtk() true.
-
-const { test } = require('node:test');
-const assert = require('node:assert/strict');
-const fs = require('node:fs');
-const os = require('node:os');
-const path = require('node:path');
-
-const rtk = require('../rtk-detect.cjs');
-
-function mkEmptyBinDir() {
-  return fs.mkdtempSync(path.join(os.tmpdir(), 'ubp-rtk-empty-bin-'));
-}
-
-function mkFakeBinDir({ succeed = true } = {}) {
-  const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'ubp-rtk-fake-bin-'));
-  const script = succeed
-    ? '#!/usr/bin/env bash\nif [ "$1" = "--version" ]; then echo "rtk 1.2.3"; exit 0; fi\nif [ "$1" = "init" ]; then echo "init ok"; exit 0; fi\nexit 0\n'
-    : '#!/usr/bin/env bash\nexit 42\n';
-  const target = path.join(dir, 'rtk');
-  fs.writeFileSync(target, script);
-  fs.chmodSync(target, 0o755);
-  return dir;
-}
-
-test('detectRtk returns true when rtk --version succeeds', () => {
-  const binDir = mkFakeBinDir({ succeed: true });
-  try {
-    const result = rtk.detectRtk({ pathOverride: binDir });
-    assert.equal(result.available, true);
-    assert.match(result.version, /rtk/i);
-  } finally {
-    fs.rmSync(binDir, { recursive: true, force: true });
-  }
-});
-
-test('detectRtk returns false when rtk missing from PATH (no throw)', () => {
-  const binDir = mkEmptyBinDir();
-  try {
-    const result = rtk.detectRtk({ pathOverride: binDir });
-    assert.equal(result.available, false);
-    assert.ok(result.reason);
-  } finally {
-    fs.rmSync(binDir, { recursive: true, force: true });
-  }
-});
-
-test('detectRtk returns false when rtk exits non-zero (no throw)', () => {
-  const binDir = mkFakeBinDir({ succeed: false });
-  try {
-    const result = rtk.detectRtk({ pathOverride: binDir });
-    assert.equal(result.available, false);
-  } finally {
-    fs.rmSync(binDir, { recursive: true, force: true });
-  }
-});
-
-test('initRtk runs rtk init -g and returns success', () => {
-  const binDir = mkFakeBinDir({ succeed: true });
-  try {
-    const result = rtk.initRtk({ pathOverride: binDir, scope: 'global' });
-    assert.equal(result.ok, true);
-    assert.match(result.stdout, /init ok/);
-  } finally {
-    fs.rmSync(binDir, { recursive: true, force: true });
-  }
-});
-
-test('initRtk reports failure when binary is missing (does not throw)', () => {
-  const binDir = mkEmptyBinDir();
-  try {
-    const result = rtk.initRtk({ pathOverride: binDir, scope: 'global' });
-    assert.equal(result.ok, false);
-    assert.ok(result.error);
-  } finally {
-    fs.rmSync(binDir, { recursive: true, force: true });
-  }
-});
-
-test('installHook: skip=true short-circuits with a skipped=true record', () => {
-  const binDir = mkEmptyBinDir();
-  try {
-    const result = rtk.installHook({ skip: true, pathOverride: binDir });
-    assert.equal(result.skipped, true);
-    assert.equal(result.available, undefined);
-  } finally {
-    fs.rmSync(binDir, { recursive: true, force: true });
-  }
-});
-
-test('installHook: missing rtk → returns {available:false} with install hint', () => {
-  const binDir = mkEmptyBinDir();
-  try {
-    const result = rtk.installHook({ skip: false, pathOverride: binDir });
-    assert.equal(result.available, false);
-    assert.ok(result.hint, 'should surface an install hint');
-  } finally {
-    fs.rmSync(binDir, { recursive: true, force: true });
-  }
-});
-
-test('installHook: present rtk → runs init and returns {available:true, initialized:true}', () => {
-  const binDir = mkFakeBinDir({ succeed: true });
-  try {
-    const result = rtk.installHook({ skip: false, pathOverride: binDir, scope: 'global' });
-    assert.equal(result.available, true);
-    assert.equal(result.initialized, true);
-  } finally {
-    fs.rmSync(binDir, { recursive: true, force: true });
-  }
-});