activerecord-libsql 0.1.6 → 0.1.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f696c91f955a35679cd2b4159bedc0a9b8429cd30397fe9631054de7f63eb510
-  data.tar.gz: 58df65dd6bfd7aad0a4f187c9935064b4d8d21d57079c44df5f51c765148aca0
+  metadata.gz: c7d43524e4ffab6a8d48512dae7494bd422a0484f4d5de4aab9638af9cc05e11
+  data.tar.gz: 19c8598dd1fe21df3a71822adfbc03b0f99b01a50d46ec0544a5c9ed54fafd76
 SHA512:
-  metadata.gz: 0a1ec1b42c809eb6cc91516796bdd7ece669396f22415f4940e5a1051780638774948e558aaeefbb22f9837052c86575061ed408ae71b24f018b726d66424ce9
-  data.tar.gz: 8a99cd050bae2dbb898bb74bb798afd7432a943f231efc10685b041ccd98b5a47449ad536f7216be53ed50d9a897bec37fcafd2f7d5f12fe7fd71e77921644ee
+  metadata.gz: ec59f0d00947a93fc541d2fbe27bbef5132208a472199ad1869187e85beb0a936d49258f3330be290ede8c01f15c3c0eae851d8518725ba2abe2d64a1d38e1a0
+  data.tar.gz: 2823e8e01b17e6553a9ae4f676d589f9e11349200bd133a0e7a20bd372480e5d46efa8098afb6b9780146efb1e5d82722dfa120400d8345c0fd5c1cd8a2c6e77

data/CHANGELOG.md

@@ -2,6 +2,26 @@
 
 All notable changes to this project will be documented in this file.
 
+## [0.1.7] - 2026-03-27
+
+### Fixed
+
+- **`NotImplementedError` on `insert_all!` / `upsert_all` with Solid Queue** (#17)
+  - Solid Queue's `ClaimedExecution.claiming` calls `insert_all!`, which internally calls
+    `self.class.quote_column_name` (a class-level method).
+  - `AbstractAdapter::ClassMethods#quote_column_name` raises `NotImplementedError` by design.
+    `LibsqlAdapter` only defined the instance-level method, leaving the class-level one unimplemented.
+  - Added `class << self` block with `quote_column_name` / `quote_table_name` (cached via `Concurrent::Map`).
+  - Removed redundant instance-level `quote_column_name` / `quote_table_name` — AR 8's `Quoting` module
+    delegates instance calls to `self.class.quote_column_name`, so only the class-level definition is needed.
+  - Added `quote_string` override: SQLite / libSQL do not treat backslash as an escape character,
+    so the default `AbstractAdapter` implementation (which doubles backslashes) would corrupt values
+    containing `\`. Now only single-quotes are escaped, matching `SQLite3Adapter`.
+  - Implemented `build_insert_sql` with `ON CONFLICT ... DO NOTHING / DO UPDATE SET` syntax
+    (same as `SQLite3Adapter`).
+  - Declared `supports_insert_on_duplicate_skip?`, `supports_insert_on_duplicate_update?`,
+    and `supports_insert_conflict_target?` as `true`.
+
 ## [0.1.6] - 2026-03-16
 
 ### Fixed

data/lib/active_record/connection_adapters/libsql_adapter.rb

@@ -18,6 +18,22 @@ module ActiveRecord
     class LibsqlAdapter < AbstractAdapter
       ADAPTER_NAME = 'Turso'
 
+      # insert_all! / upsert_all は self.class.quote_column_name（クラスメソッド）を呼ぶ。
+      # AbstractAdapter の ClassMethods はこれを NotImplementedError にするため、
+      # クラスメソッドとして上書きする。
+      QUOTED_COLUMN_NAMES = Concurrent::Map.new # :nodoc:
+      QUOTED_TABLE_NAMES  = Concurrent::Map.new # :nodoc:
+
+      class << self
+        def quote_column_name(name)
+          QUOTED_COLUMN_NAMES[name] ||= %("#{name.to_s.gsub('"', '""')}").freeze
+        end
+
+        def quote_table_name(name)
+          QUOTED_TABLE_NAMES[name] ||= %("#{name.to_s.gsub('"', '""').gsub('.', '"."')}").freeze
+        end
+      end
+
       # SQLite 互換の型マッピング（libSQL は SQLite 方言）
       # datetime / timestamp は 'datetime' を使う。
       # 'TEXT' にすると PRAGMA table_info が 'TEXT' を返し、
@@ -90,6 +106,20 @@ module ActiveRecord
         false
       end
 
+      # insert_all! / upsert_all のサポート
+      # libSQL / SQLite は ON CONFLICT 構文をサポートする
+      def supports_insert_on_duplicate_skip?
+        true
+      end
+
+      def supports_insert_on_duplicate_update?
+        true
+      end
+
+      def supports_insert_conflict_target?
+        true
+      end
+
       def write_query?(sql)
         !READ_QUERY.match?(sql)
       rescue ArgumentError
@@ -158,6 +188,32 @@ module ActiveRecord
         @raw_database&.sync
       end
 
+      # -----------------------------------------------------------------------
+      # insert_all! / upsert_all（ON CONFLICT 構文）
+      # SQLite3 adapter と同じ実装
+      # -----------------------------------------------------------------------
+
+      def build_insert_sql(insert) # :nodoc:
+        sql = +"INSERT #{insert.into} #{insert.values_list}"
+
+        if insert.skip_duplicates?
+          sql << " ON CONFLICT #{insert.conflict_target} DO NOTHING"
+        elsif insert.update_duplicates?
+          sql << " ON CONFLICT #{insert.conflict_target} DO UPDATE SET "
+          if insert.raw_update_sql?
+            sql << insert.raw_update_sql
+          else
+            sql << insert.touch_model_timestamps_unless { |column| "#{column} IS excluded.#{column}" }
+            sql << insert.updatable_columns.map { |column| "#{column}=excluded.#{column}" }.join(',')
+          end
+        end
+
+        # RETURNING 句は supports_insert_returning? が false（デフォルト）のため生成されない。
+        # perform_query は INSERT を write クエリとして扱い結果を捨てるため、
+        # RETURNING を有効にする場合は perform_query の read_query? 判定も合わせて修正が必要。
+        sql
+      end
+
       # -----------------------------------------------------------------------
       # AR 8 クエリパイプライン
       # raw_execute → perform_query → cast_result の流れ
@@ -267,12 +323,16 @@ module ActiveRecord
       # クォート
       # -----------------------------------------------------------------------
 
-      def quote_column_name(name)
-        %("#{name.to_s.gsub('"', '""')}")
-      end
+      # quote_column_name / quote_table_name はクラスメソッド（class << self）で定義済み。
+      # AR 8 の Quoting モジュールはインスタンスメソッドを self.class.quote_column_name に
+      # 委譲する設計のため、インスタンスメソッドの個別定義は不要。
 
-      def quote_table_name(name)
-        %("#{name.to_s.gsub('"', '""')}")
+      # SQLite / libSQL はバックスラッシュをエスケープ文字として扱わない。
+      # AbstractAdapter のデフォルト実装は \\ → \\\\ の変換を行うため、
+      # バックスラッシュを含む文字列の保存・取得で値が化ける。
+      # SQLite3Adapter と同様に、シングルクォートのエスケープのみ行う。
+      def quote_string(s)
+        s.gsub("'", "''")
       end
 
       def quoted_true

data/lib/activerecord/libsql/version.rb

@@ -2,6 +2,6 @@
 
 module ActiveRecord
   module Libsql
-    VERSION = '0.1.6'
+    VERSION = '0.1.7'
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: activerecord-libsql
 version: !ruby/object:Gem::Version
-  version: 0.1.6
+  version: 0.1.7
 platform: ruby
 authors:
 - aileron