Skip to content

feat: Add LangGraph integration#4727

Merged
antonpirker merged 41 commits intomasterfrom
shellmayr/feat/langgraph-integration
Sep 4, 2025
Merged

feat: Add LangGraph integration#4727
antonpirker merged 41 commits intomasterfrom
shellmayr/feat/langgraph-integration

Conversation

@shellmayr
Copy link
Copy Markdown
Member

@shellmayr shellmayr commented Aug 29, 2025 •
edited by dingsdax
Loading

  • Add LangGraph integration
  • Compilation of StateGraphs results in an agent creation span (according to OTEL semantic conventions)
  • Runtime executions are done on Pregel instances - we are wrapping their invoke & ainvoke which produces the invoke_agent spans
  • There's some internals that automatically switch between invoke & stream on CompiledStateGraph (which is a subclass of Pregel), which results in duplicate spans if both are instrumented. For now, only invoke is wrapped to prevent this duplication.
  • Agent handoffs in LangGraph are done via tools - so there is no real possibility to create handoff spans within the SDK. Looks like this will be handled in product logic instead.

Closes TET-991
Closes PY-1799

@linear
Copy link
Copy Markdown

linear Bot commented Aug 29, 2025

TET-991 Add integration for langgraph

@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 29, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 76.75676% with 43 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.75%. Comparing base (c378c2d) to head (f903562).
⚠️ Report is 1 commits behind head on master.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
sentry_sdk/integrations/langgraph.py 76.63% 18 Missing and 25 partials ⚠️
Additional details and impacted files 
@@             Coverage Diff             @@
##           master    #4727       +/-   ##
===========================================
+ Coverage   73.15%   84.75%   +11.60%     
===========================================
  Files         157      158        +1     
  Lines       16138    16323      +185     
  Branches     2744     2792       +48     
===========================================
+ Hits        11805    13835     +2030     
+ Misses       3640     1672     -1968     
- Partials      693      816      +123
Files with missing lines Coverage Δ
sentry_sdk/consts.py 99.33% <100.00%> (+<0.01%) ⬆️
sentry_sdk/integrations/__init__.py 87.01% <ø> (ø)
sentry_sdk/integrations/langgraph.py 76.63% <76.63%> (ø)

... and 40 files with indirect coverage changes

@linear
Copy link
Copy Markdown

linear Bot commented Sep 1, 2025

BE-532 Langgraph integration to sentry

PY-1799 Add integration for `langgraph`

@shellmayr shellmayr force-pushed the shellmayr/feat/langgraph-integration branch from b070632 to b5c6e37 Compare September 3, 2025 13:43
sentrivana
sentrivana reviewed Sep 3, 2025
View reviewed changes
Comment thread tox.ini Outdated
sentrivana
sentrivana reviewed Sep 3, 2025
View reviewed changes
Comment thread scripts/populate_tox/tox.jinja Outdated
sentrivana
sentrivana reviewed Sep 3, 2025
View reviewed changes
Comment thread scripts/split_tox_gh_actions/split_tox_gh_actions.py Outdated
cursor[bot]

This comment was marked as outdated.

Sign in to view

cursor[bot]

This comment was marked as outdated.

Sign in to view

cursor[bot]

This comment was marked as outdated.

Sign in to view

cursor[bot]

This comment was marked as outdated.

Sign in to view

cursor[bot]
cursor Bot reviewed Sep 4, 2025
View reviewed changes
Comment thread sentry_sdk/integrations/langgraph.py
cursor[bot]
cursor Bot reviewed Sep 4, 2025
View reviewed changes
Comment thread sentry_sdk/integrations/langgraph.py
cursor[bot]
cursor Bot reviewed Sep 4, 2025
View reviewed changes
Comment thread sentry_sdk/integrations/langgraph.py
@antonpirker antonpirker changed the title feat: add langgraph integration feat: Add langgraph integration Sep 4, 2025
@antonpirker antonpirker changed the title feat: Add langgraph integration feat: Add LangGraph integration Sep 4, 2025
cursor[bot]
cursor Bot reviewed Sep 4, 2025
View reviewed changes
Comment thread sentry_sdk/integrations/__init__.py
antonpirker
antonpirker approved these changes Sep 4, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@antonpirker antonpirker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now its ready to be merge. Thanks for the great work @shellmayr !

@antonpirker antonpirker merged commit 58a9827 into master Sep 4, 2025
134 checks passed
@antonpirker antonpirker deleted the shellmayr/feat/langgraph-integration branch September 4, 2025 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sentrivana sentrivana sentrivana left review comments

@seer-by-sentry seer-by-sentry[bot] seer-by-sentry[bot] left review comments

@cursor cursor[bot] cursor[bot] left review comments

+1 more reviewer

@antonpirker antonpirker antonpirker approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@shellmayr @antonpirker @sentrivana