ollama_chat 0.0.83 → 0.0.85

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8947d6cdb437ce85afb32cb689bad1b16de54c2afb35da9f0eac7b1f1fff19df
-  data.tar.gz: 3a03448c69d3f8883e11d5de11e9f44c455a4a8e95e2d93593d945dc99a0be63
+  metadata.gz: 4a9e61c3d789441bb11ee79fe2ec63f219cc8d938845a7dc6fe9c8f6d012d5bb
+  data.tar.gz: f453e609568a6df073fbaf865860fa1c919419b728c309ae0e19e97dc09d6b52
 SHA512:
-  metadata.gz: 02e28130adb2881c44ea053035b5b39eb43025bf0e761063e6442a031b8b8acd88fe588850438ff0ae3c9fcf039ca4cff32bcb5abae4bcca4a349852a217580b
-  data.tar.gz: 1055061e111fff2384020505b467b60ab30facc7fbf5733856061adc6834401a5414ba28a6716452b5b5fe2bb8ec5a77d54fc81d2e3ed065f0b9568847ce326c
+  metadata.gz: 24fadb41107daee4f625bb3d80743c8323624da545e6386b07ef25da5be3974afa2b0759a8db86d44eea04c9b156b92400e6a348d6bd9aae3dd71791939533bd
+  data.tar.gz: 7c43765e2da1f09781b6a8b0af70ea562cdec22fbdb4c92ab7f4eca033fdd3b9f79a55cbe3d7c3e91684c771a29cca95c76f26c44b7871cf9e142c3d0ca12179

data/CHANGES.md

@@ -1,5 +1,51 @@
 # Changes
 
+## 2026-03-29 v0.0.85
+
+- Added `list` and `rename` subcommands to the `/collection` command, allowing
+  users to display available collections and rename them via the
+  `rename_collection` method in `lib/ollama_chat/dialog.rb`.
+    - Enhanced the `/collection` command to be optional; running `/collection`
+      without arguments now displays collection statistics by invoking the
+      `collection_stats` method.
+    - Updated `README.md` to reflect new command options and adjusted help text for
+      the `/collection` command.
+    - Modified tests in `spec/ollama_chat/chat_spec.rb` to accommodate new command
+      options and collection handling.
+    - Adjusted `regexp` and `complete` configurations in `lib/ollama_chat/chat.rb`
+      for the `/collection` command.
+    - Updated the `documentrix` dependency from **0.0.4** to **0.1.0**.
+- Introduced progress feedback in `lib/ollama_chat/input_content.rb` within the
+  `provide_file_set_content` method, displaying `count/total` for batch
+  processing or just `count` for interactive mode.
+- Optimized persona loading logic to skip setting a default persona if the
+  conversation already contains messages, while retaining fallbacks from
+  command-line options for new conversations.
+- Fixed the warning condition in `OllamaChat::FollowChat` by correcting the
+  success check in `lib/ollama_chat/follow_chat.rb`.
+
+## 2026-03-26 v0.0.84
+
+- Added `OllamaChat::Utils::ValueFormatter` with helper `format_bytes` in
+  `lib/ollama_chat/utils/value_formatter.rb`.  
+    - Added `require 'ollama_chat/utils/value_formatter'` where needed.  
+    - Updated `write_file`, `patch_file`, `paste_from_clipboard` to use
+      `format_bytes` for byte‑size messages.  
+- Implemented new tool `RetrieveDocumentSnippets` in
+  `lib/ollama_chat/tools/retrieve_document_snippets.rb` and its spec
+  `spec/ollama_chat/tools/retrieve_document_snippets_spec.rb`.  
+    - Updated `default_config.yml` to enable `retrieve_document_snippets` by
+      default.  
+- Rewrote tool payloads to include a `message` key for `browse`, `search_web`,
+  `paste_from_clipboard`, `resolve_tag`, and others.  
+- `browse` now returns `"Opened \"<url>\" in browser."` or a failure message.  
+- Refactored `confirm?` in `lib/ollama_chat/dialog.rb` to accept keyword `yes:`
+  for a confirmation regex and display a coloured emoji matching the response
+  or timeout/default.  
+    - Updated all calls to `confirm?` to use `yes: /y/i` syntax.  
+    - Updated `lib/ollama_chat/message_output.rb` to add `yes:` keyword to
+      `confirm?` calls and refine the emoji shown when a file already exists.  
+
 ## 2026-03-24 v0.0.83
 
 - Added timeout support to `Dialog#confirm?`: introduced optional `timeout:`

data/README.md

@@ -209,8 +209,10 @@ The following commands can be given inside the chat, if prefixed by a `/`:
 ├──────────────────┼────────────────────┼──────────────────┼──────────────────────────────────────────────────────────┤
 │ /load            │                    │ path             │ load conversation messages                               │
 ├──────────────────┼────────────────────┼──────────────────┼──────────────────────────────────────────────────────────┤
-│ /collection      │ change﹡           │                  │ change (default) collection or clear                     │
-│                  │ clear﹡            │                  │                                                          │
+│ /collection      │ change             │                  │ display, clear (current), change, list, or rename        │
+│                  │ clear              │                  │ collection                                               │
+│                  │ list               │                  │                                                          │
+│                  │ rename             │                  │                                                          │
 ├──────────────────┼────────────────────┼──────────────────┼──────────────────────────────────────────────────────────┤
 │ /persona         │ add                │                  │ manage and load/play personae for roleplay               │
 │                  │ delete             │                  │                                                          │

data/Rakefile

@@ -42,7 +42,7 @@ GemHadar do
 
   dependency             'excon',                 '~> 1.0'
   dependency             'ollama-ruby',           '~> 1.21'
-  dependency             'documentrix',           '>= 0.0.4'
+  dependency             'documentrix',           '>= 0.1.0'
   dependency             'unix_socks',            '~> 0.3'
   dependency             'rss',                   '~> 0.3'
   dependency             'term-ansicolor',        '~> 1.11'

data/lib/ollama_chat/chat.rb

@@ -61,6 +61,7 @@ class OllamaChat::Chat
   include OllamaChat::ToolCalling
   include OllamaChat::ConfigHandling
   include OllamaChat::PersonaeManagement
+  include OllamaChat::Utils::ValueFormatter
 
   # Initializes a new OllamaChat::Chat instance with the given command-line
   # arguments.
@@ -478,11 +479,12 @@ class OllamaChat::Chat
 
   command(
     name: :collection,
-    regexp: %r(^/collection(?:\s+(clear|change))?$),
-    complete: [ 'collection', %w[ clear change ] ],
-    help: 'change (default) collection or clear'
+    regexp: %r(^/collection(?:\s+(clear|change|list|rename))?$),
+    complete: [ 'collection', %w[ clear change list rename ] ],
+    optional: true,
+    help: 'display, clear (current), change, list, or rename collection'
   ) do |subcommand|
-    case subcommand || 'change'
+    case subcommand
     when 'clear'
       loop do
         tags = @documents.tags.add('[EXIT]').add('[ALL]')
@@ -492,7 +494,7 @@ class OllamaChat::Chat
           STDOUT.puts "Exiting chooser."
           break
         when '[ALL]'
-          if confirm?(prompt: '🔔 Are you sure? (y/n) ') =~ /y/i
+          if confirm?(prompt: '🔔 Are you sure? (y/n) ', yes: /y/i)
             @documents.clear
             STDOUT.puts "Cleared collection #{bold{@documents.collection}}."
             break
@@ -508,6 +510,14 @@ class OllamaChat::Chat
       end
     when 'change'
       choose_collection(@documents.collection)
+    when 'list'
+      current_collection = @documents.collection
+      puts @documents.collections.
+        map { |c| current_collection == c ? bold { c } : c }
+    when 'rename'
+      rename_collection(@documents.collection)
+    when nil
+      collection_stats
     end
     :next
   end
@@ -537,7 +547,7 @@ class OllamaChat::Chat
       end
     when 'edit'
       if result = edit_persona and
-          confirm?(prompt: '🔔 Load new persona profile? (y/n) ') =~ /y/i
+          confirm?(prompt: '🔔 Load new persona profile? (y/n) ', yes: /y/i)
         then
         result
       else
@@ -868,7 +878,7 @@ class OllamaChat::Chat
           STDOUT.puts "Exiting chooser."
           break
         when '[ALL]'
-          if confirm?(prompt: '🔔 Are you sure? (y/n) ') =~ /y/i
+          if confirm?(prompt: '🔔 Are you sure? (y/n) ', yes: /y/i)
             links.clear
             STDOUT.puts "Cleared all links in list."
             break
@@ -916,7 +926,7 @@ class OllamaChat::Chat
       @documents.clear
       STDOUT.puts "Cleared all tags."
     when 'all'
-      if confirm?(prompt: '🔔 Are you sure to clear messages and collection? (y/n) ') =~ /y/i
+      if confirm?(prompt: '🔔 Are you sure to clear messages and collection? (y/n) ', yes: /y/i)
         messages.clear
         @documents.clear
         links.clear
@@ -1000,27 +1010,6 @@ class OllamaChat::Chat
                         [ content, Documentrix::Utils::Tags.new(valid_tag: /\A#*([\w\]\[]+)/) ]
                       end
 
-      if embedding.on? && content
-        records = @documents.find_where(
-          content.downcase.first(config.embedding.model.context_length),
-          tags:,
-          prompt:     config.embedding.model.prompt?,
-          text_size:  config.embedding.found_texts_size?,
-          text_count: config.embedding.found_texts_count?,
-        )
-        unless records.empty?
-          content << ?\n << JSON(
-            prompt: "Consider these snippets generated from retrieval when formulating your response!",
-            ollama_chat_retrieval_snippets: records.map { |r|
-              {
-                text: r.text,
-                tags: r.tags_set.map { |t| { name: t.to_s(link: false), source: t.source }.compact }
-              }
-            },
-          )
-        end
-      end
-
       runtime_info.on? && content and
         content << ?\n << {
           ollama_chat_runtime_information: runtime_information
@@ -1056,17 +1045,6 @@ class OllamaChat::Chat
           raise
         end
       end
-      if embedding.on? && !records.empty?
-        STDOUT.puts "", records.map { |record|
-          link = if record.source =~ %r(\Ahttps?://)
-                   record.source
-                 else
-                   'file://%s' % File.expand_path(record.source)
-                 end
-          [ link, ?# + record.tags.first ]
-        }.uniq.map { |l, t| hyperlink(l, t) }.join(' ')
-        debug and jj messages.to_ary
-      end
 
       case type
       when :socket_input

data/lib/ollama_chat/config_handling.rb

@@ -58,7 +58,7 @@ module OllamaChat::ConfigHandling
     unless diff_tool = OC::DIFF_TOOL?
       exit 1
     end
-    if confirm?(prompt: '🔔 Do you want to fix the config? (y/n) ') =~ /y/i
+    if confirm?(prompt: '🔔 Do you want to fix the config? (y/n) ', yes: /y/i)
       system Shellwords.join([
         diff_tool,
         @ollama_chat_config.filename,
@@ -84,7 +84,7 @@ module OllamaChat::ConfigHandling
   #    `ollama_chat` if desired.
   def edit_config
     if result = edit_file(@ollama_chat_config.filename)
-      if confirm?(prompt: "🔔 Do you want to restart #{progname}? (y/n) ") =~ /y/i
+      if confirm?(prompt: "🔔 Do you want to restart #{progname}? (y/n) ", yes: /y/i)
         save_conversation(OC::XDG_CACHE_HOME + 'backup.json')
         save_history
         exec($0, *ARGV)
@@ -101,7 +101,7 @@ module OllamaChat::ConfigHandling
   # @example Restarting the app after confirmation
   #   config.reload_config  # => restarts if user answers "y"
   def reload_config
-    if confirm?(prompt: "🔔 Do you want to restart #{progname}? (y/n) ") =~ /y/i
+    if confirm?(prompt: "🔔 Do you want to restart #{progname}? (y/n) ", yes: /y/i)
       save_conversation(OC::XDG_CACHE_HOME + 'backup.json')
       save_history
       exec($0, *ARGV)

data/lib/ollama_chat/conversation.rb

@@ -26,9 +26,12 @@ module OllamaChat::Conversation
   # @example Save conversation with explicit filename
   #   chat.save_conversation('conversations/2023-10-15_my_session.json')
   def save_conversation(filename)
-    File.exist?(filename) &&
-      confirm?(prompt: "🔔 File #{filename.to_s.inspect} already exists, overwrite? (y/n) ") !~ /y/i and
-      return
+    if File.exist?(filename)
+      confirm?(
+        prompt: "🔔 File #{filename.to_s.inspect} already exists, overwrite? (y/n) ",
+        yes: /y/i
+      ) or return
+    end
     if messages.save_conversation(filename)
       STDOUT.puts "Saved conversation to #{filename.to_s.inspect}."
     else

data/lib/ollama_chat/dialog.rb

@@ -36,22 +36,46 @@ module OllamaChat::Dialog
   #
   # @return [Object] the character entered by the user, or the `default` value
   #   if a timeout occurs
-  def confirm?(prompt:, timeout: nil, default: nil)
+  def confirm?(prompt:, timeout: nil, default: nil, yes: nil)
     return default if timeout&.zero?
     if prompt.include?('%s')
       prompt = prompt % (timeout ? ('timeout in %us' % timeout) : 'no timeout')
     end
     print prompt
     system 'stty raw'
+    keypress = nil
     c = if timeout
-          ready = IO.select([ STDIN ], nil, nil, timeout)
-          ready ? STDIN.getc : nil
+          keypress = !!IO.select([ STDIN ], nil, nil, timeout)
+          keypress ? STDIN.getc : nil
         else
+          keypress = true
           STDIN.getc
         end
     system 'stty cooked'
-    puts
-    c || default
+    answer = c || default
+    case
+    when yes.nil?
+      if keypress
+        puts "⌨️ #{answer}"
+      else
+        puts "⌛️ #{answer}"
+      end
+      answer
+    when answer =~ yes
+      if keypress
+        puts "✅ #{answer}"
+      else
+        puts "☑️  #{answer}"
+      end
+      answer
+    else
+      if keypress
+        puts "🚫 #{answer}"
+      else
+        puts "⌛️ #{answer}"
+      end
+      nil
+    end
   end
 
   private
@@ -111,6 +135,27 @@ module OllamaChat::Dialog
     info
   end
 
+  # Rename an existing collection to a new, user‑supplied name.
+  #
+  # This helper prompts the user to provide a new name for the collection
+  # identified by <code>current_collection</code>. It then renames the current
+  # collection to have the new_name and switches to it.
+  #
+  # @param current_collection [Symbol] the current collection name
+  def rename_collection(current_collection)
+    prompt = 'Rename collection %s to: ' % current_collection
+    if new_collection = ask?(prompt:).full?(:to_sym)
+      begin
+        @documents.rename_collection(new_collection)
+        STDOUT.puts "Renamed current collection #{current_collection} to #{new_collection}."
+      rescue
+        STDERR.puts "Renaming to #{new_collection} failed, it already exists."
+      end
+    else
+      STDOUT.puts "Renaming cancelled."
+    end
+  end
+
   # The change_system_prompt method allows the user to select or enter a new
   # system prompt for the chat session.
   # It provides an interactive chooser when multiple prompts match the given

data/lib/ollama_chat/follow_chat.rb

@@ -11,6 +11,7 @@ class OllamaChat::FollowChat
   include Ollama::Handlers::Concern
   include Term::ANSIColor
   include OllamaChat::MessageFormat
+  include OllamaChat::Utils::ValueFormatter
 
   # Initializes a new instance of OllamaChat::FollowChat.
   #
@@ -131,9 +132,10 @@ class OllamaChat::FollowChat
       if @chat.tool_function(name).require_confirmation?
         prompt = "🔔 I want to execute tool %s\n%s\nConfirm? (y/n) " % [
           bold { name },
-          italic { function  },
+          italic { function },
         ]
-        if @chat.confirm?(prompt:) =~ /y/i
+        prompt.gsub!('%', '%%')
+        if @chat.confirm?(prompt:, yes: /y/i)
           confirmed = :explicit
         else
           confirmed = :denied
@@ -153,7 +155,7 @@ class OllamaChat::FollowChat
           resolve: 'You **MUST** ask the user for instructions on how to proceed!!!',
         )
         STDOUT.printf(
-          "\n%s Execution of tool %s denied by user.\n", ?🚫, bold { name }
+          "\n%s Execution of tool %s denied by user.\n\n", ?🚫, bold { name }
         )
         @chat.log(:warn,"Execution of tool %s was denied by user!" % name)
       else
@@ -169,24 +171,43 @@ class OllamaChat::FollowChat
           @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
+
+      data    = nil
+      message = begin
+                   data = JSON.parse(result)
+                   @chat.log(:info, JSON.pretty_generate(data))
+                   data['message']
+                 rescue
+                   @chat.log(:info, result)
+                   nil
+                 end
+      warn =
+        begin
+          !!data.ask_and_send(:[], 'error') || data.ask_and_send(:[], 'success') == false
+        rescue
+          false
+        end
+
       tools_used[name] = {
-        'size'     => Tins::Unit.format(result.to_s.size, unit: ?B, prefix: 1024, format: '%.1f %U'),
-        'duration' => Tins::Duration.new(Time.now - start).to_s,
+        message:,
+        warn:     ,
+        size:     format_bytes(result.to_s.size),
+        duration: Tins::Duration.new(Time.now - start).to_s,
       }
     end
 
     if tools_used.full?
       infobar.reset
       tools_used.each do |name, info|
-        puts "🔧 Tool functions #{name} returned result:",
-          info.to_yaml.sub(/\A---\s*\n/, '').gsub(/^/, '  '), ""
+        feedback_message = if info[:message]
+                             "\n%s %s\n\n" % [ info[:warn] ? '⚠️' : '💡', info[:message] ]
+                           end
+        puts <<~EOT.strip, ""
+          🔧 Tool functions #{name} returned result (#{info[:size]} in #{info[:duration]}).
+          #{feedback_message}
+        EOT
         timeout = @chat.tool_function(name).result_display_timeout?
         @chat.confirm?(prompt: '⏎  Press any key to continue (%s). ', timeout:)
       end
@@ -304,7 +325,7 @@ class OllamaChat::FollowChat
   #
   # This method delegates to the messages object's show_last method, which
   # displays the most recent non-user message in the conversation history.
-  # It is typically used to provide feedback to the user about the last
+  # It is typically used to provide message to the user about the last
   # response from the assistant.
   # @return [ nil, String ] the pager command or nil if no paging was
   #   performed.

data/lib/ollama_chat/input_content.rb

@@ -161,7 +161,16 @@ module OllamaChat::InputContent
   # @yield [filename] the block that processes each filename
   # @return [String] the concatenated result of the block applied to each file
   def provide_file_set_content(patterns, all: false, &block)
+    total = 0
+    all and file_set_each(patterns, all:) { total += 1 }
+    count = 0
     file_set_each(patterns, all:).each_with_object('') do |filename, result|
+      count += 1
+      if all
+        STDOUT.puts "Handling File (#{bold{count}}/#{bold{total}}):"
+      else
+        STDOUT.puts "Handling File (#{bold{count}}):"
+      end
       result << ("%s:\n\n%s\n\n" % [ filename, block.(filename) ])
     end.full?
   end

data/lib/ollama_chat/message_output.rb

@@ -3,7 +3,7 @@
 # This module encapsulates methods for piping assistant responses to command
 # standard input and writing assistant responses to files. It handles the
 # mechanics of sending output to external processes or saving content to disk
-# while providing appropriate error handling and user feedback.
+# while providing appropriate error handling and user message.
 #
 # @example Piping a response to a command
 #   chat.pipe('cat > output.txt')
@@ -83,7 +83,7 @@ module OllamaChat::MessageOutput
   def attempt_to_write_file(filename, message)
     path = Pathname.new(filename.to_s).expand_path
     if !path.exist? ||
-        confirm?(prompt: "🔔 File #{path.to_s.inspect} already exists, overwrite? (y/n) ") =~ /y/i
+        confirm?(prompt: "🔔 File #{path.to_s.inspect} already exists, overwrite? (y/n) ", yes: /y/i)
     then
       File.open(path, ?w) do |output|
         output.write(message.content)

data/lib/ollama_chat/model_handling.rb

@@ -95,7 +95,7 @@ module OllamaChat::ModelHandling
   #                     that combines the model name and formatted size
   private def model_with_size(model)
     formatted_size = Term::ANSIColor.bold {
-      Tins::Unit.format(model.size, unit: ?B, prefix: 1024, format: '%.1f %U')
+      format_bytes(model.size)
     }
     SearchUI::Wrapper.new(model.name, display: "#{model.name} #{formatted_size}")
   end

data/lib/ollama_chat/ollama_chat_config/default_config.yml

@@ -264,3 +264,5 @@ tools:
       require_confirmation: true
       allowed:
         - './tmp'
+    retrieve_document_snippets:
+      default: true

data/lib/ollama_chat/personae_management.rb

@@ -40,7 +40,7 @@ module OllamaChat::PersonaeManagement
   # playing the persona file if it exists.
   def setup_persona_from_opts
     @default_persona and return
-    @opts[?c] and return
+    @messages.size > 0 and return
     if persona = @opts[?p].full? { Pathname.new(_1) }
       if persona.extname == '.md'
         pathname = persona
@@ -173,9 +173,9 @@ module OllamaChat::PersonaeManagement
         STDOUT.puts "Deleting '#{bold{persona.sub_ext('')}}'..."
         STDOUT.puts "Backup will be saved to: #{backup_pathname}"
 
-        if confirm?(prompt: "🔔 Are you sure? (y/n) ") =~ /y/i
+        if confirm?(prompt: "🔔 Are you sure? (y/n) ", yes: /y/i)
           FileUtils.mv pathname, backup_pathname
-          STDOUT.puts "✅ Persona #{bold{persona.sub_ext('')}} deleted successfully"
+          STDOUT.puts "Persona #{bold{persona.sub_ext('')}} deleted successfully"
           {
             success: true,
             persona: persona.sub_ext(''),

data/lib/ollama_chat/server_socket.rb

@@ -117,7 +117,7 @@ module OllamaChat::ServerSocket
       running using the same directory or that a previous process left a stale
       socket file.
     EOT
-    if confirm?(prompt: '🔔 Do you want to remove the existing socket file and continue? (y/n) ') =~ /y/i
+    if confirm?(prompt: '🔔 Do you want to remove the existing socket file and continue? (y/n) ', yes: /y/i)
       FileUtils.rm_f socket_path
       retry
     else

data/lib/ollama_chat/switches.rb

@@ -51,7 +51,7 @@ module OllamaChat::Switches
   #
   # The Switch class provides a simple way to manage boolean configuration
   # options with methods to toggle, set, and query the current state. It
-  # includes messaging capabilities to provide feedback when the state changes.
+  # includes messaging capabilities to provide message when the state changes.
   #
   # @example Creating and using a switch
   #   switch = Switch.new(value: false, msg: { true => 'Enabled', false => 'Disabled' })

data/lib/ollama_chat/tools/browse.rb

@@ -26,9 +26,9 @@ class OllamaChat::Tools::Browse
       function: Tool::Function.new(
         name:,
         description: <<~EOT,
-          Launch external viewer – Opens an HTTP/HTTPS link (or local file) in
-          the system’s web/browser app. Use when you want to hand off a page
-          for human inspection; no return value is expected.
+          Launch external viewer – Opens an HTTP/HTTPS link (or local file
+          path) in the system’s web/browser app. Use when you want to hand off
+          a page for human inspection; no return value is expected.
         EOT
         parameters: Tool::Function::Parameters.new(
           type: 'object',
@@ -58,10 +58,17 @@ class OllamaChat::Tools::Browse
   def execute(tool_call, **opts)
     url    = tool_call.function.arguments.url
     result = browse_url(url)
+
+    message = if result.success?
+                 "Opened #{url.to_s.inspect} in browser."
+               else
+                 "Could not open #{url.to_s.inspect} in browser."
+               end
+
     {
       success:    result.success?,
       exitstatus: result.exitstatus,
-      message:    'opening URL/file',
+      message:   ,
       url:        ,
     }.to_json
   rescue => e

data/lib/ollama_chat/tools/concern.rb

@@ -6,6 +6,7 @@
 # functionality to integrate tools into the chat system's tool registry.
 module OllamaChat::Tools::Concern
   extend Tins::Concern
+  include OllamaChat::Utils::ValueFormatter
 
   included do
     include Ollama

data/lib/ollama_chat/tools/copy_to_clipboard.rb

@@ -56,8 +56,8 @@ class OllamaChat::Tools::CopyToClipboard
                 "The provided text has been successfully copied to the system clipboard."
               end
     {
-      success: true,
-      message:,
+      success:  true,
+      message: ,
     }.to_json
   rescue => e
     {

data/lib/ollama_chat/tools/open_file_in_editor.rb

@@ -73,15 +73,16 @@ class OllamaChat::Tools::OpenFileInEditor
     end
 
     chat.vim.open_file(file_path, start_line, end_line)
-    if end_line
-      result_msg = "Opened #{file_path} and selected range #{start_line}-#{end_line}"
-    else
-      result_msg = "Opened #{file_path} at line #{start_line}"
-    end
+    message =
+      if end_line
+        "Opened #{file_path.to_s.inspect} and selected range #{start_line}-#{end_line}"
+      else
+        "Opened #{file_path.to_s.inspect} at line #{start_line}"
+      end
 
     {
       success:    true,
-      message:    result_msg,
+      message:   ,
       path:       file_path,
       start_line: start_line,
       end_line:   end_line,

data/lib/ollama_chat/tools/paste_from_clipboard.rb

@@ -43,10 +43,11 @@ class OllamaChat::Tools::PasteFromClipboard
 
     # Use the chat instance's clipboard paste functionality
     content = chat.perform_paste_from_clipboard
+    message = "Pasted #{format_bytes(content.to_s.size)} of content."
 
     {
-      success: true,
-      message: "Content pasted from clipboard",
+      success:  true,
+      message: ,
       content:
     }.to_json
   rescue => e

data/lib/ollama_chat/tools/paste_into_editor.rb

@@ -53,7 +53,7 @@ class OllamaChat::Tools::PasteIntoEditor
       end
 
     {
-      success: true,
+      success:  true,
       message: ,
     }.to_json
   rescue => e

data/lib/ollama_chat/tools/patch_file.rb

@@ -68,9 +68,11 @@ class OllamaChat::Tools::PatchFile
     path = assert_valid_path(path, config.tools.functions.patch_file.allowed?, check_file: true)
 
     result = apply_patch(path, diff_content)
-    message = result[:success] ?
-      "Successfully applied patch to #{path.to_s.inspect}." :
-      "Failed to apply patch to file #{path.to_s.inspect}."
+    message = if result[:success]
+                 "Successfully applied patch to #{path.to_s.inspect}."
+               else
+                 "Failed to apply patch to file #{path.to_s.inspect}."
+               end
     (result | {
       path:    path.to_s,
       message: ,