Add ActionCable connection identification

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

Author: dadachi

app/channels/application_cable/channel.rb

@@ -1,4 +1,11 @@
 module ApplicationCable
   class Channel < ActionCable::Channel::Base
+    # All current channels are public (Turbo::StreamsChannel for display pages).
+    # If an authenticated channel is added in the future, reject unauthorized
+    # connections in that channel's #subscribed method:
+    #
+    #   def subscribed
+    #     reject unless connection.current_shopkeeper
+    #   end
   end
 end

app/channels/application_cable/connection.rb

@@ -1,4 +1,24 @@
 module ApplicationCable
   class Connection < ActionCable::Connection::Base
+    identified_by :current_shopkeeper, :current_account
+
+    def connect
+      self.current_shopkeeper = find_shopkeeper
+      self.current_account = find_account
+    end
+
+    private
+
+    def find_shopkeeper
+      env["warden"]&.user(:shopkeeper)
+    end
+
+    # Display pages are public — anonymous connections are allowed.
+    # Shopkeeper auth is header-based (devise_token_auth), so most
+    # WebSocket connections will be anonymous. If an authenticated-only
+    # channel is added in the future, reject in that channel's #subscribed.
+    def find_account
+      current_shopkeeper&.accounts&.order(created_at: :asc)&.first
+    end
   end
 end

app/views/display/shops/show.html.erb

@@ -1,3 +1,4 @@
+<%# These streams are public by design — display pages are unauthenticated %>
 <%= turbo_stream_from @shop, :tb_stream_full_reload_entire_page %>
 <%= turbo_stream_from @shop, :tb_stream_update_item_tags %>
 

test/channels/application_cable/connection_test.rb

@@ -1,11 +1,23 @@
 require "test_helper"
 
 class ApplicationCable::ConnectionTest < ActionCable::Connection::TestCase
-  # test "connects with cookies" do
-  #   cookies.signed[:user_id] = 42
-  #
-  #   connect
-  #
-  #   assert_equal connection.user_id, "42"
-  # end
+  test "anonymous connection succeeds with nil shopkeeper and account" do
+    connect
+
+    assert_nil connection.current_shopkeeper
+    assert_nil connection.current_account
+  end
+
+  test "authenticated connection identifies shopkeeper and account" do
+    shopkeeper = shopkeepers(:one)
+    account = shopkeeper.create_default_account
+    warden = Minitest::Mock.new
+    warden.expect(:user, shopkeeper, [:shopkeeper])
+
+    connect env: {"warden" => warden}
+
+    assert_equal shopkeeper, connection.current_shopkeeper
+    assert_equal account, connection.current_account
+    warden.verify
+  end
 end