Skip to content

Break circular import using TYPE_CHECKING and local imports #47

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
amrit110 wants to merge 2 commits into from
Closed

Break circular import using TYPE_CHECKING and local imports #47

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
21afbb8
Fix circular imports
amrit110 Feb 12, 2026
15c852b
[pre-commit.ci] Add auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Feb 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 17 additions & 5 deletions aieng-eval-agents/aieng/agent_evals/async_client_manager.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,15 @@
"""

import logging
from typing import TYPE_CHECKING

from aieng.agent_evals.configs import Configs
from aieng.agent_evals.tools import ReadOnlySqlDatabase
from langfuse import Langfuse
from openai import AsyncOpenAI
from langfuse.openai import AsyncOpenAI


if TYPE_CHECKING:
from aieng.agent_evals.tools import ReadOnlySqlDatabase


logging.basicConfig(level=logging.INFO, format="%(asctime)s %(levelname)s %(name)s: %(message)s")
Expand Down Expand Up @@ -94,11 +98,15 @@ def openai_client(self) -> AsyncOpenAI:
if self._openai_client is None:
api_key = self.configs.openai_api_key.get_secret_value()

self._openai_client = AsyncOpenAI(api_key=api_key, base_url=self.configs.openai_base_url)
self._openai_client = AsyncOpenAI(
api_key=api_key,
base_url=self.configs.openai_base_url,
max_retries=0, # Using custom retry logic (tenacity) elsewhere
)
self._initialized = True
return self._openai_client

def report_generation_db(self, agent_name: str = "ReportGenerationAgent") -> ReadOnlySqlDatabase:
def report_generation_db(self, agent_name: str = "ReportGenerationAgent") -> "ReadOnlySqlDatabase":
"""Get or create Report Generation database connection.

Returns
Expand All @@ -107,6 +115,8 @@ def report_generation_db(self, agent_name: str = "ReportGenerationAgent") -> Rea
The Report Generation database connection instance.
"""
if self._report_generation_db is None:
from aieng.agent_evals.tools import ReadOnlySqlDatabase

if self.configs.report_generation_db is None:
raise ValueError("Report Generation database configuration is missing.")

Expand All @@ -118,7 +128,7 @@ def report_generation_db(self, agent_name: str = "ReportGenerationAgent") -> Rea

return self._report_generation_db

def aml_db(self, agent_name: str = "FraudInvestigationAnalyst") -> ReadOnlySqlDatabase:
def aml_db(self, agent_name: str = "FraudInvestigationAnalyst") -> "ReadOnlySqlDatabase":
"""Get or create AML database connection.

Returns
Expand All @@ -127,6 +137,8 @@ def aml_db(self, agent_name: str = "FraudInvestigationAnalyst") -> ReadOnlySqlDa
The Report Generation database connection instance.
"""
if self._aml_db is None:
from aieng.agent_evals.tools import ReadOnlySqlDatabase

if self.configs.aml_db is None:
raise ValueError("AML database configuration is missing.")

Expand Down
Loading