Fix reverted port defaults in environment_variables.py and deploy_handlers.py

Restore centralized port constant imports that were reverted by linter.
environment_variables.py: use _DEFAULT_ACP_PORT, _DEFAULT_AGENTEX_BASE_URL,
_DEFAULT_HEALTH_CHECK_PORT, _DEFAULT_TEMPORAL_ADDRESS from ports.py.
deploy_handlers.py: use str(ACP_SERVER_PORT) instead of hardcoded "8000".

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: SWE Destroyer

src/agentex/lib/cli/handlers/deploy_handlers.py

@@ -11,6 +11,7 @@
 from rich.console import Console
 
 from agentex.lib.utils.logging import make_logger
+from agentex.lib.constants.ports import ACP_SERVER_PORT
 from agentex.lib.cli.utils.exceptions import HelmError, DeploymentError
 from agentex.lib.cli.utils.path_utils import PathResolutionError, calculate_docker_acp_module
 from agentex.lib.environment_variables import EnvVarKeys
@@ -239,12 +240,12 @@ def add_acp_command_to_helm_values(helm_values: dict[str, Any], manifest: AgentM
     try:
         docker_acp_module = calculate_docker_acp_module(manifest, manifest_path)
         # Create the uvicorn command with the correct module path
-        helm_values["command"] = ["uvicorn", f"{docker_acp_module}:acp", "--host", "0.0.0.0", "--port", "8000"]
+        helm_values["command"] = ["uvicorn", f"{docker_acp_module}:acp", "--host", "0.0.0.0", "--port", str(ACP_SERVER_PORT)]
         logger.info(f"Using dynamic ACP command: uvicorn {docker_acp_module}:acp")
     except (PathResolutionError, Exception) as e:
         # Fallback to default command structure
         logger.warning(f"Could not calculate dynamic ACP module ({e}), using default: project.acp")
-        helm_values["command"] = ["uvicorn", "project.acp:acp", "--host", "0.0.0.0", "--port", "8000"]
+        helm_values["command"] = ["uvicorn", "project.acp:acp", "--host", "0.0.0.0", "--port", str(ACP_SERVER_PORT)]
 
 
 def merge_deployment_configs(

src/agentex/lib/environment_variables.py

@@ -1,4 +1,3 @@
-
 from __future__ import annotations
 
 import os
@@ -8,6 +7,12 @@
 from dotenv import load_dotenv
 
 from agentex.lib.utils.logging import make_logger
+from agentex.lib.constants.ports import (
+    ACP_SERVER_PORT as _DEFAULT_ACP_PORT,
+    TEMPORAL_ADDRESS as _DEFAULT_TEMPORAL_ADDRESS,
+    HEALTH_CHECK_PORT as _DEFAULT_HEALTH_CHECK_PORT,
+    AGENTEX_API_BASE_URL as _DEFAULT_AGENTEX_BASE_URL,
+)
 from agentex.lib.utils.model_utils import BaseModel
 
 PROJECT_ROOT = Path(__file__).resolve().parents[2]
@@ -56,9 +61,9 @@ class Environment(str, Enum):
 
 class EnvironmentVariables(BaseModel):
     ENVIRONMENT: str = Environment.DEV
-    TEMPORAL_ADDRESS: str | None = "localhost:7233"
+    TEMPORAL_ADDRESS: str | None = _DEFAULT_TEMPORAL_ADDRESS
     REDIS_URL: str | None = None
-    AGENTEX_BASE_URL: str | None = "http://localhost:5003"
+    AGENTEX_BASE_URL: str | None = _DEFAULT_AGENTEX_BASE_URL
     # Agent Identifiers
     AGENT_NAME: str
     AGENT_DESCRIPTION: str | None = None
@@ -68,12 +73,12 @@ class EnvironmentVariables(BaseModel):
     AGENT_INPUT_TYPE: str | None = None
     # ACP Configuration
     ACP_URL: str
-    ACP_PORT: int = 8000
+    ACP_PORT: int = _DEFAULT_ACP_PORT
     # Workflow Configuration
     WORKFLOW_TASK_QUEUE: str | None = None
     WORKFLOW_NAME: str | None = None
     # Temporal Worker Configuration
-    HEALTH_CHECK_PORT: int = 80
+    HEALTH_CHECK_PORT: int = _DEFAULT_HEALTH_CHECK_PORT
     # Auth Configuration
     AUTH_PRINCIPAL_B64: str | None = None
     # Build Information