fix(actions/parse-ci-reports-path): auto-pattern detection and parser accepted paths conflict

Signed-off-by: Emilien Escalle <emilien.escalle@escemi.com>

Author: neilime

actions/parse-ci-reports/src/parsers/AstroCheckParser.js

@@ -12,17 +12,25 @@ const ANSI_PATTERN = new RegExp(`${String.fromCharCode(27)}\\[[0-9;]*m`, "g");
  * Parser for output generated by `astro check`
  */
 export class AstroCheckParser extends BaseParser {
-	canParse(_filePath, content) {
+	canParse(filePath, content) {
 		if (!content) {
 			return false;
 		}
 
 		const sanitized = this.#stripAnsi(content);
-		if (LOCATION_PATTERN.test(sanitized)) {
+		const hasAstroMarkers =
+			LOCATION_PATTERN.test(sanitized) ||
+			ASTRO_HEADER_PATTERNS.some((pattern) => pattern.test(sanitized));
+
+		if (hasAstroMarkers) {
 			return true;
 		}
 
-		return ASTRO_HEADER_PATTERNS.some((pattern) => pattern.test(sanitized));
+		const normalizedPath = filePath.toLowerCase();
+		const hasSupportedName =
+			this.matchesAutoPatterns(filePath) || normalizedPath.includes("astro");
+
+		return hasSupportedName;
 	}
 
 	getPriority() {

actions/parse-ci-reports/src/parsers/AstroCheckParser.test.js

@@ -23,6 +23,14 @@ Result (26 files):
 - 0 hints`;
 
 describe("AstroCheckParser", () => {
+	it("keeps auto-pattern path detection synchronized", () => {
+		const parser = new AstroCheckParser();
+		const filePath = "application/humanize-astro-check-report.log";
+
+		assert.ok(parser.matchesAutoPatterns(filePath));
+		assert.ok(parser.canParse(filePath, SAMPLE_ERROR));
+	});
+
 	it("identifies astro check diagnostics", () => {
 		const parser = new AstroCheckParser();
 

actions/parse-ci-reports/src/parsers/BaseParser.js

@@ -12,6 +12,10 @@ export const ReportCategory = {
  * Follows the Strategy pattern for different report formats
  */
 export class BaseParser {
+	constructor() {
+		this._autoPatternRegexCache = null;
+	}
+
 	/**
 	 * Build glob patterns for file basenames matched anywhere in the workspace.
 	 * @param {string[]} baseNames - File basenames to match
@@ -66,6 +70,52 @@ export class BaseParser {
 		return extensions.map((extension) => `**/*.${extension}`);
 	}
 
+	/**
+	 * Check if a file path matches this parser auto-detection patterns.
+	 * @param {string} filePath - Path to evaluate
+	 * @returns {boolean} True if path matches any auto pattern
+	 */
+	matchesAutoPatterns(filePath) {
+		if (!filePath) {
+			return false;
+		}
+
+		const normalizedPath = this._normalizeFilePath(filePath).toLowerCase();
+		const matchers = this._getAutoPatternRegexes();
+
+		return matchers.some((matcher) => matcher.test(normalizedPath));
+	}
+
+	_getAutoPatternRegexes() {
+		if (this._autoPatternRegexCache) {
+			return this._autoPatternRegexCache;
+		}
+
+		const patterns = this.getAutoPatterns();
+		this._autoPatternRegexCache = patterns.map((pattern) =>
+			this._globToRegex(pattern),
+		);
+
+		return this._autoPatternRegexCache;
+	}
+
+	_normalizeFilePath(filePath) {
+		return String(filePath).replace(/\\/g, "/");
+	}
+
+	_globToRegex(pattern) {
+		const escapedPattern = pattern
+			.toLowerCase()
+			.replace(/[-[\]{}()+?.,\\^$|#\s]/g, "\\$&")
+			.replace(/\*\*\//g, "<<<ANY_DIR_PREFIX>>>")
+			.replace(/\*\*/g, "<<<ANY_DIR>>>")
+			.replace(/\*/g, "[^/]*")
+			.replace(/<<<ANY_DIR_PREFIX>>>/g, "(?:.*/)?")
+			.replace(/<<<ANY_DIR>>>/g, ".*");
+
+		return new RegExp(`^${escapedPattern}$`, "i");
+	}
+
 	/**
 	 * Parse a report file
 	 * @param {string} content - The file content

actions/parse-ci-reports/src/parsers/CheckStyleParser.js

@@ -16,12 +16,14 @@ export class CheckStyleParser extends BaseParser {
 	}
 
 	canParse(filePath, content) {
-		return (
-			(filePath.toLowerCase().includes("checkstyle") ||
-				filePath.endsWith(".xml")) &&
-			content.includes("<checkstyle") &&
-			content.includes("<file")
-		);
+		const normalizedPath = filePath.toLowerCase();
+		const hasSupportedName =
+			this.matchesAutoPatterns(filePath) ||
+			normalizedPath.includes("checkstyle") ||
+			normalizedPath.endsWith(".xml");
+		const hasCheckstyleRoot = /<\s*checkstyle\b/i.test(content);
+
+		return hasSupportedName && hasCheckstyleRoot;
 	}
 
 	getPriority() {

actions/parse-ci-reports/src/parsers/CoberturaParser.js

@@ -16,9 +16,14 @@ export class CoberturaParser extends BaseParser {
 	}
 
 	canParse(filePath, content) {
+		const normalizedPath = filePath.toLowerCase();
+		const hasSupportedName =
+			this.matchesAutoPatterns(filePath) ||
+			normalizedPath.includes("cobertura") ||
+			normalizedPath.includes("coverage");
+
 		return (
-			(filePath.toLowerCase().includes("cobertura") ||
-				filePath.toLowerCase().includes("coverage")) &&
+			hasSupportedName &&
 			content.includes("<coverage") &&
 			content.includes("line-rate")
 		);

actions/parse-ci-reports/src/parsers/ESLintParser.js

@@ -6,7 +6,15 @@ import { ReportData, LintIssue } from "../models/ReportData.js";
  * Standard format for ESLint output
  */
 export class ESLintParser extends BaseParser {
-	canParse(_filePath, content) {
+	canParse(filePath, content) {
+		const normalizedPath = filePath.toLowerCase();
+		const hasSupportedName =
+			this.matchesAutoPatterns(filePath) || normalizedPath.endsWith(".json");
+
+		if (!hasSupportedName) {
+			return false;
+		}
+
 		try {
 			const data = JSON.parse(content);
 			// ESLint format is an array of file results

actions/parse-ci-reports/src/parsers/JUnitParser.js

@@ -17,8 +17,14 @@ export class JUnitParser extends BaseParser {
 	}
 
 	canParse(filePath, content) {
+		const normalizedPath = filePath.toLowerCase();
+		const hasSupportedName =
+			this.matchesAutoPatterns(filePath) ||
+			normalizedPath.includes("junit") ||
+			normalizedPath.endsWith(".xml");
+
 		return (
-			(filePath.toLowerCase().includes("junit") || filePath.endsWith(".xml")) &&
+			hasSupportedName &&
 			(content.includes("<testsuite") || content.includes("<testsuites"))
 		);
 	}

actions/parse-ci-reports/src/parsers/JUnitParser.test.js

@@ -15,6 +15,14 @@ describe("JUnitParser", () => {
 		assert.strictEqual(parser.canParse("test-results.xml", content), true);
 	});
 
+	it("keeps auto-pattern path detection synchronized", () => {
+		const filePath = "reports/junit-report.xml";
+		const content = `<?xml version="1.0"?><testsuite name="suite"><testcase name="ok"/></testsuite>`;
+
+		assert.ok(parser.matchesAutoPatterns(filePath));
+		assert.ok(parser.canParse(filePath, content));
+	});
+
 	it("should parse simple JUnit XML", () => {
 		const content = `<?xml version="1.0"?>
       <testsuite name="Test Suite" tests="3" failures="1" skipped="1">

actions/parse-ci-reports/src/parsers/LCOVParser.js

@@ -7,8 +7,14 @@ import { ReportData, Coverage } from "../models/ReportData.js";
  */
 export class LCOVParser extends BaseParser {
 	canParse(filePath, content) {
+		const normalizedPath = filePath.toLowerCase();
+		const hasSupportedName =
+			this.matchesAutoPatterns(filePath) ||
+			normalizedPath.includes("lcov") ||
+			normalizedPath.endsWith(".info");
+
 		return (
-			(filePath.toLowerCase().includes("lcov") || filePath.endsWith(".info")) &&
+			hasSupportedName &&
 			content.includes("TN:") &&
 			(content.includes("SF:") || content.includes("DA:"))
 		);

actions/parse-ci-reports/src/parsers/PrettierParser.js

@@ -26,7 +26,15 @@ export class PrettierParser extends BaseParser {
 			return true;
 		}
 
-		if (filePath?.toLowerCase().includes("prettier")) {
+		const normalizedPath = filePath.toLowerCase();
+		const hasSupportedName =
+			this.matchesAutoPatterns(filePath) || normalizedPath.includes("prettier");
+
+		if (!hasSupportedName) {
+			return false;
+		}
+
+		if (normalizedPath.includes("prettier")) {
 			return /\[(warn|error)\]\s+.+/i.test(content);
 		}