ollama_chat 0.0.77 → 0.0.79

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 368f0f36682a4570b76fead6a43e797a8c450f983921dfab46ca7d6d4cb5759b
-  data.tar.gz: aadd0d681c682ce97a670b48d706f0dfc71a3a65ed6cd4390c3e663892db1e8a
+  metadata.gz: 982588e8a14a4bb35aebd365a9dea99fb7c7b474f4d1293436fa97d1258eec79
+  data.tar.gz: 0a0410940246c4d9d260256e02373f002aeadba778a2097aefc97f190dc17e68
 SHA512:
-  metadata.gz: e9b72a1b63e771e0574b7028a205936f2c3cf2213fb9f85c54f4d4c5d11a5dad99d0ae6461f16b656f8b36f13a42c7237b953a1be305176c0b12766881d087e4
-  data.tar.gz: 1986df4ed7e098d6561674e7d8406bc1db73b55884b4eedbd411c33f5777009ae88864828905c55f4c057b44895fef4e8d5552e89ac2c83bc4a6c0a5ffdce255
+  metadata.gz: f15c28d9980519078af9b27d5706edfa91df2c4c13ac235377dd97623c62a40c9b8014c06373f093ef473e5c9553e20a949fa4488afb7be06b94ebd006c2eb1c
+  data.tar.gz: 8bc4b5bd4880ab46ea227379ff4786681a7f7bef67d81df35628f5aea238c250f0842de5408fdb94056828c792080ce0ee0793a3a74e13d8a78c6d504b005f9d

data/CHANGES.md

@@ -1,5 +1,78 @@
 # Changes
 
+## 2026-03-13 v0.0.79
+
+- Added `tmp/*` to Rakefile ignore list for cleaner builds.  
+- Updated gemspec: changed `s.rubygems_version` to **3.6.9**.  
+- Added `logging.rb` to `extra_rdoc_files` and `files` in gemspec.  
+- Adjusted gemspec file list to reflect new structure.  
+- Removed `Runtime Directory` (`-d`) explanation from `README.md`.  
+- Updated URL in `README.md`.  
+- Added explicit check that `args.path.full?` is true before calling
+  `assert_valid_path`, raising `ArgumentError` with message `'require path to
+  file to be patched'`.  
+- Updated path assignment to first validate presence, then assert validity
+  against `config.tools.functions.patch_file.allowed?`.  
+- Updated `OllamaChat::Tools::ExecuteGrep` to accept new options `before`,
+  `after`, and `context`.  
+- Added helper method `normalize_number` for optional integer arguments.  
+- Adjusted command template in `default_config.yml` to include `-B`, `-A`, and
+  `-C` flags based on provided values.  
+- Refactored logging: replaced `logger.error` / `logger.warn` calls with
+  `log(:error, …)` or `log(:warn, …)` across multiple files.  
+- Added optional `warn: true` flag to new helper so that critical errors also
+  trigger a user‑visible warning output while still being written to the log
+  file.  
+- Implemented `OllamaChat::Logging#log(severity, msg, warn:)` in `logging.rb`;
+  it formats exceptions with backtraces and forwards messages to the underlying
+  Ruby `Logger`.  
+- Updated `spec_helper.rb` by adding a global `config.before` hook that sets
+  temporary paths for `OC::OLLAMA::CHAT::HISTORY` and `LOGFILE` so all specs
+  run against isolated files.  
+- Added `tmp/*` to `.gitignore`.  
+- Created new module `OllamaChat::Logging` that lazily builds a Logger writing
+  to the file defined by `OC::OLLAMA::CHAT::LOGFILE` in XDG STATE.  
+- Required this logging module in `lib/ollama_chat.rb` and included it in
+  `class OllamaChat::Chat` so every chat instance has access to `logger`.  
+- Logged connection messages via `@chat.logger.info` before printing the
+  “Connecting …” line in `OllamaChat::Dialog#connect_message`.  
+- Centralized tool‑call error handling: log unconfigured, unregistered or
+  disabled tools with `@chat.logger.error`, and record each executed function
+  payload using `JSON.pretty_generate`.  
+- Introduced an explicit vs implicit confirmation flow (`:explicit`,
+  `:implicite`, `:denied`) in `OllamaChat::FollowChat#follow_chat` and log the
+  outcome accordingly.  
+- Switched chat history storage from XDG CACHE to XDG STATE by updating
+  `OC::OLLAMA::CHAT::HISTORY` path, adjusting file operations in
+  `lib/ollama_chat/history.rb`, and adding a new config entry for `LOGFILE`
+  under state home in `oc.rb`.  
+- Enhanced PatchFile tool: updated documentation to specify `unified diff
+  format` and `JSON response`.  
+- Added private method `digest` that returns `MD5` of a file path.  
+- Modified `apply_patch` to compute old digest, run patch command, compare
+  digests, set `success` flag accordingly.  
+- Improved error handling: now includes `success`, clearer messages, empty
+  result on failure.
+
+## 2026-03-11 v0.0.78
+
+- Added new `patch_file` tool for applying unified diff patches to files with
+  safety checks, including specs and integration into the tool system.
+- Updated `DirectoryStructure` tool description to clarify that `max_depth`
+  limits the tree height for better LLM understanding.
+- Bumped `search_ui` gem dependency from **0.0** to **0.1**, refactored model
+  handling using `SearchUI::Wrapper.new` and updated chooser logic for safer
+  method dispatch.
+- Enhanced `run_tests` tool with a path validation whitelist in
+  `default_config.yml`, introducing `OllamaChat::Utils::PathValidator` and
+  updating `check_path` method to validate against the allowed list.
+- Simplified runtime information handling by adding new helper methods:
+  `runtime_information_values`, `runtime_information`, and updated `chat.rb` to
+  use these for displaying structured runtime data.
+- Added default persona name display using the `default_persona_name` helper,
+  updating information output and refactoring `reload_default_persona` guard
+  logic.
+
 ## 2026-03-09 v0.0.77
 
 - Add runtime Git and time placeholders: added `git_current_branch` and

data/README.md

@@ -252,14 +252,6 @@ The `ollama_chat_send` command now supports additional parameters to enhance fun
   $ echo "Hello world" | ollama_chat_send -d /tmp/my_working_dir -r
   ```
 
-- **Runtime Directory (`-d`)**: Specifies the directory where the Unix socket
-  file of `ollama_chat` was created, if you want to send to a specific
-  `ollama_chat`.
-
-  ```bash
-  $ echo "Hello world" | ollama_chat_send -d /tmp/my_runtime_dir -r
-  ```
-
 - **Help (`-h` or `--help`)**: Displays usage information and available options.
 
   ```bash
@@ -273,7 +265,7 @@ These parameters provide greater flexibility in how you interact with
 
 The homepage of this app is located at
 
-* https://github.com/flori/ollama\_chat
+* https://github.com/flori/ollama_chat
 
 ## Author
 

data/Rakefile

@@ -22,7 +22,7 @@ GemHadar do
   test_dir    'spec'
   ignore      '.*.sw[pon]', 'pkg', 'Gemfile.lock', '.AppleDouble', '.bundle',
     '.yardoc', 'doc', 'tags', 'corpus', 'coverage', '/config/searxng/*',
-    '.starscope.db', 'cscope.out', 'errors.lst'
+    '.starscope.db', 'cscope.out', 'errors.lst', 'tmp/*'
   package_ignore '.all_images.yml', '.tool-versions', '.gitignore', 'VERSION',
     '.rspec', '.github', '.contexts', '.envrc', '.yardopts'
 
@@ -52,7 +52,7 @@ GemHadar do
   dependency             'kramdown-ansi',         '~> 0.3'
   dependency             'complex_config',        '~> 0.22', '>= 0.22.2'
   dependency             'tins',                  '~> 1.52'
-  dependency             'search_ui',             '~> 0.0'
+  dependency             'search_ui',             '~> 0.1'
   dependency             'amatch',                '~> 0.4'
   dependency             'pdf-reader',            '~> 2.0'
   dependency             'bigdecimal',            '~> 3.1'

data/lib/ollama_chat/chat.rb

@@ -33,6 +33,7 @@ require 'context_spook'
 class OllamaChat::Chat
   include Tins::GO
   include Term::ANSIColor
+  include OllamaChat::Logging
   include OllamaChat::DocumentCache
   include OllamaChat::Switches
   include OllamaChat::StateSelectors
@@ -193,7 +194,7 @@ class OllamaChat::Chat
   end
 
   # The disable_content_parsing method turns off content parsing by setting
-  # @parse_content to false.
+  # `@parse_content` to false.
   #
   # This prevents automatic parsing of user input content during chat
   # processing.
@@ -264,7 +265,8 @@ class OllamaChat::Chat
       begin
         use_model
       rescue OllamaChatError::UnknownModelError => e
-        STDERR.puts "Caught #{e.class}: #{e}"
+        msg = "Caught #{e.class}: #{e}"
+        log(:error, msg, warn: true)
       end
       :next
     when %r(^/system(?:\s+(show))?$)
@@ -707,23 +709,7 @@ class OllamaChat::Chat
         end
       end
 
-      if runtime_info.on? && content
-        runtime_info_values = {
-          languages:            config.languages * ', ',
-          location:             location.on? ? location_description.inspect : 'n/a',
-          git_current_branch:   `git rev-parse --abbrev-ref HEAD 2>/dev/null`.chomp.full? || 'n/a',
-          git_remote_origin:    `git remote get-url origin 2>/dev/null`.chomp.full? || 'n/a',
-          client:               ,
-          current_directory:    Pathname.pwd.expand_path.to_path.inspect,
-          terminal_rows:        Tins::Terminal.rows,
-          terminal_cols:        Tins::Terminal.cols,
-          time:                 Time.now.iso8601,
-          voice:                voice.on? ? 'enabled' : 'disabled',
-          markdown:             markdown.on? ? 'enabled' : 'disabled',
-          tool_paths_allowed:   JSON(tool_paths_allowed),
-        }
-        content << config.prompts.runtime_info % runtime_info_values
-      end
+      content << runtime_information if runtime_info.on? && content
 
       messages << Ollama::Message.new(role: 'user', content:, images: @images.dup)
       @images.clear
@@ -777,7 +763,8 @@ class OllamaChat::Chat
         server_socket_message&.disconnect
       end
     rescue Ollama::Errors::TimeoutError
-      STDOUT.puts "#{bold('Error')}: Currently lost connection to ollama server and cannot send command."
+      msg = "Currently lost connection to ollama server and cannot send command."
+      log(:warn, msg, warn: true)
     rescue Interrupt
       STDOUT.puts "Type /quit to quit."
     ensure
@@ -785,6 +772,7 @@ class OllamaChat::Chat
     end
     0
   rescue ComplexConfig::AttributeMissing, ComplexConfig::ConfigurationSyntaxError => e
+    log(:error, e)
     fix_config(e)
   ensure
     save_history
@@ -821,6 +809,7 @@ class OllamaChat::Chat
     if server_version.version < '0.9.0'.version
       raise 'require ollama API version 0.9.0 or higher'
     end
+    log(:info, "Connection to #{base_url} established.")
     @ollama
   end
 

data/lib/ollama_chat/clipboard.rb

@@ -98,7 +98,6 @@ module OllamaChat::Clipboard
   # handles any OllamaChat::OllamaChatError exceptions by printing the error
   # message to standard error and does not re-raise the exception.
   def copy_to_clipboard
-
     perform_copy_to_clipboard
     STDOUT.puts "The last response has been successfully copied to the system clipboard."
     true

data/lib/ollama_chat/dialog.rb

@@ -30,7 +30,9 @@ module OllamaChat::Dialog
   # @param model [String] the model name to connect to
   # @param base_url [String] the base URL of the connection
   def connect_message(model, base_url)
-    STDOUT.puts green { "Connecting to #{model}@#{base_url} now…" }
+    msg = "Connecting to #{model}@#{base_url} now…"
+    log(:info, msg)
+    STDOUT.puts green { msg }
   end
 
   # The choose_collection method presents a menu to select or create a document

data/lib/ollama_chat/document_cache.rb

@@ -43,7 +43,7 @@ module OllamaChat::DocumentCache
       document_cache_class
     end
   rescue => e
-    STDERR.puts "Caught #{e.class}: #{e} => Falling back to MemoryCache."
+    log(:error, "Caught #{e.class}: #{e} => Falling back to MemoryCache.")
     Documentrix::Documents::MemoryCache
   end
 end

data/lib/ollama_chat/follow_chat.rb

@@ -99,43 +99,46 @@ class OllamaChat::FollowChat
     response.message.tool_calls.each do |tool_call|
       name = tool_call.function.name
       unless @chat.tool_configured?(name)
-        @chat.tool_call_results[name] =
-          "Error: Unconfigured tool named %s ignored => Skip.\n" % name
+        msg = "Error: Unconfigured tool named %s ignored => Skip.\n" % name
+        @chat.tool_call_results[name] = msg
+        @chat.log(:error, msg)
         next
       end
       unless @chat.tool_registered?(name)
-        @chat.tool_call_results[name] =
-          "Error: Unregistered tool named %s ignored => Skip.\n" % name
+        msg = "Error: Unregistered tool named %s ignored => Skip.\n" % name
+        @chat.tool_call_results[name] = msg
+        @chat.log(:error, msg)
         next
       end
       unless @chat.tool_enabled?(name)
-        @chat.tool_call_results[name] =
-          "Error: Disabled tool named %s ignored => Skip.\n" % name
+        msg = "Error: Disabled tool named %s ignored => Skip.\n" % name
+        @chat.tool_call_results[name] = msg
+        @chat.log(:error, msg)
         next
       end
       STDOUT.puts
-      confirmed = true
-      args = JSON.pretty_generate(tool_call.function.arguments)
+      confirmed = :implicit
+      function = JSON.pretty_generate(tool_call.function)
+      @chat.log(:info, function)
       if @chat.tool_function(name).require_confirmation?
         prompt = "I want to execute tool %s\n%s\nConfirm? (y/n) " % [
           bold { name },
-          italic { args  },
+          italic { function  },
         ]
-        confirmed = @chat.ask?(prompt:) =~ /\Ay/i
+        if @chat.ask?(prompt:) =~ /\Ay/i
+          confirmed = :explicit
+        else
+          confirmed = :denied
+        end
       else
         STDOUT.puts "Executing tool %s\n%s" % [
           bold { name },
-          italic { args  },
+          italic { function },
         ]
       end
       result = nil
-      if confirmed
-        STDOUT.printf(
-          "\n%s Execution of tool %s confirmed.\n\n", ?✅, bold { name }
-        )
-        result = OllamaChat::Tools.registered[name].
-          execute(tool_call, chat: @chat, config: @chat.config)
-      else
+      case confirmed
+      when :denied
         result = JSON(
           message: 'User denied confirmation!',
           resolve: 'You **MUST** ask the user for instructions on how to proceed!!!',
@@ -143,7 +146,25 @@ class OllamaChat::FollowChat
         STDOUT.printf(
           "\n%s Execution of tool %s denied by user.\n", ?🚫, bold { name }
         )
-        sleep 1
+        @chat.log(:warn,"Execution of tool %s was denied by user!" % name)
+      else
+        symbol = confirmed == :implicit ? '☑️ ' : '✅'
+        STDOUT.printf(
+          "\n%s Execution of tool %s confirmed.\n\n", symbol, bold { name }
+        )
+        result = OllamaChat::Tools.registered[name].
+          execute(tool_call, chat: @chat, config: @chat.config)
+        if confirmed == :explicit
+          @chat.log(:info, "Execution of tool %s was explicitly confirmed." % name)
+        else
+          @chat.log(:info, "Execution of tool %s was implicitly confirmed." % name)
+        end
+      end
+      begin
+        data = JSON.parse(result)
+        @chat.log(:info, JSON.pretty_generate(data))
+      rescue
+        @chat.log(:info, result)
       end
       @chat.tool_call_results[name] = result
     end

data/lib/ollama_chat/history.rb

@@ -17,20 +17,6 @@
 module OllamaChat::History
   private
 
-  # The chat_history_filename method constructs and returns the full file path
-  # for the chat history file.
-  #
-  # This method takes the configured chat history filename from the
-  # configuration and expands it to an absolute path using File.expand_path.
-  # This ensures that the returned path is fully qualified and can be used
-  # reliably for reading from or writing to the chat history file.
-  #
-  # @return [String] the absolute file path to the chat history file as
-  #   specified in the configuration
-  def chat_history_filename
-    File.expand_path(OC::OLLAMA::CHAT::HISTORY)
-  end
-
   # The init_chat_history method initializes the chat session by loading
   # previously saved command history from a file.
   #
@@ -40,15 +26,15 @@ module OllamaChat::History
   # loading process are caught and logged as warnings, but do not interrupt the
   # execution flow.
   def init_chat_history
-    if File.exist?(chat_history_filename)
-      File.open(chat_history_filename, ?r) do |history|
-        history_data = JSON.load(history)
-        Readline::HISTORY.clear
-        Readline::HISTORY.push(*history_data)
-      end
+    if OC::OLLAMA::CHAT::HISTORY.exist?
+      history = OC::OLLAMA::CHAT::HISTORY.read
+      history_data = JSON.load(history)
+      Readline::HISTORY.clear
+      Readline::HISTORY.push(*history_data)
     end
   rescue => e
-    warn "Caught #{e.class} while loading #{chat_history_filename.inspect}: #{e}"
+    msg = "Caught #{e.class} while loading #{OC::OLLAMA::CHAT::HISTORY.inspect}: #{e}"
+    log(:error, msg, warn: true)
   end
 
   # The save_history method persists the current command history to a file.
@@ -57,9 +43,10 @@ module OllamaChat::History
   # writes it to the chat history filename. It handles potential errors during
   # the write operation by catching exceptions and issuing a warning message.
   def save_history
-    File.secure_write(chat_history_filename, JSON.dump(Readline::HISTORY))
+    File.secure_write(OC::OLLAMA::CHAT::HISTORY, JSON.dump(Readline::HISTORY))
   rescue => e
-    warn "Caught #{e.class} while saving #{chat_history_filename.inspect}: #{e}"
+    msg = "Caught #{e.class} while saving #{OC::OLLAMA::CHAT::HISTORY.inspect}: #{e}"
+    log(:error, msg, warn: true)
   end
 
   # The clear_history method clears the Readline history array and ensures that

data/lib/ollama_chat/information.rb

@@ -112,13 +112,20 @@ module OllamaChat::Information
     think_mode.show
     think_loud.show
     location.show
-    runtime_info.show
     voice.show
     @voice.on? and @voices.show
+    if runtime_info.on?
+      STDOUT.puts "Runtime Information:"
+      STDOUT.puts runtime_information_values.transform_keys(&:to_s).to_yaml.
+        sub(/\A---\s*\n/, '').gsub(/^/, '  ')
+    else
+      runtime_info.show
+    end
     tools_support.show
     STDOUT.puts "Documents database cache is #{@documents.nil? ? 'n/a' : bold{@documents.cache.class}}"
     STDOUT.puts "Document policy for references in user text: #{bold{document_policy}}"
     STDOUT.puts "Currently selected search engine is #{bold(search_engine)}."
+    name = default_persona_name and STDOUT.puts "Default persona: #{bold{name}}"
     STDOUT.puts "Conversation length: #{bold(@messages.size.to_s)} message(s)."
     nil
   end
@@ -231,4 +238,40 @@ module OllamaChat::Information
   def server_url
     @server_url ||= ollama.base_url
   end
+
+  # The runtime_information_values method compiles a set of key runtime details
+  # such as languages, location description, default persona name, current git
+  # branch and remote origin, client agent string, current directory, terminal
+  # dimensions, timestamp, voice and markdown status, and allowed tool paths.
+  #
+  # @return [Hash] a hash containing runtime information values.
+  def runtime_information_values
+    {
+      languages:            config.languages * ', ',
+      time:                 Time.now.iso8601,
+      location:             location.on?.full? { location_description } || 'n/a',
+      default_persona:      default_persona_name.full? || 'n/a',
+      git_current_branch:   `git rev-parse --abbrev-ref HEAD 2>/dev/null`.chomp.full? || 'n/a',
+      git_remote_origin:    `git remote get-url origin 2>/dev/null`.chomp.full? || 'n/a',
+      client:               ,
+      current_directory:    Pathname.pwd.expand_path.to_path,
+      terminal_rows:        Tins::Terminal.rows,
+      terminal_cols:        Tins::Terminal.cols,
+      voice:                voice.on? ? 'enabled' : 'disabled',
+      markdown:             markdown.on? ? 'enabled' : 'disabled',
+      tool_paths_allowed:   JSON.pretty_generate(tool_paths_allowed),
+    }
+  end
+
+  # The runtime_information method generates a formatted string containing
+  # various runtime values such as languages, location description, git branch
+  # and origin, client agent, current directory, terminal size, time, voice and
+  # markdown status, and allowed tool paths.
+  # It returns the result of interpolating these values into the configured
+  # runtime_info prompt template.
+  #
+  # @return [String] the formatted runtime information string.
+  def runtime_information
+    config.prompts.runtime_info % runtime_information_values
+  end
 end

data/lib/ollama_chat/input_content.rb

@@ -37,7 +37,7 @@ module OllamaChat::InputContent
   # searches for files matching the given pattern, excludes already chosen
   # files, and presents them in an interactive chooser menu.
   #
-  # @param pattern [ Array<String> ] the glob pattern to search for files
+  # @param patterns [ Array<String> ] the glob patterns to search for files
   # @param chosen [ Set ] a set of already chosen filenames to exclude from
   #   selection
   #

data/lib/ollama_chat/logging.rb

@@ -0,0 +1,29 @@
+# Provides a simple logging facility for the OllamaChat application. It offers
+# methods to access a logger, determine the current log level, and write
+# messages at various severity levels, including debug.
+module OllamaChat::Logging
+  # The logger method returns the current Logger instance used by OllamaChat.
+  # If no Logger exists, it creates one pointing to the configured log file.
+  #
+  # @return [Logger] the active Logger instance
+  def logger
+    @logger and return @logger
+    OC::OLLAMA::CHAT::LOGFILE.dirname.mkpath
+    @logger = Logger.new(OC::OLLAMA::CHAT::LOGFILE)
+  end
+
+  # The log method records a message or exception at the specified severity
+  # level using the logger and optionally triggers a warning output
+  #
+  # @param severity [ Symbol ] the logging level to use
+  # @param msg [ String, Exception ] the message or exception to be logged
+  # @param warn [ TrueClass, FalseClass ] whether to also trigger a warning output
+  def log(severity, msg, warn: false)
+    if msg.is_a?(Exception)
+      msg = "Caught #{msg.class}: #{msg}\n#{Array(msg&.backtrace).join(?\n)}"
+    end
+    logger.send(severity, msg)
+    warn and self.warn(msg)
+    nil
+  end
+end

data/lib/ollama_chat/model_handling.rb

@@ -94,14 +94,10 @@ module OllamaChat::ModelHandling
   # @return [ Object ] a result object with an overridden to_s method
   #                     that combines the model name and formatted size
   private def model_with_size(model)
-    result = model.name
     formatted_size = Term::ANSIColor.bold {
       Tins::Unit.format(model.size, unit: ?B, prefix: 1024, format: '%.1f %U')
     }
-    result.singleton_class.class_eval do
-      define_method(:to_s) { "%s %s" % [ model.name, formatted_size ] }
-    end
-    result
+    SearchUI::Wrapper.new(model.name, display: "#{model.name} #{formatted_size}")
   end
 
   # The use_model method selects and sets the model to be used for the chat
@@ -151,12 +147,12 @@ module OllamaChat::ModelHandling
                  Regexp.new($1)
                end
     models = ollama.tags.models.sort_by(&:name).map { |m| model_with_size(m) }
-    selector and models = models.grep(selector)
+    selector and models = models.select { _1.value =~ selector }
     model =
       if models.size == 1
         models.first
       elsif cli_model == ''
-        OllamaChat::Utils::Chooser.choose(models) || current_model
+        OllamaChat::Utils::Chooser.choose(models)&.value || current_model
       else
         cli_model || current_model
       end

data/lib/ollama_chat/oc.rb

@@ -17,17 +17,23 @@ module OC
   prefix ''
 
   XDG_CONFIG_HOME = set do
-    description 'XDG Configuration directory path'
+    description 'XDG Configuration home directory path'
     default { '~/.config' }
     decode  { Pathname.new(_1).join('ollama_chat').expand_path }
   end
 
   XDG_CACHE_HOME = set do
-    description 'XDG Cache directory path'
+    description 'XDG Cache home directory path'
     default { '~/.cache' }
     decode  { Pathname.new(_1).join('ollama_chat').expand_path }
   end
 
+  XDG_STATE_HOME = set do
+    description 'XDG State home directory path'
+    default { '~/.local' }
+    decode  { Pathname.new(_1).join('state', 'ollama_chat').expand_path }
+  end
+
   PAGER = set do
     description 'Pager command to use in case terminal lines are exceeded by output'
 
@@ -133,14 +139,19 @@ module OC
 
       HISTORY = set do
         description 'File to save the chat history in'
-        default     XDG_CACHE_HOME + 'history.json'
+        default     XDG_STATE_HOME + 'history.json'
+      end
+
+      LOGFILE = set do
+        description 'File to output log messages to'
+        default     XDG_STATE_HOME + 'chat.log'
       end
 
       module TOOLS
         description 'Tool specific configuration settings'
 
         # Run Tests tool configuration
-        RUN_TESTS_TEST_RUNNER = set do
+        TEST_RUNNER = set do
           description 'Configured test runner for run_tests tool function'
           default     'rspec'
           required     true
@@ -162,6 +173,13 @@ module OC
           decode       { Pathname.new(_1).expand_path }
         end
 
+        PATCH_TOOL = set do
+          description 'Patch tool to use'
+
+          default { `which patch`.full?(:chomp) }
+          check   { value.blank? || File.exist?(value) }
+        end
+
         module JIRA
           description 'Jira tool configuration'
 

data/lib/ollama_chat/ollama_chat_config/default_config.yml

@@ -11,7 +11,7 @@ timeouts:
 languages:
   - en-US
 location:
-  enabled: false
+  enabled: true
   name: Berlin
   decimal_degrees: [ 52.514127, 13.475211 ]
   units: SI (International System of Units) # or USCS (United States Customary System)
@@ -44,10 +44,10 @@ prompts:
     There is usually no reason to mention this information to the user unless
     asked about it.
 
-    - Languages the user prefers: %{languages}.
-    - Current time is %{time}.
-    - Location is %{location}.
-    - Current directory is %{current_directory}.
+    - Languages the user prefers: %{languages}
+    - Current time is %{time}
+    - Location: %{location}
+    - Current directory is "%{current_directory}"
     - **Git**:
       - Current branch is %{git_current_branch}
       - Current remote origin is %{git_remote_origin}
@@ -185,7 +185,7 @@ tools:
     execute_grep:
       default: true
       cmd: |
-        grep #{'-i' if ignore_case} -m #{max_results} -r #{pattern} #{path}
+        grep #{'-i' if ignore_case} -m #{max_results} -r #{before.full? { "-B %u " % it }}#{after.full? { "-A %u " % it }}#{context.full? { "-C %u " % it }}#{pattern} #{path}
     browse:
       default: true
     write_file:
@@ -218,6 +218,10 @@ tools:
     run_tests:
       require_confirmation: true
       default: true
+      allowed:
+        - "./spec"
+        - "./test"
+        - "./tests"
     get_jira_issue:
       default: false
     get_rfc:
@@ -236,3 +240,8 @@ tools:
       default: true
     resolve_tag:
       default: true
+    patch_file:
+      default: true
+      require_confirmation: true
+      allowed:
+        - './tmp'

data/lib/ollama_chat/personae_management.rb

@@ -55,11 +55,22 @@ module OllamaChat::PersonaeManagement
     @default_persona ||= :none
   end
 
+  # The default_persona_name method returns the name of the currently set
+  # default persona by extracting its basename and removing the file extension,
+  # unless no persona is set.
+  #
+  # @return [ String ] the default persona name or nil if none.
+  def default_persona_name
+    if @default_persona.present? && @default_persona != :none
+      @default_persona.basename.sub_ext('').to_s
+    end
+  end
+
   # Reloads the default persona file if one is set and not :none, prompting the
   # user for confirmation before playing the persona file.
   def reload_default_persona
-    !@default_persona || @default_persona == :none and return
-    prompt = "Reload default persona #{@default_persona.basename.sub_ext('')}? (y/n) "
+    name = default_persona_name or return
+    prompt = "Reload default persona #{name}? (y/n) "
     if ask?(prompt:) =~ /\Ay/i
       play_persona_file @default_persona
     end