ollama_chat 0.0.29 → 0.0.31

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c4521bf944235fe5a3507f46e8c1ff8016faab3c178c2c8981d223c02c32ad1e
-  data.tar.gz: 8266fb46c588c5e456c00574a2c291b89da9aab453e8d33690a32c9b5e23762a
+  metadata.gz: '025319afc822ea46074061e5976aed1aa34f6ebe45e305bc570d58bc7883a30e'
+  data.tar.gz: eeed845da1c216ab6c38052655db1249c7f15bed25d10df2c8bb7a8578984325
 SHA512:
-  metadata.gz: b3bf3bc435c932230a7ac671f52fb41890abf8e673dd31bf8784eac721ef859e7726c06a164f5a15f7a474351274213bbbd82fe8973c9340ef3cd5463343799d
-  data.tar.gz: 7a05cc98e971adaac2e1ae834ac732533bfea925bc6c7e084068bad4dc4fc16292f3eac936406ff8abca9253cafc5b61f61f1597af8945595bd62a0084c85aba
+  metadata.gz: c79d81e5087a74486ec98dbad3ddc91099b6fda1a4a3c26365f7ff5a7e0c14934d806741ae26ecf5534002d4e47a46cd70abbbda681cff95d58925c882ff7cef
+  data.tar.gz: 696a4e5089a5b4d43729b2c0552721528dbc8647d66e538166e274ab422f422239d449f992d63679a440e54e88ae544b1672d484902f6ca55d2e91f1299caa25

data/CHANGES.md

@@ -1,5 +1,43 @@
 # Changes
 
+## 2025-09-15 v0.0.31
+
+- Added new test asset file `"spec/assets/example with \".html"` as
+  `spec/assets/example_with_quote.html`
+- Enhanced parsing tests in `spec/ollama_chat/parsing_spec.rb` to support file
+  URLs with spaces and quotes
+- Added cleanup code using `FileUtils.rm_f` in test cases to remove temporary
+  files
+- Modified test method names to reflect support for spaces and quotes in file
+  paths
+- Added `ghostscript` package to the Dockerfile dependencies
+- Added `fail_fast: true` to the script configuration
+- Added a check for `pbcopy` command in clipboard spec
+- Wrapped SimpleCov require in begin/rescue block to handle LoadError
+- Set default `OLLAMA_HOST` environment variable to 'localhost:11434' in spec
+  helper
+- Added `CONTENT_REGEXP` for unified parsing of URLs, tags, and file references
+- Updated `parse_content` method to use new regex and support quoted file paths
+- Introduced `check_exist:` option in `fetch_source` to validate file existence
+- Extracted file reading logic into new helper method
+  `fetch_source_as_filename`
+- Enhanced handling of file URLs with spaces and special characters
+- Added comprehensive tests for parsing file paths with spaces, quotes, and
+  encoding
+- Supported `file://` URI decoding using `URI.decode_www_form_component`
+- Refactored `fetch_source` to better handle relative, absolute, and tilde
+  paths
+- Updated spec files to use `Pathname` and improved assertions for file content
+- Updated `gem_hadar` development dependency version from **2.3** to **2.6**
+
+## 2025-09-11 v0.0.30
+
+- Changed `config.prompts.web_import` to `config.prompts.web_summarize` in default branch
+- Updated method calls from `import(url).full?` to `summarize(url).full?` in summarizing branch
+- Updated method calls from `summarize(url).full?` to `import(url).full?` in default branch
+- Added `ask_and_send_or_self(:read)` call within the block for both branches
+- Added `ruby:3.1-alpine` image configuration to CI pipeline
+
 ## 2025-09-08 v0.0.29
 
 - Refactored conversation persistence into a dedicated

data/Rakefile

@@ -34,7 +34,7 @@ GemHadar do
   executables << 'ollama_chat' << 'ollama_chat_send'
 
   dependency             'excon',                 '~> 1.0'
-  dependency             'ollama-ruby',           '~> 1.6'
+  dependency             'ollama-ruby',           '~> 1.7'
   dependency             'documentrix',           '~> 0.0', '>= 0.0.2'
   dependency             'unix_socks',            '~> 0.1'
   dependency             'rss',                   '~> 0.3'
@@ -43,7 +43,7 @@ GemHadar do
   dependency             'mime-types',            '~> 3.0'
   dependency             'reverse_markdown',      '~> 3.0'
   dependency             'xdg'
-  dependency             'kramdown-ansi',         '~> 0.1'
+  dependency             'kramdown-ansi',         '~> 0.2'
   dependency             'complex_config',        '~> 0.22', '>= 0.22.2'
   dependency             'tins',                  '~> 1.41'
   dependency             'search_ui',             '~> 0.0'

data/lib/ollama_chat/chat.rb

@@ -394,15 +394,19 @@ class OllamaChat::Chat
       end
       prompt.named_placeholders_interpolate({query:})
     elsif @document_policy == 'summarizing'
-      prompt = config.prompts.web_summarize
+      prompt = config.prompts.web_import
       results = urls.each_with_object('') do |url, content|
-        import(url).full? { |c| content << c }
+        summarize(url).full? do |c|
+          content << c.ask_and_send_or_self(:read)
+        end
       end
       prompt.named_placeholders_interpolate({query:, results:})
     else
-      prompt = config.prompts.web_import
+      prompt = config.prompts.web_summarize
       results = urls.each_with_object('') do |url, content|
-        summarize(url).full? { |c| content << c }
+        import(url).full? do |c|
+          content << c.ask_and_send_or_self(:read)
+        end
       end
       prompt.named_placeholders_interpolate({query:, results:})
     end

data/lib/ollama_chat/parsing.rb

@@ -189,6 +189,21 @@ module OllamaChat::Parsing
     )
   end
 
+  CONTENT_REGEXP = %r{
+    (https?://\S+)                         # Match HTTP/HTTPS URLs
+    |                                      # OR
+    (?<![a-zA-Z\d])                        # Negative lookbehind: not part of a larger word
+    \#                                     # Literal # character (starts a tag)
+    ([\w\]\[]+)                            # Tag content: alphanumeric, brackets, underscores
+    |                                      # OR
+    (file://(?:[^\s#]+))                   # Match file:// URLs
+    |                                      # OR
+    "((?:\.\.|[~.]?)/(?:\\"|\\|[^"\\]+)+)" # Quoted file path with escaped " quotes
+    |                                      # OR
+    (\S*\/\S+)                             # File path with at least one forward slash
+  }x
+  private_constant :CONTENT_REGEXP
+
   # Parses content and processes embedded resources based on document policy
   #
   # This method analyzes input content for URLs, tags, and file references,
@@ -203,28 +218,30 @@ module OllamaChat::Parsing
   def parse_content(content, images)
     images.clear
     tags = Documentrix::Utils::Tags.new valid_tag: /\A#*([\w\]\[]+)/
-
     contents = [ content ]
-    content.scan(%r((https?://\S+)|(?<![a-zA-Z\d])#+([\w\]\[]+)|(?:file://)?(\S*\/\S+))).each do |url, tag, file|
+    content.scan(CONTENT_REGEXP).each { |url, tag, file_url, quoted_file, file|
+      check_exist = false
       case
       when tag
         tags.add(tag)
         next
-      when file
-        file = file.sub(/#.*/, '')
-        file =~ %r(\A[~./]) or file.prepend('./')
-        file = begin
-                 File.expand_path(file)
-               rescue ArgumentError
-                 next
-               end
-        File.exist?(file) or next
-        source = file
       when url
         links.add(url.to_s)
         source = url
+      when file_url
+        check_exist = true
+        source      = file_url
+      when quoted_file
+        file = quoted_file.gsub('\"', ?")
+        file =~ %r(\A[~./]) or file.prepend('./')
+        check_exist = true
+        source      = file
+      when file
+        file =~ %r(\A[~./]) or file.prepend('./')
+        check_exist = true
+        source      = file
       end
-      fetch_source(source) do |source_io|
+      fetch_source(source, check_exist:) do |source_io|
         case source_io&.content_type&.media_type
         when 'image'
           add_image(images, source_io, source)
@@ -246,7 +263,7 @@ module OllamaChat::Parsing
           )
         end
       end
-    end
+    }
     new_content = contents.select { _1.present? rescue nil }.compact * "\n\n"
     return new_content, (tags unless tags.empty?)
   end

data/lib/ollama_chat/source_fetching.rb

@@ -52,14 +52,14 @@ module OllamaChat::SourceFetching
   # @param source [ String ] the source identifier which can be a command, URL, or file path
   #
   # @yield [ tmp ]
-  def fetch_source(source, &block)
+  def fetch_source(source, check_exist: false, &block)
     case source
-    when %r(\A!(.*))
+    when %r{\A!(.*)}
       command = $1
       OllamaChat::Utils::Fetcher.execute(command) do |tmp|
         block.(tmp)
       end
-    when %r(\Ahttps?://\S+)
+    when %r{\Ahttps?://\S+}
       links.add(source.to_s)
       OllamaChat::Utils::Fetcher.get(
         source,
@@ -70,12 +70,20 @@ module OllamaChat::SourceFetching
       ) do |tmp|
         block.(tmp)
       end
-    when %r(\Afile://(/\S*?)#|\A((?:\.\.|[~.]?)/\S*))
-      filename = $~.captures.compact.first
+    when %r{\Afile://([^\s#]+)}
+      filename = URI.decode_www_form_component($1)
       filename = File.expand_path(filename)
-      OllamaChat::Utils::Fetcher.read(filename) do |tmp|
-        block.(tmp)
-      end
+      check_exist && !File.exist?(filename) and return
+      fetch_source_as_filename(filename, &block)
+    when  %r{\A((?:\.\.|[~.]?)/(?:\ |\S+)*)}
+      filename = File.expand_path($1)
+      check_exist && !File.exist?(filename) and return
+      fetch_source_as_filename(filename, &block)
+    when %r{\A"((?:\.\.|[~.]?)/(?:\\"|\\|[^"\\]+)+)"}
+      filename = $1.gsub('\"', ?")
+      filename = File.expand_path(filename)
+      check_exist && !File.exist?(filename) and return
+      fetch_source_as_filename(filename, &block)
     else
       raise "invalid source #{source.inspect}"
     end
@@ -83,6 +91,21 @@ module OllamaChat::SourceFetching
     STDERR.puts "Cannot fetch source #{source.to_s.inspect}: #{e.class} #{e}\n#{e.backtrace * ?\n}"
   end
 
+  # Reads a file and extends it with header extension metadata. It then yields
+  # the file to the provided block for processing. If the file does not exist,
+  # it outputs an error message to standard error.
+  #
+  # @param filename [ String ] the path to the file to be read
+  #
+  # @yield [ file ] yields the opened file with header extension
+  #
+  # @return [ nil ] returns nil if the file does not exist
+  # @return [ Object ] returns the result of the block execution if the file exists
+  private def fetch_source_as_filename(filename, &block)
+    OllamaChat::Utils::Fetcher.read(filename) do |tmp|
+      block.(tmp)
+    end
+  end
 
   # Adds an image to the images collection from the given source IO and source
   # identifier.

data/lib/ollama_chat/version.rb

@@ -1,6 +1,6 @@
 module OllamaChat
   # OllamaChat version
-  VERSION         = '0.0.29'
+  VERSION         = '0.0.31'
   VERSION_ARRAY   = VERSION.split('.').map(&:to_i) # :nodoc:
   VERSION_MAJOR   = VERSION_ARRAY[0] # :nodoc:
   VERSION_MINOR   = VERSION_ARRAY[1] # :nodoc:

data/ollama_chat.gemspec

@@ -1,9 +1,9 @@
 # -*- encoding: utf-8 -*-
-# stub: ollama_chat 0.0.29 ruby lib
+# stub: ollama_chat 0.0.31 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "ollama_chat".freeze
-  s.version = "0.0.29".freeze
+  s.version = "0.0.31".freeze
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib".freeze]
@@ -13,7 +13,7 @@ Gem::Specification.new do |s|
   s.email = "flori@ping.de".freeze
   s.executables = ["ollama_chat".freeze, "ollama_chat_send".freeze]
   s.extra_rdoc_files = ["README.md".freeze, "lib/ollama_chat.rb".freeze, "lib/ollama_chat/chat.rb".freeze, "lib/ollama_chat/clipboard.rb".freeze, "lib/ollama_chat/conversation.rb".freeze, "lib/ollama_chat/dialog.rb".freeze, "lib/ollama_chat/document_cache.rb".freeze, "lib/ollama_chat/follow_chat.rb".freeze, "lib/ollama_chat/history.rb".freeze, "lib/ollama_chat/information.rb".freeze, "lib/ollama_chat/kramdown_ansi.rb".freeze, "lib/ollama_chat/message_format.rb".freeze, "lib/ollama_chat/message_list.rb".freeze, "lib/ollama_chat/message_output.rb".freeze, "lib/ollama_chat/model_handling.rb".freeze, "lib/ollama_chat/ollama_chat_config.rb".freeze, "lib/ollama_chat/parsing.rb".freeze, "lib/ollama_chat/server_socket.rb".freeze, "lib/ollama_chat/source_fetching.rb".freeze, "lib/ollama_chat/switches.rb".freeze, "lib/ollama_chat/utils.rb".freeze, "lib/ollama_chat/utils/cache_fetcher.rb".freeze, "lib/ollama_chat/utils/chooser.rb".freeze, "lib/ollama_chat/utils/fetcher.rb".freeze, "lib/ollama_chat/utils/file_argument.rb".freeze, "lib/ollama_chat/version.rb".freeze, "lib/ollama_chat/vim.rb".freeze, "lib/ollama_chat/web_searching.rb".freeze]
-  s.files = [".utilsrc".freeze, "CHANGES.md".freeze, "Gemfile".freeze, "README.md".freeze, "Rakefile".freeze, "bin/ollama_chat".freeze, "bin/ollama_chat_send".freeze, "config/searxng/settings.yml".freeze, "docker-compose.yml".freeze, "lib/ollama_chat.rb".freeze, "lib/ollama_chat/chat.rb".freeze, "lib/ollama_chat/clipboard.rb".freeze, "lib/ollama_chat/conversation.rb".freeze, "lib/ollama_chat/dialog.rb".freeze, "lib/ollama_chat/document_cache.rb".freeze, "lib/ollama_chat/follow_chat.rb".freeze, "lib/ollama_chat/history.rb".freeze, "lib/ollama_chat/information.rb".freeze, "lib/ollama_chat/kramdown_ansi.rb".freeze, "lib/ollama_chat/message_format.rb".freeze, "lib/ollama_chat/message_list.rb".freeze, "lib/ollama_chat/message_output.rb".freeze, "lib/ollama_chat/model_handling.rb".freeze, "lib/ollama_chat/ollama_chat_config.rb".freeze, "lib/ollama_chat/ollama_chat_config/default_config.yml".freeze, "lib/ollama_chat/parsing.rb".freeze, "lib/ollama_chat/server_socket.rb".freeze, "lib/ollama_chat/source_fetching.rb".freeze, "lib/ollama_chat/switches.rb".freeze, "lib/ollama_chat/utils.rb".freeze, "lib/ollama_chat/utils/cache_fetcher.rb".freeze, "lib/ollama_chat/utils/chooser.rb".freeze, "lib/ollama_chat/utils/fetcher.rb".freeze, "lib/ollama_chat/utils/file_argument.rb".freeze, "lib/ollama_chat/version.rb".freeze, "lib/ollama_chat/vim.rb".freeze, "lib/ollama_chat/web_searching.rb".freeze, "ollama_chat.gemspec".freeze, "redis/redis.conf".freeze, "spec/assets/api_show.json".freeze, "spec/assets/api_tags.json".freeze, "spec/assets/api_version.json".freeze, "spec/assets/conversation.json".freeze, "spec/assets/duckduckgo.html".freeze, "spec/assets/example.atom".freeze, "spec/assets/example.csv".freeze, "spec/assets/example.html".freeze, "spec/assets/example.pdf".freeze, "spec/assets/example.ps".freeze, "spec/assets/example.rb".freeze, "spec/assets/example.rss".freeze, "spec/assets/example.xml".freeze, "spec/assets/kitten.jpg".freeze, "spec/assets/prompt.txt".freeze, "spec/assets/searxng.json".freeze, "spec/ollama_chat/chat_spec.rb".freeze, "spec/ollama_chat/clipboard_spec.rb".freeze, "spec/ollama_chat/follow_chat_spec.rb".freeze, "spec/ollama_chat/information_spec.rb".freeze, "spec/ollama_chat/kramdown_ansi_spec.rb".freeze, "spec/ollama_chat/message_list_spec.rb".freeze, "spec/ollama_chat/message_output_spec.rb".freeze, "spec/ollama_chat/model_handling_spec.rb".freeze, "spec/ollama_chat/parsing_spec.rb".freeze, "spec/ollama_chat/server_socket_spec.rb".freeze, "spec/ollama_chat/source_fetching_spec.rb".freeze, "spec/ollama_chat/switches_spec.rb".freeze, "spec/ollama_chat/utils/cache_fetcher_spec.rb".freeze, "spec/ollama_chat/utils/fetcher_spec.rb".freeze, "spec/ollama_chat/utils/file_argument_spec.rb".freeze, "spec/ollama_chat/web_searching_spec.rb".freeze, "spec/spec_helper.rb".freeze, "tmp/.keep".freeze]
+  s.files = [".utilsrc".freeze, "CHANGES.md".freeze, "Gemfile".freeze, "README.md".freeze, "Rakefile".freeze, "bin/ollama_chat".freeze, "bin/ollama_chat_send".freeze, "config/searxng/settings.yml".freeze, "docker-compose.yml".freeze, "lib/ollama_chat.rb".freeze, "lib/ollama_chat/chat.rb".freeze, "lib/ollama_chat/clipboard.rb".freeze, "lib/ollama_chat/conversation.rb".freeze, "lib/ollama_chat/dialog.rb".freeze, "lib/ollama_chat/document_cache.rb".freeze, "lib/ollama_chat/follow_chat.rb".freeze, "lib/ollama_chat/history.rb".freeze, "lib/ollama_chat/information.rb".freeze, "lib/ollama_chat/kramdown_ansi.rb".freeze, "lib/ollama_chat/message_format.rb".freeze, "lib/ollama_chat/message_list.rb".freeze, "lib/ollama_chat/message_output.rb".freeze, "lib/ollama_chat/model_handling.rb".freeze, "lib/ollama_chat/ollama_chat_config.rb".freeze, "lib/ollama_chat/ollama_chat_config/default_config.yml".freeze, "lib/ollama_chat/parsing.rb".freeze, "lib/ollama_chat/server_socket.rb".freeze, "lib/ollama_chat/source_fetching.rb".freeze, "lib/ollama_chat/switches.rb".freeze, "lib/ollama_chat/utils.rb".freeze, "lib/ollama_chat/utils/cache_fetcher.rb".freeze, "lib/ollama_chat/utils/chooser.rb".freeze, "lib/ollama_chat/utils/fetcher.rb".freeze, "lib/ollama_chat/utils/file_argument.rb".freeze, "lib/ollama_chat/version.rb".freeze, "lib/ollama_chat/vim.rb".freeze, "lib/ollama_chat/web_searching.rb".freeze, "ollama_chat.gemspec".freeze, "redis/redis.conf".freeze, "spec/assets/api_show.json".freeze, "spec/assets/api_tags.json".freeze, "spec/assets/api_version.json".freeze, "spec/assets/conversation.json".freeze, "spec/assets/duckduckgo.html".freeze, "spec/assets/example.atom".freeze, "spec/assets/example.csv".freeze, "spec/assets/example.html".freeze, "spec/assets/example.pdf".freeze, "spec/assets/example.ps".freeze, "spec/assets/example.rb".freeze, "spec/assets/example.rss".freeze, "spec/assets/example.xml".freeze, "spec/assets/example_with_quote.html".freeze, "spec/assets/kitten.jpg".freeze, "spec/assets/prompt.txt".freeze, "spec/assets/searxng.json".freeze, "spec/ollama_chat/chat_spec.rb".freeze, "spec/ollama_chat/clipboard_spec.rb".freeze, "spec/ollama_chat/follow_chat_spec.rb".freeze, "spec/ollama_chat/information_spec.rb".freeze, "spec/ollama_chat/kramdown_ansi_spec.rb".freeze, "spec/ollama_chat/message_list_spec.rb".freeze, "spec/ollama_chat/message_output_spec.rb".freeze, "spec/ollama_chat/model_handling_spec.rb".freeze, "spec/ollama_chat/parsing_spec.rb".freeze, "spec/ollama_chat/server_socket_spec.rb".freeze, "spec/ollama_chat/source_fetching_spec.rb".freeze, "spec/ollama_chat/switches_spec.rb".freeze, "spec/ollama_chat/utils/cache_fetcher_spec.rb".freeze, "spec/ollama_chat/utils/fetcher_spec.rb".freeze, "spec/ollama_chat/utils/file_argument_spec.rb".freeze, "spec/ollama_chat/web_searching_spec.rb".freeze, "spec/spec_helper.rb".freeze, "tmp/.keep".freeze]
   s.homepage = "https://github.com/flori/ollama_chat".freeze
   s.licenses = ["MIT".freeze]
   s.rdoc_options = ["--title".freeze, "OllamaChat - A command-line interface (CLI) for interacting with an Ollama AI model.".freeze, "--main".freeze, "README.md".freeze]
@@ -24,7 +24,7 @@ Gem::Specification.new do |s|
 
   s.specification_version = 4
 
-  s.add_development_dependency(%q<gem_hadar>.freeze, ["~> 2.2".freeze])
+  s.add_development_dependency(%q<gem_hadar>.freeze, ["~> 2.6".freeze])
   s.add_development_dependency(%q<all_images>.freeze, ["~> 0.6".freeze])
   s.add_development_dependency(%q<rspec>.freeze, ["~> 3.2".freeze])
   s.add_development_dependency(%q<kramdown>.freeze, ["~> 2.0".freeze])
@@ -33,7 +33,7 @@ Gem::Specification.new do |s|
   s.add_development_dependency(%q<simplecov>.freeze, [">= 0".freeze])
   s.add_development_dependency(%q<context_spook>.freeze, [">= 0".freeze])
   s.add_runtime_dependency(%q<excon>.freeze, ["~> 1.0".freeze])
-  s.add_runtime_dependency(%q<ollama-ruby>.freeze, ["~> 1.6".freeze])
+  s.add_runtime_dependency(%q<ollama-ruby>.freeze, ["~> 1.7".freeze])
   s.add_runtime_dependency(%q<documentrix>.freeze, ["~> 0.0".freeze, ">= 0.0.2".freeze])
   s.add_runtime_dependency(%q<unix_socks>.freeze, ["~> 0.1".freeze])
   s.add_runtime_dependency(%q<rss>.freeze, ["~> 0.3".freeze])
@@ -42,7 +42,7 @@ Gem::Specification.new do |s|
   s.add_runtime_dependency(%q<mime-types>.freeze, ["~> 3.0".freeze])
   s.add_runtime_dependency(%q<reverse_markdown>.freeze, ["~> 3.0".freeze])
   s.add_runtime_dependency(%q<xdg>.freeze, [">= 0".freeze])
-  s.add_runtime_dependency(%q<kramdown-ansi>.freeze, ["~> 0.1".freeze])
+  s.add_runtime_dependency(%q<kramdown-ansi>.freeze, ["~> 0.2".freeze])
   s.add_runtime_dependency(%q<complex_config>.freeze, ["~> 0.22".freeze, ">= 0.22.2".freeze])
   s.add_runtime_dependency(%q<tins>.freeze, ["~> 1.41".freeze])
   s.add_runtime_dependency(%q<search_ui>.freeze, ["~> 0.0".freeze])

data/spec/assets/example_with_quote.html

@@ -0,0 +1,10 @@
+<!DOCTYPE html>
+<html>
+  <body>
+
+    <h1>My First Heading</h1>
+
+    <p>My first paragraph.</p>
+
+  </body>
+</html>

data/spec/ollama_chat/clipboard_spec.rb

@@ -8,6 +8,7 @@ describe OllamaChat::Clipboard do
   connect_to_ollama_server
 
   it 'can copy to clipboard' do
+    `which pbcopy`.full? or skip 'pbcopy not in path'
     expect(STDERR).to receive(:puts).with(/No response available to copy to the system clipboard/)
     expect(chat.copy_to_clipboard).to be_nil
     chat.instance_variable_get(:@messages).load_conversation(asset('conversation.json'))

data/spec/ollama_chat/parsing_spec.rb

@@ -167,9 +167,10 @@ EOT
     end
 
     it 'can parse file URLs' do
-      content, = chat.parse_content("see file://#{Dir.pwd}/spec/assets/example.html", [])
+      source_path= Pathname.pwd.join('spec/assets/example.html')
+      content, = chat.parse_content("see file://#{source_path}", [])
       expect(content).to include(<<~EOT)
-        Imported "#{Pathname.pwd.join('spec/assets/example.html')}":
+        Imported "file://#{source_path}":
 
         # My First Heading
 
@@ -177,15 +178,48 @@ EOT
       EOT
     end
 
+    it 'can parse file URLs with spaces and quotes' do
+      file_path = Pathname.pwd.join('spec/assets/example with ".html')
+      FileUtils.cp 'spec/assets/example_with_quote.html', file_path
+      file_url = "file://#{file_path.to_s.gsub(' ', '%20').gsub('"', '%22')}"
+      content, = chat.parse_content("see #{file_url}", [])
+      expect(content).to include(<<~EOT)
+        Imported "#{file_url}":
+
+        # My First Heading
+
+        My first paragraph.
+      EOT
+    ensure
+      FileUtils.rm_f file_path
+    end
+
     it 'can parse file paths' do
-      content, = chat.parse_content("see #{Dir.pwd}/spec/assets/example.html", [])
+      file_path = Pathname.pwd.join('spec/assets/example.html')
+      content, = chat.parse_content("see #{file_path}", [])
+      expect(content).to include(<<~EOT)
+        Imported "#{file_path}":
+
+        # My First Heading
+
+        My first paragraph.
+      EOT
+    end
+
+    it 'can parse quoted file paths' do
+      file_path = Pathname.pwd.join('spec/assets/example with ".html')
+      FileUtils.cp 'spec/assets/example_with_quote.html', file_path
+      quoted_file_path = file_path.to_s.gsub('"', '\"')
+      content, = chat.parse_content(%{see "#{quoted_file_path}"}, [])
       expect(content).to include(<<~EOT)
-        Imported "#{Pathname.pwd.join('spec/assets/example.html')}":
+        Imported "#{quoted_file_path}":
 
         # My First Heading
 
         My first paragraph.
       EOT
+    ensure
+      FileUtils.rm_f file_path
     end
 
     it 'can add images' do

data/spec/ollama_chat/source_fetching_spec.rb

@@ -36,4 +36,66 @@ describe OllamaChat::SourceFetching do
       'This source was now embedded: ./spec/assets/example.html'
     )
   end
+
+  describe '#fetch_source' do
+    context 'with filename' do
+      it 'can handle files without spaces' do
+        source = './spec/assets/example.html'
+        expect(chat).to receive(:fetch_source_as_filename).
+          with(File.expand_path(source))
+        chat.fetch_source(source)
+      end
+
+      it 'can handle files with \\ escaped spaces' do
+        source = './spec/assets/file\ with\ spaces.html'
+        expect(chat).to receive(:fetch_source_as_filename).
+          with(File.expand_path(source))
+        chat.fetch_source(source)
+      end
+
+      it 'can handle files with spaces in " quotes' do
+        source = '"./spec/assets/file with spaces.html"'
+        expect(chat).to receive(:fetch_source_as_filename).
+          with(File.expand_path(source[1..-2]))
+        chat.fetch_source(source)
+      end
+
+      it 'can handle files with spaces and " in " quotes' do
+        source = '"./spec/assets/file with \" and spaces.html"'
+        expect(chat).to receive(:fetch_source_as_filename).
+          with(File.expand_path('./spec/assets/file with " and spaces.html'))
+        chat.fetch_source(source)
+      end
+
+      it 'handles relative paths correctly' do
+        source = '../spec/assets/example.html'
+        expect(chat).to receive(:fetch_source_as_filename).
+          with(File.expand_path(source))
+        chat.fetch_source(source)
+      end
+
+      it 'handles tilde expansion in filename' do
+        source = '~/test.txt'
+        expect(chat).to receive(:fetch_source_as_filename).
+          with(File.expand_path(source))
+        chat.fetch_source(source)
+      end
+
+      it 'handles absolute paths in filename' do
+        source = '/tmp/test.txt'
+        expect(chat).to receive(:fetch_source_as_filename).
+          with(File.expand_path(source))
+        chat.fetch_source(source)
+      end
+    end
+
+    context 'with file:// URI' do
+      it 'fetches content from file URI with encoded paths' do
+        source = 'file:///path/with%20spaces/file.txt'
+        expect(chat).to receive(:fetch_source_as_filename).
+          with('/path/with spaces/file.txt')
+        chat.fetch_source(source)
+      end
+    end
+  end
 end

data/spec/spec_helper.rb

@@ -1,5 +1,8 @@
-require 'gem_hadar/simplecov'
-GemHadar::SimpleCov.start
+begin
+  require 'gem_hadar/simplecov'
+  GemHadar::SimpleCov.start
+rescue LoadError
+end
 require 'rspec'
 require 'tins/xt/expose'
 begin
@@ -143,6 +146,9 @@ RSpec.configure do |config|
   config.extend StubOllamaServer
 
   config.around(&ProtectEnvVars.apply)
+  config.before(:each) do
+    ENV['OLLAMA_HOST'] = 'localhost:11434'
+  end
 
   config.before(:suite) do
     infobar.show = nil

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ollama_chat
 version: !ruby/object:Gem::Version
-  version: 0.0.29
+  version: 0.0.31
 platform: ruby
 authors:
 - Florian Frank
@@ -15,14 +15,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.2'
+        version: '2.6'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.2'
+        version: '2.6'
 - !ruby/object:Gem::Dependency
   name: all_images
   requirement: !ruby/object:Gem::Requirement
@@ -141,14 +141,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.6'
+        version: '1.7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.6'
+        version: '1.7'
 - !ruby/object:Gem::Dependency
   name: documentrix
   requirement: !ruby/object:Gem::Requirement
@@ -273,14 +273,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.1'
+        version: '0.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.1'
+        version: '0.2'
 - !ruby/object:Gem::Dependency
   name: complex_config
   requirement: !ruby/object:Gem::Requirement
@@ -466,6 +466,7 @@ files:
 - spec/assets/example.rb
 - spec/assets/example.rss
 - spec/assets/example.xml
+- spec/assets/example_with_quote.html
 - spec/assets/kitten.jpg
 - spec/assets/prompt.txt
 - spec/assets/searxng.json