Skip to content

refactor: AWS Bedrock llm events refactor#3759

Merged
amychisholm03 merged 1 commit intonewrelic:mainfrom
amychisholm03:aws-llm-event-refactor
Feb 18, 2026
Merged

refactor: AWS Bedrock llm events refactor#3759
amychisholm03 merged 1 commit intonewrelic:mainfrom
amychisholm03:aws-llm-event-refactor

Conversation

@amychisholm03
Copy link
Contributor

Description

Refactors AWS Bedrock LLM events and related helper classes to use the new abstraction.

How to Test

npm run unit
npm run versioned:major aws-sdk-v3

Related Issues

Part of #3687

@codecov
Copy link

codecov bot commented Feb 17, 2026

Codecov Report

❌ Patch coverage is 99.22481% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 89.76%. Comparing base (f1e3a0e) to head (bc5e217).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
lib/llm-events/aws-bedrock/embedding.js 96.77% 1 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (f1e3a0e) and HEAD (bc5e217). Click for more details.

HEAD has 9 uploads less than BASE
Flag BASE (f1e3a0e) HEAD (bc5e217)
integration-tests-esm-20.x 1 0
versioned-tests-22.x 1 0
versioned-tests-20.x 1 0
unit-tests-24.x 1 0
unit-tests-22.x 1 0
unit-tests-20.x 1 0
integration-tests-cjs-24.x 1 0
integration-tests-cjs-22.x 1 0
integration-tests-esm-24.x 1 0
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3759      +/-   ##
==========================================
- Coverage   97.89%   89.76%   -8.13%     
==========================================
  Files         454      448       -6     
  Lines       58726    58316     -410     
  Branches        1        1              
==========================================
- Hits        57489    52348    -5141     
- Misses       1237     5968    +4731     
Flag Coverage Δ
integration-tests-cjs-20.x 74.36% <55.03%> (+0.05%) ⬆️
integration-tests-cjs-22.x ?
integration-tests-cjs-24.x ?
integration-tests-esm-20.x ?
integration-tests-esm-22.x 53.08% <55.03%> (+<0.01%) ⬆️
integration-tests-esm-24.x ?
unit-tests-20.x ?
unit-tests-22.x ?
unit-tests-24.x ?
versioned-tests-20.x ?
versioned-tests-22.x ?
versioned-tests-24.x 81.25% <99.22%> (-0.38%) ⬇️

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 17, 2026 22:07
@amychisholm03 amychisholm03 merged commit 7ff953d into newrelic:main Feb 18, 2026
39 of 42 checks passed
@github-project-automation github-project-automation bot moved this from Needs PR Review to Done: Issues recently completed in Node.js Engineering Board Feb 18, 2026
@amychisholm03 amychisholm03 deleted the aws-llm-event-refactor branch February 18, 2026 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done: Issues recently completed

Development

Successfully merging this pull request may close these issues.

2 participants

Comments