lex-lex 0.2.1 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3777b77e3f3d3225f3f6460b7657da500864831adbd5968508f5680f0a0db8ba
-  data.tar.gz: afcb223bac1af680e38c6efc4923c9d5211a6ab9e15d4ee8d52d0e3a72ec2f27
+  metadata.gz: 6dc3e76881ecf51e9b1db09057880ced19b7daffcb8b997ec3c338166a924f4a
+  data.tar.gz: 248dffd3c5999db11f7248cccd5a23330f261905eb62e32e649d7fd08364c2c7
 SHA512:
-  metadata.gz: 6e8cbd38ec2f50c1a32fae430338865aeb871441036ca68ae02db68aaa4c9c981dedefef1c97a710c5738af37361a34a250f929d63cd764e594624faba8de756
-  data.tar.gz: 7521e32b17b077f98ca453228ea797fcdc16a1982e90931b8ca5e83c1b855f4a30340bb7e31b4c825389dd27041dd8dff5fa0ef740804422df70edd53c75ca86
+  metadata.gz: 480a5427ba9ccf651a4b09c1e6f0a531cb099a4f53478969b62f41030199e00ace1a94aa89b7dde66ef82a357d293b2eb71d53f5cfea4f61501328d93bc72105
+  data.tar.gz: a0f7ac11038cdcb7b3cbe2e3b362e95389b624bb8c57e2cb92b3908d743288179a5b1ec96a1585328ac367d2aef9c32e12c1a6cf3188c2fa061cba4455ef4d22

data/.gitignore

@@ -7,5 +7,7 @@
 /spec/reports/
 /tmp/
 
+Gemfile.lock
+
 # rspec failure tracking
 .rspec_status

data/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Changelog
 
+## [0.3.0] - 2026-03-18
+
+### Fixed
+- `data_required?` now correctly overrides Core default (was instance method, framework ignored it)
+- Sync runner only increments update counter on actual DB writes
+- Sync runner no longer re-enables intentionally disabled extensions
+- Register.save guards against nil extension_id after creation failure
+
+### Changed
+- Renamed shadowed `update` local variables to `changes` in Extension, Runner, Function modules
+
 ## [0.2.1] - 2026-03-17
 
 ### Fixed

data/CLAUDE.md

@@ -12,7 +12,7 @@ Without lex-lex, the extensions/runners/functions DB tables remain empty, and th
 
 **GitHub**: https://github.com/LegionIO/lex-lex
 **License**: MIT
-**Version**: 0.2.0
+**Version**: 0.3.0
 
 ## Architecture
 
@@ -55,7 +55,7 @@ Startup (after all extensions load):
 | Path | Purpose |
 |------|---------|
 | `lib/legion/extensions/lex.rb` | Entry point, requires all runners |
-| `lib/legion/extensions/lex/version.rb` | VERSION constant (0.2.0) |
+| `lib/legion/extensions/lex/version.rb` | VERSION constant (0.3.0) |
 | `lib/legion/extensions/lex/runners/extension.rb` | Extension CRUD |
 | `lib/legion/extensions/lex/runners/runner.rb` | Runner CRUD |
 | `lib/legion/extensions/lex/runners/function.rb` | Function CRUD + build_args |

data/lib/legion/extensions/lex/runners/extension.rb

@@ -22,18 +22,18 @@ module Legion
             extension = Legion::Data::Model::Extension[extension_id]
             return { success: false, reason: 'extension not found' } if extension.nil?
 
-            update = {}
+            changes = {}
             %i[name namespace active exchange uri].each do |column|
               next unless opts.key?(column)
               next if extension.values[column] == opts[column]
 
-              update[column] = opts[column]
+              changes[column] = opts[column]
             end
 
-            return { success: true, changed: false, extension_id: extension_id } if update.empty?
+            return { success: true, changed: false, extension_id: extension_id } if changes.empty?
 
-            extension.update(update)
-            { success: true, changed: true, updates: update, extension_id: extension_id }
+            extension.update(changes)
+            { success: true, changed: true, updates: changes, extension_id: extension_id }
           end
 
           def get(extension_id: nil, name: nil, namespace: nil, **_opts)

data/lib/legion/extensions/lex/runners/function.rb

@@ -22,18 +22,18 @@ module Legion
             function = Legion::Data::Model::Function[function_id]
             return { success: false, reason: 'function not found' } if function.nil?
 
-            update = {}
-            update[:active] = opts[:active] if opts.key?(:active) && function.values[:active] != opts[:active]
+            changes = {}
+            changes[:active] = opts[:active] if opts.key?(:active) && function.values[:active] != opts[:active]
 
             if opts.key?(:formatted_args)
               args = Legion::JSON.dump(opts[:formatted_args])
-              update[:args] = args unless args == function.values[:args]
+              changes[:args] = args unless args == function.values[:args]
             end
 
-            return { success: true, changed: false, function_id: function_id } if update.empty?
+            return { success: true, changed: false, function_id: function_id } if changes.empty?
 
-            function.update(update)
-            { success: true, changed: true, updates: update, function_id: function_id }
+            function.update(changes)
+            { success: true, changed: true, updates: changes, function_id: function_id }
           end
 
           def get(function_id:, **_opts)

data/lib/legion/extensions/lex/runners/register.rb

@@ -22,6 +22,8 @@ module Legion
                   name:      runner_opts[:extension_name],
                   namespace: runner_opts[:extension_class].to_s
                 )
+                return { success: false, error: 'extension creation failed' } if ext_result[:extension_id].nil?
+
                 extension_id = ext_result[:extension_id]
               end
 

data/lib/legion/extensions/lex/runners/runner.rb

@@ -27,18 +27,18 @@ module Legion
             runner = Legion::Data::Model::Runner[runner_id]
             return { success: false, reason: 'runner not found' } if runner.nil?
 
-            update = {}
+            changes = {}
             %i[name namespace active queue uri].each do |column|
               next unless opts.key?(column)
               next if runner.values[column] == opts[column]
 
-              update[column] = opts[column]
+              changes[column] = opts[column]
             end
 
-            return { success: true, changed: false, runner_id: runner_id } if update.empty?
+            return { success: true, changed: false, runner_id: runner_id } if changes.empty?
 
-            runner.update(update)
-            { success: true, changed: true, updates: update, runner_id: runner_id }
+            runner.update(changes)
+            { success: true, changed: true, updates: changes, runner_id: runner_id }
           end
 
           def get(runner_id:, **_opts)

data/lib/legion/extensions/lex/runners/sync.rb

@@ -33,8 +33,10 @@ module Legion
                 created += 1
               else
                 ns = values[:extension_class].to_s
-                existing.update(namespace: ns, active: true) if existing.values[:namespace] != ns
-                updated += 1
+                if existing.values[:namespace] != ns
+                  existing.update(namespace: ns)
+                  updated += 1
+                end
               end
               synced += 1
             end

data/lib/legion/extensions/lex/version.rb

@@ -3,7 +3,7 @@
 module Legion
   module Extensions
     module Lex
-      VERSION = '0.2.1'
+      VERSION = '0.3.0'
     end
   end
 end

data/lib/legion/extensions/lex.rb

@@ -12,7 +12,7 @@ module Legion
     module Lex
       extend Legion::Extensions::Core if Legion::Extensions.const_defined?(:Core)
 
-      def data_required?
+      def self.data_required?
         true
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lex-lex
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.3.0
 platform: ruby
 authors:
 - Esity
@@ -108,7 +108,6 @@ files:
 - CHANGELOG.md
 - CLAUDE.md
 - Gemfile
-- Gemfile.lock
 - LICENSE.txt
 - README.md
 - lex-lex.gemspec

data/Gemfile.lock

@@ -1,91 +0,0 @@
-PATH
-  remote: .
-  specs:
-    lex-lex (0.2.1)
-
-GEM
-  remote: https://rubygems.org/
-  specs:
-    addressable (2.8.9)
-      public_suffix (>= 2.0.2, < 8.0)
-    ast (2.4.3)
-    bigdecimal (4.0.1)
-    diff-lcs (1.6.2)
-    docile (1.4.1)
-    json (2.19.1)
-    json-schema (6.2.0)
-      addressable (~> 2.8)
-      bigdecimal (>= 3.1, < 5)
-    language_server-protocol (3.17.0.5)
-    lint_roller (1.1.0)
-    mcp (0.8.0)
-      json-schema (>= 4.1)
-    parallel (1.27.0)
-    parser (3.3.10.2)
-      ast (~> 2.4.1)
-      racc
-    prism (1.9.0)
-    public_suffix (7.0.5)
-    racc (1.8.1)
-    rainbow (3.1.1)
-    rake (13.3.1)
-    regexp_parser (2.11.3)
-    rspec (3.13.2)
-      rspec-core (~> 3.13.0)
-      rspec-expectations (~> 3.13.0)
-      rspec-mocks (~> 3.13.0)
-    rspec-core (3.13.6)
-      rspec-support (~> 3.13.0)
-    rspec-expectations (3.13.5)
-      diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.13.0)
-    rspec-mocks (3.13.8)
-      diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.13.0)
-    rspec-support (3.13.7)
-    rspec_junit_formatter (0.6.0)
-      rspec-core (>= 2, < 4, != 2.12.0)
-    rubocop (1.85.1)
-      json (~> 2.3)
-      language_server-protocol (~> 3.17.0.2)
-      lint_roller (~> 1.1.0)
-      mcp (~> 0.6)
-      parallel (~> 1.10)
-      parser (>= 3.3.0.2)
-      rainbow (>= 2.2.2, < 4.0)
-      regexp_parser (>= 2.9.3, < 3.0)
-      rubocop-ast (>= 1.49.0, < 2.0)
-      ruby-progressbar (~> 1.7)
-      unicode-display_width (>= 2.4.0, < 4.0)
-    rubocop-ast (1.49.1)
-      parser (>= 3.3.7.2)
-      prism (~> 1.7)
-    rubocop-rspec (3.9.0)
-      lint_roller (~> 1.1)
-      rubocop (~> 1.81)
-    ruby-progressbar (1.13.0)
-    simplecov (0.22.0)
-      docile (~> 1.1)
-      simplecov-html (~> 0.11)
-      simplecov_json_formatter (~> 0.1)
-    simplecov-html (0.13.2)
-    simplecov_json_formatter (0.1.4)
-    unicode-display_width (3.2.0)
-      unicode-emoji (~> 4.1)
-    unicode-emoji (4.2.0)
-
-PLATFORMS
-  arm64-darwin-25
-  ruby
-
-DEPENDENCIES
-  lex-lex!
-  rake
-  rspec (~> 3.13)
-  rspec_junit_formatter
-  rubocop (~> 1.75)
-  rubocop-rspec
-  simplecov
-
-BUNDLED WITH
-   2.6.9