Use a separate action cable server for hotwire-spark

This will isolate the connection from hotwire-spark from the connection
used by the host app, which most likely will be authenticated. This fixes
the problem where hotwire-spark doesn't work with unauthenticated connections.

Fixes #19

This also lets us remove lots of complexity for the system monkeypatching
the cable server to prevent restarts, since we don't want those when code reloads
in development.

app/assets/javascripts/hotwire_spark.js

@@ -512,7 +512,7 @@ var HotwireSpark = (function () {
     }
   }
 
-  var consumer = createConsumer();
+  var consumer = createConsumer("/hotwire-spark");
 
   function assetNameFromPath(path) {
     return path.split("/").pop().split(".")[0];

app/javascript/hotwire/spark/channels/consumer.js

@@ -1,3 +1,3 @@
 import { createConsumer } from "@rails/actioncable"
 
-export default createConsumer()
+export default createConsumer("/hotwire-spark")

lib/hotwire-spark.rb

@@ -23,5 +23,11 @@ def install_into(application)
     def enabled?
       enabled && defined?(Rails::Server)
     end
+
+    def cable_server
+      @server ||= Hotwire::Spark::ActionCable::Server.new(config: ::ActionCable::Server::Base.config.dup).tap do |server|
+        server.config.connection_class = -> { ::ActionCable::Connection::Base }
+      end
+    end
   end
 end

lib/hotwire/spark/action_cable/server.rb

@@ -0,0 +1,2 @@
+class Hotwire::Spark::ActionCable::Server < ActionCable::Server::Base
+end

lib/hotwire/spark/engine.rb

@@ -12,12 +12,20 @@ class Engine < ::Rails::Engine
       html_paths: %w[ app/controllers app/helpers app/models app/views ],
       stimulus_paths: %w[ app/javascript/controllers ]
 
-    initializer "hotwire_spark.config" do |app|
+    initializer "hotwire_spark.config" do |application|
       config.hotwire.spark.each do |key, value|
         Hotwire::Spark.send("#{key}=", value)
       end
     end
 
+    initializer "hotwire_spark.action_cable.routes" do
+      config.after_initialize do |app|
+        app.routes.prepend do
+          mount Hotwire::Spark.cable_server => "/hotwire-spark", internal: true, anchor: true
+        end
+      end
+    end
+
     initializer "hotwire_spark.install" do |application|
       Hotwire::Spark.install_into application if Hotwire::Spark.enabled?
     end

lib/hotwire/spark/installer.rb

@@ -11,9 +11,6 @@ def install
   end
 
   def configure_middleware
-    ::ActionCable::Server::Base.prepend(Hotwire::Spark::ActionCable::PersistentCableServer)
-
-    middleware.insert_before ActionDispatch::Executor, Hotwire::Spark::ActionCable::PersistentCableMiddleware
     middleware.use Hotwire::Spark::Middleware
   end
 
@@ -39,7 +36,7 @@ def monitor(paths_name, action:)
     end
 
     def broadcast_reload_action(action, file_path)
-      ActionCable.server.broadcast "hotwire_spark", reload_message_for(action, file_path)
+      Hotwire::Spark.cable_server.broadcast "hotwire_spark", reload_message_for(action, file_path)
     end
 
     def reload_message_for(action, file_path)

test/test_helper.rb

@@ -16,18 +16,11 @@
 
 require "helpers/files_helper"
 
-::ActionCable::Server::Base.prepend(Hotwire::Spark::ActionCable::PersistentCableServer)
-
 class ActiveSupport::TestCase
   include FilesHelper
 
   setup do
     reload_rails_reloader
-    ActionCable.server.suppress_restarts = true
-  end
-
-  teardown do
-    ActionCable.server.suppress_restarts = false
   end
 
   private