Fix pre-commit hooks to properly lint Pro directory

The previous hooks only linted files in the root directory, but the
react_on_rails_pro/ directory has its own ESLint and Prettier configs
that need to be run separately. This caused CI failures when Pro files
had linting issues that weren't caught locally.

Changes:
1. Updated bin/lefthook/eslint-lint to:
   - Separate files into root and Pro directories
   - Run ESLint in Pro directory with Pro's config for Pro files
   - Track exit codes and fail if either check fails

2. Updated bin/lefthook/prettier-format to:
   - Separate files into root and Pro directories
   - Run Prettier in Pro directory with Pro's config for Pro files

3. Fixed Prettier formatting in commonWebpackConfig.js:
   - Condensed multi-line conditional to single line per Pro's config

This ensures local pre-commit hooks match CI behavior exactly, preventing
the frustrating situation where commits succeed locally but fail in CI.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: justin808

bin/lefthook/eslint-lint

@@ -10,18 +10,55 @@ if [ -z "$files" ]; then
   exit 0
 fi
 
-if [ "$CONTEXT" = "all-changed" ]; then
-  echo "🔍 ESLint on all changed files:"
-else
-  echo "🔍 ESLint on $CONTEXT files:"
+# Separate files into root and Pro directories
+root_files=$(echo "$files" | grep -v '^react_on_rails_pro/' || true)
+pro_files=$(echo "$files" | grep '^react_on_rails_pro/' || true)
+
+exit_code=0
+
+# Lint root files
+if [ -n "$root_files" ]; then
+  if [ "$CONTEXT" = "all-changed" ]; then
+    echo "🔍 ESLint on root changed files:"
+  else
+    echo "🔍 ESLint on root $CONTEXT files:"
+  fi
+  printf "  %s\n" $root_files
+
+  if ! yarn run eslint $root_files --report-unused-disable-directives --fix; then
+    exit_code=1
+  fi
+
+  # Re-stage files if running on staged or all-changed context
+  if [ "$CONTEXT" = "staged" ] || [ "$CONTEXT" = "all-changed" ]; then
+    echo $root_files | xargs -r git add
+  fi
 fi
-printf "  %s\n" $files
 
-# Run ESLint with auto-fix
-yarn run eslint $files --report-unused-disable-directives --fix
+# Lint Pro files (using Pro's ESLint config)
+if [ -n "$pro_files" ]; then
+  if [ "$CONTEXT" = "all-changed" ]; then
+    echo "🔍 ESLint on Pro changed files:"
+  else
+    echo "🔍 ESLint on Pro $CONTEXT files:"
+  fi
+  printf "  %s\n" $pro_files
+
+  # Strip react_on_rails_pro/ prefix for running in Pro directory
+  pro_files_relative=$(echo "$pro_files" | sed 's|^react_on_rails_pro/||')
 
-# Re-stage files if running on staged or all-changed context
-if [ "$CONTEXT" = "staged" ] || [ "$CONTEXT" = "all-changed" ]; then
-  echo $files | xargs -r git add
-  echo "✅ Re-staged fixed files"
+  if ! (cd react_on_rails_pro && yarn run eslint $pro_files_relative --report-unused-disable-directives --fix); then
+    exit_code=1
+  fi
+
+  # Re-stage files if running on staged or all-changed context
+  if [ "$CONTEXT" = "staged" ] || [ "$CONTEXT" = "all-changed" ]; then
+    echo $pro_files | xargs -r git add
+  fi
+fi
+
+if [ $exit_code -eq 0 ]; then
+  echo "✅ ESLint checks passed"
 fi
+
+exit $exit_code

bin/lefthook/prettier-format

@@ -10,17 +10,45 @@ if [ -z "$files" ]; then
   exit 0
 fi
 
-if [ "$CONTEXT" = "all-changed" ]; then
-  echo "💅 Prettier on all changed files:"
-else
-  echo "💅 Prettier on $CONTEXT files:"
+# Separate files into root and Pro directories
+root_files=$(echo "$files" | grep -v '^react_on_rails_pro/' || true)
+pro_files=$(echo "$files" | grep '^react_on_rails_pro/' || true)
+
+# Format root files
+if [ -n "$root_files" ]; then
+  if [ "$CONTEXT" = "all-changed" ]; then
+    echo "💅 Prettier on root changed files:"
+  else
+    echo "💅 Prettier on root $CONTEXT files:"
+  fi
+  printf "  %s\n" $root_files
+
+  yarn run prettier --write $root_files
+
+  # Re-stage files if running on staged or all-changed context
+  if [ "$CONTEXT" = "staged" ] || [ "$CONTEXT" = "all-changed" ]; then
+    echo $root_files | xargs -r git add
+  fi
 fi
-printf "  %s\n" $files
 
-yarn run prettier --write $files
+# Format Pro files (using Pro's Prettier config)
+if [ -n "$pro_files" ]; then
+  if [ "$CONTEXT" = "all-changed" ]; then
+    echo "💅 Prettier on Pro changed files:"
+  else
+    echo "💅 Prettier on Pro $CONTEXT files:"
+  fi
+  printf "  %s\n" $pro_files
 
-# Re-stage files if running on staged or all-changed context
-if [ "$CONTEXT" = "staged" ] || [ "$CONTEXT" = "all-changed" ]; then
-  echo $files | xargs -r git add
-  echo "✅ Re-staged formatted files"
+  # Strip react_on_rails_pro/ prefix for running in Pro directory
+  pro_files_relative=$(echo "$pro_files" | sed 's|^react_on_rails_pro/||')
+
+  (cd react_on_rails_pro && yarn run prettier --write $pro_files_relative)
+
+  # Re-stage files if running on staged or all-changed context
+  if [ "$CONTEXT" = "staged" ] || [ "$CONTEXT" = "all-changed" ]; then
+    echo $pro_files | xargs -r git add
+  fi
 fi
+
+echo "✅ Prettier formatting complete"

react_on_rails_pro/spec/dummy/config/webpack/commonWebpackConfig.js

@@ -55,10 +55,7 @@ const commonWebpackConfig = () => {
   // Shakapacker 9 defaults to namedExport: true, but our code uses default imports
   // Override to use the old behavior for backward compatibility
   config.module.rules.forEach((rule) => {
-    if (
-      rule.test &&
-      (rule.test.test('example.module.scss') || rule.test.test('example.module.css'))
-    ) {
+    if (rule.test && (rule.test.test('example.module.scss') || rule.test.test('example.module.css'))) {
       if (Array.isArray(rule.use)) {
         rule.use.forEach((loader) => {
           if (