google-adk: instrument `Workflow` nodes and workflow tool calls
Maintainer antworten meist innerhalb von 1 Tag
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 55/100
- Issue-Typ
- Feature
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- python
- Bereich
- observability
Rechercherichtung
Start in py/src/braintrust/integrations/adk/ and read the ADK integration plus test_adk.py; the issue identifies BaseNode.run and _ToolNode._run_impl as candidate patch targets. Add cassette-backed coverage for workflow spans and workflow tool-call spans, checking compatibility with ADK 1.14.1 as well as 2.10 and 2.11. Done when the tests confirm both spans are recorded and the existing version still works.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
ADK 2.x Workflow is a BaseNode, not a BaseAgent, and _ToolNode calls self.tool.run_async directly, bypassing flows/llm_flows/tools/_caller._call_tool_async. As a result:
- Workflow tool calls get no tool span (except MCP tools, via
McpToolPatcher). - Workflows get no span between the Runner and agent spans.
ADK now marks SequentialAgent / ParallelAgent as deprecated in favor of Workflow, so this gap will grow. 2.11.0 also made workflow tool nodes pause for confirmation and run plugin callbacks (_ToolNode._run_tool_with_plugin_callbacks). This predates the 2.11.0 bump.
Proposal
- Candidate patch targets in
py/src/braintrust/integrations/adk/:google.adk.workflow._base_node.BaseNode.run(workflow/node spans) andgoogle.adk.workflow._tool_node._ToolNode._run_impl(tool spans). Both exist in 2.10 and 2.11; gate them so 1.14.1 keeps working. test_adk.pyhas noWorkflowcoverage; add cassette-backed tests.- Related, lower value:
ModelConsultTool's advisor callsBaseLlm.generate_content_asyncdirectly, so it gets no ADK LLM span (the google-genai integration still traces the underlying call).
Parent: #858 (found while reviewing the 2026-10-05 weekly dependency update, #849).
🤖 Generated with Claude Code
- Vorherrschende Sprache
- Python
- Sterne
- 22
- Forks
- 24
- Ø Merge
- 16 Std. 34 Min.
- Gemergte PRs (30 T.)
- 113
Entwicklungsumgebung
- Enthält ein Dockerfile oder eine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus braintrustdata/braintrust-sdk-python
-
feature python
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
braintrustdata/braintrust-sdk-python#868 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Migrate legacy HTTPConnection callers to the policy-aware Transport incrementallyEvtl. vergeben @AbhiPrasad hat das vor 13 Tagen übernommen. Offenpython
braintrustdata/braintrust-sdk-python#839 · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
-
new-integration
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 68/100
braintrustdata/braintrust-sdk-python#808 ·
Maintainer antworten meist innerhalb von 1 Tag
-
new-integration
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 45/100
braintrustdata/braintrust-sdk-python#806 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
braintrustdata/braintrust-sdk-python#798 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in braintrustdata/braintrust-sdk-python
Ähnliche Issues
-
approved Good First Issue scope: docs
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 85/100
hiero-ledger/hiero-sdk-python#2730 · 3 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
wemake-services/wemake-django-template#3178 · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
-
md/asciidoc/csv: an explicit `encoding="utf-8"` keeps a leading BOM, the auto-detect path drops itOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
docling-project/docling#4846 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
neurostuff/NiMARE#1181 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
pytest-dev/pytest#15180 ·
Maintainer antworten meist innerhalb von 1 Tag