forest_liana 9.17.4 → 9.17.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b7a86f85b47098ca555b8f032bec32c813acc38ab3f3c4a1195d144d3a87aed1
-  data.tar.gz: 658e86bf9a50f4301de598982879ef511b022901f1f2841c48d6e244efb4dbd5
+  metadata.gz: 9b410ae815afe1b2592a38ab27cb5133198fb76b4624dee5217cdb2c65583a74
+  data.tar.gz: 2527c8944450bd5348962ef65dea48c6e7e1f1bf434e409bdbb26591d93d03c9
 SHA512:
-  metadata.gz: 87b9319960622c67bf4bbea12e4a31447daf30e640a6d85c259dc857a3acec043edbb34b10cccd676d8a050e00068e343ba778e550c79821967ae78885d39360
-  data.tar.gz: 60a672c6dd59c250d973a4afd9e1738650106eab342f1af3a260ce6437e660e1e87a20613252bd0d826cec1bb262bbf3bc6dc2822e1db42aadfc237d075d5f36
+  metadata.gz: 87d6d1e526ae2ea81c8eb5f0f4d557ae0cf65e406708cea7d998455f54d7eeee9bf9667efb6637e71b1a5deb4b084fc1fdd1dacb936499b00ab3bdcfdaec9178
+  data.tar.gz: fc0ee5ba0847140d5830715b861ccab6fe273fa7ea7c5f63ade342fa85645823c77ba51e91e2ef746dd5343c9a65e1d4012bf699fb6990e8b12dcdc344ac18db

data/app/models/forest_liana/model/collection.rb

@@ -37,6 +37,7 @@ class ForestLiana::Model::Collection
       field[:enums] = nil unless field.key?(:enums)
       field[:integration] = nil unless field.key?(:integration)
       field[:is_filterable] = true unless field.key?(:is_filterable)
+      field[:is_primary_key] = false unless field.key?(:is_primary_key)
       field[:is_read_only] = false unless field.key?(:is_read_only)
       field[:is_required] = false unless field.key?(:is_required)
       field[:is_sortable] = true unless field.key?(:is_sortable)

data/app/services/forest_liana/apimap_sorter.rb

@@ -20,6 +20,7 @@ module ForestLiana
       'enums',
       'integration',
       'is_filterable',
+      'is_primary_key',
       'is_read_only',
       'is_required',
       'is_sortable',

data/app/services/forest_liana/schema_adapter.rb

@@ -248,6 +248,7 @@ module ForestLiana
               relationship: get_relationship_type(association),
               reference: "#{association.name.to_s}.id",
               inverse_of: @model.name.demodulize.underscore,
+              is_primary_key: false,
               is_filterable: false,
               is_sortable: true,
               is_read_only: false,
@@ -278,6 +279,7 @@ module ForestLiana
               field[:field] = association.name
               field[:inverse_of] = inverse_of(association)
               field[:relationship] = get_relationship_type(association)
+              field[:is_primary_key] = false
 
               ForestLiana::SchemaUtils.disable_filter_and_sort_if_cross_db!(
                 field,
@@ -332,6 +334,7 @@ module ForestLiana
       schema = {
         field: column.name,
         type: column_type,
+        is_primary_key: primary_key?(column),
         is_filterable: true,
         is_sortable: true,
         is_read_only: false,
@@ -351,6 +354,16 @@ module ForestLiana
       add_validations(schema, column)
     end
 
+    def primary_key?(column)
+      pk = @model.primary_key
+      if pk.is_a?(Array)
+        # NOTICE: composite_primary_keys exposes an Array of column names.
+        pk.map(&:to_s).include?(column.name)
+      else
+        !pk.nil? && pk.to_s == column.name
+      end
+    end
+
     def get_schema_for_association(association)
       opts ={
         field: association.name.to_s,
@@ -358,6 +371,7 @@ module ForestLiana
         relationship: get_relationship_type(association),
         reference: "#{ForestLiana.name_for(association.klass)}.id",
         inverse_of: inverse_of(association),
+        is_primary_key: false,
         is_filterable: !is_many_association(association),
         is_sortable: true,
         is_read_only: false,

data/lib/forest_liana/collection.rb

@@ -68,6 +68,7 @@ module ForestLiana::Collection
       opts[:is_read_only] = true unless opts.has_key?(:is_read_only)
       opts[:is_read_only] = false if opts.has_key?(:set)
       opts[:is_required] = false unless opts.has_key?(:is_required)
+      opts[:is_primary_key] = false unless opts.has_key?(:is_primary_key)
       opts[:type] = "String" unless opts.has_key?(:type)
       opts[:default_value] = nil unless opts.has_key?(:default_value)
       opts[:integration] = nil unless opts.has_key?(:integration)

data/lib/forest_liana/engine.rb

@@ -59,7 +59,12 @@ module ForestLiana
     end
 
     def rake?
-      File.basename($0) == 'rake'
+      # NOTICE: `rails db:migrate` runs through Rake but exposes $0 as `rails`,
+      #         so matching the binary name alone let the bootstrapper introspect
+      #         models mid-migration (crashing on enums backed by a pending column).
+      return true if File.basename($0) == 'rake'
+
+      defined?(Rake) && Rake.application.top_level_tasks.any? { |task| task != 'default' }
     end
 
     def database_available?

data/lib/forest_liana/schema_file_updater.rb

@@ -26,6 +26,7 @@ module ForestLiana
       'enums',
       'integration',
       'is_filterable',
+      'is_primary_key',
       'is_read_only',
       'is_required',
       'is_sortable',
@@ -86,6 +87,7 @@ module ForestLiana
           field['enums'] = nil unless field.has_key?('enums')
           field['integration'] = nil unless field.has_key?('integration')
           field['is_filterable'] = false unless field.has_key?('is_filterable')
+          field['is_primary_key'] = false unless field.has_key?('is_primary_key')
           field['is_read_only'] = true unless field.has_key?('is_read_only')
           field['is_required'] = false unless field.has_key?('is_required')
           field['is_sortable'] = false unless field.has_key?('is_sortable')

data/lib/forest_liana/version.rb

@@ -1,3 +1,3 @@
 module ForestLiana
-  VERSION = "9.17.4"
+  VERSION = "9.17.6"
 end

data/spec/lib/forest_liana/collection_spec.rb

@@ -25,6 +25,7 @@ module ForestLiana
             validations: [],
             is_virtual: true,
             field: :address_type,
+            is_primary_key: false,
             is_filterable: false,
             is_sortable: false
           }

data/spec/lib/forest_liana/engine_spec.rb

@@ -0,0 +1,50 @@
+require 'rake'
+
+module ForestLiana
+  describe Engine do
+    subject(:engine) { ForestLiana::Engine.instance }
+
+    describe '#rake?' do
+      around do |example|
+        original_program_name = $0
+        example.run
+        $0 = original_program_name
+      end
+
+      context 'when invoked through the legacy `rake` binary' do
+        it 'returns true' do
+          $0 = '/usr/local/bin/rake'
+
+          expect(engine.rake?).to be(true)
+        end
+      end
+
+      context 'when invoked through `rails` while running a Rake task (e.g. rails db:migrate)' do
+        it 'returns true so the bootstrapper does not introspect models mid-task' do
+          $0 = '/usr/local/bin/rails'
+          allow(Rake.application).to receive(:top_level_tasks).and_return(['db:migrate'])
+
+          expect(engine.rake?).to be(true)
+        end
+      end
+
+      context 'when booting the application to serve (rails server / console)' do
+        it 'returns false so the bootstrapper runs and builds the apimap' do
+          $0 = '/usr/local/bin/rails'
+          allow(Rake.application).to receive(:top_level_tasks).and_return([])
+
+          expect(engine.rake?).to be(false)
+        end
+      end
+
+      context 'when only the default Rake task is present' do
+        it 'returns false' do
+          $0 = '/usr/local/bin/rails'
+          allow(Rake.application).to receive(:top_level_tasks).and_return(['default'])
+
+          expect(engine.rake?).to be(false)
+        end
+      end
+    end
+  end
+end

data/spec/services/forest_liana/schema_adapter_spec.rb

@@ -15,6 +15,7 @@ module ForestLiana
               relationship: "BelongsTo",
               reference: "addressable.id",
               inverse_of: "address",
+              is_primary_key: false,
               is_filterable: false,
               is_sortable: true,
               is_read_only: false,

data/test/services/forest_liana/schema_adapter_test.rb

@@ -36,6 +36,39 @@ module ForestLiana
       assert fields.size == 1
     end
 
+    test 'the standard primary key column is flagged is_primary_key: true' do
+      adapter = SchemaAdapter.new(IntegerField)
+      schema = adapter.send(:get_schema_for_column, IntegerField.columns_hash['id'])
+      assert_equal true, schema[:is_primary_key]
+    end
+
+    test 'a non primary key column is flagged is_primary_key: false' do
+      adapter = SchemaAdapter.new(IntegerField)
+      schema = adapter.send(:get_schema_for_column, IntegerField.columns_hash['field'])
+      assert_equal false, schema[:is_primary_key]
+    end
+
+    test 'every column of a composite primary key is flagged is_primary_key: true' do
+      adapter = SchemaAdapter.new(BelongsToField)
+      BelongsToField.define_singleton_method(:primary_key) { ['id', 'has_one_field_id'] }
+      is_pk = ->(name) do
+        adapter.send(:get_schema_for_column, BelongsToField.columns_hash[name])[:is_primary_key]
+      end
+
+      assert_equal true, is_pk.call('id')
+      assert_equal true, is_pk.call('has_one_field_id')
+      assert_equal false, is_pk.call('has_many_field_id')
+    ensure
+      BelongsToField.singleton_class.send(:remove_method, :primary_key)
+    end
+
+    test 'an association is flagged is_primary_key: false' do
+      adapter = SchemaAdapter.new(HasOneField)
+      association = HasOneField.reflect_on_association(:belongs_to_field)
+      schema = adapter.send(:get_schema_for_association, association)
+      assert_equal false, schema[:is_primary_key]
+    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.17.4
+  version: 9.17.6
 platform: ruby
 authors:
 - Sandro Munda
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-05-21 00:00:00.000000000 Z
+date: 2026-06-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -438,6 +438,7 @@ files:
 - spec/helpers/forest_liana/schema_helper_spec.rb
 - spec/lib/forest_liana/bootstrapper_spec.rb
 - spec/lib/forest_liana/collection_spec.rb
+- spec/lib/forest_liana/engine_spec.rb
 - spec/lib/forest_liana/schema_file_updater_spec.rb
 - spec/rails_helper.rb
 - spec/requests/actions_controller_spec.rb
@@ -749,6 +750,7 @@ test_files:
 - spec/helpers/forest_liana/schema_helper_spec.rb
 - spec/lib/forest_liana/bootstrapper_spec.rb
 - spec/lib/forest_liana/collection_spec.rb
+- spec/lib/forest_liana/engine_spec.rb
 - spec/lib/forest_liana/schema_file_updater_spec.rb
 - spec/rails_helper.rb
 - spec/requests/actions_controller_spec.rb