fix(client-ofetch): keep error responses from throwing

Author: brolnickij

packages/openapi-ts/src/plugins/@hey-api/client-ofetch/bundle/types.ts

@@ -39,6 +39,12 @@ export interface Config<T extends ClientOptions = ClientOptions>
    * @default globalThis.fetch
    */
   fetch?: typeof fetch;
+  /**
+   * Controls the native ofetch behaviour that throws `FetchError` when
+   * `response.ok === false`. We default to suppressing it to match the fetch
+   * client semantics and let `throwOnError` drive the outcome.
+   */
+  ignoreResponseError?: OfetchOptions['ignoreResponseError'];
   // No custom fetch option: provide custom instance via `ofetch` instead
   /**
    * Please don't use the Fetch client for Next.js applications. The `next`

packages/openapi-ts/src/plugins/@hey-api/client-ofetch/bundle/utils.ts

@@ -322,6 +322,9 @@ export const buildOfetchOptions = (
     body,
     dispatcher: opts.dispatcher as OfetchOptions['dispatcher'],
     headers: opts.headers as Headers,
+    ignoreResponseError:
+      (opts.ignoreResponseError as OfetchOptions['ignoreResponseError']) ??
+      true,
     method: opts.method,
     onRequest: opts.onRequest as OfetchOptions['onRequest'],
     onRequestError: opts.onRequestError as OfetchOptions['onRequestError'],
@@ -530,6 +533,7 @@ export const createConfig = <T extends ClientOptions = ClientOptions>(
 ): Config<Omit<ClientOptions, keyof T> & T> => ({
   ...jsonBodySerializer,
   headers: defaultHeaders,
+  ignoreResponseError: true,
   parseAs: 'auto',
   querySerializer: defaultQuerySerializer,
   ...override,