|
| 1 | +#!/usr/bin/env bash |
| 2 | +# fix-session-metrics.sh — propagate the corrected session-report.sh to every |
| 3 | +# registered project and delete the malformed JSON metrics files. |
| 4 | +# |
| 5 | +# Fixes bug discovered 2026-04-21: every JSON under ~/.claude/metrics/<slug>/*.json |
| 6 | +# was malformed due to `grep -c ... || echo "0"` double-output + arithmetic |
| 7 | +# cascade on corrupted previous files. |
| 8 | +# |
| 9 | +# Usage: |
| 10 | +# bash scripts/fix-session-metrics.sh # apply |
| 11 | +# bash scripts/fix-session-metrics.sh --dry-run # preview |
| 12 | +# |
| 13 | +# Idempotent. Safe to re-run. |
| 14 | + |
| 15 | +set -u |
| 16 | + |
| 17 | +DOTFORGE_DIR=$(cd "$(dirname "$0")/.." && pwd) |
| 18 | +TEMPLATE_HOOK="$DOTFORGE_DIR/template/hooks/session-report.sh" |
| 19 | +export REGISTRY="$DOTFORGE_DIR/registry/projects.local.yml" |
| 20 | +METRICS_ROOT="$HOME/.claude/metrics" |
| 21 | +DRY_RUN=false |
| 22 | + |
| 23 | +[ "${1:-}" = "--dry-run" ] && DRY_RUN=true |
| 24 | + |
| 25 | +if [ ! -f "$TEMPLATE_HOOK" ]; then |
| 26 | + echo "ERROR: $TEMPLATE_HOOK not found" >&2 |
| 27 | + exit 2 |
| 28 | +fi |
| 29 | +if [ ! -f "$REGISTRY" ]; then |
| 30 | + echo "ERROR: $REGISTRY not found" >&2 |
| 31 | + exit 2 |
| 32 | +fi |
| 33 | + |
| 34 | +_is_broken() { |
| 35 | + local f="$1" |
| 36 | + [ -f "$f" ] || return 1 |
| 37 | + grep -q '|| echo "0"' "$f" 2>/dev/null |
| 38 | +} |
| 39 | + |
| 40 | +_slug() { |
| 41 | + printf '%s' "$1" | tr '[:upper:]' '[:lower:]' | tr ' ' '-' | sed 's/[^a-z0-9-]//g' |
| 42 | +} |
| 43 | + |
| 44 | +PROJECTS=$(python3 - <<'PYEOF' |
| 45 | +import yaml, os |
| 46 | +with open(os.environ["REGISTRY"]) as f: |
| 47 | + d = yaml.safe_load(f) |
| 48 | +for p in d["projects"]: |
| 49 | + print(f"{p['name']}\t{p['path']}") |
| 50 | +PYEOF |
| 51 | +) |
| 52 | + |
| 53 | +if [ -z "$PROJECTS" ]; then |
| 54 | + echo "No projects found in registry." |
| 55 | + exit 0 |
| 56 | +fi |
| 57 | + |
| 58 | +printf '%s\n' "═══ FIX SESSION METRICS ═══" |
| 59 | +printf 'Template: %s\n' "$TEMPLATE_HOOK" |
| 60 | +printf 'Registry: %s\n' "$REGISTRY" |
| 61 | +printf 'Dry-run: %s\n\n' "$DRY_RUN" |
| 62 | + |
| 63 | +PATCHED=0 |
| 64 | +SKIPPED_CLEAN=0 |
| 65 | +SKIPPED_MISSING=0 |
| 66 | +CLEANED=0 |
| 67 | + |
| 68 | +while IFS=$'\t' read -r NAME RAW_PATH; do |
| 69 | + [ -z "$NAME" ] && continue |
| 70 | + |
| 71 | + PROJ_PATH="$RAW_PATH" |
| 72 | + if [ "$PROJ_PATH" = "." ]; then |
| 73 | + PROJ_PATH="$DOTFORGE_DIR" |
| 74 | + fi |
| 75 | + |
| 76 | + HOOK="$PROJ_PATH/.claude/hooks/session-report.sh" |
| 77 | + SLUG=$(_slug "$NAME") |
| 78 | + METRICS_DIR="$METRICS_ROOT/$SLUG" |
| 79 | + |
| 80 | + printf '── %-22s ' "$NAME" |
| 81 | + |
| 82 | + if [ ! -d "$PROJ_PATH" ]; then |
| 83 | + printf 'SKIP (path not found: %s)\n' "$PROJ_PATH" |
| 84 | + SKIPPED_MISSING=$((SKIPPED_MISSING + 1)) |
| 85 | + continue |
| 86 | + fi |
| 87 | + |
| 88 | + if [ ! -f "$HOOK" ]; then |
| 89 | + printf 'no session-report.sh — skipped\n' |
| 90 | + SKIPPED_MISSING=$((SKIPPED_MISSING + 1)) |
| 91 | + continue |
| 92 | + fi |
| 93 | + |
| 94 | + if _is_broken "$HOOK"; then |
| 95 | + if $DRY_RUN; then |
| 96 | + printf 'would patch hook' |
| 97 | + else |
| 98 | + ORIG_SHEBANG=$(head -n 1 "$HOOK") |
| 99 | + cp "$TEMPLATE_HOOK" "$HOOK" |
| 100 | + if [ "$ORIG_SHEBANG" != "$(head -n 1 "$TEMPLATE_HOOK")" ] && [ -n "$ORIG_SHEBANG" ]; then |
| 101 | + sed -i.bak "1s|.*|$ORIG_SHEBANG|" "$HOOK" |
| 102 | + rm -f "$HOOK.bak" |
| 103 | + fi |
| 104 | + chmod +x "$HOOK" |
| 105 | + printf 'patched' |
| 106 | + fi |
| 107 | + PATCHED=$((PATCHED + 1)) |
| 108 | + else |
| 109 | + printf 'hook clean (no fix needed)' |
| 110 | + SKIPPED_CLEAN=$((SKIPPED_CLEAN + 1)) |
| 111 | + fi |
| 112 | + |
| 113 | + if [ -d "$METRICS_DIR" ]; then |
| 114 | + MALFORMED=0 |
| 115 | + for jf in "$METRICS_DIR"/*.json; do |
| 116 | + [ -f "$jf" ] || continue |
| 117 | + if ! jq -e . "$jf" >/dev/null 2>&1; then |
| 118 | + if ! $DRY_RUN; then |
| 119 | + rm -f "$jf" |
| 120 | + fi |
| 121 | + MALFORMED=$((MALFORMED + 1)) |
| 122 | + fi |
| 123 | + done |
| 124 | + if [ "$MALFORMED" -gt 0 ]; then |
| 125 | + printf ' | cleaned %s malformed json' "$MALFORMED" |
| 126 | + CLEANED=$((CLEANED + MALFORMED)) |
| 127 | + fi |
| 128 | + fi |
| 129 | + printf '\n' |
| 130 | +done <<< "$PROJECTS" |
| 131 | + |
| 132 | +printf '\n═══ SUMMARY ═══\n' |
| 133 | +printf 'Hooks patched: %d\n' "$PATCHED" |
| 134 | +printf 'Hooks already clean: %d\n' "$SKIPPED_CLEAN" |
| 135 | +printf 'Projects skipped: %d\n' "$SKIPPED_MISSING" |
| 136 | +printf 'Malformed JSON files cleaned: %d\n' "$CLEANED" |
| 137 | + |
| 138 | +if $DRY_RUN; then |
| 139 | + printf '\n(dry-run: no changes written)\n' |
| 140 | +fi |
0 commit comments