refactor: adjust rule details format in generated markdown report

Rather than generating one big table, with lots of wide columns, this
generates one table per violation to be more extendable with new fields.
It also ends up being more consistent with how the data is displayed in
the CLI output.

Signed-off-by: Dustin Popp <[email protected]>

Author: dpopp07

packages/validator/src/markdown-report/tables/rule-violation-details.js

@@ -1,28 +1,26 @@
 /**
- * Copyright 2024 IBM Corporation.
+ * Copyright 2024 - 2025 IBM Corporation.
  * SPDX-License-Identifier: Apache2.0
  */
 
 const MarkdownTable = require('../markdown-table');
 
-function getTable({ error, warning }) {
-  const table = new MarkdownTable(
-    'Rule',
-    'Message',
-    'Path',
-    'Line',
-    'Severity'
-  );
+function getTables(violations) {
+  let tableOutput = '';
+  for (const severity of ['error', 'warning']) {
+    for (const { message, path, rule, line } of violations[severity].results) {
+      const table = new MarkdownTable('Rule', rule);
+      table.addRow('Message', message);
+      table.addRow('Path', path.join('.'));
+      table.addRow('Line', line);
+      table.addRow('Severity', severity);
 
-  error.results.forEach(({ message, path, rule, line }) => {
-    table.addRow(rule, message, path.join('.'), line, 'error');
-  });
+      tableOutput += `${table.render()}\n\n`;
+    }
+  }
 
-  warning.results.forEach(({ message, path, rule, line }) => {
-    table.addRow(rule, message, path.join('.'), line, 'warning');
-  });
-
-  return table.render();
+  // Remove the final newline characters from the string.
+  return tableOutput.trim();
 }
 
-module.exports = getTable;
+module.exports = getTables;