ollama_chat 0.0.140 → 0.0.141

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c1cff963a5116fbe9c1a4a94fff41df7128c5f31c4daaaf96e2a4334c97d77d5
-  data.tar.gz: 97ec4d9f4f90c71b4bc2eee13f2000bb4f8fb7ad6b9fe308c82342734e3d1629
+  metadata.gz: 2e928fba1669bde0639cbaba045d872c71f3630dc5ca2b00fec128dfe1217836
+  data.tar.gz: e781850be5059fd1bc79ba476557c49841177ce1965106d538f689b80a7fba70
 SHA512:
-  metadata.gz: f1abfdfaab1002d8503a7a72b204ee247f2375df5e6b3de417c483826ad88208ceca28c73921afcdd0b8ec2575e15c4cf9ade009fd7aab99b18a707a9451891f
-  data.tar.gz: ddd78c34605c7035c0d24e9ba76e59a28edd8a85f2116478b97474e4233fb47077b297134814a3e829abd4af43d343a6ce5ca191a33cc46e174d46664186dbce
+  metadata.gz: 2adcab92a8314bf93d0e322e63413321c49a85f1ecd8d6df63a70b32ed5a98bd1df6c5996302c1e2cdd4eed74d4bde4c6213596f3d16ab543c7803b29f94726f
+  data.tar.gz: 289d236ebec4be3474931ca20eeeebb8161f57260638cd19f366145fd08eb0be24b261cf1c1037b2c064c5133ca5cf209df39d937125b92903877878865c4cbe

data/CHANGES.md

@@ -1,5 +1,39 @@
 # Changes
 
+## 2026-11-06 v0.0.141
+
+### Improvements
+
+- Replaced `Tins::Duration#to_s` with `to_human(style: :abbrev)` in
+  `follow_chat.rb`, `session_management.rb`, `tool_call_processor.rb`, and
+  `generate_image.rb` to produce compact duration strings (e.g., `45s`,
+  `1m30s`) for terminal stats and JSON results.
+- Replaced hand-rolled duration formatting in `compaction.rb` with
+  `Tins::Duration#to_human` for compaction TTS reports.
+- Updated `execute_shell` tool description to instruct the LLM to separate
+  multi-command scripts with newlines and indentation rather than semicolons
+  for improved readability in the review editor.
+- Added an early-return guard in `ToolCallProcessor#handle` to skip tool
+  execution when `chat.tools_support.off?`, emitting a `tools_disabled` JSON
+  result per tool call to provide proper error feedback to the LLM.
+
+### Fixes
+
+- Set `default: false` for `get_time` and `get_location` tools in
+  `default_config.yml`, as these values are already injected into runtime info
+  each turn, making default enablement redundant.
+
+### Build & CI
+
+- Bumped `tins` dependency to **~> 1.62** in `Rakefile` and
+  `ollama_chat.gemspec`.
+
+### Testing
+
+- Added a spec example in `tool_call_processor_spec.rb` verifying that
+  `#handle` returns the expected `tools_disabled` error and message when tool
+  support is disabled.
+
 ## 2026-11-02 v0.0.140
 
 ### Improvements

data/Rakefile

@@ -49,7 +49,7 @@ GemHadar do
   dependency             'reverse_markdown',      '~> 3.0'
   dependency             'kramdown-ansi',         '~> 0.9'
   dependency             'complex_config',        '~> 0.22', '>= 0.22.2'
-  dependency             'tins',                  '~> 1.61'
+  dependency             'tins',                  '~> 1.62'
   dependency             'search_ui',             '>= 0.2'
   dependency             'amatch',                '~> 0.4'
   dependency             'pdf-reader',            '~> 2.0'

data/lib/ollama_chat/compaction.rb

@@ -322,10 +322,7 @@ module OllamaChat::Compaction
       Context:      #{result.context_before} → #{result.context_after}
       Stored total: #{result.stored_total}
     EOT
-    duration        = Tins::Duration.new(duration)
-    duration_string = duration.format(
-      duration.minutes? ? '%M minutes and %u seconds' : '%u seconds'
-    )
+    duration_string = Tins::Duration.new(duration).to_human
     speak(
       'Compressed conversation context from %.0f percent to %.0f percent in %s.' % [
         100 * result.context_filled_before,

data/lib/ollama_chat/follow_chat.rb

@@ -245,14 +245,14 @@ class OllamaChat::FollowChat
     eval_duration        = response.eval_duration.to_f / 1e9
     prompt_eval_duration = response.prompt_eval_duration.to_f / 1e9
     {
-      eval_duration:        Tins::Duration.new(eval_duration).to_s,
+      eval_duration:        Tins::Duration.new(eval_duration).to_human(style: :abbrev),
       eval_count:           response.eval_count.to_i,
       eval_rate:            "%.2f t/s" % (response.eval_count.to_i / eval_duration),
-      prompt_eval_duration: Tins::Duration.new(prompt_eval_duration).to_s,
+      prompt_eval_duration: Tins::Duration.new(prompt_eval_duration).to_human(style: :abbrev),
       prompt_eval_count:    response.prompt_eval_count.to_i,
       prompt_eval_rate:     "%.2f t/s" % (response.prompt_eval_count.to_i / prompt_eval_duration),
-      total_duration:       Tins::Duration.new(response.total_duration / 1e9).to_s,
-      load_duration:        Tins::Duration.new(response.load_duration / 1e9).to_s,
+      total_duration:       Tins::Duration.new(response.total_duration / 1e9).to_human(style: :abbrev),
+      load_duration:        Tins::Duration.new(response.load_duration / 1e9).to_human(style: :abbrev),
     }
   end
 

data/lib/ollama_chat/ollama_chat_config/default_config.yml

@@ -582,10 +582,10 @@ tools:
       require_confirmation: true
     get_time:
       result_display_timeout: 3
-      default: true
+      default: false
     get_location:
       result_display_timeout: 3
-      default: true
+      default: false
     list_directory:
       default: true
       require_confirmation: false

data/lib/ollama_chat/session_management.rb

@@ -737,7 +737,7 @@ module OllamaChat::SessionManagement
     elsif selector
       now = Time.now
       sessions = session_query.order(Sequel.desc(:updated_at)).map { |session|
-        duration = session.age(now:)
+        duration = session.age(now:).to_human(style: :abbrev)
         es       = session.estimate_tokens
         count    = session.count_messages
         locked   = if pid = session.locked?

data/lib/ollama_chat/tool_call_processor.rb

@@ -33,6 +33,19 @@ class OllamaChat::ToolCallProcessor
   def handle(response)
     return unless response.message.ask_and_send(:tool_calls)
 
+    if chat.tools_support.off?
+      response.message.tool_calls.each do |tool_call|
+        name = tool_call.function.name
+        @results[name] << JSON(
+          error:  'tools_disabled',
+          message: 'Tool support is disabled. ' \
+                   'Ask the user to re-enable with `/tools on`.'
+        )
+        chat.log(:warn, 'Tool call ignored: tools disabled',
+                 data: { tool: name })
+      end
+      return
+    end
     tools_used = {}
 
     response.message.tool_calls.each do |tool_call|
@@ -108,7 +121,7 @@ class OllamaChat::ToolCallProcessor
         warn:     ,
         size:     es.bytes_formatted,
         tokens:   es.tokens_formatted,
-        duration: Tins::Duration.new(Time.now - start).to_s,
+        duration: Tins::Duration.new(Time.now - start).to_human(style: :abbrev),
       }
     end
 

data/lib/ollama_chat/tools/execute_shell.rb

@@ -45,6 +45,9 @@ class OllamaChat::Tools::ExecuteShell
           then a mandatory y/n/i/a confirm prompt gates execution.
           Answering a remembers the command for the session
           (skips editor and confirm on repeat).
+          For multi-command scripts, separate commands with newlines
+          and proper intendation rather than `;` for readability
+          in the review editor.
           Last resort: use ONLY when no dedicated tool covers
           the task (e.g. `git commit`, `bundle install`, `make`).
           Prefer `execute_grep`, `list_directory`, `read_file`,

data/lib/ollama_chat/tools/generate_image.rb

@@ -101,7 +101,7 @@ class OllamaChat::Tools::GenerateImage
       message:  ,
       url:      view_url,
       prompt:   ,
-      duration: Tins::Duration.new(Time.now - started).to_s,
+      duration: Tins::Duration.new(Time.now - started).to_human(style: :abbrev),
     }.to_json
   rescue => e
     chat.log(:error, e, data: { tool: name, prompt: })

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.140'
+  VERSION         = '0.0.141'
   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.140 ruby lib
+# stub: ollama_chat 0.0.141 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "ollama_chat".freeze
-  s.version = "0.0.140".freeze
+  s.version = "0.0.141".freeze
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib".freeze]
@@ -43,7 +43,7 @@ Gem::Specification.new do |s|
   s.add_runtime_dependency(%q<reverse_markdown>.freeze, ["~> 3.0".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<tins>.freeze, ["~> 1.62".freeze])
   s.add_runtime_dependency(%q<search_ui>.freeze, [">= 0.2".freeze])
   s.add_runtime_dependency(%q<amatch>.freeze, ["~> 0.4".freeze])
   s.add_runtime_dependency(%q<pdf-reader>.freeze, ["~> 2.0".freeze])

data/spec/ollama_chat/tool_call_processor_spec.rb

@@ -48,4 +48,31 @@ describe OllamaChat::ToolCallProcessor do
       expect(processor.results).to be_empty
     end
   end
+
+  describe '#handle' do
+    let :tool_call do
+      double('tool_call', function: double('function', name: 'roll_dice'))
+    end
+
+    let :message do
+      double('message', ask_and_send: true, tool_calls: [tool_call])
+    end
+
+    let :response do
+      double('response', message:)
+    end
+
+    context 'when tools support is disabled' do
+      before do
+        allow(chat.tools_support).to receive(:off?).and_return(true)
+      end
+
+      it 'returns a tools_disabled result per tool call without executing' do
+        processor.handle(response)
+        data = JSON.parse(processor.results['roll_dice'].first)
+        expect(data['error']).to eq('tools_disabled')
+        expect(data['message']).to include('/tools on')
+      end
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ollama_chat
 version: !ruby/object:Gem::Version
-  version: 0.0.140
+  version: 0.0.141
 platform: ruby
 authors:
 - Florian Frank
@@ -287,14 +287,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.61'
+        version: '1.62'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.61'
+        version: '1.62'
 - !ruby/object:Gem::Dependency
   name: search_ui
   requirement: !ruby/object:Gem::Requirement