chats 0.4.2 → 0.5.1

data/lib/chats/models/message.rb

@@ -81,6 +81,77 @@ module Chats
     scope :oldest_first, -> { order(created_at: :asc, id: :asc) }
     scope :visible, -> { where(deleted_at: nil) }
 
+    # What messagers wrote: text messages that still stand. "Human" is the
+    # gem's word for a text message, written by a messager, person or not
+    # (a bot's or a headless desk's message counts), as opposed to a system
+    # line ("Alice joined", "Your ride was cancelled"), which is the app
+    # talking. A tombstone no longer says anything. A photo with no caption
+    # is still a text message and still counts. The inbox does NOT read through this,
+    # on purpose — see README "Real conversations vs system-only threads".
+    scope :human, -> { where(kind: "text").visible }
+
+    # The newest human message of each given conversation, at most one row
+    # per conversation, in ONE query: the batch twin of
+    # Conversation#last_human_message for lists (admin walls, moderation
+    # queues). +conversations+ is a page of records, ids, a single record, or
+    # a Chats::Conversation relation — an unloaded relation is read as a
+    # subquery, never loaded first, and without its `includes`/`preload`/
+    # `eager_load` (they load records; a subquery has none to load). So a
+    # relation that FILTERS on an associated table must join it explicitly
+    # (`joins` / `left_joins`): `includes(:x).references(:x).where(x: …)`
+    # loses the join and raises. Or pass the loaded page, or its ids.
+    #
+    # The result is a relation, so the columns can be narrowed:
+    #
+    #   Chats::Message.last_human_in(conversations)
+    #                 .select(:id, :conversation_id, :created_at)
+    #                 .index_by(&:conversation_id)          # { id => message }
+    #
+    # A conversation nobody wrote in has no row. ROW_NUMBER works on every
+    # supported adapter (PostgreSQL, SQLite and MySQL 8), the same ranking
+    # Chats::Inbox uses for its stacks. The ranked subquery is built unscoped,
+    # so neither a scope it is called on nor a `.select` chained onto the
+    # result leaks into it.
+    scope :last_human_in, lambda { |conversations|
+      ids = Chats::Message.conversation_ids_in(conversations)
+      next none if ids.is_a?(Array) && ids.empty?
+
+      ranking = "ROW_NUMBER() OVER (PARTITION BY chats_messages.conversation_id " \
+                "ORDER BY chats_messages.created_at DESC, chats_messages.id DESC) AS human_rank"
+      ranked = Chats::Message.unscoped.human.where(conversation_id: ids)
+                             .select(Arel.sql("chats_messages.id"), Arel.sql(ranking))
+      newest = Chats::Message.unscoped.from(ranked, :ranked_messages)
+                             .where(Arel.sql("ranked_messages.human_rank = 1"))
+                             .select(Arel.sql("ranked_messages.id"))
+      where(id: newest)
+    }
+
+    # The conversation ids +last_human_in+ ranks within: an Array for records
+    # and ids (a loaded relation included, read from memory), a subquery for
+    # an unloaded relation, as a derived table because MySQL rejects LIMIT
+    # directly inside IN, and a page is usually limited.
+    #
+    # A LIMITed or OFFSET page is its own query, word for word (its select
+    # list, DISTINCT and ORDER BY decide WHICH rows it holds, and PostgreSQL
+    # refuses a DISTINCT whose ORDER BY left the select list), so it must
+    # expose an `id` column, as the default select list does. Any other
+    # relation is just a set of ids: narrowed to `id`, unordered (the inbox's
+    # ORDER BY COALESCE(…) next to DISTINCT id is another refusal).
+    def self.conversation_ids_in(conversations) # :nodoc:
+      unless conversations.is_a?(ActiveRecord::Relation)
+        return Array(conversations).map { |c| c.is_a?(Chats::Conversation) ? c.id : c }
+      end
+      unless conversations.klass <= Chats::Conversation
+        raise ArgumentError, "expected Chats::Conversation records, ids or a relation of them, " \
+                             "got a #{conversations.klass.name} relation"
+      end
+      return conversations.map(&:id) if conversations.loaded?
+
+      page = conversations.except(:includes, :preload, :eager_load)
+      page = page.reselect(page.arel_table[:id]).reorder(nil) unless page.limit_value || page.offset_value
+      Chats::Conversation.unscoped.from(page, :conversation_page).select(Arel.sql("conversation_page.id"))
+    end
+
     # Keyset (cursor) pagination for infinite scroll-up. OFFSET pagination
     # drifts when new messages arrive mid-scroll; anchoring on the oldest
     # loaded message can't skip or duplicate. `(created_at, id)` because
@@ -93,17 +164,35 @@ module Chats
     }
 
     # Catch-up scopes for ConversationsController#refresh (the stale-thread
-    # recovery path — see the thread controller's refresh trigger). `since`
-    # is the newest `updated_at` the client has rendered:
-    #   created_since — messages that arrived while the tab was asleep
-    #                   (appended);
-    #   updated_since — messages the client HAS rendered that changed since
-    #                   (edits, soft-delete tombstones — replaced in place).
-    # `updated_since` excludes fresh rows so nothing renders twice — a Turbo
-    # append of an existing dom_id would otherwise also MOVE that bubble to
-    # the end (Turbo removes-then-appends on id collision).
+    # recovery path — see the thread controller's catch-up), on the client's
+    # two cursors. Each is a moment, to the microsecond, up to which the
+    # client KNOWS it has everything (a complete render, or a catch-up that
+    # succeeded — never a live bubble, which can arrive past one it missed),
+    # with the ids it has on screen from that moment on:
+    #   created_after(at, seen:)   — created at or after that moment and not
+    #                                on screen: appended;
+    #   created_through(at, seen:) — the complement: what the client has;
+    #   updated_after(at, seen:)   — changed after that moment, or at it and
+    #                                not among the ones seen in that version.
+    # Replacements are created_through(after).updated_after(changed): changes
+    # to what is on screen, never a second copy of what is appended — a Turbo
+    # append of an existing dom_id would MOVE that bubble to the end. Two
+    # cursors, because one can't answer both: a message created at 10:15 and
+    # missed is older than a 10:20 edit that did arrive.
     # Pattern from Basecamp's Campfire (Rooms::RefreshesController):
     # https://github.com/basecamp/once-campfire
+    scope :created_after, lambda { |at, seen: []|
+      where(arel_table[:created_at].gteq(at)).where.not(id: seen)
+    }
+    scope :created_through, lambda { |at, seen: []|
+      where(arel_table[:created_at].lt(at)).or(where(id: seen))
+    }
+    scope :updated_after, lambda { |at, seen: []|
+      where(arel_table[:updated_at].gt(at)).or(where(updated_at: at).where.not(id: seen))
+    }
+    # The one-cursor catch-up of bubbles ejected before they carried the
+    # cursors above (`data-updated-at-ms` alone, rounded up): what was
+    # created after it, and what was rendered by then and changed since.
     scope :created_since, ->(time) { where("chats_messages.created_at > ?", time) }
     scope :updated_since, lambda { |time|
       where("chats_messages.updated_at > ?", time).where("chats_messages.created_at <= ?", time)

data/lib/chats/models/reaction.rb

@@ -27,6 +27,15 @@ module Chats
     # Rails behavior: https://guides.rubyonrails.org/active_record_callbacks.html#using-both-after-create-commit-and-after-update-commit
     after_commit :broadcast_change, on: %i[create destroy]
 
+    # A reaction changes how its message renders, so it moves the message's
+    # updated_at: that is what a thread catching up on missed broadcasts asks
+    # for (Message.updated_after). update_columns, not touch: the
+    # broadcast above already re-renders the bubble, and touch would run the
+    # message's own update callbacks and broadcast it a second time. Not when
+    # the message itself is being destroyed and takes its reactions with it.
+    after_create :stamp_message
+    after_destroy :stamp_message, unless: :destroyed_by_association
+
     # Add the reaction if absent, remove it if present (the universal
     # tap-to-toggle semantic). Returns the created reaction, or false when
     # toggled off. Race-safe: a concurrent double-tap resolves through the
@@ -68,6 +77,10 @@ module Chats
       errors.add(:reactor, :not_a_participant) unless message.conversation.participant?(reactor)
     end
 
+    def stamp_message
+      message.update_columns(updated_at: Time.current)
+    end
+
     def broadcast_change
       Chats::Broadcasts.message_updated(message) unless message.destroyed?
     end

data/lib/chats/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Chats
-  VERSION = "0.4.2"
+  VERSION = "0.5.1"
 end

data/lib/generators/chats/templates/initializer.rb

@@ -226,6 +226,7 @@ Chats.configure do |config|
   #   app/views/chats/slots/_composer_actions.html.erb          your own buttons above the composer
   #   app/views/chats/slots/_locked_composer.html.erb           the locked composer's body
   #   app/views/chats/slots/_message_meta.html.erb              after each bubble's timestamp
+  #   app/views/chats/slots/_dock_panel_footer.html.erb         beside "See all messages" in the dock
   #
   # `rails generate chats:views` is still there for wholesale restyling.
 end

data/lib/generators/chats/views_generator.rb

@@ -24,13 +24,14 @@ module Chats
       # Which groups to eject. Default copies everything renderable.
       class_option :views,
                    type: :array,
-                   default: %w[conversations messages shared],
-                   desc: "Which view groups to copy (conversations, messages, shared)"
+                   default: %w[conversations messages shared dock],
+                   desc: "Which view groups to copy (conversations, messages, shared, dock)"
 
       def copy_views
         directory "chats/conversations", "app/views/chats/conversations" if include?("conversations")
         directory "chats/messages", "app/views/chats/messages" if include?("messages")
         directory "chats/shared", "app/views/chats/shared" if include?("shared")
+        directory "chats/dock", "app/views/chats/dock" if include?("dock")
       end
 
       def show_styling_tip

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: chats
 version: !ruby/object:Gem::Version
-  version: 0.4.2
+  version: 0.5.1
 platform: ruby
 authors:
 - rameerez
 bindir: exe
 cert_chain: []
-date: 2026-10-17 00:00:00.000000000 Z
+date: 2026-10-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -135,16 +135,31 @@ files:
 - app/helpers/chats/engine_helper.rb
 - app/javascript/chats/composer_controller.js
 - app/javascript/chats/debounced_submit_controller.js
+- app/javascript/chats/dock_controller.js
+- app/javascript/chats/recovery.js
 - app/javascript/chats/refresh_inbox_controller.js
 - app/javascript/chats/thread_controller.js
+- app/javascript/chats/unread_badge_controller.js
 - app/views/chats/conversations/_conversation_row.html.erb
 - app/views/chats/conversations/_group.html.erb
+- app/views/chats/conversations/_history.html.erb
 - app/views/chats/conversations/_locked_composer.html.erb
 - app/views/chats/conversations/_messages_page.html.erb
 - app/views/chats/conversations/_read_state.html.erb
+- app/views/chats/conversations/_thread.html.erb
+- app/views/chats/conversations/badge.turbo_stream.erb
+- app/views/chats/conversations/dock.html.erb
 - app/views/chats/conversations/index.html.erb
 - app/views/chats/conversations/refresh.turbo_stream.erb
 - app/views/chats/conversations/show.html.erb
+- app/views/chats/conversations/window.html.erb
+- app/views/chats/dock/_badge.html.erb
+- app/views/chats/dock/_dock.html.erb
+- app/views/chats/dock/_signal.html.erb
+- app/views/chats/dock/_update.turbo_stream.erb
+- app/views/chats/dock/_window.html.erb
+- app/views/chats/dock/_window_header.html.erb
+- app/views/chats/dock/_window_shell.html.erb
 - app/views/chats/messages/_action.html.erb
 - app/views/chats/messages/_composer.html.erb
 - app/views/chats/messages/_message.html.erb
@@ -162,6 +177,11 @@ files:
 - docs/PRD.md
 - docs/campfire_review.md
 - docs/screenshots/composer-actions.png
+- docs/screenshots/dock-bar.png
+- docs/screenshots/dock-desktop.png
+- docs/screenshots/dock-phone-button.png
+- docs/screenshots/dock-phone-panel.png
+- docs/screenshots/dock-phone-window.png
 - docs/screenshots/inbox.png
 - docs/screenshots/system-note-action.png
 - docs/screenshots/thread-actions-phone.png
@@ -176,6 +196,7 @@ files:
 - lib/chats/errors.rb
 - lib/chats/inbox.rb
 - lib/chats/inbox_group.rb
+- lib/chats/inbox_refresh_job.rb
 - lib/chats/macros.rb
 - lib/chats/models/application_record.rb
 - lib/chats/models/concerns/chat_subject.rb