Skip to content

refactor: LangGraph LlmAgent event, remove old LlmEvent base#3761

Open
amychisholm03 wants to merge 1 commit intonewrelic:mainfrom
amychisholm03:final-llm-refactor
Open

refactor: LangGraph LlmAgent event, remove old LlmEvent base#3761
amychisholm03 wants to merge 1 commit intonewrelic:mainfrom
amychisholm03:final-llm-refactor

Conversation

@amychisholm03
Copy link
Contributor

Description

Refactors the LangGraph LlmAgent event to use the new LlmEvent base (event-base.js) and adds missing assertions to LangGraph tests.

Removes old event base (event.js) and sets intended private function in LlmErrorMessage to actually be private.

How to Test

npm run unit
npm run versioned langgraph

Related Issues

Closes #3687

@codecov
Copy link

codecov bot commented Feb 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.76%. Comparing base (8503d25) to head (be5c3bc).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3761      +/-   ##
==========================================
- Coverage   97.89%   97.76%   -0.13%     
==========================================
  Files         454      452       -2     
  Lines       58749    58490     -259     
  Branches        1        1              
==========================================
- Hits        57512    57183     -329     
- Misses       1237     1307      +70     
Flag Coverage Δ
integration-tests-cjs-20.x 74.39% <72.22%> (+0.08%) ⬆️
integration-tests-cjs-22.x 74.42% <72.22%> (+0.08%) ⬆️
integration-tests-cjs-24.x 75.19% <72.22%> (+0.04%) ⬆️
integration-tests-esm-20.x 53.01% <72.22%> (-0.01%) ⬇️
integration-tests-esm-22.x 53.06% <72.22%> (-0.01%) ⬇️
integration-tests-esm-24.x 54.19% <72.22%> (-0.05%) ⬇️
unit-tests-20.x 89.44% <88.88%> (-0.05%) ⬇️
unit-tests-22.x 89.46% <88.88%> (-0.05%) ⬇️
unit-tests-24.x 89.46% <88.88%> (-0.05%) ⬇️
versioned-tests-20.x 81.33% <100.00%> (-0.36%) ⬇️
versioned-tests-22.x 81.33% <100.00%> (-0.36%) ⬇️
versioned-tests-24.x 81.24% <100.00%> (-0.37%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 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.

@amychisholm03 amychisholm03 marked this pull request as ready for review February 18, 2026 17:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Needs PR Review

Development

Successfully merging this pull request may close these issues.

Refactor LlmEvent classes to provide better abstraction/reusability

1 participant

Comments