Skip to content

feat: Update OpenAI graph runner to return AgentGraphRunnerResult with GraphMetrics#155

Draft
jsonbailey wants to merge 1 commit intojb/aic-2174/graph-tracking-refactorfrom
jb/aic-2174/openai-graph-runner
Draft

feat: Update OpenAI graph runner to return AgentGraphRunnerResult with GraphMetrics#155
jsonbailey wants to merge 1 commit intojb/aic-2174/graph-tracking-refactorfrom
jb/aic-2174/openai-graph-runner

Conversation

@jsonbailey
Copy link
Copy Markdown
Contributor

Summary

  • Removes all direct LaunchDarkly tracker calls from OpenAIAgentGraphRunner
  • Introduces _NodeMetricsAccumulator — a lightweight per-node metrics collector replacing LDAIConfigTracker inside the runner
  • Runner now returns AgentGraphRunnerResult with populated GraphMetrics (path, duration_ms, usage, node_metrics)
  • Graph-level and per-node tracking events are emitted by ManagedAgentGraph._flush_graph_tracking() from the result metrics
  • ManagedAgentGraph._flush_graph_tracking() extended to drive per-node tracking from result.metrics.node_metrics using graph node tracker factories
  • Integration tests updated to exercise the full ManagedAgentGraph.run() pipeline (tracking events now come from the managed layer)
  • Handoff-level track_handoff_success() calls removed (per spec: path field is sufficient; handoffs are not in GraphMetrics)

Depends on

Test plan

  • All existing tests pass (uv run pytest packages/ai-providers/server-ai-openai/tests/)
  • test_openai_agent_graph_runner.py: runner returns new shape, no tracker created
  • test_tracking_openai_agents.py: graph-level and per-node events emitted through managed layer

🤖 Generated with Claude Code

…h GraphMetrics

Remove all direct LaunchDarkly tracker calls from OpenAIAgentGraphRunner. The
runner now collects per-node metrics via _NodeMetricsAccumulator (a lightweight
accumulator replacing the per-node LDAIConfigTracker) and returns
AgentGraphRunnerResult with populated GraphMetrics (path, duration_ms, usage,
node_metrics). Graph-level and per-node tracking events are emitted by
ManagedAgentGraph._flush_graph_tracking() from the result.

ManagedAgentGraph._flush_graph_tracking() is extended to also drive per-node
tracking from result.metrics.node_metrics using the graph definition's node
tracker factories.

Integration tests in test_tracking_openai_agents.py are updated to run through
the full ManagedAgentGraph pipeline (ManagedAgentGraph.run()) so tracking events
are emitted by the managed layer as intended.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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