refactor!: update default timeout tiers on non-storage resource clients (#664)

## Goal

Update default timeout tiers on individual resource client methods.

## Tier philosophy

| Tier | Value | Use for |
|------|-------|---------|
| `short` (5s) | Simple metadata CRUD | get/update/delete single
resources, quick checks |
| `medium` (30s) | Moderate operations | list with pagination, create,
batch ops, triggering runs/builds |
| `long` (360s) | Heavy operations | downloading/streaming large data,
large payloads |
| `no_timeout` | Indefinite | Polling/waiting (`wait_for_finish`,
`call`, streaming logs) |

## Current state

Only storage clients (Dataset, KVS, RQ) were tiered in PR #653.
Everything else defaults to `'long'` — meaning even a simple
`actor.get()` has a 360s timeout.

## Changes description

### ActorClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `get()` | long | **short** | Simple metadata fetch |
| `update()` | long | **short** | Simple metadata update |
| `delete()` | long | **short** | Simple delete |
| `start()` | long | **medium** | Triggers a run, returns run info (some
server processing) |
| `call()` | long | **no_timeout** | Starts run AND waits for finish —
can take minutes or even much more |
| `build()` | long | **medium** | Triggers a build, returns quickly |
| `default_build()` | long | **short** | Simple metadata fetch |
| `last_run()` | long | **short** | Simple metadata fetch |
| `validate_input()` | long | **short** | Quick validation call |

### ActorCollectionClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `list()` | long | **medium** | Paginated list, could be large |
| `create()` | long | **medium** | Creates resource, some processing |

### ActorVersionClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `get()` | long | **short** | Simple metadata |
| `update()` | long | **short** | Simple metadata |
| `delete()` | long | **short** | Simple delete |

### ActorVersionCollectionClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `list()` | long | **short** | Typically small lists |
| `create()` | long | **short** | Simple create |

### ActorEnvVarClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `get()` | long | **short** | Simple metadata |
| `update()` | long | **short** | Simple metadata |
| `delete()` | long | **short** | Simple delete |

### ActorEnvVarCollectionClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `list()` | long | **short** | Small lists |
| `create()` | long | **short** | Simple create |

### BuildClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `get()` | long | **short** | Simple metadata |
| `delete()` | long | **short** | Simple delete |
| `abort()` | long | **short** | Sends abort signal, returns immediately
|
| `get_open_api_definition()` | long | **medium** | Could be a larger
JSON payload |
| `wait_for_finish()` | no_timeout | no_timeout | Already correct |

### BuildCollectionClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `list()` | long | **medium** | Paginated list |

### RunClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `get()` | long | **short** | Simple metadata |
| `update()` | long | **short** | Simple metadata |
| `delete()` | long | **short** | Simple delete |
| `abort()` | long | **short** | Sends abort signal |
| `wait_for_finish()` | no_timeout | no_timeout | Already correct |
| `metamorph()` | long | **medium** | Triggers transformation |
| `resurrect()` | long | **medium** | Restarts a run |
| `reboot()` | long | **medium** | Reboots a run |
| `get_streamed_log()` | long | **long** | Streaming, keep long |
| `charge()` | long | **short** | Quick API call |
| `get_status_message_watcher()` | long | **long** | Long-lived watcher
|

### RunCollectionClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `list()` | long | **medium** | Paginated list |

### TaskClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `get()` | long | **short** | Simple metadata |
| `update()` | long | **short** | Simple metadata |
| `delete()` | long | **short** | Simple delete |
| `start()` | long | **medium** | Triggers a run |
| `call()` | long | **no_timeout** | Starts and waits for completion |
| `get_input()` | long | **short** | Simple metadata |
| `update_input()` | long | **short** | Simple metadata update |
| `last_run()` | long | **short** | Simple metadata |

### TaskCollectionClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `list()` | long | **medium** | Paginated list |
| `create()` | long | **medium** | Creates resource |

### ScheduleClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `get()` | long | **short** | Simple metadata |
| `update()` | long | **short** | Simple metadata |
| `delete()` | long | **short** | Simple delete |
| `get_log()` | long | **medium** | Returns list of schedule invocations
|

### ScheduleCollectionClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `list()` | long | **medium** | Paginated list |
| `create()` | long | **short** | Simple create |

### WebhookClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `get()` | long | **short** | Simple metadata |
| `update()` | long | **short** | Simple metadata |
| `delete()` | long | **short** | Simple delete |
| `test()` | long | **medium** | Triggers test dispatch |

### WebhookCollectionClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `list()` | long | **medium** | Paginated list |
| `create()` | long | **short** | Simple create |

### WebhookDispatchClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `get()` | long | **short** | Simple metadata |

### WebhookDispatchCollectionClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `list()` | long | **medium** | Paginated list |

### StoreCollectionClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `list()` | long | **medium** | Paginated list |

### LogClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `get()` | long | **long** | This is not metadata call. Logs can be
large — keep as is |
| `get_as_bytes()` | long | **long** | Same |
| `stream()` | long | **long** | Streaming — long is reasonable |

### UserClient

| Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `get()` | long | **short** | Simple metadata |
| `monthly_usage()` | long | **short** | Simple metadata |
| `limits()` | long | **short** | Simple metadata |
| `update_limits()` | long | **short** | Simple metadata update |

### Storage Collection Clients (currently all `long`)

| Client.Method | Current | Recommended | Rationale |
|--------|---------|-------------|-----------|
| `DatasetCollectionClient.list()` | long | **medium** | Paginated list
|
| `DatasetCollectionClient.get_or_create()` | long | **short** | Quick
idempotent op |
| `KeyValueStoreCollectionClient.list()` | long | **medium** | Paginated
list |
| `KeyValueStoreCollectionClient.get_or_create()` | long | **short** |
Quick idempotent op |
| `RequestQueueCollectionClient.list()` | long | **medium** | Paginated
list |
| `RequestQueueCollectionClient.get_or_create()` | long | **short** |
Quick idempotent op |

## Summary of Changes

- **~60 methods** `long` → `short` (simple CRUD on non-storage
resources)
- **~20 methods** `long` → `medium` (list operations, creates, trigger
operations)
- **2 methods** `long` → `no_timeout` (`actor.call()`, `task.call()`)
- **~10 methods** stay `long` (log operations, streaming, data-heavy
ops)
- **Storage clients** (dataset, kvs, request_queue) already well-tiered
— only their collection clients need updates

## Test plan

- [x] CI passes

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Author: vdusek

src/apify_client/_resource_clients/actor.py

@@ -85,7 +85,7 @@ def __init__(
             **kwargs,
         )
 
-    def get(self, *, timeout: Timeout = 'long') -> Actor | None:
+    def get(self, *, timeout: Timeout = 'short') -> Actor | None:
         """Retrieve the Actor.
 
         https://docs.apify.com/api/v2#/reference/actors/actor-object/get-actor
@@ -129,7 +129,7 @@ def update(
         pricing_infos: list[dict[str, Any]] | None = None,
         actor_permission_level: ActorPermissionLevel | None = None,
         tagged_builds: dict[str, None | dict[str, str]] | None = None,
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'short',
     ) -> Actor:
         """Update the Actor with the specified fields.
 
@@ -209,7 +209,7 @@ def update(
         result = self._update(timeout=timeout, **actor_fields.model_dump(by_alias=True, exclude_none=True))
         return ActorResponse.model_validate(result).data
 
-    def delete(self, *, timeout: Timeout = 'long') -> None:
+    def delete(self, *, timeout: Timeout = 'short') -> None:
         """Delete the Actor.
 
         https://docs.apify.com/api/v2#/reference/actors/actor-object/delete-actor
@@ -233,7 +233,7 @@ def start(
         force_permission_level: ActorPermissionLevel | None = None,
         wait_for_finish: int | None = None,
         webhooks: list[dict] | None = None,
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'medium',
     ) -> Run:
         """Start the Actor and immediately return the Run object.
 
@@ -310,7 +310,7 @@ def call(
         force_permission_level: ActorPermissionLevel | None = None,
         wait_duration: timedelta | None = None,
         logger: Logger | None | Literal['default'] = 'default',
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'no_timeout',
     ) -> Run | None:
         """Start the Actor and wait for it to finish before returning the Run object.
 
@@ -386,7 +386,7 @@ def build(
         tag: str | None = None,
         use_cache: bool | None = None,
         wait_for_finish: int | None = None,
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'medium',
     ) -> Build:
         """Build the Actor.
 
@@ -444,7 +444,7 @@ def default_build(
         self,
         *,
         wait_for_finish: int | None = None,
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'short',
     ) -> BuildClient:
         """Retrieve Actor's default build.
 
@@ -533,7 +533,7 @@ def validate_input(
         *,
         build_tag: str | None = None,
         content_type: str | None = None,
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'short',
     ) -> bool:
         """Validate an input for the Actor that defines an input schema.
 
@@ -581,7 +581,7 @@ def __init__(
             **kwargs,
         )
 
-    async def get(self, *, timeout: Timeout = 'long') -> Actor | None:
+    async def get(self, *, timeout: Timeout = 'short') -> Actor | None:
         """Retrieve the Actor.
 
         https://docs.apify.com/api/v2#/reference/actors/actor-object/get-actor
@@ -625,7 +625,7 @@ async def update(
         pricing_infos: list[dict[str, Any]] | None = None,
         actor_permission_level: ActorPermissionLevel | None = None,
         tagged_builds: dict[str, None | dict[str, str]] | None = None,
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'short',
     ) -> Actor:
         """Update the Actor with the specified fields.
 
@@ -705,7 +705,7 @@ async def update(
         result = await self._update(timeout=timeout, **actor_fields.model_dump(by_alias=True, exclude_none=True))
         return ActorResponse.model_validate(result).data
 
-    async def delete(self, *, timeout: Timeout = 'long') -> None:
+    async def delete(self, *, timeout: Timeout = 'short') -> None:
         """Delete the Actor.
 
         https://docs.apify.com/api/v2#/reference/actors/actor-object/delete-actor
@@ -729,7 +729,7 @@ async def start(
         force_permission_level: ActorPermissionLevel | None = None,
         wait_for_finish: int | None = None,
         webhooks: list[dict] | None = None,
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'medium',
     ) -> Run:
         """Start the Actor and immediately return the Run object.
 
@@ -806,7 +806,7 @@ async def call(
         force_permission_level: ActorPermissionLevel | None = None,
         wait_duration: timedelta | None = None,
         logger: Logger | None | Literal['default'] = 'default',
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'no_timeout',
     ) -> Run | None:
         """Start the Actor and wait for it to finish before returning the Run object.
 
@@ -886,7 +886,7 @@ async def build(
         tag: str | None = None,
         use_cache: bool | None = None,
         wait_for_finish: int | None = None,
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'medium',
     ) -> Build:
         """Build the Actor.
 
@@ -944,7 +944,7 @@ async def default_build(
         self,
         *,
         wait_for_finish: int | None = None,
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'short',
     ) -> BuildClientAsync:
         """Retrieve Actor's default build.
 
@@ -1033,7 +1033,7 @@ async def validate_input(
         *,
         build_tag: str | None = None,
         content_type: str | None = None,
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'short',
     ) -> bool:
         """Validate an input for the Actor that defines an input schema.
 

src/apify_client/_resource_clients/actor_collection.py

@@ -49,7 +49,7 @@ def list(
         offset: int | None = None,
         desc: bool | None = None,
         sort_by: Literal['createdAt', 'stats.lastRunStartedAt'] | None = 'createdAt',
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'medium',
     ) -> ListOfActors:
         """List the Actors the user has created or used.
 
@@ -94,7 +94,7 @@ def create(
         actor_standby_idle_timeout: timedelta | None = None,
         actor_standby_build: str | None = None,
         actor_standby_memory_mbytes: int | None = None,
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'medium',
     ) -> Actor:
         """Create a new Actor.
 
@@ -194,7 +194,7 @@ async def list(
         offset: int | None = None,
         desc: bool | None = None,
         sort_by: Literal['createdAt', 'stats.lastRunStartedAt'] | None = 'createdAt',
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'medium',
     ) -> ListOfActors:
         """List the Actors the user has created or used.
 
@@ -239,7 +239,7 @@ async def create(
         actor_standby_idle_timeout: timedelta | None = None,
         actor_standby_build: str | None = None,
         actor_standby_memory_mbytes: int | None = None,
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'medium',
     ) -> Actor:
         """Create a new Actor.
 

src/apify_client/_resource_clients/actor_env_var.py

@@ -31,7 +31,7 @@ def __init__(
             **kwargs,
         )
 
-    def get(self, *, timeout: Timeout = 'long') -> EnvVar | None:
+    def get(self, *, timeout: Timeout = 'short') -> EnvVar | None:
         """Return information about the Actor environment variable.
 
         https://docs.apify.com/api/v2#/reference/actors/environment-variable-object/get-environment-variable
@@ -53,7 +53,7 @@ def update(
         is_secret: bool | None = None,
         name: str,
         value: str,
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'short',
     ) -> EnvVar:
         """Update the Actor environment variable with specified fields.
 
@@ -74,7 +74,7 @@ def update(
         )
         return EnvVarResponse.model_validate(result).data
 
-    def delete(self, *, timeout: Timeout = 'long') -> None:
+    def delete(self, *, timeout: Timeout = 'short') -> None:
         """Delete the Actor environment variable.
 
         https://docs.apify.com/api/v2#/reference/actors/environment-variable-object/delete-environment-variable
@@ -106,7 +106,7 @@ def __init__(
             **kwargs,
         )
 
-    async def get(self, *, timeout: Timeout = 'long') -> EnvVar | None:
+    async def get(self, *, timeout: Timeout = 'short') -> EnvVar | None:
         """Return information about the Actor environment variable.
 
         https://docs.apify.com/api/v2#/reference/actors/environment-variable-object/get-environment-variable
@@ -128,7 +128,7 @@ async def update(
         is_secret: bool | None = None,
         name: str,
         value: str,
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'short',
     ) -> EnvVar:
         """Update the Actor environment variable with specified fields.
 
@@ -149,7 +149,7 @@ async def update(
         )
         return EnvVarResponse.model_validate(result).data
 
-    async def delete(self, *, timeout: Timeout = 'long') -> None:
+    async def delete(self, *, timeout: Timeout = 'short') -> None:
         """Delete the Actor environment variable.
 
         https://docs.apify.com/api/v2#/reference/actors/environment-variable-object/delete-environment-variable

src/apify_client/_resource_clients/actor_env_var_collection.py

@@ -29,7 +29,7 @@ def __init__(
             **kwargs,
         )
 
-    def list(self, *, timeout: Timeout = 'long') -> ListOfEnvVars:
+    def list(self, *, timeout: Timeout = 'short') -> ListOfEnvVars:
         """List the available Actor environment variables.
 
         https://docs.apify.com/api/v2#/reference/actors/environment-variable-collection/get-list-of-environment-variables
@@ -49,7 +49,7 @@ def create(
         is_secret: bool | None = None,
         name: str,
         value: str,
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'short',
     ) -> EnvVar:
         """Create a new Actor environment variable.
 
@@ -90,7 +90,7 @@ def __init__(
             **kwargs,
         )
 
-    async def list(self, *, timeout: Timeout = 'long') -> ListOfEnvVars:
+    async def list(self, *, timeout: Timeout = 'short') -> ListOfEnvVars:
         """List the available Actor environment variables.
 
         https://docs.apify.com/api/v2#/reference/actors/environment-variable-collection/get-list-of-environment-variables
@@ -110,7 +110,7 @@ async def create(
         is_secret: bool | None = None,
         name: str,
         value: str,
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'short',
     ) -> EnvVar:
         """Create a new Actor environment variable.
 

src/apify_client/_resource_clients/actor_version.py

@@ -49,7 +49,7 @@ def __init__(
             **kwargs,
         )
 
-    def get(self, *, timeout: Timeout = 'long') -> Version | None:
+    def get(self, *, timeout: Timeout = 'short') -> Version | None:
         """Return information about the Actor version.
 
         https://docs.apify.com/api/v2#/reference/actors/version-object/get-version
@@ -76,7 +76,7 @@ def update(
         git_repo_url: str | None = None,
         tarball_url: str | None = None,
         github_gist_url: str | None = None,
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'short',
     ) -> Version:
         """Update the Actor version with specified fields.
 
@@ -115,7 +115,7 @@ def update(
         result = self._update(timeout=timeout, **version_fields.model_dump(by_alias=True, exclude_none=True))
         return VersionResponse.model_validate(result).data
 
-    def delete(self, *, timeout: Timeout = 'long') -> None:
+    def delete(self, *, timeout: Timeout = 'short') -> None:
         """Delete the Actor version.
 
         https://docs.apify.com/api/v2#/reference/actors/version-object/delete-version
@@ -165,7 +165,7 @@ def __init__(
             **kwargs,
         )
 
-    async def get(self, *, timeout: Timeout = 'long') -> Version | None:
+    async def get(self, *, timeout: Timeout = 'short') -> Version | None:
         """Return information about the Actor version.
 
         https://docs.apify.com/api/v2#/reference/actors/version-object/get-version
@@ -192,7 +192,7 @@ async def update(
         git_repo_url: str | None = None,
         tarball_url: str | None = None,
         github_gist_url: str | None = None,
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'short',
     ) -> Version:
         """Update the Actor version with specified fields.
 
@@ -231,7 +231,7 @@ async def update(
         result = await self._update(timeout=timeout, **version_fields.model_dump(by_alias=True, exclude_none=True))
         return VersionResponse.model_validate(result).data
 
-    async def delete(self, *, timeout: Timeout = 'long') -> None:
+    async def delete(self, *, timeout: Timeout = 'short') -> None:
         """Delete the Actor version.
 
         https://docs.apify.com/api/v2#/reference/actors/version-object/delete-version

src/apify_client/_resource_clients/actor_version_collection.py

@@ -44,7 +44,7 @@ def __init__(
             **kwargs,
         )
 
-    def list(self, *, timeout: Timeout = 'long') -> ListOfVersions:
+    def list(self, *, timeout: Timeout = 'short') -> ListOfVersions:
         """List the available Actor versions.
 
         https://docs.apify.com/api/v2#/reference/actors/version-collection/get-list-of-versions
@@ -70,7 +70,7 @@ def create(
         git_repo_url: str | None = None,
         tarball_url: str | None = None,
         github_gist_url: str | None = None,
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'short',
     ) -> Version:
         """Create a new Actor version.
 
@@ -131,7 +131,7 @@ def __init__(
             **kwargs,
         )
 
-    async def list(self, *, timeout: Timeout = 'long') -> ListOfVersions:
+    async def list(self, *, timeout: Timeout = 'short') -> ListOfVersions:
         """List the available Actor versions.
 
         https://docs.apify.com/api/v2#/reference/actors/version-collection/get-list-of-versions
@@ -157,7 +157,7 @@ async def create(
         git_repo_url: str | None = None,
         tarball_url: str | None = None,
         github_gist_url: str | None = None,
-        timeout: Timeout = 'long',
+        timeout: Timeout = 'short',
     ) -> Version:
         """Create a new Actor version.