@aws-sdk/types should be a full dependency of @aws-sdk/util-retry

[jfirebaugh]

Checkboxes for prior research

• I've gone through Developer Guide and API reference
• I've checked AWS Forums and StackOverflow.
• I've searched for previous similar issues and didn't find any solution.

Describe the bug

@aws-sdk/util-retry lists @aws-sdk/types as a devDependency, but depends on it for types, so it should be a full dependency.

SDK version number

@aws-sdk/util-retry@3.272.0

Which JavaScript Runtime is this issue in?

Node.js

Details of the browser/Node.js/ReactNative version

v16.17.1

Reproduction Steps

pnpm init
pnpm i @aws-sdk/client-secrets-manager typescript
echo "hoist=false" >.npmrc

tsconfig.json:

{
  "compilerOptions": {
    "lib": ["es2018", "dom"],
  }
}

index.ts:

import { SecretsManagerClient, GetSecretValueCommand } from '@aws-sdk/client-secrets-manager';

(async () => {
    const client = new SecretsManagerClient({ region: 'us-west-2' });
    const response = await client.send(new GetSecretValueCommand({
        SecretId: 'somekey'
    }))
    console.log(response)
})().then(console.log).catch(console.log)

Run pnpm tsc.

Observed Behavior

node_modules/.pnpm/@aws-sdk+config-resolver@3.272.0/node_modules/@aws-sdk/config-resolver/dist-types/endpointsConfig/NodeUseDualstackEndpointConfigOptions.d.ts:1:39 - error TS2307: Cannot find module '@aws-sdk/node-config-provider' or its corresponding type declarations.

1 import { LoadedConfigSelectors } from "@aws-sdk/node-config-provider";
                                        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

node_modules/.pnpm/@aws-sdk+config-resolver@3.272.0/node_modules/@aws-sdk/config-resolver/dist-types/endpointsConfig/NodeUseFipsEndpointConfigOptions.d.ts:1:39 - error TS2307: Cannot find module '@aws-sdk/node-config-provider' or its corresponding type declarations.

1 import { LoadedConfigSelectors } from "@aws-sdk/node-config-provider";
                                        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

node_modules/.pnpm/@aws-sdk+config-resolver@3.272.0/node_modules/@aws-sdk/config-resolver/dist-types/regionConfig/config.d.ts:1:59 - error TS2307: Cannot find module '@aws-sdk/node-config-provider' or its corresponding type declarations.

1 import { LoadedConfigSelectors, LocalConfigOptions } from "@aws-sdk/node-config-provider";
                                                            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

node_modules/.pnpm/@aws-sdk+middleware-retry@3.272.0/node_modules/@aws-sdk/middleware-retry/dist-types/configurations.d.ts:1:39 - error TS2307: Cannot find module '@aws-sdk/node-config-provider' or its corresponding type declarations.

1 import { LoadedConfigSelectors } from "@aws-sdk/node-config-provider";
                                        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

node_modules/.pnpm/@aws-sdk+util-retry@3.272.0/node_modules/@aws-sdk/util-retry/dist-types/AdaptiveRetryStrategy.d.ts:1:91 - error TS2307: Cannot find module '@aws-sdk/types' or its corresponding type declarations.

1 import { Provider, RetryErrorInfo, RetryStrategyV2, RetryToken, StandardRetryToken } from "@aws-sdk/types";
                                                                                            ~~~~~~~~~~~~~~~~

node_modules/.pnpm/@aws-sdk+util-retry@3.272.0/node_modules/@aws-sdk/util-retry/dist-types/StandardRetryStrategy.d.ts:1:79 - error TS2307: Cannot find module '@aws-sdk/types' or its corresponding type declarations.

1 import { Provider, RetryErrorInfo, RetryStrategyV2, StandardRetryToken } from "@aws-sdk/types";
                                                                                ~~~~~~~~~~~~~~~~


Found 6 errors in 6 files.

Errors  Files
     1  node_modules/.pnpm/@aws-sdk+config-resolver@3.272.0/node_modules/@aws-sdk/config-resolver/dist-types/endpointsConfig/NodeUseDualstackEndpointConfigOptions.d.ts:1
     1  node_modules/.pnpm/@aws-sdk+config-resolver@3.272.0/node_modules/@aws-sdk/config-resolver/dist-types/endpointsConfig/NodeUseFipsEndpointConfigOptions.d.ts:1
     1  node_modules/.pnpm/@aws-sdk+config-resolver@3.272.0/node_modules/@aws-sdk/config-resolver/dist-types/regionConfig/config.d.ts:1
     1  node_modules/.pnpm/@aws-sdk+middleware-retry@3.272.0/node_modules/@aws-sdk/middleware-retry/dist-types/configurations.d.ts:1
     1  node_modules/.pnpm/@aws-sdk+util-retry@3.272.0/node_modules/@aws-sdk/util-retry/dist-types/AdaptiveRetryStrategy.d.ts:1
     1  node_modules/.pnpm/@aws-sdk+util-retry@3.272.0/node_modules/@aws-sdk/util-retry/dist-types/StandardRetryStrategy.d.ts:1

Expected Behavior

Compiles successfully.

Possible Solution

No response

Additional Information/Context

No response

[jfirebaugh]

Please also consider testing with pnpm and hoisting disabled to ensure that aws-sdk does not accidentally add "phantom dependencies" in the future.

[spbarber]

I am also getting the same issue when using yarn.

Any way to get around this?

[andrew-rowley]

me too .. we added packageExtensions rules to our .yarnrc.yml to resolve

[spbarber]

Do you have an example of what that looks like?

[andrew-rowley]

Do you have an example of what that looks like?

packageExtensions:
  "@aws-sdk/config-resolver@*":
    dependencies:
      "@aws-sdk/node-config-provider": "*"
  "@aws-sdk/middleware-retry@*":
    dependencies:
      "@aws-sdk/node-config-provider": "*"
  "@aws-sdk/util-retry@*":
    dependencies:
      "@aws-sdk/types": "*"

[spbarber]

Worked like a charm. Thanks very much 😄

[github-actions]

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.