@robbiesrobotics/alice-agents 1.3.3 → 1.4.1

package/README.md

@@ -74,7 +74,7 @@ When you install, the installer will **auto-detect your configured model** and u
 # Interactive install
 npx @robbiesrobotics/alice-agents
 
-# Non-interactive with defaults (Sonnet, Starter tier)
+# Non-interactive with defaults (detected model if available, otherwise Sonnet; Starter tier)
 npx @robbiesrobotics/alice-agents --yes
 
 # Show help

package/bin/alice-install.mjs

@@ -4,6 +4,7 @@ import { readFileSync } from 'node:fs';
 import { createRequire } from 'node:module';
 import { runInstall, runUninstall } from '../lib/installer.mjs';
 import { runDoctor } from '../lib/doctor.mjs';
+import { runSkillsManager } from '../lib/skills.mjs';
 
 const args = process.argv.slice(2);
 const flags = new Set(args);
@@ -24,6 +25,7 @@ if (flags.has('--help') || flags.has('-h')) {
     npx @robbiesrobotics/alice-agents --update     Non-interactive upgrade to latest agents
     npx @robbiesrobotics/alice-agents --uninstall  Remove A.L.I.C.E. agents from config
     npx @robbiesrobotics/alice-agents --doctor     Run diagnostics on your A.L.I.C.E. install
+    npx @robbiesrobotics/alice-agents --skills     Manage skills (install, remove, browse)
     npx @robbiesrobotics/alice-agents --version    Show version
     npx @robbiesrobotics/alice-agents --help       Show this help
 
@@ -47,6 +49,11 @@ if (flags.has('--doctor')) {
     console.error('  ❌ Update failed:', err.message);
     process.exit(1);
   });
+} else if (flags.has('--skills')) {
+  runSkillsManager().catch((err) => {
+    console.error(`  ✗ Skills manager failed: ${err.message}`);
+    process.exit(1);
+  });
 } else if (flags.has('--uninstall')) {
   runUninstall({ yes: flags.has('--yes') }).catch((err) => {
     console.error('  ❌ Uninstall failed:', err.message);

package/lib/colors.mjs

@@ -0,0 +1,102 @@
+// lib/colors.mjs
+// ANSI color helper — no external deps, NO_COLOR + non-TTY safe
+// Brand: A.L.I.C.E. / NemoClaw green (#76b900 → \x1b[92m bright green)
+
+const enabled =
+  process.stdout.isTTY &&
+  !process.env.NO_COLOR &&
+  process.env.TERM !== 'dumb';
+
+const esc = (code) => (str) => enabled ? `\x1b[${code}m${str}\x1b[0m` : str;
+
+export const c = {
+  green:    esc('92'),
+  greenDim: esc('32'),
+  success:  esc('92'),
+  error:    esc('91'),
+  warning:  esc('93'),
+  info:     esc('96'),
+  accent:   esc('95'),
+  bold:     esc('1'),
+  dim:      esc('2'),
+  italic:   esc('3'),
+  underline:esc('4'),
+  white:    esc('97'),
+  gray:     esc('90'),
+};
+
+export const bold       = (s) => c.bold(s);
+export const dim        = (s) => c.dim(s);
+export const green      = (s) => c.green(s);
+export const greenBold  = (s) => enabled ? `\x1b[1m\x1b[92m${s}\x1b[0m` : s;
+export const red        = (s) => c.error(s);
+export const yellow     = (s) => c.warning(s);
+export const cyan       = (s) => c.info(s);
+export const gray       = (s) => c.gray(s);
+export const white      = (s) => c.white(s);
+
+export const icons = {
+  ok:    green('✔'),
+  fail:  red('✗'),
+  warn:  yellow('⚠'),
+  info:  cyan('ℹ'),
+  pkg:   '📦',
+  arrow: dim('›'),
+  bullet:dim('·'),
+  check: green('✓'),
+  cross: red('✗'),
+};
+
+const SEP_WIDTH = 50;
+const SEP_CHAR  = '─';
+
+export function separator() {
+  return dim(SEP_CHAR.repeat(SEP_WIDTH));
+}
+
+export function printSection(title) {
+  const totalDashes = SEP_WIDTH - title.length - 4;
+  const left  = SEP_CHAR.repeat(2);
+  const right = SEP_CHAR.repeat(Math.max(0, totalDashes));
+  console.log(`\n  ${dim(left)} ${greenBold(title)} ${dim(right)}`);
+}
+
+export function printSeparator() {
+  console.log(`  ${separator()}`);
+}
+
+const stripAnsi = (s) => s.replace(/\x1b\[[0-9;]*m/g, '');
+
+export function printBox(lines, { title = null, padding = 1 } = {}) {
+  const innerWidth = Math.max(...lines.map(l => stripAnsi(l).length)) + padding * 2;
+  const w = innerWidth;
+  const TL = '╭', TR = '╮', BL = '╰', BR = '╯', H = '─', V = '│';
+
+  const topBar = title
+    ? `${TL}${H} ${greenBold(title)} ${H.repeat(Math.max(0, w - stripAnsi(title).length - 4))}${TR}`
+    : `${TL}${H.repeat(w + 2)}${TR}`;
+
+  console.log(`  ${dim(topBar)}`);
+  lines.forEach(line => {
+    const pad = ' '.repeat(padding);
+    const raw = stripAnsi(line);
+    const trail = ' '.repeat(Math.max(0, w - raw.length - padding));
+    console.log(`  ${dim(V)}${pad}${line}${trail}${dim(V)}`);
+  });
+  console.log(`  ${dim(`${BL}${H.repeat(w + 2)}${BR}`)}`);
+}
+
+export function printStepDone(label, detail = '') {
+  const suffix = detail ? `  ${dim(detail)}` : '';
+  console.log(`  ${icons.ok} ${label}${suffix}`);
+}
+
+export function printStepFail(label, reason = '') {
+  const suffix = reason ? `  ${red(reason)}` : '';
+  console.log(`  ${icons.fail} ${label}${suffix}`);
+}
+
+export function printStepSkip(label, reason = '') {
+  const suffix = reason ? `  ${dim(reason)}` : '';
+  console.log(`  ${dim('–')} ${dim(label)}${suffix}`);
+}

package/lib/config-merger.mjs

@@ -6,6 +6,33 @@ import { randomBytes } from 'node:crypto';
 const OPENCLAW_DIR = join(homedir(), '.openclaw');
 const CONFIG_PATH = join(OPENCLAW_DIR, 'openclaw.json');
 
+function normalizeProviderId(provider) {
+  if (!provider) return null;
+  if (provider === 'openai-codex') return 'openai';
+  return provider;
+}
+
+function getModelProvider(model) {
+  if (!model || typeof model !== 'string' || !model.includes('/')) return null;
+  return normalizeProviderId(model.split('/')[0]);
+}
+
+function collectConfiguredProviders(config) {
+  const providers = new Set();
+
+  for (const provider of Object.keys(config?.models?.providers || {})) {
+    const normalized = normalizeProviderId(provider);
+    if (normalized) providers.add(normalized);
+  }
+
+  for (const profile of Object.values(config?.auth?.profiles || {})) {
+    const normalized = normalizeProviderId(profile?.provider);
+    if (normalized) providers.add(normalized);
+  }
+
+  return [...providers];
+}
+
 export function configExists() {
   return existsSync(CONFIG_PATH);
 }
@@ -34,7 +61,7 @@ export function detectAvailableModels() {
     const globalModel = config?.model;
 
     // Configured provider keys (e.g. ['anthropic', 'openai', 'ollama'])
-    const providers = Object.keys(config?.models?.providers || {});
+    const providers = collectConfiguredProviders(config);
 
     const primary = agentDefaults.primary || globalModel || null;
     const orchestrator = agentDefaults.orchestrator || primary;
@@ -127,6 +154,22 @@ function getModelConfig(preset, customModels, detectedModels) {
   }
 }
 
+function isCompatibleWithConfiguredProviders(modelCfg, detectedModels) {
+  const providers = new Set(detectedModels?.providers || []);
+  if (providers.size === 0) return true;
+
+  const models = [
+    modelCfg?.primary,
+    modelCfg?.orchestrator,
+    ...(modelCfg?.fallbacks || []),
+  ].filter(Boolean);
+
+  return models.every((model) => {
+    const provider = getModelProvider(model);
+    return !provider || providers.has(provider);
+  });
+}
+
 function buildAgentEntry(agent, modelCfg) {
   const entry = {
     id: agent.id,
@@ -157,7 +200,17 @@ export function mergeConfig({ agents, mode, preset, customModels }) {
   const backupPath = backupConfig();
   const config = readConfig();
   const detectedModels = detectAvailableModels();
-  const modelCfg = getModelConfig(preset, customModels, detectedModels);
+  let effectivePreset = preset;
+  let warning = null;
+  let modelCfg = getModelConfig(preset, customModels, detectedModels);
+
+  if (!modelCfg.preserve && !isCompatibleWithConfiguredProviders(modelCfg, detectedModels)) {
+    effectivePreset = detectedModels?.hasModel ? 'detected' : preset;
+    if (effectivePreset === 'detected') {
+      warning = `Preset "${preset}" does not match configured OpenClaw providers (${(detectedModels.providers || []).join(', ')}). Preserving existing model settings instead.`;
+      modelCfg = getModelConfig('detected', customModels, detectedModels);
+    }
+  }
 
   // Build agent entries
   const aliceEntries = agents.map((a) => buildAgentEntry(a, modelCfg));
@@ -221,7 +274,7 @@ export function mergeConfig({ agents, mode, preset, customModels }) {
   config.tools.agentToAgent.allow = [...aliceIds];
 
   writeConfigAtomic(config);
-  return { backupPath, agentCount: aliceEntries.length };
+  return { backupPath, agentCount: aliceEntries.length, effectivePreset, warning };
 }
 
 export function removeAliceAgents(agentIds) {

package/lib/doctor.mjs

@@ -3,9 +3,21 @@ import { join, dirname } from 'node:path';
 import { homedir, platform } from 'node:os';
 import { execSync } from 'node:child_process';
 import { fileURLToPath } from 'node:url';
+import { icons, greenBold, green, red, yellow, cyan, dim, bold,
+         printSection, printSeparator, separator } from './colors.mjs';
 
 const __dirname = dirname(fileURLToPath(import.meta.url));
 const HOME = homedir();
+
+function commandExists(cmd) {
+  const probe = process.platform === 'win32' ? 'where' : 'which';
+  try {
+    execSync(`${probe} ${cmd}`, { stdio: 'pipe' });
+    return true;
+  } catch {
+    return false;
+  }
+}
 const OPENCLAW_DIR = join(HOME, '.openclaw');
 
 const STARTER_AGENTS = [
@@ -14,9 +26,9 @@ const STARTER_AGENTS = [
 ];
 
 function check(label, ok, hint) {
-  const icon = ok ? '✓' : '✗';
-  console.log(`  ${icon}  ${label}`);
-  if (!ok && hint) console.log(`     → ${hint}`);
+  const icon = ok ? icons.ok : icons.fail;
+  console.log(`  ${icon}  ${ok ? green(label) : red(label)}`);
+  if (!ok && hint) console.log(`     ${dim('→')} ${dim(hint)}`);
   return ok;
 }
 
@@ -51,12 +63,7 @@ function checkDockerEnvironment() {
   const isLinux = platform() === 'linux';
 
   // Check if docker binary exists at all
-  let dockerInstalled = false;
-  try {
-    execSync('which docker', { stdio: 'pipe' });
-    dockerInstalled = true;
-  } catch {
-    // Docker not installed — not a blocker unless openclaw needs it
+  if (!commandExists('docker')) {
     return { present: false, accessible: false, needsSudo: false, hint: null };
   }
 
@@ -109,7 +116,10 @@ function checkDockerEnvironment() {
 }
 
 export async function runDoctor() {
-  console.log('\n  🩺 A.L.I.C.E. Doctor — Diagnostic Report\n');
+  console.log('');
+  printSection('A.L.I.C.E. Doctor');
+  console.log(`  ${dim('Diagnostic Report')}`);
+  console.log('');
   let allOk = true;
 
   // 1. Runtime check
@@ -130,7 +140,7 @@ export async function runDoctor() {
 
   if (!configExists) {
     allOk = false;
-    console.log('\n  ⚠️  Cannot continue checks — openclaw.json missing.\n');
+    console.log(`\n  ${icons.warn} ${yellow('Cannot continue checks — openclaw.json missing.')}\n`);
     return false;
   }
 
@@ -143,7 +153,7 @@ export async function runDoctor() {
   );
   if (!configValid) {
     allOk = false;
-    console.log('\n  ⚠️  Cannot continue checks — config is invalid JSON.\n');
+    console.log(`\n  ${icons.warn} ${yellow('Cannot continue checks — config is invalid JSON.')}\n`);
     return false;
   }
 
@@ -236,11 +246,16 @@ export async function runDoctor() {
       );
       // Docker permission issue is a warning, not a hard failure for A.L.I.C.E. itself
       // but it will break OpenClaw's own Docker features
-      console.log('  ℹ️  Note: This will affect OpenClaw features that use Docker.\n');
+      console.log(`  ${icons.info} ${dim('This will affect OpenClaw features that use Docker.')}\n`);
     } else {
       check('Docker daemon not running or not accessible', false, docker.hint);
     }
   }
+
+  const isWSL = !!process.env.WSL_DISTRO_NAME || !!process.env.WSLENV;
+  if (isWSL && docker.present) {
+    console.log("  ℹ️  WSL2 detected. If Docker has cgroup issues, run: nemoclaw setup-spark\n");
+  }
   // If docker not present at all, skip silently — not required for all setups
 
   // 7. License check
@@ -276,13 +291,43 @@ export async function runDoctor() {
     check('License: Starter tier (no license required)', true);
   }
 
+  // 8. Skills disk check
+  const skillsManifestPath = join(OPENCLAW_DIR, '.alice-manifest.json');
+  const skillsManifestData = (() => {
+    try {
+      if (!existsSync(skillsManifestPath)) return null;
+      return JSON.parse(readFileSync(skillsManifestPath, 'utf8'));
+    } catch { return null; }
+  })();
+
+  const installedSkills = skillsManifestData?.skills || [];
+  if (installedSkills.length > 0) {
+    const missingSkills = installedSkills.filter(
+      id => !existsSync(join(HOME, '.openclaw', 'skills', id))
+    );
+    if (missingSkills.length > 0) {
+      check(
+        `Skills missing on disk: ${missingSkills.join(', ')}`,
+        false,
+        'Run: npx @robbiesrobotics/alice-agents --skills to reinstall'
+      );
+      allOk = false;
+    } else {
+      check(`Skills installed (${installedSkills.length}): ${installedSkills.join(', ')}`, true);
+    }
+  } else {
+    console.log(`  ${dim('–')}  ${dim('No skills installed (optional)')}`);
+  }
+
   // Summary
-  console.log();
+  console.log('');
   if (allOk) {
-    console.log('  ✅ A.L.I.C.E. is healthy!\n');
+    console.log(`  ${icons.ok} ${greenBold('A.L.I.C.E. is healthy!')}`);
   } else {
-    console.log('  ⚠️  Issues found — follow the hints above to fix them.\n');
+    console.log(`  ${icons.warn} ${yellow('Issues found')} ${dim('─ follow the hints above to fix them.')}`);
   }
+  printSeparator();
+  console.log('');
 
   return allOk;
 }