-
Notifications
You must be signed in to change notification settings - Fork 601
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Missed tslib dependency in @aws-sdk/types 3.199.0 #4075
Labels
Comments
I can confirm the issue occurs for |
Still actual in 3.197.0 |
Encountered the issue in |
This should be resolved by #4128. |
3 tasks
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Checkboxes for prior research
Describe the bug
Base tsconfig for cjs has importHelpers: true, so 'tslib' must be in dependencies for any compiled package:
aws-sdk-js-v3/tsconfig.cjs.json
Lines 3 to 5 in f9bc7af
'types' package has no dependencies, only devDependencies:
aws-sdk-js-v3/packages/types/package.json
Lines 42 to 49 in f9bc7af
It could be ok for types-only (in TS terms) package, but 'middleware-sdk-s3' imports EndpointURLScheme enum from 'types' package:
aws-sdk-js-v3/packages/util-endpoints/src/lib/parseURL.ts
Lines 1 to 8 in f9bc7af
And yes, dist-cjs/index.js requires 'tslib':
![image](https://user-images.githubusercontent.com/20106607/197289095-d8893f11-fd61-4e7b-8ab2-7ba0fabb870f.png)
As a result, we have an error inside strict environment like Yarn PnP:
In my case stack trace begins from 's3-client'.
SDK version number
3.194.0
Which JavaScript Runtime is this issue in?
Node.js
Details of the browser/Node.js/ReactNative version
v19.0.0
Reproduction Steps
yarn pnp, cjs, s3-client and SelectObjectContentCommand.
Observed Behavior
'missed dependency'
Expected Behavior
no error
Possible Solution
add missed dependency to 'types' package
Additional Information/Context
No response
The text was updated successfully, but these errors were encountered: