feat: use common TSConfig for all packages

[trivikr]

Issue

Internal: JS-2844
Refs: #1307

Description

Update clients and protocol_tests to use root TSConfig

Shell script for renaming tsconfig.json to tsconfig.cjs.json

foreach parentDir ("clients" "protocol_tests")
  for dir in ./$parentDir/*; do (cd "$dir" && sed -i 's/tsconfig.json/tsconfig.cjs.json/g' ./package.json); done
  for dir in ./$parentDir/*; do (cd "$dir" && mv tsconfig.json tsconfig.cjs.json); done
end

Script used for updating tsconfig

// @ts-check
import { readdirSync, readFileSync, writeFileSync } from "fs";
import { join } from "path";

const clients = ["clients/*", "protocol_tests/*"];

clients
  .map((dir) => dir.replace("/*", ""))
  .forEach((workspacesDir) => {
    // Process each workspace in workspace directory
    readdirSync(join(process.cwd(), workspacesDir), { withFileTypes: true })
      .filter((dirent) => dirent.isDirectory())
      .map((dirent) => dirent.name)
      .forEach((workspaceDir) => {
        const cwd = join(process.cwd(), workspacesDir, workspaceDir);

        const customTsConfig = {
          "tsconfig.cjs.json": {
            outDir: "dist-cjs",
          },
          "tsconfig.es.json": {
            outDir: "dist-es",
          },
          "tsconfig.types.json": {
            declarationDir: "dist-types",
          },
        };

        const extendsTsConfig = {
          "tsconfig.cjs.json": "../../tsconfig.cjs.json",
          "tsconfig.es.json": "../../tsconfig.es.json",
          "tsconfig.types.json": "../../tsconfig.types.json",
        };

        for (const tsconfigFileName of ["tsconfig.cjs.json", "tsconfig.es.json", "tsconfig.types.json"]) {
          const tsconfigFilePath = join(cwd, tsconfigFileName);
          const tsconfig = JSON.parse(readFileSync(tsconfigFilePath, "utf8").toString());
          writeFileSync(
            tsconfigFilePath,
            JSON.stringify(
              {
                ...tsconfig,
                compilerOptions: {
                  baseUrl: ".",
                  rootDir: "src",
                  ...customTsConfig[tsconfigFileName],
                  ...(tsconfig?.compilerOptions?.lib && { lib: tsconfig.compilerOptions.lib }),
                  ...(tsconfig?.compilerOptions?.types && { types: tsconfig.compilerOptions.types }),
                },
                extends: extendsTsConfig[tsconfigFileName],
                include: ["src/"],
              },
              null,
              2
            ).concat(`\n`)
          );
        }
      });
  });

Testing

CI

Additional context

Attempt to use common TSConfig without bumping Jest #2864

---

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

[codecov-commenter]

Codecov Report

Merging #2872 (00ff8df) into main (10d64df) will decrease coverage by 25.95%.
The diff coverage is n/a.

❗ Current head 00ff8df differs from pull request most recent head b51f27d. Consider uploading reports for the commit b51f27d to get more accurate results

@@             Coverage Diff             @@
##             main    #2872       +/-   ##
===========================================
- Coverage   60.42%   34.46%   -25.96%     
===========================================
  Files         554      554               
  Lines       28886    30459     +1573     
  Branches     7072     7283      +211     
===========================================
- Hits        17453    10499     -6954     
- Misses      11433    19960     +8527     

Impacted Files	Coverage Δ
.../commands/HttpPayloadTraitsWithMediaTypeCommand.ts	34.61% <0.00%> (-65.39%)	⬇️
.../commands/HttpPayloadTraitsWithMediaTypeCommand.ts	34.61% <0.00%> (-65.39%)	⬇️
...nds/HttpPayloadWithXmlNamespaceAndPrefixCommand.ts	34.61% <0.00%> (-65.39%)	⬇️
...s/src/commands/AssumeRoleWithWebIdentityCommand.ts	36.00% <0.00%> (-64.00%)	⬇️
...l/src/commands/InputAndOutputWithHeadersCommand.ts	36.00% <0.00%> (-64.00%)	⬇️
...n/src/commands/InputAndOutputWithHeadersCommand.ts	36.00% <0.00%> (-64.00%)	⬇️
.../src/commands/FlattenedXmlMapWithXmlNameCommand.ts	36.00% <0.00%> (-64.00%)	⬇️
...src/commands/HttpPayloadWithXmlNamespaceCommand.ts	36.00% <0.00%> (-64.00%)	⬇️
...rc/commands/HttpPayloadWithMemberXmlNameCommand.ts	36.00% <0.00%> (-64.00%)	⬇️
...rc/commands/StreamingTraitsRequireLengthCommand.ts	36.00% <0.00%> (-64.00%)	⬇️
... and 263 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 10d64df...b51f27d. Read the comment docs.

[trivikr]

Superceded by #2864

[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.