@code-pushup/core 0.8.13 → 0.8.15

package/index.js

@@ -1,6 +1,5 @@
 // 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
@@ -648,6 +647,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 });
@@ -1379,6 +1386,9 @@ function withColor({ score, text }) {
 }
 
 // packages/utils/src/lib/transform.ts
+function toArray(val) {
+  return Array.isArray(val) ? val : [val];
+}
 function deepClone(obj) {
   if (obj == null || typeof obj !== "object") {
     return obj;
@@ -1545,47 +1555,55 @@ async function persistReport(report, options) {
   const { outputDir, filename, format } = options;
   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";
@@ -1599,12 +1617,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,
@@ -1659,30 +1673,31 @@ async function executePlugin(pluginConfig, onProgress) {
   };
 }
 async function executePlugins(plugins, options) {
-  const { progress = false } = options || {};
-  const progressName = "Run Plugins";
-  const progressBar = progress ? getProgressBar(progressName) : null;
+  const { progress = false } = options ?? {};
+  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}`
     );
@@ -1702,14 +1717,11 @@ function auditOutputsCorrelateWithPluginOutput(auditOutputs, pluginConfigAudits)
 
 // packages/core/package.json
 var name = "@code-pushup/core";
-var version = "0.8.13";
+var version = "0.8.15";
 
 // packages/core/src/lib/implementation/collect.ts
 async function collect(options) {
   const { plugins, categories } = options;
-  if (!plugins?.length) {
-    throw new Error("No plugins registered");
-  }
   const date = (/* @__PURE__ */ new Date()).toISOString();
   const start = performance.now();
   const pluginOutputs = await executePlugins(plugins, options);
@@ -1732,62 +1744,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) {
@@ -1810,8 +1834,8 @@ var normalizePersistConfig = (cfg) => ({
 
 // packages/core/src/lib/upload.ts
 async function upload(options, uploadFn = uploadToPortal) {
-  const persist = normalizePersistConfig(options?.persist);
-  if (!options?.upload) {
+  const persist = normalizePersistConfig(options.persist);
+  if (!options.upload) {
     throw new Error("upload config must be set");
   }
   const { apiKey, server, organization, project } = options.upload;
@@ -1827,7 +1851,7 @@ async function upload(options, uploadFn = uploadToPortal) {
     organization,
     project,
     commit: commitData.hash,
-    ...jsonToGql(report)
+    ...jsonReportToGql(report)
   };
   return uploadFn({ apiKey, server, data });
 }
@@ -1836,9 +1860,11 @@ async function upload(options, uploadFn = uploadToPortal) {
 async function collectAndPersistReports(options) {
   const { exec } = verboseUtils(options.verbose);
   const report = await collect(options);
-  const persist = normalizePersistConfig(options?.persist);
+  const persist = normalizePersistConfig(options.persist);
   const persistResults = await persistReport(report, persist);
-  exec(() => logPersistedResults(persistResults));
+  exec(() => {
+    logPersistedResults(persistResults);
+  });
   report.plugins.forEach((plugin) => {
     pluginReportSchema.parse(plugin);
   });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@code-pushup/core",
-  "version": "0.8.13",
+  "version": "0.8.15",
   "dependencies": {
     "@code-pushup/models": "*",
     "@code-pushup/utils": "*",

package/src/lib/implementation/json-to-gql.d.ts

@@ -1,6 +1,6 @@
 import { CategoryConfigRefType, IssueSourceType, IssueSeverity as PortalIssueSeverity } from '@code-pushup/portal-client';
-import { Report } from '@code-pushup/models';
-export declare function jsonToGql(report: Report): {
+import { Issue, Report } from '@code-pushup/models';
+export declare function jsonReportToGql(report: Report): {
     packageName: string;
     packageVersion: string;
     commandStartDate: string;
@@ -58,3 +58,13 @@ export declare function jsonToGql(report: Report): {
         }[];
     }[];
 };
+export declare function issuesToGql(issues: Issue[] | undefined): {
+    message: string;
+    severity: PortalIssueSeverity;
+    sourceEndColumn: number | undefined;
+    sourceEndLine: number | undefined;
+    sourceFilePath: string | undefined;
+    sourceStartColumn: number | undefined;
+    sourceStartLine: number | undefined;
+    sourceType: IssueSourceType;
+}[];

package/src/lib/implementation/runner.d.ts

@@ -1,8 +1,8 @@
-import { AuditOutputs, OnProgress, RunnerConfig, RunnerFunction } from '@code-pushup/models';
+import { OnProgress, RunnerConfig, RunnerFunction } from '@code-pushup/models';
 export type RunnerResult = {
     date: string;
     duration: number;
-    audits: AuditOutputs;
+    audits: unknown;
 };
 export declare function executeRunnerConfig(cfg: RunnerConfig, onProgress?: OnProgress): Promise<RunnerResult>;
 export declare function executeRunnerFunction(runner: RunnerFunction, onProgress?: OnProgress): Promise<RunnerResult>;