Fix empty outputs by updating format parsing logic

Co-authored-by: neilime <314088+neilime@users.noreply.github.com>

Author: Copilot

actions/parse-ci-reports/src/ReportParserCore.js

@@ -51,18 +51,21 @@ export class ReportParserCore {
    * @param {ReportData} reportData - The parsed report data
    * @param {string} reportName - Name of the report
    * @param {boolean} includePassed - Include passed tests
-   * @param {string} outputFormat - Output format (summary, markdown, both)
+   * @param {string} outputFormat - Output format (comma-separated: summary, markdown, annotations, or "all")
    * @returns {Object} Object with markdown and summary strings
    */
   generateOutput(reportData, reportName, includePassed, outputFormat) {
+    // Parse output formats
+    const formats = this.parseOutputFormats(outputFormat);
+
     let markdown = "";
     let summary = "";
 
-    if (outputFormat === "markdown" || outputFormat === "both") {
+    if (formats.includes("markdown")) {
       markdown = this.markdownFormatter.format(reportData, reportName);
     }
 
-    if (outputFormat === "summary" || outputFormat === "both") {
+    if (formats.includes("summary")) {
       summary = this.summaryFormatter.format(
         reportData,
         reportName,
@@ -72,4 +75,20 @@ export class ReportParserCore {
 
     return { markdown, summary };
   }
+
+  /**
+   * Parse output format string into array of formats
+   * @param {string} outputFormat - Output format string (comma-separated or "all")
+   * @returns {string[]} Array of output format values
+   */
+  parseOutputFormats(outputFormat) {
+    if (outputFormat === "all") {
+      return ["summary", "markdown", "annotations"];
+    }
+
+    return outputFormat
+      .split(",")
+      .map((f) => f.trim())
+      .filter((f) => f.length > 0);
+  }
 }

actions/parse-ci-reports/src/index.js

@@ -12,7 +12,7 @@ async function run({ core, glob: globModule, inputs }) {
     core.info(`Output Format: ${inputs.outputFormat}`);
 
     // Parse output formats (comma-separated or "all")
-    const outputFormats = parseOutputFormats(inputs.outputFormat);
+    const outputFormats = parserCore.parseOutputFormats(inputs.outputFormat);
     core.info(`Output formats: ${outputFormats.join(", ")}`);
 
     // Resolve report paths using the dedicated component
@@ -80,22 +80,6 @@ async function run({ core, glob: globModule, inputs }) {
   }
 }
 
-/**
- * Parse output format string into array of formats
- * @param {string} outputFormat - Output format string (comma-separated or "all")
- * @returns {string[]} Array of output format values
- */
-function parseOutputFormats(outputFormat) {
-  if (outputFormat === "all") {
-    return ["summary", "markdown", "annotations"];
-  }
-
-  return outputFormat
-    .split(",")
-    .map((f) => f.trim())
-    .filter((f) => f.length > 0);
-}
-
 /**
  * Generate GitHub annotations for failed tests and lint issues
  */