Skip to content

feat(langchain): Record run_name in on_tool_start#5925

Merged
alexander-alderman-webb merged 39 commits intomasterfrom
webb/langchain/tool-pipeline-name
Apr 22, 2026
Merged

feat(langchain): Record run_name in on_tool_start#5925
alexander-alderman-webb merged 39 commits intomasterfrom
webb/langchain/tool-pipeline-name

Conversation

@alexander-alderman-webb
Copy link
Copy Markdown
Contributor

Description

Issues

Reminders

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 31, 2026

Semver Impact of This PR

🟡 Minor (new features)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

Langchain

  • Record run_name in on_tool_start by alexander-alderman-webb in #5925
  • Record run_name in on_chat_model_start by alexander-alderman-webb in #5924

Other

  • (ci) Cancel in-progress PR workflows on new commit push by joshuarli in #5994

Bug Fixes 🐛

  • (langchain) Set agent name as gen_ai.agent.name for chat and tool spans by alexander-alderman-webb in #5877

🤖 This preview updates automatically when you update the PR.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 31, 2026

Codecov Results 📊

2115 passed | ⏭️ 153 skipped | Total: 2268 | Pass Rate: 93.25% | Execution Time: 4m 10s

All tests are passing successfully.

❌ Patch coverage is 0.00%. Project has 12506 uncovered lines.

Files with missing lines (1)
File Patch % Lines
langchain.py 3.20% ⚠️ 574 Missing

Generated by Codecov Action

@alexander-alderman-webb alexander-alderman-webb marked this pull request as ready for review March 31, 2026 13:19
@alexander-alderman-webb alexander-alderman-webb requested a review from a team as a code owner March 31, 2026 13:19
Comment thread sentry_sdk/integrations/langchain.py
Comment thread sentry_sdk/integrations/langchain.py
Comment thread tests/integrations/langchain/test_langchain.py Outdated
Copy link
Copy Markdown
Member

@ericapisani ericapisani left a comment

Choose a reason for hiding this comment

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

I'm not going to block on the test name, but generally speaking we should try and make these more descriptive

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Comment thread sentry_sdk/integrations/langchain.py
Base automatically changed from webb/langchain/pipeline-name to master April 22, 2026 08:09
@alexander-alderman-webb alexander-alderman-webb merged commit 6abe578 into master Apr 22, 2026
202 of 212 checks passed
@alexander-alderman-webb alexander-alderman-webb deleted the webb/langchain/tool-pipeline-name branch April 22, 2026 08:51
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.

2 participants