Skip to content

Conversation

@maxday
Copy link
Member

@maxday maxday commented Nov 16, 2025

Description

Bump @aws/lambda-invoke-store to 0.2.0.
Fix breaking changes and update tests

Testing

yarn g:vitest run packages/middleware-recursion-detection/src/recursionDetectionMiddleware.spec.ts

 RUN  v3.2.4 /workplace/maxday/aws-sdk-js-v3

 ✓ packages/middleware-recursion-detection/src/recursionDetectionMiddleware.spec.ts (8 tests) 7ms
   ✓ recursionDetectionMiddleware > sets X-Amzn-Trace-Id header when function name and > trace id environmental variables is set 2ms
   ✓ recursionDetectionMiddleware > sets X-Amzn-Trace-Id header when function name and > trace id value is set in InvokeStore 1ms
   ✓ recursionDetectionMiddleware > sets X-Amzn-Trace-Id header when function name and > favors trace id value from InvokeStore over that from env variable 0ms
   ✓ recursionDetectionMiddleware > should NOT set X-Amzn-Trace-Id header when function name environmental variable is NOT set 0ms
   ✓ recursionDetectionMiddleware > should NOT set X-Amzn-Trace-Id header when the header is already set 0ms
   ✓ recursionDetectionMiddleware > should NOT set X-Amzn-Trace-Id header when the header is already set with some other casing 0ms
   ✓ recursionDetectionMiddleware > should NOT set X-Amzn-Trace-Id header when the header is already set with alternating case 1ms
   ✓ recursionDetectionMiddleware > should NOT set X-Amzn-Trace-Id header when the header is already set with all uppercase 0ms

 Test Files  1 passed (1)
      Tests  8 passed (8)
   Start at  01:17:16
   Duration  426ms (transform 45ms, setup 0ms, collect 55ms, tests 7ms, environment 0ms, prepare 97ms)

Checklist

  • If the PR is a feature, add integration tests (*.integ.spec.ts).
  • If you wrote E2E tests, are they resilient to concurrent I/O?
  • If adding new public functions, did you add the @public tag and enable doc generation on the package?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@maxday maxday marked this pull request as ready for review November 16, 2025 23:06
@maxday maxday requested a review from a team as a code owner November 16, 2025 23:06
@kuhe kuhe merged commit 7454bd8 into aws:main Nov 17, 2025
3 of 4 checks passed
@kuhe kuhe changed the title fix: bump aws-lambda-invoke-store and fix breaking changes chore: upgrade aws-lambda-invoke-store Nov 17, 2025
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