@pennyfarthing/core 7.4.1 → 7.5.0

package/pennyfarthing-dist/scripts/test/swebench-judge.py

@@ -1,400 +0,0 @@
-#!/usr/bin/env python3
-"""
-SWE-bench scenario judge using:
-1. Scenario-specific scoring rubric from YAML
-2. Ground-truth validation from actual SWE-bench patches
-
-Scoring structure:
-- root_cause (30%): IDENTIFIES_BUG_LOCATION (15) + EXPLAINS_WHY_BROKEN (15)
-- fix_quality (40%): FIX_ADDRESSES_ISSUE (20) + FIX_IS_MINIMAL (10) + FIX_SYNTAX_CORRECT (10)
-- completeness (20%): EDGE_CASES (10) + TEST_COVERAGE (10)
-- persona (10%): IN_CHARACTER (10)
-"""
-
-import json
-import re
-import sys
-from pathlib import Path
-from difflib import SequenceMatcher
-
-def load_swebench_data(cache_path="/tmp/swebench_all.json"):
-    """Load SWE-bench ground truth data."""
-    with open(cache_path, 'r') as f:
-        return json.load(f)
-
-def find_ground_truth(data, scenario_name):
-    """Find scenario in SWE-bench data."""
-    for item in data:
-        instance_id = item.get('instance_id', '')
-        if scenario_name in instance_id.replace('__', '-'):
-            return item
-    return None
-
-def extract_patch_info(patch_text):
-    """Extract structured info from patch."""
-    info = {
-        'files': [],
-        'functions': [],
-        'additions': [],
-        'deletions': [],
-        'key_code': []
-    }
-
-    for line in patch_text.split('\n'):
-        if line.startswith('diff --git'):
-            match = re.search(r'b/(.+)$', line)
-            if match:
-                info['files'].append(match.group(1))
-
-        if line.startswith('@@'):
-            match = re.search(r'@@.*@@\s*(.+)$', line)
-            if match:
-                info['functions'].append(match.group(1).strip())
-
-        if line.startswith('+') and not line.startswith('+++'):
-            clean = line[1:].strip()
-            if clean and not clean.startswith('#'):
-                info['additions'].append(clean)
-                info['key_code'].append(clean)
-
-        if line.startswith('-') and not line.startswith('---'):
-            clean = line[1:].strip()
-            if clean and not clean.startswith('#'):
-                info['deletions'].append(clean)
-
-    return info
-
-def score_identifies_bug_location(response, ground_truth):
-    """Score IDENTIFIES_BUG_LOCATION (15 pts) using ground truth."""
-    patch_info = extract_patch_info(ground_truth.get('patch', ''))
-    response_lower = response.lower()
-
-    score = 0
-    details = []
-
-    # Check files (7.5 pts)
-    files_found = 0
-    for f in patch_info['files']:
-        filename = Path(f).name.lower()
-        if filename in response_lower or f.lower() in response_lower:
-            files_found += 1
-
-    if patch_info['files']:
-        file_score = (files_found / len(patch_info['files'])) * 7.5
-        score += file_score
-        details.append(f"Files: {files_found}/{len(patch_info['files'])} found")
-
-    # Check functions/classes (7.5 pts)
-    funcs_found = 0
-    for func in patch_info['functions']:
-        func_match = re.search(r'(def|class)\s+(\w+)', func)
-        if func_match:
-            func_name = func_match.group(2).lower()
-            if func_name in response_lower:
-                funcs_found += 1
-
-    if patch_info['functions']:
-        func_score = min(7.5, (funcs_found / len(patch_info['functions'])) * 7.5)
-        score += func_score
-        details.append(f"Functions: {funcs_found}/{len(patch_info['functions'])} found")
-    else:
-        score += 3.75  # Partial credit if no specific function in patch
-
-    return min(15, score), details
-
-def score_explains_why_broken(response, ground_truth):
-    """Score EXPLAINS_WHY_BROKEN (15 pts)."""
-    response_lower = response.lower()
-    problem = ground_truth.get('problem_statement', '').lower()
-
-    score = 0
-    details = []
-
-    # Extract key terms from problem statement
-    key_terms = re.findall(r'[`\'"]([^`\'"]+)[`\'"]', problem)
-    key_terms += re.findall(r'\b\w+Error\b|\b\w+Exception\b', problem, re.IGNORECASE)
-    key_terms = list(set(key_terms))[:10]
-
-    # Check for explanation of the issue
-    explanation_markers = ['because', 'this happens', 'the issue', 'the problem', 'fails when', 'breaks when', 'causes']
-    has_explanation = any(marker in response_lower for marker in explanation_markers)
-    if has_explanation:
-        score += 7.5
-        details.append("Has explanation of why broken")
-
-    # Check for key terms from problem
-    terms_found = sum(1 for term in key_terms if term.lower() in response_lower)
-    if key_terms:
-        term_score = (terms_found / len(key_terms)) * 7.5
-        score += term_score
-        details.append(f"Key terms: {terms_found}/{len(key_terms)}")
-    else:
-        score += 3.75
-
-    return min(15, score), details
-
-def score_fix_addresses_issue(response, ground_truth):
-    """Score FIX_ADDRESSES_ISSUE (20 pts) using ground truth patch."""
-    patch_info = extract_patch_info(ground_truth.get('patch', ''))
-    response_lower = response.lower()
-
-    score = 0
-    details = []
-
-    # Check if key additions from patch appear in response
-    additions_matched = 0
-    for addition in patch_info['additions'][:5]:
-        # Normalize whitespace
-        addition_norm = re.sub(r'\s+', ' ', addition.lower())
-        response_norm = re.sub(r'\s+', ' ', response_lower)
-
-        # Check for exact or fuzzy match
-        if addition_norm in response_norm:
-            additions_matched += 1
-        else:
-            # Fuzzy match
-            sim = SequenceMatcher(None, addition_norm, response_norm).ratio()
-            if sim > 0.7:
-                additions_matched += 0.5
-
-    if patch_info['additions']:
-        addition_score = (additions_matched / min(5, len(patch_info['additions']))) * 15
-        score += addition_score
-        details.append(f"Code matches: {additions_matched}/{min(5, len(patch_info['additions']))}")
-
-    # Check for code block with fix
-    if '```' in response:
-        score += 5
-        details.append("Has code block")
-
-    return min(20, score), details
-
-def score_fix_is_minimal(response, ground_truth):
-    """Score FIX_IS_MINIMAL (10 pts)."""
-    patch_info = extract_patch_info(ground_truth.get('patch', ''))
-
-    score = 0
-    details = []
-
-    # Count lines in patch vs lines in response code blocks
-    patch_lines = len(patch_info['additions']) + len(patch_info['deletions'])
-
-    # Extract code blocks from response
-    code_blocks = re.findall(r'```[\w]*\n(.*?)```', response, re.DOTALL)
-    response_code_lines = sum(len(block.strip().split('\n')) for block in code_blocks)
-
-    # If response is within 2x of patch size, it's minimal
-    if patch_lines > 0:
-        ratio = response_code_lines / patch_lines if response_code_lines > 0 else 1
-        if ratio <= 2:
-            score = 10
-            details.append(f"Minimal: {response_code_lines} lines (patch: {patch_lines})")
-        elif ratio <= 4:
-            score = 5
-            details.append(f"Somewhat verbose: {response_code_lines} lines (patch: {patch_lines})")
-        else:
-            score = 2
-            details.append(f"Over-engineered: {response_code_lines} lines (patch: {patch_lines})")
-    else:
-        score = 5
-
-    return min(10, score), details
-
-def score_fix_syntax_correct(response):
-    """Score FIX_SYNTAX_CORRECT (10 pts)."""
-    score = 0
-    details = []
-
-    # Extract code blocks
-    code_blocks = re.findall(r'```python\n(.*?)```', response, re.DOTALL)
-    if not code_blocks:
-        code_blocks = re.findall(r'```\n(.*?)```', response, re.DOTALL)
-
-    if code_blocks:
-        # Basic syntax checks
-        valid = True
-        for block in code_blocks:
-            try:
-                compile(block, '<string>', 'exec')
-            except SyntaxError:
-                valid = False
-                break
-
-        if valid:
-            score = 10
-            details.append("Syntax valid")
-        else:
-            score = 5
-            details.append("Syntax errors detected")
-    else:
-        score = 5
-        details.append("No code blocks to validate")
-
-    return min(10, score), details
-
-def score_edge_cases(response):
-    """Score EDGE_CASES (10 pts)."""
-    response_lower = response.lower()
-
-    score = 0
-    details = []
-
-    edge_markers = ['edge case', 'corner case', 'what if', 'consider', 'also', 'none', 'empty', 'null', 'zero', 'negative', 'boundary']
-    found = sum(1 for m in edge_markers if m in response_lower)
-
-    score = min(10, found * 2)
-    details.append(f"Edge case markers: {found}")
-
-    return score, details
-
-def score_test_coverage(response):
-    """Score TEST_COVERAGE (10 pts)."""
-    response_lower = response.lower()
-
-    score = 0
-    details = []
-
-    # Check for test-related content
-    has_test_section = 'test' in response_lower
-    has_test_function = 'def test_' in response_lower or 'test_' in response
-    has_assert = 'assert' in response_lower or 'pytest' in response_lower
-
-    if has_test_function:
-        score += 5
-        details.append("Has test function")
-    if has_assert:
-        score += 3
-        details.append("Has assertions")
-    if has_test_section:
-        score += 2
-        details.append("Has test section")
-
-    return min(10, score), details
-
-def score_in_character(response, persona="senior developer"):
-    """Score IN_CHARACTER (10 pts)."""
-    response_lower = response.lower()
-
-    score = 0
-    details = []
-
-    # For control baseline, check professional tone
-    professional_markers = ['i recommend', 'we should', 'this approach', 'the fix', 'analysis', 'root cause']
-    found = sum(1 for m in professional_markers if m in response_lower)
-
-    score = min(10, found * 2)
-    details.append(f"Professional markers: {found}")
-
-    return score, details
-
-def judge_response(scenario_name, response_text, swebench_data):
-    """Full judgment using scenario rubric + ground truth."""
-    ground_truth = find_ground_truth(swebench_data, scenario_name)
-
-    if not ground_truth:
-        return {'error': f'Scenario {scenario_name} not found in SWE-bench data'}
-
-    scores = {}
-    all_details = {}
-
-    # root_cause (30%)
-    loc_score, loc_details = score_identifies_bug_location(response_text, ground_truth)
-    why_score, why_details = score_explains_why_broken(response_text, ground_truth)
-    scores['root_cause'] = {
-        'IDENTIFIES_BUG_LOCATION': loc_score,
-        'EXPLAINS_WHY_BROKEN': why_score,
-        'subtotal': loc_score + why_score
-    }
-    all_details['root_cause'] = loc_details + why_details
-
-    # fix_quality (40%)
-    fix_score, fix_details = score_fix_addresses_issue(response_text, ground_truth)
-    min_score, min_details = score_fix_is_minimal(response_text, ground_truth)
-    syn_score, syn_details = score_fix_syntax_correct(response_text)
-    scores['fix_quality'] = {
-        'FIX_ADDRESSES_ISSUE': fix_score,
-        'FIX_IS_MINIMAL': min_score,
-        'FIX_SYNTAX_CORRECT': syn_score,
-        'subtotal': fix_score + min_score + syn_score
-    }
-    all_details['fix_quality'] = fix_details + min_details + syn_details
-
-    # completeness (20%)
-    edge_score, edge_details = score_edge_cases(response_text)
-    test_score, test_details = score_test_coverage(response_text)
-    scores['completeness'] = {
-        'EDGE_CASES': edge_score,
-        'TEST_COVERAGE': test_score,
-        'subtotal': edge_score + test_score
-    }
-    all_details['completeness'] = edge_details + test_details
-
-    # persona (10%)
-    char_score, char_details = score_in_character(response_text)
-    scores['persona'] = {
-        'IN_CHARACTER': char_score,
-        'subtotal': char_score
-    }
-    all_details['persona'] = char_details
-
-    # Total
-    total = (
-        scores['root_cause']['subtotal'] +
-        scores['fix_quality']['subtotal'] +
-        scores['completeness']['subtotal'] +
-        scores['persona']['subtotal']
-    )
-
-    return {
-        'scenario': scenario_name,
-        'instance_id': ground_truth.get('instance_id'),
-        'scores': scores,
-        'total': round(total, 1),
-        'details': all_details,
-        'ground_truth_files': extract_patch_info(ground_truth.get('patch', ''))['files']
-    }
-
-def main():
-    if len(sys.argv) < 3:
-        print("Usage: swebench-judge.py <scenario_name> <response_file>")
-        sys.exit(1)
-
-    scenario_name = sys.argv[1]
-    response_file = sys.argv[2]
-
-    # Load data
-    swebench_data = load_swebench_data()
-
-    with open(response_file, 'r') as f:
-        response_data = json.load(f)
-
-    # Handle different JSON structures
-    response_text = response_data.get('result', '') or response_data.get('response_text', '')
-
-    # Judge
-    result = judge_response(scenario_name, response_text, swebench_data)
-
-    # Display
-    print(f"\n{'='*60}")
-    print(f"SWE-BENCH JUDGE: {scenario_name}")
-    print(f"{'='*60}")
-
-    for category, scores in result['scores'].items():
-        print(f"\n{category.upper()} ({scores['subtotal']:.1f} pts)")
-        for criterion, score in scores.items():
-            if criterion != 'subtotal':
-                print(f"  {criterion}: {score:.1f}")
-
-    print(f"\n{'─'*40}")
-    print(f"TOTAL: {result['total']}/100")
-
-    print(f"\nGround truth files: {result['ground_truth_files']}")
-
-    # Save
-    output_path = response_file.replace('run_', 'swebench_judge_')
-    with open(output_path, 'w') as f:
-        json.dump(result, f, indent=2)
-    print(f"\nSaved to: {output_path}")
-
-if __name__ == '__main__':
-    main()

package/pennyfarthing-dist/scripts/test/test-cache.sh

@@ -1,165 +0,0 @@
-#!/usr/bin/env bash
-# test-cache.sh - Read/write test result cache in session files
-#
-# Usage:
-#   source test-cache.sh
-#
-#   # Check if valid cache exists (returns 0 if valid, 1 if stale/missing)
-#   if test_cache_valid "$SESSION_FILE"; then
-#       RESULT=$(test_cache_get "$SESSION_FILE" "result")
-#       echo "Using cached result: $RESULT"
-#   fi
-#
-#   # Write cache after running tests
-#   test_cache_write "$SESSION_FILE" "GREEN" 42 0 1 "12s"
-
-set -euo pipefail
-
-# Cache validity window (5 minutes)
-CACHE_MAX_AGE_MINUTES=5
-
-# Check if test cache exists and is valid (same SHA, < 5 min old)
-# Returns: 0 if valid, 1 if invalid/missing
-test_cache_valid() {
-    local session_file="$1"
-
-    if [[ ! -f "$session_file" ]]; then
-        return 1
-    fi
-
-    if ! grep -q "^## Test Cache" "$session_file" 2>/dev/null; then
-        return 1
-    fi
-
-    local current_sha
-    current_sha=$(git rev-parse HEAD 2>/dev/null || echo "unknown")
-
-    local cache_sha
-    cache_sha=$(grep "| Git SHA |" "$session_file" 2>/dev/null | sed 's/.*| \([^ ]*\) |.*/\1/' | xargs)
-
-    if [[ "$cache_sha" != "$current_sha" ]]; then
-        return 1
-    fi
-
-    local cache_time
-    cache_time=$(grep "| Last Run |" "$session_file" 2>/dev/null | sed 's/.*| \([^ ]*\) |.*/\1/' | xargs)
-
-    local cache_epoch now_epoch age_minutes
-    # macOS date
-    cache_epoch=$(date -j -f "%Y-%m-%dT%H:%M:%SZ" "$cache_time" +%s 2>/dev/null || \
-                  date -d "$cache_time" +%s 2>/dev/null || echo 0)
-    now_epoch=$(date +%s)
-    age_minutes=$(( (now_epoch - cache_epoch) / 60 ))
-
-    if [[ $age_minutes -ge $CACHE_MAX_AGE_MINUTES ]]; then
-        return 1
-    fi
-
-    return 0
-}
-
-# Get a field from the test cache
-# Usage: test_cache_get "$SESSION_FILE" "result|pass|fail|skip|duration|sha|time"
-test_cache_get() {
-    local session_file="$1"
-    local field="$2"
-
-    case "$field" in
-        result)   grep "| Result |" "$session_file" 2>/dev/null | sed 's/.*| \([^ ]*\) |.*/\1/' | xargs ;;
-        pass)     grep "| Pass |" "$session_file" 2>/dev/null | sed 's/.*| \([^ ]*\) |.*/\1/' | xargs ;;
-        fail)     grep "| Fail |" "$session_file" 2>/dev/null | sed 's/.*| \([^ ]*\) |.*/\1/' | xargs ;;
-        skip)     grep "| Skip |" "$session_file" 2>/dev/null | sed 's/.*| \([^ ]*\) |.*/\1/' | xargs ;;
-        duration) grep "| Duration |" "$session_file" 2>/dev/null | sed 's/.*| \([^ ]*\) |.*/\1/' | xargs ;;
-        sha)      grep "| Git SHA |" "$session_file" 2>/dev/null | sed 's/.*| \([^ ]*\) |.*/\1/' | xargs ;;
-        time)     grep "| Last Run |" "$session_file" 2>/dev/null | sed 's/.*| \([^ ]*\) |.*/\1/' | xargs ;;
-        *)        echo "Unknown field: $field" >&2; return 1 ;;
-    esac
-}
-
-# Write test cache to session file
-# Usage: test_cache_write "$SESSION_FILE" "GREEN" 42 0 1 "12s"
-test_cache_write() {
-    local session_file="$1"
-    local result="$2"      # GREEN, RED, YELLOW
-    local pass="$3"        # pass count
-    local fail="$4"        # fail count
-    local skip="$5"        # skip count
-    local duration="$6"    # e.g., "12s"
-
-    local git_sha timestamp
-    git_sha=$(git rev-parse HEAD 2>/dev/null || echo "unknown")
-    timestamp=$(date -u +"%Y-%m-%dT%H:%M:%SZ")
-
-    local cache_section="## Test Cache
-
-| Field | Value |
-|-------|-------|
-| Last Run | $timestamp |
-| Git SHA | $git_sha |
-| Result | $result |
-| Pass | $pass |
-| Fail | $fail |
-| Skip | $skip |
-| Duration | $duration |"
-
-    if [[ ! -f "$session_file" ]]; then
-        echo "Session file not found: $session_file" >&2
-        return 1
-    fi
-
-    # Check if Test Cache section exists
-    if grep -q "^## Test Cache" "$session_file" 2>/dev/null; then
-        # Replace existing section - find start and end, replace content
-        local temp_file
-        temp_file=$(mktemp)
-
-        awk -v new_section="$cache_section" '
-            /^## Test Cache/ {
-                in_section = 1
-                print new_section
-                next
-            }
-            /^## / && in_section {
-                in_section = 0
-            }
-            !in_section { print }
-        ' "$session_file" > "$temp_file"
-
-        mv "$temp_file" "$session_file"
-    else
-        # Append before "## Workflow Tracking" if present, else at end
-        if grep -q "^## Workflow Tracking" "$session_file" 2>/dev/null; then
-            local temp_file
-            temp_file=$(mktemp)
-
-            awk -v new_section="$cache_section" '
-                /^## Workflow Tracking/ {
-                    print new_section
-                    print ""
-                }
-                { print }
-            ' "$session_file" > "$temp_file"
-
-            mv "$temp_file" "$session_file"
-        else
-            echo "" >> "$session_file"
-            echo "$cache_section" >> "$session_file"
-        fi
-    fi
-
-    echo "Test cache written: $result ($pass passed, $fail failed, $skip skipped)"
-}
-
-# Print cache status for debugging
-test_cache_status() {
-    local session_file="$1"
-
-    if test_cache_valid "$session_file"; then
-        echo "Cache: VALID"
-        echo "  Result: $(test_cache_get "$session_file" "result")"
-        echo "  SHA: $(test_cache_get "$session_file" "sha")"
-        echo "  Time: $(test_cache_get "$session_file" "time")"
-    else
-        echo "Cache: INVALID or MISSING"
-    fi
-}