layered-assistant-rails 0.9.0 → 0.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 68ea0fb66720732e6e05a139e873ec6eb90912967e1a6e9c956461e76d7e347c
-  data.tar.gz: f4e561987280115a3e160a42a747ee036aab7373c8f65fcc33d36eac01dafec7
+  metadata.gz: 801e95c2e899ba36085052a8971f405f2a7d5b12f3129193cf738978d0653c2c
+  data.tar.gz: b9fefa9325aaf76a334d7f958a37cdf119f73f59bddf08cfbd77b20a62ffddcd
 SHA512:
-  metadata.gz: 56758769136c2e033813dcf466444c778643483739bcafa5048962bca0786e167a04a6a9de9085eed038a8d492cd08c2328363cc1a98f2ea58024bd6096efd99
-  data.tar.gz: 054c1b233c696bbec3d6c7e4e643c28fc69ffc378ce5ffa11b3110de59ed9cf8bbb87c9ed582b442f437a724a7d502609171c3022b67eb43f8b2a759a8190059
+  metadata.gz: cff5a257d775b91f808b1b92cf52a5c9f25be2fe5885edb8557b074c6994f1d664645807c783464552f0b409e7115f65e598fde7618b90f5f34b29c8b8d48dc7
+  data.tar.gz: 5e7bf84dafd4756af9bfa208a9711a16411a7543e7bc3b657e56f591643d2d9e748bb32b80cf8bbc3fc469a719ec79382b95847f30dd1ef13926bb8b71c672fd

data/.claude/skills/layered-assistant-rails/SKILL.md

@@ -27,12 +27,14 @@ The install generator:
 
 1. Verifies `layered-ui-rails` is installed - invokes its installer if not.
 2. Adds `import "layered_assistant"` to `app/javascript/application.js` (after the `layered_ui` import).
-3. Creates `config/initializers/layered_assistant.rb` with example `authorize` and `scope` blocks.
+3. Creates `config/initializers/layered_assistant.rb` with example `authorize` blocks, notes on record ownership with an optional `owner` block, and tool configuration examples.
 4. Mounts the engine at `/layered/assistant` in `config/routes.rb`.
 5. Copies migrations into the host app via the `layered:assistant:migrations` generator.
 
 After installation, configure the initializer (see below) - **routes return 403 until an `authorize` block is set**. Then visit `/layered/assistant` to set up providers, models, personas, skills and assistants.
 
+**Streaming:** replies are generated in an Active Job and broadcast as Turbo Streams over Action Cable. The queue adapter must run jobs in the background (not `:inline`), and if jobs run in a separate worker process (Solid Queue's `bin/jobs`, Sidekiq), `config/cable.yml` must use `solid_cable` or `redis` rather than `async`. The Rails defaults (`:async` jobs, `async` cable) work in a single development process.
+
 ## Authorization
 
 All non-public engine routes are gated by an `authorize` block. The block runs in controller context, so you have access to `current_user`, `request`, `redirect_to`, `head`, `main_app`, etc.
@@ -229,6 +231,7 @@ The engine renders inside `layered-ui-rails` layouts and uses only `l-ui-` class
 - **Provider creation fails with encryption error** - run `bin/rails db:encryption:init` and add the keys to credentials, or set `Layered::Assistant.skip_db_encryption = true` for dev/test.
 - **Panel body never loads** - `turbo-rails` must be installed and `layered-ui-rails` must be mounted in the layout. Check `import "@hotwired/turbo-rails"` is present.
 - **`layered_assistant` JS controllers missing** - ensure `import "layered_assistant"` is in `app/javascript/application.js` (added by the install generator, after the `layered_ui` import).
+- **Replies don't stream, or only appear after a refresh** - the job's broadcasts aren't reaching the browser. Avoid the `:inline` queue adapter, and if jobs run in a separate worker, switch `config/cable.yml` from `async` to `solid_cable` or `redis`.
 - **No records visible, or `MissingOwnerError` on create** - `current_owner` is nil. Check the authorize block only admits signed-in users, and that any `Layered::Assistant.owner` block returns a record.
 
 ## Further reference

data/README.md

@@ -76,6 +76,22 @@ bin/rails generate layered:assistant:migrations
 bin/rails db:migrate
 ```
 
+### Background jobs and Action Cable
+
+Assistant replies are generated in an Active Job and streamed to the browser
+as Turbo Streams over Action Cable. Streaming needs two things from the host
+app:
+
+- **A queue adapter that runs jobs in the background.** With `:inline`, the job
+  runs inside the request that sent the message, so the whole reply arrives at
+  once instead of streaming. The Rails default (`:async`) and Solid Queue both
+  work.
+- **A cable adapter the job process can reach.** The `async` cable adapter only
+  works within one process. It is fine when jobs also run in the web process
+  (the `:async` queue adapter), but if jobs run in a separate worker (Solid
+  Queue's `bin/jobs`, Sidekiq, etc.), their broadcasts never reach the browser.
+  Use `solid_cable` or `redis` in `config/cable.yml` instead.
+
 ## Authorization
 
 All non-public engine routes are **blocked by default** (403 Forbidden) until you configure an `authorize` block. The install generator creates a starter initialiser at `config/initializers/layered_assistant.rb` - uncomment one of the examples to get started.

data/app/controllers/layered/assistant/messages_controller.rb

@@ -1,15 +1,14 @@
 module Layered
   module Assistant
-    class MessagesController < ApplicationController
+    # Messages are a layered resource for listing and deleting them; only
+    # `create` - the composer posting a message - is this engine's own.
+    class MessagesController < ResourcesController
       include MessageCreation
 
-      before_action :set_conversation
-      before_action :set_message, only: [ :destroy ]
-
-      def index
-        @page_title = "Messages"
-        @pagy, @messages = pagy(@conversation.messages.includes(:model).by_created_at)
-      end
+      # Messages have no fields, which the gem reads as having no form, but
+      # `create` is the composer's rather than the gem's.
+      skip_before_action :require_layered_fields, only: [ :create ]
+      before_action :set_conversation, only: [ :create, :destroy ]
 
       def create
         model_id = scoped_model_id(message_params[:model_id])
@@ -37,9 +36,8 @@ module Layered
       end
 
       def destroy
-        @message.destroy
+        super
         @conversation.update_token_totals!
-        redirect_to layered_assistant.conversation_messages_path(@conversation), notice: "Message was successfully deleted."
       end
 
       private
@@ -48,10 +46,6 @@ module Layered
         @conversation = scoped(Conversation).find_by!(uid: params[:conversation_id])
       end
 
-      def set_message
-        @message = @conversation.messages.find(params[:id])
-      end
-
       def message_params
         params.require(:message).permit(:content, :model_id)
       end

data/app/layered_resources/layered/assistant/message_resource.rb

@@ -0,0 +1,44 @@
+module Layered
+  module Assistant
+    # Messages are listed and deleted here; they are written through the
+    # composer, so there are no fields and no form.
+    class MessageResource < Layered::Resource::Base
+      model Message
+
+      columns [
+        { attribute: :role, primary: true },
+        { attribute: :content, sortable: false,
+          render: ->(record, view) { view.truncate(record.content, length: 100) } },
+        { attribute: :model_id, label: "Model", sortable: false,
+          render: ->(record) { record.model&.name } },
+        { attribute: :tokens, label: "Tokens", sortable: false,
+          render: ->(record, view) {
+            total = record.input_tokens.to_i + record.output_tokens.to_i
+            "#{'~' if record.tokens_estimated?}#{view.number_with_delimiter(total)}" if total > 0
+          } },
+        { attribute: :tokens_per_second, label: "Tok/s", sortable: false,
+          render: ->(record) {
+            if record.output_tokens.to_i > 0 && record.response_ms.to_i >= MessagesHelper::MIN_RESPONSE_MS_FOR_TPS
+              (record.output_tokens * 1000.0 / record.response_ms).round(1)
+            end
+          } },
+        { attribute: :ttft_ms, label: "TTFT",
+          render: ->(record) { "#{record.ttft_ms}ms" if record.ttft_ms } },
+        { attribute: :created_at, label: "Created",
+          render: ->(record) { record.created_at.to_fs(:short) } }
+      ]
+
+      search_fields [ :content ]
+
+      default_sort attribute: :created_at, direction: :asc
+
+      # The conversation is resolved through the engine's own scoping, so an
+      # out-of-scope one 404s before any message is reached.
+      def self.scope(controller)
+        controller.send(:scoped, Conversation)
+          .find_by!(uid: controller.params[:conversation_id])
+          .messages.includes(:model)
+      end
+    end
+  end
+end

data/app/models/layered/assistant/conversation.rb

@@ -3,8 +3,10 @@ module Layered
     class Conversation < ApplicationRecord
       include Ownable
 
-      # UID
-      has_secure_token :uid
+      # UID: the public identifier, in the form conversation_<uuid>
+      UID_FORMAT = /\Aconversation_[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}\z/
+
+      attribute :uid, :string, default: -> { "conversation_#{SecureRandom.uuid}" }
 
       # Associations
       belongs_to :assistant, counter_cache: true
@@ -22,6 +24,7 @@ module Layered
 
       # Validations
       validates :name, presence: true
+      validates :uid, format: { with: UID_FORMAT }, uniqueness: true
 
       # Scopes
       scope :by_name, -> { order(name: :asc, created_at: :desc) }

data/config/routes.rb

@@ -16,7 +16,7 @@ Layered::Assistant::Engine.routes.draw do
     end
   end
   resources :conversations, only: [] do
-    resources :messages, only: [ :index, :create, :destroy ]
+    layered_resources :messages, namespace: "Layered::Assistant", controller: "messages", only: [ :index, :create, :destroy ]
     # Approving a tool call is the same act wherever it is rendered, and the
     # partial carrying the buttons is broadcast rather than requested, so it
     # cannot know which namespace it landed in. One route serves them all.

data/db/migrate/20261031000000_prefix_layered_assistant_conversation_uids.rb

@@ -0,0 +1,14 @@
+class PrefixLayeredAssistantConversationUids < ActiveRecord::Migration[8.0]
+  # Conversation uids become conversation_<uuid>. Existing tokens are
+  # replaced, so old conversation URLs stop resolving; the integer primary
+  # key is untouched.
+  def up
+    select_values("SELECT id FROM layered_assistant_conversations").each do |id|
+      execute "UPDATE layered_assistant_conversations SET uid = #{quote("conversation_#{SecureRandom.uuid}")} WHERE id = #{id.to_i}"
+    end
+  end
+
+  def down
+    # Irreversible data change; the prefixed uids remain valid.
+  end
+end

data/lib/layered/assistant/version.rb

@@ -1,5 +1,5 @@
 module Layered
   module Assistant
-    VERSION = "0.9.0"
+    VERSION = "0.10.0"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: layered-assistant-rails
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.10.0
 platform: ruby
 authors:
 - layered.ai
@@ -350,6 +350,7 @@ files:
 - app/jobs/layered/assistant/messages/tool_call_job.rb
 - app/layered_resources/layered/assistant/assistant_resource.rb
 - app/layered_resources/layered/assistant/conversation_resource.rb
+- app/layered_resources/layered/assistant/message_resource.rb
 - app/layered_resources/layered/assistant/model_resource.rb
 - app/layered_resources/layered/assistant/persona_resource.rb
 - app/layered_resources/layered/assistant/provider_resource.rb
@@ -387,7 +388,6 @@ files:
 - app/views/layered/assistant/messages/_message.html.erb
 - app/views/layered/assistant/messages/_tool_message.html.erb
 - app/views/layered/assistant/messages/create.turbo_stream.erb
-- app/views/layered/assistant/messages/index.html.erb
 - app/views/layered/assistant/panel/conversations/_header.html.erb
 - app/views/layered/assistant/panel/conversations/index.html.erb
 - app/views/layered/assistant/panel/conversations/new.html.erb
@@ -427,6 +427,7 @@ files:
 - db/migrate/20260901000000_create_layered_assistant_assistant_tools.rb
 - db/migrate/20260901000001_add_user_to_layered_assistant_conversations.rb
 - db/migrate/20260919000000_add_tool_status_to_layered_assistant_messages.rb
+- db/migrate/20261031000000_prefix_layered_assistant_conversation_uids.rb
 - lib/generators/layered/assistant/install_agent_skill_generator.rb
 - lib/generators/layered/assistant/install_generator.rb
 - lib/generators/layered/assistant/migrations_generator.rb

data/app/views/layered/assistant/messages/index.html.erb

@@ -1,63 +0,0 @@
-<%= l_ui_title_bar(
-  title: "Messages",
-  breadcrumbs: [["Conversations", layered_assistant.conversations_path]]
-) %>
-
-<div class="l-ui-table-container l-ui-mt-4">
-  <table class="l-ui-table">
-    <caption class="l-ui-sr-only">Messages</caption>
-    <thead class="l-ui-table__header">
-      <tr>
-        <th scope="col" class="l-ui-table__header-cell">Role</th>
-        <th scope="col" class="l-ui-table__header-cell">Content</th>
-        <th scope="col" class="l-ui-table__header-cell">Model</th>
-        <th scope="col" class="l-ui-table__header-cell">Tokens</th>
-        <th scope="col" class="l-ui-table__header-cell">Tok/s</th>
-        <th scope="col" class="l-ui-table__header-cell">TTFT</th>
-        <th scope="col" class="l-ui-table__header-cell">Created</th>
-        <th scope="col" class="l-ui-table__header-cell l-ui-table__header-cell--action">Actions</th>
-      </tr>
-    </thead>
-
-    <tbody class="l-ui-table__body">
-      <% @messages.each do |message| %>
-        <tr>
-          <th scope="row" class="l-ui-table__cell l-ui-table__cell--primary">
-            <%= message.role %>
-          </th>
-          <td class="l-ui-table__cell">
-            <%= truncate(message.content, length: 100) %>
-          </td>
-          <td class="l-ui-table__cell">
-            <%= message.model&.name %>
-          </td>
-          <td class="l-ui-table__cell">
-            <% total = message.input_tokens.to_i + message.output_tokens.to_i %>
-            <% if total > 0 %>
-              <% prefix = message.tokens_estimated? ? "~" : "" %>
-              <%= "#{prefix}#{number_with_delimiter(total)}" %>
-            <% end %>
-          </td>
-          <td class="l-ui-table__cell">
-            <% if message.output_tokens.to_i > 0 && message.response_ms.to_i >= Layered::Assistant::MessagesHelper::MIN_RESPONSE_MS_FOR_TPS %>
-              <%= (message.output_tokens * 1000.0 / message.response_ms).round(1) %>
-            <% end %>
-          </td>
-          <td class="l-ui-table__cell">
-            <%= "#{message.ttft_ms}ms" if message.ttft_ms %>
-          </td>
-          <td class="l-ui-table__cell">
-            <%= message.created_at.to_fs(:short) %>
-          </td>
-          <td class="l-ui-table__cell l-ui-table__cell--action">
-            <%= link_to "Delete", layered_assistant.conversation_message_path(@conversation, message), class: "l-ui-table__action l-ui-table__action--danger", data: { turbo_method: :delete, turbo_confirm: "Are you sure?" } %>
-          </td>
-        </tr>
-      <% end %>
-    </tbody>
-  </table>
-
-  <div class="l-ui-mt-4">
-    <%= l_ui_pagy(@pagy) %>
-  </div>
-</div>