Skip to content

Add astream_events instrumentation and tests for v1 & v2 in Langchain#1765

Draft
lrafeei wants to merge 17 commits into
mainfrom
test-astream_events_v1_v2
Draft

Add astream_events instrumentation and tests for v1 & v2 in Langchain#1765
lrafeei wants to merge 17 commits into
mainfrom
test-astream_events_v1_v2

Conversation

@lrafeei

@lrafeei lrafeei commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

This PR adds instrumentation and tests for Langchain's astream_events v1 & v2

Separate PR coming for stream_events v3 and astream_events v3

@github-actions

github-actions Bot commented Jun 12, 2026

Copy link
Copy Markdown

MegaLinter analysis: Success

Descriptor Linter Files Fixed Errors Warnings Elapsed time
✅ ACTION actionlint 9 0 0 1.18s
✅ MARKDOWN markdownlint 7 0 0 0 1.2s
✅ PYTHON ruff 1038 0 0 0 1.12s
✅ PYTHON ruff-format 1038 0 0 0 0.48s
✅ YAML prettier 23 0 0 0 1.71s
✅ YAML v8r 23 0 0 5.37s
✅ YAML yamllint 23 0 0 0.79s

Notices

📣 MegaLinter 9.5.0 is out! Discover the new features and security recommendations in the release announcement. (Skip this info by defining SECURITY_SUGGESTIONS: false)

See detailed reports in MegaLinter artifacts

MegaLinter is graciously provided by OX Security
Show us your support by starring ⭐ the repository

@mergify mergify Bot added the tests-failing Tests failing in CI. label Jun 12, 2026
@codecov-commenter

codecov-commenter commented Jun 12, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 77.27273% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.36%. Comparing base (853d8cd) to head (4292a63).

Files with missing lines Patch % Lines
newrelic/hooks/mlmodel_langchain.py 73.68% 4 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1765      +/-   ##
==========================================
- Coverage   82.07%   81.36%   -0.71%     
==========================================
  Files         215      215              
  Lines       26589    26611      +22     
  Branches     4213     4215       +2     
==========================================
- Hits        21823    21653     -170     
- Misses       3315     3519     +204     
+ Partials     1451     1439      -12     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@lrafeei lrafeei force-pushed the test-astream_events_v1_v2 branch 2 times, most recently from 9f71859 to a290052 Compare June 13, 2026 01:31
@lrafeei lrafeei force-pushed the test-astream_events_v1_v2 branch from 1e0de6e to f5f3340 Compare June 16, 2026 23:17
@mergify mergify Bot added the merge-conflicts Merge conflicts detected. label Jun 17, 2026
@mergify mergify Bot removed the merge-conflicts Merge conflicts detected. label Jun 18, 2026
@TimPansino TimPansino force-pushed the test-astream_events_v1_v2 branch from cc7b218 to fb2c9f3 Compare June 23, 2026 19:11
@lrafeei lrafeei force-pushed the test-astream_events_v1_v2 branch from b1b2146 to 1168f7b Compare June 23, 2026 20:29
@TimPansino TimPansino force-pushed the test-astream_events_v1_v2 branch from 57a3fb7 to 4292a63 Compare June 23, 2026 22:08
@mergify mergify Bot removed the tests-failing Tests failing in CI. label Jun 23, 2026
@lrafeei lrafeei marked this pull request as ready for review June 23, 2026 22:26
@lrafeei lrafeei requested a review from a team as a code owner June 23, 2026 22:26
@lrafeei lrafeei marked this pull request as draft June 25, 2026 17:21
@mergify mergify Bot added the merge-conflicts Merge conflicts detected. label Jul 6, 2026
@TimPansino TimPansino self-assigned this Jul 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-conflicts Merge conflicts detected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants