feat(aws-sdk)!: drop support for AWS SDK for JavaScript v2

[trivikr]

Which problem is this PR solving?

• Fixes [instrumentation-aws-sdk] Drop support for JS SDK v2 #2607

Short description of the changes

• Drops support for AWS SDK for JavaScript v2

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.31%. Comparing base (fcca38a) to head (18b31e6).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2623      +/-   ##
==========================================
- Coverage   90.95%   89.31%   -1.65%     
==========================================
  Files         172      172              
  Lines        8137     8046      -91     
  Branches     1649     1625      -24     
==========================================
- Hits         7401     7186     -215     
- Misses        736      860     +124     

Files with missing lines	Coverage Δ
...entelemetry-instrumentation-aws-sdk/src/aws-sdk.ts	93.12% <ø> (-1.94%)	⬇️
...opentelemetry-instrumentation-aws-sdk/src/utils.ts	93.10% <ø> (-4.20%)	⬇️

... and 5 files with indirect coverage changes

[pichlermarc]

@trivikr I changed the title to include a ! to mark this PR as a breaking change so it shows up as such in the changelog.

Please update the branch if you find the time to see if the build failures have been resolved. I think the TAV failures are resolved by #2628 and #2643 🙂

[trivikr]

Thank you @pichlermarc. I've rebased the changes.

---

The CI is failing though for the path which isn't updated

  1) MessageAttributes
       injectPropagationContext
         should inject context if there are available attributes:
     Error: expect(received).toBeGreaterThan(expected)

Test code which fails:

opentelemetry-js-contrib/plugins/node/opentelemetry-instrumentation-aws-sdk/test/MessageAttributes.test.ts

Line 84 in 6b4e532

expect(Object.keys(contextAttributes).length).toBeGreaterThan(5);

Execution https://github.com/open-telemetry/opentelemetry-js-contrib/actions/runs/12792793464/job/35664659517?pr=2623

I confirmed when debugging that propagation.inject is called but it has not impact on the contents of attributes
Source code

opentelemetry-js-contrib/plugins/node/opentelemetry-instrumentation-aws-sdk/src/services/MessageAttributes.ts

Line 81 in 6b4e532

propagation.inject(context.active(), attributes, contextSetter);

@blumamir @jj22ee Do you know what may be going wrong?

[pichlermarc]

This looks like it tested v3 - did you mean to delete these tests? 🤔

[pichlermarc]

@trivikr oh, looks like the lint step is failing.
Running npm run lint:fix should take care of it, seems to be just some replacement from " to ' . 🙂