From f8217a634dbfe5dbe215e15a500d7b2d2c57f1e2 Mon Sep 17 00:00:00 2001
From: Chuck Smith <eclectic-coding@users.noreply.github.com>
Date: Tue, 26 May 2026 10:36:32 -0400
Subject: [PATCH] feat: Solid Cable channel browser

Adds a channel list to GET /cable showing per-channel message count and
last-message time, ordered by most recent activity; fixes SQLite
datetime string-casting via pluck+map; includes empty state; adds
ActiveSupport::Testing::TimeHelpers to RSpec config for travel helper.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 CHANGELOG.md                                  |  4 +++
 .../solid_stack_web/cable_controller.rb       | 15 +++++++++-
 .../solid_stack_web/application.html.erb      |  9 ++++++
 .../solid_stack_web/cable/index.html.erb      | 16 ++++++++--
 spec/rails_helper.rb                          |  1 +
 spec/requests/solid_stack_web/cable_spec.rb   | 29 ++++++++++++++++++-
 6 files changed, 70 insertions(+), 4 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 2fe8696..441b17b 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Added
+
+- Solid Cable channel browser — `GET /cable` lists all distinct channels with per-channel message count and last-message timestamp, ordered by most recent activity; a total-messages stat and channel-count stat are shown at the top; empty state shown when no messages exist; Cable subnav (Overview) added to the layout
+
 ## [0.5.0] - 2026-05-26
 
 ### Added
diff --git a/app/controllers/solid_stack_web/cable_controller.rb b/app/controllers/solid_stack_web/cable_controller.rb
index 722f362..bbda29a 100644
--- a/app/controllers/solid_stack_web/cable_controller.rb
+++ b/app/controllers/solid_stack_web/cable_controller.rb
@@ -2,7 +2,20 @@ module SolidStackWeb
   class CableController < ApplicationController
     def index
       @total_messages = ::SolidCable::Message.count
-      @channels       = ::SolidCable::Message.distinct.pluck(:channel).sort
+      @channels       = channel_rows
+    end
+
+    private
+
+    def channel_rows
+      ::SolidCable::Message
+        .group(:channel)
+        .order(Arel.sql("MAX(created_at) DESC"))
+        .pluck(:channel, Arel.sql("COUNT(*)"), Arel.sql("MAX(created_at)"))
+        .map do |ch, cnt, last_at|
+          last_at = Time.zone.parse(last_at) if last_at.is_a?(String)
+          { channel: ch, message_count: cnt, last_message_at: last_at }
+        end
     end
   end
 end
diff --git a/app/views/layouts/solid_stack_web/application.html.erb b/app/views/layouts/solid_stack_web/application.html.erb
index 462f69a..141079e 100644
--- a/app/views/layouts/solid_stack_web/application.html.erb
+++ b/app/views/layouts/solid_stack_web/application.html.erb
@@ -35,6 +35,15 @@
     </nav>
   <% end %>
 
+  <% if current_section == :cable %>
+    <nav class="sqw-subnav">
+      <div class="sqw-subnav__inner">
+        <%= link_to "Overview", cable_path,
+              class: "sqw-subnav__link#{" sqw-subnav__link--active" if controller_name == "cable"}" %>
+      </div>
+    </nav>
+  <% end %>
+
   <% if current_section == :queue %>
     <nav class="sqw-subnav">
       <div class="sqw-subnav__inner">
diff --git a/app/views/solid_stack_web/cable/index.html.erb b/app/views/solid_stack_web/cable/index.html.erb
index 7d06a1a..86e1103 100644
--- a/app/views/solid_stack_web/cable/index.html.erb
+++ b/app/views/solid_stack_web/cable/index.html.erb
@@ -16,12 +16,24 @@
 <% if @channels.any? %>
   <table class="sqw-table">
     <thead>
-      <tr><th>Channel</th></tr>
+      <tr>
+        <th>Channel</th>
+        <th>Messages</th>
+        <th>Last Message</th>
+      </tr>
     </thead>
     <tbody>
       <% @channels.each do |channel| %>
-        <tr><td class="sqw-monospace"><%= channel %></td></tr>
+        <tr>
+          <td class="sqw-monospace"><%= channel[:channel] %></td>
+          <td><%= channel[:message_count] %></td>
+          <td class="sqw-muted"><%= channel[:last_message_at]&.strftime("%b %d %H:%M") %></td>
+        </tr>
       <% end %>
     </tbody>
   </table>
+<% else %>
+  <div class="sqw-empty">
+    <p>No cable messages.</p>
+  </div>
 <% end %>
diff --git a/spec/rails_helper.rb b/spec/rails_helper.rb
index de97119..da5676a 100644
--- a/spec/rails_helper.rb
+++ b/spec/rails_helper.rb
@@ -16,4 +16,5 @@
   config.use_transactional_fixtures = true
   config.infer_spec_type_from_file_location!
   config.filter_rails_from_backtrace!
+  config.include ActiveSupport::Testing::TimeHelpers
 end
diff --git a/spec/requests/solid_stack_web/cable_spec.rb b/spec/requests/solid_stack_web/cable_spec.rb
index a3e94e7..308cc93 100644
--- a/spec/requests/solid_stack_web/cable_spec.rb
+++ b/spec/requests/solid_stack_web/cable_spec.rb
@@ -42,9 +42,36 @@
 
       get "#{engine_root}/cable"
 
-      # 2 distinct channels, stat value should be 2 (not 3 messages)
       expect(response.body).to include("Channels")
       expect(response.body).to match(/class="sqw-stat__value">\s*2\s*</)
     end
+
+    it "shows message count and last message time per channel" do
+      SolidCable::Message.broadcast("sports:scores", "goal!")
+      SolidCable::Message.broadcast("sports:scores", "offside")
+
+      get "#{engine_root}/cable"
+
+      expect(response.body).to include("sports:scores")
+      expect(response.body).to include("Messages")
+      expect(response.body).to include("Last Message")
+    end
+
+    it "orders channels by most recent message first" do
+      SolidCable::Message.broadcast("older:channel", "first")
+      travel 1.second do
+        SolidCable::Message.broadcast("newer:channel", "second")
+      end
+
+      get "#{engine_root}/cable"
+
+      expect(response.body.index("newer:channel")).to be < response.body.index("older:channel")
+    end
+
+    it "shows empty state when no messages exist" do
+      SolidCable::Message.delete_all
+      get "#{engine_root}/cable"
+      expect(response.body).to include("No cable messages")
+    end
   end
 end