@spinnaker/scripts 2026.0.3 → 2026.1.1

package/config/rollup.config.base.js

@@ -10,7 +10,7 @@ const autoPrefixer = require('autoprefixer');
 const postCssColorFix = require('postcss-colorfix');
 const postCssUrl = require('postcss-url');
 const esbuild = require('rollup-plugin-esbuild').default;
-const { terser } = require('rollup-plugin-terser');
+const terser = require('@rollup/plugin-terser');
 const { visualizer } = require('rollup-plugin-visualizer');
 
 const ROLLUP_STATS = !!process.env.ROLLUP_STATS;
@@ -19,12 +19,17 @@ const NODE_ENV = process.env.NODE_ENV || 'development';
 const ENV_MINIFY = process.env.ROLLUP_MINIFY;
 const ROLLUP_MINIFY = ENV_MINIFY === 'true' || (NODE_ENV === 'production' && ENV_MINIFY !== 'false');
 
-// eslint-disable-next-line no-console
 console.log({ ROLLUP_STATS, ROLLUP_WATCH, ROLLUP_MINIFY, NODE_ENV });
 
 const plugins = [
   nodeResolve(),
-  commonjs(),
+  commonjs({
+    // Ensure CommonJS modules that use require() get the full module.exports
+    // rather than trying to find a default export (fixes ngimport interop)
+    defaultIsModuleExports: true,
+    // Treat external ESM dependencies correctly when they're required from CJS
+    esmExternals: true,
+  }),
   json(),
   url({
     include: ['**/*.html', '**/*.svg', '**/*.png', '**/*.jp(e)?g', '**/*.gif', '**/*.webp'],

package/index.js

@@ -3,7 +3,9 @@
 const chalk = require('chalk');
 const child_process = require('child_process');
 const fs = require('fs');
-const loadConfigFile = require('rollup/dist/loadConfigFile');
+
+const { loadConfigFile } = require('rollup/loadConfigFile');
+
 const ora = require('ora');
 const path = require('path');
 const process = require('process');
@@ -12,6 +14,30 @@ const util = require('util');
 
 const exec = util.promisify(child_process.exec);
 
+const ESM_SYNTAX = /^\s*(import\s+|export\s+(default\s+)?)/m;
+
+// Rollup 4's loadConfigFile writes temporary .cjs files (named with Date.now()) to the same
+// directory as the config file. When multiple packages build in parallel and share the same
+// config (e.g. rollup.config.base.module.js), one build can delete another's temp file before
+// it's been loaded causing "Cannot find module" errors. To work around this, CJS configs are
+// loaded directly via require() (no temp files), while ESM configs still use loadConfigFile
+// (only used in sequential builds, so safe).
+const loadConfig = async (configPath) => {
+  const source = fs.readFileSync(configPath, 'utf8');
+  if (ESM_SYNTAX.test(source)) {
+    // ESM configs must go through loadConfigFile which bundles them to CJS.
+    // These are only used in sequential builds (core, presentation) so no race condition.
+    const { options, warnings } = await loadConfigFile(configPath, { bundleConfigAsCjs: true });
+    warnings.flush();
+    return options;
+  }
+  // CJS configs can be required directly, avoiding the race condition where
+  // parallel builds sharing the same config collide on temp .cjs files.
+  delete require.cache[require.resolve(configPath)];
+  const config = require(configPath);
+  return Array.isArray(config) ? config : [config];
+};
+
 const getRollupConfigPath = (file) => {
   if (file && !fs.existsSync(path.resolve(path.join('.', file)))) {
     // If the user explicitly provides a rollup config file and if it is not available, then throw an error and exit.
@@ -65,15 +91,13 @@ const runTsc = (options, exitOnFailure) => {
 const startHandler = async ({ file, push }) => {
   process.env.ROLLUP_WATCH = true;
 
-  const { options, warnings } = await loadConfigFile(getRollupConfigPath(file));
+  const options = await loadConfig(getRollupConfigPath(file));
   // A map of `input-output` bundle key to a tracker object. This is used to quickly access a spinner object and
   // startTime when succeeding/failing that object on receiving a `BUNDLE_END`/`ERROR` event.
   const buildTracker = {};
   // Kick-starting rollup's bundling process in watch mode.
   const watcher = rollup.watch(options);
   let startTime;
-
-  warnings.flush();
   watcher.on('event', (event) => {
     // Rollup will be emitting lifecycle events for the bundling process such as start, end and error. These events are
     // used to control a spinner in the terminal for each `input-output` bundle.
@@ -137,8 +161,7 @@ const printBundleComplete = (option, completedTimeInMS) => {
 };
 
 const buildHandler = async ({ file }) => {
-  const { options, warnings } = await loadConfigFile(getRollupConfigPath(file));
-  warnings.flush();
+  const options = await loadConfig(getRollupConfigPath(file));
 
   for (const o of options) {
     const start = Date.now();

package/package.json

@@ -4,7 +4,7 @@
     "type": "git",
     "url": "https://github.com/spinnaker/spinnaker.git"
   },
-  "version": "2026.0.3",
+  "version": "2026.1.1",
   "description": "Spinnaker scripts",
   "main": "index.js",
   "bin": {
@@ -20,35 +20,35 @@
   "author": "",
   "license": "ISC",
   "dependencies": {
-    "@rollup/plugin-commonjs": "^17.0.0",
-    "@rollup/plugin-json": "4.1.0",
-    "@rollup/plugin-node-resolve": "^11.0.1",
-    "@rollup/plugin-replace": "2.4.2",
-    "@rollup/plugin-typescript": "^8.1.0",
-    "@rollup/plugin-url": "^6.0.0",
+    "@rollup/plugin-commonjs": "^28.0.0",
+    "@rollup/plugin-json": "^6.1.0",
+    "@rollup/plugin-node-resolve": "^15.0.0",
+    "@rollup/plugin-replace": "^6.0.0",
+    "@rollup/plugin-terser": "^0.4.0",
+    "@rollup/plugin-typescript": "^12.0.0",
+    "@rollup/plugin-url": "^8.0.0",
     "@spinnaker/eslint-plugin": "^3.0.2",
     "@spinnaker/styleguide": "^2.0.0",
     "@svgr/rollup": "^6.3.1",
     "autoprefixer": "^10.4.8",
     "chalk": "^4.1.1",
     "comment-json": "^4.1.1",
-    "esbuild": "^0.12.14",
+    "esbuild": "^0.18.0",
     "ora": "^5.4.0",
     "postcss": "^8.4.14",
     "postcss-colorfix": "0.0.5",
     "postcss-url": "^10.1.3",
-    "rollup": "^2.79.0",
-    "rollup-plugin-esbuild": "^4.8.2",
-    "rollup-plugin-external-globals": "0.6.1",
+    "rollup": "^4.59.0",
+    "rollup-plugin-esbuild": "^6.2.0",
+    "rollup-plugin-external-globals": "^0.13.0",
     "rollup-plugin-less": "1.1.3",
     "rollup-plugin-postcss": "^4.0.2",
-    "rollup-plugin-terser": "7.0.2",
-    "rollup-plugin-visualizer": "5.4.1",
+    "rollup-plugin-visualizer": "^5.14.0",
     "yalc": "^1.0.0-pre.53",
     "yargs": "^17.1.1"
   },
   "devDependencies": {
     "@types/yargs": "^17.0.3"
   },
-  "gitHead": "d7237d64b8696dfcdf1e1eff76b6e59720899cbd"
+  "gitHead": "f842aaebe413ea1a14ccec06dd6824e07dc7426b"
 }

package/.eslintrc.js

@@ -1,5 +0,0 @@
-module.exports = {
-  rules: {
-    'no-console': 'off',
-  },
-};