@pennyfarthing/core 7.9.1 → 7.9.2

package/pennyfarthing-dist/scripts/lib/retry.sh

@@ -1,76 +0,0 @@
-#!/usr/bin/env zsh
-# Retry utilities with exponential backoff
-# Dev: Fanny Price - "We have all a better guide in ourselves, if we would attend to it."
-
-# retry_with_backoff MAX_ATTEMPTS INITIAL_DELAY MAX_DELAY COMMAND...
-# Retry a command with exponential backoff
-#
-# Arguments:
-#   MAX_ATTEMPTS   - Maximum number of attempts (default: 3)
-#   INITIAL_DELAY  - Initial delay in seconds (default: 1)
-#   MAX_DELAY      - Maximum delay cap in seconds (default: 30)
-#   COMMAND...     - Command to execute
-#
-# Returns:
-#   0 if command succeeds within attempts
-#   1 if all attempts exhausted
-#
-# Example:
-#   retry_with_backoff 3 1 10 curl -s https://api.example.com/health
-#
-retry_with_backoff() {
-    local max_attempts=${1:-3}
-    local delay=${2:-1}
-    local max_delay=${3:-30}
-    shift 3
-
-    local attempt=1
-    while true; do
-        # Try the command
-        if "$@"; then
-            return 0
-        fi
-
-        # Check if we've exhausted attempts
-        if ((attempt >= max_attempts)); then
-            return 1
-        fi
-
-        # Wait before retry
-        sleep "$delay"
-
-        # Exponential backoff with cap
-        delay=$((delay * 2))
-        if ((delay > max_delay)); then
-            delay=$max_delay
-        fi
-
-        attempt=$((attempt + 1))
-    done
-}
-
-# command_with_fallback PRIMARY_CMD FALLBACK_CMD
-# Execute primary command, fall back to alternative if it fails
-#
-# Arguments:
-#   PRIMARY_CMD   - Primary command to try (as string)
-#   FALLBACK_CMD  - Fallback command if primary fails (as string)
-#
-# Returns:
-#   Exit code of whichever command succeeds (or fallback's exit code)
-#
-# Example:
-#   command_with_fallback "git pull --ff-only" "git pull --no-rebase"
-#
-command_with_fallback() {
-    local primary_cmd="$1"
-    local fallback_cmd="$2"
-
-    if eval "$primary_cmd"; then
-        return 0
-    else
-        eval "$fallback_cmd"
-    fi
-}
-
-# Functions available when sourced - no export needed in zsh

package/pennyfarthing-dist/scripts/maintenance/migrate-theme-schema.mjs

@@ -1,102 +0,0 @@
-#!/usr/bin/env node
-/**
- * Migration Script: Consolidate quote into catchphrases (MSSCI-12478)
- * 
- * For each theme YAML:
- * 1. For each agent: if quote exists and isn't in catchphrases, add it
- * 2. Remove the quote field from all agents
- */
-
-import { readFileSync, writeFileSync, readdirSync } from 'node:fs';
-import { join, dirname } from 'node:path';
-import { fileURLToPath } from 'node:url';
-
-const __filename = fileURLToPath(import.meta.url);
-const __dirname = dirname(__filename);
-
-const THEMES_DIR = join(__dirname, '..', '..', 'personas', 'themes');
-
-function migrateThemeFile(filePath) {
-  let content = readFileSync(filePath, 'utf-8');
-  const lines = content.split('\n');
-  const result = [];
-  
-  let i = 0;
-  while (i < lines.length) {
-    const line = lines[i];
-    const trimmed = line.trimEnd();
-    
-    // Check for quote: at 4-space indent (agent level)
-    if (/^    quote:/.test(trimmed)) {
-      // Extract the quote value
-      const quoteMatch = trimmed.match(/^    quote:\s*["']?(.+?)["']?$/);
-      if (quoteMatch) {
-        const quoteValue = quoteMatch[1];
-        
-        // Look ahead for catchphrases section
-        let catchphrasesIndex = -1;
-        let catchphrases = [];
-        
-        for (let j = i + 1; j < lines.length; j++) {
-          const checkLine = lines[j];
-          // Stop if we hit the next agent (2-space indent with colon) or end of agents
-          if (/^  \w+:/.test(checkLine) && !/^    /.test(checkLine)) break;
-          if (/^agents:/.test(checkLine) || /^theme:/.test(checkLine)) break;
-          
-          if (/^    catchphrases:/.test(checkLine)) {
-            catchphrasesIndex = j;
-            // Collect existing catchphrases
-            for (let k = j + 1; k < lines.length; k++) {
-              const catchLine = lines[k];
-              if (/^      - /.test(catchLine)) {
-                const catchValue = catchLine.replace(/^      - ["']?(.+?)["']?$/, '$1');
-                catchphrases.push(catchValue);
-              } else if (!/^\s*$/.test(catchLine) && !/^      /.test(catchLine)) {
-                break;
-              }
-            }
-            break;
-          }
-        }
-        
-        // Check if quote is already in catchphrases
-        const quoteInCatchphrases = catchphrases.some(c => c === quoteValue);
-        
-        if (!quoteInCatchphrases && catchphrasesIndex === -1) {
-          // No catchphrases section - this shouldn't happen based on exploration
-          // but handle it anyway by skipping quote (tests expect catchphrases to exist)
-          console.log(`  Warning: No catchphrases found for agent with quote in ${filePath}`);
-        }
-        
-        // Skip the quote line (don't add to result)
-        i++;
-        continue;
-      }
-    }
-    
-    result.push(line);
-    i++;
-  }
-  
-  const newContent = result.join('\n');
-  if (newContent !== content) {
-    writeFileSync(filePath, newContent);
-    return true;
-  }
-  return false;
-}
-
-// Main
-const files = readdirSync(THEMES_DIR).filter(f => f.endsWith('.yaml'));
-console.log(`Migrating ${files.length} theme files...`);
-
-let modified = 0;
-for (const file of files) {
-  const filePath = join(THEMES_DIR, file);
-  if (migrateThemeFile(filePath)) {
-    modified++;
-    console.log(`  Migrated: ${file}`);
-  }
-}
-
-console.log(`\nDone. Modified ${modified}/${files.length} files.`);

package/pennyfarthing-dist/scripts/maintenance/sidecar-health.sh

@@ -1,97 +0,0 @@
-#!/bin/bash
-# sidecar-health.sh - Check sidecar files for bloat and staleness
-#
-# Usage: sidecar-health.sh [--fix]
-#   --fix    Offer to archive bloated files
-#
-# Thresholds:
-#   gotchas.md: 50 lines max
-#   patterns.md: 50 lines max
-#   decisions.md: 40 lines max
-
-set -euo pipefail
-
-# Find project root
-if [[ -z "${PROJECT_ROOT:-}" ]]; then
-  d="$PWD"
-  while [[ ! -d "$d/.pennyfarthing" ]] && [[ "$d" != "/" ]]; do
-    d="$(dirname "$d")"
-  done
-  PROJECT_ROOT="$d"
-fi
-
-SIDECAR_DIR="$PROJECT_ROOT/.pennyfarthing/sidecars"
-FIX_MODE="${1:-}"
-
-# Thresholds
-GOTCHAS_MAX=50
-PATTERNS_MAX=50
-DECISIONS_MAX=40
-
-# Colors
-RED='\033[0;31m'
-YELLOW='\033[1;33m'
-GREEN='\033[0;32m'
-NC='\033[0m' # No Color
-
-echo "Sidecar Health Check"
-echo "===================="
-echo ""
-
-issues_found=0
-
-check_file() {
-  local file="$1"
-  local max_lines="$2"
-  local agent=$(basename "$(dirname "$file")")
-  local filename=$(basename "$file")
-
-  if [[ ! -f "$file" ]]; then
-    return
-  fi
-
-  local lines=$(wc -l < "$file" | tr -d ' ')
-
-  if [[ $lines -gt $max_lines ]]; then
-    echo -e "${RED}BLOATED${NC}: $agent/$filename ($lines lines, max $max_lines)"
-    issues_found=$((issues_found + 1))
-
-    if [[ "$FIX_MODE" == "--fix" ]]; then
-      local archive_dir="$PROJECT_ROOT/.pennyfarthing/sidecars/.archive"
-      local timestamp=$(date +%Y%m%d)
-      mkdir -p "$archive_dir"
-
-      echo "  → Archiving to .archive/${agent}-${filename%.md}-${timestamp}.md"
-      cp "$file" "$archive_dir/${agent}-${filename%.md}-${timestamp}.md"
-      echo "  → Original preserved. Manually prune $file to <$max_lines lines."
-    fi
-  elif [[ $lines -gt $((max_lines * 80 / 100)) ]]; then
-    echo -e "${YELLOW}WARNING${NC}: $agent/$filename ($lines lines, approaching $max_lines limit)"
-  else
-    echo -e "${GREEN}OK${NC}: $agent/$filename ($lines lines)"
-  fi
-}
-
-# Check all agent sidecars
-for agent_dir in "$SIDECAR_DIR"/*/; do
-  if [[ -d "$agent_dir" ]]; then
-    agent=$(basename "$agent_dir")
-    [[ "$agent" == ".archive" ]] && continue
-
-    check_file "$agent_dir/gotchas.md" $GOTCHAS_MAX
-    check_file "$agent_dir/patterns.md" $PATTERNS_MAX
-    check_file "$agent_dir/decisions.md" $DECISIONS_MAX
-  fi
-done
-
-echo ""
-if [[ $issues_found -gt 0 ]]; then
-  echo "Found $issues_found bloated file(s)."
-  if [[ "$FIX_MODE" != "--fix" ]]; then
-    echo "Run with --fix to archive and prepare for pruning."
-  fi
-  exit 1
-else
-  echo "All sidecar files within limits."
-  exit 0
-fi

package/pennyfarthing-dist/scripts/misc/README.md

@@ -1,44 +0,0 @@
-# Miscellaneous Scripts
-
-Uncategorized utilities and one-off scripts.
-
-## Scripts
-
-| Script | Purpose |
-|--------|---------|
-| `deploy.sh` | Deployment script |
-| `doctor-dogfood.sh` | Health check for dogfooding setup |
-| `run-ci.sh` | Run CI locally |
-| `statusline.sh` | Status line configuration |
-| `uninstall.sh` | Uninstall Pennyfarthing from project |
-| `backlog.sh` | Backlog operations |
-| `repo-utils.sh` | Multi-repo utilities |
-| `repo-scan.sh` | Repository scanner |
-| `add-short-names.mjs` | Add short names to themes |
-| `migrate-bmad-workflow.mjs` | BMAD workflow migration |
-| `migrate-bmad-workflow.sh` | BMAD migration wrapper |
-| `find-related-work.sh` | Find related work items |
-| `check-status.sh` | Status checking utilities |
-| `session-cleanup.sh` | Clean up old sessions |
-| `generate-skill-docs.sh` | Generate skill documentation |
-| `log-skill-usage.sh` | Log skill invocations |
-| `skill-usage-report.sh` | Generate skill usage reports |
-| `run-timestamp.sh` | Timestamp utility |
-| `validate-subagent-frontmatter.sh` | Validate subagent YAML frontmatter |
-
-## Usage
-
-```bash
-.pennyfarthing/scripts/core/run.sh misc/doctor-dogfood.sh
-.pennyfarthing/scripts/core/run.sh misc/run-ci.sh
-```
-
-## Ownership
-
-- **Primary users:** Various
-- **Maintained by:** Core Pennyfarthing team
-
-## Note
-
-Scripts here may be candidates for moving to more specific categories
-as the codebase evolves.

package/pennyfarthing-dist/scripts/misc/add-short-names.sh

@@ -1,13 +0,0 @@
-#!/usr/bin/env bash
-# add-short-names.sh - Pre-generate shortName field for theme characters
-#
-# Usage:
-#   add-short-names.sh                    # Dry run - show what would change
-#   add-short-names.sh --write            # Actually write changes
-#   add-short-names.sh --theme discworld  # Only process one theme
-
-set -euo pipefail
-
-SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
-
-exec python3 "$SCRIPT_DIR/add_short_names.py" "$@"

package/pennyfarthing-dist/scripts/misc/add_short_names.py

@@ -1,226 +0,0 @@
-#!/usr/bin/env python3
-"""
-add_short_names.py - Pre-generate shortName field for theme characters
-
-Finds the shortest unique identifier that distinguishes each character.
-
-Usage:
-  python add_short_names.py                    # Dry run - show what would change
-  python add_short_names.py --write            # Actually write changes
-  python add_short_names.py --theme discworld  # Only process one theme
-"""
-
-import argparse
-import re
-import sys
-from pathlib import Path
-
-try:
-    import yaml
-except ImportError:
-    print("Error: PyYAML required. Install with: pip install pyyaml", file=sys.stderr)
-    sys.exit(1)
-
-
-def find_project_root() -> Path:
-    """Find project root by looking for .pennyfarthing directory."""
-    current = Path.cwd()
-    while current != current.parent:
-        if (current / ".pennyfarthing").is_dir():
-            return current
-        current = current.parent
-    return Path.cwd()
-
-
-# Common titles/prefixes to strip for comparison
-SKIP_PREFIXES = {
-    'the', 'dr.', 'dr', 'captain', 'admiral', 'colonel', 'lieutenant', 'commander',
-    'president', 'lord', 'lady', 'sir', 'professor', 'inspector', 'sergeant',
-    'mr.', 'mr', 'mrs.', 'mrs', 'miss', 'ms.', 'ms', 'chief', 'major', 'general',
-    'king', 'queen', 'prince', 'princess', 'duke', 'earl', 'count', 'baron',
-    'first', 'grand', 'arch', 'high',
-    'uncle', 'aunt', 'brother', 'sister', 'father', 'mother', 'friar',
-    'avatar', 'agent', 'detective', 'officer', 'private', 'corporal',
-    'chancellor', 'ambassador', 'senator', 'governor', 'minister',
-    'master', 'young', 'old', 'elder', 'reverend', 'bishop', 'cardinal'
-}
-
-# Words that make poor short names on their own
-POOR_SHORT_NAMES = {
-    'big', 'little', 'old', 'young', 'true', 'false', 'good', 'bad',
-    'thought', 'ministry', 'situation', 'room', 'place', 'house',
-    'superintendent', 'commander', 'speaker', 'council',
-    'mode', 'narrator', 'chronicler',
-    'h.m.', 'j.f.', 'a.w.', 'e.b.', 'l.'
-}
-
-# Names that should use the full form (iconic two-word names)
-USE_FULL_NAME = {'big brother', 'sun tzu'}
-
-
-def extract_nickname(name: str) -> str | None:
-    """Extract quoted nickname from character name if present."""
-    match = re.search(r'["\']([^"\']+)["\']', name)
-    if match:
-        nickname = match.group(1).strip()
-        if nickname and ' ' not in nickname:
-            return nickname
-    return None
-
-
-def clean_name(name: str) -> str:
-    """Clean character name by removing parenthetical annotations and slash alternatives."""
-    cleaned = re.sub(r'\s*\([^)]+\)\s*', ' ', name).strip()
-    cleaned = re.sub(r'\b\w+/(\w+)\s', r'\1 ', cleaned)
-    cleaned = re.sub(r'\s*["\'][^"\']+["\']\s*', ' ', cleaned).strip()
-    cleaned = re.sub(r'\s+', ' ', cleaned).strip()
-    return cleaned
-
-
-def tokenize(name: str) -> list[str]:
-    """Tokenize a name into meaningful parts."""
-    cleaned = clean_name(name)
-    words = [w for w in cleaned.split() if w]
-
-    filtered = [
-        w for w in words
-        if w.lower() not in SKIP_PREFIXES
-        and not re.match(r'^[A-Z]\.$', w)
-        and not re.match(r'^[IVXLCDM]+$', w)
-    ]
-
-    return filtered if filtered else words
-
-
-def compute_short_names(agents: dict) -> dict[str, str]:
-    """Compute display name map for all characters in a theme."""
-    characters = [
-        a['character'] for a in agents.values()
-        if a and a.get('character')
-    ]
-
-    def is_unique(candidate: str, except_for: str) -> bool:
-        candidate_lower = candidate.lower()
-        for char in characters:
-            if char == except_for:
-                continue
-            tokens = tokenize(char)
-            if any(t.lower() == candidate_lower for t in tokens):
-                return False
-        return True
-
-    def is_good_short_name(candidate: str) -> bool:
-        return candidate.lower() not in POOR_SHORT_NAMES and len(candidate) > 1
-
-    def find_short_name(full_name: str) -> str:
-        cleaned = clean_name(full_name)
-
-        if cleaned.lower() in USE_FULL_NAME:
-            return cleaned
-
-        nickname = extract_nickname(full_name)
-        if nickname and is_good_short_name(nickname):
-            return nickname
-
-        tokens = tokenize(full_name)
-
-        if not tokens:
-            return cleaned
-
-        if len(tokens) == 1:
-            return tokens[0]
-
-        # Strategy 1: First token (if good and unique)
-        if is_good_short_name(tokens[0]) and is_unique(tokens[0], full_name):
-            return tokens[0]
-
-        # Strategy 2: Last token (surname, if good and unique)
-        last_token = tokens[-1]
-        if is_good_short_name(last_token) and is_unique(last_token, full_name):
-            return last_token
-
-        # Strategy 3: First + Last
-        if len(tokens) >= 2:
-            first_last = f"{tokens[0]} {last_token}"
-            if is_unique(first_last, full_name):
-                return first_last
-
-        return clean_name(full_name)
-
-    return {char: find_short_name(char) for char in characters}
-
-
-def process_theme(filepath: Path, dry_run: bool = True) -> dict:
-    """Process a single theme file."""
-    content = filepath.read_text()
-    theme = yaml.safe_load(content)
-
-    if not theme or 'agents' not in theme:
-        print(f"  Skipping {filepath.name} - no agents found")
-        return {'changes': 0, 'filename': filepath.name}
-
-    short_names = compute_short_names(theme['agents'])
-    changes = 0
-
-    for role, agent in theme['agents'].items():
-        if not agent or 'character' not in agent:
-            continue
-
-        short_name = short_names.get(agent['character'])
-        existing = agent.get('shortName')
-
-        if short_name and short_name != existing:
-            if dry_run:
-                existing_note = f' (was: "{existing}")' if existing else ''
-                print(f'  {role}: "{agent["character"]}" -> "{short_name}"{existing_note}')
-            agent['shortName'] = short_name
-            changes += 1
-
-    if not dry_run and changes > 0:
-        with open(filepath, 'w') as f:
-            yaml.dump(theme, f, default_flow_style=False, allow_unicode=True, width=1000)
-        print(f"  Wrote {changes} changes to {filepath.name}")
-
-    return {'changes': changes, 'filename': filepath.name}
-
-
-def main() -> int:
-    parser = argparse.ArgumentParser(
-        description="Pre-generate shortName field for theme characters"
-    )
-    parser.add_argument('--write', action='store_true',
-                        help='Actually write changes (default: dry run)')
-    parser.add_argument('--theme', type=str,
-                        help='Only process one theme')
-    args = parser.parse_args()
-
-    dry_run = not args.write
-
-    project_root = find_project_root()
-    themes_dir = project_root / 'pennyfarthing-dist' / 'personas' / 'themes'
-
-    print('DRY RUN - No files will be modified\n' if dry_run else 'WRITING CHANGES\n')
-
-    files = sorted(themes_dir.glob('*.yaml'))
-    if args.theme:
-        theme_file = f"{args.theme}.yaml" if not args.theme.endswith('.yaml') else args.theme
-        files = [f for f in files if f.name == theme_file]
-
-    total_changes = 0
-    for file in files:
-        print(f"\n{file.name}:")
-        result = process_theme(file, dry_run)
-        total_changes += result['changes']
-        if result['changes'] == 0:
-            print('  (no changes needed)')
-
-    print(f"\n{'=' * 50}")
-    print(f"Total: {total_changes} changes across {len(files)} themes")
-    if dry_run and total_changes > 0:
-        print("\nRun with --write to apply changes")
-
-    return 0
-
-
-if __name__ == "__main__":
-    sys.exit(main())

package/pennyfarthing-dist/scripts/misc/backlog.sh

@@ -1,91 +0,0 @@
-#!/usr/bin/env zsh
-# backlog.sh - Display backlog stories in a formatted table
-# Usage: backlog.sh
-# Reads sprint/current-sprint.yaml and outputs backlog stories
-
-set -euo pipefail
-
-SCRIPT_DIR="${0:A:h}"
-PROJECT_ROOT="${SCRIPT_DIR}/../.."
-
-# Find project root by looking for .pennyfarthing directory
-find_project_root() {
-    local dir="$PWD"
-    while [[ "$dir" != "/" ]]; do
-        if [[ -d "$dir/.pennyfarthing" ]]; then
-            echo "$dir"
-            return 0
-        fi
-        dir="$(dirname "$dir")"
-    done
-    echo "$PWD"
-}
-
-PROJECT_ROOT="$(find_project_root)"
-SPRINT_FILE="$PROJECT_ROOT/sprint/current-sprint.yaml"
-
-if [[ ! -f "$SPRINT_FILE" ]]; then
-    echo "Error: sprint/current-sprint.yaml not found" >&2
-    exit 1
-fi
-
-# Parse YAML and extract backlog stories
-# Output format: | Story | Title | Pts | Epic |
-
-echo "| Story | Title | Pts | Epic |"
-echo "|-------|-------|-----|------|"
-
-# Use yq if available, otherwise fall back to grep/awk
-if command -v yq &>/dev/null; then
-    yq eval '
-        .epics[] |
-        .id as $epic_id |
-        .stories[]? |
-        select(.status == "backlog") |
-        "| " + .id + " | " + .title + " | " + (.points | tostring) + " | " + $epic_id + " |"
-    ' "$SPRINT_FILE"
-else
-    # Fallback: grep-based extraction
-    current_epic=""
-    in_stories=false
-
-    while IFS= read -r line; do
-        # Detect epic id
-        if [[ "$line" =~ ^[[:space:]]*-[[:space:]]*id:[[:space:]]*(MSSCI-[0-9]+) ]]; then
-            current_epic="${match[1]}"
-            in_stories=false
-        fi
-
-        # Detect stories section
-        if [[ "$line" =~ ^[[:space:]]*stories: ]]; then
-            in_stories=true
-        fi
-
-        # Detect story with backlog status (look ahead)
-        if [[ "$in_stories" == true && "$line" =~ ^[[:space:]]*-[[:space:]]*id:[[:space:]]*(MSSCI-[0-9]+) ]]; then
-            story_id="${match[1]}"
-            story_title=""
-            story_points=""
-            story_status=""
-
-            # Read next lines for this story
-            while IFS= read -r detail; do
-                [[ "$detail" =~ ^[[:space:]]*-[[:space:]]*id: ]] && break
-                [[ "$detail" =~ ^[[:space:]]*title:[[:space:]]*[\"\']*(.+)[\"\']*$ ]] && story_title="${match[1]}"
-                [[ "$detail" =~ ^[[:space:]]*points:[[:space:]]*([0-9]+) ]] && story_points="${match[1]}"
-                [[ "$detail" =~ ^[[:space:]]*status:[[:space:]]*(.+) ]] && story_status="${match[1]}"
-
-                # If we have all fields and status is backlog, print and break
-                if [[ -n "$story_title" && -n "$story_points" && "$story_status" == "backlog" ]]; then
-                    echo "| $story_id | $story_title | $story_points | $current_epic |"
-                    break
-                fi
-
-                # If status is not backlog, skip this story
-                if [[ -n "$story_status" && "$story_status" != "backlog" ]]; then
-                    break
-                fi
-            done
-        fi
-    done < "$SPRINT_FILE"
-fi