Skip to content

Report Generation: Switching from OpenAI SDK to Google ADK and removing OpenAI dependencies#35

Open
lotif wants to merge 2 commits intomainfrom
marcelo/adk
Open

Report Generation: Switching from OpenAI SDK to Google ADK and removing OpenAI dependencies#35
lotif wants to merge 2 commits intomainfrom
marcelo/adk

Conversation

@lotif
Copy link
Collaborator

@lotif lotif commented Feb 4, 2026

Summary

Switching from OpenAI SDK to Google ADK on Report Generation agent to match the other agents. Removing OpenAI dependencies.

Some extra code was needed to parse the response of the agents which is a bit trickier with Google ADK.

Clickup Ticket(s): NA

Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📝 Documentation update
  • 🔧 Refactoring (no functional changes)
  • ⚡ Performance improvement
  • 🧪 Test improvements
  • 🔒 Security fix

Changes Made

  • Switching from OpenAI SDK to Google ADK on report generation agents
  • Modifying the demo and evaluate code accordingly
  • Added an event parser to parse the response of the google models
  • Removing OpenAI dependencies from pyproject.toml
  • Updating .env.example and the Config class to remove OpenAI env vars

Testing

  • Tests pass locally (uv run pytest tests/)
  • Type checking passes (uv run mypy <src_dir>)
  • Linting passes (uv run ruff check src_dir/)
  • Manual testing performed (describe below)

Manual testing details:
Ran the demo UI and the evaluation script

Checklist

  • Code follows the project's style guidelines
  • Self-review of code completed
  • Documentation updated (if applicable)
  • No sensitive information (API keys, credentials) exposed

@lotif lotif requested review from amrit110 and fcogidi February 4, 2026 20:52
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.

1 participant