legion-tty 0.4.41 → 0.4.42

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2e5ecb9de86922a5a3d4dd3b3eca6b4f7864103c89399a8a945bbae95e749814
-  data.tar.gz: 0d25f75d79303fd7a42a9fbf125768503c48f16e5c749b799cd2013d68bae99a
+  metadata.gz: 544115343036952adfc2f3accc8ce189ec8bb13c06f04ce383d0536549118e49
+  data.tar.gz: 806f22df6bed2c34ce6c47c6eab8c1af6bb611cc458db878087a86163f7e0248
 SHA512:
-  metadata.gz: 804bc560650b61a6987a8f9d4d02cd7a315caeaccd1823f1b11a4e1b1fe024fc0eb37a35c59bafe654c0738f2e201af9b04d7274bd9c7d8df02b563c6551336b
-  data.tar.gz: 15ab28737a1677ae28285400d1e9ae6b794f1ea021e4cc7c194e949e59a97df56c054419d21fd351c70ea97636608d09a8a6737ca7a690e17b398e87531e6c1c
+  metadata.gz: 4a90450fe8ba541b1823f43cfc89eb97ed2ef6185029afffc022771c87f8909583fab80715098789240b8bab4ee3c8c2ae240987dc9630352c76d89ee3acad6a
+  data.tar.gz: '0667478fd7867da23c0c5f4a83fdeac3056cc97ff18a2bf1dbe0c447f4a29088b0e1d6aafeb6d9b19ffc8a1784b4cd80f1a94a2d9ea5a2563fca063004020ee5'

data/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## [0.4.42] - 2026-04-08
+
+### Changed
+- `DaemonClient` now uses `Legion::Logging::Helper` (structured logging via `log.info`/`log.debug`) instead of inline `Legion::Logging.method if defined?` guards for all logging and exception handling
+- Bumped `legion-logging` minimum dependency from `>= 1.2.8` to `>= 1.5.0` to use `handle_exception` from `Legion::Logging::Helper`
+- Extracted `store_manifest` and `parse_inference_response` private helpers to reduce method complexity in `fetch_manifest` and `inference`
+
+### Fixed
+- `KerberosProbe#days_in_month` used `Time.new(year, month, -1)` which raises `ArgumentError` for day `-1`; replaced with `Date.new(year, month, -1).day` (correct Ruby idiom for last day of month)
+
 ## [0.4.41] - 2026-03-31
 
 ### Added

data/README.md

@@ -2,7 +2,7 @@
 
 Rich terminal UI for the LegionIO async cognition engine.
 
-**Version**: 0.4.35
+**Version**: 0.4.42
 
 Think Claude Code meets Codex CLI, but for LegionIO: onboarding wizard with identity detection, streaming AI chat shell with 115 slash commands, operational dashboard, extensions browser, config editor, and session persistence - all rendered with the [tty-ruby](https://ttytoolkit.org/) gem ecosystem.
 
@@ -289,8 +289,8 @@ Boot logs go to `~/.legionio/logs/tty-boot.log`.
 
 ```bash
 bundle install
-bundle exec rspec       # 1817 examples, 0 failures
-bundle exec rubocop     # 150 files, 0 offenses
+bundle exec rspec       # 1952 examples, 0 failures
+bundle exec rubocop     # 163 files, 0 offenses
 ```
 
 ## License

data/lib/legion/tty/background/kerberos_probe.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require 'date'
 require 'resolv'
 require 'shellwords'
 
@@ -185,7 +186,7 @@ module Legion
         # rubocop:enable Metrics/AbcSize
 
         def days_in_month(month, year)
-          Time.new(year, month, -1).day
+          Date.new(year, month, -1).day
         end
 
         def log_result(result, elapsed)

data/lib/legion/tty/daemon_client.rb

@@ -4,6 +4,7 @@ require 'net/http'
 require 'uri'
 require 'fileutils'
 require 'legion/json'
+require 'legion/logging'
 
 module Legion
   module TTY
@@ -12,11 +13,14 @@ module Legion
 
       # rubocop:disable Metrics/ClassLength
       class << self
+        include Legion::Logging::Helper
+
         def configure(daemon_url: 'http://127.0.0.1:4567', cache_file: nil, timeout: 5)
           @daemon_url = daemon_url
           @cache_file = cache_file || File.expand_path('~/.legionio/catalog.json')
           @timeout = timeout
           @manifest = nil
+          log.info { "TTY daemon client configured daemon_url=#{@daemon_url} timeout=#{@timeout}" }
         end
 
         def available?
@@ -26,7 +30,7 @@ module Legion
           end
           response.code.to_i == 200
         rescue StandardError => e
-          Legion::Logging.debug("daemon available? check failed: #{e.message}") if defined?(Legion::Logging)
+          handle_exception(e, level: :debug, operation: 'tty.daemon_client.available?', daemon_url: daemon_url)
           false
         end
 
@@ -37,12 +41,9 @@ module Legion
           end
           return nil unless response.code.to_i == 200
 
-          body = Legion::JSON.load(response.body)
-          @manifest = body[:data]
-          write_cache(@manifest)
-          @manifest
+          store_manifest(Legion::JSON.load(response.body)[:data])
         rescue StandardError => e
-          Legion::Logging.warn("fetch_manifest failed: #{e.message}") if defined?(Legion::Logging)
+          handle_exception(e, level: :warn, operation: 'tty.daemon_client.fetch_manifest', daemon_url: daemon_url)
           nil
         end
 
@@ -53,7 +54,7 @@ module Legion
 
           @manifest = Legion::JSON.load(File.read(@cache_file))
         rescue StandardError => e
-          Legion::Logging.warn("cached_manifest failed: #{e.message}") if defined?(Legion::Logging)
+          handle_exception(e, level: :warn, operation: 'tty.daemon_client.cached_manifest', cache_file: @cache_file)
           nil
         end
 
@@ -80,26 +81,28 @@ module Legion
 
           uri = URI("#{daemon_url}/api/llm/chat")
           payload = Legion::JSON.dump({ message: message, model: model, provider: provider })
+          log.debug { "TTY chat request model=#{model} provider=#{provider} message_length=#{message.to_s.length}" }
           response = post_json(uri, payload)
 
           return nil unless response && SUCCESS_CODES.include?(response.code.to_i)
 
           Legion::JSON.load(response.body)
         rescue StandardError => e
-          Legion::Logging.warn("chat failed: #{e.message}") if defined?(Legion::Logging)
+          handle_exception(e, level: :warn, operation: 'tty.daemon_client.chat',
+                              daemon_url: daemon_url, model: model, provider: provider)
           nil
         end
 
         def inference(messages:, tools: [], model: nil, provider: nil, timeout: 120)
+          log.debug { "TTY inference model=#{model} provider=#{provider} msgs=#{Array(messages).size}" }
           response = post_inference(messages: messages, tools: tools, model: model,
                                     provider: provider, timeout: timeout)
           return inference_error_result(response) unless SUCCESS_CODES.include?(response.code.to_i)
 
-          body = Legion::JSON.load(response.body)
-          data = body[:data] || body
-          { status: :ok, data: data }
+          parse_inference_response(response)
         rescue StandardError => e
-          Legion::Logging.warn("inference failed: #{e.message}") if defined?(Legion::Logging)
+          handle_exception(e, level: :warn, operation: 'tty.daemon_client.inference',
+                              daemon_url: daemon_url, model: model, provider: provider, timeout: timeout)
           { status: :unavailable, error: { message: e.message } }
         end
 
@@ -112,6 +115,19 @@ module Legion
 
         private
 
+        def parse_inference_response(response)
+          body = Legion::JSON.load(response.body)
+          data = body[:data] || body
+          { status: :ok, data: data }
+        end
+
+        def store_manifest(data)
+          @manifest = data
+          write_cache(@manifest)
+          log.info { "TTY fetched daemon manifest entries=#{Array(@manifest).size}" }
+          @manifest
+        end
+
         def post_inference(messages:, tools:, model:, provider:, timeout:)
           uri = URI("#{daemon_url}/api/llm/inference")
           payload = Legion::JSON.dump({ messages: messages, tools: tools,
@@ -151,8 +167,9 @@ module Legion
 
           FileUtils.mkdir_p(File.dirname(@cache_file))
           File.write(@cache_file, Legion::JSON.dump(data))
+          log.debug { "TTY daemon manifest cache updated file=#{@cache_file}" }
         rescue StandardError => e
-          Legion::Logging.warn("write_cache failed: #{e.message}") if defined?(Legion::Logging)
+          handle_exception(e, level: :warn, operation: 'tty.daemon_client.write_cache', cache_file: @cache_file)
           nil
         end
       end

data/lib/legion/tty/version.rb

@@ -2,6 +2,6 @@
 
 module Legion
   module TTY
-    VERSION = '0.4.41'
+    VERSION = '0.4.42'
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: legion-tty
 version: !ruby/object:Gem::Version
-  version: 0.4.41
+  version: 0.4.42
 platform: ruby
 authors:
 - Esity
@@ -197,14 +197,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.2.8
+        version: 1.5.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.2.8
+        version: 1.5.0
 description: Rich TUI with onboarding wizard, AI chat shell, and operational dashboards
   for LegionIO
 email: