rails_console_ai 0.23.0 → 0.24.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6f97d0243924a105c21ebd4a86049bef4fdda6ed76d738c1c2850244f8cb6e0e
-  data.tar.gz: 930e70d14ae27b0d87999b4625f299390bc928646e73417aef2fc5f7f46aaf5f
+  metadata.gz: 41b94c2b3a01cba1210242c6c8752510fc1976bf9b7195272f4a109dceadcdab
+  data.tar.gz: 6c813abfeeda804b115dfd2a385bc67e1796b0af80cc438e5f41d66886fa519e
 SHA512:
-  metadata.gz: 17d56e01106acbeee8e1299c39e2fbcc3c0f3150922b977652f8857139d8ad7c481c13d080216f492833e22287df616eb4967a6d46162d39b618ac51e74bbeb4
-  data.tar.gz: 259216bdd4cf23d0b6c8634fad2d48d520eac0b4600d16697cdf176e55a3677a5a7975304697be2f36817bf04aaa0f7317459a7e9d79da591cfe49879398cca3
+  metadata.gz: 52281835764610528b039d12ac786ec7056d035e9055d5574bee3a3143c9833e8fa5943a5932980c7ee28ae77351a178186dc15b3afb2f0e9acb7c71b1104c5f
+  data.tar.gz: d1f56cd511ff3411b6e2783c0790bbc3d012a420971fe2cd400f7f573ffc168f5b6bca5c692618bf5c4ef2eaa9d0d0283958afcf1b7ac71d426dd4f2f4377425

data/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 All notable changes to this project will be documented in this file.
 
+## [0.24.0]
+
+- Refactor thinking text display and include in Slack with more technical detail
+- Add `a` command to trigger auto-accept mode
+- Include code output in Slack server logs
+- Fix Bedrock issue after declining code execution
+- Fix `allow_code_execution` configuration
+
 ## [0.23.0]
 
 - Add `save_skill` tool

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

@@ -86,7 +86,7 @@ RailsConsoleAi.configure do |config|
   # config.channels = {
   #   'slack' => {
   #     'allowed_usernames' => ['alice', 'bob'],                          # who can use the bot (or 'ALL')
-  #     'allow_code_execution' => ['alice'],                              # who can run code (nil = everyone)
+  #     'allow_code_execution' => ['alice'],                              # who can run code directly via ``` (nil = everyone)
   #     'pinned_memory_tags' => ['sharding'],
   #     'bypass_guards_for_methods' => ['ChangeApproval#approve_by!']
   #   },

data/lib/rails_console_ai/channel/base.rb

@@ -2,7 +2,8 @@ module RailsConsoleAi
   module Channel
     class Base
       def display(text);            raise NotImplementedError; end
-      def display_dim(text);        raise NotImplementedError; end
+      def display_thinking(text);    raise NotImplementedError; end
+      def display_status(text);      raise NotImplementedError; end
       def display_warning(text);    raise NotImplementedError; end
       def display_error(text);      raise NotImplementedError; end
       def display_code(code);       raise NotImplementedError; end

data/lib/rails_console_ai/channel/console.rb

@@ -14,7 +14,11 @@ module RailsConsoleAi
         $stdout.puts colorize(text, :cyan)
       end
 
-      def display_dim(text)
+      def display_thinking(text)
+        $stdout.puts "\e[2m#{text}\e[0m"
+      end
+
+      def display_status(text)
         $stdout.puts "\e[2m#{text}\e[0m"
       end
 

data/lib/rails_console_ai/channel/slack.rb

@@ -28,23 +28,21 @@ module RailsConsoleAi
         post(strip_ansi(text))
       end
 
-      def display_dim(text)
-        raw = strip_ansi(text)
-        stripped = raw.strip
+      def display_thinking(text)
+        stripped = strip_ansi(text).strip
+        return if stripped.empty?
+        post(stripped)
+      end
+
+      def display_status(text)
+        stripped = strip_ansi(text).strip
+        return if stripped.empty?
 
         if stripped =~ /\AThinking\.\.\.|\AAttempting to fix|\ACancelled|\A_session:/
           post(stripped)
-        elsif stripped =~ /\ACalling LLM/
-          # Technical LLM round status — suppress in Slack
-          @output_log.write("#{stripped}\n")
-          STDOUT.puts "#{@log_prefix} (dim) #{stripped}"
-        elsif raw =~ /\A {2,4}\S/ && stripped.length > 10
-          # LLM thinking text (2-space indent from conversation engine) — show as status
-          post(stripped)
         else
-          # Tool result previews (5+ space indent) and other technical noise — log only
           @output_log.write("#{stripped}\n")
-          STDOUT.puts "#{@log_prefix} (dim) #{stripped}"
+          STDOUT.puts "#{@log_prefix} (status) #{stripped}"
         end
       end
 
@@ -112,9 +110,9 @@ module RailsConsoleAi
 
       def system_instructions
         <<~INSTRUCTIONS.strip
-          ## Response Formatting (Slack Channel)
+          ## Slack Channel
 
-          You are responding to non-technical users in Slack. Follow these rules:
+          You are responding in a Slack thread.
 
           ## Code Execution
           - ALWAYS use the `execute_code` tool to run Ruby code. Do NOT put code in markdown
@@ -132,11 +130,8 @@ module RailsConsoleAi
             123  John Smith        john@example.com
             456  Jane Doe          jane@example.com
             ```
-          - Use `puts` with formatted output instead of returning arrays or hashes
-          - Summarize findings in plain, simple language
-          - Do NOT show technical details like SQL queries, token counts, or class names
-          - Keep explanations simple and jargon-free
-          - Never return raw Ruby objects — always present data in a human-readable way
+          - Use `puts` with formatted output instead of returning arrays or hashes.
+          - Never return raw Ruby objects — always present data in a human-readable way.
           - The output of `puts` in your code is automatically shown to the user. Do NOT
             repeat or re-display data that your code already printed via `puts`.
             Just add a brief summary after (e.g. "10 events found" or "Let me know if you need more detail").

data/lib/rails_console_ai/context_builder.rb

@@ -97,7 +97,8 @@ module RailsConsoleAi
         - Give ONE concise answer. Do not offer multiple alternatives or variations.
         - For multi-step tasks, use execute_plan to break the work into small, clear steps.
         - For simple queries, use the execute_code tool.
-        - Include a brief one-line explanation before or after executing code.
+        - Before calling tools, briefly state what you're about to do (e.g., "Let me check the
+          user's migration status." or "I'll look up the table structure."). Keep it to one sentence.
         - Use the app's actual model names, associations, and schema.
         - Prefer ActiveRecord query interface over raw SQL.
         - For destructive operations, add a comment warning.

data/lib/rails_console_ai/conversation_engine.rb

@@ -49,7 +49,7 @@ module RailsConsoleAi
           conversation << { role: :assistant, content: @_last_result_text }
           conversation << { role: :user, content: error_msg }
 
-          @channel.display_dim("  Ran into an issue, trying a different approach...")
+          @channel.display_status("  Ran into an issue, trying a different approach...")
           exec_result, code, executed = one_shot_round(conversation)
         end
 
@@ -114,7 +114,7 @@ module RailsConsoleAi
 
       status = send_and_execute
       if status == :error
-        @channel.display_dim("  Ran into an issue, trying a different approach...")
+        @channel.display_status("  Ran into an issue, trying a different approach...")
         send_and_execute
       end
     end
@@ -232,7 +232,7 @@ module RailsConsoleAi
         @channel.display_warning("No code to retry.")
         return
       end
-      @channel.display_dim("  Retrying last code...")
+      @channel.display_status("  Retrying last code...")
       execute_direct(code)
     end
 
@@ -790,19 +790,19 @@ module RailsConsoleAi
 
       max_rounds.times do |round|
         if @channel.cancelled?
-          @channel.display_dim("  Cancelled.")
+          @channel.display_status("  Cancelled.")
           break
         end
 
         if round == 0
-          @channel.display_dim("  Thinking...")
+          @channel.display_status("  Thinking...")
         else
           if last_thinking
             last_thinking.split("\n").each do |line|
-              @channel.display_dim("  #{line}")
+              @channel.display_thinking("  #{line}")
             end
           end
-          @channel.display_dim("  #{llm_status(round, messages, total_input, last_thinking, last_tool_names)}")
+          @channel.display_status("  #{llm_status(round, messages, total_input, last_thinking, last_tool_names)}")
         end
 
         # Trim large tool outputs between rounds to prevent context explosion.
@@ -856,7 +856,7 @@ module RailsConsoleAi
                           else
                             "No matching outputs found with id(s) #{ids.join(', ')}."
                           end
-            @channel.display_dim("     #{tool_result}")
+            @channel.display_status("     #{tool_result}")
             tool_msg = provider.format_tool_result(tc[:id], tool_result)
             messages << tool_msg
             new_messages << tool_msg
@@ -865,7 +865,7 @@ module RailsConsoleAi
 
           # Display any pending LLM text before executing the tool
           if last_thinking
-            last_thinking.split("\n").each { |line| @channel.display_dim("  #{line}") }
+            last_thinking.split("\n").each { |line| @channel.display_thinking("  #{line}") }
             last_thinking = nil
           end
 
@@ -879,7 +879,7 @@ module RailsConsoleAi
 
             preview = compact_tool_result(tc[:name], tool_result)
             cached_tag = tools.last_cached? ? " (cached)" : ""
-            @channel.display_dim("     #{preview}#{cached_tag}")
+            @channel.display_status("     #{preview}#{cached_tag}")
           end
 
           if RailsConsoleAi.configuration.debug
@@ -908,10 +908,10 @@ module RailsConsoleAi
           tool_call_counts[key] += 1
 
           if tool_call_counts[key] >= LOOP_BREAK_THRESHOLD
-            @channel.display_dim("  Loop detected: #{tc[:name]} called #{tool_call_counts[key]} times with same args — stopping.")
+            @channel.display_status("  Loop detected: #{tc[:name]} called #{tool_call_counts[key]} times with same args — stopping.")
             exhausted = true
           elsif tool_call_counts[key] >= LOOP_WARN_THRESHOLD
-            @channel.display_dim("  Warning: #{tc[:name]} called #{tool_call_counts[key]} times with same args — consider a different approach.")
+            @channel.display_status("  Warning: #{tc[:name]} called #{tool_call_counts[key]} times with same args — consider a different approach.")
             messages << { role: :user, content: "You are repeating the same tool call (#{tc[:name]}) with the same arguments. This is not making progress. Try a different approach or provide your answer now." }
           end
         end
@@ -1095,6 +1095,12 @@ module RailsConsoleAi
         else
           truncate(result, 80)
         end
+      when 'execute_code'
+        lines = result.split("\n").reject { |l| l.strip.empty? }
+        output_lines = lines.select { |l| !l.start_with?('Output:') && !l.start_with?('Return value:') }
+        summary = output_lines.first(2).map { |l| l.strip }.join('; ')
+        summary = truncate(summary, 70) if summary.length > 70
+        "#{output_lines.length} lines: #{summary}"
       when 'execute_plan'
         steps_done = result.scan(/^Step \d+/).length
         steps_done > 0 ? "#{steps_done} steps executed" : truncate(result, 80)

data/lib/rails_console_ai/executor.rb

@@ -216,7 +216,20 @@ module RailsConsoleAi
 
       loop do
         case answer
-        when 'y', 'yes', 'a'
+        when 'a', 'auto'
+          RailsConsoleAi.configuration.auto_execute = true
+          if @channel
+            @channel.display_status("Auto-execute: ON")
+          else
+            $stdout.puts colorize("Auto-execute: ON", :cyan)
+          end
+          result = execute(code)
+          if @last_safety_error
+            return nil unless danger_allowed?
+            return offer_danger_retry(code)
+          end
+          return result
+        when 'y', 'yes'
           result = execute(code)
           if @last_safety_error
             return nil unless danger_allowed?
@@ -355,9 +368,9 @@ module RailsConsoleAi
     def execute_prompt
       guards = RailsConsoleAi.configuration.safety_guards
       if !guards.empty? && guards.enabled? && danger_allowed?
-        "Execute? [y/N/danger] "
+        "Execute? [y/N/a/danger] "
       else
-        "Execute? [y/N] "
+        "Execute? [y/N/a] "
       end
     end
 

data/lib/rails_console_ai/providers/bedrock.rb

@@ -121,10 +121,10 @@ module RailsConsoleAi
                     else
                       [{ text: msg[:content].to_s }]
                     end
-          # Bedrock rejects empty text blocks in content arrays
-          content.reject! { |block| block.is_a?(Hash) && block.key?(:text) && !block.key?(:tool_use) && !block.key?(:tool_result) && block[:text].to_s.empty? }
+          # Bedrock rejects empty or whitespace-only text blocks in content arrays
+          content.reject! { |block| block.is_a?(Hash) && block.key?(:text) && !block.key?(:tool_use) && !block.key?(:tool_result) && block[:text].to_s.strip.empty? }
           # Bedrock also rejects messages with completely empty content arrays
-          content << { text: ' ' } if content.empty?
+          content << { text: '.' } if content.empty?
           { role: msg[:role].to_s, content: content }
         end
 

data/lib/rails_console_ai/slack_bot.rb

@@ -443,10 +443,10 @@ module RailsConsoleAi
         Thread.current.report_on_exception = false
         Thread.current[:log_prefix] = "[#{channel_id}/#{thread_ts}]#{channel_log_tag(channel_id)} @#{user_name}"
         begin
-          channel.display_dim("_session: #{channel_id}/#{thread_ts}_")
+          channel.display_status("_session: #{channel_id}/#{thread_ts}_")
           if restored
             puts "Restored session for thread #{thread_ts} (#{engine.history.length} messages)"
-            channel.display_dim("_(session restored — continuing from previous conversation)_")
+            channel.display_status("_(session restored — continuing from previous conversation)_")
           end
           engine.process_message(text)
         rescue => e

data/lib/rails_console_ai/tools/registry.rb

@@ -368,13 +368,6 @@ module RailsConsoleAi
       def register_execute_plan
         return unless @executor
 
-        # Check per-channel code execution permission
-        if @channel
-          unless RailsConsoleAi.configuration.username_allowed?(@channel.mode, 'allow_code_execution', @channel.user_identity)
-            return
-          end
-        end
-
         register(
           name: 'execute_code',
           description: 'Execute Ruby code in the Rails console and return the result. Use this for all code execution — simple queries, data lookups, reports, etc. The output of puts/print statements is automatically shown to the user. The return value is sent back to you so you can summarize the findings.',
@@ -419,12 +412,8 @@ module RailsConsoleAi
         # Show the code to the user
         @executor.display_code_block(code)
 
-        # Slack: execute directly, suppress display (output goes back to LLM as tool result).
-        # Console: show code and confirm before executing, display output directly.
-        exec_result = if @channel&.mode == 'slack'
-                        @executor.execute(code, display: false)
-                      elsif RailsConsoleAi.configuration.auto_execute
-                        @executor.execute(code, display: false)
+        exec_result = if @channel&.mode == 'slack' || RailsConsoleAi.configuration.auto_execute
+                        @executor.execute(code)
                       else
                         @executor.confirm_and_execute(code)
                       end

data/lib/rails_console_ai/version.rb

@@ -1,3 +1,3 @@
 module RailsConsoleAi
-  VERSION = '0.23.0'.freeze
+  VERSION = '0.24.0'.freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails_console_ai
 version: !ruby/object:Gem::Version
-  version: 0.23.0
+  version: 0.24.0
 platform: ruby
 authors:
 - Cortfr