dependency-cruiser 13.0.5 → 13.1.0-beta-2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dependency-cruiser",
-  "version": "13.0.5",
+  "version": "13.1.0-beta-2",
   "description": "Validate and visualize dependencies. With your rules. JavaScript, TypeScript, CoffeeScript. ES6, CommonJS, AMD.",
   "keywords": [
     "static analysis",

package/src/main/resolve-options/normalize.mjs

@@ -1,11 +1,10 @@
 import fs from "node:fs";
-import has from "lodash/has.js";
-import omit from "lodash/omit.js";
 import enhancedResolve from "enhanced-resolve";
+import omit from "lodash/omit.js";
 import { scannableExtensions } from "../../extract/transpile/meta.mjs";
 import {
-  ruleSetHasLicenseRule,
   ruleSetHasDeprecationRule,
+  ruleSetHasLicenseRule,
 } from "../../graph-utl/rule-set.mjs";
 
 const DEFAULT_CACHE_DURATION = 4000;

package/src/meta.js

@@ -1,7 +1,7 @@
 /* generated - don't edit */
 
 module.exports = {
-  version: "13.0.5",
+  version: "13.1.0-beta-2",
   engines: {
     node: "^16.14||>=18",
   },

package/src/report/azure-devops.mjs

@@ -0,0 +1,222 @@
+import { EOL } from "node:os";
+import {
+  formatPercentage,
+  formatViolation as _formatViolation,
+} from "./utl/index.mjs";
+
+const SEVERITY2VSO_TYPE = new Map([
+  // "error" | "warn" | "info" | "ignore
+  ["error", "error"],
+  ["warn", "warning"],
+  // azure devops doesn't seem to understand 'info'. We still want to
+  // show them, though, hence:
+  ["info", "warning"],
+]);
+
+/**
+ * @param {import("../../types/shared-types.js").SeverityType} pSeverity
+ * @returns {string}
+ */
+function formatSeverity(pSeverity) {
+  return SEVERITY2VSO_TYPE.get(pSeverity) ?? "warning";
+}
+
+/**
+ * @param {import("../../types/violations.js").IViolation} pViolation
+ * @returns {string}
+ */
+function formatModuleViolation(pViolation) {
+  return `${pViolation.rule.name}: ${pViolation.from}`;
+}
+
+/**
+ *
+ * @param {import("../../types/violations.js").IViolation} pViolation
+ * @returns {string}
+ */
+function formatDependencyViolation(pViolation) {
+  return `${pViolation.rule.name}: ${pViolation.from} -> ${pViolation.to}`;
+}
+
+/**
+ * @param {import("../../types/violations.js").IViolation} pViolation
+ * @returns {string}
+ */
+function formatCycleViolation(pViolation) {
+  return `${pViolation.rule.name}: ${
+    pViolation.from
+  } -> ${pViolation.cycle.join(" -> ")}`;
+}
+
+/**
+ * @param {import("../../types/violations.js").IViolation} pViolation
+ * @returns {string}
+ */
+function formatReachabilityViolation(pViolation) {
+  return `${pViolation.rule.name}: ${pViolation.from} -> ${
+    pViolation.to
+  } (via ${pViolation.via.join(" -> ")})`;
+}
+
+/**
+ * @param {import("../../types/violations.js").IViolation} pViolation
+ * @returns {string}
+ */
+function formatInstabilityViolation(pViolation) {
+  return `${pViolation.rule.name}: ${pViolation.from} -> ${
+    pViolation.to
+  } (instability: ${formatPercentage(
+    pViolation.metrics.from.instability
+  )} -> ${formatPercentage(pViolation.metrics.to.instability)})`;
+}
+
+/**
+ * @param {import("../../types/violations.js").IViolation} pViolation
+ * @returns {string}
+ */
+function formatViolation(pViolation) {
+  const lViolationType2Formatter = {
+    module: formatModuleViolation,
+    dependency: formatDependencyViolation,
+    cycle: formatCycleViolation,
+    reachability: formatReachabilityViolation,
+    instability: formatInstabilityViolation,
+  };
+  let lFormattedViolators = _formatViolation(
+    pViolation,
+    lViolationType2Formatter,
+    formatDependencyViolation
+  );
+  return `##vso[task.logissue type=${formatSeverity(
+    pViolation.rule.severity
+  )};sourcepath=${pViolation.from}]${lFormattedViolators}${EOL}`;
+}
+
+/**
+ *
+ * @param {number} pNumberOfErrors
+ * @param {number} pNumberOfWarns
+ * @param {number} pNumberOfInfos
+ *
+ * @returns
+ */
+function formatResultStatus(
+  pNumberOfErrors,
+  pNumberOfWarns,
+  pNumberOfInfos,
+  pNumberOfIgnored
+) {
+  if (pNumberOfErrors > 0) {
+    return "Failed";
+  }
+  if (pNumberOfWarns + pNumberOfInfos + pNumberOfIgnored > 0) {
+    return "SucceededWithIssues";
+  }
+  return "Succeeded";
+}
+
+function formatMeta(pMeta) {
+  const lWarningCount = pMeta.warn + pMeta.info;
+  const lError = `${pMeta.error} error`;
+  const lWarn = `${lWarningCount} warning/ informational`;
+  const lIgnore = (pMeta?.ignore ?? 0) > 0 ? `, ${pMeta.ignore} ignored` : "";
+
+  return `${lError}, ${lWarn}${lIgnore}`;
+}
+
+function sumMeta(pMeta) {
+  return pMeta.error + pMeta.warn + pMeta.info;
+}
+
+/**
+ *
+ * @param {number} pNumberOfIgnored
+ * @returns {string}
+ */
+function formatIgnoreWarning(pNumberOfIgnored) {
+  return (pNumberOfIgnored ?? 0) > 0
+    ? ` - ${pNumberOfIgnored} violations ignored `
+    : "";
+}
+
+/**
+ *
+ * @param {import("../../types/cruise-result.js").ISummary} pSummary
+ */
+function formatResultMessage(pSummary) {
+  let lStatSummary = `${pSummary.totalCruised} modules, ${
+    pSummary?.totalDependenciesCruised ?? 0
+  } dependencies cruised`;
+
+  if (sumMeta(pSummary) > 0) {
+    return `${sumMeta(pSummary)} dependency violations (${formatMeta(
+      pSummary
+    )}). ${lStatSummary}`;
+  } else {
+    return `no dependency violations found${formatIgnoreWarning(
+      pSummary.ignore
+    )} (${lStatSummary})`;
+  }
+}
+
+/**
+ *
+ * @param {import("../../types/cruise-result.js").ISummary} pSummary
+ */
+function formatSummary(pSummary) {
+  return `##vso[task.complete result=${formatResultStatus(
+    pSummary.error,
+    pSummary.warn,
+    pSummary.info,
+    pSummary?.ignore ?? 0
+  )};]${formatResultMessage(pSummary)}${EOL}`;
+}
+
+/**
+ * Returns a bunch of Azure DevOps log messages:
+ * - for each violation in the passed results: the severity, source found, violated rule & some additional info
+ * - a summary line
+ *
+ * Background documentation:
+ * https://learn.microsoft.com/en-us/azure/devops/pipelines/scripts/logging-commands?view=azure-devops&tabs=bash#task-commands
+ *
+ * @param {import("../../types/dependency-cruiser.js").ICruiseResult} pResults
+ * @returns {import("../../types/dependency-cruiser.js").IReporterOutput}
+ */
+// eslint-disable-next-line unicorn/prevent-abbreviations
+export default function azureDevOps(pResults) {
+  const lViolations = (pResults?.summary?.violations ?? []).filter(
+    (pViolation) => pViolation.rule.severity !== "ignore"
+  );
+
+  return {
+    output: lViolations
+      .map(formatViolation)
+      .join("")
+      .concat(formatSummary(pResults.summary)),
+    exitCode: pResults.summary.error,
+  };
+}
+
+/*
+Some notes from the documentation over at https://learn.microsoft.com/en-us/azure/devops/pipelines/scripts/logging-commands?view=azure-devops&tabs=bash#task-commands
+
+Warnings and errors:
+
+##vso[task.logissue type=warning;sourcepath=consoleapp/main.cs;linenumber=1;columnnumber=1;code=100;]Found something that could be a problem.
+
+Progress
+##vso[task.setprogress]current operation
+
+Complete
+##vso[task.complete]current operation
+
+Message grouping
+##[group]Beginning of a group
+##[warning]Warning message
+##[error]Error message
+##[section]Start of a section
+##[debug]Debug text
+##[command]Command-line being run
+##[endgroup]
+*/

package/src/report/index.mjs

@@ -1,27 +1,28 @@
 import { getExternalPluginReporter } from "./plugins.mjs";
 
-const TYPE2MODULE = {
-  anon: "./anon/index.mjs",
-  csv: "./csv.mjs",
-  dot: "./dot/dot-module.mjs",
-  ddot: "./dot/dot-folder.mjs",
-  cdot: "./dot/dot-custom.mjs",
-  archi: "./dot/dot-custom.mjs",
-  fdot: "./dot/dot-flat.mjs",
-  flat: "./dot/dot-flat.mjs",
-  "err-html": "./error-html/index.mjs",
-  markdown: "./markdown.mjs",
-  "err-long": "./error-long.mjs",
-  err: "./error.mjs",
-  html: "./html/index.mjs",
-  json: "./json.mjs",
-  teamcity: "./teamcity.mjs",
-  text: "./text.mjs",
-  baseline: "./baseline.mjs",
-  metrics: "./metrics.mjs",
-  mermaid: "./mermaid.mjs",
-  null: "./null.mjs",
-};
+const TYPE2MODULE = new Map([
+  ["anon", "./anon/index.mjs"],
+  ["csv", "./csv.mjs"],
+  ["dot", "./dot/dot-module.mjs"],
+  ["ddot", "./dot/dot-folder.mjs"],
+  ["cdot", "./dot/dot-custom.mjs"],
+  ["archi", "./dot/dot-custom.mjs"],
+  ["fdot", "./dot/dot-flat.mjs"],
+  ["flat", "./dot/dot-flat.mjs"],
+  ["err-html", "./error-html/index.mjs"],
+  ["markdown", "./markdown.mjs"],
+  ["err-long", "./error-long.mjs"],
+  ["err", "./error.mjs"],
+  ["html", "./html/index.mjs"],
+  ["json", "./json.mjs"],
+  ["teamcity", "./teamcity.mjs"],
+  ["text", "./text.mjs"],
+  ["baseline", "./baseline.mjs"],
+  ["metrics", "./metrics.mjs"],
+  ["mermaid", "./mermaid.mjs"],
+  ["null", "./null.mjs"],
+  ["azure-devops", "./azure-devops.mjs"],
+]);
 
 /**
  * Returns the reporter function associated with given output type,
@@ -37,9 +38,7 @@ async function getReporter(pOutputType) {
   if (pOutputType?.startsWith("plugin:")) {
     lReturnValue = await getExternalPluginReporter(pOutputType);
   } else {
-    const lModuleToImport =
-      // eslint-disable-next-line security/detect-object-injection
-      TYPE2MODULE[pOutputType] || "./identity.mjs";
+    const lModuleToImport = TYPE2MODULE.get(pOutputType) ?? "./identity.mjs";
     const lModule = await import(lModuleToImport);
     lReturnValue = lModule.default;
   }
@@ -52,7 +51,7 @@ async function getReporter(pOutputType) {
  * @returns {import("../../types/shared-types.js").OutputType[]} -
  */
 function getAvailableReporters() {
-  return Object.keys(TYPE2MODULE);
+  return Array.from(TYPE2MODULE.keys());
 }
 
 export default {