@soleri/cli 9.3.1 → 9.5.0

package/src/main.ts

@@ -21,6 +21,7 @@ import { registerSkills } from './commands/skills.js';
 import { registerAgent } from './commands/agent.js';
 import { registerTelegram } from './commands/telegram.js';
 import { registerStaging } from './commands/staging.js';
+import { registerYolo } from './commands/yolo.js';
 
 const require = createRequire(import.meta.url);
 const { version } = require('../package.json');
@@ -82,4 +83,5 @@ registerSkills(program);
 registerAgent(program);
 registerTelegram(program);
 registerStaging(program);
+registerYolo(program);
 program.parse();

package/vitest.config.ts

@@ -6,5 +6,6 @@ export default defineConfig({
     testTimeout: 15_000,
     pool: 'forks',
     teardownTimeout: 10_000,
+    exclude: ['**/node_modules/**', '**/.claude/worktrees/**'],
   },
 });

package/src/__tests__/archetypes.test.ts

@@ -1,84 +0,0 @@
-import { describe, expect, it } from 'vitest';
-import { ARCHETYPES } from '../prompts/archetypes.js';
-import {
-  CORE_SKILLS,
-  SKILL_CATEGORIES,
-  DOMAIN_OPTIONS,
-  PRINCIPLE_CATEGORIES,
-} from '../prompts/playbook.js';
-
-const allDomainValues = DOMAIN_OPTIONS.map((d) => d.value);
-const allPrincipleValues = PRINCIPLE_CATEGORIES.flatMap((c) => c.options.map((o) => o.value));
-const allOptionalSkillValues = SKILL_CATEGORIES.flatMap((c) => c.options.map((o) => o.value));
-
-describe('Archetypes', () => {
-  it('should have unique values', () => {
-    const values = ARCHETYPES.map((a) => a.value);
-    expect(new Set(values).size).toBe(values.length);
-  });
-
-  it('should all have tier field', () => {
-    for (const a of ARCHETYPES) {
-      expect(a.tier).toMatch(/^(free|premium)$/);
-    }
-  });
-
-  it('should have at least 9 archetypes', () => {
-    expect(ARCHETYPES.length).toBeGreaterThanOrEqual(9);
-  });
-
-  it('should reference only valid domains', () => {
-    for (const a of ARCHETYPES) {
-      for (const d of a.defaults.domains) {
-        expect(allDomainValues).toContain(d);
-      }
-    }
-  });
-
-  it('should reference only valid principles', () => {
-    for (const a of ARCHETYPES) {
-      for (const pr of a.defaults.principles) {
-        expect(allPrincipleValues).toContain(pr);
-      }
-    }
-  });
-
-  it('should not include core skills in archetype skills', () => {
-    const coreSet = new Set<string>(CORE_SKILLS);
-    for (const a of ARCHETYPES) {
-      for (const s of a.defaults.skills) {
-        expect(coreSet.has(s)).toBe(false);
-      }
-    }
-  });
-
-  it('should reference only valid optional skills', () => {
-    for (const a of ARCHETYPES) {
-      for (const s of a.defaults.skills) {
-        expect(allOptionalSkillValues).toContain(s);
-      }
-    }
-  });
-
-  it('should include Accessibility Guardian', () => {
-    expect(ARCHETYPES.find((a) => a.value === 'accessibility-guardian')).toBeDefined();
-  });
-
-  it('should include Documentation Writer', () => {
-    expect(ARCHETYPES.find((a) => a.value === 'documentation-writer')).toBeDefined();
-  });
-});
-
-describe('Core Skills', () => {
-  it('should include writing-plans and executing-plans', () => {
-    expect(CORE_SKILLS).toContain('writing-plans');
-    expect(CORE_SKILLS).toContain('executing-plans');
-  });
-
-  it('should not appear in optional skill categories', () => {
-    const coreSet = new Set<string>(CORE_SKILLS);
-    for (const s of allOptionalSkillValues) {
-      expect(coreSet.has(s)).toBe(false);
-    }
-  });
-});

package/src/__tests__/create.test.ts

@@ -1,207 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { mkdirSync, rmSync, existsSync, readFileSync, writeFileSync, readdirSync } from 'node:fs';
-import { join } from 'node:path';
-import { tmpdir } from 'node:os';
-import { previewScaffold, scaffold } from '@soleri/forge/lib';
-import type { AgentConfig } from '@soleri/forge/lib';
-import { installPack } from '../hook-packs/installer.js';
-
-describe('create command', { timeout: 30_000 }, () => {
-  let tempDir: string;
-
-  const testConfig: AgentConfig = {
-    id: 'test-agent',
-    name: 'TestAgent',
-    role: 'A test agent',
-    description: 'This agent is used for testing the CLI create command.',
-    domains: ['testing', 'quality'],
-    principles: ['Test everything', 'Quality first'],
-    greeting: 'Hello! I am TestAgent, here to help with testing.',
-    outputDir: '',
-  };
-
-  beforeEach(() => {
-    tempDir = join(tmpdir(), `cli-create-test-${Date.now()}`);
-    mkdirSync(tempDir, { recursive: true });
-    testConfig.outputDir = tempDir;
-  });
-
-  afterEach(() => {
-    rmSync(tempDir, { recursive: true, force: true });
-  });
-
-  it('should preview scaffold without creating files', () => {
-    const preview = previewScaffold(testConfig);
-
-    expect(preview.agentDir).toBe(join(tempDir, 'test-agent'));
-    expect(preview.persona.name).toBe('TestAgent');
-    expect(preview.domains).toEqual(['testing', 'quality']);
-    expect(preview.files.length).toBeGreaterThan(10);
-    expect(existsSync(preview.agentDir)).toBe(false);
-  });
-
-  it('should scaffold agent successfully', () => {
-    const result = scaffold(testConfig);
-
-    expect(result.success).toBe(true);
-    expect(result.agentDir).toBe(join(tempDir, 'test-agent'));
-    expect(result.filesCreated.length).toBeGreaterThan(10);
-    expect(existsSync(join(tempDir, 'test-agent', 'package.json'))).toBe(true);
-    expect(existsSync(join(tempDir, 'test-agent', 'src', 'index.ts'))).toBe(true);
-  });
-
-  it('should fail if directory already exists', () => {
-    scaffold(testConfig);
-    const result = scaffold(testConfig);
-
-    expect(result.success).toBe(false);
-    expect(result.summary).toContain('already exists');
-  });
-
-  it('should not create facade files (v5.0 uses runtime factories from @soleri/core)', () => {
-    scaffold(testConfig);
-
-    // v5.0: facades are created at runtime by createDomainFacades() — no generated files
-    expect(existsSync(join(tempDir, 'test-agent', 'src', 'facades'))).toBe(false);
-
-    // Entry point should reference createDomainFacades
-    const entry = readFileSync(join(tempDir, 'test-agent', 'src', 'index.ts'), 'utf-8');
-    expect(entry).toContain('createDomainFacades');
-    expect(entry).toContain('"testing"');
-    expect(entry).toContain('"quality"');
-  });
-
-  it('should create intelligence data files for each domain', () => {
-    scaffold(testConfig);
-
-    const testingBundle = JSON.parse(
-      readFileSync(
-        join(tempDir, 'test-agent', 'src', 'intelligence', 'data', 'testing.json'),
-        'utf-8',
-      ),
-    );
-    expect(testingBundle.domain).toBe('testing');
-    expect(testingBundle.entries.length).toBeGreaterThanOrEqual(0);
-    if (testingBundle.entries.length > 0) {
-      expect(testingBundle.entries[0].id).toBe('testing-seed');
-      expect(testingBundle.entries[0].tags).toContain('seed');
-    }
-  });
-
-  it('should read config from file for non-interactive mode', () => {
-    const configPath = join(tempDir, 'agent.json');
-    writeFileSync(configPath, JSON.stringify(testConfig), 'utf-8');
-
-    const raw = JSON.parse(readFileSync(configPath, 'utf-8'));
-    expect(raw.id).toBe('test-agent');
-    expect(raw.domains).toEqual(['testing', 'quality']);
-  });
-
-  // ─── Hook pack integration tests ──────────────────────────────
-
-  it('should create .claude/ directory when hookPacks specified', () => {
-    const configWithHooks: AgentConfig = {
-      ...testConfig,
-      hookPacks: ['typescript-safety'],
-    };
-    const result = scaffold(configWithHooks);
-
-    expect(result.success).toBe(true);
-    expect(existsSync(join(tempDir, 'test-agent', '.claude'))).toBe(true);
-  });
-
-  it('should install hookify files to agent .claude/ via installPack', () => {
-    const configWithHooks: AgentConfig = {
-      ...testConfig,
-      hookPacks: ['typescript-safety'],
-    };
-    const result = scaffold(configWithHooks);
-    expect(result.success).toBe(true);
-
-    // Simulate what create.ts does: install packs into agent dir
-    const { installed } = installPack('typescript-safety', { projectDir: result.agentDir });
-    expect(installed.length).toBeGreaterThan(0);
-
-    // Verify hookify files exist in agent .claude/
-    const claudeDir = join(result.agentDir, '.claude');
-    const hookFiles = readdirSync(claudeDir).filter(
-      (f) => f.startsWith('hookify.') && f.endsWith('.local.md'),
-    );
-    expect(hookFiles.length).toBeGreaterThan(0);
-    expect(hookFiles.some((f) => f.includes('no-any-types'))).toBe(true);
-  });
-
-  it('should not create .claude/ when hookPacks is empty or undefined', () => {
-    const result = scaffold(testConfig);
-
-    expect(result.success).toBe(true);
-    expect(existsSync(join(tempDir, 'test-agent', '.claude'))).toBe(false);
-  });
-
-  it('should include hook packs in preview when hookPacks specified', () => {
-    const configWithHooks: AgentConfig = {
-      ...testConfig,
-      hookPacks: ['typescript-safety'],
-    };
-    const preview = previewScaffold(configWithHooks);
-
-    const hookEntry = preview.files.find((f) => f.path === '.claude/');
-    expect(hookEntry).toBeDefined();
-    expect(hookEntry!.description).toContain('typescript-safety');
-  });
-
-  it('should include Hook Packs section in CLAUDE.md when hookPacks specified', () => {
-    const configWithHooks: AgentConfig = {
-      ...testConfig,
-      hookPacks: ['typescript-safety'],
-    };
-    scaffold(configWithHooks);
-
-    const claudeMd = readFileSync(
-      join(tempDir, 'test-agent', 'src', 'activation', 'claude-md-content.ts'),
-      'utf-8',
-    );
-    expect(claudeMd).toContain('Hook Packs');
-    expect(claudeMd).toContain('typescript-safety');
-  });
-
-  it('should not include Hook Packs section in CLAUDE.md when hookPacks undefined', () => {
-    scaffold(testConfig);
-
-    const claudeMd = readFileSync(
-      join(tempDir, 'test-agent', 'src', 'activation', 'claude-md-content.ts'),
-      'utf-8',
-    );
-    expect(claudeMd).not.toContain('Hook Packs');
-  });
-
-  it('should include hook copy logic in setup.sh when hookPacks specified', () => {
-    const configWithHooks: AgentConfig = {
-      ...testConfig,
-      hookPacks: ['typescript-safety'],
-    };
-    scaffold(configWithHooks);
-
-    const setupSh = readFileSync(join(tempDir, 'test-agent', 'scripts', 'setup.sh'), 'utf-8');
-    expect(setupSh).toContain('Installing hook packs');
-    expect(setupSh).toContain('hookify.');
-    expect(setupSh).toContain('GLOBAL_CLAUDE_DIR');
-  });
-
-  it('should not include hook copy logic in setup.sh when hookPacks undefined', () => {
-    scaffold(testConfig);
-
-    const setupSh = readFileSync(join(tempDir, 'test-agent', 'scripts', 'setup.sh'), 'utf-8');
-    expect(setupSh).not.toContain('Installing hook packs');
-  });
-
-  it('should mention hook packs in scaffold summary', () => {
-    const configWithHooks: AgentConfig = {
-      ...testConfig,
-      hookPacks: ['typescript-safety', 'a11y'],
-    };
-    const result = scaffold(configWithHooks);
-
-    expect(result.summary).toContain('2 hook pack(s) bundled in .claude/');
-  });
-});

package/src/hook-packs/yolo-safety/scripts/anti-deletion.sh

@@ -1,214 +0,0 @@
-#!/usr/bin/env bash
-# Anti-Deletion Staging Hook for Claude Code (Soleri Hook Pack: yolo-safety)
-# PreToolUse -> Bash: intercepts rm, rmdir, mv (of project dirs), git clean, reset --hard
-# Copies target files to ~/.soleri/staging/<timestamp>/ then blocks the command.
-#
-# Catastrophic commands (rm -rf /, rm -rf ~) should stay in deny rules —
-# this hook handles targeted deletes only.
-#
-# Dependencies: jq (required), perl (optional, for heredoc stripping)
-
-set -euo pipefail
-
-STAGING_ROOT="$HOME/.soleri/staging"
-PROJECTS_DIR="$HOME/projects"
-INPUT=$(cat)
-
-# Extract the command from stdin JSON
-CMD=$(echo "$INPUT" | jq -r '.tool_input.command // empty' 2>/dev/null)
-
-# No command found — let it through
-if [ -z "$CMD" ]; then
-  exit 0
-fi
-
-# --- Strip heredocs and quoted strings to avoid false positives ---
-# Commands like: gh issue comment --body "$(cat <<'EOF' ... rmdir ... EOF)"
-# contain destructive keywords in text, not as actual commands.
-
-# Remove heredoc blocks: <<'EOF'...EOF and <<EOF...EOF (multiline)
-STRIPPED=$(echo "$CMD" | perl -0777 -pe "s/<<'?\\w+'?.*?^\\w+$//gms" 2>/dev/null || echo "$CMD")
-# Remove double-quoted strings (greedy but good enough for this check)
-STRIPPED=$(echo "$STRIPPED" | sed -E 's/"[^"]*"//g' 2>/dev/null || echo "$STRIPPED")
-# Remove single-quoted strings
-STRIPPED=$(echo "$STRIPPED" | sed -E "s/'[^']*'//g" 2>/dev/null || echo "$STRIPPED")
-
-# --- Detect destructive commands (on stripped command only) ---
-
-IS_RM=false
-IS_RMDIR=false
-IS_MV_PROJECT=false
-IS_GIT_CLEAN=false
-IS_RESET_HARD=false
-IS_GIT_CHECKOUT_DOT=false
-IS_GIT_RESTORE_DOT=false
-
-# Check for rm commands (but not git rm which is safe — it stages, doesn't destroy)
-if echo "$STRIPPED" | grep -qE '(^|\s|;|&&|\|\|)rm\s'; then
-  if ! echo "$STRIPPED" | grep -qE '(^|\s)git\s+rm\s'; then
-    IS_RM=true
-  fi
-fi
-
-# Check for rmdir commands
-if echo "$STRIPPED" | grep -qE '(^|\s|;|&&|\|\|)rmdir\s'; then
-  IS_RMDIR=true
-fi
-
-# Check for mv commands that move project directories or git repos
-if echo "$STRIPPED" | grep -qE '(^|\s|;|&&|\|\|)mv\s'; then
-  MV_SOURCES=$(echo "$STRIPPED" | sed -E 's/^.*\bmv\s+//' | sed -E 's/-(f|i|n|v)\s+//g')
-  if echo "$MV_SOURCES" | grep -qE "(~/projects|$HOME/projects|\\\$HOME/projects|\\.git)"; then
-    IS_MV_PROJECT=true
-  fi
-fi
-
-# Check for git clean
-if echo "$STRIPPED" | grep -qE '(^|\s|;|&&|\|\|)git\s+clean\b'; then
-  IS_GIT_CLEAN=true
-fi
-
-# Check for git reset --hard
-if echo "$STRIPPED" | grep -qE '(^|\s|;|&&|\|\|)git\s+reset\s+--hard'; then
-  IS_RESET_HARD=true
-fi
-
-# Check for git checkout -- . (restores all files, discards changes)
-if echo "$STRIPPED" | grep -qE '(^|\s|;|&&|\|\|)git\s+checkout\s+--\s+\.'; then
-  IS_GIT_CHECKOUT_DOT=true
-fi
-
-# Check for git restore . (restores all files, discards changes)
-if echo "$STRIPPED" | grep -qE '(^|\s|;|&&|\|\|)git\s+restore\s+\.'; then
-  IS_GIT_RESTORE_DOT=true
-fi
-
-# Not a destructive command — let it through
-if [ "$IS_RM" = false ] && [ "$IS_RMDIR" = false ] && [ "$IS_MV_PROJECT" = false ] && \
-   [ "$IS_GIT_CLEAN" = false ] && [ "$IS_RESET_HARD" = false ] && \
-   [ "$IS_GIT_CHECKOUT_DOT" = false ] && [ "$IS_GIT_RESTORE_DOT" = false ]; then
-  exit 0
-fi
-
-# --- Handle git clean (block outright) ---
-
-if [ "$IS_GIT_CLEAN" = true ]; then
-  jq -n '{
-    continue: false,
-    stopReason: "BLOCKED: git clean would remove untracked files. Use git stash --include-untracked to save them first, or ask the user to run git clean manually."
-  }'
-  exit 0
-fi
-
-# --- Handle git reset --hard (block outright) ---
-
-if [ "$IS_RESET_HARD" = true ]; then
-  jq -n '{
-    continue: false,
-    stopReason: "BLOCKED: git reset --hard would discard uncommitted changes. Use git stash to save them first, or ask the user to run this manually."
-  }'
-  exit 0
-fi
-
-# --- Handle git checkout -- . (block outright) ---
-
-if [ "$IS_GIT_CHECKOUT_DOT" = true ]; then
-  jq -n '{
-    continue: false,
-    stopReason: "BLOCKED: git checkout -- . would discard all uncommitted changes. Use git stash to save them first, or ask the user to run this manually."
-  }'
-  exit 0
-fi
-
-# --- Handle git restore . (block outright) ---
-
-if [ "$IS_GIT_RESTORE_DOT" = true ]; then
-  jq -n '{
-    continue: false,
-    stopReason: "BLOCKED: git restore . would discard all uncommitted changes. Use git stash to save them first, or ask the user to run this manually."
-  }'
-  exit 0
-fi
-
-# --- Handle mv of project directories (block outright) ---
-
-if [ "$IS_MV_PROJECT" = true ]; then
-  jq -n '{
-    continue: false,
-    stopReason: "BLOCKED: mv of a project directory or git repo detected. Moving project directories can cause data loss if the operation fails midway. Ask the user to run this manually, or use cp + verify + rm instead."
-  }'
-  exit 0
-fi
-
-# --- Handle rmdir (block outright) ---
-
-if [ "$IS_RMDIR" = true ]; then
-  jq -n '{
-    continue: false,
-    stopReason: "BLOCKED: rmdir detected. Removing directories can break project structure. Ask the user to confirm this operation manually."
-  }'
-  exit 0
-fi
-
-# --- Handle rm commands — copy to staging, then block ---
-
-# Create timestamped staging directory
-TIMESTAMP=$(date +%Y-%m-%d_%H%M%S)
-STAGE_DIR="$STAGING_ROOT/$TIMESTAMP"
-
-# Extract file paths from the rm command
-# Strip rm and its flags, keeping only the file arguments
-FILES=$(echo "$CMD" | sed -E 's/^.*\brm\s+//' | sed -E 's/-(r|f|rf|fr|v|i|rv|fv|rfv|frv)\s+//g' | tr ' ' '\n' | grep -v '^-' | grep -v '^$')
-
-if [ -z "$FILES" ]; then
-  jq -n '{
-    continue: false,
-    stopReason: "BLOCKED: rm command detected but could not parse file targets. Please specify files explicitly."
-  }'
-  exit 0
-fi
-
-STAGED=()
-MISSING=()
-
-mkdir -p "$STAGE_DIR"
-
-while IFS= read -r filepath; do
-  # Expand path (handle ~, relative paths)
-  expanded=$(eval echo "$filepath" 2>/dev/null || echo "$filepath")
-
-  if [ -e "$expanded" ]; then
-    # Preserve directory structure in staging
-    target_dir="$STAGE_DIR/$(dirname "$expanded")"
-    mkdir -p "$target_dir"
-    # COPY instead of MOVE — originals stay intact, staging is a backup
-    if [ -d "$expanded" ]; then
-      cp -R "$expanded" "$target_dir/" 2>/dev/null && STAGED+=("$expanded") || MISSING+=("$expanded")
-    else
-      cp "$expanded" "$target_dir/" 2>/dev/null && STAGED+=("$expanded") || MISSING+=("$expanded")
-    fi
-  else
-    MISSING+=("$expanded")
-  fi
-done <<< "$FILES"
-
-# Build response
-STAGED_COUNT=${#STAGED[@]}
-MISSING_COUNT=${#MISSING[@]}
-
-if [ "$STAGED_COUNT" -eq 0 ] && [ "$MISSING_COUNT" -gt 0 ]; then
-  # All files were missing — let the rm fail naturally
-  rmdir "$STAGE_DIR" 2>/dev/null || true
-  exit 0
-fi
-
-STAGED_LIST=$(printf '%s, ' "${STAGED[@]}" | sed 's/, $//')
-
-jq -n \
-  --arg staged "$STAGED_LIST" \
-  --arg dir "$STAGE_DIR" \
-  --argjson count "$STAGED_COUNT" \
-  '{
-    continue: false,
-    stopReason: ("BLOCKED & BACKED UP: " + ($count | tostring) + " item(s) copied to " + $dir + " — files: " + $staged + ". The originals are untouched. To proceed with deletion, ask the user to run the rm command manually.")
-  }'