@pjmendonca/devflow 1.13.2 → 1.18.0

package/tooling/scripts/create-persona.py

@@ -19,18 +19,7 @@ import sys
 from dataclasses import dataclass, field
 from pathlib import Path
 
-
-# Colors for terminal output
-class Colors:
-    RED = "\033[0;31m"
-    GREEN = "\033[0;32m"
-    YELLOW = "\033[1;33m"
-    BLUE = "\033[0;34m"
-    CYAN = "\033[0;36m"
-    MAGENTA = "\033[0;35m"
-    BOLD = "\033[1m"
-    NC = "\033[0m"
-
+from lib.colors import Colors
 
 # Persona templates
 PERSONA_TEMPLATES = {

package/tooling/scripts/create-persona.sh

@@ -44,15 +44,15 @@ prompt() {
     local message="$1"
     local default="$2"
     local result
-    
+
     if [[ -n "$default" ]]; then
         echo -n -e "${BLUE}${message}${NC} [${default}]: "
     else
         echo -n -e "${BLUE}${message}${NC}: "
     fi
-    
+
     read result
-    
+
     if [[ -z "$result" && -n "$default" ]]; then
         echo "$default"
     else
@@ -65,15 +65,15 @@ prompt_list() {
     local min_items="${2:-0}"
     local items=()
     local count=0
-    
+
     echo -e "${BLUE}${message}${NC}"
     echo "  (Enter items one per line, empty line to finish)"
-    
+
     while true; do
         ((count++))
         echo -n "  $count. "
         read item
-        
+
         if [[ -z "$item" ]]; then
             if [[ ${#items[@]} -ge $min_items ]]; then
                 break
@@ -83,10 +83,10 @@ prompt_list() {
                 continue
             fi
         fi
-        
+
         items+=("$item")
     done
-    
+
     printf '%s\n' "${items[@]}"
 }
 
@@ -96,7 +96,7 @@ prompt_list() {
 
 get_template() {
     local template_name="$1"
-    
+
     case "$template_name" in
         developer|dev)
             echo "developer|Software Developer|Writing clean, maintainable code|sonnet"
@@ -136,9 +136,9 @@ generate_agent_file() {
     local model="$4"
     shift 4
     local responsibilities=("${@}")
-    
+
     local file="$AGENTS_DIR/${name}.md"
-    
+
     cat > "$file" << EOF
 # ${role} Agent
 
@@ -147,13 +147,13 @@ You are a ${role}. ${focus}
 ## Responsibilities
 
 EOF
-    
+
     local count=0
     for resp in "${responsibilities[@]}"; do
         ((count++))
         echo "$count. $resp" >> "$file"
     done
-    
+
     cat >> "$file" << 'EOF'
 
 ## Principles
@@ -185,25 +185,25 @@ If you sense context is running low, output a warning:
 ```
 
 EOF
-    
+
     echo "## Model" >> "$file"
     echo "" >> "$file"
     echo "Recommended model: \`${model}\`" >> "$file"
     echo "" >> "$file"
-    
+
     echo "$file"
 }
 
 generate_override_file() {
     local name="$1"
-    
+
     local file="$OVERRIDES_DIR/${name}.override.yaml"
-    
+
     # Don't overwrite existing
     if [[ -f "$file" ]]; then
         return
     fi
-    
+
     cat > "$file" << EOF
 # ${name^^} Agent Override
 # Customize this agent's behavior without modifying the core agent file
@@ -228,7 +228,7 @@ critical_actions:
 # max_budget_usd: 10.00
 
 EOF
-    
+
     echo "$file"
 }
 
@@ -239,50 +239,50 @@ EOF
 list_personas() {
     echo -e "${BOLD}Available Agent Personas:${NC}"
     echo ""
-    
+
     if [[ ! -d "$AGENTS_DIR" ]]; then
         echo -e "  ${YELLOW}No agents directory found.${NC}"
         return
     fi
-    
+
     printf "  %-15s │ %-25s │ Override\n" "NAME" "ROLE"
     printf "  %s\n" "$(printf '─%.0s' {1..50})"
-    
+
     for agent_file in "$AGENTS_DIR"/*.md; do
         if [[ -f "$agent_file" ]]; then
             local name=$(basename "$agent_file" .md)
             local role=$(head -1 "$agent_file" | sed 's/# //' | sed 's/ Agent$//')
-            
+
             local has_override=" "
             if [[ -f "$OVERRIDES_DIR/${name}.override.yaml" ]]; then
                 has_override="[OK]"
             fi
-            
+
             printf "  ${GREEN}%-15s${NC} │ %-25s │ %s\n" "$name" "$role" "$has_override"
         fi
     done
-    
+
     echo ""
 }
 
 create_from_template() {
     local template_name="$1"
     local persona_name="$2"
-    
+
     local template=$(get_template "$template_name")
-    
+
     if [[ -z "$template" ]]; then
         echo -e "${RED}Unknown template: $template_name${NC}"
         echo "Available templates: developer, reviewer, architect, tester, security, devops, documentation"
         exit 1
     fi
-    
+
     # Parse template
     IFS='|' read -r tpl_name role focus model <<< "$template"
-    
+
     # Use provided name or template name
     local name="${persona_name:-$tpl_name}"
-    
+
     # Default responsibilities
     local responsibilities=(
         "Complete assigned tasks efficiently"
@@ -290,12 +290,12 @@ create_from_template() {
         "Document work appropriately"
         "Communicate progress and blockers"
     )
-    
+
     mkdir -p "$AGENTS_DIR" "$OVERRIDES_DIR"
-    
+
     local agent_file=$(generate_agent_file "$name" "$role" "$focus" "$model" "${responsibilities[@]}")
     local override_file=$(generate_override_file "$name")
-    
+
     echo -e "${GREEN}[OK] Persona created from template!${NC}"
     echo ""
     echo "  Agent file:    $agent_file"
@@ -306,16 +306,16 @@ create_from_template() {
 interactive_create() {
     echo -e "${BOLD}Let's create your custom agent persona!${NC}"
     echo ""
-    
+
     # Get name
     local name=$(prompt "Persona name (e.g., 'qa', 'frontend-dev')")
     name=$(echo "$name" | tr '[:upper:]' '[:lower:]' | tr ' ' '-')
-    
+
     if [[ -z "$name" ]]; then
         echo -e "${RED}Name is required.${NC}"
         exit 1
     fi
-    
+
     # Check if exists
     if [[ -f "$AGENTS_DIR/${name}.md" ]]; then
         echo -e "${YELLOW}Persona '$name' already exists.${NC}"
@@ -324,25 +324,25 @@ interactive_create() {
             exit 0
         fi
     fi
-    
+
     # Get details
     echo ""
     local role=$(prompt "Role (e.g., 'Senior QA Engineer')")
     local focus=$(prompt "Focus (one-line description)")
-    
+
     echo ""
     echo "Select model:"
     echo "  1. sonnet (default - balanced)"
     echo "  2. opus (complex tasks)"
     echo "  3. haiku (quick tasks)"
     local model_choice=$(prompt "Choice" "1")
-    
+
     local model="sonnet"
     case "$model_choice" in
         2|opus) model="opus" ;;
         3|haiku) model="haiku" ;;
     esac
-    
+
     echo ""
     echo "Enter responsibilities (what this agent does):"
     local responsibilities=()
@@ -361,13 +361,13 @@ interactive_create() {
         fi
         responsibilities+=("$resp")
     done
-    
+
     # Create files
     mkdir -p "$AGENTS_DIR" "$OVERRIDES_DIR"
-    
+
     local agent_file=$(generate_agent_file "$name" "$role" "$focus" "$model" "${responsibilities[@]}")
     local override_file=$(generate_override_file "$name")
-    
+
     echo ""
     echo -e "${GREEN}[OK] Persona created successfully!${NC}"
     echo ""
@@ -405,7 +405,7 @@ print_usage() {
 
 main() {
     print_header
-    
+
     case "$1" in
         --list|-l)
             list_personas

package/tooling/scripts/lib/__init__.py

@@ -2,34 +2,45 @@
 Devflow Library - Core modules for the Devflow automation system.
 
 This package provides:
+- colors: Shared terminal color codes
+- platform: Cross-platform detection utilities
 - cost_tracker: Token usage and cost tracking
 - cost_display: Terminal-based cost monitoring display
 - cost_config: Configuration management for costs
 - currency_converter: Multi-currency support
+- context_monitor: Real-time context window tracking
 - errors: Enhanced error handling
 - agent_router: Dynamic agent selection
 - agent_handoff: Structured agent transitions
 - shared_memory: Cross-agent knowledge sharing
 - swarm_orchestrator: Multi-agent collaboration
 - pair_programming: DEV + REVIEWER collaboration
+- validation_loop: Three-tier validation framework
 
 Usage:
     from lib.cost_tracker import CostTracker
+    from lib.context_monitor import ContextMonitor, StatusLine, get_status_manager
     from lib.agent_router import AgentRouter
+    from lib.colors import Colors
+    from lib.platform import get_platform, IS_WINDOWS
 """
 
-__version__ = "1.13.2"
+__version__ = "1.18.0"
 
 # Lazy imports to avoid circular dependencies
 __all__ = [
+    "colors",
+    "platform",
     "cost_tracker",
     "cost_display",
     "cost_config",
     "currency_converter",
+    "context_monitor",
     "errors",
     "agent_router",
     "agent_handoff",
     "shared_memory",
     "swarm_orchestrator",
     "pair_programming",
+    "validation_loop",
 ]

package/tooling/scripts/lib/agent_handoff.py

@@ -226,7 +226,12 @@ class HandoffGenerator:
                     )
 
             return changes
-        except Exception:
+        except subprocess.SubprocessError as e:
+            # Git command failed - likely not a git repo or git not installed
+            print(f"Warning: Could not get git changes: {e}")
+            return []
+        except OSError as e:
+            print(f"Warning: Error accessing git: {e}")
             return []
 
     def get_staged_changes(self) -> list[FileChange]:
@@ -255,7 +260,11 @@ class HandoffGenerator:
                     )
 
             return changes
-        except Exception:
+        except subprocess.SubprocessError as e:
+            print(f"Warning: Could not get staged changes: {e}")
+            return []
+        except OSError as e:
+            print(f"Warning: Error accessing git for staged changes: {e}")
             return []
 
     def extract_decisions_from_memory(self, agent: str) -> list[str]:

package/tooling/scripts/lib/agent_router.py

@@ -29,6 +29,14 @@ from enum import Enum
 from pathlib import Path
 from typing import Optional
 
+# Configuration constants (can be overridden via environment or config)
+CONFIDENCE_BASE = 0.3  # Base confidence score
+CONFIDENCE_PATTERN_WEIGHT = 0.1  # Weight per detected pattern
+CONFIDENCE_FILE_CONTEXT_WEIGHT = 0.1  # Weight per file context match
+COMPLEXITY_SIMPLE_THRESHOLD = 2  # Complexity <= this is "simple"
+COST_OPT_COMPLEXITY_THRESHOLD = 3  # Max complexity for cost optimization
+MAX_ALTERNATIVES = 3  # Maximum alternative agents to suggest
+
 
 class TaskType(Enum):
     """Types of development tasks."""
@@ -242,6 +250,12 @@ TASK_PATTERNS = {
     ],
 }
 
+# Pre-compile regex patterns for performance
+COMPILED_TASK_PATTERNS: dict[TaskType, list[re.Pattern]] = {
+    task_type: [re.compile(pattern, re.IGNORECASE) for pattern in patterns]
+    for task_type, patterns in TASK_PATTERNS.items()
+}
+
 # File extension to specialty mapping
 FILE_SPECIALTIES = {
     # Security-sensitive files
@@ -337,14 +351,14 @@ class AgentRouter:
         """
         description_lower = description.lower()
 
-        # Detect task type from patterns
+        # Detect task type from pre-compiled patterns (faster than re.findall each time)
         type_scores: dict[TaskType, int] = {}
         detected_patterns: list[str] = []
 
-        for task_type, patterns in TASK_PATTERNS.items():
+        for task_type, compiled_patterns in COMPILED_TASK_PATTERNS.items():
             score = 0
-            for pattern in patterns:
-                matches = re.findall(pattern, description_lower)
+            for pattern in compiled_patterns:
+                matches = pattern.findall(description_lower)
                 if matches:
                     score += len(matches)
                     detected_patterns.extend(matches)
@@ -368,8 +382,13 @@ class AgentRouter:
         # Estimate complexity
         complexity = self._estimate_complexity(description, files)
 
-        # Calculate confidence
-        confidence = min(1.0, 0.3 + (0.1 * len(detected_patterns)) + (0.1 * len(file_contexts)))
+        # Calculate confidence using configurable weights
+        confidence = min(
+            1.0,
+            CONFIDENCE_BASE
+            + (CONFIDENCE_PATTERN_WEIGHT * len(detected_patterns))
+            + (CONFIDENCE_FILE_CONTEXT_WEIGHT * len(file_contexts)),
+        )
 
         return TaskAnalysis(
             task_type=task_type,
@@ -562,7 +581,9 @@ class AgentRouter:
         }
 
         # Determine complexity level
-        complexity_level = "simple" if complexity.value <= 2 else "complex"
+        complexity_level = (
+            "simple" if complexity.value <= COMPLEXITY_SIMPLE_THRESHOLD else "complex"
+        )
 
         # Get routing
         rule = routing_rules.get(task_type, {}).get(complexity_level)
@@ -572,11 +593,11 @@ class AgentRouter:
         agents, workflow, reasoning = rule
 
         # Cost optimization
-        if prefer_cost and complexity.value <= 3:
+        if prefer_cost and complexity.value <= COST_OPT_COMPLEXITY_THRESHOLD:
             # Replace Opus agents with Sonnet equivalents where possible
             cost_effective_agents = []
             for agent in agents:
-                if agent == "REVIEWER" and complexity.value <= 2:
+                if agent == "REVIEWER" and complexity.value <= COMPLEXITY_SIMPLE_THRESHOLD:
                     cost_effective_agents.append("SM")  # SM can do simple reviews
                 else:
                     cost_effective_agents.append(agent)
@@ -601,7 +622,7 @@ class AgentRouter:
             if matches and agent.max_complexity >= analysis.complexity.value:
                 alternatives.append(name)
 
-        return alternatives[:3]  # Top 3 alternatives
+        return alternatives[:MAX_ALTERNATIVES]
 
     def get_workflow_for_agents(self, agents: list[str]) -> str:
         """Determine the best workflow for a set of agents."""

package/tooling/scripts/lib/colors.py

@@ -0,0 +1,106 @@
+#!/usr/bin/env python3
+"""
+Shared Colors Module for Devflow.
+
+Provides consistent ANSI color codes for terminal output across all scripts.
+
+Usage:
+    from lib.colors import Colors
+
+    print(f"{Colors.GREEN}Success!{Colors.RESET}")
+    print(f"{Colors.BOLD_RED}Error!{Colors.RESET}")
+"""
+
+import os
+import sys
+
+
+def _supports_color() -> bool:
+    """Check if the terminal supports color output."""
+    # Check for explicit NO_COLOR environment variable
+    if os.getenv("NO_COLOR"):
+        return False
+
+    # Check for explicit FORCE_COLOR
+    if os.getenv("FORCE_COLOR"):
+        return True
+
+    # Check if output is a TTY
+    if not hasattr(sys.stdout, "isatty") or not sys.stdout.isatty():
+        return False
+
+    # Windows-specific handling
+    if sys.platform == "win32":
+        try:
+            import ctypes
+
+            kernel32 = ctypes.windll.kernel32
+            # Enable ANSI escape sequences on Windows 10+
+            kernel32.SetConsoleMode(kernel32.GetStdHandle(-11), 7)
+            return True
+        except Exception:
+            return False
+
+    return True
+
+
+class Colors:
+    """ANSI color codes for terminal output."""
+
+    _USE_COLORS = _supports_color()
+
+    # Reset
+    RESET = "\033[0m" if _USE_COLORS else ""
+    END = RESET  # Alias for compatibility
+
+    # Regular colors
+    BLACK = "\033[30m" if _USE_COLORS else ""
+    RED = "\033[31m" if _USE_COLORS else ""
+    GREEN = "\033[32m" if _USE_COLORS else ""
+    YELLOW = "\033[33m" if _USE_COLORS else ""
+    BLUE = "\033[34m" if _USE_COLORS else ""
+    MAGENTA = "\033[35m" if _USE_COLORS else ""
+    CYAN = "\033[36m" if _USE_COLORS else ""
+    WHITE = "\033[37m" if _USE_COLORS else ""
+
+    # Bold colors
+    BOLD = "\033[1m" if _USE_COLORS else ""
+    BOLD_RED = "\033[1;31m" if _USE_COLORS else ""
+    BOLD_GREEN = "\033[1;32m" if _USE_COLORS else ""
+    BOLD_YELLOW = "\033[1;33m" if _USE_COLORS else ""
+    BOLD_BLUE = "\033[1;34m" if _USE_COLORS else ""
+    BOLD_MAGENTA = "\033[1;35m" if _USE_COLORS else ""
+    BOLD_CYAN = "\033[1;36m" if _USE_COLORS else ""
+    BOLD_WHITE = "\033[1;37m" if _USE_COLORS else ""
+
+    # Background colors
+    BG_RED = "\033[41m" if _USE_COLORS else ""
+    BG_GREEN = "\033[42m" if _USE_COLORS else ""
+    BG_YELLOW = "\033[43m" if _USE_COLORS else ""
+    BG_BLUE = "\033[44m" if _USE_COLORS else ""
+
+    # Styles
+    DIM = "\033[2m" if _USE_COLORS else ""
+    UNDERLINE = "\033[4m" if _USE_COLORS else ""
+
+    # Common aliases used across codebase
+    HEADER = BOLD_MAGENTA
+    OKBLUE = BLUE
+    OKCYAN = CYAN
+    OKGREEN = GREEN
+    WARNING = YELLOW
+    FAIL = RED
+    ENDC = RESET
+    NC = RESET  # No Color alias
+
+    @staticmethod
+    def strip(text: str) -> str:
+        """Remove ANSI codes from text."""
+        import re
+
+        return re.sub(r"\033\[[0-9;]*m", "", text)
+
+    @classmethod
+    def enabled(cls) -> bool:
+        """Check if colors are enabled."""
+        return cls._USE_COLORS