forest_admin_agent 1.9.0 → 1.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: db02d65763186ba3ebdb9a253ca60c7a292c00f4b52d54e841be1ab4cd05702b
-  data.tar.gz: 87bd836d1d1f461d7aed416d8bb43397067f1575331974df1ac59ec32bbb415d
+  metadata.gz: 7e8e0238988884b3db56ac66a0478dc662ea5fed4377d50b2de685f414c455cb
+  data.tar.gz: 7986a7262febae19442a08d0916362edd97ff655d2fad7f688d43babc14e8af1
 SHA512:
-  metadata.gz: 241d1f8f861ed041334c14713018b0c0609d4193e4821c1d517975b1e780b57fc5e606fae505799f73393b700117e17cc53b8218b1a6595d71d238fc9d887cda
-  data.tar.gz: 434548e00921d627fa800d10834980476afaf3bd3a3ef5e2e80bca7bd64263622284bae86c28449b3708ac507847ff8326c6029163164930564da7fd601eb385
+  metadata.gz: d1f931091e7e9e836208337f190bc0442e7543e55afd232ffc2c98331823f1f235fa71eeae574b13a907009756783fd3d17592fe7540f091a827cee9a2e44ad2
+  data.tar.gz: 69f51337fc2eb92f6eba5f0c0d1decd260f61bb2703975336cec22c857452bb87ecbc5bc1a914444e5ec38a9052430df35f579124c3ac7a55cef61fd21a0152a

data/lib/forest_admin_agent/http/router.rb

@@ -20,6 +20,7 @@ module ForestAdminAgent
           Resources::Show.new.routes,
           Resources::Store.new.routes,
           Resources::Update.new.routes,
+          Resources::UpdateField.new.routes,
           Resources::Related::CsvRelated.new.routes,
           Resources::Related::ListRelated.new.routes,
           Resources::Related::CountRelated.new.routes,

data/lib/forest_admin_agent/routes/resources/update_field.rb

@@ -0,0 +1,140 @@
+# frozen_string_literal: true
+
+require 'jsonapi-serializers'
+
+module ForestAdminAgent
+  module Routes
+    module Resources
+      class UpdateField < AbstractAuthenticatedRoute
+        include ForestAdminAgent::Builder
+        include ForestAdminDatasourceToolkit::Components::Query
+        include ForestAdminDatasourceToolkit::Validations
+        include ForestAdminDatasourceToolkit::Schema
+
+        def setup_routes
+          add_route(
+            'forest_update_field',
+            'put',
+            '/:collection_name/:id/relationships/:field_name/:index',
+            ->(args) { handle_request(args) }
+          )
+
+          self
+        end
+
+        def handle_request(args = {})
+          build(args)
+
+          record_id = Utils::Id.unpack_id(@collection, args[:params]['id'], with_key: true)
+          field_name = args[:params]['field_name']
+          array_index = parse_index(args[:params]['index'])
+
+          @permissions.can?(:edit, @collection)
+
+          field_schema = @collection.schema[:fields][field_name]
+          validate_array_field!(field_schema, field_name)
+
+          record = fetch_record(record_id)
+
+          array = record[field_name]
+          validate_array_value!(array, field_name, array_index)
+
+          new_value = parse_value_from_body(args[:params], field_schema)
+
+          updated_array = array.dup
+          updated_array[array_index] = new_value
+
+          scope = @permissions.get_scope(@collection)
+          condition_tree = ConditionTree::ConditionTreeFactory.match_records(@collection, [record_id])
+          filter = ForestAdminDatasourceToolkit::Components::Query::Filter.new(
+            condition_tree: ConditionTree::ConditionTreeFactory.intersect([condition_tree, scope])
+          )
+          @collection.update(@caller, filter, { field_name => updated_array })
+
+          records = @collection.list(@caller, filter, ProjectionFactory.all(@collection))
+
+          {
+            name: args[:params]['collection_name'],
+            content: JSONAPI::Serializer.serialize(
+              records[0],
+              is_collection: false,
+              class_name: @collection.name,
+              serializer: Serializer::ForestSerializer
+            )
+          }
+        end
+
+        private
+
+        def parse_index(index_param)
+          index = Integer(index_param)
+          raise Http::Exceptions::ValidationError, 'Index must be non-negative' if index.negative?
+
+          index
+        rescue ArgumentError
+          raise Http::Exceptions::ValidationError, "Invalid index: #{index_param}"
+        end
+
+        def validate_array_field!(field_schema, field_name)
+          FieldValidator.validate(@collection, field_name)
+          return if field_schema.column_type.to_s.start_with?('[')
+
+          raise Http::Exceptions::ValidationError,
+                "Field '#{field_name}' is not an array (type: #{field_schema.column_type})"
+        rescue ForestAdminDatasourceToolkit::Exceptions::ValidationError => e
+          raise Http::Exceptions::NotFoundError, e.message if e.message.include?('not found')
+
+          raise Http::Exceptions::ValidationError, e.message
+        end
+
+        def fetch_record(record_id)
+          scope = @permissions.get_scope(@collection)
+          condition_tree = ConditionTree::ConditionTreeFactory.match_records(@collection, [record_id])
+          filter = ForestAdminDatasourceToolkit::Components::Query::Filter.new(
+            condition_tree: ConditionTree::ConditionTreeFactory.intersect([condition_tree, scope])
+          )
+          records = @collection.list(@caller, filter, ProjectionFactory.all(@collection))
+
+          raise Http::Exceptions::NotFoundError, 'Record not found' unless records&.any?
+
+          records[0]
+        end
+
+        def validate_array_value!(array, field_name, array_index)
+          unless array.is_a?(Array)
+            raise Http::Exceptions::UnprocessableError,
+                  "Field '#{field_name}' value is not an array (got: #{array.class})"
+          end
+
+          return unless array_index >= array.length
+
+          raise Http::Exceptions::ValidationError,
+                "Index #{array_index} out of bounds for array of length #{array.length}"
+        end
+
+        def parse_value_from_body(params, field_schema)
+          body = params[:data] || {}
+          value = body.dig(:attributes, 'value') || body.dig(:attributes, :value)
+
+          coerce_value(value, field_schema.column_type)
+        end
+
+        def coerce_value(value, column_type)
+          return value unless column_type.to_s.start_with?('[') && column_type.to_s.end_with?(']')
+
+          element_type = column_type.to_s[1..-2]
+
+          if element_type == 'Number' && value.is_a?(String)
+            begin
+              return Float(value)
+            rescue ArgumentError
+              raise Http::Exceptions::ValidationError, "Cannot coerce '#{value}' to Number - wrong type"
+            end
+          end
+
+          value
+        end
+      end
+    end
+  end
+end

data/lib/forest_admin_agent/services/ip_whitelist.rb

@@ -87,7 +87,26 @@ module ForestAdminAgent
       def fetch_rules
         response = forest_api.get('/liana/v1/ip-whitelist-rules')
 
-        body = JSON.parse(response.body)
+        unless response.status == 200
+          ForestAdminAgent::Facades::Container.logger.log('Error', {
+                                                            error: "HTTP #{response.status}",
+                                                            status: response.status,
+                                                            response: response.body
+                                                          })
+          raise ForestAdminAgent::Error, ForestAdminAgent::Utils::ErrorMessages::UNEXPECTED
+        end
+
+        begin
+          body = JSON.parse(response.body)
+        rescue JSON::ParserError => e
+          ForestAdminAgent::Facades::Container.logger.log('Error', {
+                                                            error: e.message,
+                                                            status: response.status,
+                                                            response: response.body
+                                                          })
+          raise ForestAdminAgent::Error, ForestAdminAgent::Utils::ErrorMessages::UNEXPECTED
+        end
+
         ip_whitelist_data = body['data']['attributes']
 
         @use_ip_whitelist = ip_whitelist_data['use_ip_whitelist']

data/lib/forest_admin_agent/utils/id.rb

@@ -27,7 +27,7 @@ module ForestAdminAgent
           field = collection.schema[:fields][pk_name]
           value = primary_key_values[index]
           casted_value = field.column_type == 'Number' ? value.to_i : value
-          ForestAdminDatasourceToolkit::Validations::FieldValidator.validate_value(value, field, casted_value)
+          ForestAdminDatasourceToolkit::Validations::FieldValidator.validate_value(pk_name, field, casted_value)
 
           [pk_name, casted_value]
         end.to_h

data/lib/forest_admin_agent/utils/schema/schema_emitter.rb

@@ -6,7 +6,7 @@ module ForestAdminAgent
     module Schema
       class SchemaEmitter
         LIANA_NAME = "agent-ruby"
-        LIANA_VERSION = "1.9.0"
+        LIANA_VERSION = "1.10.0"
 
         def self.generate(datasource)
           datasource.collections

data/lib/forest_admin_agent/version.rb

@@ -1,3 +1,3 @@
 module ForestAdminAgent
-  VERSION = "1.9.0"
+  VERSION = "1.10.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: forest_admin_agent
 version: !ruby/object:Gem::Version
-  version: 1.9.0
+  version: 1.10.0
 platform: ruby
 authors:
 - Matthieu
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-10-20 00:00:00.000000000 Z
+date: 2025-10-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -264,6 +264,7 @@ files:
 - lib/forest_admin_agent/routes/resources/show.rb
 - lib/forest_admin_agent/routes/resources/store.rb
 - lib/forest_admin_agent/routes/resources/update.rb
+- lib/forest_admin_agent/routes/resources/update_field.rb
 - lib/forest_admin_agent/routes/security/authentication.rb
 - lib/forest_admin_agent/routes/security/scope_invalidation.rb
 - lib/forest_admin_agent/routes/system/health_check.rb