legion-data 1.8.3 → 1.8.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d9f440b5e825e600fe33f7b07f6aa7aeb11cde67069eea1960e01ed6fcfb5ee2
-  data.tar.gz: b60960ed3f6cf3f870c92a309dd8ee53d11f7b1f672e747a35c0ca11e8267074
+  metadata.gz: b825eb46e0228251730ce4c339e76eac14c67e3001d1554495bf7f72ec0f5d0c
+  data.tar.gz: a9f797bd628944c509ea230003a927e5bf3ba434a08a50bdd6a35a34d719d72e
 SHA512:
-  metadata.gz: 345dd816ac680066d8964506e9c14f72638b694ef5a3d7bc17e135328ed6131ac57ca1180b376818985248b5bbd55d89a8739ce10009f4fa444bec7b374bddee
-  data.tar.gz: 54a8d886807da8bcddf8700bac674a5c6199f92f15e6f0ee76384a4651057c8af034024c1e4ca51de852acb15c81fee3e7e0b87ffdcf8950364d24bc0a6784b9
+  metadata.gz: 1d46e826a98465155b373512068ae0edc3caf29dd5a0b39c910aa2fd68e3076f0348bd7611b94f91707d6d8b674b9637d4c4f55739712e82cba2aa485a67a602
+  data.tar.gz: c5bbda03afc750f841f22a8eb55a01a4b8a335d8167a81445cadb033e95b40d4409c704c110fefb5a42d752d9abb6d46d5778613b92a9d10a81f25c415b0c569

data/CHANGELOG.md

@@ -1,6 +1,19 @@
 # Legion::Data Changelog
 
-## [Unreleased]
+## [1.8.5] - 2026-05-09
+
+### Removed
+- Unnecessary `defined?(Legion::Logging)` guards from connection and local database setup — legion-logging is a hard gemspec dependency and always available
+
+## [1.8.4] - 2026-05-08
+
+### Removed
+- Dropped legacy postgres-only identity tables (`principals`, `identity_providers`, `identities`, `identity_groups`, `identity_group_memberships`, `identity_audit_log`) via migration 098.
+- Removed `table_available?` guards from all identity model files — models load unconditionally.
+
+### Changed
+- Renamed `portable_identity_*` tables to canonical names (`identity_principals`, `identity_providers`, `identities`, `identity_groups`, `identity_group_memberships`, `identity_audit_log`, `identity_provider_capabilities`) via migration 099.
+- Updated all identity models to reference the new table names.
 
 ## [1.8.3] - 2026-05-07
 

data/lib/legion/data/connection.rb

@@ -396,7 +396,7 @@ module Legion
             opts[:logger]              = @query_file_logger
             opts[:sql_log_level]       = :debug
             opts[:log_connection_info] = data[:log_connection_info] || false
-          elsif data[:log] && defined?(Legion::Logging)
+          elsif data[:log]
             # Standard mode: slow-query warnings through Legion::Logging domain
             opts[:logger]              = build_data_logger
             opts[:sql_log_level]       = data[:sql_log_level]&.to_sym || :debug

data/lib/legion/data/local.rb

@@ -38,7 +38,7 @@ module Legion
             @query_file_logger = Legion::Data::Connection::QueryFileLogger.new(log_path)
             opts[:logger]          = @query_file_logger
             opts[:sql_log_level]   = :debug
-          elsif data[:log] && defined?(Legion::Logging)
+          elsif data[:log]
             opts[:logger]          = build_local_logger
             opts[:sql_log_level]   = resolved_sql_log_level
             opts[:log_warn_duration] = resolved_log_warn_duration

data/lib/legion/data/migrations/098_drop_legacy_identity_tables.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+Sequel.migration do
+  up do
+    drop_table(:identity_audit_log) if table_exists?(:identity_audit_log)
+    drop_table(:identity_group_memberships) if table_exists?(:identity_group_memberships)
+    drop_table(:identity_groups) if table_exists?(:identity_groups)
+    drop_table(:identities) if table_exists?(:identities)
+
+    alter_table(:nodes) { drop_column :principal_id } if table_exists?(:nodes) && schema(:nodes).any? { |col, _| col == :principal_id }
+
+    drop_table(:principals) if table_exists?(:principals)
+    drop_table(:identity_providers) if table_exists?(:identity_providers)
+  end
+
+  down do
+    nil
+  end
+end

data/lib/legion/data/migrations/099_rename_portable_identity_tables.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+Sequel.migration do
+  up do
+    rename_table(:portable_identity_provider_capabilities, :identity_provider_capabilities)
+    rename_table(:portable_identity_audit_log, :identity_audit_log)
+    rename_table(:portable_identity_group_memberships, :identity_group_memberships)
+    rename_table(:portable_identity_groups, :identity_groups)
+    rename_table(:portable_identities, :identities)
+    rename_table(:portable_identity_principals, :identity_principals)
+    rename_table(:portable_identity_providers, :identity_providers)
+  end
+
+  down do
+    rename_table(:identity_providers, :portable_identity_providers)
+    rename_table(:identity_principals, :portable_identity_principals)
+    rename_table(:identities, :portable_identities)
+    rename_table(:identity_groups, :portable_identity_groups)
+    rename_table(:identity_group_memberships, :portable_identity_group_memberships)
+    rename_table(:identity_audit_log, :portable_identity_audit_log)
+    rename_table(:identity_provider_capabilities, :portable_identity_provider_capabilities)
+  end
+end

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

@@ -2,13 +2,11 @@
 
 require_relative 'model_helpers'
 
-return unless Legion::Data::Model::Identity::ModelHelpers.table_available?(:portable_identity_audit_log)
-
 module Legion
   module Data
     module Model
       class Identity
-        class AuditLog < Sequel::Model(:portable_identity_audit_log)
+        class AuditLog < Sequel::Model(:identity_audit_log)
           include ModelHelpers
 
           many_to_one :principal, class: 'Legion::Data::Model::Identity::Principal'

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

@@ -2,19 +2,17 @@
 
 require_relative 'model_helpers'
 
-return unless Legion::Data::Model::Identity::ModelHelpers.table_available?(:portable_identity_groups)
-
 module Legion
   module Data
     module Model
       class Identity
-        class Group < Sequel::Model(:portable_identity_groups)
+        class Group < Sequel::Model(:identity_groups)
           include ModelHelpers
 
           one_to_many :memberships, class: 'Legion::Data::Model::Identity::GroupMembership', key: :group_id
           many_to_many :principals,
                        class:      'Legion::Data::Model::Identity::Principal',
-                       join_table: :portable_identity_group_memberships,
+                       join_table: :identity_group_memberships,
                        left_key:   :group_id,
                        right_key:  :principal_id
 

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

@@ -2,13 +2,11 @@
 
 require_relative 'model_helpers'
 
-return unless Legion::Data::Model::Identity::ModelHelpers.table_available?(:portable_identity_group_memberships)
-
 module Legion
   module Data
     module Model
       class Identity
-        class GroupMembership < Sequel::Model(:portable_identity_group_memberships)
+        class GroupMembership < Sequel::Model(:identity_group_memberships)
           include ModelHelpers
 
           many_to_one :principal, class: 'Legion::Data::Model::Identity::Principal'

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

@@ -2,13 +2,11 @@
 
 require_relative 'model_helpers'
 
-return unless Legion::Data::Model::Identity::ModelHelpers.table_available?(:portable_identities)
-
 module Legion
   module Data
     module Model
       class Identity
-        class Identity < Sequel::Model(:portable_identities)
+        class Identity < Sequel::Model(:identities)
           include ModelHelpers
 
           many_to_one :principal, class: 'Legion::Data::Model::Identity::Principal'

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

@@ -11,12 +11,6 @@ module Legion
             model.extend(ClassMethods)
           end
 
-          def self.table_available?(table_name)
-            Legion::Data::Connection.sequel&.table_exists?(table_name)
-          rescue StandardError
-            false
-          end
-
           module ClassMethods
             def lookup(value)
               lookup_by_columns(value, lookup_columns)

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

@@ -2,20 +2,18 @@
 
 require_relative 'model_helpers'
 
-return unless Legion::Data::Model::Identity::ModelHelpers.table_available?(:portable_identity_principals)
-
 module Legion
   module Data
     module Model
       class Identity
-        class Principal < Sequel::Model(:portable_identity_principals)
+        class Principal < Sequel::Model(:identity_principals)
           include ModelHelpers
 
           one_to_many :identities, class: 'Legion::Data::Model::Identity::Identity'
           one_to_many :group_memberships, class: 'Legion::Data::Model::Identity::GroupMembership'
           many_to_many :groups,
                        class:      'Legion::Data::Model::Identity::Group',
-                       join_table: :portable_identity_group_memberships,
+                       join_table: :identity_group_memberships,
                        left_key:   :principal_id,
                        right_key:  :group_id
 

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

@@ -2,13 +2,11 @@
 
 require_relative 'model_helpers'
 
-return unless Legion::Data::Model::Identity::ModelHelpers.table_available?(:portable_identity_providers)
-
 module Legion
   module Data
     module Model
       class Identity
-        class Provider < Sequel::Model(:portable_identity_providers)
+        class Provider < Sequel::Model(:identity_providers)
           include ModelHelpers
 
           one_to_many :identities, class: 'Legion::Data::Model::Identity::Identity', key: :provider_id
@@ -25,7 +23,7 @@ module Legion
           end
         end
 
-        class ProviderCapability < Sequel::Model(:portable_identity_provider_capabilities)
+        class ProviderCapability < Sequel::Model(:identity_provider_capabilities)
           many_to_one :provider, class: 'Legion::Data::Model::Identity::Provider'
         end
       end

data/lib/legion/data/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: legion-data
 version: !ruby/object:Gem::Version
-  version: 1.8.3
+  version: 1.8.5
 platform: ruby
 authors:
 - Esity
@@ -244,6 +244,8 @@ files:
 - lib/legion/data/migrations/095_create_portable_identity_group_memberships.rb
 - lib/legion/data/migrations/096_create_portable_identity_audit_log.rb
 - lib/legion/data/migrations/097_add_llm_dispatch_fields.rb
+- lib/legion/data/migrations/098_drop_legacy_identity_tables.rb
+- lib/legion/data/migrations/099_rename_portable_identity_tables.rb
 - lib/legion/data/model.rb
 - lib/legion/data/models/apollo/access_log.rb
 - lib/legion/data/models/apollo/entries.rb