claude-flow-novice 2.15.11 → 2.16.0

package/.claude/skills/pre-edit-backup/backup.sh.backup

@@ -0,0 +1,107 @@
+#!/bin/bash
+
+##############################################################################
+# Pre-Edit Backup Script - Creates safe file backups before modifications
+# Version: 1.0.0
+##############################################################################
+
+set -euo pipefail
+
+# Function to create backup of a file before editing
+create_backup() {
+    local file_path="$1"
+    local agent_id="${2:-unknown}"
+    local project_root="${3:-$(pwd)}"
+
+    # Validate inputs
+    if [[ -z "$file_path" ]]; then
+        echo "Error: File path is required" >&2
+        exit 1
+    fi
+
+    # Check if file exists
+    if [[ ! -f "$file_path" ]]; then
+        echo "Warning: File does not exist: $file_path" >&2
+        # Create empty backup path for new files
+        echo "$project_root/.backups/$agent_id/new-file-$(date +%s)-$(echo "$file_path" | tr '/' '_' | tr ' ' '_')"
+        return 0
+    fi
+
+    # Create backup directory structure
+    local backup_dir="$project_root/.backups/$agent_id"
+    local timestamp=$(date +%s)
+    local file_hash=$(md5sum "$file_path" | cut -d' ' -f1)
+    local backup_name="${timestamp}_${file_hash}"
+
+    # Create full backup path
+    local full_backup_path="$backup_dir/$backup_name"
+
+    # Create backup directory
+    mkdir -p "$full_backup_path"
+
+    # Copy original file to backup location
+    cp "$file_path" "$full_backup_path/original"
+
+    # Store backup metadata
+    cat > "$full_backup_path/metadata.json" << EOF
+{
+  "timestamp": "$timestamp",
+  "agent_id": "$agent_id",
+  "original_file": "$file_path",
+  "file_hash": "$file_hash",
+  "backup_path": "$full_backup_path",
+  "created_at": "$(date -u +%Y-%m-%dT%H:%M:%SZ)"
+}
+EOF
+
+    # Create revert script
+    cat > "$full_backup_path/revert.sh" << EOF
+#!/bin/bash
+# Revert script for $file_path
+set -euo pipefail
+
+echo "Reverting file: $file_path"
+cp "$full_backup_path/original" "$file_path"
+echo "✅ File reverted successfully"
+EOF
+
+    chmod +x "$full_backup_path/revert.sh"
+
+    # Output backup path for caller
+    echo "$full_backup_path"
+
+    echo "✅ Backup created: $full_backup_path" >&2
+}
+
+# Main execution
+if [[ "${BASH_SOURCE[0]}" == "${0}" ]]; then
+    # Script called directly
+    if [[ $# -lt 1 ]]; then
+        echo "Usage: $0 <file_path> [--agent-id <id>] [--project-root <path>]" >&2
+        exit 1
+    fi
+
+    file_path="$1"
+    agent_id="unknown"
+    project_root="$(pwd)"
+
+    # Parse optional arguments
+    while [[ $# -gt 0 ]]; do
+        case $1 in
+            --agent-id)
+                agent_id="$2"
+                shift 2
+                ;;
+            --project-root)
+                project_root="$2"
+                shift 2
+                ;;
+            *)
+                # Skip unknown arguments
+                shift
+                ;;
+        esac
+    done
+
+    create_backup "$file_path" "$agent_id" "$project_root"
+fi

package/claude-assets/agents/cfn-dev-team/coordinators/cfn-frontend-coordinator.md

@@ -46,6 +46,7 @@ fi
 
 **Old (Deprecated):**
 ```bash
+```
 
 **New (Required):**
 ```bash

package/claude-assets/agents/cfn-dev-team/coordinators/consensus-builder.md

@@ -53,6 +53,7 @@ fi
 
 **Old (Deprecated):**
 ```bash
+```
 
 **New (Required):**
 ```bash

package/claude-assets/agents/cfn-dev-team/coordinators/handoff-coordinator.md

@@ -44,6 +44,7 @@ fi
 
 **Old (Deprecated):**
 ```bash
+```
 
 **New (Required):**
 ```bash

package/claude-assets/agents/cfn-dev-team/coordinators/multi-sprint-coordinator.md

@@ -45,6 +45,7 @@ fi
 
 **Old (Deprecated):**
 ```bash
+```
 
 **New (Required):**
 ```bash

package/claude-assets/agents/cfn-dev-team/dev-ops/devops-engineer.md

@@ -128,6 +128,16 @@ You are an elite DevOps and platform engineer specializing in building scalable,
 
 Remember: The best infrastructure is invisible—seamless, scalable, and empowering developers to deliver value.
 
+## Skill References
+
+### Test-Driven Development
+→ **JSON Validation**: `.claude/skills/json-validation/SKILL.md` - Defensive AGENT_SUCCESS_CRITERIA parsing with injection prevention
+→ **Test Runner**: `.claude/skills/cfn-test-runner/SKILL.md` - Unified test execution with benchmarking and regression detection
+
+### Container & Infrastructure
+→ **Docker Build**: `.claude/skills/docker-build/SKILL.md` - Fast Docker builds using Linux native storage (96% faster)
+→ **Redis Data Extraction**: `.claude/skills/cfn-redis-data-extraction/SKILL.md` - Extract and analyze CFN Loop coordination data
+
 ## Completion Protocol
 
 Complete your work and provide a structured response with:

package/claude-assets/agents/cfn-dev-team/dev-ops/docker-specialist.md

@@ -17,30 +17,31 @@ model: glm-4.6
 
 ## Success Criteria Awareness (REQUIRED - Phase 2 TDD)
 
-### 1. Read Success Criteria
-Before starting work, read test requirements from environment:
+### 1. JSON Validation & Success Criteria Parsing
+Use the centralized JSON validation skill for defensive AGENT_SUCCESS_CRITERIA parsing:
+
+**Skill Reference:** `.claude/skills/json-validation/SKILL.md`
+
 ```bash
-if [[ -n "${AGENT_SUCCESS_CRITERIA:-}" ]]; then
-    # Validate JSON before parsing
-    if ! echo "$AGENT_SUCCESS_CRITERIA" | jq -e '.' >/dev/null 2>&1; then
-        echo "❌ Invalid JSON in AGENT_SUCCESS_CRITERIA" >&2
-        exit 1
-    fi
-
-    CRITERIA=$(echo "$AGENT_SUCCESS_CRITERIA" | jq -r '.')
-    TEST_SUITES=$(echo "$CRITERIA" | jq -r '.test_suites[] // empty')
-
-    if [[ -n "$TEST_SUITES" ]]; then
-        echo "📋 Success Criteria Loaded:"
-        echo "$TEST_SUITES" | jq -r '.name // "unnamed"'
-    fi
-fi
+# Source the skill for safe JSON validation
+source .claude/skills/json-validation/validate-success-criteria.sh
+
+# Validate and parse with injection attack prevention
+validate_success_criteria || exit 1
+
+# Access parsed data
+list_test_suites
 ```
 
+**Features:**
+- Prevents JSON injection attacks (CVSS 8.2)
+- Handles missing/malformed data gracefully
+- No external dependencies beyond jq
+
 ### 2. TDD Protocol (MANDATORY)
 
 **Write Tests First (15-20 min):**
-- Extract test requirements from success criteria
+- Extract test requirements from success criteria (via skill above)
 - Write failing tests for each requirement
 - Ensure test coverage ≥80%
 
@@ -54,26 +55,25 @@ fi
 - Verify pass rate meets threshold (Standard: ≥95%)
 - Check coverage: `npm run coverage`
 
-### 3. Report Test Results (NOT Confidence)
+### 3. Test Execution & Results Parsing
+Use the centralized test runner skill for consistent test result collection:
 
-**Old (Deprecated):**
-```bash
-# Not shown - deprecated pattern
-```
+**Skill Reference:** `.claude/skills/cfn-test-runner/SKILL.md`
 
-**New (Required):**
 ```bash
-# Execute tests and capture output
-TEST_OUTPUT=$(npm test 2>&1)
-
-# Parse natively (no external dependencies)
-PASS=$(echo "$TEST_OUTPUT" | grep -oP '\d+(?= passing)' || echo "0")
-FAIL=$(echo "$TEST_OUTPUT" | grep -oP '\d+(?= failing)' || echo "0")
-TOTAL=$((PASS + FAIL))
-RATE=$(awk "BEGIN {if ($TOTAL > 0) printf \"%.2f\", $PASS/$TOTAL; else print \"0.00\"}")
-
+# Execute tests with benchmarking
+./.claude/skills/cfn-test-runner/run-all-tests.sh \
+  --suite all \
+  --benchmark \
+  --detect-regressions
 ```
 
+**Captures:**
+- Test pass/fail counts
+- Performance metrics
+- Regression detection
+- Historical comparisons
+
 # Docker Specialist Agent
 
 ## 🚨 CRITICAL: WSL2 Build Performance Requirement
@@ -150,6 +150,29 @@ Fast Docker image building using Linux native storage for 96% faster builds (755
 
 **See:** `.claude/skills/docker-build/SKILL.md` for complete documentation
 
+### cfn-redis-data-extraction
+Extract and analyze complete Redis coordination data from completed CFN Loop tasks.
+
+**Use Cases:**
+- Analyze coordinator performance metrics
+- Extract task completion timelines
+- Review agent success/failure rates
+- Audit multi-agent coordination decisions
+
+**Quick Use:**
+```bash
+# Extract coordination data from completed task
+npx claude-flow-novice skill cfn-redis-data-extraction \
+  --task-id "cfn-cli-XXXXXXX-XXXXX"
+
+# Extract with performance metrics
+npx claude-flow-novice skill cfn-redis-data-extraction \
+  --task-id "cfn-cli-XXXXXXX-XXXXX" \
+  --include-performance=true
+```
+
+**See:** `.claude/skills/cfn-redis-data-extraction/SKILL.md` for complete documentation
+
 ---
 
 ## 🚨 CRITICAL: Docker Coordinator Known Issues

package/claude-assets/agents/cfn-dev-team/dev-ops/kubernetes-specialist.md

@@ -18,63 +18,63 @@ model: glm-4.6
 
 ## Success Criteria Awareness (REQUIRED - Phase 2 TDD)
 
-### 1. Read Success Criteria
-Before starting work, read test requirements from environment:
+### 1. JSON Validation & Success Criteria Parsing
+Use the centralized JSON validation skill for defensive AGENT_SUCCESS_CRITERIA parsing:
+
+**Skill Reference:** `.claude/skills/json-validation/SKILL.md`
+
 ```bash
-if [[ -n "${AGENT_SUCCESS_CRITERIA:-}" ]]; then
-    # Validate JSON before parsing
-    if ! echo "$AGENT_SUCCESS_CRITERIA" | jq -e '.' >/dev/null 2>&1; then
-        echo "❌ Invalid JSON in AGENT_SUCCESS_CRITERIA" >&2
-        exit 1
-    fi
-
-    CRITERIA=$(echo "$AGENT_SUCCESS_CRITERIA" | jq -r '.')
-    TEST_SUITES=$(echo "$CRITERIA" | jq -r '.test_suites[] // empty')
-
-    if [[ -n "$TEST_SUITES" ]]; then
-        echo "📋 Success Criteria Loaded:"
-        echo "$TEST_SUITES" | jq -r '.name // "unnamed"'
-    fi
-fi
+# Source the skill for safe JSON validation
+source .claude/skills/json-validation/validate-success-criteria.sh
+
+# Validate and parse with injection attack prevention
+validate_success_criteria || exit 1
+
+# Access parsed data
+list_test_suites
 ```
 
+**Features:**
+- Prevents JSON injection attacks (CVSS 8.2)
+- Handles missing/malformed data gracefully
+- No external dependencies beyond jq
+
 ### 2. TDD Protocol (MANDATORY)
 
 **Write Tests First (15-20 min):**
-- Extract test requirements from success criteria
+- Extract test requirements from success criteria (via skill above)
 - Write failing tests for each requirement
 - Ensure test coverage ≥80%
 
 **Implement (30-40 min):**
 - Write minimum code to pass tests
-- Run tests continuously (`npm test --watch` or framework equivalent)
+- Run tests continuously (kubectl test, helm test, or framework equivalent)
 - Refactor for quality
 
 **Validate (5 min):**
-- Run full test suite: `npm test` (or framework command from criteria)
+- Run full test suite: `npm test` or `kubectl test` (per framework)
 - Verify pass rate meets threshold (Standard: ≥95%)
-- Check coverage: `npm run coverage`
+- Check coverage: appropriate to your testing framework
 
-### 3. Report Test Results (NOT Confidence)
+### 3. Test Execution & Results Parsing
+Use the centralized test runner skill for consistent test result collection:
 
-**Old (Deprecated):**
-```bash
-# Not shown - deprecated pattern
-```
+**Skill Reference:** `.claude/skills/cfn-test-runner/SKILL.md`
 
-**New (Required):**
 ```bash
-# Execute tests and capture output
-TEST_OUTPUT=$(npm test 2>&1)
-
-# Parse natively (no external dependencies)
-PASS=$(echo "$TEST_OUTPUT" | grep -oP '\d+(?= passing)' || echo "0")
-FAIL=$(echo "$TEST_OUTPUT" | grep -oP '\d+(?= failing)' || echo "0")
-TOTAL=$((PASS + FAIL))
-RATE=$(awk "BEGIN {if ($TOTAL > 0) printf \"%.2f\", $PASS/$TOTAL; else print \"0.00\"}")
-
+# Execute tests with benchmarking
+./.claude/skills/cfn-test-runner/run-all-tests.sh \
+  --suite all \
+  --benchmark \
+  --detect-regressions
 ```
 
+**Captures:**
+- Test pass/fail counts
+- Performance metrics
+- Regression detection
+- Historical comparisons
+
 # Kubernetes Specialist Agent
 
 ## Core Responsibilities
@@ -595,6 +595,16 @@ Before reporting high confidence:
 4. **Monitoring Setup**: Prometheus metrics, Grafana dashboards
 5. **CI/CD Integration**: GitOps workflows, ArgoCD applications
 
+## Skill References
+
+### Test-Driven Development
+→ **JSON Validation**: `.claude/skills/json-validation/SKILL.md` - Defensive AGENT_SUCCESS_CRITERIA parsing with injection prevention
+→ **Test Runner**: `.claude/skills/cfn-test-runner/SKILL.md` - Unified test execution with benchmarking and regression detection
+
+### Container & Orchestration
+→ **Docker Build**: `.claude/skills/docker-build/SKILL.md` - Fast Docker builds using Linux native storage (96% faster)
+→ **Redis Data Extraction**: `.claude/skills/cfn-redis-data-extraction/SKILL.md` - Extract and analyze CFN Loop coordination data
+
 ## Completion Protocol
 
 Complete your work and provide a structured response with:

package/claude-assets/agents/cfn-dev-team/dev-ops/monitoring-specialist.md

@@ -753,7 +753,16 @@ Complete your work and provide a structured response with:
 **Note:** Coordination handled automatically by the system.
 
 ## Skill References
+
+### Test-Driven Development
+→ **JSON Validation**: `.claude/skills/json-validation/SKILL.md` - Defensive AGENT_SUCCESS_CRITERIA parsing with injection prevention
+→ **Test Runner**: `.claude/skills/cfn-test-runner/SKILL.md` - Unified test execution with benchmarking and regression detection
+
+### Monitoring & Observability
 → **Prometheus Setup**: `.claude/skills/prometheus-monitoring/SKILL.md`
 → **Grafana Dashboards**: `.claude/skills/grafana-dashboard-creation/SKILL.md`
 → **SLO Tracking**: `.claude/skills/slo-management/SKILL.md`
 → **Distributed Tracing**: `.claude/skills/opentelemetry-tracing/SKILL.md`
+
+### Coordination & Data Analysis
+→ **Redis Data Extraction**: `.claude/skills/cfn-redis-data-extraction/SKILL.md` - Extract and analyze CFN Loop coordination data

package/claude-assets/agents/cfn-dev-team/developers/api-gateway-specialist.md

@@ -29,23 +29,18 @@ model: glm-4.6
 ## Success Criteria Awareness (REQUIRED - Phase 2 TDD)
 
 ### 1. Read Success Criteria
-Before starting work, read test requirements from environment:
+Before starting work, use the JSON validation skill:
+
+**Skill Reference:** `.claude/skills/json-validation/validate-success-criteria.sh`
+- Validates `AGENT_SUCCESS_CRITERIA` JSON safely
+- Prevents injection attacks
+- Provides error handling
+
+Usage:
 ```bash
-if [[ -n "${AGENT_SUCCESS_CRITERIA:-}" ]]; then
-    # Validate JSON before parsing
-    if ! echo "$AGENT_SUCCESS_CRITERIA" | jq -e '.' >/dev/null 2>&1; then
-        echo "❌ Invalid JSON in AGENT_SUCCESS_CRITERIA" >&2
-        exit 1
-    fi
-
-    CRITERIA=$(echo "$AGENT_SUCCESS_CRITERIA" | jq -r '.')
-    TEST_SUITES=$(echo "$CRITERIA" | jq -r '.test_suites[] // empty')
-
-    if [[ -n "$TEST_SUITES" ]]; then
-        echo "📋 Success Criteria Loaded:"
-        echo "$TEST_SUITES" | jq -r '.name // "unnamed"'
-    fi
-fi
+source .claude/skills/json-validation/validate-success-criteria.sh
+validate_success_criteria || exit 1
+list_test_suites
 ```
 
 ### 2. TDD Protocol (MANDATORY)
@@ -67,7 +62,12 @@ fi
 
 ### 3. Report Test Results (NOT Confidence)
 
-Execute tests and report objective pass/fail metrics:
+Use the test runner skill for parsing and reporting:
+
+**Skill Reference:** `.claude/skills/cfn-test-runner/run-all-tests.sh`
+- Executes test suite with native bash parsing
+- Calculates pass rates and coverage metrics
+- Handles Redis gracefully (automatic failure in Task mode)
 
 ```bash
 # Execute tests and capture output

package/claude-assets/agents/cfn-dev-team/developers/backend-developer.md

@@ -20,23 +20,18 @@ model: glm-4.6
 ## Success Criteria Awareness (REQUIRED - Phase 2 TDD)
 
 ### 1. Read Success Criteria
-Before starting work, read test requirements from environment:
+Before starting work, read test requirements from environment using the JSON validation skill:
+
+**Skill Reference:** `.claude/skills/json-validation/validate-success-criteria.sh`
+- Validates `AGENT_SUCCESS_CRITERIA` JSON safely
+- Prevents injection attacks
+- Provides centralized error handling
+
+Usage:
 ```bash
-if [[ -n "${AGENT_SUCCESS_CRITERIA:-}" ]]; then
-    # Validate JSON before parsing
-    if ! echo "$AGENT_SUCCESS_CRITERIA" | jq -e '.' >/dev/null 2>&1; then
-        echo "❌ Invalid JSON in AGENT_SUCCESS_CRITERIA" >&2
-        exit 1
-    fi
-
-    CRITERIA=$(echo "$AGENT_SUCCESS_CRITERIA" | jq -r '.')
-    TEST_SUITES=$(echo "$CRITERIA" | jq -r '.test_suites[] // empty')
-
-    if [[ -n "$TEST_SUITES" ]]; then
-        echo "📋 Success Criteria Loaded:"
-        echo "$TEST_SUITES" | jq -r '.name // "unnamed"'
-    fi
-fi
+source .claude/skills/json-validation/validate-success-criteria.sh
+validate_success_criteria || exit 1
+list_test_suites
 ```
 
 ### 2. TDD Protocol (MANDATORY)
@@ -58,10 +53,17 @@ fi
 
 ### 3. Report Test Results (NOT Confidence)
 
-**IMPORTANT:** Use the centralized completion script which handles Redis gracefully:
+Use the centralized test runner skill for parsing and reporting:
 
+**Skill Reference:** `.claude/skills/cfn-test-runner/run-all-tests.sh`
+- Executes test suite with native bash parsing (no external dependencies)
+- Calculates pass rates and coverage metrics
+- Handles Redis gracefully (automatic failure in Task mode)
+- Stores results in Redis (when available)
+
+Usage:
 ```bash
-# Execute tests and capture output
+# Execute tests and capture results
 TEST_OUTPUT=$(npm test 2>&1)
 
 # Parse natively (no external dependencies)
@@ -70,10 +72,11 @@ FAIL=$(echo "$TEST_OUTPUT" | grep -oP '\d+(?= failing)' || echo "0")
 TOTAL=$((PASS + FAIL))
 RATE=$(awk "BEGIN {if ($TOTAL > 0) printf \"%.2f\", $PASS/$TOTAL; else print \"0.00\"}")
 
-# Report completion (automatically fails gracefully in Task mode)
+# Return results (Main Chat receives automatically in Task Mode)
+echo "{\"passed\": $PASS, \"failed\": $FAIL, \"pass_rate\": $RATE}"
 ```
 
-**Note:** Redis commands automatically fail gracefully when unavailable (Task mode). No manual mode detection needed - the centralized wrapper handles this via ANTI-023 protection.
+**Note:** Redis commands fail gracefully when unavailable (Task mode). The skill handles this via ANTI-023 protection.
 
 ## Core Responsibilities
 - Design and implement scalable backend services
@@ -161,49 +164,28 @@ DO NOT report subjective confidence scores. Instead:
 
 ## Completion Protocol (Test-Driven)
 
-Complete your work and provide test-based validation:
-
-1. **Execute Tests**: Run all test suites from success criteria
-   ```bash
-   # Parse natively (no external dependencies)
-   PASS=$(echo "$TEST_OUTPUT" | grep -oP '\d+(?= passing)' || echo "0")
-   FAIL=$(echo "$TEST_OUTPUT" | grep -oP '\d+(?= failing)' || echo "0")
-   TOTAL=$((PASS + FAIL))
-   RATE=$(awk "BEGIN {if ($TOTAL > 0) printf \"%.2f\", $PASS/$TOTAL; else print \"0.00\"}")
-
-   # Return results (Main Chat receives automatically in Task Mode)
-   echo "{\"passed\": $PASS, \"failed\": $FAIL, \"pass_rate\": $RATE}"
-   ```
-
-2. **Parse Results**: Extract test counts and calculate pass rate
-
-3. **Coverage Check**: Ensure coverage meets minimum thresholds
-   - Unit tests: ≥95%
-   - Integration tests: ≥90%
-   - Coverage: ≥80%
+Complete your work and provide test-based validation using the test runner skill:
 
-4. **Store in Redis**: Use test-results key (not confidence key)
+**Skill Reference:** `.claude/skills/cfn-test-runner/run-all-tests.sh`
 
-5. **Signal Completion**: Push to completion queue
+1. **Execute Tests**: Run all test suites from success criteria
+2. **Parse Results**: Extract test counts and calculate pass rate using native bash parsing
+3. **Coverage Check**: Ensure coverage meets minimum thresholds (≥80%)
+4. **Store Results**: Use test-results key for reporting (skill handles Redis gracefully)
+5. **Signal Completion**: Push to completion queue (automatic via skill)
 
-**Example Report:**
-```text
-Test Execution Summary:
-- Unit Tests: 45/47 passed (95.7%)
-- Integration Tests: 12/12 passed (100%)
-- E2E Tests: 8/10 passed (80%)
-- Overall: 65/69 passed (94.2%)
-- Coverage: 84.3%
-- Gate Status: PASS
+**Example Implementation:**
+```bash
+source .claude/skills/cfn-test-runner/run-all-tests.sh
 
-Gate Logic (Hybrid Threshold):
-  Pass criteria: At least 2 of 3 test suites meet ≥95% threshold AND overall ≥80%
-  This example: 2 suites (Integration 100%, Unit 95.7%) meet ≥95% ✓
-                Overall 94.2% meets ≥80% ✓
-                Result: PASS
+# Run tests and get results
+TEST_OUTPUT=$(npm test 2>&1)
+PASS=$(echo "$TEST_OUTPUT" | grep -oP '\d+(?= passing)' || echo "0")
+FAIL=$(echo "$TEST_OUTPUT" | grep -oP '\d+(?= failing)' || echo "0")
+RATE=$(awk "BEGIN {if ($((PASS + FAIL)) > 0) printf \"%.2f\", $PASS/($PASS+$FAIL); else print \"0.00\"}")
 
-Note: The hybrid 2-of-3 rule applies when multiple test suites are defined in success criteria.
-      For single-suite tasks, the standard ≥95% threshold applies directly to that suite.
+# Return results
+echo "{\"passed\": $PASS, \"failed\": $FAIL, \"pass_rate\": $RATE}"
 ```
 
 **Note:** Coordination instructions and success criteria provided when spawned via CLI.

package/claude-assets/agents/cfn-dev-team/developers/data/data-engineer.md

@@ -19,23 +19,18 @@ model: glm-4.6
 ## Success Criteria Awareness (REQUIRED - Phase 2 TDD)
 
 ### 1. Read Success Criteria
-Before starting work, read test requirements from environment:
+Before starting work, use the JSON validation skill to safely parse test requirements:
+
+**Skill Reference:** `.claude/skills/json-validation/validate-success-criteria.sh`
+- Validates `AGENT_SUCCESS_CRITERIA` JSON safely
+- Prevents injection attacks
+- Provides centralized error handling
+
+Usage:
 ```bash
-if [[ -n "${AGENT_SUCCESS_CRITERIA:-}" ]]; then
-    # Validate JSON before parsing
-    if ! echo "$AGENT_SUCCESS_CRITERIA" | jq -e '.' >/dev/null 2>&1; then
-        echo "❌ Invalid JSON in AGENT_SUCCESS_CRITERIA" >&2
-        exit 1
-    fi
-
-    CRITERIA=$(echo "$AGENT_SUCCESS_CRITERIA" | jq -r '.')
-    TEST_SUITES=$(echo "$CRITERIA" | jq -r '.test_suites[] // empty')
-
-    if [[ -n "$TEST_SUITES" ]]; then
-        echo "📋 Success Criteria Loaded:"
-        echo "$TEST_SUITES" | jq -r '.name // "unnamed"'
-    fi
-fi
+source .claude/skills/json-validation/validate-success-criteria.sh
+validate_success_criteria || exit 1
+list_test_suites
 ```
 
 ### 2. TDD Protocol (MANDATORY)
@@ -57,10 +52,14 @@ fi
 
 ### 3. Report Test Results (NOT Confidence)
 
-**Old (Deprecated):**
-```bash
+Use the test runner skill for parsing and reporting results:
 
-**New (Required):**
+**Skill Reference:** `.claude/skills/cfn-test-runner/run-all-tests.sh`
+- Executes test suite with native bash parsing
+- Calculates pass rates and coverage metrics
+- Handles Redis gracefully (automatic failure in Task mode)
+
+Implementation:
 ```bash
 # Execute tests and capture output
 TEST_OUTPUT=$(npm test 2>&1)
@@ -73,7 +72,6 @@ RATE=$(awk "BEGIN {if ($TOTAL > 0) printf \"%.2f\", $PASS/$TOTAL; else print \"0
 
 # Return results (Main Chat receives automatically in Task Mode)
 echo "{\"passed\": $PASS, \"failed\": $FAIL, \"pass_rate\": $RATE}"
-
 ```
 
 # Data Engineer Agent