ollama_chat 0.0.136 → 0.0.138

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e22822d0b02f1f0dad3fe7e9b2902fc727a6dc2d13ffcd192da3fda3980fdefc
-  data.tar.gz: 2bb58b895a48bceefb16600def20f1ea8bb16deeeabd4a307f7fea27749a9139
+  metadata.gz: 170c9e2778ed0d581bda082eebc43d07a6c6e6a6cd99a0d1e0d3634a5281ca9f
+  data.tar.gz: e57e2eb4d990f10617e1574ae1f70d22741dd814cf945575cc2cb7fb882dd4be
 SHA512:
-  metadata.gz: 41e74251f4318137993e985f97beba5626b9d200b460d827ce04bddfb6dc316df6dae2d78024dd1c7d4486f3b7f4f7febd4e38d666fdf1e5f00f0e3e246b7c70
-  data.tar.gz: 1cd96acf7bca275337aaaad85c41d65c2bd75101ccfa1f0924d72eda254f4fffe04c019c9c2ee12285df6ea2e91da96cf702753059c0a7e7d7aa4bde8c13b0f4
+  metadata.gz: b5e6781664a38632da9d7a4f11900864b86b9e0049fcb66d6084777d0f8b4a85e762a57217c23817c77f0fb365592d5d659e3eab874aa270fa97b52a3e727c45
+  data.tar.gz: 6aa806be34d04830f4788dd452d6e8a078e848767978d6bf49d6fef39f3dbd92b3f61fd4dc203c9914491c1ab2d8fa432ff75bc3543bf385dffcda2333acbd83

data/CHANGES.md

@@ -1,5 +1,103 @@
 # Changes
 
+## 2026-10-28 v0.0.138
+
+### Features
+
+- Added `OllamaChat::Utils::Redact` module with `redact_password` to mask
+  credentials in URLs for user-facing display.
+- `memory_restore` now deduplicates entries by checking existing tags before
+  adding records, reporting the number of skipped duplicates.
+- `OllamaChat::Utils::AnalyzeDirectory` now returns a `Hash` with `:height` and
+  `:entries` keys instead of a bare `Array`. Entry hashes no longer include
+  `:type` or `:name`; directories are identified by a trailing `/` in `:path`.
+
+### Fixes
+
+- `use_model` now raises `OllamaChat::UnknownModelError` if the previous model
+  is missing or blank after a cancelled pull, preventing stale metadata from
+  being returned.
+- Fixed `update_collection` to use `Set` for the `seen` variable, resolving a
+  latent `NoMethodError` when upstream yields a `Set`.
+- Corrected the error message in `pull_model_unless_present` to reference the
+  `model` parameter instead of the potentially stale `@model` instance
+  variable.
+
+### Improvements
+
+- Applied password redaction to all user-visible URL displays, including
+  `Chat#inspect`, `Dialog#connect_message`, `Information#info_rag`,
+  `Information#print_welcome`, `Tools::Browse#execute`,
+  `Tools::GenerateImage#execute`, and `Fetcher.get`.
+- Standardized `Set` insertion to use the `<<` idiom across `conversation.rb`,
+  `http_handling.rb`, `rag_handling.rb`, `generate_image.rb`, and
+  `web_searching.rb`.
+- Added `<<` alias to `LinksSet` to support the standard `Set` insertion idiom.
+- `AnalyzeDirectory` now omits the `:children` key from directory entries when
+  the array is empty, both during build and pruning.
+
+### Build & CI
+
+- Added `lib/ollama_chat/utils/redact.rb` to the gemspec file list.
+
+### Testing
+
+- Updated `analyze_directory_spec.rb`, `parsing_spec.rb`, and
+  `list_directory_spec.rb` to assert the new `AnalyzeDirectory` return
+  structure.
+- Added specs for `redact_password` covering URI/String inputs, passthrough for
+  URLs without passwords, and non-mutation of the original object.
+- Added specs for `memory_restore` deduplication logic, covering partial and
+  full skip scenarios.
+- Updated `web_searching_spec.rb` to expect `<<` instead of `add` for link
+  insertion.
+
+### Documentation
+
+- Updated `execute_grep` tool description to explicitly state that the regex
+  runs against file contents.
+- Updated YARD documentation for `AnalyzeDirectory` to reflect the new return
+  value structure.
+
+## 2026-10-23 v0.0.137
+
+### Fixes
+
+- Fixed a double-scheme bug in `OC::URL` default generation by delegating to
+  `Ollama.base_url`, ensuring correct handling of `HOST` values that already
+  include `http://` or `https://`.
+- Prevented deletion of the `default` collection in `delete_collection`,
+  returning a warning feedback instead.
+- Reset `session.current_collection` to `'default'` when the active collection
+  is deleted.
+
+### Improvements
+
+- Ensured the `default` collection row always exists in the database by
+  unioning it with `chat.documents.collections` during `Collection.sync`.
+- Removed four vestigial `refresh_system_prompt` calls from
+  `create_collection`, `edit_collection`, `rename_collection`, and
+  `delete_collection`, as collection data no longer interpolates into the
+  system prompt.
+
+### Build & CI
+
+- Updated `ollama-ruby` dependency from **1.25** to **1.26**.
+- Updated `kramdown-ansi` dependency from **~> 0.6** to **~> 0.7**.
+- Updated `gem_hadar` development dependency from **2.20.1** to **2.21.0**.
+
+### Testing
+
+- Added specs for `delete_collection` to verify refusal to delete the `default`
+  collection, resetting `current_collection` when the active collection is
+  deleted, and leaving `current_collection` unchanged for non-active
+  collections.
+
+### Documentation
+
+- Updated YARD documentation for `refresh_system_prompt` to accurately reflect
+  the placeholders used (`persona`, `persona_name`, `runtime_info`).
+
 ## 2026-10-21 v0.0.136
 
 ### Features

data/Rakefile

@@ -39,7 +39,7 @@ GemHadar do
   )
 
   dependency             'excon',                 '~> 1.0'
-  dependency             'ollama-ruby',           '~> 1.25'
+  dependency             'ollama-ruby',           '~> 1.26'
   dependency             'documentrix',           '>= 0.8.0'
   dependency             'unix_socks',            '>= 0.4'
   dependency             'rss',                   '~> 0.3'
@@ -47,7 +47,7 @@ GemHadar do
   dependency             'redis',                 '~> 5.0'
   dependency             'mime-types',            '~> 3.0'
   dependency             'reverse_markdown',      '~> 3.0'
-  dependency             'kramdown-ansi',         '~> 0.6'
+  dependency             'kramdown-ansi',         '~> 0.7'
   dependency             'complex_config',        '~> 0.22', '>= 0.22.2'
   dependency             'tins',                  '~> 1.61'
   dependency             'search_ui',             '>= 0.2'

data/lib/ollama_chat/chat.rb

@@ -344,7 +344,7 @@ class OllamaChat::Chat
   #
   # @return [String] the inspected string
   def inspect
-    "#<#{self.class}: #{base_url.to_s}>"
+    "#<#{self.class}: #{redact_password(base_url)}>"
   end
 
   # Returns the tool call processor instance.

data/lib/ollama_chat/conversation.rb

@@ -86,7 +86,7 @@ module OllamaChat::Conversation
           selected.merge(remaining)
           break
         else
-          selected.add(choice)
+          selected << choice
         end
       end
     end

data/lib/ollama_chat/database/models/collection.rb

@@ -51,7 +51,8 @@ class OllamaChat::Database::Models::Collection < Sequel::Model(OllamaChat::DB)
   #
   # @param chat [OllamaChat::Chat] the chat instance providing the configuration
   def self.sync(chat)
-    chat.documents.collections.each do |name|
+    all_collections = (%w[ default ] + chat.documents.collections).to_set
+    all_collections.each do |name|
       where(name: name.to_s).first and next
       create(
         name:        name.to_s,

data/lib/ollama_chat/dialog.rb

@@ -142,7 +142,7 @@ module OllamaChat::Dialog
   #   distinguish the connection type, e.g. `"embedding "` for a dedicated
   #   embedding host (default: `""`)
   def connect_message(base_url:, prefix: '')
-    feedback green { "Connecting to #{prefix}ollama #{base_url.to_s.inspect}…" }, type: :info
+    feedback green { "Connecting to #{prefix}ollama #{redact_password(base_url).inspect}…" }, type: :info
   end
 
   # Displays a success-level feedback message confirming that a

data/lib/ollama_chat/http_handling.rb

@@ -61,7 +61,7 @@ module OllamaChat::HTTPHandling
   # @yield [IO] yields a temporary file handle for the caller to consume.
   # @return [Object] the result of the block or the fetcher.
   def get_url(url, headers: nil, cache: nil, remember: true, &block)
-    remember and links.add(url.to_s)
+    remember and links << url.to_s
     headers = config.request_headers?.to_h | headers
     OllamaChat::Utils::Fetcher.get(
       url,

data/lib/ollama_chat/information.rb

@@ -16,6 +16,7 @@
 #   chat.usage
 module OllamaChat::Information
   extend Tins::Concern
+  include OllamaChat::Utils::Redact
 
   included do
     include UserAgent
@@ -190,7 +191,7 @@ module OllamaChat::Information
   def info_rag(output: STDOUT)
     if embedding.on?
       if base_url = OC::OLLAMA::EMBEDDING_URL?
-        output.puts "🔌 Connected to embedding ollama server on: #{base_url.to_s.inspect}"
+        output.puts "🔌 Connected to embedding ollama server on: #{redact_password(base_url)}"
       end
       output.puts "🗄️ Current RAG model is #{bold{@embedding_model}}"
       if @embedding_model_options.present?
@@ -220,7 +221,7 @@ module OllamaChat::Information
   def print_welcome(output: STDOUT)
     feedback(<<~EOT, output:)
       💎 Running ollama_chat version: #{bold{OllamaChat::VERSION}}
-      🔌 Connected to ollama server version: #{bold{server_version}} on: #{bold{server_url}}
+      🔌 Connected to ollama server version: #{bold{server_version}} on: #{bold{redact_password(server_url)}}
     EOT
   end
 

data/lib/ollama_chat/links_set.rb

@@ -23,6 +23,7 @@ class OllamaChat::LinksSet
     @set.add(x)
     sync
   end
+  alias << add
 
   # Removes all links from the set and synchronizes it with the session.
   #

data/lib/ollama_chat/model_handling.rb

@@ -465,7 +465,7 @@ module OllamaChat::ModelHandling
       if model_metadata = model_present?(model, ollama:)
         return model_metadata
       end
-      raise OllamaChat::UnknownModelError, "unknown model named #{@model.inspect}"
+      raise OllamaChat::UnknownModelError, "unknown model named #{model.inspect}"
     end
   end
 
@@ -587,17 +587,25 @@ module OllamaChat::ModelHandling
     old_model   = @model
 
     if model.blank?
-      @model = choose_model('', @model)
+      new_model = choose_model('', old_model)
     else
-      @model = choose_model(model, config.model.name)
+      new_model = choose_model(model, config.model.name)
     end
 
-    if @model_metadata = model_present?(@model, ollama:)
-      session.update(current_model: @model)
-    else
-      session.update(current_model: nil)
+    unless model_metadata = model_present?(new_model, ollama:)
+      feedback("Model #{bold{new_model}} not found locally.", type: :warn)
+      unless confirm?(prompt: "Pull it from remote? (y/n) ", yes: /\Ay/i)
+        feedback("Cancelled.", type: :cancel)
+        old_model.full? && model_present?(old_model, ollama:) or
+              raise OllamaChat::UnknownModelError, "no usable model after cancel"
+        return @model_metadata
+      end
+      model_metadata = pull_model_unless_present(new_model, ollama:)
     end
 
+    @model, @model_metadata = new_model, model_metadata
+    session.update(current_model: @model)
+
     if old_model != @model
       reconfigure_model_options(profile:, keep_options:)
       if @model_metadata&.can?('thinking')

data/lib/ollama_chat/oc.rb

@@ -114,7 +114,7 @@ module OC
 
     URL = set do
       description 'Ollama base URL to connect to'
-      default     { 'http://%s' % OC::OLLAMA::HOST }
+      default     { host = OC::OLLAMA::HOST? and Ollama.base_url(host).to_s }
       sensitive   true
       decode { URI.parse(_1) if _1.present? }
       check { value.scheme =~ /\Ahttps?\z/ }

data/lib/ollama_chat/rag_handling.rb

@@ -122,7 +122,7 @@ module OllamaChat::RAGHandling
           selected.merge(remaining)
           break
         else
-          selected.add(choice)
+          selected << choice
         end
       end
     end
@@ -202,7 +202,7 @@ module OllamaChat::RAGHandling
           selected.merge(remaining)
           break
         else
-          selected.add(choice)
+          selected << choice
         end
       end
     end
@@ -213,14 +213,25 @@ module OllamaChat::RAGHandling
 
     feedback("Restoring #{selected.size} collection(s): #{selected.join(', ')}")
 
-    total = 0
+    total   = 0
+    skipped = 0
     collections.slice(*selected).sort.each do |col, group|
       create_memory_collection(col)
       switch_collection(col) do
+        existing_tags = Set.new
+        @documents.each_record do |r|
+          r.tags.each { |t| existing_tags << t.to_s(link: false) }
+        end
+
         group.each do |entry|
+          tags = Array(entry['tags'])
+          if tags.any? { |t| existing_tags.include?(t) }
+            skipped += 1
+            next
+          end
           @documents.add(
             [entry['text']],
-            tags: Array(entry['tags']),
+            tags:,
             batch_size: 1,
             source: entry['source']
           )
@@ -230,9 +241,10 @@ module OllamaChat::RAGHandling
     end
 
     log(:info, 'Memory restored',
-        data: { file: filename.to_s, collections: selected, records: total })
+        data: { file: filename.to_s, collections: selected, records: total, skipped: })
     feedback(
-      "Restored #{total} record(s) across #{selected.size} collection(s) from #{filename.to_s.inspect}.",
+      "Restored #{total} record(s) across #{selected.size} collection(s) from #{filename.to_s.inspect}." +
+        (skipped.zero? ? '' : " (#{skipped} duplicate(s) skipped)."),
       type: :success
     )
   end
@@ -374,7 +386,7 @@ module OllamaChat::RAGHandling
         source_tags = Hash.new { |h, k| h[k] = Set.new }
         @documents.records.each do |record|
           source = record.source.full? or next
-          sources.add(source)
+          sources << source
           source_tags[source].merge(record.tags)
         end
         display_map = {}
@@ -480,7 +492,6 @@ module OllamaChat::RAGHandling
         col&.update(name: new_collection.to_s)
         log(:info, "Collection renamed", data: { old_name: current_collection, new_name: new_collection })
         feedback("Renamed current collection #{current_collection} to #{new_collection}.", type: :info)
-        refresh_system_prompt
       rescue Sequel::UniqueConstraintViolation
         feedback("Renaming to #{new_collection} failed, it already exists in database.", type: :warn)
       rescue => e
@@ -600,12 +611,12 @@ module OllamaChat::RAGHandling
       end
 
       sources = {}
-      seen = {}
+      seen    = Set.new
       @documents.each_record do |record|
         source = @documents.normalize_source(record.source) or next
         if matched_sources.nil? || matched_sources.member?(source)
-          seen.key?(source) and next
-          seen[source] = true
+          seen.member?(source) and next
+          seen << source
           unless @documents.source_modified?(source)
             infobar.puts "Source #{source.to_s.inspect} is unmodified. => Skipping."
             next
@@ -616,8 +627,8 @@ module OllamaChat::RAGHandling
       end
 
       if patterns = col.patterns.full?
-        new_sources = all_file_set(patterns).map(&:to_s).reject { seen.key?(_1) }.to_set
-        new_sources.each { seen[_1] = true }
+        new_sources = all_file_set(patterns).map(&:to_s).reject { seen.member?(_1) }.to_set
+        new_sources.each { seen << _1 }
         new_sources.each { sources[_1] = [] }
       end
 
@@ -678,7 +689,6 @@ module OllamaChat::RAGHandling
       end
       feedback("Created collection '#{name}'.", type: :success)
       log(:info, "Collection created", data: { name:, description:, patterns: })
-      refresh_system_prompt
     rescue Sequel::UniqueConstraintViolation
       feedback("Collection #{name.inspect} already exists.", type: :warn)
     rescue Sequel::Error => e
@@ -737,7 +747,6 @@ module OllamaChat::RAGHandling
       end
       feedback("Updated collection '#{col.name}'.", type: :success)
       log(:info, "Collection updated", data: { name: col.name, enabled: col.enabled })
-      refresh_system_prompt
     rescue Sequel::Error => e
       feedback("Database error: #{e.message}", type: :warn)
     end
@@ -761,17 +770,31 @@ module OllamaChat::RAGHandling
           next
         end
 
+        if target_name.to_s == 'default'
+          feedback(
+            "Cannot delete #{target_name.inspect}: it is the default collection. " \
+              'Switch to another collection first.',
+            type: :warn
+          )
+          next
+        end
+
+        destroyed = false
         if confirm?(prompt: "⚠️ Are you sure you want to permanently delete #{target_name.inspect}? (y/n) ", yes: /\Ay/i)
           begin
             col.chat = self
             col.destroy
+            destroyed = true
             feedback("Deleted collection #{target_name.inspect}.", type: :success)
             log(:info, "Collection deleted", data: { name: target_name })
-            refresh_system_prompt
           rescue Sequel::Error => e
             feedback("Database error: #{e.message}", type: :warn)
           rescue => e
             feedback("Error removing from Documentrix: #{e.message}", type: :warn)
+          ensure
+            if destroyed && session.current_collection == target_name
+              session.update(current_collection: 'default')
+            end
           end
         else
           feedback("Deletion denied.", type: :denied)

data/lib/ollama_chat/system_prompt_management.rb

@@ -42,11 +42,11 @@ module OllamaChat::SystemPromptManagement
   end
 
   # Re-interpolates the current system prompt with fresh dynamic values
-  # (persona content, collection descriptions, runtime info).
+  # (persona content, persona name, runtime info).
   #
   # Call this after any operation that mutates the sources baked into
-  # the system prompt, such as creating, editing, renaming, or deleting
-  # a collection, or after a persona file has been edited on disk.
+  # the system prompt, such as switching personas or toggling runtime
+  # info.
   #
   # Safe to call when no system prompt is set; returns immediately.
   def refresh_system_prompt

data/lib/ollama_chat/tools/browse.rb

@@ -60,10 +60,10 @@ class OllamaChat::Tools::Browse
     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
+                "Opened #{redact_password(url).inspect} in browser."
+              else
+                "Could not open #{redact_password(url).inspect} in browser."
+              end
 
     {
       success:    result.success?,

data/lib/ollama_chat/tools/concern.rb

@@ -7,6 +7,7 @@
 module OllamaChat::Tools::Concern
   extend Tins::Concern
   include OllamaChat::Utils::ValueFormatter
+  include OllamaChat::Utils::Redact
   include OllamaChat::Utils::Backup
   include OllamaChat::Feedback
 

data/lib/ollama_chat/tools/execute_grep.rb

@@ -21,9 +21,9 @@ class OllamaChat::Tools::ExecuteGrep
       function: Tool::Function.new(
         name:,
         description: <<~EOT,
-          Pattern searcher – Runs a regex against all files at path, optionally
-          case‑insensitive, returning matches (up to max_results). Useful for
-          quick codebase queries or finding config values.
+          Pattern searcher – Runs a regex against all files' contents at path,
+          optionally case‑insensitive, returning matches (up to max_results).
+          Useful for quick codebase queries or finding config values.
         EOT
         parameters: Tool::Function::Parameters.new(
           type: 'object',

data/lib/ollama_chat/tools/generate_image.rb

@@ -92,10 +92,10 @@ class OllamaChat::Tools::GenerateImage
       filename:, subfolder: '', type: 'output', rand:
     )
 
-    chat.links.add(view_url.to_s)
+    chat.links << view_url.to_s
 
     message = "Image successfully generated! The user can view it here,"\
-      " give this URL to the user: #{view_url}"
+      " give this URL to the user: #{redact_password(view_url)}"
     {
       status:   'success',
       message:  ,

data/lib/ollama_chat/utils/analyze_directory.rb

@@ -44,13 +44,13 @@ module OllamaChat::Utils::AnalyzeDirectory
   #   directories are included in the tree. Defaults to `false`, which
   #   skips any entry whose base name starts with a dot.
   #
-  # @return [Array<Hash>] An array of entry hashes.  Each hash contains:
-  #   * `:type`   – "file" or "directory"
-  #   * `:name`   – Base name of the file/directory
-  #   * `:path`   – Absolute path
-  #   * `:depth`  – Depth relative to the root (root = 0)
-  #   * `:height` – The maximum depth found in the entire tree
-  #   * `:children` – Array of child entry hashes (only for directories)
+  # @return [Hash] A hash with two keys:
+  #   * `:height`  – The maximum depth found in the entire tree.
+  #   * `:entries` – Array of entry hashes. Each hash contains:
+  #     * `:path`     – Absolute path (directories end with `/`)
+  #     * `:depth`    – Depth relative to the root (root = 0)
+  #     * `:children` – Array of child entry hashes (directories only,
+  #                      omitted when empty)
   #
   # @raise [StandardError] Any exception raised during traversal is
   #   rescued and returned as a hash with `:error` and `:message`
@@ -72,15 +72,16 @@ module OllamaChat::Utils::AnalyzeDirectory
       height = e[:depth] if e[:depth] > height
     end
 
-    structure_each_entry(entries) { |e| e[:height] = height }
-
     if max_depth && max_depth <= height
       structure_each_entry(entries) do |e|
-        e[:children]&.reject! { |c| c[:depth] >= max_depth }
+        if children = e[:children].full?
+          children.reject! { |c| c[:depth] >= max_depth }
+          e.delete(:children) if children.empty?
+        end
       end
     end
 
-    entries
+    { height:, entries: }
   rescue => e
     { error: e.class, message: e.message }
   end
@@ -116,19 +117,18 @@ module OllamaChat::Utils::AnalyzeDirectory
       next if exclude.any? { |e| child.basename.fnmatch?(e.to_s, File::FNM_PATHNAME) }
 
       if child.directory?
+        children = recurse_generate_structure(
+          child, exclude:, suffix:, depth: depth + 1, include_hidden:
+        ).full?
         entries << {
-          type:     'directory',
-          name:     child.basename.to_s,
-          path:     child.expand_path.to_s,
-          children: recurse_generate_structure(child, exclude:, suffix:, depth: depth + 1, include_hidden:),
-          depth:
-        }
+          path:     child.expand_path.to_s << ?/,
+          depth:    ,
+          children: ,
+        }.compact
       elsif child.file?
         # Skip unless suffix matches
         next if extname.present? && child.extname != extname
         entries << {
-          type:  'file',
-          name:  child.basename.to_s,
           path:  child.expand_path.to_s,
           depth:
         }

data/lib/ollama_chat/utils/fetcher.rb

@@ -19,6 +19,7 @@ require 'ollama_chat/utils/cache_fetcher'
 #     # Process the fetched content
 #   end
 class OllamaChat::Utils::Fetcher
+  include OllamaChat::Utils::Redact
   # A module that extends IO objects with content type metadata and expiration
   # tracking.
   #
@@ -154,7 +155,7 @@ class OllamaChat::Utils::Fetcher
   def self.get(url, headers: {}, **options, &block)
     cache = options.delete(:cache) and
       cache = OllamaChat::Utils::CacheFetcher.new(cache)
-    cache and infobar.puts "Getting #{url.to_s.inspect} via cache…"
+    cache and infobar.puts "Getting #{OllamaChat::Utils::Redact.redact_password(url).inspect} via cache…"
     if result = cache&.get(url, &block)
       content_type = result&.content_type || 'unknown'
       infobar.puts "…hit, found #{content_type} content in cache."

data/lib/ollama_chat/utils/redact.rb

@@ -0,0 +1,32 @@
+require 'uri'
+
+# Redacts credentials from URLs for safe, user-facing display.
+#
+# Whenever a URL that may carry a password (e.g.
+# `http://user:secret@host`) is about to be printed to the terminal or
+# echoed back to the model, run it through {#redact_password} so the
+# secret is masked as `***`.
+#
+# @example
+#   OllamaChat::Utils::Redact.redact_password('http://u:secret@example.com')
+#   # => "http://u:***@example.com"
+#
+# The module can also be +include+d to gain {#redact_password} as a
+# private helper on the host class.
+module OllamaChat::Utils::Redact
+  module_function
+
+  # Returns the URL as a string with its password component replaced by
+  # `***`. Accepts a +URI+ or a plain +String+ (parsed on the fly). The
+  # original object is never mutated: a duplicate is redacted and the
+  # result returned as a string. URLs without a password are returned
+  # unchanged.
+  #
+  # @param url [URI, String] the URL whose password should be hidden
+  # @return [String] the URL with its password redacted
+  def redact_password(url)
+    uri = url.is_a?(String) ? URI.parse(url) : url.dup
+    uri.password = '***' if uri.password
+    uri.to_s
+  end
+end

data/lib/ollama_chat/utils.rb

@@ -9,6 +9,7 @@
 module OllamaChat::Utils
 end
 
+require 'ollama_chat/utils/redact'
 require 'ollama_chat/utils/analyze_directory'
 require 'ollama_chat/utils/audio_player'
 require 'ollama_chat/utils/backup'

data/lib/ollama_chat/version.rb

@@ -5,7 +5,7 @@
 # by hand; it is rewritten on every `rake version` invocation.
 module OllamaChat
   # The current version of the OllamaChat gem as a String.
-  VERSION         = '0.0.136'
+  VERSION         = '0.0.138'
   VERSION_ARRAY   = VERSION.split('.').map(&:to_i) # :nodoc:
   VERSION_MAJOR   = VERSION_ARRAY[0] # :nodoc:
   VERSION_MINOR   = VERSION_ARRAY[1] # :nodoc:

data/lib/ollama_chat/web_searching.rb

@@ -31,7 +31,7 @@ module OllamaChat::WebSearching
         log(:info, "Web search performed", data: {
           search_engine: engine, query:, results: results.size
         })
-        results.each { |url| links.add(url) }
+        results.each { |url| links << url }
       end
     else
       feedback("Search engine #{bold{engine}} not implemented!", type: :warn)