fix: keep browser routing helpers out of generated code

Restore the generated internal types file so browser routing changes stay in custom code only.
Move joinURL into lib and keep browser.fetch limited to the SDK's existing HTTPMethod union.

Made-with: Cursor

Author: rgarcia

src/internal/types.ts

@@ -1,7 +1,7 @@
 // File generated from our OpenAPI spec by Stainless. See CONTRIBUTING.md for details.
 
 export type PromiseOrValue<T> = T | Promise<T>;
-export type HTTPMethod = 'get' | 'post' | 'put' | 'patch' | 'delete' | 'head' | 'options';
+export type HTTPMethod = 'get' | 'post' | 'put' | 'patch' | 'delete';
 
 export type KeysEnum<T> = { [P in keyof Required<T>]: true };
 
@@ -64,18 +64,14 @@ type OverloadedParameters<T> =
  * [1]: https://www.typescriptlang.org/tsconfig/#typeAcquisition
  */
 /** @ts-ignore For users with \@types/node */
-// prettier-ignore
 type UndiciTypesRequestInit = NotAny<import('../node_modules/undici-types/index.d.ts').RequestInit> | NotAny<import('../../node_modules/undici-types/index.d.ts').RequestInit> | NotAny<import('../../../node_modules/undici-types/index.d.ts').RequestInit> | NotAny<import('../../../../node_modules/undici-types/index.d.ts').RequestInit> | NotAny<import('../../../../../node_modules/undici-types/index.d.ts').RequestInit> | NotAny<import('../../../../../../node_modules/undici-types/index.d.ts').RequestInit> | NotAny<import('../../../../../../../node_modules/undici-types/index.d.ts').RequestInit> | NotAny<import('../../../../../../../../node_modules/undici-types/index.d.ts').RequestInit> | NotAny<import('../../../../../../../../../node_modules/undici-types/index.d.ts').RequestInit> | NotAny<import('../../../../../../../../../../node_modules/undici-types/index.d.ts').RequestInit>;
 /** @ts-ignore For users with undici */
-// prettier-ignore
 type UndiciRequestInit = NotAny<import('../node_modules/undici/index.d.ts').RequestInit> | NotAny<import('../../node_modules/undici/index.d.ts').RequestInit> | NotAny<import('../../../node_modules/undici/index.d.ts').RequestInit> | NotAny<import('../../../../node_modules/undici/index.d.ts').RequestInit> | NotAny<import('../../../../../node_modules/undici/index.d.ts').RequestInit> | NotAny<import('../../../../../../node_modules/undici/index.d.ts').RequestInit> | NotAny<import('../../../../../../../node_modules/undici/index.d.ts').RequestInit> | NotAny<import('../../../../../../../../node_modules/undici/index.d.ts').RequestInit> | NotAny<import('../../../../../../../../../node_modules/undici/index.d.ts').RequestInit> | NotAny<import('../../../../../../../../../../node_modules/undici/index.d.ts').RequestInit>;
 /** @ts-ignore For users with \@types/bun */
 type BunRequestInit = globalThis.FetchRequestInit;
 /** @ts-ignore For users with node-fetch@2 */
-// prettier-ignore
 type NodeFetch2RequestInit = NotAny<import('../node_modules/@types/node-fetch/index.d.ts').RequestInit> | NotAny<import('../../node_modules/@types/node-fetch/index.d.ts').RequestInit> | NotAny<import('../../../node_modules/@types/node-fetch/index.d.ts').RequestInit> | NotAny<import('../../../../node_modules/@types/node-fetch/index.d.ts').RequestInit> | NotAny<import('../../../../../node_modules/@types/node-fetch/index.d.ts').RequestInit> | NotAny<import('../../../../../../node_modules/@types/node-fetch/index.d.ts').RequestInit> | NotAny<import('../../../../../../../node_modules/@types/node-fetch/index.d.ts').RequestInit> | NotAny<import('../../../../../../../../node_modules/@types/node-fetch/index.d.ts').RequestInit> | NotAny<import('../../../../../../../../../node_modules/@types/node-fetch/index.d.ts').RequestInit> | NotAny<import('../../../../../../../../../../node_modules/@types/node-fetch/index.d.ts').RequestInit>;
 /** @ts-ignore For users with node-fetch@3, doesn't need file extension because types are at ./@types/index.d.ts */
-// prettier-ignore
 type NodeFetch3RequestInit = NotAny<import('../node_modules/node-fetch').RequestInit> | NotAny<import('../../node_modules/node-fetch').RequestInit> | NotAny<import('../../../node_modules/node-fetch').RequestInit> | NotAny<import('../../../../node_modules/node-fetch').RequestInit> | NotAny<import('../../../../../node_modules/node-fetch').RequestInit> | NotAny<import('../../../../../../node_modules/node-fetch').RequestInit> | NotAny<import('../../../../../../../node_modules/node-fetch').RequestInit> | NotAny<import('../../../../../../../../node_modules/node-fetch').RequestInit> | NotAny<import('../../../../../../../../../node_modules/node-fetch').RequestInit> | NotAny<import('../../../../../../../../../../node_modules/node-fetch').RequestInit>;
 /** @ts-ignore For users who use Deno */
 type FetchRequestInit = NonNullable<OverloadedParameters<typeof fetch>[1]>;

src/lib/browser-fetch.ts

@@ -3,7 +3,7 @@ import { KernelError } from '../core/error';
 import { buildHeaders } from '../internal/headers';
 import type { FinalRequestOptions, RequestOptions } from '../internal/request-options';
 import type { HTTPMethod } from '../internal/types';
-import { joinURL } from '../internal/utils/url';
+import { joinURL } from './join-url';
 import type { Kernel } from '../client';
 
 export interface BrowserFetchInit extends RequestInit {
@@ -54,7 +54,7 @@ export async function browserFetch(
 
 function normalizeMethod(method: string): HTTPMethod {
   const methodLower = method.toLowerCase();
-  const allowed = new Set(['get', 'post', 'put', 'patch', 'delete', 'head', 'options']);
+  const allowed = new Set(['get', 'post', 'put', 'patch', 'delete']);
   if (!allowed.has(methodLower)) {
     throw new KernelError(`browser.fetch unsupported HTTP method: ${method}`);
   }

src/lib/browser-routing.ts

@@ -1,5 +1,5 @@
 import type { Fetch, RequestInfo, RequestInit } from '../internal/builtin-types';
-import { joinURL } from '../internal/utils/url';
+import { joinURL } from './join-url';
 
 export type BrowserRoute = {
   sessionId: string;

src/internal/utils/url.ts src/lib/join-url.tssrc/internal/utils/url.ts renamed to src/lib/join-url.ts

@@ -279,16 +279,11 @@ describe('browser routing', () => {
     });
   });
 
-  test('browser.fetch supports HEAD requests', async () => {
-    const calls: Array<{ url: string; init: RequestInit | undefined }> = [];
+  test('browser.fetch rejects methods outside the SDK HTTPMethod union', async () => {
     const kernel = new Kernel({
       apiKey: 'k',
       baseURL: 'https://api.example/',
-      fetch: async (input, init?: RequestInit) => {
-        const url = normalizeURL(input);
-        calls.push({ url, init });
-        return new Response(null, { status: 204 });
-      },
+      fetch: async () => new Response(null, { status: 204 }),
     });
 
     kernel.browserRouteCache.set({
@@ -297,11 +292,12 @@ describe('browser routing', () => {
       jwt: 'token-abc',
     });
 
-    const response = await kernel.browsers.fetch('sess-1', 'https://example.com/hello', { method: 'HEAD' });
-
-    expect(response.status).toBe(204);
-    expect(calls[0]?.url).toContain('http://browser-session.test/browser/kernel/curl/raw?');
-    expect(calls[0]?.init?.method).toBe('HEAD');
+    await expect(
+      kernel.browsers.fetch('sess-1', 'https://example.com/hello', { method: 'HEAD' }),
+    ).rejects.toThrow(/unsupported HTTP method/i);
+    await expect(
+      kernel.browsers.fetch('sess-1', 'https://example.com/hello', { method: 'OPTIONS' }),
+    ).rejects.toThrow(/unsupported HTTP method/i);
   });
 
   test('defaults browser routing subresources to curl when env is unset', async () => {