From 28764bc44a0092e8d5d2e82d34471e9f0c575658 Mon Sep 17 00:00:00 2001
From: Matt <matthv@gmail.com>
Date: Mon, 29 Jun 2026 11:16:00 +0200
Subject: [PATCH] test(agent): add gzip passthrough regression test [PRD-567]

Mirrors the Node gzip fix (agent-nodejs#1716): a real local server + real
Faraday confirm a gzipped executor response is decompressed and the stale
Content-Encoding is not forwarded.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 .../workflow/workflow_executor_proxy_spec.rb  | 52 +++++++++++++++++++
 1 file changed, 52 insertions(+)

diff --git a/packages/forest_admin_agent/spec/lib/forest_admin_agent/routes/workflow/workflow_executor_proxy_spec.rb b/packages/forest_admin_agent/spec/lib/forest_admin_agent/routes/workflow/workflow_executor_proxy_spec.rb
index a89c80ef4..ed39710a7 100644
--- a/packages/forest_admin_agent/spec/lib/forest_admin_agent/routes/workflow/workflow_executor_proxy_spec.rb
+++ b/packages/forest_admin_agent/spec/lib/forest_admin_agent/routes/workflow/workflow_executor_proxy_spec.rb
@@ -1,5 +1,8 @@
 require 'spec_helper'
 require 'faraday'
+require 'zlib'
+require 'stringio'
+require 'socket'
 
 module ForestAdminAgent
   module Routes
@@ -272,6 +275,55 @@ def override_config(extra)
           end
         end
 
+        # Real local server + real Faraday (no run_request 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(JSON.generate('steps' => [{ 'stepId' => 'gz' }]))
+            writer.close
+            io.string
+          end
+
+          around do |example|
+            @gz_server = TCPServer.new('127.0.0.1', 0)
+            @gz_port = @gz_server.addr[1]
+            @gz_thread = Thread.new do
+              client = @gz_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
+
+            example.run
+
+            @gz_thread.kill
+            @gz_server.close
+          end
+
+          before do
+            # Runs after the outer `before`, so it wins: use the real Faraday client (no run_request
+            # stub) against the local gzip server.
+            override_config(workflow_executor_url: "http://127.0.0.1:#{@gz_port}")
+            allow(Faraday).to receive(:new).and_call_original
+          end
+
+          it 'decompresses the body and drops the stale content-encoding' do
+            result = proxy.handle_request(method: 'GET', headers: headers, params: { 'path' => 'runs/gz' })
+
+            expect(result[:status]).to eq(200)
+            expect(result[:content]).to eq('steps' => [{ 'stepId' => 'gz' }])
+            expect(result[:headers].keys.map(&:downcase)).not_to include('content-encoding')
+          end
+        end
+
         describe 'Faraday timeout' do
           it 'applies a single large timeout regardless of the verb' do
             proxy.handle_request(method: 'POST', headers: headers, params: { 'path' => "#{run_id}/trigger" }, body: '{}')