fix: improve Rails generator output and conventions

- Remove duplicate timestamps in migration filenames by letting Rails handle them
- Follow Rails conventions by omitting default class parameters in acts_as declarations
- Replace verbose post-install markdown with concise, colorful terminal output
- Add sponsorship information to encourage community support
- Update tests to match new generator structure

Author: crmne

docs/guides/rails.md

@@ -60,7 +60,6 @@ This approach has one important consequence: **you cannot use `validates :conten
 ## Setting Up Your Rails Application
 
 ### Quick Setup with Generator
-
 {: .d-inline-block }
 
 Available in v1.4.0

lib/generators/ruby_llm/install_generator.rb

@@ -36,8 +36,12 @@ def postgresql?
 
     def acts_as_chat_declaration
       acts_as_chat_params = []
-      acts_as_chat_params << "message_class: \"#{options[:message_model_name]}\"" if options[:message_model_name]
-      acts_as_chat_params << "tool_call_class: \"#{options[:tool_call_model_name]}\"" if options[:tool_call_model_name]
+      if options[:message_model_name] != 'Message'
+        acts_as_chat_params << "message_class: \"#{options[:message_model_name]}\""
+      end
+      if options[:tool_call_model_name] != 'ToolCall'
+        acts_as_chat_params << "tool_call_class: \"#{options[:tool_call_model_name]}\""
+      end
       if acts_as_chat_params.any?
         "acts_as_chat #{acts_as_chat_params.join(', ')}"
       else
@@ -47,8 +51,8 @@ def acts_as_chat_declaration
 
     def acts_as_message_declaration
       acts_as_message_params = []
-      acts_as_message_params << "chat_class: \"#{options[:chat_model_name]}\"" if options[:chat_model_name]
-      if options[:tool_call_model_name]
+      acts_as_message_params << "chat_class: \"#{options[:chat_model_name]}\"" if options[:chat_model_name] != 'Chat'
+      if options[:tool_call_model_name] != 'ToolCall'
         acts_as_message_params << "tool_call_class: \"#{options[:tool_call_model_name]}\""
       end
       if acts_as_message_params.any?
@@ -60,7 +64,9 @@ def acts_as_message_declaration
 
     def acts_as_tool_call_declaration
       acts_as_tool_call_params = []
-      acts_as_tool_call_params << "message_class: \"#{options[:message_model_name]}\"" if options[:message_model_name]
+      if options[:message_model_name] != 'Message'
+        acts_as_tool_call_params << "message_class: \"#{options[:message_model_name]}\""
+      end
       if acts_as_tool_call_params.any?
         "acts_as_tool_call #{acts_as_tool_call_params.join(', ')}"
       else
@@ -71,19 +77,18 @@ def acts_as_tool_call_declaration
     def create_migration_files
       # Create migrations with timestamps to ensure proper order
       # First create chats table
-      timestamp = Time.now.utc
       migration_template 'create_chats_migration.rb.tt',
-                         "db/migrate/#{timestamp.strftime('%Y%m%d%H%M%S')}_create_#{options[:chat_model_name].tableize}.rb" # rubocop:disable Layout/LineLength
+                         "db/migrate/create_#{options[:chat_model_name].tableize}.rb"
 
-      # Then create tool_calls table with timestamp + 1 second
-      timestamp += 1
+      # Then create tool_calls table
+      sleep 1 # Ensure different timestamp
       migration_template 'create_tool_calls_migration.rb.tt',
-                         "db/migrate/#{timestamp.strftime('%Y%m%d%H%M%S')}_create_#{options[:tool_call_model_name].tableize}.rb" # rubocop:disable Layout/LineLength
+                         "db/migrate/create_#{options[:tool_call_model_name].tableize}.rb"
 
-      # Finally create messages table with timestamp + 2 seconds
-      timestamp += 1
+      # Finally create messages table
+      sleep 1 # Ensure different timestamp
       migration_template 'create_messages_migration.rb.tt',
-                         "db/migrate/#{timestamp.strftime('%Y%m%d%H%M%S')}_create_#{options[:message_model_name].tableize}.rb" # rubocop:disable Layout/LineLength
+                         "db/migrate/create_#{options[:message_model_name].tableize}.rb"
     end
 
     def create_model_files
@@ -97,8 +102,19 @@ def create_initializer
     end
 
     def show_install_info
-      content = ERB.new(File.read("#{source_paths.first}/INSTALL_INFO.md.tt")).result(binding)
-      say content
+      say "\n  ✅ RubyLLM installed!", :green
+
+      say "\n  Next steps:", :yellow
+      say '     1. Run: rails db:migrate'
+      say '     2. Set your API keys in config/initializers/ruby_llm.rb'
+      say "     3. Start chatting: #{options[:chat_model_name]}.create!(model_id: 'gpt-4.1-nano').ask('Hello!')"
+
+      say "\n  📚 Full docs: https://rubyllm.com", :cyan
+
+      say "\n  ❤️  Love RubyLLM?", :magenta
+      say '     • Sponsor: https://github.com/sponsors/crmne'
+      say '     • Try Chat with Work: https://chatwithwork.com'
+      say "\n"
     end
   end
 end

spec/lib/generators/ruby_llm/install_generator_spec.rb

@@ -134,35 +134,35 @@
     end
   end
 
-  describe 'INSTALL_INFO template' do
-    let(:install_info_content) { File.read(File.join(template_dir, 'INSTALL_INFO.md.tt')) }
+  describe 'show_install_info method' do
+    let(:generator_content) { File.read(generator_file) }
 
-    it 'has INSTALL_INFO template file' do
-      expect(File.exist?(File.join(template_dir, 'INSTALL_INFO.md.tt'))).to be(true)
+    it 'defines show_install_info method' do
+      expect(generator_content).to include('def show_install_info')
     end
 
     it 'includes welcome message' do
-      expect(install_info_content).to include('RubyLLM Rails Setup Complete')
-    end
-
-    it 'includes setup information' do
-      expect(install_info_content).to include('Run migrations')
+      expect(generator_content).to include('RubyLLM installed!')
     end
 
     it 'includes migration instructions' do
-      expect(install_info_content).to include('rails db:migrate')
+      expect(generator_content).to include('rails db:migrate')
     end
 
     it 'includes API configuration instructions' do
-      expect(install_info_content).to include('Set your API keys')
+      expect(generator_content).to include('Set your API keys')
+    end
+
+    it 'includes usage example with create! and ask' do
+      expect(generator_content).to include('.create!(model_id:').and include('.ask(')
     end
 
-    it 'includes usage examples' do
-      expect(install_info_content).to include('Start using RubyLLM in your code')
+    it 'includes documentation link' do
+      expect(generator_content).to include('https://rubyllm.com')
     end
 
-    it 'includes streaming response information' do
-      expect(install_info_content).to include('For streaming responses')
+    it 'includes sponsorship information' do
+      expect(generator_content).to include('https://github.com/sponsors/crmne').and include('https://chatwithwork.com')
     end
   end