Fix RBS validation issues and improve error handling

- Replace Open3.capture3 timeout option with Timeout.timeout wrapper for Ruby 3.0+ compatibility
- Remove duplicate RBS definitions for PrerenderError and JsonParseError from sig/react_on_rails.rbs
- Add comment explaining error class definitions are in separate files
- Improve test documentation with clearer explanation of RBS runtime checking
- Add timeout handling to Pro package RBS rake tasks
- Add RuboCop disable for BlockLength in Pro package RBS rake file

The timeout option for Open3.capture3 was added in Ruby 3.3, but we support Ruby >= 3.0.0.
Using Timeout.timeout provides compatibility across all supported Ruby versions.

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

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

Author: justin808

rakelib/rbs.rake

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "open3"
+require "timeout"
 
 # rubocop:disable Metrics/BlockLength
 namespace :rbs do
@@ -15,8 +16,10 @@ namespace :rbs do
     # This allows us to distinguish between actual validation errors and warnings
     # Note: Must use bundle exec even though rake runs in bundle context because
     # spawned shell commands via Open3.capture3() do NOT inherit bundle context
-    # Timeout after 60 seconds to prevent hung processes in CI environments
-    stdout, stderr, status = Open3.capture3("bundle exec rbs -I sig validate", timeout: 60)
+    # Wrap in Timeout to prevent hung processes in CI environments (60 second timeout)
+    stdout, stderr, status = Timeout.timeout(60) do
+      Open3.capture3("bundle exec rbs -I sig validate")
+    end
 
     if status.success?
       puts "✓ RBS validation passed"
@@ -46,8 +49,10 @@ namespace :rbs do
     # Use Open3 for better error handling
     # Note: Must use bundle exec even though rake runs in bundle context because
     # spawned shell commands via Open3.capture3() do NOT inherit bundle context
-    # Timeout after 60 seconds to prevent hung processes in CI environments
-    stdout, stderr, status = Open3.capture3("bundle exec steep check", timeout: 60)
+    # Wrap in Timeout to prevent hung processes in CI environments (60 second timeout)
+    stdout, stderr, status = Timeout.timeout(60) do
+      Open3.capture3("bundle exec steep check")
+    end
 
     if status.success?
       puts "✓ Steep type checking passed"

react_on_rails_pro/rakelib/rbs.rake

@@ -1,9 +1,11 @@
 # frozen_string_literal: true
 
 require "open3"
+require "timeout"
 
 # NOTE: Pro package does not include Steep tasks (:steep, :all) as it does not
 # use Steep type checker. Only RBS validation is performed.
+# rubocop:disable Metrics/BlockLength
 namespace :rbs do
   desc "Validate RBS type signatures"
   task :validate do
@@ -16,7 +18,10 @@ namespace :rbs do
     # This allows us to distinguish between actual validation errors and warnings
     # Note: Must use bundle exec even though rake runs in bundle context because
     # spawned shell commands via Open3.capture3() do NOT inherit bundle context
-    stdout, stderr, status = Open3.capture3("bundle exec rbs -I sig validate")
+    # Wrap in Timeout to prevent hung processes in CI environments (60 second timeout)
+    stdout, stderr, status = Timeout.timeout(60) do
+      Open3.capture3("bundle exec rbs -I sig validate")
+    end
 
     if status.success?
       puts "✓ RBS validation passed"
@@ -39,3 +44,4 @@ namespace :rbs do
     puts "\nTotal: #{sig_files.count} files"
   end
 end
+# rubocop:enable Metrics/BlockLength

sig/react_on_rails.rbs

@@ -9,30 +9,9 @@ module ReactOnRails
   def self.configure: () { (Configuration) -> void } -> void
   def self.configuration: () -> Configuration
 
+  # Error classes are defined in separate RBS files:
+  # - sig/react_on_rails/prerender_error.rbs
+  # - sig/react_on_rails/json_parse_error.rbs
   class Error < StandardError
   end
-
-  class PrerenderError < Error
-    attr_reader component_name: String?
-    attr_reader js_code: String?
-    attr_reader err: Hash[Symbol, untyped]?
-    attr_reader props: (Hash[Symbol, untyped] | String)?
-    attr_reader console_messages: Array[String]?
-
-    def initialize: (
-      ?component_name: String?,
-      ?js_code: String?,
-      ?err: Hash[Symbol, untyped]?,
-      ?props: (Hash[Symbol, untyped] | String)?,
-      ?console_messages: Array[String]?
-    ) -> void
-
-    def to_honeybadger_context: () -> Hash[Symbol, untyped]
-    def raven_context: () -> Hash[Symbol, untyped]
-    def to_error_context: () -> Hash[Symbol, untyped]
-  end
-
-  class JsonParseError < Error
-    def initialize: (Hash[Symbol, untyped] err) -> void
-  end
 end

spec/react_on_rails/rbs_runtime_checking_spec.rb

@@ -15,8 +15,13 @@
     it "catches invalid type assignments to configuration" do
       # This test verifies runtime checking actually works by intentionally
       # violating a type signature and expecting RBS to catch it
-      # Type signature: sig/react_on_rails/configuration.rbs:4
-      # attr_accessor server_bundle_js_file: String
+      #
+      # Type signature defined in: sig/react_on_rails/configuration.rbs
+      #   attr_accessor server_bundle_js_file: String
+      #
+      # When RBS runtime checking is enabled via rakelib/run_rspec.rake, the
+      # RBS::Test::Hook wraps all method calls to ReactOnRails classes and validates
+      # that arguments and return values match the type signatures.
       expect do
         config = ReactOnRails::Configuration.new(
           server_bundle_js_file: 123 # Invalid: should be String, not Integer