Simplify build orchestration for rescript and locale generation

- bin/dev now runs rescript build and locale generation BEFORE starting
  Procfile processes, eliminating race conditions and sleep delays
- Added res:watch npm script (watch without clean) for Procfile.dev
- Removed sleep delays and inline locale generation from Procfile.dev
- Updated build_production_command to include rescript build
- Removed precompile_hook from shakapacker.yml (handled by bin/dev and
  build commands)

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

Co-Authored-By: Claude Opus 4.5 <[email protected]>

Author: justin808

Procfile.dev

@@ -1,10 +1,14 @@
 # Procfile for development using HMR
 # You can run these commands in separate shells
-# ReScript: clean and rebuild, then watch for changes
-rescript: yarn res:dev
+#
+# Note: bin/dev runs precompile tasks (rescript + locale) BEFORE starting these processes.
+# This ensures all generated files exist before webpack starts watching.
+#
+# ReScript watch mode (no clean - bin/dev already did the clean build)
+rescript: yarn res:watch
 # redis: redis-server  # Run Redis as a system service instead (brew services start redis)
 rails: bundle exec thrust bin/rails server -p 3000
-# Sleep to allow rescript files to compile before starting webpack
-# Increased sleep time to ensure rescript initial build completes
-wp-client: sleep 15 &&  RAILS_ENV=development NODE_ENV=development bin/shakapacker-dev-server
-wp-server: sleep 15 && bundle exec rake react_on_rails:locale && HMR=true SERVER_BUNDLE_ONLY=yes bin/shakapacker --watch
+# Client webpack dev server with HMR
+wp-client: RAILS_ENV=development NODE_ENV=development bin/shakapacker-dev-server
+# Server webpack watcher for SSR bundle
+wp-server: HMR=true SERVER_BUNDLE_ONLY=yes bin/shakapacker --watch

config/initializers/react_on_rails.rb

@@ -6,10 +6,11 @@
   config.components_subdirectory = "ror_components"
   config.auto_load_bundle = true
 
-  # Build commands with locale generation
-  config.build_test_command = "RAILS_ENV=test bin/shakapacker"
-  config.build_production_command =
-    "bundle exec rake react_on_rails:locale && RAILS_ENV=production NODE_ENV=production bin/shakapacker"
+  # Build commands
+  # Note: react_on_rails:assets:webpack (run by assets:precompile) depends on react_on_rails:locale,
+  # so locale generation happens automatically. We just need rescript to run first.
+  config.build_test_command = "yarn res:build && RAILS_ENV=test bin/shakapacker"
+  config.build_production_command = "yarn res:build && RAILS_ENV=production NODE_ENV=production bin/shakapacker"
 
   # This is the file used for server rendering of React when using `(prerender: true)`
   # If you are never using server rendering, you may set this to "".

config/shakapacker.yml

@@ -21,6 +21,11 @@ default: &default
   # Use the config.build_production_command in config/initializers/react_on_rails.rb
   shakapacker_precompile: false
 
+  # Note: precompile_hook is not used here because:
+  # - In development: bin/dev runs rescript + locale before starting processes
+  # - In production: build_production_command includes rescript, and
+  #   react_on_rails:assets:webpack already depends on react_on_rails:locale
+
 development:
   <<: *default
   # This is false since we're running `bin/shakapacker -w` in Procfile.dev-setic

package.json

@@ -21,6 +21,7 @@
     "res:clean": "rescript clean",
     "res:format": "rescript format -all",
     "res:dev": "yarn res:clean && rescript build -w",
+    "res:watch": "rescript build -w",
     "res:build": "yarn res:clean && rescript build",
     "lint:eslint": "yarn eslint client --ext \".js,.jsx,.ts\"",
     "lint:prettier": "yarn prettier \"**/*.@(js|jsx)\" --list-different",