claude-flow-novice 2.14.28 → 2.14.29

package/claude-assets/agents/cfn-dev-team/utility/analyst.md

@@ -122,54 +122,14 @@ async function analyzeWithFallback(system) {
 - `agent/${AGENT_ID}/findings/${TASK_ID}`
 - `cfn/phase-${phaseId}/loop3/agent-${AGENT_ID}`
 
-## CFN Loop Redis Completion Protocol
+## Completion Protocol
 
-When participating in CFN Loop workflows, agents MUST follow this protocol:
+Complete your work and provide a structured response with:
+- Confidence score (0.0-1.0) based on work quality
+- Summary of analysis/review completed
+- List of findings or deliverables
+- Any recommendations made
 
-### Step 1: Complete Work
-Execute assigned task (system analysis, performance investigation, code quality assessment)
-
-### Step 2: Signal Completion
-```bash
-redis-cli lpush "swarm:${TASK_ID}:${AGENT_ID}:done" "complete"
-```
-
-### Step 3: Report Confidence Score
-```bash
-./.claude/skills/redis-coordination/invoke-waiting-mode.sh report \
-
-**After reporting, exit cleanly. Do NOT enter waiting mode.**
-
-**Why This Matters:**
-- Orchestrator collects confidence/consensus scores from Redis
-- Enables adaptive agent specialization for next iteration
-- Prevents orchestrator blocking on wait $PID
-- Coordinator spawns appropriate specialist based on feedback type
-
-  --task-id "$TASK_ID" \
-  --agent-id "$AGENT_ID" \
-  --confidence [0.0-1.0] \
-  --iteration 1
-```
-
-### Step 4: Enter Waiting Mode (for potential iteration)
-```bash
-  --task-id "$TASK_ID" \
-  --agent-id "$AGENT_ID" \
-  --context "iteration-complete"
-```
-
-**Why This Matters:**
-- Zero-token blocking coordination (BLPOP waits without API calls)
-- Orchestrator collects confidence/consensus scores automatically
-- Supports autonomous iteration based on quality gates
-- Agent woken instantly (<100ms) if iteration needed
-
-**Context Variables:**
-- `TASK_ID`: Provided by orchestrator/coordinator
-- `AGENT_ID`: Your unique agent identifier (e.g., "analyst-1")
-- Confidence: Your self-assessment score (0.0-1.0)
-
-See: `.claude/skills/redis-coordination/SKILL.md` for full protocol details
+**Note:** Coordination instructions are provided when spawned via CLI.
 
 Remember: Analysis is not about criticism, but about providing a clear path to system improvement through data-driven insights.

package/claude-assets/agents/cfn-dev-team/utility/code-booster.md

@@ -95,45 +95,12 @@ npx claude-flow@alpha hooks post-edit [FILE_PATH] \
   --memory-key "code-booster/${AGENT_ID}/optimization" \
   --structured
 ```
-## CFN Loop Redis Completion Protocol
+## Completion Protocol
 
-When participating in CFN Loop workflows, agents MUST follow this protocol:
+Complete your work and provide a structured response with:
+- Confidence score (0.0-1.0) based on work quality
+- Summary of analysis/review completed
+- List of findings or deliverables
+- Any recommendations made
 
-### Step 1: Complete Work
-Execute assigned task (code optimization, WASM acceleration, performance enhancement)
-
-### Step 2: Signal Completion
-```bash
-redis-cli lpush "swarm:${TASK_ID}:${AGENT_ID}:done" "complete"
-```
-
-### Step 3: Report Confidence Score
-```bash
-./.claude/skills/redis-coordination/invoke-waiting-mode.sh report   --task-id "$TASK_ID"   --agent-id "$AGENT_ID"   --confidence [0.0-1.0]   --iteration 1
-
-**After reporting, exit cleanly. Do NOT enter waiting mode.**
-
-**Why This Matters:**
-- Orchestrator collects confidence/consensus scores from Redis
-- Enables adaptive agent specialization for next iteration
-- Prevents orchestrator blocking on wait $PID
-- Coordinator spawns appropriate specialist based on feedback type
-
-```
-
-### Step 4: Enter Waiting Mode (for potential iteration)
-```bash
-```
-
-**Why This Matters:**
-- Zero-token blocking coordination (BLPOP waits without API calls)
-- Orchestrator collects confidence/consensus scores automatically
-- Supports autonomous iteration based on quality gates
-- Agent woken instantly (<100ms) if iteration needed
-
-**Context Variables:**
-- `TASK_ID`: Provided by orchestrator/coordinator
-- `AGENT_ID`: Your unique agent identifier (e.g., "code-booster-1")
-- Confidence: Self-assessment score of optimization quality and performance gain (0.0-1.0)
-
-See: `.claude/skills/redis-coordination/SKILL.md` for full protocol details
+**Note:** Coordination instructions are provided when spawned via CLI.

package/claude-assets/agents/cfn-dev-team/utility/context-curator.md

@@ -45,52 +45,12 @@ Intelligent context merging and prioritization
 - No destructive merges
 - Explainable AI principles
 
-## CFN Loop Redis Completion Protocol
+## Completion Protocol
 
-When participating in CFN Loop workflows, agents MUST follow this protocol:
+Complete your work and provide a structured response with:
+- Confidence score (0.0-1.0) based on work quality
+- Summary of analysis/review completed
+- List of findings or deliverables
+- Any recommendations made
 
-### Step 1: Complete Work
-Execute assigned task (context merging, prioritization, adaptive learning strategy)
-
-### Step 2: Signal Completion
-```bash
-redis-cli lpush "swarm:${TASK_ID}:${AGENT_ID}:done" "complete"
-```
-
-### Step 3: Report Confidence Score
-```bash
-./.claude/skills/redis-coordination/invoke-waiting-mode.sh report \
-
-**After reporting, exit cleanly. Do NOT enter waiting mode.**
-
-**Why This Matters:**
-- Orchestrator collects confidence/consensus scores from Redis
-- Enables adaptive agent specialization for next iteration
-- Prevents orchestrator blocking on wait $PID
-- Coordinator spawns appropriate specialist based on feedback type
-
-  --task-id "$TASK_ID" \
-  --agent-id "$AGENT_ID" \
-  --confidence [0.0-1.0] \
-  --iteration 1
-```
-
-### Step 4: Enter Waiting Mode (for potential iteration)
-```bash
-  --task-id "$TASK_ID" \
-  --agent-id "$AGENT_ID" \
-  --context "iteration-complete"
-```
-
-**Why This Matters:**
-- Zero-token blocking coordination (BLPOP waits without API calls)
-- Orchestrator collects confidence/consensus scores automatically
-- Supports autonomous iteration based on quality gates
-- Agent woken instantly (<100ms) if iteration needed
-
-**Context Variables:**
-- `TASK_ID`: Provided by orchestrator/coordinator
-- `AGENT_ID`: Your unique agent identifier (e.g., "context-curator-1")
-- Confidence: Your self-assessment score (0.0-1.0)
-
-See: `.claude/skills/redis-coordination/SKILL.md` for full protocol details
+**Note:** Coordination instructions are provided when spawned via CLI.

package/claude-assets/commands/CFN_LOOP_TASK_MODE.md

@@ -35,6 +35,125 @@ Task Mode: Main Chat coordinates directly and spawns agents via Task() tool with
 
 ---
 
+## ANTI-023 Memory Leak Protection (v2.14.28)
+
+**Critical Fix**: Prevents Task Mode agents from executing CLI coordination scripts that cause memory leaks.
+
+### Three-Layer Defense System
+
+#### Layer 1: Agent Documentation
+**Task Mode agents use mode-specific completion protocols:**
+```markdown
+## ⚠️ CRITICAL: Mode-Specific Completion Protocol
+
+**Task Mode (95%):** Spawned via `Task("agent", "...")` in Main Chat
+- Return structured JSON output directly
+- ❌ DO NOT: Use Redis commands, bash scripts, CLI tools
+- ✅ Main Chat receives output automatically
+
+**CLI Mode (5%):** Spawned via `npx claude-flow-novice agent-spawn`
+- Use Redis signals and completion scripts
+- ✅ CLI coordination allowed
+```
+
+#### Layer 2: Agent-Level Detection
+**Automatic detection functions prevent CLI usage:**
+```bash
+detect_task_mode_and_exit() {
+  if [[ -z "${TASK_ID:-}" || -z "${AGENT_ID:-}" ]]; then
+    echo "❌ TASK MODE DETECTED - CLI commands forbidden"
+    exit 1
+  fi
+}
+```
+
+#### Layer 3: Code-Level Runtime Blocking
+**Coordination scripts include early exit checks:**
+```bash
+# ⚠️ ANTI-023 MEMORY LEAK PROTECTION
+if [[ -z "${TASK_ID:-}" || -z "${AGENT_ID:-}" ]]; then
+    echo "❌ TASK MODE DETECTED - Redis coordination forbidden"
+    exit 1
+fi
+```
+
+### Memory Impact Resolution
+- **Before Fix**: Up to 23GB memory consumption per hanging agent
+- **After Fix**: <100MB normal usage with complete audit trails
+- **Detection Logic**: Environment variable presence indicates spawn mode
+
+---
+
+## Audit Trail Architecture (New v2.14.28)
+
+### Dual-Mode Storage System
+
+**Design Principle**: Both Task Mode and CLI Mode create complete audit trails for compliance and debugging.
+
+#### Task Mode Storage (Main Chat Managed)
+```bash
+# After Task Mode agent completion, Main Chat stores audit data
+./.claude/skills/cfn-task-audit/store-task-audit.sh \
+  --task-id "$TASK_ID" \
+  --agent-type "$AGENT_TYPE" \
+  --output "$AGENT_OUTPUT" \
+  --mode "Task"
+```
+
+**Storage Locations:**
+- **Redis**: Fast access (`swarm:${TASK_ID}:${AGENT_TYPE}:audit`)
+- **SQLite**: Permanent audit trail (`agent_audit` table)
+- **Files**: Structured logs (`.claude/logs/task-audit/`)
+
+#### CLI Mode Storage (Agent Managed)
+```bash
+# CLI agents continue using existing Redis coordination
+redis-cli HSET "swarm:${TASK_ID}:${AGENT_ID}:result" \
+  "decision" "$DECISION" \
+  "mode" "CLI"
+```
+
+### Combined Audit Retrieval
+```bash
+# Retrieve complete audit trail from both modes
+get-audit-data.sh --task-id "$TASK_ID" --mode "combined"
+
+# Returns unified decision data:
+{
+  "task_mode_data": [...],
+  "cli_mode_data": [...],
+  "complete_audit_trail": true,
+  "execution_summary": {...}
+}
+```
+
+### Audit Data Structure
+```json
+{
+  "task_id": "task-123",
+  "agent_type": "reviewer",
+  "decision": "PROCEED",
+  "reasoning": "Code meets quality standards",
+  "confidence": 0.92,
+  "mode": "Task|CLI",
+  "deliverables": ["file.ts", "test.ts"],
+  "timestamp": "2025-11-06T15:30:00Z",
+  "metadata": {
+    "stored_via": "store-task-audit.sh",
+    "version": "1.0.0"
+  }
+}
+```
+
+### Benefits
+- **✅ Complete Coverage**: Both execution modes fully audited
+- **✅ Memory Safe**: ANTI-023 protection maintained
+- **✅ Compliance Ready**: Full audit trail for enterprise requirements
+- **✅ Debugging Support**: Replay capability for complex issues
+- **✅ Performance**: Fast Redis access + persistent SQLite storage
+
+---
+
 ## Agent Specialization
 
 ### Loop 3 (Implementation)

package/claude-assets/skills/cfn-agent-spawning/spawn-agent.sh

@@ -5,6 +5,18 @@
 
 set -euo pipefail
 
+
+# ⚠️ ANTI-023 MEMORY LEAK PROTECTION: Environment Sanitization
+# Load and apply environment sanitization to prevent memory leaks
+# shellcheck source=../cfn-environment-sanitization/sanitize-environment.sh
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+if [[ -f "$SCRIPT_DIR/../cfn-environment-sanitization/sanitize-environment.sh" ]]; then
+    source "$SCRIPT_DIR/../cfn-environment-sanitization/sanitize-environment.sh" --strict
+    echo "✅ Agent spawning environment sanitized" >&2
+else
+    echo "⚠️ Environment sanitization not available - proceeding without protection" >&2
+fi
+
 # ⚠️ ANTI-023 MEMORY LEAK PROTECTION: Block Task Mode agents
 # Task Mode agents spawn via Task() tool and should NOT use agent spawning CLI
 if [[ -z "${1:-}" || -z "${TASK_ID:-}" ]]; then

package/claude-assets/skills/cfn-agent-spawning/spawn-agent.sh.backup

@@ -0,0 +1,273 @@
+#!/bin/bash
+
+# Agent Spawning CLI Wrapper
+# Enables agents to spawn other agents or stop existing agents via simple CLI interface
+
+set -euo pipefail
+
+# ⚠️ ANTI-023 MEMORY LEAK PROTECTION: Block Task Mode agents
+# Task Mode agents spawn via Task() tool and should NOT use agent spawning CLI
+if [[ -z "${1:-}" || -z "${TASK_ID:-}" ]]; then
+    echo "❌ TASK MODE DETECTED - Agent spawning CLI forbidden" >&2
+    echo "🚨 ANTI-023: This script is for CLI-spawned coordinators only" >&2
+    echo "💡 Task Mode agent spawning should be handled directly by Main Chat" >&2
+    echo "🔧 Agent spawned via Task() tool - use Task() tool for spawning instead" >&2
+    exit 1
+fi
+
+# ============================================================================
+# LOGGING FUNCTIONS
+# ============================================================================
+log_error() {
+    echo "[ERROR] $*" >&2
+}
+
+log_warning() {
+    echo "[WARNING] $*" >&2
+}
+
+log_info() {
+    echo "[INFO] $*"
+}
+
+# ============================================================================
+# DEPENDENCY CHECKS
+# ============================================================================
+
+check_dependencies() {
+  local missing_deps=()
+
+  # Check bash version
+  if [[ "${BASH_VERSINFO[0]}" -lt 4 ]]; then
+    missing_deps+=("bash>=4.0")
+  fi
+
+  # Check required command-line tools
+  local required_tools=("npx" "node" "grep" "sed")
+  for tool in "${required_tools[@]}"; do
+    if ! command -v "$tool" &> /dev/null; then
+      missing_deps+=("$tool")
+    fi
+  done
+
+  # Check required Node.js modules
+  local node_modules=(
+    "redis"
+    "dotenv"
+  )
+
+  for module in "${node_modules[@]}"; do
+    if [[ ! -d "node_modules/$module" ]]; then
+      missing_deps+=("$module")
+    fi
+  done
+
+  # Specific Claude Flow dependencies
+  local claude_deps=(
+    "Task tool"
+    "session-manager.js"
+    "redis-coordination scripts"
+  )
+
+  # Use current directory as PROJECT_ROOT if not set
+  PROJECT_ROOT="${PROJECT_ROOT:-$(pwd)}"
+
+  for dep in "${claude_deps[@]}"; do
+    if [[ ! -d "$PROJECT_ROOT/.claude" ]]; then
+      missing_deps+=("$dep")
+    fi
+  done
+
+  # Report missing dependencies
+  if [[ ${#missing_deps[@]} -gt 0 ]]; then
+    log_error "Missing Dependencies:"
+    for dep in "${missing_deps[@]}"; do
+      echo "  - $dep"
+    done
+
+    log_warning "Recommended Installation:"
+    echo "  1. Install Node.js and npm (latest LTS version)"
+    echo "  2. Run: npm install redis dotenv"
+    echo "  3. Clone Claude Flow Novice repository"
+
+    exit 1
+  fi
+}
+
+# ============================================================================
+# AGENT SPAWNING FUNCTIONS
+# ============================================================================
+
+# Spawn agents via CLI
+spawn_agents() {
+  local task="$1"
+  local agents="$2"
+  local agent_id="${3:-main}"
+  local provider="${4:-zai}"
+  local redis_channel="${5:-}"
+
+  log_info "Spawning agents: $agents"
+  log_info "Task: $task"
+
+  # Build spawn command
+  local spawn_cmd="npx claude-flow-spawn \"$task\" --agents=$agents --provider=$provider"
+
+  # Add optional Redis channel
+  if [[ -n "$redis_channel" ]]; then
+    spawn_cmd="$spawn_cmd --redis-channel=$redis_channel"
+  fi
+
+  # Execute spawn
+  eval "$spawn_cmd"
+  local exit_code=$?
+
+  if [[ $exit_code -eq 0 ]]; then
+    log_info "Agents spawned successfully"
+  else
+    log_error "Failed to spawn agents (exit code: $exit_code)"
+    exit $exit_code
+  fi
+}
+
+# Stop agent by task ID
+stop_agent() {
+  local task_id="$1"
+
+  log_info "Stopping agent: $task_id"
+  npx claude-flow-spawn --stop="$task_id"
+
+  if [[ $? -eq 0 ]]; then
+    log_info "Agent stopped successfully"
+  else
+    log_error "Failed to stop agent"
+    exit 1
+  fi
+}
+
+# Stop all agents
+stop_all_agents() {
+  log_info "Stopping all agents"
+  npx claude-flow-spawn --stop-all
+
+  if [[ $? -eq 0 ]]; then
+    log_info "All agents stopped"
+  else
+    log_error "Failed to stop all agents"
+    exit 1
+  fi
+}
+
+# Configuration handler function
+handle_config() {
+  local action="$1"
+  local key="${2:-}"
+  local value="${3:-}"
+
+  case "$action" in
+    list)
+      # Placeholder: List configuration (modify as needed)
+      echo "redis_host=localhost"
+      echo "redis_port=6379"
+      ;;
+    get)
+      # Placeholder: Return config value (modify as needed)
+      case "$key" in
+        redis_host)
+          echo "localhost"
+          ;;
+        *)
+          log_error "Unknown config key: $key"
+          exit 1
+          ;;
+      esac
+      ;;
+    set)
+      # Placeholder: Set config value (modify as needed)
+      log_info "Setting $key to $value"
+      ;;
+    *)
+      log_error "Invalid config action: $action"
+      exit 1
+      ;;
+  esac
+}
+
+# Main function to include dependency check and argument parsing
+main() {
+  check_dependencies  # Dependency check before processing
+
+  # Parse arguments
+  case "${1:-}" in
+    --check-dependencies)
+      log_info "Dependencies checked successfully"
+      ;;
+    --config)
+      shift
+      handle_config "$@"
+      ;;
+    --task)
+      shift
+      local task="$1"
+      shift
+      local agents=""
+      local agent_id="main"
+      local provider="zai"
+      local redis_channel=""
+
+      # Parse remaining arguments
+      while [[ $# -gt 0 ]]; do
+        case "$1" in
+          --agents)
+            agents="$2"
+            shift 2
+            ;;
+          --agent-id)
+            agent_id="$2"
+            shift 2
+            ;;
+          --provider)
+            provider="$2"
+            shift 2
+            ;;
+          --redis-channel)
+            redis_channel="$2"
+            shift 2
+            ;;
+          *)
+            log_error "Unknown argument: $1"
+            exit 1
+            ;;
+        esac
+      done
+
+      # Validate required arguments
+      if [[ -z "$task" ]] || [[ -z "$agents" ]]; then
+        log_error "Missing required arguments: --task and --agents"
+        exit 1
+      fi
+
+      # Spawn agents
+      spawn_agents "$task" "$agents" "$agent_id" "$provider" "$redis_channel"
+      ;;
+    --stop)
+      shift
+      stop_agent "$1"
+      ;;
+    --stop-all)
+      stop_all_agents
+      ;;
+    *)
+      log_error "Unknown argument: $1"
+      echo "Usage:"
+      echo "  spawn-agent.sh --task \"Task description\" --agents coder,tester [--provider zai] [--redis-channel channel]"
+      echo "  spawn-agent.sh --stop <task-id>"
+      echo "  spawn-agent.sh --stop-all"
+      echo "  spawn-agent.sh --check-dependencies"
+      echo "  spawn-agent.sh --config list|get|set"
+      exit 1
+      ;;
+  esac
+}
+
+# Run main function
+main "$@"