@covibes/zeroshot 5.3.0 → 5.4.1

package/scripts/record-demo.sh

@@ -1,279 +0,0 @@
-#!/bin/bash
-# record-demo.sh - Create ephemeral project, run zeroshot demo, record it, cleanup
-#
-# Usage:
-#   ./scripts/record-demo.sh              # Interactive mode (you run zeroshot manually)
-#   ./scripts/record-demo.sh --record     # Record with asciinema automatically
-#
-# Output: zeroshot-demo.cast (asciinema recording)
-# Convert to gif: agg zeroshot-demo.cast zeroshot-demo.gif --idle-time-limit 2
-
-set -e
-
-SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
-REPO_ROOT="$(cd "$SCRIPT_DIR/.." && pwd)"
-DEMO_DIR=""
-RECORD_MODE=false
-FORCE_MODE=false
-LOCK_FILE="/tmp/zeroshot-demo-recording.lock"
-CAST_FILE="$REPO_ROOT/zeroshot-demo.cast"
-
-# Check for existing recording session
-check_existing_session() {
-    # Check lock file
-    if [[ -f "$LOCK_FILE" ]]; then
-        local old_pid
-        old_pid=$(cat "$LOCK_FILE" 2>/dev/null || echo "")
-        if [[ -n "$old_pid" ]] && kill -0 "$old_pid" 2>/dev/null; then
-            echo "ERROR: Recording already in progress (PID $old_pid)"
-            echo "Kill it with: kill $old_pid"
-            exit 1
-        fi
-        # Stale lock file
-        rm -f "$LOCK_FILE"
-    fi
-
-    # Check for any asciinema processes recording to our file
-    local existing_asciinema
-    existing_asciinema=$(pgrep -f "asciinema.*zeroshot-demo.cast" 2>/dev/null || true)
-    if [[ -n "$existing_asciinema" ]]; then
-        echo "ERROR: Existing asciinema process(es) found: $existing_asciinema"
-        echo "Kill them with: pkill -f 'asciinema.*zeroshot-demo.cast'"
-        exit 1
-    fi
-
-    # Check for any running zeroshot clusters
-    local existing_zeroshot
-    existing_zeroshot=$(pgrep -f "zeroshot.*rate limiting" 2>/dev/null || true)
-    if [[ -n "$existing_zeroshot" ]]; then
-        echo "ERROR: Existing zeroshot process(es) found: $existing_zeroshot"
-        echo "Kill them with: pkill -f 'zeroshot.*rate limiting'"
-        exit 1
-    fi
-}
-
-# Parse args
-while [[ $# -gt 0 ]]; do
-    case "$1" in
-        --record) RECORD_MODE=true; shift ;;
-        --force|-f) FORCE_MODE=true; shift ;;
-        *) echo "Unknown option: $1"; exit 1 ;;
-    esac
-done
-
-# Cleanup on exit
-cleanup() {
-    # Remove lock file
-    rm -f "$LOCK_FILE"
-
-    if [[ -n "$DEMO_DIR" && -d "$DEMO_DIR" ]]; then
-        echo ""
-        echo "Cleaning up $DEMO_DIR..."
-        rm -rf "$DEMO_DIR"
-        echo "Done."
-    fi
-}
-trap cleanup EXIT
-
-# Check for conflicts before doing anything
-check_existing_session
-
-# Create temp project
-DEMO_DIR=$(mktemp -d -t zeroshot-demo-XXXXXX)
-echo "Creating demo project in $DEMO_DIR"
-
-cd "$DEMO_DIR"
-
-# Initialize package.json
-cat > package.json << 'EOF'
-{
-  "name": "demo-api",
-  "version": "1.0.0",
-  "type": "commonjs",
-  "scripts": {
-    "dev": "ts-node src/index.ts",
-    "build": "tsc",
-    "start": "node dist/index.js"
-  }
-}
-EOF
-
-# Install dependencies (quiet)
-echo "Installing dependencies..."
-npm install --silent express typescript ts-node @types/express @types/node
-
-# Create tsconfig
-cat > tsconfig.json << 'EOF'
-{
-  "compilerOptions": {
-    "target": "ES2020",
-    "module": "commonjs",
-    "strict": true,
-    "esModuleInterop": true,
-    "outDir": "dist",
-    "rootDir": "src"
-  },
-  "include": ["src/**/*"]
-}
-EOF
-
-# Create minimal server with users "database"
-mkdir -p src
-
-cat > src/db.ts << 'EOF'
-// Simple in-memory database
-export interface User {
-  id: number;
-  username: string;
-  email: string;
-  password_hash: string;
-  avatar: string;
-  created_at: Date;
-}
-
-export const users: User[] = [
-  {
-    id: 1,
-    username: "alice",
-    email: "alice@example.com",
-    password_hash: "$2b$10$X7VYKzPQ...",
-    avatar: "https://example.com/alice.jpg",
-    created_at: new Date("2024-01-15"),
-  },
-  {
-    id: 2,
-    username: "bob",
-    email: "bob@example.com",
-    password_hash: "$2b$10$Y8WZLaQR...",
-    avatar: "https://example.com/bob.jpg",
-    created_at: new Date("2024-02-20"),
-  },
-];
-EOF
-
-cat > src/index.ts << 'EOF'
-import express from "express";
-
-const app = express();
-const PORT = 3000;
-
-app.use(express.json());
-
-// Health check
-app.get("/health", (_, res) => {
-  res.json({ status: "ok" });
-});
-
-app.listen(PORT, () => {
-  console.log(`Server running on http://localhost:${PORT}`);
-});
-EOF
-
-# Initialize git repo
-git init --quiet
-git add -A
-git commit --quiet -m "Initial commit: Express API with users database"
-
-echo ""
-echo "=========================================="
-echo "Demo project ready!"
-echo "=========================================="
-echo ""
-echo "Directory: $DEMO_DIR"
-echo ""
-echo "Suggested demo task:"
-echo "  zeroshot 'Add PUT /users/:id endpoint to update user profile'"
-echo ""
-
-if [[ "$RECORD_MODE" == "true" ]]; then
-    echo "Recording with asciinema..."
-    echo ""
-
-    # Check asciinema is installed
-    if ! command -v asciinema &> /dev/null; then
-        echo "Error: asciinema not installed. Run: pip install asciinema"
-        exit 1
-    fi
-
-    # Warn if cast file exists (skip with --force)
-    if [[ -f "$CAST_FILE" ]] && [[ "$FORCE_MODE" != "true" ]]; then
-        echo "WARNING: $CAST_FILE already exists!"
-        echo "Previous recording will be OVERWRITTEN."
-        echo ""
-        read -p "Continue? [y/N] " -n 1 -r
-        echo ""
-        if [[ ! $REPLY =~ ^[Yy]$ ]]; then
-            echo "Aborted."
-            exit 1
-        fi
-    fi
-
-    # Create lock file with our PID
-    echo $$ > "$LOCK_FILE"
-
-    # Record the session
-    # CRITICAL: Use bash -l -c to get full PATH (includes npm global binaries)
-    ZEROSHOT_PATH=$(which zeroshot)
-    TASK="Add rate limiting middleware: sliding window algorithm (not fixed window), per-IP tracking with in-memory store and automatic TTL cleanup to prevent memory leaks, configurable limits per endpoint. Return 429 Too Many Requests with Retry-After header (seconds until reset) and X-RateLimit-Remaining header on ALL responses. Must handle both IPv4 and IPv6, normalizing IPv6 to consistent format."
-
-    # SIGNAL ISOLATION: Use setsid to create new session, completely immune to terminal signals
-    # The recording process will NOT be killed by Ctrl+C or signals to this script
-    # We wait for it explicitly and only cleanup AFTER it naturally completes
-    echo "Starting recording in isolated session (immune to Ctrl+C)..."
-    echo "To kill it manually: pkill -f 'asciinema.*zeroshot-demo.cast'"
-    echo ""
-
-    # Disable cleanup trap during recording - we'll handle it manually
-    trap - EXIT
-
-    # Start asciinema in new session (setsid) so it's immune to our signals
-    # Save the session leader PID so we can wait for it
-    setsid bash -c "
-        # Ignore all signals - this recording WILL NOT DIE
-        trap '' INT TERM HUP QUIT
-
-        cd '$DEMO_DIR'
-        asciinema rec \
-            --overwrite \
-            --title 'Zeroshot Demo' \
-            --command \"bash -l -c '$ZEROSHOT_PATH \\\"$TASK\\\"'\" \
-            '$CAST_FILE'
-    " &
-    RECORDING_PID=$!
-
-    # Update lock file with the actual recording PID
-    echo $RECORDING_PID > "$LOCK_FILE"
-
-    echo "Recording started (session PID: $RECORDING_PID)"
-    echo "Waiting for recording to complete..."
-    echo ""
-
-    # Wait for recording to finish - this script will NOT kill it on Ctrl+C
-    # Ignore signals while waiting
-    trap '' INT TERM HUP
-    wait $RECORDING_PID 2>/dev/null || true
-
-    # Recording finished naturally - now cleanup
-    echo ""
-    echo "Recording saved to: $CAST_FILE"
-    echo ""
-    echo "Convert to gif with:"
-    echo "  agg $CAST_FILE $REPO_ROOT/zeroshot-demo.gif --idle-time-limit 2"
-
-    # Now do cleanup
-    rm -f "$LOCK_FILE"
-    if [[ -n "$DEMO_DIR" && -d "$DEMO_DIR" ]]; then
-        echo ""
-        echo "Cleaning up $DEMO_DIR..."
-        rm -rf "$DEMO_DIR"
-        echo "Done."
-    fi
-else
-    echo "Interactive mode - run zeroshot manually:"
-    echo ""
-    echo "  cd $DEMO_DIR"
-    echo "  zeroshot 'Add PUT /users/:id endpoint to update user profile'"
-    echo ""
-    echo "Press Enter when done to cleanup..."
-    read -r
-fi

package/scripts/test-install.sh

@@ -1,40 +0,0 @@
-#!/bin/bash
-# Test zeroshot installation on any platform
-# Run: ./scripts/test-install.sh
-
-set -e
-
-echo "=== Zeroshot Install Test ==="
-echo "Platform: $(uname -s) $(uname -m)"
-echo "Node: $(node --version)"
-echo "npm: $(npm --version)"
-echo ""
-
-echo "1. Installing dependencies..."
-npm install
-
-echo ""
-echo "2. Linking CLI..."
-npm link
-
-echo ""
-echo "3. Testing CLI commands..."
-
-echo "   zeroshot --version"
-zeroshot --version
-
-echo ""
-echo "   zeroshot --help"
-zeroshot --help | head -20
-
-echo ""
-echo "   zeroshot list"
-zeroshot list 2>/dev/null || echo "   (no clusters yet - this is expected)"
-
-echo ""
-echo "   zeroshot config list"
-zeroshot config list
-
-echo ""
-echo "=== ALL TESTS PASSED ==="
-echo "Zeroshot installed successfully on $(uname -s)!"

package/scripts/validate-templates.js

@@ -1,100 +0,0 @@
-#!/usr/bin/env node
-/**
- * Validate all cluster templates for config errors
- * Run in CI to prevent broken templates from being merged
- *
- * Usage: node scripts/validate-templates.js
- * Exit codes: 0 = all valid, 1 = validation errors found
- */
-
-const fs = require('fs');
-const path = require('path');
-const { validateConfig } = require('../src/config-validator');
-
-const TEMPLATES_DIR = path.join(__dirname, '../cluster-templates');
-
-function findJsonFiles(dir) {
-  const files = [];
-  if (!fs.existsSync(dir)) return files;
-
-  for (const entry of fs.readdirSync(dir, { withFileTypes: true })) {
-    const fullPath = path.join(dir, entry.name);
-    if (entry.isDirectory()) {
-      files.push(...findJsonFiles(fullPath));
-    } else if (entry.name.endsWith('.json')) {
-      files.push(fullPath);
-    }
-  }
-  return files;
-}
-
-function validateTemplate(filePath) {
-  const relativePath = path.relative(process.cwd(), filePath);
-
-  try {
-    const content = fs.readFileSync(filePath, 'utf-8');
-    const config = JSON.parse(content);
-
-    // Skip non-cluster configs (like package.json)
-    if (!config.agents && !config.name) {
-      return { valid: true, skipped: true };
-    }
-
-    const result = validateConfig(config);
-
-    if (!result.valid) {
-      console.error(`\n❌ ${relativePath}`);
-      for (const error of result.errors) {
-        console.error(`   ERROR: ${error}`);
-      }
-    } else if (result.warnings.length > 0) {
-      console.warn(`\n⚠️  ${relativePath}`);
-      for (const warning of result.warnings) {
-        console.warn(`   WARN: ${warning}`);
-      }
-    } else {
-      console.log(`✓ ${relativePath}`);
-    }
-
-    return result;
-  } catch (err) {
-    console.error(`\n❌ ${relativePath}`);
-    console.error(`   PARSE ERROR: ${err.message}`);
-    return { valid: false, errors: [err.message], warnings: [] };
-  }
-}
-
-function main() {
-  console.log('Validating cluster templates...\n');
-
-  const templateFiles = [...findJsonFiles(TEMPLATES_DIR)];
-
-  let hasErrors = false;
-  let validated = 0;
-  let skipped = 0;
-
-  for (const file of templateFiles) {
-    const result = validateTemplate(file);
-    if (result.skipped) {
-      skipped++;
-    } else {
-      validated++;
-      if (!result.valid) {
-        hasErrors = true;
-      }
-    }
-  }
-
-  console.log(`\n${'='.repeat(60)}`);
-  console.log(`Validated: ${validated} templates, Skipped: ${skipped} files`);
-
-  if (hasErrors) {
-    console.error('\n❌ VALIDATION FAILED - Fix errors above before merging\n');
-    process.exit(1);
-  } else {
-    console.log('\n✓ All templates valid\n');
-    process.exit(0);
-  }
-}
-
-main();

package/src/agent/agent-config.js

@@ -1,193 +0,0 @@
-/**
- * AgentConfig - Agent configuration validation and defaults
- *
- * Provides:
- * - Config validation and normalization
- * - Default values for optional fields
- * - Model configuration setup
- * - Safety checks for test mode
- * - maxModel ceiling enforcement at config time
- */
-
-const { loadSettings, validateModelAgainstMax, VALID_MODELS } = require('../../lib/settings');
-
-const VALID_LEVELS = ['level1', 'level2', 'level3'];
-
-// Default max iterations (high limit - let the user decide when to give up)
-const DEFAULT_MAX_ITERATIONS = 100;
-
-// Default timeout: 0 = no timeout (task runs until completion or explicit kill)
-// Use positive number for timeout in milliseconds
-const DEFAULT_TIMEOUT = 0;
-
-// Stale detection - ENABLED by default using multi-indicator analysis (safe from false positives)
-// Multi-indicator approach checks: process state, CPU usage, context switches, network I/O
-// Only flags as stuck when ALL indicators show inactivity (score >= 3.5)
-// Single-indicator detection (just output freshness) was too risky - this is safe.
-const DEFAULT_STALE_DURATION_MS = 30 * 60 * 1000; // 30 minutes before triggering analysis
-const DEFAULT_LIVENESS_CHECK_ENABLED = true; // Safe with multi-indicator detection
-
-/**
- * Validate and normalize agent configuration
- * @param {Object} config - Raw agent configuration
- * @param {Object} options - Agent wrapper options
- * @returns {Object} Normalized configuration
- */
-function validateAgentConfig(config, options = {}) {
-  // CRITICAL: Enforce JSON schema output by default to prevent parse failures and crashes
-  // Agents MUST return structured output so hooks can safely use {{result.*}} templates
-  if (!config.outputFormat) {
-    config.outputFormat = 'json';
-  }
-
-  // If outputFormat is json but no schema defined, use a minimal default schema
-  // This prevents uncaught exceptions when parsing agent output
-  if (config.outputFormat === 'json' && !config.jsonSchema) {
-    config.jsonSchema = {
-      type: 'object',
-      properties: {
-        summary: {
-          type: 'string',
-          description: 'Brief summary of what was done',
-        },
-        result: {
-          type: 'string',
-          description: 'Detailed result or output',
-        },
-      },
-      required: ['summary', 'result'],
-    };
-  }
-
-  // Model configuration: support both static model and dynamic rules
-  // If no model specified, model is null - _selectModel() will use provider defaults
-  let modelConfig;
-  if (config.modelRules) {
-    modelConfig = { type: 'rules', rules: config.modelRules };
-  } else {
-    modelConfig = {
-      type: 'static',
-      model: config.model || null,
-      modelLevel: config.modelLevel || null,
-    };
-  }
-
-  // COST CEILING/FLOOR ENFORCEMENT: Validate model(s) against maxModel and minModel at config time
-  // Catches violations EARLY (config load) instead of at runtime (iteration N)
-  const settings = loadSettings();
-  const maxModel = settings.maxModel || 'sonnet';
-  const minModel = settings.minModel || null;
-
-  if (modelConfig.type === 'static') {
-    if (modelConfig.model && VALID_MODELS.includes(modelConfig.model)) {
-      // Static model: validate once (legacy Claude models only)
-      try {
-        validateModelAgainstMax(modelConfig.model, maxModel, minModel);
-      } catch (error) {
-        throw new Error(`Agent "${config.id}": ${error.message}`);
-      }
-    }
-
-    if (modelConfig.modelLevel && !VALID_LEVELS.includes(modelConfig.modelLevel)) {
-      throw new Error(
-        `Agent "${config.id}": invalid modelLevel "${modelConfig.modelLevel}". ` +
-          `Valid: ${VALID_LEVELS.join(', ')}`
-      );
-    }
-  } else if (modelConfig.type === 'rules') {
-    // Dynamic rules: validate ALL rules upfront (don't wait until iteration N)
-    for (const rule of modelConfig.rules) {
-      if (rule.model && VALID_MODELS.includes(rule.model)) {
-        try {
-          validateModelAgainstMax(rule.model, maxModel, minModel);
-        } catch {
-          throw new Error(
-            `Agent "${config.id}": modelRule "${rule.iterations}" requests "${rule.model}" ` +
-              `but maxModel is "${maxModel}"${minModel ? ` and minModel is "${minModel}"` : ''}. ` +
-              `Either adjust the rule's model or change maxModel/minModel settings.`
-          );
-        }
-      }
-
-      if (rule.modelLevel && !VALID_LEVELS.includes(rule.modelLevel)) {
-        throw new Error(
-          `Agent "${config.id}": modelRule "${rule.iterations}" has invalid modelLevel ` +
-            `"${rule.modelLevel}". Valid: ${VALID_LEVELS.join(', ')}`
-        );
-      }
-    }
-  }
-
-  // Prompt configuration: support static prompt OR iteration-based rules
-  // Formats:
-  //   prompt: "string"                              -> static
-  //   prompt: { system: "string" }                  -> static
-  //   prompt: { initial: "...", subsequent: "..." } -> iteration 1 vs 2+
-  //   prompt: { iterations: [...] }                 -> full control
-  let promptConfig = null;
-  if (config.prompt?.iterations) {
-    promptConfig = { type: 'rules', rules: config.prompt.iterations };
-  } else if (config.prompt?.initial || config.prompt?.subsequent) {
-    const rules = [];
-    if (config.prompt.initial) rules.push({ match: '1', system: config.prompt.initial });
-    if (config.prompt.subsequent) rules.push({ match: '2+', system: config.prompt.subsequent });
-    promptConfig = { type: 'rules', rules };
-  } else if (typeof config.prompt === 'string') {
-    promptConfig = { type: 'static', system: config.prompt };
-  } else if (config.prompt?.system) {
-    promptConfig = { type: 'static', system: config.prompt.system };
-  } else if (config.prompt) {
-    throw new Error(`Agent "${config.id}": invalid prompt format`);
-  }
-
-  // Default timeout to 0 (no timeout) if not specified
-  // Use positive number for timeout in milliseconds
-  // ROBUST: Handle undefined, null, AND string values from template resolution
-  if (config.timeout === undefined || config.timeout === null || config.timeout === '') {
-    config.timeout = DEFAULT_TIMEOUT;
-  } else {
-    // Coerce to number (handles string "0" from template resolution)
-    config.timeout = Number(config.timeout);
-  }
-  if (!Number.isFinite(config.timeout) || config.timeout < 0) {
-    throw new Error(
-      `Agent "${config.id}": timeout must be a non-negative number (got ${config.timeout}).`
-    );
-  }
-
-  // Build normalized config
-  const normalizedConfig = {
-    ...config,
-    modelConfig,
-    promptConfig,
-    maxIterations: config.maxIterations || DEFAULT_MAX_ITERATIONS,
-    timeout: config.timeout, // Defaults to 0 (no timeout) if not specified
-    staleDuration: config.staleDuration || DEFAULT_STALE_DURATION_MS,
-    enableLivenessCheck: config.enableLivenessCheck ?? DEFAULT_LIVENESS_CHECK_ENABLED, // On by default, opt-out with false
-  };
-
-  // SAFETY: In test mode, verify mock is provided for agents that execute tasks
-  // Check if this agent executes tasks (vs orchestrator agents that only publish messages)
-  const executesTask = config.triggers?.some(
-    (trigger) => !trigger.action || trigger.action === 'execute_task'
-  );
-
-  // Accept either mockSpawnFn OR taskRunner as valid mock sources
-  const hasMock = options.mockSpawnFn || options.taskRunner;
-
-  if (options.testMode && !hasMock && executesTask) {
-    throw new Error(
-      `AgentWrapper: testMode=true but no mockSpawnFn/taskRunner provided for agent '${config.id}'. ` +
-        `This would cause real Claude API calls. ABORTING.`
-    );
-  }
-
-  return normalizedConfig;
-}
-
-module.exports = {
-  validateAgentConfig,
-  DEFAULT_MAX_ITERATIONS,
-  DEFAULT_STALE_DURATION_MS,
-  DEFAULT_LIVENESS_CHECK_ENABLED,
-};