ollama_chat 0.0.111 → 0.0.113

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 54b2bf70b7f843ad09b5de5f0881bf810ec78a08c79c5e1cd93d86aa75b68696
-  data.tar.gz: 33e475ea5e115609e02ba6b92cdf6a9b1b1dc6eccbfde6978fc80623daa3040b
+  metadata.gz: e618b797bf1b5fa84728816e06b83fe3540424b48256ba107deb29c2beb8a219
+  data.tar.gz: f5abee2701f3fc6d7ba09513f0b0bd5133c7ec10ddb511ee5ea4238c14f9988e
 SHA512:
-  metadata.gz: e7dad8f668a54b9c1e89937a445894eeb9872e1541d54dfc36fcf136d5c759bde7e0c21c9b818ce2e8ffa3bd7274163ada6b18553590dc812f3e89bdae839a27
-  data.tar.gz: 8bbd695f1eacc33c6928f8a382acdb8ed68b57348a0ce5199a9203bd7070df59f71a7a37e2862297c6c7dbc12e0d00aa9c46ef90f6392fc7579b87c8de438b7d
+  metadata.gz: 2f2172cc90da0a8b8c1b7f2306abbc8480db92abeb3629e20b2471e88fdd79ca2d18c209c05690167b564b6911685222dce48496e9825330e43500ff520aab74
+  data.tar.gz: da6e5e8583b11cb7d766ff0745e97070f2a39ac8f1ca6314a4bcb93d8e9a6c3ec9ef087a055b7c9844317f831fc4a6a590e11f80eaf6b6c80980143ab8385771

data/CHANGES.md

@@ -1,5 +1,28 @@
 # Changes
 
+## 2026-08-25 v0.0.113
+
+*   Simplified hash syntax by replacing `text: text` with `text:` etc.
+*   Removed the `/conversation compact` stub, including its regexp,
+    completion list, help text, handler logic, and associated specs.
+*   Added specs for `/config edit`, `/config reload`, and `/config diff` in
+    `commands_spec.rb`, verifying the ordered sequence of `diff_config`
+    followed by the new `reload_config` for the diff command.
+*   Removed the unused `collection_descriptions: {}` entry from the
+    `embedding:` section in `default_config.yml`.
+
+## 2026-08-25 v0.0.112
+
+*   Added a `diff` subcommand to the `/config` command, including updated
+    regexp, completion, and help text.
+*   Extracted a `diff_config` method in `OllamaChat::ConfigHandling` that
+    resolves `OC::DIFF_TOOL` and launches a diff against
+    `@ollama_chat_config.filename` and
+    `@ollama_chat_config.default_config_path`.
+*   Simplified `fix_config` to delegate the diff invocation to `diff_config`.
+*   Added `#diff_config` specs covering both the missing-diff-tool path (STDERR
+    + `nil`) and the successful `system` invocation.
+
 ## 2026-08-25 v0.0.111
 
 ### Added

data/lib/ollama_chat/clipboard.rb

@@ -114,7 +114,7 @@ module OllamaChat::Clipboard
   # @return [TrueClass] if the copying has been performed successfully.
   def copy_to_clipboard(edit: false)
     text = last_message_content(content: false)
-    perform_copy_to_clipboard(text: text, edit:)
+    perform_copy_to_clipboard(text:, edit:)
     log(:info, "Copied to clipboard", data: { bytes: format_bytes(text.bytesize) }) if text
     STDOUT.puts "The last response has been successfully copied to the system clipboard."
     true

data/lib/ollama_chat/commands.rb

@@ -62,11 +62,11 @@ module OllamaChat::Commands
 
   command(
     name: :config,
-    regexp: %r(^/config(?:\s+(edit|reload))?$),
-    complete: [ 'config', %w[ edit reload ] ],
+    regexp: %r(^/config(?:\s+(edit|reload|diff))?$),
+    complete: [ 'config', %w[ edit reload diff ] ],
     optional: true,
     help: <<~EOT
-      \u2699\uFE0F View, edit, or reload configuration
+      \u2699\uFE0F View, edit, diff, or reload configuration
     EOT
   ) do |subcommand|
     case subcommand
@@ -74,9 +74,13 @@ module OllamaChat::Commands
       edit_config
     when 'reload'
       reload_config
+    when 'diff'
+      diff_config
+      reload_config
     else
       display_config
     end
+
     :next
   end
 
@@ -594,15 +598,14 @@ module OllamaChat::Commands
 
   command(
     name: :conversation,
-    regexp: %r(^/conversation\s+(clean|compact|save|load)(\s+-c)?(?:\s+([^-].*\.jsonl?))?$),
-    complete: [ 'conversation', %w[ save load clean compact ] ],
+    regexp: %r(^/conversation\s+(clean|save|load)(\s+-c)?(?:\s+([^-].*\.jsonl?))?$),
+    complete: [ 'conversation', %w[ save load clean ] ],
     options: '[-c] [FILENAME]',
     help: <<~EOT
       💾 Save/Load conversation state
          (-c to clean before saving)
          FILENAME ends with .json or .jsonl
          Or clean inplace (removes tool content, images, and thinking)
-         Or compact (summarize old messages, keep recent)
     EOT
   ) do |subcommand,opts,path|
     if %w[ save load ].include?(subcommand) && path.blank?
@@ -628,18 +631,6 @@ module OllamaChat::Commands
       else
         STDOUT.puts 'Cancelled.'
       end
-    when 'compact'
-      if confirm?(
-          prompt: '🔔 Compact conversation? Old messages will be summarized. (y/n) ',
-          yes: /\Ay/i
-        )
-      then
-        messages.compact!
-        session_sync
-        STDOUT.puts "Conversation compacted."
-      else
-        STDOUT.puts 'Cancelled.'
-      end
     end
     :next
   end

data/lib/ollama_chat/config_handling.rb

@@ -44,6 +44,22 @@ module OllamaChat::ConfigHandling
     end
   end
 
+  # Opens a diff tool to compare the current config file with the
+  # default config, allowing the user to visually inspect differences.
+  #
+  # @return [Boolean, nil] the result of +system+ if a diff tool was
+  #   found, or +nil+ if no diff tool is configured.
+  def diff_config
+    diff_tool = OC::DIFF_TOOL? or
+      return STDERR.puts 'No diff tool configured (OC::DIFF_TOOL).'
+    cmd = [
+      diff_tool,
+      @ollama_chat_config.filename,
+      @ollama_chat_config.default_config_path,
+    ].map(&:to_s)
+    system(*cmd)
+  end
+
   # The fix_config method handles configuration file errors by informing the
   # user about the exception and prompting them to fix it. It then executes a
   # diff tool to compare the current config file with the default one.
@@ -52,9 +68,9 @@ module OllamaChat::ConfigHandling
   # @param exception [Exception] the exception that occurred while reading
   #   the config file
   def fix_config(exception)
-    STDOUT.puts "When reading the config file, a #{exception.class} "\
+    STDOUT.puts "When reading the config file, a #{exception.class} " \
       "exception was caught: #{exception.message.inspect}"
-    unless diff_tool = OC::DIFF_TOOL?
+    unless OC::DIFF_TOOL?
       exit 1
     end
     if confirm?(
@@ -62,18 +78,14 @@ module OllamaChat::ConfigHandling
         yes: /\Ay/i
       )
     then
-      cmd = [
-        diff_tool,
-        @ollama_chat_config.filename,
-        @ollama_chat_config.default_config_path,
-      ].map(&:to_s)
-      system(*cmd)
+      diff_config
       exit 0
     else
       exit 1
     end
   end
 
+
   # Adjusts the command-line argument array to ensure the current session is
   # explicitly reloaded by injecting the `-l <session_id>` flag.
   #

data/lib/ollama_chat/ollama_chat_config/default_config.yml

@@ -378,7 +378,6 @@ embedding:
   batch_size: 100
   database_filename: null # ':memory:'
   collection: <%= OC::OLLAMA::CHAT::COLLECTION %>
-  collection_descriptions: {}
   found_texts_size: 16384
   found_texts_count: 10
   splitter:

data/lib/ollama_chat/rag_handling.rb

@@ -267,7 +267,7 @@ module OllamaChat::RAGHandling
         update_collection(name.to_s)
       end
       STDOUT.puts "✅ Created collection '#{name}'."
-      log(:info, "Collection created", data: { name: name, description:, patterns: })
+      log(:info, "Collection created", data: { name:, description:, patterns: })
     rescue Sequel::UniqueConstraintViolation
       STDERR.puts "❌ Collection #{name.inspect} already exists."
     rescue Sequel::Error => e

data/lib/ollama_chat/tools/patch_file.rb

@@ -170,7 +170,7 @@ class OllamaChat::Tools::PatchFile
 
     (result | {
       path:     path.to_s,
-      message:  message,
+      message:  ,
       mtime:    ,
       checksum: ,
     }).to_json

data/lib/ollama_chat/version.rb

@@ -1,6 +1,6 @@
 module OllamaChat
   # OllamaChat version
-  VERSION         = '0.0.111'
+  VERSION         = '0.0.113'
   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.111 ruby lib
+# stub: ollama_chat 0.0.113 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "ollama_chat".freeze
-  s.version = "0.0.111".freeze
+  s.version = "0.0.113".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

@@ -532,18 +532,6 @@ describe OllamaChat::Commands, protect_env: true do
       expect(chat).to receive(:confirm?).and_return false
       expect(chat.handle_input("/conversation clean")).to eq :next
     end
-
-    it 'returns :next when input is "/conversation compact"' do
-      expect(chat).to receive(:confirm?).and_return true
-      expect(chat.messages).to receive(:compact!)
-      expect(chat).to receive(:session_sync)
-      expect(chat.handle_input("/conversation compact")).to eq :next
-    end
-
-    it 'returns :next when input is "/conversation compact" and user cancels' do
-      expect(chat).to receive(:confirm?).and_return false
-      expect(chat.handle_input("/conversation compact")).to eq :next
-    end
   end
 
   describe 'tools' do
@@ -568,6 +556,22 @@ describe OllamaChat::Commands, protect_env: true do
       expect(chat).to receive(:display_config)
       expect(chat.handle_input("/config")).to eq :next
     end
+
+    it 'returns :next when input is "/config edit"' do
+      expect(chat).to receive(:edit_config)
+      expect(chat.handle_input("/config edit")).to eq :next
+    end
+
+    it 'returns :next when input is "/config reload"' do
+      expect(chat).to receive(:reload_config)
+      expect(chat.handle_input("/config reload")).to eq :next
+    end
+
+    it 'returns :next when input is "/config diff"' do
+      expect(chat).to receive(:diff_config)
+      expect(chat).to receive(:reload_config)
+      expect(chat.handle_input("/config diff")).to eq :next
+    end
   end
 
   describe '/quit' do

data/spec/ollama_chat/config_handling_spec.rb

@@ -110,6 +110,36 @@ describe OllamaChat::ConfigHandling do
     end
   end
 
+  describe '#diff_config' do
+    context 'without a diff tool' do
+      before do
+        expect(OC).to receive(:DIFF_TOOL?).and_return(nil)
+      end
+
+      it 'prints an error to STDERR and returns nil' do
+        expect(STDERR).to receive(:puts).with(/No diff tool configured/)
+        expect(chat.diff_config).to be_nil
+      end
+    end
+
+    context 'with a diff tool' do
+      before { const_conf_as('OC::DIFF_TOOL' => 'vimdiff') }
+
+      it 'launches the diff tool with config and default paths' do
+        expect(chat.instance_variable_get(:@ollama_chat_config))
+          .to receive(:filename).and_return('/cfg.yml')
+          .ordered
+        expect(chat.instance_variable_get(:@ollama_chat_config))
+          .to receive(:default_config_path).and_return('/default.yml')
+          .ordered
+        expect(chat).to receive(:system)
+          .with('vimdiff', '/cfg.yml', '/default.yml')
+          .and_return(true)
+        expect(chat.diff_config).to be true
+      end
+    end
+  end
+
   describe '#edit_config' do
     it 'edits the config and restarts on confirmation' do
       expect(chat).to receive(:edit_file).and_return(true)

metadata

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