Polyglot conformance: typed-error activity hangs on PHP worker path (#331)

Author: durable-workflow-ops

app/Polyglot/ServerClient.php

@@ -262,7 +262,15 @@ private function pollTask(string $path, string $workerId, string $taskQueue, int
 
     private function requestTimeoutForPoll(int $pollTimeoutSeconds): int
     {
-        return $pollTimeoutSeconds === 0 ? 1 : $pollTimeoutSeconds + 5;
+        if ($pollTimeoutSeconds === 0) {
+            return 1;
+        }
+
+        // Keep the client-side timeout comfortably above the server-held
+        // long-poll window. Under loaded conformance runs, request handling can
+        // overrun the nominal poll timeout; if the HTTP client gives up first,
+        // the server can still lease a task to a worker that never receives it.
+        return $pollTimeoutSeconds + 15;
     }
 
     /**

tests/Unit/PolyglotServerClientTest.php

@@ -72,10 +72,10 @@ public function test_poll_request_timeout_tracks_the_requested_poll_window(): vo
         $method->setAccessible(true);
 
         $this->assertSame(1, $method->invoke($client, 0));
-        $this->assertSame(6, $method->invoke($client, 1));
-        $this->assertSame(10, $method->invoke($client, 5));
-        $this->assertSame(35, $method->invoke($client, 30));
-        $this->assertSame(65, $method->invoke($client, 60));
+        $this->assertSame(16, $method->invoke($client, 1));
+        $this->assertSame(20, $method->invoke($client, 5));
+        $this->assertSame(45, $method->invoke($client, 30));
+        $this->assertSame(75, $method->invoke($client, 60));
     }
 
     public function test_poll_workflow_task_treats_http_timeout_as_empty_poll(): void