legion-data 1.6.25 → 1.6.27

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6d2adcdaec18362d8417478957f8093bd331e6e047fd5546b4c9c1151174935e
-  data.tar.gz: 02aed26a7474699c18956263d1ac639ca688b2e76db37905d230e7eed9af13b7
+  metadata.gz: d2fca06857473b0c009e6ab0ca7044f09b394ce3652adad2b35e57835bceb5eb
+  data.tar.gz: ea80dac3fee5e54500534c5399c4f1c5b47b9a68ee6c0d0bf0d5ebe344053531
 SHA512:
-  metadata.gz: e5343b72716892156ab69870bd50625fdf84b5b54d6792b6838b9783ccf2808c61e1ed2782c6f0a4610569ad771c09856ecc151bb1dd2c3dfdba839f8cc06bab
-  data.tar.gz: 2e780ebae82893bdbf4a91d0b6d68bce7a1fb3ed2e5dfe0630c6290b67a4ea8bf60f17bdafc0794cbe73fae2dc2a36b5a71fcc0c9955428abf1eb51fc7280538
+  metadata.gz: '009fcbd9de6e5dd5d8e74d1ccf2d21c8692a930ae15369ae19fccb2a5f7829982ba6bf79757c66af25d4cce32c5caaf4b8350316e98261676e1bc532485302d4'
+  data.tar.gz: d36ede441cdf282214ff9abc7836f2ba07f2eb02bce839f884bf074a7cacea3336ddfdb0182ff87c4e1ba63e54f0bc26c3d9eb1769b607474cc3a32197c814fd

data/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 ## [Unreleased]
 
+## [1.6.27] - 2026-04-17
+
+### Fixed
+- `load_sequel_model` now reads `settings[:models][:continue_on_load_fail]` (was erroneously reading `continue_on_fail`, a key that never existed — LoadError was always re-raised regardless of the setting). (Fixes #22)
+- `load_models` now skips model loading when `settings[:models][:autoload]` is `false`, honoring the documented knob. (Fixes #22)
+- Audit record live-path specs (`append`, `verify`, `walk`, `query_by_type`, immutability guards) are now self-sufficient: the `before` block reconnects the DB if a prior spec tore it down, eliminating 19 pending examples when running the full suite. (Fixes #22)
+- Default `preconnect` changed from `'concurrently'` to `false`. The concurrent preconnect mode spawned background threads that emitted noisy connection errors when a network adapter was unreachable before dev-fallback could catch the failure. `false` preserves identical behavior for SQLite (default) and avoids the noise for production deployments where operators can opt-in explicitly. (Fixes #22)
+
+## [1.6.26] - 2026-04-17
+
+### Fixed
+- `connection_validation_timeout` default reduced from 600s to -1 (validate every checkout) for non-SQLite adapters. The previous 10-minute window meant stale PG connections from a VPN drop, sleep/wake, or network interface change were not evicted until the next scheduled validation cycle, causing `Sequel::DatabaseDisconnectError` to repeat on every actor tick. With -1, Sequel pings the connection on every pool checkout and discards dead connections immediately. (Fixes #28)
+
 ## [1.6.24] - 2026-04-13
 
 ### Added

data/lib/legion/data/archiver.rb

@@ -158,11 +158,7 @@ module Legion
         end
 
         def json_dump(obj)
-          if defined?(Legion::JSON)
-            Legion::JSON.dump(obj)
-          else
-            ::JSON.generate(obj)
-          end
+          ::JSON.generate(obj)
         end
 
         def gzip_compress(data)

data/lib/legion/data/audit_record.rb

@@ -25,7 +25,7 @@ module Legion
           conn = Legion::Data.connection
           conn.transaction do
             parent_hash = latest_chain_hash(conn, chain_id)
-            ts          = Time.now
+            ts          = truncate_to_us(Time.now)
             ch          = compute_chain_hash(parent_hash, content_hash, ts, content_type)
             sig         = sign ? sign_record(ch) : nil
             meta_json   = metadata.empty? ? nil : Legion::JSON.dump(metadata)
@@ -104,31 +104,35 @@ module Legion
           ds.order(Sequel.desc(:created_at)).limit(limit).all.map { |r| deserialize(r) }
         end
 
-        # SHA-256 of "parent_hash:content_hash:unix_ts_ns:content_type".
+        # SHA-256 of "parent_hash:content_hash:unix_ts_us:content_type".
         #
-        # The timestamp is normalised to nanoseconds-since-epoch so the hash is
-        # independent of time zone, string formatting, and database type.
-        # Exposed as a public method so callers can independently verify a hash
-        # without querying the database.
+        # The timestamp is normalised to microseconds-since-epoch. PostgreSQL
+        # TIMESTAMP columns have microsecond precision, so nanosecond values
+        # written by Ruby would be truncated on read, causing recomputed hashes
+        # to diverge. Microsecond normalisation keeps write-time and read-time
+        # hashes identical across all supported adapters.
         def compute_chain_hash(parent_hash, content_hash, timestamp, content_type)
-          ts_ns = normalise_timestamp_ns(timestamp)
-          Digest::SHA256.hexdigest("#{parent_hash}:#{content_hash}:#{ts_ns}:#{content_type}")
+          ts_us = normalise_timestamp_us(timestamp)
+          Digest::SHA256.hexdigest("#{parent_hash}:#{content_hash}:#{ts_us}:#{content_type}")
         end
 
         private
 
-        # Normalise a timestamp to integer nanoseconds-since-epoch regardless of
-        # whether the database returned a Time, DateTime, or String.
-        def normalise_timestamp_ns(timestamp)
-          case timestamp
-          when ::Time
-            (timestamp.to_r * 1_000_000_000).to_i
-          when ::DateTime
-            (timestamp.to_time.to_r * 1_000_000_000).to_i
-          else
-            ts = ::Time.parse(timestamp.to_s)
-            (ts.to_r * 1_000_000_000).to_i
-          end
+        # Normalise a timestamp to integer microseconds-since-epoch regardless of
+        # whether the database returned a Time, DateTime, or String. Always uses
+        # the absolute epoch value so timezone differences don't affect the hash.
+        def normalise_timestamp_us(timestamp)
+          t = case timestamp
+              when ::Time     then timestamp
+              when ::DateTime then timestamp.to_time
+              else ::Time.parse(timestamp.to_s)
+              end
+          (t.to_r * 1_000_000).to_i
+        end
+
+        def truncate_to_us(time)
+          us = (time.to_r * 1_000_000).to_i
+          ::Time.at(Rational(us, 1_000_000))
         end
 
         def latest_chain_hash(conn, chain_id)

data/lib/legion/data/model.rb

@@ -37,7 +37,7 @@ module Legion
           model
         rescue LoadError => e
           handle_exception(e, level: :fatal, operation: :load_sequel_model, model: model)
-          raise e unless Legion::Settings[:data][:models][:continue_on_fail]
+          raise e unless Legion::Settings[:data][:models][:continue_on_load_fail]
         end
       end
     end

data/lib/legion/data/settings.rb

@@ -35,7 +35,7 @@ module Legion
           # Connection pool
           max_connections:               25,
           pool_timeout:                  5,
-          preconnect:                    'concurrently',
+          preconnect:                    false,
           single_threaded:               false,
           test:                          true,
           name:                          nil,
@@ -48,8 +48,9 @@ module Legion
           sql_log_level:                 'debug',
 
           # Connection health (network adapters only, ignored for sqlite)
+          # -1 means validate on every checkout, catching stale connections from VPN/sleep/network changes immediately
           connection_validation:         true,
-          connection_validation_timeout: 600,
+          connection_validation_timeout: -1,
           connection_expiration:         true,
           connection_expiration_timeout: 14_400,
 

data/lib/legion/data/version.rb

@@ -2,6 +2,6 @@
 
 module Legion
   module Data
-    VERSION = '1.6.25'
+    VERSION = '1.6.27'
   end
 end

data/lib/legion/data.rb

@@ -64,6 +64,8 @@ module Legion
       end
 
       def load_models
+        return unless Legion::Settings[:data][:models][:autoload] != false
+
         Legion::Data::Models.load
       end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: legion-data
 version: !ruby/object:Gem::Version
-  version: 1.6.25
+  version: 1.6.27
 platform: ruby
 authors:
 - Esity