activerecord-beagle-turso 0.1.3 → 0.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7b4bdeec9d75fe6297e7f42aba4ab966b31407731a5992de2af9a214483c53c7
-  data.tar.gz: bc8dcf3930ec2eb8104712b5f910b38118cd75ae0840d0d98bcb9a145d958f21
+  metadata.gz: 96e5e44a6be08be7733d3f7ea9590e6577826c43d8ecffe178f6bb6314d3396c
+  data.tar.gz: 66627c1ff6245cb9f44180c6152020248d49f58722359c7070ce883a9cbe4ed2
 SHA512:
-  metadata.gz: 7b31dbac6656267e980d1fc85b7c8ce805ea538564750b9f537b957ba1d6443cddee350d375730a64a83c8444b35fe76df90952fd76f11661494f07ae0ea5683
-  data.tar.gz: 0e77f74a3dca74225fd169e4556b6b948ae42c2702b555e19c3e034cf1301da3267428b04198eefad73da422037de6574d61ee46ca77d06541a026eccb5ff60e
+  metadata.gz: cc94306cead989dba2cf18789880c63cc4eb27b973812bec9f807fbbdf40b81baef90edcc512d099a90c2955427ed56582c51c0dd1d7fb6fc1439e973dfbbd5f
+  data.tar.gz: d15c112f354343e61bda83c8f146603ae0aa1c6da19381e158eb01ce84ee487f82e67e775d416328196d5c2cf7a4f25872954219a63f75181535318681d806a3

data/README.md

@@ -94,7 +94,7 @@ All three default their `connection` argument to `ActiveRecord::Base.connection`
 so `Beagle::Turso::SyncManager.sync!` alone is the usual call. Calling any of
 them against a **local-only** connection (no `remote_url:`/`auth_token:` in
 `database.yml`) raises `Beagle::Turso::SyncManager::NotSyncedError` — a
-distinct, rescuable type — rather than the driver's bare `RuntimeError`.
+distinct, rescuable type — rather than the driver's own `Beagle::Turso::NotSyncedError`.
 
 To sync on a timer, wire
 `ActiveRecord::ConnectionAdapters::BeagleTurso::SyncJob` (a thin delegator to

data/lib/active_record/connection_adapters/beagle_turso/sync_manager.rb

@@ -46,23 +46,14 @@ module Beagle
     module SyncManager
       # Raised by +push!+/+pull!+/+sync!+ when the given connection is
       # local-only (its database.yml entry has no +remote_url+/+auth_token+,
-      # so BeagleTursoAdapter opened it via +Database.open_local+). The
-      # underlying Client#push/#pull already raise a bare RuntimeError for
-      # this (see Beagle::Turso::Database#push/#pull); SyncManager re-raises
-      # it as this distinct, documented type so callers can rescue it
-      # specifically instead of matching on the driver's message text. A
+      # so BeagleTursoAdapter opened it via +Database.open_local+). The driver
+      # raises Beagle::Turso::NotSyncedError for this; SyncManager re-raises it
+      # as this type (kept for compatibility) with a database.yml hint. A
       # genuine sync failure against a real remote (network error, auth
-      # failure, etc.) is NOT reclassified -- it propagates as whatever error
-      # the driver raised, unchanged.
+      # failure, etc.) is NOT reclassified -- it propagates as
+      # Beagle::Turso::SyncError (or another Beagle::Turso::Error), unchanged.
       class NotSyncedError < StandardError; end
 
-      # Matches only the driver's local-only wording (see
-      # beagle_turso_core's `Error::Sync` in `Database::push`/`Database::pull`)
-      # so a real sync failure -- which also surfaces as a RuntimeError, just
-      # with different text -- is never misreported as "not synced".
-      LOCAL_ONLY_MESSAGE_REGEX = /on a local-only database/i
-      private_constant :LOCAL_ONLY_MESSAGE_REGEX
-
       module_function
 
       # Push this connection's local writes to its remote Turso database.
@@ -88,9 +79,7 @@ module Beagle
 
       def reraise_local_only
         yield
-      rescue RuntimeError => e
-        raise e unless LOCAL_ONLY_MESSAGE_REGEX.match?(e.message)
-
+      rescue Beagle::Turso::NotSyncedError => e
         raise NotSyncedError,
           "#{e.message} -- this connection is local-only; configure remote_url: " \
           "and auth_token: in database.yml to enable sync"

data/lib/active_record/connection_adapters/beagle_turso_adapter.rb

@@ -223,6 +223,65 @@ module ActiveRecord
           "#{super} AND name NOT LIKE '\\_\\_turso\\_internal\\_%' ESCAPE '\\'"
         end
 
+        # A column definition in the DDL that carries a parenthesised type
+        # argument: `"price" decimal (10, 2) NOT NULL`, `a varchar (50)`. The
+        # name may be bare or quoted with ", ` or []; the type may be several
+        # words ("double precision"); the argument is one or two integers.
+        DECLARED_TYPE_REGEX = /
+          \A\s*
+          (?:"(?<name>[^"]*)" | `(?<name>[^`]*)` | \[(?<name>[^\]]*)\] | (?<name>\w+))
+          \s+
+          (?<type>[A-Za-z_][A-Za-z0-9_\ ]*?)
+          \s*\(\s*(?<args>\d+\s*(?:,\s*\d+)?)\s*\)
+        /x
+        private_constant :DECLARED_TYPE_REGEX
+
+        # Restore type arguments that beagle_turso's `PRAGMA table_info` drops.
+        #
+        # turso reports a column's declared type as the bare type name: a
+        # `decimal(10,2)` column comes back as `decimal`, `varchar(50)` as
+        # `varchar`, `datetime(6)` as `datetime`. Rails derives precision,
+        # scale and limit by parsing exactly that string, so all three silently
+        # become nil. Real SQLite returns the full declared type here, and the
+        # information is not lost on disk -- the CREATE TABLE in sqlite_master
+        # still says `decimal (10, 2)` -- so recover it from there.
+        #
+        # Left alone this is worse than cosmetic schema.rb churn: a dump emits
+        # `t.decimal "price"` with no precision/scale, and loading that dump
+        # builds a column whose values are no longer rounded to 2 decimal
+        # places. `super` already splits the DDL into per-column strings to
+        # recover COLLATE, AUTOINCREMENT and GENERATED ALWAYS AS for the same
+        # reason; this is the same trick for the type itself.
+        #
+        # Only fills in a missing argument -- a type that already arrived with
+        # one is left untouched, so this becomes an automatic no-op if turso
+        # starts reporting declared types in full.
+        def table_structure_with_collation(table_name, basic_structure)
+          columns = super
+          return columns unless columns.any? { |column| !column["type"].to_s.include?("(") }
+
+          declared = declared_column_types(table_name, columns.filter_map { |c| c["name"] })
+          return columns if declared.empty?
+
+          columns.each do |column|
+            next if column["type"].to_s.include?("(")
+
+            full_type = declared[column["name"]]
+            column["type"] = full_type if full_type
+          end
+        end
+
+        # Maps column name -> declared type with its argument normalised back
+        # to Rails' own spelling (`decimal (10, 2)` -> `decimal(10,2)`), which
+        # is what Rails' precision/scale/limit extraction expects to see.
+        def declared_column_types(table_name, column_names)
+          table_structure_sql(table_name, column_names).each_with_object({}) do |column_string, declared|
+            next unless (match = DECLARED_TYPE_REGEX.match(column_string))
+
+            declared[match[:name]] = "#{match[:type]}(#{match[:args].gsub(/\s+/, '')})"
+          end
+        end
+
         # The single seam. AbstractAdapter#raw_execute funnels every statement
         # here and expects an ActiveRecord::Result back, with the notification
         # payload's affected_rows/row_count filled in. cast_result (inherited)

data/lib/activerecord/beagle_turso/version.rb

@@ -2,6 +2,6 @@
 
 module ActiveRecord
   module BeagleTurso
-    VERSION = "0.1.3"
+    VERSION = "0.1.5"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: activerecord-beagle-turso
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.5
 platform: ruby
 authors:
 - BeagleSoftwareUK
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-08-09 00:00:00.000000000 Z
+date: 2026-10-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.1'
+        version: '0.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.1'
+        version: '0.2'
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement