feat: use rollup for cjs distributions

Author: kuhe

.changeset/sour-seahorses-wash.md

@@ -0,0 +1,62 @@
+---
+"@smithy/service-client-documentation-generator": patch
+"@smithy/experimental-identity-and-auth": patch
+"@smithy/core": patch
+"@smithy/abort-controller": patch
+"@smithy/chunked-blob-reader": patch
+"@smithy/chunked-blob-reader-native": patch
+"@smithy/config-resolver": patch
+"@smithy/credential-provider-imds": patch
+"@smithy/eventstream-codec": patch
+"@smithy/eventstream-serde-browser": patch
+"@smithy/eventstream-serde-config-resolver": patch
+"@smithy/eventstream-serde-node": patch
+"@smithy/eventstream-serde-universal": patch
+"@smithy/fetch-http-handler": patch
+"@smithy/hash-blob-browser": patch
+"@smithy/hash-node": patch
+"@smithy/hash-stream-node": patch
+"@smithy/invalid-dependency": patch
+"@smithy/is-array-buffer": patch
+"@smithy/md5-js": patch
+"@smithy/middleware-apply-body-checksum": patch
+"@smithy/middleware-compression": patch
+"@smithy/middleware-content-length": patch
+"@smithy/middleware-endpoint": patch
+"@smithy/middleware-retry": patch
+"@smithy/middleware-serde": patch
+"@smithy/middleware-stack": patch
+"@smithy/node-config-provider": patch
+"@smithy/node-http-handler": patch
+"@smithy/property-provider": patch
+"@smithy/protocol-http": patch
+"@smithy/querystring-builder": patch
+"@smithy/querystring-parser": patch
+"@smithy/service-error-classification": patch
+"@smithy/shared-ini-file-loader": patch
+"@smithy/signature-v4": patch
+"@smithy/signature-v4a": patch
+"@smithy/smithy-client": patch
+"@smithy/types": patch
+"@smithy/url-parser": patch
+"@smithy/util-base64": patch
+"@smithy/util-body-length-browser": patch
+"@smithy/util-body-length-node": patch
+"@smithy/util-buffer-from": patch
+"@smithy/util-config-provider": patch
+"@smithy/util-defaults-mode-browser": patch
+"@smithy/util-defaults-mode-node": patch
+"@smithy/util-endpoints": patch
+"@smithy/util-hex-encoding": patch
+"@smithy/util-middleware": patch
+"@smithy/util-retry": patch
+"@smithy/util-stream": patch
+"@smithy/util-stream-browser": patch
+"@smithy/util-stream-node": patch
+"@smithy/util-uri-escape": patch
+"@smithy/util-utf8": patch
+"@smithy/util-waiter": patch
+"@smithy/uuid": patch
+---
+
+update dist-cjs generation to use rollup

package.json

@@ -36,6 +36,8 @@
   },
   "devDependencies": {
     "@microsoft/api-extractor": "7.52.7",
+    "@rollup/plugin-node-resolve": "^16.0.1",
+    "@rollup/plugin-typescript": "^12.1.4",
     "@tsconfig/recommended": "1.0.2",
     "@types/jest": "28.1.3",
     "@types/jsdom": "20.0.1",
@@ -51,6 +53,7 @@
     "jest": "29.7.0",
     "prettier": "3.2.5",
     "puppeteer": "^19.2.0",
+    "rollup": "^4.52.2",
     "ts-jest": "29.1.2",
     "turbo": "2.3.3",
     "typescript": "~5.8.3",

packages/core/src/util-identity-and-auth/memoizeIdentityProvider.ts

@@ -3,8 +3,10 @@ import type { Identity, IdentityProvider } from "@smithy/types";
 /**
  * @internal
  */
-export const createIsIdentityExpiredFunction = (expirationMs: number) => (identity: Identity) =>
-  doesIdentityRequireRefresh(identity) && identity.expiration!.getTime() - Date.now() < expirationMs;
+export const createIsIdentityExpiredFunction = (expirationMs: number) =>
+  function isIdentityExpired(identity: Identity) {
+    return doesIdentityRequireRefresh(identity) && identity.expiration!.getTime() - Date.now() < expirationMs;
+  };
 
 /**
  * @internal

packages/experimental-identity-and-auth/src/memoizeIdentityProvider.ts

@@ -3,8 +3,10 @@ import type { Identity, IdentityProvider } from "@smithy/types";
 /**
  * @internal
  */
-export const createIsIdentityExpiredFunction = (expirationMs: number) => (identity: Identity) =>
-  doesIdentityRequireRefresh(identity) && identity.expiration!.getTime() - Date.now() < expirationMs;
+export const createIsIdentityExpiredFunction = (expirationMs: number) =>
+  function isIdentityExpired(identity: Identity) {
+    return doesIdentityRequireRefresh(identity) && identity.expiration!.getTime() - Date.now() < expirationMs;
+  };
 
 /**
  * @internal

packages/service-client-documentation-generator/package.json

@@ -3,7 +3,7 @@
   "version": "4.1.0",
   "scripts": {
     "build": "concurrently 'yarn:build:cjs' 'yarn:build:es' 'yarn:build:types && yarn build:types:downlevel'",
-    "build:cjs": "node ../../scripts/inline service-client-documentation-generator",
+    "build:cjs": "yarn g:tsc -p tsconfig.cjs.json",
     "build:es": "yarn g:tsc -p tsconfig.es.json",
     "build:types": "yarn g:tsc -p tsconfig.types.json",
     "build:types:downlevel": "rimraf dist-types/ts3.4 && downlevel-dts dist-types dist-types/ts3.4",

scripts/compilation/Inliner.js

@@ -2,7 +2,9 @@ const fs = require("fs");
 const path = require("path");
 const { spawnProcess } = require("./../utils/spawn-process");
 const walk = require("./../utils/walk");
-const esbuild = require("esbuild");
+const rollup = require("rollup");
+const { nodeResolve } = require("@rollup/plugin-node-resolve");
+const typescript = require("@rollup/plugin-typescript");
 
 const root = path.join(__dirname, "..", "..");
 
@@ -18,7 +20,6 @@ module.exports = class Inliner {
     this.platform = "node";
     this.isPackage = fs.existsSync(path.join(root, "packages", pkg));
     this.isLib = fs.existsSync(path.join(root, "lib", pkg));
-    this.isClient = !this.isPackage && !this.isLib;
     this.isCore = pkg === "core";
     this.reExportStubs = false;
     this.subfolder = this.isPackage ? "packages" : this.isLib ? "lib" : "clients";
@@ -154,28 +155,68 @@ module.exports = class Inliner {
       return this;
     }
 
-    this.variantExternalsForEsBuild = this.variantExternals.map(
-      (variant) => "*/" + path.basename(variant).replace(/.js$/, "")
+    const variantExternalsForRollup = this.variantExternals.map((variant) =>
+      path.basename(variant).replace(/.js$/, "")
     );
 
-    const buildOptions = {
-      platform: this.platform,
-      target: ["node18"],
-      bundle: true,
-      format: "cjs",
-      mainFields: ["main"],
-      allowOverwrite: true,
-      entryPoints: [path.join(root, this.subfolder, this.package, "src", "index.ts")],
-      supported: {
-        "dynamic-import": false,
+    const entryPoint = path.join(root, this.subfolder, this.package, "src", "index.ts");
+
+    const inputOptions = {
+      input: entryPoint,
+      plugins: [
+        nodeResolve(),
+        typescript({
+          compilerOptions: {
+            importHelpers: true,
+            noEmitHelpers: false,
+            module: "esnext",
+            target: "es2022",
+            noCheck: true,
+            removeComments: true,
+          },
+        }),
+      ],
+      external: (id) => {
+        const relative = !!id.match(/^\.?\.?\//);
+        if (!relative) {
+          this.verbose && console.log("EXTERN (pkg)", id);
+          return true;
+        }
+
+        const local = id.includes(`/packages/`) && id.includes(`/dist-es/`);
+        if (local) {
+          this.verbose && console.log("EXTERN (local)", id);
+          return true;
+        }
+
+        if (id === entryPoint) {
+          this.verbose && console.log("INTERN (entry point)", id);
+          return false;
+        }
+
+        for (const file of variantExternalsForRollup) {
+          const idWithoutExtension = id.replace(/\.ts$/, "");
+          if (idWithoutExtension.endsWith(file)) {
+            this.verbose && console.log("EXTERN (variant)", id);
+            return true;
+          }
+        }
+
+        this.verbose && console.log("INTERN (invariant)", id);
+        return false;
       },
-      outfile: this.outfile,
-      keepNames: true,
-      packages: "external",
-      external: ["@smithy/*", "@aws-sdk/*", "node_modules/*", ...this.variantExternalsForEsBuild],
     };
 
-    await esbuild.build(buildOptions);
+    const outputOptions = {
+      file: this.outfile,
+      format: "cjs",
+      exports: "named",
+      preserveModules: false,
+    };
+
+    const bundle = await rollup.rollup(inputOptions);
+    await bundle.write(outputOptions);
+    await bundle.close();
 
     if (this.isCore) {
       const submodules = fs.readdirSync(path.join(root, this.subfolder, this.package, "src", "submodules"));
@@ -189,12 +230,18 @@ module.exports = class Inliner {
         ) {
           continue;
         }
-        await esbuild.build({
-          ...buildOptions,
-          keepNames: false,
-          entryPoints: [path.join(root, this.subfolder, this.package, "src", "submodules", submodule, "index.ts")],
-          outfile: path.join(root, this.subfolder, this.package, "dist-cjs", "submodules", submodule, "index.js"),
+
+        const submoduleBundle = await rollup.rollup({
+          ...inputOptions,
+          input: path.join(root, this.subfolder, this.package, "src", "submodules", submodule, "index.ts"),
         });
+
+        await submoduleBundle.write({
+          ...outputOptions,
+          file: path.join(root, this.subfolder, this.package, "dist-cjs", "submodules", submodule, "index.js"),
+        });
+
+        await submoduleBundle.close();
       }
     }
 
@@ -398,10 +445,33 @@ module.exports = class Inliner {
     }
 
     // check ESM compat.
-    const tmpFileContents = this.canonicalExports
-      .filter((sym) => !sym.includes(":"))
-      .map((sym) => `import { ${sym} } from "${this.pkgJson.name}";`)
-      .join("\n");
+    const tmpFileContents =
+      `import assert from "node:assert";\n` +
+      this.canonicalExports
+        .filter((sym) => !sym.includes(":"))
+        .map((sym) => {
+          if (
+            [
+              "getDefaultClientConfiguration", // renamed as an alias
+              "generateIdempotencyToken", // sometimes called v4
+              "expectInt", // aliased to expectLong
+              "handleFloat", // aliased to limitedParseDouble
+              "limitedParseFloat", // aliased to limitedParseDouble
+              "strictParseFloat", // aliased to strictParseDouble
+              "strictParseInt", // aliased to strictParseLong
+            ].includes(sym)
+          ) {
+            return `import { ${sym} } from "${this.pkgJson.name}";`;
+          }
+          return `import { ${sym} } from "${this.pkgJson.name}";
+if (typeof ${sym} === "function") {
+  if (${sym}.name !== "${sym}") {
+    throw new Error(${sym}.name + " does not equal expected ${sym}.")
+  }
+} 
+        `;
+        })
+        .join("\n");
     fs.writeFileSync(path.join(__dirname, "tmp", this.package + ".mjs"), tmpFileContents, "utf-8");
     await spawnProcess("node", [path.join(__dirname, "tmp", this.package + ".mjs")]);
     if (this.verbose) {

tsconfig.cjs.json

@@ -4,7 +4,7 @@
     "importHelpers": true,
     "module": "commonjs",
     "noEmitHelpers": false,
-    "target": "ES2018",
+    "target": "es2022",
     "noCheck": true
   }
 }

tsconfig.es.json

@@ -5,7 +5,7 @@
     "module": "ESNext",
     "moduleResolution": "bundler",
     "noEmitHelpers": false,
-    "target": "ES2020",
+    "target": "es2022",
     "noCheck": true
   }
 }