Raise review agent read timeout to 300s (#860)

actions/__init__.py

@@ -28,4 +28,4 @@
 #     ├── test_summarize_pr.py
 #     └── ...
 
-__version__ = "0.2.46"
+__version__ = "0.2.47"

actions/review_pr.py

@@ -552,7 +552,6 @@ def generate_pr_review(
             max_turns=MAX_AGENT_TURNS,
             max_cost=REVIEW_COST_SOFT_LIMIT,
             parallel_tools=True,  # review tools are read-only GitHub/diff reads, safe to batch concurrently
-            request_timeout=(30, 120),
             retries=1,  # one transient failure on any of the sequential turns would otherwise abort the whole review
             # Do not pass background=True; queued background reviews can consume the full 900s poll timeout.
         )

actions/utils/openai_utils.py

@@ -388,7 +388,7 @@ def get_agent_response(
     max_cost: float = 0.0,
     parallel_tools: bool = False,
     retries: int = 2,
-    request_timeout: tuple[int, int] = (30, 120),
+    request_timeout: tuple[int, int] = (30, 300),  # responses are unstreamed, so the read timeout caps generation
 ) -> str | dict:
     """Run an iterative OpenAI Responses API agent with application-managed function tools.
 

tests/test_first_interaction.py

@@ -175,7 +175,6 @@ def test_generate_pr_review_uses_synchronous_response(mock_get_agent_response, m
     assert kwargs["max_turns"] == review_pr.MAX_AGENT_TURNS
     assert kwargs["max_cost"] == review_pr.REVIEW_COST_SOFT_LIMIT
     assert kwargs["reasoning_effort"] == "high"
-    assert kwargs["request_timeout"] == (30, 120)
     assert "FULL FILE CONTENTS" not in mock_get_agent_response.call_args.args[0][1]["content"]
     assert (
         "Do not flag package or version availability based on web search"