fix handling activestorage attachments

Author: manuelmeurer

lib/ruby_llm/active_record/acts_as_legacy.rb

@@ -302,7 +302,9 @@ def prepare_for_active_storage(attachments)
           case attachment
           when ActionDispatch::Http::UploadedFile, ActiveStorage::Blob
             attachment
-          when ActiveStorage::Attached::One, ActiveStorage::Attached::Many
+          when ActiveStorage::Attached::One
+            attachment.blob
+          when ActiveStorage::Attached::Many
             attachment.blobs
           when Hash
             attachment.values.map { |v| prepare_for_active_storage(v) }

lib/ruby_llm/active_record/chat_methods.rb

@@ -315,7 +315,9 @@ def prepare_for_active_storage(attachments)
           case attachment
           when ActionDispatch::Http::UploadedFile, ActiveStorage::Blob
             attachment
-          when ActiveStorage::Attached::One, ActiveStorage::Attached::Many
+          when ActiveStorage::Attached::One
+            attachment.blob
+          when ActiveStorage::Attached::Many
             attachment.blobs
           when Hash
             attachment.values.map { |v| prepare_for_active_storage(v) }

spec/dummy/app/models/document.rb

@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+
+class Document < ApplicationRecord
+  has_one_attached :file
+  has_many_attached :files
+end

spec/dummy/db/migrate/20251002152808_create_documents.rb

@@ -0,0 +1,10 @@
+# frozen_string_literal: true
+
+class CreateDocuments < ActiveRecord::Migration[7.0]
+  def change
+    create_table :documents do |t|
+      t.string :title
+      t.timestamps
+    end
+  end
+end

spec/dummy/db/schema.rb

@@ -74,6 +74,12 @@
     t.index ['tool_call_id'], name: 'index_document_messages_on_tool_call_id'
   end
 
+  create_table 'documents', force: :cascade do |t|
+    t.string 'title'
+    t.datetime 'created_at', null: false
+    t.datetime 'updated_at', null: false
+  end
+
   create_table 'messages', force: :cascade do |t|
     t.integer 'chat_id'
     t.string 'role'

spec/fixtures/vcr_cassettes/activerecord_actsas_attachment_handling_handles_attachments_in_ask_method.yml renamed to spec/fixtures/vcr_cassettes/activerecord_actsas_attachment_handling_handles_a_single_attachment_in_ask_method.yml

@@ -47,30 +47,47 @@ def uploaded_file(path, type)
       expect(llm_message.content.attachments.first.mime_type).to eq('image/png')
     end
 
-    it 'handles multiple attachments' do
+    it 'handles ActiveStorage::Attached::One in ask method' do
       chat = Chat.create!(model: model)
 
-      image_upload = uploaded_file(image_path, 'image/png')
-      pdf_upload = uploaded_file(pdf_path, 'application/pdf')
-
-      response = chat.ask('Analyze these', with: [image_upload, pdf_upload])
-
-      user_message = chat.messages.find_by(role: 'user')
-      expect(user_message.attachments.count).to eq(2)
-      expect(response.content).to be_present
-    end
-
-    it 'handles attachments in ask method' do
-      chat = Chat.create!(model: model)
-
-      image_upload = uploaded_file(image_path, 'image/png')
+      document = Document.create!(title: 'Test Document')
+      document.file.attach(
+        io: File.open(image_path),
+        filename: 'ruby.png',
+        content_type: 'image/png'
+      )
 
-      response = chat.ask('What do you see?', with: image_upload)
+      response = chat.ask('What do you see?', with: document.file)
 
       user_message = chat.messages.find_by(role: 'user')
       expect(user_message.attachments.count).to eq(1)
+      expect(user_message.attachments.first.filename.to_s).to eq('ruby.png')
       expect(response.content).to be_present
     end
+
+    # it 'handles ActiveStorage::Attached::Many in ask method' do
+    #   chat = Chat.create!(model: model)
+
+    #   document = Document.create!(title: 'Test Document')
+    #   document.files.attach(
+    #     io: File.open(image_path),
+    #     filename: 'ruby.png',
+    #     content_type: 'image/png'
+    #   )
+    #   document.files.attach(
+    #     io: File.open(pdf_path),
+    #     filename: 'sample.pdf',
+    #     content_type: 'application/pdf'
+    #   )
+
+    #   response = chat.ask('Analyze these', with: document.files)
+
+    #   user_message = chat.messages.find_by(role: 'user')
+    #   expect(user_message.attachments.count).to eq(2)
+    #   filenames = user_message.attachments.map { |a| a.filename.to_s }.sort
+    #   expect(filenames).to eq(['ruby.png', 'sample.pdf'])
+    #   expect(response.content).to be_present
+    # end
   end
 
   describe 'attachment types' do