change manager class suffixes from Client to Ops

Author: sid-rl

src/runloop_api_client/sdk/__init__.py

@@ -5,13 +5,13 @@
 
 from __future__ import annotations
 
-from .sync import RunloopSDK, DevboxClient, SnapshotClient, BlueprintClient, StorageObjectClient
+from .sync import RunloopSDK, DevboxOps, SnapshotOps, BlueprintOps, StorageObjectOps
 from .async_ import (
     AsyncRunloopSDK,
-    AsyncDevboxClient,
-    AsyncSnapshotClient,
-    AsyncBlueprintClient,
-    AsyncStorageObjectClient,
+    AsyncDevboxOps,
+    AsyncSnapshotOps,
+    AsyncBlueprintOps,
+    AsyncStorageObjectOps,
 )
 from .devbox import Devbox
 from .snapshot import Snapshot
@@ -31,14 +31,14 @@
     "RunloopSDK",
     "AsyncRunloopSDK",
     # Management interfaces
-    "DevboxClient",
-    "AsyncDevboxClient",
-    "BlueprintClient",
-    "AsyncBlueprintClient",
-    "SnapshotClient",
-    "AsyncSnapshotClient",
-    "StorageObjectClient",
-    "AsyncStorageObjectClient",
+    "DevboxOps",
+    "AsyncDevboxOps",
+    "BlueprintOps",
+    "AsyncBlueprintOps",
+    "SnapshotOps",
+    "AsyncSnapshotOps",
+    "StorageObjectOps",
+    "AsyncStorageObjectOps",
     # Resource classes
     "Devbox",
     "AsyncDevbox",

src/runloop_api_client/sdk/async_.py

@@ -29,7 +29,7 @@
 from ..types.object_create_params import ContentType
 
 
-class AsyncDevboxClient:
+class AsyncDevboxOps:
     """High-level async manager for creating and managing AsyncDevbox instances.
 
     Accessed via ``runloop.devbox`` from :class:`AsyncRunloopSDK`, provides
@@ -156,7 +156,7 @@ async def list(
         return [AsyncDevbox(self._client, item.id) for item in page.devboxes]
 
 
-class AsyncSnapshotClient:
+class AsyncSnapshotOps:
     """High-level async manager for working with disk snapshots.
 
     Accessed via ``runloop.snapshot`` from :class:`AsyncRunloopSDK`, provides
@@ -205,7 +205,7 @@ def from_id(self, snapshot_id: str) -> AsyncSnapshot:
         return AsyncSnapshot(self._client, snapshot_id)
 
 
-class AsyncBlueprintClient:
+class AsyncBlueprintOps:
     """High-level async manager for creating and managing blueprints.
 
     Accessed via ``runloop.blueprint`` from :class:`AsyncRunloopSDK`, provides
@@ -275,7 +275,7 @@ async def list(
         return [AsyncBlueprint(self._client, item.id) for item in page.blueprints]
 
 
-class AsyncStorageObjectClient:
+class AsyncStorageObjectOps:
     """High-level async manager for creating and managing storage objects.
 
     Accessed via ``runloop.storage_object`` from :class:`AsyncRunloopSDK`, provides
@@ -452,10 +452,10 @@ class AsyncRunloopSDK:
     """
 
     api: AsyncRunloop
-    devbox: AsyncDevboxClient
-    blueprint: AsyncBlueprintClient
-    snapshot: AsyncSnapshotClient
-    storage_object: AsyncStorageObjectClient
+    devbox: AsyncDevboxOps
+    blueprint: AsyncBlueprintOps
+    snapshot: AsyncSnapshotOps
+    storage_object: AsyncStorageObjectOps
 
     def __init__(
         self,
@@ -489,10 +489,10 @@ def __init__(
             http_client=http_client,
         )
 
-        self.devbox = AsyncDevboxClient(self.api)
-        self.blueprint = AsyncBlueprintClient(self.api)
-        self.snapshot = AsyncSnapshotClient(self.api)
-        self.storage_object = AsyncStorageObjectClient(self.api)
+        self.devbox = AsyncDevboxOps(self.api)
+        self.blueprint = AsyncBlueprintOps(self.api)
+        self.snapshot = AsyncSnapshotOps(self.api)
+        self.storage_object = AsyncStorageObjectOps(self.api)
 
     async def aclose(self) -> None:
         """Close the underlying HTTP client and release resources."""

src/runloop_api_client/sdk/sync.py

@@ -29,7 +29,7 @@
 from ..types.object_create_params import ContentType
 
 
-class DevboxClient:
+class DevboxOps:
     """High-level manager for creating and managing Devbox instances.
 
     Accessed via ``runloop.devbox`` from :class:`RunloopSDK`, provides methods to
@@ -157,7 +157,7 @@ def list(
         return [Devbox(self._client, item.id) for item in page.devboxes]
 
 
-class SnapshotClient:
+class SnapshotOps:
     """High-level manager for working with disk snapshots.
 
     Accessed via ``runloop.snapshot`` from :class:`RunloopSDK`, provides methods
@@ -202,7 +202,7 @@ def from_id(self, snapshot_id: str) -> Snapshot:
         return Snapshot(self._client, snapshot_id)
 
 
-class BlueprintClient:
+class BlueprintOps:
     """High-level manager for creating and managing blueprints.
 
     Accessed via ``runloop.blueprint`` from :class:`RunloopSDK`, provides methods
@@ -271,7 +271,7 @@ def list(
         return [Blueprint(self._client, item.id) for item in page.blueprints]
 
 
-class StorageObjectClient:
+class StorageObjectOps:
     """High-level manager for creating and managing storage objects.
 
     Accessed via ``runloop.storage_object`` from :class:`RunloopSDK`, provides
@@ -444,10 +444,10 @@ class RunloopSDK:
     """
 
     api: Runloop
-    devbox: DevboxClient
-    blueprint: BlueprintClient
-    snapshot: SnapshotClient
-    storage_object: StorageObjectClient
+    devbox: DevboxOps
+    blueprint: BlueprintOps
+    snapshot: SnapshotOps
+    storage_object: StorageObjectOps
 
     def __init__(
         self,
@@ -481,10 +481,10 @@ def __init__(
             http_client=http_client,
         )
 
-        self.devbox = DevboxClient(self.api)
-        self.blueprint = BlueprintClient(self.api)
-        self.snapshot = SnapshotClient(self.api)
-        self.storage_object = StorageObjectClient(self.api)
+        self.devbox = DevboxOps(self.api)
+        self.blueprint = BlueprintOps(self.api)
+        self.snapshot = SnapshotOps(self.api)
+        self.storage_object = StorageObjectOps(self.api)
 
     def close(self) -> None:
         """Close the underlying HTTP client and release resources."""

tests/sdk/test_async_clients.py

@@ -18,10 +18,10 @@
 from runloop_api_client.sdk import AsyncDevbox, AsyncSnapshot, AsyncBlueprint, AsyncStorageObject
 from runloop_api_client.sdk.async_ import (
     AsyncRunloopSDK,
-    AsyncDevboxClient,
-    AsyncSnapshotClient,
-    AsyncBlueprintClient,
-    AsyncStorageObjectClient,
+    AsyncDevboxOps,
+    AsyncSnapshotOps,
+    AsyncBlueprintOps,
+    AsyncStorageObjectOps,
 )
 from runloop_api_client.lib.polling import PollingConfig
 
@@ -34,7 +34,7 @@ async def test_create(self, mock_async_client: AsyncMock, devbox_view: MockDevbo
         """Test create method."""
         mock_async_client.devboxes.create_and_await_running = AsyncMock(return_value=devbox_view)
 
-        client = AsyncDevboxClient(mock_async_client)
+        client = AsyncDevboxOps(mock_async_client)
         devbox = await client.create(
             name="test-devbox",
             metadata={"key": "value"},
@@ -50,7 +50,7 @@ async def test_create_from_blueprint_id(self, mock_async_client: AsyncMock, devb
         """Test create_from_blueprint_id method."""
         mock_async_client.devboxes.create_and_await_running = AsyncMock(return_value=devbox_view)
 
-        client = AsyncDevboxClient(mock_async_client)
+        client = AsyncDevboxOps(mock_async_client)
         devbox = await client.create_from_blueprint_id(
             "bp_123",
             name="test-devbox",
@@ -65,7 +65,7 @@ async def test_create_from_blueprint_name(self, mock_async_client: AsyncMock, de
         """Test create_from_blueprint_name method."""
         mock_async_client.devboxes.create_and_await_running = AsyncMock(return_value=devbox_view)
 
-        client = AsyncDevboxClient(mock_async_client)
+        client = AsyncDevboxOps(mock_async_client)
         devbox = await client.create_from_blueprint_name(
             "my-blueprint",
             name="test-devbox",
@@ -80,7 +80,7 @@ async def test_create_from_snapshot(self, mock_async_client: AsyncMock, devbox_v
         """Test create_from_snapshot method."""
         mock_async_client.devboxes.create_and_await_running = AsyncMock(return_value=devbox_view)
 
-        client = AsyncDevboxClient(mock_async_client)
+        client = AsyncDevboxOps(mock_async_client)
         devbox = await client.create_from_snapshot(
             "snap_123",
             name="test-devbox",
@@ -92,7 +92,7 @@ async def test_create_from_snapshot(self, mock_async_client: AsyncMock, devbox_v
 
     def test_from_id(self, mock_async_client: AsyncMock) -> None:
         """Test from_id method."""
-        client = AsyncDevboxClient(mock_async_client)
+        client = AsyncDevboxOps(mock_async_client)
         devbox = client.from_id("dev_123")
 
         assert isinstance(devbox, AsyncDevbox)
@@ -107,7 +107,7 @@ async def test_list(self, mock_async_client: AsyncMock, devbox_view: MockDevboxV
         page = SimpleNamespace(devboxes=[devbox_view])
         mock_async_client.devboxes.list = AsyncMock(return_value=page)
 
-        client = AsyncDevboxClient(mock_async_client)
+        client = AsyncDevboxOps(mock_async_client)
         devboxes = await client.list(
             limit=10,
             status="running",
@@ -129,7 +129,7 @@ async def test_list(self, mock_async_client: AsyncMock, snapshot_view: MockSnaps
         page = SimpleNamespace(snapshots=[snapshot_view])
         mock_async_client.devboxes.disk_snapshots.list = AsyncMock(return_value=page)
 
-        client = AsyncSnapshotClient(mock_async_client)
+        client = AsyncSnapshotOps(mock_async_client)
         snapshots = await client.list(
             devbox_id="dev_123",
             limit=10,
@@ -143,7 +143,7 @@ async def test_list(self, mock_async_client: AsyncMock, snapshot_view: MockSnaps
 
     def test_from_id(self, mock_async_client: AsyncMock) -> None:
         """Test from_id method."""
-        client = AsyncSnapshotClient(mock_async_client)
+        client = AsyncSnapshotOps(mock_async_client)
         snapshot = client.from_id("snap_123")
 
         assert isinstance(snapshot, AsyncSnapshot)
@@ -158,7 +158,7 @@ async def test_create(self, mock_async_client: AsyncMock, blueprint_view: MockBl
         """Test create method."""
         mock_async_client.blueprints.create_and_await_build_complete = AsyncMock(return_value=blueprint_view)
 
-        client = AsyncBlueprintClient(mock_async_client)
+        client = AsyncBlueprintOps(mock_async_client)
         blueprint = await client.create(
             name="test-blueprint",
             polling_config=PollingConfig(timeout_seconds=60.0),
@@ -170,7 +170,7 @@ async def test_create(self, mock_async_client: AsyncMock, blueprint_view: MockBl
 
     def test_from_id(self, mock_async_client: AsyncMock) -> None:
         """Test from_id method."""
-        client = AsyncBlueprintClient(mock_async_client)
+        client = AsyncBlueprintOps(mock_async_client)
         blueprint = client.from_id("bp_123")
 
         assert isinstance(blueprint, AsyncBlueprint)
@@ -182,7 +182,7 @@ async def test_list(self, mock_async_client: AsyncMock, blueprint_view: MockBlue
         page = SimpleNamespace(blueprints=[blueprint_view])
         mock_async_client.blueprints.list = AsyncMock(return_value=page)
 
-        client = AsyncBlueprintClient(mock_async_client)
+        client = AsyncBlueprintOps(mock_async_client)
         blueprints = await client.list(
             limit=10,
             name="test",
@@ -203,7 +203,7 @@ async def test_create(self, mock_async_client: AsyncMock, object_view: MockObjec
         """Test create method."""
         mock_async_client.objects.create = AsyncMock(return_value=object_view)
 
-        client = AsyncStorageObjectClient(mock_async_client)
+        client = AsyncStorageObjectOps(mock_async_client)
         obj = await client.create(name="test.txt", content_type="text", metadata={"key": "value"})
 
         assert isinstance(obj, AsyncStorageObject)
@@ -217,7 +217,7 @@ async def test_create(self, mock_async_client: AsyncMock, object_view: MockObjec
 
     def test_from_id(self, mock_async_client: AsyncMock) -> None:
         """Test from_id method."""
-        client = AsyncStorageObjectClient(mock_async_client)
+        client = AsyncStorageObjectOps(mock_async_client)
         obj = client.from_id("obj_123")
 
         assert isinstance(obj, AsyncStorageObject)
@@ -230,7 +230,7 @@ async def test_list(self, mock_async_client: AsyncMock, object_view: MockObjectV
         page = SimpleNamespace(objects=[object_view])
         mock_async_client.objects.list = AsyncMock(return_value=page)
 
-        client = AsyncStorageObjectClient(mock_async_client)
+        client = AsyncStorageObjectOps(mock_async_client)
         objects = await client.list(
             content_type="text",
             limit=10,
@@ -261,7 +261,7 @@ async def test_upload_from_file(
         http_client.put = AsyncMock(return_value=mock_response)
         mock_async_client._client = http_client
 
-        client = AsyncStorageObjectClient(mock_async_client)
+        client = AsyncStorageObjectOps(mock_async_client)
         obj = await client.upload_from_file(temp_file, name="test.txt")
 
         assert isinstance(obj, AsyncStorageObject)
@@ -281,7 +281,7 @@ async def test_upload_from_text(self, mock_async_client: AsyncMock, object_view:
         http_client.put = AsyncMock(return_value=mock_response)
         mock_async_client._client = http_client
 
-        client = AsyncStorageObjectClient(mock_async_client)
+        client = AsyncStorageObjectOps(mock_async_client)
         obj = await client.upload_from_text("test content", "test.txt", metadata={"key": "value"})
 
         assert isinstance(obj, AsyncStorageObject)
@@ -305,7 +305,7 @@ async def test_upload_from_bytes(self, mock_async_client: AsyncMock, object_view
         http_client.put = AsyncMock(return_value=mock_response)
         mock_async_client._client = http_client
 
-        client = AsyncStorageObjectClient(mock_async_client)
+        client = AsyncStorageObjectOps(mock_async_client)
         obj = await client.upload_from_bytes(b"test content", "test.bin", content_type="binary")
 
         assert isinstance(obj, AsyncStorageObject)
@@ -321,7 +321,7 @@ async def test_upload_from_bytes(self, mock_async_client: AsyncMock, object_view
     @pytest.mark.asyncio
     async def test_upload_from_file_missing_path(self, mock_async_client: AsyncMock, tmp_path: Path) -> None:
         """upload_from_file should raise when file cannot be read."""
-        client = AsyncStorageObjectClient(mock_async_client)
+        client = AsyncStorageObjectOps(mock_async_client)
         missing_file = tmp_path / "missing.txt"
 
         with pytest.raises(OSError, match="Failed to read file"):
@@ -335,10 +335,10 @@ def test_init(self) -> None:
         """Test AsyncRunloopSDK initialization."""
         sdk = AsyncRunloopSDK(bearer_token="test-token")
         assert sdk.api is not None
-        assert isinstance(sdk.devbox, AsyncDevboxClient)
-        assert isinstance(sdk.snapshot, AsyncSnapshotClient)
-        assert isinstance(sdk.blueprint, AsyncBlueprintClient)
-        assert isinstance(sdk.storage_object, AsyncStorageObjectClient)
+        assert isinstance(sdk.devbox, AsyncDevboxOps)
+        assert isinstance(sdk.snapshot, AsyncSnapshotOps)
+        assert isinstance(sdk.blueprint, AsyncBlueprintOps)
+        assert isinstance(sdk.storage_object, AsyncStorageObjectOps)
 
     @pytest.mark.asyncio
     async def test_aclose(self) -> None: