ollama_chat 0.0.138 → 0.0.139

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 170c9e2778ed0d581bda082eebc43d07a6c6e6a6cd99a0d1e0d3634a5281ca9f
-  data.tar.gz: e57e2eb4d990f10617e1574ae1f70d22741dd814cf945575cc2cb7fb882dd4be
+  metadata.gz: e1a0cfa8c0914331039a424abe408286dce18f50b3b593bc3cac04c551c9b078
+  data.tar.gz: ab96f2413e6ab996329d654d1f7dc60b75d99cf6cd2bb93f5c0b1f5bc09627a7
 SHA512:
-  metadata.gz: b5e6781664a38632da9d7a4f11900864b86b9e0049fcb66d6084777d0f8b4a85e762a57217c23817c77f0fb365592d5d659e3eab874aa270fa97b52a3e727c45
-  data.tar.gz: 6aa806be34d04830f4788dd452d6e8a078e848767978d6bf49d6fef39f3dbd92b3f61fd4dc203c9914491c1ab2d8fa432ff75bc3543bf385dffcda2333acbd83
+  metadata.gz: 987632cebff57817b71ad6f56326c85edf921a4f54c1a42fff906e93603ec7c45113be2eb0b57b372987f9050d62213f54985359fdaf84f4bbc5bcd4bcdf8392
+  data.tar.gz: 9d9cee9148bc2c3542c9eb543015fa5b16393832f2741b56926f8d6b37a0d0633abea5252d32a5d3b931e350d4e44d874e857cdf84ae72d0c70ab79d8b747183

data/CHANGES.md

@@ -1,5 +1,56 @@
 # Changes
 
+## 2026-10-30 v0.0.139
+
+### Features
+
+- Added terminal table wrapping for `/help` and `/session list` output. New
+  private methods `wrap_table_rows` and `table_column_budgets` in
+  `OllamaChat::KramdownANSI` compute per-column width budgets and wrap cell
+  content to fit within a configurable percentage of the terminal width.
+- Updated `help_message` and `list_sessions` to collect rows into arrays and
+  route them through `wrap_table_rows`, removing manual truncation and heading
+  assignment.
+
+### Fixes
+
+- Fixed a `TypeError` in `Collection.sync` when `chat.documents.collections`
+  yielded no collections (e.g., fresh database) by using `.to_a` instead of
+  relying on `Tins::NullClass#to_ary`.
+- Improved error handling in `fetch_source` for `OllamaChat::HTTPError` by
+  adding a dedicated rescue clause that logs a warning without a backtrace and
+  stamps `response_status` and `response_reason` onto the
+  `ResponseMetadata.failed` IO.
+- Added a non-2xx HTTP status guard in the `get_url` tool to short-circuit
+  processing before document policy dispatch, returning the raw body and a
+  status message instead of feeding error pages to the summarizer.
+
+### Improvements
+
+  - Updated the compaction voice report to format context fill percentages as
+    rounded integers with the unit "percent" for grammatical completeness
+    (e.g., "from 38 percent to 19 percent").
+  - Moved `configure_kramdown_ansi_styles` to a private method in
+    `OllamaChat::KramdownANSI`.
+  - Expanded YARD documentation for `OllamaChat::KramdownANSI` to describe
+    styling, parsing, and table wrapping concerns.
+
+### Build & CI
+
+  - Bumped `kramdown-ansi` dependency from `~> 0.7` to `~> 0.9` in `Rakefile`
+    and `ollama_chat.gemspec`.
+
+### Testing
+
+  - Updated `compaction_spec.rb` to expect rounded integer percentages in the
+    spoken compaction report.
+  - Updated `kramdown_ansi_spec.rb` to use a real `OllamaChat::Chat` instance
+    instead of a double and added examples for `wrap_table_rows` and
+    `table_column_budgets`.
+  - Updated `get_url_spec.rb` to stub `response_status: nil` on `SourceIO`
+    doubles and added a spec verifying that non-200 statuses skip summarization
+    and import.
+
 ## 2026-10-28 v0.0.138
 
 ### Features

data/Rakefile

@@ -47,7 +47,7 @@ GemHadar do
   dependency             'redis',                 '~> 5.0'
   dependency             'mime-types',            '~> 3.0'
   dependency             'reverse_markdown',      '~> 3.0'
-  dependency             'kramdown-ansi',         '~> 0.7'
+  dependency             'kramdown-ansi',         '~> 0.9'
   dependency             'complex_config',        '~> 0.22', '>= 0.22.2'
   dependency             'tins',                  '~> 1.61'
   dependency             'search_ui',             '>= 0.2'

data/lib/ollama_chat/command_concern.rb

@@ -6,6 +6,8 @@ module OllamaChat::CommandConcern
   extend Tins::Concern
 
   class_methods do
+    include OllamaChat::KramdownANSI
+
     # @!attribute [rw] commands
     #   @return [Hash<Symbol, Command>] A hash mapping command names to
     #   `Command` instances.
@@ -76,8 +78,8 @@ module OllamaChat::CommandConcern
         all_separators: true,
         border:         :unicode_round,
       }
-      table.headings  = %w[ CMD SUBCMD OPTS HELP ]
       sorted_commands = self.commands.sort_by { [ _2.category, _2.name ] }.map(&:last)
+      rows            = []
       cat             = nil
       sorted_commands.each do |command|
         command.help or next
@@ -93,15 +95,16 @@ module OllamaChat::CommandConcern
         }
         if cat != command.category
           cat = command.category
-          table << [ { value: bold{cat}, colspan: 4 } ]
+          rows << [ { value: bold{cat}, colspan: 4 } ]
         end
-        table << [
-          "%s" % command.command_names.map { ?/ + _1 }.join(?\n),
+        rows << [
+          '%s' % command.command_names.map { ?/ + _1 }.join(?\n),
           '%s' % subcommands,
           command.options,
           command.help,
         ]
       end
+      wrap_table_rows(rows, 4, headings: %w[ CMD SUBCMD OPTS HELP ], percentage: 95).each { |row| table << row }
       table
     end
   end

data/lib/ollama_chat/compaction.rb

@@ -327,7 +327,7 @@ module OllamaChat::Compaction
       duration.minutes? ? '%M minutes and %u seconds' : '%u seconds'
     )
     speak(
-      'Compressed conversation context from %.f to %.f in %s.' % [
+      'Compressed conversation context from %.0f percent to %.0f percent in %s.' % [
         100 * result.context_filled_before,
         100 * result.context_filled_after,
         duration_string,

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

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

data/lib/ollama_chat/kramdown_ansi.rb

@@ -1,30 +1,22 @@
-# A module that provides Kramdown::ANSI styling configuration and parsing
-# functionality for OllamaChat.
+# A module that provides Kramdown::ANSI styling, parsing, and
+# terminal table wrapping for OllamaChat.
 #
-# This module handles the setup and application of ANSI styles for markdown
-# rendering, allowing for customizable terminal output formatting. It manages
-# the configuration of ANSI styles either from environment variables or falls
-# back to default settings, and provides methods to parse content with the
-# configured styling.
+# This module handles three concerns:
+#
+# * **Styling configuration** – sets up ANSI styles either from the
+#   `KRAMDOWN_ANSI_OLLAMA_CHAT_STYLES` environment variable or from
+#   built-in defaults.
+# * **Markdown parsing** – converts raw markdown into ANSI-formatted
+#   terminal output using the configured styles, with a fallback for
+#   invalid encodings.
+# * **Table wrapping** – computes per-column width budgets and wraps
+#   cell content so that `Terminal::Table` output stays within a
+#   configurable percentage of the terminal width.
 #
 # @example Configuring custom ANSI styles via environment variable
 #   Set KRAMDOWN_ANSI_OLLAMA_CHAT_STYLES to a JSON object containing style
 #   definitions for customizing markdown output formatting in the terminal.
 module OllamaChat::KramdownANSI
-  # The configure_kramdown_ansi_styles method sets up ANSI styling for
-  # Kramdown::ANSI output by checking for specific environment variables and
-  # falling back to default styles.
-  #
-  # @return [ Hash ] a hash of ANSI styles configured either from environment
-  #   variables or using default settings
-  def configure_kramdown_ansi_styles
-    if json = OC::KRAMDOWN_ANSI_OLLAMA_CHAT_STYLES?
-      Kramdown::ANSI::Styles.from_json(json).ansi_styles
-    else
-      Kramdown::ANSI::Styles.new.ansi_styles
-    end
-  end
-
   # The kramdown_ansi_parse method processes content using Kramdown::ANSI with
   # custom ANSI styles.
   #
@@ -49,6 +41,22 @@ module OllamaChat::KramdownANSI
     retry
   end
 
+  private
+
+  # The configure_kramdown_ansi_styles method sets up ANSI styling for
+  # Kramdown::ANSI output by checking for specific environment variables and
+  # falling back to default styles.
+  #
+  # @return [ Hash ] a hash of ANSI styles configured either from environment
+  #   variables or using default settings
+  def configure_kramdown_ansi_styles
+    if json = OC::KRAMDOWN_ANSI_OLLAMA_CHAT_STYLES?
+      Kramdown::ANSI::Styles.from_json(json).ansi_styles
+    else
+      Kramdown::ANSI::Styles.new.ansi_styles
+    end
+  end
+
   # The kramdown_markdown_remove method strips markdown formatting, emoji,
   # ANSI escape sequences, and box-drawing characters from content, yielding
   # plain text suitable for TTS playback.
@@ -74,4 +82,82 @@ module OllamaChat::KramdownANSI
            .gsub(/\s{2,}/, ' ')
            .strip
   end
+
+  # Wraps the cells of a terminal table to fit within a terminal-width
+  # budget.
+  #
+  # Terminal::Table sizes each column to its widest cell, so a single long
+  # cell forces the whole table wider than the terminal. This computes a
+  # per-column budget from the widest cell in each column across all rows,
+  # then wraps each cell to it. Cells expressed as Hashes (e.g. colspan
+  # rows) are passed through untouched.
+  #
+  # @param rows [Array<Array<String, Hash>>] the data rows to wrap
+  # @param cols [Integer] the number of columns in the table
+  # @param headings [Array<String>, nil] the heading row; when given it is
+  #   measured for the budget and returned first, right-padded to each
+  #   column's allocated width so Terminal::Table sizes columns to the
+  #   full budget even when no data cell reaches it
+  # @param percentage [Numeric] the percentage of terminal width to use for
+  #   the total budget (default: 90)
+  # @return [Array<Array<String, Hash>>] wrapped rows; the padded heading
+  #   row is returned first when `headings` was supplied
+  def wrap_table_rows(rows, cols, headings: nil, percentage: 90)
+    all    = rows + (headings ? [headings] : [])
+    budget = table_column_budgets(all, cols, percentage:)
+    wrap_row = ->(row) {
+      row.each_with_index.map { |cell, i|
+        cell.is_a?(Hash) and next cell
+        Kramdown::ANSI::Width.wrap(cell.to_s, length: budget[i], reflow: false)
+      }
+    }
+    result = rows.map(&wrap_row)
+    if headings
+      padded = headings.each_with_index.map { |h, i|
+        w = Kramdown::ANSI::Width.display_width(h)
+        w < budget[i] ? h + ' ' * (budget[i] - w) : h
+      }
+      result.unshift(padded)
+    end
+    result
+  end
+
+  # Computes a per-column display-width budget for a terminal table.
+  #
+  # @param rows [Array<Array<String, Hash>>] the rows to measure (Hash cells
+  #   are ignored)
+  # @param cols [Integer] the number of columns
+  # @param percentage [Numeric] the percentage of terminal width to use for
+  #   the total budget (default: 90)
+  # @return [Array<Integer>] the per-column width budget
+  def table_column_budgets(rows, cols, percentage: 90)
+    max_w = Array.new(cols, 0)
+    rows.each do |row|
+      row.each_with_index do |cell, i|
+        cell.is_a?(Hash) and next
+        w = cell.to_s.lines.map { |l| Kramdown::ANSI::Width.display_width(l) }.max || 0
+        max_w[i] = [max_w[i], w].max
+      end
+    end
+    total  = max_w.sum
+    budget = Kramdown::ANSI::Width.width(percentage:) - (3 * cols + 1)
+    return Array.new(cols, (budget.to_f / cols).to_i) if total.zero?
+    if total <= budget
+      extra = budget - total
+      max_w.map { |w| w + ((w.to_f / total) * extra).floor }
+    else
+      share  = (budget.to_f / cols).to_i
+      alloc  = max_w.map { |w| [w, share].min }
+      unused = budget - alloc.sum
+      return alloc if unused.zero?
+      capped = (0...cols).to_a.select { |i| alloc[i] < max_w[i] }
+      need   = capped.sum { |i| max_w[i] - alloc[i] }
+      return alloc if need.zero?
+      capped.each do |i|
+        extra = ((max_w[i] - alloc[i]).to_f / need * unused).floor
+        alloc[i] = [max_w[i], alloc[i] + extra].min
+      end
+      alloc
+    end
+  end
 end

data/lib/ollama_chat/session_management.rb

@@ -107,22 +107,23 @@ module OllamaChat::SessionManagement
         all_separators: true,
         border:         :unicode_round,
       }
-      table.headings = %w[ ID NAME SIZE #TOK COUNT UPDATED ].map { |header| bold { header } }
-      now = Time.now
+      headings = %w[ ID NAME SIZE #TOK COUNT UPDATED ].map { |h| bold { h } }
+      now      = Time.now
+      rows     = []
       models::Session.order(Sequel.desc(:updated_at)).each do |s|
-        name        = truncate(s.name, length: 32)
-        name        = session.id == s.id ? bold { name } : name
-        name        = if pid = s.locked?
-                        if pid == $$
-                          "#{name} 🔓"
-                        else
-                          "#{name} 🔐"
-                        end
-                      else
-                        name
-                      end
-        es = s.estimate_tokens
-        table << [
+        name   = s.name
+        name   = session.id == s.id ? bold { name } : name
+        name   = if pid = s.locked?
+                   if pid == $$
+                     "#{name} 🔓"
+                   else
+                     "#{name} 🔐"
+                   end
+                 else
+                   name
+                 end
+        es     = s.estimate_tokens
+        rows << [
           s.id.to_s,
           name,
           es.bytes_formatted,
@@ -131,6 +132,7 @@ module OllamaChat::SessionManagement
           s.age(now:),
         ]
       end
+      wrap_table_rows(rows, 6, headings:).each { |row| table << row }
       table.align_column 0, :right
       table.align_column 2, :left
       table.align_column 3, :right

data/lib/ollama_chat/source_fetching.rb

@@ -33,6 +33,7 @@ module OllamaChat::SourceFetching
   #
   # @yield [ tmp ]
   def fetch_source(source, check_exist: false, &block)
+    confirm_notification = false
     source = source.ask_and_send_or_self(:to_path).to_s
     case source
     when %r{\A!(.*)}
@@ -72,14 +73,28 @@ module OllamaChat::SourceFetching
     else
       raise "invalid source #{source.inspect}"
     end
+  rescue OllamaChat::HTTPError => e
+    msg = "Fetching source #{source.to_s.inspect}: #{e.class} #{e}"
+    feedback(msg, type: :warn)
+    log(:warn, msg, data: { source: source.to_s, status: e.status })
+    confirm_notification = true
+    msg = OllamaChat::Utils::Fetcher::ResponseMetadata.failed(msg)
+    msg.response_status = e.status
+    msg.response_reason = e.message
+    block.(msg)
+    msg
   rescue => e
     msg = "Fetching source #{source.to_s.inspect}: #{e.class} #{e}"
     feedback("#{msg}\n#{e.backtrace * ?\n}", type: :warn)
     log(:error, msg, data: { source: source.to_s, error: e.class })
-    confirm?(prompt: '⏎  Press any key to continue (%s). ', output: STDERR, timeout: 3)
+    confirm_notification = true
     msg = OllamaChat::Utils::Fetcher::ResponseMetadata.failed(msg)
     block.(msg)
     msg
+  ensure
+    if confirm_notification
+      confirm?(prompt: "\n⏎  Press any key to continue (%s). ", output: STDERR, timeout: 3)
+    end
   end
 
   # Adds an image to the images collection from the given source IO and source

data/lib/ollama_chat/tools/get_url.rb

@@ -116,6 +116,11 @@ class OllamaChat::Tools::GetURL
 
     source, message, content = url, nil, ''
     chat.fetch_source(source, check_exist: false) do |source_io|
+      if (status = source_io&.response_status) && status != 200
+        content = source_io.read
+        message = "Received HTTP status #{status} from #{source}."
+        next
+      end
       if source_io&.content_type&.to_s&.match?(OllamaChat::Parsing::HAS_AUDIO)
         text = chat.parse_audio(source_io, language:)
         source_io = OllamaChat::Utils::Fetcher::ResponseMetadata.as_text(text)

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.138'
+  VERSION         = '0.0.139'
   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.138 ruby lib
+# stub: ollama_chat 0.0.139 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "ollama_chat".freeze
-  s.version = "0.0.138".freeze
+  s.version = "0.0.139".freeze
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib".freeze]
@@ -41,7 +41,7 @@ Gem::Specification.new do |s|
   s.add_runtime_dependency(%q<redis>.freeze, ["~> 5.0".freeze])
   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<kramdown-ansi>.freeze, ["~> 0.7".freeze])
+  s.add_runtime_dependency(%q<kramdown-ansi>.freeze, ["~> 0.9".freeze])
   s.add_runtime_dependency(%q<complex_config>.freeze, ["~> 0.22".freeze, ">= 0.22.2".freeze])
   s.add_runtime_dependency(%q<tins>.freeze, ["~> 1.61".freeze])
   s.add_runtime_dependency(%q<search_ui>.freeze, [">= 0.2".freeze])

data/spec/ollama_chat/compaction_spec.rb

@@ -351,7 +351,7 @@ describe OllamaChat::Compaction do
         .with(a_string_matching(/Summarized:\s+12 messages/), type: :success)
       expect(chat).to receive(:speak)
         .with(a_string_matching(
-          /Compressed conversation context from \d+ to \d+ in \d+ (?:seconds|minutes and \d+ seconds)\./
+          /Compressed conversation context from \d+ percent to \d+ percent in \d+ (?:seconds|minutes and \d+ seconds)\./
         ))
 
       expect(chat.compact_with_retry).to be true

data/spec/ollama_chat/kramdown_ansi_spec.rb

@@ -1,8 +1,14 @@
 describe OllamaChat::KramdownANSI do
+  let :argv do
+    chat_default_config
+  end
+
   let :chat do
-    double('Chat').extend(described_class)
+    OllamaChat::Chat.new(argv:).expose
   end
 
+  connect_to_ollama_server
+
   describe '#configure_kramdown_ansi_styles', protect_env: true do
     it 'can be configured via env var' do
       const_conf_as(
@@ -12,7 +18,6 @@ describe OllamaChat::KramdownANSI do
       expect(Kramdown::ANSI::Styles).to receive(:from_json).
         with('{"foo":"bar"}').
         and_return(double(ansi_styles: styles))
-
       expect(chat.configure_kramdown_ansi_styles).to eq(styles)
     end
 
@@ -100,4 +105,51 @@ describe OllamaChat::KramdownANSI do
       expect(result).to include('**bold**')
     end
   end
+
+  describe '#wrap_table_rows' do
+    it 'wraps long cells to fit within terminal budget' do
+      long_text = 'word ' * 50
+      rows      = [[ long_text, 'short' ]]
+      result    = chat.wrap_table_rows(rows, 2)
+      expect(result[0][0]).to include("\n")
+    end
+
+    it 'passes Hash cells (colspan) through untouched' do
+      rows   = [[ { value: 'category', colspan: 4 } ]]
+      result = chat.wrap_table_rows(rows, 4)
+      expect(result).to eq(rows)
+    end
+
+    it 'returns wrapped headings first when supplied' do
+      rows   = [[ 'a', 'b' ]]
+      result = chat.wrap_table_rows(rows, 2, headings: ['H1', 'H2'])
+      expect(result.first.map(&:strip)).to eq(['H1', 'H2'])
+      expect(result.size).to eq(2)
+    end
+
+    it 'handles nil cells by converting to empty string' do
+      result = chat.wrap_table_rows([[ nil, 'text' ]], 2)
+      expect(result[0][0]).to eq('')
+    end
+
+    it 'preserves short content without wrapping' do
+      rows   = [[ 'hi', 'there' ]]
+      result = chat.wrap_table_rows(rows, 2)
+      expect(result[0]).to eq(['hi', 'there'])
+    end
+  end
+
+  describe '#table_column_budgets' do
+    it 'returns equal budgets when total width is zero' do
+      result = chat.table_column_budgets([[ '', '' ]], 2)
+      expect(result[0]).to eq(result[1])
+    end
+
+    it 'allocates proportionally based on content width' do
+      narrow = 'a'
+      wide   = 'a' * 20
+      result = chat.table_column_budgets([[ narrow, wide ]], 2)
+      expect(result[1]).to be > result[0]
+    end
+  end
 end

data/spec/ollama_chat/tools/get_url_spec.rb

@@ -109,7 +109,7 @@ describe OllamaChat::Tools::GetURL do
 
   context 'with different document policies' do
     let(:url) { 'https://www.example.com/foo' }
-    let(:source_io) { double('SourceIO', content_type: double(media_type: 'text'), read: 'raw content') }
+    let(:source_io) { double('SourceIO', content_type: double(media_type: 'text'), read: 'raw content', response_status: nil) }
 
     before do
       expect(chat).to receive(:fetch_source).and_yield(source_io)
@@ -191,7 +191,7 @@ describe OllamaChat::Tools::GetURL do
     let(:url) { 'https://www.example.com/foo' }
 
     it 'handles image content types' do
-      source_io = double('SourceIO', content_type: double(media_type: 'image', sub_type: 'jpeg'))
+      source_io = double('SourceIO', content_type: double(media_type: 'image', sub_type: 'jpeg'), response_status: nil)
       expect(chat).to receive(:fetch_source).and_yield(source_io)
       expect(chat).to receive(:add_image).with(chat.images, source_io, URI.parse(url))
 
@@ -222,7 +222,7 @@ describe OllamaChat::Tools::GetURL do
     end
 
     it 'handles audio content types via ASR transcription' do
-      source_io = double('SourceIO', content_type: double(media_type: 'audio', to_s: 'audio/mp3'))
+      source_io = double('SourceIO', content_type: double(media_type: 'audio', to_s: 'audio/mp3'), response_status: nil)
       expect(chat).to receive(:fetch_source).and_yield(source_io)
       expect(chat).to receive(:parse_audio).with(source_io, language: 'de').and_return('transcribed text')
 
@@ -235,7 +235,7 @@ describe OllamaChat::Tools::GetURL do
     end
 
     it 'handles video content types via ASR transcription' do
-      source_io = double('SourceIO', content_type: double(media_type: 'video', to_s: 'video/mp4'))
+      source_io = double('SourceIO', content_type: double(media_type: 'video', to_s: 'video/mp4'), response_status: nil)
       expect(chat).to receive(:fetch_source).and_yield(source_io)
       expect(chat).to receive(:parse_audio).with(source_io, language: nil).and_return('video text')
 
@@ -248,7 +248,7 @@ describe OllamaChat::Tools::GetURL do
     end
 
     it 'handles unsupported media types' do
-      source_io = double('SourceIO', content_type: double(media_type: 'font'))
+      source_io = double('SourceIO', content_type: double(media_type: 'font'), response_status: nil)
       expect(chat).to receive(:fetch_source).and_yield(source_io)
 
       args = double(url:, document_policy: 'ignoring', words: nil, instruction: nil, language: nil)
@@ -258,5 +258,24 @@ describe OllamaChat::Tools::GetURL do
       json = json_object(result)
       expect(json.message).to match(/Cannot fetch.*with content type/)
     end
+
+    it 'short-circuits on non-200 HTTP status' do
+      source_io = double(
+        'FailedSourceIO',
+        response_status: 403,
+        read: 'Access Denied',
+      )
+      expect(chat).to receive(:fetch_source).and_yield(source_io)
+      expect(chat).not_to receive(:summarize_source)
+      expect(chat).not_to receive(:import_source)
+
+      args = double(url:, document_policy: 'summarizing', words: nil, instruction: nil, language: nil)
+      tool_call = double(function: double(arguments: args))
+
+      result = described_class.new(chat).execute(tool_call)
+      json = json_object(result)
+      expect(json.content).to eq('Access Denied')
+      expect(json.message).to match(/HTTP status 403/)
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ollama_chat
 version: !ruby/object:Gem::Version
-  version: 0.0.138
+  version: 0.0.139
 platform: ruby
 authors:
 - Florian Frank
@@ -253,14 +253,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.7'
+        version: '0.9'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.7'
+        version: '0.9'
 - !ruby/object:Gem::Dependency
   name: complex_config
   requirement: !ruby/object:Gem::Requirement