Skip to content

Conversation

@maxday
Copy link
Contributor

@maxday maxday commented Nov 13, 2025

Description of changes:

Internal JS-6355

Add lazy loading capabilities for node:async_hooks, update doc.
This contains a breaking change as we changed the exported object.

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

@changeset-bot
Copy link

changeset-bot bot commented Nov 13, 2025

🦋 Changeset detected

Latest commit: 7cffc76

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@aws/lambda-invoke-store Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@maxday maxday marked this pull request as ready for review November 14, 2025 00:06
@maxday maxday requested a review from a team as a code owner November 14, 2025 00:06
Copy link
Member

@trivikr trivikr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor comments on changelog

}

export const InvokeStore = instance;
export const _testing =
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you mean to keep this here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes? I could not figure out a better way of exposing a way to remove the instance, this is used only in the benchmark (as we want to actually measure the init time)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, then I recommend marking @internal

@trivikr trivikr merged commit d14bda4 into awslabs:main Nov 14, 2025
3 checks passed
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