legion-data 1.8.1 → 1.8.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9b3a56b1330f3ec71addc28b9b3b02e6ed845ff80c9993073bde8130fee363ef
-  data.tar.gz: ccfdf0a86f6408b201065d414ab6199cacf6610e0bfbb527449e59cf970fddb8
+  metadata.gz: d9f440b5e825e600fe33f7b07f6aa7aeb11cde67069eea1960e01ed6fcfb5ee2
+  data.tar.gz: b60960ed3f6cf3f870c92a309dd8ee53d11f7b1f672e747a35c0ca11e8267074
 SHA512:
-  metadata.gz: b40d2cde196b2c8fec186671ee7973ac85893239e9502445d975ccfaca43efdd8170e04082869f30bd0544cb42a9b090b1190c5ef942c6afa37fa07622874854
-  data.tar.gz: c9e5c0e72a62c423a532b1be4d34ec98fd6b7b9611232939e5e57f3e61e3f56cc14eb19c5df57855cb3735ee76f6e00338e6a610fa81989e0acfba27c4fceb6a
+  metadata.gz: 345dd816ac680066d8964506e9c14f72638b694ef5a3d7bc17e135328ed6131ac57ca1180b376818985248b5bbd55d89a8739ce10009f4fa444bec7b374bddee
+  data.tar.gz: 54a8d886807da8bcddf8700bac674a5c6199f92f15e6f0ee76384a4651057c8af034024c1e4ca51de852acb15c81fee3e7e0b87ffdcf8950364d24bc0a6784b9

data/CHANGELOG.md

@@ -2,6 +2,21 @@
 
 ## [Unreleased]
 
+## [1.8.3] - 2026-05-07
+
+### Removed
+- Legacy top-level identity model files (`identity.rb`, `principal.rb`, `identity_provider.rb`, `identity_group.rb`, `identity_group_membership.rb`, `identity_audit_log.rb`) — superseded by the portable `identity/` namespace models backed by `portable_*` tables.
+
+### Fixed
+- `TypeError: superclass mismatch for class Identity` on postgres startup caused by `model_helpers.rb` defining `class Identity` as a plain namespace before `identity.rb` tried to reopen it as `< Sequel::Model(:identities)`.
+
+## [1.8.2] - 2026-05-07
+
+### Changed
+- Refactored `Legion::Data.setup` to call `setup_global`, `setup_cache`, then `setup_local` in explicit order — eliminates the `ensure setup_local` footgun that ran local SQLite even when global setup failed.
+- Extracted `setup_global` (connection + migrate + load_models) and promoted `setup_local` and `setup_cache` to top-level public methods with their own `rescue` blocks (`fatal` for local/global, `error` for cache).
+- SQLite main database now resolves to `~/.legionio/data/legionio.db` instead of a relative path in the process working directory; existing absolute path overrides in settings are unchanged.
+
 ## [1.8.1] - 2026-05-07
 
 ### Fixed

data/lib/legion/data/connection.rb

@@ -365,7 +365,12 @@ module Legion
         end
 
         def sqlite_path
-          Legion::Settings[:data][:creds][:database] || 'legionio.db'
+          path = Legion::Settings[:data][:creds][:database] || 'legionio.db'
+          return path if File.absolute_path?(path)
+
+          base_dir = File.expand_path('~/.legionio/data')
+          FileUtils.mkdir_p(base_dir)
+          File.join(base_dir, path)
         end
 
         def connection_opts_for(adapter:, opts:)

data/lib/legion/data/model.rb

@@ -13,8 +13,7 @@ module Legion
         def models
           %w[extension function relationship chain task runner node setting digital_worker
              apollo_entry apollo_relation apollo_expertise apollo_access_log audit_log
-             audit_record identity_provider principal identity identity_group
-             identity_group_membership identity_audit_log extract_step_timing
+             audit_record extract_step_timing
              identity/identity identity/principal identity/providers identity/group
              identity/group_memberships identity/audit_log
              apollo/entries apollo/relation apollo/access_log apollo/expertise

data/lib/legion/data/version.rb

@@ -2,6 +2,6 @@
 
 module Legion
   module Data
-    VERSION = '1.8.1'
+    VERSION = '1.8.3'
   end
 end

data/lib/legion/data.rb

@@ -44,14 +44,38 @@ module Legion
 
       def setup
         log.info 'Legion::Data setup starting'
-        connection_setup
-        migrate
-        load_models
+        setup_global
         setup_cache
         setup_local
         log.info 'Legion::Data setup complete'
       end
 
+      def setup_local
+        return if Legion::Settings[:data].dig(:local, :enabled) == false
+
+        Legion::Data::Local.setup
+        log.info "Legion::Data::Local connected to #{Legion::Data::Local.db_path}"
+      rescue StandardError => e
+        handle_exception(e, level: :fatal, operation: :setup_local)
+        raise
+      end
+
+      def setup_global
+        connection_setup
+        migrate
+        load_models
+      rescue StandardError => e
+        handle_exception(e, level: :fatal, operation: :setup_global)
+      end
+
+      def setup_cache
+        cache_settings = Legion::Settings[:data][:cache]
+        setup_static_cache if cache_settings[:static_cache]
+        setup_external_cache if cache_settings[:auto_enable] && defined?(::Legion::Cache)
+      rescue StandardError => e
+        handle_exception(e, level: :error, operation: :setup_cache)
+      end
+
       def connection_setup
         return if Legion::Settings[:data][:connected]
 
@@ -133,12 +157,6 @@ module Legion
         @read_privileges = nil
       end
 
-      def setup_cache
-        cache_settings = Legion::Settings[:data][:cache]
-        setup_static_cache if cache_settings[:static_cache]
-        setup_external_cache if cache_settings[:auto_enable] && defined?(::Legion::Cache)
-      end
-
       def setup_static_cache
         [Model::Extension, Model::Runner, Model::Function].each do |model|
           model.plugin :static_cache
@@ -195,14 +213,6 @@ module Legion
 
         false
       end
-
-      def setup_local
-        return if Legion::Settings[:data].dig(:local, :enabled) == false
-
-        Legion::Data::Local.setup
-      rescue StandardError => e
-        handle_exception(e, level: :warn, operation: :setup_local)
-      end
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: legion-data
 version: !ruby/object:Gem::Version
-  version: 1.8.1
+  version: 1.8.3
 platform: ruby
 authors:
 - Esity
@@ -262,7 +262,6 @@ files:
 - lib/legion/data/models/extension.rb
 - lib/legion/data/models/extract_step_timing.rb
 - lib/legion/data/models/function.rb
-- lib/legion/data/models/identity.rb
 - lib/legion/data/models/identity/audit_log.rb
 - lib/legion/data/models/identity/group.rb
 - lib/legion/data/models/identity/group_memberships.rb
@@ -270,10 +269,6 @@ files:
 - lib/legion/data/models/identity/model_helpers.rb
 - lib/legion/data/models/identity/principal.rb
 - lib/legion/data/models/identity/providers.rb
-- lib/legion/data/models/identity_audit_log.rb
-- lib/legion/data/models/identity_group.rb
-- lib/legion/data/models/identity_group_membership.rb
-- lib/legion/data/models/identity_provider.rb
 - lib/legion/data/models/llm/conversation.rb
 - lib/legion/data/models/llm/conversation_compaction.rb
 - lib/legion/data/models/llm/message.rb
@@ -288,7 +283,6 @@ files:
 - lib/legion/data/models/llm/tool_call.rb
 - lib/legion/data/models/llm/tool_call_attempt.rb
 - lib/legion/data/models/node.rb
-- lib/legion/data/models/principal.rb
 - lib/legion/data/models/rbac/cross_team_grants.rb
 - lib/legion/data/models/rbac/model_helpers.rb
 - lib/legion/data/models/rbac/role_assignments.rb

data/lib/legion/data/models/identity.rb

@@ -1,26 +0,0 @@
-# frozen_string_literal: true
-
-return unless Legion::Data::Connection.adapter == :postgres
-
-module Legion
-  module Data
-    module Model
-      class Identity < Sequel::Model(:identities)
-        require_relative 'identity/model_helpers'
-        include ModelHelpers
-
-        many_to_one :principal, class: 'Legion::Data::Model::Principal'
-        many_to_one :provider, class: 'Legion::Data::Model::IdentityProvider', key: :provider_id
-
-        def self.lookup_columns
-          %i[id uuid provider_identity_key provider_identity]
-        end
-
-        if defined?(Legion::Data::Encryption::SequelPlugin)
-          plugin Legion::Data::Encryption::SequelPlugin
-          encrypted_column :profile
-        end
-      end
-    end
-  end
-end

data/lib/legion/data/models/identity_audit_log.rb

@@ -1,14 +0,0 @@
-# frozen_string_literal: true
-
-return unless Legion::Data::Connection.adapter == :postgres
-
-module Legion
-  module Data
-    module Model
-      class IdentityAuditLog < Sequel::Model(:identity_audit_log)
-        many_to_one :principal, class: 'Legion::Data::Model::Principal'
-        many_to_one :identity, class: 'Legion::Data::Model::Identity'
-      end
-    end
-  end
-end

data/lib/legion/data/models/identity_group.rb

@@ -1,26 +0,0 @@
-# frozen_string_literal: true
-
-require_relative 'identity/model_helpers'
-
-return unless Legion::Data::Connection.adapter == :postgres
-
-module Legion
-  module Data
-    module Model
-      class IdentityGroup < Sequel::Model(:identity_groups)
-        include Identity::ModelHelpers
-
-        one_to_many :memberships, class: 'Legion::Data::Model::IdentityGroupMembership', key: :group_id
-        many_to_many :principals,
-                     class:      'Legion::Data::Model::Principal',
-                     join_table: :identity_group_memberships,
-                     left_key:   :group_id,
-                     right_key:  :principal_id
-
-        def self.lookup_columns
-          %i[id uuid name]
-        end
-      end
-    end
-  end
-end

data/lib/legion/data/models/identity_group_membership.rb

@@ -1,22 +0,0 @@
-# frozen_string_literal: true
-
-return unless Legion::Data::Connection.adapter == :postgres
-
-module Legion
-  module Data
-    module Model
-      class IdentityGroupMembership < Sequel::Model(:identity_group_memberships)
-        many_to_one :principal, class: 'Legion::Data::Model::Principal'
-        many_to_one :group, class: 'Legion::Data::Model::IdentityGroup', key: :group_id
-
-        def expired?
-          status == 'expired' || (expires_at && Time.now >= expires_at)
-        end
-
-        def stale?
-          status == 'stale'
-        end
-      end
-    end
-  end
-end

data/lib/legion/data/models/identity_provider.rb

@@ -1,25 +0,0 @@
-# frozen_string_literal: true
-
-require_relative 'identity/model_helpers'
-
-return unless Legion::Data::Connection.adapter == :postgres
-
-module Legion
-  module Data
-    module Model
-      class IdentityProvider < Sequel::Model(:identity_providers)
-        include Identity::ModelHelpers
-
-        one_to_many :identities, class: 'Legion::Data::Model::Identity'
-
-        def self.lookup_columns
-          %i[id uuid name]
-        end
-
-        def parsed_capabilities
-          Array(capabilities)
-        end
-      end
-    end
-  end
-end

data/lib/legion/data/models/principal.rb

@@ -1,35 +0,0 @@
-# frozen_string_literal: true
-
-require_relative 'identity/model_helpers'
-
-return unless Legion::Data::Connection.adapter == :postgres
-
-module Legion
-  module Data
-    module Model
-      class Principal < Sequel::Model(:principals)
-        include Identity::ModelHelpers
-
-        one_to_many :identities, class: 'Legion::Data::Model::Identity'
-        one_to_many :group_memberships, class: 'Legion::Data::Model::IdentityGroupMembership'
-        many_to_many :groups,
-                     class:      'Legion::Data::Model::IdentityGroup',
-                     join_table: :identity_group_memberships,
-                     left_key:   :principal_id,
-                     right_key:  :group_id
-
-        def self.lookup_columns
-          %i[id uuid canonical_name employee_key employee_id]
-        end
-
-        def active_groups
-          group_memberships_dataset
-            .where(status: 'active')
-            .eager(:group)
-            .all
-            .map(&:group)
-        end
-      end
-    end
-  end
-end