From f986b352ff48f4e6810dab9b6037672c200b33a0 Mon Sep 17 00:00:00 2001 From: hemasumanth Date: Fri, 17 Apr 2026 22:45:59 -0400 Subject: [PATCH 1/2] Fix region enum mapping - send full proto enum name to API Made-with: Cursor --- gradient_adk/cli/agent/deployment/deploy_service.py | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/gradient_adk/cli/agent/deployment/deploy_service.py b/gradient_adk/cli/agent/deployment/deploy_service.py index 0c5ce19..1266da1 100644 --- a/gradient_adk/cli/agent/deployment/deploy_service.py +++ b/gradient_adk/cli/agent/deployment/deploy_service.py @@ -34,6 +34,13 @@ def _get_adk_version() -> str: return "unknown" +def _region_to_proto_enum(region: str | None) -> str | None: + """Convert a short region string to the proto enum name expected by the API.""" + if region is None: + return None + return f"AGENT_WORKSPACE_DEPLOYMENT_REGION_{region.upper()}" + + class DeployService(Protocol): """Protocol for agent deployment operations.""" @@ -322,7 +329,6 @@ async def _create_or_update_deployment( code_artifact: AgentDeploymentCodeArtifact, project_id: str, description: str | None = None, - region: str | None = None, ) -> str: """Create or update the deployment based on what exists. @@ -348,7 +354,7 @@ async def _create_or_update_deployment( project_id=project_id, library_version=_get_adk_version(), description=description, - region=region, + region=_region_to_proto_enum(region), ) workspace_output = await self.client.create_agent_workspace(workspace_input) @@ -376,7 +382,7 @@ async def _create_or_update_deployment( agent_deployment_code_artifact=code_artifact, library_version=_get_adk_version(), description=description, - region=region, + region=_region_to_proto_enum(region), ) deployment_output = await self.client.create_agent_workspace_deployment( deployment_input From 48b4709f944398682ed7464657e7e6ba9a074845 Mon Sep 17 00:00:00 2001 From: hemasumanth Date: Fri, 17 Apr 2026 22:55:40 -0400 Subject: [PATCH 2/2] Fix missing region param in function signature and handle empty strings Made-with: Cursor --- gradient_adk/cli/agent/deployment/deploy_service.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/gradient_adk/cli/agent/deployment/deploy_service.py b/gradient_adk/cli/agent/deployment/deploy_service.py index 1266da1..b2a555e 100644 --- a/gradient_adk/cli/agent/deployment/deploy_service.py +++ b/gradient_adk/cli/agent/deployment/deploy_service.py @@ -36,9 +36,9 @@ def _get_adk_version() -> str: def _region_to_proto_enum(region: str | None) -> str | None: """Convert a short region string to the proto enum name expected by the API.""" - if region is None: + if not region or not region.strip(): return None - return f"AGENT_WORKSPACE_DEPLOYMENT_REGION_{region.upper()}" + return f"AGENT_WORKSPACE_DEPLOYMENT_REGION_{region.strip().upper()}" class DeployService(Protocol): @@ -329,6 +329,7 @@ async def _create_or_update_deployment( code_artifact: AgentDeploymentCodeArtifact, project_id: str, description: str | None = None, + region: str | None = None, ) -> str: """Create or update the deployment based on what exists.