Commit 28085f3
fix: only send reasoning.effort to models that support it
Each harness-engineering-bench target agent builds its Responses API request
with a hard-coded "reasoning": {"effort": ...} on every call, regardless of the
configured model. Reasoning models (gpt-5.x, o-series, *-codex) accept it, but
gpt-4o and other non-reasoning models reject the whole request with HTTP 400:
"Unsupported parameter: 'reasoning.effort' is not supported with this model".
On a swe-atlas-qna run pointed at a gpt-4o agent model this produced a wall of
400s (60 cases died with BadRequestError). The gaia and tau3 configs point their
agent model at gpt-4o in the same setup, so they share the latent bug.
Gate reasoning.effort on the model family: send it only when the resolved model
is a reasoning model (gpt-5.x / o1 / o3 / o4 / *-codex), otherwise omit it. Each
agent keeps its own effort level (high for atlas and swe-bench-pro, medium for
gaia, officeqa and tau3). No behavior change for reasoning models; gpt-4o stops
400ing. Applied uniformly to all five bench agents.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>1 parent ccc7d07 commit 28085f3
5 files changed
Lines changed: 25 additions & 5 deletions
Lines changed: 5 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
185 | 185 | | |
186 | 186 | | |
187 | 187 | | |
188 | | - | |
189 | 188 | | |
190 | 189 | | |
191 | 190 | | |
| 191 | + | |
| 192 | + | |
| 193 | + | |
| 194 | + | |
| 195 | + | |
192 | 196 | | |
193 | 197 | | |
194 | 198 | | |
| |||
Lines changed: 5 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
189 | 189 | | |
190 | 190 | | |
191 | 191 | | |
192 | | - | |
193 | 192 | | |
194 | 193 | | |
195 | 194 | | |
| 195 | + | |
| 196 | + | |
| 197 | + | |
| 198 | + | |
| 199 | + | |
196 | 200 | | |
197 | 201 | | |
198 | 202 | | |
| |||
Lines changed: 5 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
150 | 150 | | |
151 | 151 | | |
152 | 152 | | |
153 | | - | |
154 | 153 | | |
155 | 154 | | |
156 | 155 | | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
| 159 | + | |
| 160 | + | |
157 | 161 | | |
158 | 162 | | |
159 | 163 | | |
| |||
Lines changed: 5 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
382 | 382 | | |
383 | 383 | | |
384 | 384 | | |
385 | | - | |
386 | 385 | | |
387 | 386 | | |
388 | 387 | | |
| 388 | + | |
| 389 | + | |
| 390 | + | |
| 391 | + | |
| 392 | + | |
389 | 393 | | |
390 | 394 | | |
391 | 395 | | |
| |||
Lines changed: 5 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
298 | 298 | | |
299 | 299 | | |
300 | 300 | | |
301 | | - | |
302 | 301 | | |
303 | 302 | | |
304 | 303 | | |
| 304 | + | |
| 305 | + | |
| 306 | + | |
| 307 | + | |
| 308 | + | |
305 | 309 | | |
306 | 310 | | |
307 | 311 | | |
| |||
0 commit comments