@pennyfarthing/core 11.2.0 → 11.2.2

package/pennyfarthing-dist/scripts/hooks/bell-mode-hook.sh

@@ -1,183 +1,4 @@
-#!/bin/bash
-#
-# PostToolUse Hook — Bell Mode + Tandem Injection
-#
-# Called by Claude Code after each tool execution. Handles two independent
-# injection systems:
-#
-# 1. Bell queue (Cyclist only) — injects queued user messages when Cyclist
-#    is running and bell_mode is enabled. In CLI sessions this is a no-op.
-# 2. Tandem observations (always active) — injects backseat agent observations
-#    when tandem observation files exist. No configuration required.
-#
-# Bell queue takes precedence: if a queued message exists, tandem is
-# deferred to the next hook invocation.
-#
-# Output format (when injecting):
-#   {
-#     "hookSpecificOutput": {
-#       "hookEventName": "PostToolUse",
-#       "additionalContext": "<message>"
-#     }
-#   }
-#
-# Output when nothing to inject: (nothing - exit 0)
-
-# Find project root (walk up to find .pennyfarthing)
-PROJECT_ROOT="$PWD"
-while [[ "$PROJECT_ROOT" != "/" ]]; do
-  if [[ -d "$PROJECT_ROOT/.pennyfarthing" ]]; then
-    break
-  fi
-  PROJECT_ROOT="$(dirname "$PROJECT_ROOT")"
-done
-
-if [[ ! -d "$PROJECT_ROOT/.pennyfarthing" ]]; then
-  exit 0
-fi
-
-CONFIG_LOCAL_YAML="$PROJECT_ROOT/.pennyfarthing/config.local.yaml"
-BELL_QUEUE_FILE="$PROJECT_ROOT/.pennyfarthing/bell-queue.json"
-
-# Detect Cyclist — bell queue is a Cyclist-only feature
-IS_CYCLIST=false
-CYCLIST_PORT=""
-PORT_FILE="$PROJECT_ROOT/.wheelhub-port"
-if [[ -f "$PORT_FILE" ]]; then
-  CYCLIST_PORT=$(cat "$PORT_FILE" 2>/dev/null)
-  if [[ -n "$CYCLIST_PORT" ]] && [[ "$CYCLIST_PORT" =~ ^[0-9]+$ ]]; then
-    IS_CYCLIST=true
-  fi
-fi
-
-# --- Bell queue check (Cyclist only, requires bell_mode: true) ---
-# Takes precedence over tandem injection when active.
-
-if [[ "$IS_CYCLIST" == "true" ]] && [[ -f "$CONFIG_LOCAL_YAML" ]]; then
-  BELL_ENABLED=$(grep -E '^\s*bell_mode:\s*true' "$CONFIG_LOCAL_YAML" 2>/dev/null || true)
-
-  if [[ -n "$BELL_ENABLED" ]] && [[ -f "$BELL_QUEUE_FILE" ]]; then
-    QUEUE_CONTENT=$(cat "$BELL_QUEUE_FILE" 2>/dev/null)
-    if [[ -n "$QUEUE_CONTENT" ]] && [[ "$QUEUE_CONTENT" != "[]" ]]; then
-      FIRST_MESSAGE_TEXT=$(echo "$QUEUE_CONTENT" | sed -n 's/.*"text"[[:space:]]*:[[:space:]]*"\([^"]*\)".*/\1/p' | head -1)
-
-      if [[ -n "$FIRST_MESSAGE_TEXT" ]]; then
-        cat << EOF
-{
-  "hookSpecificOutput": {
-    "hookEventName": "PostToolUse",
-    "additionalContext": "User feedback: $FIRST_MESSAGE_TEXT"
-  }
-}
-EOF
-
-        # Dequeue and notify Cyclist in background
-        (
-          if command -v jq &> /dev/null; then
-            jq 'if length > 0 then .[1:] else [] end' "$BELL_QUEUE_FILE" > "$BELL_QUEUE_FILE.tmp" 2>/dev/null && mv "$BELL_QUEUE_FILE.tmp" "$BELL_QUEUE_FILE"
-          fi
-
-          curl -s -X POST "http://localhost:$CYCLIST_PORT/api/bell-consumed" \
-            -H "Content-Type: application/json" \
-            -d "{\"text\": \"$FIRST_MESSAGE_TEXT\"}" \
-            >/dev/null 2>&1 || true
-        ) &
-
-        exit 0
-      fi
-    fi
-  fi
-fi
-
-# --- Tandem observation injection ---
-# Always active. No configuration required.
-# Checks .session/*-tandem-*.md files for new observations via mtime comparison.
-
-SESSION_DIR="$PROJECT_ROOT/.session"
-if [[ ! -d "$SESSION_DIR" ]]; then
-  exit 0
-fi
-
-# Find tandem observation files
-TANDEM_FILES=$(find "$SESSION_DIR" -maxdepth 1 -name "*-tandem-*.md" 2>/dev/null)
-if [[ -z "$TANDEM_FILES" ]]; then
-  exit 0
-fi
-
-# Check each tandem file for new content
-for TANDEM_FILE in $TANDEM_FILES; do
-  # Extract agent name from filename: *-tandem-{agent}.md
-  AGENT=$(echo "$TANDEM_FILE" | sed -n 's/.*-tandem-\([a-zA-Z_]*\)\.md$/\1/p')
-  if [[ -z "$AGENT" ]]; then
-    continue
-  fi
-
-  # Get file mtime (portable: stat -f %m on macOS, stat -c %Y on Linux)
-  if [[ "$(uname)" == "Darwin" ]]; then
-    FILE_MTIME=$(stat -f %m "$TANDEM_FILE" 2>/dev/null)
-  else
-    FILE_MTIME=$(stat -c %Y "$TANDEM_FILE" 2>/dev/null)
-  fi
-  if [[ -z "$FILE_MTIME" ]]; then
-    continue
-  fi
-
-  # Compare with saved mtime sidecar
-  MTIME_SIDECAR="$SESSION_DIR/.tandem-mtime-$AGENT"
-  SAVED_MTIME="0"
-  if [[ -f "$MTIME_SIDECAR" ]]; then
-    SAVED_MTIME=$(cat "$MTIME_SIDECAR" 2>/dev/null)
-  fi
-
-  if [[ "$FILE_MTIME" == "$SAVED_MTIME" ]]; then
-    continue
-  fi
-
-  # Read tandem file and extract latest observation
-  CONTENT=$(cat "$TANDEM_FILE" 2>/dev/null)
-  if [[ -z "$CONTENT" ]]; then
-    continue
-  fi
-
-  # Extract persona from header: **Observer:** agent (Persona Name)
-  PERSONA=$(echo "$CONTENT" | sed -n 's/.*\*\*Observer:\*\*[[:space:]]*[a-zA-Z_]*[[:space:]]*(\([^)]*\)).*/\1/p' | head -1)
-  if [[ -z "$PERSONA" ]]; then
-    PERSONA="Unknown"
-  fi
-
-  # Extract latest observation text (last ## [HH:MM] Observation block)
-  # Get text between last observation header and trailing ---
-  OBS_TEXT=$(echo "$CONTENT" | awk '
-    /^## \[[0-9]+:[0-9]+\] Observation/ { found=1; text=""; next }
-    found && /^---/ { next }
-    found && /^\*\*Trigger:\*\*/ { next }
-    found { text = text (text ? "\n" : "") $0 }
-    END { if (text) print text }
-  ' | sed 's/^[[:space:]]*//;s/[[:space:]]*$//')
-
-  if [[ -z "$OBS_TEXT" ]]; then
-    # Update mtime even if no parseable observation
-    echo "$FILE_MTIME" > "$MTIME_SIDECAR"
-    continue
-  fi
-
-  # Format as [Tandem] prefix and output
-  TANDEM_MSG="[Tandem] $PERSONA: $OBS_TEXT"
-
-  cat << EOF
-{
-  "hookSpecificOutput": {
-    "hookEventName": "PostToolUse",
-    "additionalContext": "$TANDEM_MSG"
-  }
-}
-EOF
-
-  # Save mtime sidecar
-  echo "$FILE_MTIME" > "$MTIME_SIDECAR"
-
-  # Only inject one tandem observation per hook invocation
-  exit 0
-done
-
-exit 0
+#!/usr/bin/env bash
+# Shim: delegates to Python CLI via uv run (dogfooding) or global pf (consumer).
+source "$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd -P)/../lib/run-pf.sh"
+exec_pf hooks bell-mode

package/pennyfarthing-dist/scripts/hooks/context-circuit-breaker.sh

@@ -1,95 +1,4 @@
-#!/usr/bin/env zsh
-# Context circuit breaker hook: Block tool execution at 85% context usage
-# Called by Claude Code before tool calls (PreToolUse)
-#
-# Input: JSON via stdin with tool_name, tool_input, session_id
-# Output: Exit 0 to allow, Exit 2 to block (stderr shown to Claude)
-#
-# This hook provides a hard stop when context is critically high,
-# unlike context-warning.sh which only warns.
-#
-# When triggered, automatically saves the active agent to a checkpoint
-# so /continue-session can restore it with FULL tier.
-
-# Read stdin to get session_id (required by hook protocol)
-INPUT=$(cat)
-
-# Script location for sibling script references
-SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
-
-# Get context percentage from check-context.sh
-CONTEXT_SCRIPT="$SCRIPT_DIR/../check-context.sh"
-if [[ ! -x "$CONTEXT_SCRIPT" ]]; then
-    # graceful degradation: if check-context.sh not found, allow the tool
-    exit 0
-fi
-
-# Run check-context.sh and parse output
-eval "$("$CONTEXT_SCRIPT" 2>/dev/null)" 2>/dev/null
-
-# graceful degradation: if we can't get context, allow the tool
-if [[ -z "$CONTEXT_PERCENT" ]] || [[ "$CONTEXT_PERCENT" == "null" ]]; then
-    exit 0
-fi
-
-# Load critical threshold (default 80%)
-CRITICAL_THRESHOLD="${CRITICAL_THRESHOLD:-80}"
-
-# Check if at or above critical threshold
-if [[ "$CONTEXT_PERCENT" -ge "$CRITICAL_THRESHOLD" ]] 2>/dev/null; then
-    # Auto-save active agent to checkpoint before blocking
-    # This allows /continue-session to restore with FULL tier
-    source "$SCRIPT_DIR/../lib/checkpoint.sh" 2>/dev/null || true
-
-    # Get session_id from input JSON
-    SESSION_ID=""
-    if command -v jq &>/dev/null && [[ -n "$INPUT" ]]; then
-        SESSION_ID=$(echo "$INPUT" | jq -r '.session_id // empty' 2>/dev/null)
-    fi
-
-    # Look up active agent for this session
-    ACTIVE_AGENT=""
-    AGENT_FILE="${CLAUDE_PROJECT_DIR:-.}/.session/agents/${SESSION_ID}"
-    if [[ -n "$SESSION_ID" && -f "$AGENT_FILE" ]]; then
-        ACTIVE_AGENT=$(cat "$AGENT_FILE" 2>/dev/null)
-    fi
-
-    # Save agent checkpoint if we found one
-    if [[ -n "$ACTIVE_AGENT" ]]; then
-        checkpoint_save "circuit_breaker_agent" "$ACTIVE_AGENT" 2>/dev/null || true
-    fi
-
-    # Send error message to stderr (Claude will see this)
-    cat >&2 << EOF
-CONTEXT CIRCUIT BREAKER TRIGGERED
-
-Context usage: ${CONTEXT_PERCENT}% - CRITICAL (threshold: ${CRITICAL_THRESHOLD}%)
-
-Tool execution BLOCKED. You must stop and hand off.
-EOF
-
-    # Include agent info if available
-    if [[ -n "$ACTIVE_AGENT" ]]; then
-        cat >&2 << EOF
-
-Active agent saved: ${ACTIVE_AGENT}
-The agent will be restored with FULL context when you run /continue-session.
-EOF
-    fi
-
-    cat >&2 << EOF
-
-Required actions:
-1. Commit any pending changes
-2. Tell user to start fresh session with /continue-session
-
-DO NOT attempt further tool calls. This is a hard stop.
-
-To resume later: /continue-session
-EOF
-    # Exit 2 blocks the tool execution
-    exit 2
-fi
-
-# Below threshold: allow the tool
-exit 0
+#!/usr/bin/env bash
+# Shim: delegates to Python CLI via uv run (dogfooding) or global pf (consumer).
+source "$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd -P)/../lib/run-pf.sh"
+exec_pf hooks context-breaker

package/pennyfarthing-dist/scripts/hooks/context-warning.sh

@@ -1,65 +1,4 @@
-#!/usr/bin/env zsh
-# Context warning hook: Warn agent when context usage is high
-# Called by Claude Code before tool calls (PreToolUse)
-#
-# Input: JSON via stdin with tool_name, tool_input
-# Output: Warning message to stdout if context > 60%
-# Always exits 0 (warning only, never blocks)
-
-# Read and discard stdin (required by hook protocol)
-cat > /dev/null
-
-# Script location for sibling script references
-SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
-
-# Get context percentage from check-context.sh
-CONTEXT_SCRIPT="$SCRIPT_DIR/../check-context.sh"
-if [[ ! -x "$CONTEXT_SCRIPT" ]]; then
-    exit 0  # Can't check, allow silently
-fi
-
-# Run check-context.sh and parse output
-eval "$("$CONTEXT_SCRIPT" 2>/dev/null)" 2>/dev/null
-
-# Check if we got a valid percentage
-if [[ -z "$CONTEXT_PERCENT" ]] || [[ "$CONTEXT_PERCENT" == "null" ]]; then
-    exit 0  # Can't determine, allow silently
-fi
-
-# Load thresholds from settings (defaults match check-context.sh)
-WARNING_THRESHOLD="${WARNING_THRESHOLD:-60}"
-CRITICAL_THRESHOLD="${CRITICAL_THRESHOLD:-85}"
-
-# Output warning if above threshold
-if [[ "$CONTEXT_PERCENT" -ge "$CRITICAL_THRESHOLD" ]] 2>/dev/null; then
-    cat << EOF
-
----
-CONTEXT WARNING: ${CONTEXT_PERCENT}% (CRITICAL)
-
-Context usage is critically high. Recommended actions:
-1. Complete current task immediately
-2. Commit any pending changes
-3. Hand off to next agent or ask user to start fresh session
-
-Do NOT start new subtasks. Wrap up and hand off.
----
-
-EOF
-elif [[ "$CONTEXT_PERCENT" -ge "$WARNING_THRESHOLD" ]] 2>/dev/null; then
-    cat << EOF
-
----
-CONTEXT WARNING: ${CONTEXT_PERCENT}%
-
-Context usage is high. Consider:
-- Wrapping up current task soon
-- Preparing for handoff to next agent
-- Avoiding large file reads or complex operations
----
-
-EOF
-fi
-
-# Always allow the tool (we're warning, not blocking)
-exit 0
+#!/usr/bin/env bash
+# Shim: delegates to Python CLI via uv run (dogfooding) or global pf (consumer).
+source "$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd -P)/../lib/run-pf.sh"
+exec_pf hooks context-warning

package/pennyfarthing-dist/scripts/hooks/cyclist-pretooluse-hook.sh

@@ -1,32 +1,4 @@
 #!/usr/bin/env bash
-#
-# Cyclist PreToolUse Hook - Shell wrapper for Python implementation
-#
-# Routes tool approval requests through WheelHub when Cyclist is running.
-# Falls back to Claude Code's built-in permissions when Cyclist is not active.
-#
-# Input (stdin): JSON with tool_name, tool_input, session_id, etc.
-# Output (stdout): JSON decision (allow/deny/ask)
-#
-# Story: MSSCI-14320
-
-set -euo pipefail
-
-SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd -P)"
-source "$SCRIPT_DIR/../lib/find-root.sh"
-
-# Set PYTHONPATH for pennyfarthing_scripts
-PENNYFARTHING_SCRIPTS=""
-if [[ -d "$PROJECT_ROOT/pennyfarthing" ]]; then
-    # Dogfooding: framework inlined in orchestrator
-    PENNYFARTHING_SCRIPTS="$PROJECT_ROOT/pennyfarthing"
-elif [[ -d "$PROJECT_ROOT/node_modules/@pennyfarthing/core" ]]; then
-    # Normal install
-    PENNYFARTHING_SCRIPTS="$PROJECT_ROOT/node_modules/@pennyfarthing/core"
-fi
-
-if [[ -n "$PENNYFARTHING_SCRIPTS" ]]; then
-    export PYTHONPATH="$PENNYFARTHING_SCRIPTS:${PYTHONPATH:-}"
-fi
-
-python3 -m pennyfarthing_scripts.pretooluse_hook
+# Shim: delegates to Python CLI via uv run (dogfooding) or global pf (consumer).
+source "$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd -P)/../lib/run-pf.sh"
+exec_pf hooks cyclist-pretooluse

package/pennyfarthing-dist/scripts/hooks/otel-auto-config.sh

@@ -1,7 +1,7 @@
 #!/bin/bash
 # otel-auto-config.sh - Auto-configure OTEL for Cyclist web mode (Story 20-1)
 #
-# This hook checks for a .wheelhub-port file in the project directory.
+# This hook checks for a .bikerack-port file in the project directory.
 # If found, it sets the OTEL environment variables to connect Claude Code
 # telemetry to the running Cyclist server.
 #
@@ -9,15 +9,15 @@
 #   source /path/to/otel-auto-config.sh
 #
 # Prerequisites:
-#   - Cyclist must be running in web mode (writes .wheelhub-port file)
+#   - Cyclist must be running in web mode (writes .bikerack-port file)
 #   - CLAUDE_PROJECT_DIR must be set (standard in Claude Code environment)
 
 # Determine project directory
 PROJECT_DIR="${CLAUDE_PROJECT_DIR:-$(pwd)}"
 
-# Read port from .wheelhub-port (used by both Cyclist and BikeRack)
+# Read port from .bikerack-port (used by both Cyclist and BikeRack)
 PORT=""
-PORT_FILE="$PROJECT_DIR/.wheelhub-port"
+PORT_FILE="$PROJECT_DIR/.bikerack-port"
 if [[ -f "$PORT_FILE" ]]; then
   PORT=$(cat "$PORT_FILE" 2>/dev/null)
   # Validate port is a number
@@ -28,6 +28,7 @@ fi
 
 # Configure OTEL if a valid port was found
 if [[ -n "$PORT" ]]; then
+  export CLAUDE_CODE_ENABLE_TELEMETRY=1
   export OTEL_EXPORTER_OTLP_PROTOCOL="http/json"
   export OTEL_EXPORTER_OTLP_ENDPOINT="http://localhost:$PORT"
 

package/pennyfarthing-dist/scripts/hooks/pre-commit.sh

@@ -3,7 +3,7 @@
 #
 # Checks:
 # 1. Prevents direct commits to protected branches (main, develop)
-#    Exception: sprint/ folder commits allowed on develop
+#    Exception: sprint/ folder commits allowed on protected branches
 # 2. Validates agent files when pennyfarthing-dist/agents/*.md is modified
 # 3. Validates file references when pennyfarthing-dist/ files are modified (warn only)
 # 4. Validates sprint YAML files when sprint/*.yaml is modified
@@ -34,40 +34,34 @@ BRANCH=$(git rev-parse --abbrev-ref HEAD 2>/dev/null)
 PROTECTED_BRANCHES="^(main|develop)$"
 
 if [[ $BRANCH =~ $PROTECTED_BRANCHES ]]; then
-    # Special case: Allow sprint/ folder commits on develop branch
-    if [[ $BRANCH == "develop" ]]; then
-        STAGED_FILES=$(git diff --cached --name-only)
-        NON_SPRINT_FILES=$(echo "$STAGED_FILES" | grep -v "^sprint/")
-
-        # If all staged files are in sprint/, allow the commit
-        if [ -z "$NON_SPRINT_FILES" ] && [ -n "$STAGED_FILES" ]; then
-            # Continue to agent validation check
-            :
-        else
-            echo ""
-            echo "COMMIT BLOCKED"
-            echo ""
-            echo "You are trying to commit directly to: $BRANCH"
-            echo "This violates the git workflow rules."
-            echo ""
-            echo "Protected branches: main, develop"
-            echo ""
-            echo "What to do:"
-            echo "1. Create a feature branch:"
-            echo "   git checkout -b <type>/<epic-story>-<description>"
-            echo ""
-            echo "2. Example:"
-            echo "   git checkout -b feat/8-2-add-authentication"
-            echo ""
-            echo "3. Then commit your changes on the feature branch"
-            echo ""
-            echo "Exception: Sprint tracking files (sprint/*) can be committed to develop"
-            echo ""
-            exit 1
-        fi
+    # Special case: Allow sprint/ folder commits on protected branches
+    # (orchestrator uses trunk-based on main; pennyfarthing uses gitflow on develop)
+    STAGED_FILES=$(git diff --cached --name-only)
+    NON_SPRINT_FILES=$(echo "$STAGED_FILES" | grep -v "^sprint/")
+
+    # If all staged files are in sprint/, allow the commit
+    if [ -z "$NON_SPRINT_FILES" ] && [ -n "$STAGED_FILES" ]; then
+        # Continue to agent validation check
+        :
     else
         echo ""
-        echo "COMMIT BLOCKED - Cannot commit directly to $BRANCH"
+        echo "COMMIT BLOCKED"
+        echo ""
+        echo "You are trying to commit directly to: $BRANCH"
+        echo "This violates the git workflow rules."
+        echo ""
+        echo "Protected branches: main, develop"
+        echo ""
+        echo "What to do:"
+        echo "1. Create a feature branch:"
+        echo "   git checkout -b <type>/<epic-story>-<description>"
+        echo ""
+        echo "2. Example:"
+        echo "   git checkout -b feat/8-2-add-authentication"
+        echo ""
+        echo "3. Then commit your changes on the feature branch"
+        echo ""
+        echo "Exception: Sprint tracking files (sprint/*) can be committed directly"
         echo ""
         exit 1
     fi
@@ -140,7 +134,8 @@ fi
 SPRINT_YAML_FILES=$(git diff --cached --name-only -- 'sprint/*.yaml' 'sprint/archive/*.yaml' 2>/dev/null \
     | grep -v 'sprint-template\.yaml$' \
     | grep -v 'sprint/completed\.yaml$' \
-    | grep -v 'sprint/context/archived/' || true)
+    | grep -v 'sprint/context/archived/' \
+    | grep -v 'sprints\.yaml$' || true)
 
 if [[ -n "$SPRINT_YAML_FILES" ]]; then
     echo "Sprint YAML files staged for commit:"

package/pennyfarthing-dist/scripts/hooks/pre-edit-check.sh

@@ -1,71 +1,4 @@
-#!/usr/bin/env zsh
-# Pre-edit hook: Verify edits don't touch protected files
-# Called by Claude Code before Edit/Write tool calls
-#
-# Input: JSON via stdin with tool_name, tool_input
-# Output: Exit 0 to allow, Exit 2 to block (stderr shown to Claude)
-
-set -euo pipefail
-
-# Read input from stdin
-input=$(cat)
-
-# Extract file path from tool input
-file_path=$(echo "$input" | jq -r '.tool_input.file_path // .tool_input.path // ""')
-
-# Protected patterns (files that should never be edited automatically)
-protected_patterns=(
-    "*.env"
-    "*.pem"
-    "*.key"
-    "*credentials*"
-    "*secrets*"
-    ".git/*"
-    "node_modules/*"
-    "vendor/*"
-)
-
-# Pennyfarthing managed files protection
-# These are managed by `pennyfarthing update`, don't edit directly
-# EXCEPTION: If we ARE in the pennyfarthing library itself, allow edits
-is_pennyfarthing_lib=false
-if [[ -d "${CLAUDE_PROJECT_DIR:-$PWD}/pennyfarthing-dist" ]]; then
-    is_pennyfarthing_lib=true
-fi
-
-if [[ "$file_path" == *".claude/pennyfarthing/"* ]] && [[ "$is_pennyfarthing_lib" == "false" ]]; then
-    echo "BLOCKED: Cannot edit managed pennyfarthing files." >&2
-    echo "File: $file_path" >&2
-    echo "" >&2
-    echo "These files are managed by pennyfarthing and will be overwritten on update." >&2
-    echo "Instead:" >&2
-    echo "  - Put project-specific customizations in .claude/project/" >&2
-    echo "  - For framework changes, edit the pennyfarthing repo and run 'pennyfarthing update'" >&2
-    exit 2
-fi
-
-# Check if file matches any protected pattern
-for pattern in "${protected_patterns[@]}"; do
-    if [[ "$file_path" == $pattern ]]; then
-        echo "BLOCKED: Cannot edit protected file matching pattern: $pattern" >&2
-        echo "File: $file_path" >&2
-        exit 2
-    fi
-done
-
-# Additional check: warn on certain file types but allow
-warn_patterns=(
-    "*.md"
-    "*.json"
-)
-
-# For warnings, just log to stderr but allow (exit 0)
-for pattern in "${warn_patterns[@]}"; do
-    if [[ "$file_path" == $pattern ]]; then
-        # Could log warnings here if needed
-        :
-    fi
-done
-
-# Allow the edit
-exit 0
+#!/usr/bin/env bash
+# Shim: delegates to Python CLI via uv run (dogfooding) or global pf (consumer).
+source "$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd -P)/../lib/run-pf.sh"
+exec_pf hooks pre-edit-check

package/pennyfarthing-dist/scripts/hooks/question-reflector-check.sh

@@ -1,20 +1,4 @@
 #!/usr/bin/env bash
-#
-# Question Reflector Enforcement Hook (Stop hook / PreToolUse hook)
-#
-# Thin wrapper that delegates to question_reflector_check.py for the actual logic.
-# This allows the hook to be written in Python for easier testing and maintenance.
-#
-# Input (stdin): JSON with transcript_path, stop_hook_active, etc.
-# Output (stdout): JSON decision to allow or block
-#
-# Story: MSSCI-12393
-#
-
-set -euo pipefail
-
-# Get the directory where this script lives
-SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
-
-# Delegate to Python implementation
-exec python3 "$SCRIPT_DIR/question_reflector_check.py"
+# Shim: delegates to Python CLI via uv run (dogfooding) or global pf (consumer).
+source "$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd -P)/../lib/run-pf.sh"
+exec_pf hooks reflector-check

package/pennyfarthing-dist/scripts/hooks/schema-validation.sh

@@ -1,30 +1,4 @@
-#!/usr/bin/env zsh
-# Schema Validation Hook
-# Validates XML schema for session, skill, and workflow step files on Write operations
-# Called by Claude Code PreToolUse hook
-#
-# Input: JSON via stdin with tool_name, tool_input
-# Output: JSON with decision (allow/deny)
-
-set -euo pipefail
-
-# Find script directory (zsh compatible)
-SCRIPT_DIR="${0:A:h}"
-source "$SCRIPT_DIR/../lib/find-root.sh"
-
-# Set PYTHONPATH for pennyfarthing_scripts
-PENNYFARTHING_SCRIPTS=""
-if [[ -d "$PROJECT_ROOT/pennyfarthing" ]]; then
-    # Dogfooding: framework inlined in orchestrator
-    PENNYFARTHING_SCRIPTS="$PROJECT_ROOT/pennyfarthing"
-elif [[ -d "$PROJECT_ROOT/node_modules/@pennyfarthing/core" ]]; then
-    # Normal install
-    PENNYFARTHING_SCRIPTS="$PROJECT_ROOT/node_modules/@pennyfarthing/core"
-fi
-
-if [[ -n "$PENNYFARTHING_SCRIPTS" ]]; then
-    export PYTHONPATH="$PENNYFARTHING_SCRIPTS:${PYTHONPATH:-}"
-fi
-
-# Run Python validation hook
-python3 -m pennyfarthing_scripts.schema_validation_hook
+#!/usr/bin/env bash
+# Shim: delegates to Python CLI via uv run (dogfooding) or global pf (consumer).
+source "$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd -P)/../lib/run-pf.sh"
+exec_pf hooks schema-validation