legion-settings 1.3.2 → 1.3.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 905fefabffcdb64f2d654827742feb7c2651a3603625eb7f4e2eb71930416c8e
-  data.tar.gz: e19c105841be33b885565d5dac97d2549ee9813897ea4286bf68977e35e95e63
+  metadata.gz: 220738db670fe9bbda99acc5d64d903de88268afdc83bb0ef6c482f550b7b769
+  data.tar.gz: b46f3ff6ed58a9a7a0269230814ea2a9204953970c93877747c5d28c23410c8b
 SHA512:
-  metadata.gz: 71a352c5cd2a45145619ae24d31f9b9f3280c33c0d2f864c21c9549f7a78a4a48b45f06d4172bc6ed30d314a355ab1548bc141efa63638917734816b1c66c0ba
-  data.tar.gz: 6a7a29a655bc77c13352631a62763bc20c8cf9d2a7e84efeaa43b92cc70301425081aa8ed82e6d34a2c77aac9944500ff241ce404f09d0af76bb4de9aaf68baf
+  metadata.gz: 2e8a71bf3705c3f2a809b454c1bc676f7ab1f7c2e3c2dd1b3d6b317f5126ba84bde201b75a1f65e5111d31f1b67408e173eef8da3683d4d19ee48af83c69bdc0
+  data.tar.gz: a537874627f4d55ee13bb3ec4cbce0e869337a15c0f6c1bfd012a16f081c79f8bb7a56f39d7431c7eb6c1e6e525cda8a98155f534bfac32177a135b1aa96e7cd

data/CHANGELOG.md

@@ -1,5 +1,24 @@
 # Legion::Settings Changelog
 
+## [1.3.5] - 2026-03-19
+
+### Added
+- DNS-based bootstrap discovery: auto-detect corporate config from `legion-bootstrap.<search-domain>`
+- `Settings[:dns]` populated with FQDN, default domain, search domains, and nameservers
+- `DnsBootstrap` class with DNS resolution, HTTPS fetch, local caching, and background refresh
+- First boot blocks on fetch; subsequent boots use cache with async refresh
+- Opt-out via `LEGION_DNS_BOOTSTRAP=false` environment variable
+
+## [1.3.4] - 2026-03-18
+
+### Fixed
+- Added `logger` gem to test dependencies for Ruby 4.0 compatibility
+
+## [1.3.3] - 2026-03-17
+
+### Fixed
+- Config JSON parse error now includes the filename at ERROR level instead of burying it in DEBUG
+
 ## [1.3.2] - 2026-03-17
 
 ### Added

data/CLAUDE.md

@@ -8,6 +8,7 @@
 Hash-like configuration store for the LegionIO framework. Loads settings from JSON files, directories, and environment variables. Provides a unified `Legion::Settings[:key]` accessor used by all other Legion gems. Includes schema-based validation with type inference, enum constraints, and cross-module checks.
 
 **GitHub**: https://github.com/LegionIO/legion-settings
+**Version**: 1.3.4
 **License**: Apache-2.0
 
 ## Architecture

data/Gemfile

@@ -5,6 +5,7 @@ source 'https://rubygems.org'
 gemspec
 group :test do
   gem 'legion-logging'
+  gem 'logger'
   gem 'rake'
   gem 'rspec'
   gem 'rspec_junit_formatter'

data/README.md

@@ -2,6 +2,8 @@
 
 Configuration management module for the [LegionIO](https://github.com/LegionIO/LegionIO) framework. Loads settings from JSON files, directories, and environment variables. Provides a unified `Legion::Settings[:key]` accessor used by all other Legion gems.
 
+**Version**: 1.3.4
+
 ## Installation
 
 ```bash

data/docs/plans/2026-03-17-config-error-filename-design.md

@@ -0,0 +1,56 @@
+# Design: Include Filename in JSON Parse Error Messages
+
+## Problem
+
+When a user edits a config file and introduces a JSON syntax error, the settings loader logs:
+
+```
+ERROR config file must be valid json
+```
+
+The filename is only logged at DEBUG level on the next line:
+
+```ruby
+Legion::Logging.error('config file must be valid json')
+Legion::Logging.debug("file:#{file}, error: #{e}")
+```
+
+This makes it nearly impossible for users to identify which file is broken, especially when multiple config files exist in `~/.legionio/settings/`. The settings loader silently skips the broken file, and downstream code fails with confusing errors (e.g., RubyLLM complaining about missing API keys because `llm.json` wasn't loaded).
+
+## Proposed Solution
+
+Change `load_file` in `loader.rb` to include the filename and parse error in the ERROR message:
+
+```ruby
+rescue Legion::JSON::ParseError => e
+  Legion::Logging.error("config file must be valid json: #{file}")
+  Legion::Logging.error("  parse error: #{e.message}")
+end
+```
+
+This is a one-line change (plus one added line) in `loader.rb:140-142`.
+
+### Before
+
+```
+[2026-03-17 18:27:27 -0500] ERROR config file must be valid json
+```
+
+### After
+
+```
+[2026-03-17 18:27:27 -0500] ERROR config file must be valid json: /Users/matt/.legionio/settings/llm.json
+[2026-03-17 18:27:27 -0500] ERROR   parse error: unexpected token at '{ "llm": { ... '
+```
+
+## Alternatives Considered
+
+1. **Raise instead of logging** — rejected; the current behavior of skipping broken files and continuing is correct for resilience. But the user needs to know what happened.
+2. **Add a `config validate` pre-check** — already exists (`legion config validate`), but users won't run it proactively. The error message at load time is the first line of defense.
+3. **Return the broken files in `loaded_files`** — would require API changes for minimal benefit.
+
+## Constraints
+
+- Do not change the error-handling behavior (continue loading other files)
+- Do not change method signatures
+- The debug line can be removed since the info is now in the error message

data/docs/plans/2026-03-17-config-error-filename-implementation.md

@@ -0,0 +1,34 @@
+# Implementation: Include Filename in JSON Parse Error Messages
+
+## Phase 1: Update Error Logging
+
+### Files to Modify
+
+- `lib/legion/settings/loader.rb` — lines 140-142
+
+### Changes
+
+Replace:
+
+```ruby
+rescue Legion::JSON::ParseError => e
+  Legion::Logging.error('config file must be valid json')
+  Legion::Logging.debug("file:#{file}, error: #{e}")
+```
+
+With:
+
+```ruby
+rescue Legion::JSON::ParseError => e
+  Legion::Logging.error("config file must be valid json: #{file}")
+  Legion::Logging.error("  parse error: #{e.message}")
+```
+
+### Spec Coverage
+
+- Add spec in `spec/legion/loader_spec.rb` that writes invalid JSON to a temp file, calls `load_file`, and asserts the error was logged with the filename
+
+### Version Bump
+
+- Bump patch version in `lib/legion/settings/version.rb`
+- Update CHANGELOG.md

data/lib/legion/settings/dns_bootstrap.rb

@@ -0,0 +1,80 @@
+# frozen_string_literal: true
+
+require 'fileutils'
+require 'json'
+require 'net/http'
+require 'resolv'
+require 'uri'
+
+module Legion
+  module Settings
+    class DnsBootstrap
+      CACHE_FILENAME = '_dns_bootstrap.json'
+      HOSTNAME_PREFIX = 'legion-bootstrap'
+      URL_PATH = '/legion/bootstrap.json'
+      HTTP_TIMEOUT = 10
+
+      attr_reader :default_domain, :hostname, :url, :cache_path
+
+      def initialize(default_domain:, cache_dir: nil)
+        @default_domain = default_domain
+        @hostname = "#{HOSTNAME_PREFIX}.#{default_domain}"
+        @url = "https://#{@hostname}#{URL_PATH}"
+        dir = cache_dir || File.expand_path('~/.legionio/settings')
+        @cache_path = File.join(dir, CACHE_FILENAME)
+      end
+
+      def resolve?
+        Resolv.getaddress(@hostname)
+        true
+      rescue Resolv::ResolvError, Resolv::ResolvTimeout
+        false
+      end
+
+      def fetch
+        return nil unless resolve?
+
+        uri = URI.parse(@url)
+        http = Net::HTTP.new(uri.host, uri.port)
+        http.use_ssl = true
+        http.open_timeout = HTTP_TIMEOUT
+        http.read_timeout = HTTP_TIMEOUT
+        response = http.request(Net::HTTP::Get.new(uri))
+        return nil unless response.is_a?(Net::HTTPSuccess)
+
+        ::JSON.parse(response.body, symbolize_names: true)
+      rescue StandardError
+        nil
+      end
+
+      def write_cache(config)
+        FileUtils.mkdir_p(File.dirname(@cache_path))
+        payload = config.merge(
+          _dns_bootstrap_meta: {
+            fetched_at: Time.now.utc.iso8601,
+            hostname:   @hostname,
+            url:        @url
+          }
+        )
+        tmp = "#{@cache_path}.tmp"
+        File.write(tmp, ::JSON.pretty_generate(payload))
+        File.rename(tmp, @cache_path)
+      end
+
+      def read_cache
+        return nil unless File.exist?(@cache_path)
+
+        raw = ::JSON.parse(File.read(@cache_path), symbolize_names: true)
+        raw.delete(:_dns_bootstrap_meta)
+        raw
+      rescue ::JSON::ParserError
+        FileUtils.rm_f(@cache_path)
+        nil
+      end
+
+      def cache_exists?
+        File.exist?(@cache_path)
+      end
+    end
+  end
+end

data/lib/legion/settings/loader.rb

@@ -1,7 +1,9 @@
 # frozen_string_literal: true
 
+require 'resolv'
 require 'socket'
 require 'legion/settings/os'
+require_relative 'dns_bootstrap'
 
 module Legion
   module Settings
@@ -19,6 +21,17 @@ module Legion
         @loaded_files = []
       end
 
+      def dns_defaults
+        resolv_config = read_resolv_config
+        {
+          fqdn:           detect_fqdn,
+          default_domain: resolv_config[:search_domains]&.first,
+          search_domains: resolv_config[:search_domains] || [],
+          nameservers:    resolv_config[:nameservers] || [],
+          bootstrap:      { enabled: true }
+        }
+      end
+
       def client_defaults
         {
           hostname: system_hostname,
@@ -38,7 +51,25 @@ module Legion
             vault:                  { connected: false }
           },
           cache:                      { enabled: true, connected: false, driver: 'dalli' },
-          extensions:                 {},
+          extensions:                 {
+            core:              %w[
+              lex-node lex-tasker lex-scheduler lex-health lex-ping
+              lex-telemetry lex-metering lex-log lex-audit
+              lex-conditioner lex-transformer lex-exec lex-lex lex-codegen
+            ],
+            ai:                %w[lex-claude lex-openai lex-gemini],
+            gaia:              %w[lex-tick lex-mesh lex-apollo lex-cortex],
+            categories:        {
+              core:    { type: :list, tier: 1 },
+              ai:      { type: :list, tier: 2 },
+              gaia:    { type: :list, tier: 3 },
+              agentic: { type: :prefix, tier: 4 }
+            },
+            blocked:           [],
+            reserved_prefixes: %w[core ai agentic gaia],
+            reserved_words:    %w[transport cache crypt data settings json logging llm rbac legion],
+            agentic:           { allowed: nil, blocked: [] }
+          },
           reload:                     false,
           reloading:                  false,
           auto_install_missing_lex:   true,
@@ -53,7 +84,8 @@ module Legion
           },
           transport:                  { connected: false },
           data:                       { connected: false },
-          role:                       { profile: nil, extensions: [] }
+          role:                       { profile: nil, extensions: [] },
+          dns:                        dns_defaults
         }
       end
 
@@ -98,6 +130,27 @@ module Legion
         load_api_env
       end
 
+      def load_dns_bootstrap(cache_dir: nil)
+        return if ENV['LEGION_DNS_BOOTSTRAP'] == 'false'
+
+        domain = @settings.dig(:dns, :default_domain)
+        return unless domain
+        return unless @settings.dig(:dns, :bootstrap, :enabled)
+
+        dir = cache_dir || File.expand_path('~/.legionio/settings')
+        bootstrap = DnsBootstrap.new(default_domain: domain, cache_dir: dir)
+
+        config = if bootstrap.cache_exists?
+                   load_dns_from_cache(bootstrap)
+                 else
+                   load_dns_first_boot(bootstrap)
+                 end
+
+        return unless config
+
+        merge_dns_config(config, bootstrap)
+      end
+
       def load_module_settings(config)
         @settings = deep_merge(config, @settings)
       end
@@ -120,8 +173,8 @@ module Legion
             # @indifferent_access = false
             @loaded_files << file
           rescue Legion::JSON::ParseError => e
-            Legion::Logging.error('config file must be valid json')
-            Legion::Logging.debug("file:#{file}, error: #{e}")
+            Legion::Logging.error("config file must be valid json: #{file}")
+            Legion::Logging.error("  parse error: #{e.message}")
           end
         else
           Legion::Logging.warn("Config file does not exist or is not readable file:#{file}")
@@ -166,6 +219,39 @@ module Legion
 
       private
 
+      def load_dns_from_cache(bootstrap)
+        config = bootstrap.read_cache
+        start_dns_background_refresh(bootstrap) if config
+        config
+      end
+
+      def load_dns_first_boot(bootstrap)
+        Legion::Logging.debug("DNS bootstrap: first boot, fetching from #{bootstrap.url}")
+        config = bootstrap.fetch
+        bootstrap.write_cache(config) if config
+        config
+      end
+
+      def merge_dns_config(config, bootstrap)
+        @settings = deep_merge(config, @settings)
+        @settings[:dns] ||= {}
+        @settings[:dns][:corp_bootstrap] = {
+          discovered: true,
+          hostname:   bootstrap.hostname,
+          url:        bootstrap.url
+        }
+        @indifferent_access = false
+      end
+
+      def start_dns_background_refresh(bootstrap)
+        Thread.new do
+          fresh = bootstrap.fetch
+          bootstrap.write_cache(fresh) if fresh
+        rescue StandardError
+          # Background refresh is best-effort
+        end
+      end
+
       def setting_category(category)
         @settings[category].map do |name, details|
           details.merge(name: name.to_s)
@@ -283,6 +369,25 @@ module Legion
         Legion::Logging.error(message)
         raise(Error, message)
       end
+
+      def read_resolv_config
+        config = Resolv::DNS::Config.default_config_hash
+        {
+          search_domains: config[:search]&.map(&:to_s)&.uniq,
+          nameservers:    config[:nameserver]&.map(&:to_s)&.uniq
+        }
+      rescue StandardError
+        { search_domains: [], nameservers: [] }
+      end
+
+      def detect_fqdn
+        fqdn = Addrinfo.getaddrinfo(Socket.gethostname, nil).first&.canonname
+        return nil if fqdn.nil?
+
+        fqdn.include?('.') ? fqdn : nil
+      rescue StandardError
+        nil
+      end
     end
   end
 end

data/lib/legion/settings/version.rb

@@ -2,6 +2,6 @@
 
 module Legion
   module Settings
-    VERSION = '1.3.2'
+    VERSION = '1.3.5'
   end
 end

data/lib/legion/settings.rb

@@ -17,6 +17,7 @@ module Legion
       def load(options = {})
         @loader = Legion::Settings::Loader.new
         @loader.load_env
+        @loader.load_dns_bootstrap
         @loader.load_file(options[:config_file]) if options[:config_file]
         @loader.load_directory(options[:config_dir]) if options[:config_dir]
         options[:config_dirs]&.each do |directory|

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: legion-settings
 version: !ruby/object:Gem::Version
-  version: 1.3.2
+  version: 1.3.5
 platform: ruby
 authors:
 - Esity
@@ -42,8 +42,11 @@ files:
 - Gemfile
 - LICENSE
 - README.md
+- docs/plans/2026-03-17-config-error-filename-design.md
+- docs/plans/2026-03-17-config-error-filename-implementation.md
 - legion-settings.gemspec
 - lib/legion/settings.rb
+- lib/legion/settings/dns_bootstrap.rb
 - lib/legion/settings/loader.rb
 - lib/legion/settings/os.rb
 - lib/legion/settings/resolver.rb