forest_liana 9.18.0 → 9.18.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bf3f27387c51c6e72f363fa0f92a788ed62b57874e685bc26b58642d5b57ed64
-  data.tar.gz: 87d8c7ee8492972fcc9172714b8056f4bd6003842bdaa64c590b07744735517a
+  metadata.gz: 38c7944d7c156c137b196297c69f904cc8e2506003dcf490cebe6f533f0f51e9
+  data.tar.gz: 7738c3daeaf5d9278920974e9d2c3c3f83d1a237b3323bc8f00d289335e113ee
 SHA512:
-  metadata.gz: e642957d6f0925fd6a2567c44c3fce1f386e85da1ebc0abfc1fac9c36c19ee5fdc38ed33430d8ad2fd4e7e1a4a72f64c3c7894c187bb3d8445c511bcd3c2eead
-  data.tar.gz: 66098b227f8abcd58e3e8da41954e560d8dae0e32ad8f4c995cb1f5daab4a2481af0c4b3c1f9e35bd7f7a816a68c826dc4df161f6b83eec78be1b592b4834896
+  metadata.gz: 420b641ccd725591b6413fe4be8e680e77bf97582ddeb75e5f4dee0a7c534fb1564017f5b9df19adc3d0829a04b56ce46147b288e22172cd8f95d72f828271a6
+  data.tar.gz: 8b339928ecf3bd9c6f81957b78962936a02510d831510233fb159af39f4fc09228ea0b83f2745557548551171fe7ad101314809e96d72659e984811cb5b04f44

data/app/services/forest_liana/record_findable.rb

@@ -14,13 +14,18 @@ module ForestLiana
       end
     end
 
+    # Composite ids reach us in two shapes, both ordered like the model's primary_key:
+    #   - JSON array "[v1,v2]"  -> from the Forest frontend
+    #   - pipe-joined "v1|v2"   -> from the agent-client (workflow executor, Forest convention)
+    # Either way the values stay in primary_key order, so find_record can zip them directly.
     def parse_composite_id(id)
       return id if id.is_a?(Array)
 
-      if id.to_s.start_with?('[') && id.to_s.end_with?(']')
-        JSON.parse(id.to_s)
+      str = id.to_s
+      if str.start_with?('[') && str.end_with?(']')
+        JSON.parse(str)
       else
-        raise ForestLiana::Errors::HTTP422Error.new("Composite primary key ID must be in format [value1,value2], received: #{id}")
+        str.split('|')
       end
     end
   end

data/app/services/forest_liana/schema_adapter.rb

@@ -279,7 +279,8 @@ module ForestLiana
               field[:field] = association.name
               field[:inverse_of] = inverse_of(association)
               field[:relationship] = get_relationship_type(association)
-              field[:is_primary_key] = false
+              # NOTICE: Preserve is_primary_key from the underlying column; a
+              #         composite PK made of FKs would otherwise lose its flag.
 
               ForestLiana::SchemaUtils.disable_filter_and_sort_if_cross_db!(
                 field,

data/lib/forest_liana/version.rb

@@ -1,3 +1,3 @@
 module ForestLiana
-  VERSION = "9.18.0"
+  VERSION = "9.18.2"
 end

data/spec/services/forest_liana/composite_primary_keys_spec.rb

@@ -30,14 +30,14 @@ module ForestLiana
           expect(helper.parse_composite_id('["a","b"]')).to eq(['a', 'b'])
         end
 
-        it 'returns array as-is when already an array' do
-          expect(helper.parse_composite_id([1, 2])).to eq([1, 2])
+        it 'parses composite ID in pipe-joined format (agent-client / workflow executor)' do
+          expect(helper.parse_composite_id('1|2')).to eq(['1', '2'])
+          expect(helper.parse_composite_id('10|20')).to eq(['10', '20'])
+          expect(helper.parse_composite_id('a|b')).to eq(['a', 'b'])
         end
 
-        it 'raises error for invalid composite ID format' do
-          expect {
-            helper.parse_composite_id('invalid')
-          }.to raise_error(ForestLiana::Errors::HTTP422Error)
+        it 'returns array as-is when already an array' do
+          expect(helper.parse_composite_id([1, 2])).to eq([1, 2])
         end
       end
 

data/test/services/forest_liana/schema_adapter_test.rb

@@ -69,6 +69,23 @@ module ForestLiana
       assert_equal false, schema[:is_primary_key]
     end
 
+    test 'a foreign key composing the primary key keeps is_primary_key: true once turned into an association' do
+      name = ForestLiana.name_for(BelongsToField)
+      original = ForestLiana.apimap.find { |c| c.name.to_s == name }
+      ForestLiana.apimap.delete(original) if original
+      BelongsToField.define_singleton_method(:primary_key) { ['id', 'has_one_field_id'] }
+
+      schema = SchemaAdapter.new(BelongsToField).perform
+      field = schema.fields.find { |f| f[:field] == :has_one_field }
+
+      assert_equal true, field[:is_primary_key]
+    ensure
+      BelongsToField.singleton_class.send(:remove_method, :primary_key)
+      rebuilt = ForestLiana.apimap.find { |c| c.name.to_s == name }
+      ForestLiana.apimap.delete(rebuilt) if rebuilt
+      ForestLiana.apimap << original if original
+    end
+
     test 'belongsTo relationship' do
       schema = SchemaAdapter.new(BelongsToField).perform
       fields = schema.fields.select do |field|

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: forest_liana
 version: !ruby/object:Gem::Version
-  version: 9.18.0
+  version: 9.18.2
 platform: ruby
 authors:
 - Sandro Munda
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-06-04 00:00:00.000000000 Z
+date: 2026-06-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails