test: add tsup/dts and metro bundler compatibility tests (#8122)

kuhe · web-flow · commit 88ae83fcff33 · 2026-06-23T14:47:20.000-04:00
* test: add tsup/dts and metro bundler compatibility tests

* fix: bump DTS worker memory to 8GB for CI
diff --git a/Makefile b/Makefile
@@ -101,6 +101,7 @@ bundles: build-s3-browser-bundle build-signature-v4-multi-region-browser-bundle
 
 test-bundlers:
 	node ./tests/bundlers/bundler-canary.mjs
+	(cd ./tests/bundler-compat && npm install && node ./run.mjs)
 
 build-s3-browser-bundle:
 	node ./clients/client-s3/test/browser-build/esbuild
diff --git a/lib/lib-transfer-manager/transfer-manager.d.ts b/lib/lib-transfer-manager/transfer-manager.d.ts
@@ -2,6 +2,4 @@
  * Do not edit:
  * This is a compatibility redirect for contexts that do not understand package.json exports field.
  */
-declare module "@aws-sdk/lib-transfer-manager/transfer-manager" {
-  export * from "@aws-sdk/lib-transfer-manager/dist-types/submodules/transfer-manager/index.d";
-}
+export * from "./dist-types/submodules/transfer-manager/index";
diff --git a/lib/lib-transfer-manager/worker.d.ts b/lib/lib-transfer-manager/worker.d.ts
@@ -1,9 +1,5 @@
-
-  /**
-   * Do not edit:
-   * This is a compatibility redirect for contexts that do not understand package.json exports field.
-   */
-  declare module "@aws-sdk/lib-transfer-manager/worker" {
-    export * from "@aws-sdk/lib-transfer-manager/dist-types/submodules/worker/index.d";
-  }
-  
+/**
+ * Do not edit:
+ * This is a compatibility redirect for contexts that do not understand package.json exports field.
+ */
+export * from "./dist-types/submodules/worker/index";
diff --git a/lib/lib-transfer-manager/worker.js b/lib/lib-transfer-manager/worker.js
@@ -1,7 +1,5 @@
-
-  /**
-   * Do not edit:
-   * This is a compatibility redirect for contexts that do not understand package.json exports field.
-   */
-  module.exports = require("./dist-cjs/submodules/worker/index.js");
-  
+/**
+ * Do not edit:
+ * This is a compatibility redirect for contexts that do not understand package.json exports field.
+ */
+module.exports = require("./dist-cjs/submodules/worker/index.js");
diff --git a/packages-internal/core/account-id-endpoint.d.ts b/packages-internal/core/account-id-endpoint.d.ts
@@ -2,6 +2,4 @@
  * Do not edit:
  * This is a compatibility redirect for contexts that do not understand package.json exports field.
  */
-declare module "@aws-sdk/core/account-id-endpoint" {
-  export * from "@aws-sdk/core/dist-types/submodules/account-id-endpoint/index.d";
-}
+export * from "./dist-types/submodules/account-id-endpoint/index";
diff --git a/packages-internal/core/client.d.ts b/packages-internal/core/client.d.ts
@@ -2,6 +2,4 @@
  * Do not edit:
  * This is a compatibility redirect for contexts that do not understand package.json exports field.
  */
-declare module "@aws-sdk/core/client" {
-  export * from "@aws-sdk/core/dist-types/submodules/client/index.d";
-}
+export * from "./dist-types/submodules/client/index";
diff --git a/packages-internal/core/httpAuthSchemes.d.ts b/packages-internal/core/httpAuthSchemes.d.ts
@@ -2,6 +2,4 @@
  * Do not edit:
  * This is a compatibility redirect for contexts that do not understand package.json exports field.
  */
-declare module "@aws-sdk/core/httpAuthSchemes" {
-  export * from "@aws-sdk/core/dist-types/submodules/httpAuthSchemes/index.d";
-}
+export * from "./dist-types/submodules/httpAuthSchemes/index";
diff --git a/packages-internal/core/protocols.d.ts b/packages-internal/core/protocols.d.ts
@@ -2,6 +2,4 @@
  * Do not edit:
  * This is a compatibility redirect for contexts that do not understand package.json exports field.
  */
-declare module "@aws-sdk/core/protocols" {
-  export * from "@aws-sdk/core/dist-types/submodules/protocols/index.d";
-}
+export * from "./dist-types/submodules/protocols/index";
diff --git a/packages-internal/core/util.d.ts b/packages-internal/core/util.d.ts
@@ -2,6 +2,4 @@
  * Do not edit:
  * This is a compatibility redirect for contexts that do not understand package.json exports field.
  */
-declare module "@aws-sdk/core/util" {
-  export * from "@aws-sdk/core/dist-types/submodules/util/index.d";
-}
+export * from "./dist-types/submodules/util/index";
diff --git a/packages-internal/middleware-sdk-s3/s3-control.d.ts b/packages-internal/middleware-sdk-s3/s3-control.d.ts
@@ -2,6 +2,4 @@
  * Do not edit:
  * This is a compatibility redirect for contexts that do not understand package.json exports field.
  */
-declare module "@aws-sdk/middleware-sdk-s3/s3-control" {
-  export * from "@aws-sdk/middleware-sdk-s3/dist-types/submodules/s3-control/index.d";
-}
+export * from "./dist-types/submodules/s3-control/index";
diff --git a/packages-internal/middleware-sdk-s3/s3.d.ts b/packages-internal/middleware-sdk-s3/s3.d.ts
@@ -2,6 +2,4 @@
  * Do not edit:
  * This is a compatibility redirect for contexts that do not understand package.json exports field.
  */
-declare module "@aws-sdk/middleware-sdk-s3/s3" {
-  export * from "@aws-sdk/middleware-sdk-s3/dist-types/submodules/s3/index.d";
-}
+export * from "./dist-types/submodules/s3/index";
diff --git a/packages-internal/nested-clients/cognito-identity.d.ts b/packages-internal/nested-clients/cognito-identity.d.ts
@@ -2,6 +2,4 @@
  * Do not edit:
  * This is a compatibility redirect for contexts that do not understand package.json exports field.
  */
-declare module "@aws-sdk/nested-clients/cognito-identity" {
-  export * from "@aws-sdk/nested-clients/dist-types/submodules/cognito-identity/index.d";
-}
+export * from "./dist-types/submodules/cognito-identity/index";
diff --git a/packages-internal/nested-clients/signin.d.ts b/packages-internal/nested-clients/signin.d.ts
@@ -2,6 +2,4 @@
  * Do not edit:
  * This is a compatibility redirect for contexts that do not understand package.json exports field.
  */
-declare module "@aws-sdk/nested-clients/signin" {
-  export * from "@aws-sdk/nested-clients/dist-types/submodules/signin/index.d";
-}
+export * from "./dist-types/submodules/signin/index";
diff --git a/packages-internal/nested-clients/sso-oidc.d.ts b/packages-internal/nested-clients/sso-oidc.d.ts
@@ -2,6 +2,4 @@
  * Do not edit:
  * This is a compatibility redirect for contexts that do not understand package.json exports field.
  */
-declare module "@aws-sdk/nested-clients/sso-oidc" {
-  export * from "@aws-sdk/nested-clients/dist-types/submodules/sso-oidc/index.d";
-}
+export * from "./dist-types/submodules/sso-oidc/index";
diff --git a/packages-internal/nested-clients/sso.d.ts b/packages-internal/nested-clients/sso.d.ts
@@ -2,6 +2,4 @@
  * Do not edit:
  * This is a compatibility redirect for contexts that do not understand package.json exports field.
  */
-declare module "@aws-sdk/nested-clients/sso" {
-  export * from "@aws-sdk/nested-clients/dist-types/submodules/sso/index.d";
-}
+export * from "./dist-types/submodules/sso/index";
diff --git a/packages-internal/nested-clients/sts.d.ts b/packages-internal/nested-clients/sts.d.ts
@@ -2,6 +2,4 @@
  * Do not edit:
  * This is a compatibility redirect for contexts that do not understand package.json exports field.
  */
-declare module "@aws-sdk/nested-clients/sts" {
-  export * from "@aws-sdk/nested-clients/dist-types/submodules/sts/index.d";
-}
+export * from "./dist-types/submodules/sts/index";
diff --git a/packages-internal/xml-builder/scripts/benchmark.js b/packages-internal/xml-builder/scripts/benchmark.js
@@ -0,0 +1,90 @@
+const { XMLParser } = require("fast-xml-parser");
+const { parseXML } = require("../dist-cjs/xml-parser");
+
+const fxpParser = new XMLParser({
+  attributeNamePrefix: "",
+  processEntities: { enabled: true, maxTotalExpansions: Infinity },
+  htmlEntities: true,
+  ignoreAttributes: false,
+  ignoreDeclaration: true,
+  parseTagValue: false,
+  trimValues: false,
+  tagValueProcessor: (_, val) => (val.trim() === "" && val.includes("\n") ? "" : undefined),
+  maxNestedTags: Infinity,
+});
+
+function generateListBuckets(n) {
+  const bucket = `<Bucket>
+<CreationDate>2023-06-15T12:00:00.000Z</CreationDate>
+<Name>my-example-bucket-name-here</Name>
+<BucketRegion>us-east-1</BucketRegion>
+</Bucket>`;
+  return `<?xml version="1.0" encoding="UTF-8"?>
+<ListAllMyBucketsResult xmlns="http://s3.amazonaws.com/doc/2006-03-01/">
+<Owner><ID>abc123def456</ID><DisplayName>user@example.com</DisplayName></Owner>
+<Buckets>${bucket.repeat(n)}</Buckets>
+</ListAllMyBucketsResult>`;
+}
+
+function generateListObjects(n) {
+  const obj = `<Contents>
+<Key>photos/2024/vacation/IMG_${Math.random().toString(36).slice(2, 10)}.jpg</Key>
+<LastModified>2024-03-15T09:30:00.000Z</LastModified>
+<ETag>&quot;d41d8cd98f00b204e9800998ecf8427e&quot;</ETag>
+<Size>1048576</Size>
+<StorageClass>STANDARD</StorageClass>
+<Owner><ID>abc123</ID><DisplayName>owner</DisplayName></Owner>
+</Contents>`;
+  return `<?xml version="1.0" encoding="UTF-8"?>
+<ListBucketResult xmlns="http://s3.amazonaws.com/doc/2006-03-01/">
+<Name>my-bucket</Name>
+<Prefix>photos/</Prefix>
+<MaxKeys>1000</MaxKeys>
+<IsTruncated>false</IsTruncated>
+${obj.repeat(n)}
+</ListBucketResult>`;
+}
+
+function bench(name, xml, iterations = 100) {
+  // Warmup
+  for (let i = 0; i < 10; i++) {
+    parseXML(xml);
+    fxpParser.parse(xml, true);
+  }
+
+  const t0 = performance.now();
+  for (let i = 0; i < iterations; i++) parseXML(xml);
+  const internalMs = performance.now() - t0;
+
+  const t1 = performance.now();
+  for (let i = 0; i < iterations; i++) fxpParser.parse(xml, true);
+  const fxpMs = performance.now() - t1;
+
+  const ratio = (fxpMs / internalMs).toFixed(2);
+  const sizeKB = (Buffer.byteLength(xml) / 1024).toFixed(1);
+  console.log(
+    `${name.padEnd(30)} ${sizeKB.padStart(7)} KB | internal: ${(internalMs / iterations)
+      .toFixed(3)
+      .padStart(8)} ms | fxp: ${(fxpMs / iterations).toFixed(3).padStart(8)} ms | ratio: ${ratio}x`
+  );
+}
+
+console.log(
+  "Payload".padEnd(30) +
+    "   Size".padStart(7) +
+    "    |" +
+    " internal (avg)".padStart(20) +
+    " | fxp (avg)".padStart(15) +
+    "      | speedup"
+);
+console.log("-".repeat(110));
+
+for (const n of [1, 5, 10, 50, 100, 250, 500, 1000]) {
+  bench(`ListBuckets (${n})`, generateListBuckets(n));
+}
+
+console.log("");
+
+for (const n of [1, 5, 10, 50, 100, 250, 500, 1000]) {
+  bench(`ListObjects (${n})`, generateListObjects(n));
+}
diff --git a/packages/config/credentials.d.ts b/packages/config/credentials.d.ts
@@ -2,6 +2,4 @@
  * Do not edit:
  * This is a compatibility redirect for contexts that do not understand package.json exports field.
  */
-declare module "@aws-sdk/config/credentials" {
-  export * from "@aws-sdk/config/dist-types/submodules/credentials/index.d";
-}
+export * from "./dist-types/submodules/credentials/index";
diff --git a/packages/config/logger.d.ts b/packages/config/logger.d.ts
@@ -2,6 +2,4 @@
  * Do not edit:
  * This is a compatibility redirect for contexts that do not understand package.json exports field.
  */
-declare module "@aws-sdk/config/logger" {
-  export * from "@aws-sdk/config/dist-types/submodules/logger/index.d";
-}
+export * from "./dist-types/submodules/logger/index";
diff --git a/packages/config/package.json b/packages/config/package.json
@@ -45,6 +45,14 @@
     },
     "./requestHandler": {
       "types": "./dist-types/submodules/requestHandler/index.d.ts",
+      "react-native": {
+        "import": "./dist-es/submodules/requestHandler/index.browser.js",
+        "require": "./dist-cjs/submodules/requestHandler/index.browser.js"
+      },
+      "browser": {
+        "import": "./dist-es/submodules/requestHandler/index.browser.js",
+        "require": "./dist-cjs/submodules/requestHandler/index.browser.js"
+      },
       "module": "./dist-es/submodules/requestHandler/index.js",
       "node": "./dist-cjs/submodules/requestHandler/index.js",
       "import": "./dist-es/submodules/requestHandler/index.js",
@@ -119,5 +127,12 @@
     "type": "git",
     "url": "https://github.com/aws/aws-sdk-js-v3.git",
     "directory": "packages/config"
+  },
+  "browser": {
+    "./dist-es/submodules/requestHandler/index.js": "./dist-es/submodules/requestHandler/index.browser.js"
+  },
+  "react-native": {
+    "./dist-es/submodules/requestHandler/index.js": "./dist-es/submodules/requestHandler/index.browser.js",
+    "./dist-cjs/submodules/requestHandler/index.js": "./dist-cjs/submodules/requestHandler/index.browser.js"
   }
 }
diff --git a/packages/config/protocol.d.ts b/packages/config/protocol.d.ts
@@ -2,6 +2,4 @@
  * Do not edit:
  * This is a compatibility redirect for contexts that do not understand package.json exports field.
  */
-declare module "@aws-sdk/config/protocol" {
-  export * from "@aws-sdk/config/dist-types/submodules/protocol/index.d";
-}
+export * from "./dist-types/submodules/protocol/index";
diff --git a/packages/config/requestHandler.d.ts b/packages/config/requestHandler.d.ts
@@ -2,6 +2,4 @@
  * Do not edit:
  * This is a compatibility redirect for contexts that do not understand package.json exports field.
  */
-declare module "@aws-sdk/config/requestHandler" {
-  export * from "@aws-sdk/config/dist-types/submodules/requestHandler/index.d";
-}
+export * from "./dist-types/submodules/requestHandler/index";
diff --git a/packages/config/retryStrategy.d.ts b/packages/config/retryStrategy.d.ts
@@ -2,6 +2,4 @@
  * Do not edit:
  * This is a compatibility redirect for contexts that do not understand package.json exports field.
  */
-declare module "@aws-sdk/config/retryStrategy" {
-  export * from "@aws-sdk/config/dist-types/submodules/retryStrategy/index.d";
-}
+export * from "./dist-types/submodules/retryStrategy/index";
diff --git a/packages/config/src/submodules/requestHandler/index.browser.ts b/packages/config/src/submodules/requestHandler/index.browser.ts
@@ -0,0 +1,9 @@
+const no = Symbol.for("node-only");
+
+export { HttpRequest, HttpResponse } from "@smithy/core/protocols";
+
+export { FetchHttpHandler, type FetchHttpHandlerOptions } from "@smithy/fetch-http-handler";
+export { type NodeHttpHandlerOptions, type NodeHttp2HandlerOptions } from "@smithy/node-http-handler";
+
+export const NodeHttpHandler = no;
+export const NodeHttp2Handler = no;
diff --git a/packages/config/typecheck.d.ts b/packages/config/typecheck.d.ts
@@ -2,6 +2,4 @@
  * Do not edit:
  * This is a compatibility redirect for contexts that do not understand package.json exports field.
  */
-declare module "@aws-sdk/config/typecheck" {
-  export * from "@aws-sdk/config/dist-types/submodules/typecheck/index.d";
-}
+export * from "./dist-types/submodules/typecheck/index";
diff --git a/scripts/validation/submodules-linter.js b/scripts/validation/submodules-linter.js
diff --git a/tests/bundler-compat/.gitignore b/tests/bundler-compat/.gitignore
diff --git a/tests/bundler-compat/babel.config.cjs b/tests/bundler-compat/babel.config.cjs
diff --git a/tests/bundler-compat/metro-build.cjs b/tests/bundler-compat/metro-build.cjs
diff --git a/tests/bundler-compat/package.json b/tests/bundler-compat/package.json
diff --git a/tests/bundler-compat/run.mjs b/tests/bundler-compat/run.mjs
diff --git a/tests/bundler-compat/src/dts-check.ts b/tests/bundler-compat/src/dts-check.ts
diff --git a/tests/bundler-compat/src/sample-app.ts b/tests/bundler-compat/src/sample-app.ts
diff --git a/tests/bundler-compat/tsconfig.json b/tests/bundler-compat/tsconfig.json
diff --git a/tests/bundler-compat/tsup-dts.config.ts b/tests/bundler-compat/tsup-dts.config.ts
diff --git a/tests/bundlers/bundler-output-analysis.cjs b/tests/bundlers/bundler-output-analysis.cjs
