chore(internal): codegen related update (#80)

Author: stainless-app[bot]

src/core.ts

@@ -163,7 +163,7 @@ export abstract class APIClient {
     maxRetries = 2,
     timeout = 60000, // 1 minute
     httpAgent,
-    fetch: overriddenFetch,
+    fetch: overridenFetch,
   }: {
     baseURL: string;
     maxRetries?: number | undefined;
@@ -176,7 +176,7 @@ export abstract class APIClient {
     this.timeout = validatePositiveInteger('timeout', timeout);
     this.httpAgent = httpAgent;
 
-    this.fetch = overriddenFetch ?? fetch;
+    this.fetch = overridenFetch ?? fetch;
   }
 
   protected authHeaders(opts: FinalRequestOptions): Headers {
@@ -522,19 +522,9 @@ export abstract class APIClient {
 
     const timeout = setTimeout(() => controller.abort(), ms);
 
-    const fetchOptions = {
-      signal: controller.signal as any,
-      ...options,
-    };
-    if (fetchOptions.method) {
-      // Custom methods like 'patch' need to be uppercased
-      // See https://github.com/nodejs/undici/issues/2294
-      fetchOptions.method = fetchOptions.method.toUpperCase();
-    }
-
     return (
       // use undefined this binding; fetch errors if bound to something else in browser/cloudflare
-      this.fetch.call(undefined, url, fetchOptions).finally(() => {
+      this.fetch.call(undefined, url, { signal: controller.signal as any, ...options }).finally(() => {
         clearTimeout(timeout);
       })
     );

tests/index.test.ts

@@ -122,23 +122,6 @@ describe('instantiate client', () => {
     expect(spy).toHaveBeenCalledTimes(1);
   });
 
-  test('normalized method', async () => {
-    let capturedRequest: RequestInit | undefined;
-    const testFetch = async (url: RequestInfo, init: RequestInit = {}): Promise<Response> => {
-      capturedRequest = init;
-      return new Response(JSON.stringify({}), { headers: { 'Content-Type': 'application/json' } });
-    };
-
-    const client = new Browserbase({
-      baseURL: 'http://localhost:5000/',
-      apiKey: 'My API Key',
-      fetch: testFetch,
-    });
-
-    await client.patch('/foo');
-    expect(capturedRequest?.method).toEqual('PATCH');
-  });
-
   describe('baseUrl', () => {
     test('trailing slash', () => {
       const client = new Browserbase({ baseURL: 'http://localhost:5000/custom/path/', apiKey: 'My API Key' });
@@ -194,7 +177,7 @@ describe('instantiate client', () => {
     expect(client.apiKey).toBe('My API Key');
   });
 
-  test('with overridden environment variable arguments', () => {
+  test('with overriden environment variable arguments', () => {
     // set options via env var
     process.env['BROWSERBASE_API_KEY'] = 'another My API Key';
     const client = new Browserbase({ apiKey: 'My API Key' });