Skip to content

fix(client-sts): update imports to new module locations#8025

Merged
kuhe merged 1 commit into
mainfrom
kuhe/chore/imports
May 15, 2026
Merged

fix(client-sts): update imports to new module locations#8025
kuhe merged 1 commit into
mainfrom
kuhe/chore/imports

Conversation

@kuhe

@kuhe kuhe commented May 15, 2026

Copy link
Copy Markdown
Contributor

Issue

#8027
#8004

Description

  • regenerate the imports for client-sts
  • clean up imports in the ./private folder's non-code-generated packages.

Testing

CI

Checklist

  • If the PR is a feature, add integration tests (*.integ.spec.ts) or E2E tests.
    • It's not a feature.
  • My E2E tests are resilient to concurrent i/o.
    • I didn't write any E2E tests.
  • I added access level annotations e.g. @public, @internal tags and enabled doc generation on the package. Remember that access level annotations go below the description, not above.
    • I didn't add any public functions.
  • Streams - how do they work?? My WebStream readers/locks are properly lifecycled. Node.js stream backpressure is handled. Error handling.
    • No streams here.

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

@kuhe kuhe requested a review from a team as a code owner May 15, 2026 17:59
@kuhe kuhe force-pushed the kuhe/chore/imports branch 3 times, most recently from 18008cd to 7a7b15d Compare May 15, 2026 19:20
# Conflicts:
#	private/aws-client-api-test/package.json
#	private/aws-client-retry-test/package.json
#	yarn.lock
@kuhe kuhe force-pushed the kuhe/chore/imports branch from 7a7b15d to ed4e831 Compare May 15, 2026 20:35
@kuhe kuhe changed the title chore(private): update imports in private test utils fix(client-sts): update imports to new module locations May 15, 2026
// https://github.com/aws/aws-sdk-js-v3/blob/main/codegen/smithy-aws-typescript-codegen/src/main/resources/software/amazon/smithy/aws/typescript/codegen/sts-client-defaultStsRoleAssumers.ts
import { setCredentialFeature } from "@aws-sdk/core/client";
import { stsRegionDefaultResolver } from "@aws-sdk/region-config-resolver";
import { setCredentialFeature, stsRegionDefaultResolver } from "@aws-sdk/core/client";

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The template was already updated

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

it's possible that changes to how often the Java codegen is recompiled caused the resource file which contains the import path not to be picked up.

In the future I'd like to remove the resource file and use Java string templates instead. The resource file doesn't integrate well with TS tools in its current location anyway.

@kuhe kuhe merged commit be183b6 into main May 15, 2026
7 checks passed
@kuhe kuhe deleted the kuhe/chore/imports branch May 15, 2026 21:09
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