Skip to content

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

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

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
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.

3 participants