ollama_chat 0.0.126 → 0.0.127

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 93612bbed949bff6bad6cf89f581b324165c0aa672f6f92b675b63a6a2dce7e0
-  data.tar.gz: 79aac9f7dada8df87035af5bdfd2913d16012af44a781b7a20c44a457163b313
+  metadata.gz: a72761f0d0b34c0ba21859589410e914dda692d669a39629e4e6bb8f6e2857ea
+  data.tar.gz: 374fa95c62c5510e94c6a7624200a6193a15e9ed842a4b49b5364d43eeecac52
 SHA512:
-  metadata.gz: 2eb715b50338ddb686e6aab7debd264591e1018216adef57146237a862a5f37f217a31bb481a724b4bb8fcc99144c1a40679e3048c64cb88f0eaaabd546661fc
-  data.tar.gz: 8fa30b91ba125939cd506078bcaf6c9549bb43aa6323f9186e0acae218c7814fd8add54aa75b39f907d4affc78889c52a45b2479ec57c4218d76e61e67843ed3
+  metadata.gz: ea024b4607101da7037f0234970cb6e655c6d2b1af46fc1eac1b8dc901aebd5ef95a091d079653bfb58e86d381c4bb56173172f9fb0861e9d8a3424667bfdd32
+  data.tar.gz: dfbbe6cddbb4aac1f047cb39719ed58e8920e22a42775cb9f18a4d3731d0c53cc7c632733241f58f6cd6c132e5979b0a3117bf38e7e0095417310e839fb7421b

data/CHANGES.md

@@ -1,5 +1,67 @@
 # Changes
 
+## 2026-10-06 v0.0.127
+
+## Changes
+
+*   **Switches and Configuration**
+    *   Added `memory_trigger` switch, registered in `/toggle` (regexp,
+        completion list, help string, and fallback feedback).
+    *   Refactored `Switch` to accept callable values alongside plain values;
+        plain values are now wrapped via a new private `adhoc_closure` method
+        using a `UNSET` sentinel for read/write disambiguation.
+    *   Hardened `Switch#initialize` with an arity guard: callable values
+        responding to `:arity` must return `-1` (optional single parameter
+        defaulting to `UNSET`), otherwise a `TypeError` is raised.
+    *   Updated `set` methods in `OllamaChat::Switches` to explicitly return
+        `nil` for consistent behavior.
+    *   Added `@raise [TypeError]` tag to `Switch#initialize` and expanded
+        `@param value` YARD documentation for `CombinedSwitch`.
+    *   Added arity guard (`ask_and_send(:arity) <= 0`) to
+        `CombinedSwitch#initialize`.
+
+*   **Session and Model Updates**
+    *   Added `trigger_enabled?` and `set_trigger_enabled` to the `Session`
+        model to encapsulate trigger hash shape.
+    *   Added YARD documentation for `adhoc_closure` and the `:memory_trigger`
+        reader.
+
+*   **Conversation and Compaction**
+    *   Added `/conversation compact summary` command.
+    *   Introduced `show_compaction_summary` in `Compaction` to retrieve the
+        most recent summary message via `messages.find_summary` and render its
+        content (narrative + `tool_calls` lookup index) through the pager,
+        prefixed with a size header and entry count.
+    *   Registered the new `compact summary` subcommand in the `/conversation`
+        regexp alternation, completion list, help text, and dispatch block.
+    *   Excluded system-injected messages (`runtime_information`,
+        `trigger_inject`) from the compaction summary's `tool_calls` block to
+        keep the index focused on user-initiated tool calls.
+
+*   **RAG and Search**
+    *   Moved `set_documents_collection` above the `private` keyword in
+        `rag_handling.rb` to make it callable from outside `RAGHandling`.
+    *   Updated `choose_collection` to call `set_documents_collection` instead
+        of assigning `@documents.collection` directly.
+    *   Replaced direct `chat.documents.collection` reads/writes in `chat.rb`
+        and `search_knowledge.rb` with public accessors `chat.collection` and
+        `chat.set_documents_collection`.
+    *   Consolidated two `chat.log` calls in `search_knowledge.rb` into one,
+        adding `rerank:` and `pre_rerank:` to the structured data hash with
+        `.compact`.
+    *   Removed `String#%` interpolation in the old rerank log line and
+        eliminated the `collection_name` local variable.
+    *   Updated `trigger_inject` in `collection_search.rb` to include the
+        surviving `records` array (text, similarity, tags) in the log `data:`
+        hash instead of only logging the count.
+
+*   **Shell Execution**
+    *   Forced pager for all `execute_shell` output by passing `force: true` to
+        `use_pager`, ensuring consistent UX with other pager-based tools.
+    *   Updated test-mode spec examples to include a `stub_success` call with
+        sample stdout to exercise the always-active pager path with real
+        content.
+
 ## 2026-10-05 v0.0.126
 
 *   **Tool Registry Refactor**: Refactored the tool registry to store tool

data/lib/ollama_chat/chat.rb

@@ -712,7 +712,7 @@ class OllamaChat::Chat
   # @note Progress is reported to STDOUT during processing
   def add_documents_from_argv(document_list)
     if document_list.any?(&:empty?)
-      feedback("Clearing collection #{bold{documents.collection}}.", type: :info)
+      feedback("Clearing collection #{bold{collection}}.", type: :info)
       documents.clear
       document_list.reject!(&:empty?)
     end
@@ -724,7 +724,7 @@ class OllamaChat::Chat
           File.expand_path(doc)
         end
       end
-      feedback("Collection #{bold{documents.collection}}: Adding #{document_list.size} documents…", type: :info)
+      feedback("Collection #{bold{collection}}: Adding #{document_list.size} documents…", type: :info)
       count = 1
       document_list.each_slice(25) do |docs|
         docs.each do |doc|

data/lib/ollama_chat/collection_search.rb

@@ -130,8 +130,20 @@ module OllamaChat::CollectionSearch
       records.empty? and return
       pre_rerank = records.size
       records    = rerank_records(content, records, prompt_name:)
-      log(:info, 'Trigger: %d/%d passed rerank for %s' % [ records.size, pre_rerank, collection ],
-          data: { collection:, prompt_name: })
+      log(
+        :info,
+        'Trigger: %d/%d passed rerank for %s' % [ records.size, pre_rerank, collection ],
+        data: {
+          collection:,
+          prompt_name:,
+          records: records.map { |r|
+            {
+              text:       r.text,
+              similarity: r.similarity&.round(4),
+              tags:       r.tags_set.to_s(link: false)
+            }
+          }
+        })
       records.empty? and return
     end
     snippets = records.map { |record|

data/lib/ollama_chat/commands.rb

@@ -161,13 +161,13 @@ module OllamaChat::Commands
 
   command(
     name: :toggle,
-    regexp: %r(^/toggle(?:\s+(markdown|stream|location|runtime_info|voice|think_loud|think_strip|embedding)(?:\s+(-[yn]))?)?$),
-    complete: [ 'toggle', %w[ markdown stream location runtime_info voice think_loud think_strip embedding ] ],
+    regexp: %r(^/toggle(?:\s+(markdown|stream|location|runtime_info|voice|think_loud|think_strip|embedding|memory_trigger)(?:\s+(-[yn]))?)?$),
+    complete: [ 'toggle', %w[ markdown stream location runtime_info voice think_loud think_strip embedding memory_trigger ] ],
     options: '[-y|-n]',
     help: <<~EOT
       🎛️ Toggle feature switches
-      (markdown, stream, location, runtime_info,
-      voice, think_loud, think_strip, embedding)
+       (markdown, stream, location, runtime_info,
+       voice, think_loud, think_strip, embedding, memory_trigger)
       Options: -y (on), -n (off)
     EOT
   ) do |toggle_name, flag|
@@ -191,7 +191,7 @@ module OllamaChat::Commands
       end
     else
       feedback(
-        "Available toggles: markdown|stream|location|runtime_info|voice|think_loud|think_strip|embedding"
+        "Available toggles: markdown|stream|location|runtime_info|voice|think_loud|think_strip|embedding|memory_trigger"
       )
     end
     :next
@@ -391,14 +391,15 @@ module OllamaChat::Commands
 
   command(
     name: :conversation,
-    regexp: %r(^/conversation\s+(clean|compact|save|load|summarize|report)((?:\s+-[sc])*)(?:\s+([^-].*\.jsonl?))?$),
-    complete: [ 'conversation', %w[ save load clean compact summarize report ] ],
+    regexp: %r(^/conversation\s+(clean|compact summary|compact|save|load|summarize|report)((?:\s+-[sc])*)(?:\s+([^-].*\.jsonl?))?$),
+    complete: [ 'conversation', %w[ compact clean report compact\ summary summarize save load ] ],
     options: '[-s|-c] [FILENAME]',
     help: <<~EOT
       💾 Manage conversation content:
          - save/load: Export/import as .json or .jsonl
          - clean: Remove tool content, images, thinking
          - compact: Summarize old messages, keep recent
+         - compact summary: Show last compaction summary
          - summarize: Per-message narrative (-s sentence)
          - report: Generate a session report document
     EOT
@@ -426,6 +427,8 @@ module OllamaChat::Commands
       else
         feedback("Denied.", type: :denied)
       end
+    when 'compact summary'
+      show_compaction_summary
     when 'compact'
       compact_with_retry
     when 'summarize'

data/lib/ollama_chat/compaction.rb

@@ -134,6 +134,33 @@ module OllamaChat::Compaction
     false
   end
 
+  # Displays the most recent compaction summary in the pager.
+  #
+  # Shows the narrative and the tool-call lookup index (the
+  # `lookup_group` entries with short UUIDs) carried in the summary
+  # message, prefixed with size metadata.
+  #
+  # @return [nil] always returns nil (command block handles :next).
+  def show_compaction_summary
+    unless msg = messages.find_summary
+      feedback('No compaction summary yet.', type: :info)
+      return
+    end
+    es = OllamaChat::TokenEstimator.estimate(msg.content.to_s)
+    use_pager do |output|
+      output.puts <<~EOT
+        #{bold{'Compaction Summary'}}
+        #{'─' * Tins::Terminal.columns}
+
+        #{kramdown_ansi_parse(msg.content.to_s)}
+
+        #{'─' * Tins::Terminal.columns}
+        Size:   👾#{es.bytes_formatted} 🧩#{es.tokens_formatted}
+        Tools:  #{msg.tool_calls&.size || 0} lookup_group entries
+      EOT
+    end
+  end
+
   private
 
   # Serializes non-system messages into a readable block for the prompt.
@@ -172,7 +199,7 @@ module OllamaChat::Compaction
   def build_tool_entries(messages)
     messages.filter_map do |msg|
       next unless tool_name = msg.tool_name.full?
-      next if tool_name == 'runtime_information'
+      next if %w[ runtime_information trigger_inject ].include?(tool_name)
       uuid = msg.group_uuid.to_s[-8..]
       summary = OllamaChat::Compaction.tool_summary_line(
         tool_name, msg.content.to_s

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

@@ -65,6 +65,27 @@ class OllamaChat::Database::Models::Session < Sequel::Model(OllamaChat::DB)
     messages.to_s.count(?\n)
   end
 
+  # Returns the `enabled` flag of the first configured trigger entry.
+  #
+  # @return [Boolean, nil] true when the trigger is active, false when
+  #   explicitly disabled, nil when no trigger is configured
+  def trigger_enabled?
+    trigger&.values&.first&.[]('enabled')
+  end
+
+  # Sets the `enabled` flag of the first configured trigger entry and
+  # persists the change.
+  #
+  # No-op when no trigger is configured.
+  #
+  # @param value [Boolean] the new enabled state
+  def set_trigger_enabled(value)
+    if hash = trigger&.values&.first
+      hash['enabled'] = value
+      save
+    end
+  end
+
   # @!attribute [v] id
   #   @return [Integer] The primary key for the session.
   #

data/lib/ollama_chat/rag_handling.rb

@@ -66,6 +66,14 @@ module OllamaChat::RAGHandling
     @documents.collection
   end
 
+  # Sets the documents collection to collection.
+  #
+  # @param collection [String, Symbol] the name of the collection to set
+  # @return [String, Symbol] the newly set collection name
+  def set_documents_collection(collection)
+    @documents.collection = collection
+  end
+
   # Retrieves a hash of collection names and their descriptions from the
   # database.
   #
@@ -264,14 +272,6 @@ module OllamaChat::RAGHandling
     end
   end
 
-  # Sets the documents collection to collection.
-  #
-  # @param collection [String, Symbol] the name of the collection to set
-  # @return [String, Symbol] the newly set collection name
-  def set_documents_collection(collection)
-    @documents.collection = collection
-  end
-
   # The choose_collection method presents a menu to select or create a document
   # collection. It displays existing collections along with options to create a
   # new one or exit.
@@ -290,12 +290,12 @@ module OllamaChat::RAGHandling
     case collection = collection&.to_s
     when '[NEW]'
       if name = create_collection
-        @documents.collection = name
+        set_documents_collection(name)
       end
     when nil, '[EXIT]'
       feedback("Exiting chooser.")
     when /./
-      @documents.collection = collection
+      set_documents_collection(collection)
     end
   ensure
     if collection

data/lib/ollama_chat/switches.rb

@@ -99,31 +99,70 @@ module OllamaChat::Switches
   #   switch.off?    # Returns false
   #   switch.on?     # Returns true
   class Switch
+    UNSET = Object.new.freeze
+
     # Initializes a new Switch instance.
     #
     # @param msg [Hash{Boolean => String}] a hash containing true and false messages
-    # @param value [Object] the initial state of the switch (coerced to boolean)
+    # @param value [Object, Proc] the initial state of the switch. A plain
+    #   value is coerced to a boolean and wrapped in an internal closure;
+    #   a callable is used directly and must be invokable both bare (read)
+    #   and with an explicit boolean (write). If the callable responds to
+    #   `arity`, it must return -1 (a `TypeError` is raised otherwise);
+    #   callables without an `arity` method are accepted as-is.
     # @param callbacks [Hash{[Boolean, Boolean] => Proc}] optional mapping of
     #   state transitions to callback procs. Keys are `[old_value, new_value]`.
+    # @raise [TypeError] if `value` is a callable that responds to `arity`
+    #   and the arity is not -1
     def initialize(msg:, value:, callbacks: {})
-      @value     = !!value
+      if value.respond_to?(:call)
+        if arity = value.ask_and_send(:arity)
+          arity != -1 and raise TypeError, 'value Proc requires arity -1'
+        end
+        @value = value
+      else
+        @value = adhoc_closure(value)
+      end
       @msg       = msg
       @callbacks = callbacks.to_h
     end
 
+    # Wraps a plain (non-callable) value in a closure so that every switch
+    # state is stored and accessed uniformly via a `Proc`.
+    #
+    # The returned closure reads the coerced (`!!`) boolean state when called
+    # with no argument (or the `UNSET` sentinel) and rebinds the captured
+    # `original_value` when called with an explicit value.
+    #
+    # @param original_value [Object] the initial value to be coerced to a
+    #   boolean on read
+    # @return [Proc] a closure that reads or writes the switch state
+    def adhoc_closure(original_value)
+      -> new_value = UNSET {
+        if new_value == UNSET
+          !!original_value
+        else
+          original_value = new_value
+        end
+      }
+    end
+    private :adhoc_closure
+
     # @!attribute [r] value
     #   @return [Boolean] the current state of the switch
-    attr_reader :value
+    def value
+      @value.()
+    end
 
     # Assigns a boolean value to the switch and optionally displays the result.
     #
     # @param value [Object] the value to be coerced to a boolean and assigned
     # @param show [Boolean] determines whether to display the status message
     # @param output [IO] the output stream to write the message to
-    # @return [String, nil, Boolean] the result of the display operation or the show flag
     def set(value, show: false, output: STDOUT)
-      @value = !!value
+      @value.(value)
       show && self.show(output:)
+      nil
     end
 
     # Toggles the current boolean value and optionally displays the result.
@@ -131,7 +170,7 @@ module OllamaChat::Switches
     # @param show [Boolean] determines whether to show the value after toggling
     # @return [String, nil, Boolean] the result of the display operation or the show flag
     def toggle(show: true)
-      @value = !@value
+      @value.(!@value.())
       show && self.show
     end
 
@@ -175,10 +214,10 @@ module OllamaChat::Switches
     # @param value [Object] the value to be coerced to a boolean and saved
     # @param show [Boolean] whether to show the updated value
     # @param output [IO] the output stream to use when showing the value
-    # @return [String, nil, Boolean] the result of the display operation or the show flag
     def set(value, show: false, output: STDOUT)
       @chat.session.update("#{attribute}": !!value)
       show && self.show(output:)
+      nil
     end
 
     # Toggles the value of a session attribute and optionally displays the new state.
@@ -207,9 +246,18 @@ module OllamaChat::Switches
   class CombinedSwitch
     # Initializes a new CombinedSwitch instance.
     #
-    # @param value [Proc] the proc used to determine the current state
-    # @param msg [Hash{Boolean => String}] the message hash containing true and false keys
+    # @param value [Proc] the proc used to determine the current state.
+    #   Must be invokable with no arguments. If the proc responds to
+    #   `arity`, it must return 0 or -1 (a `TypeError` is raised
+    #   otherwise); procs without an `arity` method are accepted as-is.
+    # @param msg [Hash{Boolean => String}] the message hash containing
+    #   true and false keys
+    # @raise [TypeError] if `value` responds to `arity` and returns a
+    #   value greater than 0 (the proc must be invokable with no args)
     def initialize(value:, msg:)
+      if arity = value.ask_and_send(:arity)
+        arity <= 0 or raise TypeError, 'value Proc requires arity <= 0'
+      end
       @value = value
       @msg   = msg
     end
@@ -279,6 +327,14 @@ module OllamaChat::Switches
   #   @return [OllamaChat::Switches::DatabaseSwitch] the tools support setting switch
   attr_reader :tools_support
 
+  # Controls whether the session's memory trigger is active. The enabled
+  # state is read from and written to the session's `trigger` hash; the
+  # switch is a no-op when no trigger is configured.
+  #
+  # @!attribute [r] memory_trigger
+  #   @return [OllamaChat::Switches::Switch] the memory trigger switch instance
+  attr_reader :memory_trigger
+
   # Initializes the various switches for configuring the application's behavior.
   #
   # This method creates and configures the database and local switch objects
@@ -387,5 +443,19 @@ module OllamaChat::Switches
         false => "Tools support disabled.",
       }
     )
+    @memory_trigger = Switch.new(
+      value: -> new_value = OllamaChat::Switches::Switch::UNSET {
+        if new_value == OllamaChat::Switches::Switch::UNSET
+          chat.session.trigger_enabled?
+        else
+          chat.session.set_trigger_enabled(new_value)
+          nil
+        end
+      },
+      msg: {
+        true  => "Memory trigger active.",
+        false => "Memory trigger inactive.",
+      },
+    )
   end
 end

data/lib/ollama_chat/tools/execute_shell.rb

@@ -139,7 +139,7 @@ class OllamaChat::Tools::ExecuteShell
 
   # Display command output through the pager.
   def display(stdout, stderr, exit_code)
-    chat.use_pager do |io|
+    chat.use_pager(force: true) do |io|
       if stdout.present?
         io.puts bold { "stdout:" }, ""
         io.puts stdout

data/lib/ollama_chat/tools/search_knowledge.rb

@@ -123,28 +123,29 @@ class OllamaChat::Tools::SearchKnowledge
         raise OllamaChat::ToolFunctionArgumentError,
           "Collection #{collection} is disabled."
       end
-      old_collection            = chat.documents.collection
-      chat.documents.collection = collection
+      old_collection = chat.collection
+      chat.set_documents_collection(collection)
     end
 
     records = chat.find_document_records(query, tags:, text_size:, text_count:, min_similarity:)
 
+    pre_rerank  = records.size
+    prompt_name = 'default'
     if rerank && records.any?
-      pre_rerank  = records.size
-      prompt_name = 'default'
       records = chat.rerank_records(query, records, prompt_name:)
-      chat.log(:info, 'Tool: %d/%d passed rerank for %s' % [ records.size, pre_rerank, collection ],
-          data: { collection:, prompt_name: })
     end
 
     chat.log(:info, "Snippets retrieved", data: {
-      tool: name, collection: chat.documents.collection, hits: records.size
-    })
+      tool: name,
+      collection: chat.collection,
+      rerank:,
+      pre_rerank:,
+      hits: records.size
+    }.compact)
 
-    collection_name = chat.documents.collection
     message =
       if records.any?
-        "Retrieved #{records.size} relevant snippets from collection #{collection_name.inspect} for query #{query.inspect}. See snippets below:\n\n" +
+        "Retrieved #{records.size} relevant snippets from collection #{chat.collection.inspect} for query #{query.inspect}. See snippets below:\n\n" +
           records.map { |record|
             link = if record.source =~ %r(\Ahttps?://)
                      record.source
@@ -155,14 +156,14 @@ class OllamaChat::Tools::SearchKnowledge
             [ link, ?# + record.tags.first ]
           }.flat_map { |l, t| chat.hyperlink(l, t) }.join(' ')
       else
-        "No relevant snippets found for query #{query.inspect} in collection #{collection_name.inspect}."
+        "No relevant snippets found for query #{query.inspect} in collection #{chat.collection.inspect}."
       end
 
     {
       prompt: chat.prompt('snippets_retrieval').to_s,
       collection: {
-        name:        collection_name,
-        description: chat.database_collection?(collection_name)&.description&.to_s,
+        name:        chat.collection,
+        description: chat.database_collection?(chat.collection)&.description&.to_s,
       },
       snippets: records.map do |record|
         {
@@ -183,7 +184,7 @@ class OllamaChat::Tools::SearchKnowledge
     chat.log(:error, e, data: { tool: name, query: args.query })
     { error: e.class.name, message: e.message }.to_json
   ensure
-    old_collection and chat.documents.collection = old_collection
+    old_collection and chat.set_documents_collection(old_collection)
   end
 
   self

data/lib/ollama_chat/version.rb

@@ -1,6 +1,6 @@
 module OllamaChat
   # OllamaChat version
-  VERSION         = '0.0.126'
+  VERSION         = '0.0.127'
   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.126 ruby lib
+# stub: ollama_chat 0.0.127 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "ollama_chat".freeze
-  s.version = "0.0.126".freeze
+  s.version = "0.0.127".freeze
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib".freeze]

data/spec/ollama_chat/commands_spec.rb

@@ -111,6 +111,21 @@ describe OllamaChat::Commands, protect_env: true do
       expect(chat.stream).to receive(:set).with(false, show: true)
       expect(chat.handle_input("/toggle stream -n")).to eq :next
     end
+
+    it 'returns :next when input is "/toggle memory_trigger"' do
+      expect(chat.memory_trigger).to receive(:toggle)
+      expect(chat.handle_input("/toggle memory_trigger")).to eq :next
+    end
+
+    it 'returns :next when input is "/toggle memory_trigger -y"' do
+      expect(chat.memory_trigger).to receive(:set).with(true, show: true)
+      expect(chat.handle_input("/toggle memory_trigger -y")).to eq :next
+    end
+
+    it 'returns :next when input is "/toggle memory_trigger -n"' do
+      expect(chat.memory_trigger).to receive(:set).with(false, show: true)
+      expect(chat.handle_input("/toggle memory_trigger -n")).to eq :next
+    end
   end
 
   describe '/voice' do
@@ -794,6 +809,11 @@ describe OllamaChat::Commands, protect_env: true do
       expect(chat.handle_input("/conversation compact")).to eq :next
     end
 
+    it 'returns :next when input is "/conversation compact summary"' do
+      expect(chat).to receive(:show_compaction_summary)
+      expect(chat.handle_input("/conversation compact summary")).to eq :next
+    end
+
     it 'returns :next when input is "/conversation summarize"' do
       expect(chat).to receive(:summarize_conversation)
         .with(sentence: false)

data/spec/ollama_chat/switches_spec.rb

@@ -95,6 +95,34 @@ describe OllamaChat::Switches do
         }.from(true).to(false)
       end
     end
+
+    context 'callable value (non-adhoc closure)' do
+      let(:store) { { enabled: false } }
+      let :switch do
+        described_class.new(
+          value: -> nv = described_class::UNSET do
+            nv == described_class::UNSET ? store[:enabled]
+                                         : (store[:enabled] = nv; nil)
+          end,
+          msg: { true => 'On', false => 'Off' }
+        )
+      end
+
+      it 'reads without writing' do
+        expect(switch.on?).to eq false
+        expect(store[:enabled]).to eq false
+      end
+
+      it 'set calls the closure with the explicit value' do
+        expect { switch.set(true) }
+          .to change { store[:enabled] }.from(false).to(true)
+      end
+
+      it 'toggle reads then writes the negation' do
+        expect { switch.toggle(show: false) }
+          .to change { store[:enabled] }.from(false).to(true)
+      end
+    end
   end
 
   describe OllamaChat::Switches::DatabaseSwitch do

data/spec/ollama_chat/tools/execute_shell_spec.rb

@@ -134,6 +134,7 @@ describe OllamaChat::Tools::ExecuteShell do
     end
 
     it 'skips editor and confirm in test mode' do
+      stub_success(stdout: "file1\nfile2", stderr: '')
       expect(OllamaChat).to receive(:test_mode?).and_return(true)
       expect(chat).not_to receive(:edit_text)
       expect(chat).not_to receive(:confirm?)

data/spec/ollama_chat/tools/search_knowledge_spec.rb

@@ -118,17 +118,14 @@ describe OllamaChat::Tools::SearchKnowledge do
       )
     )
 
-    mock_docs = double('Documents')
-    expect(chat).to receive(:documents).and_return(mock_docs).at_least(:once)
+    expect(chat).to receive(:collection).
+      and_return('default_collection').at_least(:once)
+    expect(chat).to receive(:set_documents_collection).with('tolkien').ordered
+    expect(chat).to receive(:set_documents_collection).
+      with('default_collection').ordered
     expect(chat).to receive(:database_collection?)
       .and_return(double('Col', description: 'Desc', enabled: true))
       .at_least(:once)
-
-    expect(mock_docs).to receive(:collection).and_return('default_collection').
-      at_least(:once)
-    expect(mock_docs).to receive(:collection=).with('tolkien').ordered
-    expect(mock_docs).to receive(:collection=).with('default_collection').ordered
-
     tool = described_class.new(chat)
     expect(chat).to receive(:find_document_records).and_return([])
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ollama_chat
 version: !ruby/object:Gem::Version
-  version: 0.0.126
+  version: 0.0.127
 platform: ruby
 authors:
 - Florian Frank