fix(providers): let a deadline while reading an error body propagate

Author: waleedlatif1

apps/sim/providers/openai/core.transport-phase.test.ts

@@ -174,6 +174,25 @@ describe('OpenAI transport phase annotation', () => {
     expect(fetchMock).toHaveBeenCalledTimes(2)
   })
 
+  /**
+   * Reading the error body of a non-OK response can itself hit the deadline or be
+   * cancelled. Swallowing that would report the HTTP status as the failure and lose both
+   * the transport detail and the fact that the user aborted.
+   */
+  it('propagates a deadline hit while reading a non-OK error body', async () => {
+    const unreadable = {
+      ok: false,
+      status: 502,
+      headers: new Headers(),
+      text: () => Promise.reject(timeoutError()),
+    }
+
+    const error = await run(vi.fn().mockResolvedValue(unreadable)).catch((e) => e)
+
+    expect(error.message).toContain('The operation timed out.')
+    expect(error.message).not.toContain('502')
+  })
+
   it('leaves a healthy response entirely unaffected', async () => {
     const fetchMock = vi.fn().mockResolvedValue({
       ok: true,

apps/sim/providers/openai/core.ts

@@ -292,9 +292,12 @@ export async function executeResponsesProviderRequest(
    * block error, so it is bounded and falls back to `statusText`. A structured provider
    * message is returned untruncated on purpose: the reasoning-summary strip-and-retry
    * fallback matches on its text.
+   *
+   * A failed body read is deliberately not caught: a deadline or a cancellation here must
+   * stay distinguishable from an error response that simply carried no body.
    */
   const parseErrorResponse = async (response: Response): Promise<string> => {
-    const text = await response.text().catch(() => '')
+    const text = await response.text()
     try {
       const payload = JSON.parse(text)
       if (payload?.error?.message) return payload.error.message