chore(scripts): replace tsc by esbuild (#3425)

Author: millotp

eslint/package.json

@@ -2,12 +2,12 @@
   "name": "eslint-plugin-automation-custom",
   "version": "1.0.0",
   "description": "Custom rules for eslint",
-  "main": "dist/src/index.js",
+  "main": "dist/index.js",
   "files": [
     "src/**.ts"
   ],
   "scripts": {
-    "build": "rm -rf dist/ && tsc",
+    "build": "esbuild --bundle --minify --platform=node --outdir=dist --log-level=error src/index.ts",
     "lint": "eslint --ext=ts .",
     "lint:fix": "eslint --ext=ts --fix .",
     "test": "jest"
@@ -17,6 +17,7 @@
     "@babel/preset-env": "7.24.8",
     "@babel/preset-typescript": "7.24.7",
     "@types/jest": "29.5.12",
+    "esbuild": "0.23.0",
     "eslint": "8.57.0",
     "jest": "29.7.0",
     "typescript": "5.5.3"

scripts/ci/actions/restore-artifacts/action.yml

@@ -11,4 +11,4 @@ inputs:
       List of languages to restore (use the pipe character to put each language on its own line)
 runs:
   using: 'node20'
-  main: './builddir/index.js'
+  main: './builddir/index.cjs'

scripts/ci/actions/restore-artifacts/builddir/index.cjs

@@ -4,9 +4,8 @@ import { setOutput } from '@actions/core';
 import { CLIENTS, createClientName, GENERATORS, LANGUAGES } from '../../common.js';
 import { getLanguageFolder, getTestExtension, getTestOutputFolder } from '../../config.js';
 
-import { COMMON_DEPENDENCIES, DEPENDENCIES } from './setRunVariables.js';
 import type { ClientMatrix, CreateMatrix, ToRunMatrix } from './types.js';
-import { isBaseChanged } from './utils.js';
+import { COMMON_DEPENDENCIES, DEPENDENCIES, isBaseChanged } from './utils.js';
 
 // This empty matrix is required by the CI, otherwise it throws
 const EMPTY_MATRIX = { client: ['no-run'] };

scripts/ci/actions/restore-artifacts/builddir/index.js

@@ -1,75 +1,7 @@
 /* eslint-disable no-console */
 import * as core from '@actions/core';
 
-import { LANGUAGES } from '../../common.js';
-import { getLanguageFolder } from '../../config.js';
-import type { Language } from '../../types.js';
-
-import { isBaseChanged } from './utils.js';
-
-export const COMMON_DEPENDENCIES = {
-  GITHUB_ACTIONS_CHANGED: ['.github/actions', '.github/workflows'],
-  SCRIPTS_CHANGED: [
-    'scripts',
-    'eslint',
-    'yarn.lock',
-    '.eslintrc.cjs',
-    'config/generation.config.mjs',
-    'config/clients.config.json',
-    'config/release.config.json',
-    'generators',
-    'tests/CTS',
-    '.nvmrc',
-  ],
-  COMMON_SPECS_CHANGED: ['specs/common'],
-};
-
-export function getVersionFileForLanguage(lang: Language): string {
-  // js rely on the nvmrc of the repo
-  if (lang === 'javascript') {
-    return '.nvmrc';
-  }
-
-  // jvm lang rely on the same java version
-  if (lang === 'kotlin' || lang === 'java' || lang === 'scala') {
-    return 'config/.java-version';
-  }
-
-  return `config/.${lang}-version`;
-}
-
-/**
- * This dependency array is generated to match the "external" dependencies of a generated client.
- *
- * Those variables are used to determine if jobs should run, based on the changes
- * made in their respective dependencies.
- *
- * Negative paths should start with `:!`.
- *
- * The variable will be accessible in the CI via `steps.diff.outputs.<name>`.
- *
- * Variables starting by `LANGUAGENAME_` will be used in the `createMatrix` to determine
- * if a job should be added.
- */
-export const DEPENDENCIES = LANGUAGES.reduce(
-  (finalDependencies, lang) => {
-    const key = `${lang.toUpperCase()}_CLIENT_CHANGED`;
-    const langFolder = getLanguageFolder(lang);
-
-    finalDependencies[key] = [
-      ':!**node_modules',
-      `templates/${lang}`,
-      // language related files
-      langFolder,
-      getVersionFileForLanguage(lang),
-      `:!${langFolder}/.github`,
-      `:!${langFolder}/README.md`,
-    ];
-
-    return finalDependencies;
-  },
-  { ...COMMON_DEPENDENCIES } as Record<string, string[]>,
-);
+import { DEPENDENCIES, isBaseChanged } from './utils.js';
 
 /**
  * Outputs variables used in the CI to determine if a job should run.

scripts/ci/actions/restore-artifacts/builddir/index.js.map

@@ -1,8 +1,75 @@
 /* eslint-disable no-console */
 import * as core from '@actions/core';
 
+import { LANGUAGES } from '../../common.js';
+import { getLanguageFolder } from '../../config.js';
+import type { Language } from '../../types.js';
 import { getNbGitDiff } from '../utils.js';
 
+export const COMMON_DEPENDENCIES = {
+  GITHUB_ACTIONS_CHANGED: ['.github/actions', '.github/workflows'],
+  SCRIPTS_CHANGED: [
+    'scripts',
+    'eslint',
+    'yarn.lock',
+    '.eslintrc.cjs',
+    'config/generation.config.mjs',
+    'config/clients.config.json',
+    'config/release.config.json',
+    'generators',
+    'tests/CTS',
+    '.nvmrc',
+  ],
+  COMMON_SPECS_CHANGED: ['specs/common'],
+};
+
+/**
+ * This dependency array is generated to match the "external" dependencies of a generated client.
+ *
+ * Those variables are used to determine if jobs should run, based on the changes
+ * made in their respective dependencies.
+ *
+ * Negative paths should start with `:!`.
+ *
+ * The variable will be accessible in the CI via `steps.diff.outputs.<name>`.
+ *
+ * Variables starting by `LANGUAGENAME_` will be used in the `createMatrix` to determine
+ * if a job should be added.
+ */
+export const DEPENDENCIES = LANGUAGES.reduce(
+  (finalDependencies, lang) => {
+    const key = `${lang.toUpperCase()}_CLIENT_CHANGED`;
+    const langFolder = getLanguageFolder(lang);
+
+    finalDependencies[key] = [
+      ':!**node_modules',
+      `templates/${lang}`,
+      // language related files
+      langFolder,
+      getVersionFileForLanguage(lang),
+      `:!${langFolder}/.github`,
+      `:!${langFolder}/README.md`,
+    ];
+
+    return finalDependencies;
+  },
+  { ...COMMON_DEPENDENCIES } as Record<string, string[]>,
+);
+
+function getVersionFileForLanguage(lang: Language): string {
+  // js rely on the nvmrc of the repo
+  if (lang === 'javascript') {
+    return '.nvmrc';
+  }
+
+  // jvm lang rely on the same java version
+  if (lang === 'kotlin' || lang === 'java' || lang === 'scala') {
+    return 'config/.java-version';
+  }
+
+  return `config/.${lang}-version`;
+}
+
 /**
  * Determines if changes have been found in the `dependencies`, compared to the `baseBranch`.
  *