Rename action to parse-ci-reports and refactor to use GitHub Actions script

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

Author: Copilot

.github/dependabot.yml

@@ -26,6 +26,18 @@ updates:
         patterns:
           - "*"
 
+  - package-ecosystem: npm
+    open-pull-requests-limit: 20
+    directory: "/actions/parse-ci-reports"
+    schedule:
+      interval: weekly
+      day: friday
+      time: "04:00"
+    groups:
+      npm-dependencies:
+        patterns:
+          - "*"
+
   - package-ecosystem: "devcontainers"
     open-pull-requests-limit: 20
     directory: "/"

.gitignore

@@ -0,0 +1 @@
+node_modules/

README.md

@@ -32,7 +32,7 @@ Opinionated GitHub Actions and reusable workflows for foundational continuous-in
 
 ### Repository insights & utilities
 
-- [Parse test reports](actions/parse-test-reports/README.md) — parses check, lint, test reports, and coverage into GitHub summaries and Markdown for PR comments.
+- [Parse CI reports](actions/parse-ci-reports/README.md) — parses CI reports (tests, linting, coverage) into GitHub summaries and Markdown for PR comments.
 - [Repository owner is organization](actions/repository-owner-is-organization/README.md) — checks whether the repository owner is an organization.
 - [Slugify](actions/slugify/README.md) — converts free-form strings into GitHub-friendly slugs.
 

actions/parse-test-reports/.jscpd.json actions/parse-ci-reports/.jscpd.jsonactions/parse-test-reports/.jscpd.json renamed to actions/parse-ci-reports/.jscpd.json

@@ -1,6 +1,6 @@
-# Parse Test Reports
+# Parse CI Reports
 
-Parse check, lint, test reports, and coverage into GitHub summary and Markdown for PR comments.
+Parse CI reports (tests, linting, coverage) into GitHub summary and Markdown for PR comments.
 
 ## Description
 
@@ -34,7 +34,7 @@ It supports multiple common report standards out of the box.
 
 ```yaml
 - name: Parse test reports
-  uses: hoverkraft-tech/ci-github-common/actions/parse-test-reports@v1
+  uses: hoverkraft-tech/ci-github-common/actions/parse-ci-reports@v1
   with:
     report-paths: |
       **/junit.xml
@@ -48,7 +48,7 @@ It supports multiple common report standards out of the box.
 ```yaml
 - name: Parse test reports
   id: parse-reports
-  uses: hoverkraft-tech/ci-github-common/actions/parse-test-reports@v1
+  uses: hoverkraft-tech/ci-github-common/actions/parse-ci-reports@v1
   with:
     report-paths: "**/test-results/*.xml"
     report-name: "Test Results"
@@ -65,7 +65,7 @@ It supports multiple common report standards out of the box.
 
 ```yaml
 - name: Parse coverage
-  uses: hoverkraft-tech/ci-github-common/actions/parse-test-reports@v1
+  uses: hoverkraft-tech/ci-github-common/actions/parse-ci-reports@v1
   with:
     report-paths: "coverage/cobertura-coverage.xml"
     report-name: "Coverage Report"
@@ -76,7 +76,7 @@ It supports multiple common report standards out of the box.
 
 ```yaml
 - name: Parse test reports
-  uses: hoverkraft-tech/ci-github-common/actions/parse-test-reports@v1
+  uses: hoverkraft-tech/ci-github-common/actions/parse-ci-reports@v1
   with:
     report-paths: "**/junit.xml"
     report-name: "Test Results"
@@ -120,7 +120,7 @@ Parse test results, coverage, and linting in one action:
   run: npm run lint -- --format json --output-file eslint-report.json
 
 - name: Parse all reports
-  uses: hoverkraft-tech/ci-github-common/actions/parse-test-reports@v1
+  uses: hoverkraft-tech/ci-github-common/actions/parse-ci-reports@v1
   with:
     report-paths: |
       test-results/junit.xml
@@ -136,7 +136,7 @@ Only comment on PRs if there are failures:
 ```yaml
 - name: Parse test reports
   id: parse-reports
-  uses: hoverkraft-tech/ci-github-common/actions/parse-test-reports@v1
+  uses: hoverkraft-tech/ci-github-common/actions/parse-ci-reports@v1
   with:
     report-paths: "**/test-results/*.xml"
     report-name: "Test Results"
@@ -162,7 +162,7 @@ Only comment on PRs if there are failures:
   run: mvn test
 
 - name: Parse all test reports
-  uses: hoverkraft-tech/ci-github-common/actions/parse-test-reports@v1
+  uses: hoverkraft-tech/ci-github-common/actions/parse-ci-reports@v1
   with:
     report-paths: |
       pytest-results.xml
@@ -197,7 +197,7 @@ src/
 ### Running Tests
 
 ```bash
-cd actions/parse-test-reports
+cd actions/parse-ci-reports
 npm install
 npm test
 ```

actions/parse-test-reports/README.md actions/parse-ci-reports/README.mdactions/parse-test-reports/README.md renamed to actions/parse-ci-reports/README.md

@@ -1,5 +1,5 @@
-name: "Parse Test Reports"
-description: "Parse check, lint, test reports and coverage into GitHub summary and markdown for PR comments"
+name: "Parse CI Reports"
+description: "Parse CI reports (tests, linting, coverage) into GitHub summary and Markdown for PR comments"
 author: hoverkraft
 branding:
   icon: check-circle
@@ -57,27 +57,47 @@ outputs:
 runs:
   using: "composite"
   steps:
-    - name: Parse reports
-      id: parse-reports
-      shell: bash
-      run: |
-        node ${{ github.action_path }}/src/index.js \
-          "${{ inputs.report-paths }}" \
-          "${{ inputs.report-name }}" \
-          "${{ inputs.include-passed }}" \
-          "${{ inputs.output-format }}"
+    - name: Setup Node.js
+      uses: actions/setup-node@60edb5dd545a775178f52524783378180af0d1f8 # v4.0.2
+      with:
+        node-version: "18"
 
-    - name: Write to GitHub Step Summary
-      if: inputs.output-format == 'summary' || inputs.output-format == 'both'
+    - name: Install dependencies
       shell: bash
       run: |
-        if [ -f /tmp/test-report-summary.md ]; then
-          cat /tmp/test-report-summary.md >> $GITHUB_STEP_SUMMARY
-        fi
+        cd ${{ github.action_path }}
+        npm ci --omit=dev
+
+    - name: Parse reports
+      id: parse-reports
+      uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
+      env:
+        REPORT_PATHS: ${{ inputs.report-paths }}
+        REPORT_NAME: ${{ inputs.report-name }}
+        INCLUDE_PASSED: ${{ inputs.include-passed }}
+        OUTPUT_FORMAT: ${{ inputs.output-format }}
+        ACTION_PATH: ${{ github.action_path }}
+      with:
+        script: |
+          const fs = require('fs');
+          const path = require('path');
+
+          // Import the action runner
+          const actionPath = process.env.ACTION_PATH;
+          const { runAction } = require(path.join(actionPath, 'src', 'index-action.js'));
+
+          const inputs = {
+            reportPaths: process.env.REPORT_PATHS,
+            reportName: process.env.REPORT_NAME,
+            includePassed: process.env.INCLUDE_PASSED === 'true',
+            outputFormat: process.env.OUTPUT_FORMAT
+          };
+
+          await runAction({ core, glob, inputs });
 
     - name: Fail if errors detected
       if: inputs.fail-on-error == 'true' && steps.parse-reports.outputs.has-errors == 'true'
-      shell: bash
-      run: |
-        echo "::error::Test failures or errors detected in reports"
-        exit 1
+      uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
+      with:
+        script: |
+          core.setFailed('Test failures or errors detected in reports');

actions/parse-test-reports/action.yml actions/parse-ci-reports/action.ymlactions/parse-test-reports/action.yml renamed to actions/parse-ci-reports/action.yml

@@ -1,7 +1,7 @@
 {
-  "name": "parse-test-reports",
+  "name": "parse-ci-reports",
   "version": "1.0.0",
-  "description": "Parse test, lint, and coverage reports into GitHub summaries and markdown",
+  "description": "Parse CI reports (tests, linting, coverage) into GitHub summaries and Markdown",
   "main": "src/index.js",
   "type": "module",
   "scripts": {
@@ -10,12 +10,16 @@
   },
   "keywords": [
     "github-action",
+    "ci-reports",
     "test-reports",
     "coverage",
+    "linting",
     "junit",
     "tap",
     "lcov",
-    "cobertura"
+    "cobertura",
+    "eslint",
+    "checkstyle"
   ],
   "author": "hoverkraft",
   "license": "MIT",

…ons/parse-test-reports/package-lock.json …tions/parse-ci-reports/package-lock.jsonactions/parse-test-reports/package-lock.json renamed to actions/parse-ci-reports/package-lock.json actions/parse-test-reports/package-lock.json renamed to actions/parse-ci-reports/package-lock.json

@@ -0,0 +1,75 @@
+import { readFileSync } from "fs";
+import { ParserFactory } from "./parsers/ParserFactory.js";
+import { SummaryFormatter } from "./formatters/SummaryFormatter.js";
+import { MarkdownFormatter } from "./formatters/MarkdownFormatter.js";
+import { ReportData } from "./models/ReportData.js";
+
+/**
+ * Core report parsing logic shared between CLI and Action
+ */
+export class ReportParserCore {
+  constructor() {
+    this.parserFactory = new ParserFactory();
+    this.summaryFormatter = new SummaryFormatter();
+    this.markdownFormatter = new MarkdownFormatter();
+  }
+
+  /**
+   * Parse all report files
+   * @param {string[]} files - Array of file paths
+   * @param {Function} logger - Logging function (info)
+   * @param {Function} errorLogger - Error logging function
+   * @returns {ReportData} Aggregated report data
+   */
+  parseReports(files, logger = console.log, errorLogger = console.error) {
+    const aggregatedData = new ReportData();
+
+    for (const file of files) {
+      try {
+        logger(`Parsing ${file}...`);
+        const content = readFileSync(file, "utf-8");
+        const reportData = this.parserFactory.parse(file, content);
+
+        // Merge data
+        aggregatedData.tests.push(...reportData.tests);
+        aggregatedData.lintIssues.push(...reportData.lintIssues);
+
+        if (reportData.coverage) {
+          aggregatedData.setCoverage(reportData.coverage);
+        }
+      } catch (error) {
+        errorLogger(`Error parsing ${file}: ${error.message}`);
+        // Continue with other files
+      }
+    }
+
+    return aggregatedData;
+  }
+
+  /**
+   * Generate formatted output
+   * @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)
+   * @returns {Object} Object with markdown and summary strings
+   */
+  generateOutput(reportData, reportName, includePassed, outputFormat) {
+    let markdown = "";
+    let summary = "";
+
+    if (outputFormat === "markdown" || outputFormat === "both") {
+      markdown = this.markdownFormatter.format(reportData, reportName);
+    }
+
+    if (outputFormat === "summary" || outputFormat === "both") {
+      summary = this.summaryFormatter.format(
+        reportData,
+        reportName,
+        includePassed,
+      );
+    }
+
+    return { markdown, summary };
+  }
+}