fix(tunnels): allow tunnel removal (#8257)

Author: stainless-app[bot]

.stats.yml

@@ -1,4 +1,4 @@
 configured_endpoints: 117
-openapi_spec_url: https://storage.googleapis.com/stainless-sdk-openapi-specs/runloop-ai%2Frunloop-3d441e9c2074aef7bd1e87b2ebfbe5c25e340084dbdfeed8a2e4dc9384561e3b.yml
-openapi_spec_hash: 250bfdf2c2270ade53c058419e23c5dc
+openapi_spec_url: https://storage.googleapis.com/stainless-sdk-openapi-specs/runloop-ai%2Frunloop-64b2ded3d3cc58131c7ebf36c411f43c3d8dabae0b5b1edd65dd7dab307c54a1.yml
+openapi_spec_hash: 80fabb5e89a61299203113505a894dc2
 config_hash: de99cfce88e2d1f02246dc6c2f43bc6c

api.md

@@ -155,7 +155,7 @@ Methods:
 - <code title="post /v1/devboxes/{id}/keep_alive">client.devboxes.<a href="./src/runloop_api_client/resources/devboxes/devboxes.py">keep_alive</a>(id) -> object</code>
 - <code title="get /v1/devboxes/disk_snapshots">client.devboxes.<a href="./src/runloop_api_client/resources/devboxes/devboxes.py">list_disk_snapshots</a>(\*\*<a href="src/runloop_api_client/types/devbox_list_disk_snapshots_params.py">params</a>) -> <a href="./src/runloop_api_client/types/devbox_snapshot_view.py">SyncDiskSnapshotsCursorIDPage[DevboxSnapshotView]</a></code>
 - <code title="post /v1/devboxes/{id}/read_file_contents">client.devboxes.<a href="./src/runloop_api_client/resources/devboxes/devboxes.py">read_file_contents</a>(id, \*\*<a href="src/runloop_api_client/types/devbox_read_file_contents_params.py">params</a>) -> str</code>
-- <code title="post /v1/devboxes/{id}/remove_tunnel">client.devboxes.<a href="./src/runloop_api_client/resources/devboxes/devboxes.py">remove_tunnel</a>(id, \*\*<a href="src/runloop_api_client/types/devbox_remove_tunnel_params.py">params</a>) -> object</code>
+- <code title="post /v1/devboxes/{id}/remove_tunnel">client.devboxes.<a href="./src/runloop_api_client/resources/devboxes/devboxes.py">remove_tunnel</a>(id) -> object</code>
 - <code title="post /v1/devboxes/{id}/resume">client.devboxes.<a href="./src/runloop_api_client/resources/devboxes/devboxes.py">resume</a>(id) -> <a href="./src/runloop_api_client/types/devbox_view.py">DevboxView</a></code>
 - <code title="get /v1/devboxes/{id}/usage">client.devboxes.<a href="./src/runloop_api_client/resources/devboxes/devboxes.py">retrieve_resource_usage</a>(id) -> <a href="./src/runloop_api_client/types/devbox_resource_usage_view.py">DevboxResourceUsageView</a></code>
 - <code title="post /v1/devboxes/{id}/shutdown">client.devboxes.<a href="./src/runloop_api_client/resources/devboxes/devboxes.py">shutdown</a>(id, \*\*<a href="src/runloop_api_client/types/devbox_shutdown_params.py">params</a>) -> <a href="./src/runloop_api_client/types/devbox_view.py">DevboxView</a></code>

src/runloop_api_client/resources/devboxes/devboxes.py

@@ -30,7 +30,6 @@
     devbox_download_file_params,
     devbox_enable_tunnel_params,
     devbox_execute_async_params,
-    devbox_remove_tunnel_params,
     devbox_snapshot_disk_params,
     devbox_wait_for_command_params,
     devbox_read_file_contents_params,
@@ -1257,14 +1256,10 @@ def read_file_contents(
             cast_to=str,
         )
 
-    @typing_extensions.deprecated(
-        "remove_tunnel is deprecated; V2 tunnels cannot be removed and close on devbox shutdown."
-    )
     def remove_tunnel(
         self,
         id: str,
         *,
-        port: int,
         # Use the following arguments if you need to pass additional parameters to the API that aren't available via kwargs.
         # The extra values given here take precedence over values defined on the client or passed to this method.
         extra_headers: Headers | None = None,
@@ -1273,14 +1268,10 @@ def remove_tunnel(
         timeout: float | httpx.Timeout | None | NotGiven = not_given,
         idempotency_key: str | None = None,
     ) -> object:
-        """[Deprecated] V2 tunnels cannot be removed and close on devbox shutdown.
-
-        This endpoint
-        removes a legacy tunnel.
+        """
+        Remove an existing V2 tunnel from the Devbox.
 
         Args:
-          port: Devbox port that tunnel will expose.
-
           extra_headers: Send extra headers
 
           extra_query: Add additional query parameters to the request
@@ -1295,7 +1286,6 @@ def remove_tunnel(
             raise ValueError(f"Expected a non-empty value for `id` but received {id!r}")
         return self._post(
             path_template("/v1/devboxes/{id}/remove_tunnel", id=id),
-            body=maybe_transform({"port": port}, devbox_remove_tunnel_params.DevboxRemoveTunnelParams),
             options=make_request_options(
                 extra_headers=extra_headers,
                 extra_query=extra_query,
@@ -2901,14 +2891,10 @@ async def read_file_contents(
             cast_to=str,
         )
 
-    @typing_extensions.deprecated(
-        "remove_tunnel is deprecated; V2 tunnels cannot be removed and close on devbox shutdown."
-    )
     async def remove_tunnel(
         self,
         id: str,
         *,
-        port: int,
         # Use the following arguments if you need to pass additional parameters to the API that aren't available via kwargs.
         # The extra values given here take precedence over values defined on the client or passed to this method.
         extra_headers: Headers | None = None,
@@ -2917,14 +2903,10 @@ async def remove_tunnel(
         timeout: float | httpx.Timeout | None | NotGiven = not_given,
         idempotency_key: str | None = None,
     ) -> object:
-        """[Deprecated] V2 tunnels cannot be removed and close on devbox shutdown.
-
-        This endpoint
-        removes a legacy tunnel.
+        """
+        Remove an existing V2 tunnel from the Devbox.
 
         Args:
-          port: Devbox port that tunnel will expose.
-
           extra_headers: Send extra headers
 
           extra_query: Add additional query parameters to the request
@@ -2939,7 +2921,6 @@ async def remove_tunnel(
             raise ValueError(f"Expected a non-empty value for `id` but received {id!r}")
         return await self._post(
             path_template("/v1/devboxes/{id}/remove_tunnel", id=id),
-            body=await async_maybe_transform({"port": port}, devbox_remove_tunnel_params.DevboxRemoveTunnelParams),
             options=make_request_options(
                 extra_headers=extra_headers,
                 extra_query=extra_query,
@@ -3481,10 +3462,8 @@ def __init__(self, devboxes: DevboxesResource) -> None:
         self.read_file_contents = to_raw_response_wrapper(
             devboxes.read_file_contents,
         )
-        self.remove_tunnel = (  # pyright: ignore[reportDeprecated]
-            to_raw_response_wrapper(
-                devboxes.remove_tunnel,  # pyright: ignore[reportDeprecated],
-            )
+        self.remove_tunnel = to_raw_response_wrapper(
+            devboxes.remove_tunnel,
         )
         self.resume = to_raw_response_wrapper(
             devboxes.resume,
@@ -3584,10 +3563,8 @@ def __init__(self, devboxes: AsyncDevboxesResource) -> None:
         self.read_file_contents = async_to_raw_response_wrapper(
             devboxes.read_file_contents,
         )
-        self.remove_tunnel = (  # pyright: ignore[reportDeprecated]
-            async_to_raw_response_wrapper(
-                devboxes.remove_tunnel,  # pyright: ignore[reportDeprecated],
-            )
+        self.remove_tunnel = async_to_raw_response_wrapper(
+            devboxes.remove_tunnel,
         )
         self.resume = async_to_raw_response_wrapper(
             devboxes.resume,
@@ -3687,10 +3664,8 @@ def __init__(self, devboxes: DevboxesResource) -> None:
         self.read_file_contents = to_streamed_response_wrapper(
             devboxes.read_file_contents,
         )
-        self.remove_tunnel = (  # pyright: ignore[reportDeprecated]
-            to_streamed_response_wrapper(
-                devboxes.remove_tunnel,  # pyright: ignore[reportDeprecated],
-            )
+        self.remove_tunnel = to_streamed_response_wrapper(
+            devboxes.remove_tunnel,
         )
         self.resume = to_streamed_response_wrapper(
             devboxes.resume,
@@ -3790,10 +3765,8 @@ def __init__(self, devboxes: AsyncDevboxesResource) -> None:
         self.read_file_contents = async_to_streamed_response_wrapper(
             devboxes.read_file_contents,
         )
-        self.remove_tunnel = (  # pyright: ignore[reportDeprecated]
-            async_to_streamed_response_wrapper(
-                devboxes.remove_tunnel,  # pyright: ignore[reportDeprecated],
-            )
+        self.remove_tunnel = async_to_streamed_response_wrapper(
+            devboxes.remove_tunnel,
         )
         self.resume = async_to_streamed_response_wrapper(
             devboxes.resume,

src/runloop_api_client/sdk/_types.py

@@ -26,7 +26,6 @@
     NetworkPolicyListParams,
     DevboxDownloadFileParams,
     DevboxEnableTunnelParams,
-    DevboxRemoveTunnelParams,
     DevboxSnapshotDiskParams,
     GatewayConfigCreateParams,
     GatewayConfigUpdateParams,
@@ -127,7 +126,7 @@ class SDKDevboxEnableTunnelParams(DevboxEnableTunnelParams, LongRequestOptions):
     pass
 
 
-class SDKDevboxRemoveTunnelParams(DevboxRemoveTunnelParams, LongRequestOptions):
+class SDKDevboxRemoveTunnelParams(LongRequestOptions):
     pass
 
 

src/runloop_api_client/sdk/async_devbox.py

@@ -840,7 +840,7 @@ async def remove_tunnel(
         :rtype: object
 
         Example:
-            >>> await devbox.net.remove_tunnel(port=8080)
+            >>> await devbox.net.remove_tunnel()
         """
         warnings.warn(
             "remove_tunnel is deprecated; V2 tunnels cannot be removed and close on devbox shutdown.",

src/runloop_api_client/sdk/devbox.py

@@ -843,7 +843,7 @@ def remove_tunnel(
         :rtype: object
 
         Example:
-            >>> devbox.net.remove_tunnel(port=8080)
+            >>> devbox.net.remove_tunnel()
         """
         warnings.warn(
             "remove_tunnel is deprecated; V2 tunnels cannot be removed and close on devbox shutdown.",

src/runloop_api_client/types/__init__.py

@@ -99,7 +99,6 @@
 from .devbox_download_file_params import DevboxDownloadFileParams as DevboxDownloadFileParams
 from .devbox_enable_tunnel_params import DevboxEnableTunnelParams as DevboxEnableTunnelParams
 from .devbox_execute_async_params import DevboxExecuteAsyncParams as DevboxExecuteAsyncParams
-from .devbox_remove_tunnel_params import DevboxRemoveTunnelParams as DevboxRemoveTunnelParams
 from .devbox_snapshot_disk_params import DevboxSnapshotDiskParams as DevboxSnapshotDiskParams
 from .scenario_list_public_params import ScenarioListPublicParams as ScenarioListPublicParams
 from .benchmark_definitions_params import BenchmarkDefinitionsParams as BenchmarkDefinitionsParams

src/runloop_api_client/types/devbox_remove_tunnel_params.py

@@ -727,21 +727,16 @@ def test_path_params_read_file_contents(self, client: Runloop) -> None:
 
     @parametrize
     def test_method_remove_tunnel(self, client: Runloop) -> None:
-        with pytest.warns(DeprecationWarning):
-            devbox = client.devboxes.remove_tunnel(
-                id="id",
-                port=0,
-            )
-
+        devbox = client.devboxes.remove_tunnel(
+            "id",
+        )
         assert_matches_type(object, devbox, path=["response"])
 
     @parametrize
     def test_raw_response_remove_tunnel(self, client: Runloop) -> None:
-        with pytest.warns(DeprecationWarning):
-            response = client.devboxes.with_raw_response.remove_tunnel(
-                id="id",
-                port=0,
-            )
+        response = client.devboxes.with_raw_response.remove_tunnel(
+            "id",
+        )
 
         assert response.is_closed is True
         assert response.http_request.headers.get("X-Stainless-Lang") == "python"
@@ -750,27 +745,23 @@ def test_raw_response_remove_tunnel(self, client: Runloop) -> None:
 
     @parametrize
     def test_streaming_response_remove_tunnel(self, client: Runloop) -> None:
-        with pytest.warns(DeprecationWarning):
-            with client.devboxes.with_streaming_response.remove_tunnel(
-                id="id",
-                port=0,
-            ) as response:
-                assert not response.is_closed
-                assert response.http_request.headers.get("X-Stainless-Lang") == "python"
+        with client.devboxes.with_streaming_response.remove_tunnel(
+            "id",
+        ) as response:
+            assert not response.is_closed
+            assert response.http_request.headers.get("X-Stainless-Lang") == "python"
 
-                devbox = response.parse()
-                assert_matches_type(object, devbox, path=["response"])
+            devbox = response.parse()
+            assert_matches_type(object, devbox, path=["response"])
 
         assert cast(Any, response.is_closed) is True
 
     @parametrize
     def test_path_params_remove_tunnel(self, client: Runloop) -> None:
-        with pytest.warns(DeprecationWarning):
-            with pytest.raises(ValueError, match=r"Expected a non-empty value for `id` but received ''"):
-                client.devboxes.with_raw_response.remove_tunnel(
-                    id="",
-                    port=0,
-                )
+        with pytest.raises(ValueError, match=r"Expected a non-empty value for `id` but received ''"):
+            client.devboxes.with_raw_response.remove_tunnel(
+                "",
+            )
 
     @parametrize
     def test_method_resume(self, client: Runloop) -> None:
@@ -2350,21 +2341,16 @@ async def test_path_params_read_file_contents(self, async_client: AsyncRunloop)
 
     @parametrize
     async def test_method_remove_tunnel(self, async_client: AsyncRunloop) -> None:
-        with pytest.warns(DeprecationWarning):
-            devbox = await async_client.devboxes.remove_tunnel(
-                id="id",
-                port=0,
-            )
-
+        devbox = await async_client.devboxes.remove_tunnel(
+            "id",
+        )
         assert_matches_type(object, devbox, path=["response"])
 
     @parametrize
     async def test_raw_response_remove_tunnel(self, async_client: AsyncRunloop) -> None:
-        with pytest.warns(DeprecationWarning):
-            response = await async_client.devboxes.with_raw_response.remove_tunnel(
-                id="id",
-                port=0,
-            )
+        response = await async_client.devboxes.with_raw_response.remove_tunnel(
+            "id",
+        )
 
         assert response.is_closed is True
         assert response.http_request.headers.get("X-Stainless-Lang") == "python"
@@ -2373,27 +2359,23 @@ async def test_raw_response_remove_tunnel(self, async_client: AsyncRunloop) -> N
 
     @parametrize
     async def test_streaming_response_remove_tunnel(self, async_client: AsyncRunloop) -> None:
-        with pytest.warns(DeprecationWarning):
-            async with async_client.devboxes.with_streaming_response.remove_tunnel(
-                id="id",
-                port=0,
-            ) as response:
-                assert not response.is_closed
-                assert response.http_request.headers.get("X-Stainless-Lang") == "python"
+        async with async_client.devboxes.with_streaming_response.remove_tunnel(
+            "id",
+        ) as response:
+            assert not response.is_closed
+            assert response.http_request.headers.get("X-Stainless-Lang") == "python"
 
-                devbox = await response.parse()
-                assert_matches_type(object, devbox, path=["response"])
+            devbox = await response.parse()
+            assert_matches_type(object, devbox, path=["response"])
 
         assert cast(Any, response.is_closed) is True
 
     @parametrize
     async def test_path_params_remove_tunnel(self, async_client: AsyncRunloop) -> None:
-        with pytest.warns(DeprecationWarning):
-            with pytest.raises(ValueError, match=r"Expected a non-empty value for `id` but received ''"):
-                await async_client.devboxes.with_raw_response.remove_tunnel(
-                    id="",
-                    port=0,
-                )
+        with pytest.raises(ValueError, match=r"Expected a non-empty value for `id` but received ''"):
+            await async_client.devboxes.with_raw_response.remove_tunnel(
+                "",
+            )
 
     @parametrize
     async def test_method_resume(self, async_client: AsyncRunloop) -> None:

tests/api_resources/test_devboxes.py

@@ -186,7 +186,6 @@ async def test_remove_tunnel(self, mock_async_client: AsyncMock) -> None:
         devbox = AsyncDevbox(mock_async_client, "dbx_123")
         with pytest.warns(DeprecationWarning, match="remove_tunnel is deprecated"):
             result = await devbox.net.remove_tunnel(
-                port=8080,
                 extra_headers={"X-Custom": "value"},
                 extra_query={"param": "value"},
                 extra_body={"key": "value"},