legionio 1.7.13 → 1.7.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 42d28272b4f314a736fdd339106511ef743107d7df2b1a771d403a8312d66135
-  data.tar.gz: aa616bda3a27ae36e9524a5dcdc4f027d730274cd04f4e45a1142accc638365c
+  metadata.gz: 86fc5b0df46813866069443960f455bfd54c68eab894e96e23c763ef1c3356e2
+  data.tar.gz: f3881012c84c5eaf039ae0c46243e9a97d5aa2ef53092a5b733a1cdeeacf9d08
 SHA512:
-  metadata.gz: e1b1d5265860f317faf0da4a23e5e25373e290e4b4456acc0ceffdd3b6f3b8f65f97a5d78c0ca199f7882ab1776e9a17cc585a11dd8eb3ac8f75d6b9ef4b7772
-  data.tar.gz: bd82d4424862d625639be08c340686b0eb830f1cd4002116c69dd525116576685761c0624b6dddfa19a789b6c98561444a88c89e8a8a002a846c3ca01494aa8a
+  metadata.gz: f191aabb91a76ac66d6d71890c2fcdc273f4cbd60f77768a024064b8f58386a6e2791073364c6deb93dfd95b5a8fb8485b0b26c7d0dad67eb5c3d40a1c337289
+  data.tar.gz: bb61732fe43b41f23e8bb96defb087c206d5d397bd165e3c6eef5c16f9b12e542f88de846639fb767300c5050960f0b7298d83be3d720ab80d5d11096a1008ae

data/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 ## [Unreleased]
 
+## [1.7.14] - 2026-04-03
+
+### Fixed
+- Actor boot ordering: once → poll → every → loop → subscriptions, preventing timer actors from competing with AMQP channel setup
+- Builder now respects `remote_invocable? false` and skips auto-generated subscription actors for local-only extensions
+- Catalog exchange cached and reused instead of creating a new channel + exchange_declare per transition
+- Catalog SQLite persists batched into a single transaction at end of boot instead of per-transition writes from concurrent threads
+
 ## [1.7.13] - 2026-04-03
 
 ### Changed

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

@@ -37,6 +37,11 @@ module Legion
         end
 
         def build_meta_actor_list
+          if lex_class.respond_to?(:remote_invocable?) && !lex_class.remote_invocable?
+            log.debug "[Actors] skipping meta actors for #{lex_class} (remote_invocable=false)"
+            return
+          end
+
           @runners.each do |runner, attr|
             next if @actors[runner.to_sym].is_a? Hash
 

data/lib/legion/extensions/catalog.rb

@@ -61,6 +61,40 @@ module Legion
           @warned_missing_extension_catalog = false
         end
 
+        def flush_persisted_transitions
+          pending = nil
+          @pending_persists_mutex ||= Mutex.new
+          @pending_persists_mutex.synchronize do
+            return if @pending_persists.nil? || @pending_persists.empty?
+
+            pending = @pending_persists.dup
+            @pending_persists.clear
+          end
+
+          return unless defined?(Legion::Data::Local) &&
+                        Legion::Data::Local.respond_to?(:connected?) &&
+                        Legion::Data::Local.connected?
+
+          ensure_local_migration_registered!
+          return warn_missing_extension_catalog_once unless extension_catalog_table_available?
+
+          model = Legion::Data::Local.model(:extension_catalog)
+          now = Time.now
+          Legion::Data::Local.connection.transaction do
+            pending.each do |lex_name, new_state|
+              existing = model.where(lex_name: lex_name).first
+              if existing
+                existing.update(state: new_state.to_s, updated_at: now)
+              else
+                model.insert(lex_name: lex_name, state: new_state.to_s, created_at: now, updated_at: now)
+              end
+            end
+          end
+          Legion::Logging.info "Catalog persisted #{pending.size} transitions" if defined?(Legion::Logging)
+        rescue StandardError => e
+          Legion::Logging.warn { "Catalog flush failed: #{e.class}: #{e.message}" } if defined?(Legion::Logging)
+        end
+
         private
 
         def entries
@@ -78,30 +112,25 @@ module Legion
             timestamp: Time.now.to_i
           )
 
-          exchange = Legion::Transport::Exchange.new('legion.catalog')
-          exchange.publish(payload, routing_key: "legion.catalog.#{lex_name}.#{new_state}",
-                                    content_type: 'application/json', persistent: true)
+          catalog_exchange.publish(payload, routing_key: "legion.catalog.#{lex_name}.#{new_state}",
+                                            content_type: 'application/json', persistent: true)
         rescue StandardError => e
+          @catalog_exchange = nil
           Legion::Logging.warn { "Catalog publish failed for #{lex_name}=#{new_state}: #{e.class}: #{e.message}" } if defined?(Legion::Logging)
         end
 
-        def persist_transition(lex_name, new_state)
-          return unless defined?(Legion::Data::Local) &&
-                        Legion::Data::Local.respond_to?(:connected?) &&
-                        Legion::Data::Local.connected?
+        def catalog_exchange
+          return @catalog_exchange if @catalog_exchange&.channel&.open?
 
-          ensure_local_migration_registered!
-          return warn_missing_extension_catalog_once unless extension_catalog_table_available?
+          @catalog_exchange = Legion::Transport::Exchange.new('legion.catalog')
+        end
 
-          model = Legion::Data::Local.model(:extension_catalog)
-          existing = model.where(lex_name: lex_name).first
-          if existing
-            existing.update(state: new_state.to_s, updated_at: Time.now)
-          else
-            model.insert(lex_name: lex_name, state: new_state.to_s, created_at: Time.now, updated_at: Time.now)
+        def persist_transition(lex_name, new_state)
+          @pending_persists_mutex ||= Mutex.new
+          @pending_persists_mutex.synchronize do
+            @pending_persists ||= {}
+            @pending_persists[lex_name] = new_state
           end
-        rescue StandardError => e
-          Legion::Logging.warn { "Catalog persist failed for #{lex_name}=#{new_state}: #{e.class}: #{e.message}" } if defined?(Legion::Logging)
         end
 
         def extension_catalog_table_available?

data/lib/legion/extensions.rb

@@ -279,16 +279,18 @@ module Legion
 
         Legion::Logging.info "Hooking #{@pending_actors.size} deferred actors"
 
-        sub_actors = []
-        @pending_actors.each do |actor|
-          if actor[:actor_class].ancestors.include?(Legion::Extensions::Actors::Subscription)
-            sub_actors << actor
-          else
-            hook_actor(**actor)
-          end
-        end
+        groups = group_pending_actors
 
-        hook_subscription_actors_pooled(sub_actors) unless sub_actors.empty?
+        %i[once poll every loop].each do |type|
+          next if groups[type].empty?
+
+          Legion::Logging.info "Starting #{type} actors (#{groups[type].size})"
+          groups[type].each { |actor| hook_actor(**actor) }
+        end
+        unless groups[:subscription].empty?
+          Legion::Logging.info "Starting subscription actors (#{groups[:subscription].size})"
+          hook_subscription_actors_pooled(groups[:subscription])
+        end
         dispatch_local_actors(@local_tasks) unless @local_tasks.empty?
 
         @pending_actors.clear
@@ -301,6 +303,33 @@ module Legion
           "local:#{@local_tasks.count}"
         )
         @loaded_extensions&.each { |name| Catalog.transition(name, :running) }
+        Catalog.flush_persisted_transitions
+      end
+
+      ACTOR_TYPE_MAP = {
+        Once:         :once,
+        Poll:         :poll,
+        Every:        :every,
+        Loop:         :loop,
+        Subscription: :subscription
+      }.freeze
+
+      def group_pending_actors
+        groups = { once: [], poll: [], every: [], loop: [], subscription: [] }
+        @pending_actors.each do |actor|
+          type = resolve_actor_type(actor[:actor_class])
+          groups[type] << actor
+        end
+        groups
+      end
+
+      def resolve_actor_type(actor_class)
+        anc = actor_class.ancestors
+        ACTOR_TYPE_MAP.each do |const, type|
+          return type if anc.include?(Legion::Extensions::Actors.const_get(const))
+        end
+        Legion::Logging.warn "Unknown actor type for #{actor_class}, defaulting to loop"
+        :loop
       end
 
       def hook_actor(extension:, extension_name:, actor_class:, size: 1, **opts)

data/lib/legion/version.rb

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

metadata

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