@code-pushup/cli 0.8.13 → 0.8.14

package/index.js

@@ -1713,7 +1713,7 @@ function auditOutputsCorrelateWithPluginOutput(auditOutputs, pluginConfigAudits)
 
 // packages/core/package.json
 var name = "@code-pushup/core";
-var version = "0.8.13";
+var version = "0.8.14";
 
 // packages/core/src/lib/implementation/collect.ts
 async function collect(options2) {
@@ -1868,6 +1868,10 @@ async function readCodePushupConfig(filepath) {
   return coreConfigSchema.parse(await importEsmModule({ filepath }));
 }
 
+// packages/cli/src/lib/constants.ts
+var CLI_NAME = "Code PushUp CLI";
+var CLI_SCRIPT_NAME = "code-pushup";
+
 // packages/cli/src/lib/implementation/only-plugins-options.ts
 var onlyPluginsOption = {
   describe: "List of plugins to run. If not set all plugins are run.",
@@ -1902,10 +1906,10 @@ function yargsAutorunCommandObject() {
         }
       };
       await collectAndPersistReports(optionsWithFormat);
-      if (!options2.upload) {
-        console.warn("Upload skipped because configuration is not set.");
-      } else {
+      if (options2.upload) {
         await upload(options2);
+      } else {
+        console.warn("Upload skipped because configuration is not set.");
       }
     }
   };
@@ -1930,19 +1934,13 @@ function yargsCollectCommandObject() {
 
 // packages/cli/src/lib/implementation/filter-kebab-case-keys.ts
 function filterKebabCaseKeys(obj) {
-  const newObj = {};
-  Object.keys(obj).forEach((key) => {
-    if (key.includes("-")) {
-      return;
-    }
-    if (typeof obj[key] === "string" || typeof obj[key] === "object" && Array.isArray(obj[key])) {
-      newObj[key] = obj[key];
-    }
-    if (typeof obj[key] === "object" && !Array.isArray(obj[key]) && obj[key] != null) {
-      newObj[key] = filterKebabCaseKeys(obj[key]);
-    }
-  });
-  return newObj;
+  return Object.entries(obj).filter(([key]) => !key.includes("-")).reduce(
+    (acc, [key, value]) => typeof value === "string" || typeof value === "object" && Array.isArray(obj[key]) ? { ...acc, [key]: value } : typeof value === "object" && !Array.isArray(value) && value != null ? {
+      ...acc,
+      [key]: filterKebabCaseKeys(value)
+    } : { ...acc, [key]: value },
+    {}
+  );
 }
 
 // packages/cli/src/lib/print-config/print-config-command.ts
@@ -2041,10 +2039,10 @@ function logErrorBeforeThrow(fn) {
   return async (...args) => {
     try {
       return await fn(...args);
-    } catch (err) {
-      console.error(err);
+    } catch (error) {
+      console.error(error);
       await new Promise((resolve) => process.stdout.write("", resolve));
-      throw err;
+      throw error;
     }
   };
 }
@@ -2072,10 +2070,10 @@ async function configMiddleware(processArgs) {
     // therefore this can't live in option defaults as the order would be `config`->`provided options`->default
     // so we have to manually implement the order
     persist: {
-      outputDir: cliOptions?.persist?.outputDir || importedRc?.persist?.outputDir || PERSIST_OUTPUT_DIR,
-      filename: cliOptions?.persist?.filename || importedRc?.persist?.filename || PERSIST_FILENAME,
+      outputDir: cliOptions.persist?.outputDir || importedRc.persist?.outputDir || PERSIST_OUTPUT_DIR,
+      filename: cliOptions.persist?.filename || importedRc.persist?.filename || PERSIST_FILENAME,
       format: coerceArray(
-        cliOptions?.persist?.format || importedRc?.persist?.format || PERSIST_FORMAT
+        cliOptions.persist?.format ?? importedRc.persist?.format ?? PERSIST_FORMAT
       )
     },
     plugins: filterPluginsByOnlyPluginsOption(importedRc.plugins, cliOptions),
@@ -2173,19 +2171,16 @@ import chalk9 from "chalk";
 import yargs from "yargs";
 function yargsCli(argv, cfg) {
   const { usageMessage, scriptName, noExitProcess } = cfg;
-  let { commands: commands2, options: options2, middlewares: middlewares2 } = cfg;
-  commands2 = Array.isArray(commands2) ? commands2 : [];
-  middlewares2 = Array.isArray(middlewares2) ? middlewares2 : [];
-  options2 = options2 || {};
+  const commands2 = cfg.commands ?? [];
+  const middlewares2 = cfg.middlewares ?? [];
+  const options2 = cfg.options ?? {};
   const cli2 = yargs(argv);
   cli2.help().version(false).alias("h", "help").parserConfiguration({
     "strip-dashed": true
-  }).array("persist.format").coerce("config", (config) => {
-    if (Array.isArray(config)) {
-      return config[config.length - 1];
-    }
-    return config;
-  }).options(options2);
+  }).array("persist.format").coerce(
+    "config",
+    (config) => Array.isArray(config) ? config.at(-1) : config
+  ).options(options2);
   if (usageMessage) {
     cli2.usage(chalk9.bold(usageMessage));
   }
@@ -2201,9 +2196,7 @@ function yargsCli(argv, cfg) {
   commands2.forEach((commandObj) => {
     cli2.command({
       ...commandObj,
-      ...commandObj.handler && {
-        handler: logErrorBeforeThrow(commandObj.handler)
-      },
+      handler: logErrorBeforeThrow(commandObj.handler),
       ...typeof commandObj.builder === "function" && {
         builder: logErrorBeforeThrow(commandObj.builder)
       }
@@ -2216,8 +2209,6 @@ function yargsCli(argv, cfg) {
 }
 
 // packages/cli/src/lib/cli.ts
-var CLI_NAME = "Code PushUp CLI";
-var CLI_SCRIPT_NAME = "code-pushup";
 var cli = (args) => yargsCli(args, {
   usageMessage: CLI_NAME,
   scriptName: CLI_SCRIPT_NAME,
@@ -2227,4 +2218,4 @@ var cli = (args) => yargsCli(args, {
 });
 
 // packages/cli/src/index.ts
-cli(hideBin(process.argv)).argv;
+await cli(hideBin(process.argv)).argv;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@code-pushup/cli",
-  "version": "0.8.13",
+  "version": "0.8.14",
   "bin": {
     "code-pushup": "index.js"
   },

package/src/lib/cli.d.ts

@@ -1,3 +1 @@
-export declare const CLI_NAME = "Code PushUp CLI";
-export declare const CLI_SCRIPT_NAME = "code-pushup";
 export declare const cli: (args: string[]) => import("yargs").Argv<unknown>;

package/src/lib/constants.d.ts

@@ -0,0 +1,2 @@
+export declare const CLI_NAME = "Code PushUp CLI";
+export declare const CLI_SCRIPT_NAME = "code-pushup";