Refine locale generation improvements

Changes:
- Fix CHANGELOG PR reference (2091 → 2092)
- Add safety guard for empty locale_files array to prevent nil comparison errors
- Improve ENV variable parsing to accept force=true|1|yes (more flexible)
- Remove unnecessary sleep in test (timestamp difference already guaranteed)
- Add RBS type signatures for Locales module and classes

All tests pass and RuboCop clean.

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

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

Author: justin808

CHANGELOG.md

@@ -25,7 +25,7 @@ Changes since the last non-beta release.
 
 #### Improved
 
-- **Idempotent Locale Generation**: The `react_on_rails:locale` rake task is now idempotent, automatically skipping generation when locale files are already up-to-date. This makes it safe to call multiple times (e.g., in Shakapacker's `precompile_hook`) without duplicate work. Added `force=true` option to override timestamp checking. [PR 2091](https://github.com/shakacode/react_on_rails/pull/2091) by [justin808](https://github.com/justin808).
+- **Idempotent Locale Generation**: The `react_on_rails:locale` rake task is now idempotent, automatically skipping generation when locale files are already up-to-date. This makes it safe to call multiple times (e.g., in Shakapacker's `precompile_hook`) without duplicate work. Added `force=true` option to override timestamp checking. [PR 2092](https://github.com/shakacode/react_on_rails/pull/2092) by [justin808](https://github.com/justin808).
 
 ### [v16.2.0.beta.12] - 2025-11-20
 

lib/react_on_rails/locales/base.rb

@@ -64,6 +64,8 @@ def exist_files
       end
 
       def files_are_outdated
+        return true if locale_files.empty?
+
         latest_yml = locale_files.map { |file| File.mtime(file) }.max
         earliest = exist_files.map { |file| File.mtime(file) }.min
         latest_yml > earliest

lib/tasks/locale.rake

@@ -15,7 +15,7 @@ namespace :react_on_rails do
                    Example: rake react_on_rails:locale force=true
   DESC
   task locale: :environment do
-    force = ENV["force"] == "true"
+    force = %w[true 1 yes].include?(ENV["force"]&.downcase)
     ReactOnRails::Locales.compile(force: force)
   end
 end

sig/react_on_rails/locales.rbs

@@ -0,0 +1,46 @@
+module ReactOnRails
+  module Locales
+    def self.compile: (?force: bool) -> (ToJs | ToJson)
+    def self.check_config_directory_exists: (directory: String?, key_name: String, remove_if: String) -> void
+
+    class Base
+      def initialize: (?force: bool) -> void
+
+      private
+
+      def file_format: () -> String?
+      def obsolete?: () -> bool
+      def exist_files: () -> Array[String]
+      def files_are_outdated: () -> bool
+      def file_names: () -> Array[String]
+      def files: () -> Array[String]
+      def file: (String name) -> String
+      def locale_files: () -> Array[String]
+      def i18n_dir: () -> String?
+      def i18n_yml_dir: () -> String?
+      def default_locale: () -> String
+      def convert: () -> void
+      def generate_file: (String template, String path) -> void
+      def generate_translations: () -> [String, String]
+      def format: (untyped input) -> Symbol
+      def flatten_defaults: (Hash[untyped, untyped] val) -> Hash[Symbol, Hash[Symbol, untyped]]
+      def flatten: (Hash[untyped, untyped] translations) -> Hash[Symbol, untyped]
+      def template_translations: () -> String
+      def template_default: () -> String
+    end
+
+    class ToJs < Base
+      private
+
+      def file_format: () -> String
+    end
+
+    class ToJson < Base
+      private
+
+      def file_format: () -> String
+      def template_translations: () -> String
+      def template_default: () -> String
+    end
+  end
+end

spec/react_on_rails/locales_to_js_spec.rb

@@ -56,7 +56,6 @@ module ReactOnRails
           initial_mtime = File.mtime(translations_path)
 
           # Touch files to make them newer than YAML (up-to-date)
-          sleep 0.01 # Ensure timestamp difference
           future_time = Time.current + 1.minute
           FileUtils.touch(translations_path, mtime: future_time)
           FileUtils.touch(default_path, mtime: future_time)