Skip to content

fix: initialize BaseToolset and add missing subagent dependencies - #140

Open
ramooncamacho wants to merge 2 commits into
GoogleCloudPlatform:mainfrom
ramooncamacho:fix/adk-toolset-init-and-missing-deps
Open

fix: initialize BaseToolset and add missing subagent dependencies#140
ramooncamacho wants to merge 2 commits into
GoogleCloudPlatform:mainfrom
ramooncamacho:fix/adk-toolset-init-and-missing-deps

Conversation

@ramooncamacho

Copy link
Copy Markdown

Fixes #139

Summary of Changes

  1. agent_bar_v2/tools/context_based_toolset.py:

    • Added super().__init__() call in ContextBasedToolset to ensure proper BaseToolset initialization from Google ADK.
  2. pyproject.toml:

    • Declared missing direct runtime dependencies required by existing subagents and the Cloud Run session backend:
      • google-api-python-client & google-auth-oauthlib
      • langchain-community & langchain-core
      • python-dotenv
      • pyyaml
      • sqlparse
      • aiosqlite
      • pydantic

Verification

  • Confirmed subagent imports succeed without ModuleNotFoundError.
  • Verified FastAPI session backend initialization using aiosqlite.
  • Verified dynamic tool resolution in ContextBasedToolset.

Ensure base class initialization is executed properly when instantiating
ContextBasedToolset inheriting from ADK's BaseToolset.
Declare direct dependencies required by various subagents and Cloud Run:
- google-api-python-client & google-auth-oauthlib (Drive Agent)
- langchain-community & langchain-core (Knowledge Graph Builder)
- python-dotenv (environment variable loading across subagents)
- pyyaml (Patient Handover and SQL translation)
- sqlparse (Knowledge Graph and Retail subagents)
- aiosqlite (async session database for ADK FastAPI server)
- pydantic (type validation across tools and configurations)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: ContextBasedToolset missing BaseToolset init and undeclared subagent dependencies

1 participant