legion-settings 1.3.1 → 1.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cb0e2c44578e9e5b6cbb495c5160e1a37c067ea5692979e50e89494d33da9888
-  data.tar.gz: ebca4fdb1fb58e2fa315620209ccbe5f5302057a7dc0d4a256f9689ea8d5d8b0
+  metadata.gz: f66aca7decaa9e60f84abf3d4b2538f42a90978289916f535395991161bb8e24
+  data.tar.gz: 41c5f1bac7ec1e12d0034ad872816405588d880542efbe7c420ec958697f1991
 SHA512:
-  metadata.gz: 738f2ced6bf58fcb2d9ad4fca307d63abea4241bd6c5e788ff3ee47346f8f1054fd1cceb94d86e2f393f7985a0a835076513b94de730a89e9e233d923d5c1775
-  data.tar.gz: ac60c369c84bd0dcb53b84f5fcdf9440fcd25ea00d15bb32386b67a4b999ceead7dde040fa66de53c9e6b06a5e99628b674923d0503dcbbe945a5b30a7d88bb6
+  metadata.gz: 967b78c6ca4e9e2bee7a2ac1e53c6198a0b0606985e16fca8734c061be6b5b5b4f30e7ed882c387af00554f3323e9008269e036eaec09e9b7ee12d1e1c8ad349
+  data.tar.gz: 2451834575d97a57f1f11c41556f48cf581358624c793d8ac011158b174df2072988d1135af0b9fc3656d1fd608734ab155a2f5b7419d71f77a613e5d8677994

data/CHANGELOG.md

@@ -1,5 +1,20 @@
 # Legion::Settings Changelog
 
+## [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
+- `dig(*keys)` method on `Legion::Settings` and `Legion::Settings::Loader` for nested key access
+
 ## [1.3.1] - 2026-03-16
 
 ### Added

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

@@ -36,12 +36,13 @@ Each Legion module registers its own defaults via `merge_settings` during startu
 
 ### Secret Resolution
 
-Settings values can reference external secret sources using URI syntax. Two schemes are supported:
+Settings values can reference external secret sources using URI syntax. Three schemes are supported:
 
 | Scheme | Format | Resolution |
 |--------|--------|------------|
-| `vault://` | `vault://path/to/secret#key` | Reads from HashiCorp Vault via `Legion::Crypt` |
+| `vault://` | `vault://path/to/secret#key` | Reads static KV secrets from HashiCorp Vault via `Legion::Crypt` |
 | `env://` | `env://ENV_VAR_NAME` | Reads from environment variable |
+| `lease://` | `lease://name#key` | Reads from dynamic Vault leases via `Legion::Crypt::LeaseManager` |
 
 Array values act as fallback chains — the first non-nil result wins:
 
@@ -59,7 +60,7 @@ Call `Legion::Settings.resolve_secrets!` to resolve all URIs in-place. In the Le
 
 ```ruby
 Legion::Settings.resolve_secrets!
-# All vault:// and env:// references are now replaced with their resolved values
+# All vault://, env://, and lease:// references are now replaced with their resolved values
 ```
 
 ### Schema Validation

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/loader.rb

@@ -38,7 +38,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,
@@ -69,6 +87,10 @@ module Legion
         to_hash[key]
       end
 
+      def dig(*keys)
+        to_hash.dig(*keys)
+      end
+
       def []=(key, value)
         @settings[key] = value
         @indifferent_access = false
@@ -116,8 +138,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}")

data/lib/legion/settings/version.rb

@@ -2,6 +2,6 @@
 
 module Legion
   module Settings
-    VERSION = '1.3.1'
+    VERSION = '1.3.4'
   end
 end

data/lib/legion/settings.rb

@@ -37,6 +37,13 @@ module Legion
         nil
       end
 
+      def dig(*keys)
+        @loader = load if @loader.nil?
+        @loader.dig(*keys)
+      rescue NoMethodError, TypeError
+        nil
+      end
+
       def set_prop(key, value)
         @loader = load if @loader.nil?
         @loader[key] = value

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: legion-settings
 version: !ruby/object:Gem::Version
-  version: 1.3.1
+  version: 1.3.4
 platform: ruby
 authors:
 - Esity
@@ -42,6 +42,8 @@ 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/loader.rb