Skip to content

feat: add google-adk node-weather ai-observability fixture - #3711

Open
marco-g-pm wants to merge 2 commits into
mainfrom
marco/aio-google-adk-fixture
Open

feat: add google-adk node-weather ai-observability fixture#3711
marco-g-pm wants to merge 2 commits into
mainfrom
marco/aio-google-adk-fixture

Conversation

@marco-g-pm

Copy link
Copy Markdown
Contributor

Problem

The wizard has no fixture to prove the Google ADK path. PostHog/posthog-js#4687 adds PostHogADKPlugin, and no ai-observability app exercises an ADK Runner plugin.

for PostHog/posthog-js#4687

Changes

  • Adds apps/ai-observability/google-adk/node-weather: the same two-turn get_weather round trip as the other weather apps, on @google/adk, so a diff against any of them isolates the framework.
  • Its README grades the plugin path: PostHogADKPlugin in the Runner's plugins, identity inherited from ADK's own userId and sessionId, two traces in one session, flush before exit.
  • The README also states the plugin emits no tool spans, so a missing $ai_span is not graded as a failure.
  • Mechanical: the apps README lists the fixture and adds it to the wrapper-first exceptions.

Testing

🤖 Agent context

Written by Claude Code from PostHog/posthog-js#4687, mirroring #3675 (the Go fixture rollout). Skills invoked: writing-pr-descriptions, writing-user-facing-copy, writing-code-comments.

marco-g-pm and others added 2 commits September 1, 2026 10:31
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Same demo values as the sibling ai-observability fixtures. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@marco-g-pm marco-g-pm self-assigned this Sep 1, 2026
@marco-g-pm
marco-g-pm marked this pull request as ready for review September 1, 2026 17:56
@marco-g-pm
marco-g-pm requested a review from a team as a code owner September 1, 2026 17:56
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