feat: Add ManagedGraphResult, GraphMetricSummary, and AgentGraphRunnerResult types#151
Merged
jsonbailey merged 4 commits intomainfrom May 1, 2026
Merged
Conversation
37bad31 to
b708885
Compare
d847d83 to
9c9e802
Compare
b708885 to
c553fbd
Compare
9c9e802 to
5fb879e
Compare
c553fbd to
5df809b
Compare
5fb879e to
4319954
Compare
5df809b to
c6e35a4
Compare
4319954 to
ba64a62
Compare
c6e35a4 to
08dfcb7
Compare
ba64a62 to
49e95a3
Compare
08dfcb7 to
ca37e74
Compare
49e95a3 to
80d4e11
Compare
ca37e74 to
9c0003c
Compare
80d4e11 to
05e1f4c
Compare
069c0ee to
1c0255f
Compare
jsonbailey
commented
Apr 29, 2026
| content=result.output, | ||
| metrics=summary, | ||
| raw=result.raw, | ||
| evaluations=None, |
Contributor
Author
There was a problem hiding this comment.
This will be added in a follow up PR #153
1c0255f to
0355872
Compare
05e1f4c to
0a1ee15
Compare
0355872 to
796bda5
Compare
0a1ee15 to
72f5a37
Compare
796bda5 to
a2db8cb
Compare
72f5a37 to
8e60f79
Compare
34598f5 to
36cd239
Compare
8e60f79 to
1b47300
Compare
Contributor
Author
|
@cursor review |
1b47300 to
3928030
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 1b47300. Configure here.
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 3928030. Configure here.
keelerm84
approved these changes
May 1, 2026
7aae3e2 to
1e61d42
Compare
…rResult types - Add GraphMetrics dataclass (runner-layer return type for graph runs) - Add GraphMetricSummary dataclass (managed-layer metrics, analogous to LDAIMetricSummary for single-model invocations) - Add ManagedGraphResult dataclass (managed-layer return type from ManagedAgentGraph) - Add AgentGraphRunnerResult dataclass (future runner return type, no evaluations field) - ManagedAgentGraph.run() now returns ManagedGraphResult with GraphMetricSummary built from the runner's AgentGraphResult metrics - Export all new types from ldai package Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ent_graph Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
f9e947b to
75172d8
Compare
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Summary
Adds the managed-layer return type and metric summary for agent graph execution, mirroring the per-config types introduced in PR 8.
GraphMetrics— runner-layer metrics (success, path, duration_ms, usage, node_metrics).GraphMetricSummary— managed-layer summary, analogous toLDAIMetricSummary(adds resumption_token).ManagedGraphResult— return type ofManagedAgentGraph.run()(content, metrics, raw, evaluations task).AgentGraphRunnerResult— pure runner-layer return type (noevaluationsfield).ManagedAgentGraph.run()now returnsManagedGraphResultand builds aGraphMetricSummaryfrom the underlying runner'sLDAIMetrics(path/node_metrics will be populated once graph runners migrate in PR 11).Stack
PR 7 → PR 8 → PR 8-openai → PR 8-langchain → PR 9 (this) → PR 10 → PR 11 → PR 11-openai → PR 11-langchain → PR 12
Test plan
make test— 261 tests pass (server-ai 140, langchain 81, openai 40)make lint— mypy clean🤖 Generated with Claude Code
Note
Medium Risk
Introduces new public result/metrics types and changes
ManagedAgentGraph.run()to returnManagedGraphResult(breaking callers expectingAgentGraphResult.output). Risk is mainly API/typing compatibility and downstream integrations relying on the old return shape.Overview
Adds new agent-graph result/metrics dataclasses (
GraphMetrics,GraphMetricSummary,ManagedGraphResult,AgentGraphRunnerResult) and exports them fromldaiandldai.providers.Updates
ManagedAgentGraph.run()to return a managed-layerManagedGraphResult(withcontent+GraphMetricSummary) by wrapping the underlying runner’sAgentGraphResult, and adjusts tests to assert the new return type/field names.Reviewed by Cursor Bugbot for commit af60d27. Bugbot is set up for automated code reviews on this repo. Configure here.