forest_liana 9.20.0 → 9.20.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c37d0a61be7a37b80bab933cf4892cef1aceb25a5f8351d56c66dbd4d07e6c4d
-  data.tar.gz: d857a4ebb09833f6c99afb44d4daf2e06b396689331c6e28359fe78117e78727
+  metadata.gz: 7e6590ae4597f44757af9a98c3f4d8ba9e83652eedd857f45aeb1fe365e8a8ae
+  data.tar.gz: 5aa5f3150171dd65dac411007c311b9cb5409c9eb97d1a415d31e908001b0d34
 SHA512:
-  metadata.gz: d8e83baf30e64e0c1a0fed567e62c7f1ed8b1763924a813967090616cf703e103b5f53609b0579d811bd286fd107aef17fa50ec14d92dac67ed8cc77cfc51b12
-  data.tar.gz: d2c6655ba0c37f0a917018e32ac3afdc0bbcbc3de9ebb82bce59cf2e4043a53145c7897184322a1c372ea9e51c223cc03c59714d312b728b7466044bd8a575a1
+  metadata.gz: 6d848b970f6f68edd2a162f40e546740878c6fc7e2b298a3c494f6641fce908b41e2412a9e3457d13edef7d6433ce9f5704b77604851b2fd7aeca5eb60cfe691
+  data.tar.gz: 2828ff99f57028970cc2ec4e1662b70203d0d077ca0bbb45350dd4d29d09bffaffd1ef907b923e666cfc28704f97db62358bdf6709cf4b7c16bac822cd1afa15

data/app/controllers/forest_liana/actions_controller.rb

@@ -58,7 +58,7 @@ module ForestLiana
       begin
         # action.hooks[:change] is a hashmap here
         # to do the validation, only the hook names are require
-        change_hooks_name = action.hooks[:change].nil? ? nil : action.hooks[:change].keys
+        change_hooks_name = action.hooks && action.hooks[:change] ? action.hooks[:change].keys : nil
         ForestLiana::SmartActionFieldValidator.validate_smart_action_fields(result[:fields], action.name, change_hooks_name)
       rescue ForestLiana::Errors::SmartActionInvalidFieldError => invalid_field_error
         FOREST_LOGGER.warn invalid_field_error.message
@@ -107,10 +107,15 @@ module ForestLiana
         # Get the smart action hook load context
         context = get_smart_action_load_ctx(action.fields)
 
-        # Call the user-defined load hook.
-        result = action.hooks[:load].(context)
+        if action.hooks && action.hooks[:load].present?
+          # Call the user-defined load hook.
+          result = action.hooks[:load].(context)
 
-        handle_result(result, action)
+          handle_result(result, action)
+        else
+          # No load hook declared: answer the static form (Node agent parity) instead of a 404.
+          handle_result(context[:fields], action)
+        end
       end
     end
 

data/config/routes/actions.rb

@@ -1,9 +1,8 @@
 ForestLiana.apimap.each do |collection|
   if !collection.actions.empty?
     collection.actions.each do |action|
-      if action.hooks && action.hooks[:load].present?
-        post action.endpoint.sub('forest', '') + '/hooks/load' => 'actions#load', action_name: ActiveSupport::Inflector.parameterize(action.name)
-      end
+      # Unconditional: clients probe /hooks/load even when no load hook is declared.
+      post action.endpoint.sub('forest', '') + '/hooks/load' => 'actions#load', action_name: ActiveSupport::Inflector.parameterize(action.name)
       if action.hooks && action.hooks[:change].present?
         post action.endpoint.sub('forest', '') + '/hooks/change' => 'actions#change', action_name: ActiveSupport::Inflector.parameterize(action.name)
       end

data/lib/forest_liana/version.rb

@@ -1,3 +1,3 @@
 module ForestLiana
-  VERSION = "9.20.0"
+  VERSION = "9.20.1"
 end

data/spec/dummy/lib/forest_liana/collections/island.rb

@@ -28,6 +28,19 @@ class Forest::Island
 
   action 'test'
 
+  action 'static_form_action',
+    fields: [{
+      field: 'static_field',
+      type: 'String',
+      default_value: nil,
+      enums: nil,
+      is_required: false,
+      is_read_only: false,
+      reference: nil,
+      description: nil,
+      widget: nil
+    }]
+
   action 'my_action',
     fields: [foo],
     hooks: {

data/spec/requests/actions_controller_spec.rb

@@ -130,6 +130,25 @@ describe 'Requesting Actions routes', :type => :request  do
         expect(response.status).to eq(422)
       end
 
+      it 'should respond 200 with the static form when the action has no load hook' do
+        post '/forest/actions/static_form_action/hooks/load', params: JSON.dump(params), headers: headers
+        action = island.actions.select { |action| action.name == 'static_form_action' }.first
+        static_field = action.fields.select { |field| field[:field] == 'static_field' }.first
+        expect(response.status).to eq(200)
+        expect(JSON.parse(response.body)).to eq({'fields' => [static_field.merge({:value => nil}).transform_keys { |key| key.to_s.camelize(:lower) }.stringify_keys]})
+
+        first_body = response.body
+        post '/forest/actions/static_form_action/hooks/load', params: JSON.dump(params), headers: headers
+        expect(response.status).to eq(200)
+        expect(response.body).to eq(first_body)
+      end
+
+      it 'should respond 200 with empty fields when the action has no form at all' do
+        post '/forest/actions/test/hooks/load', params: JSON.dump(params), headers: headers
+        expect(response.status).to eq(200)
+        expect(JSON.parse(response.body)).to eq({'fields' => []})
+      end
+
       it 'should respond 500 with bad hook result type' do
         post '/forest/actions/fail_action/hooks/load', params: JSON.dump(params), headers: headers
         expect(response.status).to eq(500)

data/spec/requests/workflow_executions_spec.rb

@@ -1,4 +1,7 @@
 require 'rails_helper'
+require 'zlib'
+require 'stringio'
+require 'socket'
 
 describe 'Workflow executor proxy', type: :request do
   let(:run_id) { 'run_abc123' }
@@ -207,6 +210,53 @@ describe 'Workflow executor proxy', type: :request do
     end
   end
 
+  # Real local server + real HTTParty (no stub) to exercise actual gzip decompression.
+  describe 'when the executor gzip-compresses the response (nginx in front)' do
+    let(:gz_body) do
+      io = StringIO.new
+      writer = Zlib::GzipWriter.new(io)
+      writer.write({ 'id' => run_id, 'state' => 'pending' }.to_json)
+      writer.close
+      io.string
+    end
+
+    around do |example|
+      server = TCPServer.new('127.0.0.1', 0)
+      port = server.addr[1]
+      thread = Thread.new do
+        client = server.accept
+        client.readpartial(4096)
+        client.write(
+          "HTTP/1.1 200 OK\r\nContent-Type: application/json\r\n" \
+          "Content-Encoding: gzip\r\nContent-Length: #{gz_body.bytesize}\r\n" \
+          "Connection: close\r\n\r\n"
+        )
+        client.write(gz_body)
+        client.close
+      rescue IOError, Errno::ECONNRESET, Errno::EPIPE
+        nil
+      end
+
+      original = ForestLiana.workflow_executor_url
+      ForestLiana.workflow_executor_url = "http://127.0.0.1:#{port}"
+      example.run
+    ensure
+      ForestLiana.workflow_executor_url = original
+      thread&.kill
+      server&.close
+    end
+
+    before { allow(HTTParty).to receive(:get).and_call_original }
+
+    it 'decompresses the body and drops the stale content-encoding' do
+      get "/forest/_internal/executor/runs/#{run_id}", headers: auth_headers
+
+      expect(response.status).to eq(200)
+      expect(JSON.parse(response.body)).to eq('id' => run_id, 'state' => 'pending')
+      expect(response.headers['content-encoding']).to be_nil
+    end
+  end
+
   describe 'when ForestLiana.workflow_executor_url is blank' do
     around do |example|
       original = ForestLiana.workflow_executor_url

metadata

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