Restore server bundle support and fix failing tests

- Add back server bundle logic that respects server_bundle_output_path configuration
- Server bundles with configured output path skip manifest lookup for better performance
- Server bundles without configured path use fallback logic like client bundles
- Fix test mocking to properly test fallback behavior when server_bundle_output_path is nil
- All utils tests now pass (53/53)

This integrates the simpler remote approach while preserving the essential
server bundle functionality that the configuration and tests expect.

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

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

Author: justin808

lib/react_on_rails/utils.rb

@@ -71,6 +71,17 @@ def self.server_bundle_path_is_http?
     end
 
     def self.bundle_js_file_path(bundle_name)
+      # Check if this is a server bundle with configured output path - skip manifest lookup
+      if server_bundle?(bundle_name)
+        config = ReactOnRails.configuration
+        root_path = Rails.root || "."
+
+        # Use configured server_bundle_output_path if present
+        if config.server_bundle_output_path.present?
+          return File.expand_path(File.join(root_path, config.server_bundle_output_path, bundle_name))
+        end
+      end
+
       # Either:
       # 1. Using same bundle for both server and client, so server bundle will be hashed in manifest
       # 2. Using a different bundle (different Webpack config), so file is not hashed, and
@@ -117,6 +128,12 @@ def self.bundle_js_file_path(bundle_name)
       File.expand_path(fallback_locations.first)
     end
 
+    private_class_method def self.server_bundle?(bundle_name)
+      config = ReactOnRails.configuration
+      bundle_name == config.server_bundle_js_file ||
+      bundle_name == config.rsc_bundle_js_file
+    end
+
     def self.server_bundle_js_file_path
       return @server_bundle_path if @server_bundle_path && !Rails.env.development?
 

spec/react_on_rails/utils_spec.rb

@@ -244,6 +244,9 @@ def mock_dev_server_running
               it "returns the standard location path" do
                 server_bundle_name = "server-bundle.js"
                 mock_bundle_configs(server_bundle_name: server_bundle_name)
+                # Override server_bundle_output_path to test fallback behavior
+                allow(ReactOnRails).to receive_message_chain("configuration.server_bundle_output_path")
+                  .and_return(nil)
                 mock_missing_manifest_entry(server_bundle_name)
 
                 # Mock File.exist? to return false for environment-specific path but true for standard path
@@ -264,6 +267,9 @@ def mock_dev_server_running
               it "returns the environment-specific path as final fallback" do
                 server_bundle_name = "server-bundle.js"
                 mock_bundle_configs(server_bundle_name: server_bundle_name)
+                # Override server_bundle_output_path to test fallback behavior
+                allow(ReactOnRails).to receive_message_chain("configuration.server_bundle_output_path")
+                  .and_return(nil)
                 mock_missing_manifest_entry(server_bundle_name)
 
                 # Mock File.exist? to return false for all paths