fix(agent_runtime): use effective config for workspace name resolution

PR #102 Copilot review surfaced two issues in the new workspace_name
resolution path:

1. workspace_name resolution only used the method-level `config`, ignoring
   `self.config`. When a caller built `AgentRuntimeClient(config=...)` and
   then called `create(input)` without a method-level config, the resolver
   fell back to env credentials/region while the subsequent OpenAPI call
   used `self.config` — possibly resolving a workspace under the wrong
   account/region. Now passes `Config.with_configs(self.config, config)`
   to the resolver, matching the merge done in `ControlAPI._get_client`.

2. `if input.workspace_name:` silently skipped resolution for empty
   strings instead of surfacing the resolver's existing
   `ValueError("workspace_name must be non-empty")`. Switched the
   singular checks to `is not None` so empty strings now raise.

Applied to create / create_async / list / list_async in client.py and
mirrored into the codegen template. Plural `workspace_names` keeps the
truthy check since empty-string plural is a benign "no filter".

Added 8 unit tests covering effective-config propagation (client.config
used when method config=None; method config overrides client.config) and
empty-string ValueError across all four entry points. agent_runtime
suite 262 passed / 2 skipped; `_workspace.py` 95.24% line/branch.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Signed-off-by: Sodawyx <sodawyx@126.com>

Author: Sodawyx

agentrun/agent_runtime/__client_async_template.py

@@ -88,9 +88,10 @@ async def create_async(
                 "workspace_id and workspace_name are mutually exclusive; please"
                 " only set one of them."
             )
-        if input.workspace_name:
+        if input.workspace_name is not None:
+            cfg = Config.with_configs(self.config, config)
             input.workspace_id = await resolve_workspace_id_by_name_async(
-                input.workspace_name, config
+                input.workspace_name, cfg
             )
             input.workspace_name = None
 
@@ -241,18 +242,22 @@ async def list_async(
                     "workspace_ids and workspace_names are mutually exclusive;"
                     " please only set one of them."
                 )
-            if input.workspace_name:
-                input.workspace_id = await resolve_workspace_id_by_name_async(
-                    input.workspace_name, config
-                )
-                input.workspace_name = None
-            if input.workspace_names:
-                input.workspace_ids = (
-                    await resolve_workspace_ids_by_names_async(
-                        input.workspace_names, config
+            if input.workspace_name is not None or input.workspace_names:
+                cfg = Config.with_configs(self.config, config)
+                if input.workspace_name is not None:
+                    input.workspace_id = (
+                        await resolve_workspace_id_by_name_async(
+                            input.workspace_name, cfg
+                        )
                     )
-                )
-                input.workspace_names = None
+                    input.workspace_name = None
+                if input.workspace_names:
+                    input.workspace_ids = (
+                        await resolve_workspace_ids_by_names_async(
+                            input.workspace_names, cfg
+                        )
+                    )
+                    input.workspace_names = None
 
             results = await self.__control_api.list_agent_runtimes_async(
                 ListAgentRuntimesRequest().from_map(input.model_dump()),

agentrun/agent_runtime/client.py

@@ -98,9 +98,10 @@ async def create_async(
                 "workspace_id and workspace_name are mutually exclusive; please"
                 " only set one of them."
             )
-        if input.workspace_name:
+        if input.workspace_name is not None:
+            cfg = Config.with_configs(self.config, config)
             input.workspace_id = await resolve_workspace_id_by_name_async(
-                input.workspace_name, config
+                input.workspace_name, cfg
             )
             input.workspace_name = None
 
@@ -156,9 +157,10 @@ def create(
                 "workspace_id and workspace_name are mutually exclusive; please"
                 " only set one of them."
             )
-        if input.workspace_name:
+        if input.workspace_name is not None:
+            cfg = Config.with_configs(self.config, config)
             input.workspace_id = resolve_workspace_id_by_name(
-                input.workspace_name, config
+                input.workspace_name, cfg
             )
             input.workspace_name = None
 
@@ -390,18 +392,22 @@ async def list_async(
                     "workspace_ids and workspace_names are mutually exclusive;"
                     " please only set one of them."
                 )
-            if input.workspace_name:
-                input.workspace_id = await resolve_workspace_id_by_name_async(
-                    input.workspace_name, config
-                )
-                input.workspace_name = None
-            if input.workspace_names:
-                input.workspace_ids = (
-                    await resolve_workspace_ids_by_names_async(
-                        input.workspace_names, config
+            if input.workspace_name is not None or input.workspace_names:
+                cfg = Config.with_configs(self.config, config)
+                if input.workspace_name is not None:
+                    input.workspace_id = (
+                        await resolve_workspace_id_by_name_async(
+                            input.workspace_name, cfg
+                        )
                     )
-                )
-                input.workspace_names = None
+                    input.workspace_name = None
+                if input.workspace_names:
+                    input.workspace_ids = (
+                        await resolve_workspace_ids_by_names_async(
+                            input.workspace_names, cfg
+                        )
+                    )
+                    input.workspace_names = None
 
             results = await self.__control_api.list_agent_runtimes_async(
                 ListAgentRuntimesRequest().from_map(input.model_dump()),
@@ -450,16 +456,18 @@ def list(
                     "workspace_ids and workspace_names are mutually exclusive;"
                     " please only set one of them."
                 )
-            if input.workspace_name:
-                input.workspace_id = resolve_workspace_id_by_name(
-                    input.workspace_name, config
-                )
-                input.workspace_name = None
-            if input.workspace_names:
-                input.workspace_ids = resolve_workspace_ids_by_names(
-                    input.workspace_names, config
-                )
-                input.workspace_names = None
+            if input.workspace_name is not None or input.workspace_names:
+                cfg = Config.with_configs(self.config, config)
+                if input.workspace_name is not None:
+                    input.workspace_id = resolve_workspace_id_by_name(
+                        input.workspace_name, cfg
+                    )
+                    input.workspace_name = None
+                if input.workspace_names:
+                    input.workspace_ids = resolve_workspace_ids_by_names(
+                        input.workspace_names, cfg
+                    )
+                    input.workspace_names = None
 
             results = self.__control_api.list_agent_runtimes(
                 ListAgentRuntimesRequest().from_map(input.model_dump()),