brute_cli 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 209050c59aa0893b1c8eb87a31a17af8c162857f6ed45d87bca6ed3e575d15c1
-  data.tar.gz: e35271f8f8bfa9fae282d46a12e561406a16ffa3aacdd3ab214c626c7f3f91a0
+  metadata.gz: d25022bbf25ee324c11c0517e9186a65e13b4838d33dde2dc4250b9803783631
+  data.tar.gz: 1831cd59d1263dd3099854dc58cc5ecfb73f1448ac180cac73271ab836541fb6
 SHA512:
-  metadata.gz: d3d9daa8c885f9f117fa0e1d9a36a14c3e603927be0058f382fdc2551fbe99f6143a0f81d3316884ba5a227e0a716db479240ce50b2c429861c967d0356a3b54
-  data.tar.gz: 9dd409dc9e0bfea51c99a29720b00d5727b8e8e189bffb33ab5d92c574283a8b72461e445f7c52df27d8737cd3f31c4666e87055a605b60ed69891131dd1eda4
+  metadata.gz: c03d4a80b353b12ab4e28cdddd4aed2c4a3645cbbb8f9d96d9636d6e889873f509ec3ca8dc813013ebbed1f8d38bdb4c6f41cc554998a0fcbfbd27f167e6e5eb
+  data.tar.gz: e3dc1f1741c4bbb83a406164034788564ce6d5a1c45a286702b2a36470997789ed0a41632d02c467b12ac88dd4f279b4f613b6bddeb4a9aecd49b3de58b540a0

data/lib/brute_cli/repl.rb

@@ -11,7 +11,16 @@ require "brute_cli/question_screen"
 
 module BruteCLI
   class REPL
-    AGENTS = %w[build plan].freeze
+    AGENTS = %w[build plan bash ruby python nix].freeze
+
+    # Shell-mode agents: agent name → shell interpreter (model name).
+    # These agents use the Shell provider instead of the current LLM provider.
+    SHELL_AGENTS = {
+      "bash"   => "bash",
+      "ruby"   => "ruby",
+      "python" => "python",
+      "nix"    => "nix",
+    }.freeze
 
     def initialize(options = {})
       @options = options
@@ -20,10 +29,12 @@ module BruteCLI
       @session = nil
       @selected_model = nil    # user-chosen model override (nil = provider default)
       @models_cache = nil      # cached model list from provider API
+      @saved_provider = nil    # stashed LLM provider when in shell-mode agent
       @width = TTY::Screen.width
       @content_buf = +""
       @streamer = StreamFormatter.new(width: @width)
       @spinner = nil
+      @last_output = nil  # :separator, :content, or :tool — used to deduplicate separators
       @mu = Mutex.new
     end
 
@@ -75,23 +86,38 @@ module BruteCLI
         end
       }
 
-      # Rebind Tab (^I = byte 9) to cycle agents when the buffer is empty,
-      # otherwise fall through to normal completion. We define a custom method
-      # on the singleton LineEditor instance and point the emacs keymap at it.
+      # Force Reline's config to load now (reads inputrc, registers ANSI
+      # default key bindings).  Without this, the defaults are lazily
+      # applied on the first readmultiline call and overwrite our overrides.
+      unless Reline.core.config.loaded?
+        Reline.core.config.read
+        Reline::IOGate.set_default_key_bindings(Reline.core.config)
+      end
+
+      # Rebind Tab (^I = byte 9) to cycle agents forward when the buffer is
+      # empty, otherwise fall through to normal completion.
+      # Shift+Tab (ESC [ Z = bytes 27,91,90) cycles agents backward.
       repl = self
       Reline.line_editor.define_singleton_method(:cycle_or_complete) do |key|
         if current_line.empty?
-          repl.send(:cycle_agent)
-          # Reline caches prompt_list based on (whole_lines, mode_string).
-          # Since neither changed, the cache returns the stale prompt.
-          # Clear it so the next rerender re-evaluates prompt_proc.
+          repl.send(:cycle_agent, :forward)
           @cache.delete(:prompt_list)
           @cache.delete(:wrapped_prompt_and_input_lines)
         else
           complete(key)
         end
       end
+
+      Reline.line_editor.define_singleton_method(:reverse_cycle_agent) do |_key|
+        if current_line.empty?
+          repl.send(:cycle_agent, :backward)
+          @cache.delete(:prompt_list)
+          @cache.delete(:wrapped_prompt_and_input_lines)
+        end
+      end
+
       Reline.core.config.add_default_key_binding_by_keymap(:emacs, [9], :cycle_or_complete)
+      Reline.core.config.add_default_key_binding_by_keymap(:emacs, [27, 91, 90], :reverse_cycle_agent)
     end
 
     # Reline completion callback.
@@ -139,9 +165,14 @@ module BruteCLI
     # ── Provider ──
 
     def resolve_provider_info
-      provider = Brute.provider rescue nil
-      @provider_name = provider&.name&.to_s
-      @model_name = @selected_model || provider&.default_model&.to_s
+      if (shell_model = SHELL_AGENTS[@current_agent])
+        @provider_name = "shell"
+        @model_name = shell_model
+      else
+        provider = Brute.provider rescue nil
+        @provider_name = provider&.name&.to_s
+        @model_name = @selected_model || provider&.default_model&.to_s
+      end
     end
 
     def model_short
@@ -170,6 +201,14 @@ module BruteCLI
       return if @agent
 
       ensure_session!
+
+      # Shell-mode agents swap the provider to Shell with the right interpreter.
+      if (shell_model = SHELL_AGENTS[@current_agent])
+        activate_shell_agent!(shell_model)
+      else
+        restore_llm_provider!
+      end
+
       @agent = Brute.agent(
         cwd: @options[:cwd] || Dir.pwd,
         model: @selected_model,
@@ -185,6 +224,26 @@ module BruteCLI
       @session.restore(@agent.context) if @options[:session_id]
     end
 
+    # Swap the global provider to Shell with the given interpreter model.
+    # Saves the current LLM provider so it can be restored later.
+    def activate_shell_agent!(shell_model)
+      current = Brute.provider
+      unless current.is_a?(Brute::Providers::Shell)
+        @saved_provider = current
+      end
+      Brute.provider = Brute::Providers::Shell.new
+      @selected_model = shell_model
+    end
+
+    # Restore the saved LLM provider when leaving a shell-mode agent.
+    def restore_llm_provider!
+      if @saved_provider
+        Brute.provider = @saved_provider
+        @saved_provider = nil
+        @selected_model = nil
+      end
+    end
+
     # Force the agent to be recreated on next ensure_agent! call.
     # Used after changing provider, model, or agent.
     def reset_agent!
@@ -196,10 +255,25 @@ module BruteCLI
       "%"
     end
 
-    def cycle_agent
-      idx = (AGENTS.index(@current_agent) + 1) % AGENTS.size
+    def cycle_agent(direction = :forward)
+      step = direction == :backward ? -1 : 1
+      idx = (AGENTS.index(@current_agent) + step) % AGENTS.size
       @current_agent = AGENTS[idx]
       reset_agent!
+
+      # Pre-resolve provider info for the status line.
+      # Shell agents show "shell" provider + interpreter model;
+      # LLM agents show the current LLM provider + model.
+      if (shell_model = SHELL_AGENTS[@current_agent])
+        @provider_name = "shell"
+        @model_name = shell_model
+      else
+        # Peek at what the LLM provider will be (saved or current).
+        provider = @saved_provider || (Brute.provider rescue nil)
+        @provider_name = provider&.name&.to_s
+        @model_name = @selected_model || provider&.default_model&.to_s
+      end
+
       # Rewrite the model/status line sitting one line above the prompt.
       # Save cursor, move up, clear line, print, restore cursor.
       parts = []
@@ -314,8 +388,8 @@ module BruteCLI
           resolve_provider_info
           puts separator
           puts "Provider changed to: #{provider_name.colorize(ACCENT)}"
-          puts "Model: #{@model_name.colorize(ACCENT)}"
-          puts separator
+          puts "Select a model:".colorize(DIM)
+          cmd_model
         else
           puts "Failed to initialize provider: #{provider_name}".colorize(ERROR_FG)
         end
@@ -392,6 +466,7 @@ module BruteCLI
     def execute(prompt)
       @content_buf = +""
       @streamer.reset
+      @last_output = nil
 
       start_spinner("Thinking...")
 
@@ -442,7 +517,8 @@ module BruteCLI
 
     def start_spinner(label)
       stop_spinner
-      puts separator
+      puts separator unless @last_output == :separator
+      @last_output = :separator
       @spinner = TTY::Spinner.new(
         ":spinner #{label}",
         frames: nyan_frames,
@@ -469,6 +545,7 @@ module BruteCLI
         stop_spinner
         @content_buf << text
         @streamer << text
+        @last_output = :content
       end
     end
 
@@ -502,8 +579,9 @@ module BruteCLI
         stop_spinner
         tool = @pending_tool || { name: name, args: {} }
 
-        puts separator
+        puts separator unless @last_output == :separator
         print_tool_result(tool, result)
+        @last_output = :tool
 
         @pending_tool = nil
         start_spinner("Thinking...")
@@ -530,6 +608,7 @@ module BruteCLI
       unless @content_buf.strip.empty?
         @streamer.flush
         @content_buf = +""
+        @last_output = :content
       end
     end
 
@@ -638,8 +717,7 @@ module BruteCLI
       parts << stat_span("out", (tokens[:total_output] || 0).to_s)
       parts << stat_span("time", format_time(timing[:total_elapsed] || 0))
       parts << stat_span("tools", tool_calls.to_s) if tool_calls > 0
-      puts
-      puts separator
+      puts separator unless @last_output == :separator
       puts parts.join(sep)
       puts thick_separator
     end
@@ -660,7 +738,7 @@ module BruteCLI
 
     def print_banner
       puts separator
-      puts BruteCLI::LOGO.chomp.colorize(ACCENT)
+      puts BruteCLI::LOGO.chomp.colorize(DIM)
       puts separator
       puts "Version #{Brute::VERSION}".colorize(DIM)
       if @session
@@ -707,11 +785,11 @@ module BruteCLI
     end
 
     def separator
-      ("─" * [@width, 40].max).colorize(ACCENT)
+      ("─" * [@width, 40].max).colorize(DIM)
     end
 
     def thick_separator
-      ("═" * [@width, 40].max).colorize(ACCENT)
+      ("═" * [@width, 40].max).colorize(DIM)
     end
 
     def detect_width

data/lib/brute_cli/styles.rb

@@ -22,7 +22,7 @@ module BruteCLI
 
   # Named color/style constants for use with "string".colorize(CONST).
   # Compound styles (bold + color, background + foreground) use the hash form.
-  DIM         = :light_black
+  DIM         = :grey
   ACCENT      = COLOR
   ACCENT_BOLD = { color: COLOR, mode: :bold }
   ACCENT_BG   = { color: :black, background: COLOR, mode: :bold }

data/lib/brute_cli/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module BruteCli
-  VERSION = "0.2.0"
+  VERSION = "0.3.0"
 end

data/lib/brute_cli.rb

@@ -28,7 +28,8 @@ module BruteCLI
  888   888  888      888   888    888 . 888    .o 
  `Y8bod8P' d888b     `V88V"V8P'   "888" `Y8bod8P' 
   LOGO
-                                                  
+
+  # yolo, bruv...
 
   def self.error(message)
     $stderr.puts "#{"ERROR".colorize(ERROR_BG)} #{message.colorize(ERROR_FG)}"

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: brute_cli
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Brute Contributors
 bindir: exe
 cert_chain: []
-date: 1980-01-01 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: brute
@@ -15,14 +15,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.1.9
+        version: 0.2.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.1.9
+        version: 0.2.0
 - !ruby/object:Gem::Dependency
   name: brute_flow
   requirement: !ruby/object:Gem::Requirement