lex-llm-ollama 0.1.5 → 0.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a2dc0a8753b2476cdcbfc6828b668c237e90a3cab9cbdc54871318c5278f5584
-  data.tar.gz: 816cfdf03f47f05413992fb71ff1753b49a6451197ca753965e077a94e2c7626
+  metadata.gz: a9b4ccf479be5fe7a785b15c5a8f5cc328c3e916736d75b22584af2a7e3488b5
+  data.tar.gz: 8ee2de3a028a8a877a4055e83019652feb8208f1d359db91584b9a795b5b20ed
 SHA512:
-  metadata.gz: 3f2c7ac5d68f8d1ef5a95d9a20eed2dfa186d759a590c385b7b325c39fa973c149622c30373f8f03acd882915487cacb4815cf332cd3079b4df3629d0bd52d2a
-  data.tar.gz: eb0a2212c0e4623d253e702204cbfb0733aa27948399ef238d34bd8539b19cc757b11662cc35b0e4ca8b731b815d2994b38c6f7b19db8b8a5cb93fa5013bc7fa
+  metadata.gz: de11691a812a9a45ec23ee8b2391cdc902c77043acd368f6b48d3ace4e53acd20da23a8d8d2b66d90a6878542ec186d6b2bf809a86e660b545ff5fe2aeed8577
+  data.tar.gz: 8080ef3565c62e8c587036d70fce6f08ac15c4890b3d5b4316a1cf8f85ac9407483c0638292d1bd24857451490a9a49d4287db40c75f5a921d1a4d5dfffa6a6e

data/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## 0.1.6 - 2026-04-30
+
+- Add `Legion::Logging::Helper` to Ollama module, RegistryPublisher, and RegistryEventBuilder.
+- Replace all bare rescue blocks with `handle_exception` calls including level, handled, and operation.
+- Add info-level action logging to Provider key actions: list_running_models, readiness, list_models, show_model, pull_model.
+- Add info-level logging to RegistryPublisher publish methods.
+- Add rescue-with-handle_exception to Provider#list_running_models, show_model, and pull_model.
+- Update README to reflect current architecture and file layout.
+
 ## 0.1.5 - 2026-04-28
 
 - Publish best-effort provider/model availability events to `llm.registry` from Ollama readiness and model discovery.

data/README.md

@@ -1,6 +1,6 @@
 # lex-llm-ollama
 
-LegionIO LLM provider extension for Ollama.
+LegionIO LLM provider extension for [Ollama](https://ollama.ai).
 
 This gem lives under `Legion::Extensions::Llm::Ollama` and depends on `lex-llm` for shared provider-neutral routing, fleet, and schema primitives.
 
@@ -10,14 +10,27 @@ Load it with `require 'legion/extensions/llm/ollama'`.
 
 - `Legion::Extensions::Llm::Provider` registration as `:ollama`
 - Ollama-native chat requests through `POST /api/chat`
-- streaming chat support
-- model discovery through `GET /api/tags`
-- running model inspection through `GET /api/ps`
-- model details through `POST /api/show`
-- model download helper through `POST /api/pull`
-- embeddings through `POST /api/embed`
-- best-effort `llm.registry` availability events from readiness and model discovery when Legion Transport is loaded
-- shared fleet/default settings via `Legion::Extensions::Llm.provider_settings`
+- Streaming chat support
+- Model discovery through `GET /api/tags`
+- Running model inspection through `GET /api/ps`
+- Model details through `POST /api/show`
+- Model download helper through `POST /api/pull`
+- Embeddings through `POST /api/embed`
+- Best-effort `llm.registry` availability events from readiness and model discovery when Legion Transport is loaded
+- Shared fleet/default settings via `Legion::Extensions::Llm.provider_settings`
+- Full `Legion::Logging::Helper` integration with structured `handle_exception` in every rescue block
+
+## Architecture
+
+```
+Legion::Extensions::Llm::Ollama
+├── Provider               # Ollama provider (chat, stream, embed, models, readiness)
+├── RegistryPublisher      # Best-effort async llm.registry event publishing
+├── RegistryEventBuilder   # Sanitized lex-llm registry envelope construction
+└── Transport/
+    ├── Exchanges::LlmRegistry     # Topic exchange for llm.registry
+    └── Messages::RegistryEvent    # AMQP message wrapper for registry events
+```
 
 ## Defaults
 
@@ -46,3 +59,16 @@ Legion::Extensions::Llm.configure do |config|
   config.default_embedding_model = "nomic-embed-text:latest"
 end
 ```
+
+## Development
+
+```bash
+bundle install
+bundle exec rspec       # 0 failures
+bundle exec rubocop -A  # auto-fix
+bundle exec rubocop     # lint check
+```
+
+## License
+
+Apache-2.0

data/lib/legion/extensions/llm/ollama/provider.rb

@@ -47,27 +47,42 @@ module Legion
           def version_url = '/api/version'
 
           def list_running_models
+            log.info { "listing running models from #{api_base}#{running_models_url}" }
             connection.get(running_models_url).body.fetch('models', [])
+          rescue StandardError => e
+            handle_exception(e, level: :error, handled: true, operation: 'ollama.list_running_models')
+            []
           end
 
           def readiness(live: false)
+            log.info { "checking readiness live=#{live} at #{api_base}" }
             super.tap do |metadata|
               self.class.registry_publisher.publish_readiness_async(metadata) if live
             end
           end
 
           def list_models
+            log.info { "discovering models from #{api_base}#{models_url}" }
             super.tap do |models|
+              log.info { "discovered #{models.size} model(s) from Ollama" }
               self.class.registry_publisher.publish_models_async(models, readiness: readiness(live: false))
             end
           end
 
           def show_model(model)
+            log.info { "fetching model details for #{model}" }
             connection.post(show_model_url, { model: model }).body
+          rescue StandardError => e
+            handle_exception(e, level: :error, handled: true, operation: 'ollama.show_model')
+            raise
           end
 
           def pull_model(model, stream: false)
+            log.info { "pulling model #{model} stream=#{stream}" }
             connection.post(pull_url, { model: model, stream: stream }).body
+          rescue StandardError => e
+            handle_exception(e, level: :error, handled: true, operation: 'ollama.pull_model')
+            raise
           end
 
           private

data/lib/legion/extensions/llm/ollama/registry_event_builder.rb

@@ -6,6 +6,8 @@ module Legion
       module Ollama
         # Builds sanitized lex-llm registry envelopes for Ollama provider state.
         class RegistryEventBuilder # rubocop:disable Metrics/ClassLength
+          include Legion::Logging::Helper
+
           def readiness(readiness)
             registry_event_class.public_send(
               readiness[:ready] ? :available : :unavailable,
@@ -129,7 +131,8 @@ module Legion
             configured_node = (::Legion::Settings.dig(:node, :canonical_name) if defined?(::Legion::Settings))
             value = configured_node.to_s.strip
             value.empty? ? :ollama : value.to_sym
-          rescue StandardError
+          rescue StandardError => e
+            handle_exception(e, level: :debug, handled: true, operation: 'ollama.registry.provider_instance')
             :ollama
           end
 

data/lib/legion/extensions/llm/ollama/registry_publisher.rb

@@ -6,6 +6,8 @@ module Legion
       module Ollama
         # Best-effort publisher for Ollama provider availability events.
         class RegistryPublisher
+          include Legion::Logging::Helper
+
           APP_ID = 'lex-llm-ollama'
 
           def initialize(builder: RegistryEventBuilder.new)
@@ -13,10 +15,12 @@ module Legion
           end
 
           def publish_readiness_async(readiness)
+            log.info { 'publishing readiness event to llm.registry' }
             schedule { publish_event(@builder.readiness(readiness)) }
           end
 
           def publish_models_async(models, readiness:)
+            log.info { "publishing #{Array(models).size} model event(s) to llm.registry" }
             schedule do
               Array(models).each do |model|
                 publish_event(@builder.model_available(model, readiness:))
@@ -33,10 +37,10 @@ module Legion
               Thread.current.abort_on_exception = false
               yield
             rescue StandardError => e
-              log_publish_failure(e, level: :debug)
+              handle_exception(e, level: :debug, handled: true, operation: 'ollama.registry.schedule_thread')
             end
           rescue StandardError => e
-            log_publish_failure(e, level: :debug)
+            handle_exception(e, level: :debug, handled: true, operation: 'ollama.registry.schedule')
             false
           end
 
@@ -45,7 +49,7 @@ module Legion
 
             message_class.new(event:, app_id: APP_ID).publish(spool: false)
           rescue StandardError => e
-            log_publish_failure(e)
+            handle_exception(e, level: :warn, handled: true, operation: 'ollama.registry.publish_event')
             false
           end
 
@@ -56,7 +60,8 @@ module Legion
             return true unless ::Legion::Transport::Connection.respond_to?(:session_open?)
 
             ::Legion::Transport::Connection.session_open?
-          rescue StandardError
+          rescue StandardError => e
+            handle_exception(e, level: :debug, handled: true, operation: 'ollama.registry.publishing_available?')
             false
           end
 
@@ -70,7 +75,8 @@ module Legion
 
             require 'legion/extensions/llm/ollama/transport/messages/registry_event'
             message_class_defined?
-          rescue LoadError
+          rescue LoadError => e
+            handle_exception(e, level: :debug, handled: true, operation: 'ollama.registry.transport_load')
             false
           end
 
@@ -81,18 +87,6 @@ module Legion
           def message_class
             ::Legion::Extensions::Llm::Ollama::Transport::Messages::RegistryEvent
           end
-
-          def log_publish_failure(error, level: :warn)
-            message = "[lex-llm-ollama] llm.registry publish failed: #{error.class}: #{error.message}"
-            logger = ::Legion::Extensions::Llm.logger if defined?(::Legion::Extensions::Llm)
-            if logger.respond_to?(level)
-              logger.public_send(level, message)
-            elsif logger.respond_to?(:debug)
-              logger.debug(message)
-            end
-          rescue StandardError
-            nil
-          end
         end
       end
     end

data/lib/legion/extensions/llm/ollama/version.rb

@@ -4,7 +4,7 @@ module Legion
   module Extensions
     module Llm
       module Ollama
-        VERSION = '0.1.5'
+        VERSION = '0.1.6'
       end
     end
   end

data/lib/legion/extensions/llm/ollama.rb

@@ -12,6 +12,7 @@ module Legion
       # Ollama provider extension namespace.
       module Ollama
         extend ::Legion::Extensions::Core if ::Legion::Extensions.const_defined?(:Core, false)
+        extend Legion::Logging::Helper
 
         PROVIDER_FAMILY = :ollama
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lex-llm-ollama
 version: !ruby/object:Gem::Version
-  version: 0.1.5
+  version: 0.1.6
 platform: ruby
 authors:
 - LegionIO