Skip to content

LangChain Tool SubComponent Gating#1791

Open
TimPansino wants to merge 3 commits into
mainfrom
fix-langchain-tool-subcomponents
Open

LangChain Tool SubComponent Gating#1791
TimPansino wants to merge 3 commits into
mainfrom
fix-langchain-tool-subcomponents

Conversation

@TimPansino

@TimPansino TimPansino commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Overview

  • Remove subcomponent attributes from LangChain tools when a known remote tools is found.
    • This prevents duplicate tool entities from being generated in the UI when the remote server is also instrumented.
  • Add tests against a real MCP server to LangChain to verify remote tool behavior.

@TimPansino TimPansino requested a review from a team as a code owner July 8, 2026 20:43
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

MegaLinter analysis: Success

Descriptor Linter Files Fixed Errors Warnings Elapsed time
✅ ACTION actionlint 9 0 0 1.17s
✅ MARKDOWN markdownlint 7 0 0 0 1.5s
✅ PYTHON ruff 1065 0 0 0 1.19s
✅ PYTHON ruff-format 1065 0 0 0 0.37s
✅ YAML prettier 23 0 0 0 1.71s
✅ YAML v8r 23 0 0 6.9s
✅ YAML yamllint 23 0 0 0.8s

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

@TimPansino TimPansino changed the title LangChain Tool SubComponent Gating [DO NOT MERGE] LangChain Tool SubComponent Gating Jul 8, 2026
@mergify mergify Bot added the tests-failing Tests failing in CI. label Jul 8, 2026
@codecov-commenter

codecov-commenter commented Jul 8, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 82.35294% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.99%. Comparing base (d9a1079) to head (651931a).

Files with missing lines Patch % Lines
newrelic/hooks/mlmodel_langchain.py 82.35% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1791      +/-   ##
==========================================
- Coverage   82.06%   81.99%   -0.07%     
==========================================
  Files         216      216              
  Lines       26786    26799      +13     
  Branches     4238     4242       +4     
==========================================
- Hits        21981    21973       -8     
- Misses       3339     3356      +17     
- Partials     1466     1470       +4     

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

@TimPansino TimPansino force-pushed the fix-langchain-tool-subcomponents branch from 9643678 to 2055367 Compare July 8, 2026 23:38
@TimPansino TimPansino changed the title [DO NOT MERGE] LangChain Tool SubComponent Gating LangChain Tool SubComponent Gating Jul 8, 2026
@mergify mergify Bot removed the tests-failing Tests failing in CI. label Jul 9, 2026
@TimPansino TimPansino requested a review from umaannamalai July 9, 2026 21:52
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