@catladder/cli 0.0.0-refactor-needs-c896b98e → 0.0.0-refactor-args-30342529

package/dist/bundles/cli/index.js

@@ -159,7 +159,7 @@ exports["default"] = (function (vorpal) { return __awaiter(void 0, void 0, void
                 .command("cloud-sql-restore-db", "restore a db from one source to another target")
                 .action(function restoreDb() {
                 return __awaiter(this, void 0, void 0, function () {
-                    var sourceInstance, sourceProxy, targetProxy, sourcePort, targetPort, sourceLocalPort, sourcePassword, sourceDbName, targetInstance, targetLocalPort, targetPassword, targetDbName, shouldContinue, targetPSQL, copyDBScript;
+                    var sourceInstance, sourceProxy, targetProxy, sourcePort, targetPort, sourceLocalPort, sourceUsername, sourcePassword, sourceDbName, targetInstance, targetLocalPort, targetUsername, targetPassword, targetDbName, shouldContinue, targetPSQL, copyDBScript;
                     return __generator(this, function (_a) {
                         switch (_a.label) {
                             case 0: return [4 /*yield*/, this.prompt({
@@ -188,82 +188,98 @@ exports["default"] = (function (vorpal) { return __awaiter(void 0, void 0, void
                                 _a.label = 5;
                             case 5: return [4 /*yield*/, this.prompt({
                                     type: "input",
-                                    name: "sourcePassword",
-                                    message: "Source Password? 🤔 "
+                                    name: "sourceUsername",
+                                    "default": "postgres",
+                                    message: "Source Username? 🤔 "
                                 })];
                             case 6:
+                                sourceUsername = (_a.sent()).sourceUsername;
+                                return [4 /*yield*/, this.prompt({
+                                        type: "input",
+                                        name: "sourcePassword",
+                                        message: "Source Password? 🤔 "
+                                    })];
+                            case 7:
                                 sourcePassword = (_a.sent()).sourcePassword;
                                 return [4 /*yield*/, this.prompt({
                                         type: "input",
                                         name: "sourceDbName",
                                         message: "Source DB name? 🤔 "
                                     })];
-                            case 7:
+                            case 8:
                                 sourceDbName = (_a.sent()).sourceDbName;
                                 return [4 /*yield*/, this.prompt({
                                         type: "input",
                                         name: "targetInstance",
                                         message: "Targe INSTANCE (connection string or 'local')? 🤔  "
                                     })];
-                            case 8:
+                            case 9:
                                 targetInstance = (_a.sent()).targetInstance;
-                                if (!(targetInstance === "local")) return [3 /*break*/, 10];
+                                if (!(targetInstance === "local")) return [3 /*break*/, 11];
                                 return [4 /*yield*/, this.prompt({
                                         type: "number",
                                         name: "targetLocalPort",
                                         "default": 5432,
                                         message: "Local Port for target? 🤔 "
                                     })];
-                            case 9:
+                            case 10:
                                 targetLocalPort = (_a.sent()).targetLocalPort;
                                 targetPort = targetLocalPort;
-                                return [3 /*break*/, 12];
-                            case 10:
+                                return [3 /*break*/, 13];
+                            case 11:
                                 targetPort = 54499;
                                 return [4 /*yield*/, createProxy(targetInstance, targetPort)];
-                            case 11:
+                            case 12:
                                 targetProxy = _a.sent();
-                                _a.label = 12;
-                            case 12: return [4 /*yield*/, this.prompt({
+                                _a.label = 13;
+                            case 13: return [4 /*yield*/, this.prompt({
                                     type: "input",
-                                    name: "targetPassword",
-                                    message: "Target Password? 🤔 "
+                                    name: "targetUsername",
+                                    "default": "postgres",
+                                    message: "Target Username? 🤔 "
                                 })];
-                            case 13:
+                            case 14:
+                                targetUsername = (_a.sent()).targetUsername;
+                                return [4 /*yield*/, this.prompt({
+                                        type: "input",
+                                        name: "targetPassword",
+                                        message: "Target Password? 🤔 "
+                                    })];
+                            case 15:
                                 targetPassword = (_a.sent()).targetPassword;
                                 return [4 /*yield*/, this.prompt({
                                         type: "input",
                                         name: "targetDbName",
                                         message: "Target DB name? 🤔 "
                                     })];
-                            case 14:
+                            case 16:
                                 targetDbName = (_a.sent()).targetDbName;
                                 return [4 /*yield*/, this.prompt({
                                         type: "confirm",
                                         name: "shouldContinue",
                                         message: "This will drop ".concat(targetInstance, "/").concat(targetDbName, " and replace it with ").concat(sourceInstance, "/").concat(sourceDbName, ". Continue? \uD83E\uDD14 ")
                                     })];
-                            case 15:
+                            case 17:
                                 shouldContinue = (_a.sent()).shouldContinue;
                                 if (!shouldContinue) {
                                     return [2 /*return*/];
                                 }
                                 targetPSQL = function (command) {
-                                    return "PGPASSWORD=".concat(targetPassword, " psql -p ").concat(targetPort, " --host=localhost --user=postgres -q ").concat(command);
+                                    return "PGPASSWORD=".concat(targetPassword, " psql -p ").concat(targetPort, " --host=localhost --user=").concat(targetUsername, " -q ").concat(command);
                                 };
-                                copyDBScript = "\n      set -e\n     \n    \n\n      dumptmp=$(mktemp /tmp/dump.XXXXXX)\n\n      echo \"Dumping file to $dumptmp\"\n      pg_dump --dbname=postgres://postgres:".concat(sourcePassword, "@localhost:").concat(sourcePort, "/").concat(sourceDbName, " --no-owner --no-privileges > $dumptmp\n      echo \"dump done\"\n      ").concat(targetPSQL("-c 'drop database \"".concat(targetDbName, "\" WITH (FORCE)' 1> /dev/null || true")), "\n      ").concat(targetPSQL("-c 'create database \"".concat(targetDbName, "\"' 1> /dev/null")), "\n          echo \"Restoring dump...\"\n      ").concat(targetPSQL("\"".concat(targetDbName, "\" < $dumptmp 1> /dev/null")), "\n  \n\n      echo \"Clean up...\"\n      set +e\n      rm $dumptmp\n      echo \"\n\uD83D\uDC31 Done!\"\n      ");
-                                _a.label = 16;
-                            case 16:
-                                _a.trys.push([16, , 18, 19]);
+                                copyDBScript = "\n      set -e\n     \n    \n\n      dumptmp=$(mktemp /tmp/dump.XXXXXX)\n\n      echo \"Dumping file to $dumptmp\"\n      pg_dump --dbname=postgres://".concat(sourceUsername, ":").concat(sourcePassword, "@localhost:").concat(sourcePort, "/").concat(sourceDbName, " --no-owner --no-privileges > $dumptmp\n      echo \"dump done\"\n      ").concat(targetPSQL("-c 'drop database \"".concat(targetDbName, "\" WITH (FORCE)' 1> /dev/null || true")), "\n      ").concat(targetPSQL("-c 'create database \"".concat(targetDbName, "\"' 1> /dev/null")), "\n          echo \"Restoring dump...\"\n      ").concat(targetPSQL("\"".concat(targetDbName, "\" < $dumptmp 1> /dev/null")), "\n  \n\n      echo \"Clean up...\"\n      set +e\n      rm $dumptmp\n      echo \"\n\uD83D\uDC31 Done!\"\n      ");
+                                _a.label = 18;
+                            case 18:
+                                _a.trys.push([18, , 20, 21]);
                                 return [4 /*yield*/, (0, child_process_promise_1.spawn)(copyDBScript, [], { shell: "bash", stdio: "inherit" })];
-                            case 17:
+                            case 19:
                                 _a.sent();
-                                return [3 /*break*/, 19];
-                            case 18:
+                                return [3 /*break*/, 21];
+                            case 20:
                                 sourceProxy === null || sourceProxy === void 0 ? void 0 : sourceProxy.stop();
                                 targetProxy === null || targetProxy === void 0 ? void 0 : targetProxy.stop();
                                 return [7 /*endfinally*/];
-                            case 19: return [2 /*return*/];
+                            case 21: return [2 /*return*/];
                         }
                     });
                 });
@@ -1466,7 +1482,7 @@ exports["default"] = (function (vorpal) { return __awaiter(void 0, void 0, void
                     .action(function (_a) {
                     var envComponent = _a.envComponent;
                     return __awaiter(this, void 0, void 0, function () {
-                        var _b, env, componentName, context, proxyInfo, DB_PASSWORD, DB_NAME, instanceName, localPort, cloudsqlProxyCredentials;
+                        var _b, env, componentName, context, proxyInfo, DB_PASSWORD, DB_NAME, DB_USER, instanceName, localPort, cloudsqlProxyCredentials;
                         var _this = this;
                         return __generator(this, function (_c) {
                             switch (_c.label) {
@@ -1491,7 +1507,7 @@ exports["default"] = (function (vorpal) { return __awaiter(void 0, void 0, void
                                     return [3 /*break*/, 6];
                                 case 5: throw new Error("unsupported environment");
                                 case 6:
-                                    DB_PASSWORD = proxyInfo.DB_PASSWORD, DB_NAME = proxyInfo.DB_NAME, instanceName = proxyInfo.instanceName;
+                                    DB_PASSWORD = proxyInfo.DB_PASSWORD, DB_NAME = proxyInfo.DB_NAME, DB_USER = proxyInfo.DB_USER, instanceName = proxyInfo.instanceName;
                                     return [4 /*yield*/, this.prompt({
                                             type: "number",
                                             name: "localPort",
@@ -1503,7 +1519,7 @@ exports["default"] = (function (vorpal) { return __awaiter(void 0, void 0, void
                                     this.log("");
                                     this.log("postgres-PW: ".concat(DB_PASSWORD));
                                     this.log("");
-                                    this.log("POSTGRESQL_URL=postgresql://postgres:".concat(DB_PASSWORD, "@localhost:").concat(localPort, "/").concat(DB_NAME, "?schema=public"));
+                                    this.log("POSTGRESQL_URL=postgresql://".concat(DB_USER, ":").concat(DB_PASSWORD, "@localhost:").concat(localPort, "/").concat(DB_NAME, "?schema=public"));
                                     this.log("");
                                     return [4 /*yield*/, (0, getProjectConfig_1.getGitlabVar)(this, env, componentName, "cloudsqlProxyCredentials")];
                                 case 8:
@@ -1570,13 +1586,14 @@ var getProxyInfoForKubernetes = function (context) { return __awaiter(void 0, vo
                 return [2 /*return*/, {
                         instanceName: instanceName,
                         DB_PASSWORD: DB_PASSWORD,
-                        DB_NAME: DB_NAME
+                        DB_NAME: DB_NAME,
+                        DB_USER: "postgres"
                     }];
         }
     });
 }); };
 var getProxyInfoForCloudRun = function (context) { return __awaiter(void 0, void 0, void 0, function () {
-    var envVars, DB_PASSWORD, DB_NAME;
+    var envVars, DB_PASSWORD, DB_USER, DB_NAME;
     return __generator(this, function (_a) {
         switch (_a.label) {
             case 0:
@@ -1588,11 +1605,13 @@ var getProxyInfoForCloudRun = function (context) { return __awaiter(void 0, void
             case 1:
                 envVars = _a.sent();
                 DB_PASSWORD = envVars === null || envVars === void 0 ? void 0 : envVars.DB_PASSWORD;
+                DB_USER = envVars === null || envVars === void 0 ? void 0 : envVars.DB_USER;
                 DB_NAME = context.environment.envVars.DB_NAME;
                 return [2 /*return*/, {
                         instanceName: context.componentConfig.deploy.cloudSql.instanceConnectionName,
                         DB_PASSWORD: DB_PASSWORD,
-                        DB_NAME: DB_NAME
+                        DB_NAME: DB_NAME,
+                        DB_USER: DB_USER
                     }];
         }
     });
@@ -350421,9 +350440,15 @@ var __spreadArray = this && this.__spreadArray || function (to, from, pack) {
   }
   return to.concat(ar || Array.prototype.slice.call(from));
 };
+var __importDefault = this && this.__importDefault || function (mod) {
+  return mod && mod.__esModule ? mod : {
+    "default": mod
+  };
+};
 exports.__esModule = true;
 exports.getNextCache = exports.getNodeCache = exports.getNodeModulesCache = exports.getYarnCache = void 0;
 var path_1 = __nccwpck_require__(71017);
+var slugify_1 = __importDefault(__nccwpck_require__(44035));
 var getYarnCache = function (context, policy) {
   var _a;
   if (policy === void 0) {
@@ -350435,7 +350460,7 @@ var getYarnCache = function (context, policy) {
     policy: policy,
     paths: [".yarn"]
   } : {
-    key: context.componentConfig.dir + "-yarn",
+    key: (0, slugify_1["default"])(context.componentConfig.dir) + "-yarn",
     policy: policy,
     paths: [(0, path_1.join)(context.componentConfig.dir, ".yarn")]
   }];
@@ -350447,9 +350472,11 @@ var getNodeModulesCache = function (context, policy) {
     policy = "pull-push";
   }
   var componentIsInWorkspace = (_a = context.packageManagerInfo) === null || _a === void 0 ? void 0 : _a.componentIsInWorkspace;
+  // We intentionally do not use the contents of yarn.lock as a cache key, as yarn install should always guarantee that the files are updated, but it can still use part of the cache if not all packages are up-to-date.
+  // It would slow down all pipelines whenever one adds a new dependency as it will need to download all node_modules again.
   return [{
     // if component is in a shared workspace, use workspace cache. use individual cache else
-    key: componentIsInWorkspace ? "node-modules-workspace" : context.componentConfig.dir + "-node-modules",
+    key: componentIsInWorkspace ? "node-modules-workspace" : (0, slugify_1["default"])(context.componentConfig.dir) + "-node-modules",
     policy: policy,
     paths: __spreadArray([], __read(componentIsInWorkspace ? __spreadArray(["node_modules"], __read((_c = (_b = context.packageManagerInfo) === null || _b === void 0 ? void 0 : _b.workspaces.map(function (w) {
       return (0, path_1.join)(w.location, "node_modules");
@@ -350949,8 +350976,9 @@ var createRailsTestJobs = function (context) {
   var bundlerCache = {
     key: {
       files: ["Gemfile.lock"],
-      prefix: "$CI_JOB_IMAGE"
+      prefix: "$CI_JOB_IMAGE" // a changed image might have different OS libraries which no longer work with the cached gems
     },
+
     paths: [bundlerCacheDir]
   };
   var auditJob = buildConfig.audit !== false ? __assign(__assign({
@@ -351171,7 +351199,7 @@ exports.readConfigSync = readConfigSync;
 
 exports.__esModule = true;
 exports.DOCKER_REGISTRY = exports.PIPELINE_IMAGE_TAG = void 0;
-exports.PIPELINE_IMAGE_TAG = "refactor-needs-c896b98e" || 0;
+exports.PIPELINE_IMAGE_TAG = "refactor-args-30342529" || 0;
 exports.DOCKER_REGISTRY = "git.panter.ch:5001/catladder/catladder" || 0;
 
 /***/ }),
@@ -352228,7 +352256,7 @@ var getBaseDeploymentJob = function (context) {
   return {
     name: exports.DEPLOY_JOB_NAME,
     envMode: "stagePerEnv",
-    needs: context.componentConfig.deploy ? (_c = (_b = context.componentConfig.deploy.waitFor) === null || _b === void 0 ? void 0 : _b.map(function (c) {
+    needsOtherComponent: context.componentConfig.deploy ? (_c = (_b = context.componentConfig.deploy.waitFor) === null || _b === void 0 ? void 0 : _b.map(function (c) {
       return {
         componentName: c,
         job: exports.DEPLOY_JOB_NAME,
@@ -352379,9 +352407,9 @@ var createGoogleCloudRunDeployJobs = function (context) {
       value = _b[1];
     return "".concat(key, "=").concat(value);
   }).join(",");
-  var commonArgs = "--project ".concat(deployConfig.projectId, " --region=").concat(deployConfig.region);
+  var commonArgs = "--project=".concat(deployConfig.projectId, " --region=").concat(deployConfig.region);
   var cloudRunArgs = deployConfig.cloudSql ? "--set-cloudsql-instances=".concat(deployConfig.cloudSql.instanceConnectionName) : "";
-  var commonDeployArgs = "--image ".concat(gcloudImageName, " ").concat(commonArgs, " ").concat(cloudRunArgs, " --labels ").concat(labelsString);
+  var commonDeployArgs = "--image=".concat(gcloudImageName, " ").concat(commonArgs, " ").concat(cloudRunArgs, " --labels=").concat(labelsString);
   var serviceName = context.environment.fullName.toLowerCase();
   var getFullJobName = function (name) {
     return (0, jobName_1.getCloudRunJobName)(context.environment.fullName, name);
@@ -352574,7 +352602,7 @@ var jobName_1 = __nccwpck_require__(77311);
 exports.GCLOUD_DEPLOY_CREDENTIALS_KEY = "GCLOUD_DEPLOY_credentialsKey";
 exports.GCLOUD_RUN_CANONICAL_HOST_SUFFIX = "GCLOUD_RUN_canonicalHostSuffix";
 var getCloudSqlVariables = function (_a) {
-  var _b, _c;
+  var _b, _c, _d;
   var deployConfigRaw = _a.deployConfigRaw,
     environmentSlug = _a.environmentSlug,
     env = _a.env,
@@ -352591,7 +352619,7 @@ var getCloudSqlVariables = function (_a) {
     return {
       CLOUD_SQL_INSTANCE_CONNECTION_NAME: deployConfigRaw.cloudSql.instanceConnectionName,
       DB_NAME: DB_NAME,
-      DB_USER: "postgres",
+      DB_USER: (_d = deployConfigRaw.cloudSql.dbUser) !== null && _d !== void 0 ? _d : "postgres",
       DB_PASSWORD: "$" + (0, context_1.getSecretVarName)(env, componentName, "DB_PASSWORD"),
       DATABASE_URL: "postgresql://$DB_USER:$DB_PASSWORD@localhost/$DB_NAME?host=/cloudsql/$CLOUD_SQL_INSTANCE_CONNECTION_NAME".concat(additionalQueryParamsString),
       DATABASE_JDBC_URL: "jdbc:postgresql:///$DB_NAME?cloudSqlInstance=$CLOUD_SQL_INSTANCE_CONNECTION_NAME&socketFactory=com.google.cloud.sql.postgres.SocketFactory&user=$DB_USER&password=$DB_PASSWORD".concat(additionalQueryParamsString)
@@ -353257,6 +353285,16 @@ exports.createKubeEnv = createKubeEnv;
 "use strict";
 
 
+var __assign = this && this.__assign || function () {
+  __assign = Object.assign || function (t) {
+    for (var s, i = 1, n = arguments.length; i < n; i++) {
+      s = arguments[i];
+      for (var p in s) if (Object.prototype.hasOwnProperty.call(s, p)) t[p] = s[p];
+    }
+    return t;
+  };
+  return __assign.apply(this, arguments);
+};
 var __rest = this && this.__rest || function (s, e) {
   var t = {};
   for (var p in s) if (Object.prototype.hasOwnProperty.call(s, p) && e.indexOf(p) < 0) t[p] = s[p];
@@ -353265,6 +353303,28 @@ var __rest = this && this.__rest || function (s, e) {
   }
   return t;
 };
+var __read = this && this.__read || function (o, n) {
+  var m = typeof Symbol === "function" && o[Symbol.iterator];
+  if (!m) return o;
+  var i = m.call(o),
+    r,
+    ar = [],
+    e;
+  try {
+    while ((n === void 0 || n-- > 0) && !(r = i.next()).done) ar.push(r.value);
+  } catch (error) {
+    e = {
+      error: error
+    };
+  } finally {
+    try {
+      if (r && !r.done && (m = i["return"])) m.call(i);
+    } finally {
+      if (e) throw e.error;
+    }
+  }
+  return ar;
+};
 exports.__esModule = true;
 exports.createKubeValues = void 0;
 var lodash_1 = __nccwpck_require__(23255);
@@ -353305,6 +353365,14 @@ var createAppConfig = function (context, application) {
   );
 };
 
+var removeFalsy = function (record) {
+  if (!record) return undefined;
+  return Object.fromEntries(Object.entries(record).filter(function (_a) {
+    var _b = __read(_a, 2),
+      value = _b[1];
+    return value !== false;
+  }));
+};
 var createKubeValues = function (context) {
   var _a, _b;
   var deployConfig = context.componentConfig.deploy;
@@ -353321,13 +353389,18 @@ var createKubeValues = function (context) {
   // eslint-disable-next-line @typescript-eslint/no-unused-vars
   var _c = values !== null && values !== void 0 ? values : {},
     application = _c.application,
-    rest = __rest(_c, ["application"]);
+    jobs = _c.jobs,
+    cronjobs = _c.cronjobs,
+    rest = __rest(_c, ["application", "jobs", "cronjobs"]);
   var env = (0, kubeEnv_1.createKubeEnv)(context);
   var defaultKubeValues = (0, lodash_1.merge)({
     env: env,
     application: createAppConfig(context, application)
   }, (0, cloudSql_1.hasKubernetesCloudSQL)(context) ? (0, cloudSql_1.createCloudsqlBaseConfig)(context) : {}, ((_b = (_a = deployConfig.values) === null || _a === void 0 ? void 0 : _a.mongodb) === null || _b === void 0 ? void 0 : _b.enabled) ? (0, mongodb_1.createMongodbBaseConfig)(context) : {});
-  var kubeValues = (0, processSecretsAsFiles_1.processSecretsAsFiles)((0, utils_1.mergeWithMergingArrays)(defaultKubeValues, rest));
+  var kubeValues = (0, processSecretsAsFiles_1.processSecretsAsFiles)((0, utils_1.mergeWithMergingArrays)(defaultKubeValues, __assign({
+    jobs: removeFalsy(jobs),
+    cronjobs: removeFalsy(cronjobs)
+  }, rest)));
   return kubeValues;
 };
 exports.createKubeValues = createKubeValues;
@@ -354376,14 +354449,19 @@ var getFullReferencedJobName = function (referencedJobName, componentName, env,
 var replaceReferences = function (job, componentName, env, allRawJobs) {
   var _a, _b, _c;
   var stage = job.envMode === "stagePerEnv" ? "".concat(job.stage, " ").concat(env) : job.stage;
-  var cleanedNeeds = __spreadArray(__spreadArray([], __read((_a = job.needs) !== null && _a !== void 0 ? _a : []), false), __read((_b = job.needsOtherComponent) !== null && _b !== void 0 ? _b : []), false);
-  var needs = cleanedNeeds === null || cleanedNeeds === void 0 ? void 0 : cleanedNeeds.map(function (n) {
-    var _a;
+  var needsFromNeeds = (_a = job.needs) === null || _a === void 0 ? void 0 : _a.map(function (n) {
     return (0, lodash_1.isObject)(n) ? {
-      job: getFullReferencedJobName(n.job, (_a = n.componentName) !== null && _a !== void 0 ? _a : componentName, env, allRawJobs),
+      job: getFullReferencedJobName(n.job, componentName, env, allRawJobs),
       artifacts: n.artifacts
     } : getFullReferencedJobName(n, componentName, env, allRawJobs);
   });
+  var needsFromOtherComponents = (_b = job.needsOtherComponent) === null || _b === void 0 ? void 0 : _b.map(function (other) {
+    return {
+      artifacts: other.artifacts,
+      job: getFullReferencedJobName(other.job, other.componentName, env, allRawJobs)
+    };
+  });
+  var needs = __spreadArray(__spreadArray([], __read(needsFromNeeds !== null && needsFromNeeds !== void 0 ? needsFromNeeds : []), false), __read(needsFromOtherComponents !== null && needsFromOtherComponents !== void 0 ? needsFromOtherComponents : []), false);
   return __assign(__assign({}, job), {
     stage: stage,
     needs: needs,
@@ -354496,37 +354574,6 @@ var __rest = this && this.__rest || function (s, e) {
   }
   return t;
 };
-var __read = this && this.__read || function (o, n) {
-  var m = typeof Symbol === "function" && o[Symbol.iterator];
-  if (!m) return o;
-  var i = m.call(o),
-    r,
-    ar = [],
-    e;
-  try {
-    while ((n === void 0 || n-- > 0) && !(r = i.next()).done) ar.push(r.value);
-  } catch (error) {
-    e = {
-      error: error
-    };
-  } finally {
-    try {
-      if (r && !r.done && (m = i["return"])) m.call(i);
-    } finally {
-      if (e) throw e.error;
-    }
-  }
-  return ar;
-};
-var __spreadArray = this && this.__spreadArray || function (to, from, pack) {
-  if (pack || arguments.length === 2) for (var i = 0, l = from.length, ar; i < l; i++) {
-    if (ar || !(i in from)) {
-      if (!ar) ar = Array.prototype.slice.call(from, 0, i);
-      ar[i] = from[i];
-    }
-  }
-  return to.concat(ar || Array.prototype.slice.call(from));
-};
 exports.__esModule = true;
 exports.makeGitlabJob = void 0;
 var lodash_1 = __nccwpck_require__(23255);
@@ -354543,9 +354590,9 @@ var makeGitlabJob = function (_a) {
     rest = __rest(_a, ["envMode", "needsStages", "needsOtherComponent", "name", "needs"]);
   return __assign(__assign({}, rest), {
     // sort in a predictable manner for snapshot tests
-    needs: needs ? __spreadArray([], __read(needs), false).sort(function (a, b) {
+    needs: needs === null || needs === void 0 ? void 0 : needs.sort(function (a, b) {
       return getJobName(a).localeCompare(getJobName(b));
-    }) : undefined,
+    }),
     retry: defaults_1.BASE_RETRY,
     interruptible: true
   });