@code-pushup/cli 0.8.14 → 0.8.15

package/index.js

@@ -7,8 +7,7 @@ import { hideBin } from "yargs/helpers";
 import chalk5 from "chalk";
 
 // packages/core/src/lib/implementation/persist.ts
-import { existsSync, mkdirSync } from "node:fs";
-import { stat as stat2, writeFile } from "node:fs/promises";
+import { mkdir as mkdir2, stat as stat2, writeFile } from "node:fs/promises";
 import { join as join2 } from "node:path";
 
 // packages/utils/src/lib/execute-process.ts
@@ -656,6 +655,14 @@ async function fileExists(path) {
     return false;
   }
 }
+async function directoryExists(path) {
+  try {
+    const stats = await stat(path);
+    return stats.isDirectory();
+  } catch {
+    return false;
+  }
+}
 async function ensureDirectoryExists(baseDir) {
   try {
     await mkdir(baseDir, { recursive: true });
@@ -1556,47 +1563,55 @@ async function persistReport(report, options2) {
   const { outputDir, filename, format } = options2;
   const sortedScoredReport = sortReport(scoreReport(report));
   console.info(generateStdoutSummary(sortedScoredReport));
-  const results = [];
-  if (format.includes("json")) {
-    results.push({
-      format: "json",
-      content: JSON.stringify(report, null, 2)
-    });
-  }
-  if (format.includes("md")) {
-    const commitData = await getLatestCommit();
-    validateCommitData(commitData);
-    results.push({
-      format: "md",
-      content: generateMdReport(sortedScoredReport, commitData)
-    });
-  }
-  if (!existsSync(outputDir)) {
+  const results = await Promise.all(
+    format.map(async (reportType) => {
+      switch (reportType) {
+        case "json":
+          return {
+            format: "json",
+            content: JSON.stringify(report, null, 2)
+          };
+        case "md":
+          const commitData = await getLatestCommit();
+          validateCommitData(commitData);
+          return {
+            format: "md",
+            content: generateMdReport(sortedScoredReport, commitData)
+          };
+      }
+    })
+  );
+  if (!await directoryExists(outputDir)) {
     try {
-      mkdirSync(outputDir, { recursive: true });
-    } catch (e) {
-      console.warn(e);
+      await mkdir2(outputDir, { recursive: true });
+    } catch (error) {
+      console.warn(error);
       throw new PersistDirError(outputDir);
     }
   }
   return Promise.allSettled(
-    results.map(({ format: format2, content }) => {
-      const reportPath = join2(outputDir, `${filename}.${format2}`);
-      return writeFile(reportPath, content).then(() => stat2(reportPath)).then((stats) => [reportPath, stats.size]).catch((e) => {
-        console.warn(e);
-        throw new PersistError(reportPath);
-      });
-    })
+    results.map(
+      (result) => persistResult(
+        join2(outputDir, `${filename}.${result.format}`),
+        result.content
+      )
+    )
   );
 }
-function logPersistedResults(persistResults) {
-  logMultipleFileResults(persistResults, "Generated reports");
-}
 function validateCommitData(commitData) {
   if (!commitData) {
     console.warn("no commit data available");
   }
 }
+async function persistResult(reportPath, content) {
+  return writeFile(reportPath, content).then(() => stat2(reportPath)).then((stats) => [reportPath, stats.size]).catch((error) => {
+    console.warn(error);
+    throw new PersistError(reportPath);
+  });
+}
+function logPersistedResults(persistResults) {
+  logMultipleFileResults(persistResults, "Generated reports");
+}
 
 // packages/core/src/lib/implementation/execute-plugin.ts
 import chalk4 from "chalk";
@@ -1610,12 +1625,8 @@ async function executeRunnerConfig(cfg, onProgress) {
     args,
     observer: { onStdout: onProgress }
   });
-  let audits = await readJsonFile(
-    join3(process.cwd(), outputFile)
-  );
-  if (outputTransform) {
-    audits = await outputTransform(audits);
-  }
+  const outputs = await readJsonFile(join3(process.cwd(), outputFile));
+  const audits = outputTransform ? await outputTransform(outputs) : outputs;
   return {
     duration,
     date,
@@ -1670,30 +1681,31 @@ async function executePlugin(pluginConfig, onProgress) {
   };
 }
 async function executePlugins(plugins, options2) {
-  const { progress = false } = options2 || {};
-  const progressName = "Run Plugins";
-  const progressBar = progress ? getProgressBar(progressName) : null;
+  const { progress = false } = options2 ?? {};
+  const progressBar = progress ? getProgressBar("Run plugins") : null;
   const pluginsResult = await plugins.reduce(async (acc, pluginCfg) => {
-    const outputs = await acc;
-    progressBar?.updateTitle(`Executing  ${chalk4.bold(pluginCfg.title)}`);
+    progressBar?.updateTitle(`Executing ${chalk4.bold(pluginCfg.title)}`);
     try {
       const pluginReport = await executePlugin(pluginCfg);
       progressBar?.incrementInSteps(plugins.length);
-      return outputs.concat(Promise.resolve(pluginReport));
-    } catch (e) {
+      return [...await acc, Promise.resolve(pluginReport)];
+    } catch (error) {
       progressBar?.incrementInSteps(plugins.length);
-      return outputs.concat(
-        Promise.reject(e instanceof Error ? e.message : String(e))
-      );
+      return [
+        ...await acc,
+        Promise.reject(error instanceof Error ? error.message : String(error))
+      ];
     }
   }, Promise.resolve([]));
   progressBar?.endProgress("Done running plugins");
-  const errorsCallback = ({ reason }) => console.error(reason);
+  const errorsCallback = ({ reason }) => {
+    console.error(reason);
+  };
   const results = await Promise.allSettled(pluginsResult);
   logMultipleResults(results, "Plugins", void 0, errorsCallback);
   const { fulfilled, rejected } = groupByStatus(results);
-  if (rejected.length) {
-    const errorMessages = rejected.map(({ reason }) => reason).join(", ");
+  if (rejected.length > 0) {
+    const errorMessages = rejected.map(({ reason }) => String(reason)).join(", ");
     throw new Error(
       `Plugins failed: ${rejected.length} errors: ${errorMessages}`
     );
@@ -1713,14 +1725,11 @@ function auditOutputsCorrelateWithPluginOutput(auditOutputs, pluginConfigAudits)
 
 // packages/core/package.json
 var name = "@code-pushup/core";
-var version = "0.8.14";
+var version = "0.8.15";
 
 // packages/core/src/lib/implementation/collect.ts
 async function collect(options2) {
   const { plugins, categories } = options2;
-  if (!plugins?.length) {
-    throw new Error("No plugins registered");
-  }
   const date = (/* @__PURE__ */ new Date()).toISOString();
   const start = performance.now();
   const pluginOutputs = await executePlugins(plugins, options2);
@@ -1743,62 +1752,74 @@ import {
   IssueSourceType,
   IssueSeverity as PortalIssueSeverity
 } from "@code-pushup/portal-client";
-function jsonToGql(report) {
+function jsonReportToGql(report) {
   return {
     packageName: report.packageName,
     packageVersion: report.version,
     commandStartDate: report.date,
     commandDuration: report.duration,
-    plugins: report.plugins.map((plugin) => ({
-      audits: plugin.audits.map((audit) => ({
-        description: audit.description,
-        details: {
-          issues: audit.details?.issues.map((issue) => ({
-            message: issue.message,
-            severity: transformSeverity(issue.severity),
-            sourceEndColumn: issue.source?.position?.endColumn,
-            sourceEndLine: issue.source?.position?.endLine,
-            sourceFilePath: issue.source?.file,
-            sourceStartColumn: issue.source?.position?.startColumn,
-            sourceStartLine: issue.source?.position?.startLine,
-            sourceType: IssueSourceType.SourceCode
-          })) || []
-        },
-        docsUrl: audit.docsUrl,
-        formattedValue: audit.displayValue,
-        score: audit.score,
-        slug: audit.slug,
-        title: audit.title,
-        value: audit.value
-      })),
-      description: plugin.description,
-      docsUrl: plugin.docsUrl,
-      groups: plugin.groups?.map((group) => ({
-        slug: group.slug,
-        title: group.title,
-        description: group.description,
-        refs: group.refs.map((ref) => ({ slug: ref.slug, weight: ref.weight }))
-      })),
-      icon: plugin.icon,
-      slug: plugin.slug,
-      title: plugin.title,
-      packageName: plugin.packageName,
-      packageVersion: plugin.version,
-      runnerDuration: plugin.duration,
-      runnerStartDate: plugin.date
+    plugins: pluginReportsToGql(report.plugins),
+    categories: categoryConfigsToGql(toArray(report.categories))
+  };
+}
+function pluginReportsToGql(plugins) {
+  return plugins.map((plugin) => ({
+    audits: auditReportsToGql(plugin.audits),
+    description: plugin.description,
+    docsUrl: plugin.docsUrl,
+    groups: plugin.groups?.map((group) => ({
+      slug: group.slug,
+      title: group.title,
+      description: group.description,
+      refs: group.refs.map((ref) => ({ slug: ref.slug, weight: ref.weight }))
     })),
-    categories: report.categories.map((category) => ({
-      slug: category.slug,
-      title: category.title,
-      description: category.description,
-      refs: category.refs.map((ref) => ({
-        plugin: ref.plugin,
-        type: ref.type === "audit" ? CategoryConfigRefType.Audit : CategoryConfigRefType.Group,
-        weight: ref.weight,
-        slug: ref.slug
-      }))
+    icon: plugin.icon,
+    slug: plugin.slug,
+    title: plugin.title,
+    packageName: plugin.packageName,
+    packageVersion: plugin.version,
+    runnerDuration: plugin.duration,
+    runnerStartDate: plugin.date
+  }));
+}
+function auditReportsToGql(audits) {
+  return audits.map((audit) => ({
+    description: audit.description,
+    details: {
+      issues: issuesToGql(audit.details?.issues)
+    },
+    docsUrl: audit.docsUrl,
+    formattedValue: audit.displayValue,
+    score: audit.score,
+    slug: audit.slug,
+    title: audit.title,
+    value: audit.value
+  }));
+}
+function issuesToGql(issues) {
+  return issues?.map((issue) => ({
+    message: issue.message,
+    severity: transformSeverity(issue.severity),
+    sourceEndColumn: issue.source?.position?.endColumn,
+    sourceEndLine: issue.source?.position?.endLine,
+    sourceFilePath: issue.source?.file,
+    sourceStartColumn: issue.source?.position?.startColumn,
+    sourceStartLine: issue.source?.position?.startLine,
+    sourceType: IssueSourceType.SourceCode
+  })) ?? [];
+}
+function categoryConfigsToGql(categories) {
+  return categories.map((category) => ({
+    slug: category.slug,
+    title: category.title,
+    description: category.description,
+    refs: category.refs.map((ref) => ({
+      plugin: ref.plugin,
+      type: ref.type === "audit" ? CategoryConfigRefType.Audit : CategoryConfigRefType.Group,
+      weight: ref.weight,
+      slug: ref.slug
     }))
-  };
+  }));
 }
 function transformSeverity(severity) {
   switch (severity) {
@@ -1821,8 +1842,8 @@ var normalizePersistConfig = (cfg) => ({
 
 // packages/core/src/lib/upload.ts
 async function upload(options2, uploadFn = uploadToPortal) {
-  const persist = normalizePersistConfig(options2?.persist);
-  if (!options2?.upload) {
+  const persist = normalizePersistConfig(options2.persist);
+  if (!options2.upload) {
     throw new Error("upload config must be set");
   }
   const { apiKey, server, organization, project } = options2.upload;
@@ -1838,7 +1859,7 @@ async function upload(options2, uploadFn = uploadToPortal) {
     organization,
     project,
     commit: commitData.hash,
-    ...jsonToGql(report)
+    ...jsonReportToGql(report)
   };
   return uploadFn({ apiKey, server, data });
 }
@@ -1847,9 +1868,11 @@ async function upload(options2, uploadFn = uploadToPortal) {
 async function collectAndPersistReports(options2) {
   const { exec } = verboseUtils(options2.verbose);
   const report = await collect(options2);
-  const persist = normalizePersistConfig(options2?.persist);
+  const persist = normalizePersistConfig(options2.persist);
   const persistResults = await persistReport(report, persist);
-  exec(() => logPersistedResults(persistResults));
+  exec(() => {
+    logPersistedResults(persistResults);
+  });
   report.plugins.forEach((plugin) => {
     pluginReportSchema.parse(plugin);
   });
@@ -2175,9 +2198,12 @@ function yargsCli(argv, cfg) {
   const middlewares2 = cfg.middlewares ?? [];
   const options2 = cfg.options ?? {};
   const cli2 = yargs(argv);
-  cli2.help().version(false).alias("h", "help").parserConfiguration({
+  cli2.help().version(false).alias("h", "help").check((args) => {
+    const persist = args["persist"];
+    return persist == null || validatePersistFormat(persist);
+  }).parserConfiguration({
     "strip-dashed": true
-  }).array("persist.format").coerce(
+  }).coerce(
     "config",
     (config) => Array.isArray(config) ? config.at(-1) : config
   ).options(options2);
@@ -2207,6 +2233,20 @@ function yargsCli(argv, cfg) {
   }
   return cli2;
 }
+function validatePersistFormat(persist) {
+  try {
+    if (persist?.format != null) {
+      persist.format.forEach((format) => formatSchema.parse(format));
+    }
+    return true;
+  } catch {
+    throw new Error(
+      `Invalid persist.format option. Valid options are: ${Object.values(
+        formatSchema.Values
+      ).join(", ")}`
+    );
+  }
+}
 
 // packages/cli/src/lib/cli.ts
 var cli = (args) => yargsCli(args, {

package/package.json

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