Skip to content

Commit 02a2f59

Browse files
committed
refactor: inline browser resource passthrough returns
Remove the leftover result/page wrappers from browser create, retrieve, update, and list now that route cache warming lives in the shared response hooks. Keep the actual curl and raw HTTP browser routing surface unchanged. Made-with: Cursor
1 parent a873a18 commit 02a2f59

1 file changed

Lines changed: 8 additions & 16 deletions

File tree

src/kernel/resources/browsers/browsers.py

Lines changed: 8 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -226,7 +226,7 @@ def create(
226226
227227
timeout: Override the client-level default timeout for this request, in seconds
228228
"""
229-
result = self._post(
229+
return self._post(
230230
"/browsers",
231231
body=maybe_transform(
232232
{
@@ -249,7 +249,6 @@ def create(
249249
),
250250
cast_to=BrowserCreateResponse,
251251
)
252-
return result
253252

254253
def retrieve(
255254
self,
@@ -279,7 +278,7 @@ def retrieve(
279278
"""
280279
if not id:
281280
raise ValueError(f"Expected a non-empty value for `id` but received {id!r}")
282-
result = self._get(
281+
return self._get(
283282
path_template("/browsers/{id}", id=id),
284283
options=make_request_options(
285284
extra_headers=extra_headers,
@@ -292,7 +291,6 @@ def retrieve(
292291
),
293292
cast_to=BrowserRetrieveResponse,
294293
)
295-
return result
296294

297295
def update(
298296
self,
@@ -334,7 +332,7 @@ def update(
334332
"""
335333
if not id:
336334
raise ValueError(f"Expected a non-empty value for `id` but received {id!r}")
337-
result = self._patch(
335+
return self._patch(
338336
path_template("/browsers/{id}", id=id),
339337
body=maybe_transform(
340338
{
@@ -350,7 +348,6 @@ def update(
350348
),
351349
cast_to=BrowserUpdateResponse,
352350
)
353-
return result
354351

355352
def list(
356353
self,
@@ -393,7 +390,7 @@ def list(
393390
394391
timeout: Override the client-level default timeout for this request, in seconds
395392
"""
396-
page = self._get_api_list(
393+
return self._get_api_list(
397394
"/browsers",
398395
page=SyncOffsetPagination[BrowserListResponse],
399396
options=make_request_options(
@@ -414,7 +411,6 @@ def list(
414411
),
415412
model=BrowserListResponse,
416413
)
417-
return page
418414

419415
@typing_extensions.deprecated("deprecated")
420416
def delete(
@@ -788,7 +784,7 @@ async def create(
788784
789785
timeout: Override the client-level default timeout for this request, in seconds
790786
"""
791-
result = await self._post(
787+
return await self._post(
792788
"/browsers",
793789
body=await async_maybe_transform(
794790
{
@@ -811,7 +807,6 @@ async def create(
811807
),
812808
cast_to=BrowserCreateResponse,
813809
)
814-
return result
815810

816811
async def retrieve(
817812
self,
@@ -841,7 +836,7 @@ async def retrieve(
841836
"""
842837
if not id:
843838
raise ValueError(f"Expected a non-empty value for `id` but received {id!r}")
844-
result = await self._get(
839+
return await self._get(
845840
path_template("/browsers/{id}", id=id),
846841
options=make_request_options(
847842
extra_headers=extra_headers,
@@ -854,7 +849,6 @@ async def retrieve(
854849
),
855850
cast_to=BrowserRetrieveResponse,
856851
)
857-
return result
858852

859853
async def update(
860854
self,
@@ -896,7 +890,7 @@ async def update(
896890
"""
897891
if not id:
898892
raise ValueError(f"Expected a non-empty value for `id` but received {id!r}")
899-
result = await self._patch(
893+
return await self._patch(
900894
path_template("/browsers/{id}", id=id),
901895
body=await async_maybe_transform(
902896
{
@@ -912,7 +906,6 @@ async def update(
912906
),
913907
cast_to=BrowserUpdateResponse,
914908
)
915-
return result
916909

917910
def list(
918911
self,
@@ -955,7 +948,7 @@ def list(
955948
956949
timeout: Override the client-level default timeout for this request, in seconds
957950
"""
958-
page = self._get_api_list(
951+
return self._get_api_list(
959952
"/browsers",
960953
page=AsyncOffsetPagination[BrowserListResponse],
961954
options=make_request_options(
@@ -976,7 +969,6 @@ def list(
976969
),
977970
model=BrowserListResponse,
978971
)
979-
return page
980972

981973
@typing_extensions.deprecated("deprecated")
982974
async def delete(

0 commit comments

Comments
 (0)