add not_found variant so callers don't need a try/catch

Adds `{ status: 'not_found' }` to AcquireOutcome and wraps NotFoundError
internally so all three documented outcomes — acquired, timed_out,
not_found — are reachable via the same `result.status` switch.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: Jayko001

src/lib/browser-pools.ts

@@ -1,10 +1,12 @@
+import { NotFoundError } from '../core/error';
 import type { Kernel } from '../client';
 import type { RequestOptions } from '../internal/request-options';
 import type { BrowserPoolAcquireParams, BrowserPoolAcquireResponse } from '../resources/browser-pools';
 
 export type AcquireOutcome =
   | { status: 'acquired'; browser: BrowserPoolAcquireResponse }
-  | { status: 'timed_out' };
+  | { status: 'timed_out' }
+  | { status: 'not_found' };
 
 /**
  * Long-polling acquire that surfaces the HTTP outcome as a typed result.
@@ -14,18 +16,26 @@ export type AcquireOutcome =
  * - `{ status: 'acquired', browser }` — a browser was leased from the pool.
  * - `{ status: 'timed_out' }` — the long poll expired without a browser becoming
  *   available. Retry to keep waiting.
+ * - `{ status: 'not_found' }` — no pool exists with the given id or name.
  *
- * Rejects with `NotFoundError` if the pool does not exist.
+ * Other API errors (auth, server errors, etc.) still reject.
  */
 export async function acquire(
   client: Kernel,
   idOrName: string,
   body: BrowserPoolAcquireParams = {},
   options?: RequestOptions,
 ): Promise<AcquireOutcome> {
-  const { data, response } = await client.browserPools.acquire(idOrName, body, options).withResponse();
-  if (response.status === 204) {
-    return { status: 'timed_out' };
+  try {
+    const { data, response } = await client.browserPools.acquire(idOrName, body, options).withResponse();
+    if (response.status === 204) {
+      return { status: 'timed_out' };
+    }
+    return { status: 'acquired', browser: data };
+  } catch (err) {
+    if (err instanceof NotFoundError) {
+      return { status: 'not_found' };
+    }
+    throw err;
   }
-  return { status: 'acquired', browser: data };
 }

tests/lib/browser-pools.test.ts

@@ -1,4 +1,4 @@
-import Kernel, { NotFoundError } from '@onkernel/sdk';
+import Kernel from '@onkernel/sdk';
 
 import { acquire, type AcquireOutcome } from '../../src/lib/browser-pools';
 
@@ -32,10 +32,11 @@ describe('browser pool typed acquire', () => {
     expect(result.status).toBe('timed_out');
   });
 
-  test('rejects with NotFoundError on 404', async () => {
+  test('resolves to not_found on 404', async () => {
     const client = clientWith(async () =>
       Response.json({ code: 'not_found', message: 'pool not found' }, { status: 404 }),
     );
-    await expect(acquire(client, 'missing')).rejects.toBeInstanceOf(NotFoundError);
+    const result = await acquire(client, 'missing');
+    expect(result.status).toBe('not_found');
   });
 });