claude-flow-novice 2.5.2 → 2.8.1

package/.claude/skills/webapp-testing/init-storage.sh

@@ -0,0 +1,150 @@
+#!/bin/bash
+# Webapp Testing Skill - Initialize Storage
+# Purpose: Create SQLite schema and directory structure for screenshot management
+
+set -e
+
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+PROJECT_ROOT="$(cd "$SCRIPT_DIR/../../.." && pwd)"
+cd "$PROJECT_ROOT"
+
+FORCE=false
+
+# Parse arguments
+while [[ $# -gt 0 ]]; do
+  case $1 in
+    --force) FORCE=true; shift ;;
+    *) echo "Unknown parameter: $1"; exit 1 ;;
+  esac
+done
+
+DB_PATH="${HOME}/.claude/memory/adaptive-context.db"
+
+echo "Initializing webapp-testing storage..." >&2
+
+# Create directory structure
+echo "Creating directory structure..." >&2
+mkdir -p .screenshots/baselines
+mkdir -p .screenshots/current
+mkdir -p .screenshots/diffs
+mkdir -p .screenshots/archive
+echo "✅ Directories created" >&2
+
+# Check if tables already exist
+TABLES_EXIST=$(sqlite3 "$DB_PATH" \
+  "SELECT COUNT(*) FROM sqlite_master WHERE type='table' AND name IN ('webapp_screenshots', 'screenshot_audit_log');" 2>/dev/null || echo "0")
+
+if [ "$TABLES_EXIST" -eq 2 ] && [ "$FORCE" = false ]; then
+  echo "⚠️  SQLite tables already exist. Use --force to recreate (will drop existing data)" >&2
+  exit 0
+fi
+
+if [ "$FORCE" = true ] && [ "$TABLES_EXIST" -gt 0 ]; then
+  echo "⚠️  --force flag set. Dropping existing tables..." >&2
+  sqlite3 "$DB_PATH" <<EOF
+DROP TABLE IF EXISTS webapp_screenshots;
+DROP TABLE IF EXISTS screenshot_audit_log;
+EOF
+fi
+
+# Create SQLite schema
+echo "Creating SQLite schema..." >&2
+sqlite3 "$DB_PATH" <<'EOF'
+-- Main screenshots table
+CREATE TABLE IF NOT EXISTS webapp_screenshots (
+  id INTEGER PRIMARY KEY AUTOINCREMENT,
+  screenshot_key TEXT UNIQUE NOT NULL,
+  project TEXT NOT NULL,
+  component TEXT NOT NULL,
+  viewport TEXT NOT NULL,
+  state TEXT NOT NULL,
+  variant TEXT NOT NULL DEFAULT 'default',
+  file_path TEXT NOT NULL,
+  file_hash TEXT NOT NULL,
+  baseline BOOLEAN DEFAULT 0,
+  captured_at INTEGER NOT NULL,
+  task_id TEXT,
+  agent_id TEXT,
+  metadata TEXT,
+  UNIQUE(project, component, viewport, state, variant, baseline)
+);
+
+-- Indexes for efficient querying
+CREATE INDEX IF NOT EXISTS idx_baseline ON webapp_screenshots(project, component, viewport, state, variant, baseline);
+CREATE INDEX IF NOT EXISTS idx_task ON webapp_screenshots(task_id);
+CREATE INDEX IF NOT EXISTS idx_component ON webapp_screenshots(project, component);
+CREATE INDEX IF NOT EXISTS idx_captured ON webapp_screenshots(captured_at DESC);
+CREATE INDEX IF NOT EXISTS idx_screenshot_key ON webapp_screenshots(screenshot_key);
+
+-- Audit log table
+CREATE TABLE IF NOT EXISTS screenshot_audit_log (
+  id INTEGER PRIMARY KEY AUTOINCREMENT,
+  screenshot_key TEXT NOT NULL,
+  action TEXT NOT NULL,
+  reason TEXT,
+  old_file_hash TEXT,
+  new_file_hash TEXT,
+  approved_by TEXT,
+  created_at INTEGER NOT NULL,
+  task_id TEXT,
+  metadata TEXT
+);
+
+-- Indexes for audit log
+CREATE INDEX IF NOT EXISTS idx_audit_screenshot ON screenshot_audit_log(screenshot_key);
+CREATE INDEX IF NOT EXISTS idx_audit_created ON screenshot_audit_log(created_at DESC);
+CREATE INDEX IF NOT EXISTS idx_audit_task ON screenshot_audit_log(task_id);
+CREATE INDEX IF NOT EXISTS idx_audit_action ON screenshot_audit_log(action);
+EOF
+
+echo "✅ SQLite schema created" >&2
+
+# Verify tables created
+VERIFY=$(sqlite3 "$DB_PATH" \
+  "SELECT COUNT(*) FROM sqlite_master WHERE type='table' AND name IN ('webapp_screenshots', 'screenshot_audit_log');" 2>/dev/null)
+
+if [ "$VERIFY" -eq 2 ]; then
+  echo "✅ Verification passed: 2 tables created" >&2
+else
+  echo "❌ Verification failed: Expected 2 tables, found $VERIFY" >&2
+  exit 1
+fi
+
+# Create .gitignore for ephemeral directories
+cat > .screenshots/.gitignore <<'EOF'
+# Ephemeral screenshot directories (not version controlled)
+current/
+diffs/
+archive/
+
+# Version control baselines only
+!baselines/
+EOF
+
+echo "✅ .gitignore created for .screenshots/" >&2
+
+# Output summary
+cat <<EOF
+
+✅ Webapp-testing storage initialized successfully
+
+Directory Structure:
+  .screenshots/
+  ├── baselines/    (version controlled - reference images)
+  ├── current/      (ephemeral - test captures)
+  ├── diffs/        (ephemeral - comparison outputs)
+  └── archive/      (ephemeral - historical captures)
+
+SQLite Tables:
+  - webapp_screenshots      (screenshot metadata and file references)
+  - screenshot_audit_log    (baseline change audit trail)
+
+Database Location:
+  ${DB_PATH}
+
+Next Steps:
+  1. Capture screenshot:  ./.claude/skills/webapp-testing/capture-screenshot.sh --help
+  2. Set baseline:        ./.claude/skills/webapp-testing/set-baseline.sh --help
+  3. Compare screenshots: ./.claude/skills/webapp-testing/compare-screenshots.sh --help
+
+EOF

package/.claude/skills/webapp-testing/set-baseline.sh

@@ -0,0 +1,196 @@
+#!/bin/bash
+# Webapp Testing Skill - Set Baseline
+# Purpose: Set a captured screenshot as the baseline reference for future comparisons
+
+set -e
+
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+PROJECT_ROOT="$(cd "$SCRIPT_DIR/../../.." && pwd)"
+cd "$PROJECT_ROOT"
+
+REASON=""
+
+# Parse arguments
+while [[ $# -gt 0 ]]; do
+  case $1 in
+    --screenshot-key) SCREENSHOT_KEY="$2"; shift 2 ;;
+    --reason) REASON="$2"; shift 2 ;;
+    --current-file) CURRENT_FILE="$2"; shift 2 ;;  # Optional: explicit file path
+    *) echo "Unknown parameter: $1"; exit 1 ;;
+  esac
+done
+
+# Validate required parameters
+if [ -z "$SCREENSHOT_KEY" ]; then
+  echo "Error: Missing required parameter --screenshot-key" >&2
+  echo "Usage: $0 --screenshot-key <key> [--reason <reason>] [--current-file <path>]" >&2
+  echo "" >&2
+  echo "Required:" >&2
+  echo "  --screenshot-key   Screenshot identifier (e.g., 'auth-system/login-form/1920x1080/default/light-mode')" >&2
+  echo "" >&2
+  echo "Optional:" >&2
+  echo "  --reason           Reason for setting baseline (for audit trail)" >&2
+  echo "  --current-file     Explicit path to screenshot file (default: auto-detect latest)" >&2
+  exit 1
+fi
+
+DB_PATH="${HOME}/.claude/memory/adaptive-context.db"
+
+# Parse screenshot key components
+IFS='/' read -r PROJECT COMPONENT VIEWPORT STATE VARIANT <<< "$SCREENSHOT_KEY"
+
+if [ -z "$PROJECT" ] || [ -z "$COMPONENT" ] || [ -z "$VIEWPORT" ] || [ -z "$STATE" ] || [ -z "$VARIANT" ]; then
+  echo "Error: Invalid screenshot key format. Expected: project/component/viewport/state/variant" >&2
+  exit 1
+fi
+
+# Get current screenshot from SQLite (if not explicitly provided)
+if [ -z "$CURRENT_FILE" ]; then
+  CURRENT_PATH=$(sqlite3 "$DB_PATH" \
+    "SELECT file_path FROM webapp_screenshots WHERE screenshot_key = '${SCREENSHOT_KEY}' AND baseline = 0 ORDER BY captured_at DESC LIMIT 1" 2>/dev/null)
+
+  if [ -z "$CURRENT_PATH" ]; then
+    echo "Error: No current screenshot found for key: $SCREENSHOT_KEY" >&2
+    echo "Hint: Run capture-screenshot.sh first to create a capture" >&2
+    exit 1
+  fi
+else
+  CURRENT_PATH="$CURRENT_FILE"
+fi
+
+# Verify current screenshot exists
+if [ ! -f "$CURRENT_PATH" ]; then
+  echo "Error: Current screenshot file not found: $CURRENT_PATH" >&2
+  exit 1
+fi
+
+# Create baseline directory structure
+BASELINE_DIR=".screenshots/baselines/${PROJECT}/${COMPONENT}/${VIEWPORT}/${STATE}"
+mkdir -p "$BASELINE_DIR"
+BASELINE_PATH="${BASELINE_DIR}/${VARIANT}.png"
+
+echo "Setting baseline:" >&2
+echo "  Source:   $CURRENT_PATH" >&2
+echo "  Baseline: $BASELINE_PATH" >&2
+
+# Check if baseline already exists
+if [ -f "$BASELINE_PATH" ]; then
+  echo "Warning: Baseline already exists at $BASELINE_PATH" >&2
+  echo "This will overwrite the existing baseline. Continue? (y/n)" >&2
+  read -r CONFIRM
+  if [ "$CONFIRM" != "y" ] && [ "$CONFIRM" != "Y" ]; then
+    echo "Aborted" >&2
+    exit 1
+  fi
+
+  # Archive old baseline
+  ARCHIVE_DIR=".screenshots/archive/$(date +%Y-%m)"
+  ARCHIVE_PATH="${ARCHIVE_DIR}/${PROJECT}/${COMPONENT}/${VIEWPORT}/${STATE}/${VARIANT}_$(date +%Y%m%d%H%M%S).png"
+  mkdir -p "$(dirname "$ARCHIVE_PATH")"
+  cp "$BASELINE_PATH" "$ARCHIVE_PATH"
+  echo "  Archived: $ARCHIVE_PATH" >&2
+
+  # Get old hash for audit log
+  OLD_FILE_HASH=$(sha256sum "$BASELINE_PATH" | awk '{print $1}')
+else
+  OLD_FILE_HASH=""
+fi
+
+# Copy to baselines directory (remove timestamp from filename)
+cp "$CURRENT_PATH" "$BASELINE_PATH"
+
+# Calculate file hash
+FILE_HASH=$(sha256sum "$BASELINE_PATH" | awk '{print $1}')
+
+# Get metadata from current screenshot
+METADATA=$(sqlite3 "$DB_PATH" \
+  "SELECT metadata FROM webapp_screenshots WHERE file_path = '${CURRENT_PATH}' LIMIT 1" 2>/dev/null || echo '{}')
+
+TASK_ID=$(sqlite3 "$DB_PATH" \
+  "SELECT task_id FROM webapp_screenshots WHERE file_path = '${CURRENT_PATH}' LIMIT 1" 2>/dev/null || echo '')
+
+AGENT_ID=$(sqlite3 "$DB_PATH" \
+  "SELECT agent_id FROM webapp_screenshots WHERE file_path = '${CURRENT_PATH}' LIMIT 1" 2>/dev/null || echo '')
+
+# Update SQLite (set as baseline)
+# First check if baseline entry already exists
+BASELINE_EXISTS=$(sqlite3 "$DB_PATH" \
+  "SELECT COUNT(*) FROM webapp_screenshots WHERE screenshot_key = '${SCREENSHOT_KEY}' AND baseline = 1" 2>/dev/null || echo "0")
+
+if [ "$BASELINE_EXISTS" -gt 0 ]; then
+  # Update existing baseline
+  sqlite3 "$DB_PATH" <<EOF
+UPDATE webapp_screenshots
+SET file_path = '${BASELINE_PATH}',
+    file_hash = 'sha256:${FILE_HASH}',
+    captured_at = $(date +%s),
+    task_id = '${TASK_ID}',
+    agent_id = '${AGENT_ID}',
+    metadata = '${METADATA}'
+WHERE screenshot_key = '${SCREENSHOT_KEY}' AND baseline = 1;
+EOF
+  ACTION="updated"
+else
+  # Insert new baseline entry
+  sqlite3 "$DB_PATH" <<EOF
+INSERT INTO webapp_screenshots (
+  screenshot_key, project, component, viewport, state, variant,
+  file_path, file_hash, baseline, captured_at, task_id, agent_id, metadata
+) VALUES (
+  '${SCREENSHOT_KEY}',
+  '${PROJECT}',
+  '${COMPONENT}',
+  '${VIEWPORT}',
+  '${STATE}',
+  '${VARIANT}',
+  '${BASELINE_PATH}',
+  'sha256:${FILE_HASH}',
+  1,
+  $(date +%s),
+  '${TASK_ID}',
+  '${AGENT_ID}',
+  '${METADATA}'
+);
+EOF
+  ACTION="created"
+fi
+
+# Insert audit log entry
+REASON_ESCAPED=$(echo "$REASON" | sed "s/'/''/g")  # Escape single quotes for SQL
+sqlite3 "$DB_PATH" <<EOF
+INSERT INTO screenshot_audit_log (
+  screenshot_key, action, reason, old_file_hash, new_file_hash, approved_by, created_at, task_id
+) VALUES (
+  '${SCREENSHOT_KEY}',
+  '${ACTION}',
+  '${REASON_ESCAPED}',
+  ${OLD_FILE_HASH:+'sha256:$OLD_FILE_HASH'}${OLD_FILE_HASH:-NULL},
+  'sha256:${FILE_HASH}',
+  '${AGENT_ID:-system}',
+  $(date +%s),
+  '${TASK_ID}'
+);
+EOF
+
+# Clear Redis baseline cache
+redis-cli del "screenshot:baseline:${SCREENSHOT_KEY}" > /dev/null 2>&1 || \
+  echo "Warning: Failed to clear Redis cache" >&2
+
+# Cache new baseline hash (TTL: 24 hours)
+redis-cli setex "screenshot:baseline:${SCREENSHOT_KEY}" 86400 "sha256:${FILE_HASH}" > /dev/null 2>&1
+
+# Output result
+cat <<EOF | jq '.'
+{
+  "status": "baseline-set",
+  "action": "${ACTION}",
+  "screenshot_key": "${SCREENSHOT_KEY}",
+  "baseline_path": "${BASELINE_PATH}",
+  "file_hash": "sha256:${FILE_HASH}",
+  "reason": "${REASON}",
+  "archived_previous": ${OLD_FILE_HASH:+true}${OLD_FILE_HASH:-false},
+  "created_at": $(date +%s)
+}
+EOF
+
+echo "✅ Baseline ${ACTION}: $SCREENSHOT_KEY" >&2

package/.claude/skills/webapp-testing/test-webapp-testing.sh

@@ -0,0 +1,233 @@
+#!/bin/bash
+# Webapp Testing Skill - Test Suite
+# Purpose: Validate webapp-testing skill functionality
+
+set -e
+
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+PROJECT_ROOT="$(cd "$SCRIPT_DIR/../../.." && pwd)"
+cd "$PROJECT_ROOT"
+
+# Colors for output
+RED='\033[0;31m'
+GREEN='\033[0;32m'
+YELLOW='\033[1;33m'
+NC='\033[0m' # No Color
+
+TESTS_PASSED=0
+TESTS_FAILED=0
+TESTS_SKIPPED=0
+
+# Test result tracking
+test_result() {
+  local TEST_NAME="$1"
+  local STATUS="$2"
+  local MESSAGE="$3"
+
+  case "$STATUS" in
+    PASS)
+      echo -e "${GREEN}✓${NC} $TEST_NAME"
+      TESTS_PASSED=$((TESTS_PASSED + 1))
+      ;;
+    FAIL)
+      echo -e "${RED}✗${NC} $TEST_NAME: $MESSAGE"
+      TESTS_FAILED=$((TESTS_FAILED + 1))
+      ;;
+    SKIP)
+      echo -e "${YELLOW}⊘${NC} $TEST_NAME: $MESSAGE"
+      TESTS_SKIPPED=$((TESTS_SKIPPED + 1))
+      ;;
+  esac
+}
+
+echo "=========================================="
+echo "Webapp Testing Skill - Test Suite"
+echo "=========================================="
+echo ""
+
+# Test 1: Storage Initialization
+echo "Test 1: Storage Initialization"
+if "$SCRIPT_DIR/init-storage.sh" --force >/dev/null 2>&1; then
+  if [ -d ".screenshots/baselines" ] && [ -d ".screenshots/current" ]; then
+    test_result "Storage initialization" "PASS"
+  else
+    test_result "Storage initialization" "FAIL" "Directories not created"
+  fi
+else
+  test_result "Storage initialization" "FAIL" "init-storage.sh failed"
+fi
+
+# Test 2: SQLite Schema Validation
+echo "Test 2: SQLite Schema Validation"
+DB_PATH="${HOME}/.claude/memory/adaptive-context.db"
+TABLE_COUNT=$(sqlite3 "$DB_PATH" \
+  "SELECT COUNT(*) FROM sqlite_master WHERE type='table' AND name IN ('webapp_screenshots', 'screenshot_audit_log');" 2>/dev/null)
+
+if [ "$TABLE_COUNT" -eq 2 ]; then
+  test_result "SQLite schema creation" "PASS"
+else
+  test_result "SQLite schema creation" "FAIL" "Expected 2 tables, found $TABLE_COUNT"
+fi
+
+# Test 3: Playwright Availability
+echo "Test 3: Playwright Availability"
+if command -v npx >/dev/null 2>&1 && npx playwright --version >/dev/null 2>&1; then
+  test_result "Playwright installation" "PASS"
+else
+  test_result "Playwright installation" "SKIP" "Playwright not installed (run: npx playwright install chromium)"
+fi
+
+# Test 4: Node Dependencies Check
+echo "Test 4: Node Dependencies Check"
+NODE_DEPS_MISSING=0
+for PKG in playwright pngjs pixelmatch; do
+  if ! node -e "require('$PKG')" 2>/dev/null; then
+    test_result "$PKG dependency" "SKIP" "Not installed (run: npm install $PKG)"
+    NODE_DEPS_MISSING=1
+  else
+    test_result "$PKG dependency" "PASS"
+  fi
+done
+
+# Test 5: Redis Connectivity
+echo "Test 5: Redis Connectivity"
+if redis-cli ping >/dev/null 2>&1; then
+  test_result "Redis connectivity" "PASS"
+else
+  test_result "Redis connectivity" "SKIP" "Redis not running"
+fi
+
+# Test 6: Screenshot Capture (requires test server)
+echo "Test 6: Screenshot Capture (Integration Test)"
+TEST_URL="https://example.com"  # Use public URL for testing
+TASK_ID="test-$(date +%s)"
+AGENT_ID="test-agent"
+
+if [ "$NODE_DEPS_MISSING" -eq 0 ]; then
+  CAPTURE_RESULT=$("$SCRIPT_DIR/capture-screenshot.sh" \
+    --project "test-project" \
+    --component "test-component" \
+    --viewport "800x600" \
+    --state "default" \
+    --variant "light-mode" \
+    --url "$TEST_URL" \
+    --task-id "$TASK_ID" \
+    --agent-id "$AGENT_ID" 2>&1)
+
+  if [ $? -eq 0 ] && echo "$CAPTURE_RESULT" | jq -e '.screenshot_key' >/dev/null 2>&1; then
+    SCREENSHOT_KEY=$(echo "$CAPTURE_RESULT" | jq -r '.screenshot_key')
+    test_result "Screenshot capture" "PASS"
+  else
+    test_result "Screenshot capture" "FAIL" "Capture failed or invalid output"
+  fi
+else
+  test_result "Screenshot capture" "SKIP" "Node dependencies missing"
+fi
+
+# Test 7: Baseline Setting
+echo "Test 7: Baseline Setting"
+if [ -n "$SCREENSHOT_KEY" ]; then
+  BASELINE_RESULT=$("$SCRIPT_DIR/set-baseline.sh" \
+    --screenshot-key "$SCREENSHOT_KEY" \
+    --reason "Test baseline" 2>&1)
+
+  if [ $? -eq 0 ] && echo "$BASELINE_RESULT" | jq -e '.status' >/dev/null 2>&1; then
+    test_result "Baseline setting" "PASS"
+  else
+    test_result "Baseline setting" "FAIL" "set-baseline.sh failed"
+  fi
+else
+  test_result "Baseline setting" "SKIP" "No screenshot to baseline"
+fi
+
+# Test 8: Screenshot Comparison
+echo "Test 8: Screenshot Comparison"
+if [ -n "$SCREENSHOT_KEY" ]; then
+  # Capture second screenshot for comparison
+  CAPTURE2_RESULT=$("$SCRIPT_DIR/capture-screenshot.sh" \
+    --project "test-project" \
+    --component "test-component" \
+    --viewport "800x600" \
+    --state "default" \
+    --variant "light-mode" \
+    --url "$TEST_URL" \
+    --task-id "$TASK_ID" \
+    --agent-id "$AGENT_ID" 2>&1)
+
+  if [ $? -eq 0 ]; then
+    COMPARE_RESULT=$("$SCRIPT_DIR/compare-screenshots.sh" \
+      --screenshot-key "$SCREENSHOT_KEY" \
+      --task-id "$TASK_ID" \
+      --threshold 0.95 2>&1)
+
+    if [ $? -eq 0 ] && echo "$COMPARE_RESULT" | jq -e '.similarity_score' >/dev/null 2>&1; then
+      SIMILARITY=$(echo "$COMPARE_RESULT" | jq -r '.similarity_score')
+      if (( $(echo "$SIMILARITY >= 0.95" | bc -l) )); then
+        test_result "Screenshot comparison" "PASS"
+      else
+        test_result "Screenshot comparison" "FAIL" "Low similarity: $SIMILARITY"
+      fi
+    else
+      test_result "Screenshot comparison" "FAIL" "compare-screenshots.sh failed"
+    fi
+  else
+    test_result "Screenshot comparison" "SKIP" "Second capture failed"
+  fi
+else
+  test_result "Screenshot comparison" "SKIP" "No screenshot to compare"
+fi
+
+# Test 9: SQLite Data Persistence
+echo "Test 9: SQLite Data Persistence"
+SCREENSHOT_COUNT=$(sqlite3 "$DB_PATH" \
+  "SELECT COUNT(*) FROM webapp_screenshots WHERE project = 'test-project';" 2>/dev/null)
+
+if [ "$SCREENSHOT_COUNT" -gt 0 ]; then
+  test_result "SQLite data persistence" "PASS"
+else
+  test_result "SQLite data persistence" "FAIL" "No test screenshots in database"
+fi
+
+# Test 10: Redis Queue Integration
+echo "Test 10: Redis Queue Integration"
+if [ -n "$TASK_ID" ]; then
+  QUEUE_LENGTH=$(redis-cli llen "screenshot:queue:${TASK_ID}" 2>/dev/null || echo "0")
+  if [ "$QUEUE_LENGTH" -gt 0 ]; then
+    test_result "Redis queue integration" "PASS"
+  else
+    test_result "Redis queue integration" "SKIP" "Redis queue empty (may have been processed)"
+  fi
+else
+  test_result "Redis queue integration" "SKIP" "No task ID available"
+fi
+
+# Cleanup test data
+echo ""
+echo "Cleaning up test data..."
+if [ -n "$SCREENSHOT_KEY" ]; then
+  sqlite3 "$DB_PATH" "DELETE FROM webapp_screenshots WHERE project = 'test-project';" 2>/dev/null
+  sqlite3 "$DB_PATH" "DELETE FROM screenshot_audit_log WHERE screenshot_key LIKE 'test-project/%';" 2>/dev/null
+  rm -rf .screenshots/baselines/test-project 2>/dev/null
+  rm -rf .screenshots/current/test-project 2>/dev/null
+  rm -rf .screenshots/diffs/"$TASK_ID" 2>/dev/null
+  redis-cli del "screenshot:queue:${TASK_ID}" >/dev/null 2>&1
+  echo "✓ Test data cleaned"
+fi
+
+# Summary
+echo ""
+echo "=========================================="
+echo "Test Summary"
+echo "=========================================="
+echo -e "${GREEN}Passed:${NC}  $TESTS_PASSED"
+echo -e "${RED}Failed:${NC}  $TESTS_FAILED"
+echo -e "${YELLOW}Skipped:${NC} $TESTS_SKIPPED"
+echo ""
+
+if [ "$TESTS_FAILED" -eq 0 ]; then
+  echo -e "${GREEN}✓ All tests passed${NC}"
+  exit 0
+else
+  echo -e "${RED}✗ Some tests failed${NC}"
+  exit 1
+fi

package/.claude/spawn-pattern-examples.md

@@ -242,7 +242,7 @@ Task("validator", `
 **Main Chat Thinks:**
 - Task type: Feature implementation
 - Topology: Mesh (coordinator + 5 agents, some parallel)
-- Agents needed: analyst, architect, backend-dev, frontend-dev, tester
+- Agents needed: analyst, architect, backend-dev, react-frontend-engineer, tester
 - Dependencies: analyst → architect → (backend + frontend parallel) → tester
 
 **Single Message Spawn:**
@@ -261,10 +261,10 @@ Task("coordinator-hybrid", `
 
   **Redis Channel:** swarm:profile-edit:coordinator
 
-  **Agents:** analyst → architect → (backend-dev || frontend-dev) → tester
+  **Agents:** analyst → architect → (backend-dev || react-frontend-engineer) → tester
 
   **Parallel Phase:**
-  - backend-dev and frontend-dev work in parallel after architect
+  - backend-dev and react-frontend-engineer work in parallel after architect
   - tester waits for BOTH to complete
 
   Monitor and aggregate results.