legionio 1.6.29 → 1.6.31

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f3a73a5c562d8c349f10592682acdfebc23bd0d5bc3cea80286be00d8b72aff4
-  data.tar.gz: a789defca2fbe696da75046a45f003eb49e6247fb102e5994d3a9bb57c8b9edf
+  metadata.gz: 56ff571f5c8480be8a6e783549dd9cd377805dd5590e0e07a7fa60f2f0da39da
+  data.tar.gz: 6d6251a3b04602caffb8e76b12fc1677c7caf1981a5e7f9cd73c1bea52bf97d6
 SHA512:
-  metadata.gz: 734077acd813bbc111cb6613f8046873370d5db6f1c6208127eefefa318ea4afbe947205f721cbd2d2e08b47166f315735fbdb6fd64a69a376bff8cee9ec7986
-  data.tar.gz: ed8bbee89db76679597caade809ebc1a6ceb5ac2c7664456f01e693ac3cf899d28716dc493de2911bf723e02c15ec56d135326a8fbdaf4fb7870434b8bfef3a7
+  metadata.gz: 9b40b7e385bfa89fd4642ea18bc0bc014ba6407dc182f3ffbb8462963e45c9afffa0471015cda08a77c78ad71eabb31709cfe69a967653ae531c3457e5b90419
+  data.tar.gz: c6fa233e90846bf0296f1f9ba3adf171d9ccfa34d224de96fcd721e1e013218c3cfc778988a80d661d57a93ca1002a1b57ffba0a75033aa40b657b1f92c2e9f3

data/CHANGELOG.md

@@ -2,8 +2,23 @@
 
 ## [Unreleased]
 
+## [1.6.31] - 2026-03-28
+
+### Fixed
+- `build_hook_list` in `Builders::Hooks` now calls `runner_class` on a hook instance (instance method) instead of the class, preventing the `TypeError: no implicit conversion of nil into String` boot crash caused by `Helpers::Base#runner_class` being inherited at the class level and calling `sub!` on a string that contains no `'Actor'` substring
+- `Helpers::Base#runner_class` changed `sub!` to `sub` (non-destructive) as a defensive fix — `sub!` returns `nil` when no substitution is made, which caused `Kernel.const_get(nil)` to raise `TypeError`
+- Runner reference returned by `hook_class.new.runner_class` is now resolved safely: string class names are resolved via `Kernel.const_defined?` + `Kernel.const_get`; Class objects are used directly; `nil` falls back to `hook_class`
+
+## [1.6.30] - 2026-03-28
+
+### Fixed
+- `Legion::Extensions::Hooks::Base` now defines the `mount(path)` DSL method and `mount_path` reader — fixes `NoMethodError` boot crash in any extension hook that calls `mount` (e.g. `lex-microsoft_teams` `Hooks::Auth`)
+
 ## [1.6.29] - 2026-03-28
 
+### Removed
+- `ClassMethods` module (`expose_as_mcp_tool`, `mcp_tool_prefix`) from `Legion::Extensions::Helpers::Lex` — deprecated since the definition DSL was introduced; zero extensions use them
+
 ### Fixed
 - Fallback route guards in `api.rb` now check `router.library_names.include?` instead of `defined?` — prevents 404s when gem modules are loaded but routes are not yet mounted (fixes #53)
 

data/lib/legion/extensions/builders/hooks.rb

@@ -29,7 +29,7 @@ module Legion
             next unless hook_class < Legion::Extensions::Hooks::Base
 
             route_path = "#{extension_name}/#{hook_name}"
-            runner = hook_class.respond_to?(:runner_class) ? hook_class.runner_class : nil
+            runner = resolve_hook_runner(hook_class)
 
             @hooks[hook_name.to_sym] = {
               extension:      lex_class.to_s.downcase,
@@ -61,6 +61,17 @@ module Legion
         def hook_files
           @hook_files ||= find_files('hooks')
         end
+
+        private
+
+        def resolve_hook_runner(hook_class)
+          ref = hook_class.new.runner_class
+          if ref.is_a?(String)
+            Kernel.const_defined?(ref) ? Kernel.const_get(ref) : nil
+          elsif ref.is_a?(Class)
+            ref
+          end
+        end
       end
     end
   end

data/lib/legion/extensions/helpers/base.rb

@@ -83,7 +83,7 @@ module Legion
         end
 
         def runner_class
-          @runner_class ||= Kernel.const_get(actor_class.to_s.sub!('Actor', 'Runners'))
+          @runner_class ||= Kernel.const_get(actor_class.to_s.sub('Actor', 'Runners'))
         end
 
         def runner_name

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

@@ -12,32 +12,6 @@ module Legion
         include Legion::JSON::Helper
         include Legion::Extensions::Helpers::Secret
 
-        module ClassMethods
-          # @deprecated Use mcp_exposed: flag in definition DSL instead
-          def expose_as_mcp_tool(value = :_unset)
-            if value == :_unset
-              return @expose_as_mcp_tool unless @expose_as_mcp_tool.nil?
-
-              if defined?(Legion::Settings) && Legion::Settings.respond_to?(:dig)
-                Legion::Settings.dig(:mcp, :auto_expose_runners) || false
-              else
-                false
-              end
-            else
-              @expose_as_mcp_tool = value
-            end
-          end
-
-          # @deprecated Use mcp_exposed: flag in definition DSL instead
-          def mcp_tool_prefix(value = :_unset)
-            if value == :_unset
-              @mcp_tool_prefix
-            else
-              @mcp_tool_prefix = value
-            end
-          end
-        end
-
         def runner_desc(desc)
           settings[:runners] = {} if settings[:runners].nil?
           settings[:runners][actor_name.to_sym] = {} if settings[:runners][actor_name.to_sym].nil?
@@ -47,7 +21,6 @@ module Legion
         def self.included(base)
           base.send :extend, Legion::Extensions::Helpers::Core if base.instance_of?(Class)
           base.send :extend, Legion::Extensions::Helpers::Logger if base.instance_of?(Class)
-          base.extend ClassMethods if base.instance_of?(Class)
           base.extend base if base.instance_of?(Module)
         end
 

data/lib/legion/extensions/hooks/base.rb

@@ -47,8 +47,14 @@ module Legion
             @verify_config = { header: header.upcase.tr('-', '_'), secret: secret }
           end
 
+          # DSL: declare a sub-path suffix appended to the auto-generated hook route
+          #   mount '/callback'  # e.g. /api/extensions/microsoft_teams/hooks/auth/callback
+          def mount(path)
+            @mount_path = path
+          end
+
           attr_reader :route_type, :route_header_name, :route_field_name,
-                      :route_mapping, :verify_type, :verify_config
+                      :route_mapping, :verify_type, :verify_config, :mount_path
         end
 
         # Instance methods called by the API layer

data/lib/legion/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Legion
-  VERSION = '1.6.29'
+  VERSION = '1.6.31'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: legionio
 version: !ruby/object:Gem::Version
-  version: 1.6.29
+  version: 1.6.31
 platform: ruby
 authors:
 - Esity