legion-data 1.8.1 → 1.8.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9b3a56b1330f3ec71addc28b9b3b02e6ed845ff80c9993073bde8130fee363ef
-  data.tar.gz: ccfdf0a86f6408b201065d414ab6199cacf6610e0bfbb527449e59cf970fddb8
+  metadata.gz: 38a6acdfd8b43c7f28928e2c972b292d5f4dac76fe62049600c3cafe617e270c
+  data.tar.gz: 00615b710b087d2d71683eca0424b44200cf9d2934d68169196ee9b13252f225
 SHA512:
-  metadata.gz: b40d2cde196b2c8fec186671ee7973ac85893239e9502445d975ccfaca43efdd8170e04082869f30bd0544cb42a9b090b1190c5ef942c6afa37fa07622874854
-  data.tar.gz: c9e5c0e72a62c423a532b1be4d34ec98fd6b7b9611232939e5e57f3e61e3f56cc14eb19c5df57855cb3735ee76f6e00338e6a610fa81989e0acfba27c4fceb6a
+  metadata.gz: 6dbf52624a9780173930c096f4fcd58216e6bcd59ffaa832d137446753dbb39eb293e17c6e2288dbce1451bc77cf5fa366d55d6ce99901794e82d871ca679ad7
+  data.tar.gz: 50ea8e3806bdf7de608af572802c8edf6ba40c77c2c42857ed4d7a7ec85470c29c3345771aefa62bbb92d7fd466fad1f3fce1d8b9ab85d404fd42bed70e70554

data/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 ## [Unreleased]
 
+## [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/version.rb

@@ -2,6 +2,6 @@
 
 module Legion
   module Data
-    VERSION = '1.8.1'
+    VERSION = '1.8.2'
   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.2
 platform: ruby
 authors:
 - Esity