@pjmendonca/devflow 1.13.1 → 1.18.0

package/tooling/scripts/memory_summarize.py

@@ -29,11 +29,15 @@ SCRIPT_DIR = Path(__file__).parent.resolve()
 PROJECT_ROOT = SCRIPT_DIR.parent.parent
 MEMORY_DIR = PROJECT_ROOT / ".automation" / "memory"
 
+# Pre-compiled regex patterns for performance
+_ENTRY_WITH_TIMESTAMP_RE = re.compile(r"^-\s*(\d{4}-\d{2}-\d{2}\s+\d{2}:\d{2}):\s*(.+)$")
+_ENTRY_WITHOUT_TIMESTAMP_RE = re.compile(r"^-\s*(.+)$")
+
 
 def parse_memory_entry(line: str) -> tuple[Optional[datetime], str]:
     """Parse a memory entry line into timestamp and content."""
     # Format: - YYYY-MM-DD HH:MM: content
-    match = re.match(r"^-\s*(\d{4}-\d{2}-\d{2}\s+\d{2}:\d{2}):\s*(.+)$", line.strip())
+    match = _ENTRY_WITH_TIMESTAMP_RE.match(line.strip())
     if match:
         try:
             timestamp = datetime.strptime(match.group(1), "%Y-%m-%d %H:%M")
@@ -42,7 +46,7 @@ def parse_memory_entry(line: str) -> tuple[Optional[datetime], str]:
             pass
 
     # Format without timestamp: - content
-    match = re.match(r"^-\s*(.+)$", line.strip())
+    match = _ENTRY_WITHOUT_TIMESTAMP_RE.match(line.strip())
     if match:
         return None, match.group(1).strip()
 
@@ -137,6 +141,9 @@ def find_duplicates(
 
 def summarize_group(entries: list[tuple[Optional[datetime], str]]) -> str:
     """Create a summary for a group of similar entries."""
+    if not entries:
+        return ""
+
     # Keep the most recent entry's content
     sorted_entries = sorted(entries, key=lambda x: x[0] or datetime.min, reverse=True)
     most_recent = sorted_entries[0][1]

package/tooling/scripts/new-doc.py

@@ -19,16 +19,9 @@ import sys
 from pathlib import Path
 
 SCRIPT_DIR = Path(__file__).parent
+sys.path.insert(0, str(SCRIPT_DIR / "lib"))
 
-
-def get_platform():
-    """Detect the current platform."""
-    if sys.platform == "win32":
-        return "windows"
-    elif sys.platform == "darwin":
-        return "macos"
-    else:
-        return "linux"
+from platform import get_platform
 
 
 def run_windows(args):

package/tooling/scripts/personalize_agent.py

@@ -18,18 +18,7 @@ import sys
 from pathlib import Path
 from typing import Optional
 
-
-# Colors for terminal output
-class Colors:
-    HEADER = "\033[95m"
-    BLUE = "\033[94m"
-    CYAN = "\033[96m"
-    GREEN = "\033[92m"
-    YELLOW = "\033[93m"
-    RED = "\033[91m"
-    END = "\033[0m"
-    BOLD = "\033[1m"
-
+from lib.colors import Colors
 
 # Find project paths
 SCRIPT_DIR = Path(__file__).parent.resolve()

package/tooling/scripts/rollback-migration.sh

@@ -82,16 +82,16 @@ create_rollback_point() {
     local timestamp=$(date '+%Y%m%d_%H%M%S')
     local rollback_name="${timestamp}_${name}"
     local rollback_path="$ROLLBACK_DIR/$rollback_name"
-    
+
     mkdir -p "$rollback_path"
-    
+
     echo -e "${CYAN}Creating rollback point: $rollback_name${NC}"
-    
+
     # Save git state
     local current_branch=$(git rev-parse --abbrev-ref HEAD 2>/dev/null || echo "unknown")
     local current_commit=$(git rev-parse HEAD 2>/dev/null || echo "unknown")
     local has_changes=$(git status --porcelain 2>/dev/null | wc -l | tr -d ' ')
-    
+
     # Create manifest
     cat > "$rollback_path/manifest.json" << EOF
 {
@@ -103,7 +103,7 @@ create_rollback_point() {
     "type": "migration-rollback"
 }
 EOF
-    
+
     # Save uncommitted changes if any
     if [[ $has_changes -gt 0 ]]; then
         info "Saving uncommitted changes..."
@@ -111,33 +111,33 @@ EOF
         git stash show -p > "$rollback_path/uncommitted.patch" 2>/dev/null || true
         git stash pop 2>/dev/null || true
     fi
-    
+
     # Save package state
     if [[ -f "$PROJECT_ROOT/package.json" ]]; then
         cp "$PROJECT_ROOT/package.json" "$rollback_path/"
         info "Saved package.json"
     fi
-    
+
     if [[ -f "$PROJECT_ROOT/package-lock.json" ]]; then
         cp "$PROJECT_ROOT/package-lock.json" "$rollback_path/"
         info "Saved package-lock.json"
     fi
-    
+
     if [[ -f "$PROJECT_ROOT/pubspec.yaml" ]]; then
         cp "$PROJECT_ROOT/pubspec.yaml" "$rollback_path/"
         info "Saved pubspec.yaml"
     fi
-    
+
     if [[ -f "$PROJECT_ROOT/pubspec.lock" ]]; then
         cp "$PROJECT_ROOT/pubspec.lock" "$rollback_path/"
         info "Saved pubspec.lock"
     fi
-    
+
     if [[ -f "$PROJECT_ROOT/requirements.txt" ]]; then
         cp "$PROJECT_ROOT/requirements.txt" "$rollback_path/"
         info "Saved requirements.txt"
     fi
-    
+
     success "Rollback point created: $rollback_path"
     echo "$rollback_name"
 }
@@ -146,25 +146,25 @@ EOF
 list_rollback_points() {
     echo -e "${BOLD}Available Rollback Points:${NC}"
     echo ""
-    
+
     if [[ ! -d "$ROLLBACK_DIR" ]] || [[ -z "$(ls -A "$ROLLBACK_DIR" 2>/dev/null)" ]]; then
         info "No rollback points found."
         return 0
     fi
-    
+
     printf "%-25s %-20s %-15s %s\n" "NAME" "CREATED" "COMMIT" "BRANCH"
     printf "%s\n" "$(printf '─%.0s' {1..80})"
-    
+
     for dir in "$ROLLBACK_DIR"/*/; do
         if [[ -d "$dir" ]]; then
             local name=$(basename "$dir")
             local manifest="$dir/manifest.json"
-            
+
             if [[ -f "$manifest" ]]; then
                 local created=$(grep '"created"' "$manifest" | sed 's/.*: *"\([^"]*\)".*/\1/' | cut -c1-19)
                 local commit=$(grep '"commit"' "$manifest" | sed 's/.*: *"\([^"]*\)".*/\1/' | cut -c1-8)
                 local branch=$(grep '"branch"' "$manifest" | sed 's/.*: *"\([^"]*\)".*/\1/')
-                
+
                 printf "%-25s %-20s %-15s %s\n" "$name" "$created" "$commit" "$branch"
             fi
         fi
@@ -175,17 +175,17 @@ list_rollback_points() {
 # Delete old rollback points (keep last N)
 cleanup_rollback_points() {
     local keep="${1:-10}"
-    
+
     if [[ ! -d "$ROLLBACK_DIR" ]]; then
         return 0
     fi
-    
+
     local count=$(ls -d "$ROLLBACK_DIR"/*/ 2>/dev/null | wc -l | tr -d ' ')
-    
+
     if [[ $count -gt $keep ]]; then
         local to_delete=$((count - keep))
         info "Cleaning up $to_delete old rollback point(s)..."
-        
+
         ls -d "$ROLLBACK_DIR"/*/ 2>/dev/null | head -n "$to_delete" | while read dir; do
             rm -rf "$dir"
             success "Deleted: $(basename "$dir")"
@@ -200,35 +200,35 @@ cleanup_rollback_points() {
 # Find migration spec file
 find_migration_spec() {
     local migration_id="$1"
-    
+
     # Search in common locations
     local search_paths=(
         "$MIGRATIONS_DIR/${migration_id}.md"
         "$PROJECT_ROOT/docs/${migration_id}.md"
         "$PROJECT_ROOT/tooling/docs/migrations/${migration_id}.md"
     )
-    
+
     for path in "${search_paths[@]}"; do
         if [[ -f "$path" ]]; then
             echo "$path"
             return 0
         fi
     done
-    
+
     # Search by pattern
     local found=$(find "$PROJECT_ROOT" -name "*${migration_id}*.md" -path "*/migrations/*" 2>/dev/null | head -1)
     if [[ -n "$found" ]]; then
         echo "$found"
         return 0
     fi
-    
+
     return 1
 }
 
 # Parse rollback steps from migration spec
 parse_rollback_steps() {
     local spec_file="$1"
-    
+
     # Extract rollback steps section
     awk '/^### Rollback Steps/,/^##[^#]/' "$spec_file" 2>/dev/null | \
         grep -E '^\s*[0-9]+\.' | \
@@ -239,21 +239,21 @@ parse_rollback_steps() {
 rollback_to_commit() {
     local commit="$1"
     local dry_run="$2"
-    
+
     if [[ "$dry_run" == "true" ]]; then
         info "Would reset to commit: $commit"
         git log --oneline -1 "$commit"
         return 0
     fi
-    
+
     info "Rolling back to commit: $commit"
-    
+
     # Check for uncommitted changes
     if [[ -n "$(git status --porcelain)" ]]; then
         warning "You have uncommitted changes. These will be stashed."
         git stash push -m "pre-rollback-$(date +%s)"
     fi
-    
+
     # Perform rollback
     git checkout "$commit" -- .
     success "Rolled back to commit: $commit"
@@ -264,73 +264,73 @@ rollback_from_point() {
     local point_name="$1"
     local dry_run="$2"
     local point_path="$ROLLBACK_DIR/$point_name"
-    
+
     if [[ ! -d "$point_path" ]]; then
         error "Rollback point not found: $point_name"
     fi
-    
+
     local manifest="$point_path/manifest.json"
     if [[ ! -f "$manifest" ]]; then
         error "Invalid rollback point (no manifest): $point_name"
     fi
-    
+
     local commit=$(grep '"commit"' "$manifest" | sed 's/.*: *"\([^"]*\)".*/\1/')
-    
+
     echo ""
     info "Rollback Point Details:"
     cat "$manifest" | grep -E '"(name|created|branch|commit)"' | sed 's/^/  /'
     echo ""
-    
+
     if [[ "$dry_run" == "true" ]]; then
         info "[DRY RUN] Would restore from rollback point: $point_name"
-        
+
         if [[ -f "$point_path/package.json" ]]; then
             info "[DRY RUN] Would restore package.json"
         fi
         if [[ -f "$point_path/pubspec.yaml" ]]; then
             info "[DRY RUN] Would restore pubspec.yaml"
         fi
-        
+
         return 0
     fi
-    
+
     if ! confirm "This will rollback to the saved state. Continue?"; then
         info "Rollback cancelled."
         return 0
     fi
-    
+
     # Restore files
     if [[ -f "$point_path/package.json" ]]; then
         cp "$point_path/package.json" "$PROJECT_ROOT/"
         success "Restored package.json"
     fi
-    
+
     if [[ -f "$point_path/package-lock.json" ]]; then
         cp "$point_path/package-lock.json" "$PROJECT_ROOT/"
         success "Restored package-lock.json"
     fi
-    
+
     if [[ -f "$point_path/pubspec.yaml" ]]; then
         cp "$point_path/pubspec.yaml" "$PROJECT_ROOT/"
         success "Restored pubspec.yaml"
     fi
-    
+
     if [[ -f "$point_path/pubspec.lock" ]]; then
         cp "$point_path/pubspec.lock" "$PROJECT_ROOT/"
         success "Restored pubspec.lock"
     fi
-    
+
     if [[ -f "$point_path/requirements.txt" ]]; then
         cp "$point_path/requirements.txt" "$PROJECT_ROOT/"
         success "Restored requirements.txt"
     fi
-    
+
     # Rollback git if commit is available
     if [[ -n "$commit" && "$commit" != "unknown" ]]; then
         info "Rolling back git state..."
         rollback_to_commit "$commit" "false"
     fi
-    
+
     # Reinstall dependencies
     echo ""
     info "Dependencies may need to be reinstalled:"
@@ -343,7 +343,7 @@ rollback_from_point() {
     if [[ -f "$PROJECT_ROOT/requirements.txt" ]]; then
         echo "  pip install -r requirements.txt"
     fi
-    
+
     echo ""
     success "Rollback complete!"
 }
@@ -353,43 +353,43 @@ rollback_migration() {
     local migration_id="$1"
     local dry_run="$2"
     local force="$3"
-    
+
     echo -e "${BOLD}Rolling back migration: $migration_id${NC}"
     echo ""
-    
+
     # Look for a rollback point for this migration
     local rollback_point=$(ls -d "$ROLLBACK_DIR"/*"$migration_id"* 2>/dev/null | tail -1)
-    
+
     if [[ -n "$rollback_point" && -d "$rollback_point" ]]; then
         info "Found rollback point: $(basename "$rollback_point")"
         rollback_from_point "$(basename "$rollback_point")" "$dry_run"
         return 0
     fi
-    
+
     # Look for migration spec with rollback instructions
     local spec_file=$(find_migration_spec "$migration_id")
-    
+
     if [[ -n "$spec_file" ]]; then
         info "Found migration spec: $spec_file"
-        
+
         local steps=$(parse_rollback_steps "$spec_file")
-        
+
         if [[ -n "$steps" ]]; then
             echo ""
             echo -e "${BOLD}Rollback Steps:${NC}"
             echo "$steps" | nl
             echo ""
-            
+
             if [[ "$dry_run" == "true" ]]; then
                 info "[DRY RUN] Would execute the above rollback steps"
                 return 0
             fi
-            
+
             if ! confirm "Execute these rollback steps?"; then
                 info "Rollback cancelled."
                 return 0
             fi
-            
+
             # Execute steps (basic implementation - just displays them)
             info "Please execute the following steps manually:"
             echo "$steps" | nl
@@ -399,7 +399,7 @@ rollback_migration() {
     else
         warning "No migration spec found for: $migration_id"
     fi
-    
+
     # Offer git-based rollback
     echo ""
     info "You can also rollback using git:"
@@ -448,7 +448,7 @@ main() {
     local create_name=""
     local restore_point=""
     local cleanup_keep=""
-    
+
     # Parse arguments
     while [[ $# -gt 0 ]]; do
         case "$1" in
@@ -498,12 +498,12 @@ main() {
         esac
         shift
     done
-    
+
     print_header
-    
+
     # Ensure rollback directory exists
     mkdir -p "$ROLLBACK_DIR"
-    
+
     case "$command" in
         list)
             list_rollback_points

package/tooling/scripts/run-collab.ps1

@@ -68,20 +68,20 @@
 param(
     [Parameter(Position=0)]
     [string]$StoryKey,
-    
+
     [switch]$Swarm,
     [switch]$Pair,
     [switch]$Auto,
     [switch]$Sequential,
-    
+
     [string]$Agents,
     [int]$MaxIterations = 3,
-    
+
     [ValidateSet("opus", "sonnet", "haiku")]
     [string]$Model = "opus",
-    
+
     [double]$Budget = 20.0,
-    
+
     [switch]$Memory,
     [string]$Query,
     [switch]$RouteOnly,
@@ -140,11 +140,11 @@ function Show-Usage {
 # Build Python command arguments
 function Build-PythonArgs {
     $args = @()
-    
+
     if ($StoryKey) {
         $args += $StoryKey
     }
-    
+
     if ($Swarm) {
         $args += "--swarm"
     } elseif ($Pair) {
@@ -154,44 +154,44 @@ function Build-PythonArgs {
     } else {
         $args += "--auto"
     }
-    
+
     if ($Agents) {
         $args += "--agents"
         $args += $Agents
     }
-    
+
     if ($MaxIterations -ne 3) {
         $args += "--max-iterations"
         $args += $MaxIterations.ToString()
     }
-    
+
     if ($Model -ne "opus") {
         $args += "--model"
         $args += $Model
     }
-    
+
     if ($Budget -ne 20.0) {
         $args += "--budget"
         $args += $Budget.ToString()
     }
-    
+
     if ($Memory) {
         $args += "--memory"
     }
-    
+
     if ($Query) {
         $args += "--query"
         $args += $Query
     }
-    
+
     if ($RouteOnly) {
         $args += "--route-only"
     }
-    
+
     if ($Quiet) {
         $args += "--quiet"
     }
-    
+
     return $args
 }