lex-codegen 0.2.4 → 0.2.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: eed547e1328af8116c12a43b51f3dc68ddafdc0d1d17bd7736df863c126c6fba
-  data.tar.gz: 275a664f737f601f95ad61ae9b73c10a4274ec36f8915dddc555e9ac9e966093
+  metadata.gz: 5ae1fad2e903314b7587a1d1173e336b9ab35a1cd6ba69190d2c1a4fb05a4018
+  data.tar.gz: acbad453e8e835facdcebe94d16ccfdea26423a1ab0e17266ac4323ba0c08511
 SHA512:
-  metadata.gz: f38f7774034f1f24428d4181fa3ef593d648ffdc62932d1eb3cba6c216167678c2441f3e8f6769d8016f4e1aeecaca9dbd27506126b6a62bc339277fced8fd5a
-  data.tar.gz: 607c285280a1429d2bcdad8ab3be42279a1bb13b3e5816554a178e65c08b5396633b2c0e5510c3b1b0eab766ef331433afe964adf5a76f71e5ac6abeb77d7e4e
+  metadata.gz: cd2748dd1d307b4333af2e4770c61f6a178172f66fd581ef4b431a565aaaef44990d29376a78782ec1eeb831500ebcc71c7f8e2ac0d38ecd1773585bc2c4f019
+  data.tar.gz: f0f5c698c6333dbd9f576e712f9948446307fa615ad53722c799b9ae0791b33484370ef796def7b9b75a8bf7fb703cb7fba0787d36d5934e5ffaef3a5aa45d98

data/CHANGELOG.md

@@ -1,5 +1,39 @@
 # Changelog
 
+## [0.2.10] - 2026-03-27
+
+### Fixed
+- Capture exception in `GeneratedRegistry#db_available?` rescue block (`rescue StandardError => e`) and log debug message to satisfy CI Rescue Logging lint
+
+## [0.2.9] - 2026-03-27
+
+### Fixed
+- Replace all `Legion::Logging.*` direct calls with `log.*` helper calls across actors, runners, and helpers to satisfy CI Helper Migration lint check
+- Fix `log` method in actors to return a null logger fallback instead of `nil` when `Legion::Logging` is undefined, so `log.warn` never raises `NoMethodError`
+- Add private `log` method to `Runners::Generate`, `Runners::ReviewHandler`, and `Helpers::GeneratedRegistry` (all `extend self` / `module_function` modules)
+- Update `RUNNER_TEMPLATE` in `Helpers::Constants` to use `log.debug` instead of `Legion::Logging.debug` in generated runner stubs
+
+## [0.2.8] - 2026-03-27
+
+### Fixed
+- GapSubscriber and ReviewSubscriber: replace `log&.warn`/`log&.debug` in rescue blocks with `Legion::Logging.warn`/`Legion::Logging.debug` so the CI rescue-logging lint script matches the pattern correctly
+
+## [0.2.7] - 2026-03-27
+
+### Fixed
+- GapSubscriber and ReviewSubscriber: added `runner_class` and `runner_function` overrides so `Subscription#prepare` can resolve the runner constant
+
+## [0.2.6] - 2026-03-27
+
+### Fixed
+- Actors: `GapSubscriber` and `ReviewSubscriber` changed from modules to classes inheriting `Actors::Subscription`
+- Actor files guarded with `return unless defined?` for standalone spec compatibility
+
+## [0.2.5] - 2026-03-26
+
+### Changed
+- set remote_invocable? false for local dispatch
+
 ## [0.2.4] - 2026-03-26
 
 ### Security

data/lib/legion/extensions/codegen/actors/gap_subscriber.rb

@@ -1,12 +1,17 @@
 # frozen_string_literal: true
 
+return unless defined?(Legion::Extensions::Actors::Subscription)
+
 module Legion
   module Extensions
     module Codegen
       module Actor
-        module GapSubscriber
+        class GapSubscriber < Legion::Extensions::Actors::Subscription
           QUEUE = Transport::Queues::GapDetected if defined?(Transport::Queues::GapDetected)
 
+          def runner_class = self.class
+          def runner_function = 'action'
+
           def action(payload)
             gap = normalize_gap(payload)
 
@@ -20,7 +25,7 @@ module Legion
 
             result
           rescue StandardError => e
-            log&.warn("GapSubscriber failed: #{e.message}")
+            log.warn("GapSubscriber failed: #{e.message}")
             { success: false, error: e.message }
           end
 
@@ -47,7 +52,7 @@ module Legion
               source:  { provider: node_name, channel: 'gap_detector' }
             )
           rescue StandardError => e
-            log&.debug("GapSubscriber: Apollo ingest failed: #{e.message}")
+            log.debug("GapSubscriber: Apollo ingest failed: #{e.message}")
           end
 
           def query_corroboration(gap)
@@ -62,7 +67,7 @@ module Legion
             results = result[:results] || []
             results.map { |r| r.dig(:source, :provider) }.compact.uniq.size
           rescue StandardError => e
-            log&.debug("GapSubscriber: Apollo query failed: #{e.message}")
+            log.debug("GapSubscriber: Apollo query failed: #{e.message}")
             0
           end
 
@@ -106,9 +111,11 @@ module Legion
           end
 
           def log
-            return unless defined?(Legion::Logging)
+            return Legion::Logging if defined?(Legion::Logging)
 
-            Legion::Logging
+            @log ||= Object.new.tap do |nl|
+              %i[debug info warn error fatal].each { |m| nl.define_singleton_method(m) { |*| nil } }
+            end
           end
         end
       end

data/lib/legion/extensions/codegen/actors/review_subscriber.rb

@@ -1,12 +1,17 @@
 # frozen_string_literal: true
 
+return unless defined?(Legion::Extensions::Actors::Subscription)
+
 module Legion
   module Extensions
     module Codegen
       module Actor
-        module ReviewSubscriber
+        class ReviewSubscriber < Legion::Extensions::Actors::Subscription
           QUEUE = Transport::Queues::ReviewCompleted if defined?(Transport::Queues::ReviewCompleted)
 
+          def runner_class = self.class
+          def runner_function = 'action'
+
           def action(payload)
             review = {
               generation_id: payload[:generation_id],
@@ -18,16 +23,18 @@ module Legion
 
             Runners::ReviewHandler.handle_verdict(review: review)
           rescue StandardError => e
-            log&.warn("ReviewSubscriber failed: #{e.message}")
+            log.warn("ReviewSubscriber failed: #{e.message}")
             { success: false, error: e.message }
           end
 
           private
 
           def log
-            return unless defined?(Legion::Logging)
+            return Legion::Logging if defined?(Legion::Logging)
 
-            Legion::Logging
+            @log ||= Object.new.tap do |nl|
+              %i[debug info warn error fatal].each { |m| nl.define_singleton_method(m) { |*| nil } }
+            end
           end
         end
       end

data/lib/legion/extensions/codegen/helpers/constants.rb

@@ -252,7 +252,7 @@ module Legion
             <% methods.each do |m| -%>
 
                       def <%= m[:name] %>(<%= m[:params].join(', ') %><%= m[:params].empty? ? '**' : ', **' %>)
-                        Legion::Logging.debug "[<%= gem_name_underscored %>] <%= m[:name] %> called"
+                        log.debug "[<%= gem_name_underscored %>] <%= m[:name] %> called"
                         { success: true }
                       end
             <% end -%>

data/lib/legion/extensions/codegen/helpers/generated_registry.rb

@@ -86,7 +86,7 @@ module Legion
                 Kernel.load(record[:file_path])
                 loaded += 1
               rescue StandardError => e
-                Legion::Logging.warn("GeneratedRegistry: failed to load #{record[:file_path]}: #{e.message}") if defined?(Legion::Logging)
+                log.warn("GeneratedRegistry: failed to load #{record[:file_path]}: #{e.message}")
               end
             end
 
@@ -101,9 +101,18 @@ module Legion
             @store ||= {}
           end
 
+          def log
+            return Legion::Logging if defined?(Legion::Logging)
+
+            @log ||= Object.new.tap do |nl|
+              %i[debug info warn error fatal].each { |m| nl.define_singleton_method(m) { |*| nil } }
+            end
+          end
+
           def db_available?
             defined?(Legion::Data::Local) && Legion::Data::Local.respond_to?(:db) && !Legion::Data::Local.db.nil?
-          rescue StandardError
+          rescue StandardError => e
+            log.debug("db_available? check failed: #{e.message}")
             false
           end
 

data/lib/legion/extensions/codegen/runners/generate.rb

@@ -32,7 +32,7 @@ module Legion
             files = build_scaffold_files(engine, variables, helpers, runner_methods, spec_gen, module_name, underscored, runner_names)
             writer.write_all(files)
 
-            Legion::Logging.info "[codegen] scaffolded #{gem_name} with #{files.size} files at #{ext_path}"
+            log.info "[codegen] scaffolded #{gem_name} with #{files.size} files at #{ext_path}"
             { success: true, path: ext_path, files_created: files.size, name: gem_name }
           rescue ArgumentError => e
             { success: false, error: e.message }
@@ -50,6 +50,14 @@ module Legion
 
           private
 
+          def log
+            return Legion::Logging if defined?(Legion::Logging)
+
+            @log ||= Object.new.tap do |nl|
+              %i[debug info warn error fatal].each { |m| nl.define_singleton_method(m) { |*| nil } }
+            end
+          end
+
           def build_variables(gem_name:, underscored:, module_name:, description:, helpers:, runner_names:, extra_deps:)
             {
               gem_name:             gem_name,

data/lib/legion/extensions/codegen/runners/review_handler.rb

@@ -28,6 +28,14 @@ module Legion
 
           private
 
+          def log
+            return Legion::Logging if defined?(Legion::Logging)
+
+            @log ||= Object.new.tap do |nl|
+              %i[debug info warn error fatal].each { |m| nl.define_singleton_method(m) { |*| nil } }
+            end
+          end
+
           def approve(record, review)
             Helpers::GeneratedRegistry.update_status(id: record[:id], status: 'approved')
 
@@ -35,7 +43,7 @@ module Legion
               begin
                 Kernel.load(record[:file_path])
               rescue StandardError => e
-                Legion::Logging.warn("ReviewHandler: load failed: #{e.message}") if defined?(Legion::Logging)
+                log.warn("ReviewHandler: load failed: #{e.message}")
               end
             end
 

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

@@ -3,7 +3,7 @@
 module Legion
   module Extensions
     module Codegen
-      VERSION = '0.2.4'
+      VERSION = '0.2.10'
     end
   end
 end

data/lib/legion/extensions/codegen.rb

@@ -31,6 +31,10 @@ module Legion
   module Extensions
     module Codegen
       extend Legion::Extensions::Core if Legion::Extensions.const_defined? :Core
+
+      def self.remote_invocable?
+        false
+      end
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lex-codegen
 version: !ruby/object:Gem::Version
-  version: 0.2.4
+  version: 0.2.10
 platform: ruby
 authors:
 - Esity