chore(smithy-client): build-fix, hide submodule export from api-extractor (#1596)

Author: kuhe

.changeset/pink-bats-explode.md

@@ -0,0 +1,2 @@
+---
+---

packages/core/api-extractor.json

@@ -0,0 +1,4 @@
+{
+  "extends": "../../api-extractor.packages.json",
+  "mainEntryPointFilePath": "./dist-types/index.d.ts"
+}

packages/core/package.json

@@ -11,6 +11,7 @@
     "clean": "rimraf ./dist-* && rimraf *.tsbuildinfo || exit 0",
     "lint": "npx eslint -c ../../.eslintrc.js \"src/**/*.ts\" --fix && node ./scripts/lint",
     "format": "prettier --config ../../prettier.config.js --ignore-path ../../.prettierignore --write \"**/*.{ts,md,json}\"",
+    "extract:docs": "api-extractor run --local",
     "test": "yarn g:vitest run",
     "test:cbor:perf": "node ./scripts/cbor-perf.mjs",
     "test:watch": "yarn g:vitest watch"

packages/smithy-client/package.json

@@ -11,7 +11,7 @@
     "clean": "rimraf ./dist-* && rimraf *.tsbuildinfo || exit 0",
     "lint": "eslint -c ../../.eslintrc.js \"src/**/*.ts\"",
     "format": "prettier --config ../../prettier.config.js --ignore-path ../../.prettierignore --write \"**/*.{ts,md,json}\"",
-    "extract:docs": "api-extractor run --local",
+    "extract:docs": "node ./scripts/fix-api-extractor && api-extractor run --local && node ./scripts/fix-api-extractor --unset",
     "test": "yarn g:vitest run",
     "test:watch": "yarn g:vitest watch"
   },

packages/smithy-client/scripts/fix-api-extractor.js

@@ -0,0 +1,34 @@
+const fs = require("node:fs");
+const path = require("path");
+
+const file = path.join(__dirname, "..", "dist-types", "index.d.ts");
+
+const contents = fs.readFileSync(file, "utf-8");
+
+/**
+ * This build script temporarily excludes the statement
+ * ```
+ * export * from "@smithy/core/serde";
+ * ```
+ * from the package's type declarations during running api-extractor.
+ *
+ * This has no effect on the runtime, but api-extractor at this time cannot understand
+ * the package.json exports field nor the compatibility redirect (packages/core/serde.d.ts).
+ */
+
+module.exports = {
+  source: `export * from "@smithy/core/serde";`,
+  replacement: `/* export * from "@smithy/core/serde"; */`,
+  unset() {
+    fs.writeFileSync(file, contents.replace(module.exports.replacement, module.exports.source));
+  },
+  set() {
+    fs.writeFileSync(file, contents.replace(module.exports.source, module.exports.replacement));
+  },
+};
+
+if (process.argv.includes("--unset")) {
+  module.exports.unset();
+} else {
+  module.exports.set();
+}