From 65f25d1568fedb392a7e85054376fed6be1b9342 Mon Sep 17 00:00:00 2001 From: Quratulain-bilal Date: Tue, 4 Aug 2026 01:04:29 +0500 Subject: [PATCH] fix: use logger instead of print() for warning in _agent_config.py Replace print(..., file=sys.stderr) with logger.warning() for consistent logging behavior. Removes unused sys import. --- src/specify_cli/_agent_config.py | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/src/specify_cli/_agent_config.py b/src/specify_cli/_agent_config.py index 0f82824271..cfc57bfb2d 100644 --- a/src/specify_cli/_agent_config.py +++ b/src/specify_cli/_agent_config.py @@ -1,10 +1,12 @@ """Agent configuration constants derived from the integration registry.""" from __future__ import annotations +import logging import os -import sys from typing import Any +logger = logging.getLogger(__name__) + def _build_agent_config() -> dict[str, dict[str, Any]]: from .integrations import INTEGRATION_REGISTRY @@ -41,11 +43,11 @@ def resolve_default_init_integration() -> str: return DEFAULT_INIT_INTEGRATION if override in AGENT_CONFIG: return override - print( - f"Warning: {DEFAULT_INIT_INTEGRATION_ENV_VAR}='{override}' is not a " - f"recognized integration; falling back to '{DEFAULT_INIT_INTEGRATION}'. " - f"Choose from: {', '.join(sorted(AGENT_CONFIG.keys()))}.", - file=sys.stderr, + logger.warning( + "%s='%s' is not a recognized integration; falling back to '%s'. " + "Choose from: %s.", + DEFAULT_INIT_INTEGRATION_ENV_VAR, override, + DEFAULT_INIT_INTEGRATION, ', '.join(sorted(AGENT_CONFIG.keys())), ) return DEFAULT_INIT_INTEGRATION