Testing skip hooks option

Author: justin808

.lefthook.yml

@@ -0,0 +1,23 @@
+# .lefthook.yml
+# Fast pre-commit hooks that only check changed files
+# Install with: bundle exec lefthook install
+
+pre-commit:
+  parallel: true
+  commands:
+    autofix:
+      run: bin/lefthook/ruby-autofix staged
+
+    rubocop:
+      run: bin/lefthook/ruby-lint staged
+
+    prettier:
+      run: bin/lefthook/prettier-format staged
+
+    trailing-newlines:
+      run: bin/lefthook/check-trailing-newlines staged
+
+pre-push:
+  commands:
+    branch-lint:
+      run: bin/lefthook/ruby-lint branch

CLAUDE.md

@@ -12,6 +12,18 @@ This file provides guidance to Claude Code (claude.ai/code) when working with co
 
 These requirements are non-negotiable. CI will fail if not followed.
 
+**🚀 RECOMMENDED: Install Git hooks to automate these checks:**
+
+```bash
+# Install Lefthook hooks (already included in Gemfile)
+bundle install
+bundle exec lefthook install
+```
+
+This will automatically run linting on **only the files you changed** before each commit - making it fast!
+
+**Note:** Git hooks are for React on Rails gem developers only, not for users who install the gem.
+
 ## Development Commands
 
 ### Essential Commands

CONTRIBUTING.md

@@ -7,6 +7,16 @@
 ## Prerequisites
 
 - [Yalc](https://github.com/whitecolor/yalc) must be installed globally for most local development.
+- **Git hooks setup** (REQUIRED for all contributors):
+
+```sh
+cd react_on_rails/
+bundle install
+bundle exec lefthook install
+```
+
+This sets up automatic linting that runs **only on files you changed** - making commits fast while preventing CI failures.
+
 - After updating code via Git, to prepare all examples:
 
 ```sh
@@ -457,7 +467,9 @@ This approach:
 
 ## Pre-Commit Requirements
 
-**CRITICAL**: Before committing any changes, always run the following commands to ensure code quality:
+**AUTOMATED**: If you've set up Lefthook (see Prerequisites), linting runs automatically on changed files before each commit.
+
+**MANUAL OPTION**: If you need to run linting manually:
 
 ```bash
 # Navigate to the main react_on_rails directory
@@ -476,14 +488,14 @@ rake lint:rubocop
 rake lint
 ```
 
-**Automated checks:**
+**Git hooks automatically run:**
 
-- Format all JavaScript/TypeScript files with Prettier
+- Format JavaScript/TypeScript files with Prettier (on changed files only)
 - Check and fix linting issues with ESLint
-- Check and fix Ruby style issues with RuboCop
-- Ensure all tests pass before pushing
+- Check and fix Ruby style issues with RuboCop (on changed files only)
+- Ensure trailing newlines on all files
 
-**Tip**: Set up your IDE to run these automatically on save to catch issues early.
+**Setup once**: `bundle exec lefthook install` (see Prerequisites above)
 
 ## 🤖 Best Practices for AI Coding Agents
 

Gemfile.development_dependencies

@@ -38,6 +38,7 @@ group :development, :test do
   gem "rubocop-rspec", "~>2.26", require: false
   gem "scss_lint", require: false
   gem "spring", "~> 4.0"
+  gem "lefthook", require: false
 end
 
 group :test do

Gemfile.lock

@@ -156,6 +156,7 @@ GEM
     launchy (3.0.1)
       addressable (~> 2.8)
       childprocess (~> 5.0)
+    lefthook (1.13.1)
     listen (3.9.0)
       rb-fsevent (~> 0.10, >= 0.10.3)
       rb-inotify (~> 0.9, >= 0.9.10)
@@ -412,6 +413,7 @@ DEPENDENCIES
   jbuilder
   jquery-rails
   launchy
+  lefthook
   listen
   package_json
   pry

bin/lefthook/check-trailing-newlines

@@ -0,0 +1,34 @@
+#!/usr/bin/env bash
+# Check for trailing newlines on all changed files
+set -euo pipefail
+
+CONTEXT="${1:-staged}"
+files="$(bin/lefthook/get-changed-files "$CONTEXT" '.*')"
+
+if [ -z "$files" ]; then
+  echo "✅ No files to check for trailing newlines"
+  exit 0
+fi
+
+echo "🔍 Checking trailing newlines on $CONTEXT files..."
+
+failed_files=""
+for file in $files; do
+  if [ -f "$file" ] && [ -s "$file" ]; then
+    if ! tail -c 1 "$file" | grep -q '^$'; then
+      echo "❌ Missing trailing newline: $file"
+      failed_files="$failed_files $file"
+    fi
+  fi
+done
+
+if [ -n "$failed_files" ]; then
+  echo ""
+  echo "❌ Trailing newline check failed!"
+  echo "💡 Add trailing newlines to:$failed_files"
+  echo "🔧 Quick fix: for file in$failed_files; do echo >> \"\$file\"; done"
+  echo "🚫 Skip hook: git commit --no-verify"
+  exit 1
+fi
+
+echo "✅ All files have proper trailing newlines"

bin/lefthook/get-changed-files

@@ -0,0 +1,22 @@
+#!/usr/bin/env bash
+# Get changed files based on context (staged, branch, or all)
+set -euo pipefail
+
+CONTEXT="${1:-staged}"
+PATTERN="${2:-.*}"
+
+case "$CONTEXT" in
+  staged)
+    git diff --cached --name-only --diff-filter=ACM | grep -E "$PATTERN" || true
+    ;;
+  branch)
+    # Find base branch (prefer main over master)
+    base="origin/main"
+    git rev-parse --verify --quiet "$base" >/dev/null || base="origin/master"
+    git diff --name-only --diff-filter=ACM "$base"...HEAD | grep -E "$PATTERN" || true
+    ;;
+  *)
+    echo "Usage: $0 {staged|branch} [pattern]" >&2
+    exit 1
+    ;;
+esac

bin/lefthook/prettier-format

@@ -0,0 +1,22 @@
+#!/usr/bin/env bash
+# Format JS/TS/JSON/MD files with Prettier
+set -euo pipefail
+
+CONTEXT="${1:-staged}"
+files="$(bin/lefthook/get-changed-files "$CONTEXT" '\.(js|jsx|ts|tsx|json|md|yml|yaml)$')"
+
+if [ -z "$files" ]; then
+  echo "✅ No files to format with Prettier"
+  exit 0
+fi
+
+echo "💅 Prettier on $CONTEXT files:"
+printf "  %s\n" $files
+
+yarn run prettier --write $files
+
+# Re-stage files if running on staged context
+if [ "$CONTEXT" = "staged" ]; then
+  echo $files | xargs -r git add
+  echo "✅ Re-staged formatted files"
+fi

bin/lefthook/ruby-autofix

@@ -0,0 +1,22 @@
+#!/usr/bin/env bash
+# Auto-fix Ruby files using rake autofix
+set -euo pipefail
+
+CONTEXT="${1:-staged}"
+files="$(bin/lefthook/get-changed-files "$CONTEXT" '\.(rb|rake|ru)$')"
+
+if [ -z "$files" ]; then
+  echo "✅ No Ruby files to autofix"
+  exit 0
+fi
+
+echo "🎨 Autofix on $CONTEXT Ruby files:"
+printf "  %s\n" $files
+
+bundle exec rake autofix
+
+# Re-stage files if running on staged context
+if [ "$CONTEXT" = "staged" ]; then
+  echo $files | xargs -r git add
+  echo "✅ Re-staged formatted files"
+fi

bin/lefthook/ruby-lint

@@ -0,0 +1,23 @@
+#!/usr/bin/env bash
+# Lint Ruby files with RuboCop
+set -euo pipefail
+
+CONTEXT="${1:-staged}"
+files="$(bin/lefthook/get-changed-files "$CONTEXT" '\.(rb|rake|ru)$')"
+
+if [ -z "$files" ]; then
+  echo "✅ No Ruby files to lint"
+  exit 0
+fi
+
+echo "🔍 RuboCop on $CONTEXT Ruby files:"
+printf "  %s\n" $files
+
+if ! bundle exec rubocop --force-exclusion --display-cop-names -- $files; then
+  echo ""
+  echo "❌ RuboCop check failed!"
+  echo "💡 Auto-fix: bundle exec rubocop --auto-correct --force-exclusion -- $files"
+  echo "🚫 Skip hook: git commit --no-verify"
+  exit 1
+fi
+echo "✅ RuboCop checks passed for Ruby files"