build: update chalk

Updates to the latest version of Chalk which was tripping up the bot. As a result of this, some scripts had to be converted to `.mts`.

Author: crisbeto

.github/workflows/pr.yml

@@ -149,7 +149,7 @@ jobs:
       - name: Verify tooling setup
         run: pnpm check-tooling-setup
       - name: Build and create package artifacts
-        run: ./scripts/create-package-archives.js --suffix "pr${{github.event.number}}-$(git rev-parse --short HEAD)"
+        run: ./scripts/create-package-archives.mjs --suffix "pr${{github.event.number}}-$(git rev-parse --short HEAD)"
       - name: Upload artifacts
         uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
         with:

package.json

@@ -29,13 +29,13 @@
     "lint": "pnpm -s tslint && pnpm -s stylelint && pnpm -s ownerslint && pnpm -s ng-dev format changed --check",
     "e2e": "bazel test //src/... --build_tag_filters=e2e --test_tag_filters=e2e --build_tests_only",
     "deploy-dev-app": "node ./scripts/deploy-dev-app.js",
-    "breaking-changes": "ts-node --project scripts/tsconfig.json scripts/breaking-changes.ts",
-    "check-entry-point-setup": "node ./scripts/check-entry-point-setup.js",
-    "check-package-externals": "ts-node --project scripts/tsconfig.json scripts/check-package-externals.ts",
+    "breaking-changes": "node --no-warnings=ExperimentalWarning --loader ts-node/esm/transpile-only scripts/breaking-changes.mts",
+    "check-entry-point-setup": "node ./scripts/check-entry-point-setup.mjs",
+    "check-package-externals": "node --no-warnings=ExperimentalWarning --loader ts-node/esm/transpile-only scripts/check-package-externals.mts",
     "format": "pnpm -s ng-dev format changed",
     "cherry-pick-patch": "ts-node --project tools/cherry-pick-patch/tsconfig.json tools/cherry-pick-patch/cherry-pick-patch.ts",
-    "ownerslint": "ts-node --project scripts/tsconfig.json scripts/ownerslint.ts",
-    "detect-component-id-collisions": "ts-node --project scripts/tsconfig.json scripts/detect-component-id-collisions.ts",
+    "ownerslint": "node --no-warnings=ExperimentalWarning --loader ts-node/esm/transpile-only scripts/ownerslint.mts",
+    "detect-component-id-collisions": "node --no-warnings=ExperimentalWarning --loader ts-node/esm/transpile-only scripts/detect-component-id-collisions.mts",
     "tslint": "tslint -c tslint.json --project ./tsconfig.json",
     "stylelint": "stylelint \"(src|docs)/**/*.+(css|scss)\" --config .stylelintrc.json",
     "resync-caretaker-app": "ts-node --project scripts/tsconfig.json scripts/caretaking/resync-caretaker-app-prs.ts",
@@ -103,7 +103,7 @@
     "@types/yargs": "^17.0.8",
     "autoprefixer": "^10.4.2",
     "axe-core": "^4.10.3",
-    "chalk": "^4.1.0",
+    "chalk": "^5.4.1",
     "dgeni": "^0.4.14",
     "dgeni-packages": "^0.30.0",
     "esbuild": "^0.25.0",

pnpm-lock.yaml

@@ -5,8 +5,10 @@ import ts from 'typescript';
 
 const projectRoot = process.cwd();
 
+const packageJson = JSON.parse(readFileSync(join(projectRoot, 'package.json'), 'utf8'));
+
 // Current version from the package.json. Splits it on the dash to ignore pre-release labels.
-const packageVersion = require(join(projectRoot, 'package.json')).version.split('-')[0];
+const packageVersion = packageJson.version.split('-')[0];
 
 // Regex used to extract versions from a string.
 const versionRegex = /\d+\.\d+\.\d+/;

scripts/breaking-changes.ts renamed to scripts/breaking-changes.mts

@@ -7,15 +7,15 @@
  * script through a manifest file (generated by Bazel)
  */
 
-const {join, dirname} = require('path');
-const {sync: globSync} = require('glob');
-const minimatch = require('minimatch');
-const fs = require('fs');
-const chalk = require('chalk');
+import {join, dirname} from 'path';
+import {sync as globSync} from 'glob';
+import {readFileSync} from 'fs';
+import minimatch from 'minimatch';
+import chalk from 'chalk';
 
 const [entryPointManifest] = process.argv.slice(2);
-const entryPoints = JSON.parse(fs.readFileSync(entryPointManifest, 'utf8'));
-const packagesDir = join(__dirname, '../src');
+const entryPoints = JSON.parse(readFileSync(entryPointManifest, 'utf8'));
+const packagesDir = join(process.cwd(), 'src');
 
 /**
  * Globs that matches directories which should never be considered

scripts/check-entry-point-setup.js renamed to scripts/check-entry-point-setup.mjs

@@ -14,7 +14,7 @@ import minimatch from 'minimatch';
 import {join, relative} from 'path';
 import ts from 'typescript';
 
-const projectRoot = join(__dirname, '../');
+const projectRoot = process.cwd();
 const args = process.argv.slice(2);
 
 if (args.length !== 1) {

scripts/check-package-externals.ts renamed to scripts/check-package-externals.mts

@@ -9,12 +9,11 @@
  * the primary `cdk` and `material` packages, and also don't run for pull requests.
  */
 
-const {join} = require('path');
-const {rm, mkdir, test, ls, set, exec, cd} = require('shelljs');
-const {red, green} = require('chalk');
-const yargs = require('yargs');
+import {join} from 'path';
+import sh from 'shelljs';
+import chalk from 'chalk';
+import yargs from 'yargs';
 
-const projectDir = join(__dirname, '../');
 const archivesDir = 'dist/release-archives';
 const releasesDir = 'dist/releases';
 const {suffix} = yargs(process.argv.slice(2))
@@ -23,34 +22,35 @@ const {suffix} = yargs(process.argv.slice(2))
   .parseSync();
 
 // Fail if any ShellJS command fails.
-set('-e');
+sh.set('-e');
 
-cd(projectDir);
-
-if (!test('-e', releasesDir)) {
-  console.error(red('The release output has not been built.'));
+if (!sh.test('-e', releasesDir)) {
+  console.error(chalk.red('The release output has not been built.'));
   process.exit(1);
 }
 
-rm('-Rf', archivesDir);
-mkdir('-p', archivesDir);
+sh.rm('-Rf', archivesDir);
+sh.mkdir('-p', archivesDir);
 
-const builtPackages = ls(releasesDir)
+const builtPackages = sh
+  .ls(releasesDir)
   .map(name => ({name, path: join(releasesDir, name)}))
-  .filter(pkg => test('-d', pkg.path));
+  .filter(pkg => sh.test('-d', pkg.path));
 
 // If multiple packages should be archived, we also generate a single archive that
 // contains all packages. This makes it easier to transfer the release packages.
 if (builtPackages.length > 1) {
   console.info('Creating archive with all packages..');
-  exec(`tar --create --gzip --directory ${releasesDir} --file ${archivesDir}/all-${suffix}.tgz .`);
+  sh.exec(
+    `tar --create --gzip --directory ${releasesDir} --file ${archivesDir}/all-${suffix}.tgz .`,
+  );
 }
 
 for (const pkg of builtPackages) {
   console.info(`Creating archive for package: ${pkg.name}`);
-  exec(
+  sh.exec(
     `tar --create --gzip --directory ${pkg.path} --file ${archivesDir}/${pkg.name}-${suffix}.tgz .`,
   );
 }
 
-console.info(green(`Created package archives in: ${archivesDir}`));
+console.info(chalk.green(`Created package archives in: ${archivesDir}`));

scripts/create-package-archives.js renamed to scripts/create-package-archives.mjs

@@ -9,7 +9,7 @@ import {sync as glob} from 'glob';
 
 const errors: string[] = [];
 const seenMetadata = new Map<string, ts.ClassDeclaration>();
-const fileToCheck = join(__dirname, '../src/**/!(*.spec).ts');
+const fileToCheck = join(process.cwd(), 'src/**/!(*.spec).ts');
 const ignoredPatterns = [
   '**/components-examples/**',
   '**/dev-app/**',
@@ -141,13 +141,16 @@ function serializeValue(node: ts.Node): string {
       .slice()
       // Sort the fields since JS engines preserve the order properties in object literals.
       .sort((a, b) => (a.name?.getText() || '').localeCompare(b.name?.getText() || ''))
-      .reduce((accumulator, prop) => {
-        if (ts.isPropertyAssignment(prop)) {
-          accumulator[prop.name.getText()] = serializeValue(prop.initializer);
-        }
-
-        return accumulator;
-      }, {} as Record<string, string>);
+      .reduce(
+        (accumulator, prop) => {
+          if (ts.isPropertyAssignment(prop)) {
+            accumulator[prop.name.getText()] = serializeValue(prop.initializer);
+          }
+
+          return accumulator;
+        },
+        {} as Record<string, string>,
+      );
 
     return JSON.stringify(serialized);
   }

scripts/detect-component-id-collisions.ts renamed to scripts/detect-component-id-collisions.mts

@@ -1,6 +1,6 @@
 import chalk from 'chalk';
 import {readdirSync, readFileSync, statSync} from 'fs';
-import {IMinimatch, Minimatch} from 'minimatch';
+import minimatch from 'minimatch';
 import {join} from 'path';
 
 /**
@@ -23,7 +23,7 @@ const ownedPaths = readFileSync(ownersFilePath, 'utf8')
   // Split off just the path glob.
   .map(line => line.split(/\s+/)[0])
   // Turn paths into Minimatch objects.
-  .map(path => new Minimatch(path, {dot: true, matchBase: true}));
+  .map(path => new minimatch.Minimatch(path, {dot: true, matchBase: true}));
 
 const ignoredPaths = readFileSync(gitIgnorePath, 'utf8')
   .split('\n')
@@ -32,7 +32,7 @@ const ignoredPaths = readFileSync(gitIgnorePath, 'utf8')
   // Remove empty lines and comments.
   .filter(line => line && !line.startsWith('#'))
   // Turn paths into Minimatch objects.
-  .map(path => new Minimatch(path, {dot: true, matchBase: true}));
+  .map(path => new minimatch.Minimatch(path, {dot: true, matchBase: true}));
 
 for (let paths = getChildPaths('.'); paths.length; ) {
   paths = Array.prototype.concat(
@@ -72,7 +72,7 @@ if (errors) {
 }
 
 /** Check if the given path is owned by the given owned path matcher. */
-function isOwnedBy(path: string, ownedPath: IMinimatch) {
+function isOwnedBy(path: string, ownedPath: minimatch.IMinimatch) {
   // If the owned path ends with `**` its safe to eliminate whole directories.
   if (ownedPath.pattern.endsWith('**') || statSync(path).isFile()) {
     return ownedPath.match('/' + path);